Add L2CAP service

Including fixed channels.
Note: This proto is not compatible with l2cap.proto on AOSP, since the API is not compatible with Bumble(and even most hosts).

I like your direction 👍 I had things to do today, let me review that tomorrow

For server side, it needs to keep global queues because data may be sent right after opening, but client may not start receiving yet.

It's true for the queue (be channel), but not really a big deal I think. Also it will better align on other services like A2dp.

For client side, disconnected will always be known as events instead of something need to be handled as exception or implicit wait in the background

I was seeing the disconnect as the end of the receive stream or as an error on the send, do we really need a specific event ?

Right, so it should still be better to have a synchronized model. I will propose another version tomorrow.

Sorry that was a draft sync...

But for the server questions:

I there a requirements to wait for a server at the device level instead of the connection level ? To avoid the Transport enum.
This is relatively impossible, especially for fixed channels which are usually established right after ACL established. If servers must be registered on connection level, there might be some cases hard to handle.
An alternative way is to define a ChannelType enum instead of Transport enum, so that we can also descritbe other channel types better such as Credit/Retransmission/Connectionless channels.

This is relatively impossible, especially for fixed channels which are usually established right after ACL established. If servers must be registered on connection level, there might be some cases hard to handle.
An alternative way is to define a ChannelType enum instead of Transport enum, so that we can also descritbe other channel types better such as Credit/Connectionless channels.

Humm maybe we could do something like this ?
message DynamicChannelRequest {
  // BR/EDR or Low-Energy connection.
  Connection connection = 1;
  // Protocol/Service Multiplexer.
  uint16 psm = 2;
  // Maximum transmission unit.
  uint16 mtu = 4;
  // Maximum PDU payload size.
  // Required for dynamic channels over LE.
  // For BR/EDR connections, if set (and supported by the host), a credits based channel is opened.
  uint16 mps = 5;
}

message ConnectRequest {
  oneof type {
    // Create a fixed channel from a fixed channel CID.
    uint16 fixed = 1;
    // Create a dynamic channel over BR/EDR or Low-Energy.
    DynamicChannelRequest dynamic = 2;
  }
}

message WaitConnectionRequest {
  oneof type {
    // Create a fixed channel from a fixed channel CID.
    uint16 fixed = 1;
    // Create a dynamic channel over BR/EDR or Low-Energy.
    DynamicChannelRequest dynamic = 2;
  }
}

Not for my requirement(in fact I only need fixed channels for the near future), but on many platforms(like Android), dynamic channels cannot be registered without allocation(though there is a hidden API but not promised for usage).

Ok, I guess we can implement this in a follow-up PR, what do you think ?

This is not fully applicable because ...

Can you try changing the package name package pandora to package pandora.l2cap ?
It may require some modifications in the code generator, but it's the way to go I think.
Thank you for your patience, I known it's not easy to handle such a review with our different timelines :/ ..

Generally it's good enough, but one thing left is that we still need a flag to handle whether enhanced credit based connection is used in LE.

Ok, I guess we can implement this in a follow-up PR, what do you think ?

Sure

Can you try changing the package name package pandora to package pandora.l2cap ?
It may require some modifications in the code generator, but it's the way to go I think.

NP

Left some small nitpicks, I really like this interface thanks @zxzxwu and @uael !

Modified based on comments.
I will provide an implementation for verification soon so we are not hurry to merge this.

@DeltaEvo Could you review this as I will be OOO ?

Sample implementations:

https://github.com/zxzxwu/bumble/commit/c0e53e3a244efb51613fa38d6fd3c55e1b9aa999
https://github.com/zxzxwu/avatar/commit/05f3e77e57da61d11f21895e94ff414fe7cfd0e3


Cards summary page should show the card's "record"

This is now present on the individual card page. It would be very nice to have it on /cards/ but naïve generation is far too costly on user time. It's a very cacheable thing, though, to be updated regularly or whenever a deck is added. Or we can probably just get it out of the db with one horking great SQL query of some sort.

