Improve logging

Now that the project is being refined across multiple platforms, there are some important steps that are happening before logging starts. For example on Fly.io, we're creating a project and a database before building log dirs. These steps really should be logged.

[ ] Log all steps of the deployment process, regardless of whether we've created a log dir yet.
[ ] See picologging

Implementation

[ ] Store early log entries in a string, and write them to the log file as soon as it's created.

Other notes

[ ] Consider using Django's logger, rather than general Python logging module.

[ ] See notes from djangocon, from Django Logging Demysitified

👋 I would love to help out remotely during the sprints (and after, as well).
If travel goes smoothly I will be around Thursday and Friday. Worst case I will only be around on Friday.

Hi Lee, did you see the two discussions that were opened? If you want to jump in on those, we'll bring you on board with what's happening here.

Hi Lee, did you see the two discussions that were opened? If you want to jump in on those, we'll bring you on board with what's happening here.

I got in super late Thursday (2am local) and I am just now getting caught back up with everything. Sorry I missed the dedicated sprint working days. I followed up on the other threads... I had not seen those so thank you for linking!

See #260.


Circular Import Detected in Venue Booking System

We are encountering a circular import issue in our Venue Booking System project. The circular import is causing conflicts and preventing the application from running as expected.
Details

Problem Description:
When trying to run the application, we are getting import errors that indicate a circular dependency between certain modules. This is causing the application to fail to start and preventing users from logging in or accessing other features.

Steps to Reproduce:

Clone the repository and set up the environment as per the README instructions.
Run the login script (python login_page.py).
Observe the import error messages related to circular dependencies.

Error Messages:
ImportError: cannot import name 'ModuleA' from 'moduleB' due to a circular import

Affected Files:

login_page.py
common.py
Other modules that import these files

Current Behavior:

The application fails to start due to import errors.
Users cannot log in or make bookings, which affects the usability of the system.

Expected Behavior:

The application should run without import errors, allowing users to log in, make bookings, and administrators to manage the system.

Analysis

The circular import occurs because common.py is being imported in multiple files, causing a loop of imports.
This issue disrupts the flow of the program and prevents proper initialization of the application components.

Potential Solutions

Redesign of Imports:

Restructure the module dependencies to break the circular import.
Consider moving common functions and variables to a separate utility module that can be imported where needed without causing a circular reference.

Conditional Imports:

Use if __name__ == "__main__" to avoid circular imports during script execution.
Ensure that modules that don’t need to interact directly don’t import each other, or if they must, it should be done in a controlled manner.

Additional Information

Provide any relevant screenshots of the errors or the code that is causing the issue.
Mention any debugging steps you’ve already taken to diagnose this problem.

Tasks to Resolve:

[x] Review the current module import structure.
[x] Identify modules that are causing the circular import.
[x] Apply the suggested solutions to eliminate the circular dependency.
[x] Test the changes to ensure the application starts without errors.

Issue has been resolved !


Add debug verbose

Codecov Report

Merging #8 into master will decrease coverage by 0.16%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   44.33%   44.17%   -0.17%
==========================================
  Files           5        5
  Lines         212      206       -6
==========================================
- Hits           94       91       -3
+ Misses         96       93       -3
  Partials       22       22

Impacted Files
Coverage Δ

internal/phpmyadmin/phpmyadmin.go
54.83% <ø> (-0.39%)
:arrow_down:

internal/phpmyadmin/request.go
51.42% <100%> (+4.36%)
:arrow_up:

internal/requests/request.go
35% <100%> (-5.91%)
:arrow_down:

Continue to review full report at Codecov.

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


Make a Windows Installer

Dragging the files from the .zip is fine, but it would be easier if you just had an installer that put all of the stuff in C:\Program Files (x86)\Championify with a single click.

I think NSIS would be my best bet for setting this up as they have a command line setup. I'll see about creating a gulp module in another repo sometime soon, and then integrate it with Championify release flow.


fix: respect classname for aria-expanded initial state

Description
Fix for the issue described in https://github.com/zurb/foundation-sites/issues/11747
I hope I covered everything in this one 🤔 ... Feel free to hate me if no 🙈

Closes #11747

Types of changes

[ ] Documentation
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (anything that would change an existing functionality)
[ ] Maintenance (refactor, code cleaning, development tools...)

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] The pull request title and template are correctly filled.
[x] The pull request targets the right branch (develop or develop-v...).
[x] My commits are correctly titled and contain all relevant information.
[ ] I have updated the documentation accordingly to my changes (if relevant).
[x] I have added tests to cover my changes (if relevant).

Build failure is not due to this PR, but rather the dependency issues fixed in https://github.com/zurb/foundation-sites/pull/11792 so going to merge even with the failure.


Start testing on Postgres 15 and drop Postgres 10

@paolobarbolini base changed to 0.7-dev, can you rebase please?


🐛 Fix crash on serializing a union of RootModel and BaseModel

Change Summary
Added tests reproducing the issue. The actual fix is in pydantic-core https://github.com/pydantic/pydantic-core/pull/679
Related issue number
Fix #6113
Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review


Change the second output of LRN (unit_scale) to be Opaque

Current construction of gradient parts for LRN node is a little irregular, and it causes some error when recomputation. This PR fixes this issue, though I'm not confident if this method is the best one.

I found #60 is a better way to fix this issue. I think this actually solves a few other problems. For example, I think we can remove "NoCtx" ops with #60 .

Since #60 fixed this problem, I will close this PR.


Usage Issue

Unable to get canvas to render sample content

Updated To Blazor 0.5.1
Created a new Project
pulled down Blazor.Extensions.Canvas 0.1.6
Created class

`    public class IndexComponent : BECanvasComponent
{
private Canvas2dContext _context;
    protected BECanvasComponent _canvasReference;

    protected override void OnAfterRender()
    {
        this._context = this._canvasReference.CreateCanvas2d();
        this._context.FillStyle = "green";

        this._context.FillRect(10, 100, 100, 100);

        this._context.Font = "48px serif";
        this._context.StrokeText("Hello Blazor---!!!", 10, 100);
    }
}`

Replaced Index.cshtml with
@page "/"
@inherits IndexComponent
BECanvas ref="@_canvasReference"></BECanvas
Then In _viewimports.cshtml added
@using Blazor.Extensions.Canvas @addTagHelper *, Blazor.Extensions.Canvas
It complies and runs but it shows nothing in the canvas. But if i inspect i see
<canvas id="22f7aebb-6bcb-486b-b62d-1c46c0e1f3d8" width="0" height="0" _bl_2=""></canvas>

Was able to fix my mistakes after looking at the sample in the repo under test. Thank you.


Fix device message to stop confusing multiple schedules

The subquery is timely-beliefs functionality modified here to fit our old data model. It is tested in timely-beliefs (in test_belief_query). I now added an inline todo explaining how this code should be replaced when moving to our new data model.


fail to detect chrome browser v1.3.5

the function getDeviceInfo() on DeviceDetectorService returns an incorrect browser when using chrome.
happened on version 1.3.5, but works until v1.3.4

@rafaelwalter, it seems a bit late to reach out, 🤷♂ But do you still face this?
Can you share the object that's returned from the service for you?

Closing the issue being stale.


Stage errors for repo integration

TODOS (ongoing):

[x] Refactor test directory to follow repo conventions
[x] Unit and system tests pass
[x] Fix linting errors
[x] Switch from diagnostics-common to gcp-common

cc @GoogleCloudPlatform/node-team

This PR brings in the Stackdriver Error Reporting library as a client library. This is currently available as the @google/cloud-errors module. It would good to convert this to be the actual client library for Stackdriver error reporting (in the interest of minimizing the number of 'official' modules that work with Stackdriver error reporting). That is, we want to rename the module to be @google-cloud/error-reporting
At the moment this only has support for the 'write' side of the Error Reporting API, but we expect to add the 'read' side of the API in follow-ons.

@GoogleCloudPlatform/node-team ready for a look

@stephenplusplus ready for another look

@cristiancavalli thanks! As far as the code of the library itself, I'm going to remain hands-off there and let you guys run it the way you see fit. As long as you're happy with how it looks and acts, I'm not concerned if it doesn't follow the conventions of the other parts of our library. I'm mostly concerned with the details of the integration when it comes to testing, releasing, and docs. Those are the types of questions I've brought up in comments above, attached to their relevant files in the PR. PTAL for those and feel free to elaborate on the expectations for these integrations.

Moving the convo about how we're planning to integrate out here instead of in the code comments section
RE: https://github.com/GoogleCloudPlatform/google-cloud-node/pull/2017#discussion_r108980300
Let me know if this sounds right. Before we merge, this is the extent of our process integration.
The GCN CI:

will not include error-reporting in our coverage report for the repo
will not run error-reporting unit tests
will not run error-reporting system tests
will include error-reporting docs on https://googlecloudplatform.github.io/google-cloud-node/#/

@stephenplusplus Your proposal seems like the best given the current constraints

Cool, then @callmehiphop, can you step in and figure out what we need to do for docs?
Also cc @lukesneeringer for any thoughts on the "big picture" plan outlined above.

For the purposes of making progress on this PR, it makes sense to skip CI for now, but I would expect a follow-on refactor to add the unit tests and system tests back into the CI. At least that is my understanding of the integration here.

For the purposes of making progress on this PR, it makes sense to skip CI for now, but I would expect a follow-on refactor to add the unit tests and system tests back into the CI. At least that is my understanding of the integration here.

That makes me all kinds of nervous. Is there a hard deliverable or OKR to get this done?
What is the challenge here? Is it just moving the tests up one level, or something else?

@lukesneeringer The idea was to keep the review burden on y'all for the initial PR low. If that is not a problem, then we can add the CI refactoring here too. I am not too worried though as the origin repo for this code is still running CI, and my expectation is that we will immediately follow-up on with CI support this repo once this PR lands.

@ofrobots Sold.

I am guessing that the 'sold' refers to landing + following up with a PR to enable CI support. @cristiancavalli, @stephenplusplus anything else that remains to be done before proceeding here?

@ofrobots 👍

@ofrobots It essentially meant, "I accept what you said and am happy with it." Sorry. :-)

I believe we still need to figure out the docs, which is why this is assigned to @callmehiphop.

Adding onto the comment about examples in index.js, is it possible to move this file to the src directory?

