No connectivity from client to server on the same host via kube-proxy ClusterIP when Cilium E/W LB is disabled

client-pod: <IP_ADDRESS>:XXX
service-ip: <IP_ADDRESS>:9090
backend: <IP_ADDRESS>:9090
tcpdump -i any:
00:41:31.413413 IP <IP_ADDRESS>.40106 > <IP_ADDRESS>.9090: Flags [S], seq 179494632, win 64390, options [mss 1370,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
00:41:31.413492 IP <IP_ADDRESS>.40106 > <IP_ADDRESS>.9090: Flags [S], seq 179494632, win 64390, options [mss 1370,sackOK,TS val<PHONE_NUMBER> ecr 0,nop,wscale 7], length 0
00:41:31.413513 IP <IP_ADDRESS>.9090 > <IP_ADDRESS>.40106: Flags [S.], seq<PHONE_NUMBER>, ack 179494633, win 65184, options [mss 1370,sackOK,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>,nop,wscale 7], length 0
00:41:31.413527 IP <IP_ADDRESS>.9090 > <IP_ADDRESS>.40106: Flags [S.], seq<PHONE_NUMBER>, ack 179494633, win 65184, options [mss 1370,sackOK,TS val<PHONE_NUMBER> ecr<PHONE_NUMBER>,nop,wscale 7], length 0
00:41:31.413537 IP <IP_ADDRESS>.40106 > <IP_ADDRESS>.9090: Flags [R], seq 179494633, win 0, length 0
00:41:31.413544 IP <IP_ADDRESS>.40106 > <IP_ADDRESS>.9090: Flags [R], seq 179494633, win 0, length 0

monitor output:
Policy verdict log: flow 0xdece3b7d local EP ID 2591, remote ID 2, dst port 9090, proto 6, ingress false, action allow, match all, <IP_ADDRESS>:39772 -> <IP_ADDRESS>:9090 tcp SYN
-> stack flow 0xdece3b7d identity 7644->2 state new ifindex 0 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:39772 -> <IP_ADDRESS>:9090 tcp SYN
Policy verdict log: flow 0x492eb6dd local EP ID 2591, remote ID 56645, dst port 39772, proto 6, ingress true, action allow, match all, <IP_ADDRESS>:9090 -> <IP_ADDRESS>:39772 tcp SYN, ACK
-> endpoint 2591 flow 0x492eb6dd identity 56645->7644 state new ifindex lxc10371e843f08 orig-ip <IP_ADDRESS>: <IP_ADDRESS>:9090 -> <IP_ADDRESS>:39772 tcp SYN, ACK

cilium config map
  auto-direct-node-routes: "false"
  bpf-ct-global-any-max: "262144"
  bpf-ct-global-tcp-max: "524288"
  bpf-nat-global-max: "841429"
  cluster-name: default
  debug: "false"
  enable-external-ips: "false"
  enable-host-reachable-services: "false"
  enable-ipv4: "true"
  enable-ipv6: "false"
  enable-metrics: "true"
  enable-node-port: "false"
  enable-remote-node-identity: "true"
  enable-well-known-identities: "false"
  enable-xt-socket-fallback: "true"
  identity-allocation-mode: crd
  install-iptables-rules: "true"
  k8s-require-ipv4-pod-cidr: "true"
  kube-proxy-replacement: disabled
  masquerade: "true"
  monitor-aggregation: medium
  monitor-aggregation-flags: all
  monitor-aggregation-interval: 5s
  node-port-mode: hybrid
  operator-api-serve-addr: <IP_ADDRESS>:9234
  operator-prometheus-serve-addr: :6942
  policy-audit-mode: "false"
  preallocate-bpf-maps: "false"
  prometheus-serve-addr: :9090
  sidecar-istio-proxy-image: cilium/istio_proxy
  synchronize-k8s-nodes: "true"
  tofqdns-enable-poller: "false"
  tunnel: vxlan
  wait-bpf-mount: "false"

iptables rules
$ sudo iptables-save | grep 9090
-A KUBE-SEP-VTRHTEKTWBKBIVAE -p tcp -m tcp -j DNAT --to-destination <IP_ADDRESS>:9090
-A KUBE-SERVICES -d <IP_ADDRESS>/32 -p tcp -m comment --comment "kube-system/prometheus:webui cluster IP" -m tcp --dport 9090 -j KUBE-SVC-SMBNPD2J27EUPM6V
$ sudo iptables-save | grep KUBE-SVC-SMBNPD2J27EUPM6V
:KUBE-SVC-SMBNPD2J27EUPM6V - [0:0]
-A KUBE-SERVICES -d <IP_ADDRESS>/32 -p tcp -m comment --comment "kube-system/prometheus:webui cluster IP" -m tcp --dport 9090 -j KUBE-SVC-SMBNPD2J27EUPM6V
-A KUBE-SVC-SMBNPD2J27EUPM6V -j KUBE-SEP-VTRHTEKTWBKBIVAE
$ sudo iptables-save | grep KUBE-SEP-VTRHTEKTWBKBIVAE
:KUBE-SEP-VTRHTEKTWBKBIVAE - [0:0]
-A KUBE-SEP-VTRHTEKTWBKBIVAE -s <IP_ADDRESS>/32 -j KUBE-MARK-MASQ
-A KUBE-SEP-VTRHTEKTWBKBIVAE -p tcp -m tcp -j DNAT --to-destination <IP_ADDRESS>:9090
-A KUBE-SVC-SMBNPD2J27EUPM6V -j KUBE-SEP-VTRHTEKTWBKBIVAE

commit a734d81edc5aafd2d58532fa2c71f2984c9c080a

When --kube-proxy-replacement=disabled and --disable-k8s-services=true, the datapath is using the old service mechanism for ClusterIP services. It seems that the rev-DNAT xlation was not applied for the SYN-ACK packet, therefore we see the TCP RST.
@aanm Did it happen in the CI? The CI run for the commit which you linked to failed because of the different test case (the managed ETCD). If yes, can you attach the test dump?
I was expecting that we have tests which check client @ host A -> Cluster IP -> server @ host A (https://github.com/cilium/cilium/blob/master/test/k8sT/Services.go#L192), but they sent a request to ClusterIP from the host netns.

Created an issue to extend the tests: https://github.com/cilium/cilium/issues/10568.

Backend is on same node where svc request is processed, Cilium is not handling any services at all since kube-proxy-replacement: disabled.

I've just managed to reproduce the issue on the kernel 5.6.0-rc2+ (ubuntu-next v45). The problem is that the rev-DNAT xlation is not performed for a reply. This happens because the service endpoint does skb_redirect to the client pod, which makes the reply to bypass the netfilter's conntrack module which is responsible for the xlation when --disable-k8s-service=true.
Interestingly, I cannot reproduce the issue on the kernel 5.5.6-arch1-1.

Did we apply any fix to resolve this issue?
Are we seeing it in CI? These days we should be running effectively a Linux v5.7 RC kernel there so I would expect us to observe it regularly if it is some kind of kernel regression.

Did we apply any fix to resolve this issue?

AFAIK, noup.

Are we seeing it in CI?

We don't have any test which would test podA@host1 -> clusterIP -> podB@host1 with --disable-k8s-service=true.

Context: Cilium 1.7 had hit this because of initial disablement of k8s services entirely. We released a fix for this.
We have already deprecated --disable-k8s-services flag due to this issue.
Unless something changes, we do not intend to fix this issue.

Removing blocker for v1.9. @brb if you disagree, please reach out :-)

Closing this issue in favor of #16197.


[STARKs] Fix panic when FRI folding is zero

Description
Closes #216, but with a different solution. We don't need to resample beta, we just needed to not panic when obtaining the last coefficient/last FRI evaluation.
Type of change
Please delete options that are not relevant.

[x] Bug fix

Checklist

[x] Linked to Github Issue

Codecov Report

Merging #232 (d169509) into main (bd9d77b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #232   +/-   ##
=======================================
  Coverage   94.01%   94.01%
=======================================
  Files          78       78
  Lines        9202     9208    +6
=======================================
+ Hits         8651     8657    +6
  Misses        551      551

Impacted Files
Coverage Δ

proving_system/stark/src/fri/fri_decommit.rs
98.18% <100.00%> (+0.14%)
:arrow_up:

proving_system/stark/src/fri/mod.rs
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM


v1

At v1.0, Riak-Cpp has everything it needs to solve some subset of people's C++/Riak problems. Our target audience is high-performance server programmers using Riak as a backend.
Definition of Done:

First production system using Riak-Cpp has been built and deployed.
Missing features that make the library useful for our target customers have been added.
Enabling documentation accompanies the tested (obviously) library.

Note that, for (2), we refer not to those features that were missing from 0.x implementations due to oversight, but rather those features that weren't considered for inclusion because those requirements had not been expressed. The former belongs in a 0.x release, and the set of those things may be empty.

Idiotic. I had meant to create a milestone for this.


feat: Create Website for Pentesting-and-Hacking-Scripts Project

Description
This pull request adds a website that allows users to view and search hacking scripts from the repository. Additionally, I have implemented a special route for admin users to update the content whenever a new script is added.
Here are the changes included in this pull request:

Added a new website for the project.
Implemented functionality to display and search hacking scripts.
Created an admin route for updating the content.
Handled the addition of new scripts and updating the website accordingly.
I have deployed frontend on vercel and backend server on render.com

I believe this website will greatly enhance the user experience and make it easier for users to find and access the hacking scripts they need. The admin route ensures that the content remains up to date as new scripts are added to the repository.
I have thoroughly tested the website and it is working smoothly. The code is well-documented and follows best practices. Please review the changes and consider merging this pull request into the main branch.
Main Deployed website: https://pentesting-and-hacking-scripts.vercel.app/
Deployed server: https://hacking-5br1.onrender.com
Fixes #(#285)
Replace issue_no with the issue number which is fixed in this PR
https://github.com/avinashkranjan/Pentesting-and-Hacking-Scripts/assets/85815172/ad332b99-5bce-40c2-9853-6514aeb5401b
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I've edited the SCRIPTS.md and link to my code.
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@avinashkranjan , please review and merge this pull request  for website


added managed prometheus alert manager config for java/jmx modules. A…

…dded additional output variable to get the managed prometheus workspace id
What does this PR do?

added managed prometheus alert manager config for java/jmx modules. Added additional output variable to get the managed prometheus workspace id
Motivation

More

[x ] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)
[ ] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[ ] Yes, I have updated the docs for this feature
[ ] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

