feat(falcosidekick)!: support Ingress API version networking.k8s.io/v1

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #273
Special notes for your reviewer:
$ helm version
version.BuildInfo{Version:"v3.6.3", GitCommit:"d506314abfb5d21419df8c7e7e68012379db2354", GitTreeState:"clean", GoVersion:"go1.16.5"}

Kubernetes <= 1.17:
$ helm template --namespace falcosidekick --name-template falcosidekick -s templates/ingress.yaml -s templates/ingress-ui.yaml --set ingress.enabled=true --set ingress.hosts[0].host=sidekick.falco.org --set ingress.hosts[0].paths[0].path=/ --set webui.enabled=true --set webui.ingress.enabled=true --kube-version 1.17 .
---
# Source: falcosidekick/templates/ingress.yaml
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: falcosidekick
  namespace: falcosidekick
  labels:
    app.kubernetes.io/name: falcosidekick
    helm.sh/chart: falcosidekick-0.3.17
    app.kubernetes.io/instance: falcosidekick
    app.kubernetes.io/managed-by: Helm
spec:
  rules:
    - host: "sidekick.falco.org"
      http:
        paths:
        - path: /
          backend:
            serviceName: falcosidekick
            servicePort: http
---
# Source: falcosidekick/templates/ingress-ui.yaml
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: falcosidekick-ui
  namespace: falcosidekick
  labels:
    app.kubernetes.io/name: falcosidekick-ui
    helm.sh/chart: falcosidekick-0.3.17
    app.kubernetes.io/instance: falcosidekick-ui
    app.kubernetes.io/managed-by: Helm
spec:
  rules:
    - host: "falcosidekick-ui.local"
      http:
        paths:
        - path: /ui
          backend:
            serviceName: falcosidekick-ui
            servicePort: http
        - path: /events
          backend:
            serviceName: falcosidekick-ui
            servicePort: http
        - path: /healthz
          backend:
            serviceName: falcosidekick-ui
            servicePort: http
        - path: /ws
          backend:
            serviceName: falcosidekick-ui
            servicePort: http

Kubernetes 1.18 (supports pathType with an implicit default, from 1.19 the pathType field in mandatory):
$ helm template --namespace falcosidekick --name-template falcosidekick -s templates/ingress.yaml -s templates/ingress-ui.yaml --set ingress.enabled=true --set ingress.hosts[0].host=sidekick.falco.org --set ingress.hosts[0].paths[0].path=/ --set webui.enabled=true --set webui.ingress.enabled=true --kube-version 1.18 .
---
# Source: falcosidekick/templates/ingress.yaml
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: falcosidekick
  namespace: falcosidekick
  labels:
    app.kubernetes.io/name: falcosidekick
    helm.sh/chart: falcosidekick-0.3.17
    app.kubernetes.io/instance: falcosidekick
    app.kubernetes.io/managed-by: Helm
spec:
  rules:
    - host: "sidekick.falco.org"
      http:
        paths:
        - path: /
          pathType: ImplementationSpecific
          backend:
            serviceName: falcosidekick
            servicePort: http
---
# Source: falcosidekick/templates/ingress-ui.yaml
apiVersion: networking.k8s.io/v1beta1
kind: Ingress
metadata:
  name: falcosidekick-ui
  namespace: falcosidekick
  labels:
    app.kubernetes.io/name: falcosidekick-ui
    helm.sh/chart: falcosidekick-0.3.17
    app.kubernetes.io/instance: falcosidekick-ui
    app.kubernetes.io/managed-by: Helm
spec:
  rules:
    - host: "falcosidekick-ui.local"
      http:
        paths:
        - path: /ui
          pathType: ImplementationSpecific
          backend:
            serviceName: falcosidekick-ui
            servicePort: http
        - path: /events
          pathType: ImplementationSpecific
          backend:
            serviceName: falcosidekick-ui
            servicePort: http
        - path: /healthz
          pathType: ImplementationSpecific
          backend:
            serviceName: falcosidekick-ui
            servicePort: http
        - path: /ws
          pathType: ImplementationSpecific
          backend:
            serviceName: falcosidekick-ui
            servicePort: http

Kubernetes 1.19+:
$ helm template --namespace falcosidekick --name-template falcosidekick -s templates/ingress.yaml -s templates/ingress-ui.yaml --set ingress.enabled=true --set ingress.hosts[0].host=sidekick.falco.org --set ingress.hosts[0].paths[0].path=/ --set webui.enabled=true --set webui.ingress.enabled=true --kube-version 1.19 .
---
# Source: falcosidekick/templates/ingress.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: falcosidekick
  namespace: falcosidekick
  labels:
    app.kubernetes.io/name: falcosidekick
    helm.sh/chart: falcosidekick-0.3.17
    app.kubernetes.io/instance: falcosidekick
    app.kubernetes.io/managed-by: Helm
spec:
  rules:
    - host: "sidekick.falco.org"
      http:
        paths:
        - path: /
          pathType: ImplementationSpecific
          backend:
            service:
              name: falcosidekick
              port:
                name: http
---
# Source: falcosidekick/templates/ingress-ui.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: falcosidekick-ui
  namespace: falcosidekick
  labels:
    app.kubernetes.io/name: falcosidekick-ui
    helm.sh/chart: falcosidekick-0.3.17
    app.kubernetes.io/instance: falcosidekick-ui
    app.kubernetes.io/managed-by: Helm
spec:
  rules:
    - host: "falcosidekick-ui.local"
      http:
        paths:
        - path: /ui
          pathType: ImplementationSpecific
          backend:
            service:
              name: falcosidekick-ui
              port:
                name: http
        - path: /events
          pathType: ImplementationSpecific
          backend:
            service:
              name: falcosidekick-ui
              port:
                name: http
        - path: /healthz
          pathType: ImplementationSpecific
          backend:
            service:
              name: falcosidekick-ui
              port:
                name: http
        - path: /ws
          pathType: ImplementationSpecific
          backend:
            service:
              name: falcosidekick-ui
              port:
                name: http

Checklist

[x] Chart Version bumped
[x] CHANGELOG.md updated

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

b051154 feat(falcosidekick)!: support Ingress API version networking.k8s.io/v1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sboschman
To complete the pull request process, please assign cpanato after the PR has been reviewed.
You can assign the PR to them by writing /assign @cpanato in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @sboschman! It looks like this is your first PR to falcosecurity/charts 🎉

New chart version is 1.0.0 following semver with a breaking change or ?
Just having a bit of a fight with CircleCI atm, I want to see the failing test results, without giving CircleCI read/write permissions to my entire github or bitbucket account!
They are insane imo requesting those permissions, they don't need those permissions for pure authentication against a github or bitbucket account and they for sure do not need those permissions to show me some test results for a public repo (which I do not even own).

We ask for read/write access to make your experience seamless on CircleCI.

Requesting:

Administer your repositories
Read and modify your repositories

no way in hell, sorry

Can you also embed this fix please: https://gitlab.qonto.co/devops/kubernetes-resources/kyverno-policies/-/merge_requests/15

I can't access that site/gitlab instance:
$ nslookup gitlab.qonto.co; curl https://gitlab.qonto.co
Non-authoritative answer:
gitlab.qonto.co canonical name = gitlab-external-elb-1924446322.eu-west-3.elb.amazonaws.com.
Name:   gitlab-external-elb-1924446322.eu-west-3.elb.amazonaws.com
Address: <IP_ADDRESS>
Name:   gitlab-external-elb-1924446322.eu-west-3.elb.amazonaws.com
Address: <IP_ADDRESS>

curl: (28) Failed to connect to gitlab.qonto.co port 443: Connection timed out

@sboschman and you will never, I f*** up my copy/paste, sorry.
I was talking about #267

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sboschman
To complete the pull request process, please ask for approval from issif after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sboschman and you will never, I f*** up my copy/paste, sorry.
I was talking about #267

 ✖ falco => (version: "1.15.7", path: "falco") > Chart version not ok. Needs a version bump!
 ✔ falcosidekick => (version: "0.4.0", path: "falcosidekick")

Not sure this is the right way @Issif to do this. #267 is a falco chart change, not a falcosidekick chart change. Seems I have to bump the falco chart version as well. Isn't it clearer to embed #267 in a PR for the falco chart to keep commits and merges separate?

@sboschman and you will never, I f*** up my copy/paste, sorry.
I was talking about #267

 ✖ falco => (version: "1.15.7", path: "falco") > Chart version not ok. Needs a version bump!
 ✔ falcosidekick => (version: "0.4.0", path: "falcosidekick")

Not sure this is the right way @Issif to do this. #267 is a falco chart change, not a falcosidekick chart change. Seems I have to bump the falco chart version as well. Isn't it clearer to embed #267 in a PR for the falco chart to keep commits and merges separate?

You're totally right, I answered too quickly (again), sorry. I'll stop answering to Github issues from phone in a train 😅

LGTM label has been added.  Git tree hash: 395da85f0caf0b14191c8fa6bb629131d1fb4b55

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, sboschman
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[DOC][0.9] Invalid port/typo in backup and restore section

Hello,
I believe there is a typo in influxdb v0.9 documentation in section related to backup[1].

influxd backup -host <IP_ADDRESS>:8808 /tmp/mysnapshot

But the port is 8088, isn't it? If you tell me where the docs are, I can create a PR for this. :)
EDIT: as a matter of fact, I don't think I can do that(sign CLA and stuff) at the moment. sorry :(
Thanks!
[1] https://docs.influxdata.com/influxdb/v0.9/administration/backup_and_restore/

I found out in another issue this isn't the right place to report docs issues. Perhaps it'd be good idea to put this information into guidelines? Although, I admit I've read them after I've created the issue :)


Programmer mode: Add shortcuts for switching to non-decimal input

Problem Statement
The programmer mode has no hotkeys to switch between decimal / hexdecimal / octal and binary input.
Evidence or User Insights
It's quite annoying for programmers to be able to launch the calculator and switch to programmer mode all by using the keyboard and then taking a hand off and use the mouse to click on HEX or BIN or OCT and then switching back to keyboard to input the value.
We also do have keyboard shortcuts for pretty much anything else which kinda makes sense to support this as well.
Proposal
Add shortcuts to switch between decimal / hexdecimal / octal and binary input.
Goals
User can continue using the keyboard to enter non decimal values without taking hand off and using the mouse.
Low-Fidelity Concept
I don't think it's required for this request.
Requested Assignment
I'm just suggesting this idea.  I don't want to implement it.

Figured it already exists but is nowhere documented and the GUI doesn't expose them like it does with anything else: https://github.com/microsoft/calculator/issues/157

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.


If a regex removes an entire subtitle, make condensed mode skip that subtitle

