SQLDatabase introspection fails with NoInspectionAvailable

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
import os
from langchain.llms import OpenAI
from langchain.utilities import SQLDatabase
... Your database credentials ...
os.environ["OPENAI_API_KEY"] = "sk-myapikey"
db_uri = f"postgresql://{user}:{password}@{host}/{database}"
db = SQLDatabase(db_uri)  # Error occurs here
Error Message and Stack Trace (if applicable)
NoInspectionAvailable: No inspection system is available for object of type <class 'str'>
Traceback:
File "C:\Python311\Lib\site-packages\streamlit\runtime\scriptrunner\script_runner.py", line 542, in _run_script
exec(code, module.dict)
File "C:\opensql\gemSQL.py", line 19, in
db = SQLDatabase(db_uri)
^^^^^^^^^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\langchain_community\utilities\sql_database.py", line 69, in init
self._inspector = inspect(self._engine)
^^^^^^^^^^^^^^^^^^^^^
File "C:\Python311\Lib\site-packages\sqlalchemy\inspection.py", line 71, in inspect
raise exc.NoInspectionAvailable(
Description
I'm encountering a NoInspectionAvailable error when using the SQLDatabase class in LangChain. However, basic SQLAlchemy connections and introspection work correctly.
System Info
OS: windows 10
Python: 3.19
LangChain: 0.1.12
SQLAlchemy: 1.4.47

@jotatx,
Can you try to upgrade SQLAlchemy to 2.0.27 ?


maximumCaptureDuration not working when pausing/resuming

I've set maximumCaptureDuration to 5 seconds. If I long press and hold for 5 seconds, maximumCaptureDuration works and the video recording ends, but if I let go of my long press, then press again to keep recording, maximumCaptureDuration will not work on it's own and you can keep recording for as long as you want.

hey @mitchellporter i'm unable to reproduce this with the sample project. are you always calling startVideoCapture following the maximumCaptureDuration "end"?

@piemonte Sorry for the delay in reply! I'm unable to recreate this as well now that I've been further integrating PBJVision. Not sure what I was doing wrong before, but looks like this was a false bug.


Add docker with FSL for integration tests

Closes #40 .

Changes proposed in this pull request:

Add nipype docker to run integration tests with FSL.

Try sudo?

Apparently, the issue is that once the job is finished, the conda environment is removed. Locally, when I exit the docker image and get back in, the conda environment is no longer there. The same thing is happening with CircleCI.
I have no idea why this is happening with Aroma but it doesn't happen with other repos. I actually use nipype/nipype in a repo of mine and have no issues with it.
Any ideas @tsalo @vinferrer ?

Maybe something's wrong with the caching? That should be the step that makes the environment persist within the Docker image, right?

Maybe something's wrong with the caching? That should be the step that makes the environment persist within the Docker image, right?

Right. But I honestly do not know how to check what's going wrong with the caching.

I think the environment is still there, but conda is not recognizing it for whatever reason.
Try using the complete path to the environment when activating it, like in this commit.

Wow, thank you @notZaki ! I'm surprised that made the trick. When I tested locally I could see the environment under miniconda-latest. I don't know why saving the environment in miniconda-latest and reading it from conda works. This is pretty weird but it works now. Thank you!

Well, that was a crazy ride with the conda environments and the environment variables, but it works now!
I would like to know what you think @vinferrer @notZaki @tsalo

I think the environment is still there, but conda is not recognizing it for whatever reason. Maybe it's because the environments are in /opt/conda instead of /opt/miniconda-latest, but the cache is using the latter path? Not really sure what's going on there.

It looks like this is not the case in the main branch. I will open a PR with a quick fix for it. This is really weird anyway.


TypeError: unhashable type: 'list'

I'm getting an error running this in python 3.9.
$ find . -name compile_commands.json
./projects/.../.../compile_commands.json
./projects/.../.../compile_commands.json
./projects/.../compile_commands.json
./projects/.../compile_commands.json

$ compile-commands --merge --dir projects
Traceback (most recent call last):
  File "/home/hamish/.local/bin/compile-commands", line 8, in <module>
    sys.exit(main())
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 370, in main
    data = [dict(t) for t in {tuple(d.items()) for d in data}]
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 370, in <setcomp>
    data = [dict(t) for t in {tuple(d.items()) for d in data}]
TypeError: unhashable type: 'list'

$ compile-commands --dir projects/...
Traceback (most recent call last):
  File "/home/hamish/.local/bin/compile-commands", line 8, in <module>
    sys.exit(main())
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 370, in main
    data = [dict(t) for t in {tuple(d.items()) for d in data}]
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 370, in <setcomp>
    data = [dict(t) for t in {tuple(d.items()) for d in data}]
TypeError: unhashable type: 'list'

With the latest code I now have
$ compile-commands --dir . --merge
Traceback (most recent call last):
  File "/home/hamish/.local/bin/compile-commands", line 8, in <module>
    sys.exit(main())
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 373, in main
    args.files = get_compile_dbs(args.dir)
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 202, in get_compile_dbs
    return [p for p in paths if Path(p).parent.parts[-1] != Path(directory).parts[-1]]
  File "/home/hamish/.local/lib/python3.9/site-packages/src/compile_commands.py", line 202, in <listcomp>
    return [p for p in paths if Path(p).parent.parts[-1] != Path(directory).parts[-1]]
IndexError: tuple index out of range

Although if I leave out --dir . it seems to work.

Forgot an os.path.abspath() while refactoring, should work again with relative paths.

Thanks, merging is working fine now.


View Resource Owner - 'NoneType' object has no attribute 'lower'

When attempting to view resource owner profile infromation from the resource listing page under the Owner column link, it errors with type: 'NoneType' object has no attribute 'lower'
using alpha.hydroshare.org as an example: http://alpha.hydroshare.org/user/admin/

mark this issue as "Fix pushed to github." Now the fix has been merged into develop branch. Once it makes its way into alpha, will mark it as ready for testing.

In my testing this appears to be resolved, but I do not know exactly what Michael did to generate this error, so leave it for @mjstealey to test and close.


I cannot run it the browser fs.readFileSync is getting called

I am trying to run the examples on the readme. I am using webpack but just the act of requiring swarmlog  is giving me an error because it is calling fs.readFileSync()
module.exports = protobuf(fs.readFileSync(__dirname + '/../schema.proto', 'utf-8'))

Managed to get this working temporarily by hardcoding contents /../schema.proto in hyperlog here:
https://github.com/mafintosh/hyperlog/blob/master/lib/messages.js
Am I missing something, is there some way to get fs.readFileSync() to work in browser with webpack / browserify?

OK found brfs and got it working with webpack


Don't emit filename in diff mode, add filename to diff metadata

Addresses #807 - the added test doesn't actually test the changed code, but since its a minor detail and this is not an issue about correctness then I thought it'd be ok.

What does the output from --diff look like now? Is it valid input for patch?

nope, the only thing that changed is that the output is blank for no diff.
I wasn't sure what was the expected output of the issue, but if the goal is to have it follow patch format then I'd be happy to work on that

@juicejitsu that would be ideal, it would be great if you wanted to work on that. I'm afraid I don't know what it would entail since I'm not familiar with the diff crate, but if I can help in any way, let me know.
This PR seems like a short-term improvement, so I'll land this also.


frame_begin should be initialized after csp_buffer_get()

As you can see from the following code, without csp_id_prepend(packet);, a segmentation fault occurs because frame_begin in the packet is not properly initialized but set to 0x0.
#include <csp/csp.h>
#include <csp/csp_debug.h>
#include <csp/csp_id.h>
#include <csp/crypto/csp_hmac.h>

int main(int argc, char * argv[])
{
	csp_packet_t * packet;

	csp_init();

	packet = csp_buffer_get_always();
	csp_id_prepend(packet);

	csp_hmac_append(packet, true);
	csp_hmac_verify(packet, true);

	return 0;
}

This issue was reported by @poomravint in #730.

Fixed by #735


Allow // as an alternative separator to -- for user CLI arguments

This is required when using startx to start a Godot project, as -- is used by startx for its own arguments (and there is no way to escape it). This is specifically needed by https://github.com/godotengine/godot-benchmarks for automation purposes.
Testing project: test_cli_separator_double_slash.zip

// looks pretty awkward to me, especially on Unix command line where it is a valid path ls //home works like ls /home.
Maybe we could use --- instead?
What's the startx command you need to run exactly? Is there really no way to separate startx arguments from the launch command's arguments?

Yea, I'd rather see something other than //, since it is a valid path or looks like a comment in C derived languages.
--- seems like a reasonable workaround.  The other options suggested by akien-mga seem fine too.

--- seems like a reasonable workaround. The other options suggested by akien-mga seem fine too.

--- might be intercepted by some apps, and looks like a typo at a glance.

Thanks!


New Minecket 4.0

Dear Minecket fans,

we are excited to announce that we will soon be adding a powerful database system to our website, which will enhance your browsing and shopping experience. Additionally, we will be cleaning up our existing codes and improving our shopping cart and checkout systems to make the entire process smoother and more efficient for you.

By Henry8K and team.

TODO

[x] JavaScript clean up.
[x] Better CSS
[x] Responsive Fixies
[ ] New 404 e 505 Pages
[ ] Database Support.
[ ] Languages Support.
[ ] And others.

Perhaps a css reorganization?

Sure.

For 404 pages, I have an idea to create an animation. What do you think? Do you like it?

I like the idea, but what kind of animation are you thinking of?

I'm still not sure. Are you thinking of something like a confused Steve animation?

I'm still not sure. what about a confused Steve animation?

Cool, do it

Css in the same page of html?

Idk, maybe yes...

Done.


Find pull requests merged between revisions

I would like to have capistrano-fiesta use the current revision and the previous revision to find pull requests merged rather than the previous deployment time because

The release doesn't necessarily include all the PRs merged after the previous deployment.
I think the revisions are more likely to be available in a broader range of cases than the previous deployment time.
The revisions we deploy may not be in the master branch.

The downside of this change is we have to call GitHub API for each PR. We might be able to mitigate this by squashing multiple requests into a Search Issues API request by passing multiple SHA hashes of commits as the q parameter, but we still need to call the API multiple times if we have PRs that doesn't fit into a single search request.

I'm closing this since this implementation doesn't work with squash/rebase merge as pointed out in https://github.com/balvig/capistrano-fiesta/pull/16#discussion_r220389723 and I can't come up with another solution 🙇


update clap to last version: clap = { version = "3.1.8",

Summary

Motivation

Additional Information

[dependencies]
clap = { version = "3.1.8", features = ["derive"] }

https://github.com/clap-rs/clap/discussions/3611

@cindRoberta I am working on this PR. I'll do it soon

@cindRoberta I am working on this PR. I'll do it soon

I'm too, do you want to exchange information? did you open a fork?

@cindRoberta Oh, then keep working, I'll take another task)

Thanks!! @cindRoberta @ikrivosheev

@cindRoberta do you still working on this ?


[java] Tech Debt - Add JUnit tests for templates

Add JUnit tests for below templates, you may use HiveToGCSTest.java as example.
Goal is write unit test for all the business logic before spark session is created.
JDBCToGCS.java
JDBCToBigQuery.java
SpannerToGCS.java
DataplexGCStoBQ.java

JUnit tests for JDBCToGCS.java and JDBCToBigQuery.java have been added as part of issue https://github.com/GoogleCloudPlatform/dataproc-templates/issues/464

Closing out this issue, as different folks are working on different unit tests. Thus, created different issues separately for different templates :-
https://github.com/GoogleCloudPlatform/dataproc-templates/issues/464
https://github.com/GoogleCloudPlatform/dataproc-templates/issues/466
https://github.com/GoogleCloudPlatform/dataproc-templates/issues/530


Updating for the latest SDK and to match changes to the Getting Start…

…ed pages and to match changes to the Getting Started pages.
Updated the README.md file

@VTCoco can you accept this PR?

Done (avengers names have also been removed :P)


Add skeleton for drill down in table results

The code looks good and produced the necessary table outputs. I see you decided to use mui ExpandMoreIcon instead of caret. That works fine.
When I expand the upside caret, I see 3 things. Shouldn't be just 2 things there - drugs and weights?
Overall nice and clean design.

Yes, the drugs and and weights will go in that table, I just filled it in with a placeholder for now to make sure it would work.
Also, the ExpandMoreIcon should probably be swapped with the ExpandLessIcon or rotated 180 degrees when the expansion actually happens.
But those will wait for another commit.
Thanks for the feedback!


CDK Drag-Drop - Animation when entering and exiting items

Feature Description
When we delete an action, add or even pass an item from one container to another, the item goes sharply pushing forward the others, and it would be much better to have some animation in entering and exiting items, I even tried to put myself, but I noticed that the library uses the position of the items at the time of the drop to calculate the movement, so it would even animate, but then dragging would get weird. Having such a function would make the animations much more consistent and beautiful.
Use Case
When CSS animation is enabled it appears incomplete when deleting, adding or moving items. This would improve the look of any implementation that uses transition CSS animation.

I have a similar issue. In my case I have a list of lists that are separated by a header like:

And when you move between a list another the effect is not smooth.
I've tried to fixed it with different approaches:

Add :enter/:leave height animation to the placeholder, but it seems that it has some limitation, it doesn't work.
Try to create a transition with css. It is possible on leave effect, but not on enter.


Expose NotebookScreenshot docstring

References and relevant issues
Closes napari/docs#376
Description
Exposes the docstring of NotebookScreenshot, which now apppears as a link in the nbscreenshot docstring.
I understand NotebookScreenshot may not be something we want to expose. In that case, I would suggest moving the docstring to nbscreenshot as an alternative.
Also fixes a few syntax errors in the NotebookScreenshot docstring.

Here's the rendered html:
https://output.circle-artifacts.com/output/job/ca65b81d-f8d8-4645-8d3b-d88d5af2d532/artifacts/0/docs/docs/_build/html/api/napari.utils.NotebookScreenshot.html


footer in the feedback section

Describe the bug
there is no footer section in the feedback as it makes unattractive  in the website
To Reproduce
Steps to reproduce the behavior:

Go to '.navbar section..'
Click on '...feedback.'
Scroll down to
See error

here is the image:

please assign this to me.

thanks @ANSHIKA-26

I have completed the task and make a pr #326

@ANSHIKA-26
Please make review and give me feedback

hey @ANSHIKA-26
i updated again footer and makes changes to all other webpages
here's are the screenshot of the same:
before:

After:

here's are the videos of the same:
Before:
https://github.com/user-attachments/assets/9a793b8e-a0d0-4590-8754-3f2b3ddacf10
After:
https://github.com/user-attachments/assets/b32f0bba-b5fe-4327-b432-de557445c3e2
Now i am unable to make pr please do help me.
how i can make these changes in the websites??
please help me!!


Didn't find op for builtin opcode 'CONV_2D' version '5'

Hi, I'm trying to run inference of a custom trained mobilenet_v2_coco17_320x320_tpu-8 model on a Raspberry pi. I have succesfully trained, saved and converted the model to tf.lite. However, when trying to run the inttepreter on a Raspberry pi 4, I'm getting the following error: ValueError: Didn't find op for builtin opcode 'CONV_2D' version '5' Registration failed.
But when running the interpreter on google colab notebook it works just fine.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Google colab
TensorFlow installed from (source or binary): tf-nightly on colab. tf-2.2 on Rapsberry pi

Provide the text output from tflite_convert
converter = tf.lite.TFLiteConverter.from_saved_model('/content/drive/My Drive/Deteccion_escaleras/models/research/Mobilenet_entrenada/saved_model',signature_keys=['serving_default'])
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.experimental_new_converter = True
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS]
tflite_model = converter.convert()

with tf.io.gfile.GFile('/content/drive/My Drive/training_mobilenet/model_3.tflite', 'wb') as f:
  f.write(tflite_model)
  print("Model saved")

Code use for training,evaluation,conversion:
https://colab.research.google.com/drive/1M7SBWcA_gRe4xEww3q8QYdyblV8Ic2Te?usp=sharing
Any other info / logs
Generated model files (saved_model.pb & model.tflite): https://drive.google.com/drive/folders/1-JHyeg268jQWyXF_AO348SNN8fgU1TCT?usp=sharing

The saved model is incomplete. Can you share a saved model dir that we can load directly?

@bhack Sorry what do you mean when you say the model is incomplete? That folder attached contains a saved_model.pb file that I've loaded directly using the tf.saved_model.load() command. Did you get a problem when loading the model?
However here is the complete directory containning the saved model folder and  the checkpoints folder:
https://drive.google.com/drive/folders/1-JHyeg268jQWyXF_AO348SNN8fgU1TCT?usp=sharing

Ok.. what TF version are you using on Raspberry?

I'm running tf 2.2

Can you try to align the exporting version and the one running on Raspberry?

When I try to do that I run in to a problem with the exporter beacuse of the dynamic input tensor in model. ValueError: None is only supported in the 1st dimension. Tensor 'input_tensor' has invalid shape '[1, None, None, 3]'.
And when I try to export using concretefunc I run into another error: Exception: :0: error: loc("Func/StatefulPartitionedCall/input/_1"): requires all operands to be either same as or ref type of results
Here(https://colab.research.google.com/drive/1Uwj0Boede08W0e7gpb3evQp3RSQysqtr?usp=sharing) I attach a notebook where I downgraded TF to 2.2 and try to run an export job by the 2 different means mentioned.

I think It Is hard to get a precomp nightly for Raspberry to quickly test the exported model cause we don't have a new link for nightly: https://github.com/tensorflow/tensorflow/issues/21469
Can you try to build https://www.tensorflow.org/install/source_rpi

@juanpbotero98 It looks like TF 2.2 version does not have the CONV_2D version '5'. You need to use a recent version of TF runtime or at least the same TF version, used for conversion in Raspberry.

@bhack I managed to upgrade tf on my Raspberry pi to the latest available version (2.3) however i couldn't find any data for installing tf-nightly and when running the interpreter I still get the same error.

@abattery Thanks I will now try to convert using tf 2.3 since now i have that version running in my raspberry pi.

If 2.3 will not convert your model (if it requires ft-nightly) you will need to build the PI with the same instructions but from the master branch.

When trying to convert using tf 2.3 I ran into the following error:
:0: error: loc("Func/StatefulPartitionedCall/input/_0"): requires all operands and results to have compatible element types
:0: note: loc("Func/StatefulPartitionedCall/input/_0"): see current operation: %1 = "tf.Identity"(%arg0) {_class = ["loc:@Func/StatefulPartitionedCall/StatefulPartitionedCall/input/_335"], device = ""} : (tensor<1x?x?x3x!tf.quint8>) -> tensor<1x?x?x3xui8>

@bhack Could you please elaborate? what do you mean by building the Pi from the master branch?

Yes, same instruction but from the master branch:
https://www.tensorflow.org/install/source_rpi

@bhack I tried to run the instruction for the master branch but after all the compiling was done I wasn't able to find the .whl file. I don't really know what went wrong.
the only thing is that when I run the commad: git checkout branch_name I get an error saying: git checkout branch name did not match any file known to git.

<branch_name> is e.g. master
The wheel is in the outputdir defined at https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/ci_build/pi/build_raspberry_pi.sh#L128

@bhack I'm running the commands listed in https://www.tensorflow.org/install/source_rpi, and it seems to run without any problem however the out-artifacts directory doesn't get created. Hence no .whl file gets created either. I don't know what might be the problem.

I found the error i was running out of memory before the .whl file could be generated. I'm going trough the buil once again using a digital ocean virtual machine.

@juanpbotero98 You can also add some option to limit the RAM https://www.tensorflow.org/install/source#bazel_build_options

@bhack Now I'm getting a really strange error.
Execution platform: @local_execution_config_platform//:platform arm-rpi-linux-gnueabihf-gcc: internal compiler error: Killed (program cc1plus)
and after that the console prints: FAILED: Build did NOT complete successfully

@juanpbotero98 Sorry I don't have currently enought free resources to rebuild It locally.
In the meantime can I suggest you to open a ticket mentioning this one in https://github.com/tensorflow/build? That one is to request if they can publish again regularly Raspberry tf-nightly wheels.
Cause at some point we lost It as you can see from #21469
It think that could useful for other devs that need to tf-nightly to convert their model to be aligned at inference time on the device.

@bhack Thanks for all the help, I really appriciate it! . I will definitely post a ticket requesting the tf-nigthly wheels for Raspberry pi. However I was wondering if in the meantime there is anything I can do, like  train the model from scratch using a certain TF version or something like that.

I think in the meantime you have also two other actions:

open a new ticket here about the error you got building Rasberry wheel on master
try to find a TF version for the model training and conversion that works with your model but that is also compatible with an already available precompiled Raspberry wheel (<= 2.3)

I have the same problem with tflite_runtime 2.5.0 on Raspberry pi 4. Is there any new solution?

I have the same problem with tflite_runtime 2.5.0 on Raspberry pi 4. Is there any new solution?

Hi @HMellor  !
We are checking to see whether you still need help in this issue .
Have you  checked from 2.10/2.11 tensorflow lite run time. I can see Conv2D in built in ops allow list now .
If issue still persists, Please check with work around in below similar issues or post in TFlite-micro repo for further assistance.
Attached similar issues for reference.
1, 2
Thank you!

try this it solved my issue
check
!pip show tensorflow
now try to add  implementation of same version if i have tf 2.12.0 then use below
eg: implementation 'org.tensorflow:tensorflow-lite:2.12.0'


Linux packages

I'm in the process of updating some web apps to mod_python 3.5.x on debian bullseye. On several occasions I've found the need to take patches from here and apply them to the debian files. I'm wondering if anyone knows of a linux distribution that maintains a more up-to-date mod_python package or should I be using the git master?
I cringe at the prospect of changing to wsgi. Although there are some native python projects that could be used, there aren't any that I've seen that can do everything that mod_python can. Anyone have any suggestions?
Thanks to everyone who has been maintaining this git repo. It's been very helpful.

IIRC people from Ubuntu have opened a pull request or two lately, which might suggest that they are keeping an up-to-date version. (This isn't a very good answer to your question, but it's all I know :) )

How to report facebook id, using kali linux, (please tell me)

I see things have progressed in debian bookworm, they've posted a new version which seems to include recent git code.
3.5.0+git20211031.e6458ec-1
Thanks to who ever is responsible :+1:


Add reusable Encoder API, similar to encoding/json

This also includes an async encoder that will encode objects from several goroutines to a single io.Writer.
The example on the from page would look like this - much cleaner, and similar to encoding/json.
import "github.com/pquerna/ffjson/ffjson"

func EncodeItems(items []Item, out io.Writer) {
    // We declare the buffer here, so it can be reused.
    var encoder := ffjson.NewEncoder(out)

    for _, item := range items {
        // Encode into the buffer
        _ = enc.Encode(&buffer)
    }
}
```.

As a side note, the async encoder is designed so it is possible to extend it later and have several encoders running at once.
But for now, this allows you to encode while other stuff is running, as well as sending data from multiple goroutines. This could be used for logging, message processing, etc.

Encoder makes sense to me.
EncoderAsync I'm having a harder time philosophically aligning with ffjson:

The standard-library doesn't include any equivalent, so we would be educating the world on why this interface is a good one.
It could be a separate library, with no performance penalty (uses no internals)
I can see some use cases for EncoderAsync, but it seems like often times you would want to have more direct control over concurrency and ordering in a real world application.

I see your point. Actually, as you say, it could be a fine generic package, and it could even be used with encoding/json.
I will take it out of the PR tomorrow, and probably do a separate package.

Async removed. I will merge the PR.


[Bug]: Inconsistent OperationStatusResult model in tsp

Describe the bug
The current OperationStatusResult model in typespec has mandatory opeartions parameter, but according to this. It must be optional.

Reproduction
Inspect OperationStatusResult model from Azure.ResourceManager.Foundations.OperationStatusResult
Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

Additional information: in swagger, it is optional as well.
https://github.com/Azure/azure-rest-api-specs/blob/b5512186602fbe490d273aab7ebddc21f8e7f855/specification/common-types/resource-management/v5/types.json#L463-L468

Fixed in pending PR. https://github.com/Azure/typespec-azure/pull/945


Nick: Watch beedatimes.com

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

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

Approved by Ollie in Charcoal HQ


feat(Sqllab): Enabling selection and copying of columns and rows in sql lab and dataset view

SUMMARY
Problem

Currently, the users are not able to copy column headers in Sql Lab results section.
The table columns and rows are not selectable while viewing the datasets metadata before creation.

UseCase

The copy column name/rows is required to utilise the column names easily while creating the metrics and calculated columns.

Technical Details

[x] Enable user-select to text in AntTable and VirtualTables in superset-frontend/src/components/Table/index.tsx and superset-frontend/src/components/Table/VirtualTable.tsx

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE

AFTER

TESTING INSTRUCTIONS
SQL LAB

Go to Sql Lab
Select any relevant database
Write and execute and sql query
In the results section, the query results should appear
Try selecting column from column header content in the results section

DATASET

Go to Dataset and create a new dataset
Select relevant database,schema, table. A dataset metadata view will get generated
Try selecting the columns or rows from the table view

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] 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


Store snapshots in localStorage and prefetch from gateway

Tweak snapshot ui to show timestamp and give enough space to see full hash, and use a monospace font to make the length predictable.
Store and load snapshots from localStorage to persist them across sessions.
Prefetch snapshots from gateway to speed up first render.
Add local createdAt time property to snapshots. (ISO 8601 string)
Adds title property which is now the mutable doc title. name is immutable and part of the url.  See: #62

Fixes #56
Fixes #54

👓

Some of my best friends are human, and they love ISO timestamps. and by friends, I mean me. and by human, I mean 🤖 👍


Add Scorecard Action to monitor Keras' security posture

Fixes #18906.
This PR adds the Scorecard Action to monitor Keras' security posture.
I can also add a badge to the project's README if you're interested:

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

Comparison is base (ef339e8) 79.44% compared to head (b296afc) 55.95%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18907       +/-   ##
===========================================
- Coverage   79.44%   55.95%   -23.49%
===========================================
  Files         336      336
  Lines       34861    34867        +6
  Branches     6852     6855        +3
===========================================
- Hits        27694    19511     -8183
- Misses       5585    13886     +8301
+ Partials     1582     1470      -112

Flag
Coverage Δ

keras
55.95% <ø> (-23.35%)
:arrow_down:

keras-jax
?

keras-numpy
55.95% <ø> (-0.01%)
:arrow_down:

keras-tensorflow
?

keras-torch
?

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

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


storage/sql: "signing_key_public" is spelled wrong

2016/10/14 10:00:50 failed to rotate keys: update: no such column: singing_key_pub

closed in #615


Error running fitting example

Hi Peter,
I updated to Version 2.3.2 and trying to run the fitting example. I got two errors in fitting.py:
First:
Traceback (most recent call last):
File ".\example.py", line 28, in
Cl = af.fit_xfoil_data_Cl(plot_fit=True)
File "c:\users\lenovo\anaconda3\lib\site-packages\aerosandbox\tools\airfoil_fitter\airfoil_fitter.py", line 154, in fit_xfoil_data_Cl
verbose=self.verbose,
File "c:\users\lenovo\anaconda3\lib\site-packages\aerosandbox\tools\fitting.py", line 85, in fit
weights /= np.sum(weights)
TypeError: No loop matching the specified signature and casting was found for ufunc true_divide_
I changed line 85 with:
np.divide(weights, np.sum(weights), out=weights, casting='unsafe')
which removed the TypeError. All weights are zero ???
Second:
This was a data lenght mismatch at line 113
Traceback (most recent call last):
File ".\example.py", line 28, in
Cl = af.fit_xfoil_data_Cl(plot_fit=True)
File "c:\users\lenovo\anaconda3\lib\site-packages\aerosandbox\tools\airfoil_fitter\airfoil_fitter.py", line 154, in fit_xfoil_data_Cl
verbose=self.verbose,
File "c:\users\lenovo\anaconda3\lib\site-packages\aerosandbox\tools\fitting.py", line 113, in fit
f"The supplied data series "{key}" has length {series_length}, but y_data has length {n_datapoints}.")
ValueError: The supplied data series "alpha_indices" has length 37, but y_data has length 868.
Can you please have a look at these errors.
Regards,
John

Hi @joyo44 !
Thank you for the bug report! We've looked at this issue and fixed it in our develop branch; changes will be pushed to master in a few days as part of the AeroSandbox v3.0.0 release. :)
Thanks,
Peter


Consolidate and document environment variables

These were spread out before, now they're in one place.
Fixes #1538
r? @alexcrichton
I'm also missing an entry on CARGO_PROFILE, what does it do?

CARGO_PROFILE info added

I don't believe that all of these environment variables are inputs to the build script, some of them are only set for build scripts (e.g. OUT_DIR). In that sense it may make sense to have one page for environment variables but there should probably be sections along the lines of:

Environment variables Cargo reads (aka CARGO_HOME)
Environment variables Cargo sets for build scripts
Environment variables Cargo sets for crates

What's the status on this? I was just looking for this information in the cargo docs.

@alexcrichton updated for the feedback. I am 99% sure I got these right, let me know if I missed something.

Looks good to me, thanks @steveklabnik! Just a few dangling links but otherwise r=me

@bors: r=alexcrichton

:pushpin: Commit e596ed3 has been approved by alexcrichton

:hourglass: Testing commit e596ed3 with merge 288d458...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64

Nice that this finally got merged :)
http://doc.crates.io/environment-variables.html seems to 404 right now. Is this just a cache thing?


Can't set toolbox category colour

I want to change the toolbox category color like described here: https://developers.google.com/blockly/guides/configure/web/toolbox#categories
What I did so far:
Ardublockly.TOOLBOX_XML =
'<xml id="toolbox" style="display: none">' +
 '<category name="Control" colour="210">' +
  '<block type="controls_if"></block>' +
  '<block type="controls_whileUntil"></block>' +
  '<block type="controls_for">' +
'</category>' +
'</xml>'

What it displays:

There should be a colored block next to the category name.
Thanks in advance

I was able to resolve the issue by myself:
You need to modify .blocklyTreeRow in ardublockly.css to:
.blocklyTreeRow {
  background: none !important;
  /*border: none !important;*/
  margin: 0px 0px !important;
  padding: 7px 16px 7px 0px !important;
  box-sizing: content-box !important;
}


fix(tabs): update mat-tab-link to set aria-current when active

mat-tab-header already does this:
https://github.com/angular/material2/blob/abc3d38c57146443c848d5ba26fd2fab8ca185d6/src/lib/tabs/tab-group.html#L11
And you can see aria-selected is being set properly on the main tabs demo page
https://material.angular.io/components/tabs/overview.
But mat-tab-nav-bar (on the same page, but no live demo) doesn't set aria-selected at all.

LGTM.
The tab-link should have role (#11410) and aria-selected.

Thank you! Rebased and ready for further review.

Updated from aria-selected to aria-current per discussion on https://github.com/angular/material2/pull/11410. nav > link pattern should use aria-current to indicate the current active nav link.

Per offline discussion, Angular Material is not interested in this fix for now pending further analysis. Users are expected to implement their own indicator on which of the mat-tab-link is active. A good candidate seems to be https://www.w3.org/TR/wai-aria-1.1/#aria-current.

Reopening since it looks like this is the way we want to go.


Missing NuGet setup (OSOE-456)

Lombiq.Hosting.MediaTheme.Deployer has no NuGet metadata configured in its csproj.
NuGet shields apart from Lombiq.Hosting.MediaTheme's are missing from the Readme.

Jira issue

This was fixed under https://github.com/Lombiq/Hosting-Media-Theme/pull/15/files.


Question about top auc metric

Thank you for sharing your code and your work on benchmarking.
I have a question regarding the top_auc metrics especially this part:
https://github.com/wenhao-gao/mol_opt/blob/caa98f339fb11b3a7b12f3fd5df1422daea27fb2/main/optimizer.py#L46-L48.
If I understand correctly, considering we have a dictionary of smiles and their corresponding scores, the conditions will always evaluate to true when the uniqueness is less than 1. This could potentially create issues if the model generates a large number of duplicate molecules.
One edge example would be a model for which when he generates a smiles, will necessarily generate 2 other identical smiles. This means the model will have 1/3 of the 10,000 molecules unique. Let's assume this model learns and so the score of each new unique molecule generated increases across oracle calls. In this case I think the current implementation of the top_auc include a biais. I think it's clearer with graphs:
top_auc = 0.5831

top_auc = 0.4995

values added to sum consist of [(top_k_scores + prev) / 2] * update_frequency in the for loop (len(scores) - limit_update) * [(top_k_scores + prev) / 2] and (len(results) - len(scores))  * [top_k_scores].
In this case, I think it is apparent that removing duplicates creates a bias to the top_auc computation. I am wondering if I am missing something and if this is expected? Could you enlighten me on this ? What is the intuition behind it ?

Thank you for your comment. The storage buffer for molecules in the form of canonical SMILES is implemented as a dictionary. This ensures that identical molecules do not appear more than once.

@wenhao-gao Thank you for your answer.
My main concern is that it does not track molecules appearing more than once. As I tried to explain above, the fact that you are keeping only one identical molecule and not every molecule creates bias in my opinion.
Right now, duplicate molecules are treated as if they have the best score of the run. which creates the final plateau on the graph above.
Having every molecule makes it so that a model which increases its score linearly over time has a score of 0.5 (in this corner case we are reaching the best score at the last call). A score of 0.5 for this corner case makes sense to me. As I understand it, the top auc will measure how quickly a model will converge to a high score. So linear convergence can be seen as a random scenario for regular auc, ie one of the worst case scenarios. The issue with the current top auc is that the score is of 0.5 only if there are no duplicates.
Another example would be if the model stagnates on a subset of molecules at a small score. and then manage to reach high scoring molecules at the end. This will look like the following graph. You can clearly see that the plateau is not taken into account for the current auc. And this plateau could have been at the start with the score of 0 or at the end with a score of 1, the top_auc metric would not have changed.
top_auc = 0.959725

top_auc = 0.584775

I understand the examples I am showing are corner cases since they present a lot of molecule duplicates. However, I think this issue might impart the score in real cases as well, it is just less visible.
Once again, I am not 100% confident which is why I would like your opinion on this. If you could explain your reasoning for keeping only the duplicates it would be appreciated.


0.6.9

Coverage increased (+0.04%) to 21.68% when pulling d2f77237a1fda0d649800081b5f62e779596f0e1 on 0.6.9 into 889315abe4649131a44119f30fea504008cc9672 on master.


Samples:Host Exerciser Enhancement

Checking the capability set or not for the binary.
If capability not set with non-root user then display the error.
If capability set with non-root user then read the performance counter and display.
Added libcap-devel for capability support.
Signed-off-by: ramyavv<EMAIL_ADDRESS>

Coverage decreased (-0.04%) to 63.877% when pulling 1ce71099d134acacb28ea7644d0f0d45bfeb46e4 on ramya/host_exerciser_rev6 into 20331e26bf6f58df02e2c594668354a61e133537 on master.


What is a terminal?

Hey guys, I know it might sound silly at first, please don’t take this as snark :) I couldn’t find a definition for terminal anywhere in the Readme and I genuinely want to know what it opens on the different platforms. On windows is it command prompt, powershell or ubuntu for windows? Can I choose which one in config? It’d be great to have that communicated somewhere in the “terminal” section of the readme. Thanks for reading.
I

G'day @DannyMcGillick !
On Windows, the default behavior is to open powershell, although you can set it to use cmd by default (which is what I do).
On Linux, macOS, and other *Nixes the default is bash - again, you can change it in the Settings >> Shell Override.

The relevant code snippet is below (from lib/termination.coffee):
        shell:
          title: 'Shell Override'
          description: 'Override the default shell instance to launch.'
          type: 'string'
          default: do ->
            if process.platform is 'win32'
              path = require 'path'
              path.resolve(process.env.SystemRoot, 'System32', 'WindowsPowerShell', 'v1.0', 'powershell.exe')
            else
              process.env.SHELL || '/bin/bash'

We really should document this - thanks for pointing it out!

Thankyou for being so understanding, and helpful!
On Tue, 27 Feb 2018 at 14:13, Fred Barclay<EMAIL_ADDRESS>wrote:

G'day @DannyMcGillick https://github.com/dannymcgillick !
On Windows, the default behavior is to open powershell, although you can
set it to use cmd by default (which is what I do).
On Linux, macOS, and other *Nixes the default is bash - again, you can
change it in the Settings >> Shell Override.
[image: shell]
https://user-images.githubusercontent.com/11165995/36708595-5f5ed66e-1b39-11e8-9dd9-895f42868aec.png
The relevant code snippet is below (from lib/termination.coffee):
    shell:
      title: 'Shell Override'
      description: 'Override the default shell instance to launch.'
      type: 'string'
      default: do ->
        if process.platform is 'win32'
          path = require 'path'
          path.resolve(process.env.SystemRoot, 'System32', 'WindowsPowerShell', 'v1.0', 'powershell.exe')
        else
          process.env.SHELL || '/bin/bash'

We really should document this - thanks for pointing it out!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fred-Barclay/Termination/issues/70#issuecomment-368732956,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC0j1cP0kLx6FfC5iu4EOY4Dmay10LZWks5tY3LvgaJpZM4SURAg
.

No worries mate. Cheers!


Missing required props warning for

https://github.com/oku-ui/motion/blob/584e368e13cd3a63f56072f0a7c4c4e48731f822/packages/vue/src/components/Presence.vue#L23-L27
I suppose exitBeforeEnter and initial should be declared optional to suppress this warning and align with the docs.

Any update for this @YeSuX ? 😁

@zernonia :D He is very busy and can't take care of the library, so I will take a look at this.

In 2 days I will release the new version.


[Suggestion] Change repair material

It would be great, if you could change the repair material of the construction wands. For example, I changed the Stone Wand to a Netherite Wand (I changed the recipe, the texture, the name and tweaked the numbers in your config). The issue now is that you can still repair it with Cobblestone instead of Netherite. So it would definitely be a cool feature for Modpack developers to have a list for each tier in the config which defines the items to repair the Construction Wands, instead of the Vanilla tier system.

I found a solution using PropertyModifier


Cache params ordering issue

In the first commit, I represented the current problem with cache. Now, query params {a: 1, b: 2} and {b: 2, a: 1} are resolved as different ones.
In the second commit, I solved the problem with fast-json-stable-stringify library as an example. I'm opened to discussion: maybe we will find another solution or hardcode stable-json-hash inside core package.

Will close https://github.com/igorkamyshev/farfetched/issues/156


"Redux Store has not been initialized" if UiFramework initialized with custom store

Maybe I'm doing something wrong here, but I am initializing UiFramework with a custom store instance, and as a result the Viewer component throws this error:
BentleyError.ts:400 Uncaught Error: Redux Store has not been initialized
    at Function.get store [as store] (StateManager.ts:89)
    at IModelLoader.tsx:247
    at renderWithHooks (react-dom.development.js:14803)
    at updateFunctionComponent (react-dom.development.js:17034)
    at updateSimpleMemoComponent (react-dom.development.js:16972)
    at beginWork (react-dom.development.js:18687)
    at HTMLUnknownElement.callCallback (react-dom.development.js:188)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:237)
    at invokeGuardedCallback (react-dom.development.js:292)
    at beginWork$1 (react-dom.development.js:23203)
    at performUnitOfWork (react-dom.development.js:22157)
    at workLoopSync (react-dom.development.js:22130)
    at performSyncWorkOnRoot (react-dom.development.js:21756)
    at react-dom.development.js:11089
    at unstable_runWithPriority (scheduler.development.js:653)
    at runWithPriority$1 (react-dom.development.js:11039)
    at flushSyncCallbackQueueImpl (react-dom.development.js:11084)
    at flushSyncCallbackQueue (react-dom.development.js:11072)
    at scheduleUpdateOnFiber (react-dom.development.js:21199)
    at dispatchAction (react-dom.development.js:15660)
    at getModelConnection (IModelLoader.tsx:161)

My cursory understanding is that itwin-viewer-react erroneously uses StateManager's store reference when it should be using UiFramework's store reference which allows both. Otherwise I don't know much about StateManager's role.
I am using this workaround in the meanwhile:
await UiFramework.initialize(store, IModelApp.i18n);
// SEE: https://github.com/imodeljs/viewer-components-react/issues/74
(StateManager as any)._singletonStore = store;

Im surprised you're hitting this error, as the itwin-viewer should in fact be initializing UiFramework, look here.
One thing we should probably be doing is exposing the StateManager so consumers can add their own stores to it.

this seems to be the culprit to me https://github.com/imodeljs/itwin-viewer/blob/f255deea50a678140a4a6ac28b04dcb615650614/packages/modules/itwin-viewer-react/src/components/iModel/IModelLoader.tsx#L247
If you can use the UiFramework.store instead, it would default to StateManager.store if no custom store is used. But that might be naive, I'm not sure what else is going on

@aruniverse hit the nail on the head here. The main idea behind the viewer is that consumers should not need to worry about initializing all of the statics in iModel.js. It assumes that consumers would not be calling initialize. That said, we still want to support parameters for the statics where necessary. It seems like maybe we should allow for users to pass in custom stores?
We can also change the above code if you think that will fix your issues in the short term. Have you verified that this resolves it?

I think the best option is to be able to pass a custom store which is what I initially thought would be a prop of the viewer. I do initialize everything myself which I sort of didn't realize I didn't need to do, I have custom rpcs and packages that need to be initialized, and I'm not sure which are or aren't already initialized by itwin-viewer. I believe I need the custom store for itwin-saved-views to work, although I should double check that.

I think Mike should initialize his customs rpcs and static in the post-startup callback in order to not initialize any of the core imjs statics. If theres specific parameters to the core statics he needs to override, we should expose that.
Rather than passing in a custom store to use I think we should try to make use of Core's ReducerRegistry so we can dynamically add to the StateManager.

Out of scope of this thread but while I'm changing it, is it not possible to pass a custom i18n instance to IModelApp.startup using itwin-viewer? It looks like it's always explicitly created for you. I'd like to be able to use translations before the Viewer component is rendered, any suggestions? I18next is static so it's possible but I'm not sure how that interacts with IModelApp's usage of it.

Closing and will register reducers myself as @aruniverse suggested.


Conditional fields don't clear form data when their parent changes

If a field is shown conditionally based on another field's value (e.g., "Other"), then if the controlling field changes value (e.g., from "Other" to something else), the conditional field will be hidden in the UI, but its value remains set in the underlying form property.
Similarly, if one field controls the options in another (e.g., "State" -> "County"), if you change the controlling field value (e.g., change the state), the conditional field's value is not reset and remains the same -- even if it is now an invalid value (e.g., it will show the selected county from the last state rather than clear it).

There are a few fields where we handled this correctly, but many are still not correct.


Adding missing #include

this fixes #2958

@chinz07 Please verify whether this solves your issue.

@hkaiser This solves the issue, thanks!


[9.4.x] ISPN-10248 Allow users to customize NearCache behavior

https://issues.jboss.org/browse/ISPN-10248
Backport of https://github.com/infinispan/infinispan/pull/6999

Merged, thanks


Provide an API for the back button handling

When running an app in Mojo shell, the back button is handled by the application.  The relevant code snippet is https://github.com/vanadium/syncslides/blob/master/dart/lib/main.dart#L41  We need to implement the same functionality in the new Navigator API, but Navigator.pop() requires a context instance which we may not have when the back button callback is invoked.

You should just call ui.window.onPopRoute.  That will send the event to the appropriate listeners.

With @abarth's help, I implemented the back button behavior without any API changes on the Flutter side.  Closing the bug.


SDL binding lacks the button mask constants

Summary
SDL_BUTTON_LMASK, SDL_BUTTON_MMASK, SDL_BUTTON_RMASK, SDL_BUTTON_X1MASK and SDL_BUTTON_X2MASK are not present in the Silk.NET.SDL package (while the related constants SDL_BUTTON_LEFT and such are contained in the Sdl class.)

The SDL bindings are quite incomplete yes, this will be fixed in 3.0 by removing SDL support, but in the meantime: the reason they didnt get generated is likely (this is complete conjecture) due to it calling into a macro to get those, and clangsharp just isnt handling that and isnt giving us the final value

You should be able to just calculate it by hand no problem, and add the constants in your code

This is expected as this is how they are defined in SDL (i.e. as macros rather than enums). We only generate enums where SDL has actually defined an enum, so them appearing in Sdl as consts is to be expected.


text can be hard to read in darkmode

Darkmode integration with student forum makes text hard to read as the color of the text is changed to white while the background is white. See picture below for demonstration. I have marked part of the text that is supposed to be readable.

Chrome extension tab text is hard to read:

Will be fixed in #35


Documentation typos in web/guides/code-splitting

In the 6th & 7th paragraphs you have what looks like a double negative. In paragraph 6:
Why bundle loader, and not import()?
Do you mean Why bundle loader, and not require.ensure()?
OR
Why bundle loader, and not System.import()?
Then in paragraph 7:

We’ve been using it for years and it continues to work while TC39 continues to come up with an official dynamic import. The latest proposal is import(), and we could adjust our Bundle component to use import() instead:

I believe you mean "...and we could adjust our Bundle component to use require.ensure() instead:" ?

In the code example you have:
class App extends React.Component {
  componentDidMount() {
    // preloads the rest
    loadAbout(() => {})
    loadDashbaord(() => {})
  }

  render() {
    return (

Welcome!

        <Route path="/about" component={About}/>
        <Route path="/dashboard" component={Dashboard}/>

    )
  }
}

I believe loadDashbaord(() => {}) should be loadDashboard(() => {}); the "o" and "a" in "Dashboard" are mixed up.

Ah nevermind, I see that you're comparing bundle-loader and import(), so it's not a typo in the 6th and 7th paragraphs. Not sure why I was confused there!

For typos, please just do a PR. Issues are only add noise.
That being said, thank you for the PR! 😄


not actually published to bower

The purescript-node-redis package is not registered with bower, and this repo has no tags.
$ bower install --save purescript-node-redis
bower ENOTFOUND     Package purescript-node-redis not found

I've not had time to finish this, I can release in it's current state, but it's got about 4 commands, hence why I haven't released it yet.
Also I'm working on a better release in the commands branch #2, I'm about 50% done on this.

I saw that branch, it was looking good. I was just pointing out that it says to install it using that name on bower and that doesn't work.

@michaelficarra you spurred me to start back on it :+1:

I have trouble building the version published to bower. I'm getting the error
Error in module Database.Redis.Redis:
Error at bower_components/purescript-node-redis/src/Database/Redis/Redis.purs line 53, column 8 - line 53, column 23:
  Unknown value parseFileURI
See https://github.com/purescript/purescript/wiki/Error-Code-UnknownValue for more information, or to contribute content related to this error.

@michaelficarra I've managed to update the master branch. It's now using the commands. Note that it uses pipelining be default, which is what we wanted to use, it wouldn't be too difficult to not do this I guess, but pipelining gives the ability to pipe multiple queries to the redis instance.
consider:
commands :: Query
commands = do
  set (name "list") "hello"
  get $ name "list"
  mset [tuple2 (name "x") "y", tuple2 (name "a") "b"]
  mget [name "x", name "a"]

:cool:

@SimonRichardson I don't see any git tags. Is that why you've re-opened?

Yeah, I'm not happy to release this until #5 is done.

I've created an initial release, I'll start implementing more commands soon.


New game Language Learner Adventure

PR Description 📜
Objective:
The goal of the game is to help players learn new vocabulary by matching words with their correct translations. Players answer multiple-choice questions and receive feedback on their answers. A scoring system tracks their progress.
Fixes #4257

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗Objective:
The goal of the game is to help players learn new vocabulary by matching words with their correct translations. Players answer multiple-choice questions and receive feedback on their answers. A scoring system tracks their progress.
Fixes https://github.com/kunjgit/GameZone/issues/4257

Thank you  @bhaveshnegi , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Configuration via GIT

Make the path custom/conf a repository inside gogs so you can push configuration changes via git instead of relying on a user logging in via ssh and editing the configuration

It sounds a good idea, but I have security concerns on it.

gitolite uses this technique

Thanks @Kruemmelmonster , it's good to know, but it doesn't necessary mean that Gogs has to have this feature as well.
The better way I prefer is(on the todo list) changing config options through web UI.


Regexp for raising blacklist connection error

We have an update query which raises a Mysql2::Error due to query data string being too long, which is fine since we are running mysql in strict mode. However, the problem was that the error was wrapped by blacklist connection error. Hence the connection was blacklisted, which it shouldn't be. I traced the code and found that the original exception message from Mysql2::Error (which has a huge amount of text) was handled gracefully due to the regexp: /connection[^:]+closed/
https://github.com/taskrabbit/makara/blob/v0.3.10/lib/active_record/connection_adapters/makara_abstract_adapter.rb#L25
The exception message is so big that somewhere it has the word "connection" and elsewhere it has the word "closed". Hence /connection[^:]+closed/ matches it.
So my question is, what does the above regexp and this one:
https://github.com/taskrabbit/makara/blob/v0.3.10/lib/active_record/connection_adapters/makara_abstract_adapter.rb#L21
attempt to match? I'd like to change them to more specific pattern matching but I need to know what the target strings they were designed for. Thanks!

Thanks!


problem with python3.6 use

Hello,
I'm usingp ython3.6 on bastion host with some permissions and getting error with __future__  package not having annotations
error is below.
from python_on_whales import docker
  File "/home/sddc-user/.local/lib/python3.6/site-packages/python_on_whales/components/buildx.py", line 1
    from __future__ import annotations
                                      ^
SyntaxError: future feature annotations is not defined

is there some dependencies needed ? can i define annotations manually ?

Hi! Python-on-whales only works with python 3.7 and above :)

yes - I 've realize that: i'd suggest to make it part of dependencies

The end of life for python 3.6 was one year ago. We can't try to adapt the codebase for all dead versions of python

You are correct, yet many places are still stuck with older versions, thus there always will be someone for whom the project will not work. So for you to have less work and not support those older version, make py3.7 to be part of requirement/dependency.

It's weird because it's already declared there: https://github.com/gabrieldemarmiesse/python-on-whales/blob/dcf2e6e5cbe048d0466932d4f0b1f079428c8207/setup.py#L22
Maybe the declaration is not working?

probably, because i was able to install the package but can not import the object in python shell

@zero-pytagoras Could you check the version of python-on-whales you have installed, with e.g. python3 -m pip freeze | grep whales? My guess is pip is resolving to an older version that does not have the restriction (but perhaps still doesn't work with python 3.6).


Unit tests

We should definetly add some unit tests to validate if the directories are being successfully created

Agreed!

We will enable TravisCI and possibly Tox for this issue


Use container offset in bounds computation

Hello,
When the bounds are provided as html selector, their computations doesn't use the container position. So the draggable can move in the selector area. The area size is the same than the selector moreover the position is different because the bounds don't use the selector offset.
I made the required changes and added a test.

I just spent a few hours tracking down why I couldn't my my div to drag partially offscreen to the left with a top: -200px on the bounding parent and tracked it down to this same thing. Really wish I had seen this PR before spending that time. Would love to see this get merged!

This depends on whether the containing element uses position: relative or not, yes? If so, there should be no need to use boundNode.offset{Left,Top} because the draggable's absolute position is relative to the container.

Yes if the container is a parent of the draggable.
In my case of use, the draggable element is not a child of the html element pointed by the bounds.

It seems that we should perhaps accept a function as bounds, returning coords.

I can confirm that this is basically needed to work properly with CSS grids as well it seems.


Demo: Dialog buttons all open the same dialog

Describe the bug
On the demo page for Dialog, all of the buttons only open the same dialog "Initial Focus"
To Reproduce
Steps to reproduce the behavior:

Go to https://mwc-demos.glitch.me/demos/dialog.html
Click on any of the buttons
See "Initial Focus" dialog

Expected behavior
I think this is obvious. But nevertheless: It should open the respective dialog related to that button.
Screenshots

Browser Version (please complete the following information):

OS: elementary OS 5.1.2 Hera
Browser: Firefox
Version: Dev 74.0 (64-bit)
Also reproduced on Chrome

oops


Cannot expect(setTimeout) without legacy mode of useFakeTimers()

💥 Regression Report
I upgraded to jest 27.0 and found my tests failed due to the change in useFakeTimers().
The error is:
Matcher error: received value must be a mock or spy function

Received has type:  function
Received has value: [Function setTimeout]

It does work when I use the "legacy" option, however my usage is the recommended approach. I don't think it constitutes the "subtle implementation differences" described in your blog.
Last working version
Worked up to version: 26.6.3
Stopped working in version: 27.0.4 (Probably 27.0.0)
To Reproduce
Steps to reproduce the behavior:

In a test file, call jest.useFakeTimers("modern"); (or jest.useFakeTimers() in v27+)
Attempt to use setTimeout as a mock, such as expect(setTimeout).toHaveBeenCalledTimes(0);

The problem might only happen on Windows, as I couldn't reproduce it in repl and everything else was the same.
Expected behavior
It should still work, as this is the technique described in your docs.
Link to repl or repo (highly encouraged)
repl.it demo
NOTE: The repl runs without error, but fails locally on my Windows 10 workstation. I confirmed I have an identical package-lock.json and have cleaned my npm cache etc. Might it be only an issue on Windows?
Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: Windows 10 10.0.19041
    CPU: (8) x64 Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz
  Binaries:
    Node: 14.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - C:\Program Files\nodejs\yarn.CMD
    npm: 6.14.11 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    jest: ^27.0.4 => 27.0.4

seems like a duplicate of #11500

@Smrtnyk They might be related, but I'm describing a different symptom. I don't know if setTimeout is working or not. I don't get past expect(setTimeout) because the fake version of setTimeout does not appear to be a valid mock.

@Smrtnyk I should have read through the other thread before responding. The discussion does seem to be going in the direction I thought this one might, so there is some duplication in that sense. It's the same underlying cause, but reported via a different use case. I see pros and cons to "duplicate" vs "related" but I don't care to split hair over it. :) I'm just glad that others are seeing the same thing.
It's weird that it works in my repl.it though.

Duplicate of #11447


Parsing error

Trading with CRV not yet available on Coinbase: https://pro.coinbase.com/trade/CRV-USD
I added more clear error. Will be included in the next app v2.29.


Add session properties to set task max memories

It would be also very nice if we can control task max memories through session properties for testing and multi tenancy purpose.

I see, it does make sense. I'm closing this request.


load_module() method is deprecated

When I launch my server after migrate to Python 3.11.5, xhtml2pdf raise an error
DeprecationWarning: the load_module() method is deprecated and slated for removal in Python 3.12; use exec_module() instead
File "C:\Users\albat\Documents\DEV\pepienv\Lib\site-packages\xhtml2pdf\pisa.py", line 25, in
from xhtml2pdf.default import DEFAULT_CSS
File "C:\Users\albat\Documents\DEV\pepienv\Lib\site-packages\xhtml2pdf\default.py", line 17, in
from reportlab.lib.pagesizes import (A0, A1, A2, A3, A4, A5, A6,A7, A8, A9, A10,
File "C:\Users\albat\Documents\DEV\pepienv\Lib\site-packages\reportlab_init_.py", line 40, in
fake_import(os.path.expanduser(os.path.join('~','.reportlab_mods')),'reportlab_mods')
File "C:\Users\albat\Documents\DEV\pepienv\Lib\site-packages\reportlab_init.py", line 24, in _fake_import
sys.modules[name] = m.load_module(name)
OS version: Windows 11 (Local Test)
Python version: 3.11.5
XHTML2PDF version: 0.2.11

Which version of reportlab are you using? This rather looks like a reportlab issue (as it occurs when importing the page sizes from reportlab), maybe because of an old version?

Indeed, load_module() is not used in this repository, but only in reportlab.
The bug has been fixed upstream recently and will probably be part of the next official release:
https://hg.reportlab.com/hg-public/reportlab/rev/18ae86d14b0a


May I have your modlist and most recent log?

With the limited knowledge about this issue, I'm assuming it's a duplicate to
https://github.com/MerchantPug/cursedorigins/issues/4


issue #409 API should allow integration with ExecutorService to allow…

https://github.com/binance-exchange/binance-java-api/issues/409
When you make an asynchronous call there is no control of how many threads are used for the requests and responses. Particularly when responses are processed the client needs to ensure data integrity, this could be done using synchronization but its better achieved using intelligent threading and dispatching. This is even more important when connecting to more than one exchange and you need to ensure that responses for the same ticker across multiple exchanges are processed in order.
The most simple way to achieve this is to control the threading model for the async calls, rather than relying on the http library.

Also available here as a bolt on library until (unless) this is merged into master :
https://github.com/antlen/binance-java-api-extensions


Add donate Now button to the Help section.

Add a menu option that will open a web browser to a donate page.

Donate menu option has been added.


InvalidCastExceptions

This is a great little library - thanks for sharing!
One issue I've run across is that PublishImpl calls Subscription.Handle for each subscriber of any message type, and relies on swallowing the InvalidCastException generated for subscribers of message types other than the one being published.  With the debugger attached, this affects performance greatly.  I'd suggest a cleaner way of filtering subscribers based on the message type.  Thanks!

Thanks for pointing this out, this should never happen and if it does then it is indeed a problem.
The exception handling in PublishImpl is there to ensure that a thrown exception by a subscriber's handler does not mess up all the other handlers and it is not there to filter the subscribers.
Let me write some tests to try to reproduce this.

Thanks!  I've fixed it locally by making this change in the Subscribe method, testing that msg is of type TMsg before doing the explicit cast:
                _subscriptions.Add(new Subscription(
                    token,
                    new Handler<TMsgBase>(msg => { if (msg is TMsg) handler.Handle((TMsg)msg); } )));

Yes you can use that as a temp fix however it is still throwing errors. In the UsageExample tests if you subscribe to the OnError you will see the errors.
Actually you raising this issue has prompted me to re-visit some of the decisions I had made, I have now modified and improved the API slightly and also improved the way I subscribe and publish to both fix your issue as well as improving the throughput.
I am testing it at the moment and will be pushing it shortly.

Verified.  Thanks for the quick turnaround!


Change docs to use Pydata theme

Description
This is the theme used by numpy, scipy, pandas, etc and looks much better than our current one.
Also did a little bit of reformatting, started a user guide, etc, but there is more to be done
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Docs look like this on latest build: https://pybamm--2618.org.readthedocs.build/en/2618/
Don't know why the pydata theme isn't being used

Almost ready to go, final problem is the code is not displaying on this page https://pybamm--2618.org.readthedocs.build/en/2618/source/user_guide/fundamentals/index.html (it's not displaying locally either)

Found the problem, I was doing .. code-block::python instead of .. code-block:: python 🤦


uv_fs_write vs UV_FS_O_APPEND

Hello!
If uv_fs_write is equivalent to pwritev(2) according to the manual, it seems to be incompatible with UV_FS_O_APPEND on POSIX-compliant systems.
https://stackoverflow.com/questions/22268115/using-pwrite-to-append-to-a-file
Please note that on Linux pwritev(2) ignores offset when a file is opened with O_APPEND only due to a bug in the kernel - https://bugzilla.kernel.org/show_bug.cgi?id=43178
libuv does not seem to deal with this issue internally, for example, using writev(2) instead of pwritev(2) when a uv_file was opened with UV_FS_O_APPEND.
As a side note, why not just use writev(2) and have an lseek(2) equivalent like uv_fs_lseek to avoid the above issue along with the hassle to track file position when writing? It's always possible to have a separate uv_fs_pwrite for seek&write requests if preferred.

lseek has a separate issue already: https://github.com/libuv/libuv/issues/2378

using writev(2) instead of pwritev

Wouldn't that exhibit exactly the same behavior on Linux, and only diverge if that issue (from 2012) got closed?
Given how we set the flags in CreateFile, it looks like libuv-on-Windows may follow Linux here:

If the caller sets only the FILE_APPEND_DATA and SYNCHRONIZE flags, it can write only to the end of the file, and any offset information about write operations to the file is ignored. The file will automatically be extended as necessary for this type of operation.
https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/content/ntifs/nf-ntifs-ntcreatefile

Anyone know what happens on macOS (and other platforms)?

Yes, I can see now that lseek(2) is a bad idea given the thread pooled nature of libuv's file operations. Forget the side note then.

Nevertheless, the use of writev(2) instead of pwritev(2) seems inevitable to reliably support O_APPEND.

#include <stdio.h>
#include <fcntl.h>
#include <unistd.h>
#include <errno.h>

int main() {
	int fd = open("a.txt", O_CREAT | O_TRUNC | O_WRONLY | O_APPEND, 0755);
	if (fd == -1) {
		perror("open()");
		return 1;
	}
	pwrite(fd, "aaa\n", 4, 0);
	pwrite(fd, "bbb\n", 4, 0);
	close(fd);
	return 0;
}

On Linux and FreeBSD (!!!):
$ cat a.txt
aaa
bbb

On macOS:
$ cat a.txt
bbb

xnu's vfs layer passes the O_APPEND flag to the fs driver so I suppose it's possible that different file systems exhibit different behavior.
The nfs client driver, for example, simply forwards the flag to the server (when it speaks nfsv4.)


About add other loss to the bootstrapped cross entropy

May I ask you how to do the scaling as mentioned in this answer? https://github.com/hkchengrex/STCN/issues/63#issuecomment-928091282
The bootstrapped cross entropy is about 0.04-0.06 before 50k steps, after that is about 0.1. So should I keep the added loss between 0.04-0.06? Or I should keep it around 0.1?

Can you please give me some guidance about how to monitor the gradient? It seems that Keras can use write_grads flag, and tensorflow can use log_histograms flag. But I didn't find any information about pytorch. Does pytorch have similar flags? I still need to implement this in code by myself?


Failed to compile

error[E0432]: unresolved import `std`
 --> /Users/filiplazovic/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ff-0.13.0/src/helpers.rs:3:5
  |
3 | use std::println;
  |     ^^^ use of undeclared crate or module `std`

For more information about this error, try `rustc --explain E0432`.
error: could not compile `ff` (lib) due to 1 previous error

believe this is fixed. If not comment back


Unable to get Fabric Network with TLS Chain of Trust of multiple Fabric CA Servers working!

Repost for publicity: https://github.com/hyperledger/fabric-ca/issues/266
Please comment only on the linked Issue!
Thank you very much!

Please reserve github issues for Fabric code issues.
For getting help, see the community help resources mentioned at:
https://hyperledger-fabric.readthedocs.io/en/latest/CONTRIBUTING.html#getting-help


Oops typo in the image

Sorry everyone, i've typed ] instead of } in all images. This is the fix.

Closed. See #77.


Investigate Visual Studio Integration

An item to discuss/consider/prototype Visual Studio Integration:
An integration could take one of two paths:

A standalone VSIX that has a dependency on https://github.com/adamdriscoll/poshtools/
A library that the aforementioend poshtools has dependencies upon

A completely standalone VSIX would not work if we intent to handle DSC, due to bitness restrictions.
To maintain agility, and to get to a minimum viable product sooner, we recommend the first option.
A dependency can be checked both at install time [as part of the VSIX Manifest], or at Runtime, by Querying the Visual Studio Service provider for the IPowerShellService.  See https://github.com/microsoft/poshtools/blob/dev/PowerShellTools.PublicContract/IPowershellService.cs
The fastest way to a Proof of Concept, is to use the IPowerShell service and craft the powershell command to run upon the selected file.  The output would be emitted as text.
The next step could be an output to a separate window, in a more friendly and usable manner
A further step could be real time analysis, but, to be performant, it would work nicer if the service exposed the most recent AST [See https://github.com/Microsoft/poshtools/issues/496 ]
General Information about VS Extensibility can be found at https://msdn.microsoft.com/en-us/library/bb165336.aspx but our team is always happy to help with advice :)

We are working with PowerShell EditorServices team to add ScriptAnalyzer support in Visual Studio

VS Code has ScriptAnalyzer integration now

@raghushantha - This was titled "Visual Studio Integration" and that is still missing in VS 2017 Ent and other editions.
If its not missing, please tell me why its not flagging errors and warnings in an open script.


Redo pr 649

@aspeddro this should work, only needs cleaning up the changelog.

I thought it was fixed in master. Thanks @cristianoc.

These are 2 different issues:
1 references to elements defined in interface files
2 references from implementation files which also happen to have interface files

1 in master
2 here

All done now.


Update README

Coverage remained the same at 100.0% when pulling 07fc0125da52b9b4dd767c2430ceaa2bdfa6221d on update-readme into c938584bb5aac248e8736caf493f7bf601091aa7 on main.


COPR: Fix build and remove unused files

Makefile: add mock-rpm target
Signed-off-by: Lokesh Mandvekar<EMAIL_ADDRESS>Let's wait for netavark copr build to succeed.
@Luap99 @baude @rhatdan @flouthoc @mheon PTAL

LGTM

/lgtm

thanks, i'll remove work-in-progress label once the netavark build succeeds.


strike_wizard in Simple displayMode click event incorrect parameters

When leveraging the click event in displayMode="Simple", the event does not populate the results correctly. In the function "renderSimpleMode", you can see the for loop should not be setting the results.current unless it finds the chevron the user has just clicked on. The single line below should not exist so that the correct "current" chevron is passed to the event.
for (var i = 0; i < chevrons.length; i++) {
var chevron = chevrons[i];
chevron.class = '';
if (i == targetChevron) {
chevron.active = true;
chevron.current = false;
results.current = i || 0;
chevron.class += 'slds-is-active slds-is-incomplete';
} else {
chevron.active = false;
chevron.current = false;
results.current = i || 0; // REMOVE THIS LINE!
chevron.class += 'slds-is-incomplete';
}
}

i'm not seeing this happen when i test this. could you be more specific as to what is happening and what your issue is because i'm always getting the correct current chevron in the event.


Adding Realism to Simulated Sensor Data

Purpose

Tasks the progress of development of error generation classes or functionality. The high-level details are outlined on confluence.
Noise (error) and delay will be added to any data that is output from sensors and published to ROS2 topics. The data that will be impacted by noise and delay is listed here on confluence.
This is representative of real sensors just having occasional spikes in readings or delays in readings when they simply don't update for a short period of time.
Tasks

Noise Generation

[ ] Design Sensor classes: WindSensor and GPS and update the Output Interface section on Confluence
[ ] Write boilerplate Sensor class based on specs
[ ] Implement noise generation function

Delay Generation

TBD...
[ ] Implement delay generation function

We want the Sensor to be responsible for only sensing data and applying noise then sending the data elsewhere for it to be sent over a ROS topic.
Noise will come from distribution (specific to each type of data). Could be multivariate, univariate, or constant. Come from the generator classes we've created.
I learned that delay is a function of the Sensors and not the act of publishing. That is, the delay is caused by sensors not updating their values right away.


Why did i get banned

I got banned for xray For no reason i never use xray and some how i got banned for having 6 stacks of diamonds So i my user name is OVmaster123 i would like a answer

You were banned for directly digging down through multiple layers of stone to diamonds, and amassing 7 stacks of diamonds, while attempting to sell them to players in exchange for money.

Over a month with no reply. Closing.


Can you specify the order plugins run in relation to each other?

Describe the bug
This may not be a bug. Likely isn't a bug. It's a question that I can't find any documentation on. We are upgrading from 1.0.0-beta.111 to the new release version 1.0.0.
We have two preprocessor plugins that we've developed and have been using with the beta versions for the last 2 years or so. They've been working great.
The first allows us to write something like this instead of a description
get:
  operationId: getSomething
  summary: Get Something
  x-description-ref:
    $ref: ../description.md
  # We used an x-description-ref rather than a `definition`

Which allows us to type a longer description right in a markdown file. The plugin then resolves the reference to the file and puts the contents of the file as the value of a description and then deletes the x-description-ref key.
The second is called replace-macros and it replaces constants like _OUR_MACRO_ with a predetermined string. (I would imagine something like this is already built into redocly and maybe we don't need this plugin any more?)
The first is configured to run on enter of a node and the second one is configured to run on leave of a node. It's important that for each relevant node that the replace-macro run after the other preprocessor. If it doesn't then none of the macros in the referenced file will get replaced.
What is the algorithm for running plugins?  If so, can they be ordered? Or alternatively do they get interleaved?
What we are seeing is that all of replace-macros runs first before the first invocation of the x-description-ref plugin.
If we revert back to the old version it continues to work as desired.

It should work in the order you expect. enter should be always executed before leave. Something must be wrong.
Could you share more details on your preprocessors? The code of the plugin would be amazing but if you can't share it for some reason then the general layout would be enough.

Do you visit the graph multiple times: once for each plugin? Or do you visit the graph just one time and on each node you run the skip, then enter then leave methods of each visitor?

The latter one is how we do it.

"visit the graph just one time and on each node you run the skip, then enter then leave methods of each visitor".

But we do separate traversal for every stage: preprocess, lint, decorate. Are both of your visitors registered as preprocessors indeed?
This is the only reason I can think of for the behaviour you observe (other than us having some bug for sure 😄).
If they are, could you please share some minimal reproducible sample please?

We haven't had an update on this for a while so I'll close, but please let us know if you have more information on this one. If we can replicate it, we'll take another look.


Is it possible to generate glue code from an existing C++ header?

I understand the approach according to documentation is to:

define you djinni file
generate hpp/objc/java
add cpp implementation

But for my use case, I'd love to be able to:

given an existing C++ .h file
generate objc/java glue code

Is what I'm explaining currently possible?

Hi Dominic,
no, this is not possible.
You require either a djinni IDL file, or a yaml file in which you can define types that are not in the IDL language available.
But feeding in a header and magically get Java / ObjC bindings is not possible.

Thanks for the quick response @a4z


Fixed typo in deliver options.rb

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
screeenshot has too many 'e'
Description
screeenshot => screenshot

@googlebot I signed it!


[BUG] beforeHandle needs to be wrapped in array when used in a global Guard

I'm trying to use guard on global schema method to authenticate users
app.guard({
    beforeHandle: async ({ cookie, accessToken, set }) => {
      const token = cookie.accessToken
      const user = await accessToken.verify(token)
      if (!user) {
        set.status = 401
        return {
          error: "Unauthorized"
        }
      }
    }
  })

but for some reason it throws the following error:
TypeError: (Y?.beforeHandle??[]).map is not a function. (In '(Y?.beforeHandle??[]).map(X)', '(Y?.beforeHandle??[]).map' is undefined)
      at C1 (/Users/axeloehrli/Projects/ganadexx-api/node_modules/elysia/dist/bun/index.js:8:12945)
      at guard (/Users/axeloehrli/Projects/ganadexx-api/node_modules/elysia/dist/bun/index.js:272:23994)
      at /Users/axeloehrli/Projects/ganadexx-api/src/users.ts:23:3
      at processTicksAndRejections (:1:2602)

The solution is to wrap beforeHandle in an array, which i believe is a bug:
  beforeHandle: [async ({ cookie, accessToken, set }) => {
      const token = cookie.accessToken
      const user = await accessToken.verify(token)
      if (!user) {
        set.status = 401
        return {
          error: "Unauthorized"
        }
      }
    }]

https://github.com/elysiajs/elysia/blob/741ffe7659cf049c8627d30b91cd1f05cfbab53e/src/utils.ts#L68-L71
vs
https://github.com/elysiajs/elysia/blob/741ffe7659cf049c8627d30b91cd1f05cfbab53e/src/utils.ts#L260-L263
Maybe mergeObjectArray could take an argument that will apply a mapping after it converts singular items to arrays?

Seems not to happen in the latest version, closing.
Feels free to reopen if the issue still persists.

i'm getting this in v0.8.8 currently


qemuarm64-secureboot: Add UEFI Secure Boot v2

Hi,
Addressing comments from patch series v1 plus other comments from Mikko Rapeli<EMAIL_ADDRESS>A backport from meta-ts with the minimal changes to add UEFI Secure Boot into qemuarm64-secureboot machine.
Requirements:

Create a UEFI disk partition to copy EFI apps.

Add UEFI settings to U-Boot, systemd-boot, and Linux kernel.

UEFI keys are to be stored in U-Boot and used to sign systemd-boot and Linux kernel images.

Add systemd as Init manager to auto-mount efivarfs.

Introduces uefi-secureboot machine feature.
UEFI keys must be genereated in order to be added to U-Boot. Sign both systemd-boot EFI app and Linux kernel image.
Build and verification steps:
$ kas build 'ci/qemuarm64-secureboot.yml:ci/testimage.yml'

Changes since v1:

Rework all subject commits to follow OE, Yocto, and meta-arm guidelines.
Add gen-uefi-sb-keys.bb recipe to generate UEFI keys.
Add an OE test to validate UEFI Secure Boot.
Simplify gen_uefi_keys.sh to avoid code repetition.
Replace grub with systemd-boot.
Simplify signing binary images with sbsign class.
Set OE branch to Scarthgap

Changes since the v0:

Remove u-boot recipe.
Split the change in several commits.
Remove sample UEFI keys.
Validate UEFI keys exist before building.
Insolate most of changes under uefi-secureboot machine feature.

@

Subject should be "linux-yocto: sign kernel image..."

Done. Thanks!

Could you prefix each patch subject with recipe it actually changes?
That's the pattern in poky and meta-arm. Then if possible, keep
changes separate to each recipe and main config file (machine,
kas etc).

Done. Thanks!

Subject should have "systemd:" prefix.

Done

Is there some way to test this in oeqa runtime with ssh that
the boot was really done with secure binaries?
I think this is quite brittle and test should verify that
boot was secure.

I have added a test to validate UEFI Secure Boot is enabled. Thanks!

@mikkorapeli-linaro I believe all your comments have been addressed. Please, take another look. Thanks

One small issue found. With that fixed, please submit to meta-arm. Lets see what they say...

One small issue found. With that fixed, please submit to meta-arm. Lets see what they say...

Submitted to meta-arm. I have added another commit to set in kas the release branch to scarthgap because Secure Boot is broken in master. Maybe is the Linux kernel.

Replaced by https://github.com/jetm/meta-arm/pull/2


安装过程问题求助

CUDA完成安装 环境报错
NVIDIA-SMI has failed because it couldn't communicate with the NVIDIA driver. Make sure that the latest NVIDIA driver is installed and running. This can also be happening if non-NVIDIA GPU is running as primary display, and NVIDIA GPU is in WDDM mode.

补充

@Childish-Ghost 检查一下你的显卡驱动安装好了吗


Scaling security monkey

Can we have a section in the documentation on how to scale security monkey in the production environment? Has there been any testing performed to check to see how well the tool performs in each category.
Examples of architectures we can probably investigate -
- Architecture# 1 (probably worst case scenario and would definitely work): create separate instances of security monkey and rds databases for every couple of dozens of accounts (depending on the load)

.
. (many models in between)
.

- Architecture# N (probably best case scenario and need to research this from various angles): think of scheduler and web as micro instances. Run web on less number of nodes with Security monkey and scheduler on the nodes given the # of AWS accounts that scales based on the instance utilization

Hey @pklanka,
@ollytheninja did some incredible work in this area: #318
The ApScheduler v3 should allow us to have a cluster of security_monkeys each running separate tasks.  The tasks would be persisted in an RDS datastore.  As you can see, I haven't had time to merge that in yet.  For the current architecture, I feel that is the way it must go.  This is very similar to your "Architecture # N".  If anyone has the skills and the time, I'd love to see contributions here.
We're working on a complete re-architecture that should eliminate scaling problems.  lambda functions are triggered by eventrule cloudtrail changes.  The re-architecture is mostly server-less except for the web UI.  Polling the AWS API's will mostly be eliminated.  We don't have a release date for that yet.  The new architecture may not work in environments where lambda isn't approved.  (I don't think gov-cloud has lambda, and I'm not sure if healthcare organizations can use it either. -- I should find someone and verify.)


Send 2 Unreal - cannot import name 'make_remote'

Bug
cannot import name 'make_remote' from 'send2ue.dependencies.rpc.factory' (C:\Users\Dragosh\AppData\Roaming\Blender Foundation\Blender\4.2\scripts\addons\send2ue\dependencies\rpc\factory.py)
When trying to turn on the addon.
Blender 4.2.1 - MS Store version

This sounds like an issue with python modules not reloading after installing a new addon code. Can you:

Deactivate and remove send2ue addon
Shutdown blender
Start blender
install send2ue addon and activate it

Let me know if that works thanks

Yeah, seems it worked now for some reason after doing it on and off like 3 times, weird.
However now a different issue arose.
After trying to push assets I get this:
Python: RuntimeError: Error: Image is not editable

TIL blender is on the MS store. I will see if I can replicate the issue

light_collection_test_1.zip
Here

Hmm okay I'm not getting that error still. Can you please try again and screenshot/paste contents of the console?

I wonder if it's a file permissions issue?

Just tried opening my "main" file, which is 400 megs big, pushing assets from that gives me reliably the said error.
Here's a drive link to it.
https://drive.google.com/file/d/11RlWohtDnvO5f9Z7tOC2CLV97Op9WadW/view?usp=drive_link

Hmm ye still not getting any issues aside from a couple empty objects. I'm thinking it could be happening when trying to load Blender's FBX module. Since you are using the Microsoft Store version, I'm thinking there are permission issues.
My current suggestions are to find where the MS store version gets installed and change the permissions to allow full access, or you can try downloading Blender from blender.org (or Steam if you still want the easier updates)

I think this is related to the unpacking of textures to disk so that FBX can read them correctly on export.
https://github.com/poly-hammer/BlenderTools/blob/eb00c749664ee794b4c6918c05884157926f30c9/src/addons/send2ue/core/utilities.py#L1625
My guess is since this is only happening with this certain file, you might just try unpacking and see if that gives the same permission error about the image texture.

I would check the folder permission where the .blend file is located

Tried unpacking, stull the same error occurs

Could be permissions issue with either textures location, blender install, or project folder

Well i use a Blender UWA from the MS store because i really dislike the normal install version. The UWA apps are sandboxed and heavily walled up from the rest of the system and some other access restriction, so it could be that. This might be something a Windows developer could know more about.


Future-proof against addition of Data.List.!?

base-4.19 (and GHC 9.8) will export Data.List.!?, which will clash with Data.List.Extra.!? in
https://github.com/ndmitchell/extra/blob/374ebbc12cb4b9580394556c5e5c7b1bf250d817/src/Data/List/Extra.hs#L164-L170
This PR wraps the definition of Data.List.Extra.!? in CPP to prevent the name clash.
See https://github.com/haskell/core-libraries-committee/issues/110#issuecomment-1359037078 for CLC discussion of the change.

Thanks!


Source jenkins/utils.sh in public cloud CI scripts

These scripts call the check_and_cleanup_docker_build_cache function.

@XinShuYang I validated this change for all 3 cloud jobs

/skip-all


How to treat in the network bids with anonymous bidders?

There are tenders like this one that have winning bids but the winners are not listed.
How should we treat these in the network? Should we even show them?
Currently the contracts add up to the buyer node and the bidder node and their edge is not drawn.
I think this is fine but there are case like this one when there is no other bidder to draw an edge to and we'll have a lonely buyer node:

Tbh I have no problem with that but I'm not sure there is any value for the journalists to see only the buyer.

Look like in the new version of data that tender does have the winners listed.
I think this calls for a check if we still have lots that awarded but whose winners are not disclosed.


Add File Based support for C

Platform: linux
In Linux Environment I have noticed, atom script extension not works for C. When I run script I got an error:
File Based runner not available for C.
If it should exist, add an issue on GitHub, or send your own pull request.
It only works in Mac, that's not good :-(

If you've got a crafty way to make it work with gcc, feel free to submit a PR.


Item Window Will not Open

Describe the bug
When I go to Items → Crafting Systems it does not do anything.
To Reproduce
Steps to reproduce the behavior:

Go to Items
Click on Crafting Systems

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

https://user-images.githubusercontent.com/29180106/210413114-bc1f5350-4208-4b5b-969e-8b7926ba2a4c.mp4

Version information

Foundry version: [e.g. 10]
Fabricate version: [e.g. 0.7.2]

Additional context
I am running on FATE which is not one of the "big" rule-sets but I don't think that could cause it.

I can confirm it works on D&D but not FATE.

That's fun! Thanks for the report 🐛
I suspect that it is trying to load Alchemist's Supplies, which depends on 5E. The system data is good, so it's looking for the compendium Items. I don't think foundry is loading them, as the compendium also depends upon 5E.
This can be fixed in a few ways:

Loading crafting systems and not displaying those that fail, or showing them as being in error in the UI
Tagging systems with the game system they are created in, and then not loading them if the game system isn't present

Arguably I could do both.

Is there anything else I can do to help you? And thanks again for your support your add-in is pretty awesome.

That's all the information I need to work the Issue, thanks! The only thing I could ask for above and beyond than this excellent, detailed bug report is the code to fix the issue 😉

The fix is simpler than I first thought in one way, but more complex in another.
The way I'm going about solving it is to no longer store bundled systems in world data. That wasn't necessary, as the data ships with the module and cannot be edited. This requires:

Deleting copies of alchemist's supplies in world data on startup
A change to system duplication to use the data in memory, rather than in world settings
Combining bundled system data and stored data when loading all systems
Filtering out bundled systems with game system dependencies

I'm pretty optimistic about getting this sorted quickly though. Will keep you updated.

Now opening in pf2e.

And in fate-core-official

And in dnd5e, where Alchemist's Supplies v1.6 is no longer stored in world settings:

But there's another minor, unrelated issue to resolve now. It won't (or should not) impact you and is described in #73


Fix reactions-avatars conflict with new style

Fix for #4674

Test URLs
Regular comment: https://github.com/sindresorhus/refined-github/issues/4674
Review: https://github.com/sindresorhus/refined-github/pull/4667#pullrequestreview-730139417
Review comments: https://github.com/sindresorhus/refined-github/pull/4667#discussion_r689119705
Native

Refined GitHub before this PR

Refined GitHub in this PR

On white

While loading

Maybe the same padding between the number and the avatars as the number and the emoji? It looks a bit too close

Good suggestion. Also updated the screenshot in the readme (one of the oldest ones!)

Funny to still see the bug I just fixed locally 😩 😂


new app: letterboxd

Description
Displays the 3 most recent movie reviews from your letterboxd friends.

Copilot

copilot:all

I have read the CLA Document and I hereby sign the CLA


how to stop colloect trace info  when pinpoint-agent is working

@JsonSong89
hot-update [profiler.sampling.rate],
Is there a solution?


Hotfix/yaml and paths

Fixes for strict yaml decrypt, display images, and realtive paths not working correctly. Added additional categories and tags.

Fix for issue:
https://github.com/EncoreTechnologies/vscode-hiera-eyaml/issues/9
https://github.com/EncoreTechnologies/vscode-hiera-eyaml/issues/10
https://github.com/EncoreTechnologies/vscode-hiera-eyaml/issues/11
https://github.com/EncoreTechnologies/vscode-hiera-eyaml/issues/12