@askulkarni2 - ran pre-commit and updated the PR. can you pls review and approve?


[5.1] Backport #21320

As mentioned here, this fix should be back-ported to 5.1. This brings both the final changes and the added tests to ensure their correctness.

@mcordingley tests failing

I saw that. It looks like HHVM doesn't like something to do with something mocking PDO statements. I'm not sure what to make of that, since I'm using the same mocking calls as the pre-existing tests.

@mcordingley I think it's ok to exclude the problematic test. 5.1 is closed for changes anyways.

Looks like test 247 (where the job fails) is DatabaseConnectionTest::testSelectProperlyCallsPDO

From my phpunit --debug output:
Starting test 'DatabaseConnectionTest::testSettingDefaultCallsGetDefaultGrammar'.
W  244 / 1757 ( 13%)

Starting test 'DatabaseConnectionTest::testSettingDefaultCallsGetDefaultPostProcessor'.
W
Starting test 'DatabaseConnectionTest::testSelectOneCallsSelectAndReturnsSingleResult'.
W
Starting test 'DatabaseConnectionTest::testSelectProperlyCallsPDO'.

This took about 17 PRs to actually get right the first time. I don't want to go through that again on an existing stable release.

That us perfectly fine by me. Just felt a responsibility to backport it to the old LTS, per the updates policy. Otherwise, I'm also happy to be done with this issue for the same reason as you.

I'll just hold off on this. The requirement to actually have cracked the AES-256 encryption key of the entire application before this is useful I think makes it less plausible as an attack.


modal and button text is updated.Issue no  #239

please resolve conflicts

I have deleted the repo, forked it again, and created a pull request Again.
On Mon, Oct 17, 2022 at 10:23 PM Avinash Singh @.***>
wrote:

delete the repo and fork again to do your changes
—
Reply to this email directly, view it on GitHub
https://github.com/avinash201199/To-Do-List/pull/243#issuecomment-1281167797,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATY7ASJRQVAWHGOSXT2NHY3WDWACJANCNFSM6AAAAAARGTSX2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

please, check it.
On Mon, Oct 17, 2022 at 11:03 PM vikash kathait @.***>
wrote:

I have deleted the repo, forked it again, and created a pull request
Again.
On Mon, Oct 17, 2022 at 10:23 PM Avinash Singh @.***>
wrote:

delete the repo and fork again to do your changes
—
Reply to this email directly, view it on GitHub
https://github.com/avinash201199/To-Do-List/pull/243#issuecomment-1281167797,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATY7ASJRQVAWHGOSXT2NHY3WDWACJANCNFSM6AAAAAARGTSX2Q
.
You are receiving this because you authored the thread.Message ID:
@.***>


Set account tree item

Going to wait to merge this until after releasing v1.7.1
Fixes #3191
Is this too hacky? We could properly expose this in the host api, but I don't think it belongs there.

Is this too hacky? We could properly expose this in the host api.

Hacky yes, but I don't think it's something we should put on the API.

We're on the same page then 😄


docs(samples): added samples and test for recaptcha key operations

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #642  ☕

@eaball35 PTAL


Negative loss in unbalanced Sinkhorn

Unbalanced Sinkhorn results in negative reg_ot_cost.
import jax
from ott.core.sinkhorn import Sinkhorn
from ott.geometry.pointcloud import PointCloud
import jax.numpy as jnp
from ott.core import LinearProblem

n = 1000
dim = 30

rng = jax.random.PRNGKey(0)
rng, *rngs = jax.random.split(rng, 5)
x = jax.random.uniform(rngs[0], (n, dim))
y = jax.random.uniform(rngs[1], (n, dim)) + 0.1
a = jax.random.uniform(rngs[2], (n,))
b = jax.random.uniform(rngs[3], (n,))
a_scaled = a / jnp.sum(a)
b_scaled = b / jnp.sum(b)

lp = LinearProblem(PointCloud(x, y, epsilon=1), a_scaled, b_scaled, tau_a=0.5, tau_b=0.6)
s = Sinkhorn()(lp)

print(s.reg_ot_cost)

The regularised OT cost is -8.937. The more unbalanced, the more negative the OT cost is, which should not be the case in general.
Installation ott-jax: from source.

Hi Dominik, thanks for this.
If you look at the primal cost, which is returned by reg_ot_cost, this includes terms in KLs (those should >0) but also a contribution term corresponding to minus the entropy.
In OTT, we compute the objective using the dual formulation
https://github.com/ott-jax/ott/blob/d7521fdbe3e461a2192c8fc1e4080558dd453ee3/ott/core/sinkhorn.py#L142
and that also includes some negative contributions, notably in the div_a and div_b terms, and the total_sum term substratcted at the end.
So, having a negative ent_reg_cost is not a contradiction per se. However, this does not preclude a bug. Have you looked at the transportation matrix that is returned? do you see entries that are unusual?

