Incompatibility with themer library

Hello
I'm using the qirolab/laravel-themer library. In this case, the translation files are created in public/themes/{them-name}/js instead of in the public/js. So in order for the text in the components to be translated correctly, I have to manually copy and paste the files.

Hey @ali-salavati can you share where you are using the plugin?
Thanks.


feat(promql): add lookback parameter

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
add lookback parameter to Prometheus proto declarations
Checklist

[ ]  I have written the necessary comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
https://github.com/GreptimeTeam/greptimedb/pull/3630
https://github.com/GreptimeTeam/greptimedb/issues/3453

@tisonkun could you please check if that's what needed when you have time?


Fb lineage graph testing

Rationale
This updates some test code to use new lineage graph component; we anticipate replacing the existing graphviz component with this newer one
Related Pull Requests
The new components are in testAutomation/fb_lineageGraphTesting
Similar changes are in provenance/fb_lineageGraphTesting
Changes
Where we have the new lineage graph component (on Experiment Run page) refactor tests to use that instead

@labkey-chrisj
Regarding the "Related Pull Requests" section of the template; please provide actual links to the related pull requests. GitHub will even automatically format them all nice for you:
https://github.com/LabKey/testAutomation/pull/314
https://github.com/LabKey/provenance/pull/17


CY's weekly update

Hi @navyabattula, I've merged your pull request and updated my notes, .gitignore, and clearer tasks in the main Read Me.
CC'ing: @shereenElSayed @ckrintz

I will merge this direct so the Read Me is updated on time for Shereen to see.


KeyError: '1' - Score method

Hello, everyone.
I'm trying to run the score method but during the execution, this error occurs:
File "ldpred/lib/python3.7/site-packages/ldpred/validate.py", line 100, in get_prs     [util.opp_strand_dict[g_nt[0]], util.opp_strand_dict[g_nt[1]]])
Can anybody help me?
LDpred Version: 1.0.10
Best.

HI,
Same problem occurs.
Did you solve it?


[BEAM-9699] Add test verifying we can use ZetaSQL in Python SqlTransform

Adds a simple tests that sends the planner_name option through to the SQL expansion service, and verifies it worked with a ZetaSQL-specific query.
Includes changes from #11574
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

R: @robertwb
cc: @mxm

Run XVR_Flink PostCommit

The changes for this PR alone are in https://github.com/apache/beam/pull/11575/commits/3cc5424ab4b56d62ab076e55035fa336ce5a7381

Run XVR_Flink PostCommit

The change looks good to me, once all its prerequisites get in.

I just backed out the changes that relied on pipeline options, and instead plumbed through the local planner class name configuration added in #11647. Could you take another look @robertwb?

Run Python2_PVR_Flink PreCommit

Run XVR_Flink PostCommit

I plan to merge when CI is green

XVR Flink [passed](https://builds.apache.org/job/beam_PostCommit_XVR_Flink_PR/100] before the package-info run, so I won't do it again

Run Python2_PVR_Flink PreCommit


different feature from the same image

Hello, I meet a very strange problem. I find that the app obtains a little poor performance than that in PC. When digging into the code, I find that for the same image, the features obtained in training and in recognition is a little different. But I can not figure out why.
In training, the feature is loaded by cv::imread. In recognition, the image is loaded by BitmapFactory.
BitmapFactory.Options options = new BitmapFactory.Options();
options.inPreferredConfig = Bitmap.Config.ARGB_8888;
Bitmap bmp = BitmapFactory.decodeFile(filename,options);

And then covert to cv::Mat by
 jniutils::ConvertBitmapToRGBAMat(env, bitmap, rgbaMat, true);
 cv::cvtColor(rgbaMat, bgrMat, cv::COLOR_RGBA2BGR);

Is there any difference between these two ways? thanks.

@xueliang did you find out about this?


Group Policy to control default Terminal setting

Since a year or longer I can choose if WT is the default conhost.
Is it possible to implement a policy that allows company admins to control this centralized?

Right now, enterprise administrators can use Group Policy to roll out registry changes as documented in this knowledgebase article.


231 version 0010 updates

Enhancements

[x] #186
[x] tune memory curation
[x] shadows should contain proxy endpoint (story-shadows currently = /shadow
[x] openai runs need canceling when errored out, otherwise thread is locked
[x] #234

Fixes

[x] bot-bar lost click action set active bot (botId ("data-bot-id") => bot_id
[x] team popups only .contains(target) not working

Cosmetic

[x] finish popup css (bots.css)

so much work... you are a machine


ginkgo local: cilium-etcd-operator: unable to create Cilium etcd cluster CR

Cilium master (0222bc85f439). cilium-etcd-operator v2.0.3
I've been trying to run one specific test in the k8s scope locally, but cilium-etcd-operator gets stuck:
$ git diff
diff --git a/test/k8sT/Services.go b/test/k8sT/Services.go
index 6a47cee9459c..8f8fa8017dcf 100644
--- a/test/k8sT/Services.go
+++ b/test/k8sT/Services.go
@@ -124,7 +124,7 @@ var _ = Describe("K8sServicesTest", func() {
                        _ = kubectl.Delete(demoYAML)
                })

-               It("Checks service on same node", func() {
+               FIt("Checks service on same node", func() {
                        err := kubectl.WaitforPods(helpers.DefaultNamespace, "-l zgroup=testapp", 300)
                        Expect(err).Should(BeNil())
                        clusterIP, _, err := kubectl.GetServiceHostPort(helpers.DefaultNamespace, serviceName)

Ginkgo commandline:
$ K8S_VERSION=1.8 NFS=0 ginkgo -failFast -v -v -v -- -cilium.holdEnvironment -cilium.skipLogs --cilium.testScope=k8s

cilium-etcd-operator logs:
time="2018-12-21T18:13:50Z" level=info msg="Deploying secret kube-system/cilium-etcd-peer-tls..."
time="2018-12-21T18:13:50Z" level=info msg=Done
time="2018-12-21T18:13:50Z" level=info msg="Deploying secret kube-system/cilium-etcd-server-tls..."
time="2018-12-21T18:13:50Z" level=info msg=Done
time="2018-12-21T18:13:50Z" level=info msg="Deploying secret kube-system/cilium-etcd-client-tls..."
time="2018-12-21T18:13:50Z" level=info msg=Done
time="2018-12-21T18:13:50Z" level=info msg="Deriving etcd client from cilium-etcd-client-tls to cilium-etcd-secrets..."
time="2018-12-21T18:13:50Z" level=info msg="Updating cilium-etcd-secrets secret..."
time="2018-12-21T18:13:50Z" level=info msg=Done
time="2018-12-21T18:13:50Z" level=info msg="Deploying etcd-operator CRD..."
time="2018-12-21T18:13:50Z" level=info msg="Done!"
time="2018-12-21T18:13:50Z" level=info msg="Deploying etcd-operator deployment..."
time="2018-12-21T18:13:50Z" level=info msg="Done!"
time="2018-12-21T18:13:50Z" level=info msg="Deploying Cilium etcd cluster CR..."
time="2018-12-21T18:13:50Z" level=error msg="unable to create Cilium etcd cluster CR: the server could not find the requested resource (post etcdclusters.etcd.database.coreos.com)"

No actual etcd pods are running at this point.
If I restart (ie, kill) the cilium-etcd-operator pod, it gets past this point but seems to do nothing:
time="2018-12-21T18:17:49Z" level=info msg="Deploying secret kube-system/cilium-etcd-peer-tls..."
time="2018-12-21T18:17:49Z" level=info msg=Done
time="2018-12-21T18:17:49Z" level=info msg="Deploying secret kube-system/cilium-etcd-server-tls..."
time="2018-12-21T18:17:49Z" level=info msg=Done
time="2018-12-21T18:17:49Z" level=info msg="Deploying secret kube-system/cilium-etcd-client-tls..."
time="2018-12-21T18:17:49Z" level=info msg=Done
time="2018-12-21T18:17:49Z" level=info msg="Deriving etcd client from cilium-etcd-client-tls to cilium-etcd-secrets..."
time="2018-12-21T18:17:49Z" level=info msg="Updating cilium-etcd-secrets secret..."
time="2018-12-21T18:17:49Z" level=info msg=Done
time="2018-12-21T18:17:49Z" level=info msg="Deploying etcd-operator CRD..."
time="2018-12-21T18:17:49Z" level=info msg="Done!"
time="2018-12-21T18:17:49Z" level=info msg="Deploying etcd-operator deployment..."
time="2018-12-21T18:17:49Z" level=info msg="Done!"
time="2018-12-21T18:17:49Z" level=info msg="Deploying Cilium etcd cluster CR..."
time="2018-12-21T18:17:49Z" level=info msg=Done
time="2018-12-21T18:17:49Z" level=info msg="Sleeping for 5m0s to allow cluster to come up..."

From kube-dns logs:
I1221 18:29:30.429822       1 dns.go:866] Not a federation query: len(["cilium-etcd-2nwspbn6qq" "cilium-etcd" "kube-system" "svc" "svc" "cluster" "local"]) != 4+len(["local" "cluster"])
I1221 18:29:30.429836       1 dns.go:757] Found 0 records for [local cluster svc svc kube-system cilium-etcd cilium-etcd-2nwspbn6qq] in the cache
I1221 18:29:30.429843       1 dns.go:764] getRecordsForPath retval=[], path=[local cluster svc svc kube-system cilium-etcd cilium-etcd-2nwspbn6qq]
I1221 18:29:30.429850       1 dns.go:670] No record found for cilium-etcd-2nwspbn6qq.cilium-etcd.kube-system.svc.svc.cluster.local.

After some time (5-10 minutes?), cilium was finally able to resolve cilium-etcd-client.kube-system.svc.


Add tracking for accordion open all link

We currently track users who click on and interact with accordions but what we don't track is the number of users who interact with the expand and collapse all feature.
Adding tracking to this will help us determine if users that click this option have their journey altered in some way or don't make it through a large proportion of the page.
The event structure should follow the accordion structure currently in place.
Event category: pageElementInteraction
Event action: accordionOpened/accordionClosed
Event label: Expand all/Collapse all
It's done when:
We can see when users interact with the expand and collapse all portions of the accordion interaction.
https://trello.com/c/so5IN6ZA/157-add-tracking-to-the-expand-all-option-on-accordions

Looks good. Does it need a test?

You’re right, I’ll get on that


renameToCamelCase for columns with names starting with capital letter should lowercase it

@belovrv what kind of CamelCase do you want?

CamelCase?
camelCase? (style lowerCamelCase)

For now, we have no documentation for this function, please give some examples of usage or share some insights

@Jolanrensen how do you estimate this code?
For me, it looks like a strange 2-3 level repeating function s with recursive call.
If it's a common pattern to apply changes in the dataframe, could you advise how to interpret correctly the code?

public fun <T> DataFrame<T>.renameToCamelCase(): DataFrame<T> {
    return rename {
        dfs { it.isColumnGroup() && it.name() matches DELIMITED_STRING_REGEX }
    }.toCamelCase()
        .rename {
            dfs { !it.isColumnGroup() && it.name() matches DELIMITED_STRING_REGEX }
        }.toCamelCase()
        .update {
            dfsOf<AnyFrame>()
        }.with { it.renameToCamelCase() }
}

@zaleslaw
It's strange actually, the first two statements seem to be useless but I cannot merge them into 1.
It says:
First, recursively rename all column groups to camel case,
then recursively rename all other columns to camel case,
then finally, take all frame columns and call renameToCamelCase() on all dataframes inside.
It might look clearer after my rename:
public fun <T> DataFrame<T>.renameToCamelCase(): DataFrame<T> =
    rename {
        cols { it.isColumnGroup() && it.name() matches DELIMITED_STRING_REGEX }.recursively()
    }.toCamelCase()

    .rename {
        cols { !it.isColumnGroup() && it.name() matches DELIMITED_STRING_REGEX }.recursively()
    }.toCamelCase()

    .update {
        colsOf<AnyFrame>().recursively()
    }.with { it.renameToCamelCase() }


[imgui] Update to 1.76

Update dear imgui library to version 1.76.

What does your PR fix? Fixes #

N/A

Which triplets are supported/not supported? Have you updated the CI baseline?

Should be no changes.

Does your PR follow the maintainer guide?

I believe so.

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

Could be mistaken, but the failing PR checks all look like they might be transient infrastructure failures.

Thank you! How do I get this merged?

You wait for working hours pacific time ;)
Thanks @michaelquigley :)

No problem! Just wanted to make sure there wasn't something else I needed to do! I appreciate it...


Add markdown support for the documentation body.

It would be nice for mxmldoc to support a generic format for the body (intro) text, and Markdown would seem to be well-suited to the task.

Initial support is part of:
[master c6bda8e] Add markdown support for HTML/EPUB/DOCSET output (Issue #194)


xmrig-proxy: 6.14.0 -> 6.15.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/xmrig-proxy/versions.
meta.description for xmrig-proxy is: "Monero (XMR) Stratum protocol proxy"
meta.homepage for xmrig-proxy is: "https://github.com/xmrig/xmrig-proxy"
meta.changelog for xmrig-proxy is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/23bgs333sjx8fg7rbz0q1v5lfqrwmnn7-xmrig-proxy-6.15.0/bin/xmrig-proxy passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 6.15.0 with grep in /nix/store/23bgs333sjx8fg7rbz0q1v5lfqrwmnn7-xmrig-proxy-6.15.0

found 6.15.0 in filename of file in /nix/store/23bgs333sjx8fg7rbz0q1v5lfqrwmnn7-xmrig-proxy-6.15.0

directory tree listing: https://gist.github.com/64ce8e37459e5e44ae604ee5056dd185

du listing: https://gist.github.com/0ce1e31ac697991a4cb2980325ede789

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
xmrig-proxy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/23bgs333sjx8fg7rbz0q1v5lfqrwmnn7-xmrig-proxy-6.15.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xmrig-proxy https://github.com/r-ryantm/nixpkgs/archive/72800727fe743f47525df882c89e14990abba32a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/23bgs333sjx8fg7rbz0q1v5lfqrwmnn7-xmrig-proxy-6.15.0
ls -la /nix/store/23bgs333sjx8fg7rbz0q1v5lfqrwmnn7-xmrig-proxy-6.15.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

xmrig-proxy

Maintainer pings
cc @aij for testing.

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

1 package built:

xmrig-proxy


Fixes #7178 Access control support for TestCase entities

Describe your changes :

See #7178 for details

Type of change :

[x] New feature

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Cypress failures seem unrelated.


Docs: Explain how MaxItems is applied in more detail

This update to the botocore docs adds to the paginators chapter information explaining that the CommonPrefixes list doesn't count toward the limit set by MaxItems, with example code.
A few other minor cleanup bits are also included.

Converted this to a draft. Forgot the editorial and translation steps because I've gotten out of the habit due to work on other guides.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.05%. Comparing base (8fd0160) to head (18173f7).
Report is 153 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3151      +/-   ##
===========================================
- Coverage    93.14%   93.05%   -0.10%
===========================================
  Files           66       66
  Lines        14237    14245       +8
===========================================
- Hits         13261    13255       -6
- Misses         976      990      +14

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

Edit request has been submitted. Once that's done I'll submit for L10n.

Edit has been approved. This is now ready to merge and deploy.


Enable indexing on non-spell indicators

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #1069

[x] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] HISTORY.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

[ ] bumpversion patch has been called on this branch
[x] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Enable indexing on most indicators. I left all the spells indicator, because I believe further discussion is needed.
I merged _preciptemp.py into _precip.py.

Does this PR introduce a breaking change?
Not a clear one.  Many indicators have updated signatures.

Coverage increased (+0.0005%) to 91.605% when pulling ebc73c3cb8d49854bbe94826a9c92ed4ff3badfa on enable-indexer-where-easy into 6f51e7a8c397a58e04e3f98191357746382b3836 on master.


Testing prompts

Hello.
I am trying since a while to determine how I am supposed to create tests for the prompts. Using Enquirer directly, I can use its events, but using Listr, I don't have access to the underlying Enquirer object.
Is there currently any way to test prompts? Thanks!
EDIT - I think the ability to pass an instance of Enquirer, the same as logger, to a Listr object, would solve the issue, because we would be able to pass a fake Enquirer object, hence, to test it.

Hi Mr. Innocenzi,
I think that idea is feasible and doable. I will have a look at that this weekend because at last, I was also planning to add tests for renderers. I was planning to that with jest-mock-process. Maybe that can be the answer since you can manipulate stdout as well as stdin. But you are right, unfortunately, at the moment there is no easy way to spy on the instance of enquirer since I converted it to be a dynamic import.

Hi, thank you for your answer! I look forward to a change in that direction. Let me know if I can be of any help.

Can you try the latest beta from listr2@beta. I was also writing something basing this as a dependency had to do some minor fixes, so hopefully added your functionality before weekend as well, weeee.

:tada: This issue has been resolved in version 2.3.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

You went fast! And it looks good, I'll take a look tomorrow. Thank you!

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add E2B AI Analyst

What does this PR do?
This PR adds an AI data analyst that uses Llama APIs or Ollama to analyze data and generate charts. It uses Llama to generate the Python code and E2B to run the actual code.
A live demo is available here: https://ai-analyst.e2b.dev/

The app is a self contained NextJS project that can be easily self-hosted on Vercel and integrated with API providers and Ollama via the .env file.
We have also signed the CLA.

Thanks @jamesmurdza for the PR! its a cool recipe, if there is no hard dependency,  can we please have short readme + the online version link instead of the submodule.

Great idea, done. Let me know if I should make the README shorter or change anything.

Hello @HamidShojanazeri , is there anything else to update on our (E2B) end for this PR to be merged? Thanks for update :)
T.

