TxGroup::assign_group_id should take a &mut [Transaction] or &mut Vec<_> instead of a Vec<&mut Transaction>.

Is your feature request related to a problem? Please describe.
As a general rule, so far as I can tell, interfaces should use either an iterator or a slice unless you're taking ownership of the data held within. In this case, since assign_group_id only modifies the data provided, I believe this is a better approach.
Describe the solution you'd like
Replace the parameter type of assign_group_id's  (Vec<&mut Transaction>) with &mut [Transaction]
Describe alternatives you've considered
It's possible that this is still kind of restrictive. A more obtuse, but more flexible approach is to take in an impl IntoIterator<Item = &mut Transaction>. Both Vec<&mut Transaction> and &mut [Transaction] (and &[&mut Transaction]) should implement this or have conversion methods readily available.
Again, this is a bit obtuse, though.
Additional context
To recap, I'd like:
pub fn assign_group_id(txns: &mut [Transaction]) -> Result<(), TransactionError> {
    let gid = TxGroup::compute_group_id(txns)?;
    for tx in txns {
        tx.assign_group_id(gid);
    }
    Ok(())
}

(I haven't checked if tweaking compute_group_id would affect any other functions.)
OR
pub fn assign_group_id<'a>(txns: impl IntoIterator<Item = &'a mut [Transaction]>) -> Result<(), TransactionError> {
    let txns: Vec<_> = txns.into_iter().collect();
    let gid = TxGroup::compute_group_id(txns.as_slice())?;
    for tx in txns {
        tx.assign_group_id(gid);
    }
    Ok(())
}

OR
pub fn assign_group_id<'a>(txns: impl IntoIterator<IntoIter = impl Iterator<Item = &'a mut Transaction> + Clone>) -> Result<(), TransactionError> {
    let txn_iter = txns.into_iter();
    let gid = TxGroup::compute_group_id(txn_iter.clone())?;
    for tx in txn_iter {
        tx.assign_group_id(gid);
    }
    Ok(())
}

(I... don't think I can recommend this last one for a general purpose interface. The Clone potentially hides a lot of work. Maybe a Copy might be better? You can't just pass a reference since the iterator will get consumed.)
EDIT: Changed the IntoIterator<Item = &'a mut [Transaction]> into IntoIterator<Item = &'a mut Transaction>.

We certainly should abandon the  'Vec<&mut Transaction>` parameter. I gave a try to your slice suggestion and I find it a good solution. The Iterator solution maybe would make the API less readable? Do you have any example of this approach in the standard library or popular libraries?

The reason that option 1) is not being used IIRC is that you transfer the transactions ownership to the vector, so to continue using them after assigning the id, you've to get them back from it:
let txs = &mut vec![t1, t2];
TxGroup::assign_group_id(txs)?;
let signed_t1 = account.sign_transaction(&txs[0])?;
let signed_t2 = account.sign_transaction(&txs[1])?;

which is slightly error prone and not so nice in general.
Also, having to declare the transactions themselves as mutable, while a bit tedious, signalizes intent better IMO.
The owned vector parameter, while not idiomatic can be seen as a throw away vehicle to mutate the transactions.
Might be missing something.
The second variant allows to keep the caller side similar to what we've now, but it seems to do more work than assign_group_id(txns: Vec<&mut Transaction>), which is likely negligible due to the number of transactions normally passed / the max group size, but makes me question it as a replacement (combined with the more complicated signature).
The purpose of the third variant isn't quite clear.

Would something like this be a reasonable compromise?
pub fn assign_group_id(txns: &mut [&mut Transaction]) -> Result<(), TransactionError> {
        let gid = TxGroup::compute_group_id(txns)?;
        for tx in txns {
            tx.assign_group_id(gid);
        }
        Ok(())
    }

only difference in the atomic_swap example would be a mutable slice instead of a vec! macro
    TxGroup::assign_group_id(&mut [t1, t2])?;

    let signed_t1 = account1.sign_transaction(&t1)?;
    let signed_t2 = account2.sign_transaction(&t2)?;

which does not move t1 and t2.
Notice that something like
    let mut txs = vec![t1, t2];
    TxGroup::assign_group_id(txs.as_mut_slice())?;

    let signed_t1 = account1.sign_transaction(&txs[0])?;
    let signed_t2 = account2.sign_transaction(&txs[1])?;

would work and the user is explicitly deciding to move the transactions within a Vec.

The Iterator solution maybe would make the API less readable? Do you have any example of this approach in the standard library or popular libraries?

It's mostly found in libraries built for iterator manipulation. One example is itertools::join. I've also seen it in nalgebra::from_iterator. It's definitely not common, but it's a pattern I use extensively throughout my code. Perhaps we could create variants for ease of use? (Copying how the std would create specialized functions for certain common types.)
fn assign_group_id<'a>(txns: Vec<&'a mut Transaction>) {
    assign_group_id_iter(txns)
}

fn assign_group_id_iter<'a>(iterator: impl IntoIterator<Item = &'a mut Transaction> + Clone) {
    // ...
}

It's rather similar to an API taking impl AsRef<str>.

The purpose of the third variant isn't quite clear.

Uh, just realized, I meant IntoIterator<IntoIter = impl Iterator<Transaction<Item = &mut Transaction> + Clone>> not IntoIterator<Item = &mut [Transaction]> + Clone. Does that change things?
The third variant exists to avoid that additional work mentioned about the second variant. As a general rule, Iterators over references are cheap to clone. To make it more restricted, the parameter would IntoIterator<Item = &mut Transaction> + Copy
I know that variant 2 & 3 are obtuse, which is why I'd suggested the variant 1 instead of 2 & 3.
Another note is that the signature would accept all of the following:
TxGroup::assign_group_id([&mut t1, &mut t2]);
let txn: Vec<Transaction> = vec![];
TxGroup::assign_group_id(txns); // Probably wrong. Also disallowed by `Copy` variant
TxGroup::assign_group_id(&mut txns);
TxGroup::assign_group_id(txns.iter_mut()); // Disallowed by `Copy` variant
TxGroup::assign_group_id(txns.as_mut_slice());
let txn: Vec<&mut Transaction> = vec![];
TxGroup::assign_group_id(txns); // Disallowed by `Copy` variant
TxGroup::assign_group_id(&txns);
TxGroup::assign_group_id(txns.iter()); // Disallowed by `Copy` variant
TxGroup::assign_group_id(txns.as_mut_slice());

Would something like this be a reasonable compromise?

I like it!

In my case, I'm orchestrating transactions of many NFT/limited count tokens between multiple parties, so my use case ends up looking a little like this:
fn attempt_transfer(transfer: Transfer) -> {
    let mut txns = transfer.construct_transactions();
    // What I do now
    TxGroup::assign_group_id(txns.iter_mut().collect());
    // This would be nice
    TxGroup::assign_group_id(&mut txns);
    // This was mentioned, and I like it.
    TxGroup::assign_group_id(txns.as_mut_slice());

    // Does a bit of look up & stuff to figure out who is supposed to sign what.
    // Also takes ownership of the transaction, and pumps out the signed versions for broadcast.
    let signed_txns = transfer.sign_transactions(txns);
}

And the other use case I was imagining
    // But maybe it's a HashMap<_, Vec<Transaction>> instead, so it'd be nice to
    let txn_refs = txns.values_mut().flat_map(|txns| txns.iter_mut());
    TxGroup::assign_group_id(txn_refs);
    // But as is, I need to collect.
    let txn_refs = txns.values_mut().flat_map(|txns| txns.iter_mut()).collect();
    TxGroup::assign_group_id(txn_refs);

In the long run, the compiler probably knows about how to handle it, but I'd like to avoid potentially costly collects.
I'm simulating guilds of players pooling on chain resources to do things at the moment, so there are a lot of transactions going on. Maybe I should look at a different way to do things?

+1 for &mut [&mut Transaction]. It seems the "correct version" of what we've now and provide enough flexibility.

I'm simulating guilds of players pooling on chain resources to do things at the moment, so there are a lot of transactions going on. Maybe I should look at a different way to do things?

This goes a bit beyond the scope of this issue / possibly the SDK. I'd recommend asking first in the Algorand forum or Discord.

Okay. I'll make that change, then, since it looks like we're all agreed.

This goes a bit beyond the scope of this issue...

Ah, yeah, I wasn't really asking for help with it, more that I'm trying to establish why I'd like something more generic.

Looking forward to the PR on this, just got stuck on this exact issue.

I'll do it in a bit (just leaving the note to prevent duplicated work possibly). Not sure when @AlterionX will have time.

@AlterionX @manuelmauro feel free to double check that it's what we agreed.
@dallin-shogun you can reference the PR branch for now.

Looks good to me.

Thanks everyone for the collaboration!

Thanks everyone!


fix crontab link and clarify use of cron for renewal

Should fix documentation problem I noted in #18

👍 Thanks for the fix!


Add Laravel 11 support

@brandonb927 @shaun-mac this is ready to go and has been tested with my Laravel 11 branch.
https://github.com/echosec/phoenix/pull/13372

I would love to see us replace this library with https://github.com/intlify/vue-i18n in the near-future.

I would love to see us replace this library with https://github.com/intlify/vue-i18n in the near-future.

I'll keep this in mind and I'll try to squeeze this in as a maintenance work in the near future. We're already using vue-i18n, but still consuming the strings from the generated files.


Stop video recording called in the end test (robot listener) not working

I got one issue when import ScreenCapLibrary from my robot listener py file.
Video get start to record from start_test() method and want to stop form end_test()
I just checked the output video files, just first video file can play. Another file seems didn't
stop correctly.  File size remain 555 bytes and can't play correctly.
Could you help check if this caused by Close Video Recording keyword is taking time to respond. Is there any way if I want to run from listener python file and I can know start/end video is actually finished ?  Thank you so much

Try ScreenCapLibrary 1.3.1 alpha 1 to see if the problem still occurs. You can install it with:
pip install --pre --upgrade robotframework-screencaplibrary.

Cool, It works now after upgrade to 1.3.1 alpha 1. Many thanks for your answer.


Any support for updating existing libraries?

There are maybe more than 100 libraries (just by the fork count of this repo) that have been built based on the different versions of ProjectScaffold. Which was great, it has been an excellent template. But then came .NET Core and .NET Standard and new project files, FAKE 5 and so-on. Gradually some parts are not working from the old templates: SourceLinks, document generation, FSharpTargetsPath is not detected correctly, etc.
Many of the maintainers are not keen to do these big changes to their libraries, maybe their interest has shifted to other projects. This is too-much of the burden for new people who would like to create some simple PR-fixes, but they cannot get the whole projects to compile anymore.
I think this is dangerous for the whole community and the F# ecosystem. Could we have here in ProjectScaffold a guidance, a WIKI, or something, to help the migration: What is the current best practices of .NET projects and how the different old Scaffold templates (or F# code-blocks) should be migrated to the latest best practices?