Hi Marco,
Thanks for your response. I might be wrong but reading equation 24 in https://arxiv.org/pdf/1910.12958.pdf the term multiplied by epsilon in the entropic penalization should be non-positive because  and hence every element of the right matrix in the inner product is non-positive whereas each term in the left matrix is non-negative.
This holds true for total_sum in the code, in the example above total_sum = -11.217885.
On the other hand, we add this term in the return statement, i.e. the return value (https://github.com/ott-jax/ott/blob/d7521fdbe3e461a2192c8fc1e4080558dd453ee3/ott/core/sinkhorn.py#L198) is div_a + div_b + ot_prob.epsilon * (jnp.sum(ot_prob.a) * jnp.sum(ot_prob.b) - total_sum). Hence, I don't know where the minus before the epsilon in equation (24) is incorporated.
Hence, I would assume that the return statement starting in line 198 should read
div_a + div_b - ot_prob.epsilon * (
      jnp.sum(ot_prob.a) * jnp.sum(ot_prob.b) - total_sum
  )

Apologies if I am wrong.

Thanks for checking this more closely, I must admit that part of the code was not extensively reviewed, so i am very grateful for you taking a closer look!!
here total_sum corresponds to $\langle\alpha \otimes \beta, e^{\frac{f\oplus g-C}{\varepsilon}}\rangle$, since the elementwise product of these two matrices, i.e. $[\alpha_i \beta_i, e^{\frac{f_i+ g_j-C_{ij}}{\varepsilon}}]_ij$ corresponds to the transport.
Since that term has a minus in front of it in equation 24, I think this should still be a $-\varepsilon \times$ total_sum.
Similarly the jnp.sum(ot_prob.a) * jnp.sum(ot_prob.b) corresponds to $\langle\alpha \otimes \beta, 1\rangle$ and that should be added (with +) because of the two -. So with a quick check I think that's still a + there in front of ot_prob.epsilon, anything mistaken on my end? Thanks again for checking!
As for marginals, I think you are right, there is definitely some scaling factor needed here in the stopping criterion. I think that's a very good idea. For instance, we could have a tolerance equal by default to jnp.sum(a) * tolerance. I hope this does not have complicated side effects when optimizing over $a$ in an unbalanced setting, but I think that's reasonable.


Exception has occurred. MissingPluginException (MissingPluginException(No implementation found for method run on channel com.pichillilorenzo/flutter_headless_inappwebview))

what can i fix this error?
Exception has occurred.
MissingPluginException (MissingPluginException(No implementation found for method run on channel com.pichillilorenzo/flutter_headless_inappwebview))

stop currently running app
flutter pub get
run app


General cleanup of host module code

This issue overlaps issues #43 and #49

Tasks:
[ ] Bring from_index() up to date
[ ] Consistent internal structure among modules
[ ] Document all arguments
[ ] Consistent calling interface


Really slow on large files?

When opening a large file (in my current example 10k lines) the file takes forever to load... This doesn't happen using the default js highlighter.
I'm using Sublime Text 2221

I have same problem but only on large line.
Thanks.

ST2 isn't supported. @petersirka are you also using ST2?

I found the problem. The problem persist when I have enabled detect slow plugins.
Everything works correctly.
:100: sorry.

@thomasmattheussen Can you give an example file?

I've since upgraded to ST3 and not experiencing the problem anymore.
An example file would just be a large javascript file (like 10k+ loc)...

Well good to know the problem fixed itself, that doesn't happen often :)


I put CSS file under the module，How should I quote？Can asset() be used?

I put CSS file under the Modules，How should I quote？Can asset() be used?
I make the Modules --Web,Create folders /Modules/Web/Public/css/index.css
How should I quote it in my views? please help me,Thank you very much.
at first run the commands
php artisan module:publish Web
I use
{{ Module::asset('web:Public/css/index.css') }}
it can make the url  //<IP_ADDRESS>:8003/modules/web/Public/css/index.css
but 404 error。

Hi,
You should publish these assets in the public folder, and then use asset() to reference them.
There's a module publish command to do this.

嗨，
您应该在公用文件夹中发布这些资产，然后使用asset()它们来引用它们。
有一个模块发布命令来执行此操作。

i used edition is v2,my laravel is 5.5.
i see theArticle  https://github.com/nWidart/laravel-modules/issues/151，
Do the same， but not ok。。。

I'll close this as there hasn't been a reply in a while, feel free to comment if needed.

Hello nWidart,
I am able to publish assets in the public folder but I need to type-in /public in my url in-order to get the complete url; though I have remove the /public from my url.
Can you please help me on this.
Thanks

Hello nWidart,
I am able to publish assets in the public folder but I need to type-in /public in my url in-order to get the complete url; though I have remove the /public from my url.
Can you please help me on this.
Thanks

You need to configure nginx/apache to point the directory root, to /public. You can find information about this on google.


Add RPC

I still don’t get it. Let sync in zoom. You can merge it if you really want
to for now to unblock the progrss. but I don’t agree that this will be the
API that will be published 😅
On Mon, Nov 20, 2023 at 4:43 PM Kaili Zhu @.***> wrote:

@.**** commented on this pull request.
In src/main/java/io/xdb/core/communication/Communication.java
https://github.com/xcherryio/sdk-java/pull/13#discussion_r1399901818:

@@ -50,4 +55,13 @@ public void publishToLocalQueue(final String queueName, final String dedupId, fi

     localQueueMessagesToPublish.add(message);
 }

/**

* Trigger new state decision from @RPC methods.

*

* @param stateDecision     the state decision to trigger.

*/

public void triggerNewStateDecision(final StateDecision stateDecision) {

I think we can expose as many completed funtions as possible so that users
can choose which functionality they like.
As for the SDK API naming, maybe you can read the current API for
AsyncState execute first, then look back to this one? I don't wanna make
the two inconsistent for users.
—
Reply to this email directly, view it on GitHub
https://github.com/xcherryio/sdk-java/pull/13#discussion_r1399901818,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCQPM4OUQCQRI6FNDRNINDYFP2KNAVCNFSM6AAAAAA7RPVYUWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBQHEZDCMRXG4
.
You are receiving this because your review was requested.Message ID:
@.***>


[WIP] - Include skip(0) to subqueries with order by to avoid client eval

DO NOT COMMIT THIS - work in progress.
I've taken a stab at solving #10102. Wanted to get some feedback on the change and some help resolving the last few unit tests failures it introduced.
I updated the subQueryModelVisitor to add a Skip(0) into the subquery if an order by is present. This works a treat for the simple examples. It all gets a bit fruity on the Join_customers_orders_with_subquery_anonymous_property_method unit test.  Which now produces the following two sql statements which to me look wrong:
SELECT [t].[OrderID], [t].[CustomerID], [t].[EmployeeID], [t].[OrderDate]
FROM (
    SELECT [o2].[OrderID], [o2].[CustomerID], [o2].[EmployeeID], [o2].[OrderDate]
    FROM [Orders] AS [o2]
    ORDER BY [o2].[OrderID]
    OFFSET 0 ROWS
) AS [t]

SELECT [c].[CustomerID]
FROM [Customers] AS [c]

The other issue I can't get my head around is SimpleQuerySqlServerTest.OrderBy_Join tests fail with what seems like the order of Customers being correct but the suborder of the Orders are reversed every other customer.
PS, ignore my branch name, seems I was looking at the wrong issue when I created it.

The first problem happens due to https://github.com/aspnet/EntityFrameworkCore/issues/8094 - we can't bind to the anonymous type in the subquery. It's OK to leave it as is for now.

@Muppets looks good otherwise - please rebase-squash on the latest master and it should be ready to go.

merged 4a2b27f8fbf484fa2ea09bfb7fb785a170189b42
@Muppets thank you for the contribution!


Caching

Is there any way to cache responses/resources the Rails way? I don't see anything in docs.

Not yet. #712 does some caching and it does improve the performance noticeably. I'm hoping we can get it merged soon.

@Keqi try the current master now, see the README documentaion regarding "resource caching".

@DavidMikeSimon Just a thought, but I recently tried to use the caching behavior described in the README, and I get:
undefined method `resource_cache=' for #<JSONAPI::Configuration:0x007f917396d208>`

unless I point my gemfile towards the github URL. None of the gems on Rubygems (0.7.0 - 0.8.0.beta2) have the config item.
The way its presented in the README makes me think it should be in there. Might be worth putting a message in the README about this?

The problem is the readme displayed is for the master branch, but this feature hasn't been released yet. The plan is to release 0.8.0 this coming week and then caching will go in a 0.9.0.beta1 release along with a few other changes. We need a different solution for release documentation as the current way is obviously confusing.

@lgebhardt is this still being worked on?


Updated lists

Updated hbbtv->smarttv list, since its name changed by maintainer.
Removed RansomWare Tracker, since it's no longer maintained nor accesible.
Updated dnsbl.sh accordingly.
This is my first pull request here, sorry if I did it wrong.

@marcushoff can we have #1712 merged into this one PR ?

Sure, I just wasn't sure how to make that in one PR

@marcushoff you can add more changes to the same patch-2 branch :)

got it, I'll try that next time :)

For the sake of review and clarity I have to insist on asking you to merge the two PRs...

Created new PR #1716


Release a version with OpenTSDB support

I would love to see a release, complete with container image, that includes OpenTSDB support, now that it's present in the master branch. Alternatively, I would appreciate an estimate on when we could expect it, in case we need to go ahead and make unofficial builds and images in the mean time.

Rather than unofficial builds/images, we'd love contributions for that to heapster itself if you're up for that?

I am unclear about how I would go about making a build that gets pushed out through the official channels.
The code for OpenTSDB is there (not mine), there are no bug reports against it as far as I can see, and it's just not made available in any of the official images.

Heapster v0.20.0 (currently we have an alpha release candidate) will have OpenTSDB enabled.

Is there an official docker image about OpenTSDB?

Heapster in versions 1.0, 1.1, 1.2 support OpenTSDB. Here you can find an info how to configure the sink. Providing image to with OpenTSDB itself is our of the scope of Heapster project.


Use non-template-template parameter in UniqueHandle

If needed, UniqueHandle's template-template argument can be rewritten as regular typed argument. There could be subjective pros and cons to each, but in the end same things are possible using either of two semantics.
We can change it while both changes are in betas (Identity and Core, Identity depends on Core).
But the Core Beta has already shipped, while Identity Beta ships tomorrow, so don't merge this until tomorrow after the Identity release.
Regular template parameters are supported by current ApiView parser implementation, while for template-template parameters, ApiView needs to be updated.

I'd leave the decision mostly up to @LarryOsterman, but any team member feedback is appreciated. As for me, I'm fine either way. There are cases when you can't do without template-template parameters, this one is not one of them.


Remove scaling limited from error map

Problem:
The ScalingLimited will set to true when the target workload's
cpu useage is zero.
In this case, we should not set state to error.
Solution:
Remove ScalingLimited from reverseErrorMap.
Related issue:
https://github.com/rancher/rancher/issues/20955

LGTM

LGTM

lgtm


Fix week calculation issue

Assigning to @bp323 for review

Coverage remained the same at 100.0% when pulling 01d87f42733d0fa5963ffe45d4172996918cd045 on Coenego:weekCalculation into 09192e1e34fe31ed01bd07f1ee2c3580400b7d46 on CUL-DigitalServices:master.

Verified and merged.


A Communication Group （建一个交流群）

It's a great job! How about creating a community group ?
I have created a Discord group, click to join. I suggest the author include this link in the Readme. There’s no ill intent in creating this group, and if the author needs it, I’m willing to transfer the group admin position.
https://discord.gg/h57gYkQ38A
For Chinese users, you can also Join the Wechat group. (Chinese: 中国同胞可以加下面这个群，失效请联系微信 dreamingforhope )

Thank you for your support ! But I'm not sure if the team have the bandwidth to handle a discord cc @taprosoft . Maybe enable Github Discussion instead ?

Both is ok! I just found many trending projects have their community group. Eg: https://github.com/nocodb/nocodb
Sometimes people's real-time discussion may bring new ideas

Yeah I understand and also have thought about it. But the team is really thin, it might mostly be users helping each other I'm afraid. But ultimately, let's have @taprosoft and the team decide this.

May be we have to assign a community manager for this @lone17 ... But let see what us can do.

Good suggestion, @chmod777john
We'll work on that.

群过期呢


Adds data_uri helper which seems more in line with the Rails asset_data_uri method.

The existing helper outputs "url(...)" which is kind of weird, and uses "url" which isn't totally obvious.

What is the use case for not having a the uri wrapped in a url()?
The policy here has been to follow the rails convention so that a single set of documentation and knowledge is portable between views, javascripts, and stylesheets so deviating from their naming concerns me.


recreate control if Kendo API not available

if you run the sample of the recreate branch, you can see that when you click on the 'change icon button' (see image below), that it should change the icon of the first button, but this doesn't happen. There is also no API from kendo (like $(button).icon("newIcon")) to let kendo update the icon.

I agree that KendoUI does not provide API to change button image (see http://demos.telerik.com/kendo-ui/button/images as their own demo buttons with images). I believe that Charles did not follow exactly the Kendo's original demo, but did just this

So, no need for change button (and if there was one, it should be on the "API" tab, not "Images" tab)

The "change icon" button is just for demonstrating the issue. When you click this button, our wrapper should "see" that the icon property of the first button has changed, and it should re-initialize the kendo button with the new icon, but it does not do this.

So, you are saying that this issue is a bug against Aurelia Component?

Yes, if we allow you to bind to a icon property, then we should also let kendo know when the icon changes. since this can't be done with an API, we have to recreate the control perhaps

That I agree with - as it would be bad to hide behind incomplete Kendo API when questioned by Aurelia developers. However, that might be an expensive price to pay  as I am not sure what all controls would have such deficiencies. Kendo folks probably decided that if someone wants to change the icon, let him recreate the whole button - as this could be a reasonable way to address such issue

You have made a good point, but I think this could be very confusing for developers using our wrappers. We need to find out if other controls have the same problem, so we can figure whether or not it's important to fix this.
After we know how we're going to deal with the templating stuff, we can look into this a bit more. But i'd like to do this before our first release

We need to find out if other controls have the same problem, so we can figure whether or not it's important to fix this.

Yes, that is the proper response :-) - it is the exact opposite of what @jedd-ahyoung advertised (while he was still around): let's just go do something rather than "looking around".
Leaving this issue open as a reminder

Mail conversation with Telerik:

Hello Petyo,
We are now wondering why there is no 'API' method to change the 'icon' property on a button. The button has an "enable" function, but nothing to change the icon. Is this a design decision, or has it just not been implemented yet?
We see these "missing" API methods in many controls, and for many properties. If we understand why this is the way it is, then we can decide on whether or not to fix this in our wrappers.
Thanks,
Jeroen Vinke.

Response:

Hi Jeroen,
this is more or less a heritage from the 2011 jQuery plugin era, which we get a lot of flak for :(.
Most widget configuration options can be set only initially, and can't be changed afterwards. Back then, the rendering mechanism of the widget was not very sophisticated; We also had to support progressive enhancement, which has gone out of fashion since then.
We work around this in our angular wrappers, using "rebinding" (read - destroy and recreate) of the widget when an option is changed. I would not recommend that, though.

I propose the following: do not recreate the entire control when a single property changes, but allow the user to call a function on our wrapper to trigger it to recreate.

I propose the following: do not recreate the entire control when a single property changes, but allow the user to call a function on our wrapper to trigger it to recreate.

This is a very reasonable proposal.

I may have said this before, but I also want to expose the Kendo widget, so users can talk to it directly if necessary.
If we implement an enable function in the button wrapper, which would just call enable on the Kendo control, then the user could do this:
// my-page.html
<button au-kendo-button="enable.bind: enabled" au-kendo-button.ref="button">Click me</button>

// my-page.js
somethingHappened() {
  button.enable(true);
}

If we somehow forget to implement this function, then users could do this still do this via a widget property:
// my-page.html
<button au-kendo-button="enable.bind: enabled" au-kendo-button.ref="button">Click me</button>

// my-page.js
somethingHappened() {
  button.widget.enable(true);
}

Notice that button.enable(true); has changed to button.widget.enable(true);. We should have both options available, a function proxy inside our wrappers, and have a widget property on our wrappers that contains the actual Kendo control.

You continue to amaze me with the depth of your thinking - even in the midst of the excitement that we are so close to show this puppy to the world.
Don't let me spoil your focus with my signs of appreciation, though :+1: So, while I do not remember ever discussing this, please implement it by all means.

Recreating the control is now possible through the recreate function, and the instead of the _component property we had, it is now widget

I knew that you will soon overrun me in my plans to document what we have !!


Slow DOM updates if using unshift on an array

I have a posts property that is an array of post objects displayed on the page. I have navigation buttons on the page that will make an api request and add multiple posts to the array, either at the start or the end of the array depending on which way they are navigating:
// Insert results
if(page == 'prev') {
  this.posts.unshift(...response.posts);
} else {
  this.posts.push(...response.posts);
}

When Array.push() is used it looks like the new appended objects get new appended elements in the DOM, this is really fast.
Alternatively when Array.unshift() is used it seems new elements don't get prepended, instead it looks like the current elements stay the same and any extra elements are appended to the DOM so the count of DOM elements match the array length. The properties for each object in the DOM are then updated to be in sync. This seems much slower.
This is causing issues as I have an array observer listening for updates on the posts array. When it updates I need to check some stuff in the DOM. This works fine for Array.push() but with Array.unshift() rendering hasn't finished so I get incorrect DOM results.
Example:
this.arrayObserver(this.posts).subscribe(() => {
  this.taskQueue.queueMicroTask(() => /* DOM stuff here */);
});

This worked in older versions of Aurelia. It would appear this is some sort of performance improvement to try and reuse elements, however it's definitely degrading performance in this use case.
Is there some sort of Aurelia event I can hook into that will fire when all Aurelia DOM stuff is finished for this array update? Or would I need to downgrade Aurelia to get this working again?

@niieani I just sent you a core team invitation. Pending your agreement to join, this seems like a great issue for you to start work on 😄

@martingust @jdanyow Would you assist @niieani as needed?

Just to add to this, it would be great if there was an event when rendering was done but it would also be great if you could set templating to not reuse DOM elements as that can sometimes be undesirable.
For example, when you change an image source in some browsers they will display the old image until the new image is loaded. So reusing an img element with a binded src attribute appears really laggy, you get duplicate images and then after a short delay one updates to the correct image.
Alternatively there amy be a better way of handling this scenario such as removing the src attribute before updating it.

@lukechilds I've tried recreating your issue, but so far I'm seeing that Aurelia does in fact prepend DOM when doing .unshift(...). Aurelia does also have in place processing (_inPlaceProcessItems) for array repeats, but it is only used when both of these conditions are satisfied:

views don't require a lifecycle (i.e. not a component, no animation, etc.)
you splice the array by removing and adding values at the same time / very quickly in order (or recently also when you replace the array with a new instance that has some different elements)

This is what happens when you insert an element into the array - note the insertNodesBefore is there.
unshift() should behave similarly to push(), with the difference of the index, i.e. it adds a new view at an index provided, which is 0. What happens if you replace your unshift by splice(0, 0, ...response.posts) ?
Could you provide a more comprehensive piece of code showing this problem upon forking this?
As for your other issue, to force a View lifecycle on your img element you could create a simple custom attribute for just that. Take a look at the if attribute. What you'd want to do is whenever the value changes you remove the View from the ViewSlot and add it back.

@niieani Thanks for looking into this. The the elements data gets prepended eventually but the actual extra DOM nodes appear to get appended and then afterward the properties are all updated.
I've added a comprehensive example here
I'm using an alert to halt execution so you can see what's going on at that point in time. You should see in the example that if you append items every things good but if you prepend there is duplicate data when the alert halts execution. As soon as you continue execution all the values are recalculated.

Just to clarify, the actual issue here isn't the way the templating works, it's that I don't think there's a way to reliably check the DOM after adding elements. If you remove the alert everything happens fast enough that both ways look fine.
It's just trying to check the DOM by adding cod to the task queue after the update works if you're appending but not if you're prepending. You could fix this at the template level by doing some sort of deep compare on all of the objects and working out where the new elements should be inserted in the DOM but I'm guessing that would have a pretty steep performance penalty.
Ideally I'd say the templating should be left as is and there should be some sort of event added to bind to that fires when templating has finished for a property.
So instead of doing this:
this.arrayObserver(this.posts).subscribe(() => {
  this.taskQueue.queueMicroTask(() => /* DOM stuff here */);
});

You could do something like this:
this.arrayObserver(this.posts).subscribeRenderUpdate(() => /* DOM stuff here */);

Maybe you would have to subscribe to a view rather the actual posts object for this to work but you get the idea.

To answer your question above I get the issue with both Array.splice and Array.unshift.

Have you tried taskQueue.queueTask(() => /* DOM stuff here */)?

@lukechilds Okay, sure. Thanks for your example. I see both the problem and room for improvement. As for a workaround, I'll send you something in a moment.
@EisenbergEffect indeed he has.

@EisenbergEffect  @niieani
No I haven't, queueTask() does indeed fix the problem.
Thank you very much!

@niieani I didn't see any use queueTask, only queueMicroTask.

@lukechilds But, is there still a performance issue?

@EisenbergEffect that's right, my mistake. As for the performance issue, we could add special a way to handle splices when there are only insertions happening and/or a way to manually force the repeat to enable view-lifecycle. Depending on how changes happen in the DbMonster, the first solution might slightly lower its refresh speed.

No, that's working perfectly. I mean it must be slightly slower than before the update because it's no longer rendered in time for queueMicroTask() but using queueTask() there is no visible lag.
There is a perceived performance issue when image elements are reused due to default browser behaviour mentioned above but I'll open a separate issue in the templating repo for that.
If it's not too much trouble could you briefly explain when you should use queueTask() over queueMicroTask() and vice versa? I've checked the docs but it just says it's "ASAP execution" vs "Turn-based execution". Could you expand on that?

@niieani Talk with @jdanyow and @martingust about that to see what they think.
@lukechilds Micro tasks are put on to a draining queue that runs as soon as possible, before rendering. Macro tasks (or just tasks) are executed after the next "paint" or as soon after that as the browser will allow.

@lukechilds If you want to force the element to be added instead of appending & reusing the old ones you can make the element use a view-model implementing attached method. Like this: https://gist.run/?id=78e6ce914568a9b336d05eb301dc0c14
Now it works even with queueMicroTask.

@niieani Thanks, but the above gists were very simplified examples.
In my actual application I have a custom element that has array of objects that are child custom elements. I add multiple child custom elements in one go and this should trigger some logic on the parent custom element once per batch add.
If I understand the attach() method correctly, it would fire once on the parent element when it is added and once on each child element when they are added. There would be no way to use it to fire once when a single batch of children have been added would there?

I agree that manually adding to the taskQueue doesn't seem ideal though. If there's a better way to achieve this functionality with Aurelia I'm all ears 👍

@lukechilds I see. As far as I know, there isn't any event that fires after a repeat has finished re-rendering. I suppose your way is the closest to achieving that.
However it would be trivial to add something like that to the repeat custom attribute to achieve the following:
<div repeat.for="item of items" repeat-items-changed.bind="someMethod">
  ${item}
</div>

someMethod from the view-model would fire every time the repeater finished updating all its views. Would something like that solve your case?
@EisenbergEffect Do you think this is something worth implementing?

That's an interesting idea....the repeater could just raise a custom DOM event. @martingust @jdanyow What do you guys think?

@niieani That would be perfect!

@lukechilds Here's a monkey-patched implementation:
https://gist.run/?id=6d2d446e55947d564daae507800da5c4
Can you try if this works for you?

@niieani Yeah, that works well :)

@EisenbergEffect note that a custom DOM event doesn't really help here because its not clear on which element it should be raised. Instead the above implementation simply triggers a method from the ViewModel, similarly to a how a matcher definition is extracted from the bindingContext.
If @martingust and @jdanyow agree that this is something useful, I'll clean up the code, add tests and make a pull request to templating-resources.

Good point. Let's see what the others say.


cmake: modules: add zephyr_library_relocate, to relocate zephyr library

Add zephyr_library_relocate, a helper macro intended to allow entire Zephyr libraries to be relocated to a new memory region.
This is implemented via a two step process, to ensure that sources added to a library after relocation is requested still are relocated:

zephyr_library_relocate adds a custom target property to the library with the relocation region and flags
the root level CMakeLists.txt captures this property while parsing all Zephyr libraries, and adds all library source files to the code relocation target

Signed-off-by: Daniel DeGrasse<EMAIL_ADDRESS>

Please update docs and samples

@tejlmand I've taken another look at this PR and updated zephyr_code_relocate to accept CMake generator expressions directly. I've also extended the tests/application_development/code_relocation to test relocating a library, as well as relocating files using a generator expression. Welcome to any feedback you have for these changes!

@carlocaione can you please take a look?

@tejlmand please re-review


Population Density 2011 web app needs more information

Is this item supposed to be in Release Review? This item (id:f1ded0d8cbef405582e189c50698dfab) cannot be released without a description and thumbnail.

If this isn't resolved by 1300 pst this item will not be released with the Ops Server. Upon review of the app I am guessing this item was mistakenly shared with release review.

Removed from Release Review - was definitely shared by mistake.


Investigation - Replace Ray

Investigate the possibilities of replacing ray with some other form of distributed execution.
@Adaickalavan Please fill out with more information

Hey @Gamenot @Adaickalavan -- I'm from the Ray team, just dropping by to say hi!
I would love to better understand this issue -- could you tell me what are your current issues with Ray?


Allow mutation of Descriptor documents to add device Config fields before leaving the RunEngine

Context
As discussed at NOBUGS, Will Smith is mutating the StartDocument to add NeXus specific metadata about the devices (their NXClass, metadata fields e.g. pixel size) [with the inject_md preprocessor?]. This metadata is device specific and invariant over the lifespan of the scan: a close fit to a Configuration field on a DescriptorDocument. Allowing mutation of a Descriptor document would allow us to iterate over the devices in a stream, reducing the amount of foreknowledge required to assemble the metadata. Allowing this as a pre-processor or hook  would allow the data to remain external to the Device class (coming from a file/database/service etc.).
However, pre-processors mutate Msgs leaving the RunEngine and not the event-model Documents leaving the RunEngine to any subscriptions.
Expected Behavior
There is a hook for mutating Descriptor documents prior their emission. Any documents that reference the Descriptor (Events, StreamDatums) should not be able to be emitted until the Descriptor document is finished. The hook may only allow adding configuration fields and construct the appropriate DataKey for the field.
Current Behavior
This may only be possible through an involved usage of a Subscription to the RunEngine, which would then itself need to be subscribed to by any subscription that requires the mutated document.
Possible Solutions

A Dispatcher implementation that can mutate the documents prior to emission, and the ability to pass a specific Dispatcher into the RunEngine?
A pre-subscribe method on the RunEngine installs a mutation method onto the Dispatcher?

Could also be a usage for the DocumentRouter in event_model?


soc: infineon_cat1: Added FPU definition for PSoC6

Added FPU definition to the PSoC6 soc to enable the definition of hardabi for our boards
cc: @ifyall @npal-cy

@carlescufi , i think we can merge this, CI failures are not related to PR changes/platform.
Regards,
Nazar


XML grammar autodetection failed for Visual Studio project files

Description
Atom doesn't detect Visual Studio project file as XML file and doesn't apply XML syntax highlight automatically.
Steps to Reproduce

Open any *.vcproj file

Expected behavior: [What you expected to happen]
XML syntax highlight applied
Actual behavior: [What actually happened]
No XML syntax highlight
Versions
Atom 1.5.3

Thanks for the report! The fix is available on master, and will become available in a future release.

Side note: The vcproj extension was deprecated a few years ago, and was replaced with vcxproj (which the language-xml package already recognises). Visual Studio is supposed to automatically convert any vcproj file to vcxproj when opened.


thats not an issue. I just want to say Thank you. lots of loveeeeee

i am not able to understand the final step please help me

Which final step?

try "resumeio2pdf " link"

or you can just do
go run main.go https://resume.io/r/...


Migrate to GitHub actions #135

hello
I implemented a GitHub action CI pipeline which does the same thing as the existing Travis CI pipeline..
overall it includes dependency installation, linting (flake8), tests (pytest), doc-build (sphinx), test coverage report (codecov).
so please take a look at it and merge it if possible..
Thank you..

hello I implemented a GitHub action CI pipeline which does the same thing as the existing Travis CI pipeline.. overall it includes dependency installation, linting (flake8), tests (pytest), doc-build (sphinx), test coverage report (codecov). so please take a look at it and merge it if possible.. Thank you..

Amazing! Thank you. I will try to work on this ASAP.

Thank you..


MOB-20. Details screen.

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).