Thanks @tizkovatereza LGTM, can you pls fix the spellcheck errors, by adding those words to here, https://github.com/meta-llama/llama-recipes/blob/main/.github/scripts/spellcheck_conf/wordlist.txt.

Done.

Hey the CI seems to be failing on a Markdown link that we didn't add. Is there anything we should fix?

Hey @HamidShojanazeri  could you please check the Mardown link as James says? We would love to see this PR merged and make an announcement. :)
Thank you very much!
T


Account menu

The account menu is used for services that are part of the personal tax account. It is part of the account header.

Language toggle
Overview
we currently believe the positioning was chosen based on this document by the Welsh language commissioner: http://www.comisiynyddygymraeg.cymru/hybu/SiteCollectionDocuments/CYG1754 Canllaw Dylunio ENGLISH.pdf
Different positioning
currently two different positionings for the toggle one being in the account header the other being in the main body

Issues

current colour contract on the account heading doesn't meet the required standards
@philsherry has raised issues around how the positioning works for user with magnifier

Actions

find out why the Welsh language commissioner recommended the right-hand corner
rather the research from PTA header positioning @goldensheldon


easy way to get header?

Is there an easy way to get a response header that ignores string/atom, uppercase/lowercase, etc?

Response headers are always normalized to non-capitalized strings.

ok thanks


CI: Make Travis call common setup

Update the .travis.yml config file to call the common CI setup script.
This ensures Travis jobs have the correct set of packages installed for running all the static checks.
Fixes: #648.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

/test


Not able to upgrade vespa 8 version in my current cluster because of permission issue.

Describe the bug
So basically we already have a vespa 7 setup and wanted to upgrade to vespa 8 and we have a constraint that our scripts can't be run by root user ergo the vespa commands can't be run by root user, so we are doing the below steps to run the same without root user on vespa 7.559.12.

In the docker file adding the below commands :
RUN chown -R nobody:nobody  /bin /opt /usr
USER nobody
RUN chmod -R 755 /opt &&
chmod -R 755 /usr &&
chmod -R 755 /bin
In the default environment text file adding below lines:
override VESPA_USER nobody
override VESPA_UNPRIVILEGED yes
Adding runAsUser: 99(id for nobody user) in kubernetes yml files.

so using the above config we are able to smoothly run vespa 7.559.12 in our cluster without root user.
But on trying to update to vespa 8: latest version we are facing below issues :
The id of nobody user which was 99 is changed to 65534 in vespa 8 maybe because of linux centos 8 version change and we have updated the same in our kubernetes yaml files, but on creating the container with vespa latest image version.
I am getting the below issue :
warning bad VESPA_USER '99' (ignored)
warning bad VESPA_USER '99' (ignored)
Executing /opt/vespa/libexec/vespa/stop-configserver
warning bad VESPA_USER '99' (ignored)
FATAL error: user '99' missing in passwd file
Here VESPA_USER as mentioned above was kept nobody.
So my issues is why the id of nobody user which is supposed to be 65534 for nobody is being printed as 99, as in the vespa 8:latest container 99 is not a defined user:
warning bad VESPA_USER '99' (ignored)
warning bad VESPA_USER '99' (ignored)
Executing /opt/vespa/libexec/vespa/stop-configserver
warning bad VESPA_USER '99' (ignored)
FATAL error: user '99' missing in passwd file
Vespa version
vespa 8:latest