@stephenplusplus @callmehiphop Thanks for the feedback gentlemen! Changes are posted

@cristiancavalli LGTM!
For anyone interested I've posted a demo of how the documentation will look.

cc @ofrobots, next steps?

@stephenplusplus Can this be landed now?
@cristiancavalli I believe the next step would be to add this to the CI and the requisite changes for that.

@ofrobots CI runs are still going to be in a follow-on pr right?

Yep, should have been more clear. Follow-on PR for the CI changes is the agreed upon plan.

Looks like the lint task is failing. Can you run 'npm run lint' from the root directory to take a look?

@stephenplusplus  updated

@stephenplusplus @callmehiphop
I'm having some trouble with the snippet test, specifically this error from the ci:
../packages/error-reporting/node_modules/@google-cloud/error-reporting

Would you gentlemen know where this coming from and how to fix it?

@stephenplusplus @lukesneeringer bump. It would be good to be able to land this soon.

@cristiancavalli sorry I didn't catch this one in my docs review! When we write examples we don't include any snippets on how to require the module itself. This code usually is generated and included within an overview. After which, we assume any requiring of @google-cloud packages is showing how to integrate with other modules - so we do some magic to require the local instance of whatever package.
Long story short, the fix would be to remove the require('@google-cloud/error-reporting') parts of the examples from the docs.

@callmehiphop Thanks for the info! Updated the PR and added fake injection dependencies for snippet tests.

Changes Unknown when pulling 475eedbf53abcc0a1414d7ec6dc24393e147daa7 on cristiancavalli:error-library-integration into ** on GoogleCloudPlatform:master**.

Build is now passing on CircleCI
cc @stephenplusplus @ofrobots

Hooray!! 🎉 🍰


ViewPropTypes Error on Expo SDK 46 and 47

Please update all ViewPropTypes import to: import { ViewPropTypes } from 'deprecated-react-native-prop-types' .
ViewPropTypes was deprecated after React Native version 0.63

Hi bro,
If the Expo SDK version is below 46 it will throw a warning.
SDK version 46 and above will throw an error and the app won't start at all.
Thanks,
Jordan


Support encryption using KMS key

/feature
Hey! It would be very useful to have support for encryption using KMS key. Currently on S3 bucket policies we have below controls implemented:

https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingServerSideEncryption.html
https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingKMSEncryption.html

{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "DenyIncorrectEncryptionHeader",
            "Effect": "Deny",
            "Principal": "*",
            "Action": "s3:PutObject",
            "Resource": "arn:aws:s3:::mybucket/*",
            "Condition": {
                "StringNotEquals": {
                    "s3:x-amz-server-side-encryption": "AES256"
                }
            }
        },
        {
            "Sid": "DenyUnEncryptedObjectUploads",
            "Effect": "Deny",
            "Principal": "*",
            "Action": "s3:PutObject",
            "Resource": "arn:aws:s3:::mybucket/*",
            "Condition": {
                "Null": {
                    "s3:x-amz-server-side-encryption": "true"
                }
            }
        }
    ]
}

Even though we could read from S3/volumes, due to this limitation we cannot write to volumes/S3 buckets as there is no way to explicitly say that object should be encrypted when writing.
/app # echo "test" > /data/test.txt
sh: write error: I/O error

On s3-csi-node daemonset` i cannot find any related errors, even with increased verbosity.

I see there is https://github.com/awslabs/mountpoint-s3/issues/534 in progress already to add support under mountpoint-s3, though once it is released i guess https://github.com/awslabs/mountpoint-s3-csi-driver will have to be updated

Also, i see https://github.com/awslabs/mountpoint-s3/pull/715 was already merged, but i cannot see --sse nor --sse-kms-key-id flags in v1.4.1:
bash-4.2# mountpoint-s3/bin/mount-s3  --version
mount-s3 1.4.1

I am assuming those features were not released yet?

Thank you for the request. That's correct this feature is still in development in mountpoint itself and the issue you linked is the correct place to get updates on the current development. Once mountpoint is released, the CSI driver will make any necessary changes and release support shortly after.

@jjkr, hi, sorry for the stupid question here, but do you know when it may be released or which release it may go? thanks in advance

For reading from an S3 Bucket with S3 AWS Customer KMS Encryption, was there any specific mountoptions required to enable you to read from the Bucket. I'm having issues reading a simple file. The S3 bucket is mounted successfully and I can see all the folders and files but when I go to read(cat, more, tail, vi) a simple text file I get an Input/output error. I see that reading files was reported as an issue here also https://github.com/awslabs/mountpoint-s3-csi-driver/issues/194

CSI Driver v1.6.0 release contained the new feature allowing specific encryption modes and KMS key IDs to be specified outside of the bucket default: https://github.com/awslabs/mountpoint-s3-csi-driver/releases/tag/v1.6.0
Closing this, the feature is now available!


Socket Implementation

Have you a plan to implement sockets in Ldpl?, that will be awesome to start coding some network protocols, maybe a telnet or basic HTTP server...

@dvkt you have more experience than I do with this. Would it be possible to add some kind of socket statements to LDPL?

Would it be possible to add some kind of socket statements to LDPL?

Definitely! We could start with something simple and grow from there.
Maybe three statements: one to send, one to receive, and one to reply.
Send statement:
SEND <TEXT-EXPR> TO <TEXT-EXPR> AT <NUMBER-EXPR> IN <TEXT-VAR>

Receive statement:
LISTEN ON <TEXT-EXPR> AT <NUMBER-EXPR> AND CALL <SUB-PROCEDURE> WITH <TEXT-VAR>

Reply statement:
REPLY WITH <TEXT-EXPR>

Here's what a Gopher request would look like:
send crlf to "gopher.black" at 70 in response
display response crlf

Raw HTTP request:
in request join "GET /get HTTP/1.1" crlf "Host:httpbin.org" crlf crlf
send request to "httpbin.org" at 80 in response
display response crlf

Echo server:
sub-procedure echo-handler
    if data is equal to "shutdown" then
        exit
    else
        reply with data
    end if
end sub-procedure

listen on "localhost" at 12345 and call echo-handler with data

Echo client:
display ">> "
accept request
send request to "localhost" at 12345 in response
display "== " response crlf

I think that would get us pretty far.

Yes! This is wonderful! I couldn't have made it simpler, I love it. Yes, we could definitely do that!

Today I was working with Python SSH Tunnels and I discovered that the "library" it uses is just a separate .exe file that the library calls and does all the SSH handling for you. Couldn't we do something along these lines for sockets?

Some socket support has been added using the LDPL Net Server library. A net client library has yet to be made, but this is a step forward towards the completition of this issue.

A socket client library for LDPL should have the following things:

It should expect an socket_new_message sub to be declared by the user, to handle new messages.
It should expect an socket_on_conneect sub to be declared by the user, to be executed when a socket connects to somewhere.
It should expect an socket_on_disconneect sub to be declared by the user, to be executed when a socket is disconnected.
It should provide a socket <socketNumber> connect to <ip> port <port> to create and connect a socket to a server.
It should provide a socket <socketNumber> disconnect to close a socket connection (and free it).

Well, it took me a few months, but I've got another attempt at this based on all the new features in LDPL and Lartu's amazing ldpl-net-server project. It only works for simple reading/writing over TCP right now, but it's able to serve as an echo client to ldpl-net-server's echo server example:
https://github.com/dvkt/ldpl-socket
It also works with this simple echoserver in C as long as you modify it to close and reopen the connection after each echo message. So, that's two echo servers it works with. Production ready networking solution.
Instead of using callbacks, I just kept close to the C socket API. Since that API is synchronus, this package adds these synchronous LDPL statements:

SOCKET CONNECT TO <hostname> AT <port> IN <number variable>
SOCKET CLOSE <socket number>
SOCKET SEND <text> TO <socket number>
SOCKET READ FROM <socket number> IN <text variable>

As far as the library itself goes, maybe it should be called ldpl-net-client? Or the two should be merged into a ldpl-net package that can be community maintained, or in the std?
I'm going to keep testing this - it might be a dead end like last time, but since it's based on Lartu's code this time I have a good feeling about it!

Holy sh*t you are alive :scream: welcome back! We've missed you!

It only works for simple reading/writing over TCP right now, but it's able to serve as an echo client [...]

This is wonderful! That's exactly what's needed right now!
I love the logo you made. I simply love it, haha. About the name of the library, I think ldpl-socket is perfect. It's 100% descriptive of what it is, so I'd leave it the way it is now.
I'm about to release (maybe today, it's already tested and working) an ncurses library for LDPL (used to make lute), so this comes extremely in handy! We can now write IRC clients in LDPL, MUD clients in LDPL, chat apps and whatnot! This is amazing! I'm super exited, I will test-drive it in some project asap!
Thank you for your awesome contribution, as always. I'll add a link to it (and to your todo app, that's also wonderful) to the LDPL website and the LDPL readme also asap. And again, welcome back! :beach_umbrella:

And I will add your library to LPM, if that's ok with you.

Thank you! I've missed everyone here too. Especially dino. 🦕

I love the logo you made.

haha, I'm not much of an illustrator. If you ever wanted to draw an upgraded logo in the style of the ldpl-net-server one that would be greatly appreciated. But this one has some charm.

I'm about to release ... an ncurses library for LDPL

That sounds great! LDPL would is a wonderful fit for terminal apps.

And I will add your library to LPM, if that's ok with you.

Yes that's great! Thank you. I just updated Gild to use the new socket library and it seems to work fine: https://github.com/dvkt/gild/commit/de9e3d50f13811231e1b182cc09d12f807c5a889
We can probably close this issue now and direct new issues (bugs) to the appropriate libraries. :+1:

We missed you too! LDPL has grown a lot since we last saw you here (we have a new website, LPM, we added multicontainers, etc) if you need help catching up or something do ask! :smile:

haha, I'm not much of an illustrator. If you ever wanted to draw an upgraded logo in the style of the ldpl-net-server one that would be greatly appreciated. But this one has some charm.

I do really love you made though, haha. If you want I can upgrade it to the style of the other LDPL library logos (no problem at all), but this one is cute. Haha!

That sounds great! LDPL would is a wonderful fit for terminal apps.

We are trying to find a niche for it and I think that we've somewhat struck some gold here. Let's see how that evolves!

Yes that's great! Thank you. I just updated Gild to use the new socket library and it seems to work fine.

Woah, wonderful! I'm super excited to try your library, there are so many things I wanted to make with LDPL that required client sockets that we didn't have. No sleep for me tonight!

We can probably close this issue now and direct new issues (bugs) to the appropriate libraries.

I shall close this issue then. Once more, thank you for your marvelous contributions and welcome back!


[DCS] - Calling conn.dcs.delete_instance_backup won't delete the backup

Calling conn.dcs.delete_instance_backup(backup_id, instance_id) won't give an error but also won't delete the backup.

Check the following code to verify the functionality:
"""
Delete a backup of a Distributed Cache Service instance.
"""
import openstack
openstack.enable_logging(True)
conn = openstack.connect(cloud='otc')
backup_id = '88495318-c388-476c-9cf4-7ffb1bb30123'
instance = 'dcs-t6eg'
instance = conn.dcs.find_instance(name_or_id=instance)
conn.dcs.delete_instance_backup(
backup=backup_id,
instance=instance,
ignore_missing=True
)
For me it is working fine.

Thanks :)


destroy() function not necessary anymore or just missing in v3?

v2 had the destroy() function which could free up memory associated with the image instance before the PHP script ended. Is this not necessary anymore for some reason, or just currently not possible with v3?

Intervention Image is a wrapper providing a unified API for GD and Imagick Library. The library is therefore dependent on the functions provided by the extensions.
GD Library's imagedestroy() no longer has an effect since PHP 8, so I have decided to remove support.
I would suggest just trying to set the image variable to null or use unset() to free memory.

@olivervogel
I think it makes sense to mention it in the upgrade manual (https://image.intervention.io/v3/introduction/upgrade).

I think it makes sense to mention it in the upgrade manual (https://image.intervention.io/v3/introduction/upgrade).

You're right. I will accept pull requests: https://github.com/Intervention/docs


About the ros topics

Problem Description
Hello, I am using your ros package to perform hand-eye calibration. In your launch file, there are four ROS topics: ARTagTF, cameraTF,EETF,baseTF. But now I know the transformation from base link to end effector, and the transformation from world to camera, how can I extract four ros topics from these transformation?

I do not understand the underlying problem described in this question, could you please reword it?


Use config-drive instead of guestfish on s390x

This adds support for using the config-drive mechanism to deliver Ignition configs and replaces the guestfish method. Using libguestfish to inject the Ignition config directly into the root disk proved to be difficult to implement when targeting a remote libvirtd instance. This approach instead uses the OpenStack config-drive mechanism which is similar to how cloud-configs are provided to guests. In order for this to work, the Ignition instance in the guest needs to be configured to read its config from the OpenStack provider.

I'm proposing this as an alternative to #666. This approach has the advantage that the content in the coreos_ignition resource definitions can just be plain text, as it is on other platforms today. In #666, the content needs to be a properly formatted config-drive.

@crawford  thx for pr ! :+1: I will look at soon

Yeah, I can absolutely add some examples. Let me see if I can get that done today.

I made a basic test with this change.
Using terraform apply, it could successfully create a VM in my env for both local and remote libvirtd instance.

@MalloZup I wasn't able to come up with an example that could demonstrate this functionality (the existing examples should work), but I did add a bit of documentation to explain how it works and the dependencies. It looks like @xkwangcn was successful with testing, so I think you can merge this once you are happy with it.

@crawford thx ok from my side I will merge this end of the day thx sofar!

@crawford @xkwangcn thx for this pr . Merged. Let me know if I can help for other stuff. enjoy


Cannot install spotlight via pip

Hi,
I am using an environment where conda install is not possible. I have tried installing via pip but it doesn't work. Has anyone tried this? Would appreciate any tips.

change directory inside the root folder of project then: pip install .

@mexangel I think you can also use pip install git+https://github.com/maciejkula/spotlight.git@master#egg=spotlight as a single command (or add it to the requirements.txt of your project, if you're trying to automate it)

Thanks for your help, @artdgn!

NB: https://github.com/maciejkula/spotlight.git@master#egg=spotlight doesn't exist

@mexangel I think you can also use pip install git+https://github.com/maciejkula/spotlight.git@master#egg=spotlight as a single command (or add it to the requirements.txt of your project, if you're trying to automate it)

It helped a lot!


fix access level of generated azure.core.foundations.error model

Right now, we generate an error model from Azure.Core.Foundations.Error if it's used as a property in another model, see here. However, it is always generated as internal, even if it's marked as public. We want to make sure this model follows the access decorator
cc @kristapratico

Could you share the tsp with problem? I could not repro. The following two situations work well with current emitter.
First:
model User {
  @key
  @doc("The user's id.")
  @visibility("read")
  id: int32;

  @doc("The user's name.")
  name: string;

  @doc("The user's order list")
  orders?: UserOrder[];
  ...global.Azure.Core.EtagProperty;

  @doc("for test")
  test: global.Azure.Core.Foundations.Error;
}

Second:
model Error {
  type: string;
  message: string;
  param: string | null;
  code: string | null;
}

@error
model ErrorResponse {
  error: Error;
}

@scenario
@scenarioDoc("""
  Scenario defining how an explicit required body parameter is specified.

  Expected request body:
  ```json
  { "name": "foo" }

