Getting error while running default rasa model

$ rasa train
/home/times-67/rasa/rasa/core/training/dsl.py:416: UserWarning: Found unknown intent 'inform' on line 4. Please, make sure that all intents are listed in your domain yaml.
f"Found unknown intent '{intent_name}' on line {line_num}. "
Traceback (most recent call last):
File "/home/times-67/anaconda3/bin/rasa", line 11, in
load_entry_point('rasa', 'console_scripts', 'rasa')()
File "/home/times-67/rasa/rasa/main.py", line 76, in main
cmdline_arguments.func(cmdline_arguments)
File "/home/times-67/rasa/rasa/cli/train.py", line 76, in train
kwargs=extract_additional_arguments(args),
File "/home/times-67/rasa/rasa/train.py", line 46, in train
kwargs=kwargs,
File "uvloop/loop.pyx", line 1417, in uvloop.loop.Loop.run_until_complete
File "/home/times-67/rasa/rasa/train.py", line 97, in train_async
kwargs,
File "/home/times-67/rasa/rasa/train.py", line 140, in _train_async_internal
file_importer.get_stories(), file_importer.get_nlu_data()
File "/home/times-67/rasa/rasa/importers/importer.py", line 269, in get_nlu_data
nlu_data = await asyncio.gather(*nlu_data)
File "/home/times-67/rasa/rasa/importers/rasa.py", line 61, in get_nlu_data
return utils.training_data_from_paths(self._nlu_files, language)
File "/home/times-67/rasa/rasa/importers/utils.py", line 9, in training_data_from_paths
training_datas = [loading.load_data(nlu_file, language) for nlu_file in paths]
File "/home/times-67/rasa/rasa/importers/utils.py", line 9, in
training_datas = [loading.load_data(nlu_file, language) for nlu_file in paths]
File "/home/times-67/rasa/rasa/nlu/training_data/loading.py", line 67, in load_data
data_sets = [_load(f, language) for f in files]
File "/home/times-67/rasa/rasa/nlu/training_data/loading.py", line 67, in
data_sets = [_load(f, language) for f in files]
File "/home/times-67/rasa/rasa/nlu/training_data/loading.py", line 138, in _load
return reader.read(filename, language=language, fformat=fformat)
File "/home/times-67/rasa/rasa/nlu/training_data/formats/readerwriter.py", line 15, in read
return self.reads(rasa.utils.io.read_file(filename), **kwargs)
File "/home/times-67/rasa/rasa/nlu/training_data/formats/markdown.py", line 76, in reads
self._set_current_section(header[0], header[1])
File "/home/times-67/rasa/rasa/nlu/training_data/formats/markdown.py", line 195, in _set_current_section
"".format(section, "', '".join(available_sections))
ValueError: Found markdown section 'entity' which is not in the allowed sections 'intent', 'synonym', 'regex', 'lookup'.

Thanks for the question, our Github issue board is for bugs and feature requests only, though 😕
Please ask this question in the forum✨
If you were describing a bug, please create a report and fill out the template.


Getting Started

So i did the tutorial but i'm still confused about how i download a copy of the game to work on.
-Stephen Docy

Ok i downloaded the game and got it working so now i'm gonna try and work on a grenade counter to be used for a high score.


Json serializer does not support on redis?

Hi,
I use cache manager with redis on mvc project, when I add serializerType Json attribute on config (for some performance concerns), nothing writes to redis. I realized that CacheManager.Serialization.Json package missing and I added it but did not work again.
Json serializer does not support on redis?
<managers>
    <cache name="xxx" updateMode="Up" enableStatistics="true" enablePerformanceCounters="true" serializerType="Json">
       <handle ref="redisCache" name="redisConnectionString" expirationMode="Sliding" timeout="2h" />
    </cache>
</managers>

It causes LoadConfiguration method, when add serializerType this method throws exception.


translateX, translateY, rotateX, rotateY, etc.

I noticed that in the input example (http://lvivski.com/anima/example/keyboard_mixed.html) you have code to make sure that only one axis is affected at a time:

a.translate = translate.map(function(t, i) { return t || a.translate[i] })

It's a functionality that I often need when I'm coding for user interaction. It might be nice if that were built into the library so that a user wouldn't have to do it themselves each time they wanted to affect only a single axis. If there was the option to translate, rotate, scale, etc. for each axis (translateX, scaleY, rotateZ, etc.), it could really help.

I've never actually designed anima for interactive animations (when you change them based on user input) but I like the idea. I think I can modify existing .translate() .rotate(). scale() methods to support a second argument as dutation. So you'll get the same API, but if you provide a duration it would modify existing or add a new animation.

Thanks for the response. Actually, my point wasn't about duration, it was about the ability to isolate one axis instead of having to pass all the values at once. The way it currently is, you have to calculate the value for each axis, making sure not to change ones that don't need changing. If you could just change one axis without affecting the others at all, that would be something I would find very useful.


Providers Setup Section

Please update the contribution.md guide to include a reminder for developers to follow the steps for creating an app for each service provider before proceeding with the setup. This update should reference the detailed instructions available in the Provider Configuration section.
Proposed Change
Add the following reminder to the Providers Setup section:

Providers Setup
Before proceeding with the setup, please ensure you have created an app for each service provider. Detailed instructions can be found in the Provider Configuration section.
Create files from templates that contain secrets for specific providers. If you don't want to set up certain providers, you can leave their values empty.
# In apps/sample-app/android
cp local.properties.sample local.properties

# In apps/sample-app
cp .env.sample .env

Rationale
This reminder will help developers avoid setup issues by ensuring they complete the necessary steps for provider configuration. It will make the setup process smoother and more efficient.

Hey @dzuluaga, thanks for bringing this to our attention! This has been addressed and should be fixed in https://github.com/openmobilehub/react-native-omh-auth/pull/70/commits/04a42c530620a9674eaa4b13dfc351ebbbdb67ca.

LGTM. Thanks.


Publishing with a relative symlink

Changes the publishing symlink to be relative instead of absolute. This update fixes publishing which doesn't work when the deployTo option is a relative path.
Relative links are better if the directories they target are moved or mounted elsewhere for example.

@Jaconil I see, but tests are not passing, and we don't need to expose relativeReleasePath

@neoziro I made some changes following your comment ;)

@Jaconil good thanks


Handle renderer options size and webgl in bokeh

Adds support for global scaling of bokeh plots via size output magic option and setting webgl option based on global renderer.

There are really three separate changes here but they all look good. Merging.


ApplySecret: Don't erase injected Data for ServiceAccounts

ApplySecret was fighting with kube controller manager - service account token controller when used for service accounts. It was constantly deleting the injected data. This PR prevents it.
/cc @deads2k
needed for https://github.com/openshift/cluster-kube-apiserver-operator/pull/663

proof PR in https://github.com/openshift/cluster-kube-apiserver-operator/pull/663

/lgtm


Update remix plugin to latest version 1.7.5, support for remix stacks

Can we give the remix plugin some attention and update to the latest release
Release v1.7.5 · remix-run/remix (github.com)
Also add support for remix stacks

We will. The Remix plugin is still a labs plugin so that's why we cannot dedicate as much time as we do to other plugins atm.
Also note that you can update the remix version by yourself, shouldn't have an impact on the actual correct functioning of the plugin.

Also add support for remix stacks

That's something we've been thinking about. Ideally though you would be able to pass existing Remix stacks templates to the Nx plugin generator and it would just proxy stuff through. Otherwise we need to maintain the stacks which we def don't want to.

Dear @juristr ,
is there an ETA on the updated @nrwl/remix plugin? Or an ETA when to declare it as an official plugin?
I just fiddled around a bit, and i really like it. However, i noticed a few things:

app generation misses a stack property (as mentioned above)
update to the latest version of remix (currently 1.9.0)
i had to manually add npm i remix, as the generator did not add this dependency
the generator does not add proper targets to the project.json file of the generated app. I.e., i cannot run npx nx serve demo to start the app. I need to start the dev/prod server via the package.json file within the app folder. This seems a bit "alien" in this case as it does not feels like working with a real nx plugin

All the best and thanks for your hard work on making nx better every day :)
Johannes


feat: adjust header to design

Check https://386.hub.asap.science/.storybook/?path=/story/pages-team--about-tab on mobile landscape and tablet landscape - Last updated wraps which is not ideal. Other than that LGTM :sweat_smile:


Variadic Macros

https://gcc.gnu.org/onlinedocs/cpp/Variadic-Macros.html#Variadic-Macros

Added in #41


Draft: updating protobuf modules

Updating protobuf libs used in he project.

Enables building code on M1 macs
Updated protobuf used in builds

Combined with #26


ci: use public github ci job to verify pr title

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


VC++ compatibility?

Can I build GLFFT in a way that lets me use it in my VC++2010 app?

I made a fork with VS project files: https://github.com/ActuallyaDeviloper/GLFFT

@ActuallyaDeviloper Great, thanks!


Missing one warning in the error list after changing the property from "inner" to "outer"

OS: Win10/Mac
Build Version: 20200731.2
Repro Steps:

Open the nested-sql-server.json file in VS Code.
Change the following property from "inner" to "outer":
"expressionEvaluationOptions": {  	"scope": "inner"  },
Check the errors.

Expect:
Display the following errors:

Actual:
Missing one warning in the error list.

Fixed by #894


[FEATURE REQ]Add throughput control support in gateway mode

Add throughput control in gateway mode as well.

Thanks for opening this issue. In case this is related to our case<PHONE_NUMBER>000494, just so that you know it's OK for us to use direct mode for now. In the long run, this feature could be useful for Microsoft to support other institutional clients where gateway mode is more desirable and they also want to do some throughput control.

@stevenylai  - agree, we should add the throughput control for gateway as well. Thanks for letting us know that you are able to use direct mode and at least unblocked currently :)


Fix 'Transfer-Encoding: chunked' bug when upload an empty file

Though we have the stdout (which is also a fileno type with length=0) to care about, when we post or put a normal empty file, we still can not force to set the Transfer-Encoding to chunked.
I think it is clearly meaning that "I don't want to transfer the file in chunked type", when someone set the Content-length=0 in headers. requests does too much in this case, and it will bring out some problem when requesting some servers which not support chunked type. The Transfer-Encoding parameter should be user's response to add into the headers.

I am :-1: on this change.
The reality is that I don't care about servers that don't support chunked transfer encoding. RFC 7230 Section 4.1 says:

A recipient MUST be able to parse and decode the chunked transfer coding.

And this is hardly a new requirement. RFC 2616, the original HTTP/1.1 specification that was published nearly 15 years ago (in 1999), says in Section 3.6.1 that:

All HTTP/1.1 applications MUST be able to receive and decode the "chunked" transfer-coding