From ZyphDoz on Discord:
If a regex removes an entire subtitle, make condensed mode skip that subtitle. Would be useful because you could filter out stuff like はあっ！|くっ！|はっ！|あっ！ to fly past action scenes

I'm guessing that was implied but...
That'd be true for the auto-pause feature as well =]

The subtitle is removed from the list when only whitespace is left
58d8ff0a1ffc7709cbe6e6eea3c0f65845ce3e6b


Issue with React-Officegraph sample web part

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
Category

[ ] Question
[X ] Bug
[ ] Enhancement

Expected or Desired Behavior
React-Officegraph SPFx web part is not compiling with latest SPFx  tools
https://github.com/SharePoint/sp-dev-fx-webparts/tree/master/samples/react-officegraph
Observed Behavior
There are many error like below:
Could not find implementations for the following rules specified in the configuration:
prefer-const
Tried updating TSLint but no success.
Steps to Reproduce

Downloaded latest SPFx tools
Download sample
Gulp Build -- gives the error

Submission Guidelines
_

@dharmenb if I recall it correctly, the issue you mention is a warning not an error that's preventing the compilation, or has it changed? Have you tried running gulp serve to see if you can start the workbench and add the web parts?

Yeah I tried gulp serve, it is not working. Lot of other samples seems to broken as well.
Regards,
Dharmendra
Sent from my iPhone
On 12-Nov-2017, at 2:19 AM, Waldek Mastykarz <notifications<EMAIL_ADDRESS>wrote:
@dharmenbhttps://github.com/dharmenb if I recall it correctly, the issue you mention is a warning not an error that's preventing the compilation, or has it changed? Have you tried running gulp serve to see if you can start the workbench and add the web parts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/SharePoint/sp-dev-fx-webparts/issues/358#issuecomment-343693229, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALVWKmqXX2bRCQBAh5u-33ubyhG6uHGVks5s1ghngaJpZM4QTaRp.

@waldekmastykarz I can confirm that it doesn't compile because the references in the package.json are outdated. This is the result of the update check:

After I upgraded all versions there were some deprecated warnings but overall it compiles and the web part can be found.
This are the problems I got:

Node Version: v6.12.0
Npm Version: 5.5.1

Thanks for confirming, @StfBauer. Regarding this particular sample, the question is if it would be worth upgrading, given that Office Graph is deprecated.

git mv react-officegraph deprecated/react-officegraph maybe ;)

I updated the Readme file to indicate that the sample uses deprecated packages. If someone wants to create a newer version of this sample, please feel free to do so.


Increase pagination limits

I've merged earlier today https://github.com/xataio/xata/pull/3117 which significantly increases the pagination limits:

max page size from 200 to 1000
max offset from 800 to 49000 (for a total of 50k)

e2e tests failed on staging and it makes sense because the client / e2e tests expected the old limits. I think this should fix it, but I didn't try it yet. I'll try running the tests against staging.

Tested from local against staging and that passed. I'm not sure about the merge order, though?
We force-merge this, then retry the deployment in the xata repo?

Sounds good


.IsFixedLength() creates/alters varchar instead of char

Version 2.1.1
The following:
builder.Property(x => x.Password);
builder.Property(x => x.Salt).IsFixedLength();
will create a migration:
password = table.Column<string>(maxLength: 64, nullable: false),
salt = table.Column<string>(fixedLength: true, maxLength: 64, nullable: false),
The migration will create a table with:
password varchar(64) NOT NULL,
salt varchar(64) NOT NULL, <- This should be char(64).

Thanks for raising, fixed for 2.1.2.


Invalid json

The time series objects like Confirmed cases, Deaths and recovered are all invalid json. Reasoning behind is that the supposed json contains an extra } at the very end.

https://github.com/nat236919/Covid2019API/commit/6344eb3346abc9b4ee5ea9497cbb211b223e014d I've wrapped the case result in an array, I hope it will do.


Flawed implementation of selectrows(model, rows, data)

using MLJBase
using MLJModels

EvoLinearRegressor = @load EvoLinearRegressor
model = EvoLinearRegressor()
X, y = make_regression()
mach = machine(model, X, y) |> fit!
predict(mach, X);

# julia> predict(mach, rows=1:2)
# ERROR: MethodError: no method matching predict(::EvoLinear.EvoLinearRegressor{Float32, Int64, Symbol}, ::EvoLinear.EvoLinearModel{EvoLinear.MSE, Vector{Float32}, Float32}, ::SubArray{Float64, 2, Matrix{Float64}, Tuple{UnitRange{Int64}, Base.Slice{Base.OneTo{Int64}}}, false}, ::Vector{Symbol})
# Closest candidates are:
#   predict(::Machine{<:Static}, ::Any, ::Any...) at ~/MLJ/MLJBase/src/operations.jl:141
#   predict(::EvoLinear.EvoLinearRegressor, ::Any, ::Any) at ~/.julia/packages/EvoLinear/S3Tc0/src/MLJ.jl:44
#   predict(::NetworkComposite, ::Any, ::Any) at ~/MLJ/MLJBase/src/operations.jl:224
#   ...
# Stacktrace:
#  [1] predict(mach::Machine{EvoLinear.EvoLinearRegressor{Float32, Int64, Symbol}, true}; rows
# ::UnitRange{Int64})
#    @ MLJBase ~/MLJ/MLJBase/src/operations.jl:86
#  [2] top-level scope
#    @ REPL[49]:1
#  [3] top-level scope
#    @ ~/.julia/packages/CUDA/DfvRa/src/initialization.jl:52

Took me a while to sort this one out but here's the diagnosis:
Xnew = mach.data[1]

# julia> MLJBase.selectrows(model, 1:2, Xnew) isa Tuple
# false

The output of selectrows(models, rows, data...) (and reformat) must always be a tuple with the same length as data, even if this is one. See the items marked as "important" in the spec.
(This was corrected done in EvoTrees.jl.)

Sorry to pester. Is this still on your radar?

Thanks for the reminder!
Yes the MLJ fix is already present in the spline branch. Only a few extra experiments were needed to assess the reasonability of the spline options, I should be able to push new release within 1-2 days.


Move extensive task docstring to getting-started.md

Also fix "Examples:" sections breaking mkdocs rendering

Pull Request Checklist

[x] Fixes bad doc rendering
[ ] ~Tests added~
[x] Documentation/examples added
[x] Good commit messages and/or PR title

Description of PR
Currently, mkdocs is having trouble rendering the doc website
This PR fixes some of the docstrings escaping the rendering which results in sections like

ToC now fixed all the way down 😎
Might just update links as they're not clickable which I think need to be markdown [text](link) style


Remove EC:<IP_ADDRESS> as xref to phosphatase activity (GO:0016791)?

EC:<IP_ADDRESS> = 4-nitrophenylphosphatase (4-nitrophenyl phosphate + H(2)O <=> 4-nitrophenol + phosphate)
https://enzyme.expasy.org/EC/<IP_ADDRESS>
I'm not sure if this should really be an xref (and included in the def) of phosphatase activity (GO:0016791)?  The other partial xref (EC:3.1.3.-) is fine.

This was due to a merge; that ec should have been removed; done

Thanks Harold.
Can/did you remove this EC from the definition line as well?  Current full definition reads:
"Catalysis of the hydrolysis of phosphoric monoesters, releasing inorganic phosphate." [EC:3.1.3, EC:<IP_ADDRESS>, GOC:curators, GOC:pg]

yes; it is gone gone


Update Dropdown Options for Phases Form

-Update dropdown for engagement phases to update the recommendation text
-Requested by Marie Anik
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1039 (471dac0) into main (3406e67) will increase coverage by 0.64%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1039      +/-   ##
==========================================
+ Coverage   74.42%   75.07%   +0.64%
==========================================
  Files         217      217
  Lines        6013     5949      -64
  Branches      388      357      -31
==========================================
- Hits         4475     4466       -9
+ Misses       1495     1428      -67
- Partials       43       55      +12

Flag
Coverage Δ

metweb
69.08% <100.00%> (+1.20%)
:arrow_up:

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

Impacted Files
Coverage Δ

...ement/form/EngagementWidgets/Phases/PhasesForm.tsx
78.72% <ø> (ø)

met-web/src/models/engagementPhases.tsx
100.00% <100.00%> (ø)

met-web/src/services/widgetService/index.tsx
23.07% <0.00%> (ø)

met-web/src/apiManager/httpRequestHandler/index.ts
58.82% <0.00%> (ø)

...rvices/widgetService/DocumentService.tsx/index.tsx
20.58% <0.00%> (ø)

met-api/src/met_api/services/comment_service.py
71.42% <0.00%> (+0.46%)
:arrow_up:

met-web/src/services/feedbackService/index.tsx
26.66% <0.00%> (+1.66%)
:arrow_up:

met-web/src/services/contactService/index.tsx
25.00% <0.00%> (+3.12%)
:arrow_up:

met-web/src/services/engagementService/index.ts
27.77% <0.00%> (+5.05%)
:arrow_up:

...web/src/services/emailVerificationService/index.ts
31.25% <0.00%> (+6.25%)
:arrow_up:

... and 4 more


Group project sources according to folders on filesystem

It would be really nice if the source groups generated for IDE projects (e.g. Visual Studio) could be the same as on the filesystem. Fortunately, there is a command for that: source_group(TREE ...). Would you mind adding this to your CMakeLists.txt?
We use it as follows (where PROJECT_SOURCE_DIR points to the root directory of your source code):
file (GLOB_RECURSE ALL_FILES RELATIVE ${PROJECT_SOURCE_DIR} "*.cpp" "*.cxx" "*.h" "*.hxx")
source_group(" " FILES ${all_files})
source_group(TREE ${PROJECT_SOURCE_DIR} FILES ${all_files})

@bernhardmgruber  Feeling free to implement this. We don't usw visual studio, so I cant Test this by myself .


CMakeLists: Use the jsoncpp submodule included in VRPN.

We've already checked out this submodule, so why not use it?
Depends on https://github.com/vrpn/vrpn/pull/109.

I noticed this PR currently only works when building the server, but not when you're only building the client. I'll have a closer look at it next week.

@rpavlik This PR works correctly now, but it depends on https://github.com/vrpn/vrpn/pull/109.