SQL for getting this efficiently rather than loading all decks and summing is:
SELECT card, SUM(wins) AS wins, SUM(losses) AS losses
FROM deck_card AS dc
INNER JOIN deck AS d ON dc.deck_id = d.id
-- optionally: WHERE d.created_date > {season_start_date
GROUP BY card

add0534.


Signature Error moving to production storage on DynamoDB

After setting up credentials for access on us-east-1 and following instructions to switch to storage from local DynamoDB to production DynamoDB, I get the following error when trying to open a graph:
AmazonServiceException The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details.

The Canonical String for this request should have been
'POST
/

content-length:39
content-type:application/x-amz-json-1.0
host:dynamodb.us-east-1.amazonaws.com
user-agent:aws-sdk-java/1.10.26 Mac_OS_X/10.10.5 Java_HotSpot(TM)_64-Bit_Server_VM/25.40-b25/1.8.0_40
x-amz-date:20151015T054332Z
x-amz-target:DynamoDB_20120810.DescribeTable

content-length;content-type;host;user-agent;x-amz-date;x-amz-target
bexxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx9d'

The String-to-Sign should have been
'AWS4-HMAC-SHA256
20151015T054332Z
20151015/us-east-1/dynamodb/aws4_request
c5xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx94'
 (Service: AmazonDynamoDBv2; Status Code: 400; Error Code: InvalidSignatureException; Request ID: 47XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXJG)  com.amazonaws.http.AmazonHttpClient.handleErrorResponse (AmazonHttpClient.java:1181)

I switched to the ClasspathPropertiesFileCredentialsProvider instead of the DefaultAWSCredentialsProviderChain and put the access key and secret key in a file called AwsCredentials.properties and was able to open a graph using production DynamoDB as the store.
I don't know why the default credentials provider chain was not working - I copied the access key and secret key for the new credentials file from the .sh file that I used to set those in the environment variables for the default provider chain.
But for now, I'm past this issue.


Switch to SBC in KFF (incl. HashPlace)

Removes complexity within HashPlace/KffDataObjectHandler to just use the SBC path. Adds additional tests to confirm behaviour and functionality.

We are going to need to see the performance impact of this at full scale. My gut feeling is that using a SBC here will be wasteful and impact throughput.

PR#204 ("Large data support") only calculated the hashes using the SeekableByteChannel. PR#204 spent months being tested on the development system with no observable performance impact by either myself or anyone else on the team. It was not until PR#362 ("KFFHashPlace should prefer byte[] to SeekableByteChannelFactory") that this behavior was changed to calculating hashes using the byte[]. Therefore, I believe the performance impact of exclusively calculating the hashes using the SeekableByteChannel has already been shown to be negligible.

@jdcove2 I hear your points but it has not been tested at full scale. You quote no observable performance differences. Do you have any data to back up those claims?
But as a side note. Emissary is a configuration-driven framework. Why should users of emissary be forced to use SBC if they don't want to?

Given drivenflywheel's comment in PR#362 that "PRs work great for discussion, and they leave a much better historical record than a verbal discussion", it seems that this is the appropriate place to discuss your above concerns.
Therefore, I have the following questions...

If testing on the non-production system is insufficient to be "tested at full scale", then what is sufficient?
If BDO's can now hold data that is significantly larger than what a Java byte array can hold using a SeekableByteChannel, then why would we want to keep the byte array interface on the BDO knowing that it might return only a small subset of the actual data?

Thanks for the extra details Dave. As for the full scale I personally would have liked to see a statistical analysis of the performance on a full scale system. That can be interpolated from a smaller system with a different resource profile.
But it appears that the team is ok accepting this how it is. And not allowing for it to be configurable so I will accept that moving forward.

From my questions above...

It sounds like you have specific performance information you would like to see. I can work with you to produce this performance information. Based on what you said above, please let me know what metrics you want collected, what statistical analysis you want performed and what system you would like the metrics gathered on.

In addition to what I said in my #2 above, every data representation (e.g. byte array, SeekableByteChannel, InputStream, etc.) on the BDO will need to be supported by every place that uses BDO data. That is, if the BDO supports more than one data representation then each place would need to determine which data representation is being used and then process that data representation appropriately. This is why I understand the goal to be staying with one data representation by migrating from byte array to SeekableByteChannel. Eventually the byte array data methods will be removed from the BDO.

The current code in this PR is configurable only because the BDO currently supports both byte array and SeekableByteChannel data representations during the transition from byte array to SeekableByteChannel. All code that uses byte arrays from the BDO will need to be converted to using SeekableByteChannels from the BDO. This PR addresses the code that manages hashes on the BDO and it has code for both data representations. Therefore, this PR is just eliminating the byte array code since the byte array data representation will be eventually removed from the BDO.

@ukengineer925 @jdcove2
I think we need more discussions on this approach. Maybe it will be discussed at our meeting on Friday.
My main concern is the removal of being able to use KFF without sbc.
We should not force all emissary users to use sbc.
I know of use cases where even if you get a large file the hash from a truncated input is useful.
Or we might have flavors of emissary that by design never get large files so why should they have to use sbc in places that don't need them.
This might require some rollback of previous work but a more acceptable and easier to integrate approach would be to have a new KFF hash place or handler that uses sbc.
Then a running server can be configured to use one or the other. Or maybe in the world of testing or a unique use case use both. The framework is very configurable for places and routing. We should leverage it more.
And maybe something changes later in time and it is realized that the old non sbc KFF hash place is never used and not needed then it can go through the deprecation and removal process.

@cfkoehler - I've added a convenience method to allow users to pass in a byte array of data if they really don't want to create their own SBCF (though as you'll see, there's a neat helper method to do that). I don't think maintaining two copies of KffDataObjectHandler, or two paths within one class, is optimal hence this PR.

https://github.com/NationalSecurityAgency/emissary/pull/930


Add protocol Dockerfile

Allows for viewing and creating ASCII network packet diagrams. See
http://www.luismg.com/protocol/

seems cool, thanks!


[Android]How To Prevent And Allow Taking Screenshot In An Android App Using Kotlin To Improve App Security

Topic Suggestion
[Android]How To Prevent Taking Screenshot In An Android App Using Kotlin To Improve App Security
Proposed title of article
How To Prevent And Allow Taking Screenshot In An Android App Using Kotlin To Improve App Security
Proposed article introduction
Certain apps contains confidential information for instance, PayPal android app which is a finance or banking app that restricts the users from taking screenshots. This mechanism helps in securing the app's data. For example if a user wants to share the information of the app via a screenshot.
Implementing this feature in Android applications hasn't been easy for developers. With this article I hope it's going to help the developers task to implement this feature in their apps and secure the user's data.
Key takeaways

What is an app screenshot
How to prevent taking screenshot
How to allow taking screenshot
Advantages of using this feature

Article quality
This article is going to be in Kotlin. As I was doing research on this topic I also happened to realize that the issue has been challenging as per this question on StackOverflow. This article is therefore going to be unique to help in solving the issue fully.
References
Preventing taking of screenshots in Android

Unfortunately, his topic appears to be adequately covered in numerous documentations. Feel free to suggest another topic. Thanks


[question]convert tensorflow models through UFF to TensorRT engines

hi, for me to use the SSD to TRT converter the model must be trained on a specific version of tensorflow or any tensorflow works (tensorflow 1.x and even tensorflow 2.x).

As stated in the README, the "Demo #3: SSD" code in this repo was mainly tested against tensorflow-1.12.  I think tensorflow-1.8 up to 1.15 would all work.  But it would not work for tensorflow-2.x due to API changes.

thanks


Harness for interfacing with Calyx memories

Opening an issue to keep track of this, since it is what I'm currently working on and it seems important for general Filament use as well.
My current idea is to compile a Filament design to Calyx and instantiate it, along with whatever memories it needs to interact with, in a top-level Calyx program. I'm not sure how well this approach would scale/if there are more convenient approaches (like adding standard memory components to Filament that get compiled to Calyx memories?), but I'll post updates/discussions here.

Awesome! One deeper, language-level problem here is that every port in Filament represents a "stream" of data and Filament programs transform the streams in some way. Here, we seem to want an outer "controller" that decides how to send the data for the stream.
Maybe we can do this within the language itself? Spade uses the entity abstraction (which I don't deeply understand) to represent such controllers. The other question is that such controllers probably don't have the same kind of type system as Filament in general

I've been working with this quite a bit and I think there is a strong argument for adding this to the language itself. It feels like you do all this work writing things in Filament so that the timing is correct, but then that sort of goes out the window when you want your design to use any memories. Would it make sense to try writing a Filament signature for the Verilog implementation of the Calyx memories?

Absolutely! You may even consider trying to implement a multi ported Verilog memory and giving it a filament. I’d be curious to see how it works and if we can do the whole darn thing in filament!


Encountering an exception when attempting to play/pause media via DLNA.

Please investigate the play/pause exception that occurs when hitting the cast screen button and selecting media mode. The exception has a status code of 500.
Exception: request http://<IP_ADDRESS>:2869/upnphost/udhisapi.dll?control=uuid:71fbf567-c9ce-4f84-a025-1c9e88e260c4+urn:upnp-org:serviceId:AVTransport error , status 500
I/flutter ( 2316): <SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/" SOAP-ENV:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/">SOAP-ENV:BodySOAP-ENV:FaultSOAP-ENV:ClientUPnPError<u:UPnPError xmlns:u="urn:schemas-upnp-org:control-1-0"><u:errorCode>501</u:errorCode><u:errorDescription>Action Failed</u:errorDescription></u:UPnPError></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope>

it seems your screen cast server Action Failed, maybe it has some differences , I do not know


Add perl and tailwindcss support.

About tailwind, what do you mean by new json.lua? https://github.com/rxi/json.lua hasn't seen any commits for around 4 years. Is there some fork I should look at?

Me and Guldoman tried tailwindcss with lite-xl-lsp and it freezes when you start writing class names. Guldoman observed an enourmous amount of data being sent from the tailwindcss server to lite-xl and said (if I remember correctly) that the freeze could be solved by re-implementing something related to json and lua.
Actually I'm not sure, as I'm a beginner when it comes to lite-xl internals.

Ok, I just consulted Guldoman and he says that a new C implementation of the json lib is needed because tailwindcss sends MBs to lite-xl-lsp, slowing it down.

Thanks for the explanation, also you should adjust your perlnavigator addition to look like this https://github.com/pragtical/lsp/commit/5082ffda6f26291eac526103bf577ddd3f300883 basically the same patter as the other entries on config.lua

I have a request regarding this PR: I made a mistake in setting up this PR, because I didn't create a branch specific to the perl/tailwind addition and now that I want to add the R language I can't avoid mixing commits and making a mess.
My request is to accept (if the PR is acceptable) this PR so that I can update the master branch and create the proper new branch for adding R support.

Thank you!


Utoipa users poll: How do you use responses and how would you like them to work? Answer would help.

Responses poll
Currently while I am working on refactoring multiple aspects of the utoipa lib for the coming release 5.0.0 this is good place to introduce some improvements to current workflow within boundaries set by its users. So here comes the question.
How do you currently use the responses with utoipa lib?

Only tuples
Tuples with  derived ToResponse
Derived IntoResponses
Any other way, please describe.

Second question is how would you like the responses work, or is there any aspect you would like to change in order to them to be more user friendly?

Hi, first of all huge thank you for this crate!
To answer your questions. I use dedicated structs and deriving from ToSchema: https://github.com/agnesoft/agdb/blob/main/agdb_api/rust/src/api_types.rs
I find it quite nice and I would not change it. There are two aspects I think would be nice if improved but I am not sure if possible:

I miss validation of the stuff in the derived against actual code (route signature / types). Obviously things like status codes cannot be validated but I hope types could be. Often I missed the change in return type in the derive annotation and nothing complained. Luckily I have comprehensive test suite so I caught it but a compile error would be nice if there was a mismatch. Probably not possible though.

More comprehensive documentation on the derive macro. I use a lots of stuff from utoipa and sometimes it was difficult to find out how to achieve what I wanted. E.g. https://github.com/agnesoft/agdb/blob/main/agdb_server/src/routes/db.rs#L82 Since I generate apis for different languages and they require things like tags, operation_id etc. That is a common struggle, no such thing as enough docs. :-)

Great, thanks for a reply.

There is improvement for this actually coming in the 5.0.0 release of utoipa. However if you completely forget the add the response(...) declaration then the improvement will not help there yet since the return types are not inferred. But could be with certain limitations. In the 5.0.0 release the request_body and response body schemas will be correctly spanned thus it will give compile error if an item does not exists and will also get renamed if you rename the type as well.
True, more docs, that is forever target of improvement. :slightly_smiling_face:

Hey, not really an answer on how I uses it, but I would really like to be able to share responses between routes.

Responses can be shared if used with ToResponse. Types that implement ToResponse can be added to the responses section of OpenApi. and then they can be references with response = MyResponse.

struct MyResponse;

impl<'s> ToResponse<'s> for MyResponse {
  fn response() -> (&'s str, Response) -> {
    // add all the examples here programmatically
    ("MyResponse", Response::builder().build())
  }
}

#[utoipa::path(
    post,
    path = "/{taskUid}",
    tag = "Tasks",
    params(("taskUid", format = UInt32, example = 0, description = "The task identifier", nullable = false)),
    responses(
        (status = 200, response = inline(MyResponse))
    )
)]
async fn get_task(} {}

You could also derive the ToResponse but I am most likely going to change the ToResponse implementation. And I am not sure about the current behavior of the derive. I think it lacking something but I am not sure what, and perhaps I am going to remove that completely if I find some better way to do shareable responses.
example and examples attributes could be allowed to use method references though nonetheless but hasn't been done so far.

Humm, yes, I tried to use it, but it didn’t work, and I don’t really have the time to understand why, It's probably an issue on my side.
But in terms of API, I find it strange that the ResponseBuilder doesn’t contain the HTTP status code and the schema 🤔
But it’s pretty nice to be able to define multiple examples for one type.
Defining and using the examples through a structure is also pretty good, IMO
Ideally, in our case, almost every route can have three different errors that share the same error code: If you forget to specify an API key, we don’t understand the format of your API key, or your API key doesn’t give you the right to access this route.
Iirc the first two returns an HTTP 400 while the last one return a 401 or something.
Then, they all share the same schema.
It would be really nice to be able to define all that on a single struct and then use it easily in most of my routes:
#[utoipa::path(
    post,
    path = "/{taskUid}",
    tag = "Tasks",
    params(("taskUid", format = UInt32, example = 0, description = "The task identifier", nullable = false)),
    responses(
        inline(ApiKeyError), // here I can define multiple examples with multiple HTTP status code
        inline(SummarizedTaskView), // here I only have one example
        (status = BAD_REQUEST, schema = ResponseError, example = json!(...)),
    )
)]
async fn get_task(} {}

I hope that can help you in your decisions!

Spend way too much time looking into this 😆
What I'd care deeply about is the Good Path Response being as easy to create as possible.

My reasoning/Viewpoint
For us usually following is true:

Unless an error has technical implications and should
cause the caller to do anything but log, we usually don't bother documenting these.
This e.g. on Bad Requests, Bad Auth, Internal Errors, Not Found etc.

Why:

Because IMO on most apis these things can and will not be Programatically handled by the caller,
just logged. As such spending time on perfect docs can be spend on something more effective.
Because each documented error asks the caller to think about how to handle this case specifically
so it adds mental fatigue and makes apis look much harder to implement than they actually are.

However: There are some rare cases where I want percise control over the errors, I would be fine having a a litte more clunky api for these cases.

At the moment, we usually do the tuple based approach with impl Responder as the return - so e.g.
    #[derive(Debug, serde::Serialize, serde::Deserialize, ToSchema)]
    pub struct Task {
        ...
    }

    /// Retrieves a certain task by id
    #[utoipa::path(
        responses(
            (status = 200, body = dto::Task),
        )
    )]
    #[get("/tasks/{id}")]
    pub async fn get_task_by_id(id : web::Path<i32>) -> impl Responder {
        println!("Should now get a task with id {}", id.into_inner());
        Ok(Json(Task:rand()))
    }

My dream would be:

    #[derive(Debug, serde::Serialize, serde::Deserialize, ToSchema, IntoResponses)]
    #[response(status = default_ok)] // Maybe allow this to be optional ? No defined = default ok - So Get 200, Post 201...
    pub struct Task {
        ...
    }

    /// Retrieves a certain task by id
    #[utoipa::path]
    #[get("/tasks/{id}")]
    pub async fn get_task_by_id(id : web::Path<i32>) ->  Result<Json<dto::Task>> {
        println!("Should now get a task with id {}", id.into_inner());
        Ok(Json(Task:rand()))
    }