This means that any server that cannot handle chunked transfer encoding is either a) a HTTP/1.0-only server, which is incompatible with requests (a HTTP/1.1 library); or b) deliberately non-compliant with the now fifteen year old HTTP/1.1 specification. Requests has no duty to bow to the whims of server developers that chose not to support 15 year old protocols or to ignore specifications when they do so, or to operators of websites that are stuck in the 1990s.
And even if that were not true, your reasoning about chunked transfer encoding:

The Transfer-Encoding parameter should be user's response to add into the headers.

is not now and has never been how requests operates. The requests team has quite consistently told users not to add their own content-length or transfer-encoding headers to the headers dictionary, because requests will ignore them.
The upshot is that sending an empty chunked body is far and away the nicest way to deal with this situation.


Add print_layer_shapes function

This adds a new utility function print_layer_shapes that can be used to print the output shape of each layer in a model.
I am not sure this is the most efficient way to do it, but it seems to work.
For example :
from keras.models import Sequential
from keras.layers.core import Layer, Dense, Dropout, Activation, Flatten, Reshape
from keras.layers.convolutional import Convolution2D, MaxPooling2D
from keras.utils.model_utils import print_layer_shapes

model = Sequential()

model.add(Convolution2D(32, 1, 3, 3, border_mode='valid'))
model.add(Convolution2D(32, 32, 3, 3, border_mode='valid'))
model.add(MaxPooling2D(poolsize=(2, 2)))
model.add(Flatten())
model.add(Dense(32*12*12, 128))
model.add(Dense(128, 10))

print_layer_shapes(model, (1, 1, 28, 28))

Will print
input shape :  (1, 1, 28, 28)
shape after Convolution2D : [ 1 32 26 26]
shape after Convolution2D : [ 1 32 24 24]
shape after MaxPooling2D : [ 1 32 12 12]
shape after Flatten : [   1 4608]
shape after Dense : [  1 128]
shape after Dense : [ 1 10]

It looks like this only works for Sequential models, any chance this could also be made to work with a Graph?
Also, a significantly harder and more niche request: how about handling unknown input shapes by propagating symbolic names for those dimensions (e.g. make the input dims SymPy symbols with names like "x", "y", "z", "w" or "input0".."input3" and build up algebraic expressions for dim calculations that depend on them).

Definitely useful for debugging.
Indeed, if we were to merge this it would need to be generic, i.e. work for any model (including Sequential models that include Merge layers, and Graph models).

@iskandr This is an interesting suggestion, but I think you usually know your input shape. For example in a CNN, even if the Convolution2D layer doesn't have to know the size of the images, the Dense layer that follows will have the image size specified. Do you have an example where having algebraic dimensions would be useful ?

@fchollet I added a separate function that works for Graph model. For Graph models, it would be nice to be able to print a computation tree for each output. But it seems there is currently no generic way to walk the graph. For example, for a given layer l, its parent is at graph.nodes[l.previous.name] but :

Not all layers implement set_name (Merge doesn't implement it for example)
Layer.set_name doesn't set self.name but just modifies the parameters.

Would it make sense to fix this by having all layers implement set_name and add a Layer.name attribute ?

@julienr I was thinking this would be useful for debugging shape mismatch errors, especially those arising from incorrect assumptions about reshape/flatten/embedding. Instead of tracking constants through the computation graph, it might be easier to see that "wait, I didn't expect 'height * width' to come out of that node".
It might, however, be sufficient just to see the all the constant shapes (bypassing the complexity of building up symbolic computations).

I had worked on this, as a part of my dimensionality inference code. I think it can be solved by having an optional attribute called input_shape  and output_shape  for each layer. When a new layer is added, it can get input shape to it by get_input_shape  method, which fetches the output shapes of all previous layers. And then we can set layer's output_shape after performing the calculations, which can be subsequently used further. This will create a recursive, on demand call, hence no symbolic stuff necessary. This can be used in graphs also, with a minor change.
I've the code to calculate the output shapes of almost all layers. But, I'm really busy, so I can share it with anyone willing to work on it.
Also, this is a much simpler way to help the user in this regard, without modifying the API significantly. So :+1:
On Thu, Jul 23, 2015 at 10:15 PM, Alex Rubinsteyn
<EMAIL_ADDRESS>wrote:

@julienr I was thinking this would be useful for debugging shape mismatch errors, especially those arising from incorrect assumptions about reshape/flatten/embedding. Instead of tracking constants through the computation graph, it might be easier to see that "wait, I didn't expect 'height * width' to come out of that node".
It might, however, be sufficient just to see the all the constant shapes (bypassing the complexity of building up symbolic computations).
Reply to this email directly or view it on GitHub:
https://github.com/fchollet/keras/pull/416#issuecomment-124164388

@pranv I think what you propose is close to what Lasagne does with its lasagne.layers.Layer.get_output_shape. This is a great approach, the only downside is that this require duplicating the whole shape computation in keras. But it might be worth it as it makes it easier than my solution to get the shape of a specific layer.

Merged. Had to rewrite it almost completely :( https://github.com/fchollet/keras/blob/master/keras/utils/layer_utils.py#L88

@fchollet Much better after rewrite :) Thanks


Doesn't take healthcheck-protocol annotation

Using 1.0-alpha.8
Unable to set healthcheck-protocol to HTTP when backend-protocol is HTTPS.
Zarrar

Can you give me an example ingress definition of what you are trying to do?


sequencer: concurrency retrieve balances for each asset in check_balance_for_total_fees_and_transfers

an optimization, see https://github.com/astriaorg/astria/pull/1196#discussion_r1662247215

This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue
be closed in 7 days.

This issue was closed because it was stale


Bump Mixin version to 0.7

This PR bumps the stable-5 Mixin version to 0.7 and resolves incompatibilities which arise as a result.

I'm not going to review this until @Mumfrey lets me know that 0.7 is fine for general use (I'm aware it is released).

I'm not going to review this until @Mumfrey lets me know that 0.7 is fine for general use (I'm aware it is released).

Yeah I'd actually recommend going to 0.7 for general stability and feature improvements.

@caseif I have backported the update to Mixin 0.6.15 for now to fix https://github.com/SpongePowered/SpongeVanilla/issues/338 which has caused conflicts in your PR. Would you mind fixing them?
I personally think we should do the update to Mixin 0.7 in bleeding first before we backport it to the stable branches eventually.

Should be resolved now.

Thanks @caseif for the PR but Minecrell went ahead and took care of this among some other house cleaning.


API calls should all focus around a dot ID

The API calls centre around the concept of a branch. Any action that would function at the dot level - e.g. AddCollaborator - take the MasterBranchID to reference which "dot" we are working on.
While technically a dot may be identifiable via its master branch ID, someone using the API will think in terms of the basic unit of usage, which is the dot. Forcing them into a mindset of, "just use the master branch" violates the "principle of least surprise." It makes it harder for developers to adopt.
Perhaps more importantly, it locks us into the "branch as central unit" mindset, which is distinct from how customers will think about it.
Even if every dot's ID "happens" to match the master branch ID, we need them to be distinct concepts... and to require a dot ID on everything that operates at a dot level.
/cc @lukemarsden per our discussion earlier today.

Marking this as a feature rather than a broken window.

Marking this as a feature rather than a broken window

No, it isn't a "broken window".
I think, though, that "feature" vs "broken-window" may be too extreme a dichotomy. This isn't requesting a new feature to so we can appeal to new customers, as much as changing the UX so it can be adopted and understood by the broader market. We also don't want it to sit for too long, as fundamental API interfaces are much harder to change.
Do we need a new tag, perhaps, "product" or "fundamental" or something similar?

Note: Check if the master branchID is always a stable ID in cases such as divergence resolution.


Update demo

[x] Update CMake
[x] Use GitHub actions

Travis fails, but GitHub actions work as expected: https://github.com/ENCCS/gtest-demo/actions/runs/588932471
This is ready for review.

Thanks! I will remove the Travis part if it still needs to be removed.


Packaging opentrons API

This PR:

Refactored packaging and information
Upgraded pyserial to 3.2.1 as the result of conda packaging process
Converted README from md to rst
Added versioning using tags using versioneer
Publishing tagged builds into PyPI (https://pypi.python.org/pypi/opentrons/) and Conda (https://anaconda.org/opentrons/opentrons)
Updated setup documentation with Anaconda Navigator experience

Changes Unknown when pulling be2e0064810d268b4fdc3823e9513ac8fccc7a9d on 194-l-publishing-in-pypi-and-conda into ** on master**.

Coverage decreased (-3.8%) to 90.793% when pulling a7a923db279988d051e1f1cf275f46b205ba4af6 on 194-l-publishing-in-pypi-and-conda into a9305e85e149cc3168754520b811a162fd8523fd on master.

Coverage increased (+0.004%) to 94.589% when pulling cb0f0c1f9adc9e02f99f79e78a62daa79590b92f on 194-l-publishing-in-pypi-and-conda into a9305e85e149cc3168754520b811a162fd8523fd on master.


Charts Integration testing framework

Investigate helm test module for unit testing
Investigate adding integration testing into the CI in charts repo

official docs: https://helm.sh/docs/topics/chart_tests/
base example: https://github.com/bitnami/charts/blob/master/bitnami/wordpress/templates/tests/test-mariadb-connection.yaml

Likely bumping this to 2.5.7 timeframe.

Likely bumping this to 2.5.7 timeframe.

Need to ensure the testing stuff makes its way into charts-build-scripts and resolves https://github.com/rancher/charts-build-scripts/issues/30.

@bmdepesa fyi during hack week we built a solution in https://github.com/aiyengar2/charts-testing that integrates with our existing scripts in rancher/charts (at the time we built it; we'll need to make slight modifications now due to a new repository structure) to set up a Go "unit" testing framework for Helm charts. There is more information on what this framework is in https://github.com/aiyengar2/charts-testing/blob/dev-v2.5-source/tests/template/tests/README.md and example tests that could be written in this way in https://github.com/aiyengar2/charts-testing/tree/dev-v2.5-source/tests/template/tests and https://github.com/aiyengar2/charts-testing/tree/dev-v2.5-source/tests/common.
At the time, the way we defined "unit" testing on a Helm chart was a test you could run on the manifest outputted by a helm template command on one or more values.yamls associated with the chart, which differs from integration testing that was defined to be a test you could run on a chart that has been deployed onto a Kubernetes cluster.
So while that might not completely satisfy the goals of this issue (e.g. the solution built would not be able to test something like ensuring an operator is creating resources that are Windows compliant, which is something that only integration testing could test), there's definitely some overlap with the type of automated testing we could do with this framework that might fit in nicely with what QA is working on for this issue.
Issue tracking integrating this with rancher/charts: https://github.com/rancher/charts-build-scripts/issues/30


Old version of the Rust library on crates.io

The version on the Rust package repository crates.io is from 2018: https://crates.io/crates/open-location-code
It would be good to publish a newer version once #555 lands

Someone will need to take over that package and push a new version.

The package is owned by @JamesFysh so the easiest thing would be for them to transfer ownership. Rust packages ownership is based on GitHub accounts / teams, so you wouldn't have to create a new account for it.

Happy to transfer ownership, just let me know who to.


Diarization

Hi!!
First of all great library!
So thanks for that!
Second, is there an API  for diarization?

No, as far as I'm aware whisper.cpp (which this package is based on) doesn't support diarization itself.
There seems to be an open issue here, but nothing has been officially added to whisper.cpp.


Some Missing Movies either not detected or are detected but not added to UI/RSS

Describe the bug
I have identified multiple instances where I have multiple movies missing from a collection. Some will show in the UI but other will be listed under missing on the movies showing but do not have their own entry in the UI or the RSS feed. Hard to explain, below is a screenshot

So Garage Sale Mysteries: Murder In D Minor is noted as missing and is displayed in the UI/RSS feeds. However Garage Sale Mysteries: Searched & Seized is noted as missing but does not have it's own entry in the UI/RSS feeds. On a number of these instances it appears the movie that does not have a UI/RSS entry is not a released film, they are all planned/post-production etc... so it makes sense not to include those. However I have a bunch where the missing movie is actually released. Listing them below.
Garage Sale Mysteries: Searched & Seized
Tales from the Hood 3
Captain Underpants: Epic Choice-o-rama
The SpongeBob Movie: Sponge on the Run
One Cut of the Dead - Mission: Remote (Foreign)
Klovn the Final (Foreign)
The Amityville Harvest
Skylin3s
Father of the Bride Part 3 (ish)
Tremors: Shrieker Island
Deep Blue Sea 3
I also have two movies that are missing that don't show up on the UI/RSS at all. I assume because I already have all the other movies from their collection, if one of the others was missing I assume they would show just as the above ones do.
Aurora Teagarden Mysteries: Heist and Seek
Aurora Teagarden Mysteries: Reunited and It Feels So Deadly
While making this list I've noticed it appears that the ones missing all released in 2020. It appears it may be keying off release date greater than or equal to year 2020 and not adding, instead of Status: Released.
To Reproduce
Steps to reproduce the behavior:

Add a movie to library which has a collection item released in 2020
Run Scan

Expected behavior
Any movies from a collection that are Status: Released to be present in the missing UI/RSS.
Screenshots
Attached above.
Desktop (please complete the following information):

OS: [e.g. iOS] Docker on Ubuntu 18.04
Browser [e.g. chrome, safari]
Gaps Version [e.g. 22] v0.8.4

Additional context
Add any other context about the problem here.

Here is a set of logs from last nights scan.
docker01_log.zip

Thanks for all the information. I'll check it out soon.

Any progress on this? We've moved into 2021 so not sure if it's going to affect even more releases now if it's a date based issue?

Any progress on this? We've moved into 2021 so not sure if it's going to affect even more releases now if it's a date based issue?


Not connecting to Wi-Fi in GoPro 8Black

Problem:
Hello, I'm trying to use the endpoint
http://<IP_ADDRESS>/gp/gpControl/command/wireless/ssid/select?ssid=WIFI_SSID&auth_type=AUTH_TYPE&pw=SSID_PASSWORD
And I get this error
{
"version": "6.00",
"path_info": "command/wireless/ssid/select",
"query_string": "ssid=XXX-XXXX",
"error_code":-3,
"error_msg": "500 Internal Server Error
",
"function": "gpcontrol_cgi_handler",
"line": 190
}

I've double checked that the SSID and Password are the right ones and the Auth too after doing
http://<IP_ADDRESS>/gp/gpControl/command/wireless/ssid/scan?p=1 and
http://<IP_ADDRESS>/gp/gpControl/command/wireless/ssid/list
Can I get some help please?
Details:

GoPro Camera(s): 8 Black
Firmware Version: HD<IP_ADDRESS>.00
Steps to reproduce: Connect to GoPro hotspot and hit the endpoint with the valid SSID - AUTH - PW
Happens every time? [Y/N]: Y

Hi, I got a similar problem.
When I want to start a recording session with
http://xxx.xx.xxx.xx/gp/gpControl/command/shutter?p=1
I get the same error
{
"version": "6.00",
"path_info": "command/shutter",
"query_string": "p=1",
"error_code":-3,
"error_msg": "500 Internal Server Error
",
"function": "gpcontrol_cgi_handler",
"line": 190
}

I've connected my camera via the new GoPro connect feature.
Details:

GoPro Camera(s): 9 Black
Firmware Version: HD<IP_ADDRESS>.00
Steps to reproduce: Connect camera via GoPro connect/USB and send the http request.
Happens every time? [Y/N]: Y

I already found out that this problem is related with the newest GoPro Firmware update from the 17. Dec 2020, because this error occured for the first time after I updated the camera to the latest version. In the previous version everything worked as intended.
So I see that @fcaraballo is also using the latest version of the firmware and I think GoPro messed something up with this new version. I asume that this problem only can be solved by GoPro themself but I am really thankfull for every help we can get.


update pfm

tidy
bump and tidies

hey thanks for the PR @faddat, could you get this updated with recent changes to main? feel free to ping me afterwards and we can get this merged!
Also I think you meant async-icq since the pfm is not in the repo (yet)

Sure thing.  For whatever reason I sometimes confuse the names async-icq and pfm....  Should be good to go now.


Custom macro expansion issues in falco_rules.local.yaml

Greetings,
I an trying to define some custom conditions in the user_read_sensitive_file_conditions macro. That macro is defined in falco_rules.yaml and I'm overriding it in falco_rules.local.yaml.
To make my YAML file cleaner, I'm defining the following macro:
- macro: puppet_access_sudoers
  condition: proc.name=puppet and proc.cmdline="puppet /usr/bin/puppet agent" and fd.name startswith "/etc/sudoers"

Then, I'm using the puppet_access_sudoers in the override of user_read_sensitive_file_conditions:
- macro: user_read_sensitive_file_conditions
  condition: puppet_access_sudoers

The result is the following error:
Fri Jan 25 22:02:20 2019: Runtime error: Error loading rules: /usr/share/falco/lua/compiler.lua:78: Undefined macro 'puppet_access_sudoers' used in filter.. Exiting.

The puppet_access_sudoers is defined above the override user_read_sensitive_file_conditions, but the Lua compiler fails to find it.
A workaround is the following:
- macro: user_read_sensitive_file_conditions
  condition: proc.name=puppet and proc.cmdline="puppet /usr/bin/puppet agent" and fd.name startswith "/etc/sudoers"

It works, but it will be more complicated to maintain. :)
Thanks for your help!

@mstemm any thoughts on why the Lua compiler isn't working for this macro?

Hello! I think we need this one because it will allow users to make their rules cleaner 🙂

Ok, reopening.

Related to #706

I think we can close this since it had been detailed a lot more in #706 - wdyt @JPLachance  ?

Have fun! :)


[Automation] Shellscript Linting

Proposed changes
Shellscript Linting should adhere to known standards. Look into:

github.com/koalaman/shellcheck

This is already implemented in Go and needs to be ported over:
https://github.com/deepgram/deepgram-go-sdk/pull/208
Context

Why is this change important to you? How would you use it? How can it benefit other users?

Possible Implementation

Not obligatory, but suggest an idea for implementing addition or change

Other information

Anything else we should know? (e.g. detailed explanation, related issues, links for us to have context, eg. stack overflow, codepen, etc)

We don't have any shellscripts :/

I will open a PR soon that will introduce some, but if we can add some of these things without shell scripts, we can close.
Reference to this:
https://github.com/deepgram/deepgram-js-sdk/issues/284


Provide the integration of tools for monitoring the application metrics

Problem description
Clients should be able to integrate the guardian with external monitoring tools such as Prometheus, Graphana, New Relic, and so on, open source or not.
Requirements
We need to provide the documentation necessary in order to allow the client to integrate the Guardian application with a monitoring tool in order to extract metrics about the application including but not limited to:  active requests, CPU usage, event loop lag, active handlers, external memory, heap size (total, available, used), process memory.
Definition of done
The possibility of integrating The Guardian application with external monitoring tools that will allow us to track the application metrics such as active requests, memory, heap size ,and so on.
Acceptance criteria
All documentation available to integrate the chosen monitoring tool with docker, and/or all configuration files available internally to provide integrations with at least one of these technologies.
Graphana example:

This is not necessary at the moment.


RAYLIB_VERSION

RAYLIB_VERSION is defined in two source files:
#define RAYLIB_VERSION  "3.7"

but I can't access it directly by including raylib.h. Is there another way to find the current version at compile time? Can you add RAYLIB_VERSION definition to raylib.h?

@pauljurczak Actually, it was not in raylib.h for some reason I can't remember now... In any case, I just moved it to raylib.h.


dhcp: SV Test to validate dhcp option 60 is being logged

Feature 4587
This PR adds a test to validate that dhcp option 60 is being logged in dhcp extended logging mode.
Redmine ticket: 4587

@inashivb, can I squash the commits and force push to this branch itself or should I open a new PR ?

Force pushing for now...happy to put my changes into a new branch and open a new PR if that's preferable.

That's always preferable. ;)

Force pushing for now...happy to put my changes into a new branch and open a new PR if that's preferable.

That's always preferable. ;)

Got it ! I'll create a new one.

Continued in https://github.com/OISF/suricata-verify/pull/1359


[MXNET-1443] Change gluon contrib base event handlers to mixins

Description
(Brief description on what this PR is about)
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes
incubator-mxnet/python/mxnet/gluon/contrib/estimator/event_handler.py
incubator-mxnet/python/mxnet/gluon/contrib/estimator/estimator.py
incubator-mxnet/tests/python/unittest/test_gluon_event_handler.py
incubator-mxnet/tests/python/unittest/test_gluon_estimator.py
incubator-mxnet/docs/python_docs/python/tutorials/packages/gluon/training/fit_api_tutorial.md

@szha Sounds good to me!


Move codespell to a separate GitHub Action

Contributors get too confused when typos are raised by the lint_python so let's make codespell its own GitHub Action.

Could you add paths to lint_python too, and change to name python

I would like to have only one GitHub action by programming language

I would like to have only one GitHub action by programming language


Make consume and ContextConsumer return undefined rather than null

Fixes #29
cc @kevinkucharczyk
I'm not sure I've done the change correctly - running npm lint locally seems to lead to a lot of TypeScript errors which I'm unsure how to fix. Please take a look and let me know if I've done something wrong.