I'd feel better just having the build done locally, inside OSVR's build system, but using the submodule, if the version in the submodule is sufficient - I haven't compared that version against the one I typically use recently. Doing so in the simplest way possible would leave some consumers of a pre-compiled library high and dry (or at least in need of building jsoncpp on their own), but there's no reason we couldn't support an external jsoncpp with the internal as fallback - the Linux distros whose packaging rules I'm familiar with would require this anyway (Debian really does not like vendored-in versions of libraries, and jsoncpp is widely used and widely packaged - and we're not terribly picky about the version we use).

I'd feel better just having the build done locally, inside OSVR's build system, but using the submodule

Would a PR that implements that be acceptable?
Also, currently we depend on HID API being compiled in VRPN, the only reason I had to make jsoncpp a separate project is because it isn't compiled with VRPN if we're just building the client. So should the same be done for HID API at some point?


use case range instead of cap [skip ci]

unnecessary. if you really need this just have the change locally but it doesnt do anything

I'd like to use this repo for debugging SBMLToolkit. It wastes hours running hundreds of cases just for checking if the bugfix for one case worked. What is your rational for having an upper but not lower boundary? In particular, what disadvantages come with a lower boundary?

why aren't you just running on the file you're testing? it makes no sense to run all of them for a single test file

Cumbersome API with lower_one requiring a dataframe as argument. Thanks for changing that, that helps already. Also a bit cumbersome to add that both, lower_one and verify_case both need the whole path as input. Could be solved by adding a utility function get_case_dir("00001") so you can verify_case(get_case_dir("00001")). How does that sound? Defo the README also needs better instructions how to use this repo.


Support for spaces in paths on Windows systems added.

Fixes issue: #6891

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

Seems good to me. Care to take a look @mhegazy?

I just saw that my changes are missing in the current jakefile.js.
@mhegazy
Hasn't this been merged yet?

@DanielRosenwasser
I noticed that my changes have been accidentally reversed in a later contribution.
I have now created a successor pull request @ #8267.


Simplify

Coverage increased (+0.6%) to 89.655% when pulling 64346761ebec566f4eced7fdad25d3e2d793c1e3 on simplify into db25fcf30a8f9047c80775be7a5eb7e20eeb06c1 on master.


[WIP] Refactoring + support for testing

[x] rename HandlerBehaviour -> Handler
[x] move DummyHandler to examples dir from lib
[x] don't change for handlers when starting up
[x] add RestAuth.Configure plug that stores configuration in conn
[x] move all auth configuration from app env to handler
[x] add tests
[x] figure out what to do with CacheService

@olivermt I think this is ready to merge.


Flutter Development in android studio is extremely painful.

I've been developing a flutter app in android studio for about 4 months now. and more of my time is spent debugging why android studio is crashing/breaking than actually developing my application.
flutter needs a more stable environment.
things I've run into:

android studio can't seem to remember that the project is a flutter app, requiring every startup to resync gradle files and re-enable dart support.
flutter fails to properly build a release APK (see issues already filed around missing libflutter.so from APK)
android studio marking files as read only, even when I created them inside android-studio (wtf?) I have yet to figure out how to fix this.
android studio can't seem to handle the java aspects of the project simultaneously with the dart without opening a new window specifically for the java portion.

Please create a new issues for every concrete problem.
If you create an issue, please provide as much information as possible that allows to reproduce that concrete problem.
IntelliJ/Android Studio integration is maintained in https://github.com/flutter/flutter-intellij. Please create related issues there.
Issues need to be about concrete problems. General rants are not actionable.

its hard to give concrete details when there are so many continuous problems =) but I'll start over in the intellij repo.

I fully understand that such issues are hard to reproduce and report accordingly
and we really appreciate your effort.

flutter fails to properly build a release APK (see issues already filed around missing libflutter.so from APK)

I'm pretty sure this is unrelated to AS/IntelliJ and will have the same behavior on command line as from within AS/IntelliJ.


2024-11-20_01-35-31_master-next_python3-boto3

Automatically created.

Backport failed for kirkstone-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin kirkstone-next
git worktree add -d .worktree/backport-10428-to-kirkstone-next origin/kirkstone-next
cd .worktree/backport-10428-to-kirkstone-next
git switch --create backport-10428-to-kirkstone-next
git cherry-pick -x ec129dd44f53cb43c9eeae5dcdc3e3fd46cb19ee

Backport failed for scarthgap-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin scarthgap-next
git worktree add -d .worktree/backport-10428-to-scarthgap-next origin/scarthgap-next
cd .worktree/backport-10428-to-scarthgap-next
git switch --create backport-10428-to-scarthgap-next
git cherry-pick -x ec129dd44f53cb43c9eeae5dcdc3e3fd46cb19ee

Backport failed for styhead-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin styhead-next
git worktree add -d .worktree/backport-10428-to-styhead-next origin/styhead-next
cd .worktree/backport-10428-to-styhead-next
git switch --create backport-10428-to-styhead-next
git cherry-pick -x ec129dd44f53cb43c9eeae5dcdc3e3fd46cb19ee


Change the --wait option to wait for the command

This changes the meaning of the --wait option.
I find this alternate behaviour much more useful. I don't ever want to have two copies of my tests running at the same time. I suspect that for most/all people using this tool with --wait X this will be a compatible replacement.
If you're interested but want to keep backwards compatibility I'd be happy to write the extra code behind a new option flag.
Thanks for looking :)

This seems like a useful feature, but, I don't think we should make it a breaking change to the current --wait flag.  I'd expect this to be a new option.
In the end, the design is up to @mikeal on this one.

Sure, I'd be happy to rework it as an alternative option. I just thought I'd put it up here first in its simplest form to see if you were interested :)

Hi @mikeal, would you be interested in this feature? Either as-is or as a separate option?
I'm also curious what was the intention behind the existing --wait feature and how people use it right now.
Thanks :)

@jbunton-atlassian does this work with child processes?
execshell uses sh -c which can result in not waiting for orphans
Example:
cli.js ... script.sh
script.sh

ls
tree &
exit

Would this code wait for tree to finish before executing it again?
I implemented something similar by checking for child processes in #128

@hbt, no, it only waits for the parent process in that case. In your example you should be able to change exit to wait to have the same effect.
I had a quick look at your code and noticed that it discards modification events while waiting rather than queueing them. I use watch to run tests, so if I make changes while the tests are running I want them to rerun immediately. Did you consider this use case?

Hi @mikeal, any thoughts on this feature?


Crash with Collector's Reap Mod

Hi, I've updated seasonals to 4.1.0 but my game crashed and I speculate that it is because of an incompatibility with collector's reap mod which is a mod that expands Farmers Delight along with Seasonals.
There's the crash report file.
crash-2023-08-17_15.16.02-client.txt
I hope there is a solution.
I appreciate the feedback.

Hey, thanks for reporting this. Unfortunately I cannot fix this problem on my end. I will look into fixing the problem on the end of the collector's reap mod


ArgumentOutOfRangeException: The DateTimeOffset specified cannot be converted into a Zip file timestamp - Part Two

What You Are Seeing?