Anyone who can help with this? It is blocking vespa 8 upgrade for us while running as unprivileged user
[cc: @bratseth ]

Looks like an issue at our end while trying out various values in default-env.txt where 99 was an older value; reverting this back to nobody did not reflect in our artifact repo at the time of image creation. Tried re publishing the artifact containing default-env.txt with the right value, and it worked.
Thanks for your observation.


[WIP] Add FORMAT_RULES.md

Closes #3

Closing this PR as I don't have a strong motivation to write this down right now.


add towhee.glob

Signed-off-by: jie.hou<EMAIL_ADDRESS>

Codecov Report

Merging #924 (aaa6cae) into main (72b97c4) will increase coverage by 0.06%.
The diff coverage is 100.00%.

:exclamation: Current head aaa6cae differs from pull request most recent head c8eb617. Consider uploading reports for the commit c8eb617 to get more accurate results

@@            Coverage Diff             @@
##             main     #924      +/-   ##
==========================================
+ Coverage   69.62%   69.68%   +0.06%
==========================================
  Files         203      203
  Lines        8846     8851       +5
  Branches     1452     1453       +1
==========================================
+ Hits         6159     6168       +9
+ Misses       2264     2261       -3
+ Partials      423      422       -1

Impacted Files
Coverage Δ

towhee/__init__.py
82.75% <100.00%> (+0.40%)
:arrow_up:

towhee/dc/__init__.py
66.66% <100.00%> (+66.66%)
:arrow_up:

towhee/hparam/hyperparameter.py
49.80% <100.00%> (+0.19%)
:arrow_up:

towhee/dataframe/dataframe.py
73.10% <0.00%> (-1.05%)
:arrow_down:

towhee/hub/repo_manager.py
80.18% <0.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 72b97c4...c8eb617. Read the comment docs.

/lgtm


More complex view/track arrangement

Codecov Report

Merging #1 into master will decrease coverage by 0.09%.
The diff coverage is 3.57%.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   24.36%   24.26%   -0.10%
==========================================
  Files          73       74       +1
  Lines        2467     2477      +10
  Branches      514      515       +1
==========================================
  Hits          601      601
- Misses       1779     1789      +10
  Partials       87       87

Impacted Files
Coverage Δ

src/core/gemini.schema.ts
0.00% <ø> (ø)

src/core/layout/layout-circular.ts
0.00% <0.00%> (ø)

src/core/layout/layout.ts
0.00% <0.00%> (ø)

src/core/layout/view.ts
0.00% <0.00%> (ø)

src/core/mark/line.ts
6.66% <ø> (ø)

src/core/mark/outline-circular.ts
2.63% <0.00%> (-0.31%)
:arrow_down:

src/core/utils/spec-preprocess.ts
0.00% <0.00%> (ø)

src/editor/example/circos-2.ts
0.00% <0.00%> (ø)

src/editor/example/circos-many-rows.ts
0.00% <ø> (ø)

src/editor/example/circos.ts
0.00% <ø> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Fix input arg name for Python example

The input argument name is arg_name and not name  in python.

@ggailey777

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ggailey777 - Can you review this old PR and determine whether it needs to be closed or merged?

PR review team emailed author today


Install cmake-cargo-gen from cargo if it's not already installed

The generator executable will not be published to crates.io - it will ship with the release of Corrosion.


Use Julia 1.10 in CI

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f30e16f) 73.83% compared to head (102c497) 71.49%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   73.83%   71.49%   -2.35%
==========================================
  Files           6        6
  Lines         237      221      -16
==========================================
- Hits          175      158      -17
- Misses         62       63       +1

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


The spatial example is broken

As of commit a55cac7 the spatial example does not emit any sound. It was broken by part of #191.

Seems fixed by 198


Share component code cleanup and refactor

Jira
http://vjira2:8080/browse/BDS-311
Summary
Refactored share component's code to be more efficient.
Details

Cleaned up the CSS and twig file.
Removed as much overrides as possible.
Followed the Bolt conventions and renamed certain props and variables.
Button version has been removed because it's not being used.
Latest 2.x has been pulled in.
Icon component has been changed: small icon size has been increased and stroke has been decreased.
Copy to clipboard has been changed: misc. code clean ups.

View demo

How to test
Pull down the branch and run locally. Navigate to pattern lab and view the share component. Test all the links and copy to clipboard function.
Notes

Web component side of things has not been done.

When you deprecate something in a PR, it's important to follow all the steps in part I of the  deprecation wiki page
Missing pieces here:

Update your schema to specifically exclude the old properties.  This lets us quickly find any incorrect usages in pattern lab:

not:
  required:
    - copyToClipboard

Add a description of the deprecations to you PR description.  This will be used by the Drupal guys when pulling this in.


Let pkg also be the home for common knative API and binaries

Both serving and eventing start to have some functionalities that could benefit more than just the belonging repo. E.g. https://github.com/knative/eventing/pull/2333 ConfigMap propagation API could be beneficial to both repos but the fact it only comes with eventing seems wrong. @grantr also mentioned that serving Certificate API could be another example.
pkg currently only have common libs. I wonder if could also be the home for common knative utility APIs and binaries?

We have a few common APIs in the "duck typing" package in apis today, so I think this has been done before.
In general, I think we'll end up needing to use duck typing to share these components.

What about the controllers for such APIs? E.g. ConfigMap propagation. I think a single controller is good enough.


Box vs Box SAT Check

For calculating the projection of Box on to an axis, I tried this method that  I found in
cyclone/src/collide_fine.cpp
static inline real transformToAxis(
    const CollisionBox &box,
    const Vector3 &axis
    )
{
    return
        box.halfSize.x * real_abs(axis * box.getAxis(0)) +
        box.halfSize.y * real_abs(axis * box.getAxis(1)) +
        box.halfSize.z * real_abs(axis * box.getAxis(2));
}

But it is not giving me correct results.
I made a small modification by taking absolute values of axis and box axis,
and that gives me correct results.
Can some one please verify this? I am not sure if I am doing something wrong elsewhere in my code.

I recommend you to read the Real-Time Collision Detection Book, the collision detection of cyclone code is based on the book.
you will find the related information on the AABB, OBB intersection tests.

This may be related to this isssue.
The only problem I'm seeing (so far) with the BOX SAT is that it fails when the axes being compared are exactly parallel.
The comparison happens here:
https://github.com/idmillington/cyclone-physics/blob/master/src/collide_fine.cpp#L111-L119
and
https://github.com/idmillington/cyclone-physics/blob/master/src/collide_fine.cpp#L80-L84
If the two boxes are oriented on the same plane (e.g. the ground), their X, Y and Z-axis will be parallel and the resulting cross product will be 0.
I've worked around this changing the overlap comparison from < to <=
return (distance <= oneProject + twoProject);

This covers the case where distance is exactly 0 due to axis being a 0-vector. Obviously with this modification the SAT now succeeds in the rare case where the axis projections are exactly adjacent and not overlapping, but I find this to be an acceptable compromise for my own engine. YMMV.
Finally, note that the box intersection test is commented out in the contact generation code:
https://github.com/idmillington/cyclone-physics/blob/master/src/collide_fine.cpp#L415
Not sure why.


Install Xcode from a local DMG image or server URL

Hi,
Thanks for this tool, it looks really great.
Could you please add additional options such as:

installing from a local DMG image

xcode-install install /path/to/xcode.dmg

installing from a different URL

xcode-install install http://storage.server.com/xcode/xcode7.dmg

This might be an interesting enhancement — you can already somewhat achieve it by copying the DMG into ~/Library/Caches/XcodeInstall where it will be automatically picked up if it matches the one on the DevCenter.

thanks :+1:

This was implemented in #59


Can't change the status of an Issue

Currently jiracli doesn't model the Jira project workflow, so there's no way to move from, say, Backlog to Done.
Jira manages this with a prescribed workflow, so only certain status transitions are valid. jiracli will also need to model these workflow transitions for a project in order to fix.

when is this gonna be done?been waiting for more than a year for this stuff bro, run out of VB

Pull requests welcome, check the collaborators guide


Tutorial about how to structure data in a relational database system

I've had the chance in the last few years to work in collaboration with biodiversity professionals, and from my IT-oriented perspective there's often a lack of knowledge about how to efficiently organise / structure the data in a relational database systems. To be more precise: there's generally enough knowledge to make something "that works", but a better structured database would be much more future-proof (less data errors, easier to reuse the data in other contexts such as data publication, web portals, ...)
That tutorial would not cover SQL nor the technicalities of a given database engine (SQLite, PostgreSQL, ...), but rather help answering questions such as

should a given piece of information be placed in a new field or in a new table?
how should I link tables X, Y and Z so they can be queried to answer a wide range of questions?
what constraint can I configure early when I create a database so human errors (i.e. typos when entering data) are detected as early as possible (and the database doesn't get messier when it gets more used/bigger)

Is there any demand for this from scientists, or is it just me?
If so, I'd be happy to help contributing to a tutorial (but I think it can be a pretty large task, so I'd like to have an idea of the interest first).

Hi Nico, this sounds great, and indeed seems very useful to many scientsts (INBO and non-INBO) - thanks so much for your enthusiasm!
IMHO it would be interesting that you connect with @fredericpiesschaert and @gertvanspaendonk in order to see which material they already have in that direction (it may be in Dutch, and SQL Server oriented, but still honouring the ideas that you describe). Ideally they could participate in this effort.
Moreover, this has relationships with issue #10 and the associated (still open) PR #140 (designing databases), for which further collaboration with @ThierryO was suggested in order to make it more technology-agnostic, similar to what you suggest.
Maybe @niconoe it would be great if you could bring the involved people together for a dedicated brainstorm and to refresh plans?

Moreover, this has relationships with issue #10 and the associated (still open) PR #140 (designing databases), for which further collaboration with @ThierryO was suggested in order to make it more technology-agnostic, similar to what you suggest.

Update: PR #140 has been merged since the authors wish to limit the scope to MQ SQL Server. So there's still room to tackle issue #10 further.
Advice on how to structure data in a (any) RDBMS is most welcome indeed.


[DPUL/staging] RedisClient::ReadTimeoutError: Waited 2.0 seconds

Backtrace
View full backtrace and more info at honeybadger.io

Looking at https://github.com/sidekiq/sidekiq/wiki/Using-Redis to see if it has any advice.