What I would love to see:

A "default_ok" status code which could be used with the IntoResponse derive.
Why? We often use the same Response dto for all C/R/U/D ops.
Thus mapping the status code 1:1 to my dto doesn't make sense for how I work.

Ideally there would be a blanket

impl<T:ToSchema/ToResponse> IntoResponses for Result<T, E> or
impl<T:ToSchema/ToResponse, E:ToSchema/ToResponse> IntoResponses for Result<T, E>
The 2nd would ofc be much more correct than the first. However would require that every error we return implements ToResponse.
This would be annoying as for us at least, most errors are from upstream crates, thus making it impossible for us to implement the ToResponse directly.
This would be required as the user of utoipa can not make these implementations themselves.

For actix_extras there should e.g. be a impl IntoResponses/ToResponse for Json<T> as this can not be implemented by the user.

IDK if this is possible - but it would also be nice if everything which implements ToResponse
could be used with the responses attribute.
They would then use the default_ok status code for the operation.

Until I started looking into the topic specifically I did not understand the usage of the ToResponse and that
with actix_extras I can automatically infer the response

This Could be me being slow, or maybe the Usage docs being hard to follow.

How I might start using this now
I might implement my own Result type like:
pub enum GetResult<Ok> {
    Ok(actix_web::web::Json<Ok>),
    Err(actix_web::Error),
}

impl<Ok:Serialize> Responder for GetResult<Ok> {
    type Body =  EitherBody<<actix_web::web::Json<Ok> as Responder>::Body>;

    fn respond_to(self, req: &actix_web::HttpRequest) -> actix_web::HttpResponse<Self::Body> {
        match self {
            GetResult::Ok(t) => t.respond_to(req).map_into_left_body(),
            GetResult::Err(e) => e.error_response().map_into_right_body(),
        }
    }
}

impl<Ok: for <'a> ToResponse<'a>> IntoResponses for GetResult<Ok> {
    fn responses() -> std::collections::BTreeMap<String, utoipa::openapi::RefOr<utoipa::openapi::response::Response>> {
        let mut map = std::collections::BTreeMap::new();
        map.insert("200".to_string(), Ok::response().1);
        map
    }
}

/// ... PostResult

Don't love it as I'll have to into() all my responses this way, but not sure if there is a different idea.
Or possibly might add some post processing to the generated OpenAPI
which maps all the OK responses to the correct counterpart, and only require a single custom Result


feat(ONYX-742): notifications empty state redesign

The type of this PR is: Feat

This PR solves ONYX-742
Description
Update Notifications empty state design, added links "Artists" and "Galleries" that lead to their pages.
https://github.com/artsy/force/assets/17580625/5badc5dd-2329-44f8-8d04-ccf05784ed1f
https://github.com/artsy/force/assets/17580625/0b3a7ad3-bee5-4ff2-88b8-89e9cddd18ee

@MounirDhahri The "Offer" pill is conditional so when there is an empty state - it is not visible, unlike "Alerts" and "Follows"

Perfect 👌


Could you show us the project's BP?

Our team are very interested in this project and Could you show us the project's BP?

@matthewei I'm not sure what you mean under "BP", could you elaborate? You can always contact me on Slack, but please note: this project is in a very early stage. It's not ready for production yet.

"BP" I mean "blueprint" I am very interested in this project and I want to verify it. If given the chance, I would like to contribute something to this project.

@matthewei one thing I wanna do is to write a formal proposal against metal3-docs. Otherwise, the closest I have is https://github.com/metal3-io/ironic-standalone-operator/blob/main/ARCHITECTURE.md.
If you want to help, the first and foremost work item is https://github.com/metal3-io/ironic-standalone-operator/issues/29. It's not my strongest area, so if you feel like you're up to the task - feel free to grab it.

@dtantsur I will try it！

@dtantsur This scripts are generated by kubebuilder. Why do you think we should modify it?