What is Expected?
It should silently correct the date being out of range. This behavior was fixed originally in #2803.
What version of Cake are you using?
0.38.4
Are you running on a 32 or 64 bit system?
64
What environment are you running on?  Windows? Linux? Mac?
Windows 10
Version 10.0.18363 Build 18363
Are you running on a CI Server?  If so, which one?
This is only showing up locally so far. I'm in US Central Time.
How Did You Get This To Happen? (Steps to Reproduce)
I run Cake locally.
Output Log
Preparing to run build script...
Running build script...
0.38.4
Module directory does not exist.
NuGet Config not specified. Will use NuGet default mechanism for resolving it.
Analyzing build script...
Analyzing C:/Users/thomas.sobieck/Documents/GitHub/school/build.cake...
Processing build script...
Installing tools...
Installing addins...
The addin Cake.FileHelpers will reference Cake.FileHelpers.dll.
Verifying assembly 'Cake.FileHelpers, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Adding assembly reference to mscorlib.dll...
Adding assembly reference to System.Core.dll...
Adding assembly reference to Microsoft.CSharp.dll...
Adding assembly reference to Cake.Core.dll...
Adding assembly reference to Cake.Common.dll...
Adding assembly reference to Cake.exe...
Adding assembly reference to System.dll...
Adding assembly reference to System.Xml.dll...
Adding assembly reference to System.Xml.Linq.dll...
Adding assembly reference to System.Data.dll...
Adding assembly reference to System.Runtime.dll...
Adding assembly reference to System.Collections.dll...
Adding assembly reference to System.Net.Http.dll...
Adding assembly reference to netstandard.dll...
Adding assembly reference to Cake.FileHelpers.dll...
Importing namespace Cake.Common...
Importing namespace Cake.Common.Build...
Importing namespace Cake.Common.Build.AppVeyor...
Importing namespace Cake.Common.Build.AppVeyor.Data...
Importing namespace Cake.Common.Build.AzurePipelines...
Importing namespace Cake.Common.Build.AzurePipelines.Data...
Importing namespace Cake.Common.Build.Bamboo...
Importing namespace Cake.Common.Build.Bamboo.Data...
Importing namespace Cake.Common.Build.BitbucketPipelines...
Importing namespace Cake.Common.Build.BitbucketPipelines.Data...
Importing namespace Cake.Common.Build.Bitrise...
Importing namespace Cake.Common.Build.Bitrise.Data...
Importing namespace Cake.Common.Build.ContinuaCI...
Importing namespace Cake.Common.Build.ContinuaCI.Data...
Importing namespace Cake.Common.Build.GitHubActions...
Importing namespace Cake.Common.Build.GitHubActions.Data...
Importing namespace Cake.Common.Build.GitLabCI...
Importing namespace Cake.Common.Build.GitLabCI.Data...
Importing namespace Cake.Common.Build.GoCD...
Importing namespace Cake.Common.Build.GoCD.Data...
Importing namespace Cake.Common.Build.Jenkins...
Importing namespace Cake.Common.Build.Jenkins.Data...
Importing namespace Cake.Common.Build.MyGet...
Importing namespace Cake.Common.Build.TeamCity...
Importing namespace Cake.Common.Build.TFBuild...
Importing namespace Cake.Common.Build.TFBuild.Data...
Importing namespace Cake.Common.Build.TravisCI...
Importing namespace Cake.Common.Build.TravisCI.Data...
Importing namespace Cake.Common.Diagnostics...
Importing namespace Cake.Common.IO...
Importing namespace Cake.Common.IO.Paths...
Importing namespace Cake.Common.Net...
Importing namespace Cake.Common.Security...
Importing namespace Cake.Common.Solution...
Importing namespace Cake.Common.Solution.Project...
Importing namespace Cake.Common.Solution.Project.Properties...
Importing namespace Cake.Common.Solution.Project.XmlDoc...
Importing namespace Cake.Common.Text...
Importing namespace Cake.Common.Tools...
Importing namespace Cake.Common.Tools.Cake...
Importing namespace Cake.Common.Tools.Chocolatey...
Importing namespace Cake.Common.Tools.Chocolatey.ApiKey...
Importing namespace Cake.Common.Tools.Chocolatey.Config...
Importing namespace Cake.Common.Tools.Chocolatey.Download...
Importing namespace Cake.Common.Tools.Chocolatey.Features...
Importing namespace Cake.Common.Tools.Chocolatey.Install...
Importing namespace Cake.Common.Tools.Chocolatey.New...
Importing namespace Cake.Common.Tools.Chocolatey.Pack...
Importing namespace Cake.Common.Tools.Chocolatey.Pin...
Importing namespace Cake.Common.Tools.Chocolatey.Push...
Importing namespace Cake.Common.Tools.Chocolatey.Sources...
Importing namespace Cake.Common.Tools.Chocolatey.Uninstall...
Importing namespace Cake.Common.Tools.Chocolatey.Upgrade...
Importing namespace Cake.Common.Tools.DotCover...
Importing namespace Cake.Common.Tools.DotCover.Analyse...
Importing namespace Cake.Common.Tools.DotCover.Cover...
Importing namespace Cake.Common.Tools.DotCover.Merge...
Importing namespace Cake.Common.Tools.DotCover.Report...
Importing namespace Cake.Common.Tools.DotNetCore...
Importing namespace Cake.Common.Tools.DotNetCore.Build...
Importing namespace Cake.Common.Tools.DotNetCore.BuildServer...
Importing namespace Cake.Common.Tools.DotNetCore.Clean...
Importing namespace Cake.Common.Tools.DotNetCore.Execute...
Importing namespace Cake.Common.Tools.DotNetCore.MSBuild...
Importing namespace Cake.Common.Tools.DotNetCore.NuGet.Delete...
Importing namespace Cake.Common.Tools.DotNetCore.NuGet.Push...
Importing namespace Cake.Common.Tools.DotNetCore.NuGet.Source...
Importing namespace Cake.Common.Tools.DotNetCore.Pack...
Importing namespace Cake.Common.Tools.DotNetCore.Publish...
Importing namespace Cake.Common.Tools.DotNetCore.Restore...
Importing namespace Cake.Common.Tools.DotNetCore.Run...
Importing namespace Cake.Common.Tools.DotNetCore.Test...
Importing namespace Cake.Common.Tools.DotNetCore.Tool...
Importing namespace Cake.Common.Tools.DotNetCore.VSTest...
Importing namespace Cake.Common.Tools.DupFinder...
Importing namespace Cake.Common.Tools.Fixie...
Importing namespace Cake.Common.Tools.GitLink...
Importing namespace Cake.Common.Tools.GitReleaseManager...
Importing namespace Cake.Common.Tools.GitReleaseManager.AddAssets...
Importing namespace Cake.Common.Tools.GitReleaseManager.Close...
Importing namespace Cake.Common.Tools.GitReleaseManager.Create...
Importing namespace Cake.Common.Tools.GitReleaseManager.Discard...
Importing namespace Cake.Common.Tools.GitReleaseManager.Export...
Importing namespace Cake.Common.Tools.GitReleaseManager.Label...
Importing namespace Cake.Common.Tools.GitReleaseManager.Open...
Importing namespace Cake.Common.Tools.GitReleaseManager.Publish...
Importing namespace Cake.Common.Tools.GitReleaseNotes...
Importing namespace Cake.Common.Tools.GitVersion...
Importing namespace Cake.Common.Tools.ILMerge...
Importing namespace Cake.Common.Tools.ILRepack...
Importing namespace Cake.Common.Tools.InnoSetup...
Importing namespace Cake.Common.Tools.InspectCode...
Importing namespace Cake.Common.Tools.MSBuild...
Importing namespace Cake.Common.Tools.MSpec...
Importing namespace Cake.Common.Tools.MSTest...
Importing namespace Cake.Common.Tools.NSIS...
Importing namespace Cake.Common.Tools.NuGet...
Importing namespace Cake.Common.Tools.NuGet.Add...
Importing namespace Cake.Common.Tools.NuGet.Delete...
Importing namespace Cake.Common.Tools.NuGet.Init...
Importing namespace Cake.Common.Tools.NuGet.Install...
Importing namespace Cake.Common.Tools.NuGet.List...
Importing namespace Cake.Common.Tools.NuGet.Pack...
Importing namespace Cake.Common.Tools.NuGet.Push...
Importing namespace Cake.Common.Tools.NuGet.Restore...
Importing namespace Cake.Common.Tools.NuGet.SetApiKey...
Importing namespace Cake.Common.Tools.NuGet.SetProxy...
Importing namespace Cake.Common.Tools.NuGet.Sources...
Importing namespace Cake.Common.Tools.NuGet.Update...
Importing namespace Cake.Common.Tools.NUnit...
Importing namespace Cake.Common.Tools.OctopusDeploy...
Importing namespace Cake.Common.Tools.OpenCover...
Importing namespace Cake.Common.Tools.ReportGenerator...
Importing namespace Cake.Common.Tools.ReportUnit...
Importing namespace Cake.Common.Tools.Roundhouse...
Importing namespace Cake.Common.Tools.SignTool...
Importing namespace Cake.Common.Tools.SpecFlow...
Importing namespace Cake.Common.Tools.SpecFlow.StepDefinitionReport...
Importing namespace Cake.Common.Tools.SpecFlow.TestExecutionReport...
Importing namespace Cake.Common.Tools.TextTransform...
Importing namespace Cake.Common.Tools.VSTest...
Importing namespace Cake.Common.Tools.VSWhere...
Importing namespace Cake.Common.Tools.VSWhere.All...
Importing namespace Cake.Common.Tools.VSWhere.Latest...
Importing namespace Cake.Common.Tools.VSWhere.Legacy...
Importing namespace Cake.Common.Tools.VSWhere.Product...
Importing namespace Cake.Common.Tools.WiX...
Importing namespace Cake.Common.Tools.WiX.Heat...
Importing namespace Cake.Common.Tools.XBuild...
Importing namespace Cake.Common.Tools.XUnit...
Importing namespace Cake.Common.Xml...
Importing namespace Cake.Core...
Importing namespace Cake.Core.Diagnostics...
Importing namespace Cake.Core.IO...
Importing namespace Cake.Core.Scripting...
Importing namespace Cake.FileHelpers...
Importing namespace System...
Importing namespace System.Collections.Generic...
Importing namespace System.IO...
Importing namespace System.Linq...
Importing namespace System.Text...
Importing namespace System.Threading.Tasks...
Compiling build script...

========================================
Clean
========================================
Executing task: Clean
Cleaning directory C:/Users/thomas.sobieck/Documents/GitHub/school/artifacts
Finished executing task: Clean

========================================
Restore-NuGet-Packages
========================================
Executing task: Restore-NuGet-Packages
Could not resolve path for tool "dotnet" using these directories: C:/Windows/system32,C:/Windows,C:/Windows/System32/Wbem,C:/Windows/System32/WindowsPowerShell/v1.0,C:/Windows/System32/OpenSSH,C:/Program Files/dotnet,C:/Program Files/Microsoft SQL Server/130/Tools/Binn,C:/Program Files/Microsoft SQL Server/Client SDK/ODBC/170/Tools/Binn,C:/Program Files (x86)/Microsoft SQL Server/150/DTS/Binn,C:/Program Files/nodejs,C:/ProgramData/chocolatey/bin,C:/Program Files/Docker/Docker/resources/bin,C:/ProgramData/DockerDesktop/version-bin,C:/Users/thomas.sobieck/AppData/Local/Microsoft/WindowsApps,C:/Users/thomas.sobieck/AppData/Local/hyper/app-3.0.2/resources/bin,C:/Users/thomas.sobieck/AppData/Local/GitHubDesktop/bin,C:/Users/thomas.sobieck/AppData/Local/Programs/Microsoft VS Code/bin,C:/Users/thomas.sobieck/AppData/Local/Programs/Git/cmd,C:/Users/thomas.sobieck/.dotnet/tools,C:/Users/thomas.sobieck/AppData/Roaming/npm,C:/Users/thomas.sobieck/AppData/Local/Programs/Fiddler,C:/tools,C:/tools
Resolved tool to path C:/Program Files/dotnet/dotnet.exe
Executing: "C:/Program Files/dotnet/dotnet.exe" restore "C:/Users/thomas.sobieck/Documents/GitHub/school/school.sln"
  Determining projects to restore...
  Restored C:\Users\thomas.sobieck\Documents\GitHub\school\src\KnightSchool.Host\KnightSchool.Host.csproj (in 244 ms).
Finished executing task: Restore-NuGet-Packages

========================================
Build
========================================
Executing task: Build
Could not resolve path for tool "dotnet" using these directories: C:/Windows/system32,C:/Windows,C:/Windows/System32/Wbem,C:/Windows/System32/WindowsPowerShell/v1.0,C:/Windows/System32/OpenSSH,C:/Program Files/dotnet,C:/Program Files/Microsoft SQL Server/130/Tools/Binn,C:/Program Files/Microsoft SQL Server/Client SDK/ODBC/170/Tools/Binn,C:/Program Files (x86)/Microsoft SQL Server/150/DTS/Binn,C:/Program Files/nodejs,C:/ProgramData/chocolatey/bin,C:/Program Files/Docker/Docker/resources/bin,C:/ProgramData/DockerDesktop/version-bin,C:/Users/thomas.sobieck/AppData/Local/Microsoft/WindowsApps,C:/Users/thomas.sobieck/AppData/Local/hyper/app-3.0.2/resources/bin,C:/Users/thomas.sobieck/AppData/Local/GitHubDesktop/bin,C:/Users/thomas.sobieck/AppData/Local/Programs/Microsoft VS Code/bin,C:/Users/thomas.sobieck/AppData/Local/Programs/Git/cmd,C:/Users/thomas.sobieck/.dotnet/tools,C:/Users/thomas.sobieck/AppData/Roaming/npm,C:/Users/thomas.sobieck/AppData/Local/Programs/Fiddler,C:/tools,C:/tools
Resolved tool to path C:/Program Files/dotnet/dotnet.exe
Executing: "C:/Program Files/dotnet/dotnet.exe" build "C:/Users/thomas.sobieck/Documents/GitHub/school/school.sln" --configuration Release /property:Version=0.0.1 /fileLogger /fileloggerparameters:LogFile="C:/Users/thomas.sobieck/Documents/GitHub/school/msbuild.log" /warnaserror
Microsoft (R) Build Engine version 16.6.0+5ff7b0c9e for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  Restored C:\Users\thomas.sobieck\Documents\GitHub\school\src\KnightSchool.Host\KnightSchool.Host.csproj (in 244 ms).
  KnightSchool.Host -> C:\Users\thomas.sobieck\Documents\GitHub\school\src\KnightSchool.Host\bin\Release\netcoreapp3.1\KnightSchool.Host.dll
  KnightSchool.Host -> C:\Users\thomas.sobieck\Documents\GitHub\school\src\KnightSchool.Host\bin\Release\netcoreapp3.1\KnightSchool.Host.Views.dll

Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:00:03.02
Finished executing task: Build