closing in favor of #1487


Cards in the features section of the homepage should have equal height

Is your feature request related to a problem? Please describe.
Cards in each row of the features section of the homepage should have equal height
Screenshot

Assign this to me please....

@Officialbube all yours!


Cleanup not running in 4.0

Summary
cleanup_old_data() is no longer called as part of the backup routine.
Automatic cleanup that worked fine under v3 stopped working since v4. I realised when a disk ran out of storage from being filled up with database backups that had stopped being cleaned up. I tried changing my old environment variable from DB_CLEANUP_TIME to DEFAULT_CLEANUP_TIME, and also DB01_CLEANUP_TIME to no effect. No logs in the console about cleanup running.
The cleanup_old_data function no longer appears to be referenced anywhere in this repo, so I guess the step got accidentally dropped from the backup script when it was migrated.

Wow - this is embarrassing that I omitted this. Fixed in 4.0.15
https://github.com/tiredofit/docker-db-backup/commit/34abe8815903f1e9cbc35568fe96ff3cfd81fe7f

Updating now. I just noticed the same lack of cleanup as my disk is now at 90%. Thanks!


Install REST Assured into Complaint Manager

Getting Started: https://github.com/rest-assured/rest-assured/wiki/GettingStarted
Downloads: https://github.com/rest-assured/rest-assured/wiki/Downloads

not needed anymore as we are not using REST assured for 2.0


"too many open files" in static sync

I got a "too many open files" error on OS X, probably related to doing lots of /static syncing. Suspect file opened, but not closed.

Not related to #1870

It would be nice if this was run automatically for OSX users, I am not sure of the purpose of the command currently.

There is (or was) a tweek in Hugo to set this limit on OS X. Not sure if this is related.