""")
@route("/required-explicit")
@post
op requiredExplicit(@body body: BodyModel): OkResponse | ErrorResponse;

I'm sorry, I sent the wrong link, here is the correct link: https://github.com/kristapratico/azure-rest-api-specs/blob/azopenai-python/specification/cognitiveservices/OpenAI.Inference/models/completions.common.tsp#L130

Is there any update for this? I think the generation code should be right.

@kristapratico do you have the generated code with the issue, and what you think it should be? When looking at @kristapratico's code before, I could confirm that we were not publicly exposing the generated Error model

We're marking the operation internal and then using the accessors to expose some of the operation's models: https://github.com/kristapratico/azure-rest-api-specs/blob/azopenai-python/specification/cognitiveservices/OpenAI.Inference/client.python.tsp#L32
I cannot find a way to bring Error back into the public surface, even though I am marking the model that contains it public (ContentFilterResults): https://github.com/kristapratico/azure-rest-api-specs/blob/azopenai-python/specification/cognitiveservices/OpenAI.Inference/client.python.tsp#L38
I've also tried adding @@access(Azure.Core.Foundations.Error, Access.public); to my client.python.tsp, but this has no effect.

What gets generated can be found here: https://github.com/kristapratico/azure-rest-api-specs/tree/azopenai-python/specification/cognitiveservices/OpenAI.Inference/tsp-output/%40azure-tools/typespec-python

@kristapratico , if you want to override models with access propagation value of internal, you need to add decorator
@@access(Azure.Core.Foundations.Error, Access.public);
@@usage(Azure.Core.Foundations.Error, Usage.input | Usage.output);

Also, this override will not be propagated to any other models. So, you need to add the decorators for all the models that you want to export to user.

@tadelesh thanks. I tried the access/usage decorators in this commit and regenerated: https://github.com/Azure/azure-rest-api-specs/commit/b43cc3172e41b3c0b4da566bf019445ebb1ebd23
I'm probably missing something obvious, but it doesn't look like it had an effect for Error. I upgraded to version 0.15.4 for the python emitter before generating.

@kristapratico thanks for the details. First, I'm apologize that my previous answer is not right. I missed that the Error is used in the API payload, so the usage override is not needed. Second, I reproduced the issue locally. The root cause should be a TSP compile problem. Emitter could not get the access decorator info for the Error. I don't know if it is caused by importing a third party model. I've opened an issue for TSP here.

Fixed in latest TSP version.


Kafka README: A command should be all on one line

The lines belong together

Comment to check my signing got through.

LGTM


Communicate "why it matters" on each testcase

Pointed out by @reaperhulk: the current metadata for each testcase communicates whether it passes or fails, but doesn't adequately communicate why or how severe a disagreeing result would be.
We should figure out some way to communicate that, e.g. "failing this testcase means that an implementation is likely to accept certificates for unintended domains."

#236 adds the Importance model and qualifier to each testcase. Leaving this open for the actual fill-in.


400 Bad Request error gets raised for a 301 Redirect

I'm getting a 400 Bad Request error for
RestClient.get "http://web.archive.org/web/20110201164948cs_/https://dannyhopefootballcamps.com/styles.css"

However, this URL gives the proper Location header when doing
require 'uri'
require 'net/http'

uri = URI.parse "http://web.archive.org/web/20110201164948cs_/https://dannyhopefootballcamps.com/styles.css"
r = Net::HTTP.get_response(uri)

r.header['Location']     # proper end url

Any idea why?

Not able to reproduce this on master.


Misleading benchmarks

The benchmarks only include inference latency, but the actual latency is much larger. For example, they say it takes 18 seconds on the 32c M1 Max, which I have validated. However, there's an additional 22-second latency before that where it says Sampling.... I pulled it up in Activity Monitor, and here's what's happening:

Loading resources and creating pipeline - 2 seconds, because I've already run the model several times
Sampling... - 99% CPU, ~0% GPU, which means one CPU core utilized through this entire step (not multi-core), 22 seconds
Step 50 of 50  [mean: 0.99, median: 1.56, last 1.55] step/sec - ~0% CPU, 88% GPU, which means the actual model is running, 18 seconds
Total time: 40 seconds

Is anyone else getting these wierd results? Is it the same, or much larger than 22 seconds? I don't know whether it's because I used the Swift CLI instead of the Python CLI. I cannot get the Python CLI to work: https://github.com/apple/ml-stable-diffusion/issues/43#issuecomment-1344970169.

Variations in execution time based on batch size:
swift run StableDiffusionSample "a photo of an astronaut riding a horse on mars" --resource-path ../mlpackages/Resources --seed 93 --output-path ../outputs --compute-units cpuAndGPU --disable-safety --image-count=IMAGE_COUNT

Batch Size
Loading Resources
Sampling
Inference (50 steps)

I guess the benchmarks aren't entirely wrong. The throughput for batched images is 16 seconds/image - probably smaller than Apple's 18 sec because I disabled the NSFW filtering model.
However, Apple should warn users about the ~20 second static overhead. This would be important for people making one-off images where the 40-second feedback loop is their bottleneck, not absolute theoretical batched throughput.

Curious what your setting is for the compute units. Try setting it to .all
@Option(help: "Compute units to load model with {all,cpuOnly,cpuAndGPU,cpuAndNeuralEngine}")     var computeUnits: ComputeUnits = .all
I've not noticed the large sampling every startup - until I changed it to cpuAndGPU and then I can reproduce your findings.  - referring to the CLI. With setting to .all, it starts up rather quickly just a second or two.
The behavior might be different with different settings based on device and the memory capabilities.
I would imagine, if you were building an application, that you could account for the setup time. Maple/Native Diffusion implementation has a similar initial startup penalty. I've not fully tested the recommended settings with ANE with this on devices yet, but maybe it can be faster on device with that setup? My guess is with such large models there is a cost loading all the weights to the GPU.

referring to the CLI. With setting to .all, it starts up rather quickly just a second or two.

It worked! I had compiled the attention implementation to be GPU-friendly (ORIGINAL), although I did see ANECompilerService compiling something for the neural engine. Perhaps the original sampling pass occurred on the ANE, and the inference pass occurred on the GPU (with 70% utilization).
 I'll switch back to v1.5 and provide an updated table of latencies.

swift run StableDiffusionSample "a photo of an astronaut riding a horse on mars" --resource-path ../mlpackages/Resources/ --seed 93 --output-path ../outputs --disable-safety --compute-units all

With attention set to ORIGINAL:

Batch Size
Loading Resources
Sampling
Inference (50 steps)
Total

I've predicted the likely (actual) fastest implementation on each M1 model, and adjusted the numbers to match CLI latencies.

 You may want to use the neural engine when on battery. I accounted for variation in power among model stages.

--compute-units
--attention-implementation
Runtime
Energy (J)
Inferences/Charge

.cpuAndGPU
ORIGINAL
37
???
???

.all
ORIGINAL
24
???
???

Assuming a 100 watt-hour battery at 90% health, or 324,000 joules.

The benchmark is still misleading. They said they could generate an image with M1 Ultra 48-core GPU within 13 seconds. And they didn't even use the swift package and neural engine!

The executed program is python_coreml_stable_diffusion.pipeline for macOS devices and a minimal Swift test app built on the StableDiffusion Swift package for iOS and iPadOS devices.

Stage	Timestamp (s)	CPU (mW)	GPU (mW)	ANE (mW)

How do you obtain these detailed mW readings of the process running?

sudo powermetrics —sample_rate 100

Wow! I recall trying to use something like that a couple years ago but didn’t seem to exist on Macs. Was this recently re-added...

On Dec 20, 2022, at 8:33 AM, Philip Turner @.***> wrote:
powermetrics


[download_dsyms] Fix wait for dSYM file to appear on App Store Connect

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

When running action download_dsyms right after action upload_to_testflight, the dSYMs are not yet compiled and download_dsyms throws:
No dSYM files found on App Store Connect - this usually happens when no recompiling has happened yet
Full logs:
[09:52:32]: Successfully finished processing the build 1.0.0 - 2647 for IOS
[09:52:34]: Successfully set the changelog for build
[09:52:34]: Distributing new build to testers: 1.0.0 - 2647
[09:52:34]: Successfully distributed build to Internal testers 🚀
[09:52:34]: ----------------------------
[09:52:34]: --- Step: download_dsyms ---
[09:52:34]: ----------------------------
[09:52:34]: Login to App Store Connect<EMAIL_ADDRESS>[09:52:35]: Login successful
[09:52:35]: Looking for dSYM files for '***.****.*****.*****' on platform ios v1.0.0 (2647)
[09:52:42]: No dSYM URL for 2647 (1.0.0)
[09:52:42]: No dSYM files found on App Store Connect - this usually happens when no recompiling has happened yet
[09:52:42]: -------------------------------------------
[09:52:42]: --- Step: upload_symbols_to_crashlytics ---
[09:52:42]: -------------------------------------------
[09:52:42]: Error setting value 'build/***.****.*****.*****-1.0.0-2647.dSYM.zip' for option 'dsym_path'
[09:52:42]: You passed invalid parameters to 'upload_symbols_to_crashlytics'.
[09:52:42]: Check out the error below and available options by running `fastlane action upload_symbols_to_crashlytics`

Related PR: https://github.com/fastlane/fastlane/pull/14137
Related issue: https://github.com/fastlane/fastlane/issues/14393
Description

Same logic as wait_for_build, simply sleep for 30 seconds and try again, with a timeout of 5 minutes.

I have tested the change locally, with 3 different scenarios:

The dSYMs are already there -> no waiting time, success.
The dSYMs are not yet there -> waits about a minute (2 times 30 seconds), success.
The dSYMs are not there (i.e. wrong build number) -> no waiting time, failure.

I can provide the logs if needed.

Here are some logs:
[...]
DEBUG [2019-07-22 16:37:58.41]: Found build version: 2651 , comparing to supplied build_number: 2651
DEBUG [2019-07-22 16:37:58.41]: Build_version: 2651 matches 2651, grabbing dsym_url
DEBUG [2019-07-22 16:38:01.39]: dsym_url:
INFO [2019-07-22 16:38:01.39]: Waiting for dSYM file to appear...
DEBUG [2019-07-22 16:38:33.34]: dsym_url: http://iosapps.itunes.apple.com/itunes-assets/*****
INFO [2019-07-22 16:38:35.63]: 🔑  Successfully downloaded dSYM file for 1.0.0 - 2651 to 'build/*********.***.***-1.0.0-2651.dSYM.zip'
[...]

I have a question @joshdholtz, are you running bundle exec fastlane generate_swift_api before releasing?
So I don't need to update the Swift files, right?

@jhandguy Nope nope, the swift generation is handled during our release process 💪 Thanks for asking though!

@joshdholtz anything else I might have missed?

Waiting for this one!

@joshdholtz I just rebased my branch along with your requested changes, can you have a last look? 👀

Hi!
We've just updated fastlane to 2.130.0 and this seems to break our build, the underlying cause is that we have builds where the dsyms can't be downloaded from appstoreconnect, i.e. the download_url is empty.
Earlier this was handled without erroring out: https://github.com/fastlane/fastlane/commit/06217dabe89372bce03b9de268297689198f5758#diff-65dfcad8f098c3c96688110dade15ed7L102 but now this results in an error instead.
I'm not sure if this is something that should be solved with another flag (fail_build_on_missing_dsym?) or if this was an unintentional side effect of this PR.
It would of course be preferable to fix this at the appstoreconnect side, but I doubt that'll be an easy fix.

Hey @cheif!
Have you tried using the wait_for_dsym_processing flag?

If so I’m sure we can simply replace the Error with a Warning instead 👍
Then I’m curious why aren’t your builds having any dSYM 🤔

@jhandguy the build missing dSYMs is an old one (from March 2019), so unfortunately they won't be coming in anytime soon. We're running a nightly job that fetches all dsyms and then uploads them to crashlytics, just because there was no wait_for_dsym_processing earlier, so we could possibly go down that route now instead.
Yeah, I'm also curious about why the dSYM is missing, but I guess we'll never know due to apple ;)

Yeah I suggest you guys try using wait_for_dsym_processing, maybe you could even get rid of your nightly job then! 🚀
Otherwise, I still pushed a PR to prevent this action from failing your builds, let's see if that is something we want or not @joshdholtz 👍


Add OVMF workaround to workaround.md

Since that package is technically a nixpkgs issue, That'll have to be documented.

Related to #11


Make ChannelOutboundBuffer an interface and so allow to write Channel…

… implementations that not extend AbstractChannel
Motivation:
ChannelOutboundBuffer was a final class that had a package-private constructor that took AbstractChannel as argument. Due of this it was impossible to write a Channel implementation that not extend AbstractChannel.
Modifications:

Make ChannelOutboundBuffer an interface
Add DefaultChannelOutboundBuffer implementation that is used by AbstractChannel
Factor out code that produced ByteBuffer to NioSocketChannel as it is the only place we use it
Add unit tests

Result:
API cleanup

This is a first step... I think as a followup we might want to see if we can remove the whole ChannelOutboundBuffer from the public API. IMHO its kind of an implementation detail of AbstractChannel and other Channel implementation might not use a ChannelOutboundBuffer at all. @chrisvest WDYT ?

Actually lets wait a bit on this... I think I might be able to get rid of it completely.

@normanmaurer Ping me when this is ready to be looked at.

Let's not do this..


Error decoding some part of the packet

I used flowmeter with recent pcap files from CIC but I always I get this errors and no csv file created
Error decoding some part of the packet:Packet decoding error: Layer type not currently supported
Error decoding some part of the packet:Packet decoding error: Unable to decode EthernetType 3

Hello @batool-ali  — Can you provide a link to the pcap files ? Also, can you provide some more details about your use case ?

Hello @batool-ali — Can you provide a link to the pcap files ? Also, can you provide some more details about your use case ?

I use CIC IoT 2023 dataset

I try many of the pcap files in this dataset for example
DDos ICMP Flood
Dos HTTP flood
Recon
And almost all the pcap files I downloaded
I want to get statistics about the flow as features in CSV file

Hello @batool-ali
Thanks for reporting the issue
I see 4 Recon datasets - Any Recon files would do?

@batool-ali I was able to repro, here is a fix:
https://github.com/deepfence/FlowMeter/pull/11
The issue was that some packets were either ipv6 or https, and we were not supporting those protocols. I have added support to both.
I have also changed the architecture of the project a bit, should see some performance improvements and also easier build instructions.


Support for configurations consisting of multiple files

Vocabularies are in separate file(s) from the actual rules file. This version will consider a given directory for uploading, and add every yaml and csv file to the uploaded package.
Also a small change to app id passing is made, so that it is always given as a flag instead of argument. This is mostly to enhance the automatic usage texts printed by the tool.

But once tar is uploaded from cli, it cannot (at the moment) be opened from dashboard. Should we warn user about that in docs or somewhere?

Good point, we should mention that somehow.


Login Page

Login pages are the most used pages on any kind of website. But designing a responsive page isn't easy. I would like to design a responsive login page using HTML, CSS, and JavaScript.

@gantavyamalviya @Adarsh-jaiss @TKirtan kindly assign me this issue

@ahanavish yare you planning to create a website for codemistic ?

Currently, I wanted to design the login page only. But if you need a website, I can definitely build a small one. Please let me know the same!


check-sensor.rb doesn't check IPMI sensors on local machine

It would be useful if check-sensor.rb would be able to check IPMI sensors on the local machine too.

I submitted a request to the rubyipmi module project to do this, since it currently only supports remote access. This is now being worked on for the 0.10.0 release: https://github.com/logicminds/rubyipmi/issues/24
Once this is done I plan to patch check-sensors.rb to support this.

When the time comes please open a PR at the new repo
https://github.com/sensu-plugins/sensu-plugins-ipmi
Thanks !


Initialize the superset database to utf8

MySQL is initialized to latin1.
This prevents us from adding Arabic or other Unicode.'
On the other hand edxapp is being created correctly:
CREATE DATABASE `edxapp` /*!40100 DEFAULT CHARACTER SET utf8 */

@Ian2012 would you mind taking a look?
Context
We're aiming to add full Arabic support for both Aspect and superset for @nelc:

RTL: https://github.com/apache/superset/issues/25258
Unicode support (this issue)
Translations of the Aspect dashboard UI (Needs contribution)
Translations for the Superset UI: https://github.com/apache/superset/issues/25258

Also we should use utf8mb4 instead of utf8 to get the full suite of code points!


ENH: DataStoreSqlite summary_not_completed now works with defaults

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
21 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.03%) to 91.279%

Files with Coverage Reduction
New Missed Lines
%

.nox/test-3-10/Lib/site-packages/cogent3/maths/matrix_exponential_integration.py
1
92.98%

.nox/test-3-10/Lib/site-packages/cogent3/app/io_new.py
2
95.15%

.nox/test-3-10/Lib/site-packages/cogent3/maths/scipy_optimize.py
3
84.14%

.nox/test-3-10/Lib/site-packages/cogent3/app/data_store_new.py
7
95.83%

.nox/test-3-10/Lib/site-packages/cogent3/app/sqlite_data_store.py
8
96.28%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
177546

Relevant Lines:
194510

💛  - Coveralls


Updated Azure Storage libraries to v12

Description
Updated Azure Storage library to v12, because v11 is deprecated
Microsoft.Azure.Storage.Blob (v11.2.2) ==> Azure.Storage.Blobs (v12.8.0)

Related issues
Addresses [issue #].
Testing
Existing tests pass.
FHIR Team Checklist

[ ] Update the title of the PR to be succinct and less than 50 characters
[ ] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[ ] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[ ] Tag the PR with Azure API for FHIR if this will release to the managed service
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

/azp run

/azp run

/azp run

@anyname2 Apologies for the late response. Thanks for bringing this up. This SDK update is something we definitely need to do. But this update also introduces breaking changes and we have to update our build/test pipelines and our managed service to handle this change. We are planning on working on this later rather than now. @CaitlinV39 FYI

Closing this out as currently we don't have any plan in near future to update azure storage libraries but we can definitely refer this PR whenever we plan this in long run.


Documentation regarding RegisterTransform

Using: go version go1.11.1 linux/amd64 in a known-working environment.
On the page/section: https://godoc.org/gopkg.in/qntfy/kazaam.v3#Config.RegisterTransform
The example block shown, when copied and used verbatim, produces:
cannot use func literal (type func(*"gopkg.in/qntfy/kazaam.v3/transform".Config, []byte) ([]byte, error)) as type kazaam.TransformFunc in argument to kc.RegisterTransform

FWIW, this does not seem to happen when using "github.com/qntfy/kazaam" in ones import (...) statement(s), as opposed to "gopkg.in/qntfy/kazaam.v3".   This might confuse people, since ones search results on the web may vary, and lead users to one over the other.  This is what happened to me.
Given the documentation itself is accurate, then the title of this issue may be worthy of a rename, not sure.  But something was fishy for certain.

I just ran into this.  Key is to switch your import from gopkg.in with v3 to a plain github.com` import.


New feature in education: supervisors & thesis

Hi folks,
I appreciate this awesome theme for a personal website. I would like to request a new feature for the education section. How about adding a subsection to list supervisors' names and thesis topics? I do see there are rooms for publication, but it would be nice to have separate rooms for the thesis. Likewise, adding supervisor names would make this theme more attractive for those in academia.
Thank you.

@IhnwhiHeo Can you give an example of how you envision this looking? I would assume that since you can add a URL to the publication you would just link to the copy and allow the additional information to be presented there. However, if you are able to give an example of what you are expecting I can see what I can do.

And Why not another tab like Languages ?

@Rihoj Hello! Currently, there are subsections like CGPA, Publications, and Taken Courses. I would like to add a custom-made subsection such as Advisor(s), Honors and Awards, etc. Tried to add it but it failed (seems like I can only add those that are hard-wired).


Collapse component

Describe the feature in detail (code, mocks, or screenshots encouraged)
In addition to the accordeon, I would love to see a collapse component.
What type of pull request would this be?
New Feature
Provide relevant links or additional information.
Similar to Bootstraps Collapse component

@bartcuijpers what would the collapse component be able to do, that the accordion can't do? can you give some more explanation or examples for how you are planning to use it?

@endigo9740 I would like to work on this feature. if this is ok and you have some expectations, please let me know.

@Mahmoud-zino the main thing is I want to try to reduce redundancy. There's a lot of overlap with <AccordionItem>, but at the same time it's not 1:1 since it doesn't have the overhead of needing to work within a group.
For now I say we treat it as a brand new standalone component, similar to how other libraries separate Accordion and Collapse:

https://mantine.dev/core/accordion/
https://mantine.dev/core/collapse/

After further discussion on Discord, we found out that the component is too trivial to include in the library.
Here is a simple example on how to implement it (using svelte only):
https://svelte.dev/repl/d8a84263d60d4c5e9100df1f6d896b15?version=4.2.1


NIFI-4950 Defining behavior for MergeContent when more than 1 FlowFil…

…e has the same fragment.index value
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Reviewed and tested; behavior in the presence of a duplicate fragment is fixed; duplicate is sent to failure. I fixed a typo and used FragmentAttributes.FRAGMENT_INDEX enum in place of "fragment.index" strings.
+1, merging.  Thanks for fixing this @markobean.


CAMEL-20281 - Camel-AWS Components: Make it possible to use AwsSessio…

…nCredentials to support temporary credentials - AWS MSK
Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test aws


casdoor facebook login error

@wu-jj hi, can you provide more details like the config, the screenshot and demo video?

@wu-jj hi, can you provide more details like the config, the screenshot and demo video?

@hsluoyz Hi, this bug appeared to be an intranet environment issue and has been resolved, thanks for noticing!


Step size

Hi,
Please help me, where to set the step size value or epoch value.

config  file maybe  your want

HI,
Thanks for your comment.
I find that in config file.
I am facing another issue,
python: relocation error: /media/administrator/D/Objectdetection/objenv/lib/python3.5/site-packages/PyQt5/Qt/plugins/platforms/../../lib/libQt5DBus.so.5: symbol dbus_message_set_allow_interactive_authorization, version LIBDBUS_1_3 not defined in file libdbus-1.so.3 with link time reference
please help me on this..

i thinkt  because of version of PyQt,your env is right?

Hi,
1,         I am using python 3.6.8 and i create virtual environment for that and try the below cmds in
ubuntu for pyqt5 ,
Python 3 + Qt5 (Recommended)
sudo apt-get install pyqt5-dev-tools
sudo pip3 install -r requirements/requirements-linux-python3.txt
make qt5py3
python3 labelImg.py
python3 labelImg.py [IMAGE_PATH] [PRE-DEFINED CLASS FILE]
 but i am facing the same issue.

2,           I am using your dataset and run the object_detection_images.py file but i am unable to see
the detected class names and score. The text is very small in the image.


add ipaddress input field to 'Add network to VM' form

Description

On the 'NICs' tab, when a user clicks the 'Add network to VM' button to add a network to a VM, the 'Add network to VM' popup screen does not have an IP Address input field to allow a user to define a specific IP Address for a NIC. A user can specify the IP address for the first NIC when creating a VM instance, but cannot do that for subsequent NICs when adding a network to a VM.

To Reproduce:
Go to the 'Instances' screen by clicking the 'Instances' tab on the lefthand side.
On the 'Instances' screen click on a specific VM instance name.
This will open the 'Details' tab for the specific VM instance.
Click on the 'NICs' tab and then click on the 'Add network to VM' button to add a network to a VM.
The 'Add network to VM' popup screen will display.
Actual Behaviour:
The 'Add network to VM' popup screen does not have an IP Address input field to allow a user to define a specific IP Address for a NIC.
Expected behaviour:
The 'Add network to VM' popup screen must have an IP Address input field to allow a user to define a specific IP Address for a NIC.
Since the addNicToVirtualMachine API's ipaddress field is not required, the IP Address input field is also not a required field.
The IP Address input field must be validated for a valid IPv4 formatted value if the user enters anything into the field.
The valid user-specified IPv4 IP Address value must be allocated to the NIC if it is within the acceptable IP range for the chosen Network.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

GitHub Issue/PRs

Screenshots (if appropriate):

How Has This Been Tested?
Manually (See screenshots)

Go to the 'Instances' screen by clicking the 'Instances' tab on the lefthand side.
On the 'Instances' screen click on a specific VM instance name.
This will open the 'Details' tab for the specific VM instance.
Click on the 'NICs' tab and then click on the 'Add network to VM' button to add a network to a VM.
The 'Add network to VM' popup screen will display.
The 'Add network to VM' popup screen now has an IP Address input field to allow a user to define a specific IP Address for a NIC.
Since the addNicToVirtualMachine API's ipaddress field is not required, the IP Address input field is also not a required field.
The IP Address input field is validated for a valid IPv4 formatted value if the user enters anything into the field.
The valid user-specified IPv4 IP Address value is allocated to the NIC since it is within the acceptable IP range for the chosen Network.

Environment:
CloudStack version 4.11, KVM hypervisor.

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
Testing
[ ] I have added tests to cover my changes.
[ ] All relevant new and existing integration tests have passed.
 I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2223


Update README.md

test

:tada: This PR is included in version 1.0.48 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Quota enable config to user/domain scope

Description
Currently, ACS only has the option to enable the Quota plugin globally. However, for some use cases having the plugin active to specific accounts and domains may be more interesting. Therefore, this PR aims to add this functionality through the quota config quota.account.enabled. Moreover, a new column was added to the quota summary section in the UI showing if the account has the plugin enabled. In the future, a filter will be added for accounts with the plugin enabled, disabled and for removed accounts.
Finally, for the config to be used in a domain scope, when it cannot be found at user level, it is required that the following global configurations are enabled: enable.account.settings.for.domain and enable.domain.settings.for.child.domain.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Screenshots (if appropriate):
List of accounts with the plugin enabled and disabled:

How Has This Been Tested?
This was tested in a local lab setting the quota.account.enabled to true and false to random accounts. In the quota summary section the accounts where correctly displayed: accounts with the plugin disabled had the (quota-disabled) string attached and accounts with the plugin enabled were normally displayed.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4138

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-4833)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43715 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6690-t4833-kvm-centos7.zip
Smoke tests completed. 100 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_08_upgrade_kubernetes_ha_cluster
Failure
640.10
test_kubernetes_clusters.py

@BryanMLima can you look at the conflicts

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6690 (SL-JID-2543)

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6690 (SL-JID-2544)

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6690 (SL-JID-2564)

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 4550

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-5205)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38339 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6690-t5205-kvm-centos7.zip
Smoke tests completed. 104 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

@GutoVeronezi , I trust all your comments have been met?

@DaanHoogland, yes, they have been met.
I will merge this one based on the packaging and tests results  (https://github.com/apache/cloudstack/pull/6690#issuecomment-1291608946 and https://github.com/apache/cloudstack/pull/6690#issuecomment-1292631838) and on the approvals.


Update purge.js

Changed so it doesnt return undefined instead it shows DBH! in example/ usage.

Why close?

Why close?

#186

Why close?

#186

revert & make new changes


ERROR reading clientDir with macOS MS Remote Desktop.app with devel

xrdp version
0.9.80
Detailed xrdp version, build options
  Configure options:
      --localstatedir=/var
      --enable-strict-locations
      --enable-pam-config=freebsd
      --enable-jpeg
      --enable-pixman
      --enable-rfxcodec
      --enable-painter
      --enable-imlib2
      --prefix=/opt/xrdp
      --enable-fuse
      --enable-devel-logging
      --disable-silent-rules
      CC=clang16
      CFLAGS= -I/usr/local/include
      LDFLAGS= -L/usr/local/lib -L/usr/lib
      OPENSSL_CFLAGS=-I/usr/include
      OPENSSL_LIBS=-lssl -lcrypto

  Compiled with OpenSSL 1.1.1t-freebsd  7 Feb 2023

Operating system & version
FreeBSD 13.2
Installation method
git clone & make install
Which backend do you use?
xorgxrdp
What desktop environment do you use?
fvwm3
Environment xrdp running on
FreeBSD 13.2 jail
What's your client?
macOS Microsoft Remote Desktop
Area(s) with issue?
No response
Steps to reproduce
RDP to the server with macOS MS Remote Desktop.app
✔ Expected Behavior
To be presented with the RDP login screen.
❌ Actual Behavior
With macOS MS RDP application and devel 50cff2eb fails to present the login screen with the following error in xrdp.log:
[2023-11-11T23:19:09.411+0000] [DEBUG] [xrdp_sec_process_logon_info(xrdp_sec.c:1087)] Client supplied directory:
[2023-11-11T23:19:09.423+0000] [TRACE] [ts_info_utf16_in(xrdp_sec.c:799)] ts_info_utf16_in: uni_len 26, dst_len 256
[2023-11-11T23:19:09.442+0000] [TRACE] [ts_info_utf16_in(xrdp_sec.c:799)] ts_info_utf16_in: uni_len -2, dst_len 256
[2023-11-11T23:19:09.456+0000] [ERROR] [ts_info_utf16_in(xrdp_sec.c:821)] ts_info_utf16_in: bad terminator. Expected 0, got 300
[2023-11-11T23:19:09.470+0000] [ERROR] [xrdp_sec_process_logon_info(xrdp_sec.c:1113)] ERROR reading clientDir

It appears if ts_info_utf16_in is called when src_bytes is 0 uni_len is -2... this didn't seem correct.  Added a check to not perform term check if src_bytes is <= 0.  This appears to allow the macOS MS RDP client to login however I'm not sure if this is a client bug or possible issue with ts_info_utf16_in or previous processing of ip data from the stream.
Anything else?
No response

Like the source and spec says this should have a null term so disabling the check shouldn't be it.

@matt335672 Can you look into this

Looks like this is a client issue.
iOS MS RDP does the same thing and MS RDP Android did (or does?) the same.
https://github.com/FreeRDP/FreeRDP/pull/3276/commits/4ae77b8273b2e17c3cc207f409c221ae9350364c
So maybe disabling the check or doing something "extra" if len_ldd is zero might be desired until MS fixes their own clients?

Thanks for raising this @derekschrock
This appears to be a regression introduced by #2794.
It doesn't seem to affect v0.9.x which is probably why we haven't seen it before. The UTF16 conversion code there is adding 2 back on to the values which have had 2 subtracted from them. Unterminated strings will cause an error, but missing strings won't.
There's a bit of length checking missing from this code. Also seems to be no reason to parse fields we are not using. I've made a few changes to address this, including using unsigned variables.
Derek - can you try the attached patch? I don't have easy access to a client which exhibits this behaviour.
patch.txt
I've reproduced the patch below as well
--- a/common/ms-rdpbcgr.h
+++ b/common/ms-rdpbcgr.h
@@ -131,6 +131,9 @@
 #define RDP_LOGON_LEAVE_AUDIO          0x2000
 #define RDP_LOGON_RAIL                 0x8000

+/* Extended Info Packet: clientAddress (<IP_ADDRESS>.1.1.1) */
+#define EXTENDED_INFO_MAX_CLIENT_ADDR_LENGTH 80
+
 /* Extended Info Packet: performanceFlags (<IP_ADDRESS>.1.1.1) */
 /* TODO: to be renamed */
 #define RDP5_DISABLE_NOTHING           0x00
diff --git a/libxrdp/xrdp_sec.c b/libxrdp/xrdp_sec.c
index 231176cd..20665dfc 100644
--- a/libxrdp/xrdp_sec.c
+++ b/libxrdp/xrdp_sec.c
@@ -833,18 +833,15 @@ static int
 xrdp_sec_process_logon_info(struct xrdp_sec *self, struct stream *s)
 {
     int flags = 0;
-    int len_domain = 0;
-    int len_user = 0;
-    int len_password = 0;
-    int len_program = 0;
-    int len_directory = 0;
-    int len_ip = 0;
-    int len_dll = 0;
-    char tmpdata[256];
+    unsigned int len_domain = 0;
+    unsigned int len_user = 0;
+    unsigned int len_password = 0;
+    unsigned int len_program = 0;
+    unsigned int len_directory = 0;
+    unsigned int len_clnt_addr = 0;
+    unsigned int len_clnt_dir = 0;
     const char *sep;

-    /* initialize (zero out) local variables */
-    g_memset(tmpdata, 0, sizeof(char) * 256);
     if (!s_check_rem_and_log(s, 8, "Parsing [MS-RDPBCGR] TS_INFO_PACKET"))
     {
         return 1;
@@ -1097,22 +1094,33 @@ xrdp_sec_process_logon_info(struct xrdp_sec *self, struct stream *s)
         }
         /* TS_EXTENDED_INFO_PACKET required fields */
         in_uint8s(s, 2);         /* clientAddressFamily */
-        in_uint16_le(s, len_ip);
-        if (ts_info_utf16_in(s, len_ip - 2, tmpdata, sizeof(tmpdata)) != 0)
+        in_uint16_le(s, len_clnt_addr);
+        if (len_clnt_addr > EXTENDED_INFO_MAX_CLIENT_ADDR_LENGTH ||
+                !s_check_rem(s, len_clnt_addr))
         {
-            LOG(LOG_LEVEL_ERROR, "ERROR reading ip");
+            LOG(LOG_LEVEL_ERROR, "clientAddress is too long (%u bytes)",
+                len_clnt_addr);
             return 1;
         }
+        // The clientAddress is currently unused. [MS-RDPBCGR] requires
+        // a mandatory null terminator, but some clients set
+        // len_clnt_addr == 0 if this field is missing. Allow for this
+        // in any future implementation.
+        in_uint8s(s, len_clnt_addr); // Skip Unicode clientAddress
+
         if (!s_check_rem_and_log(s, 2, "Parsing [MS-RDPBCGR] TS_EXTENDED_INFO_PACKET clientDir"))
         {
             return 1;
         }
-        in_uint16_le(s, len_dll);
-        if (ts_info_utf16_in(s, len_dll - 2, tmpdata, sizeof(tmpdata)) != 0)
+        in_uint16_le(s, len_clnt_dir);
+        if (len_clnt_dir > INFO_CLIENT_MAX_CB_LEN ||
+                !s_check_rem(s, len_clnt_dir))
         {
-            LOG(LOG_LEVEL_ERROR, "ERROR reading clientDir");
+            LOG(LOG_LEVEL_ERROR, "clientDir is too long (%u bytes)", len_clnt_dir);
             return 1;
         }
+        in_uint8s(s, len_clnt_dir); // Skip Unicode clientDir
+
         LOG_DEVEL(LOG_LEVEL_TRACE, "Received [MS-RDPBCGR] TS_EXTENDED_INFO_PACKET "
                   "<Required Fields> clientAddressFamily (ignored), "
                   "cbClientAddress (ignored), clientAddress (ignored), "

Looks good tested from macOS and iOS. Thanks.


tools: add script to find similar bits

Used to find these undefined bits within minutes: https://github.com/commaai/opendbc/commit/4e3899cbba7188116e5919923a7dc24264fa3d38
You just need to find a good proxy for what you want to find. In my case, I saw in PJ that the blocked cnt increased right around where the stock system faulted, so I filtered for that.
TODO:

[ ] Clean up
[ ] Somehow make a CLI interface? It might be hard to do everything you want over CLI. Sometimes you want bit_to_find to be latching, sometimes you want to compare cereal fields, sometimes you want to compare other bits/signals on CAN

Added in cabana. Still want this in?

It's still useful to compare to openpilot signals, so I'll still use this script. I'll see about adding that functionality to cabana in the future


PHP (composer) fails to run Windows executables in path when run in system()

Hi Team!
Firstly, many thanks to all of the Microsoft team that made this possible! I am proud to say that BoUoW/WSL/BashOnWindows has successfully allowed me to switch from OS X to Windows for all of my day to day development 👍
I am currently running Windows 10 build 14986.1001.
I have WampServer 64 bit installed in C:\wamp.
My .bashrc has been edited to include the path of the PHP provided with WampServer in the path:
export PATH=/mnt/c/wamp/bin/php/php7.0.10/:$PATH

In /mnt/c/wamp/bin/php/php7.0.10/, I have made a symlink from php.exe to php using the following command:
ln -s /mnt/c/wamp/bin/php/php7.0.10/php.exe /mnt/c/wamp/bin/php/php7.0.10/php

I can run the php command from bash:
root@timgws:~# which php
/mnt/c/wamp/bin/php/php7.0.10/php

root@timgws:~# php -i | grep -i Windows
System => Windows NT timgws 10.0 build 14986 (Windows 10) AMD64

I also have the PHP from Ubuntu installed:
root@timgws:~# /usr/bin/php -i | grep -i Linux
System => Linux timgws 4.4.0-43-Microsoft #1-Microsoft Wed Dec 31 14:42:53 PST 2014 x86_64

This is handy! It means that I can run PHP apps from the command line in Bash, but it uses the version of PHP that I have running inside WampServer.
However, when I run a composer create-project for Laravel (after downloading Composer from getcomposer.org) the composer command fails to run.
root@timgws:/mnt/c/wamp/code# ./composer.phar create-project --prefer-dist laravel/laravel will-this-work
Installing laravel/laravel (v5.4.3)
  - Installing laravel/laravel (v5.4.3) Loading from cache
Created project in will-this-work
> php -r "file_exists('.env') || copy('.env.example', '.env');"

'php' is not recognized as an internal or external command,
operable program or batch file.

Script php -r "file_exists('.env') || copy('.env.example', '.env');" handling the post-root-package-install event returned with error code 1

That looks like an error from cmd.exe.
The strace from this error looks like this:
execve("/mnt/c/wamp/bin/php/php7.0.10//php", ["php", "./composer.phar", "create-project", "--prefer-dist", "laravel/laravel", "will-this-work"], [/* 17 vars */]) = 0
brk(0)                                  = 0x7fffe46a9000
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f53b49d0000
access("/etc/ld.so.preload", R_OK)      = -1 ENOENT (No such file or directory)
open("/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=23029, ...}) = 0
mmap(NULL, 23029, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f53b49d9000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P \2\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0755, st_size=1840928, ...}) = 0
mmap(NULL, 3949248, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f53b4230000
mprotect(0x7f53b43ea000, 2097152, PROT_NONE) = 0
mmap(0x7f53b45ea000, 24576, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1ba000) = 0x7f53b45ea000
mmap(0x7f53b45f0000, 17088, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7f53b45f0000
close(3)                                = 0
mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f53b49c0000
mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f53b49b0000
arch_prctl(ARCH_SET_FS, 0x7f53b49b0740) = 0
mprotect(0x7f53b45ea000, 16384, PROT_READ) = 0
mprotect(0x7f53b4c09000, 4096, PROT_READ) = 0
mprotect(0x7f53b4822000, 4096, PROT_READ) = 0
munmap(0x7f53b49d9000, 23029)           = 0
getpid()                                = 2585
open("/dev/kmsg", O_WRONLY|O_CLOEXEC)   = 3
getcwd("/mnt/c/wamp/code", 4096)        = 17
open("/dev/lxssclient", O_RDWR)         = 4
ioctl(4, 0x22002f, 0x7fffeb2ec600)      = 0
open("/mnt/c/wamp/bin/php/php7.0.10//php", O_RDONLY) = 6
brk(0)                                  = 0x7fffe46a9000
brk(0x7fffe46ca000)                     = 0x7fffe46ca000
ioctl(4, 0x22003f, 0x7fffeb2ec4b0)      = 0
close(6)                                = 0

I am guessing that maybe PHP needs to be on my PATH in Windows, as well? (This would also mean that I could drop it from my bashrc, as the Windows path is automatically included)
That being said, I thought that the whole path would be sent to lxssclient for the executable?
I would however think this would 'just work', especially because running php anywhere else works as expected.

The Linux path is not shared with NT processes launched via interop.  If you are launching an NT binary that you will be launching another NT binary the second one will have to either be in the NT %PATH% environment variable or be a fully-qualified NT path.

TRUE THAT.
Didn't even think that the PHP I was launching was the Windows one, so of course I need it in the Windows path.
Sorry @benhillis 💯 ... goes to show how easy it is to run Windows apps in WSL

Taking that as a compliment :)


Mssql 2012  Sqlx 0.6.2 MssqlPool::connect

Password complexity is not supported. For example, the password is QAZwsx123@#

Info

SQLx version: 0.6.2
SQLx features enabled: "runtime-async-std-native-tls" ,"mssql", "chrono"
Database server and version:Mssql Server 2012
Operating system: [windows 2008 R
rustc --version: 1.58

That seems to be a  bug in crate Url. servo/rust-url#396. You an use function connect_with.

@HTROY What should I do?

You can see this, and here is an example
    let option = MssqlConnectOptions::new()
        .host("<IP_ADDRESS>")
        .username("sa")
        .password("QAZwsx123@#")
        .database("Your Database");
    let pool:Pool<Mssql> = PoolOptions::new()
        .max_connections(5)
        .connect_with(option)
        .await
        .unwrap();

Thank you !

Thank you !

@HTROY Can you ask a question?，Use connect_with, how to use query_file! ？

If you really need to use connect with the database URL, you can URL encode the passord: QAZwsx123%40%23


Error No disks were found

When I run the script I get the error that no disks were found.
pbxadmin@gc-custom-us-central1-2018-10-18-01-43-41-xxevdaqts7igji:~$ /opt/google-compute-snapshot/gcloud-snapshot.sh -n
[2018-11-22 21:28:42]: Start of google-compute-snapshot
[DEBUG]: OLDER_THAN=7
[DEBUG]: REMOTE_CLAUSE=
[DEBUG]: FILTER_CLAUSE=
[DEBUG]: PREFIX=gcs
[DEBUG]: OPT_ACCOUNT=
[DEBUG]: OPT_PROJECT=
[DEBUG]: DRY_RUN=true
[DEBUG]: DATE_TIME=1542940122
[DEBUG]: DELETION_DATE=20181115
[DEBUG]: INSTANCE_NAME=gc-custom-us-central1-2018-10-18-01-43-41-xxevdaqts7igji
WARNING: Argument NAME is deprecated. Use --filter="name=( 'NAME' ... )" instead.
WARNING: --filter : operator evaluation is changing for consistency across Google APIs.  name=gc-custom-us-central1-2018-10-18-01-43-41-xxevdaqts7igji currently does not match but will match in the near future.  Run `gcloud topic filters` for details.
[ERROR]: No disks were found - please check your script options / account permissions.

However, when I run gcloud compute disks list I can see them all.
pbxadmin@gc-custom-us-central1-2018-10-18-01-43-41-xxevdaqts7igji:~$ gcloud compute disks list
NAME                                                           ZONE                    SIZE_GB  TYPE         STATUS
gc-custom-australia-southeast1-2018-10-18-05-00-39-v7jzw-disk  australia-southeast1-c  10       pd-standard  READY
gc-custom-us-central1-2018-10-18-01-43-41-xxevdaqts7igji-disk  us-central1-c           10       pd-standard  READY

My API scope is set to:
Compute Engine - Read/Write
gcloud auth list also shows that the correct service account is being used.
Have this sucessfully running on two other servers - just not playing nice on this one.
What am I missing?

Very strange @andy-vdg !
I've just released a new version to fix the Depreciation warning - can you install the script again and give it a go?
I was changing the --filter clause, so whilst I was there I replaced the = operator for : (as the second warning suggested).
If it still doesn't work, can you execute the following command manually:
gcloud compute disks list --filter "name:gc-custom-us-central1-2018-10-18-01-43-41-xxevdaqts7igji" --format='value(name,zone,id)'

Hi same problem here.
I modified this part of your script
getDeviceList()

{
    # echo -e "$(gcloud $OPT_INSTANCE_SERVICE_ACCOUNT compute disks list --filter "users~instances/$1\$ $FILTER_CLAUSE" --format='value(name)')"

    local filter=""

    # if using remote instances (-r), then no name filter is required
    if [ "$REMOTE_CLAUSE" = true ]; then

        # check if $FILTER_CLAUSE exists
        if [[ ! -z $FILTER_CLAUSE ]]; then
            filter="${FILTER_CLAUSE}"
        fi
    else
        # check if $FILTER_CLAUSE exists
        if [[ ! -z $FILTER_CLAUSE ]]; then
            filter="name:$1 AND ${FILTER_CLAUSE}"
        else
            filter="name:$1"
        fi
    fi

    echo -e "$(gcloud $OPT_ACCOUNT compute disks list --filter "${filter}" --format='value(name,zone,id)' $OPT_PROJECT)"
}

by this to make it work as I only have one instance with one disk
getDeviceList()
{
    echo -e "$(gcloud compute disks list --format='value(name,zone,id)')"

    local filter=""

    # if using remote instances (-r), then no name filter is required
    if [ "$REMOTE_CLAUSE" = true ]; then

        # check if $FILTER_CLAUSE exists
        if [[ ! -z $FILTER_CLAUSE ]]; then
            filter="${FILTER_CLAUSE}"
        fi
    else
        # check if $FILTER_CLAUSE exists
        if [[ ! -z $FILTER_CLAUSE ]]; then
            filter="name:$1 AND ${FILTER_CLAUSE}"
        else
            filter="name:$1"
        fi
    fi

    # echo -e "$(gcloud $OPT_ACCOUNT compute disks list --filter "${filter}" --format='value(name,zone,id)' $OPT_PROJECT)"
}

This part worked But when it actually runs the command (dry run mode)
createSnapshot()
{
    if [ "$DRY_RUN" = true ]; then
        printCmd "gcloud ${OPT_ACCOUNT} compute disks snapshot $1 --snapshot-names $2 --zone $3 ${OPT_PROJECT}"
    else
        $(gcloud $OPT_ACCOUNT compute disks snapshot $1 --snapshot-names $2 --zone $3 ${OPT_PROJECT})
    fi
}

It only outputs this on the shell
[2018-11-28 13:30:35]: Start of google-compute-snapshot
[DEBUG]: OLDER_THAN=7
[DEBUG]: REMOTE_CLAUSE=
[DEBUG]: FILTER_CLAUSE=
[DEBUG]: PREFIX=gcs
[DEBUG]: OPT_ACCOUNT=
[DEBUG]: OPT_PROJECT=
[DEBUG]: DRY_RUN=true
[DEBUG]: DATE_TIME=1543397435
[DEBUG]: DELETION_DATE=20181121
[DEBUG]: INSTANCE_NAME=runew-virtualbox-73237c60
[DEBUG]: DEVICE_LIST=vol-8796758176650--dev-sda-a92b0d8e        europe-west1-b<PHONE_NUMBER>339443868
[2018-11-28 13:30:36]: Handling Snapshots for vol-8796758176650--dev-sda-a92b0d8e
[CMD]: gcloud  compute disks snapshot vol-8796758176650--dev-sda-a92b0d8e --snapshot-names gcs-vol-8796758176650--dev-sda-a92b0d8e-1543397435 --zone europe-west1-b
Listed 0 items.
[2018-11-28 13:30:37]: End of google-compute-snapshot

but it does not actually run the snapshot command
And when I run the command manually in the shell
gcloud  compute disks snapshot vol-8796758176650--dev-sda-a92b0d8e --snapshot-names gcs-vol-8796758176650--dev-sda-a92b0d8e-1543397435 --zone europe-west1-b
The snapshot is well created.
Thanks for the input :-) I am running the last version avalaible as of today :-)

Hi @Barbouze83 - thanks for your feedback.
What output did you get before you manually changed the script (getDeviceList)?
It would be very helpful if you could provide the output when you run the script normally and also in dry run mode?
Please include all of the output, including the execution of the script (so I can see which inputs were passed).
Thanks,
Jack

Hi @jacksegal ,
I just noticed I ve been retarded, I did not notice the end of the explication of dry run mode
-n    Dry run: causes script to print debug variables and doesn't execute any"
So yeah the modification I made to your script worked for me. my bad, anyway here are the outputs of your unmodified script,
dry-run mode output :
/opt/google-compute-snapshot/gcloud-snapshot.sh -n >> /var/log/cron/snapshot.log 2>&1
[2018-11-28 14:57:12]: Start of google-compute-snapshot
[DEBUG]: OLDER_THAN=7
[DEBUG]: REMOTE_CLAUSE=
[DEBUG]: FILTER_CLAUSE=
[DEBUG]: PREFIX=gcs
[DEBUG]: OPT_ACCOUNT=
[DEBUG]: OPT_PROJECT=
[DEBUG]: DRY_RUN=true
[DEBUG]: DATE_TIME=1543402632
[DEBUG]: DELETION_DATE=20181121
[DEBUG]: INSTANCE_NAME=runew-virtualbox-73237c60
[ERROR]: No disks were found - please check your script options / account permissions.

normal mode output
[2018-11-28 14:59:07]: Start of google-compute-snapshot
[ERROR]: No disks were found - please check your script options / account permissions.

Hope this help. Sorry for the noob question but how do I proceed to give you the last bit you asked?
include all of the output, including the execution of the script (so I can see which inputs were passed).
Thanks

No worries about the "dry run mode"!  And don't worry about the "include all of the output" bit either.
Can you run the following command manually and let me know the output:
sudo gcloud compute disks list --filter "name:runew-virtualbox-73237c60" --format='value(name,zone,id)'

Your command does not work with sudo
sudo: gcloud: command not found
without sudo, it does not produce any output and no error.
thanks for your help :-)

I mean the gcloud command only works when not used in sudo mode


fix bug: ninja is exist

fix bug: https://github.com/istio/istio/issues/24765
make rpm/builder-image failed to create symbolic link '/usr/bin/ninja'
because ninja is exist

Hi @synuwxy. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

@googlebot I signed it!

@googlebot I signed it!

/ok-to-test

@synuwxy: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test integ-multicluster-k8s-tests_istio

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

/retest

@synuwxy: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test integ-multicluster-k8s-tests_istio

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

/retest

@synuwxy: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test gencheck_istio

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

@synuwxy: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test gencheck_istio

integ-security-k8s-tests_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test integ-security-k8s-tests_istio

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

/retest

@synuwxy: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test integ-multicluster-k8s-tests_istio

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

@synuwxy: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test integ-multicluster-k8s-tests_istio

analyze-tests_istio
061f59276f394f5eef1ea86a8a7389f847de9d9c
link
/test analyze-tests_istio

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

Obviously, tools/packaging/rpm/istio/istio.spec Many of the contents of the ' make rpm ' command are not compatible with the version above 1.6.0, and the ' make rpm ' command cannot run successfully

@synuwxy thanks for your interest in this area. Due to some changes to how we are building RPMs now, this will no longer apply. See https://github.com/istio/istio/pull/25837 for more info. Thanks!


[New game]: Virtual Piano

🎮 Game Request
The Virtual Piano Project is an exciting digital platform that provides users with a realistic and immersive piano-playing experience. With a lifelike keyboard interface ,users of all skill levels can enjoy playing the piano from the comfort of their computer or mobile device. Whether you're a beginner or an experienced pianist, the Virtual Piano Project offers a convenient and inspiring way to explore and express your musical talents.

Point down the features
High Quality sounds: The game uses high-quality sounds of a real drum kit, providing a realistic playing experience.
Sample Song : The game provides you with a selection of sample songs that you can use to practice playing the piano.
Recording Feature : The game allows you to record your piano lessons.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

I am highly enthusiastic about working on this project, so if you find my idea appealing, I would be delighted if you assign me the task.

Hey @Chirag-04 ! Thank you for raising an issue 💗 You have been assigned this issue you can start working on the issue 🎮

Hey @Chirag-04 ! Thank you for raising an issue 💗 You have been assigned this issue you can start working on the issue 🎮

Thank you sir for assigning the task to me .

@kunjgit Heya Sir, could you please provide me with guidance on what to do next? I've finished the task assigned to me.

Hello @Chirag-04, Time's Uppp!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @Chirag-04 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


How does one integrate plugins to django-summernote?

For instance this one:
https://github.com/nilobarp/summernote-ext-emoji

You can add custom css/js files in your django-summernote settings via css / js.
At your settings.py;
SUMMERNOTE_CONFIG = {
    'toolbar': [
        # more toolbar items here ...
        ['misc', ['emoji']],
    ],
    'js': (  # This is for SummernoteWidget
        static_url('some/path/to/summernote-ext-emoji.js'),
    ),
    'js_for_inplace': (  # Also for SummernoteInplaceWidget
        static_url('some/path/to/summernote-ext-emoji.js'),
    ),
}

Thanks!
On Sun, Apr 10, 2016 at 2:16 PM Hyunwoo Park<EMAIL_ADDRESS>wrote:

You can add custom css/js files in your django-summernote settings via css
/ js.
At your settings.py;
SUMMERNOTE_CONFIG = {
'toolbar': [
# more toolbar items here ...
['misc', ['emoji']],
],
'js': (  # This is for SummernoteWidget
static_url('some/path/to/summernote-ext-emoji.js'),
),
'js_for_inplace': (  # Also for SummernoteInplaceWidget
static_url('some/path/to/summernote-ext-emoji.js'),
),
}
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/summernote/django-summernote/issues/129#issuecomment-208034807

@dep if this works well, then please close this issue :)

Is their a minimum version for this as it just does not work for me, no request is made for the external files but I am testing against version 0.6.6.

@StevenMapes Sorry for inconvenience. settings of external files had been changed several times. Please use latest version or see changes on https://github.com/summernote/django-summernote/commit/03e3ac4594dce5585607b623d7a1b279bdc55187#diff-04c6e90faac2675aa89e2176d2eec7d8 .

I can't get this to work.
Is there anything else that needs to be done other than adding the plugin to the toolbar, setting css and js in the settings?
I'm using v0.8.5

@gdvalderrama Please refer current settings. It uses different options for SummernoteWidget and SummernoteInplaceWidget.

@lqez yes, thank you.
I've already tried both though, with no luck.

@gdvalderrama ,
Recently, we added the subdirectory djs_playground which is Django sandbox project for testing.
https://github.com/summernote/django-summernote/blob/master/djs_playground/settings.py#L124-L139
You may refere the code as linked above. If you still encounter the issue, let us know your settings.
Otherwise, we would like you to show the same error with our djs_playground subproject.
Thank you.

@gdvalderrama ,
Recently, we added the subdirectory djs_playground which is Django sandbox project for testing.
https://github.com/summernote/django-summernote/blob/master/djs_playground/settings.py#L124-L139
You may refere the code as linked above. If you still encounter the issue, let us know your settings.
Otherwise, we would like you to show the same error with our djs_playground subproject.
Thank you.

Hi I am trying to implement the print option in summernote in my django project. But it is showing some error in chrome browser developer tools as:
Refused to execute script from 'http://localhost:8000/summernote/editor/id_body/static/summernote-ext-print.js/' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled.

You can add custom css/js files in your django-summernote settings via css / js.
At your settings.py;
SUMMERNOTE_CONFIG = {
    'toolbar': [
        # more toolbar items here ...
        ['misc', ['emoji']],
    ],
    'js': (  # This is for SummernoteWidget
        static_url('some/path/to/summernote-ext-emoji.js'),
    ),
    'js_for_inplace': (  # Also for SummernoteInplaceWidget
        static_url('some/path/to/summernote-ext-emoji.js'),
    ),
}

I tried what you have said but in the browser it is showing this error:
Refused to execute script from 'http://localhost:8000/summernote/editor/id_body/static/summernote-ext-print.js/' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled.


Error: Cannot find module 'assertion-error'

I've just done a completely clean build to get the latest release of pgtest, and I'm now getting told that "assertion-error" is missing. I believe this is because it's a devDependency, even though it's required in "lib/pgtest.js"

Oh I messed up, I mistakenly moved that package to devDependencies. Fixing! Sorry :-(