@boris-petrov what TypeScript errors are you seeing? When I run it locally I see a few warnings:

But they were before - nothing related to your changes. There are no errors and the linting passes correctly.

Well, running npm run lint leads to these:
[lint:types] app/components/a-component.ts(2,25): error TS2307: Cannot find module 'ember-provide-consume-context' or its corresponding type declarations.
[lint:types] app/components/custom-provider.ts(2,25): error TS2307: Cannot find module 'ember-provide-consume-context'or its corresponding type declarations.
[lint:types] app/components/custom-provider.ts(3,34): error TS2307: Cannot find module 'ember-provide-consume-context/context-registry' or its corresponding type declarations.
[lint:types] app/components/custom-provider.ts(28,16): error TS2664: Invalid module name in augmentation, module 'ember-provide-consume-context/context-registry' cannot be found.
[lint:types] app/components/test-component.ts(2,25): error TS2307: Cannot find module 'ember-provide-consume-context' or its corresponding type declarations.
[lint:types] app/components/test-component.ts(3,34): error TS2307: Cannot find module 'ember-provide-consume-context/context-registry' or its corresponding type declarations.
[lint:types] app/controllers/application.ts(33,16): error TS2664: Invalid module name in augmentation, module 'ember-provide-consume-context/context-registry' cannot be found.
[lint:types] tests/integration/components/decorators-test.ts(7,34): error TS2307: Cannot find module 'ember-provide-consume-context' or its corresponding type declarations.
[lint:types] tests/integration/components/test-support-test.ts(8,8): error TS2307: Cannot find module 'ember-provide-consume-context/test-support' or its corresponding type declarations.
[lint:types] types/global.d.ts(3,47): error TS2307: Cannot find module 'ember-provide-consume-context/template-registry' or its corresponding type declarations.

And below there's a lot more. One of them:
[lint:types] app/templates/application.hbs:75:13 - error TS7053: Unknown name 'ContextProvider'. If this isn't a typo,you may be missing a registry entry for this value; see the Template Registry page in the Glint documentation for moredetails.
[lint:types]   Element implicitly has an 'any' type because expression of type '"ContextProvider"' can't be used to index type 'Globals'.
[lint:types]     Property 'ContextProvider' does not exist on type 'Globals'.
[lint:types]
[lint:types]  75             <ContextProvider @key="NumberContext" @value={{9}}>
[lint:types]                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[lint:types]  76               <BComponent />
[lint:types]     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[lint:types] ...
[lint:types]  79               </ContextConsumer>
[lint:types]     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[lint:types]  80             </ContextProvider>
[lint:types]     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

That doesn't happen when you clone the repo, npm i and then npm run lint?

@boris-petrov I am not seeing those on my machine! What versions of node and npm are you on?

Node v22.8.0, npm 10.8.3. I don't think these should matter in that case though.

@boris-petrov I can't seem to reproduce what you're seeing. You said you weren't sure if your changes are correct - does this mean running npm lint against the main branch works fine for you?

No, the same thing happens on main, that's why I can't be sure that my changes are correct. It's really strange that you don't see what I see. As I said, my setup is pretty ordinary - Arch Linux, latest Node/npm, but these things shouldn't really matter. But in any case - CI here is happy so the problem should be somewhere on my machine. Not sure if we can also add/change a test somewhere for this PR?

@boris-petrov

Not sure if we can also add/change a test somewhere for this PR?

We can definitely add some tests!
There are integration test under test-app:

a file for the built in components
a file for the decorators

It would make sense to add a test case in each to test what the result is when no provider is present.
Would you like to add those?

Sorry for the delay. I’ll try to do that end of next week.
Message ID: @.***>

@kevinkucharczyk I've added two tests for the two changes. I also found why I was getting these errors. Apparently one must build an addon before one can use it. :smile: Didn't know that. So once I ran npm run build I could get both linting and tests running fine.


Emoji V2

@emanuele45 I fixed this for SCE V2 but making the PR back to the original branch is a bit messy since I had to rebase this to current dev.   If you rebase your branch then I'll PR this to that, or just use this one ... don't care either way!
----- Original PR
@Spuds I picked your addon and stuffed in 2.0. 👼
Let me know what you think 😁
https://www.elkarte.net/community/index.php?topic=4695.0

Scrutinizer — Analysis: 2700 Issues, 34 Patches – Tests: passed Sweet Baby Moses !!  Unfortunately, the changeset was too large and we could not compute changes, but only the current state. And here I though I finally broke Scrutinizer 😇

Scrutinizer — Analysis: 2700 Issues, 34 Patches – Tests: passed Sweet Baby Moses !!  Unfortunately, the changeset was too large and we could not compute changes, but only the current state. And here I though I finally broke Scrutinizer 😇


feat(explore): Show confirmation modal if user exits Explore without saving changes

SUMMARY
This PR adds a browser confirmation modal (similar to the one that we have in dashboard edit mode) when user makes a change in control panel and then tries to close Explore (either by closing the tab or moving to another page) without clicking Save button.
The modal that displays when user navigates out of SPA context (some external page or a Superset page that hasn't been migrated to SPA yet) or closes the tab, is not customizable.
In the case of navigating within SPA context, we can easily display a custom message in the modal thanks to react-router's Prompt component. However, displaying a custom modal instead of the default one rendered by <Prompt /> requires more effort and will be handled in a separate PR.
CC @kasiazjc
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
https://user-images.githubusercontent.com/15073128/181581060-fc6c1408-9788-4864-8174-d5c8e9913d44.mov
TESTING INSTRUCTIONS

Open a chart and make some changes in control panel
Try to leave the page - a confirmation modal should appear
Save your changes (save as/overwrite/save & go to dashboard)
Now leaving the page should not trigger the confirmation modal

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

@kgabryje I'm not sure if this is fixable or if it was like this before, but I noticed that if you have unsaved changes and try to navigate to Dashboard, even if you click "cancel" the navigation indicator stays on Dashboard:

@kgabryje Also not sure if this was intentional, but I noticed that if you edit the chart title but clicking directly on the rendered title and edit it, the "Altered" tag appears but it doesn't block navigation away

@kgabryje Also not sure if this was intentional, but I noticed that if you edit the chart title but clicking directly on the rendered title and edit it, the "Altered" tag appears but it doesn't block navigation away.

That's an existing bug - you can try it by going to charts list, then, dashboards list, then click "back" - dashboards will still be highlighted

@kgabryje I'm not sure if this is fixable or if it was like this before, but I noticed that if you have unsaved changes and try to navigate to Dashboard, even if you click "cancel" the navigation indicator stays on Dashboard

That's an existing bug - you can try it by going to charts list, then, dashboards list, then click "back" - dashboards will still be highlighted

@kgabryje Also not sure if this was intentional, but I noticed that if you edit the chart title but clicking directly on the rendered title and edit it, the "Altered" tag appears but it doesn't block navigation away.

Oops, well spotted!

@jinghua-qa can we test what happens when the chart is running on a query? Does it show the correct save chart modal with the dialog to save a dataset?

/testenv up

in the confirmation modal, is it make sense to have a cancel option or close dialog option? Looks like now only have discard and save, as user i may need to think about double check my change before i save, but i dont have an option currenlty.

I also noticed that there are different confirmation dialog, and this one did not have save option, only cancel and leave, is this expected? are we trying to have same confirmation dialog modal when user leave explore without save?

in the confirmation modal, is it make sense to have a cancel option or close dialog option? Looks like now only have discard and save, as user i may need to think about double check my change before i save, but i dont have an option currenlty.

I think it makes sense! @kasiazjc wdyt?

I also noticed that there are different confirmation dialog, and this one did not have save option, only cancel and leave, is this expected? are we trying to have same confirmation dialog modal when user leave explore without save?

Yes - our custom confirmation modal is possible to display only when navgating between SPA routes. When you try to close the tab or go to an external page, a default browser modal appears - it's not customizable in any way (it's a browser security measure)

in the confirmation modal, is it make sense to have a cancel option or close dialog option? Looks like now only have discard and save, as user i may need to think about double check my change before i save, but i dont have an option currenlty.

I think it makes sense! @kasiazjc wdyt?

Good point, I'm thinking - maybe for consistency we should follow chrome flow actually instead of adding this one.
Copy:
Your chart is not saved
If you leave and don't save, changes will be lost.
[Cancel][Leave]
Thoughts?

I think it makes sense! @kasiazjc wdyt?

Good point, I'm thinking - maybe for consistency we should follow chrome flow actually instead of adding this one. Copy:
Your chart is not saved If you leave and don't save, changes will be lost. [Cancel][Leave]
Thoughts?

Do we have a final plan for this design? My personal feeling is will be great to have all, [Cancel][Leave][Save], if that is too complicated, i think i will vote for [Cancel][Leave] which is consistent with the chrome flow, and i can always cancel and do save on explore.

I think it makes sense! @kasiazjc wdyt?

Good point, I'm thinking - maybe for consistency we should follow chrome flow actually instead of adding this one. Copy:
Your chart is not saved If you leave and don't save, changes will be lost. [Cancel][Leave]
Thoughts?

Do we have a final plan for this design? My personal feeling is will be great to have all, [Cancel][Leave][Save], if that is too complicated, i think i will vote for [Cancel][Leave] which is consistent with the chrome flow, and i can always cancel and do save on explore.

I try to avoid adding 3 buttons as much as possible, as it can get quite confusing. After clicking "save" we would still have to open "save" modal, so I would say let's go with [Cancel][Leave] @kgabryje

I think it makes sense! @kasiazjc wdyt?

Good point, I'm thinking - maybe for consistency we should follow chrome flow actually instead of adding this one. Copy:
Your chart is not saved If you leave and don't save, changes will be lost. [Cancel][Leave]
Thoughts?

Do we have a final plan for this design? My personal feeling is will be great to have all, [Cancel][Leave][Save], if that is too complicated, i think i will vote for [Cancel][Leave] which is consistent with the chrome flow, and i can always cancel and do save on explore.

I try to avoid adding 3 buttons as much as possible, as it can get quite confusing. After clicking "save" we would still have to open "save" modal, so I would say let's go with [Cancel][Leave] @kgabryje

[Cancel] [Leave] makes the most sense to me too!  As a user I think I'd expect there to be an option to just cancel the close.

Updated with Cancel/Leave

Updated with Cancel/Leave

lgtm, thankss

/testenv up


adding support for specifying headers in ctx.onerror

https://github.com/koajs/ratelimit/pull/13

what can't be done with middleware? streaming errors? if anything, i'd prefer the .onerror handler to be configurable.

+1 to having .onerror configurable

another +1 for configurable