========================================
Run-Tests
========================================
Executing task: Run-Tests
Finished executing task: Run-Tests

========================================
Package
========================================
Executing task: Package
Could not resolve path for tool "dotnet" using these directories: C:/Windows/system32,C:/Windows,C:/Windows/System32/Wbem,C:/Windows/System32/WindowsPowerShell/v1.0,C:/Windows/System32/OpenSSH,C:/Program Files/dotnet,C:/Program Files/Microsoft SQL Server/130/Tools/Binn,C:/Program Files/Microsoft SQL Server/Client SDK/ODBC/170/Tools/Binn,C:/Program Files (x86)/Microsoft SQL Server/150/DTS/Binn,C:/Program Files/nodejs,C:/ProgramData/chocolatey/bin,C:/Program Files/Docker/Docker/resources/bin,C:/ProgramData/DockerDesktop/version-bin,C:/Users/thomas.sobieck/AppData/Local/Microsoft/WindowsApps,C:/Users/thomas.sobieck/AppData/Local/hyper/app-3.0.2/resources/bin,C:/Users/thomas.sobieck/AppData/Local/GitHubDesktop/bin,C:/Users/thomas.sobieck/AppData/Local/Programs/Microsoft VS Code/bin,C:/Users/thomas.sobieck/AppData/Local/Programs/Git/cmd,C:/Users/thomas.sobieck/.dotnet/tools,C:/Users/thomas.sobieck/AppData/Roaming/npm,C:/Users/thomas.sobieck/AppData/Local/Programs/Fiddler,C:/tools,C:/tools
Resolved tool to path C:/Program Files/dotnet/dotnet.exe
Executing: "C:/Program Files/dotnet/dotnet.exe" publish "C:/Users/thomas.sobieck/Documents/GitHub/school/src/KnightSchool.Host/KnightSchool.Host.csproj" --output "C:/Users/thomas.sobieck/Documents/GitHub/school/artifacts/Host" --configuration Release /property:Version=0.0.1-dev
Microsoft (R) Build Engine version 16.6.0+5ff7b0c9e for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  Restored C:\Users\thomas.sobieck\Documents\GitHub\school\src\KnightSchool.Host\KnightSchool.Host.csproj (in 279 ms).

Error: System.AggregateException: One or more errors occurred. ---> System.ArgumentOutOfRangeException: The DateTimeOffset specified cannot be converted into a Zip file timestamp.
Parameter name: value
   at System.IO.Compression.ZipArchiveEntry.set_LastWriteTime(DateTimeOffset value)
   at Cake.Common.IO.Zipper.Zip(DirectoryPath rootPath, FilePath outputPath, IEnumerable`1 paths)
   at Submission#0.Zip(DirectoryPath rootPath, FilePath outputPath) in :line 2423
   at Submission#0.<<Initialize>>b__0_4() in C:\Users\thomas.sobieck\Documents\GitHub\school\build.cake:line 110
   at Cake.Core.CakeTaskBuilderExtensions.<>c__DisplayClass32_0.<Does>b__0(ICakeContext x)
   at Cake.Core.CakeTask.<Execute>d__43.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.DefaultExecutionStrategy.<ExecuteAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.CakeEngine.<RunTask>d__28.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Cake.Core.CakeEngine.<RunTargetAsync>d__27.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Scripting.BuildScriptHost.<RunTargetAsync>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.Scripting.ScriptHost.RunTarget(String target)
   at Submission#0.<<Initialize>>d__0.MoveNext() in C:\Users\thomas.sobieck\Documents\GitHub\school\build.cake:line 135
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.Scripting.ScriptExecutionState.<RunSubmissionsAsync>d__9`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.Scripting.Script`1.<RunSubmissionsAsync>d__21.MoveNext()
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at Cake.Scripting.Roslyn.RoslynScriptSession.Execute(Script script)
   at Cake.Core.Scripting.ScriptRunner.Run(IScriptHost host, FilePath scriptPath, IDictionary`2 arguments)
   at Cake.Commands.BuildCommand.Execute(CakeOptions options)
   at Cake.CakeApplication.Run(CakeOptions options)
   at Cake.Program.Main()