Update validation for the cluster name length

Issue #, if available:
#7302
#8016
Description of changes:
Updated the cluster config validation for the cluster name length to be not more than 35 characters.
Testing (if applicable):
Tested locally with a cluster name length of more than 35 characters on docker provider
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/approve

/cherry-pick release-0.19


Question&Request：how to find voice files？

Hello, I now know that SaintCoinach can export game files with known path with raw instruction, but I don't know how to obtain the path of the desired file. Where can I find the overall file directory structure of the game? Or, if possible, may this tool add the function of export all known files?
At present, I can deduce some 2.0-2.5 files according to ffxivDataExplorer and godbert, but the same method does not seem to be applicable to 3.0 and later files, so I fell into a deaend. As shown in the picture I provided, I tried to export different patchs of voice files, but only 2.0 successfully found the corresponding file.
This is just a personal request. I would appreciate it if you could reply.

Well, I found the file I wanted. The voice file is stored in the updated ex package of each expansion since 3.0, so I can't find the corresponding file by following the path of cut / FFXIV.
But I still have a question. Is there an instruction for me to export .scd files as .wav?

I solved this myself.


lock.py: allow selection of machines to list by os-type/os-version

Also, clean up the status selection a tiny bit with winnow()
Signed-off-by: Dan Mick<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1491/
Tests passed for this pull request.