@jonathanong what would this look like? would we just extend the onErrors API to allow passing in headers (keys) that should stay in the response this.res._headers? is the end goal for the user to be able to specify what headers should stay, what the values of the headers that stay are, and/or just if headers should be unset or not?

i'm not sure. i'm not sure what the problem reported is

At the bare minimum I think the user should be allowed to keep the headers that have been set up to when the error has been thrown. Then, whether this should be a default behavior, or set as an option it's debatable. Imho should be default but other people might think differently.

Related https://github.com/koajs/koa/issues/571

i think i like https://github.com/koajs/koa/issues/571's solution better. if you agree @dead-horse , wanna close this?
otherwise, onerror could be configurable too. technically it should already be configurable - we just need to make it more user-friendly and document it.

SGTM


Version 1.9.0

openai 1.9.0
Destination channel: defaults
Links

PKG-3808
Upstream repository
Upstream changelog/diff

Explanation of changes:

Update to 1.9.0
Dependency updates

Was the 3.12 build not possible?

I forgot to add it back before putting this up for review. Looks like I need to update anyio first.
  The following packages are incompatible 08:06:06 INFO build     ├─ anyio <5,>=3.5.0  is installable with the potential options 08:06:06 INFO build     │  ├─ anyio 3.5.0 would require 08:06:06 INFO build     │  │  └─ python >=3.8,<3.9.0a0 , which can be installed; 08:06:06 INFO build     │  ├─ anyio 3.5.0 would require 08:06:06 INFO build     │  │  └─ python >=3.10,<3.11.0a0 , which can be installed; 08:06:06 INFO build     │  ├─ anyio 3.5.0 would require 08:06:06 INFO build     │  │  └─ python >=3.11,<3.12.0a0 , which can be installed; 08:06:06 INFO build     │  └─ anyio 3.5.0 would require 08:06:06 INFO build     │     └─ python >=3.9,<3.10.0a0 , which can be installed; 08:06:06 INFO build     └─ python >=3.12,<3.13.0a0  is not installable because it conflicts with any installable versions previously reported.


Freeze dependencies in CI

I don't have time to keep fixing unrelated issues in every PR because packages keep getting updated. This is dramatically lowering my velocity when working on blaze. If someone wants to own a dependency upgrade that would be great, but right now it just falls on whoever decides to write a fix for anything. I would also like to do this for odo.
ping @kwmsmith @stefanseefeld

@llllllllll agreed 100%.  I'm working on freezing the deps currently.  Working around the current build failures is slowing down development.
Because we have other projects that use Blaze with sometimes-overlapping dependencies, my preference is to use open-ended dependencies (i.e., pymongo<=3.0 rather than pymongo=3.0) unless constrained otherwise.

For the CI I think they should truly be frozen to make sure that we can all reproduce a test setup. Even minor patches could affect behavior in an unexpected way. I think for installs of blaze|odo|datashape we should use soft constraints. This is what we do in zipline and has been very successful. Basically the requirements are all specified as >=n, the ci runs on one particular frozen env, and then iff we discover that something breaks on a newer version we add an upper bound.

Is a "bump version bot" (like https://dependabot.com/) an option?


All labels should have descriptions

Problem

Currently the nixpkgs repository has 167 labels and most of them lack a description of what each of them mean. This is especially confusing with labels such as 8.has: package (new), 8.has: package (update) which aren't clear, even if you look at the list of issues.
Checklist

[x] checked latest Nixpkgs manual (source) and latest NixOS manual (source)
[x] checked open documentation issues for possible duplicates
[x] checked open documentation pull requests for possible solutions

Proposal
Every single label must have a clear and concise description on what they mean and what kind of issues / pull requests should have them.

This would be great to see and helpful for new maintainers.
For example:
At the moment struggling to workout if 12.approvals: 1 is anyone has commented saying the approve/tested the PR or if it's a maintainer that approves. A description to the label would help clarify.

At the moment struggling to workout if 12.approvals: 1 is anyone has commented saying the approve/tested the PR or if it's a maintainer that approves. A description to the label would help clarify.

"12.approvals: x" is based on hitting the approve button in Github's UI. If a maintainer approves, it will also add the "12.approved-by: package-mantainer" label. See #264633. I definitely agree that it should be documented.


Fix flaky test_quantization.test_get_optimal_thresholds

Description
Fix for #11456.
Checklist
Essentials

[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Changed dtype of input and raised tolerance level

Comments
Passed 10000 times on CPU:
python tools/flakiness_checker.py test_quantization.test_get_optimal_thresholds -n 10000
INFO:root:Testing: /home/ubuntu/incubator-mxnet/tests/python/quantization/test_quantization.py:test_get_optimal_thresholds
INFO:root:No test seed provided, using random seed
/home/ubuntu/anaconda3/lib/python3.6/site-packages/urllib3/contrib/pyopenssl.py:46: DeprecationWarning: OpenSSL.rand is deprecated - you should use os.urandom instead
  import OpenSSL.SSL
test_quantization.test_get_optimal_thresholds ... ok

----------------------------------------------------------------------
Ran 1 test in 66564.359s

OK

@marcoabreu

@reminisce can you have a look please

@marcoabreu Should be ready for merge.


You can no longer disable TLS route verification

You have not been able to since 2.4

Thanks, @ameowlia ! I'll go ahead and backport this through 2.7.


Pass -f to frontend/Dockerfile rm run so that missing files do not cause errors

Problem

It isn't possible to run the frontend Dockerfile build locally any more because it assumes that it is running in a context where the CI/CD workflow has copied certain files into the frontend directory. Because rm fails if files do not exist, when you run the build locally (assuming you didn't also needlessly copy the files in question), the build will fail.
Description

Add -f to this line in frontend/Dockerfile:
RUN rm ./frontend/.npmrc ./frontend/.pnpmfile.cjs ./frontend/pnpm-lock.yaml

should read:
RUN rm -f ./frontend/.npmrc ./frontend/.pnpmfile.cjs ./frontend/pnpm-lock.yaml

Alternatives
Change the way the CI/CD workflow runs so that it doesn't rely on these files being moved around at all. Then we can remove this CI implementation detail from the Dockerfile entirely, which ideally wouldn't know anything about it.
To do this, we probably need to more heavily leverage pnpm's workspace stuff, but I'm not sure. It's almost certainly a lot more work to untangle the assumptions there.

Historical context for why files are moved around in CI: https://github.com/WordPress/openverse/pull/446.
In short:

Nuxt doesn't work well without shamefully-hoist
pnpm deploy doesn't respect shamefully-hoist
Replicating the workspace-like layout inside Docker needs files from outside the Docker context
They can be brought into the context by cp before Docker build (which CI does)

A simpler approach, for now, might be to have a build-img recipe for the frontend that can do copy the files from the root, build the Docker image and then clean those files up once built.

Recently learned about additional build contexts that remove the need for this file-juggling, implemented #2048 to resolve this issue.


📃:  adding GSAP  animation library projects

:red_circle: GSAP animation :
:red_circle: Aim : Learn about animation libraries like gsap, three.js
:red_circle: Tech Stack Used : JS , GSAP
Can you add  Gsoc ext Level 2 to this project
Screenshots 📷

These all cards are animated and have a transition effect

:white_check_mark: To be Mentioned while taking the issue :

Full name : Mani ( GSOC ext )
What is your participant role?
( GSOC ext )

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney can you add gsoc ext level 2 or 3 to this


Issue with verifying Custom Datasets fields

During the development and with the code, there were some major issues with our own Formulars which we tried to connect and send data.
Until I noticed on RecordCommands.cs | Line 216 "  recordFields[f.FieldName] = f;"
It created only entries in the array with only the type of field instead of the Name of the field. Which makes it harder to distuingish the mulitple custom text fields.
I suggest to fix this issue only to change line 216 to recordFields[f.FieldLabel], which solved my issue on working with it.

Sorry for the late answer. Yes, you are right this code may not work with the custom record types.
It will be fixed in the next release v1.0.7


Refactor deleteFileByPath() function to check for the existence of a file before attempting to delete

The complete context of this is in this thread. Pull request #41 was approved not to postpone merging new features in code.
Still, it is suggested that the code review in PR be solved in this issue.

@JohnathonBowers,
I just opened PR #48. I would appreciate your review before merging.


Broken link to 159865

Hello, the link to https://docs.microsoft.com/EN-US/help/159865 is broken. Under the "The Target Disk" heading.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9fa148d7-c2ce-dc38-8da3-957bafa62683
Version Independent ID: 70e3cc32-72eb-524a-fb61-7f1b67eaa4db
Content: Troubleshoot the Event ID 50 Error Message
Content Source: WindowsServerDocs/storage/file-server/Troubleshoot/troubleshoot-event-id-50-error.md
GitHub Login: @Deland-Han
Microsoft Alias: delhan

Confirmed. The sad part is that the link in itself is not the culprit. At first, I thought it could be an issue that the link is language specific or that /en-us/ is entered in CAPS (/EN-US/). That is apparently not a problem. The target page is lost and gone), the page appears to have been removed, which makes it a distributed issue because the link exists in multiple pages around in the microsoft.com universe.

See also https://support.microsoft.com/help/159865/ (Sorry, page not found)
404

Browsing through some old Microsoft Forum pages, it appears the KB article must have dated back to 2007 (Windows Vista) or earlier, people have asked for Windows Vista and Windows 7 support related to KB 159865, but seemingly not received any. It is most likely removed, deleted and never again revived.
The best course of action here is to completely remove the sentence referring to it, and seek other venues of support for this type of issue, like PowerShell or WMI.

Search results limited to site:microsoft.com :
https://www.bing.com/search?q="How+to+Distinguish+a+Physical+Disk+Device+from+an+Event+Message"+site%3Amicrosoft.com&qs=n&form=QBRE&sp=-1&pq="how+to+distinguish+a+physical+disk+device+from+an+event+message"&sc=0-65&sk=&cvid=2CB38ADF89AC4EAF834998F940C0A21E

There you go. PR #4973 to remove the text and the broken link is now available for review, comments and suggestions.


BungeeChannelInitializer errror

What is the output link of /viaversion dump?
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
Custom bungeecord
How does this error happen? login? Using an item?
When force close channel before initilization
Is there an error in the console? Use pastebin.com. Is there a kick message?
https://pastebin.com/h86Lj6KM
Changed PipelineUtils: https://pastebin.com/PKxkytUr LINE: 52-55
ConnectionDropper: https://pastebin.com/JQLtV1UF

Either change the injection order, or just run the code to add the handlers and close gracefully.
This isn't really a ViaVersion issue.


[Feature] Text Comparison Node

Hello, it would be handy to have a text comparison node. I hacked together a real simple one that seems to work just based off python operations but don't really think its worth a pull request especially if I'm missing a node that already does something like that.
Thanks.

