Resolve validateQueryFiles with the list of queries validated

I had a look at getting more information when using the validateQueryFiles function, i.e. see which queries the glob matched.
A quick solution is to do callback(null, queries) : resolve(queries) when validation is completed successfully.
Can I open a PR?

Sure.  That would be great.  Look forward to getting additional contributors


Add cargo-auditable docs.

I'd apparently set this aside with git stash and forgot to commit it with #1528. :sweat:

The "supply chain security" page links to the issue for adding cargo-auditable. Should it link to the relevant bit of the book instead?

@Shnatsel good catch! Updated that link and removed the :soon:. :slightly_smiling_face:


tcpflow: fix build

Motivation for this change
It won't build following https://github.com/NixOS/nixpkgs/commit/c3255fe8ec326d2c8fe9462d49ed83aa64d3e68f.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg tcpflow

@grahamc I just built it locally :)


Support full qualified names in XAddr

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Currently an IPv4 is needed in discovery messages in order to form a functional connection.
Expected Behavior
We would like to allow a full qualified name in order to allow working e.g. from a docker container or other virtualized environments
Minimal Reproducible Example
try to pass in a FQDN into the stack
Solution proposal
make sure there are no constraints that require a numerical ipv4 address
Python Version
3.x
Operating system
all
Sdc11073 Version
all
Link to sdc11073 Logs
No response
Further Information
No response
Participation

[x] I am willing to submit a pull request to fix this bug.

I am not sure whether we want to support this.

You could assign a static ip to a docker container.
One cannot ensure that the hostname will be resolvable for everyone that we send the hostname to.
This introduces overhead if the hostname has to be resolved first before sending a message.

it's not so much about if it's a good or bad idea to use hostnames instead of ip addresses in the xaddr (i think there are arguments for both). the underlying standard makes no constraints about what can be used in the xaddr, therefore we should support in the stack to use both in order to be flexible in testing
regarding the resolve: this is all up to the operating system, we dont need to do anythign on the application layer

I am not sure what standard you mean by "the underlying standard". but just because we dont have to do anything does not mean that there will be no overhead and performance decrease.
If the standard makes no constraint that does not mean we should actually use it if its not a good idea (this is what we try to do before accepting a feature request).
Also, I dont really get the use case. Why cant you use the ipaddress?