works for me (to list only rhel 7.1 machines)!

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1498/
Tests passed for this pull request.


In DFDC data set, there will be moving face images in some videos. How do you deal with it?

Looking forward to your reply.

What do you mean by "moving face images"?


^^ sbt-crossbuilding does not set scalaVersion correctly

steps

create an sbt plugin: sbt new sbt/sbt-autoplugin.g8
set sbt.version=0.13.16-M1
try to cross-compile: ^^ 1.0.0-M5 compile

problem
the scripted plugin isn't resolved for 1.0 because the scalaVersion is incorrect:
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: org.scala-sbt#scripted-sbt_2.10;1.0.0-M5: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::

expectation
The scalaVersion is set to a version compatible with the sbt version targeted

Can someone add this to the sbt 1.0 RC milestone? Solving this issue inside sbt is a must before a RC.

This is correlated to the request to requiring scalaVersion be set, aka not defaulting scalaVersion: https://github.com/sbt/sbt/issues/2795
FWIW I solved this for myself by defining a SbtVersionSeries sum type:
import sbt._, Keys._

sealed trait SbtVersionSeries
case object Sbt013 extends SbtVersionSeries
case object Sbt1   extends SbtVersionSeries

object CrossBuildingPlugin extends AutoPlugin {
  override def requires = plugins.JvmPlugin
  override def trigger  = allRequirements

  object autoImport {
    val sbtPartV = settingKey[Option[(Int, Int)]]("")
    val sbtVersionSeries = settingKey[SbtVersionSeries]("")
  }
  import autoImport._

  override def globalSettings = Seq(
    sbtPartV := CrossVersion partialVersion (sbtVersion in pluginCrossBuild).value,
    sbtVersionSeries := (sbtPartV.value match {
      case Some((0, 13)) => Sbt013
      case Some((1, _))  => Sbt1
      case _             => sys error s"Unhandled sbt version ${(sbtVersion in pluginCrossBuild).value}"
    })
  )
}

Thanks for that snippet. I saw something similar in sbt-structure and I wanted to use it, but was not sure it solved the issue.
Do you have any idea how we can solve this upstream?

I'll give it some thought.

I had a long look and attempt to fix this, but, depending on what kind of fix we want, I don't see an obvious way to fix this. Here are some notes:
The way the plugin cross-building functionality works is by defining sbtVersion in GlobalScope in pluginCrossBuild here: https://github.com/sbt/sbt/blob/v1.0.0-M6/main/src/main/scala/sbt/PluginCross.scala#L44
My attempts were to define scalaVersion in accordance to the sbt version given, but there's no clear and simply way to do that. For instance this massive block of code is how ++2.12.2 changes scala version: https://github.com/sbt/sbt/blob/v1.0.0-M6/main/src/main/scala/sbt/Cross.scala#L191-L297. While this other is how set every works: https://github.com/sbt/sbt/blob/v1.0.0-M6/main/src/main/scala/sbt/internal/SettingCompletions.scala#L27-L58.
One other solution I looked at is that of defining, defaulting and using a set of scala version keys, like so:
val scala29 = SettingKey[String]("scala29", "The version of Scala 2.9.x to use", BSetting)
val scala210 = SettingKey[String]("scala210", "The version of Scala 2.10.x to use", BSetting)
val scala211 = SettingKey[String]("scala211", "The version of Scala 2.11.x to use", BSetting)
val scala212 = SettingKey[String]("scala212", "The version of Scala 2.12.x to use", BSetting)
val scala213 = SettingKey[String]("scala213", "The version of Scala 2.13.x to use", BSetting)
val scala214 = SettingKey[String]("scala214", "The version of Scala 2.14.x to use", BSetting)
val scala215 = SettingKey[String]("scala215", "The version of Scala 2.15.x to use", BSetting)

That way a plugin author wouldn't define scalaVersion any more, but instead would define:
sbtPlugin := true
 scala210 := "2.10.5"
 scala212 := "2.12.1"

and then use ^^ without issue.
Un-assigning myself for now.