That's a good idea. I don't have a text comparison node that I remember making. Maybe it could have modes. Absolute comparison and similarity.

I am already working on a similarity function. I'll get that node in here in a few

That would be great to see implemented. I was playing around with dynamic prompts and creating folders and stuff based on the prompts and not having text comparisons made it way more messy than it should be. Love these extra nodes so far though.

How does something like this sound:
def similarity(str1, str2, threshold=1):
    m = len(str1)
    n = len(str2)
    dp = [[0 for x in range(n+1)] for x in range(m+1)]
    similar_words = set()
    for i in range(m+1):
        for j in range(n+1):
            if i == 0:
                dp[i][j] = j
            elif j == 0:
                dp[i][j] = i
            elif str1[i-1] == str2[j-1]:
                dp[i][j] = dp[i-1][j-1]
            else:
                dp[i][j] = 1 + min(dp[i][j-1],      # Insert
                                   dp[i-1][j],      # Remove
                                   dp[i-1][j-1])    # Replace
            # Check for similar words
            if dp[i][j] <= threshold and i > 0 and j > 0:
                word1_start = max(0, i-dp[i][j])
                word2_start = max(0, j-dp[i][j])
                word1_end = i
                word2_end = j
                while word1_start > 0 and str1[word1_start-1] != " ":
                    word1_start -= 1
                while word2_start > 0 and str2[word2_start-1] != " ":
                    word2_start -= 1
                while word1_end < m and str1[word1_end] != " ":
                    word1_end += 1
                while word2_end < n and str2[word2_end] != " ":
                    word2_end += 1
                word1 = str1[word1_start:word1_end]
                word2 = str2[word2_start:word2_end]
                similar_words.add(word1)
                similar_words.add(word2)
    similarity_score = 1 - (dp[m][n]/max(m,n))
    return (similarity_score, list(similar_words))

str1 = "Hello world! How are you doing today?"
str2 = "Hello starshine! I am doing well!"

from pprint import pprint
pprint(similarity(str1, str2, 1), indent=4)

Which would give us an output like this:
(   0.4054054054054054,
    ['world!', 'Hello world!', 'starshine!', 'Hello starshine!', 'Hello']  )

Cool. Probably may be helpful to have an opposite mode (a difference between strings) as well.

It's added. Let me know if there are any issues.


Doubts on working of Netflix Conductor

I am totally new to Conductor and experimenting with it to orchestrate a workflow, but have doubts regarding following topics:
A) What is role of inputKeys and outputKeys in Task metadata definition ? Why we used them?
B) What is role of Input Template in Task Meta data defintions ?
C) When writing a Workflow definition, it contains Task definition as JSON object. Task's JSON object contains two keys i.e. input parameters and output Parameters. How they can be used ?
D) Can any person provide me a link to a working example on git to understand its working.

A) inputKeys are input keyNames the workflow is expecting, I believe outputKeys have a similar purpose. You need inputKeys for defining what inputKeys the workflow needs to expect while being triggered
B) Not sure what you mean.
C) inputParameters and outputParameters are used to essentially provide a way to exchange information between tasks. If task3 wants to use task2's output, then you need assign the task2's output using JSONPATH within the inputParameters of task3 definition.
D) Have you taken a look at the kitchenSink Example within the codebase.

As @saidatta said, I believe kitchensink example is the best way to go.  It helped me understand data flows between tasks and how input and output parameters work.


Story Television (WJLP-DT7)

Source
https://www.tvpassport.com/tv-listings/stations/story-network/37281
Channels
WJLP-DT7 (Story Television)
Notes
No response

You could use WCIUDT5.us that's already in there. They're both on O&O sticks, carrying the full schedule.


Created createAmplitudeFromStream

Moved amplitude logic from createAmplitudeStream to createAmplitudeFromStream So it can be used with third party MediaStream. Also removed audio sample rate so it will defaults to device's preferred sample rate.

Before merging, please run pnpm run changeset for a minor update with a change log entry.


Address bounce castle cve alert

chore: Update bouncy castle depdendency to address the following vulnerability:

CVE-2023-33201: Bouncy Castle For Java LDAP injection vulnerability

Motivation
Modifications
Result

@spolti we're on java 17, does bcpkix-jdk18on work ok with that?

Hi @njhill, 18on is for JDK 1.8 and later: https://www.bouncycastle.org/latest_releases.html

@spolti aah nice, I thought it might be 18 not 1.8 lol


Add compatibility tests for implicit (un)tupling in patterns.

Resolves SR-11212. See the issue description on JIRA for an explanation behind the test cases.

@swift-ci please smoke test

@swift-ci please smoke test

So, hang on, if these all already work, what's different about SR-11160?

Good point! Case 1
switch Optional<(Int, Int)>((5, 6)) {
case .some(let b): print(b)
case .none:        print(-1)
}

compiles fine but the supposedly equivalent code Case 2
switch Optional<(Int, Int)>((5, 6)) {
case let b?: print(b)
case .none:  print(-1)
}

does not.
The reason this is happening is because the pattern decomposition is incorrect (this is the 3-line fix around conArgSpace in the SR-11160 PR):
_: Optional<(Int, Int)> --> DISJOIN(.none | .some(_: Int, _: Int)) // wrong on master

On master, there is another bug (which I just found out during this investigation), where projecting .some(let b) gives .some(_: Int, _: Int). I will fix this in the SR-11160 PR (turns out, I ended up changing something else, which was wrong).
In Case 1, the two bugs cancel out as the overall operation becomes
_: Optional<(Int, Int)> - DISJOIN(.none | .some(_: Int, _: Int))

In Case 2, the pattern that gets formed for let b? is the correct one (because the desugaring does the right thing) so you end up subtracting DISJOIN(.none | .some(let b: (_: Int, _: Int))).
When we try to subtract that from _: Optional<(Int, Int)> we get an error.

Oh wow, so SR-11160 really is specific to the Optional sugar syntax, even if accidentally. Okay, thanks for the explanation, and this looks good to merge!

No wonder we didn't catch it earlier.

I think the SR-11160 PR is actually introducing another error because it is allowing illegal subtractions instead of fixing the wrong pattern projection. Investigating that right now.

Hello! I have following warning in Xcode 11.4b1 and not sure what does it mean.

$0 is of type Result<(username: String, sessionKey: String), Error>
What exactly should I fix in this code?

@okla you can fix it by writing
.success((username, sessionKey))

Sorry for the trouble. We have PR https://github.com/apple/swift/pull/28076 up for improving the diagnostic but we haven't merged that yet.

Now I understand, thanks!


Geodistance search not working

Hello again,
this time i'm trying to repair my geodistance filter. It use to work but during last 2 weeks i've made some changes in the code.
From the beggining...
Specialist index:
ThinkingSphinx::Index.define :specialist, with: :real_time do
  indexes :name
  indexes description
  indexes :service_names
  has is_awarded?, as: :is_awarded, type: :boolean
  has created_at, as: :join_date, type: :timestamp
  has number_of_comments, as: :number_of_comments, type: :integer
  has integer_ratings, as: :integer_ratings, type: :integer
  has float_ratings, as: :float_ratings, type: :float
  has comments_text, as: :comments_text, type: :string
  has ratings, as: :ratings, type: :float
  has 'RADIANS(latitude)', as: :latitude, type: :float
  has 'RADIANS(longitude)', as: :longitude, type: :float
  has services.ids, as: :service_ids, multi: true, type: :integer
  has places_text, as: :places_text, type: :string
  has full_address, as: :full_address, type: :string
  has random_comment, as: :random_comment, type: :string
end

Latitude and longitude are stored in DB in degrees.
First problem what I have is I can't properly generate indexes. Each time I remove db/sphinx/development directory and run rails ts:regenerate it stucks on "fsevent_watch <- ruby bin/rails ts:regenerate". I've been waiting over 10 minutes and nothing. I have to stop the task and run again "rails ts:regenerate". Then it takes about 5s and I get:

searchd is not currently running.
Stopped searchd daemon (pid: 4523).
Generating configuration to /Users/dk/Desktop/rails/xyz/config/development.sphinx.conf
Started searchd successfully (pid: 4691).
Generating index files for specialist_core
...

Another problem is when I'm filtering by geo-position I get no results.

Sample Sphinx Query:

Sphinx Query (1.0ms)  SELECT GEODIST(0.8737929831039081, 0.34810556849908936, latitude, longitude) AS geodist, * FROM specialist_core WHERE geodist BETWEEN 0.0 AND 25000.0 AND sphinx_deleted = 0 ORDER BY is_awarded DESC, number_of_comments DESC LIMIT 0, 15
Sphinx  Found 0 results

It looks like the index definition was once SQL-based, but now is realtime - which means the SQL snippets for your latitude & longitude attributes are invalid. What you'll want to do instead is create methods in your model that translate the values to radians:
def latitude_in_radians
  Math::PI * latitude / 180.0
end

def longitude_in_radians
  Math::PI * longitude / 180.0
end

And then use these methods in your index definition:
has latitude_in_radians, as: :latitude, type: :float
has longitude_in_radians, as: :longitude, type: :float

Thanks Pat! It works.

Great to hear :)

This one needs to be in the docs, would not have been able to get this to work without this post


Restart sending Packet / STREAM after received  MAX_DATA/MAX_STREAM_DATA frame

Part of https://github.com/apache/trafficserver/commit/f0548d651a252600eabeea49ecba99dca6883efc fix this for connection.
https://github.com/apache/trafficserver/commit/1418c4f569f87fff9e938fc8554c93286c3b25cb fix this for stream.


doc: correct comment indentation in rbdVolume

correct comment indentation in rbdvolume struct
Signed-off-by: Yug<EMAIL_ADDRESS>See: #https://github.com/ceph/ceph-csi/pull/2105#discussion_r647359226

cc @humblec


Move common SQL generation code into unified SQL generator

Basic refactoring for #2493

:shipit: after interface change


[SPARK-48730][SQL] Implement CreateSQLFunctionCommand for SQL Scalar and Table Functions

What changes were proposed in this pull request?

This PR implements the CreateSQLFunctionCommand to support the creation of SQL scalar and table functions.
Note that the logic for storing and resolving SQL UDFs will be implemented in subsequent PRs. And more SQL tests will be added once it can work E2E.
Why are the changes needed?

To support SQL UDFs.
Does this PR introduce any user-facing change?

Yes. After this PR, users can create persistent SQL UDFs.
How was this patch tested?

New UTs.
Was this patch authored or co-authored using generative AI tooling?

No

cc @cloud-fan


SearXNG service: environmentFile is not effective for uWSGI vassal

Describe the bug
When SearXNG in set to run in uWSGI and specifying services.searx.environmentFile, the environment variables within that file are not set for the SearXNG uWSGI vassal, but only for the searx-init systemd service.
Steps To Reproduce