I absolutely agree. Just the lack of response to your issue is testimony to the problem. A continuous and consistent lack of support and upgrading of a lot of core F# libraries. The same applies to F# tooling.  I completely understand that this cannot be thrown at any ones feet and this is no personal attack to library maintainers. This is an observation and could be a major reason why F# hasn't been gaining more popularity. If F# is to survive as a major programming language, this general issue has to be fixed.
At the same time I realize that F# is free of charge, a great language and that I am not able to contribute in any meaningful way (being not even a real programmer).


[Feature Request] Support encode string for decimal

The discussion is from JS decimal support,  for decimal type in typespec not every language has the native support or has the ability to handle any precision either calculation or serialization/deserialization.
So if the service team strongly cares about the precision of decimal it should use @encode to encode it as a string.
Here is the draft proposal:
model Widget {
  @encode("decimalString") // ???
  value: decimal;
}

/cc @bterlson @timotheeguerin

Can you move this issue to the public repo as this is not really azure specific

closed azure one and open it in microsoft.


Add conventional-changelog-cli and…

What type of PR is this? (Check all applicable)

[ ] 🚀 Feature
[ ] 🐛 Bug Fix
[ ] 📚 Documentation
[ ] 💻 Blog post
[ ] 🎨 Styles
[ ] 🛠 Refactoring
[ ] 🕵♀ Code Review
[ ] 🚑 Hotfix
[ ] 🧪 Test
[ ] 🌐 Localization/Internationalization
[ ] ⚙  Configuration and Environment
[ ] 🚫 Deprecation or Removal
[ ] 📦 Release
[ ] ⏪ Revert

Description
Please describe the changes in this pull request:
Replace this text with a clear and concise description of what you've done.
Related Issues
Please link related issues by removing lines that are not applicable.

Fixes #123
Addresses #456

Checklist
Before submitting this pull request, please make sure you have done the following:
Code Quality

[ ] Reviewed your code for any potential issues.
[ ] Checked for coding style and formatting.

Updated documentation?

[ ] ✅ Yes
[ ] 🙅 No, because there is nothing to update
[ ] 🙋♂ No, I need help

Added tests for your changes?

[ ] ✅ Yes
[ ] 🙅 No, because they are not needed
[ ] 🙋♂ No, I need help

UI Changes

[ ] Added screenshots or recordings to demonstrate UI-related changes (if applicable).

Additional Information
Replace this text with any additional information.

@ehdlg Look documentation and make it separate the types of commits automatically.
There are examples in the documentation.


elementum 0.1.58 refuse to load after viewing home screen. Gives the "elementum is starting give it some time"

Hi! First of all I want to give my thanks for still kind of maintaining elementum. It is the best kodi addon. And the only I use tbth. So, issue is pretty much covered in the title. Kodi starts normal and elementum loads. I can view every section normally. But as soon as I "back" all the way out to home screen and then tries to enter elementum again it gives the "elementum is starting up give it time". Waiting does not help. Restarting Kodi works. So, it is not a big deal in that sense.
Kodi is Krypton 17.1 on Linux dist. Devuan 2.1. That thumbnail it tries to load in the log does not exist.
kodi.log

Hi everyone
same issue here
Anyone can help?
kodi.log

@ThurahT you still have this issue? You can try to re-install plugin from zip file.

Closing without getting response.

Nope, you fixed it some releases back! Thanks for all your hard work!

Nope, you fixed it some releases back! Thanks for all your hard work!


SwaggerSpringMvcPlugin returning JSON with "/default/" in the path