Note that this affects not just ^^ usage but also ^ (and therefore crossSbtVersions) usage (https://github.com/sbt/sbt/issues/3305)

btw this actually works if you just comment out scalaVersion:
// scalaVersion := "2.10.6"

We can potentially override Scala version in ^^ again.

Fixed in https://github.com/sbt/sbt/pull/3362.


Overlapping partitions in DataFrames

For resampling and sliding window operations on series we could use logic to overlap partitions slightly.  This would be analogous to dask.array.ghost but would overlap based on index value (e.g. overlap by 10 minutes) rather than by number of elements (e.g. overlap by 10 items).
For time series sliding window operations often only need ghosting in one direction, so that might be worth inserting.
It's still not clear to me that people have time series long enough to warrant spending time on this.  @ssanderson and I were talking about this on gitter.

It's still not clear to me that people have time series long enough to warrant spending time on this.

I think high-granularity financial data and/or analytics data would fall into this category.
What's less obvious to me is whether people who have time-series data this large are willing to pay the additional overhead of working with it in a richer dataframe-like format versus working with it in a format more like dask.array.
In our case, because we care about the market calendar, it's more often correct to think about trailing windows on N bars of data, rather than N days of data.  For example, a 3-day trailing window is not a very useful construct when you don't have data on weekends.
You had also mentioned that anisotropic ghosting for dask.array might produce performance wins in a distributed context.  I take it you think there are less substantial wins to be had in a single-machine context?

I think high-granularity financial data and/or analytics data would fall into this category.

How many entries do these datasets have?
N bars of data is certainly easier than N days.  If there isn't value to the N days case then we can happily ignore this application.
Regarding anisotropic ghosting, this significantly reduces the number of necessary communications between tasks (the graph is less tangled.)  In a single machine communication is mostly free, so we don't care about a tangled graph.  It might be that in a distributed context communication is more of an issue.  I would want someone to report that it's slow before I devoted time to anisotropic ghosting though; it's entirely possible that it's not a bottleneck.

How many entries do these datasets have?

For something like a 10 year OHLCV dataset, minutely data will have on the order of (10 years * ~250 market days per year * ~400 minutes per day * 5 fields) = ~5 million data points per asset.  At any given time you have between 8000 and 10000 equities if you only care about the US, so that leaves you with on the order of at least 40 billion data points, which translates to 160 GB

Do you care about sliding window computations that cross securities?

Do you care about sliding window computations that cross securities?

For the most part no.  There are use-cases for things like pairwise correlation matrices, but since there isn't really a natural order on that axis of the data, it doesn't really make sense to do rolling windows.

Then do people usually split up the 160GB into 10000 pieces, one per security?

I guess I'm trying to figure out if people want to parallelize within each time series or simply across securities.  We can support both but across securities is simpler.

We currently block our data by time, but that's partly a historical artifact, and partly for ease-of-deployment reasons: it's much easier to update the most recent data file than it is to update a file for every security.
As far as parallelization goes, I'd imagine that per-security (or per-N-securities) would be the way to go.  I think if dask supported moving-window computations, the user benefits would come more from handling nitty-gritty boundary-traversal details than from parallelization, which might be a problem you're less interested in solving.

OK, the fact that time series are often appended to on a daily basis seems like a decent reason to support moving window computations, even if not made necessary by memory constraints.

We've managed this is relevant computations by bringing together two or three neighboring dataframes and then loc-ing down.  This has proven sufficiently effective that I'm now closing this.


Make the repo standalone, not a fork, to i.e. allow code search

Description:
This repo is a fork of yet another dead (2 years) repository. One of the issues is some github features are not supported.
Current behaviour:
Code is not searchable

Expected behaviour:
Code can be searchable.
Steps to reproduce:
Just try to search some phrase in the repo, then click on the "code" in the left top corner.

Just sent support request as described here: https://help.github.com/articles/why-are-my-contributions-not-showing-up-on-my-profile/#commit-was-made-in-a-fork

They make magul/volontulo separate network, not CodeForPoznan/volontulo :/

Done.


gitleaks: 8.8.4 -> 8.8.5

Description of changes
https://github.com/zricethezav/gitleaks/releases/tag/v8.8.5

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 174787 run on x86_64-linux 1

1 package built:

gitleaks


Remove/delete shape

Thanks for a great package - good documentation and a clean implementation!
I wonder if a remove/delete API for shapes is on the roadmap? In my intended usecase the world is mostly static, but sometimes a line it split in two. It seems like a waiste of resources to rebuild the entire datastructure when just one node is changed/deleted. What's your thoughts on that?

I agree that this would be helpful. I could imagine methods to update/remove single shapes, and/or a static flag on shapes (so they would not be affected by clear).
Due to the recursive nature, skipping the rebuild could result in nodes without objects, but the benefits should outweight that.

https://github.com/jonit-dev/quadtree-ts

I forked and added the remove behavior

I refactored your code a bit and added tests and docs. In the simple example there are now "remove" buttons:
https://timohausmann.github.io/quadtree-ts/examples/simple/
I also added a helper function "join" that can safely cleanup the vacant subnodes. This can be skipped with a "fast" flag.
Docs:
https://timohausmann.github.io/quadtree-ts/documentation/classes/Quadtree.html#remove
If you have the time I'd be happy to hear your feedback.
npm install -S @timohausmann/quadtree-ts@latest

Alright fine, I will slap on an update method and example aswell:
https://timohausmann.github.io/quadtree-ts/examples/update/
https://timohausmann.github.io/quadtree-ts/documentation/classes/Quadtree.html#update
Please let me know if your issue isn't resolved.


Refreshed maintenance status list

Current documentation state did not reflect well statuses that customers see in the emails and ARG explorer. Latest CRI resulted with documentation repair item to address this better.

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


feat: checkbox

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Broken link

https://github.com/Azure/azure-storage-python/tree/master/azure-storage-queue links to API Reference which doesn't exist.

Hi @letmaik, thanks for spotting this issue!
I'll fix them right away.

This is fixed.


Change manager server getServerAddr to configure STUDIO_IP

Proposed changes
Problem Summary:
When the server has multiple IP addresses, the server sends a startup command to the client, and the server address is invalid,Configure the manager IP directly into the manger configure.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

choose configure write fixed STUDIO_IP or automatically ip address


How to get rid of the validator set?

Hello!
I'm a member of the Kusama Society and we are developing a dedicated interface as a development plan for the society. We want to create some integration tests and for that we want to run a development node reflecting the state of Society members at some point in time.
I'm trying to run a development chain forked off from Kusama. So far I synched an archived node and created the fork against said node. Apparently all went well, but I'm unable to produce blocks and execute extrinsics. I suppose it's because the chain is waiting for the current Kusama validator set. How do I change the validator set and get control over the chain?
I tried editing fork.json but I don't understand its structure.

The script automatically replaces the validators with Alice (unless some recent changes to substrate broke this).
Make sure you running the binary with --alice for it to generate blocks?


HBASE-22947 Client Should not be able to disable Systems table

This change is to  restrict clients to  perform a DDL/disable operation on a system table

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 27s
Docker mode activated.

For me, sometimes I will disable a table and then enable it to fix the incorrect state of its regions. This could also be used for system tables other than meta table. So I think we need to make sure that only admin users can disable system tables?

For me, sometimes I will disable a table and then enable it to fix the incorrect state of its regions. This could also be used for system tables other than meta table. So I think we need to make sure that only admin users can disable system tables?

Looks like Admin access is taken care of by
getMaster().getMasterCoprocessorHost().preDisableTable(tableName);

It is used by HMaster:
  @Override
  public long disableTable(final TableName tableName, final long nonceGroup, final long nonce)
      throws IOException {
    checkInitialized();

    return MasterProcedureUtil.submitProcedure(
        new MasterProcedureUtil.NonceProcedureRunnable(this, nonceGroup, nonce) {
      @Override
      protected void run() throws IOException {
        getMaster().getMasterCoprocessorHost().preDisableTable(tableName);

        LOG.info(getClientIdAuditPrefix() + " disable " + tableName);

        // Execute the operation asynchronously - client will check the progress of the operation
        // In case the request is from a <1.1 client before returning,
        // we want to make sure that the table is prepared to be
        // enabled (the table is locked and the table state is set).
        // Note: if the procedure throws exception, we will catch it and rethrow.
        //
        // We need to wait for the procedure to potentially fail due to "prepare" sanity
        // checks. This will block only the beginning of the procedure. See HBASE-19953.
        final ProcedurePrepareLatch prepareLatch = ProcedurePrepareLatch.createBlockingLatch();
        submitProcedure(new DisableTableProcedure(procedureExecutor.getEnvironment(),
            tableName, false, prepareLatch));
        prepareLatch.await();

        getMaster().getMasterCoprocessorHost().postDisableTable(tableName);
      }

      @Override
      protected String getDescription() {
        return "DisableTableProcedure";
      }
    });
  }

Looks like Admin access is taken care of by
getMaster().getMasterCoprocessorHost().preDisableTable(tableName);

It is used by HMaster:
  @Override
  public long disableTable(final TableName tableName, final long nonceGroup, final long nonce)
      throws IOException {
    checkInitialized();

    return MasterProcedureUtil.submitProcedure(
        new MasterProcedureUtil.NonceProcedureRunnable(this, nonceGroup, nonce) {
      @Override
      protected void run() throws IOException {
        getMaster().getMasterCoprocessorHost().preDisableTable(tableName);

        LOG.info(getClientIdAuditPrefix() + " disable " + tableName);

        // Execute the operation asynchronously - client will check the progress of the operation
        // In case the request is from a <1.1 client before returning,
        // we want to make sure that the table is prepared to be
        // enabled (the table is locked and the table state is set).
        // Note: if the procedure throws exception, we will catch it and rethrow.
        //
        // We need to wait for the procedure to potentially fail due to "prepare" sanity
        // checks. This will block only the beginning of the procedure. See HBASE-19953.
        final ProcedurePrepareLatch prepareLatch = ProcedurePrepareLatch.createBlockingLatch();
        submitProcedure(new DisableTableProcedure(procedureExecutor.getEnvironment(),
            tableName, false, prepareLatch));
        prepareLatch.await();

        getMaster().getMasterCoprocessorHost().postDisableTable(tableName);
      }

      @Override
      protected String getDescription() {
        return "DisableTableProcedure";
      }
    });
  }

AccessController has this:
  @Override
  public void preDisableTable(ObserverContext<MasterCoprocessorEnvironment> c, TableName tableName)
      throws IOException {
    if (Bytes.equals(tableName.getName(), PermissionStorage.ACL_GLOBAL_NAME)) {
      // We have to unconditionally disallow disable of the ACL table when we are installed,
      // even if not enforcing authorizations. We are still allowing grants and revocations,
      // checking permissions and logging audit messages, etc. If the ACL table is not
      // available we will fail random actions all over the place.
      throw new AccessDeniedException("Not allowed to disable " + PermissionStorage.ACL_TABLE_NAME
          + " table with AccessController installed");
    }
    requirePermission(c, "disableTable",
        tableName, null, null, Action.ADMIN, Action.CREATE);
  }

Which is covered by
getMaster().getMasterCoprocessorHost().preDisableTable(tableName);

from HMaster

AccessController has this:
  @Override
  public void preDisableTable(ObserverContext<MasterCoprocessorEnvironment> c, TableName tableName)
      throws IOException {
    if (Bytes.equals(tableName.getName(), PermissionStorage.ACL_GLOBAL_NAME)) {
      // We have to unconditionally disallow disable of the ACL table when we are installed,
      // even if not enforcing authorizations. We are still allowing grants and revocations,
      // checking permissions and logging audit messages, etc. If the ACL table is not
      // available we will fail random actions all over the place.
      throw new AccessDeniedException("Not allowed to disable " + PermissionStorage.ACL_TABLE_NAME
          + " table with AccessController installed");
    }
    requirePermission(c, "disableTable",
        tableName, null, null, Action.ADMIN, Action.CREATE);
  }

Which is covered by
getMaster().getMasterCoprocessorHost().preDisableTable(tableName);

from HMaster

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
4m  5s
Docker mode activated.


CI: bump react-native-android docker image to 5.0

Summary
Bump react-native-android docker image to 5.0, which includes JDK 11.
Changelog
[Internal] [Changed] - bump react-native-android docker image to 5.0, which includes JDK 11.
Test Plan
Android CI must be green

Seems like the CI is failing as we specify UseCGroupMemoryLimitForHeap and we should instead specify UseContainerSupport:
https://github.com/facebook/react-native/blob/1465c8f3874cdee8c325ab4a4916fda0b3e43bdb/.circleci/config.yml#L44
More on this here: https://support.circleci.com/hc/en-us/articles/360048152911
Don't you mind giving it a try?

@cortinico this PR is not ready yet

@cortinico this PR is not ready yet

Thanks for pointing out. I've converted it to a draft PR. Feel free to ping me once you're ready to land this 👍

When it's going to be ready can it be possible to also bump AGP then to 7.0.2 with this PR?

@cortinico could you please look into CI failure with JDK 11. It's strange.

solved the CI issue 👍


integration-cli: remove emptyfs

I attempted to run integration tests with the containerd feature flag enabled and noticed that testing hangs after a failed attempt to load the emptyfs image.
The image is created by the .ensure-emptyfs helper script - the problem is that it uses the old legacy v1 image format, which is not supported by the containerd and fails to import.
My initial reaction was to just change the script to create an emptyfs with a newer layout. It kinda works - there are few problems though:

The emptyfs has an architecture hardcoded to x86_64 - which with the current docker load implementation makes the image impossible to load in some scenarios (in my case it doesn't load anything on M1 because it's arm64). I can set the architecture based on the uname -m output, but it changes the configuration and so it hashes to a different hash than expected which impacts the image id.
Even when changing the docker load to import not-matching platform image, the image id is still not 11f64303... because containerd doesn't derive image id form config content. In this case the image is stored as application/vnd.docker.distribution.manifest.v2+json which is addressed by the sha256 sum of the manifest content.

I see that the emptyfs is a legacy of the ancient times when scratch was a fixed image and given that it's only used in some integration-cli tests I think we can just remove it completely and switch the tests to use some of the frozen images.
I replaced it with busybox, because it's already widely used in the integration tests.
Also I'm not sure what really to do with the TestInspectImage test. It checks whether the id of the equivalent of the legacy scratch image doesn't change for backwards compatibility -  which sounds like something we should still maintain for backwards compatibility.. but maybe we don't need it?
- What I did
Removed emptyfs which unblocks setup of integration tests with the containerd feature enabled

Closing in favor of https://github.com/rumpl/moby/pull/31


Reference-count ipset entries

To avoid crashing on error from ipset when we receive an Add before a Delete; fixes #2792
Note that a real, long-lasting, IP duplication is a real problem, and this change makes it much less likely that we would spot that.

I have manually tested this as described at https://github.com/weaveworks/weave/issues/2792#issuecomment-278941846

LGTM assuming:
// note no locking is required as all operations are serialised in the controller

indeed holds, but you may want to have someone else with more knowledge of Weave Net review it.


chore: Improve repr output

E.g., when using verbose hypothesis output, it will reduce the noise

Codecov Report

Merging #802 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #802   +/-   ##
=======================================
  Coverage   99.14%   99.14%
=======================================
  Files          59       59
  Lines        3876     3876
  Branches      668      668
=======================================
  Hits         3843     3843
  Misses         15       15
  Partials       18       18

Flag
Coverage Δ

#unittests
99.14% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/schemathesis/specs/openapi/links.py
100.00% <100.00%> (ø)

src/schemathesis/stateful.py
100.00% <100.00%> (ø)

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 b32216e...d5e376e. Read the comment docs.


Make $colors variable overwritable

The first 4200 lines of my CSS are generated by the @each-loop in materializecss's "_color.scss", containing every possible color shade from the MD palette.
The intention of MD colors, however, is to pick a primary color + an accent color -> 17 out of 19 colors are never needed once you have chosen 'your' colors.
My idea would be to overwrite the $colors variable with a list of only the needed colors.
But I can only overwrite variables which are declared as !default.
=> CHANGE REQUEST: please declare $colors in "_color.scss" as !default.
Regards,
Detlef

Please test #4784.

Is this a duplicate? Please check https://github.com/Dogfalo/materialize/issues/3612, https://github.com/Dogfalo/materialize/pull/4088 and https://github.com/Dogfalo/materialize/pull/4556

Not a duplicate imo, just adds the default keyword.

Closed via #4784


Make previosly executed config selected

After executing "Select and Start Debugging" this config becomes selected in "Run and Debug" tab. Need to provide similar feature for "Select and Run Without Debug".

There is not API for this: https://github.com/microsoft/vscode/issues/77138


Option to show horizontal scrollbar on top of div?

Make sure these boxes are checked before submitting your issue -- thank you!

[x] Check FAQ and Caveats
[x] Search if there's already one reported in Issues
[ ] Prepare a JSFiddle reproducing the issue

perfect-scrollbar JSFiddle

[ ] Provide a page or source code where the issue can be checked

You can change position of the horizontal scrollbar by simply changing position of the element with the ps__rail-x class.
For this specific case:
.ps__rail-x {
  bottom: auto;
  top: 0px;
}

JSFiddle


[ACS JobRouter] Add misssing descriptions

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[x] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

@jhendrixMSFT this is ready to be merged


How to setup Python to compile using Windows Bash

I want to use Ubuntu Environment for Compiling Python in Code, and i have installed Bash on Windows.
Is there any way i can compile the python using the Python in Bash and use all the libraries of python that are installed in Bash.
If anyone has JSON i can use, please share

This isn't the right place to ask those questions

@jrieken Why not? Looks to me like a more general question;
"How do I redirect calls to Bash on Windows instead of cmd.exe?". E.g a call to Python should use Bash on Windows when "terminal.integrated.shell.windows": "C:/Windows/sysnative/bash.exe".


Better type check when using array + union

Search Terms
type check
array
union
Suggestion
If we define a type like [null, string] | [number, null], we know that, if the first element is null, the second element is string, and if the second element is null, the first element is number. But, the type check can't understand it, what is less accurate than desired.
Use Cases/Example
When we creating a new function, we could want to return a value that is returned in success case, or an error if it failed. A simple way to do it is using an array, for example:
const createUser = (): [User, null] | [null, Error]  => {
  ...

Then, this function can return [User, null] or [null, Error]. But, the type check can't work very well:
const [user, error] = createUser()
if (user === null) {
  console.log(error.message) // type check mark a warning where,
                             // saying that "error" could be null, but it is impossible!
  return
}

// type check say that "user" never will be null. It is ok.

or...
const [user, error] = createUser()
if (error !== null) {
  console.log(error.message) // no warning
  return
}

// type check say that "user" could be null, but it is impossible

I think this is a duplicate #27497 (and others). Narrowing does not work across destructuring like this.

@jack-williams @weswigham Does we have an issue or pull request which we can track the fix about this issue?

I think #12184 is the canonical one.


Request: Allow only a part of the library to be imported

Dependabot created a pull request to one of our repositories to update from v6 to v7, and the changes in our output were as follows (as reported by bundlesize):
 PASS  ./bundlesize/app.js: 46.65KB < maxSize 75KB gzip (same as master)

 WARN  ./bundlesize/vendor.js: 289.01KB < maxSize 350KB gzip (5.59KB larger than master, careful!)

 PASS  ./bundlesize/app.css: 22.66KB < maxSize 50KB gzip (same as master)

 WARN  ./bundlesize/vendor.css: 3.97KB < maxSize 10KB gzip (864B larger than master, careful!)

This means updating sweetalert2 resulted in 5.59KB additional JS and 864B additional CSS in our build, while we don't use the new features. Granted this is not much on its own, but I do see this increasing over time.
Please allow us to only import a part of the codebase, so that we don't have to include code that we're not going to use.

Yep, that recalls me the discussions we've had about ESM and tree-shaking... but that requires important refactor/API changes. I think we, and our users, aren't ready for that, but it's a good thing to take that into consideration more seriously.
I'll try to take a look at the code base at some point to see what could be done with the current state of things.
But also, just to be sure, please check that you don't import the .all.js build of SweetAlert2 (that's the new entry point) and the CSS file at the same time.

But also, just to be sure, please check that you don't import the .all.js build of SweetAlert2 (that's the new entry point) and the CSS file at the same time.