---> (Inner Exception #0) System.ArgumentOutOfRangeException: The DateTimeOffset specified cannot be converted into a Zip file timestamp.
Parameter name: value
   at System.IO.Compression.ZipArchiveEntry.set_LastWriteTime(DateTimeOffset value)
   at Cake.Common.IO.Zipper.Zip(DirectoryPath rootPath, FilePath outputPath, IEnumerable`1 paths)
   at Submission#0.Zip(DirectoryPath rootPath, FilePath outputPath) in :line 2423
   at Submission#0.<<Initialize>>b__0_4() in C:\Users\thomas.sobieck\Documents\GitHub\school\build.cake:line 110
   at Cake.Core.CakeTaskBuilderExtensions.<>c__DisplayClass32_0.<Does>b__0(ICakeContext x)
   at Cake.Core.CakeTask.<Execute>d__43.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.DefaultExecutionStrategy.<ExecuteAsync>d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Cake.Core.CakeEngine.<ExecuteTaskAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.CakeEngine.<RunTask>d__28.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Cake.Core.CakeEngine.<RunTargetAsync>d__27.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Scripting.BuildScriptHost.<RunTargetAsync>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Cake.Core.Scripting.ScriptHost.RunTarget(String target)
   at Submission#0.<<Initialize>>d__0.MoveNext() in C:\Users\thomas.sobieck\Documents\GitHub\school\build.cake:line 135
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.Scripting.ScriptExecutionState.<RunSubmissionsAsync>d__9`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.Scripting.Script`1.<RunSubmissionsAsync>d__21.MoveNext()<---

I believe these are the files causing the problems.

I vaguely remember hitting this a while ago. Update the SDK? See https://github.com/NuGet/Home/issues/7001#issuecomment-640350856 and https://github.com/dotnet/extensions/issues/2750.

@augustoproiete It has nothing to do with the actual build script. Somehow the above files had a "invalid" "LastWriteTime" timestamp. Not even Windows was capable of zipping those files.
At first I tried to "modify" those timestamps like this:
Get-ChildItem  C:\foo.dll | % {$_.LastWriteTime = '03/11/2021 18:30:00'}

Which "worked", but seemed "wrong". After the SDK update everything was fine. Maybe a bug inside the SDK.

I'm starting to think it's some edge case like a TimeZone difference or something, we do have Min/Max dates.
https://github.com/cake-build/cake/blob/a1893a2bba26f49e089d66beeb78421cedf6126d/src/Cake.Common/IO/Zipper.cs#L24-L26
A workaround could be if we could repro we could tweak Min to a day later, sorting if these happened to be just on the border of valid from/to.
Reading zip spec

The format does not support dates prior to 1980-01-01 0:00 UTC. Avoid file dates 1980-01-01 or earlier (local or UTC time).

So maybe just change to the 2nd of January and that addresses potential UTC +/- issues, and validate day too
https://github.com/cake-build/cake/blob/a1893a2bba26f49e089d66beeb78421cedf6126d/src/Cake.Common/IO/Zipper.cs#L238
The issue with the SDK was that they shipped some packages with the SDK where some assemblies had below ZIP possible min value date, which indeed seems as an SDK bug as it should be when assemblies compiled.


[Bug] Memory may leak when EventInvalidException occurs

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
This issue may cause the memory of the shuffle server to never be released.

[2024-03-08 17:51:25.737] [clearResourceThread] [INFO] HybridStorageManager.removeResources - Start to remove resource of AppPurgeEvent{appId='application_1703049085550_5925694_1709883294114', user='tdwadmin', shuffleIds=[0, 1, 2]}
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorageManager.cleanupStorageSelectionCache - Cleaning the storage selection cache costs: 4(ms) for event: AppPurgeEvent{appId='application_1703049085550_5925694_1709883294114', user='tdwadmin', shuffleIds=[0, 1, 2]}
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorage.removeResources - Start to remove resource of application_1703049085550_5925694_1709883294114/0
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorage.removeResources - Finish remove resource of application_1703049085550_5925694_1709883294114/0, disk size is<PHONE_NUMBER>90 and 2 shuffle metadata
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorage.removeResources - Start to remove resource of application_1703049085550_5925694_1709883294114/1
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorage.removeResources - Finish remove resource of application_1703049085550_5925694_1709883294114/1, disk size is<PHONE_NUMBER>90 and 2 shuffle metadata
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorage.removeResources - Start to remove resource of application_1703049085550_5925694_1709883294114/2
[2024-03-08 17:51:25.743] [clearResourceThread] [INFO] LocalStorage.removeResources - Finish remove resource of application_1703049085550_5925694_1709883294114/2, disk size is 134246092 and 1 shuffle metadata
[2024-03-08 17:51:25.764] [LocalFileFlushEventThreadPool-25] [WARN] ShuffleFlushManager.processFlushEvent - AppId application_1703049085550_5925694_1709883294114 was removed already, event:ShuffleDataFlushEvent: eventId=38029, appId=application_1703049085550_5925694_1709883294114, shuffleId=2, startPartition=13682, endPartition=13682, retryTimes=0, underStorage=LocalStorage, isPended=false, ownedByHugePartition=false should be dropped
[2024-03-08 17:51:25.770] [LocalFileFlushEventThreadPool-25] [WARN] ShuffleFlushManager.processFlushEvent - AppId application_1703049085550_5925694_1709883294114 was removed already, event:ShuffleDataFlushEvent: eventId=38030, appId=application_1703049085550_5925694_1709883294114, shuffleId=2, startPartition=13683, endPartition=13683, retryTimes=0, underStorage=LocalStorage, isPended=false, ownedByHugePartition=false should be dropped

This is due to the multithreading concurrency issue between the scheduled cleanup task and the flush event.
Affects Version(s)
master
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

We already have a fix, still working on it.

cc @zuston


data:ianujvarshney.json

Pull Request: Adding Anuj Varshney's Bio
Description:
This pull request introduces the bio data for Anuj Varshney, providing a comprehensive overview of his skills, current learning focus, and notable milestones. The added information includes details on social media profiles, links, and a milestone related to the "Biodrop" open-source project.
Changes Made:

Added Anuj Varshney's bio data to the profile directory.
Included information on skills, social media profiles, and a significant milestone.
Verified that the data adheres to the specified structure and formatting.

Context:
The inclusion of Anuj Varshney's bio enhances the profile section by offering a detailed snapshot of his professional journey, skills, and contributions to open source.
Testing:

Manually verified the added data for accuracy.
Checked the rendering of the bio information in the profile section.

Dependencies:

No dependencies for this pull request.

Checklist:

[ ] The added data aligns with the project's data structure and standards.
[ ] No breaking changes introduced.
[ ] The bio information renders correctly in the profile display.

You already have a profile that you manage via the forms https://www.biodrop.io/ianujvarshney so I will close this PR as it will have no effect on your profile

@eddiejaoude
Working with Event Page on Localhost
I am eager to work with the Event page, but I encounter authentication hurdles every time I try to access it. Upon reaching the final authentication step, the sign-in process fails to work, and I'm perplexed about what might be causing this issue.
Upon closer inspection, I discovered that my username.json file is missing from the data folder. Realizing this, I added the missing file. that's why added.
Could you kindly guide me on how to successfully work with the Event page on localhost? I appreciate your assistance in resolving this matter.

You can create your profile using the forms again locally, so the data will be in your local database

You can create your profile using the forms again locally, so the data will be in your local database

Let me try


Consumer registration and rebalancing

Are there plans for sarama to directly support automated consumer registration and rebalancing, as described on http://kafka.apache.org/documentation.html#distributionimpl.
Use case: assuming I have N topics with M partitions, and I would to run a consumer group and distribute these across O consumers as evenly as possible (with each message being consumed only once).
The process steps are nicely documented, but is purely based on ZK communication. Should this be implemented directly into sarama or would it be better to create a wrapper library?
I volunteer to help with the development, but I would like to know if there as any existing work or plans I can (should) rely on?

I think it's better to keep this separate from the main sarama code and not pollute it with zk dependencies. I have created a new project for it: https://github.com/bsm/sarama-cluster

@dim now that zookeeper isn't a dependency to implement consumer groups, what are the chances you can move this into the official sarama repo?

@arianitu maybe, but there is quite a bit of effort involved and the integration is not 100% straight-forwards as the test cases/setups are more complicated. more importantly: I probably won't be able to find the time for such an effort any time soon, but help is more than welcome


Add-In installation issue in ArcMap 10.8

@pcgosling great job on the tool and paper!
I am trying to install the tool for my ArcMap 10.8. I successfully installed the Add-In and it shows in Add-In Manager. However, it seems that the toolbar is missing GUI components. Do you perhaps know what I am missing? Thanks!

Thanks and sorry it isn’t working. I have just upgraded my ArcMap to 10.8 too but haven’t had time to test the tool yet. It was originally created in version 10.4 but was working fine in 10.6.1 when I posted to GitHub. I have also seen this missing GUI effect before and it is normally something simple that has gone wrong. I will take a look as soon as I can.

I have successfully tested the tool using ArcMap 10.8 so thankfully no issues!
On checking the online ESRI Help I now understand that I should probably also be sharing the .esriaddin file that I create. For some reason I thought that people I share with would have to run the makeaddin.py themselves. Therefore, I have uploaded the .esriaddin file so please give that a go and let me know if that works.
Many thanks.

Yes, that resolve the issue. Probably the python script did not generated the right .esriaddin file or something. Thanks!


Add Skip button on mnemonic confirmation page for testnet

@Peersyst could you attach the PR please.

Fixed in gh1431-harvesting-fix

Fixed in integration branch.


Support for Foundation Apps (v6)

Hi all,
Are there any plans to support Foundation Apps as well?
http://foundation.zurb.com/apps.html
Thanks!

:+1:

Perhaps we could add //= require foundation/apps into the JS manifest, and similarly namespace SCSS accordingly.  Thoughts?

:+1:

AFAIK, there are currently no plans to officially support a gem version of Foundation Apps. You may be able to gemify the Foundation for Apps assets via RailsAssets and include them inside your Rails apps that way.


Build: Setup initial hardhat compilations

build: add hardhat as devDep
build: initial hardhat config w/ solidity settings
refactor: add artifacts & cache to gitignore
build: add npm hardhat compile script
refactor: port hardhat config from js to ts
refactor: rm unused calculateContractBytecode script
build: add calculate-contract-bytecode npm script

Why removing calculateContractBytecode script ? I sometime use it to check contract bytecode size, I think it is useful.

This is good, I think it will be great if u do a walkthrough the setup or hardhat overall in a learning session before merging this.

Ah, my bad @haythem96 . It looked like an unused script. Will re-add.

Use this instead of the script:
https://hardhat.org/plugins/hardhat-contract-sizer.html

@antoncoding the output of the script that plugin runs is different than the calculateContractBytecode script.
It outputs a table of each contract's bytecode size in KB but doesn't show how much space is left.
What do you think @haythem96?
Also, IIRC, by default ganache-cli maintains the EIP-170 limit. A contract breaching the limit should not deploy locally.

@antoncoding the output of the script that plugin runs is different than the calculateContractBytecode script.
hardhat-contract-sizer outputs a table of each contract's bytecode size in KB but doesn't show how much space is left.
What do you think @haythem96?
Also, IIRC, by default ganache-cli maintains the EIP-170 limit. A contract breaching the limit should not deploy locally.

Yes that's true, I think it is fine to keep the script, it is already working and doesn't need any maintenance even if we don't mainly use it, so I don't see a cons of keeping it.

@haythem96 the issue I see with keeping is that it's very specific to truffle's framework. However, if we decide to remove truffle as a devDependency, it shouldn't be difficult to modify the hardhat plugin script to also calculate and provide similar output.

@CruzMolina can u change the hardhat.config.ts file to typescript format?
Based on the example here: https://hardhat.org/guides/typescript.html
export default {
  solidity: "0.7.3",
};

@haythem96 the issue I see with keeping is that it's very specific to truffle's framework. However, if we decide to remove truffle as a devDependency, it shouldn't be difficult to modify the hardhat plugin script to also calculate and provide similar output.

I think we can keep it for now, and open another PR to modify all truffle script to hardhat

Do not merge


Accept '0' as a level which doesn't suffix the shortname.

Some specifications don't actually have levels. "Fetch", for instance. And "html5".
This patch accepts "level": 0 as a hint that the shortname shouldn't be suffixed. An alternative approach would be to make the level attribute optional, but since you'd explicitly made it required, I'm not sure you'd accept that patch. :)
WDYT?

I dropped this on the floor long enough that I have no idea what the complaint was. If it crops up again, I'll file a new PR. :)

I'm running the command line tool with the --force option to allow building without a level. I'm not sure why levels are required at level 1.

Because it's still leveled (level 1) and if you ever publish a second level, you'll want to reserve the unleveled name for referring to the "most recent" version.  We've found this to be good practice in the CSSWG, and Bikeshed generally tries to encode good practices.

Well, can't you still do that when you start working on level 2?

If you're willing to go back and repub the old one, sure.  That's effort that most people won't do, and they shouldn't have to remember to do so.

That makes sense.


Try to detect multiple entries with the same ID

Current Behavior
Sometimes it's possible to accidentally assign the same ID multiple times, like if you're working
on an entry in a branch and work on another entry in a different branch, and the IDs get generated on different machines with different existing indexes.
Possible Solution
One possible approach: when scanning an entry, if the entry record already exists and has a different pathname, see if the other pathname is still valid. If so, throw an error so it ends up in the log. Or something.
Another possibility: stop using autoincrement indexes, you silly
Context
This seems unlikely to happen on real sites but I've run into it a few times while developing the example site content.

How about: scan entry, see if record exists and if so and the existing record’s path is valid and different, treat this entry as if it has no ID. Meaning it’ll get a new ID, possibly after being deferred into the fix up queue. (Obvs generate a warning in this case.)
Testing this will be obnoxious, and one possible result is permalinks getting reassigned, which is why having a better sequence generator in the first place is a good idea.

Possible improved sequence generator:
maxid = pow(10, .5 + log(entries.count()))
candidate = rand(maxid)
for entry in entries.select(id >= candidate):
    if entry.id > candidate:
        break
    candidate += 1

Idea being that it generates a random number with an upper bound based on the number of digits in the entry count (maybe it should use the max entry ID instead of the count?) and then increments it while it collides with an existing entry. Will have to do a quick test to see what the collision rate is with this approach.

(Also I like the idea of randomizing the sequence, as it makes it harder for people to guess about future entries. Great for comics etc)

Fixed in #42


Implement kill query

Fixes #130

@ajnavarro Added lock to connection map

Rebased with the latest changes.


Remove vision support

Checklist

[x] I have updated the relevant documentation
[x] I have added the semver-major, semver-minor or semver-patch label as appropriate
[x] I would like multiple reviewers to approve my code before merging

Why do you want to make these changes?
Following discussions in #715, I believe the best way forward is to move vision code into a j5-zoloto module.
I intend to directly copy all of the removed code to a separate repo, and will attempt to preserve history if possible.
De-coupling zoloto and j5 will allow us to resolve issues with duplicated data structures and suspected issues converting between them.
By removing untested code, this PR increases test coverage to 97%. We will be able to test the zoloto integration
Which parts of the codebase do your changes affect?
Remove all vision code. Remove dependency on pyquaternion and cached-property
How do your changes affect student or volunteer experience?
Vision is a complex problem and is often very specific to a particular competition. I would be interested in re-integrating parts of common vision code in the future, but only once we know it works and without adding a dependency on Zoloto.
This change will allow us to focus on the current codebase and move towards 1.0. Vision is best left to the experts for now.
Are your changes related to any existing issues or PRs? How so?
Closes #402 and closes #346

That said, it looks like there are some pretty useful looking tests for some of the constructs. Are these definitely also covered upstream? It'd be great to not lose that work!

We definitely don't want to lose any of this code, we're just moving it elsewhere :) https://github.com/j5api/j5-zoloto now exists so most of it will go there. I'm trying to work out how we can preserve the author history of the code removed in the new repo this morning


Fixed inability to clone due to unable to copy session.lock

This PR should fix #2337. Since when cloning a world, the 2 files uid.dat and session.lock is not needed, I added a check to ignore these files when copying the world folder. Not sure how good is my implementation but it works xD.

Ah IntelliJ added it automatically,  I will change it.


Make a 3 gorgeous front end desgin

Making some of the most astonishing and mind boggling front end designs known to man

Got one web template done

Got two templates done. I will probably pause here and start writing code


Update: Converting Landing Page Dropdown Menu to Cards

https://app.clickup.com/t/1aqjp0p
Changed SDK Demo site landing page layout so that instead of a dropdown menu, partner presets are chosen using "cards" with the three options 'Generic' corresponding to default, 'Ad Portal' corresponding to Spectrum, and custom (although this flow may be removed in the future). Updated variable names accordingly with the updated option names. A random video thumbnail picture was used as a placeholder for the current cards to be replaced with icons representing each flow.

Two other things

I'm getting an error trying to create an account when the cards are selected
Randomly noticed some styling updates we need to make on the account page


add location data to model

allows for increased visualisation capabilities (maps, etc)
makes future globalisation work possible
increases utility of model as people can predict for a specific area

The current proposal includes:

Collecting and integrating new FIRMS (MODIS) data for Australia, between 2000 and 2023
Ceasing the usage of localised BOM temperature data

Explanation: For now, it is believed to be impractical to enter the temperature conditions for the potentially millions of pixels, particularly when inferencing the model to generate visualisations. It may be possible for scientific applications, however, this detracts from Firetrace's goal as an easily accessible awareness tool, aimed at use by the general public.
Instead, temperatures will be averaged from multiple weather stations across the country. Users will then be asked to provide this average, and it is hoped that the model will learn to recognise each region's individual responses to a change in average temperatures.

Addition of a 12 month and 24 month Southern Oscillation Index (ENSO) moving average

Explanation: The SOI may provide additional useful information regarding humidity and temperature over the long term, which could be used to estimate fuel loads and how readily a fuel source is burned on a particular day.

Obsolete with new implementation in https://github.com/jtpotato/firetrace/releases/tag/v2.0.0


Support to NixOS

Is this issue related to an issue?.
SI does not work with NixOS.
Describe the outcome you'd like from us
Provide support to the NixOS operating system.
Describe alternatives you've considered
Are there some workaround to run SI in NixOS environment?
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Additional context
No response

Thank for you for the feature-request! We will take this under advisement.

Tested with latest main, NixOS works following our development environment documentation.


Bounding Box Functions do not respect Orientation

Was going to use flib bounding box function to do some transformations for a mod I am working on when I noticed that they do not preserve orientation.
With flib_bounding_box.ceil(box) for example:
local area = {
      left_top = { x = 0.5, y = 0.5 },
      right_bottom = { x = 0.5, y = 0.5},
      orientation = 0.1,
    }
game.print(flib_bounding_box.ceil(area))

will result in
{      left_top = { x = 0, y = 0 },      right_bottom = { x = 1, y = 1},    }
Proposed solution for Ceil.
function flib_bounding_box.ceil(box)
  if box.left_top then
    return {
      left_top = { x = math.floor(box.left_top.x), y = math.floor(box.left_top.y) },
      right_bottom = { x = math.ceil(box.right_bottom.x), y = math.ceil(box.right_bottom.y) },
      orientation = box.orientation,
    }
  else
    return {
      { math.floor(box[1][1]), math.floor(box[1][2]) },
      { math.ceil(box[2][1]), math.ceil(box[2][2]) },
    }
  end
end

I haven't looked at a solution for the other functions yet as I wanted to see if this is something that would be accepted as a PR or if not preserving the orientation was an intention design choice because it seems like a lot of functions don't properly use orientation and can be confusing as to what ceil would actually mean when applied to a rotated object's coordinate system.

I completely forgot about orientation. A PR would be welcome, but if not then I'll get to it eventually.

I can get a PR done.
I've already implemented some of the BB functions from the mod I'm working on so can do the rest once soon. Will just get my mod stuff done first.
After working with orientation I found it's not often used or considered (even in some base game functions). Given no one else has noticed / interacted with this PR I don't imagine it's a common issue.


New version: GNU.Octave version 9.1.0

Created by 🥟 Dumplings.


feat: Adding SESSION_COOKIE_PARTITIONED

This pull request is related to Add support for partitioned session cookies
fixes #5472
Regarding the commit Adding code to handle SESSION_COOKIE_SECURE I see that in the test_basic.py in line 20 is using import flask, I believe this is the reason I got this error as it's using the package installed via pip install flask and not the code contained in /src/flask/
E       AssertionError: assert 'partitioned' in 'session=eyj0zxn0aw5nijo0mn0.zmzhyw.hgj68cffl4hz1avfswp08uya5ya; domain=example.com; secure; path=/; samesite=lax'

I'd like to learn if I'm doing something wrong with this to correct it and modify the PR

[x] Test case modified for the behavior of the change
[x] Updated relevant docs, based on the appearances of SESSION_COOKIE_SECURE
[x] Added entry to CHANGES.rst

I believe this is the reason I got this error as it's using the package installed via pip install flask and not the code contained in /src/flask/

I'm not sure what you mean here. You should follow our contributing guide to set up your development environment. It instructs you to do pip install -e . during the setup. You can also run tox to set up test envs for you. The failure does not appear to be related.

Hello, thank you for your response, I went ahead and applied changes without taking a look at the contribution guide, I just went through it and will make the process correctly, my apologies.

So the immediate reason this wouldn't work is because it needs to depend on Werkzeug 3.1, which is not yet released. But after that, you didn't actually use the get_cookie_partitioned method to pass the parameter to set_cookie. I've corrected that, and will merge this once Werkzeug 3.1 is available.

is this still open?


Running list of unexpected outputs

Hey all,
spaCy is awesome, but occasionally it gives unexpected answers. Sometimes these unexpected outputs are handled better by NLTK, for instance. I'm sure it goes the other way too sometimes.
I thought it might be useful to start an issue with a running list of unexpected outputs. This may help folks notice patterns to start debugging models.
I'll start.
import spacy
nlp = spacy.English()
doc = nlp(u"Thankyou Emeril- I couldnt find a crust without added flour, yours is Just cornmeal...... I didnt have any shortening so i substituted solid bacon fat - it worked great for my quiche!! your recipe was so easy - my 1st homemade crust! Anika in California")
for sent in doc.sents:
    print(sent)

Gives
Thankyou Emeril- I couldnt find a crust without added flour, yours is Just cornmeal...... I didnt have any shortening
so i substituted solid bacon fat - it worked great for my quiche!!
your recipe was so easy - my 1st homemade crust!
Anika in California

It should not break on the so, in my opinion.

Another:
import spacy
nlp = spacy.English()
doc = nlp(u"The quick brown fox jumps.The slow brown fox crawls")
for tok in doc:
    print(str(tok) + tok.tag_)

gives
The DT
quick JJ
brown JJ
fox NN
jumpsNNS
..
The DT
slow JJ
brown NN
fox NN
crawlsVBZ

jumps isn't a noun, and neither is brown.

Heya,
I also noticed the sentence splitting on "so" and reported it. Take a look at #231 to see what they said about it (quite a long thread!).

Thanks @Terrilocks. Quite a good discussion!


dino: add tests

Description of changes
Added tests built by dino.

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

The last commit does not fit the requirements of CONTRIBUTING.md. It is unclear from the commit message what exactly is touched.

Sh*t, that's totally my bad. I am sorry. I just checked the diff before merging, it never occurred to me that such diff would not be in a single commit.


activate or mention in the doc when learning async discarded_futures and unawaited_futures lint rules

Use case
I did not understand correctly how to you async and Futur the right way because these two linter rules are not activated by default
  - discarded_futures
  - unawaited_futures

Proposal
I think they should be activated by default or at least mentioned in https://dart.dev/codelabs/async-await
that would have avoided a lot of pain in my learning...

I remember when I was a beginner Flutter dev, and was confused by it as well. Then I learned about these 2 rules and it "clicked" :)

@stephane-archer
Are you proposing to activate these lints in flutter_lints package ?

I think they should be activated by default or at least mentioned in https://dart.dev/codelabs/async-await

This will probably go in dart-lang's website repo from documentation point of view.

@darshankawar exactly :)

https://github.com/dart-lang/site-www/issues/5740

Retitling to reflect what is being requested.


Quality drop when transcoding h265 to h264 - Is there a way to increate bitrate?

I see quite a noticable quality drop when transcoding h265 to h264. From what I understand its bound to happen with the transcode but can be reduced by increasing the bitrate of the h264. How can this be done with go2rtc?
  streams:
    frontyard_fullres:
      - rtsp://user:password@<IP_ADDRESS>:554/h265Preview_01_main
      - ffmpeg:rtsp://user:password@<IP_ADDRESS>:554/h265Preview_01_main#video=h264#hardware#audio=opus
  log:
    level: trace
  api:
    listen: ":1984"  # HTTP API port ("" - disabled)
  rtsp:
    listen: ":8554"
  webrtc:
    listen: ":8555"
    candidates:
      - <IP_ADDRESS>:8555 # <--- enter frigate host IP here
      - stun:8555

Example: left stream is the rtsp from http://localhost:8554/frontyard_fullres and right stream is from http://localhost:8554/frontyard_fullres?video=h264

Which GPU is used?

I got an h265 camera to work very well in MSE mode with almost no delay by doing this:
  cam_internal:
    - dvrip://usr:pass@<IP_ADDRESS>:34567
  cam:
    - ffmpeg:cam_internal#audio=opus#async
    - ffmpeg:cam_internal#video=copy#async

For whatever reason, they HAVE to be 2 separate streams, and cam needs to grab them like this and using #async.
I think the reason for which the original stream doesn't "just work" is because MSE doesn't like the original audio codec, and webrtc doesn't like h265. Furthermore, just adding an opus audio stream to an existing stream makes the video stop every 2 seconds. This is the result of a lot of trial and error so I quite know why it works and if it would work for you too

@skrashevich Running in the frigate docker container (on Unraid) with NVIDIA 3060.
@dbuezas I use iOS so MSE is no good for me unfortunetley. Thats why I use the below, which if my understanding is correct, provides a webrtc compatible h265 & h264 stream. Any other combination results in webrtc fine on PC browser but broken on iOS HomeAssistant APP, or the reverse.
frontyard_fullres:

rtsp://user:password@<IP_ADDRESS>:554/h265Preview_01_main
ffmpeg:rtsp://user:password@<IP_ADDRESS>:554/h265Preview_01_main#video=h264#hardware#audio=opus

You can do anything with exec source. ffmpeg source generate some template exec command. You can check it it logs and change for your needs.


CommDev branch only compiles on Raspi due to WiringSerial headers

Must mock out the WiringSerial headers for non-pi dev

What do you want these to actually do? All functions return 0 at the moment.

Fixed in 3a6c2ce655b2e0efdf26de08e743734b95d9592d


Modify CSS

Hey! Thanks for this great theme!
How to properly modify CSS? I found CSS and SCSS and created a custom_style.css to make my changes. But what the best way to do modifications? For instance "btn-primary", how to change it's colors? I changed the CSS and SCSS, is this the right way?
Thanks!

well, I won't change the original file. maybe this will help you https://stackoverflow.com/a/3679478/3410328

Hi Boby
I found the solution. It's here: https://getbootstrap.com/docs/4.0/getting-started/theming/
Thanks

I am delighted to hear that! (Not really) Now you've found a solution, you can close this issue.


MTV-921 release notes 2 5 5

Jira:

MTV-921

Preview:

@ahadas - please can you review for us?
Thanks


error. showing error: login(description: "Authentication Failed")

when I try to log in it is showing
error: login(description: "Authentication Failed")
currently trying to login with iCloud

This just started happening to me too. It was working fine in January, and little if anything changed, and now this error on this line:
let postal = Postal(configuration: .gmail(login: login, password: .accessToken(accessToken)))
Tried previous working commits. Tried different Google app registration. Tried different Apple cert. Nothing worked.

In my case, the root cause was something changing on my google cert side, and the gmail api scope wasn't part of that google cert anymore. Once the gmail api scope was added back, it worked. Adding the scope via google console web UI never made a difference. Only worked through their api client me:
func ensureGoogleScope(user: GIDGoogleUser?) {
let gmailScope = "https://mail.google.com/"
let grantedScopes = user?.grantedScopes
if grantedScopes == nil || !grantedScopes!.contains(gmailScope) {
// Request additional Drive scope.
let additionalScopes = [gmailScope]
GIDSignIn.sharedInstance.addScopes(additionalScopes, presenting:  getRootViewController()) { user, error in
guard error == nil else { return }
guard let user = user else { return }
        // Check if the user granted access to the scopes you requested.
        print("granted scopes: \(String(describing: user.grantedScopes))")
      }

    }
  }

Suggestion for Postal:
Show the complete error text from google. That would have indicated the exact cause of "Authentication Failed", such as not having the correct scope, or whatever the issue is.


Creating an abomination.

I will report this repo until it gets banned.

no

Delete this when? (Now)

You mean this issue? Yeah sure it should've never existed.


Ubuntu uses the moniker Rubbish Bin not Trash

VSCode Version: 1.51.1
OS Version: 20.04

Steps to Reproduce:

Delete a file from within VSCode while on Ubuntu
Observe that the word 'Trash' is used in the confirmation dialog box.
Realise you're on Ubuntu and it uses the moniker 'Rubbish Bin'.
Fume a bit and open an issue on Github.

Does this issue occur when all extensions are disabled?: Yes
Ideally VSCode should not use its own names for OS elements. They should be called whatever the OS called them. In this case, on Ubuntu, VSCode uses Trash where the OS uses Rubbish Bin. On macOS, which uses Trash, it should be called Trash.
If someone points me to the right files etc I am happy to attempt a fix for this. I suspect the easiest but probably not the best is to maintain a hash of OSes and assiciated trash names.

@jaapie yeah makes sense. However it would be cool if all Linux disitrbuitons use the same moniker, then this would be easier.
Anyways, here's a code pointer https://github.com/microsoft/vscode/blob/c6f4565a20360272365db74b7c08a31826548acc/src/vs/workbench/contrib/files/browser/fileActions.ts#L123
You can do a full text search for "trash" inside the src/vs/workbench/contrib/files and see where it makes sense and where it does not.

Thanks for the pointer. I may have some time over the Christmas period to try this out.


Fix tests

@rmosolgo I was able to fix all of the tests except for one, I also believe this should fix the actions. Opened a PR on my own fork, but the actions would not run, so I'm not sure how to test them.
The one test that has an error is this:
TestQueryResult#test_relay_connection_enumerator:
GraphQL::Schema::DuplicateNamesError: Found two visible definitions for `Query.users.first`: #<GraphQL::Schema::Argument Query.users.first: Int @description="Returns the first _n_ elements from the list.">, #<GraphQL::Schema::Argument Query.users.first: Int>

Looks like it is caused when parsing this GraphQL, any ideas?
    Temp.const_set :Query, @client.parse(<<-'GRAPHQL')
      {
        users(first: 2) {
          edges {
            cursor
            node {
              login
            }
          }
        }
      }
    GRAPHQL

Update, I was able to get the CI tests to run on my local for, ruby 2.7, 3.0 and 3.1 just have the one test failure. 3.2 has this odd failure, one which I cannot reproduce locally when running on 3.2:
Bundle complete! 11 Gemfile dependencies, 45 gems now installed.
Use `bundle info [gemname]` to see where a bundled gem is installed.
rake aborted!
NoMethodError: undefined method `=~' for #<Proc:0x00007f411504ce48 /opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/gems/3.2.0/gems/rake-11.3.0/lib/rake/application.rb:390 (lambda)>
/opt/hostedtoolcache/Ruby/3.2.2/x64/bin/bundle:25:in `load'
/opt/hostedtoolcache/Ruby/3.2.2/x64/bin/bundle:25:in `<main>'
(See full trace by running task with --trace)

Hey, thanks so much for your work on this! It's awesome to have the build passing again. I'm going to wait and see about getting access to github/graphql-client before I merge this PR here.

I merged this into the new canonical repository over at https://github.com/github-community-projects/graphql-client/pull/2


incorporate variant-level summary statistics

incorporate variant-level summary statistics, including effect size and direction, standardized to permit cross-study comparisons

@mkarmona and I decided to create a separate node where to load sumstats data.
@edm1 has already cleaned the data as specified in https://github.com/opentargets/sumstat_data
schema suggested lives at: https://docs.google.com/document/d/1YKgLLEz_P-WmdKcGXt4dqDNd18x5e9j96GwSP2Hc2jE/edit


Simple entry of server{} to location{} path. :)

I might just be confused as I'm good with Nginx but baffled a bit by the GUI.  That being said, have no idea how to simply do server variable to internal path. :(

Is your feature request related to a problem? Please describe.

In NGINX configuration file, it's easy to say server and location. Example:
server {
  listen 80;
  server_name example.com;
location /images/ {
    try_files $uri /images/default.gif;

}
Describe the solution you'd like

Tab with entry for server name (hostheader) and directory.
Server [funwebsite.example.com]    Internal Path [ /var/www/funwebsite/ ]
Describe alternatives you've considered

Additional context

I'm a little confused what you are trying to do.
Seems to me you want to use nginx-proxy-manager as a GUI for setting up Nginx as a simple webserver.
Since nginx-proxy-manager is a GUI for managing the proxy part of Nginx, the only way I can see you being able to do what you request, is setup an Nginx container to hold the images and either proxy fully to that container, or use the custom locations GUI to forward /images to the Nginx container hosting the images.
Am I not understanding you correctly?


Extract method to drive PageIterator -> RecordReader

Which issue does this PR close?
Relates to #171
Rationale for this change
The logic for driving a RecordReader from a PageIterator is currently duplicated in PrimitiveArrayReader and NullArrayReader. This duplication will only increase with new ArrayReader implementations added as part of #171
What changes are included in this PR?
Extracts a read_records function to handle this logic
Are there any user-facing changes?
This no longer eagerly populates the RecordReader with the first PageReader from the PageIterator. If for example the first page of the first row group contained a compression codec that was not supported, this would currently error in the constructor, whereas with this change it will error in ArrayReader::next_batch. I personally think this makes more sense. FWIW I think this is the only possible error case and so it is unlikely user-code would be impacted...

Codecov Report

Merging #1031 (75920d0) into master (239cba1) will decrease coverage by 0.00%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master    #1031      +/-   ##
==========================================
- Coverage   82.31%   82.30%   -0.01%
==========================================
  Files         168      168
  Lines       49031    49022       -9
==========================================
- Hits        40360    40350      -10
- Misses       8671     8672       +1

Impacted Files
Coverage Δ

parquet/src/arrow/array_reader.rs
76.66% <93.75%> (-0.10%)
:arrow_down:

arrow/src/datatypes/datatype.rs
65.95% <0.00%> (-0.43%)
:arrow_down:

arrow/src/datatypes/field.rs
53.37% <0.00%> (-0.31%)
:arrow_down:

parquet_derive/src/parquet_field.rs
65.98% <0.00%> (-0.23%)
:arrow_down:

arrow/src/array/transform/mod.rs
85.24% <0.00%> (+0.13%)
: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 239cba1...75920d0. Read the comment docs.

Merged, thanks!


How to map userId to connectionId?

Hi all,
How to map userId to conenctionId in SignalR core?
In previous version there was PrincipalUserIdProvider with it i was able to map userId to connectionId.
What is the way to do it in SignalR Core?

Do you want to change how the user ID is assigned? Our docs has a section on Users in SignalR that explains how you can do this. ASP.NET Core SignalR has an IUserIdProvider as well and you can provide your own implementation of that interface to customize how the user ID is assigned.

Do you want to change how the user ID is assigned? Our docs have a section on Users in SignalR that explains how you can do this. ASP.NET Core SignalR has an IUserIdProvider as well and you can provide your own implementation of that interface to customize how the user ID is assigned.

Thank you. But how
public class NameUserIdProvider : IUserIdProvider
{
public string GetUserId(HubConnectionContext connection)
{
return connection.User?.FindFirst(ClaimTypes.Name)?.Value;
}
}

Do you want to change how the user ID is assigned? Our docs have a section on Users in SignalR that explains how you can do this. ASP.NET Core SignalR has an IUserIdProvider as well and you can provide your own implementation of that interface to customize how the user ID is assigned.

Thank you!


ListView reorder produces two SelectedItems

Describe the bug
Two items are selected after ListView reorder
Steps to reproduce the bug
See github demo project

Select Item 1
Reorder item 2 above item 1
repeat a couple of times and both items 1 & 2 will be selected - SelectionMode="Single"

Expected behavior

Only the original selected item should be selected, reordering should not produce two selected items

Screenshots

NuGet package version
WinUI 3 - Windows App SDK 1.3.2: 1.3.230602002
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

I believe this may have been the wrong place to post this issue, I have also posted the issue to the WindowsAppSDK repo here


Sword app decals should be cleaned up

In the silsword app:

Related: https://github.com/webaverse/app/issues/3488

Fix: https://github.com/webaverse/silsword/pull/11


[BasicLayoutsCodelab] only render AlignYourBodyRow when executing on emulator or physical device

Cloned git repo, and "end" branch checkedout. But the project does not work, it only renders AlignYourBodyRow function. Previews are working fine.

Emultaor: default pixel 3a with API 33

I am having the same issue, any solution for this?

Fixed, ensure you are building and deploying "app" instead of "AlignYourBodyRow"

Fixed, ensure you are building and deploying "app" instead of "AlignYourBodyRow"

that was my mistake. Thanks!


Update dependency aar to release 10.2.8-1

Update to the next patch release of 10.2.8-1 when released.

Verified done