[1.1.x] set package long description (#2407) | fix dashboard graph metric list icon paths with URL_PREFIX (#2424) | docs: for sql db migration to 1.1 recommend --fake-initial (#2425) | add tag formatting docs (#2426) | Update tags.rst (#242

Backports the following commits to 1.1.x:

set package long description (#2407)
fix dashboard graph metric list icon paths with URL_PREFIX (#2424)
docs: for sql db migration to 1.1 recommend --fake-initial (#2425)
add tag formatting docs (#2426)
Update tags.rst (#2426)
add tag character validator (#2426)
Correct this split (#2431)
Add document.body.dataset.baseUrl to these calls (#2433)
Accept IPv6 addresses in CARBONLINK_HOSTS (#2436)
lint fix (#2436)
fixed developed (#2443)
Update render_api.rst (91ed1c0b)
Copy requestContext() and empty prefetch so we don't override prior fetched data (#2450)
Change all evaluateTarget() calls to include prefetch-less requestContext (#2450)
update aggregation function docs for aggregate and groupbytags (#2451)
Add Statusengine to list of integrations (Forwarding) (#2452)
Resolve failing test (#2452)
Django22 compatibility (#2462)
Update to Xenial on Travis for a newer SQLite version. (#2462)
Fix a broken link to structured_metrics in doc (#2463)
Python 3.8 support (#2464)
Fix "SyntaxWarning: invalid escape sequence" (#2464)
New functions: add, sigmoid, logit, exp (#2466)
add functions to validate params based on what functions declare they expect (#2467)
add tests for the parameter validation functions (#2467)
add customer input parameter error and handle it in the render view (#2467)
raise input parameter error if evaluator detects an issue (#2467)
raise InputParameterError from functions when appropriate, instead of ValueError (#2467)
enforce param checks, handle integer values for float params, handle missing default params, better error messages (#2467)
make input parameter validation optional (#2467)
consider an unknown function name an input error (#2467)
Pass maxDataPoints to the requestContext for Finder (#2479)
Add redis password support for tagdb (#2483)
added space before \  (#2487)
Created issue template (#2488)
docs: add netdata to 'tools that work with graphite' (#2490)
remove duplicate 'and' (#2490)
Updated minimumBelow() docstring (#2493)
xFilesFactor is an optional parameter for removeEmptySeries (#2495)
fix functions that aggregate to include the aliases in their params (#2496)
make py3 happy also (#2496)
the callback parameter for groupByNode is optional (#2497)
fix order (#2496)
Removing  function (#2499)
Removing test also (#2499)
Removing blank line (#2499)
Remove SMTP_SERVER and unused imports (#2499)
Upgrading minimal Django version (#2502)
Cleanup of tox.ini (#2502)

Codecov Report

Merging #2503 into 1.1.x will decrease coverage by 0.13%.
The diff coverage is 81.13%.

@@            Coverage Diff             @@
##            1.1.x    #2503      +/-   ##
==========================================
- Coverage   79.79%   79.66%   -0.14%
==========================================
  Files          85       85
  Lines        8910     9091     +181
  Branches     1906     1948      +42
==========================================
+ Hits         7110     7242     +132
- Misses       1537     1573      +36
- Partials      263      276      +13

Impacted Files
Coverage Δ

webapp/graphite/tags/redis.py
91.82% <ø> (ø)
:arrow_up:

webapp/graphite/composer/views.py
23.8% <ø> (-31.75%)
:arrow_down:

webapp/graphite/app_settings.py
70.96% <ø> (-6.46%)
:arrow_down:

webapp/graphite/composer/urls.py
100% <ø> (ø)
:arrow_up:

webapp/graphite/settings.py
75.74% <100%> (-1.87%)
:arrow_down:

webapp/graphite/carbonlink.py
51.92% <100%> (-1.42%)
:arrow_down:

webapp/graphite/errors.py
100% <100%> (ø)
:arrow_up:

webapp/graphite/render/evaluator.py
90.34% <50%> (-9.66%)
:arrow_down:

webapp/graphite/tags/utils.py
81.08% <55.55%> (-7.39%)
:arrow_down:

webapp/graphite/util.py
90% <61.11%> (-3.4%)
:arrow_down:

... and 24 more

Continue to review full report at Codecov.

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


Initializing Sentry in an AWS Lambda function handler fails with TypeError

Environment
SaaS (https://sentry.io/)
Steps to Reproduce

Create a Lambda function that initializes Sentry like this:

import sentry_sdk
import sentry_sdk.integrations.aws_lambda as aws_lambda

sentry_sdk.init(
        dsn="...",
        environment="test",
        integrations=[aws_lambda.AwsLambdaIntegration()],
        release="...",
        debug=True
)

requirements.txt:
boto3>=1.35.56,<2
pydantic>=2.9.2,<3
sentry_sdk>=2.19.0,<3

Lambda runtime: Python 3.12
Expected Result
Sentry should be initialized and start capturing errors. In fact, my code works AS IS when run outside AWS Lambda (e.g., on my laptop).
Actual Result
The init method raises a TypeError. Here is the according log statement from CloudWatch:

Product Area
APIs
Link
No response
DSN
No response
Version
No response

Hi @pat-youda, unfortunately the AwsLambdaIntegration is only supported on Python 3.9. We already have an issue open to add support for Python 3.12, but since we are busy with other priorities, it will take us some time to get around to implementing this.
I noticed that we are missing documentation that states that Python 3.9 is the only version we support for the AwsLambdaIntegration, so I created https://github.com/getsentry/sentry-docs/issues/11952 to make sure we add this information to the docs page.
Also, I am opening #3825 to add a warning when the integration is initialized with an unsupported Python version.
I am going to close this issue in favor of the other issues linked in this comment. Please reach out if you have any further questions.


mysql example only works with mysql 5.7

In the mysql example (examples/mysql/docker-compose.yml) the mysql image referenced is 'mysql' which defaults to latest (v8.0.20 at this moment).
However this configuration doesn't work with mysql 8, it only works with mysql 5 (I think this has something to do with the mysql npm module not supporting a new form of authentication?).
Please adjust to image: mysql:5.7

Hi, @holygekko Thank you for the info. If you feel like opening a PR to help out that would be cool :D
Will try to update when I get some time ;)

Hello, I have update the example to avoid misleading users. Thanks for your help !


Download a list (or text file) of all available courses on codecourse.com

Option to download a list (or text file) of all available courses on codecourse.com
Use-case:
Getting this list periodically will allow me to see which courses have been added or changed.

Hey @connecteev, you are pretty active :), anyways thank you for sharing your feedback and ideas as well in order to improve the downloader
There is already a file with all courses called search-courses.json here https://github.com/muhamed-didovic/ccdown/blob/main/json/search-courses.json
This list of all courses will be shown if you choose the option not to download all courses, basically, you can filter out courses that you want

Thanks @muhamed-didovic! I have seen this file and tried that option - works great for filtering / downloading a specific course. I have also tried to edit the file and feed it to ccdown (basically what I tried here: #3)
How is this static list / json file kept updated as new courses get downloaded? It would be nice to allow the user to get an exhaustive list of all courses. See step 1 here:
https://github.com/muhamed-didovic/ccdown/issues/3#issuecomment-1602608106

file is updated either by me or another way is if user selects to download one course and then he gets this question: 'Do you want to search for a courses from a local file (which is faster)' if he chooses no, I hit the API and get all courses and update the file

Hey there @muhamed-didovic just checking in - is there a way to do this?


when deploying a revision with a wasm module that does not exist (or bad path) the reverse-proxy shutdown

2022-08-23T04:27:10.472 app[048da343] cdg [info] 🖐[RegisterURL] {"code":"OK","function":"hey","message":"URL added to the revision","revision":"green","url":"http://localhost:10009"}

2022-08-23T04:27:10.472 app[048da343] cdg [info] 🟡[scale] Add a new wasm module to an existing revision of an existing function: green

2022-08-23T04:27:10.555 app[048da343] cdg [info] 📥 file to download https://capsule-faas.fly.dev:7043/demo/hey/0.0.0/hey.wasm

2022-08-23T04:27:10.555 app[048da343] cdg [info] 🙂 file downloaded ./tmp/07b2d625-5cdd-4e03-916c-9a3272372c1d.wasm

2022-08-23T04:27:10.556 app[048da343] cdg [info] 💊 Capsule ( v0.1.6 🦞 ) http server is listening on: 10009 🌍

2022-08-23T04:27:15.174 app[048da343] cdg [info] 2022/08/23 04:27:15 http: panic serving <IP_ADDRESS>:33596: runtime error: index out of range [5] with length 3

2022-08-23T04:27:15.174 app[048da343] cdg [info] goroutine 2556 [running]:

2022-08-23T04:27:15.174 app[048da343] cdg [info] net/http.(*conn).serve.func1()

2022-08-23T04:27:15.174 app[048da343] cdg [info] /usr/local/go/src/net/http/server.go:1825 +0xbf

2022-08-23T04:27:15.174 app[048da343] cdg [info] panic({0x892800, 0xc000276600})

2022-08-23T04:27:15.174 app[048da343] cdg [info] /usr/local/go/src/runtime/panic.go:844 +0x258

2022-08-23T04:27:15.174 app[048da343] cdg [info] github.com/bots-garden/capsule/capsule-reverse-proxy/reverse-proxy.redirect({0xc000414c80?, 0xc00039c9c0?, 0x403f69?}, 0xc000145300)

2022-08-23T04:27:15.174 app[048da343] cdg [info] /github/workspace/capsule-reverse-proxy/reverse-proxy/reverse-proxy.go:36 +0x170

2022-08-23T04:27:15.174 app[048da343] cdg [info] github.com/bots-garden/capsule/capsule-reverse-proxy/reverse-proxy.proxyRevision(0xc000145300)

2022-08-23T04:27:15.174 app[048da343] cdg [info] /github/workspace/capsule-reverse-proxy/reverse-proxy/reverse-proxy.go:82 +0x126

TODO:

[ ]  don't start the capsule launcher instance if the wasm file does not exist
[ ]  don't update the capsule reverse-proxy routes map
[ ]  check if there is an impact on the worker (don't update the capsule worker routes map)


How can we set the Sort By as default setting

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @auniyal3 , thank you for getting in touch with us!
Could you please elaborate a little more on your request? What the screen you are experiencing the issue on?


add an option to ignore import statements

It would be very useful to have an option to ignore changes to import statements. In particular for the Java language.

I'm afraid difftastic can't do this. It has a simple, generic representation of syntax trees and can't do language-specific or syntax-specific tweaks.


Question Config.py

Hi,
Just a quick question about config.py. I assume the curiosity module should reside in config.py, but I am not able to find it. Did you upload the correct file for this? Where can I find that intrinsic reward?

There are two config files, world_models_sonic/config.py with some hard coded data paths. And world_models_sonic/helpers/deep_rl/config.py which has the object which is configured in the notebook 04_train_PPO_v4-all_curious.ipynb to use curiosity.
Where is curiosity actually coded? If curiosity is set to true, it's used in the PPO agent, so see where just cntrl-f for if config.curiosity.


Breaking changes when replacing data.series with context.panel.data.series

I recently migrated from Grafana 9.5.3 to Grafana 10.4.0 in 3rd week of March
The volkovlabs-echarts-panel renered well with a few minor changes that hadd null checks.
But recently when I re deployed my grafana containers, and tried to push changes to dashboard through provisioning, the plugin blindly prefixed context.panel to anything that had data.series.
Sensing that I need to add this change for all dashboards, I edited the dashboard JSON externally to add context.panel.
On re-deploying the panel I hoped to see it without breaking errors.
However I think it is adding context.panel without checking if its already there in the code.
The screenshot below can explain this

Is there a way to use volkovlabs 5.0.0 with grafana version 10.4.0

@padmakr That's expected behaviour. In Business Charts v6 we introduced breaking change to use context for all parameters.
Migration happens automatically and depends on the pluginVersion property, which set to v6 when it's migrated automatically.
When you edited the dashboard JSON manually, please add pluginVersion": "6.0.0" and it will work as expected.
Depends on your installation, you can specify to use the specific version of the plugin.

@mikhail-vl Thank you for the response.
The following things are still little unclear to me.

What if I chose to add 'context.panel' prefix manually in the dashboard jsons?
What if I still want to use volkov-echarts-panel 5.0.0? Is that possible by changing "pluginVersion": "5.0.0" in dashboard jsons?

Depends on your installation, you can specify to use the specific version of the plugin.
If I am building a custom grafana image can I specify this way
GF_INSTALL_PLUGINS: volkovlabs-echarts-panel 5.0.0

@padmakr Let me explain:

Adding context.panel manually should be done together with updating pluginVersion for each panel.
Dashboard will displayed using installed plugin in Grafana. Only configuration will be migrated if version differs.
Specific plugin version should be installed by specifying archive directly: https://grafana.com/docs/grafana/latest/administration/plugin-management/


move tf to ros

This PR is an alternative to PR 664 which also resolves many of the TF issues in 681.  It changes the way we publish TF information on the ROS side by:

adding a JointStatePublisher plugin to the wamv and bridging the topic to ROS to make dynamic joint states available on the ROS side
modifying the PosePublisher plugin to only publish sensor transforms, as those are only available via the sdf and will not be published when parsing the urdf
replacing most of the functionality in the custom pose_tf_broadcaster node with a robot_state_publisher node, which provides the robot description and tf tree in a format that can be ingested by ROS applications such as RViz

A launch file to start an instance of RViz with a custom config file is included for testing and for the tutorial.
UNKNOWN, help needed: Is any other process on the Gazebo side using the transforms from the sdf that PosePublisher was producing previously?  I think not, as we have access to the truth data, but if so this PR will likely break something and needs to be reworked.
To test:
ros2 launch vrx_gz competition.launch.py world:=stationkeeping_task
ros2 launch vrx_gazebo rviz.launch.py
Confirm the wamv appears with all components drawn and in the right place.
ros2 topic pub /wamv/thrusters/left/pos std_msgs/msg/Float64 "{data: -1}"
Watch the left engine/propeller move in RViz as it moves in Gazebo.
ros2 run tf2_tools view_frames
Confirm the new TF tree is structured correctly.
I still haven't looked into the extra "wamv/" prepended on the PosePublisher's output, so this code adds the prefix to all ROS-generated link names instead, in order to make the sensor sdf links connect correctly.  This is minor but annoying, so if we decide to implement this I will take another look at it.

@M1chaelM @caguero
This is ready for a look, but we'll need to discuss.  I do think it fixes the tf issues but I'm not sure if/when we want to make a significant change.  Marked both this and the other option (tutorial hack only) as draft pending decisions.

I don't think the extra cameras and lidar are necessary - it's super easy to add them, and one topic is shown as an example - but there's no reason not to have them all if you think that would look better for the demo.  Whichever you prefer works for me!


Use std::vector in smartPointers and stl exercises ?

It is my strong feeling that the use of raw C arrays makes the syntax uselessly complex, and divert the learners from what we want to demonstrate. It would rather use std::vector, provided it has been previously presented in the morning lessons.

We have std::vector in the STL chapter, which comes before smart pointers, so we can use it for the exercises.

Do we also have std::array ?

We have a slide that lists all STL containers, including std::array. I told the students that std::array is similar to a C array but with a std::vector like interface. So in principle, they should not be too surprised seeing it.
However, if we now put STL algorithms and std::vector/std::array all in one or two exercises, that might be a bit too much.

I rather think that using the type double[] in the exercises, with the unusual way to use make_unique/shared, and the use of array addresse together with "address+size", instead of begin() and end(), is doing a great disservice to beginners. They should use std::array from the beginning, never more C arrays.

With vector, there is no need to put it on the heap. I would generally say that people should use vector, but you see that it defies the point of the exercise.
I'm very happy though if you want to replace the array with a std::array on the heap! Thoughts?

... the same for the array in LargeObject, problem2.

I generally use valarray in such a case, as a replacement for the stillborn gsl::dyn_array... but I understand that some developers dislike std::valarray.


RFC 06: Supply a locale to ManagerInterface.createState

Proposal
Require a locale to be passed to Session.CreateContext.
Rationale
Not 100% on this one, but it might be useful. It would allow a manager to associate the state with a particular session before its first use. For example, creating multiple contexts as part of a multi-shot workflow.
A better solution may be to keep this as-is, and introduce a more formal "configure" or "authenticate" set of verbs to use post-fact.

Closing as the session object is no more, and this is something we can add later once a workflow demands it.


Task 20, task 21, Task 22 and Task 23

@Bhimendhar
Task -20
Code verified.
Task -21
Code verified.
Task -22
Code verified.
Task -23
Code verified.
Task -24
Code verified.
Task -25
Code verified.
Task -26
Code verified.
Task -27
Code verified.


Old version on /multi

I'm trying to retrieve the newest version of dom-console
If i'm doing that in any way but /multi it works.
Also https://wzrd.in/status/dom-console shows version 4.0.1, but trying to fetch it via a POST request on the /multi route somehow gives me the old 3.2.0 version.
I already cleaned my local cache and hard reset reloaded the page - but no changes.
Not sure if https://wzrd.in/admin/cull/dom-console would be a quick fix ( i dont have admin credentials ) but maybe there is something in the code that shouldnt work like it currently works.

Not sure if the culling module can handle multibundles, which are cached iirc at a separate layer. Sorry, I should have mentioned that over twitter.

The hard-coded TTL is apparently 2 weeks https://github.com/jfhbrook/wzrd.in/blob/master/bundler/cache.js#L104
You might also be able to get around this by specifying 4.0.1 exactly when making your post, rather than latest (if that's not what's going on, I'm at a bit of a loss).

looks like it's technically supported https://github.com/jfhbrook/wzrd.in/blob/master/admin.js#L34 but I think you need to know the bundle key https://github.com/jfhbrook/wzrd.in/blob/master/bundler/cache.js#L126-L132 for that to work...

I'm not sure how to fix things or what exactly i need to do.
How would i figure out what the latest module is?
Basically I made a little script for codepen.io to use require directly.
It fetches the latest modules from wzrd.in and caches them in the browser.
This seems to work, but once i update the module on npm and wzrd.in did cache the module already, it wont pick up the new version from npm unless i'm specifying the exact version, so i added require('dom-console', '5.1.0') but i would rather not specify the version and have it always pick up the 'latest' version automatically :-)
...of course unless it's already cached locally - if it was, the user would need to clear his cache first, but that's fine for now i think...


Useful combinators

Useful combinators for:

Ordering pairs
Distinct pairs
Ordered lists

Plus tests.
@raulraja @juanpedromoreno would you be able to take a look please?

Codecov Report

Merging #51 into master will decrease coverage by 13.99%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master      #51    +/-   ##
========================================
- Coverage   97.16%   83.17%   -14%
========================================
  Files          10       10
  Lines         106      107     +1
  Branches        0        4     +4
========================================
- Hits          103       89    -14
- Misses          3       18    +15

Impacted Files
Coverage Δ

...ysevendeg/scalacheck/combinators/Combinators.scala
100% <100%> (ø)
:arrow_up:

...ysevendeg/scalacheck/datetime/instances/jdk8.scala
0% <0%> (-100%)
:arrow_down:

...scalacheck/datetime/jdk8/granularity/package.scala
73.33% <0%> (-26.67%)
:arrow_down:

...scalacheck/datetime/joda/granularity/package.scala
73.33% <0%> (-26.67%)
:arrow_down:

...rtysevendeg/scalacheck/datetime/joda/GenJoda.scala
76% <0%> (-18.45%)
:arrow_down:

...ysevendeg/scalacheck/datetime/instances/joda.scala
100% <0%> (ø)
:arrow_up:

...rtysevendeg/scalacheck/datetime/jdk8/GenJdk8.scala
100% <0%> (ø)
:arrow_up:

...ortysevendeg/scalacheck/datetime/Granularity.scala
100% <0%> (ø)
:arrow_up:

...ala/com/fortysevendeg/scalacheck/magic/Magic.scala
100% <0%> (ø)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


[AVM Module Issue]: custom_block_response_status_code should be optional but validation requires a value

Check for previous/existing GitHub issues

[x] I have checked for previous/existing GitHub issues

Issue Type?
Bug
(Optional) Module Version
0.1.3
(Optional) Correlation Id
No response
Description
In the Front Door Firewall Policies variable the custom block response status code is an optional number. When not supplied in calling the module the validation produces an error that makes this field required.

Error:

Thanks we will review and Correct it.
CC: @didayal-msft


fix release build for jgroups-greeter

fix release build for jgroups-greeter

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/239/
Test PASSed.


Move versioned generator configs to API group folders

Description of your changes

Fixes #
Consumes https://github.com/aws-controllers-k8s/code-generator/pull/266 that implements support for specifying API versions of the generated CRDs.
Move generator-config.yaml files to unversioned API group parent folders, because an API Group may now contain multiple versions. Please also note that https://github.com/aws-controllers-k8s/code-generator/pull/266 introduces a backwards-incompatible behavior change: The default location of the generator configuration files, if one is not specified via the command-line with the -- option, is now apis/<API Group>/generator-config.yaml.
When https://github.com/aws-controllers-k8s/code-generator/pull/266 is merged, I will update the consumed code-generator version accordingly.
I have:

[x] Read and followed Crossplane's [contribution process].
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

I have tried running provider-aws with a modified configuration for the Cloudfront's Distribution resource that moves it to the version v1beta1.
[contribution process]: https://git.io/fj2m9

i wonder if we first can merge https://github.com/crossplane/provider-aws/pull/1027 and then update this PR

@ulucinar This probably needs a rebase to show the correct diff as https://github.com/crossplane/provider-aws/pull/1027 is merged, which should cover most of the generated file diffs already.

I get the following when I run make reviewable and it seems check-diff also fails because of that:
16:41:57 [ .. ] Generating apigatewayv2 controllers and CRDs
# sigs.k8s.io/json/internal/golang/encoding/json
../pkg/pkg/mod/sigs.k8s.io/json@v0.0.0-20211020170558-c049b76a60c6/internal/golang/encoding/json/encode.go:1249:12: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
../pkg/pkg/mod/sigs.k8s.io/json@v0.0.0-20211020170558-c049b76a60c6/internal/golang/encoding/json/encode.go:1255:18: sf.IsExported undefined (type reflect.StructField has no field or method IsExported)
make[3]: *** [Makefile:129: services] Error 1
make[2]: *** [Makefile:143: services.all] Error 2
make[1]: *** [build/makelib/common.mk:418: generate] Error 2
make: *** [build/makelib/common.mk:424: reviewable] Error 2

Do you get it in your local?


0 is not 0 for Quaternion

because range is  [-0.707107,+0.707107] 0 might not fall on step, instead write sign as 1 bit, then rest of value in [0,0.707107] range

fixed with https://github.com/James-Frowen/BitPacking/commit/b802c7f25e37c90d6296daa811d9776abb5a88eb


ui for filter n search added

Issue that this pull request solves
Closes: #320
Proposed changes
ui for filter n search added
Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots

#302
important Keys

The menu closes when user clicks away from the menu
The dropdown are React components so they are reusable

As told:

[x] Filter by Year(This should show the list of years)[You can keep it as select box]
[x] Filter by Month(This should show the list of months)[You can keep it as select box]
[x] Filter by name(This should have a search box)
[x] Keep this options just below the heading.

All done 😁😁😁

@Kajol-Kumari  I am a DWoC and SWoC Participant

[x] The search box and filter buttons should appear below the heading i.e Broadcast section.
[x] Remove the background color
[x] same linear gradient for search box input filed as the one used for other form input fields in contact us forms
[x] Better color for filter button and a nice hover effect
[x] dropdown improved to look nicer.

@Kajol-Kumari , All done😁

mam, actually I looked for designs and colors on dribbble. if you dont find the current UI good-looking then it would be great if you could give me some example link or pics from dribbble or something.
you see, I am not so good with making UI design myself on my own😅. But, I can make exact material design clones if you show me some examples😁😁.

mam, actually I looked for designs and colors on dribbble. if you dont find the current UI good-looking then it would be great if you could give me some example link or pics from dribbble or something.
you see, I am not so good with making UI design myself on my own. But, I can copy designs if you show me some design examples.

Hey for button, you can use the already present design for buttons from here and for dropdown also, you can use the already present design which you can find here (you will have to make few modifications tho, but it will worth it :)) Also, please reduce the top margin of the new section that you added. It would look consistent and nicer this way :)

I used button.css. Now, the button looks nice. but, the dropdown looks ugly

So i am just pushing the new styles for button only.
@Kajol-Kumari , mam, please Tell me what do you think about this.
I Hope time it looks okay😅

I Hope you like the UI changes this time😅. I have used the button.css file's styles. Now the button looks beautiful. I did not use dropdown.css coz the dropdown looked ugly. so, I made some few changes. fixed a UI bug caused by a recent merge.

that dropdown with dropdown.css(i didnt use dropdown.css):

Hello @Kajol-Kumari , can you add labels for DWoC and SWoC both.


Cleanup observed reward accounts

Context
To fix #2005 a immediate solution is to observe the rewards of a Set ChimericAccounts on every tip change.
The not yet merged #2034 is only adding to the Set, but never removing.
We suspect this is acceptable for the moment because

The Set is cleared when the wallet restarted
Creating and deleting say 100s of wallets in a single session seem like a extremely rare user behaviour
Querying even 100 extra reward accounts are likely completely fine performance-wise (to be confirmed)
Getting out a fix for #2005 is more important.

But nonetheless we should make sure to deal with this properly.
Decision

Stop fetching the rewards of wallets when they are deleted.
Acceptance Criteria

[ ] The reward accounts we fetch on tip-changes must not grow uncontrolled.

Development

QA

Tracked as: https://jira.iohk.io/browse/ADP-410


test: Test  cos

Adds "-e PYTHONUNBUFFERED=1" to snakemake container options to flush snakemake stdout/stderr so they show up in google logs

Oops I am trying to pull this into main. I can change that...

@cademirch this is the correct branch to PR to - either you can keep your branch (but use commits from the other so they are properly attributed) or rebase in this context. What I discourage is to take someone else's code and commit/present it as your own.

And if you intended to rebase against main, I don't think you did that because changes from the testing-cos branch were present!

@vsoch Agh, sorry about that. Definitely did not intend and don't want to take credit for other's commits!  For my sake, how should have I done this? My quick google led me to do this:
git checkout -b test--cos upstream/testing-cos
git rebase main
... add my changes

My guess is that your main branch has commits not present here, meaning at some point you did a PR against snakemake from your main branch that was squashed (and now you have extra commits).
What you want to do to fix this is to clone snakemake fresh, force push the main branch to your main, and then moving forward never do a PR from your main to snakemake main. Always work from feature branches.

Also if you are new to rebasing, I made a very dumb video a few years ago, haha. https://youtu.be/9F4RE2_yn6I
But rebasing only works if your main is in sync with the upstream's main!

Awesome. Thank you for explaining this and bearing with my lack of git skills. Weirdly my main branch is not ahead of this one - so I'm not even sure what I did, but some how messed up anyways 😅.  If it's okay, I'll close this and make a new draft. Quick question - do I even need to rebase against main? Or will you able to rebase when you merge testing-cos into main?

Quick question - do I even need to rebase against main? Or will you able to rebase when you merge testing-cos into main?

If you checkout your new feature branch directly from main you should not need to! Rebasing is only when you have commits you want to fixup or squash relative to some other branch.

I guess I'm confused since if I checkout from main I wouldn't have the testing-cos code since its not in main?
Sorry for all the questions. I've created a draft #47, which just adds my two changes on top of the testing-cos branch, which I think is appropriate.

I guess I'm confused since if I checkout from main I wouldn't have the testing-cos code since its not in main?

Every new feature branch is generally checked out from main, but you are right if you are doing a PR to testing-cos it's ok to checkout from there too.


add fori_loop op to all 3 backends

fori_loop is extremely useful for keeping compile times down in jax by defining single layers as loops with a repeated body (instead of unrolling & compiling separately for each block in a sequence).
adding the primitive allows doing this in a low-level way, though eventually something more similar to flax's scan might be a nicer API for users to consume

added shape inference test and a docstring, tests passed

the symbolic call for fori_loop could probably use some work, in particular right now it looks like the graph for the loop body is going to get completely ignored? I'm curious what you think the right way to handle it would be.


Add New Archean Egg

Description

Adding a new egg for Archean.
Checklist for all submissions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you tested and reviewed your changes with confidence that everything works?
[x] Did you branch your changes and PR from that branch and not from your master branch?

If not, why?:

[x] You verify that the start command applied does not use a shell script

[x] If some script is needed then it is part of a current yolk or a PR to add one

[x] The egg was exported from the panel

New egg Submissions

[x] Does your submission pass tests (server is connectable)?
[ ] Does your egg use a custom docker image?
[x] Have you added the egg to the main README.md and any other README files in subdirectories of the egg (e.g /game_eggs) according to the alphabetical order?
[x] Have you added a unique README.md for the egg you are adding according to the alphabetical order?
[x] You verify that the start command applied does not use a shell script

[x] If some script is needed then it is part of a current yolk or a PR to add one

[x] The egg was exported from the panel

wy the close?

wy the close?

The game requires more packages than the base Ubuntu package in the yolks repo. I was unsure how to add to that and I also realized that the game itself has a file for an egg already but it's not built the exact way that this repo wants (built in a better way in my opinion)


mergejoin memory leak when Depleting incoming batches throw exception

Before submitting a bug report, please verify that you are using the most current version of Drill.
Describe the bug
mergejoin memory leak when Depleting incoming batches throw exception.
because we could close rightIterator when leftIterator throw exception
To Reproduce
Steps to reproduce the behavior:

prepare data for tpch 1s
20 concurrent for tpch sql8
set direct memory 5g
when it had OutOfMemoryException , stopped all sql.
5.finding memory leak

Expected behavior
when all  sql sop , we should find direct memory is 0 AND  could not find leak log like following.
Allocator(op:2:0:11:MergeJoinPOP) 1000000/73728/4874240/10000000000 (res/actual/peak/limit)
Error detail, log output or screenshots
Unable to allocate buffer of size XX (rounded from XX) due to memory limit (). Current allocation: xx
Drill version
The version of Drill you encountered the issue in.
Additional context
// code placeholder
select o_year, sum(case when nation = 'CHINA' then volume else 0 end) / sum(volume) as mkt_share from ( select extract(year from o_orderdate) as o_year, l_extendedprice * 1.0 as volume, n2.n_name as nation from hive.tpch1s.part, hive.tpch1s.supplier, hive.tpch1s.lineitem, hive.tpch1s.orders, hive.tpch1s.customer, hive.tpch1s.nation n1, hive.tpch1s.nation n2, hive.tpch1s.region where p_partkey = l_partkey and s_suppkey = l_suppkey and l_orderkey = o_orderkey and o_custkey = c_custkey and c_nationkey = n1.n_nationkey and n1.n_regionkey = r_regionkey and r_name = 'ASIA' and s_nationkey = n2.n_nationkey and o_orderdate between date '1995-01-01' and date '1996-12-31' and p_type = 'LARGE BRUSHED BRASS') as all_nations group by o_year order by o_year

i fixed it see attachment
0001-mergejoin-leak.patch

https://github.com/apache/drill/issues/2871

scenario  is reliably repeated by above test ,i fixed 。i can not find leak


fix: Add default value for missing code 098 in SII CTE Form 29

Add code 098 to file used by schema validator
:func:cte_f29_datos_schema_best_effort_validator, since this code is currently missing in the tipos and glosa sections of the F29 file.

Fixed exception message when the keys of campos and glosa are different

Ref: https://app.shortcut.com/cordada/story/3852/
Ref: FD-CL-DATA-230
Fixes: FD-CL-DATA-230

@jtrobles-cdd The changelog does not contain functionality updates related to our current use case. Having said that, it will not break the current implementation.

@jtrobles-cdd The error is due to codecov as I edited the error message in src/cl_sii/cte/f29/parse_datos_obj.py:185

@jtrobles-cdd The error is due to codecov as I edited the error message in src/cl_sii/cte/f29/parse_datos_obj.py:185

I'll force the merge to bypass the GitHub check.


Configuration for nodes (zookeeper & config files) is confusion and inconsistent between node types

Configuing nodes is faily confusing! For example, brokers and servers require -zkAddress parameters, whether or not they are provided a -configFileName. Meanwhile, controllers do not allow that parameter if a -configFileName is provided and instead has controller.zk.str configuration value. Same for the the -clusterName and controller.helix.cluster.name configuration value.
This difference makes orchestration confusing.
Also, as a user the zk.str configuration name is pretty meaninless and doesn't help with understanding.
I imagine comitting to a consistent pattern among the launcher script commands would help: require -zkAddress on all types. Additinally, I imagine having a clear config value that is consistent for all of them, eg zookeeper.address (making the config agnostic of controller vs. broker vs. server)

related issue https://github.com/apache/incubator-pinot/issues/5580

Looks like there's plenty of conversation over there that's already covering the topic. Happy to close this!


Fix path concatenation and set vsync to true

Fix concatenation of the argument "meshDataRootDir" of the URDF parser and object description files.
Setting vsync to true helps to decrease CPU usage of the viewer.

Merged in devel.


templates – add default-skeleton keyword

I've noticed, that there is no way to get rid of the default skeleton
with files README.txt, {name}.asd, {name}.lisp and package.lisp.
This will be useful, because thanks to that user will be able to create his own custom templates from scratch (indirectly fixes #2).

(and it extracts default-skeleton implementation from the quicklisp.lisp into default-template.lisp)

the last commits add possibility to rewrite file names (with the very same rules used in the files)

@inaimathi Does your work obsolete this, or is it complementary?

It's a conflicting change. Looks like this is an approach that keeps code-defined generators for the README.txt, {project}.asd and {project}.lisp file. It looks like there are changes that let you override these with a template you can write.
I'm not entirely clear on how this lets you have a file named {project}.asd and {project}.lisp (short of fiddling with your template per generated project), and I'm not sure I like having the default templates defined in code using format (instead of the way users would have to define theirs), but it seems otherwise equivalent.


Fail to start API if database is not connected

The provider will let you start with an undefined database and then run the API. This causes the API to emit errors like cannot read getDatasetDetails of undefined because it's doing this.db.getDatasetDetails(datasetId) within the API task.
We need to prevent the API from running when there is no db or only allow API calls to run that do not involve the DB, returning an error otherwise.

make a getter which checks whether the db is defined and throw if not


feat(hesai): add node name to diag name of hesai hw_monitor

PR Type

Improvement

Related Links
https://tier4.atlassian.net/browse/RT0-34026
Description
Added namespace so that hessai_hw_monitor can diag each LiDAR
You can verify with a bench test that a diag is published containing the namespace of each LiDAR
Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

Thank you! I was very confused why ROS2 includes the node name and provides no option to make it fully qualified (include the namespace like you did). Digging into the source code, there it was:
https://github.com/ros/diagnostics/blob/bb392ee850b7d549e621c265b214c47ad9f2dd8f/diagnostic_updater/src/diagnostic_updater.cpp#L72-L81
The parameter diagnostic_updater.use_fqn makes the node name fully qualified without any effort on our side!

Before
After

@kotaro-hihara Since this is a super useful feature, I've changed the PR to be merged into main and will cherry-pick the commit to also apply to develop-eval. Thanks again for contributing!

Cherry-picked into develop-eval: d04eac5


The Start of World Generation

Works.
'nuf said.

discussion in #45

So what changes need to be made to this at this point?

Alright so, I think the WorldGen thing is too specific and having a simpler API like just generateChunk is simpler and better.
But your superflat generation works, so could you add it there https://github.com/mhsjlw/flying-squid/tree/master/src/lib/worldGenerations ? (you don't need WorldGen at all for superflat)

Hmm well I guess an options object should be passed to the generation though. (so we can configure the 138 of https://github.com/mhsjlw/flying-squid/blob/master/src/lib/worldGenerations/diamond_square.js#L108 for example)

Alright so I added options. It should be possible to add that superflat.

Okay, done in c4974f8e3388dd8e63843d110d9dd5889717bfc4 , thanks !


problem with example demo

Hi, I am using node v0.5.6-pre and npm 1.0.27.
Scenario:
sudo npm install libxmljs
node
> var libxmljs = require("libxmljs");
> var xml =  '<?xml version="1.0" encoding="UTF-8"?>' +
...            '<root>' +
...                '<child foo="bar">' +
...                    '<grandchild baz="fizbuzz">grandchild content</grandchild>' +
...                '</child>' +
...                '<sibling>with content!</sibling>' +
...            '</root>';
> var xmlDoc = libxmljs.parseXmlString(xml);
> var gchild = xmlDoc.get('//grandchild');

Now, when I do > console.log(gchild.text());, I get:
TypeError: Cannot call method 'text' of undefined
at repl:1:20
at Interface.<anonymous> (repl.js:168:22)
at Interface.emit (events.js:67:17)
at Interface._onLine (readline.js:153:10)
at Interface._line (readline.js:408:8)
at Interface._ttyWrite (readline.js:585:14)
at ReadStream.<anonymous> (readline.js:73:12)
at ReadStream.emit (events.js:88:20)
at ReadStream._emitKey (tty_posix.js:306:10)
at ReadStream.onData (tty_posix.js:69:12)

But the rest of the demo seems to work fine:
> var children = xmlDoc.root().childNodes();
> var child = children[0];
> console.log(child.attr('foo').value());
bar

Is this still a problem using the latest 'develop' branch?

Using node v0.6.3, npm 1.1.0-alpha-2 and develop branch of libxmljs, same scenario as before, everything works out as expected.
Thanks for fixing this!


Adding an in memory cache for all heapster data.

As of now the cache is populated by a housekeeping loop.
This cache can serve REST APIs in the future.
Sources will directly write to the cache in the future.
This is a work in progress.
cc @vmarmol

Sent comments, but I will patch this and build off of it :)

Unit tests for the cache is on its way. Otherwise this PR is good to go.

Note the new comments btw :)

Fixed the new set of comments as well. Thanks for the through review @vmarmol. I apologize for the size of this PR

LGTM, thanks for fixing all the nits @vishh!


problem occurs when push a new controller and go back

hi,I add three controllers to the pagemenu and everything goes well.

Then I click the button to push a new controller,it works well too.

But if I go back,the tabs seem to be added again and app will exit abnormally.

Are you trying to dynamically add more items to your page menu? Right now this functionality isn't supported, but it will be part of version 3 of PageMenu which should be out soon.

Sorry, maybe I didn't express my problem clearly. My root view controller is UINavigationController, and I added three child view controllers to the page menu. One of the controller contained a button, I clicked it to go to another controller. But when I went back, the menu seemed to recreate again. So it had six tabs.
But I try to create the menu in viewDidLoad, the problem solved. Thanks for your reply and look forward to version 3.


feat(AgmMap): EventEmitter for maptypeid_changed event

Add the event emitter for google map's maptypeid_changed event

thx!


Link failure problem

https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/notebooks/README.md#attacks
This link is no longer working!

Hi @Zengyf-CVer Thank you very much for reporting this broken link! We will fix it as soon as possible. The notebook is still available at https://github.com/Trusted-AI/adversarial-robustness-toolbox/tree/main/notebooks/adversarial_patch


Diff view: 'Last changed by:' is unknown

Hello,
I noticed that the 'Last changed by:' information has the value 'unknown' if you just clone an empty repository and want to do a diff view before pushing objects for the first time.
It's a minor issue but I suspect it should show the local last change user, not the one from the repository (being not available at this moment).
Happens on NW 740 with abapGit v1.100.0. The changed objects are a simple report and a basic ABAP class.
Best Regards,

"last changed" is always a local user. however, some objects don't support it (like .abapgit.xml) and will always show "unknown".
I tried to reproduce it with a new repo and a program and diff correctly shows my user id as last changed. do you have a screenshot?

Ah, I didn't notice that it's a per-file field; I thought this information is only shown once for the whole set of changes.
Nevermind, I will close this issue.


Modified the JavaTextFileSource so that a remote file can be loaded via HTTPS protocol (or HTTP)

JavaTextFileSource.java provides a selection based on the file URL's protocol.
If https or http is used instead of file, we use an HTTPConnection Object to load the data.
The test class is JavaTextFileSourceTest.java.
The test runs on my system in the branch rel/0.7.1.

I reverted this PR because it was merging into a release branch.
Release branches are automatically generated by Maven commands, and should not be altered manually.
Please, merge the changes on the main branch


Issue: agent tool has been set with return_direct, when using agent.acall, just return Thought to user

Issue you'd like to raise.
when using agent.run, tool's observation is returned.
when using agent.acall, just returned agent's Thought to user
Suggestion:
No response

the question is how to streaming the observation form return_direct tool


[MM-37108] Only make request if server is enterprise ready

Summary
Making the getPrevTrialLicense() request was causing a panic server side when in team edition.
Ticket Link
https://mattermost.atlassian.net/browse/MM-37108
Release Note
NONE

Related PRs
https://github.com/mattermost/mattermost-server/pull/17933

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   249c43da3..8f8279c22  cloud      -> upstream/cloud
   bf9cfe5f8..f45f7abc4  feature/integrate-compass-components -> upstream/feature/integrate-compass-components
   78fa7570e..2cb5e8b53  master     -> upstream/master
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
!!! 'upstream/release-5.38' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

+++ Returning you to the master branch and cleaning up.

/cherry-pick cloud

Cherry pick is scheduled.

/cherry-pick release-5.37

Cherry pick is scheduled.


Copy-edits for clarity/grammar/spelling on the CIMPL in this branch

Copyediting on strings in the CIMPL for the may-2018-ballot-topic-context branch

Did it manually.


[Action] WG Comms Rebrand

Description
After discussion in the main TAG meeting and the WG meeting we are agreed that re-branding (new naming) would be useful for the WG Comms team. This will help us align our goals better and give us a better direction. It will also be easier for others to know what we do!
The suggestion is "WG Environmental Sustainability Advocacy" or "WG Sustainability Advocacy"
Outcome
New name for the WG which is changed in all relevant locations. This should also guide the WG roles as well as new plans for what we can achieve together! Things like the sustainability week fall nicely under this WG especially with the new name.
To-Do

[ ] Change name in WG meetings and TAG meetings
[ ] Change name in the calendar
[ ] Change name across the website
[ ] Change name on github in tags etc.

Code of Conduct

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

Comments
No response

+1 for  WG Sustainability Advocacy

Agree with Valeria - WG Sustainability Advocacy sounds nice😊

+1 for WG Sustainability Advocacy

+1 WG Sustainability Advocacy 🥳 🌱

+1 WG Sustainability Advocacy! Or WG Planet Advocacy :)

That is confirmed! We will start the rebrand after the roadmap meeting and kick off. We also announced it at the sustainability week event which was nice.

I have added the items we came up with in our planning meeting to the To-do list.

Weekly meeting naming is updated in Google calendar. Created service desk ticket for CNCF to update naming and update the meeting time and frequency as per Google calendar. Ticket number: CNCFSD-2545

I can work on these two points if that's okay

[ ]  Change name across the website
[ ] Change in any media/presentations

cc @claire-fletcher @guidemetothemoon @Nancy-Chauhan

Change name in the TOC report/update template --> does anyone know where I can find the TOC report/update template so I can update it? cc @claire-fletcher @guidemetothemoon @Nancy-Chauhan


[BUG]Longhorn doesn't work on K3S + CentOS

Describe the bug
The engine-image pods have no permission to access the mounted volumes, then the engine image cannot become state ready.
To Reproduce
Steps to reproduce the behavior:

Deploy K3S with CentOS nodes
Deploy the latest longhorn. The engine-image pod will get stuck in CrashLoopBackOff status.

Expected behavior
Longhorn should work fine.
Log
[root@ip-172-31-10-68 ~]# k -n longhorn-system logs engine-image-ei-62c02f63-kndkx
cp: cannot create regular file '/data/longhorn': Permission denied

Environment:

Longhorn version: v0.8.1
Kubernetes version: v1.17.5-rc2+k3s1
Node OS type and version: CentOS Linux release 7.7.1908 (Core)

Additional context

This issue is more like a docker issue rather than a Longhorn bug. Not sure if we need to track it here. Directly deploying a pod using host volumes can also reproduce this issue.
Setting privileged security context may help fix this issue. But users have no way to set it for engine image pods since the related DaemonSet is dynamically deployed by longhorn manager.
There is a similar issue here: https://stackoverflow.com/questions/24288616/permission-denied-on-accessing-host-directory-in-docker

@ShylajaDevadiga
Can you provide the steps to deploy the k3s for this issue?

This is probably an issue where selinux is properly enforcing the types and permissions on the host path.
k3s does not use docker by default, it uses containerd with selinux support. To verify it is not an issue with recent containerd changes the install should be compared to k3s using docker with selinux enabled.
If there is no difference with k3s using docker w/selinux or containerd I think the correct solution is that an admin should relabel the directory with the correct type, but it could be that longhorn can have an option to relabel for us.

@shuo-wu To install k3s
curl -fL https://get.k3s.io | INSTALL_K3S_VERSION=v1.17.5-rc2+k3s1 sh -s -

Can confirm same issue on Fedora 32 today.
k3s version v1.17.4+k3s1 (3eee8ac3)
kubectl version
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.4+k3s1", GitCommit:"3eee8ac3a1cf0a216c8a660571329d4bda3bdf77", GitTreeState:"clean", BuildDate:"2020-03-25T16:13:25Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.4+k3s1", GitCommit:"3eee8ac3a1cf0a216c8a660571329d4bda3bdf77", GitTreeState:"clean", BuildDate:"2020-03-25T16:13:25Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

Did some investigating, and this is what I've been able to find out:

Longhorn attempts to install the Engine Image to /var/lib/longhorn/engine-binaries/longhornio-longhorn-engine-${ENGINE_VERSION}, which appears to be under the SELinux context of system_u:object_r:container_var_lib_t:s0. The container runs as container_t, which doesn't seem to have write access on container_var_lib_t based on this comment under containers/libpod#1782.
Docker without SELinux support installs the Engine Image just fine, but fails with the same permission error if SELinux support is enabled (I installed k3s with this command to use Docker: curl -sfL https://get.k3s.io | INSTALL_K3S_EXEC="server --docker" sh -)
Based on kubernetes/kubernetes#27282, it doesn't look like we can just use Kubernetes to relabel the directory for us.

The proper way seems to be as Erik mentioned, which is to have an admin relabel the directory (with perhaps some documentation or a script for them to use/reference during Longhorn installation). The Engine Image Pod will also be able to access the directory successfully if run as Privileged, though that probably is unappealing to an admin who's interested in running with SELinux to begin with.

@ttpcodes
We currently have longhorn-manager, instance-manager, and cronjob pods running as privileged. So I am fine with running engine-image daemonset as privileged as well, given Longhorn is system software.
If https://github.com/kubernetes/kubernetes/issues/27282 can be resolved in the future, we can use Kubernetes to correctly label the volumes. But we cannot wait for that fix at the moment.

Manual test steps:

Set up a node with CentOS and make sure that the output of sestatus indicates that SELinux is enabled and set to Enforcing.
Run the k3s installation script.
Install Longhorn.
The system should come up successfully. The logs of the Engine Image pod should only say installed, and the system should be able to deploy a Volume successfully from the UI.

Note: I ran into some problems with running k3s on CentOS, presumably due to the firewalld rules. This seems to be reported in rancher/k3s#977. I ended up disabling firewalld with systemctl stop firewalld in order to get k3s working.

Pre-merged Checklist

[X] Does the PR include the explanation for the fix or the feature?

[X] Is the backend code merged?
The PR is at longhorn/longhorn-manager#597.

[X] Is the reproduce steps/test steps documented?

[ ] If the fix introduces the code for backward compatibility Has an separate issue been filed with the label release/obsolete-compatibility?

[ ] If labeled: area/ui Has the UI issue filed or ready to be merged?

[ ] if labeled: require/doc Has the necessary document PR submitted or merged?

[ ] If labeled: require/automation-e2e Has the end-to-end test plan been merged? Have QAs agreed on the automation test case?

[ ] if labeled: require/automation-engine Has the engine integration test been merged?

[X] if labeled: require/manual-test-plan Has the manual test plan been documented?
The updated manual test plan is at

Verified on k3s cluster on Centos with SELinux enabled with Master build - 06-11-2020, it is working. Longhorn can be deployed successfully on this setup.
Verification - Passed

Set up k3s cluster on Centos 7.

# cat /etc/os-release
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
# sestatus
SELinux status:                 enabled
SELinuxfs mount:                /sys/fs/selinux
SELinux root directory:         /etc/selinux
Loaded policy name:             targeted
Current mode:                   enforcing
Mode from config file:          enforcing
Policy MLS status:              enabled
Policy deny_unknown status:     allowed
Max kernel policy version:      31

Installed longhorn.
Engine image came up healthy.
Verified the P1 worklows.


Ignore all errors in pyproject.toml files when --no-project is used in uv run

Closes https://github.com/astral-sh/uv/issues/6550

Going with @charliermarsh's solution


Increase the wait latency of WebSocket ping

Problem:
The read latency of WebSocket is too short for Windows agent, 10s read
deadline might not enough for the peer to receive ping.
Solution:
Increase the wait duration of ping from 10s to 30s.
Issue:
https://github.com/rancher/rancher/issues/22543

lgtm

Darren observed this issue in a user's setup (linux) and has a more comprehensive fix. It rearranges the startup logic so that the plan monitor logic does not occur in between opening the websocket and starting to send pings. Hopefully he gets that PR in soon and we can do that instead.

I'm going to close this now as darren has merged fixes for this here https://github.com/rancher/remotedialer/commit/0badea25b7d37a4072531b6e320b3df7f57a51b7

@cjellick , thanks


Pode.Web Tutorial page is hosed.

https://badgerati.github.io/Pode.Web/Getting-Started/Tutorials/Basics
Icons consume entire browser page.

All links to pages (i,e New-PodeWebIcon, etc) return 404

Hey @rob3rt-keating,
The link which I'm guesing you've clicked is wired up wrong and should be pointing at https://badgerati.github.io/Pode.Web/Tutorials/Basics/ - this is fixed in #61/v0.5.0 and should be out very soon.

v0.5.0 has now been released, and the link causing the issue has now being fixed. If it's still an issue please feel free to re-open.


Add cross build action

Codecov Report

Merging #7 (9af4d24) into master (4887bd6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   48.13%   48.13%
=======================================
  Files           8        8
  Lines         322      322
=======================================
  Hits          155      155
  Misses        167      167

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 4887bd6...9af4d24. Read the comment docs.


Invalid reference of ProtoPackageIsVersion4 in v1.25.0

What version of protobuf and what language are you using?
Language: GoLang
Version:

protoc: libprotoc 3.13.0
protoc-gen-go: protoc-gen-go v1.25.0

What did you do?
I followed the below steps.
mkdir protoc-learning
go mod init proto-gen-hello
cat <<EOF >hello.proto
> syntax = "proto3";
> package hello;
> option go_package = ".;hello";
> message Greet {
>     string hello = 1;
> }
> EOF

cat <<EOF >main.go
> package main
> import "log"
> func main() {
>	log.Print("Hello protoc")
> }

go get<EMAIL_ADDRESS>
⚠ This above step is resulting in the following known errors though
https://github.com/golang/protobuf/issues/1033
https://github.com/golang/protobuf/issues/1224
The workaround I used is adding the module manually in the go.mod file.
mkdir out

 protoc --proto_path . -I=. hello.proto --hello_out=./out --go_out=./out

As expected, the above command creates the pb.go file. But it creates the file with proto "github.com/golang/protobuf/proto" dependency.
With this also, the file refers proto.ProtoPackageIsVersion4 constant.
What did you expect to see?
I expect to get v1.25.0 dependency resolved and the pb.go file should be created with the corresponding constants/variables. Because the proto.ProtoPackageIsVersion4 is not available in the v1.25.0
What did you see instead?
I see the dependency as proto "github.com/golang/protobuf/proto" and referred const _ = proto.ProtoPackageIsVersion4 constant which is not available in v1.25.0 as I have added that in the go,mod.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment?

Click to see go.mod file

module protoc-gen-hello
go 1.15
require (
google.golang.org/protobuf v1.25.0
)

Click to see proto file

syntax = "proto3";
package hello;
option go_package = ".;hello";
message Greet {
    string hello = 1;
}

Click to see pb.go file

// Code generated by protoc-gen-go. DO NOT EDIT.
// versions:
// 	protoc-gen-go v1.25.0
// 	protoc        v3.13.0
// source: hello.proto
package hello
import (
proto "github.com/golang/protobuf/proto"
protoreflect "google.golang.org/protobuf/reflect/protoreflect"
protoimpl "google.golang.org/protobuf/runtime/protoimpl"
reflect "reflect"
sync "sync"
)
const (
// Verify that this generated code is sufficiently up-to-date.
_ = protoimpl.EnforceVersion(20 - protoimpl.MinVersion)
// Verify that runtime/protoimpl is sufficiently up-to-date.
_ = protoimpl.EnforceVersion(protoimpl.MaxVersion - 20)
)
// This is a compile-time assertion that a sufficiently up-to-date version
// of the legacy proto package is being used.
const _ = proto.ProtoPackageIsVersion4
type Greet struct {
state         protoimpl.MessageState
sizeCache     protoimpl.SizeCache
unknownFields protoimpl.UnknownFields
Hello string `protobuf:"bytes,1,opt,name=hello,proto3" json:"hello,omitempty"`

}
func (x *Greet) Reset() {
*x = Greet{}
if protoimpl.UnsafeEnabled {
mi := &file_hello_proto_msgTypes[0]
ms := protoimpl.X.MessageStateOf(protoimpl.Pointer(x))
ms.StoreMessageInfo(mi)
}
}
func (x *Greet) String() string {
return protoimpl.X.MessageStringOf(x)
}
func (*Greet) ProtoMessage() {}
func (x *Greet) ProtoReflect() protoreflect.Message {
mi := &file_hello_proto_msgTypes[0]
if protoimpl.UnsafeEnabled && x != nil {
ms := protoimpl.X.MessageStateOf(protoimpl.Pointer(x))
if ms.LoadMessageInfo() == nil {
ms.StoreMessageInfo(mi)
}
return ms
}
return mi.MessageOf(x)
}
// Deprecated: Use Greet.ProtoReflect.Descriptor instead.
func (*Greet) Descriptor() ([]byte, []int) {
return file_hello_proto_rawDescGZIP(), []int{0}
}
func (x *Greet) GetHello() string {
if x != nil {
return x.Hello
}
return ""
}
var File_hello_proto protoreflect.FileDescriptor
var file_hello_proto_rawDesc = []byte{
0x0a, 0x0b, 0x68, 0x65, 0x6c, 0x6c, 0x6f, 0x2e, 0x70, 0x72, 0x6f, 0x74, 0x6f, 0x12, 0x05, 0x68,
0x65, 0x6c, 0x6c, 0x6f, 0x22, 0x1d, 0x0a, 0x05, 0x47, 0x72, 0x65, 0x65, 0x74, 0x12, 0x14, 0x0a,
0x05, 0x68, 0x65, 0x6c, 0x6c, 0x6f, 0x18, 0x01, 0x20, 0x01, 0x28, 0x09, 0x52, 0x05, 0x68, 0x65,
0x6c, 0x6c, 0x6f, 0x42, 0x09, 0x5a, 0x07, 0x2e, 0x3b, 0x68, 0x65, 0x6c, 0x6c, 0x6f, 0x62, 0x06,
0x70, 0x72, 0x6f, 0x74, 0x6f, 0x33,
}
var (
file_hello_proto_rawDescOnce sync.Once
file_hello_proto_rawDescData = file_hello_proto_rawDesc
)
func file_hello_proto_rawDescGZIP() []byte {
file_hello_proto_rawDescOnce.Do(func() {
file_hello_proto_rawDescData = protoimpl.X.CompressGZIP(file_hello_proto_rawDescData)
})
return file_hello_proto_rawDescData
}
var file_hello_proto_msgTypes = make([]protoimpl.MessageInfo, 1)
var file_hello_proto_goTypes = []interface{}{
(*Greet)(nil), // 0: hello.Greet
}
var file_hello_proto_depIdxs = []int32{
0, // [0:0] is the sub-list for method output_type
0, // [0:0] is the sub-list for method input_type
0, // [0:0] is the sub-list for extension type_name
0, // [0:0] is the sub-list for extension extendee
0, // [0:0] is the sub-list for field type_name
}
func init() { file_hello_proto_init() }
func file_hello_proto_init() {
if File_hello_proto != nil {
return
}
if !protoimpl.UnsafeEnabled {
file_hello_proto_msgTypes[0].Exporter = func(v interface{}, i int) interface{} {
switch v := v.(*Greet); i {
case 0:
return &v.state
case 1:
return &v.sizeCache
case 2:
return &v.unknownFields
default:
return nil
}
}
}
type x struct{}
out := protoimpl.TypeBuilder{
File: protoimpl.DescBuilder{
GoPackagePath: reflect.TypeOf(x{}).PkgPath(),
RawDescriptor: file_hello_proto_rawDesc,
NumEnums:      0,
NumMessages:   1,
NumExtensions: 0,
NumServices:   0,
},
GoTypes:           file_hello_proto_goTypes,
DependencyIndexes: file_hello_proto_depIdxs,
MessageInfos:      file_hello_proto_msgTypes,
}.Build()
File_hello_proto = out.File
file_hello_proto_rawDesc = nil
file_hello_proto_goTypes = nil
file_hello_proto_depIdxs = nil
}

Please let me know if I am missing anything obvious.

the ProtoPackageIsVersion4 has nothing to do with the google.golang.org/protobuf. Therefore, it's expected that you won't find it in v1.25.0 (in fact you won't find it on version of that module).
The identifier is declared in the github.com/golang/protobuf module at v1.4.0 and above.

Closing as working as intended.

As a bit of additional detail here, this is was an incompatibility between the protoc-gen-go binary installed on your machine, and the package version that you’re depending on, which was a regular source of issues.
It is not sufficient to just have the intended dependency in go.mod the actual binary used for your protoc-gen-go has to match that dependency.
This issue has mostly been fixed for this standalone protobuf package by making the protobuf package itself more backwards-forwards compatible, and moving the lock-step-change requirement over to protoc-gen-go-grpc.

Thanks @puellanivis  for the explanation.


Add support for Zcoin wallet

This change depends on https://github.com/OpenBazaar/openbazaar-go/pull/899

We will not be merging the corresponding server PR until it uses an SPV wallet because relying on a full node is something we've found 99% of users won't do and/or don't have the technical skill level to do. If the server PR is updated to support an SPV wallet, please feel free to resubmit this PR.
Thanks for the the interest in contributing to OB!


Indentation problem in online document

The indentation is not working correctly in the online document.

http://keras.io/models/

That is more like a psuedo-code. Look at #1638 for a code snippet on how to make a proper data generator.

My point is not how to make a generator, it's the document is not generated properly. The document is generated from the docstring in the code.
For example, you can see the following code in the file: https://github.com/fchollet/keras/blob/657b9fb48e93b59083d2e0b8a5e4daf237179dbc/keras/models.py
        # Examples
        ```python
            def generate_arrays_from_file(path):
                while 1:
                    f = open(path)
                    for line in f:
                        # create numpy arrays of input data
                        # and labels, from each line in the file
                        x, y = process_line(line)
                        yield x, y
                    f.close()
            model.fit_generator(generate_arrays_from_file('/my_file.txt'),
                                samples_per_epoch=10000, nb_epoch=10)
        ```

Oh okay, when you said, The indentation is not working correctly in the online document.. I thought when you convert that psuedo-code into an actual one, it was not working for you.

Thanks all the same and sorry for the misleading.

Any other places in the documentation where you're seeing this?

This is fixed.


Adds Reactotron

On the form for adding a component, there's only spot for 1 lib.  Reactotron has 7, but I didn't want to submit 7 additions as that didn't feel right.
So here we are!
Thanks for the huge compilation!

didn't know about it, it looks neat
yes, I wouldn't see the point to add all 7 npm packages related to Reactotron. One should be enough for people to discover Reactotron
Why didn't you put Reactotron into Dev Tools > Inspect ?
I'll merge after we clarified the categorization.

For the most part it can inspect, but you can actually dispatch actions through it as well.  So there's an element of remote control.  One of the next big features is exploring more of that.  :)
You know your categorization better than me.  I'm happy to be on your list, you can decide the best spot for Reactotron.  🎉

sounds exciting  🎉
merged :)


"OS/X" instructions are valid for all unix-like operating systems

So in the readme and angular docs you should rename "OS/X (bash)" and put only "bash" or "posix", "unix-like", etc.

This repository is now deprecated. The Angular Quickstart project was a nice starting point for creating Angular applications. Now we recommend using the Angular CLI to create new Angular projects.
Starting from 1 November 2017, all the Angular documentation, at angular.io, is based on the Angular CLI.
Let's get started


Create chartie-gmaps-heatmap.js

This js script is a plugin that extends gmaps-heatmap.js allowing to process and display signal data on heatmap.
It makes use of a call to the Chartie services (www.chartie.io) a free web API that mines the relevant trend patterns from your signal.
Thanks to this plugin it is possible to create rich live dashboards see http://www.chartie.io/TrentinoOpenData/ for an example.
For any question contact us at<EMAIL_ADDRESS>

This plugin doesn't add any value to the already existing gmaps overlay, it could be easily done externally. Also it would introduce a jquery dependency


Empty title

I haven't spent much time debugging this but it appears some titles are retuned empty while they are not from the Amazon product API
product = AmazonAPI(AMAZON_ACCESS_KEY, AMAZON_SECRET_KEY, AMAZON_ASSOC_TAG, region="UK").lookup(ItemId='B00PK4LLLS')
print product.title

this prints None
product on amazon.co.uk https://www.amazon.co.uk/Downton-Abbey-Moorland-Holiday-Christmas/dp/B00PK4LLLS
Version: 2.2.11

Is it returning any values?

Hi Jason,
Just tried it again today and it looks like it fixed itself, maybe a amazon api bug...
I checked the raw response yesterday and it seemed to be containing all informations but I didn't try to access any other attribute from the library as I only use the title...

Glad it sorted itself out...


Fails to initialize

The cover fails to add to HA due to line 110. I changed locally to below to get it working
return True ##self.current_cover_position <= 0

The cover fails to add to HA due to line 110. I changed locally to below to get it working
return True ##self.current_cover_position <= 0

This worked for me.
Thanks!

Does this fix my situation where I cannot update?
Regards

On Jan 13, 2022, at 10:08 AM, isewise @.***> wrote:
The cover fails to add to HA due to line 110. I changed locally to below to get it working
return True ##self.current_cover_position <= 0
This worked for me.
Thanks!
—
Reply to this email directly, view it on GitHub https://github.com/alextud/ha-switchbot-curtain/issues/28#issuecomment-1012334573, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHOJDTN5PSLVIQWKU7T7TU3UV4BJLANCNFSM5L2CIKYQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.

Actually I started using this because the switchbot integration provided in HA wasn't working for my setup, but I decided to try that integration again and it is now working for me (and better then this integration ever did). If you want to give that a try, make sure you completely remove this integration and restart HA before adding with the default integration (otherwise they collide and neither works).

I just tried removing the old HACs version that works, installing the Integration (didn’t work) removed the integration, installed latest HACS version (didn’t work) and went back to the old HACs version.  Curtain works, but battery is wrong.

On Jan 20, 2022, at 5:01 PM, apollo1220 @.***> wrote:
Actually I started using this because the switchbot integration provided in HA wasn't working for my setup, but I decided to try that integration again and it is now working for me (and better then this integration ever did). If you want to give that a try, make sure you completely remove this integration and restart HA before adding with the default integration (otherwise they collide and neither works).
—
Reply to this email directly, view it on GitHub https://github.com/alextud/ha-switchbot-curtain/issues/28#issuecomment-1018032454, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHOJDTOYU4FYLK2C6SZEWEDUXCO5TANCNFSM5L2CIKYQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.

I'm also seeing this:
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 382, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 614, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 801, in add_to_platform_finish
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 553, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 590, in _async_write_ha_state
    state = self._stringify_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 559, in _stringify_state
    if (state := self.state) is None:
  File "/usr/src/homeassistant/homeassistant/components/cover/__init__.py", line 240, in state
    if (closed := self.is_closed) is None:
  File "/config/custom_components/switchbot-curtain/cover.py", line 110, in is_closed
    return self.current_cover_position <= 0
TypeError: '<=' not supported between instances of 'NoneType' and 'int'


Record Security

In multi user environment,it is needed to know how added this record , how updated this record (the last operation of update).
i suggest add new DataAnnotations fields for CreatedBy ,UpdatedBy,CreatedOn,UpdatedON.

Fluidity already supports timestamp properties which you can find docs on
here
https://umco.github.io/umbraco-fluidity/api/collections/#defining-time-stamp-properties
We don't yet have support for who created / edited though so we will need
to think about that.
Thanks for the suggestion
Matt
On Fri, 25 May 2018, 12:53 pm mrafea-sa<EMAIL_ADDRESS>wrote:

In multi user environment,it is needed to know how added this record , how
updated this record (the last operation of update).
i suggest add new DataAnnotations fields for CreatedBy
,UpdatedBy,CreatedOn,UpdatedON.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/umco/umbraco-fluidity/issues/47, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgLyf_5YsXtwiWCYf2xbc9XtJ3wfAfdks5t1-KVgaJpZM4UNxdw
.


IssueFiles controller not callable

Controller "Ojs\JournalBundle\Controller\IssueFileController::indexAction" for URI "/journal/issue-file/" is not callable.

http://ojs.io/journal/issue-file/


Changing TerrainProvider on the go

Hey Mike,
Is it possible to change the terrainprovider without reloading the library?
Thank you

Done!
http://openglobus.org/examples/terrainSwitch/terrainSwitch.html
P.S.
Not the best decision...but it works!


Update for ghc-7.10

The new Prelude includes mconcat and foldMap, so the export list was declared ambiguous. Fortunately, since there is no ambiguous use within the file, this expedient
, Control.Foldl.mconcat
, Control.Foldl.foldMap

seems to work fine.

Thanks!


Update output-management.md

clean-webpack-plugin has released 3.0.0
https://github.com/johnagan/clean-webpack-plugin/releases
“Replaced default export with named export CleanWebpackPlugin”
const { CleanWebpackPlugin } = require('clean-webpack-plugin')

Thanks!


add my file

looks good lol idk what im doing

New comment here!  everything looks good i guess it errored out on me.


Allow to setup an independent nfsserver for cinder

When testing the upgrade from 6 to 7 we need to use a cinder backend that doesn't use storage that is located on any of the cloud nodes (or the admin node) to have it available during the whole upgrade process. This PR adds the ability to have mkcloud setup a "lonelynode" with an NFS server running that is exporting /srv/nfs/cinder.

The code it still a bit rough a round the edge, but as we need this somewhat urgently I wanted to submit as early as possible.

Typo in commit message: resolveable

 S57P4 (5) Adapt mkcloud / virtual upgrade jobs to setup and use nfs server for cinder

Typo in commit message: resolveable

fixed

I refactored it a bit to remove some redundancy with the crowbar_register code

@rhafer please add an mkcloud gating run that deploys the nfsserver

please add an mkcloud gating run that deploys the nfsserver

Ok. It's running. If will fail in the testsetup step, because of: https://github.com/crowbar/crowbar-openstack/pull/869, but that should not matter here.

Ok. It's running. If will fail in the testsetup step, because of: crowbar/crowbar-openstack#869, but that should not matter here.

@jdsn The job failed as expected. But that is really a bug in the tempest barclamp, and fixed already. Are you ok with this PR now?

Ok. It's running. If will fail in the testsetup step, because of: crowbar/crowbar-openstack#869, but that should not matter here.

@rhafer well if you know already, then why did you include the step in the gating run?


Fabulous logo for Fabulous

Today on twitter we started discussion about logo candidates for Fabulous
https://twitter.com/kunjee/status/1093008978332864512
Option 1:
Currently we have one proposal with 3 positive feedbacks:

Happy to hear other options/opinions.

I find the angles of the F really sharp.
If you look at the Xamarin logo, you can see that they put a small border-radius.

@MangelMaxime Good catch! (said my wife)
Option 2 (with border radius):

I do prefer this version :)

Good initiative!
Option 2 is better. :)
Perhaps with a different color?
It's a little too close to the Xamarin logo, it might confuse people.
Also I'd like to avoid a "Xamarin" logo, as lots of people are asking support for other frameworks like Avalonia/Uno as well.
Something we might be able to do in a distant future.
I must say I've never been a fan of the Xamarin logo, so partial here. :)

Also I'd like to avoid a "Xamarin" logo, as lots of people are asking support for other frameworks like Avalonia/Uno as well.

Yes.... I said "yes" to Xamarin-like logo on twitter, I thought it over and wondered the same thing....   But equally I'm no good at logo design.....
Note also that the React/Flux/Flow... technologies all use logos which carry a "flowing information around the app, reacting to changes".  That might be a place to start.

I was thinking:
Maybe we should play on the word "Fabulous" for the logo.
Make it memorable for people, instead of an abstract logo.
Do like Fable and use something related to "fabulous". Like a mythical beast for example.
There's a lot of phoenix logo on the web, it seems... :)

Not particularly serious suggestions these, just playing around with ideas...

What about the signature F# operator?

Credits: Good ol' Paint on Windows 10

Closing since Fabulous as a logo now.


[BUG] Playing games

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] My answer given in the checkbox above is a lie.

What problem did you encounter?
Whenever i try to run roblox itll load then just close not giving a reason why.

https://github.com/pizzaboxer/bloxstrap/wiki/Roblox-crashes-or-does-not-launch

I'll reopen once you've confirmed you've read the above link.


⚠ T:Riza Community Forum has degraded performance

In 1c280d0, T:Riza Community Forum (https://community.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 3669 ms

Resolved: T:Riza Community Forum performance has improved in 2d69f10.


[YBUG.IO] The page vis not showing

The page vis not showing
Source url: https://test.sovryn.app/
Reported by<EMAIL_ADDRESS>Reported at: 27 Apr at 15:03 UTC
Location: IN, Kerala, Kochi
Browser: Chrome 89.0.4389.114
OS: Android 11
Screen: 360x800
Viewport: 360x720
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/mg8rt5kq058rd7bsnngh

Issue should be resolved now


When Kafka is temporarily unavailable during startup, OMRS cohorts never get initialized

I have put together a fix for this issue which I will be submitting for review shortly.
We have seen this in a few cases where the Kafka pod gets into a weird state in our environment.  This is not a common scenario, but we have seen it enough times that it makes sense to do what we can to automate the recovery when Kafka does come back up.
What we have done here is put in a retry mechanism that kicks in if the initial connection to the cohort fails.  The retry mechanism is totally configurable and can be turned off If desired.  With the changes, the first initialization attempt is done synchronously, the same way it is now.  If the initialization attempt fails, then an additional attempt is scheduled to run in a background thread if certain conditions are met.  If that fails, we schedule an additional retry attempt.  We do this until the configured maximum number of retries have been exhausted.  The default behavior is that there is one retry thread shared between all of the cohorts.  This can be configured programmatically by applications that embed egeria.   In addition, the default number of retries is 20, and by default we wait 10 seconds between retries.  These are also configurable.  There are certain kinds of initialization errors we exclude from the mechanism.  These are for cases where we are sure that doing a retry will not resolve the issue.
The changes also introduce a new CohortConnectionStatus: AWAITING_RETRY.  If the intialization of the cohort manager fails in a way that is retryable (and the max retries have not been exhausted), the connection state is set to AWAITING_RETRY.  When the retry starts, the state will go to INITIALIZING and resume its usual progression.  If the initialization fails with an error that is not retryable, the state will stay at CONFIGURATION_ERROR (and no retries will be scheduled)

I've created https://github.com/odpi/egeria/pull/2596

I added some comments to PR #2596 - seen before I saw the issue (catching up)

We believe #2989 will address this scenario, and the PR in #2990 is now merged.
Please re-open if the fix is insufficient


Small change of the home-learn.svg for the eye

Just a small change of the home-learn.svg on en.serlo.org

awesome thanks!


Empty query parameters are sent

Rapidoc sends empty query params. so if i don't fill it out, i'll get an error..
Example:
It calls:
https://api.infinimesh.io/namespaces/joe/objects?recurse=
it should omit this param completely..in my case my backend will reject it because the value is not a legit boolean. if it's omitted, it's fine.
Swagger-ui omits this param completely, if nothing is entered into the textbox

Thanks for reporting this issue. Let me think it over a little.
However, there is difference in the approaches of swagger-ui and RapiDoc's API console
Swagger-UI puts many validations for its console,  RapiDoc avoids putting any validation, the goal is not to make a perfect client for the API, but to allow users/developers to checkout the api with various values (including the negative ones)
The case you presented above is actually a good test case on how the api behaves when provided with an empty value, With swagger-ui you can never test this case.
Having said that we are having a little discussion on our end on what to do with empty params.
I will update this issue once we come to a conclusion.
I hope you understand our philosophy of not doing any validation

Like you suggested , we decided to not submit empty values. Just  published a new release 4.1.0, can you confirm, if it takes care of your issue ?

i can confirm, it now works as i expected. when empty, it's omitted.
perfect, thanks so much for acting so quickly! :+1:


docs: reference pages for Evaluation and Metrics definitions and providers

Closes https://github.com/keptn/lifecycle-toolkit/issues/1080

Create reference pages out of the content in the concepts/evaluations file
Create reference pages and implementing/metrics out of the content in the concepts/metrics file
Delete the concepts/evaluations and concepts/metrics files
Update content for 0.8.0 changes

This PR is corrupted and so I am deleting it.
This PR is replaced by https://github.com/keptn/lifecycle-toolkit/pull/1385


Normalize API endpoints to allow for shorthand

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #190

PR #191

PR #192   👈

This stack of pull requests is managed by Graphite.

big


Aync HTTP Requests + Notificatiions

I made the POST request run asynchronously. The submissionSet holds on to all unsuccessfully submitted names and gets cleared out after a 200 response code is received. This free ups the hashset h to collect fresh player names without getting bogged down in the main thread.
I also added some notification pop-ups to notify the user of how their submission attempts are going in the background.


Add instanceInitializer

This PR is to pave the way for fixing https://github.com/simplabs/ember-simple-auth/issues/517. I think it is logical to split the fix into multiple commits.

+1

@marcoow Any comment on this?

This would also need to make sure that the existing initializer doesn't use the container's lookup method when that's done in the instance initializer, otherwise you'd still get the deprecation notice. Also there's #532 which takes some more steps to solve the problem which I'm not sure are all actually necessary.

+1

+1

See https://github.com/simplabs/ember-simple-auth/pull/532#issuecomment-113643062 - would be cool to get something like this for the ember-cli-addon branch.

see #564, thanks for the effort!


Change how keywords are handled

Instead of parsing things as keywords without knowing whether the token needed is a keyword or not, let's instead have a parse_keyword(keyword: &str) method that calls as a parse_ident() method and then checks if the return identifier matches the expected keyword.

This turned out to be harder than it looked.  I'll probably have to create a small toy parser to see if I can figure out the right pattern before implementing it for real.


General repository cleanup

Description

These are some odds-and-ends that can be addressed as a single pull request.
Specifics

[x] remove root platform.yaml file
[x] remove storage.py and tests from Python library
[x] remove event.py and tests from Python library
[ ] build Rust CloudEvents resources in Rust library if it does not exist
[ ] unify all Alpaca key references

[ ] ensure Rust references are "wrapped" by the ALPACA_* environment variable names

Closing due to inactivity.


Fix start-containers script

Recent changes to fedimintd broke the start-containers script which brings up a federation using docker and docker-compose. This PR makes the small changes to fix the script.

This is going to be very difficult to keep up-to-date over next couple months as things rapidly change ...

I think we will largely iron out the command line / env. var calling conventions in next two weeks, after which I don't think we will be changing them much. :crossed_fingers:


fatal error: 'gnu/stubs.h' file not found

./tools/android-build
In file included from /usr/android-sdk-linux/ndk/21.2.6472646/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/c++/v1/__config:215:
/usr/include/features.h:485:10: fatal error: 'gnu/stubs.h' file not found
#include <gnu/stubs.h>
^~~~~~~~~~~~~
1 error generated.
ninja: build stopped: subcommand failed.

Duplicated with #1872


Work with generics

Microsoft/vscode-python#2446
E.g.
from typing import List

def func(a: List[str]):
    pass

Currently a has no completions instead of having completions for a list. And obviously taking it farther, supporting indexing on a so the result is str would be great!
See generics from PEP 484 for some details.

Yeah, this works for both of those two cases:

Actually, reading through that vscode-python bug, the user is expecting to see list functions on a, not just __getitem__ and __iter__. I'm not sure if that's exactly correct, though, since I don't think the generic type assumes anything other than "I can be indexed" and "I can be iterated over".

Reading through that thread also shows that def func(a: List) will show completions for a as though it were list, but def func(a: List[str]) will not, which seems to indicate that there's something else going on here.

typing.py says:
https://github.com/python/typing/blob/9443f261f5b03c2f0535cdd9c1e7a14c0fdb5952/src/typing.py#L1883-L1891
Which to me implies that it should be inheriting everything from list.

I have updated my comment to drop the a[0] case as that case works as demonstrated, but as @jakebailey realized the a case is not working as it should be showing all completions that list would show.


Add config-dir flag to change .driftctl location

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#555

❓ Documentation
yes - i must add this flag to the documentation

Description
I add a flag 'config-dir' to override the .driftctl default path

What if the config-dir is not a directory but a file ?

DEBU[0024] Bound environment variable to flag env=disable_telemetry flag=disable-telemetry value=true
DEBU[0024] New provider library created
DEBU[0024] Starting gRPC client alias=eu-west-3
fork/exec testconfigdir/.driftctl/plugins/darwin_amd64/terraform-provider-aws_v3.19.0: not a directory
Exiting.

LGTM
❯  DCTL_TF_PROVIDER_VERSION=3.44.0 driftctl-0.10.0 scan --config-dir ./tmp
[...]
❯  tree tmp/.driftctl/
tmp/.driftctl/
└── plugins
    └── linux_amd64
        └── terraform-provider-aws_v3.44.0_x5

Just added a companion issue: https://github.com/cloudskiff/driftctl/issues/669


How do I change the JDBC driver MariaDB4j uses?

I notice from the debug logging of dbunit that it is always using jdbc driver version=mysql-connector-java-6.0.6 and setting the driver jar in my maven pom has no effect.
The only reference I can find to the MySQL Connector J is in the license.txt where there is a hint that I can use the mariadb-java-client if I want to avoid license issues, however I can't see how I would do that.
Any advice please?

Sorry, it's just my IDE caching everything and not letting go until I reboot the ?"!?* computer.
I see it's now logging that I'm using the MariaDB connector/J 2.1.2 so I guess it has nothing to do with MariaDB4j.


Opening editor fails with process XXX not running

Trying around, i cleared ~/.emacs.d/elpa. So (m)elpa packages should be up-to-date. I have only intero installed right now. My project is https://github.com/janesser/rsync-imap/commit/a91808b29f6916c4d1c1d8e5fb4efe11e69165dd.
(setq debug-on-error t) is active.
Often emacs just freezes when i try to open any *.hs file after emacs start.
Then i occasionally clear rm .stack-work/intero -fR. Not really changing much.
Finally i get that stacktrace:
Debugger entered--Lisp error: (error "Process /home/jan/projs/rsync-imap not running")
  process-send-string(#<process /home/jan/projs/rsync-imap> "check /home/jan/projs/rsync-imap/app/Main.hs\n")
  ghc-with-process("check /home/jan/projs/rsync-imap/app/Main.hs\n" ghc-check-callback #[nil "\301\211\207" [mode-line-process " -:-"] 2])
  ghc-check-syntax()
  ghc-init()
  (lambda nil (ghc-init))()
  run-hooks(change-major-mode-after-body-hook prog-mode-hook haskell-mode-hook)
  apply(run-hooks (change-major-mode-after-body-hook prog-mode-hook haskell-mode-hook))
  run-mode-hooks(haskell-mode-hook)
  haskell-mode()
  set-auto-mode-0(haskell-mode nil)
  set-auto-mode()
  normal-mode(t)
  after-find-file(nil t)
  find-file-noselect-1(#<buffer Main.hs> "~/projs/rsync-imap/app/Main.hs" nil nil "~/projs/rsync-imap/app/Main.hs" (548264 47))
  find-file-noselect("/home/jan/projs/rsync-imap/app/Main.hs" nil nil nil)
  find-file("/home/jan/projs/rsync-imap/app/Main.hs")
  dired-find-file()
  call-interactively(dired-find-file nil nil)
  command-execute(dired-find-file)

I can get past this with M-x debugger-continue.
Then saving a changed file again freezes emacs. I let it run for more than 20 minutes, which should be enough even on my legacy netbook (amd k125). Emacs will not come back.

actually same setup on another computer works perfectly.
any hints what to look for?

https://emacs.stackexchange.com/questions/21643/what-do-i-do-when-emacs-is-frozen
found hint to try C-g which resolves my issue.
I don't understand what "Spam" exactly means in this context. Weird somehow.

I don't understand what "Spam" exactly means in this context. Weird somehow.

If you think of "spamming" as sending something incessantly whether the receiver wants it or not, then it makes sense for keypresses too. :-)


Type incompatible error when using optional properties in interface

Link to flow try:
/* @flow */
interface X {
  a?: string
};
type Y = {
  a?: string
};

const y : Y = {
  a: 'arst'
};

// Errors out here
(y : X);

Error:
14: (y : X);
     ^ Cannot cast `y` to `X` because undefined [1] is incompatible with string [2] in property `a`.
References:
6:   a?: string         ^ [1]
3:   a?: string         ^ [2]

There are no errors when a is made not optional (link).
It might have something to do with the variance of the property since setting a to write-only works, though read-only doesn't work (which I imagine is more important for most users).

Bumping this back up, in case it got forgotten.

bump

It now works


Fixes vimeo autoplay

This PR fixes Vimeo autoplay. Fixes #21

Wow that was fast! :-O
I will check that this morning :-)

There's no hurry, review whenever you have time :)
I'll see if I can tackle some more issues.