That's a good point.

Changing the imports makes the javascript increase go from +5.59KB to +450B, a tenth of the original change!
Now it looks like this:
import 'sweetalert2/src/sweetalert2.scss';
import sweetAlert from 'sweetalert2/dist/sweetalert2.js';

I'd prefer to take both from src, but doing that would lead to an increased javascript size again. Reason for using src would be that we can run it through our own webpack configuration, which also generated sourcemaps for node_modules.

I'd prefer to take both from src, but doing that would lead to an increased javascript size again. Reason for using src would be that we can run it through our own webpack configuration, which also generates sourcemaps for node_modules.

You should be able to import from src:
import 'sweetalert2/src/sweetalert2.scss';
import sweetAlert from 'sweetalert2/src/sweetalert2.js';

When using src for the javascript, the css is not extracted correctly in our build which leaves styles inside both the javascript and css bundles. (var styles = {...}), so that's why I'm taking dist. It's a bit strange, but it works.
The tool is https://github.com/siddharthkp/bundlesize, and it's quite good but we're working on our own replacement. I just have to convince management to allow us to open source some of our tools :D

When using src for the javascript, the css is not extracted correctly

I wonder why is that? Is that because of SweetAlert2 or your build configuration? There shouldn't be anything CSS-related in the sweetalert2/src/sweetalert2.js.

The tool is https://github.com/siddharthkp/bundlesize, and it's quite good but we're working on our own replacement. I just have to convince management to allow us to open source some of our tools :D

Thanks for the link :+1:

I wonder why is that? Is that because of SweetAlert2 or your build configuration? There shouldn't be anything CSS-related in the sweetalert2/src/sweetalert2.js.

probably our configuration, it's highly optimised to our usage and sweetalert2 is currently the only dependency that has styles, so it's not worth it at this time to find out where exactly the magic is happening. If it works, it works :-)

If it works, it works :-)