I'm using the example CustomJavaPluginConfig that uses a SwaggerSpringMvcPlugin in my SpringMVC framework, and have annotated my controller as such:
@Api(value = "orghierarchies", description="Org Hierarchies Resource")
@Controller
@RequestMapping(value = "/v1/orgHierarchies")
public class OrgHierarchyControllerV1 implements Serializable
{
@RequestMapping(value = "/generate", method = RequestMethod.POST, produces = MediaType.APPLICATION_JSON_VALUE)
@ApiOperation(value = "Generates an Org Hierarchy", notes = "notes for generating an org hierarchy", response = String.class)
public @ResponseBody
StatusResponse generateOrgHierarchy(
@ApiParam(required = true, name = "orgHerarchy", value = "The Org Hierarchy Object!")
@RequestBody OrgHierarchy orgHierarchy)
{
...
}
but the resultant JSON from /api-docs is:
{
"apiVersion": "1.0",
"swaggerVersion": "1.2",
"apis": [
{
"path": "/default/orghierarchies",
"description": "orghierarchies"
}
]
}
notice the /default/ in the path. Why is '/default' being added to my path, and furthermore, why isn't my description being used from my @Api annotation? Also why am I not getting ANY other relevant documentation for my @ApiOperations and @ApiParams?

Hi,
I am new to swagger.
I have the same issue.
I am accessing index.html , with entry http://localhost:8080/rest/api-docs.json
I do get the response as :i tried this call seprately in browser :http://localhost:8080/rest/api-docs.json
{"apiVersion":"1.0","apis":[{"description":"Location Controller","path":"/default/location-controller","position":0}],"authorizations":{},"info":{"contact":"Contact Email","description":"Api Description","license":"Licence Type","licenseUrl":"License URL","termsOfServiceUrl":"Api terms of service","title":"default Title"},"swaggerVersion":"1.2"}
The problem is when i hit the index.html its saying this
Unable to read api 'location-controller' from path http://localhost:8080/tomcat-mysql-2.0.0/api-docs.json/default/location-controller (server returned undefined)
Is the problem with the URL that is formed ?
/api-docs.json/default/ should be api-docs/default ?
I am using swagger 1.2 , i tried with the latest 2.0 UI and 1.2 UI.
Also i noticed that in my UI, the basePath is defined as
base url: http://localhost:8080/tomcat-mysql-2.0.0/api-docs.json , api version: 1.0
Thanks in Advance
Regards
Joshita

@joshita14 You dont need the .json extension.

It's not working if I don't append. Json.
Also in index.html, the discovery url should have ApApi-docs.json as entry?

Can you share your project or at the very least share your configuration. Its hard to help without knowing any details about how you've set things up. See here for issue submission guidelines. Have you looked at the demo project?

And I want to use swagger 2.0 version. Is there support for spring mvc?  Although I am using the latest dependency,  why it is showing 1.2 instead 2.0?

Here is the project configuration
I added this
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.ComponentScan;
import org.springframework.context.annotation.Configuration;
import org.springframework.web.servlet.config.annotation.EnableWebMvc;
import com.mangofactory.swagger.configuration.SpringSwaggerConfig;
import com.mangofactory.swagger.models.dto.ApiInfo;
import com.mangofactory.swagger.plugin.EnableSwagger;
import com.mangofactory.swagger.plugin.SwaggerSpringMvcPlugin;
@Configuration
@EnableWebMvc
@EnableSwagger
@ComponentScan("com.package.repositories")
public class CustomJavaPluginConfig {
private SpringSwaggerConfig springSwaggerConfig;

@Autowired
public void setSpringSwaggerConfig(SpringSwaggerConfig springSwaggerConfig) {
	this.springSwaggerConfig = springSwaggerConfig;
}

@Bean
// Don't forget the @Bean annotation
public SwaggerSpringMvcPlugin customImplementation() {
	return new SwaggerSpringMvcPlugin(this.springSwaggerConfig).apiInfo(
			apiInfo()).includePatterns(".*api.*");
}

private ApiInfo apiInfo() {
	ApiInfo apiInfo = new ApiInfo("Application",
			"Api Docs For application", "My Apps API terms of service",
			"My Apps API Contact Email", "My Apps API Licence Type",
			"My Apps API License URL");
	return apiInfo;
}

}

Swagger.props
documentation.services.basePath=http://localhost:8080/rest
documentation.services.version=1.0

pom.xml

com.mangofactory
swagger-springmvc
0.9.5

<spring.version>3.2.3.RELEASE</spring.version>

code snippet for Index.html

@joshita14 I dont think you need to create the property file. That isnt even used. Why dont you create a new issue rather than tagging on to an old issue that may not be your problem?
Also what does your web.xml... or your corresponding java config look like? You shouldnt need to add the .json extension


Update opentelemetry-add-modify.md

Using the kind: SpanKind.SERVER is not required anymore after the version 1.8.0.
Related PR: https://github.com/Azure/azure-sdk-for-js/pull/31417

@AaronMaxwell
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Implement handling of missing/invalid Bearer Token authentication

This PR adds support for handling missing/invalid Bearer Token authentication by providing AuthenticationEntryPoint that uses information provided by BearerTokenError to set HTTP response status code and populate WWW-Authenticate HTTP header.
This resolves spring-projects/spring-security#5125.

I've updated the PR last week but failed to drop a note, test naming was addressed as well as use of AssertJ instead of ExpectedException.

I've updated the PR with change to remove static helper method and simply build WWW-Authenticate inside BearerTokenAuthenticationEntryPoint#commence. For other points feel free to address them on merge as you see fit.

Thank you @vpavic


Inconsistent field names when numbers included

From https://github.com/grpc-ecosystem/protoc-gen-grpc-gateway-ts/issues/52
When the message filed contains number it's name in the generated typescript is not consistent with the JSON Mapping done by the grpc-gateway runtime. See example below.
Example input message:
syntax = "proto3";

package example.v1;

message Example { string k8s_field = 1; }

Result:
/* eslint-disable */
// @ts-nocheck
/*
* This file is a generated Typescript file for GRPC Gateway, DO NOT MODIFY
*/
export type Example = {
  k8SField?: string
}

Expected result
export type Example = {
  k8sField?: string
}

Generator uses fieldName function that effectively uses strcase.ToLoweCamel that also uppers the case of characters after any number.

A fix was made in this fork: https://github.com/cadencerpm/protoc-gen-grpc-gateway-ts/pull/1


Error message too heavy when table type is not iceberg

when trying to load a table whose type is not iceberg, the error message is too heavy with all meta info printed. Only db name and table name is needed here. The current error message is like:
java.lang.IllegalArgumentException: Invalid tableName, not Iceberg: tpcds_parquet_10.Table(tableName:web_sales, dbName:tpcds_parquet_10, owner:root, createTime:1552464655, lastAccessTime:0, retention:0, sd:StorageDescriptor(cols:[FieldSchema(name:ws_sold_time_sk, type:int, comment:null), FieldSchema(name:ws_ship_date_sk, type:int, comment:null), FieldSchema(name:ws_item_sk, type:int, comment:null), FieldSchema(name:ws_bill_customer_sk, type:int, comment:null), FieldSchema(name:ws_bill_cdemo_sk, type:int, comment:null), FieldSchema(name:ws_bill_hdemo_sk, type:int, comment:null), FieldSchema(name:ws_bill_addr_sk, type:int, comment:null), FieldSchema(name:ws_ship_customer_sk, type:int, comment:null), FieldSchema(name:ws_ship_cdemo_sk, type:int, comment:null), FieldSchema(name:ws_ship_hdemo_sk, type:int, comment:null), FieldSchema(name:ws_ship_addr_sk, type:int, comment:null), FieldSchema(name:ws_web_page_sk, type:int, comment:null), FieldSchema(name:ws_web_site_sk, type:int, comment:null), FieldSchema(name:ws_ship_mode_sk, type:int, comment:null), FieldSchema(name:ws_warehouse_sk, type:int, comment:null), FieldSchema(name:ws_promo_sk, type:int, comment:null), FieldSchema(name:ws_order_number, type:bigint, comment:null), FieldSchema(name:ws_quantity, type:int, comment:null), FieldSchema(name:ws_wholesale_cost, type:decimal(7,2), comment:null), FieldSchema(name:ws_list_price, type:decimal(7,2), comment:null), FieldSchema(name:ws_sales_price, type:decimal(7,2), comment:null), FieldSchema(name:ws_ext_discount_amt, type:decimal(7,2), comment:null), FieldSchema(name:ws_ext_sales_price, type:decimal(7,2), comment:null), FieldSchema(name:ws_ext_wholesale_cost, type:decimal(7,2), comment:null), FieldSchema(name:ws_ext_list_price, type:decimal(7,2), comment:null), FieldSchema(name:ws_ext_tax, type:decimal(7,2), comment:null), FieldSchema(name:ws_coupon_amt, type:decimal(7,2), comment:null), FieldSchema(name:ws_ext_ship_cost, type:decimal(7,2), comment:null), FieldSchema(name:ws_net_paid, type:decimal(7,2), comment:null), FieldSchema(name:ws_net_paid_inc_tax, type:decimal(7,2), comment:null), FieldSchema(name:ws_net_paid_inc_ship, type:decimal(7,2), comment:null), FieldSchema(name:ws_net_paid_inc_ship_tax, type:decimal(7,2), comment:null), FieldSchema(name:ws_net_profit, type:decimal(7,2), comment:null)], location:hdfs://<IP_ADDRESS>:9000/tpcds/parquet/10/web_sales, inputFormat:org.apache.hadoop.hive.ql.io.parquet.MapredParquetInputFormat, outputFormat:org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat, compressed:false, numBuckets:-1, serdeInfo:SerDeInfo(name:null, serializationLib:org.apache.hadoop.hive.ql.io.parquet.serde.ParquetHiveSerDe, parameters:{path=hdfs://<IP_ADDRESS>:9000/tpcds/parquet/10/web_sales, serialization.format=1}), bucketCols:[], sortCols:[], parameters:{}, skewedInfo:SkewedInfo(skewedColNames:[], skewedColValues:[], skewedColValueLocationMaps:{}), storedAsSubDirectories:false), partitionKeys:[FieldSchema(name:ws_sold_date_sk, type:int, comment:null)], parameters:{EXTERNAL=TRUE, spark.sql.sources.schema.numPartCols=1, spark.sql.sources.schema.part.0={"type":"struct","fields":[{"name":"ws_sold_time_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_ship_date_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_item_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_bill_customer_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_bill_cdemo_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_bill_hdemo_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_bill_addr_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_ship_customer_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_ship_cdemo_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_ship_hdemo_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_ship_addr_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_web_page_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_web_site_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_ship_mode_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_warehouse_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_promo_sk","type":"integer","nullable":true,"metadata":{}},{"name":"ws_order_number","type":"long","nullable":true,"metadata":{}},{"name":"ws_quantity","type":"integer","nullable":true,"metadata":{}},{"name":"ws_wholesale_cost","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_list_price","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_sales_price","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_ext_discount_amt","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_ext_sales_price","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_ext_wholesale_cost","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_ext_list_price","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_ext_tax","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_coupon_amt","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_ext_ship_cost","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_net_paid","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_net_paid_inc_tax","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_net_paid_inc_ship","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_net_paid_inc_ship_tax","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_net_profit","type":"decimal(7,2)","nullable":true,"metadata":{}},{"name":"ws_sold_date_sk","type":"integer","nullable":true,"metadata":{}}]}, spark.sql.sources.schema.partCol.0=ws_sold_date_sk, transient_lastDdlTime=1552464691, spark.sql.partitionProvider=catalog, spark.sql.sources.schema.numParts=1, spark.sql.sources.provider=parquet, spark.sql.create.version=2.3.0}, viewOriginalText:null, viewExpandedText:null, tableType:EXTERNAL_TABLE)`

@rdblue could you review this small change --> https://github.com/apache/incubator-iceberg/pull/405? Thanks!


Replace py3.5 with py3.7 in Travis CI + update to xenial distribution

This PR updates Travis CI to use python 2.7, 3.6 and 3.7 instead of 2.7, 3.5 and 3.6.  The main motivation is to start testing against python 3.7 while at the same time keeping the number of checks to be unchanged (9 variants total).
This also updates the linux distribution from trusty to xenial (required for Python 3.7).

Looks good to me.
FWIW in the Slycot Travis setup we don't use the Travis-provided Python at all; we just install the desired version from Conda, so we didn't need to specify Xenial to get Python 3.7.

Looks good to me.
FWIW in the Slycot Travis setup we don't use the Travis-provided Python at all; we just install the desired version from Conda, so we didn't need to specify Xenial to get Python 3.7.

Hey, I found an error in the travis configuration for Slycot, where Python 3.5 and 3.6 are not tested at the moment.  I created a pull request to fix this, and an additional error in setup.py that prevents Slycot to be installed from source in a python 2.7 environment.
Pull request : https://github.com/python-control/Slycot/pull/62


ERROR Can't Use XSS Tools ?!

I was testing the tools that you created, then I got a tools error in the xss section. I've even reinstalled the tools, but the results are still like that

you need to install selenium module


Simplify transfer of Razor assets to C#/C# DevKit

Our insertion process is currently complicated; we should simplify it to match dotnet/roslyn so that we have equivalent processes and it is easier to follow for all involved.

Finished with https://github.com/dotnet/razor/pull/10453


Add preference to avoid symlink creation

Each time the default release is changed, the installer symlinks /usr/local/bin/erl (and others) with the ones in the default release.
This should be optional. A boolean flag (probably ticked by default) should appear in the preferences window so that the user can choose if that is what he wants or not.
Upon ticking the checkbox, the symlink should be created as well. When unticking, nothing should happen.

Uses a path override in .profile now - no symlinking - reopen if I'm incorrect.


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

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

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

Result of nixpkgs-review pr 124255 at 4af47234 run on aarch64-linux 1

23 packages marked as broken and skipped:

python38Packages.cnvkit
python38Packages.dask-ml
python38Packages.dask-xgboost
python38Packages.dftfit
python38Packages.ignite
python38Packages.mlxtend
python38Packages.optuna
python38Packages.osmnx
python38Packages.pygbm
python38Packages.shap
...

3 packages failed to build:

python38Packages.scikit-bio
python39Packages.qiskit-ignis
python39Packages.scikit-bio

22 packages skipped due to time constraints:

python38Packages.dask-glm
python38Packages.graspologic
python38Packages.hdbscan
python38Packages.hyppo
python38Packages.nilearn
python38Packages.pytorch-metric-learning
python38Packages.qiskit
python38Packages.qiskit-aqua
python38Packages.qiskit-ignis
python38Packages.scikit-optimize
...

28 packages built successfully:

python38Packages.batchgenerators
python38Packages.bayesian-optimization
python38Packages.hmmlearn
python38Packages.imbalanced-learn
python38Packages.kmapper
python38Packages.librosa
python38Packages.lightgbm
python38Packages.mlrose
python38Packages.persim
python38Packages.ppscore
python38Packages.pynndescent
python38Packages.ripser
python38Packages.scikitlearn (python38Packages.scikit-learn)
python38Packages.seqeval
python38Packages.word2vec
python39Packages.batchgenerators
python39Packages.bayesian-optimization
python39Packages.hmmlearn
python39Packages.imbalanced-learn
python39Packages.kmapper
python39Packages.lightgbm
python39Packages.mlrose
python39Packages.persim
python39Packages.ppscore
python39Packages.ripser
python39Packages.scikitlearn (python39Packages.scikit-learn)
python39Packages.seqeval
python39Packages.word2vec

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

Package itself certainly builds for me now, macos 10.15, sandbox enabled.

I'd love to get this one merged, and then we can see if hydra likes it (fingers crossed!) and fix the downstream builds that are broken before the release. Thanks so much to @nbren12  and @DavHau for all the hard work -- this was not an easy PR.

Thanks for taking this over @DavHau! Looking forward to the new release.


fixed issue 200 Input not found

Illuminate\Support\Facades\Input is deprecated in Laravel 6 and we have to use Illuminate\Support\Facades\Request instead.
If you use older versions of Laravel you have to add the following line to the app configuration.
'Request' => Illuminate\Support\Facades\Input::class,
Closes #200

Hi!
Thanx for your PR.
There are some issues that should be fixed before I can merge it:

there is a difference between Request::get() and Input::get(). When you use Input::get() the input method is called on the request. So, when using Request facade the correct equivalent is Request::input() not the Request::get()

Considering previous experience, it's better to use Facade directly (with namespace, Illuminate\Support\Facades\Request) instead of short class alias.

Is that facade available in Laravel 4.2? There should be backward compatibility with Laravel 4.2+

@ahb360 Are you still working on this pull request? Can you finalize it :D ?

@Nayjest Request::input might not be available in 4.2 -- do we really have to keep compat with 4.2 ?
https://laravel.com/docs/4.2/requests

https://github.com/Nayjest/Grids/pull/215

Hi
I'm still here. Yes, we need compat with 4.2, so I can't merge it

This PR needs fallback code for L4.2

We probably switch to Laravel Backpack, feel free to adapt my PR. I probably cannot follow it up anymore, sorry

Looks more like emotional decision if you already have code with this lib used.
Fallback to L4.2 can be done easy, just one if-else statement, example:
https://github.com/Nayjest/Grids/commit/5137e9a6306bcccea9cbdcd72715fd38bbb25816
Also you can use grids from your forked repository, just specify custom repository in composer.json of your project.

We still like the library, but for other reasons we switch to Laravel backpack, because we need a full admin backend, that'S why. Thanks for the amazing work.

Fixed in v2.0.2


Allow automated testing on openshift and k8s

The current odr examples can be used only via the UI, which is less useful for automatic testing.
The current subscription can be used on openshift, but it points to specific vm and use specific namespace.
The current dr kustomization can be used on openshift, but it uses specific namespace.
Example usage - starting 3 vm configurations concurrently:
kubectl apply -k odr/regional/pvc/subscription
kubectl apply -k odr/regional/dv/subscription
kubectl apply -k odr/regional/dvt/subscription

Will start 3 vms in namespaces kubevirt-sample-pvc, kubevirt-sample-dv, kubevirt-sample-dvt.
DR flows can be run at the same time for all vms.
From OpenShift console, the app path will be something like odr/regional/pvc/vm or odr/metro/pvc/vm.
All commands example in README.md should be the same for any vm - the only difference is the namespace.
Similar change for ocm-ramen-samples:
https://github.com/RamenDR/ocm-ramen-samples/pull/41
The readme gives nice overview how this work:
https://github.com/RamenDR/ocm-ramen-samples/blob/ecdd1f6292881a300605599b29c9b98ce35aa693/README.md
Changes:

[ ] Use one sample vm per configuration (pvc, dv, dvt, ...)
[ ] Remove mac address from the vm so we can run multiple instances at the same time
[ ] Use same appname in all vms so we don't need to change the drpc for every vm
[ ] Use unique namespaces per example so allow running multiple vms in the same time
[ ] Use overlays to create metro/regional variants for each vm
[ ] Use overlays to customize the subscription for every vm
[ ] Use overlays to customize the dr namespace for each example

We work on this in https://github.com/RamenDR/ocm-ramen-samples.


Compliance: no license

The repo has no LICENSE file and does not declare a license in any other way. Therefore it is all-rights-reserved, not open source.

The license is specified here https://github.com/oprogramador/deep-equal-in-any-order/blob/a520180917cd2aad0b260f66f09f452940e8bcfd/package.json#L28

There are two ways to apply the MIT license.

Put the text into a header in each source code file
Copy the standard template into a file named LICENSE at the top of your repo.

I have created a pull request to implement method 2:
https://github.com/oprogramador/deep-equal-in-any-order/pull/77

A license file is added in the<EMAIL_ADDRESS>version.

Thank you, @oprogramador !


Palettes are hard to close

In Sugar palettes are closed when the mouse leaves the palette area.
They should be gone at least when clicking elsewhere.
(Edited to clarify, move z-index order to new issue).

I implemented a fix in Jappy. Will make a corresponding patch.

Closing per discussion in https://github.com/sugarlabs/sugar-web/pull/129


Add TOSHIBA TZ10xx target / Cerevo BlueNinja board supports.

We added the target "TOSHIBA TZ1000 Series" and added the board "Cerevo BlueNinja".
Please merge.
Best regards.

Sorry, "TZ10xx" class's super class was inappropriate.
Fix the super class from "CortexM" to "CoreSightTarget", redo the pull request.


MediaTypeHeaderValue constructor throws if given a media type with params

From @SteveSandersonMS on Thursday, March 30, 2017 5:15:43 AM
Currently, new MediaTypeHeaderValue("application/vnd.entity; v=2") throws, giving the error The format of value 'application/vnd.entity; v=2' is invalid. Is this by design? It seems like it should be valid for an incoming Content-Type header to specify params.
Extra strange: MediaTypeHeaderValue.TryParse("application/vnd.entity; v=2", out _) returns true, implying that the same string is valid.
(Somewhat related to #718, in that it's also about extending the range of media types we parse)
Copied from original issue: aspnet/HttpAbstractions#805

I ran into the same problem using the FHIR mediatype (application/fhir+json; fhirVersion=4.0). It is very confusing that you cannot simply pass that to new StringContent(..).


tags.news.com.au is neither adware nor malware

https://github.com/StevenBlack/hosts/blob/5b0420f1038072d80a53ac5e74a087f2d7c1172f/data/StevenBlack/hosts#L84
tags.news.com.au appears to be used by News Corp as a way of tracking visits to legitimate news articles linked via social media campaigns. Because it isn't fakenews, it should be either removed or re-categorised as 'social'.

Hello anonymous and welcome here,
Here is the commit that introduced it: https://github.com/StevenBlack/hosts/commit/36fab9ad8f6518024e2c4def34a3b1f1a73e2f59.
So let me ping @ValbonneConsulting, @mys721tx for more context before trying to do anything else.
Cheers

b831f6b is the first commit introduced so I really won't know why it is there.

Ups my bad @mys721tx ... I just did a git blame ...

So Steve @StevenBlack, it's yours...

Steve @StevenBlack. Sorry,not really related to the issue here but let me ask. Do we really need those comments there:

Thanks @9e3fce0a.
You say yourself, *tags.news.com.au appears to be used by News Corp as a way of tracking visits to legitimate news articles linked via social media campaigns. *
That's reason enough to block.  This domain is present in the ad-hoc list, which is tracking, adware, malware, etc.  Nothing to do with Fake News anyway.
We block google domains for the same reasons this one stays blocked.
Closing.

Thanks @9e3fce0a.
You say yourself, *tags.news.com.au appears to be used by News Corp as a way of tracking visits to legitimate news articles linked via social media campaigns. *
That's reason enough to block. This domain is present in the ad-hoc list, which is tracking, adware, malware, etc. Nothing to do with Fake News anyway.
We block google domains for the same reasons this one stays blocked.
Closing.

This decision effectively blocks access to a legitimate news sources. Links generated by many News Corp outlet websites, including The Australian, redirect to the real article via tags.news.com.au . This domain is not just a transparent tracker - it's used as redirector that cannot be bypassed. Blocking this effectively blocks access to theaustralian.com.au. Please reconsider. Blocking other trackers, like Google's trackers, does not have this effect - i.e. it does not block access to Google services.
Otherwise, I guess all Australians who want to access legitimate news sources will have to stop using your hosts file, or create a fork that's suitable for Australians?

I think, that's where the whitelist takes the spotlight.
AdGuard Tracking Protection also blocks that domain (ublock blocked it on my Chrome).

@9e3fce0a The view from here is, if so-called "legitimate news sources" are tracking visitors and serving ads using the same domains and subdomains as the legitimate news, then those domains and subdomains are blocked by this hosts file.
This is core to what we do here: block adware, user tracking, and other malfeasance.

a way of tracking visits

Yes, it should be blocked.

Otherwise, I guess all Australians who want to access legitimate news sources will have to stop using your hosts file, or create a fork that's suitable for Australians?

This comment is also for you. It touches the way bait_sites are setup to make it appears that your "Hosts/DNS recursor" is breaking there site.... Don't never ever forget who actually broke there site....
A: who added the unwanted_ware (malicious code) on same domain as needed code?
So dear Australia, your burned of criminals (holistically correct), and apparently some still are criminals against the mankind. Stop helping them, and boycott news.com.au and other bait sites like theaustralian.com.au...
Don't forget they are doing there criminal actions because nobody punish them.
What would happens if they in the physically world was looking through your windows? (And this is was, as they also are looking through your toilet door)

Thanks @9e3fce0a.
You say yourself, *tags.news.com.au appears to be used by News Corp as a way of tracking visits to legitimate news articles linked via social media campaigns. *
That's reason enough to block. This domain is present in the ad-hoc list, which is tracking, adware, malware, etc. Nothing to do with Fake News anyway.
We block google domains for the same reasons this one stays blocked.
Closing.

This decision effectively blocks access to legitimate news sources. Links generated by many News Corp outlet websites, including The Australian, redirect to the real article via tags.news.com.au . This domain is not just a transparent tracker - it's used as redirector that cannot be bypassed. Blocking this effectively blocks access to theaustralian.com.au. Please reconsider. Blocking other trackers, like Google's trackers, does not have this effect - i.e. it does not block access to Google services.
Otherwise, I guess all Australians who want to access legitimate news sources will have to stop using your hosts file, or create a fork that's suitable for Australians?

It's Newscorp so saying legitimate is a very big stretch anyway

redirector that cannot be bypassed

Sounds like a bug on theaustralian . com. Trackers / redirectors are malware. You should complain to their site editors.


Fallback to opened date when ordering projects

Pull Request Description
Fixes #6787
Important Notes
I can't get Project Manager compilation to work locally so I guess I'll be relying on CI to verify that it's working correctly?
Of course, QA should be able to catch any problems too - the websocket API hasn't been changed so it should work out of the box with the current dashboard.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

this feels like it should be in the changelog but i'm not sure which section it would belong in


Focus and Bang are broken for DescribeSpec

Focus
Focus Foo describe closure:
class Foo : DescribeSpec({
  describe("f:Foo") {
    it("foo") {
      assertEquals(1, 2)
    }
  }

  describe("Bar") {
    it("bar") {
      assertEquals(1, 2)
    }
  }

  describe("Baz") {
    it("baz") {
      assertEquals(1, 2)
    }
  }
})

Expected: only describe Foo and it foo executed
Actual: foo ignored. Bar and Baz are executed and didn't fail?!

Bang
class Foo : DescribeSpec({
  describe("!Foo") {
    it("foo") {
      assertEquals(1, 2)
    }
  }

  describe("Bar") {
    it("bar") {
      assertEquals(1, 2)
    }
  }

  describe("Baz") {
    it("baz") {
      assertEquals(1, 2)
    }
  }
})

Expected: Foo is skipped. Bar and Baz failed
Actual: Bar and Baz failed. Foo is passed?

A first analysis tells me that the test is being correctly ignored (if you throw an exception inside it, for example, it won't fail), but it's marked incorrectly in IntelliJ.
This for both scenarios as per some tests I made.
I'm trying to see if this has anything to do with DescribeSpec itself or the way we show IntelliJ the results


glibc

Hi can it be possible to create a port of the GNU C library to Windows?  *** If you are interested in seeing glibc on this platform visit *** the "How to submit a new port" in the wiki: ***   https://sourceware.org/glibc/wiki/#Development *** and join the community!'
This error was on MSYS2 I think I didn't have my Visual C/C++ tools installed in MSYS2.
I'm on Windows 10 2016 Enterprise LTSB I would like to use Visual C/C++ or MSYS2 and maybe a Mac port. And I just sent a message to the community listserv.
Would I have to install cross compilation tools on Debian/Ubuntu?
sudo apt-get install mingw32
I get the same configure error when I use the cross compilation tools on my Debian VMWare virtual machine.
~/Downloads/glibc-2.27/build$ ../configure --host=i586-mingw32msvc checking build system type... x86_64-pc-linux-gnu checking host system type... i586-pc-mingw32msvc checking for i586-mingw32msvc-gcc... i586-mingw32msvc-gcc checking for suffix of object files... o checking whether we are using the GNU C compiler... yes checking whether i586-mingw32msvc-gcc accepts -g... yes checking for gcc... gcc checking for i586-mingw32msvc-readelf... i586-mingw32msvc-readelf checking for i586-mingw32msvc-g++... i586-mingw32msvc-g++ checking whether we are using the GNU C++ compiler... yes checking whether i586-mingw32msvc-g++ accepts -g... yes checking whether i586-mingw32msvc-g++ can link programs... yes checking for sysdeps preconfigure fragments... aarch64 alpha arm hppa i386 m68k microblaze mips nios2 powerpc riscv s390 sh sparc tile x86_64  configure: error:  *** The GNU C library is currently unavailable for this platform. *** If you are interested in seeing glibc on this platform visit *** the "How to submit a new port" in the wiki: ***   https://sourceware.org/glibc/wiki/#Development *** and join the community!

I think you can use vcpkg_configure_make and vcpkg_install_make.
If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Incorrect error message in add_group when non-positive integer member index supplied

Currently, the app shows the error message For input string: "ddfdsf" when an illegal member index is supplied along with the add_group and edit_group commands. Eg add_group n/sfsfsd m/ddfdsf

This was solved by #163


refactor: use log.Panic instead of panic

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

LGTM

LGTM


Out-of-source CMake build for libraries

SFML, libpng

#302 #296

SFML and libpng are not used anymore.
Won't fix.


provision package individual windows 11 home?

Awesome info! Simple question (for maintainers @lizgt2000, @aczechowski?): does this work for an individual using windows 11 home edition?
I like to reset my home computer once a quarter or so to keep my development environments from conflicting. I don't want my old user or system environment, but I do want some apps to stick around. I've used imaging, but I want a bit more control over my system reinstalls. According to the docs package provisioning does exactly what I want:

Quickly configure a new device without going through the process of installing a new image.

The scope of this configure process seems appropriate for an individual doing routine resets too:

It is best suited for small- to medium-sized businesses with deployments that range from tens to a few hundred computers.

I've installed the windows configuration designer as admin, created a simple desktop .ppkg that included removing pre-installed software and installing firefox nightly, and began to apply the package via usb drive during initial setup, HOWEVER it fails with 4 errors:

Set or change network connectivity settings - ...
Cause the device to reboot - failed
Reset system and remove pre-installed software - failed
Run Scripts (0) - ...

Am I missing something important? Is there an easier or more canonical Microsoft tool to accomplish these customized quarterly resets?
Thanks!
Alec
-p.s. an hour of customer service only got me so far
¯_༼ ಥ ‿ ಥ ༽_/¯
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bc6a7f82-c972-6aea-f045-08fb57a99ae6
Version Independent ID: 384fe96b-ace9-3e34-e5e9-24bb7409e0e8
Content: Provisioning packages overview on Windows 10/11 - Configure Windows
Content Source: windows/configuration/provisioning-packages/provisioning-packages.md
Product: w10
Technology: windows
GitHub Login: @lizgt2000
Microsoft Alias: lizlong

Thanks @aguthrie19 for taking the time to share this with the Windows documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this issue:

Post to Windows on Q&A. These are moderated by community members who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Unified Support team, such as a Customer Engineer or Customer Success Account Manager.

I'm closing this issue as there's nothing actionable for docs at this time.


Summary of Package Contents

Comment:
Please provide a brief summary expressing what is in the various packages.
I am developing a clang plugin to extract information from the AST.
In particular, I need the clang headers.
// tag::visitor_hdr[]
#include "clang/AST/ASTConsumer.h"
#include "clang/AST/RecursiveASTVisitor.h"
#include "clang/Frontend/CompilerInstance.h"
#include "clang/Frontend/FrontendAction.h"
#include "clang/Tooling/CommonOptionsParser.h"
#include "clang/Tooling/Tooling.h"
// end::visitor_hdr[]

Are these present in any of the packages?

I want to consume these files via cmake.
To that end, https://llvm.org/docs/CMake.html , https://cmake.org/cmake/help/latest/command/find_package.html , and https://github.com/banach-space/clang-tutor look relevant.
$mambaInfo = mamba info --json | ConvertFrom-Json
Get-ChildItem -path $mambaInfo.'env location' -Filter 'LLVMConfig.cmake' -Recurse -ErrorAction SilentlyContinue -Force

Directory: C:\Users\<self>\micromamba\envs\<proj>\lib\cmake\llvm
Name: LLVMConfig.cmake

I presume there is some environment variable which needs to be set for the find_package( LLVM ...  to work correctly.

I want to consume these files via cmake.

llvm has a shared cmake infastructure, which doesn't necessarily match with the way we need to split our packaging (not least due to the 6h timeout imposed by CI). So it's possible you need both llvmdev and clangdev

Closing this as answered


[chart/redis-ha][BUG] Cannot specify resources for split-brain-fix conainter

Describe the bug
Cannot specify resources for the new split-brain-fix container, part of redis-ha-statefulset. This is required to be able to deploy Redis into namespaces with defined resource quotas.
Expected behavior
A way to define container resources for split-brain-fix container.

@artur-reaboi Adding now.

Right, sorry I initially didn't add it: it consumes little to no resources and I don't run tightly restricted k8s clusters.

@zerkms No worries. Just an fyi incase your chart uses the old snake case syntax.

@DandyDeveloper I'm switching back to this one as long as I have a spare minute :-)


README.md: Adds instructions how to obtain bootloader-details

Fixes ARMmbed/mbed-bootloader#23

Thanks for your changes.
@ARMmbed/pdmc-integrators please review.

@ wajahat-ublox for your information.

Instead is best to point developers to the corresponding json file that will contain the information.

I can't find the same information from the corresponding json file

This is the manual way of generating the hash. Alternatively you can find the information as part of bootloader releases https://github.com/ARMmbed/mbed-bootloader/releases/tag/v4.1.1
There is a text-file which contain this information.

Are you saying this is a kind of random value that's generated at compile time?

Are you saying this is a kind of random value that's generated at compile time?
It would be great to understand the logic behinds this.

If i am not wrong, this is the address where bootloader lies. It is was earlier printed on serial but was later removed to reduce memory footprint. Now the only way is by reading it from .map file of the image.

Are you saying this is a kind of random value that's generated at compile time?
It would be great to understand the logic behinds this.

Please see the bootloader documentation about the subject

@VeijoPesonen thanks for the link to the docs. It looks we're still referring to information generated on the serial port.
I'm ok to leave the comments on the docs on how to find out the hash info. In the future, I'll be looking at generating this automatically after the compile/link process.

@MarceloSalazar that would most likely be a feature request for Mbed CLI to provide such functionality.

@VeijoPesonen can you rebase this?

Rebased.


SSL verification error in conda in Mac OS X Sierra

I'm running conda ver. 4.1.12 on a Mac Pro running Mac OS X Sierra (10.12.1). I was attempting to install the package astroplan. I got error messages saying that the SSL module was not available.
Now this is the first time running under Mac OS X Sierra that I got SSL verification error messages. I did check the settings in my .condarc file and found that ssl_verify: true.
I've attached a file containing the error message and the reults of using the conda info -a command.
Please advise.
Sam Dupree.
SSL_verificatin_error_conda.txt

I think this issue belongs here: https://github.com/conda/conda
CC @kalefranz


fix: changed to speed-up peaks search

@Naww137 , please have a check. I just added dummy fix to speed up work with a resulting dataframe.

I pulled this in.
I have been leaving branches open after merging the pull request, would you like me to delete them? Or, if you have access to delete them I will just leave them alone and you can delete when you are done with them?

Noah, as for me personally - it's not important right now. It's up to you.
I will add some new parameters to the PolePos soon. But when I will be
ready - I can create a new branch - it's not a problem really.
The last project I had - we had rules on code/documentation versioning
(drafts):
the owner/contributor who made changes decides to delete or not the
branches he created, or previous versions he had.
Then some automation was done (I don't know how - I just saw it) - old
branches were reviewed on monthly basis and deleted by the owners if they
are done - not to multiply the instances.
I think it would be nice if a contributor will decide when to delete the
branch.
On Wed, Feb 8, 2023 at 3:48 PM Naww137 @.***> wrote:

I pulled this in.
I have been leaving branches open after merging the pull request, would
you like me to delete them? Or, if you have access to delete them I will
just leave them alone and you can delete when you are done with them?
—
Reply to this email directly, view it on GitHub
https://github.com/Naww137/ATARI/pull/10#issuecomment-1423220463, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AN5AOMX5KSXGCIHQX57ZLEDWWQBC5ANCNFSM6AAAAAAUVXJZSI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Best regards,
Oleksii Zivenko
З повагою,
Зівенко Олексій Васильович
С ув.,
Зивенко Алексей Васильевич

+380-63-303-85-20
+48-784-982-529
Additional mail: @.** @.**>


[20.03] p11-kit: 0.23.19 -> 0.23.20

Small bug fix release.
Need this to improve flatpak's function.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

The topic of a backport of this PR came up on discourse https://discourse.nixos.org/t/flatpak-error-the-name-org-freedesktop-accounts-was-not-provided-by-any-service-file/6000/10

@hedning
There’s some issues with flatpak and the p11-kit on unstable/20.03 (apps are unable to connect to the internet etc), though your issue sounds a bit different. A p11-kit update is on its way to master: https://github.com/NixOS/nixpkgs/pull/80698.k


URL too long

What happened?
My WebStorm crashed for (I think) unrelated reasons. Upon reopening, there was a red bell at the bottom-right, and I clicked it. I've clicked "Report and clear" without really reading the thing, so I can't tell you much about that.
Anyway, my Chrome opened about 15 pages, all of which pointed to a "URL too long" GitHub page. Here's one URL that I've saved.
Runtime Information
Plugin version : 3.1.0
IDE: WebStorm 2023.2.4 (WS-232.10203.14)
OS: Linux 6.2.0-36-generic
Stacktrace
com.intellij.openapi.diagnostic.RuntimeExceptionWithAttachments: PSI and index do not match.
Please report the problem to JetBrains with the files attached
 file=JSFile:StorageApi.ts, file.class=class com.intellij.lang.javascript.psi.impl.JSFileImpl, file.lang=Language: TypeScript, modStamp=1
 tree consistent
 stub debugInfo=created in getStubTree(), with AST = false; with backReference
 viewProvider=com.intellij.lang.javascript.JSFileViewProviderFactory$JSFileViewProvider{vFile=file:///home/lazar/elements-web/src/sdk/src/apis/StorageApi.ts, vFileId=665542, content=VirtualFileContent{size=147305}, eventSystemEnabled=true}
 viewProvider stamp: 0; file stamp: 0; file modCount:<PHONE_NUMBER>986; file length: 147305
 doc saved: true; doc stamp: 0; doc size: 147305; committed: true
indexing info: indexing timestamp =<PHONE_NUMBER>785, binary = false, byte size = 149645, char size = 149645
ref: 50cf572587cf
latestIndexedStub=null
	at com.intellij.psi.stubs.StubTreeLoader$StubTreeAndIndexUnmatchCoarseException.doCreateCompleteException(StubTreeLoader.java:203)
	at com.intellij.psi.stubs.StubTreeLoader$StubTreeAndIndexUnmatchCoarseException.lambda$createCompleteException$0(StubTreeLoader.java:178)
	at com.intellij.openapi.progress.impl.CoreProgressManager.registerIndicatorAndRun(CoreProgressManager.java:679)
	at com.intellij.openapi.progress.impl.CoreProgressManager.computeUnderProgress(CoreProgressManager.java:635)
	at com.intellij.openapi.progress.impl.CoreProgressManager.lambda$computeInNonCancelableSection$4(CoreProgressManager.java:230)
	at com.intellij.openapi.progress.Cancellation.computeInNonCancelableSection(Cancellation.java:64)
	at com.intellij.openapi.progress.impl.CoreProgressManager.computeInNonCancelableSection(CoreProgressManager.java:230)
	at com.intellij.psi.stubs.StubTreeLoader$StubTreeAndIndexUnmatchCoarseException.createCompleteException(StubTreeLoader.java:177)
	at com.intellij.psi.impl.source.PsiFileImpl.loadTreeElement(PsiFileImpl.java:228)
	at com.intellij.psi.impl.source.PsiFileImpl.calcTreeElement(PsiFileImpl.java:727)
	at com.intellij.psi.impl.source.PsiFileImpl.getChildren(PsiFileImpl.java:732)
	at com.github.yunabraska.githubworkflow.helper.PsiElementHelper.getAllElements(PsiElementHelper.java:290)
	at com.github.yunabraska.githubworkflow.helper.PsiElementHelper.getAllElements(PsiElementHelper.java:291)
	at com.github.yunabraska.githubworkflow.helper.PsiElementHelper.getAllElements(PsiElementHelper.java:282)
	at com.github.yunabraska.githubworkflow.helper.PsiElementChangeListener.lambda$childrenChanged$3(PsiElementChangeListener.java:33)
	at java.base/java.util.Optional.map(Optional.java:260)
	at com.github.yunabraska.githubworkflow.helper.PsiElementChangeListener.childrenChanged(PsiElementChangeListener.java:33)
	at com.intellij.psi.impl.PsiManagerImpl.notifyPsiTreeChangeListener(PsiManagerImpl.java:407)
	at com.intellij.psi.impl.PsiManagerImpl.fireEvent(PsiManagerImpl.java:350)
	at com.intellij.psi.impl.PsiManagerImpl.childrenChanged(PsiManagerImpl.java

It's up to you if you want to dedicate this issue to the "too long" part or the actual problem part. I really have no idea what happened and nothing was really disrupted in my workflow. I didn't even have a single yaml file open at the time.
Relevant log output
Uuuh see URL?
Operating System
Ubuntu 22.04
Plugin Version
3.1.0, update to 3.2.1 pending (not installed -- will do now)
IDE Name and Version
WebStorm 2023.2.4
Build #WS-232.10203.14, built on October 25, 2023

Hi, thanks for reporting. Unfortunately, I can't give any support for an older version.
The thing is, that the older version is scanning unfortunately all files also node_modules which is really too much, and then it's also blocking. That's fixed in the newer version. The best and easiest way is to update to the newer version ;)

Sure! Thought you might want to know that the URLs are too long sometimes and that might mean missing some feedback completely.
It feels like I've installed it just yesterday. They grow up so quickly 🥲 Cheers! 🍻

Ah yes, you are right, i have to do something about the too long URL :)


[AutoPR keyvault/data-plane] [KeyVault] Adding crv to KeyProperties for EC certificate support

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3198

(message created by the CI based on PR content)

Deprecated and conflicts


fix(gridfs): Adapt code to refactored Mongo driver GridFS implementation

Closes #903

Dear @mdomke ,
Do you have plan to release 4.3.X soon or should I consider backporting this PR into Debian, as mongomock is currently not functional there?
Both are fine, I'd just rather use a new release if it's coming soon.
Bests,
PEB


eth: some docs and cleanups

Codecov Report

Merging #108 (26cfe0a) into main (53d7f36) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   99.74%   99.71%   -0.03%
==========================================
  Files          67       67
  Lines        3848     3848
==========================================
- Hits         3838     3837       -1
- Misses         10       11       +1

Impacted Files
Coverage Δ

lib/eth.rb
100.00% <ø> (ø)

lib/eth/contract/event.rb
100.00% <ø> (ø)

lib/eth/contract/function_input.rb
100.00% <ø> (ø)

lib/eth/contract/function_output.rb
100.00% <ø> (ø)

lib/eth/contract/initializer.rb
100.00% <ø> (ø)

lib/eth/chain.rb
100.00% <100.00%> (ø)

lib/eth/client.rb
100.00% <100.00%> (ø)

lib/eth/contract.rb
100.00% <100.00%> (ø)

lib/eth/contract/function.rb
100.00% <100.00%> (ø)

lib/eth/client/ipc.rb
93.33% <0.00%> (-6.67%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


UI edge cases are not accomodated

The SessionLoginField doesn't currently cater to the following edge cases:

Member without any LoginSession
ReadOnly view
LoginSession for which you do not have Delete rights

@silverstripeux

I think that for all those situations it makes sense. @maxime-rainville for these scenarios does the section appear as empty or is it completely removed?
I would imagine there would be a message saying there is no login sessions or that area would be completely missing. My preference is that the area is present with a message but that would mean potentially different messages for different cases unless we can create a generic one.

Noting that I don't see this issue being a priority to resolve before the initial beta/ stable release.
When docs are created in #57 that outline the possibility of altering the behaviour to give privileged users access to other user sessions, we could make the recommendation that both CanView and CanDelete permissions should be provided, as a read-only view is not supported.

A quick win could just be to not display logout session when the field is read-only. That's probably a 1-2 point job and would lead to less confusion than having some convoluted explanation in the doc.


Adding PyBlock

https://twitter.com/__B__T__C__/status/1308454565952585731?s=20

Merged PR with basics - final TODO: Add to update/recover provisioning.

Recovery added. Tests look good - will be part of v1.6.1 release - closing issue.


Introduce ContainerFactory

This is a split from #333 to try to make smaller steps more quickly :)
Goals of this feature from that PR description:

Centralize the code building the container into a factory. This makes it easier to change the build process (instead of having to update both the tests and guides "binary"), and allows to add some custom layer for DI extensions.
Give extensions full control by allowing them to add compiler passes, create configuration trees and load services. This means the container initialization no longer has to register compiler passes that actually belong to a specific package.

This looks good! Thanks a lot!


Add Windows support

Should fix https://github.com/tomitrescak/meteor-uploads/issues/66

I've just fixed it again. I just copy/pasted from a fix I made outside this package, so all the variable names were wrong. Sorry.

Superb! Merging and publishing. Thanks!


etcdmain: do not use defer in for loop

remove defer statements in for loop, it's hard to read and using defer in for loop may lead source leak

We want to close the listener when there is an error returned. This commit changes the behavior to close the listener when etcd fails to listen. This does not make sense to me.


Kernel version 5.18.2-1 not supported

os： CentOS Linux release 7.9.2009
problem：After upgrading the latest kernel with elrepo, the installation of tetragon prompts that the kernel version is not supported
log：
time="2022-06-07T08:10:36Z" level=info msg="Loaded config from directory" config-dir=/etc/tetragon
time="2022-06-07T08:10:36Z" level=info msg="Starting tetragon" version=v0.8.0
time="2022-06-07T08:10:36Z" level=info msg="config settings" config="map[bpf-lib:/var/lib/tetragon/ btf: cilium-bpf: config-dir:/etc/tetragon config-file: debug:false enable-cilium-api:false enable-export-aggregation:false enable-k8s-api:true enable-process-ancestors:true enable-process-cred:false enable-process-ns:false export-aggregation-buffer-size:10000 export-aggregation-window-size:15s export-allowlist:{"event_set":["PROCESS_EXEC", "PROCESS_EXIT", "PROCESS_KPROBE"]} export-denylist:{"health_check":true}\n{"namespace":["", "cilium", "kube-system"]} export-file-compress:false export-file-max-backups:5 export-file-max-size-mb:10 export-file-rotation-interval:0s export-filename:/var/run/cilium/tetragon/tetragon.log export-rate-limit:-1 force-small-progs:false ignore-missing-progs:false kernel: log-format:text log-level:info metrics-server::2112 netns-dir:/var/run/docker/netns/ process-cache-size:65536 procfs:/procRoot run-standalone:false server-address:localhost:54321 verbose:0]"
time="2022-06-07T08:10:36Z" level=info msg="Available sensors" sensors=
time="2022-06-07T08:10:36Z" level=info msg="Registered tracing sensors" sensors="kprobe sensor, tracepoint sensor"
time="2022-06-07T08:10:36Z" level=info msg="Registered probe types" types="kprobe sensor, tracepoint sensor"
time="2022-06-07T08:10:36Z" level=info msg="candidate btf file does not exist" file=/sys/kernel/btf/vmlinux
time="2022-06-07T08:10:36Z" level=info msg="candidate btf file does not exist" file=/var/lib/tetragon/metadata/vmlinux-5.18.2-1.el7.elrepo.x86_64
time="2022-06-07T08:10:36Z" level=info msg="candidate btf file does not exist" file=/var/lib/tetragon/btf
time="2022-06-07T08:10:36Z" level=fatal msg="Failed to start tetragon" error="tetragon, aborting kernel autodiscovery failed: Kernel version "5.18.2-1.el7.elrepo.x86_64" BTF search failed kernel is

You need to compile your kernel with BTF info. Check for the existence of /sys/kernel/btf.

Alternatively, you can generate the BTF yourself and load it into the Tetragon pod but that's a bit more of an advanced use case.

Is there any demo or guide of fixing this problem?

No sys/kernel/btf。Is there any relevant documentation? Thank you

https://www.kernel.org/doc/html/latest/bpf/btf.html#btf-generation This explains how to generate the BTF manually. You could then set the helm values such that this is mounted into the Tetragon container and used by Tetragon.
Alternatively, you'll need to recompile your kernel with BTF support. That involves setting some kconfig flags and compiling it with a recent version of pahole installed on your system.

If you end up going the first route, here are the helm values you're looking for:
https://github.com/cilium/tetragon/blob/402d16f084669bccf1562dc4d2fda8554c0f3715/install/kubernetes/values.yaml#L14
https://github.com/cilium/tetragon/blob/402d16f084669bccf1562dc4d2fda8554c0f3715/install/kubernetes/values.yaml#L118

BTF is enabled by default using cenots 8 。Using CentOS 8, no unsupported kernel is reported
https://github.com/aquasecurity/btfhub/blob/main/docs/supported-distros.md

If you end up going the first route, here are the helm values you're looking for:
https://github.com/cilium/tetragon/blob/402d16f084669bccf1562dc4d2fda8554c0f3715/install/kubernetes/values.yaml#L14
https://github.com/cilium/tetragon/blob/402d16f084669bccf1562dc4d2fda8554c0f3715/install/kubernetes/values.yaml#L118

I download a pre-built btf file from BTF-Hub and place it to /var/lib/tetragon/.
Then I read the files in install/kubernetes/templates and change the install/kubernetes/values.yaml to
extraHostPathMounts:
- name: "btf-dir"
  mountPath: /var/lib/tetragon/
  mountPropagation: HostToContainer
  readOnly: true
  ...
  btf: "/var/lib/tetragon/4.18.0-147.5.1.el8_1.x86_64.btf"

However, this directory is not mounted successfully, and tetragon reports the btf file does not exists.
Here are related logs:
> ls /var/lib/tetragon/

4.18.0-147.5.1.el8_1.x86_64.btf

> kubectl logs -f "$(kubectl get pod -n kube-system | grep tetragon | awk '{print $1}')" -n kube-system -c tetragon

time="2022-06-20T02:07:35Z" level=warning msg="BPF filesystem is going to be mounted automatically in /run/cilium/bpffs. However, it probably means that Cilium is running inside container and BPFFS is not mounted on the host. for more information, see: https://cilium.link/err-bpf-mount"
time="2022-06-20T02:07:35Z" level=info msg="Available sensors" sensors=
time="2022-06-20T02:07:35Z" level=info msg="Registered tracing sensors" sensors="kprobe sensor, tracepoint sensor"
time="2022-06-20T02:07:35Z" level=info msg="Registered probe types" types="tracepoint sensor, kprobe sensor"
**time="2022-06-20T02:07:35Z" level=fatal msg="Failed to start tetragon" error="tetragon, aborting kernel autodiscovery failed: User specified BTF does not exist: stat /var/lib/tetragon/4.18.0-147.5.1.el8_1.x86_64.btf: no such file or directory"**

> kubectl describe pod "$(kubectl get pod -n kube-system | grep tetragon | awk '{print $1}')" -n kube-system
tetragon:
   Mounts:
      /etc/tetragon from tetragon-config (ro)
      /procRoot from host-proc (rw)
      /sys/fs/bpf from bpf-maps (rw)
      /var/lib/tetragon/ from btf-dir (rw)
      /var/run/cilium from cilium-run (rw)
      /var/run/cilium/tetragon from export-logs (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-c462t (ro)

However, the file is not detected and tetragon reports the btf file does not exists.

Does the file var/lib/tetragon/4.18.0-147.5.1.el8_1.x86_64.btf exist on the host or in the tetragon pod? If it exists on the host, but not in the pod then the tetragon agent will not be able to find it.

Does the file var/lib/tetragon/4.18.0-147.5.1.el8_1.x86_64.btf exist on the host or in the tetragon pod? If it exists on the host, but not in the pod then the tetragon agent will not be able to find it.

Sure, the file is not in the pod. But is it expected?
The extraHostPathMounts has mounted the host path /var/lib/tetragon/ to the tetragon pod. I thought the btf file should be shared in this way.
Did I misunderstand the effects of extraHostPathMounts?

If it exists on the host, but not in the pod then the tetragon agent will not be able to find it.

Sure, the file is not in the pod. But is it expected?
The extraHostPathMounts has mounted the host path /var/lib/tetragon/ to the tetragon pod. I thought the btf file should be shared in this way.
Did I misunderstand the effects of extraHostPathMounts?

Ah right! I think you are correct!
In that case, I'm not sure why we are getting that error if the file exists:
https://github.com/cilium/tetragon/blob/1b7d2315df37b8f88c13e1d15dfdb62f7d425629/pkg/btf/btf.go#L72-L74
https://github.com/cilium/tetragon/blob/1b7d2315df37b8f88c13e1d15dfdb62f7d425629/pkg/btf/btf.go#L25-L27
Would it be possible to kubectl exec into the tetragon pod and check that the file is indeed there?

Could it be that there the btf value is not properly passed?
At the beginning of the tetragon logs, there should be a line that prints the full configuration:
https://github.com/cilium/tetragon/blob/1b7d2315df37b8f88c13e1d15dfdb62f7d425629/cmd/tetragon/main.go#L107-L108
Could you please check it out?

Is this a KinD cluster by any chance? If so you would need to make sure the file is present in the KinD container in order for extraHostPathMounts to work properly.

Is this a KinD cluster by any chance? If so you would need to make sure the file is present in the KinD container in order for extraHostPathMounts to work properly.

Yes, it'a local cluster created via KinD. Thanks, I understand it now. The host path is in fact the KinD container in this situation


added dotted key export notation to to-js.

ember-i18n has dotted key definition along with nested object definition. https://github.com/jamesarosen/ember-i18n/wiki/Doc:-Defining-Translations
This request adds an option(--dotted) which allows dotted key export.

This looks great!. If you could squash your commits, I will merge ASAP.

Thanks.
I've squashed commits.

Sorry I have to revert. There are a lot of failing tests, for some reason the CI wasn't failing.

filed https://github.com/kellyselden/ember-i18n-csv/issues/59

I think I fixed the testing issue. Can you resubmit the PR after rebasing master please?


Updated #2573 to Volto 18: Change text color to black for input:focus

The PR #2573 is quite old, and since then, the structure of Volto 18 has changed. This PR aims to bring it up to date.
It was approved 3 years ago.

This PR solves both https://github.com/plone/volto/issues/2570 and https://github.com/plone/volto/issues/1505 issues.


Do not attempt quota checks for admin operations.

Codecov Report

Merging #1920 (047883c) into master (8a9e6a6) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #1920   +/-   ##
=========================================
  Coverage     15.40%   15.41%
- Complexity     2272     2275    +3
=========================================
  Files           690      691    +1
  Lines         53206    53225   +19
  Branches       6629     6633    +4
=========================================
+ Hits           8199     8206    +7
- Misses        44531    44544   +13
+ Partials        476      475    -1

Impacted Files
Coverage Δ

...om/github/ambry/frontend/AmbrySecurityService.java
0.00% <0.00%> (ø)

...c/main/java/com/github/ambry/quota/QuotaUtils.java
0.00% <0.00%> (ø)

...n/java/com/github/ambry/store/PersistentIndex.java
90.42% <0.00%> (-0.10%)
:arrow_down:

...main/java/com/github/ambry/config/QuotaConfig.java
0.00% <0.00%> (ø)

...va/com/github/ambry/quota/QuotaChargeCallback.java
0.00% <0.00%> (ø)

...java/com/github/ambry/quota/AmbryQuotaManager.java
0.00% <0.00%> (ø)

...java/com/github/ambry/router/GetBlobOperation.java
0.00% <0.00%> (ø)

...in/java/com/github/ambry/quota/QuotaTestUtils.java
0.00% <0.00%> (ø)

...com/github/ambry/frontend/NamedBlobPutHandler.java
0.00% <0.00%> (ø)

.../github/ambry/tools/admin/ConcurrencyTestTool.java
0.00% <0.00%> (ø)

... and 10 more

Continue to review full report at Codecov.

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


There are closing tags on both my p elements yet saying there isn't

Challenge http://freecodecamp.com/challenges/waypoint-change-the-font-size-of-an-element has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

<p2> is not a valid javascript tag. Please post in the Gitter help channel before submitting a bug report!


feat: allow single file remappings

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (9e3ab9b 2024-01-04T00:18:01.892563000Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug

Correct path: lib/openzeppelin-contracts/contracts/token/ERC721/extensions/IERC721Metadata.sol
Required Import: @openzeppelin/contracts/token/ERC721/IERC721Metadata.sol
This is not possible, since this exception is only for one file: @openzeppelin/contracts/token/ERC721/=lib/openzeppelin-contracts/contracts/token/ERC721/extensions/

remappings.txt

@openzeppelin/contracts/token/ERC721/IERC721Metadata.sol=lib/openzeppelin-contracts/contracts/token/ERC721/extensions/IERC721Metadata.sol

Error:

failed to resolve file: "/Users/foundry/Developer/foundry/packages/contracts/lib/openzeppelin-contracts/contracts/token/ERC721/extensions/IERC721Metadata.sol/": Not a directory (os error 20); check configured remappings.

remappings.txt

@openzeppelin/contracts/token/ERC721/=lib/openzeppelin-contracts/contracts/token/ERC721/extensions/IERC721Metadata.sol

Error:

failed to resolve file: "/Users/foundry/Developer/foundry/packages/contracts/lib/openzeppelin-contracts/contracts/token/ERC721/extensions/IERC721Metadata.sol/IERC721Metadata.sol": Not a directory (os error 20); check configured remappings.

remappings.txt

@openzeppelin/contracts/token/ERC721/IERC721Metadata.sol=lib/openzeppelin-contracts/contracts/token/ERC721/extensions/

Error:

failed to resolve file: "/Users/foundry/Developer/foundry/packages/contracts/lib/openzeppelin-contracts/contracts/token/ERC721/extensions/": Is a directory (os error 21); check configured remappings.

could you please provide a repro for this?

@tash-2s I wonder if this is related to the recent path changes?

I believe this is the reproduction of the issue:
$ mkdir issues6706 && cd issues6706
$ mkdir src my-lib

$ cat << EOF > src/A.sol
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.23;
import<EMAIL_ADDRESS>contract A is B {}
EOF

$ cat << EOF > my-lib/B.sol
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.23;
contract B {}
EOF

$ cat << EOF > remappings.txt
@my-lib/B.sol=my-lib/B.sol
EOF

$ forge build
[⠊] Compiling...
Error:
failed to resolve file: "/Users/t/bench/issues6706/my-lib/B.sol/": Not a directory (os error 20); check configured remappings.
    --> "/Users/t/bench/issues6706/src/A.sol"
<EMAIL_ADDRESS>
This change makes it work:
diff --git a/remappings.txt b/remappings.txt
index 134e3a5..ceb4244 100644
--- a/remappings.txt
+++ b/remappings.txt
@@ -1 +1 @@
<EMAIL_ADDRESS>+@my-lib/=my-lib/

This issue is not related to the recent changes (https://github.com/foundry-rs/compilers/pull/35, https://github.com/foundry-rs/compilers/pull/36). I can confirm the same behavior occurs with an older version of forge. (foundryup --commit 67ab8704476d55e47545cf6217e236553c427a80)
The current implementation of remapping resolution does not support this case. See:
https://github.com/foundry-rs/compilers/blob/1cde7fe2559815f40255f7338c7998df4f55ca7d/src/config.rs#L344-L345
To simulate that code:
fn main() {
    use std::path::Path;

    let p1 = Path::new("@my-lib/A.sol").strip_prefix(Path::new("@my-lib/A.sol"));
    let p2 = Path::new("A.sol").join(Path::new(""));

    println!("{:?}, {:?}", p1, p2); // => Ok(""), "A.sol/"
}

This is correct - you cannot remap individual files, only directories. Not sure if we should count this as a bug or as a feature request, in any case, it might make sense to support this

The reason is that from 3.4.2 openzepplin has change the path of ERC72Metadata.sol.
so i solve it by install a low version of openzepplin.
forge install<EMAIL_ADDRESS>--no-commit

The reason is that from 3.4.2 openzepplin has change the path of ERC72Metadata.sol.
so i solve it by install a low version of openzepplin.
forge install<EMAIL_ADDRESS>--no-commit.

Not directly related but it is generally not recommended to install older versions of OpenZeppelin as they may have bugs and vulnerabilities.

This is correct - you cannot remap individual files, only directories. Not sure if we should count this as a bug or as a feature request, in any case, it might make sense to support this

Updated the ticket to reflect that this is a feature request, not a bug


Update error messaging to be more descriptive and consistent

Summary
This PR updates our error messages to follow a more consistent format and to provide suggested resolutions when possible.

This looks great 💯
Are there plans to push these error message strings to i18n bundles so they could one day be translated?

@jonspalmer I hadn't thought about i18n, but that would be great! One day 😄


feat: Symfony 7 support

Q
A

Type
feature

BC Break
no

Fixed issues
n/a

Summary
Wanted to get this started.

Wanted to get this started.

Awesome, thank you @kbond!


Files being uploaded are being padded to the nearest 16 bytes

Hi,
When uploading a file to telegram using WTelegramClient, and downloading the file using an official telegram client, the file size has been padded to the nearest 16 bytes.
I have tried this with 4 different file types, jpg, png, mp4 & tar.gz, and the result is always the same.
End of file before uploading / downloading.

End of file after uploading / downloading

I have sent the same file to myself using official clients for both the the sending and downloading, and the files is without padding, so im pretty sure this is a WTelegramClient issue.
Is this known behavior?

I'm now investigating. It may be due to recent developments around Secret Chats.
Maybe try a version before 3.0.0 to confirm this theory.

confirmed as regression since v3.0.0 working to fix it...

Thank you for the quick diagnosis and response

v3.0.2 now released with the fix
Thanks a lot of your great bug report 👍


Update React Native Documentation

showsVerticalScrollIndicator = {false}
showsHorizontalScrollIndicator = {false}
It's not on https://facebook.github.io/react-native/docs/webview but it works!
It makes the scroll bar invisible on the webview.
It should be nice to have an updated documentation!!

this documentation lives under the docs repository on this repo directly! For an updated reference check it out :). If anything is missing from there please PR ❤


In both kinetic and Melodic, the source didn't work

Thanks for sharing great work you've done. I almost gave up do related works as I thought making omniwheel model is kind of hard time consuming.
As I tried to launch as below,
$ roslaunch open_base velocity\ controller.launch

In two version shows like:
[Kinetic] git checked out to gazebo7 branch
Controller Spawner couldn't find the expected controller_manager ROS interface. (seems like gazebo warning) and open_base/controller_spawner has finished cleanly.
[Melodic]
No error or warning
[Both]
After the model brought up to gazebo space, it shows white color (AFAIK, this occurs when the model might be not appropriately brought up)
it seems like uses /open_base/command to control, so when I pubbed as you directed in README.md with few edit in wheel/v_left or generic/target/x or bezier/targetTranslation/x:1.0,  didn't work either.
When I put rostopic echo on /open_base/right_joint_velocity_controller/command, it always shows 0.0, so I don't know exactly what main node does for, but anyway it publishes.
I used teleop omni (you've tagged), but doesn't work as well.
.sh file, also.
Do you have any suggestion?
Thanks

Hi there.

Controller Spawner couldn't find the expected controller_manager ROS interface: might be that you have a missing ROS package, or some issue about something in the URDF that needs to be written differently depending on the ROS distro being used.
About the white color, it's a known issue. It certainly doesn't work in Indigo, it certainly works in Lunar, but I never tested in either Kinetic or Melodic.
About teleop, it was created by YugAjmera and I never tested it, so you might have better luck asking about it there.
The .sh files also certainly doesn't work in Indigo and certainly works in Lunar. I was recently investigating how to make them work in Indigo to test something else.
As for the rest, I'll have to take a further look and make some tests, which might take a while.

I've decided to use an old HD to have several different Ubuntu distros in order to have easy access to several different ROS distros, starting from Indigo towards the newer ones.
Testing this project on Kinetic, I had the same problem that you did. I managed to fix it by installing every package that is listed by apt-cache search ros-kinetic-. Most likely most of them aren't needed, but I lack the time to find out which is which.
Please test it out and, if it works, you can close the issue already.
During this, I had an idea to make some breaking changes to this repository, like having some branches only contain the project for standalone Gazebo and other branches only containing the project for ROS. For the latter, make separate branches if there are significant differences regarding different ROS distros.
I'll look into it in the following days and will push some commits regarding the color and the movement scripts.

I just remembered. It's because I stopped working with ROS that I forgot some pretty basic information. It's even written in the read me!
The file package.xml lists the project's dependencies. Each dependency can be met by running apt-get install ros-<distro>-<dependency-name>, replacing _ by - in the dependency name when needed.


feat: Left-hand Navigation: Group stub packages under parent

See cl/561379976 for staging link.

@alicejli : Staging complete. Ready for final review.


Admin Portal

Ready for review.

@mjudeikis could you take a look through this please?

@mjudeikis review feedback addressed with additional commits, ptal again.

/azp run E2E

Tested in INT - merging!


ISR versions of mutex_try_enter and mutex_exit

#1453 Extends mutex API to support holding the spin lock. This can be used to prevent deadlock from competing core.

Note this does cause possible priority inversion, if used incorrectly.

Does disabling interrupts on core 1, disable them on core 0? This patch does not cover the case of core 0 ISR and core 1 ISR racing against each other. In which case the patch is not worth much. (Even though this is horrible practice.) This would be new behavior and is capable of creating an issue like 1453.
Note the ISR version is capable of creating a slight blocking behavior in non ISR version, which may need to be documented.

See #1580

What happened to my changes? spin_lock_blocking is not supposed to be there.
Note a queue makes more sense.

if you want a mutex which holds the spin lock, use critical_section


fix(docs): update docs to point to right GPU docs

The branch is being deprecated and docs will be updated in https://github.com/google-research/t5x/pull/1385/files#diff-4e75ae9dd8ace0dc2db4d1a3916dca63f59ceb595160f756f35710a0f5efd482


Refresh token

How do you get the refresh token?
It seems to be null when retrieving $user->refreshToken

If I remember correctly, you need to request at least one scope to receive a refresh_token from the CCP OAuth server.
Be aware that eveonline-socialite (and Socialite in general) do not handle refresh_token completly and you'll need to do some manual work to get a new token from the refresh one.


Support IndexedDB for browser

Background
There was Web SQL in the past, but it has been deprecated since November 2010. Now there are two storage options for the web. One is Web Storage and the other is Indexed DB. And the latter is suitable for large, structured data.
IndexedDB can store a large amount of data asynchronously.  Also, unlike sql.js, data is persisted and continue to work while offline. But we can not use SQL for it. So it is the right place where GlueSQL can come in and do a good job.
Consideration
There won't be much difference in implementation with other storage like memory storage. But there are two things we should consider before we start.

JS Binding
We need a way to interact with IndexedDB API. For that, rexie might be a good option.
Schemaless Data
@panarch is planning to rewrite some of the code at the execution layer in GlueSQL. So works before it might need to be rewritten to match the new Storage layer.

Plan

Implement Store and StoreMut trait

https://github.com/gluesql/gluesql/blob/1f0f84d59102b5bda01dad7fefeb84bcc7949d67/core/src/store/mod.rs#L74-L98

Developer of rexie here, I think a better alternative of rexie for low level use cases is idb.

resolved in #1067


How do I train a model on my own datasets

Hello, thank you very much for the author can open source such an excellent algorithm model, but I have a few questions to consult you:
I have found that using the pre-training weights provided by the model does not work very well in image restoration tasks in other scenes. What should I do if I want to train a special model on my own industrial datasets? How should the datasets be configured and processed? Can you provide the code for model training?
Looking forward to the author's reply!

Hi, you may find similar issues in the closed issues, e.g., https://github.com/wyhuai/DDNM/issues/3.

Hi, you may find similar issues in the closed issues, e.g., #3.

OK, thanks for your reply ！