Add this to system config:

services.searx = {
  enable = true;
  runInUwsgi = true;
  environmentFile = ./searxng.env;
};

Next to system config, create file searxng.env:

SEARXNG_SECRET="myverysecretsecret"

This should have the same effect as specifying services.searx.settings.server.secret_key = "myverysecretsecret".
3. Rebuild
Expected behavior
The environment variables should be passed to the SearXNG process. In the example above, this would mean that SearXNG would be starting. However, since SEARXNG_SECRET has not been set, it does not, and the uWSGI logs show this error thrown by SearXNG:
ERROR:searx.webapp: server.secret_key is not changed. Please use something else instead of ultrasecretkey.

Additional context
Since I cannot specify the order of categories via services.searx.settings, I wanted to use services.searx.settingsFile instead, together with services.searx.environmentFile, to separate my secret into a git-crypt encrypted file. Apart from that, I feel like it would be cleaner to be able to utilize this feature that is already built-in to SearXNG (reading the secret from env) instead of the replacement suggested in the services.searx.settings docs.
I feel like it should be possible to read the environment file and then feed that into services.uwsgi.instance.vassals.searx.env array. I'm not 100% sure if the syntax would be the same in all cases.
Notify maintainers
@rnhmjoj @999eagle
Metadata
$ nix run nixpkgs#nix-info -- -m
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.77, NixOS, 24.05 (Uakari), 24.05.20240226.13aff9b`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

I have added this to my personal config:
services.uwsgi.instance.vassals.searx.env = lib.strings.splitString "\n" (lib.readFile ./searxng.env);

and it does work.

I also use git-crypt and to do this I have set settings.server.secret_key = "@SEARXNG_SECRET@".
Anyway, I agree it make sense to import the environment into uwsgi, but I'm not sure how.
Using the systemd environment file works but would expose the secrets to all other vassals, and
reading the file into searx.env would copy the secrets to the Nix store; so both options are unsafe.

I also use git-crypt and to do this I have set settings.server.secret_key = "@SEARXNG_SECRET@".
Anyway, I agree it make sense to import the environment into uwsgi, but I'm not sure how. Using the systemd environment file works but would expose the secrets to all other vassals, and reading the file into searx.env would copy the secrets to the Nix store; so both options are unsafe.

Fair. uWSGI also has an envdir option, here is the format spec for that. The searx-init script could have a small section that converts from systemd's EnvironmentFile style to envdir. Maybe something like:
(
  umask 077
  mkdir -p "${runDir}/env"
  # read will parse POSIX backslashes, like (probably similar to) systemd
  while read line; do
    # split $line into $line_arr at "="
    IFS='=' read -ra line_arr <<< "$line"
    # sed removes leading and trailing quotes, if present
    echo "${line_arr[1]}" > "${runDir}/env/$(echo "${line_arr[0]}" | sed -e "s/^[\"']//' -e 's/[\"']$//")"
  done <"${environmentFile}"
)

I'm not sure whether this will do all the edge cases correctly, but it should be fine most of the time.
Alternatively, searx-init could parse the output from printenv and put that into an envdir.

I'm running it without runInUwsgi and thought I ran into the same issue but in my case it was a # in the secret (generated by Bitwarden) that the sed expression didn't like.


[RFE] OVHcloud documentation

Current situation
OVHcloud is powered by OpenStack (for the virtual machines) but there is no documentation to show how to import and use Flatcar OpenStack images on OVH.
Impact
It's "hard" to get Flatcar running on OVH.
Ideal future situation

[x] Flatcar documentation (https://github.com/flatcar/flatcar-website/pull/334)
[ ] Terraform examples

Additional information
Some notes I wrote during initial tests:

SSH keys injection works (afterburn)
coreos-metadata works (afterburn)
import the image with --property image_original_user=core and hw_qemu_guest_agent=yes
user-data (Ignition) can be added with Post-installation script from the portal

FYI @sbraz

@tormath1 What is the additional property for?
I ran a couple tests the other week, and it seemed to work without it on OVH.
Haven't tested any specific OVH tooling, but I was able to start VMs with some internal code based on gophercloud.

Yeah, I set the agent in all openstack clouds I use. Still works as advertised! 🥳

Do you also want to look into deploying it on OVHcloud bare metal? We have a terraform provider and we also support cloud-init: https://help.ovhcloud.com/csm/en-dedicated-servers-bringyourownimage?id=kb_article_view&sysparm_article=KB0043281

If I understand correctly, Ignition is some kind of cloud-init alternative?
In that case, yes, you'd need to implement something to fetch data from cloud-init's Config Drive (we use v2 with an ISO 9660 partition labelled config-2).
FYI, @bapt implemented something similar in Lua for FreeBSD and called it nuageinit.

Hi, we use flatcar+terraform on OVH for quite some time.
Here our implementation working in production
resource "openstack_images_image_v2" "flatcar" {
  for_each = var.os_image

  name             = "Flatcar ${each.value["arch"]} - ${each.value["version"]} ${each.value["release_channel"]}"
  region           = var.region
  image_source_url = "https://${each.value["release_channel"]}.release.flatcar-linux.net/${each.value["arch"]}-usr/${each.value["version"]}/flatcar_production_openstack_image.img.gz"
  # XXX do not use it, OVH openstack seems to not handle this well :(
  # web_download      = false
  verify_checksum  = true
  decompress       = true
  container_format = "bare"
  disk_format      = "qcow2"
  protected        = false
  hidden     = false
  visibility = "private"

  # See: https://docs.openstack.org/glance/stein/admin/useful-image-properties.html
  # See: https://wiki.openstack.org/wiki/VirtDriverImageProperties
  properties = {
    architecture              = each.value["arch"] == "amd64" ? "x86_64" : each.value["arch"]
    image_original_user       = "core"
    distro_family             = "gentoo"
    os_distro                 = "gentoo"
    os_version                = each.value["version"]
    os_release_channel        = each.value["release_channel"]
    os_arch                   = each.value["arch"]
    os_type                   = "linux"
    hw_disk_bus               = "scsi"
    hw_scsi_model             = "virtio-scsi"
    hypervisor_type           = "qemu"
    hw_qemu_guest_agent       = true
    hw_vif_model              = "virtio"
    hw_vif_multiqueue_enabled = true
    hw_time_hpet              = true
  }

  timeouts {
    create = "5m"
  }
}

The for_each is used to get different versions running
  os_image = {
    main = {
      distro          = "flatcar"
      arch            = "amd64"
      release_channel = "stable"
      version         = "3815.2.5"
    }
    canary = {
      distro          = "flatcar"
      arch            = "amd64"
      release_channel = "beta"
      version         = "current"
    }
  }

@tormath1 I'll do it later
We do not have that much beta nodes, we just want to check flatcar upgrades on them before the whole cluster. We had some bad surprises before :grimacing:
We use baremetal with flatcar too but not with the OVH MAAS.

@tormath1 Added example here: https://github.com/flatcar/flatcar-terraform/pull/27
Do you prefer a separate openstack-ovhcloud folder ?


⚠ IMAP has degraded performance

In e7b2905, IMAP (https://news.ycombinator.com) experienced degraded performance:

HTTP code: 200
Response time: 5983 ms

Resolved: IMAP performance has improved in d17b666 after 5 minutes.


fix:label's value & web-utils

Just few bug fixes , I joined today's call , looking to contribute more


Create query-kb-with-metadata

Hi @DishaAgarwal - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide, and new articles need to be submitted in the private repo. Would you open a new pull request in the private repo for your new article? Thank you! We'll close this PR.


Populate an enum with the task names

Mainly for use with task callbacks, see: https://github.com/rtfm-rs/cortex-m-rtfm/issues/247#issuecomment-571972834

In #371 I've included code which generates an enum like the following for all task names:
enum Tasks {
    foo,
    bar,
}

Hi @AfoHT, I notice that although your PR was merged, it seems the task list code is no longer in master/0.6-alpha?  Any reason for the removal?

Hi!
This seems to have disappeared during refactoring, most likely since there were no tests/examples requiring the task-list.
I'll look into reintroducing it shortly :+1:
It would be interesting to hear about how these task callbacks work, maybe you could provide some example?

It would be interesting to hear about how these task callbacks work, maybe you could provide some example?

To be honest I am still looking for a better way of doing this, but essentially I am using it for inter-task communication. This task list simplifies the manual work I have to do.
Take 3 tasks, Sntp client, Http client and Dns Client. Both the Sntp & Http client might need the Dns client to resolve some hostnames before proceeding, the thing is:

How does the DnsClient report that the Dns Look up has completed
How does the DnsClient know where the request from?

I am currently solving this via a manually curated TaskList enum, and adding a source: TaskList parameter which can be attached to the request. E.g Http client would set source: TaskList::HttpClient. Then I have a callback trait:
pub trait DnsCallback {
    fn dns_result(&mut self, res: Result<Ipv4Address, DnsStatus>);
}

which I implement on all the client structs, and call the appropriate ones based on the task lisk:
// match the source task and give back the result
match item.1 {
    TaskList::SntpClient => {
        sntp_client.lock(|sntp_client| sntp_client.dns_result(res))
    }
    TaskList::HttpClient => http_client.dns_result(res),
    _ => unreachable!(),
}

Hope that clears up my use case a bit, I welcome more questions or suggestions.
In other RTOS's I've used you can get task/queue handles which can be passed around to tell where to put or take things from. I don't think this model fits in RTIC though as the resource proxies can only last the task duration? I.e they can't be moved out of taskX::Resources.

Closing this with reason found in https://github.com/rtic-rs/rtic-syntax/issues/54


Report grouping

Is your feature request related to a problem? Please describe.
Endpoints that are invalid from multiple perspectives are represented multiple times in the report, even though the certificate may be the same (e.g. thumbprint)
Describe the solution you'd like
Group endpoints that have the same uri and thumbprint and represent the perspectives that they are invalid from in a single row.

Resolved by #16


Custom Base subclasses are automatically registered for deserialization

docs(example/base_registrar.py): add example implementation for SpeckleBase class register
style(black config): added explicit black config to pyproject.toml
refactor(test_base.py): refactor base object invalid prop tests
feat(base model and deserialization): base model subclasses can now be automatically deserialized
.

Changes

add private base model for type registry
update BaseObject serializer

Checklist

[x] Unit tests
[ ] Documentation

References
fix #50

Hey @izzylys I've added some docstrings for devs, but package reuse docs I have not done.

@izzylys I consider this ready for review. I added a showcase file in example/using_speckle_base.py.

This looks great Gergő! Just had a look through this and I think you came up with the right solution for this issue. The example file is also plenty clear. I also appreciate the light refactoring you've sprinkled in here and there. Thanks again for all your fab work on this! So pleased to get to merge in this new feature ✨
The only change I would request would be the falling back to a vanilla Base if the type is unrecognised as per my comments above.
Other than that, this looks good to merge - thanks again!

Hey, sorry I could not get this over the line yet.
Some tests keep failing after some seemingly unrelated code changes.
I'm gonna merge the latest fixes from main and try again.

Your :bug: fixes did fix my branch, thanks @izzylys.
I've uncovered one case I'm not sure how to handle. The test below currently fails, since deserailization defaults to Base. This becomes a base with speckle_type value None.
I think the sensible step here should be to fall back to a plain dict object rather than to Base.
def test_no_speckle_type():
    untyped = '{"foo": "bar"}'
    deserialised = operations.deserialize(untyped)

    assert deserialised == {"foo": "bar"}

ah right, good catch! yes I agree - in this case it should default to a dictionary

ah right, good catch! yes I agree - in this case it should default to a dictionary

Its fixed now. I ok with merging this.

amazing, thanks so much for this 🥳


Automatic CASE WHEN wrapping of boolean expressions

We should add support for plain boolean expressions in entity view mappings like e.g.:
@Mapping("age > 10")

Since JPQL does not support plain boolean expressions in select items, the user currently needs to manually formulate such mappings as CASE WHEN:
@Mapping("CASE WHEN age > 10 THEN true ELSE false END")

This CASE WHEN wrapping should happen automatically.

Actually I think we should do this in core already instead. Entity views should not have to change anything and only the final JPQL output should show the transformed expression.

Maybe we can also allow boolean expressions such as x OR y without having to write x = true OR y = true.

Although well aware of this limitation, I recently stumbled upon this again by surprise with a x IN (subquery) expression and a SUM(x) IS NOT NULL NULL expression in the select clause.


Server processing unauthorized error

Hello, what's causing this ?
Could we chat a bit?

more context please and if the repo has been edited by you is it a forked one ?

Could we chat somewhere else ? @lcfidev
Btw, the ws is sent the way you did, maybe I could send a code snippet

discord.gg/hasurei


Bjorncs/access control

@gjoranv Please do a post-merge review (we need this fix out quickly to production). I will follow up with a new PR for any requested changes from review.

I would prefer using volatiles instead of synchronized where appropriate. It generates less code noise, and is less prune to deadlocks, although the latter is not relevant here.


Add user accessor to AlexaEvent

We also need to update the documentation on this
El 1 mar. 2017 5:30 PM, "Mitchell Harris"<EMAIL_ADDRESS>escribió:

You can view, comment on, or merge this pull request online at:
https://github.com/mediarain/voxa/pull/1
Commit Summary

Add user accessor to AlexaEvent

File Changes

M lib/AlexaEvent.js
https://github.com/mediarain/voxa/pull/1/files#diff-0 (4)
M test/AlexaEvent.spec.js
https://github.com/mediarain/voxa/pull/1/files#diff-1 (11)

Patch Links:

https://github.com/mediarain/voxa/pull/1.patch
https://github.com/mediarain/voxa/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mediarain/voxa/pull/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAvDaaxinBrTwOnONeFqvp2d1l22o30rks5rhf-lgaJpZM4MQWSe
.

Coverage increased (+0.007%) to 97.196% when pulling 3eb1c39f0b74df4129ead7c06ef249fbca2b1567 on heneryville:master into 5e1e77d01eb28edfb2d22717af732d3b1cfd8dca on mediarain:master.

Coverage increased (+0.01%) to 93.541% when pulling 73ea8e139f6448cf48a1a6633c3c5188f92a7cdf on heneryville:master into 2442d9b2eaeff795febce91b20947161634d672e on mediarain:master.

Coverage increased (+0.01%) to 93.541% when pulling a3af30439175d15f5cf892762618349e96831627 on heneryville:master into 2442d9b2eaeff795febce91b20947161634d672e on mediarain:master.


Fix MaterializedViewMetadata when base table name is case sensitive (…

…JAVA-953).

This looks good to me :+1:


Populate Allowed Parents

https://github.com/mjmlio/mjml/blob/master/packages/mjml-preset-core/src/dependencies.js


How to disable javascript in splash json http api ?

Why would you disable JavaScript? Do you want to take a screenshot without JavaScript? Otherwise using Splash seems pointless to me.

yes i want to take screenshot .

Rendering a page without JS can be useful to improve speed, to check how page looks without JS, or to avoid popups which are triggered by JS.
But I think it's not exposed in the HTTP API - you'd need to write a lua script, see https://splash.readthedocs.io/en/stable/scripting-ref.html#splash-js-enabled

Thanks :)


How to hide a skip button in the last page?

I'm looking for the function which enables to hide a skip button only in the last page.

Found fadeSkipButtonOnLastPage.


120s Timeout results in None response. for long running commands

2024-02-15T20:32:35.673Z : level : ERROR , message : move_rdfg_devices_from_src_rdfg failed: error:Bad or unexpected response from the storage volume backend API: Error PUT symmetrix resource. The status code received is None and the message is None.
"/opt/XXXXXX/lib/python3.8/site-packages/PyU4V/common.py", line 180, in check_status_code_success\n raise exception.VolumeBackendAPIException(\nPyU4V.utils.exception.VolumeBackendAPIException: Bad or unexpected response from the storage volume backend API: Error PUT symmetrix resource. The status code received is None and the message is None."

Published fix for <IP_ADDRESS>

Closed

I am facing the same issue with 9.x version. Is there any API in <IP_ADDRESS> for setting up customized timeout?
Thanks,
Sacheen Birhade

Sacheen, this fix wasn't backported.  you have the option to review the changes in this pull request and create your own fork of the code for your own install.
You can change the timeout in the new version in your connection string,  conn = PyU4V.U4VConn(username='smc', password='smc',  server_ip='ipaddress', verify=None,  array_id='000220200226', timeout=300) .  PyU4V 10.1 has this code and is capable of supporting older arrays, you can upgrade an external Unipshere to this code to avail of the fix.


add trivial missing parameter

@homu r+
thanks!

:pushpin: Commit 7f7bdfa has been approved by kvark

:zap: Test exempted - status


Removing unused import from the owner repository.

@abdurraffaysyed Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@abdurraffaysyed Thank you for signing the Contributor License Agreement!


List requests should return an empty array for items and not null

Example, if I request a list of service classes and there are none, the response should not have items: null it should have items: [] to be consistent with all k8s APIs.

@jpeeler interest in this?

I'll take it.

remember to add a test for it as well.

@jpeeler @pmorie can you add a milestone to this issue?


fix: add missing import that prevents compiling generated code

@JsonProperty annotations can be added to some instance variables in the model class, but this breaks maven builds because JsonProperty is missing from the imports.

/au

/rtm

/rtm


Some minor improvements

split github-pl-colors into a standalone file
simplify subtitle composition
allow multiple languages in develop-item

LGTM.


WIP: Version 2.0.0

This PR is not ready to be merged yet until all the checks below are done
It is merely used to keep track of numbers (like commits, changed files, checks and amount of changed lines)
> 2.0.0 Milestone with all remaining issues <
Checks:

[x] Implement GitHub CI #24
[ ] Implement everything in the TODO list
[ ] Resolve everything marked with TODO or DEBUG in the code
[x] Fix all bugs

[x] Issue #13
[x] Issue #22
[x] Issue #29

[ ] Do a bunch of Q/A (Testing) #28

This pull request introduces 17 alerts and fixes 18 when merging 1d2c818 into 5111ceb - view on LGTM.com
new alerts:
* 10 for Useless assignment to local variable

* 3 for Identical operands

* 1 for Variable not declared before use

* 1 for Hard-coded credentials

* 1 for Invocation of non-function

* 1 for Access to let-bound variable in temporal dead zone

fixed alerts:
* 6 for Useless conditional

* 5 for Unused variable, import, function or class

* 4 for Missing variable declaration

* 1 for Useless assignment to local variable

* 1 for Information exposure through a stack trace

* 1 for Unneeded defensive code

Damn that's a lot, thanks mister bot

This pull request introduces 1 alert and fixes 18 when merging 43a0715 into 5111ceb - view on LGTM.com
new alerts:
* 1 for Hard-coded credentials

fixed alerts:
* 6 for Useless conditional

* 5 for Unused variable, import, function or class

* 4 for Missing variable declaration

* 1 for Useless assignment to local variable

* 1 for Information exposure through a stack trace

* 1 for Unneeded defensive code

I set you to ignore that alert you intelligence--

This pull request fixes 18 alerts when merging 709fe29 into 5111ceb - view on LGTM.com
fixed alerts:
* 6 for Useless conditional

* 5 for Unused variable, import, function or class

* 4 for Missing variable declaration

* 1 for Useless assignment to local variable

* 1 for Information exposure through a stack trace

* 1 for Unneeded defensive code

You don't have to comment on every single fucking commit


Fix i18n _get_default_locale_path

Fixes #105
Changes proposed in this pull request:

Use importlib.resources available since Python 3.9 to find the location of the locale folder containing the translations.

I tried to use __package__ initially, but the docs suggest to use __spec__.parent instead. https://docs.python.org/3/reference/import.html#package__
The tests cover when __spec__ is None or even if it's missing. As __spec__.parent is read-only , I cannot make a test which sets a different value for it.

Hi,
Thank you for the review. I had not considered the CLI aspect of humanize. That's a good point. Now a CLI user will pay the price of importing importlib.resources at some point obviously. But I can see how
if TYPE_CHECKING:
    import os
    import pathlib

is really useful here.
Thank you for showing me tuna. I did not know this tool, and I've used snakeViz in the past, so it's great to see a better alternative. Also this is the first time I see python -X importtime -c "..." in use. I learnt something new today! :clap:

I can also imagine some CLI users don't use i18n so might not run touch this. Even for code that might run more often, lazy imports can make --help a bit faster too.
Yeah, tuna is really nice for visualising import times along with -X importtime :)
Thanks for the PR!


What time we can update version to v3.8.0 at grafana web?

Thank you.

@X-Mars Grafana team is on holidays since Dec, 15 and it took a while for them to review the plugin.
They come back to me with some questions related to breaking changes we introduced. I am working with them to get it published, I will let you know when published. I expect to get and answer earlier next week.

@X-Mars We released a new version 4.0.0 introducing breaking changes instead of 3.8.0 and documentation updates.
It's resubmitted to the Grafana team for review. I expect it to be reviewed later this week.
Thank you for your contributions.

@X-Mars Apache ECharts 4.0.0 is available in the Grafana Catalog: https://grafana.com/grafana/plugins/volkovlabs-echarts-panel/

@mikhail-vl  I have upgraded，thanks.👍👍👍