I think we just finish `baremetal-operator
/ironic-deployment/ and we can use it.

Could we take this discussion to the slack channel @matthewei ? I can't really decide how to triage this issue ad the original question was answered by @dtantsur .

/close


Correctly focus the last tab when closing Neogit* buffers

Is your feature request related to a problem? Please describe.
Would be great if the kind "tab" worked correctly with closing buffers, or at least provide a way to customize the generic Buffer mappings. The feature request is that when closing a tabbed neogit buffer and there is no alternate buffer on the  current tab, of any kind the last focused tab should be opened i.e g functionality. The current behaviour would just focus the next tab instead.
Describe the solution you'd like
Either a user configuration, or implemented in neogit
Describe alternatives you've considered
Overriding the Buffer:close but the buffer file has this at the top package.loaded["neogit.buffer"] = nil which would prevent us from overriding in a robust way.
Additional context
None

I use <Cmd>tabnext # <Bar> tabclose #<CR> to close Diffview's tab, something similar can be done for Neogit too.


Add different projects, links and a new section for security

I started with a new list here https://github.com/Karneades/awesome-ebpf but found yours today. So I integrated missing links into your list now and will delete my own afterwards. The section security will focus on the security / malware aspects of eBPF.
If I should move the security tools added to the projects section to this new security section to better find them, just say it.

Thanks for your review! Will work through all of your suggestions, which will improve all my additions and update this PR accordingly.

I updated everything as suggested and make some comments above in @qmonnet comments. Furthermore, I addes some more links/tools.

Thanks again for your suggestions.
Additions in summary

Add new section for eBPF in security.
Add Cloudflare's and Brendan Gregg's eBPF links to generic article section.
Add Cilium.io link in addition to the existing GitHub link for Cilium.
Add bpftrace cheat sheet.
Add different links related to security.

Tools: Falco, Sysmon4Lnx, Tracee, eBPF4Win, Red Canary's eBPF sensor and bpflock.
Offensive security: Embrace The Red blog series about how eBPF could be abused, post about malware blocking using BPF LSM and eBPF rootkit talks and the corresponding tools.

Merged, thanks again!


Using inference from a virtual environment

Hi, I'm trying to build and run the project from a virtual environment (I tried both with conda and virtualenv, the results are the same)
I ran the whole building procedure successfully, and when I ran 'cmake ../' the process even found the Python3.6 version in the conda virtual environment:
 detecting Python 3.6...
-- found Python version:  3.6 (3.6.13)
-- found Python include:  /home/lews/archiconda3/envs/tf/include/python3.6m
-- found Python library:  /home/lews/archiconda3/envs/tf/lib/libpython3.6m.so
-- CMake module path:  /home/lews/projects/jetson-inference/utils/cuda;/home/lews/projects/jetson-inference/python/bindings;/home/lews/projects/jetson-inference/python/bindings/../../utils/python/bindings
-- NumPy ver. 1.19.5 found (include: /home/lews/archiconda3/envs/tf/lib/python3.6/site-packages/numpy/core/include)
-- found NumPy version:  1.19.5
-- found NumPy include:  /home/lews/archiconda3/envs/tf/lib/python3.6/site-packages/numpy/core/include

I can use detectnet from the system python installation, but when switching to another virtual environment I get the error
"ModuleNotFoundError: No module named 'jetson'"

I tried following the procedure described here, but with no success so far

I haven't used virtual environments with this project before, so I'm of limited help with this, sorry about that.
However after installing jetson-inference, can you make the virtualenv with virtualenv --system-site-packages ?

@dusty-nv @LewsTherin511
I could get the jetson.inference and jetson.utils working inside my virtual environment.  As correctly mentioned in the build procedure, the installation happens under /usr/lib/pythonXX/dist-packages location. The key is not just copy the .so files from this directory to site-packages inside virtual environment but also 2 other folders namely the jetson and Jetson folder. Do this and it should work -
cp /usr/lib/pythonX.X/dist-packages/jetson_utils_python.so <your_virtual_env>/lib/pythonX.X/site-packages
cp /usr/lib/pythonX.X/dist-packages/jetson_inference_python.so <your_virtual_env>/lib/pythonX.X/site-packages
cp -r /usr/lib/pythonX.X/dist-packages/jetson <your_virtual_env>/lib/pythonX.X/site-packages
cp -r /usr/lib/pythonX.X/dist-packages/Jetson <your_virtual_env>/lib/pythonX.X/site-packages
Once you move both the .so files and the 2 Jetson folders inside the virtual environment, activate your virtual environment and try importing jetson.inference and jetson.utils. They should work well. Attaching a SS for reference:


SqlWindowsFirewall: Added localization

Pull Request (PR) description

Changes to SqlSetup

Made the error message more descriptive when the Set-TargetResource
function calls the Test-TargetResource function to verify the desired
state.

Changes to SqlWindowsFirewall

Added en-US localization (issue #614).

Changes to SqlRS

Fixed one of the error handling to use localization, and made the
error message more descriptive when the Set-TargetResource function
calls the Test-TargetResource function to verify the desired
state. This was done prior to adding full en-US localization.

This Pull Request (PR) fixes the following issues

Fixes #614

Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

Merging #1336 into dev will increase coverage by <1%.
The diff coverage is 100%.

@@          Coverage Diff          @@
##            dev   #1336    +/-   ##
=====================================
+ Coverage    98%     98%   +<1%
=====================================
  Files        36      36
  Lines      4889    4917    +28
=====================================
+ Hits       4817    4845    +28
  Misses       72      72


Append the running timer to the Day’s total

Describe the new feature
Today's total should include the running timer. If this can be added elsewhere instead, that's ok too.
How does this help you?
Gives user a tally of total hours logged thus far similar to web-app.
If we couldn't add this feature, is there a compromise you can think of?

Case reference 1
Case reference 2


new attribute on root level created

What would you like to be added?
if there would be an attribute created, which is a datetime (format to be specified)
Alternatitivly or additionally there could be a revision field added as an integer to represent a version of the file
Why is this needed?
this would allow to track several versions of the same export and make sure the latest one is imported / this can be shown during an import process.

[!WARNING]
The attribute revision maybe will be used for a revision number auf der standard

created is a good idea


CLI: add install --from-download for sssp and pseudo-dojo

Fixes #92
Our workaround for installing an "established" family like a SsspFamily or PseudoDojoFamily required using the install family command, but this is explicitly forbidden since we do not want the user to be able to install an arbitrary set of pseudopotentials as one of these classes. This means there is currently no way to install the archive and metadata obtained with the --download-only option of the automated install commands as a proper SsspFamily or PseudoDojoFamily.
Here, we add the --from-download option to the automated install commands to allow the user to install the downloaded pseudopotentials and set the recommended cutoffs automatically for an SsspFamily or PseudoDojoFamily. The --download-only option now also produces a single .aiida_pseudo archive that contains:

The archive containing the pseudos
The corresponding metadata
The configuration in JSON format

The user can pass such an archive to the from-download option to install the corresponding pseudopotential family.

@sphuber still have to update the documentation instructions. Here's a quick example of the usage:
(aiida-pseudo) mbercx@theospc46:~/envs/aiida-pseudo/data$ aiida-pseudo install pseudo-dojo -x LDA -p standard -f upf --download-only
Report: downloading selected pseudopotentials archive...  [OK]
Report: downloading selected pseudopotentials metadata archive...  [OK]
Success: Pseudopotential archive written to: PseudoDojo_0.4_LDA_SR_standard_upf.aiida_pseudo
(aiida-pseudo) mbercx@theospc46:~/envs/aiida-pseudo/data$ ls
PseudoDojo_0.4_LDA_SR_standard_upf.aiida_pseudo
(aiida-pseudo) mbercx@theospc46:~/envs/aiida-pseudo/data$ aiida-pseudo install pseudo-dojo --from-download PseudoDojo_0.4_LDA_SR_standard_upf.aiida_pseudo
Report: unpacking archive and parsing pseudos...  [OK]
Report: unpacking metadata archive and parsing metadata.../home/mbercx/envs/aiida-pseudo/code/aiida-pseudo/src/aiida_pseudo/groups/family/pseudo_dojo.py:278: UserWarning: filename Ge.djrepo_old does not have a supported extension. Skipping...
  warnings.warn(f'filename {filename} does not have a supported extension. Skipping...')
 [OK]
Success: installed `PseudoDojo/0.4/LDA/SR/standard/upf` containing 70 pseudopotentials

If we still agree on the --download-only and --from-download approach, I'll update the docs as well.

If we still agree on the --download-only and --from-download approach, I'll update the docs as well.

Remind me, did we any alternatives? Did we figure out any obvious problems of this approach? If not, it seems reasonable to me.

I think your comment here presents some of the alternatives discussed: https://github.com/aiidateam/aiida-pseudo/pull/94#issuecomment-839145418. You can also read the two comments below regarding the choice to switch to --from-download.
My one idea was still to extend this .aiida_pseudo archive idea to installing other families, so it's easier for people to share pseudo families with specified cutoffs, but that's beyond the scope of this PR. If you think it's a good idea I'll open an issue for it.

I think your comment here presents some of the alternatives discussed: #94 (comment). You can also read the two comments below regarding the choice to switch to --from-download.

I remember now. I was wondering if only accepting a tarball with everything was not too limited. But I think I was maybe getting ahead of myself. Maybe let's first merge this and see if people actually need it and whether they would require even more control.

My one idea was still to extend this .aiida_pseudo archive idea to installing other families, so it's easier for people to share pseudo families with specified cutoffs, but that's beyond the scope of this PR. If you think it's a good idea I'll open an issue for it.

I like the concept, feature request would be good to discuss the details.

@sphuber docs should be updated! So this PR is ready for review.

I totally missed this review, sorry @sphuber! I'll be picking up this PR soon since it's important for rapid QEapp cloud deployments.

@mbercx I am very much interested in getting this feature in for a use-case we have. Do you have time soon to finalize this? If not, I could also take over if you are ok with that.

@mbercx I am very much interested in getting this feature in for a use-case we have. Do you have time soon to finalize this? If not, I could also take over if you are ok with that.

I'm moving from EPFL to PSI the next two days, but I could have a look at it over the weekend? Would be good to have it merged for sure.

@sphuber thanks for the review! I think I addressed all your comments. I've moved the download_only logic up, which does lead to some code duplication of:

The download call.
Creating the label.
Checking if the configuration is valid.

I've also moved the check if the configuration is already installed before the download call in https://github.com/aiidateam/aiida-pseudo/pull/135/commits/3882e06d93884d34643959449b65b642c64f0eb4. This is to avoid pointlessly downloading the pseudos, so the user doesn't see the following:
❯ aiida-pseudo install sssp
Report: downloading patch versions information...  [OK]
Report: downloading selected pseudopotentials archive...  [OK]
Report: downloading selected pseudopotentials metadata...  [OK]
Report: SsspFamily<SSSP/1.3/PBE/efficiency> is already installed

But instead
❯ aiida-pseudo install sssp
Report: SsspFamily<SSSP/1.3/PBE/efficiency> is already installed

@mbercx @sphuber could you make a patch release with this? I need it for the AiiDAlab QE, Thanks.

@unkcpz done: https://pypi.org/project/aiida-pseudo/1.3.0/

Thanks a lot!


Verify the convection-diffusion equation

Verifying only the convection-diffusion equation (the energy equation) is not so straightforward.
This issue was raised when natural convection was not working. This was supposed to be an intermediate verification step. Now natural convection is working; so, for the time being, I'm not interested in this intermediate step. I'm closing this for now.


Select an audio input device: it is empty.

Set up on a CentOS server following the tutorial.
Select an audio input device: it is empty. JS error: Cannot read properties of undefined (reading 'enumerateDevices').

Make sure you connect with localhost/<IP_ADDRESS> instead of <IP_ADDRESS>


Change pagination constant to 10 and off by one in pages_after

Just to be clear - the ammount of pages to show in each side should be 10 (instead of 7), results per page should still be 50.

Irrelevant - new API implements pagination using DRF


install_demo_configuration.sh won't run if elasticsearch.yml contains lines with "opendistro_security"

It won't run because inside my elasticsearch.yml file I have
# opendistro_security.disabled: true
After hours of trying to figure out what went wrong I found this in install_demo_configuration.sh
if $SUDO_CMD grep --quiet -i opendistro_security "$ES_CONF_FILE"; then   echo "$ES_CONF_FILE seems to be already configured for Security. Quit."   exit $skip_updates fi
IMO this is a valid issue, it's not unusual during testing to have a config file with commented out lines for commonly used options, and the main use case for install_demo_configuration.sh is for testing/learning. Having commented out options alter the behavior of the docker container was confusing.
My suggested solution is to add another option to ignore this check and use it in docker-entrypoint.sh. Something like
 -x ignore existing opendistro_security config entries

@starcatter install_demo_configuration.sh is used to generate all required TLS certificates and add the Security Plugin Configuration to your elasticsearch.yml file. When those configurations have already existed in your elasticsearch.yml file, the script is expected not to duplicate the existing configurations. This is the whole point of the check. If the comments containing opendistro_security is not a must-have, I'd suggest to remove them. Otherwise, please feel free to reopen the issue. Thanks!

We are doing some "spring cleaning in the fall", and to make sure we focus our energies on the right issues and we get a better picture of the state of the repo, we are closing all issues that we are carrying over from the ODFE era (ODFE is no longer supported/maintained, see post here).
If you believe this issue should still be considered for current versions of OpenSearch, apologies! Please let us know by re-opening it.
Thanks!


Trying to use Spring Cloud Config Server with Docker but got an error at Docker-build time

Hello, i'm recently started to use Spring Cloud with Docker, but i'm stuck at Config Server and i don't know why. My app worked fine when it was just Spring and others microservices involved, but when i'm trying to build a Dockerfile for Config server a got an error. I'm using 	<spring-cloud.version>2020.0.1</spring-cloud.version>
[ERROR] contextLoads  Time elapsed: 0.013 s  <<< ERROR! java.lang.IllegalStateException: Failed to load ApplicationContext
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'healthContributorRegistry' defined in class path resource [org/springframework/boot/actuate/autoconfigure/health/HealthEndpointConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.boot.actuate.health.HealthContributorRegistry]: Factory method 'healthContributorRegistry' threw exception; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'configServerHealthIndicator' defined in class path resource [org/springframework/cloud/config/server/config/EnvironmentRepositoryConfiguration.class]: Unsatisfied dependency expressed through method 'configServerHealthIndicator' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.springframework.cloud.config.server.config.CompositeConfiguration': Unsatisfied dependency expressed through method 'setEnvironmentRepos' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'defaultEnvironmentRepository' defined in class path resource [org/springframework/cloud/config/server/config/DefaultRepositoryConfiguration.class]: Invocation of init method failed; nested exception is java.lang.IllegalStateException: You need to configure a uri for the git repository.
Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.boot.actuate.health.HealthContributorRegistry]: Factory method 'healthContributorRegistry' threw exception; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'configServerHealthIndicator' defined in class path resource [org/springframework/cloud/config/server/config/EnvironmentRepositoryConfiguration.class]: Unsatisfied dependency expressed through method 'configServerHealthIndicator' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.springframework.cloud.config.server.config.CompositeConfiguration': Unsatisfied dependency expressed through method 'setEnvironmentRepos' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'defaultEnvironmentRepository' defined in class path resource [org/springframework/cloud/config/server/config/DefaultRepositoryConfiguration.class]: Invocation of init method failed; nested exception is java.lang.IllegalStateException: You need to configure a uri for the git repository.
Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'configServerHealthIndicator' defined in class path resource [org/springframework/cloud/config/server/config/EnvironmentRepositoryConfiguration.class]: Unsatisfied dependency expressed through method 'configServerHealthIndicator' parameter 0; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.springframework.cloud.config.server.config.CompositeConfiguration': Unsatisfied dependency expressed through method 'setEnvironmentRepos' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'defaultEnvironmentRepository' defined in class path resource [org/springframework/cloud/config/server/config/DefaultRepositoryConfiguration.class]: Invocation of init method failed; nested exception is java.lang.IllegalStateException: You need to configure a uri for the git repository.
Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.springframework.cloud.config.server.config.CompositeConfiguration': Unsatisfied dependency expressed through method 'setEnvironmentRepos' parameter 0; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'defaultEnvironmentRepository' defined in class path resource [org/springframework/cloud/config/server/config/DefaultRepositoryConfiguration.class]: Invocation of init method failed; nested exception is java.lang.IllegalStateException: You need to configure a uri for the git repository.
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'defaultEnvironmentRepository' defined in class path resource [org/springframework/cloud/config/server/config/DefaultRepositoryConfiguration.class]: Invocation of init method failed; nested exception is java.lang.IllegalStateException: You need to configure a uri for the git repository.
Caused by: java.lang.IllegalStateException: You need to configure a uri for the git repository.

but my bootstrap yml is:
spring:
  application:
    name: CONFIG-SERVER
  profiles:
    active:
    - git
  cloud:
    config:
      server:
        git:
          default-label: main
          uri: https://github.com/hideyourname/cloud-config-server.git

I have @EnableConfigServer on my main class and config server dependency on my POM
Soo.. idk where is the problem, why it can't see my git repo?

How did you solve this?


Trigger Error Handling

Hello,
When writing a trigger and completing a try catch for my DML I would do something similar to the following:
            Map<Id, Account> accounts = new Map<Id, Account>((List<Account>) Records);

            try
            {
                update onlyTheAccountsIWantToUpdate;
            }
            catch(DmlException dmlException)
            {
                for (Integer i = 0; i < dmlException.getNumDml(); i++)
                {
                    Account acc = accounts.get(dmlException.getDmlId(i));
                    acc.addError(dmlException.getDmlMessage(i));
                }
            }

I do this to remove all the stacktrace details so that a more user friendly error message can be displayed to the user (such as when declarative validations are hit).
Currently in this library I am unable (or not sure how to) do this. Can someone please let me know if this is an enhancement or if it is already possible to do?
Thanks!
Tyler

If this is in a Domain layer class (which among other things, implements the trigger handlers), then take a look at this post  where I had a related issue on processing the error message.  In the link, you would enhance my verror(...) method to cleanse the exception message of noise-not relevant to the user.
In my orgs, I remove the text 'FIELD_CUSTOM_VALIDATION_EXCEPTION,' and strip out the stack trace except for certain exceptions where I prefer it surfaces right away to aid in diagnostic response time


Use proper annotation for JSON fields

Describe the pull request:

[x] Bug fix
[ ] Functional change
[ ] New feature
[ ] Code cleanup
[ ] Build system change
[ ] Documentation change
[ ] Language translation

Pull request long description:
Use proper annotation for JSON fields

Pull Request Test Coverage Report for Build 61

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.971%

Totals

Change from base Build 59:
0.0%

Covered Lines:
126

Relevant Lines:
137

💛  - Coveralls


Conflict assigning cla manager role for Project error appears when user start the cla process

Steps:

Login corporates console
Search & open project say : Veer project Staging 01
Click on easyCLA
Click on Start the cla Process button
Click on Yes at "No Signed CLA Found" prompt
See-Activity CLAS for my account section appears
Now search & open easyCLA page of another project
Reopen easyCLA of project Veer project Staging 01
Now click on  Start the cla Process button then Yes

Actual Results :Conflict assigning cla manager role for Project error appears
Refer video clip for more details : https://images.zenhubusercontent.com/194341141/515d2252-41d4-416a-a7bf-5fd9444f7769/conflicterror.mp4

Updated logic to not fail on conflict/409 when the role is already set.  Will create PR shortly.

Updated logic to not fail on conflict/409 when the role is already set.  Will create PR shortly.

This should be available on Dev, Staging & PROD.

This should be available on Dev, Staging & PROD.

Issue is tested on Dev & staging site & found that issue is not reproducible , So making issue status to QA-Passed
Refer video clip for more details
https://images.zenhubusercontent.com/194341141/20411052-c103-4049-9e17-08c0d1c3c27b/2439_dev.mp4
https://images.zenhubusercontent.com/194341141/34e23398-6938-4780-8223-504a9719f1f8/2439.mp4

Issue is tested on Dev & staging site & found that issue is not reproducible , So making issue status to QA-Passed
Refer video clip for more details
https://images.zenhubusercontent.com/194341141/20411052-c103-4049-9e17-08c0d1c3c27b/2439_dev.mp4
https://images.zenhubusercontent.com/194341141/34e23398-6938-4780-8223-504a9719f1f8/2439.mp4


Table card

Summary

Fix on column width when icon + expanded row

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-beta.9 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.86.0 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


Office-Addins & Blazor: Possible?

Is it possible to combine Blazor and Office Addins? Would like to be able to execute .NET standard libraries from the task pane.
My apologies if this is already listed somewhere, I've been able to find anything in searches.

Thank you for your suggestion, this is a good one @mclean25! We are looking into this support. If you want to enter a UserVoice for this suggestion, the community can also vote on it. Cheers!


Add (recursive) processing of files in directories to the command-line tool

It would require fixing of #78 too, and a special value called maybe “selfsamefile” for the “--output” argument or another argument for having the selfsame files as output rather than the standard output.
We could differentiate whether files in subfolders are affected by an additional “--recursive” argument.
Actually, processing of whole folders is the crucial feature one desires from the command-line tool in ftfy, because one is hardly slower just opening and resaving files with medit or an other encoding aware editor for the purpose of fixing files ony by one.

This and #80 both sound out-of-scope. You could use ftfy to create a recursive file-fixing tool, but we shouldn't add a lot of file management to ftfy for one particular use case.
But also, if opening and saving with an editor is sufficient to fix the file, that sounds like a case where ftfy is hardly necessary -- it sounds like it's just in an unknown encoding, not mojibaked.


Syntax  error in generated code $selected{$_}++ for $self->param($name);

Syntax  error in generated code $selected{$_}++ for $self->param($name);

Fixed in v1.021


define jitter_amps as single amplitude in default config

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 90.558%

Files with Coverage Reduction
New Missed Lines
%

sed/calibrator/energy.py
2
91.86%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
4297

Relevant Lines:
4745

💛  - Coveralls

With such a default, one does not need to redefine it for different number of jitter cols in other configs


How to force download file types?

Hi my question is how can I download a file type without showing it in my browser I need to force download the file when I click on that file.
What is the correct configuration for this to place in my caddyfile?

Basically this?
https://stackoverflow.com/questions/10615797/utility-of-http-header-content-type-application-force-download-for-mobile
Possibly with this?
https://caddyserver.com/docs/header
Or that?
https://caddyserver.com/docs/mime
Which file type are you trying to force download for?
I think this question should be asked here: https://caddy.community/latest

Yep, as @magikstm said, this kind of question should be asked on the forums. Closing this since it's not an issue, but feel free to continue discussion here if you like.


7582 fix clusterrolebindings

Summary
Fixes #7582

Occurred changes and/or fixed issues

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Adding milestone

@Shavindra is this PR linked to the right issue? #7582 looks like a bug in the Ember UI

I did think about this. It would require some modifications to ResourceTable component. groupby namespace / flatlist is hardcoded in. I was bit apprehensive to make the change there. If you think I should remove it and should do within this PR. I will do it

I think since the table is used in sooo many places and it'd would take time to validate any changes to it, it would be fine to file a follow-up issue to stop hardcoding group by namespace.

I couldn't quite replicate this?
Ahhh ok I tried again if I wait on the page it shows up in a few seconds, I guess the cluster I was using was just slower than yours.


documentation - FunctionReturn isn't in docs

FunctionReturn isn't in api.binary.ninja - You can use this to indicate an instruction will return:
example, in "perform_get_instruction_info" for the DEX architecture:
if instr in ["return-void", "return", "return-wide", "return-object"]:         result.add_branch(FunctionReturn)

A small bug in the table prevented the relevant snippet of documentation from showing up, it's maybe not quite where it should be, but it is searchable at least now along with all the other branch types:
http://api.binary.ninja/python.html?highlight=functionreturn#binaryninja.Architecture.perform_get_instruction_info


buy option grayed

Hi, in ASDK, when create a virtual machine, everything goes OK, the summary page shows validation passed, but option 5 "buy" is grayed, can´t create the virtual machine. No error message, just option 5 grayed. Any ideas? Thanks for your help!

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

ID: 51a36e4c-e20e-e8fa-2944-f004ef046e09
Version Independent ID: 0aec3f91-5789-7ffc-55ed-50bdab79d7eb
Content: Make virtual machines available to your Azure Stack users
Content Source: azure-stack/operator/azure-stack-tutorial-tenant-vm.md
Service: azure-stack
GitHub Login: @Justinha
Microsoft Alias: justinha

@juajac Thank you for bringing this to our attention. We are currently investigating and will update you when we have greater clarity surrounding the issue.

@juajac First, I would make sure that everything in your ASDK deployment is working as expected. You can do this by validating your installation using test-azurestack.  If that shows everything as green, make sure that the subscription has adequate quotas for the deployment.

@juajac Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


fix #2062

Added an option to ignore the pointer clicking delay in a button over event.

It can be simplified as
if(!fastClick && pointer.justReleased()) { ..

Anyway, I wonder what the primary point of the timeout check is/was..

Then it makes me wonder what the use case(s) is(are) for 'slow clicks' - and maybe if such should be handled in the button itself (and thus handled per button, with the ability to recover from a re-hover) vs deferral to the input handling.

After looking at it a bit more, I can't seem to understand what use there is in delaying the clicks. Perhaps, even the deletion of the condition might be okay?

I'm not sure why it was added either .. but it is there, so presumably there was a case for it. I suspect it might have been / might be to prevent 'resetting to hover state' if a click is immediate followed by a hover-move.

I'm actually inline to say the check should be removed entirely - ie. revert https://github.com/photonstorm/phaser/commit/96fd0ade
I cannot find any mention of what bug it fixes, if any. This includes that commit and the following Phaser (2.3.1) release documentation.

Anyway, this commit won't work right (and I should have spotted it sooner) because the fastClick parameter, coming from an Input event, will never be anything but undefined.

So, it's probably in best interest to delete the condition down right.


Revert "Rebuild for gnuradio_core394"

Reverts conda-forge/gqrx-feedstock#27

Hi! This is the friendly automated conda-forge-webservice.
It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.
Please close this pull request and remake it from a fork of this feedstock.
Have a great day!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Bump lightbend/config to include patch allowing long strings of numbers as map keys

The current version included of https://github.com/lightbend/config includes a bug that prevents using long strings of numbers as map keys, as it tries converting them to an integer for comparison purposes. This doesn't work due to the length, which is checked in newer versions.

Hi! In Configurate 3.7 snapshots the lightbend/config dep has been bumped up to 1.4.0, which should fix this issue. That release is pretty much complete, so I'll try to get a final version pushed in the next few days.


Can't see over walls

With just Wall Height enabled, you can see over all the 50ft walls in the picture below. And you can see the token that is 50ft in the air nearby. With Elevated Vision installed you can neither see the token nor see through the wall. This seems like it should be possible considering both tokens are above the top of the wall.

This is what it looks like with just Wall Height turned on and Elevated Vision turned off:

That is happening with me too, but only with the WebGL option.

Fixed in v0.5.13. Thanks!


Enable playwright test coverage reporting for ubuntu Google chrome

Adds playwright test coverage using vite-plugin-istanbul into the Playwright worfflow.
Note:
I've had to rename vite.config.ts to vite.config.mts as per their doco on their website , in order to get around the issue below:
Error: R] Failed to resolve "vite-plugin-istanbul". This package is ESM only but it was tried to load by `require`. See https://vitejs.dev/guide/troubleshooting.html#this-package-is-esm-only for more details. [plugin externalize-deps]

    node_modules/esbuild/lib/main.js:1225:27:
      1225 │         let result = await callback({
           ╵                            ^

    at file:///home/runner/work/modeling-app/modeling-app/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:66924:27
    at requestCallbacks.on-resolve (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:1225:28)
    at handleRequest (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:647:17)
    at handleIncomingPacket (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:672:7)
    at Socket.readFromStdout (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:600:7)
    at Socket.emit (node:events:519:28)
    at addChunk (node:internal/streams/readable:559:12)
    at readableAddChunkPushByteMode (node:internal/streams/readable:510:3)
    at Readable.push (node:internal/streams/readable:390:5)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23)

  This error came from the "onResolve" callback registered here:

    node_modules/esbuild/lib/main.js:[11](https://github.com/KittyCAD/modeling-app/actions/runs/10366835465/job/28696993233#step:17:12)50:20:
      1150 │       let promise = setup({
           ╵                     ^

    at setup (file:///home/runner/work/modeling-app/modeling-app/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:66899:18)
    at handlePlugins (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:1[15](https://github.com/KittyCAD/modeling-app/actions/runs/10366835465/job/28696993233#step:17:16)0:21)
    at buildOrContextImpl (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:873:5)
    at Object.buildOrContext (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:699:5)
    at /home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:2023:15
    at new Promise (<anonymous>)
    at Object.build (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:2022:25)
    at build (/home/runner/work/modeling-app/modeling-app/node_modules/esbuild/lib/main.js:[18](https://github.com/KittyCAD/modeling-app/actions/runs/10366835465/job/28696993233#step:17:19)73:51)
    at bundleConfigFile (file:///home/runner/work/modeling-app/modeling-app/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:66854:24)

  The plugin "externalize-deps" was triggered by this import

    vite.config.ts:6:27:
      6 │ import IstanbulPlugin from "vite-plugin-istanbul"
        ╵                            ~~~~~~~~~~~~~~~~~~~~~~

If this gets annoying or becomes problematic, we can disable it by turning off the GENERATE_PLAYWRIGHT_COVERAGE in the workflow

I really wanted this in, but haven't been able to get this to produce the reports. it seems that twindow.__coverage__ is always undefined, and I'm not sure why because Ry had this working at one point and I don't think anything has changed.
Going to put it down for now.


Support API Version Interleaving Across Types

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: dawaji sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

The full support across all platforms with comprehensive tests was merged as part of #396, so I'm going to go ahead an close this PR. Thanks for taking a stab at it.


Cache global pwsh completions across terminals and reloads

Fixes #222302

approved, but looks like integration test failed


bugfix: replaces React_2 for React from type generation

Fixes https://github.com/floating-ui/floating-ui/issues/2863

manually replace React_2 for React from type generation at build-api script
adds build-api script as global dependency on turbo configuration
bumps api-extractor to latest version
adds changeset for patching:

@floating-ui/react
@floating-ui/react-dom
@floating-ui/react-native

Just wanted to make it clear, this is not an ideal solution! we should subscribe to https://github.com/microsoft/rushstack/issues/2479 and once that is fixed on API extractor we can roll this PR back


fix(app): refactor instrument cards for bad instruments

fix RQA-1478, RQA-1315, RQA-1252

Overview
Instrument cards for bad instruments were rendered outside of the GripperCard and PipetteCard components, which is where attachment flows are rendered. This caused flows to close when an instrument that needed a subsystem update was attached during a flow.
This PR refactors GripperCard and PipetteCard to handle instruments that need subsystem updates so that no longer happens and reflects in progress updates on both desktop and ODD.

Test Plan
Go through an attachment flow with an instrument that will need a firmware update. See that the attachment flow no longer closes. See that an in progress modal appears on the ODD.

Changelog

Refactor GripperCard and PipetteCard to handle instruments that need subsystem updates
Show dynamic text and color for subsystem update banner if an update is already in progress
Show ODD in progress modal anytime an externally initiated update is in progress

Review requests
Ensure refactor looks sound

Risk assessment
Low

During firmware update, pipette card doesn't show a pipette image?


ValueError: model_config not of type model_pb2.DetectionModel.

When I run train.py I get this error:
(tensorflow1) C:\TensorFlow\models\research\object_detection>python train.py --logtostderr --train_dir=training/ --pipeline_config_path=training/faster_rcnn_inception_v2_pets.config
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\tensorflow1\lib\site-packages\tensorflow\python\platform\app.py:124: main (from main) is deprecated and will be removed in a future version.
Instructions for updating:
Use object_detection/model_main.py.
W1031 16:28:00.182125 12248 tf_logging.py:118] From C:\ProgramData\Anaconda3\envs\tensorflow1\lib\site-packages\tensorflow\python\platform\app.py:124: main (from main) is deprecated and will be removed in a future version.
Instructions for updating:
Use object_detection/model_main.py.
Traceback (most recent call last):
File "train.py", line 184, in
tf.app.run()
File "C:\ProgramData\Anaconda3\envs\tensorflow1\lib\site-packages\tensorflow\python\platform\app.py", line 124, in run
_sys.exit(main(argv))
File "C:\ProgramData\Anaconda3\envs\tensorflow1\lib\site-packages\tensorflow\python\util\deprecation.py", line 136, in new_func
return func(*args, **kwargs)
File "train.py", line 180, in main
graph_hook_fn=graph_rewriter_fn)
File "C:\TensorFlow\models\research\object_detection\legacy\trainer.py", line 248, in train
detection_model = create_model_fn()
File "C:\TensorFlow\models\research\object_detection\builders\model_builder.py", line 113, in build
raise ValueError('model_config not of type model_pb2.DetectionModel.')
ValueError: model_config not of type model_pb2.DetectionModel.

I am experiencing the same issue

Iam also facing the same issue

i have same issue. can someone please help.

please helpppppp

pleaseeeeee

@sneha-v which model you are using for train the data

Faster rcnn.

I also had same issue i have used this it started working........
Faster R-CNN with Inception Resnet v2, Atrous version;
Configured for Open Images V4 Dataset.
Users should configure the fine_tune_checkpoint field in the train config as
well as the label_map_path and input_path fields in the train_input_reader and
eval_input_reader. Search for "PATH_TO_BE_CONFIGURED" to find the fields that
should be configured.
model {
faster_rcnn {
num_classes: 6
image_resizer {
keep_aspect_ratio_resizer {
min_dimension: 600
max_dimension: 1024
}
}
feature_extractor {
type: "faster_rcnn_inception_resnet_v2"
first_stage_features_stride: 8
}
first_stage_anchor_generator {
grid_anchor_generator {
scales: 0.25
scales: 0.5
scales: 1.0
scales: 2.0
aspect_ratios: 0.5
aspect_ratios: 1.0
aspect_ratios: 2.0
height_stride: 8
width_stride: 8
}
}
first_stage_atrous_rate: 2
first_stage_box_predictor_conv_hyperparams {
op: CONV
regularizer {
l2_regularizer {
weight: 0.0
}
}
initializer {
truncated_normal_initializer {
stddev: 0.01
}
}
}
first_stage_nms_score_threshold: 0.0
first_stage_nms_iou_threshold: 0.7
first_stage_max_proposals: 300
first_stage_localization_loss_weight: 2.0
first_stage_objectness_loss_weight: 1.0
initial_crop_size: 17
maxpool_kernel_size: 1
maxpool_stride: 1
second_stage_box_predictor {
mask_rcnn_box_predictor {
use_dropout: false
dropout_keep_probability: 1.0
fc_hyperparams {
op: FC
regularizer {
l2_regularizer {
weight: 0.0
}
}
initializer {
variance_scaling_initializer {
factor: 1.0
uniform: true
mode: FAN_AVG
}
}
}
}
}
second_stage_post_processing {
batch_non_max_suppression {
score_threshold: 0.0
iou_threshold: 0.6
max_detections_per_class: 100
max_total_detections: 100
}
score_converter: SOFTMAX
}
second_stage_localization_loss_weight: 2.0
second_stage_classification_loss_weight: 1.0
}
}
train_config: {
batch_size: 1
optimizer {
momentum_optimizer: {
learning_rate: {
manual_step_learning_rate {
initial_learning_rate: 0.00006
schedule {
step: 6000000
learning_rate: .000006
}
schedule {
step: 7000000
learning_rate: .0000006
}
}
}
momentum_optimizer_value: 0.9
}
use_moving_average: false
}
gradient_clipping_by_norm: 10.0
fine_tune_checkpoint_type: "detection"
from_detection_checkpoint: true
fine_tune_checkpoint: "C:/Automation/models-master/research/object_detection/training/model.ckpt"
num_steps: 10000000
data_augmentation_options {
random_horizontal_flip {
}
}
}
train_input_reader: {
tf_record_input_reader {
input_path: "C:/Automation/models-master/research/object_detection/train.record"
}
label_map_path: "C:/Automation/models-master/research/object_detection/training/labelmap.pbtxt"
}
eval_config: {
metrics_set: "open_images_V2_detection_metrics"
}
eval_input_reader: {
sample_1_of_n_examples: 67
tf_record_input_reader {
input_path: "C:/Automation/models-master/research/object_detection/test.record"
}
label_map_path: "C:/Automation/models-master/research/object_detection/training/labelmap.pbtxt"
shuffle: false
num_readers: 1
}


Update to log4j 2.17.1.

Motivation:
log4j 2.17 is vulnerable to RCE via JDBC Appender when attacker controls configuration ( CVE-2021-44832 ).
Modification:
Bump log4j property to 2.17.1.
Result:
Updates log4j to 2.17.1, which includes a patch for the issue.

More CVEs coming the Log4j way?

@johnou Thanks!


Do not suspend network reads in an HTTP/2 connection.

Armeria currently suspends network reads when more than 128KiB are unconsumed by the stream subscribers. (See: InboundTrafficController)
However, we should not disable network reads at all in an HTTP/2 connection because HTTP/2 has its own traffic control mechanism (WINDOW_UPDATE frame. See: The Flow-Control Window) Actually, arbitrarily disabling network reads can cause an unexpected permanent stall (i.e. dead lock) of an HTTP/2 connection because it can prevent the HTTP/2 implementation from reading a WINDOW_UPDATE frame.
Therefore, we should use InboundTrafficController only for HTTP/1 connections. We may also have some opportunity to gain additional performance thanks to this because 1) we do not need to suspend network reads and 2) we do not need to count the number of pending inbound bytes.

Finished implementing this but seeing some test failures related with streaming.

I realized that this is not going to work because Armeria will keep reading the traffic even if a subscriber consumes the traffic slowly, i.e. data will be buffered in the DecodedHttp(Request|Response) and eventually will overflow without InboundTrafficController.
However, while implementing this, I realized we are using smaller watermarks (128KiB) than initial connection-level window size (1MiB). We could use higher watermark values so that network reads are not suspended too often.


[Enhancement] Verify Image Rules: Ignoring Tlog Documentation Update

Description
The example on the website for disabling tlog verification check is out of date, it should be updated.
error: error validating "image-policy.yaml": error validating data: ValidationError(ClusterPolicy.spec.rules[1].verifyImages[0].attestors[0].entries[0].keys.rekor): unknown field "ignoreTlog" in io.kyverno.v1.ClusterPolicy.spec.rules.verifyImages.attestors.entries.keys.rekor; if you choose to ignore these errors, turn validation off with --validate=false

To disable the check, the field must be completely removed as it said in the API reference
Rekor provides configuration for the Rekor transparency log service. If the value is nil, Rekor is not checked. If an empty object is provided the public instance of Rekor (https://rekor.sigstore.dev/) is used.

Slack discussion
No response

I suspect that it was not the documentation, actually, the API reference is out of date.
After the 1.10 release new verification types are added to the package.
https://github.com/kyverno/kyverno/blob/release-1.10/api/kyverno/v1/image_verification_types.go#L175


[req] Let tasks access pyproject values

Hi
I'd like to have a task that builds a docker image and tags it with the current version. I'm using Poetry, so I could use the version from [tool.poetry]/version.
I know I could use a variable, but I bump the version with poetry version minor and it's easy to forget about other places (no need to have version in init.py any more).
Thanks!

I can think of 3 ideas how to get this done:

We could extend the variable feature with pyproject variables though the variable names would be really long, something like pyproject.tool.poetry.version :grimacing:
Alternatively only expand tool.poetry section so variables names would be pyproject.version but then extending this in the future could be problematic.
Give access to pyproject variables to taskipy variable definition:

[tool.taskipy.variables]
version = "{pyproject.tool.poetry.version}"

[tool.taskipy.tasks]
version = "echo {version}"

I think pyproject.tool.poetry.version is ambiguous or you'd need a special case for poetry to handle tool.poetry as a single and not nested key.
On the other hand, you don't need to use dot as a separator. TOML is a representation of JSON structure - dicts and lists, and Python is not known to use dot to access dict values.
Also, you could replace pyproject with some special character:
[tool.taskipy.tasks]
version = "echo {$[tool.poetry][version]}"

Now, only thing remains to write a parser for it 🫣

I was able to solve my particular problem, thank to taskipy supporting shell expressions 👍
[tool.taskipy.tasks]
build = "docker buildx build -t project_name:$(poetry version | cut -d' ' -f2) ."

Really cool workaround, @rafalkrupinski! Glad it worked out :)
While you were able to solve this particular requirement, please do feel free to open feature requests in the future!


download validates package

Installing a pecl package does not work because of an bug in it. Now I just wanted to download the package and fix it, but even when just downloading a package it validates the archive:
$ pyrus-5.3.5 download pecl/xdebug
Pyrus version 2.0.0a2 SHA-1: CA3946A5C2C7DE7CD6ED89CAC7CCC06606E006D3
Using PEAR installation found at /opt/phpfarm/inst/php-5.3.5/pear
Downloading pecl/xdebug...
Connected...
Mime-type: application/octet-stream

failed! (Invalid package downloaded, package name changed from Xdebug to xdebug)=====================>] 100% (282/282 kb)

Expected result:
I want a downloaded file that I can fix. Pyrus should not verify it.

I can't reproduce this. I tried downloading 1.3.0 - 2.1.3 and all versions were able to download. What version were you trying to download?

Cannot reproduce anymore with 2.0.0a3. Probably fixed since my 2.0.0a2


Add DevC Hanoi to DevCGlobalDirectory

Summary
This pull request add missing developer circles in DevCGlobalDirectory #59
This pull request add DevC Hanoi to DevCGlobalDirectory.md and the markdown file has been following the new design in #57
This pull request follow the #68 and remove all the facebook links and change to github account

Great. Thanks!


Unclear how to run az pipelines release create

I am trying to use the az pipelines release create command. It is unclear what I need to pass to the artifact-metadata-list param.
I get the following error:
The --artifact_meta_data_list argument should consistof space separated "alias=version_id" pairs.

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

ID: de490e1f-04b9-ee52-103c-1fa17f178290
Version Independent ID: 18ee8e02-79d9-c0ac-1066-727ba97999ed
Content: az pipelines release
Content Source: latest/docs-ref-autogen/pipelines/release.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@yonzhan Is this being looked into?

devops service team should take a look

@rguptar we are trying to figure out the same and will share the updates asap.

Hi @rguptar,
az pipelines release create is based on this REST API.
we found the similar issue link (mentioned below) which has the solution to try it out. Please check the issue link.[https://github.com/Azure/azure-devops-cli-extension/issues/872].
Thanks!

@v-soujanya The solution presented there does not work for us. We see the following error:
Cannot retrieve the default version for artifact '' because default version type is 'selectDuringReleaseCreationType'

Hi @rguptar
I am able to create a release successfully in my machine. Please find the image for reference.

In our case, please go through the below information and try to set the artifacts version.
This check is introduced to prevent customers ending up choosing wrong version of artifact in their releases. When default version of an artifact (say A1) is specified as “Specify at the time of release creation” and a release gets auto-triggered due to another artifact (say A2) within the release definition, we had been choosing latest version of A1 for the release. However, this might not be what the user intended. So it will start failing saving of release definition if we find this combination among artifacts in the release definition.
In your case, since you have set an auto-trigger on one of your artifacts, you should change the default version type for the other two artifacts to pick the version that you intend for them to pick, when the release is auto-triggered.

@v-soujanya
Given the following URLs,

build pipeline: https://dev.azure.com/ORG/PROJECT/_build?definitionId=BUILD_DEFINITION_ID
specific build: https://dev.azure.com/ORG/PROJECT/_build/results?buildId=BUILD_ID&view=results
release pipeline: https://dev.azure.com/ORG/PROJECT/_release?_a=releases&view=mine&definitionId=RELEASE_DEFINITION_ID
edit release pipeline: https://dev.azure.com/ORG/PROJECT/_releaseDefinition?definitionId=RELEASE_DEFINITION_ID&_a=environments-editor-preview and then I click on the artifact to find the property called "Source alias" with value SOURCE_ALIAS

With the following command, I am able to create the release for some projects but not all.
az pipelines release create --artifact-metadata-list "$SOURCE_ALIAS=$BUILD_ID" --definition-id=$RELEASE_DEFINITION_ID --organization https://dev.azure.com/$ORG --project $PROJECT
When it fails, I get the following error:
Page not found.  Operation returned a 404 status code.

Hi @rguptar,
For Some of the projects you are able to create a release then I think issue is not with cli command. Please guide me how to repro the issue in our machine for those projects in which you are facing issue.
Thanks!

@rguptar Yes Please! Please create a support ticket.


Fix link to config.json

The link to config.json is broken since the folder fonts does not exist. This PR addresses the issue.

Thanks a lot.


MB-14450-Display-times-related-to evaluation-travel

Jira ticket for this change
Summary
The following fields: time departed for evaluation, time evaluation started, and time evaluation ended are displayed on the  evaluation report, the report PDF, or its preview for submission given a QAE/CSR user logged into MilMove.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Verification Steps for Author
These are to be checked by the author.

[ ] Login as a QAE/CSR user, and navigate to the Quality Assurance tab.
[ ] Create a shipment report and select Evaluation Type: Physical, Evaluation Location: Origin
[ ] Press the Review and Submit Button to verify fields (time departed for evaluation, time evaluation started, and time evaluation ended) are added to preview for submission.
[ ] Submit report, then select view report and verify fields (time departed for evaluation, time evaluation started, and time evaluation ended) are in evaluation report.
[ ] Download the report, then verify fields (time departed for evaluation, time evaluation started, and time evaluation ended) are in report PDF
[ ] Please redo steps 1-5 for a shipment report with selected options Evaluation Type: Physical, Evaluation Location: Destination

Verification Steps for Reviewers
These are to be checked by a reviewer.
https://user-images.githubusercontent.com/111006369/205708637-6f37c74d-a535-48bc-8b49-0a5ef651ec78.mov
Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Screenshots

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-14450

  Generated by :no_entry_sign: dangerJS against 8ff022cf7afda965894cf732531c0948b842c856


[macOS] Attempt to terminate process normally before using forceTerminate.

Fixes https://github.com/godotengine/godot/issues/95167
We probably should add an extra force flag to kill method, and change the editor logic. Killing running project processes unconditionally seems like a bad idea, it probably should try killing it normally and only kill on a timer if a child process is still running.

Thanks!


Metrics Advisor Content Validation Issue for github.io website

Section links:

https://azuresdkdocs.blob.core.windows.net/$web/python/azure-ai-metricsadvisor/1.0.0/genindex.html
https://azuresdkdocs.blob.core.windows.net/$web/python/azure-ai-metricsadvisor/1.0.0/azure.ai.metricsadvisor.models.html#azure.ai.metricsadvisor.models.AnomalyIncidentStatus --
https://azuresdkdocs.blob.core.windows.net/$web/python/azure-ai-metricsadvisor/1.0.0/azure.ai.metricsadvisor.models.html#azure.ai.metricsadvisor.models.DataFeedStatus

Reason:
Format validation issue: Text exceeds page size.

Section links:

https://azuresdkdocs.blob.core.windows.net/$web/python/azure-ai-metricsadvisor/1.0.0/azure.ai.metricsadvisor.models.html#azure.ai.metricsadvisor.models.AnomalyAlert

Reason:
Typing annotation issue: Please unify the format, update to **kwargs:Any or **kwargs.(Note: Multiple occurrences)
@rohit-ganguly, @lmazuel, @achandmsft. @mayurid, @iscai-msft for notification.

Close this issue as we'll put it into a summarzied one.


High level apis typos fix

Thanks for the fixes.  I actually ran the codespell spell checker and ChaptGPT and they all missed these typos.   I am probably Dyslexic, though undiagnosed, so unfortunately don't spot lots of typos when writing or even reviewing what I've written.  Thankfully compiler catch many typos so C++ programming isn't so bad, but writing texts like this I do need help from the community to make up for my lack of skills in this area, so I appreciate you taking the time to correct the errors.
I've merged the commits one by one rather than as single PR.  I think a some of the changes are debatable as they take us into preferences rather than clear cut typo fixes.  I have accepted them all for now, but I'll need to reflect on which conventions are preferences are worth following.

Thank you for the kind words. I've been reading the docs for the vsg and it really good. My idea is to help as much as I can, and have some real contribution. Until I'm more familiar with the code-base and the docs, my contributions will probably be nitpicking but I hope to have some real contribution in the future.
Your work is phenomenal so keep up the good work and I will try to help as much as I can.


EmbeddedScheduledCallbackTests: violates EmbeddedEventLoop requirements

EmbeddedScheduledCallbackTests uses EmbeddedEventLoop as if it were thread safe, that's not right.

Resolved in #2951.


Update pandas dependency

Can we update the Pandas dependency to a newer version such as 1.5.3? Installing this library via pip, it wants a much older version.

It's updated now, thank you

Fantastic, thanks for the speedy response!


[WIP] Feature/api factory cache

This pr adds cache middleware to the project factory. Which allows partial processing

This item looks good!

This seems to be an excellent starting point; I do wonder whether we should have the middleware invoke a command and then have the command perform the caching. As this is a business logic affair it would seem logical but I am interested in your opinion.

I updated this pr, major changes are done by removing the middleware and strategies dependencies from the Factory. This was posible due the introduction of File interface.

Only very minor style things were found during review and one possible clash in the caching key which should be addressed in a subsequent PR if this is indeed a problem.
Aside from that :+1:


Fix for issue #33 https://github.com/glucaci/mongodb-quartz-net/issues/33

PR contains fix for issue https://github.com/glucaci/mongodb-quartz-net/issues/33

@glucaci Hi, please take a look at the PR. Thanks in advance!

Thanks, I changed the base branch to v4, this will be for the next major release.
I will trigger a preview release which will include this also.


Refactor the CodeBooker logo

The CodeBooker Logo I intend to use is at src/assets/codebooker.png
It can be used in the app.
I'm no good with any of the image programs like GIMP
It has a watermarked background that says LOGO.  I don't know if the background could be extracted from the logo .png?
I am sure the logo could look a lot nicer and more useable.
The icons need to be 192x192 and 512x512.  The browser favicon nedds to be 16x16 and 32x32.  They could be separate images.

@gbowne1  I've assigned myself this issue.

@gbowne1 I have a few questions to ask concerning this issue:

You suggested using 192 x 192 and 512 x 512 for the  logo size. Which size should we use from either of the two ?
P.S: I have tested both sizes and have found size 192 x 192 the most fitting.
You also spoke about using favicon but the app dosent have a favicon icon. what do we do about that ?

@BlackBond06
From my limited research on compatibility.
Different browsers and devices require different sizes of these sorts of graphics, however these sizes stood out to me to be the most required.
It also suggested the use of .png vs .svg
Google Lighthouse & Google Pagespeed also suggested it wanted a specific size for SEO, etc.
The icons and images will have to be also added to the manifest.json file as well. It is still the default.
192x192 does work with most browsers.  But we should also include 512x512 just to be sure.
As for the .ico file, I am not sure.

@BlackBond06
From my limited research on compatibility.
Different browsers and devices require different sizes of these sorts of graphics, however these sizes stood out to me to be the most required.
It also suggested the use of .png vs .svg
Google Lighthouse & Google Pagespeed also suggested it wanted a specific size for SEO, etc.
The icons and images will have to be also added to the manifest.json file as well. It is still the default.
192x192 does work with most browsers. But we should also include 512x512 just to be sure.
As for the .ico file, I am not sure.

Yea. I totally agree. But we have to agree on one of the two sizes to use tho. I noticed that size 512 x 512 doesn't fit with the current height of the apps navbar.

Yeah, the bigger one may have to be used in a media query, but that would take some more browser / device testing. For now we can use the 192x192 one.
I forget how to do icon sizing in a media query.  Might go ask perplexity.ai about this.
I think some browsers and devices may automatically resize. I dont have great information on this.

/* Define a media query for smaller screens */

@media screen and (max-width: 600px) { /* Adjust the max-width of the image for smaller screens */

img {

     max-width: 100%; /* Ensure the image is responsive */

     height: auto; /* Allow the image to scale proportionally */
}
}

Yea, for really large devices we could use the 512 x 512, otherwise the 192 x 192 fits well with the current height of the app's navbar. Now, what do we do about the favicon for the app ? can I go ahead and create something and the share with you to see if you like it ?

Sure, you can always share ideas etc with myself, @LOGESH-B and collaborators.


Missing mssql dependency

There must be some dependency missing as if I try to run django as described in the readme (python manage.py runserver), I get this error:
django.core.exceptions.ImproperlyConfigured: 'mssql' isn't an available database backend or couldn't be imported. Check the above exception. To use one of the built-in backends, use 'django.db.backends.XXX', where XXX is one of:
    'mysql', 'oracle', 'postgresql', 'sqlite3'

I guess something like pyodbc and mssql-django must be installed also, right? Make sure to include them into the requirements.txt file and call them out in the readme too.

Yes, you are absolutely right, we need to have pyodbc and mssql-django installed too.  Did you follow these steps in README.md? Will include them in requirement.txt too.
Configure Azure SQL connectivity with Django
Dependencies

pyodbc 3.0 or newer

Installation

Install pyodbc:
pip install pyodbc

Install mssql-django:
pip install mssql-django

Configuration
Configure the Database ConnectionString in the settings.py file used by your Django application or project:
DATABASES = {
    'default': {
        'ENGINE': 'mssql',
        'NAME': 'Database_name',
        'HOST': "xyz.database.windows.net",
        'PORT': '1433',
        'USER': 'User_name',
        'PASSWORD': 'db_pwd',
        'OPTIONS': {
                'driver': 'ODBC Driver 17 for SQL Server',
            },
    }
}

To connect Azure SQL DB using MSI (Managed Service Identity), you can have settings as below:
DATABASES = {
    'default': {
         'ENGINE': 'mssql',
         'HOST': 'xyz.database.windows.net',
         'NAME': 'Database_name',
         'PORT': '',
         'Trusted_Connection': 'no',
         'OPTIONS': {
             'driver': 'ODBC Driver 17 for SQL Server',
             'extra_params': "Authentication=ActiveDirectoryMsi;Encrypt=yes;TrustServerCertificate=no" }
     }
}

[!WARNING]
mssql-django doesn't support using time zones so the recommendation is to ensure the USE_TZ option is set to False.

DATABASES = {
...
}

# set this to False if the backend does not support using time zones
USE_TZ = False

Yeah, I did not see that part of the readme. ss soon as I tried to run the clone repo, I got those errors and stopped reading further. I think that now that we have reorganized the readme is clear enough.


nthreads not working in R on linux

Hi, seems like nthreads is not working in the R package for parallel processing on a linux server -- the CPU is always capped at 100%:
sessioninfo():
R version 4.0.0 (2020-04-24)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 16.04.6 LTS
Matrix products: default
BLAS:   /usr/lib/atlas-base/atlas/libblas.so.3.0
LAPACK: /usr/lib/atlas-base/atlas/liblapack.so.3.0
locale:
[1] LC_CTYPE=C.UTF-8       LC_NUMERIC=C           LC_TIME=C.UTF-8        LC_COLLATE=C.UTF-8     LC_MONETARY=C.UTF-8
[6] LC_MESSAGES=C.UTF-8    LC_PAPER=C.UTF-8       LC_NAME=C              LC_ADDRESS=C           LC_TELEPHONE=C
[11] LC_MEASUREMENT=C.UTF-8 LC_IDENTIFICATION=C
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] xgboost_<IP_ADDRESS>
script:
xgb.fit=xgb.train(
params=params,
data=xgb.train,
nrounds=10000,
nthreads=1,
early_stopping_rounds=10,
watchlist=list(val1=xgb.train,val2=xgb.test),
verbose=0
)
Any thoughts why this happened? I saw a bunch of old issues similar to this opened and closed but none of them actually got quite a good answer.

I think it's because of nthreads=1 in your script. Try changing this value to a bigger number.

There's no nthreads, instead it's nthread.
Put it in params instead of an argument for train.  In 1.1 release, there's parameter validation for R package.  @hcho3 Is there any difficulty on submitting it?

@trivialfis: @hetong007 is working on it.

Got it, thanks.  Please let me know if there's anything I can help.