The first rule of sys admins :D But not for JS developers, we are here to BREAK THINGS! :smiling_imp:

This issue can be closed I guess, ping @StephanBijzitter

Well, even though the actual impact was much less than what I first saw, it's still something to consider for the next major version.
I'd prefer to be able to write import {sweetAlert, sweetToast} from 'sweetalert2' or  import sweetAlert from 'sweetalert2/sweetAlert', so that I don't have the code for toasts being imported in a project where I'm not using toasts. It could be done in the same major version, but that'd be more work.

I'm still not sure if it's worth to split those entities. The difference between alerts and toasts is insignificant: https://github.com/limonte/sweetalert2/pull/672
We will keep this suggestion in mind, but I'll close this issue for now.


Add TreePickerNode component

Modularise more variables for greater reuse.

https://trello.com/c/TYreuTXI/4730-alexandria-tree-picker-node

LGTM :christmas_tree:

LGTM.

Green! :christmas_tree: Merging.


Encryption

Added constants for max phone number alowed

Codecov Report

Merging #310 into release-1.14 will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release-1.14     #310   +/-   ##
===============================================
  Coverage           41.15%   41.15%
  Complexity            762      762
===============================================
  Files                 110      110
  Lines                7423     7423
  Branches              843      843
===============================================
  Hits                 3055     3055
  Misses               4043     4043
  Partials              325      325

Impacted Files
Coverage Δ
Complexity Δ

...n/java/org/sunbird/common/models/util/JsonKey.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Docs: nodes outside lan

Could you please provide the description about the problem and what you would like to have documented?

Say I want to have one node as a VPS at Hetzner, one at K-Space, one at a VPS in US. Will it magically just work?

Yes, it should work. I had nodes spread across public cloud and private
datacenter
On Wed, Aug 5, 2020 at 1:54 PM jtagcat<EMAIL_ADDRESS>wrote:

Say I want to have one node as a VPS at Hetzner, one at K-Space, one at a
VPS in US. Will it magically just work?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/overnode-org/overnode/issues/22#issuecomment-668927597,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6JSVIBW7IMYRWFX22KRSTR7C3XDANCNFSM4PUZB4HA
.


Remove default methods in cursor manager

👍

Hopefully, in the future, we are able to rewind and fast-forward subscription cursors. And reset them to BEGIN. :)

:+1:


Move order of pages. Import Pages

Would be nice to reorder the pages in a way that suits user preferences. From my experience user experience is limited when having to scroll back and forward mane pages to find the right one. Another nice future would be importing pages into a user scheme

We are working on something. Planned for v3.


Mast: Watch toptrendi.com

Mast requests the watch of the watch_keyword toptrendi\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

toptrendi\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Daniil in Charcoal HQ


Fix publish widget path

WHY
BEFORE - What was wrong? What was happening before this PR?
Widgets were published to old /base/ path.
AFTER - What is happening after this PR?
They are now published to the new ui folder.
@tabacitu there is still an issue that I am trying to come up with a solution.
The current publish script is not "theme aware", it was previously only a 1-1 (vendor - user) relation.
At the moment, if you do for example: backpack:widget myCard --from=card, since tabler overwrites the card widget it will correctly pick the file from the theme to publish.
The problem is now where to publish this file in user land.
If the theme does not overwrite the widget, it will publish the widget from the core ui ... to the ui (in userland) 🟢
If the theme overwrites the widget, it will correctly pick the widget from theme🟢 but publish it to the ui (in userland) 🔴
I am working on a solution for this.

Excellent job on this one Pedro 👏👏👏
I think this works as expected now:

I've done php artisan backpack:widget myCard --from=card, expected it to get published in tabler theme... it did ✅
I've done php artisan backpack:widget myCard --from=div, expected it to get published in UI... it did ✅

It's a little clunky to get published in different directories... but I think most devs won't even notice, they can click the file path and that's where they edit.
LFG! 💪🎉


Fix typos and segregate Element and SingleElement into another file

Earlier both classes (Element and SingleElement) and tags classes were in the same file. I created a new file (tags.py) for the tag classes so that it is easier to edit in the future. Further I renamed tag.py to element.py.
There was frequent use of the word childrens but children itself is plural. Further I renamed children in for loops (such as for children in self.children) to child as the former seemed confusing.
I fixed the typo for the formatting, formatted etc.
Next I renamed the Props variable to Attrs(short for attribute) as that is the general terminology in web development.

To contribute further I would like to ask some questions about the code choices (if you don't mind)

Why are classing and Attrs needed. The classes can just be converted to string from the beginning of initialisation. Further I don't feel any concrete use for Attrs class. It may be helpful while developing but it's very confusing about what it does while the first time reading it.

Thanks and feel free to ask me about the mentioned changes 😄

Thanks for your big contribution but
I want to name it Props because It will be like React.js and use syntax like jsx
https://github.com/Unviray/pyplater/commit/8dbceb0451d4b4c87c9a23a5d8b5f2b7ce2b8026
https://github.com/Unviray/pyplater/commit/3c6743ca12ca5ba1ea871075c4ca0820edcc314d

Could you please add the "hacktoberfest-accepted" label. Thanks 😄

Sure


Error when reading mdocs with "...VS" entries of length 3

Hi, I'm starting to work with SerialEM data and stumbled upon this great package :)
I wanted to report a problem I encountered: mdocfile.read('example.mrc.mdoc') complains about the following:
ValidationError: 3 validation errors for MdocSectionData
XedgeDxyVS
  wrong tuple length 3, expected 2 (type=value_error.tuple.length; actual_length=3; expected_length=2)
YedgeDxyVS
  value is not a valid tuple (type=type_error.tuple)
AlignedPieceCoordsVS
  wrong tuple length 3, expected 2 (type=value_error.tuple.length; actual_length=3; expected_length=2)

Indeed the entries for "XedgeDxyVS" and "AlignedPieceCoordsVS" in my file are of length 3. While on one side I'm trying to understand how to interpret these three values, I'd guess mdocfile would want to read them (since my mdoc should be a valid SerialEM output).
Cheers!

As a follow up, if I modify the types in section_data as such:
    XedgeDxyVS: Optional[Tuple[float, float, float]]
I'm getting a different error instead:
ValidationError: 1 validation error for MdocSectionData
YedgeDxyVS
  value is not a valid tuple (type=type_error.tuple)

Thanks for the kind words and the report @m-albert ! I'm sorry the package wasn't quite working out of the box with your data, hopefully we can get you up and running quickly - I'll take a look tomorrow
Could you share your file so I can use it for debugging? Ideally as a GitHub gist or similar 🙂

Thanks for your quick response @alisterburt :)
Here's the file as a gist.

glad to help! Thanks for opening the issue, issues like this have probably come up before but until we know about them it's tough to do much about it
merged #10 with a fix and v0.0.8 should be released shortly, progress on deployment can be seen at
https://github.com/teamtomo/mdocfile/actions/runs/4628299599

just fyi, your mdocs look a little different to ones I've seen before in that they have two types of section data ZValue and MontSection -> these contain different data so now a few columns will contain some NaN values

merged https://github.com/teamtomo/mdocfile/pull/10 with a fix and v0.0.8 should be released shortly, progress on deployment can be seen at
https://github.com/teamtomo/mdocfile/actions/runs/4628299599

Thanks so much for this extremely quick fix :) I'll start using it right away from the main branch.

glad to help! Thanks for opening the issue, issues like this have probably come up before but until we know about them it's tough to do much about it

You're welcome, and thanks for sparing me and others redundant efforts!

just fyi, your mdocs look a little different to ones I've seen before in that they have two types of section data ZValue and MontSection -> these contain different data so now a few columns will contain some NaN values

Exactly, the last entry type describes the montage of everything above (and I'd guess would not go to values higher than 0).  You even added a new column to the output dataframe :)


DecimalConverter bug

Due to some reason, when you convert BoltData from MID0106 sometimes the data is not well converted (for example when the tool loosens and returns a negative angle).
public override decimal Convert(string value)
{
    decimal decimalValue = 0;
    if (value != null)
        decimal.TryParse(value.ToString(), out decimalValue);

    return decimalValue;
}

I fixed it adding the following replace.
public override decimal Convert(string value)
{
    decimal decimalValue = 0;
    if (value != null)
        decimal.TryParse(value.ToString().Replace('.', ','), out decimalValue);

    return decimalValue;
}

Perhaps can be also fixed using CultureInfo, the language of my client is es-ES.

It's not common to see decimal data with commas, but since there's the possibility of having commas instead of dots It's cool to handle both cases.
Going to implement it on next version. Thanks for providing info and code.

@Raykler Tried to emulate a negative angle but without success, do you have any example of it?

Here you have, for Mid0106 Rev.1, replace the _ with space.
03370106____________01030201030000200614040105RA_ST5.1_PwMACS_____062020-06-26:09:07:10075008Mode_50_____________09010211________________________________________1202130114015_16117_______18-573.0819_______20_______21_______22_______130214015_16117_______18-574.3019_______20_______21_______22_______2301Data_No_Station_____I_100000004009

Have you been able to test the issue?
I would like to update to the last version.

Solved in version 3.2.2, please download on NuGet!


Differential Expression not running with lassoed selections

Ubuntu 20.04 cirro serve 1.1.22post1.
Differential Expression with jobs submit but never run if group 1 and group 2 are based on lassoed selections. DE runs fine if cells selected with levels of existing categories (eg clusters).

Fixed in 1.1.22.post2. Thanks.


extension not signed in Firefox 43+

Starting with Firefox 43, add-on extensions and multi-item add-ons that include extensions need to be signed by Mozilla before they can install in release and beta versions of Firefox.

More info on mozilla's site

bug in firefox, not in this extension. sorry!


Remove version number from general name

Name has version 8, even though it is used for both 8 and 11. Removed 8.

[ ] Pull Request title is properly formatted: [CLOUD-XYA] Subject
[ ] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issues
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Attached commits represent units of work and are properly formatted
[x] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[x] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

thanks!


States Daily confirmed value is inaccurate

The values in this file https://github.com/covid19india/api/blob/gh-pages/states_daily_csv/confirmed.csv is not accurate.
If you scroll down to last row, and see total, it says 22718.
This means total confirmed cases for all states is only 22,718. This confirmed value is not accurate.

It's that particular dates' all-India confirmed count (number of new cases added today). Like for today right now