We still are seeing this issue on OS X. We have attempted to use hugo check ulimit but that doesn't solve for it the only fix is to run
 $  sudo sysctl -w kern.maxfilesperproc=65536
 $  ulimit -n 65536 65536```

This doesn't persist after a reboot, so it's a bit of a problem. Especially since we have dozens of contributors.

See https://github.com/devopsdays/devopsdays-web to repro.

Persistent sysctl settings are usually in /etc/sysctl.conf.
echo kern.maxfiles=65536 >> /etc/sysctl.conf
echo kern.maxfilesperproc=65536 >> /etc/sysctl.conf

Regardless, it's still a problem that we have to give all of our contributors persistent sysctl settings in order to contribute content :(
This is definitely an improved workaround, but I am still curious as to why hugo check ulimit doesn't work?

For some background, there are 4,696 files in the /static directory (and subdirectories). Yes, I know that's horrid, but it was an artifact of migration (we had so much legacy stuff to migrate that the expedient workaround was to take the output HTML from the former site generator and dump it into /static)

So what are the default values for kern.maxfiles and kern.maxfilesperproc on OSX?
Hugo attempts to increase the resources limits, but you may have to run Hugo with elevated privileges for that to actually work.
The only "real" solution I can think of from Hugo's perspective would be to limit the number of open file descriptors to keep them under the process limits, but that could get very complicated.

These are the defaults:
kern.maxfiles: 12288
kern.maxfilesperproc: 10240

I'm curious to know why hugo wants to keep so many files open at once? Surely it shouldn't have to when generating static content? I understand that it might be an issue with hugo serve without being able to subscribe to filesystem updates.
Strangely it works when I run it a couple of times in a row. I have 11045 pages, and it creates 5773 each of tags and categories.

I'm curious to know why hugo wants to keep so many files open at once?

Parallelism.

From what I can tell it's when I have many categories/tags. Drop the amount back and it stops being an issue. Noone's got ~10k cores, so "paralleism" is probably not the best answer at that point.

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.

@bep I've recently experienced this issue on a new project due to a large number of folders/files in /static. I'm not sure of the threshold (number of files) that prompts the error, but it doesn't seem excessive (~2000 files across a folder tree causes it in my case).
I only experience the issue running hugo server. The build (hugo) is of course fine.
So far I have tried all the suggestions relating to ulimit in this issue, and also other suggestions I've found elsewhere (updating homebrew and updating hugo), without success.
Of course, an alternative would be to find a different work flow for development, but this issue seems persistent, so is perhaps worth resolving.


ar_track_alvar 0.5.1-0 in 'indigo/distribution.yaml'. Release repo change. Metapkg removed.

Addresses https://github.com/sniekum/ar_track_alvar/issues/48
Related to https://github.com/sniekum/ar_track_alvar/pull/50

https://travis-ci.org/ros/rosdistro/builds/58406340#L2717

rosdistro: DEBUG: Skipped "git_manifest_provider()": Unable to fetch package.xml: Specified tag "release/indigo/ar_track_alvar/0.5.0-0" is not a git tag of repository "https://github.com/ros-gbp/ar_track_alvar-release.git"

I might have to make a release into the release repository first.

Done


Auto-advance date of the next tag

Removes softprops/action-gh-release in favor of the gh CLI.
Fixes https://github.com/web-platform-dx/web-features/issues/2068
Closes https://github.com/web-platform-dx/web-features/pull/2425

@Elchi3 would you be interested in reviewing this? It's primarily a maintainer quality-of-life thing, to stop dismissing the Dependabot upgrade for the GitHub Action. I've tested it on my fork, so I've got fairly high confidence on this one.

@ddbeck Hello!
I am watching releases.
Now I get a notification on every commit to main.
Please advise.

Oh that sounds very annoying. Sorry about that, @szepeviktor—I'll open a PR to revert this until I find another workaround.

Thank you.

I've opened https://github.com/web-platform-dx/web-features/pull/2453.


Conditional operator inside object literal causes inconsistent highlighting

.prop : shouldn't be highlighted here:

var test = { prop: (true ? a.prop : b.prop) };

Turns out multi-line ternaries can be matched after all. There are some grouping issues, i.e. the expression before ? and after : is unknown, but I think this is hardly needed.


No audio on output codec not found

you were right, it is creating the _tmp files and below is the output i am getting:
Traceback (most recent call last):File "/Users/jogswyer1/venv-totk-speedometer/lib/python3.9/site-packages/moviepy/video/VideoClip.py", line 262, in write_videofilecodec = extensions_dict[ext]['codec'][0]KeyError: 'codec'During handling of the above exception, another exception occurred:Traceback (most recent call last):File "/Users/jogswyer1/Movies/TotK-Speedometer-main/totk-speedometer.py", line 466, in main()File "/Users/jogswyer1/Movies/TotK-Speedometer-main/totk-speedometer.py", line 451, in mainexport_video_with_overlay(f)File "/Users/jogswyer1/Movies/TotK-Speedometer-main/totk-speedometer.py", line 302, in export_video_with_overlayclip_with_audio.write_videofile(output_filename, audio_codec='aac')File "", line 2, in write_videofileFile "/Users/jogswyer1/venv-totk-speedometer/lib/python3.9/site-packages/moviepy/decorators.py", line 54, in requires_durationreturn f(clip, *a, **k)File "", line 2, in write_videofileFile "/Users/jogswyer1/venv-totk-speedometer/lib/python3.9/site-packages/moviepy/decorators.py", line 135, in use_clip_fps_by_defaultreturn f(clip, *new_a, **new_kw)File "", line 2, in write_videofileFile "/Users/jogswyer1/venv-totk-speedometer/lib/python3.9/site-packages/moviepy/decorators.py", line 22, in convert_masks_to_RGBreturn f(clip, *a, **k)File "/Users/jogswyer1/venv-totk-speedometer/lib/python3.9/site-packages/moviepy/video/VideoClip.py", line 264, in write_videofileraise ValueError("MoviePy couldn't find the codec associated "ValueError: MoviePy couldn't find the codec associated with the filename. Provide the 'codec' parameter in write_videofile.
Edit: if it helps im on a mac and have been doing everything from the terminal, i did do the files for the visual interface but couldnt figure out how to use it

It looks like the problem is with the video codec:
valueerror-moviepy-couldnt-find-the-codec-associated-with-the-filename-provid
I made a commit to fix that (hopefully) https://github.com/miguelqncosta/TotK-Speedometer/commit/66f506eb66c88291d1d710c2b41e7e4192fbedf0
It is already included in the new release. Let me know if that fixed the issue for you.

So excuse my ignorance but how would I go about updating?

I know you already got it working but just in case someone sees this and has the same question, to update you only need to download the zip from the latest release. Cheers

Yes! And if it helps I didn't replace the file, I copied the contents of the update and then pasted them into the previous folder and just said to replace anything that was already there, not sure if it matters but figured just in case there was data in there from before that I needed, thanks again for your help!


How can I stop the screensaver?

Trying to use photoviewer to display Menu in a restaurant, but the screen saver keeps rotating them, how can I stop it?

and app keeps closing and going back to main roku screen??

Are you inside the app running a slideshow? Do you have it paused?   IF YES to both, you would need to disable your screensaver for Roku. The screensaver will trigger when there is no activity detected after the "wait time.

Yes I am in the app running a slide show, I have the show paused on the image I want.  I have disabled the screensaver on the roku, but still after a while it exits the app and returns to the mail screen now.
From: Chris Taylor<EMAIL_ADDRESS>Reply-To: chtaylo2/Roku-GooglePhotos<EMAIL_ADDRESS>Date: Wednesday, September 25, 2019 at 12:12 PM
To: chtaylo2/Roku-GooglePhotos<EMAIL_ADDRESS>Cc: Jeff Raben<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [chtaylo2/Roku-GooglePhotos] How can I stop the screensaver? (#260)
Are you inside the app running a slideshow? Do you have it paused? IF YES to both, you would need to disable your screensaver for Roku. The screensaver will trigger when there is no activity detected after the "wait time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/chtaylo2/Roku-GooglePhotos/issues/260?email_source=notifications&email_token=AAIRUE4AXWHQ6RTQETBZKNDQLOZ23A5CNFSM4I2A3BZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7TASYY#issuecomment-535169379, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAIRUE3GB7BL66SWHOOZ6OTQLOZ23ANCNFSM4I2A3BZQ.

@jraben - I'm not able to duplicate what you're reporting. Can you please take a short video of this and email me?
My email<EMAIL_ADDRESS>Thanks


Error: formValues() must be used inside a React tree decorated with reduxForm()

Bug report
Demo: https://stackblitz.com/edit/redux-form8?file=Form.js
But demo doesn't work because of https://github.com/erikras/redux-form/issues/4293
Steps to reproduce
$ npx create-react-app form
$ cd form
$ yarn add redux redux-form react-redux
# Copy code from demo index.js and Form.js
$ yarn start

So, about bug:
import { reduxForm, formValues } from 'redux-form';
/* ... */
export default reduxForm({ form: 'Form' })(formValues('name')(Form));

After update to version 8.0.1 using formValues decorator results in an error:
Uncaught Error: formValues() must be used inside a React tree decorated with reduxForm()

The issue still exists in 8.0.4, working (broken) example: https://codesandbox.io/s/21m68rvozn

Confirmed. Will investigate tomorrow.

It looks like the consumer is being called before the provider, even though the provider is at a higher level in the tree. I have no idea how/why.

It's got to be something to do with using Context.Provider rather than just a HOC to create the context. Hmm... You'll have to put the component connected to formValues() deeper in the tree.
Until I can figure it out, here's a little render-prop workaround for you.

Here is a version of that using recompose's toRenderProps and fromRenderProps that allows the same usage as before.
This is very strange. I wonder if it's a bug in React?

@erikras Removing the withReduxForm from this line fixes the issue.

This is very strange. I wonder if it's a bug in React?

I assume there's no way that @gaearon can have the time to check his GH mentions, but...?

@tyscorp You workaround works mostly perfectly except it lose "children" prop when you do fromRenderProps(toRenderProps(

@erikras will this be fixed any time soon?

Also hitting this when updating to v8. Reverting back to 7 now!

Hi guys.
Any updates from a redux-form team?

It's time for an HoC-free alternative API.  HoCs need to die already.

The following hook works inside a reduxForm():
import * as React from 'react'
import { useSelector } from 'react-redux'
import { ReduxFormContext } from 'redux-form/es/ReduxFormContext'
import formValueSelector from 'redux-form/es/formValueSelector'
import { mapValues } from 'lodash'

export default function useFormValues(values, deps = []) {
  const { form, sectionPrefix } = React.useContext(ReduxFormContext)
  deps = [form, sectionPrefix, ...deps]
  const mapStateToProps = React.useMemo(() => {
    const valueSelector = formValueSelector(form)
    const finalValues = sectionPrefix
      ? mapValues(values, v => `${sectionPrefix}.${v}`)
      : values
    return state => valueSelector(state, ...finalValues)
  }, deps)
  return useSelector(mapStateToProps, deps)
}

If I replace withReduxForm with this, it works:
    const ConnectedFormValues = props => (
      <FormValues {...props} _reduxForm={React.useContext(ReduxFormContext)} />
    )
    return ConnectedFormValues

This is why I hate HoCs and forwardRef (which was a bad solution to just one of HoCs' flaws).  As @tyscorp has pointed out, something within withReduxForm (and I'm guessing it has to do with forwardRef) is causing the problem.

It's time for an HoC-free alternative API. HoCs need to die already.

There's a pretty large installed base for the current API, so the bugs still need to be fixed.

@tyscorp

I'm not sure what other effects it has, but there is a lot of console.errors when I run the test suite.

Turns out those errors hit the console in master; they are from formValues.spec.js expecting errors in certain cases, but not suppressing the log output. Not sure how to fix that, but it's out of scope. I'm testing out your suggested fix in my codebase, and will make a PR if it works.

Published in v8.2.5.


Changed HTTP statuses to http package constants

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##           master      #12   +/-   ##
=======================================
  Coverage   38.53%   38.53%
=======================================
  Files           5        5
  Lines         301      301
=======================================
  Hits          116      116
  Misses        181      181
  Partials        4        4

Impacted Files
Coverage Δ

primaryHandler.go
52.06% <66.66%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


About the way of displaying series with the New Calendar

Is there any way for the calendar to show the series in your region?
Because with the implementation of the new calendar there are series that are not in my region, nor the schedules are the same and it has already happened to me that I have skipped episodes because when the day passes if they are all, but as I look throughout the day there are episodes that have come out with another schedule as they appear in the calendar sorted and of course those episodes I skip them because they appear later and then appear sorted above the one I already have downloaded when here that has come out before the other that said it came out before, skipping those episodes for not realizing.
Apart from watching episodes that are not available in my region and having to scroll to the ones that are  available in my region.

As you can see the one of love live, that if this in my region comes out higher than the one of sangri-la but has not been issued yet, apart from the other series that appear as Fayri Tale that is not licensed in my region.

you can always just disable the upcoming episodes - also the times get more accurate if you have the series in the history and refreshed them at least once since the update
i now also changed the sorting


Fix formatting of timestamp on streamed market prices

Streamed price data is returning the wrong snapshotTime. For example 10:22pm would return a 10:22:00 rather than 22:22:00

Thanks @TweededBadger for fixing this! 👍

Thanks @TweededBadger for fixing this! 👍

No problem at all.
I'd like to introduce some better error handling, as at the moment any API errors other than 'error.public-api.exceeded-api-key-allowance', 'error.security.oauth-token-invalid' & 'error.security.client-token-missing' seem to get ignored. Would you be open to a PR for that?


CI: releasing CLI to PyPI

Which issue does this PR close?

Closes #9294.
Rationale for this change

Documentation states that datafusion-cli can be installed through pip; however, that is not the case. This adds a workflow to deploy datafusion-cli (and only the CLI, not the python module) to PyPI , allowing installation using pip.
What changes are included in this PR?

This is meant to be a minimal CI, to be reviewed and edited as necessary (handle licensing, tag names, maybe maintainers prefer manual releases, etc)
CC @andygrove as he's involved in arrow-datafusion-python workflows and can have some comments.
A final note: builds for 32-bit systems fail because of  src/main.rs:304 which expects 1 << 40 to be usize. This causes cargo to panic and abort builds.
Are these changes tested?

You can try testing the deployment yourself using pip install datafusion-cli. I thought I changed the name and the repo to reference my fork, but apparently not.
I'll remove it from PyPI once the PR is ready to merge, to be redeployed by the maintainers.
Are there any user-facing changes?

Should be able to use pip to install datafusion-cli

Thanks @MohamedAbdeen21. I will review this in the next day or two.

@MohamedAbdeen21 My first question on this is why would we not do this from the arrow-datafusion-python repo? Many (perhaps most) of the maintainers of this crate are not interested in Python support and this would add an extra maintenance burden here.

Hey @andygrove this is not an issue of Python support, this is making DF more accessible to users through a more popular package manager i.e. pip, and it just happens that the easiest way to do so is through maturin.
Can we do it without maturin and the whole python bindings thing? probably, but that will be harder to maintain
Can we move this to arrow-datafusion-python? maybe, but I don't see what difference that makes given that there is no rust/python code involved in the deployment process.

Maybe once we require to manually bind some parts of the CLI before releasing we will have to move it to the python repo, but as for now  I see no point in doing so

Hey @andygrove this is not an issue of Python support, this is making DF more accessible to users through a more popular package manager i.e. pip, and it just happens that the easiest way to do so is through maturin.

My point is that DataFusion is a Rust project and datafusion-cli is already available via cargo, which is the default packaging manager for Rust. If we want to use Python packaging for DataFusion, it seems logical to do that in the DataFusion Python repository.

Logically it makes sense, but my point is that it doesn't have to be; as long as it can be deployed from the source why go through an extra layer of indirection?
Feel free to close this PR and re-open it in the Python repo, I'm happy to have DF on pip regardless of where it's deployed from.
I'll also remove the test from PyPI once the PR is closed.

Removed the release from PyPI and closing this issue, to be re-opened in the Python repo by the maintainers.


Embedded Manager's dark mode clashes with Dashboard's dark mode

One of the great things about the embedding is how it meshes together
However In dark mode the Dashboard's dark dark navy clashes with Manager's washed out black

vs

(raised as part of https://github.com/rancher/dashboard/pull/3031#pullrequestreview-669200556)

Tested on rancher/rancher:master-head c1151f6
The Legacy/Catalog page from summary now looks better:

But

Seems that even original Ember Light theme has been changed - now the colours are a bit more pastel like but some button icons were not adapted (same for dark theme - see next point + screenshot)
The Dark background in iframe is now closer to the dark theme in Dashboard but I see some glitches now - the same stands for Light theme.

Reopening

The glitches are tracked in https://github.com/rancher/rancher/issues/33449 and https://github.com/rancher/rancher/issues/33450
This can be closed then.


Cant reload from checkpoint when using SWA

🐛 Bug
My model worked just fine until I tried some optimisation using SWA.
from pytorch_lightning.callbacks import  StochasticWeightAveraging

weighting = StochasticWeightAveraging()

The problem is not even clear to understand :
KeyError                                  Traceback (most recent call last)
<ipython-input-20-2d36fa4eaad0> in <module>()
     16
     17
---> 18 trainer.fit(module, data_module, ckpt_path="./checkpoints/best-checkpoint.ckpt")
     19
     20 wandb.finish()

7 frames
/usr/local/lib/python3.7/dist-packages/torch/optim/lr_scheduler.py in load_state_dict(self, state_dict)
    233         """
    234
--> 235         lr_lambdas = state_dict.pop('lr_lambdas')
    236         self.__dict__.update(state_dict)
    237         # Restore state_dict keys in order to prevent side effects

KeyError: 'lr_lambdas'

To Reproduce
https://colab.research.google.com/github/PytorchLightning/pytorch-lightning/blob/master/pl_examples/bug_report/bug_report_model.ipynb
Expected behavior
Run from checkpoint with SWA.
Environment

CUDA:

GPU:

Tesla V100-SXM2-16GB

available:         True
version:           11.1

Packages:

numpy:             1.19.5
pyTorch_debug:     False
pyTorch_version:   1.10.0+cu111
pytorch-lightning: 1.5.9
tqdm:              4.62.3

System:

OS:                Linux
architecture:

64bit

processor:         x86_64
python:            3.7.12
version:           #1 SMP Tue Dec 7 09:58:10 PST 2021

cc @tchaton @rohitgr7 @akihironitta

Hi! Can I take this issue?

hey @BttMA can you update the reproducible colab link? currently it points to the one in the repo which doesn't have any of your update code.

Hello :) @rohitgr7
I hope this one will work :)
https://colab.research.google.com/drive/1JHaHvQ5PhfaYil0HnIkEOcF1MoindYcK?usp=sharing

hey @BttMA !
can you share an actual failing script?
I tried updating your example with:
def run(max_epochs, ckpt_path=None):
    train_data = DataLoader(RandomDataset(32, 64), batch_size=2)
    val_data = DataLoader(RandomDataset(32, 64), batch_size=2)
    test_data = DataLoader(RandomDataset(32, 64), batch_size=2)

    model = BoringModel()
    trainer = Trainer(
        default_root_dir=os.getcwd(),
        limit_train_batches=1,
        limit_val_batches=1,
        limit_test_batches=1,
        num_sanity_val_steps=0,
        max_epochs=max_epochs,
        enable_model_summary=False,
        callbacks = weighting ############
    )
    trainer.fit(model, train_dataloaders=train_data, val_dataloaders=val_data, ckpt_path=ckpt_path)
    return trainer

trainer = run(max_epochs=5, ckpt_path=None)
trainer.save_checkpoint('best_checkpoint.ckpt')
ckpt_path = 'lightning_logs/version_0/checkpoints/epoch=4-step=4.ckpt'
trainer = run(max_epochs=20, ckpt_path=ckpt_path)

and it worked fine... so I guess I am unable to reproduce your issue.

Sorry me neither I couldn't share the bug with you using the boring model.  is any either way to share it ?

share the notebook/script that is failing. You can attach it here too, in case someone else want to look at it.

it has personal/confidential data :/ I cant share it with everybody :o

maybe you can mimic the data. for starters, return random tensors of the same shape as your original data returns and use a small model.

hi @rohitgr7 :)
You can see the fail with this random generation :) hope you manage to solve it soon :)
https://colab.research.google.com/drive/1JHaHvQ5PhfaYil0HnIkEOcF1MoindYcK?usp=sharing#scrollTo=sNm4IkAefdkL&uniqifier=1
PS : maybe it has something to do with "UserWarning: SWA is currently only supported every epoch." or "Swapping scheduler LambdaLR for SWALR" ? but they are just warnings and cant lead to a fail loading!

yes! that might be the case.. We need to save and load the states for this callback to enable proper resuming.

What callback? Sorry didn't get you.
as you can see in the dummy code, when I get the "UserWarning: SWA is currently only supported every epoch." the program skip it right away and continue other epoch? also in my real version of code I get the same warning after like 10 to 15 epochs! it does not make any sense :dizzy_face:
Maybe the SWA does not support many epochs 🤔 ?? even in this doc it is not very clear about the epoch and the SWA. We have to dig deep into this! 😜

I am talking about StochasticWeightAveraging
the warning isn't reliable. Should be improved. I only got to know what it means by looking at the code.

UserWarning: SWA is currently only supported every epoch.

ideally it means if you are configuring interval='step' or frequency>1 inside scheduler configuration, it should work as per configuration expectation.
as you have done here:
return dict(lr_scheduler=dict(scheduler=scheduler, interval='step'),
                    optimizer=optimizer)

also in my real version of code I get the same warning after like 10 to 15 epochs! it does not make any sense 😵also in my real version of code I get the same warning after like 10 to 15 epochs! it does not make any sense 😵

check out the default parameters. by default it starts at when epoch=0.8*max_epochs.

check out the default parameters. by default it starts at when epoch=0.8*max_epochs.

now I see! for exemple if I have 100 epochs then the SWA callback will be activated at the 40th epoch (since 40=0.8*100).
In my case, the SWA callback is just skipped because of the scheduler LambdaLR. It is never executed and when I try to load a checkpoint it gets messy and I got that error ? correct me if I am wrong please ?

40th epoch (since 40=0.8*100)

80th epoch.
In your example, during the first run, it switch to SWALR at 40th epoch and saved the checkpoint at 50th epoch with SWALR state_dict. But when you reloaded the checkpoint, the trainer loaded them with LAmbda LR configured. Something like LambdaLR is trying to load the state_dict of SWALR, which is causing this error.

80th epoch.

OH!! sure yes yes!!

In your example, during the first run, it switch to SWALR at 40th epoch and saved the checkpoint at 50th epoch with SWALR state_dict. But when you reloaded the checkpoint, the trainer loaded them with LAmbda LR configured. Something like LambdaLR is trying to load the state_dict of SWALR, which is causing this error.

Now it makes sense :) Thank a lot!
Can you suggest any thing for me to fix this, please?
Should I change my scheduler in the plModel from LambdaLR for SWALR ? something like this :
swa_scheduler = torch.optim.swa_utils.SWALR(optimizer, anneal_strategy="linear", anneal_epochs=5, swa_lr=0.05)

Should I change my scheduler in the plModel from LambdaLR for SWALR ? something like this :

I'm not sure if this will work. I am not super familiar with every detail for SWA but I don't think that replacing the scheduler is all that's required to perform SWA. There's a lot more happening inside the callback. For the fix, I think we need to create states for this callback that can be stored and reloaded from the checkpoint while resuming the training. Need to investigate what all is required to make this work.

For the fix, I think we need to create states for this callback that can be stored and reloaded from the checkpoint while resuming the training.

Actually I was going to suggest that but I don't know what held me 😅
I will keep the issue open for further investigation (it will be helpful if you could mention other members.)
thanks a lot!

For the fix, I think we need to create states for this callback that can be stored and reloaded from the checkpoint while resuming the training

This is correct. Saving and loading is not implemented.

Should I change my scheduler in the plModel from LambdaLR for SWALR?

This is done by the callback automatically.


Stats is empty

Hello!
How can I see stats?
src/nutcracker -d --mbuf-size=1024 --stats-interval=30000 --verbose=6 --output=/var/log/nginx/proxy.log --stats-port=22222
Then I use this connection on production for 1 minute (5k qps).
Logs:
...
[2017-12-06 11:05:52.734] nc_connection.c:360 recv on sd 10 eof rb 567 sb 124
[2017-12-06 11:05:52.734] nc_request.c:430 c 10 is done
[2017-12-06 11:05:52.734] nc_core.c:237 close c 10 '<IP_ADDRESS>:20964' on event 00FF eof 1 done 1 rb 567 sb 124
[2017-12-06 11:05:52.734] nc_request.c:96 req 76105 done on c 8 req_time 0.226 msec type REQ_REDIS_ZREVRANGEBYSCORE narg 7 req_len 86 rsp_len 97 key0 'DB11' peer '<IP_ADDRESS>:20970' done 1 error 0
[2017-12-06 11:05:52.734] nc_request.c:96 req 76107 done on c 11 req_time 0.181 msec type REQ_REDIS_SETEX narg 4 req_len 79 rsp_len 5 key0 '456:51:d8437695e7a446210fc356b730d5e909:tmp' peer '<IP_ADDRESS>:20968' done 1 error 0
[2017-12-06 11:05:52.734] nc_request.c:96 req 76111 done on c 8 req_time 0.140 msec type REQ_REDIS_HGET narg 3 req_len 100 rsp_len 13 key0 'tables:domain' peer '<IP_ADDRESS>:20970' done 1 error 0
[2017-12-06 11:05:52.734] nc_connection.c:360 recv on sd 11 eof rb 456 sb 159
[2017-12-06 11:05:52.734] nc_request.c:430 c 11 is done
[2017-12-06 11:05:52.734] nc_core.c:237 close c 11 '<IP_ADDRESS>:20968' on event 00FF eof 1 done 1 rb 456 sb 159
[2017-12-06 11:05:52.734] nc_request.c:96 req 76114 done on c 8 req_time 0.156 msec type REQ_REDIS_HINCRBY narg 4 req_len 90 rsp_len 7 key0 'imps:420155' peer '<IP_ADDRESS>:20970' done 1 error 0
[2017-12-06 11:05:52.735] nc_request.c:96 req 76116 done on c 8 req_time 0.151 msec type REQ_REDIS_HINCRBY narg 4 req_len 81 rsp_len 8 key0 'campaigns:ads:stats_day:420144' peer '<IP_ADDRESS>:20970' done 1 error 0
[2017-12-06 11:05:52.735] nc_request.c:96 req 76118 done on c 8 req_time 0.139 msec type REQ_REDIS_SETEX narg 4 req_len 79 rsp_len 5 key0 '459:51:489c90dd70cc6bf3292a1722ac0f46ee:tmp' peer '<IP_ADDRESS>:20970' done 1 error 0
[2017-12-06 11:05:52.735] nc_connection.c:360 recv on sd 8 eof rb 436 sb 130
[2017-12-06 11:05:52.735] nc_request.c:430 c 8 is done
....

Stats (I checked it every 5 second):
src/nutcracker --describe-stats
This is nutcracker-0.4.1

pool stats:
  client_eof          "# eof on client connections"
  client_err          "# errors on client connections"
  client_connections  "# active client connections"
  server_ejects       "# times backend server was ejected"
  forward_error       "# times we encountered a forwarding error"
  fragments           "# fragments created from a multi-vector request"

server stats:
  server_eof          "# eof on server connections"
  server_err          "# errors on server connections"
  server_timedout     "# timeouts on server connections"
  server_connections  "# active server connections"
  server_ejected_at   "timestamp when server was ejected in usec since epoch"
  requests            "# requests"
  request_bytes       "total request bytes"
  responses           "# responses"
  response_bytes      "total response bytes"
  in_queue            "# requests in incoming queue"
  in_queue_bytes      "current request bytes in incoming queue"
  out_queue           "# requests in outgoing queue"
  out_queue_bytes     "current request bytes in outgoing queue"

ps aux | grep nut
root      5985  1.0  0.0  18584  2660 ?        Sl   11:04   0:44 src/nutcracker -d --mbuf-size=1024 --stats-interval=30000 --verbose=6 --output=/var/log/nginx/proxy.log --stats-port=22222

netstat -tulpn | grep 22222
tcp        0      0 <IP_ADDRESS>:22222           <IP_ADDRESS>:*               LISTEN      5985/nutcracker

BTW connection with redis took 0.15ms, connection with twemproxy took 0.03ms, but 4 pipes with redis was faster (1.0-1.15ms) than the same operations with twemproxy (1.0-1.5ms). I was hoping for more acceleration.
And also how to understand the logs?

Nowhere is it written how to look at the statistics, so look it like this: curl <IP_ADDRESS>:22222

One plus is fast connection.
Big minus that parts where there were about 100 HGET in 1 PIPE began to be executed 10+ times longer.

{"service":"nutcracker", "source":"ip-172-31-39-219", "version":"0.4.1", "uptime":1142, "timestamp":1512577733, "total_connections":2031818, "curr_connections":3080, "alpha": {"client_eof":2028692, "client_err":0, "client_connections":3123, "server_ejects":1, "forward_error":258, "fragments":0, "<IP_ADDRESS>:6379": {"server_eof":0, "server_err":0, "server_timedout":1, "server_connections":1, "server_ejected_at":1512577448998842, "requests":22937146, "request_bytes":1443893806, "responses":22935535, "response_bytes":340001499, "in_queue":520, "in_queue_bytes":30655, "out_queue":1023, "out_queue_bytes":60007}}}

It's raw tcp, not http. You can fetch the stats data with nc <IP_ADDRESS> 22222 < /dev/null


prevValue does not compare multi-attribute json

test_and_set(key, value, value)
Compare failed : [{"apiVersion":"v1","kind":"Endpoints","metadata":{"creationTimestamp":"2016-09-01T22:57:56Z","name":"kubernetes","namespace":"default","uid":"85c41ccb-7097-11e6-aea4-080027a1460e"},"subsets":[{"addresses":[{"ip":"<IP_ADDRESS>"},{"ip":"<IP_ADDRESS>"}],"ports":[{"name":"https","port":6443,"protocol":"TCP"}]}]} != {"kind":"Endpoints","apiVersion":"v1","metadata":{"name":"kubernetes","namespace":"default","uid":"85c41ccb-7097-11e6-aea4-080027a1460e","creationTimestamp":"2016-09-01T22:57:56Z"},"subsets":[{"addresses":[{"ip":"<IP_ADDRESS>"},{"ip":"<IP_ADDRESS>"}],"ports":[{"name":"https","port":6443,"protocol":"TCP"}]}]}
The value was fetched, decrypted and encrypted with json module.
It will compare correctly if the value is fetched and compared without running through json en/decrypt.

How is this a bug of python-etcd with this amount of information.


Implement Info Tab

In order to change what is shown in the Info Tab, the ./web/info.md needs to be modified.

Should the Info Tab be the Home tab as well or should they be separated?

If the original pdf mentions Info & Home tab yes, just let me know if it does so I can rename the issues and PR. @maikkundev

IIRC there's no mention of a Home tab on the PDF but we could include one or rename Info to Home to make the app feel more like an actual thing. @actuallylost

Then it's out of the scope of this application and this pull request.


options: update --bare help message

The current --bare help message is a little bit confusing: It says
that it will not work with --tags properly, but in fact, I would
assume it means that it may not work properly instead.
With --bare, the name will be ${KO_DOCKER_REPO}:<tag>.

Codecov Report

Merging #313 (bc35bbc) into main (8e1aaa1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #313   +/-   ##
=======================================
  Coverage   38.07%   38.07%
=======================================
  Files          33       33
  Lines        1513     1513
=======================================
  Hits          576      576
  Misses        844      844
  Partials       93       93

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 8e1aaa1...bc35bbc. Read the comment docs.

cc @mattmoor who added this iirc

This seems fine to me.


Main page sort order incorrect

Leftovers are not sorted by spoils-soonest, which is what it should be.

Fixed in 6df428a691b116c0c9bcbb8c28a842aac34b47a6


[Incremental] Minor cleanups to IncrementalCompilationState

In preparation for wave integration.

@swift-ci please test

👍

Thank you


Support raw Matlab script/functions embedded in master equations and mechanisms

Feature request: Support blocks of Matlab code embedded in equations and mechanism files.
Context: there are many things that one might want to do that can not be achieved using expressions on single lines. For instance, for loops, if-then statements, and subfunctions.
Current status: only single line expressions/statements are supported at present.
Implementation: the user should be able to indicate a block of code is raw Matlab and then implement whatever they need the same way they would in a Matlab script. On the development side, this will involve adding parameter/variable/function namespaces and inserting the block of raw Matlab code directly into the solve_file. Functions declared in the block of raw Matlab code should be appended to the end of the solve_file; everything else should be inserted into the solve_file immediately after the proceeding expression (note: order matters in this case).

I've just realized that this is way more important than it seems and once implemented I would encourage its usage against current functions and conditions, which in fact I would suggest to remove.
I know this is BOLD, but the reasons are the following:

Conditionals:

They are very limited (no elseif option) and faulty (an if-else conditional didn't work for me). Tried to fix both but it is very complicated thing to do, which made me realize the alternative suggested in this issue would be a much better approach.

Functions:

The way current functions work is by substitution, which is very nice and smart, yet pretty much limited (because once substituted there is no room for any change) and complex/convoluted codewise speaking (hence also difficult to follow and prompt to bugs). For instance one cannot define a function and then change its values with conditionals. And although I didn't try,  I'm almost sure one cannot define the a function following an if-else conditional.
You may wonder why (oh why!) removing instead of simply deprecating:
I would in principle suggest deprecating instead of removing of course. The problem of this is that I anticipate the complexity of this implementation being much higher (much longer time to deliver and with lots of issues to fix) if we are going to keep current conditionals and functions, because it's not going to be the same to "simply" deal with variable names according to scope, than to handle current functions and conditionals as well.
Also note that if we do, then all limitations about current functions and conditionals will still be potentially affecting this new implementation as well, and we won't have an easy way to help users design their models differently (that is combining functions and conditionals with matlab snippets).
To conclude, I would suggest implementing this issue in the most simple and natural way, assuming that all mechanisms are in fact in matlab language unless they appear in '' as we do in the main file. I think that for this to work, we would only need to:

define ODEs as we do outside mechanisms, that is like this: 'ODE', instead of simply ODE (that would make it actually more coherent)
apply proper scope to variables
replace current functions/conditions by expressions just the way they are naturally used in Matlab
move actual matlab functions within mechanisms to the end of the solve file

@jasonsherfey, what do you think?

Actually, perhaps the best option, in terms of modularity and clarity also for the solve file, and arguably the easiest implementation, might be to associate a matlab function to each mechanism and plug this to the end of the solve file.
This would allow forgetting about scopes inside the mechanisms, that is inside these functions.
Scope check would instead be applied to generate the arguments of the function but would not be applied inside the function itself.

Creating separate functions for each mechanism (which is how I originally implemented mechanisms; i.e., like NEURON .mod files) can dramatically increase simulation time in Matlab, probably due to overhead from Matlab passing variables between workspaces. It is much quicker to integrate the equations without external function calls. The same is true (even more so) for anonymous function calls, which is why I designed the integrator to evaluate expressions created after function substitution.
I think eliminating conditional makes sense for the reasons you stated (limited, buggy). Functions are tricky because the way they are handled has a very significant impact on how long the simulation takes to run.

I can see it would be generally slower.
However:

DynaSim is not (all) about speed (other alternatives like C++ and Julia are better)
I think using the coder would alleviate the overhead, although not sure how variables are passed in the mex file made by the coder (hopefully by reference)
DynaSim would be much easier to maintain and develop, without all the complexity of DynaSim functions and conditionals
Solve files would end up being much more human-friendly

But more importantly perhaps:

It would allow combining functions and conditionals, which is now not possible

Also quite important for allowing flexibility while optimizing speed:

Functions could have on-the-fly evaluation so if the anticipated result is zero (e.g., because for instance the conductance of a mechanism is zero), the body of the function does not need to be evaluated and the function can even return a 0 scalar value right away.

This is now not easily doable and these embedded mechanisms into expressions are evaluated even if conductances are set to 0.
How is that conductances are set to 0 and why is this important? Perhaps it's just me, but when "playing" with DynaSim models, it's not unusual (for me at least) to test adding mechanisms, so the complexity of the model increases rapidly, which has a strong impact on speed (and readibility). It may be that later on some of these mechanisms is not used but usually I don't remove the mechanisms but set them to "0" (on the course of a project one is not totally certain if that conductance will later change to a different than 0 value again).
Having them into functions, would allow setting a conditional at the head of its body, so if the conductance is zero, instead of overhead, this would imply saving time.

Nice summary from here:
https://www.mathworks.com/matlabcentral/answers/80001-speed-functions-and-overheads#answer_89745

When you call functions, you cannot avoid the overhead.
When you inline the code instead, the overhead vanishes, but other difficulties arise.
Tertium non datur.

I know that you have invested a lot of time implementing conditionals and functions as they are, for the benefit of speed. I really appreciate that effort.
I realized about all this complexity by the effort I needed to make to first understand and then improve current version of conditionals in DynaSim.
They now work with (as many) elseif conditions and a final else condition. Yet, I would encourage replacing them as well as DynaSim functions, to embrace the simplicity, flexibility and elegance of the language conditionals and functions (at the cost of speed; which can be improved using the coder as well as checking conductance values).

In my experience with development, most of the convoluted complexity that is frustrating pertains to the implementation of conditionals, not functions. Relatedly, most of the bugginess that I've experienced involves conditionals (or monitors, but that's another issue). I would be very happy to replace conditionals (and clean up the code) with the more powerful capabilities enabled by supporting raw Matlab. I also think it's very important to support conditional functions, which is currently not supported, as you discuss.
However, DynaSim was intended to enable model specification using conventional mathematical notation (i.e., like XPP), in addition to modular specification. That is the primary reason I think we should continue supporting functions specified as f(x) = .... Secondarily, every model of mine will break if we eliminate that form of specification.
Function substitution, which results in faster simulation but more obscure solve files, is already optional. There is an option reduce_function_calls_flag in dsWriteDynaSimSolver that can be set to 0 to produce a more readable solve file with functions defined/applied using anonymous function handles. There may be ways to clean up the way that functions are handled in DynaSim without eliminating support for f(x) = ...; I'll need to think about this more.
In summary:

I'd like to eliminate conditionals, which will make the code easier to understand/develop,
support raw Matlab for defining conditionals, functions, condition-dependent functions, etc,
and to continue supporting functions specified using conventional math notation.

It's also worth noting that a design goal of mine has always been to make it as easy as possible to prototype complex models, which is why I've focused on both intuitive notation and modular specification. I'm strongly in favor of anything that furthers that goal.

Alright, that may be a nice compromise actually. If we eliminate conditionals and allow current DynaSim functions as well as Matlab functions (supporting all flexibility of a programming language), then it may be the best of the two worlds.
By the way I totally agree with:

It's also worth noting that a design goal of mine has always been to make it as easy as possible to prototype complex models, which is why I've focused on both intuitive notation and modular specification. I'm strongly in favor of anything that furthers that goal.

But thought this was sort of orthogonal to this discussion and that we could manage to support this anyway with parsing.

I'm not sure what you are asking? What is "this"?

You mentioned about intuitive notation and modular specification
Intuitive notation is what we use for in the model script for describing the main ODE's
Not sure what you really mean by modular specification. If you mean mechanisms, then matlab functions are also modular. So I meant we could support parsing of mathematical expressions in mechanisms implemented otherwise as Matlab functions, similar to how the main script represents Matlab code with mathematical ODE's.
Not trying to convince you anymore about this, just saying that this seems orthogonal to whether or not use DynaSim functions as they are now.

In other words, I wasn't against mathematical notation. My proposal was to introduce them within single quotes as ODE's are introduced in the main model script and use parsing.
The rest of the content in the mechanism would then assumed to be Matlab code that we could directly used (I think this could simplify parsing, perhaps and allow for further flexibility?), jus as it is for matlab code in the main model script.


UUID field validation

We are using the dropwizard library for our microservices framework and would like to know if there is any support to validate the UUID fields. Consider the following DTO and the controller method attribute -
@JsonIgnoreProperties(ignoreUnknown = true)
public class SomeDto {
    @NotNull
    private final UUID id;
    @NotNull
    private final String type;

    public SomeDto(@JsonProperty("id") final UUID id,
                       @JsonProperty("type") final String type) {
        this.id = id;
        this.type = type;
    }

    public UUID getId() {
        return id;
    }

    . . . . .

Calling mechanism -
public Response create(
          @PathParam("id") final UUID someId,
          @Valid final SomeDto someDto) throws Exception {

Please let us know if there is any valid support to address such a requirement.

@beena-yatin-kanyal The type of a parameter will be validated in the sense that the request will fail if the parameter someId  of create(...) cannot be deserialized into a UUID parameter.

@joschi - No.  Consider that if from some request I pass the invalid UUID we need custom validation that can validate the UUID format and throw a custom message. Is there any such support available?

@beena-yatin-kanyal You can write a custom validator or a custom *Param annotation.

https://eclipse-ee4j.github.io/jersey.github.io/documentation/2.33/user-guide.html#d0e2273
https://docs.oracle.com/javaee/7/api/javax/ws/rs/ext/ParamConverter.html
https://github.com/dropwizard/dropwizard/blob/v2.0.20/dropwizard-jersey/src/main/java/io/dropwizard/jersey/validation/FuzzyEnumParamConverter.java

@joschi - I think it won't be a good idea to writing a param annotation to support multiple uuid's formats and would increase the code maintenance. Is there any support in the framework for such validation?? If not is there any milestone to have this added in dropwizard 2.x?

@beena-yatin-kanyal You can also validate the specific format of the UUID field in your constructor or in a @JsonCreator method.
There are no plans to implement something like this in Dropwizard.


AuthZ: Combine needs to use the PolicyProvider

cc @blowdart @divega

Hi @HaoK, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

c257c9528f16db78a33eed810b9a66f796146b3c


Add references to stats tests methods

One of the functionality in probatus allows to perform the statistical tests on two samples of data:

'ES': Epps-Singleton,
'KS': Kolmogorov-Smirnov statistic,
'PSI': Population Stability Index,
'SW': Shapiro-Wilk based difference statistic,
'AD': Anderson-Darling TS.

In multiple files we mention these tests in the docstrings, however, there are no references to the page/paper describing these tests. the references need to be added in:

probatus.stats_tests.distribution_statistics.py both DistributionStatistic class and AutoDist class
files in probatus.stats_tests: ad.py. es.py, ks.py, psi.py, sw.py
docs.tutorials.nb_distribution_statistics.ipynb

Make sure that reference is added correctly as in MD file format. They should appear in the docs as a hyperlinks.

Hello, may I work on this issue? Please let me know if the correct way to add a hyperlink to MD file format is the following:
[Text to be displayed](link to the website)

Hi! Great to hear you want to pick it up.
I suppose that this is the correct format. You can test if it works by first pip installing the following:
mkdocs-material>=6.1.0
mkdocs-git-revision-date-localized-plugin>=0.7.2
mkdocs-git-authors-plugin>=0.3.2
mkdocs-table-reader-plugin>=0.4.1
mkdocs-enumerate-headings-plugin>=0.4.3
mkdocs-awesome-pages-plugin>=2.4.0
mkdocs-minify-plugin>=0.3.0
mknotebooks>=0.6.2
mkdocs-print-site-plugin>=0.8.2
mkdocs-markdownextradata-plugin>=0.1.9

(these will be added to requirements.txt at some point in the future)
And then running mkdocs serve command in the to simulate how the probatus docs website will look like. It will show you an ip that you can open in a browser. On that website, you can check if the hyperlinks show up in the correct places e.g. API page for distribution statistics.
In order to check which exactly test is used you can check the code in files probatus.stats_tests: ad.py. es.py, ks.py, psi.py, sw.py, which method was used.

@dattanibedita How is it going with implementation of this issue? Do you need some help?
In case you are stuck, feel free to make a PR, and I can have a look and give you some tips in the comments.

Hello, I will open a PR soon. Where will I find the page/paper links that need to be mentioned in the docstring?

Completed by #158


Cannot run a miri test on a glommio application

Basically glommio runs some OS functions, like:
175 | /             libc::sched_setaffinity(
176 | |                 pid.into(),
177 | |                 mem::size_of::<CpuSet>() as libc::size_t,
178 | |                 &cpuset.cpu_set,
179 | |             )
    | |_____________^ can't call foreign function `sched_setaffinity` on OS `linux`

Is adding that a possibility? if it is, and you do have time, I'll gladly look into implementing this myself.

Impelementing sched_setaffinity should be easy. It did just be a no-op. Implementing iouring on the other hand is a lot more complex. Especially because the intention is that user code and a kernel thread access the same linked list concurrently. While I think it would be valid to block on the completion of the entire list of commands before returning from the syscall that triggers running the commands, even this is already non-trivial and it may hide a decent amount of bugs.

Duplicate of https://github.com/rust-lang/miri/issues/2749


Warning: comparison 'unsigned int' <=<PHONE_NUMBER> is always true

While compiling my code, I'm getting this warning.
hopscotch_hash.h:1743:39:  warning: comparison 'unsigned int' <=<PHONE_NUMBER> is always true [-Wtautological-constant-compare]"

Could you try the branch fix_issue_45?
Which version of Clang are you using? I can't reproduce the bug with my version even if compile in 32 bits.

Clang 8.2, I'll give your branch a try.

Looks like that branch did the trick.

I merged the change.


[11.x] feat: improve Factory generics, add generics to HasFactory

Hello!
This PR updates the Factory generics to make the types more accurate. I also added generics to the HasFactory trait, as well as allow the user to define a static $factory property in lieu of overriding the newFactory method which helps slim down models. Of course the method can still be overridden if desired.
// before
class User extends Authenticatable
{
    use HasFactory;

    protected function newFactory(): UserFactory
    {
        return UserFactory::new();
    }
}

// after
class User extends Authenticatable
{
    /** @use HasFactory<UserFactory> */
    use HasFactory;

    protected static string $factory = UserFactory::class;
}

Thanks!

Because I know, that PHP-CS-Fixer and Pint can have problems with formatting / ordering traits with comments in between, I wanted to ask, if the following is possible too:
/** @use HasFactory<UserFactory> */
class User extends Authenticatable
{
    use HasFactory;
}

@Jubeki, no it looks like placing @use on the class level does not work:
 ------ --------------------------------------------------------------------------------------------------------------------------
  Line   Autoload.php
 ------ --------------------------------------------------------------------------------------------------------------------------
  15     Class User uses generic trait Illuminate\Database\Eloquent\Factories\HasFactory but does not specify its types: TFactory
         🪪  missingType.generics
 ------ --------------------------------------------------------------------------------------------------------------------------

For what it's worth though, I've never had ordering issues with Pint (uses PHP-CS-Fixer) so that could be fixed.

@calebdw I checked again, it seems I remembered incorrectly. It was a spacing issue and not an ordering issue: https://github.com/laravel/pint/issues/278
So should be no problem!

Thanks @taylorotwell!


PrefixListIds not present in table

Describe the bug
The aws_vpc_security_group_rule does not contain a field for prefixlistids so when querying any rule that references a prefixlistid as the source is not displayed in the output
Steampipe version (steampipe -v)
Example: v0.7.3
Plugin version (steampipe plugin list)
Example: v0.41.0
To reproduce
Steps to reproduce the behavior (please include relevant code and/or commands).
Query security group containing references to prefix lists
steampipe query "select
  instance_id,
  group_name,
  sg ->> 'GroupId' as group_id,
  rule.from_port,
  rule.to_port,
  rule.ip_protocol,
  rule.cidr_ip,
  rule.pair_group_id,
  type
from
  aws_ec2_instance,
  jsonb_array_elements(security_groups) as sg
join
  aws_vpc_security_group_rule as rule on sg ->> 'GroupId' = rule.group_id
where
   rule.from_port <= 443 and rule.to_port >= 443
   and group_id = 'sg-2sdfw093423rqwr'"

+---------------------+--------------------------------+----------------------+-----------+---------+-------------+------------------+----------------------+---------+
| instance_id         | group_name                     | group_id             | from_port | to_port | ip_protocol | cidr_ip          | pair_group_id        | type    |
+---------------------+--------------------------------+----------------------+-----------+---------+-------------+------------------+----------------------+---------+
| i-2309iweqnfw3r231wd | name-sg | sg-2sdfw093423rqwr | 443       | 443     | tcp         | <IP_ADDRESS>/8       | <null>               | ingress |
| i-2309iweqnfw3r231wd | name-sg | sg-2sdfw093423rqwr | 443       | 443     | tcp         | <IP_ADDRESS>/25   | <null>               | ingress |
| i-2309iweqnfw3r231wd | name-sg | sg-2sdfw093423rqwr | 443       | 443     | tcp         | <IP_ADDRESS>/25 | <null>               | ingress |
| i-2309iweqnfw3r231wd | name-sg | sg-2sdfw093423rqwr | 443       | 443     | tcp         | <IP_ADDRESS>/25   | <null>               | ingress |
| i-2309iweqnfw3r231wd | name-sg | sg-2sdfw093423rqwr | 443       | 443     | tcp         | <null>           | sg-asdf092u3jroi2nwef | ingress |
+---------------------+--------------------------------+----------------------+-----------+---------+-------------+------------------+----------------------+---------+

Above output should include rules that reference perfixlistIds
Expected behavior
A clear and concise description of what you expected to happen.
Expected output contains the full list of rules for sources of all types
Additional context
Add any other context about the problem here.
An additional table allowing the CIDR blocks of the prefixlistIds would be of great benefit from an audit perspective.

@jonesy1234 Thank you for using Steampipe. Definitely interesting feedback here. Appreciate opening the issue, we are looking into it.
Looking forward if any other feedback/contribution to improve.