add support for decompressing files based on filename extension before opening in read_pdb

Uses filename suffix to detect which compression method to use.  This way PDB files can be read that are not already decompressed.
Addresses #33

Thanks for the PR! There were some issues in the github actions after I upgraded the pre-commit hooks. First it was about the tests for the Windows version but later some Ubuntu versions had that too. I spent some time to investigate the problem but failed to find out (partially because I don't have a Windows machine to test locally). Are the tests passed in your local tests?

Hey Ruibin,
This was just a quick PR I made and I didn't extensively test yet.
Not sure if related to the pre-commit hook failures; but I just realised that my code doesn't work when using .pdb.gz files when using category_names=['_atom_site'] (I was only testing with category_names=['_seq_res']) and that works).
For some reason I get an IndexError.  Let me have a look and see what's wrong


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_f6dom2w82l/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_f6dom2w82l/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_f6dom2w82l/clone/recipe/meta.yaml:42: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_f6dom2w82l/clone/recipe/meta.yaml:42: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_f6dom2w82l/clone/recipe/meta.yaml:42: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_f6dom2w82l/clone/recipe/meta.yaml:42: missing_description: The recipe is missing a description
Errors were found


[DEFECT] CashFlow doesn't work in windows

Defect Description
Describe the defect
What did you expect to see happen?
Cashflow to work in windows
What did you see instead?
it fails when trying to create a directory called aux.
Do you have a suggested fix for the development team?
Update to the newest version of cashflow.
Platform (please complete the following information):

OS: windows

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

Closure checklist passed...
Approved to be closed via #1158


Fix dissonance between function signature parameters and expanded parameter names

Hello! This is a documentation request.
The function signatures are super helpful. So are the "EXPAND PARAMETERS" descriptions. However, it is confusing to compare the two (the signature and the expanded parameters) because the argument names differ.

Thanks for the thorough response, @CrossEye ! Great linked resources as well. Definitely a newcomer, but you have me convinced on Hindley-Milner.


feat: support wasm environments for postgresql

This disables the home directory loading for WASM environments, as a filesystem to load files from is not available, as well as conditionally adding home on non WASM environments. With this, using sqlx-postgres is possible on wasm32-unknown-unknown targets.

I think there's actually quite a bit more work involved getting wasm32-unknown-unknown working properly -- I'll investigate in this PR when I have some more free time available 🕵

Yeah, you also can't dial a TCP connection from normal Webassembly which unfortunately makes this a non-starter.
You'd have to compile for wasm32-wasi which is not a stable target IIRC and not supported in most WASM environments.


Allow bit masks in searches

Unless I'm missing it in the program and the code, we can't search for a string like "07 00 ## 08 00" where ## is any hex character. Can this be added?

Closing. Over 5 years old, and there are no plans to add this feature. If anyone is interested in developing this feel free to submit a PR!


what are FASTFortranPointer, FASTFortranRecord, FASTFortranStructure ?

Belongs to FAST-Esope ?

If we don't need them, let's remove them for now.
Better to keep the meta-model as minimal as possible.
Having them without knowing what they do or being able to import them is no use anyway.

merged


Tyle8552/animate 3d expressions

updates heading expressions for 3D

@hardikc07 Can you review this on mac/linux?

Checked w/ Hardik, sample working on Linux. Mac has another issue impeding this but should work.


[WIP] Sort regions alphabetically

This has been bothering me
Downside: extra work on the backend for every user call [O(n) where n is number of countries ~200, so minimal work, but work nonetheless] for essentially no real gain
Upside: Looks neater and satisfies my OCD
Potential future improvement: Store them sorted in the database. People don't tend to visit new countries too often, and it's even rarer that they unvisit them, so one time work per user/insert for permanent sorting. Honestly, every user could have a bit vector of size ~200. Might even be more space efficient than the current built-out model, though would require non-trivial dev work every time a country is added to/removed from existence.
Backend checklist

[ ] Formatted my code by running autoflake -r -i --remove-all-unused-imports src && isort . && black . in app/backend
[ ] Added tests for any new code or added a regression test if fixing a bug
[ ] All tests pass
[ ] Run the backend locally and it works
[ ] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Doing it in the backend is fine since the number of regions is upper bounded. We could also just do it in the database (you can define the sort order in the SA relationship)

Doing it in the backend is fine since the number of regions is upper bounded. We could also just do it in the database (you can define the sort order in the SA relationship)

Oh cool, I'm going to do that. I'll get to learn something new.

O(n) where n is number of countries

sorting is O(n log n)
sorry my OCD just needed to get this corrected :p

😂 Fair point.
But really, it's constant time, since n ~200


Laravel 5.6 Shift

This pull request includes the changes for upgrading to Laravel 5.6. Feel free to commit any additional changes to the shift-6476 branch.
Before merging, you should:

Checkout the shift-6476 branch
Review all pull request comments for additional changes
Update your dependencies for Laravel 5.6
Run composer update (if the pre-scripts fail, add --no-scripts)
Thoroughly test your application

If you need help with your upgrade, check out the Shift Human Services or join the Shifty Coders Slack room to get answers to all your Laravel questions.

⚠ Shift could not upgrade the following project files since they differed from the default Laravel version. You should compare these project files against the default Laravel 5.6 versions and merge any changes:

[ ] .env.example

⚠ Shift could not upgrade the TrustProxies Middleware since it differed from the default Laravel version. You should compare your file to the 5.6 version and upgrade your project accordingly.

❌ Shift could not upgrade the following configuration files since they differed from the default Laravel version. You will need to compare these configuration files against the Laravel 5.6 configuration files and merge any changes:

[ ] config/app.php
[ ] config/services.php

ℹ Laravel 5.6 introduced a logging configuration file. You should ensure the log and log_level settings previously in the config/app.php are now set in config/logging.php.

⚠ Looks like you already have a config/hashing.php. Laravel 5.6 added this as a core config file. You should add the 5.6 version to your project and merge your customizations.

⚠ Looks like you already have a config/logging.php. Laravel 5.6 added this as a core config file. You should add the 5.6 version to your project and merge your customizations.

❌ Laravel 5.6 requires PHP 7.1.3. You have a modified dependency for PHP in your composer.json. Ensure it meets the new version requirement of >=7.1.3.

❌ Laravel 5.6 has a dependency for laravel/scout of ~4.0. You have a modified dependency for laravel/scout in your composer.json. Ensure it meets the new version requirement of ~4.0.

ℹ In Laravel 5.6, Blade will double encode special characters by default. If you would like to maintain the previous behavior of preventing double encoding, you may add Blade::withoutDoubleEncoding() to your AppServiceProvider boot method.

ℹ The previously deprecated optimize Artisan command has been removed. With recent improvements to PHP itself including the OPcache, the optimize command no longer provides any relevant performance benefit.

ℹ All of the underlying Symfony components used by Laravel have been upgraded to Symfony 4.0. If you are directly interacting with Symfony components, you should review the Symfony change log and update your application accordingly.

ℹ Laravel 5.6 uses Bootstrap 4 by default. Shift did not update your front-end resources or dependencies as this could impact your UI. If you are using Bootstrap and wish to continue using Bootstrap 3, you should add Paginator::useBootstrapThree() to your AppServiceProvider boot method.

ℹ If you have manually upgraded your app from previous versions of Laravel, you should run the Laravel Linter to ensure your app is fully upgraded to the latest version of Laravel.


Client Allocation Request for: bjwswang

Client Allocation Questions
Core Information

Name: bjwswang
Website / Social Media: https://github.com/bjwswang
DataCap Requested: 1TiB
Addresses to be Notarized: f1lszblkedtcee3kqhk7uxluit23xhn4opkzzobhi
Notary Requested: philippbanhardt

Hey @bjwswang - thanks for your request. Can you kindly add more information / context here on:

yourself , e.g. include links to social media / news mentions etc
your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc
type, size, source and your relationship with the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data, who owns the data / license, etc
how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

Thanks


[Beta] Preview is improper during JS warning

Issue Description
On typing/editing, a JS warning comes up in the preview pane which distorts the preview. Here are some screenshots to clarify that. I'm seeing this issue since PR #16737 was merged.
When there's no JS warning: The preview is complete as all the eleven hellos are printed properly.

During JS warning: The tenth and eleventh hellos are not properly printed.

Browser Information

Browser Name, Version: Chrome & Firefox.
Operating System: Windows 10 Pro 32 Bit.
Mobile, Desktop, or Tablet: Desktop.

@raisedadead Input on this?

Thanks @vkWeb !

@raisedadead Any idea how can this be solved? I tried tweaking the CSS but during JS warning it adds up a second scroll bar which is not desired.


Discord link points to wrong server

We need to fix the discord link on the website. It is sending people to the wrong discord!

Sure @drakedanner. I'll fix this.


V4 beta

Beta version for API V4. Only matters, documents and contacts tested. Added refresh token support.

Added folder support and document upload support.


Logs don't print to console with default DatadogLoggerConfiguration

Describe the bug
Logs do not print to console when customConsoleLogFunction is not specified.
Reproduction steps
WidgetsFlutterBinding.ensureInitialized();
final dataDogConfig = DatadogConfiguration(
  clientToken: 'CLIENT_TOKEN',
  env: dotenv.env["ENV"] ?? 'dev',
  site: DatadogSite.us1,
  nativeCrashReportEnabled: true,
  loggingConfiguration: DatadogLoggingConfiguration(),
  rumConfiguration: DatadogRumConfiguration(
    applicationId: 'APPLICATION_ID',
  ),
);
final originalOnError = FlutterError.onError;
FlutterError.onError = (details) {
  DatadogSdk.instance.rum?.handleFlutterError(details);
  originalOnError?.call(details);
};
final platformOriginalOnError = PlatformDispatcher.instance.onError;
PlatformDispatcher.instance.onError = (e, st) {
  DatadogSdk.instance.rum?.addErrorInfo(
    e.toString(),
    RumErrorSource.source,
    stackTrace: st,
  );
  return platformOriginalOnError?.call(e, st) ?? false;
};
await DatadogSdk.instance.initialize(dataDogConfig, TrackingConsent.granted);
final loggerConfiguration = DatadogLoggerConfiguration();
logger = DatadogSdk.instance.logs?.createLogger(loggerConfiguration);
logger?.debug("A debug message");
logger?.info("Some relevant info");
logger?.warn("An important warning…");
logger?.error("An error was met!");

However, if you set customConsoleLogFunction logs do appear.
  final loggerConfiguration = DatadogLoggerConfiguration(
    customConsoleLogFunction: simpleConsolePrintForLevel(LogLevel.debug),
  );

SDK logs
There aren't any console logs which is the problem.
Expected behavior
Console logs are expected to appear
Affected SDK versions
2.3.0
Latest working SDK version
No response
Did you confirm if the latest SDK version fixes the bug?
Yes
Flutter Version
3.19.3
Setup Type
Flutter Application
Device Information
iOS and Web
Other relevant information
No response

Hi @nwarriner,
Correct here as well. We accidentally overwrite the default with null if no print function is provided during configuration.
I'll try to get this fix ASAP.


Wording changes to docs

I eagerly read through the (excellent, thanks!) documentation for channels, and had to re-read one or two sentences. So being a good citizen, I'm suggesting a few fixes here and there.

Thanks for the quick review, and sorry for the late response. I pushed a new commit, hopefully addressing your comments. Could you have another look?

Perfect, thanks!


Install dnvm into custom directory

is it possible to have dnvm install into a custom place that is not ~/.dnx

@glennc @BrennanConroy ?

You could put it where ever you want, but you'll have to do it yourself, also make sure to add source <path>/dnvm.sh to your bash_profile or zschrc or whatever terminal you use.

sounds good thanks


[SPARK-36452][SQL]: Add the support in Spark for having group by map datatype column for the scenario that works in Hive

What changes were proposed in this pull request?
Add the support in Spark for having group by map datatype column for the scenario that works in Hive.
In hive this scenario works fine
describe extended complex2;
OK
id                  string
c1                  map<int, string>
Detailed Table Information Table(tableName:complex2, dbName:default, owner:abc, createTime:1627994412, lastAccessTime:0, retention:0, sd:StorageDescriptor(cols:[FieldSchema(name:id, type:string, comment:null), FieldSchema(name:c1, type:map<int,string>, comment:null)], location:/user/hive/warehouse/complex2, inputFormat:org.apache.hadoop.mapred.TextInputFormat, outputFormat:org.apache.hadoop.hive.ql.io.HiveIgnoreKeyTextOutputFormat, compressed:false, numBuckets:-1

select * from complex2;
OK
1 {1:"u"}
2 {1:"u",2:"uo"}
1 {1:"u",2:"uo"}
Time taken: 0.363 seconds, Fetched: 3 row(s)

Working Scenario in Hive -:

select id, c1, count(*) from complex2 group by id, c1;
OK
1 {1:"u"} 1
1 {1:"u",2:"uo"} 1
2 {1:"u",2:"uo"} 1
Time taken: 1.621 seconds, Fetched: 3 row(s)

Failed Scenario in Hive -:
When map type is present in aggregated expression
select id, max(c1), count(*) from complex2 group by id, c1;
FAILED: UDFArgumentTypeException Cannot support comparison of map<> type or complex type containing map<>.

But in spark where the group by map column failed for this scenario where the map column is used in the select without any aggregation, The one that works in hive.
scala> spark.sql("select id,c1, count(*) from complex2 group by id, c1").show
org.apache.spark.sql.AnalysisException: expression spark_catalog.default.complex2.`c1` cannot be used as a grouping expression because its data type map<int,string> is not an orderable data type.;
Aggregate [id#1, c1#2], [id#1, c1#2, count(1) AS count(1)#3L]
+- SubqueryAlias spark_catalog.default.complex2
 +- HiveTableRelation [`default`.`complex2`, org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe, Data Cols: [id#1, c1#2], Partition Cols: []]
at org.apache.spark.sql.catalyst.analysis.CheckAnalysis.failAnalysis(CheckAnalysis.scala:50)

Why are the changes needed?
There is need to add the this scenario where grouping expression can have map type if aggregated expression does not have the that map type reference. This helps in migrating the user from hive to Spark.
After the code change
scala> spark.sql("select id,c1, count(*) from complex2 group by id, c1").show
+---+-----------------+--------+
| id|               c1|count(1)|
+---+-----------------+--------+
|  1|         {1 -> u}|       1|
|  2|{1 -> u, 2 -> uo}|       1|
|  1|{1 -> u, 2 -> uo}|       1|
+---+-----------------+--------+

Does this PR introduce any user-facing change?
No
How was this patch tested?
Added the unit test and also tested using spark-shell the scenario

Can one of the admins verify this patch?

I thought @maropu is still working on this? (https://github.com/apache/spark/pull/32552)

I thought @maropu is still working on this? (#32552)

I was not aware, that there is already a jira for this map issue, Yes this PR (https://github.com/apache/spark/pull/32552) will fix the use case that I am trying in this PR.


No useful log in some error conditions

In conditions like #100 there seem to be no information available in the logs.
cc @jlhawn

The logging situation is a little thin and #86 was filed to address this. Without modifying the registry, the error messages provided in the response should provide some insight.
Closing this a duplicate of #86.


Olt postgis sub queries

Additional test cases and minor code formatting changes to #478

Pull Request Test Coverage Report for Build 1397

17 of 19 (89.47%)  changed or added relevant lines in 2 files are covered.
7 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.1%) to 44.254%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

provider/postgis/util.go
10
12
83.33%

Files with Coverage Reduction
New Missed Lines
%

provider/gpkg/binary_header.go
2
100.0%

cmd/tegola/cmd/cache.go
5
27.07%

Totals

Change from base Build 1395:
0.1%

Covered Lines:
5183

Relevant Lines:
11712

💛  - Coveralls


partly resolve #759

some versions of bash (e.g. 4.3.11) complain about clobbering
/dev/null, use bash's explicit >| operator to ignore noclober
not sure if the original #759 report has the same cause
https://github.com/rbenv/rbenv/issues/759#issuecomment-289326891

Good catch; thank you!


Target Pulumi Destroy on Multiple Targets Results in Orphaned Physical Resource

Hello!

Vote on this issue by adding a 👍 reaction
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)

Issue details
When using a targeted destroy of two related resources, such as listener and a target group, both resources are removed from Pulumi's state but only one resources is physically destroyed resulting in an orphaned resource.

Steps to reproduce

Create a load balancer, listener, and target group
Attach the target group tot he listener and listener to the load balancer
Use a targeted destroy for both listener and target group

Expected:  Listener and target group should be physically destroyed and removed from the stack's statefile.
Actual:  Both are removed from the stack's statefile but only the listener is physically destroyed, resulting in an orphaned target group
Command: pulumi destroy -t {target_group_urn} -t {listener_group_urn}
Code:
import * as aws from "@pulumi/aws";
import * as pulumi from "@pulumi/pulumi";

export = async () => {
    const nlb = new aws.lb.LoadBalancer("network-load-balancer", {
        internal: true,
        loadBalancerType: "network",
        subnets: ["subnet-id],
        enableDeletionProtection: false,
        ipAddressType: "ipv4"
    });

    const protocol = "TCP";

    const tg = new aws.lb.TargetGroup("target-group", {
        name: "tg-1",
        port: 80,
        protocol: protocol,
        targetType: "ip",
        vpcId: "vpc-id",
        healthCheck: {
            protocol: protocol,
            port: "80",
            matcher: undefined
        }
    }, {
        deleteBeforeReplace: true
    });

    const listener = new aws.lb.Listener("listener", {
        protocol: protocol,
        port: 80,
        loadBalancerArn: nlb.arn,
        defaultActions: [{
            type: "forward",
            targetGroupArn: tg.arn
        }]
    }, {
        deleteBeforeReplace: true
    });

    return {
        url: alb.dnsName
    }
}

Using a targeted destroy on the listener also removes the target group from the statefile and orphans the target group.
pulumi destroy -t {listener_urn}


Changing Nuget Artifact greet() Method Access Role

Pull Request Description
This pull request closes #1662
Acceptance Test

[x] Building the code with mvn clean install -Dintegration.tests still works.
[x] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[x] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[x] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[x] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[x] No

@anki2189 can you please rebase it with latest master, this should fix the Unit Tests

Thanks for these fixes, @anki2189 , and thanks for reviewing, @sbespalov ! :)


Fix hespori KC tracking

Description:
Hespori only giving 1 kc per harvest, regardless how many are planted.
Changes:

Adds patchType.lastQuantity to the numKillsToAdd parameter of incMonsterScore

Other checks:

[ ] I have tested all my changes thoroughly.

you can only have 1 hespori planted at a time..?

You can plant up to 6 hespori on BSO with increased patches (master farmer outfit, scroll of farming).


Add unmodifiable bool flag to Iterable.toList.

Currently we can create a growable or fixed-length list with Itearable.toList. Consider making it possible to make an unmodifiable list as well.

Does every toList implementation need to check and throw on this case?
For the purpose of whole program type analysis in dart2js, it is preferable if as many methods as possible return objects of a single type. Then the type becomes less data-dependent and more optimization opportunities are found. So .toList(), .toFixedList(), .toUnmodifiableList() would help produce better optimized code. Since we already have growable:, I don't think unmodifiable: makes things worse for type analysis.
Would  new List<T>.unmodifiable(...).toList(unmodifiable: true) be defined to return the receiver?
This would be valuable in freezing data without creating copies.

I personally liked @rakudrama suggestion on another thread about different methods:
abstract class Iterable<T> {
  List<T> toList();
  List<T> toFixedList();
  List<T> toUnmodifiableList();
}

That being said, unmodifiable lists aren't used that often because of the performance overhead of having to wrap a list, so I don't know if making it simpler to create one is a great pattern for Dart.

Changing the Iterable API is a non-starter at this point.


Supply chain and delivery can set serviceAccountName

Closes #301

This needs documentation.
At least in the reference doc
(I don't think the serviceAccount in the workload has been documented there, this would be a good time to add it)

Rebased on main and force-pushed.
Re-requesting a review because I switched from using serviceAccountName to serviceAccountRef per the discussion in #301


404-error-removed_farmer_service

Hello @apu52 @anuragverma108 @heysagnik
Title and Issue number

Title :Farmers service webapp 404
Issue No. : #898
Code Stack : Html css Javascript
done editing in html file shifted css code and corrected tons of error in code, improved padding n all
import loads of js and css files and made a lots of changes.
Obliged if you could mark it as a level 2.
Close #898

Please assign this under GSSOC and the difficulty label as well!
Video (mandatory)

https://github.com/apu52/METAVERSE/assets/129287425/ff4d756e-af4e-400e-b23d-04c9a9c925ba
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[x] I'm a GSSOC'24 contributor
[ ] I'm a SSOC'24 contributor

@sakshiglaze Your contributions are really valuable for this repo .But As I have merged your 3 PR in a row , so just want to inform you that there is a high limit of maybe 150 points which u can gather from 1 project in GSSOC. So just keep that in mind. It will be helpful for u
Otherwise keep contributing. Looking forward to see more of your contributions.


DC Plate App

Based on the 2D cylinder app, we need a plate app! The main impact here is examining the impact of shape on the charges and sensitivity

Parameters:

inclination
sigma1
sigma2
dx
dz
center
A, B electrode locations

x is the reference
To view:

currents
charges
potentials
potential along a line at the surface (similar to the DC layered earth app)

Also want to integrate the charges along the surface to get a total charge for each face of the plate

What is the total positive charge built up. With the negative charge we now have a dipole. What is the distance between the charges and the magnitude of the dipole?  Each of these will make an electric dipole. What is the strength of the dipole.

also need to be able to replace the plate with a dipole

Repository link to DC lab (with 2D cylinder anomalie and 2-layer Earth with Fudge Factor):
https://github.com/ubcgif/gpgLabs/tree/master/DC
Associated Binder:
http://mybinder.org/repo/ubcgif/gpgLabs/notebooks/DC/DCLab.ipynb

Working 2D and 2.5D versions of this app in em_examples.


Remove afni_vcheck from container?

It seems afni_vcheck will run at least once somehow during creating a single subject pipeline, which can be really annoying when https://afni.nimh.nih.gov/pub/dist/AFNI.version doesn't response.  In my case, the whole computing cluster waited for hours before any real job started running.  This version check is unnecessary for containers, because the AFNI version is already specified in the Dockerfile, and we don't expect to upgrade AFNI during running fmriprep even if a newer version is available.
I removed /usr/lib/afni/bin/afni_vcheck in my image and that fixed this problem (with an additional warning though).

This is being resolved in nipy/nipype#2196 and #723.

Great!


Where is pico_micro_ros_example.uf2?

I follow the instructions (3 times) but I cant find the file "pico_micro_ros_example.uf2" inside the build folder (of course after make)

I have replicated the instructions and the file is in build/pico_micro_ros_example.uf2.
Could you provide more details?

Even I am facing the same issue. I followed the steps as mentioned in README

Could you provide more details? Steps, build output?

I am using Fedora-35.
I installed all the packages using dnf and built gcc-arm-none-eabi from source.
Then I cloned the repository in my home directory. Made a build directory inside the pico-sdk dir.
Then i ran
cmake ..
and
make all
Here is the output:
~/pico-sdk/build> make all
[  0%] Built target bs2_default
[  0%] Performing build step for 'ELF2UF2Build'
Consolidate compiler generated dependencies of target elf2uf2
[100%] Built target elf2uf2
[  0%] No install step for 'ELF2UF2Build'
[  0%] Completed 'ELF2UF2Build'
[  1%] Built target ELF2UF2Build
[  1%] Built target bs2_default_padded_checksummed_asm
Consolidate compiler generated dependencies of target pico_stdlib_test
[  8%] Built target pico_stdlib_test
Consolidate compiler generated dependencies of target pico_time_test
[ 15%] Built target pico_time_test
Consolidate compiler generated dependencies of target pico_divider_test
[ 21%] Built target pico_divider_test
Consolidate compiler generated dependencies of target pico_divider_nesting_test_with_dirty_check
[ 28%] Built target pico_divider_nesting_test_with_dirty_check
Consolidate compiler generated dependencies of target pico_divider_nesting_test_with_disable_irq
[ 35%] Built target pico_divider_nesting_test_with_disable_irq
Consolidate compiler generated dependencies of target pico_float_test
[ 42%] Built target pico_float_test
Consolidate compiler generated dependencies of target pico_double_test
[ 49%] Built target pico_double_test
Consolidate compiler generated dependencies of target kitchen_sink
[ 57%] Built target kitchen_sink
Consolidate compiler generated dependencies of target kitchen_sink_extra_stdio
[ 69%] Built target kitchen_sink_extra_stdio
Consolidate compiler generated dependencies of target kitchen_sink_copy_to_ram
[ 78%] Built target kitchen_sink_copy_to_ram
Consolidate compiler generated dependencies of target kitchen_sink_no_flash
[ 86%] Built target kitchen_sink_no_flash
Consolidate compiler generated dependencies of target hardware_pwm_test
[ 93%] Built target hardware_pwm_test
Consolidate compiler generated dependencies of target cmsis_test
[100%] Built target cmsis_test

I can rebuild it if it helps.

have you tested make instead make all?

Yeah

CMAKE Output:
~/pico-sdk/build> cmake ..
PICO_SDK_PATH is /home/blebot/pico-sdk
Defaulting PICO_PLATFORM to rp2040 since not specified.
Defaulting PICO platform compiler to pico_arm_gcc since not specified.
-- Defaulting build type to 'Release' since not specified.
PICO compiler is pico_arm_gcc
-- The C compiler identification is GNU 10.3.1
-- The CXX compiler identification is GNU 10.3.1
-- The ASM compiler identification is GNU
-- Found assembler: /usr/share/gcc-arm-none-eabi-10.3-2021.10/bin/arm-none-eabi-gcc
Defaulting PICO target board to pico since not specified.
Using board configuration from /home/blebot/pico-sdk/src/boards/include/boards/pico.h
-- Found Python3: /usr/bin/python3.10 (found version "3.10.1") found components: Interpreter
TinyUSB available at /home/blebot/pico-sdk/lib/tinyusb/src/portable/raspberrypi/rp2040; enabling build support for USB.
-- Configuring done
-- Generating done
-- Build files have been written to: /home/blebot/pico-sdk/build

As you can see, in Ubuntu it is working in our CI: https://github.com/micro-ROS/micro_ros_raspberrypi_pico_sdk/runs/4891919559?check_suite_focus=true
Also, you have the steps here: https://github.com/micro-ROS/micro_ros_raspberrypi_pico_sdk/blob/8dd69132bc16ec0d76f61e51a442ee26329b3b8f/.github/workflows/ci.yml#L36

Ok! I'll compare the output and zero down on the issue

Any update on this?

Solved here: https://github.com/micro-ROS/micro_ros_raspberrypi_pico_sdk/pull/413


Replace MessageTracker search results table

Problem Statement
After completing the Custom Table component, the existing website tables need to be replaced with the new custom table component. Therefore, this ticket is to replace the table on the MessageTracker search results page.
Need to Know

Link?

Acceptance Criteria

[ ] Table on MessageTracker search results page uses the new custom table component
[ ] Table on MessageTracker search results page has at least the same functionality as it did before replacing with the custom table component

PR: https://github.com/CDCgov/prime-reportstream/pull/9624

Joseph helping to unblock

Unblocked by Joseph and Etan.

PR #9624


Notify a service inside k8s cluster on certificate renewal

I am running Cert manager in a kubernetes cluster using this helm chart https://github.com/jetstack/cert-manager/tree/master/deploy/charts/cert-manager
I can successfully get the certificates I need.
I do the ssl termination on my application and I would like my application to be notified when the certificate is automatically renewed so that application can reload the new certificate.
Is it possible to do that?
Environment details::

Kubernetes version - v1.16.7
Cloud-provider/provisioner - Azure Kubernetes Service
cert-manager version -v0.15.0
Install method - helm3

/triage support

There are efforts in k8s to support some kind of notification mechanism. It doesn't exist yet.
For now, what I do is load an inotify based sidecar to watch the secret for changes and send a signal within the pod to the things that need signaling

There are efforts in k8s to support some kind of notification mechanism. It doesn't exist yet.
For now, what I do is load an inotify based sidecar to watch the secret for changes and send a signal within the pod to the things that need signaling

Is it possible that you can show how you go about this ?

you add a sidecar container like:
        - command:
          - /bin/sh
          - -cxe
          - |
            while true; do
              inotifywait -e delete_self /secret/tls.crt && killall -SIGHUP nginx
            done
          image: pnnlmiscscripts/inotify-tools:latest
          name: inotify
          volumeMounts:
            - mountPath: /secret
              name: secret

and on the pod:
      shareProcessNamespace: true

Many programming languages/frameworks also allow to handle this for you with being able to live-reload certificates when changed on disk. If you have control over the code of the application running I suggest going that route.

I agree that this would be a better route, however I dont have that much time to get familiar with that codebase and even alter the code to handle this the correct way,
But I definitely agree this would be a great change to that application


Hard to select small components when zoomed in

I'm trying to right click and delete the small circle at the top, but it keeps selecting the long channel.
@JLippai

Should be addressed by the selection tool in commit b3eced69b67822855a9530696fc2e4a62ef99c80


[MultiParentchain] add CI test including a Target A&B parentchains

This is not tested yet, although a demo script already exists. It should be a minimal effort to launch a second node with the docker compose and run the already existing script

Astar uses zombienet in CI: https://github.com/AstarNetwork/Astar/blob/298a8e413874b0bcd36fe1316a55217ee3906d4d/.github/workflows/zombienetRpcTest.yml
Moonbeam too:
https://github.com/moonbeam-foundation/moonbeam/tree/master/test


Release latest Readme to npm

@broofa please check this thread: https://github.com/gatsbyjs/gatsby/pull/15407#discussion_r300392112
Seems like the current readme on v1 UUIDs on npm is confusing. Could we release a new bugfix release to get the latest README published on npm as well? As far as I can tell from https://github.com/kelektiv/node-uuid/compare/v3.3.2...master it's only documentation and maintenance changes, so bugfix release should be fine.

@broofa may I bring this to your attention again? Would be a pretty low hanging fruit and apparently resolve some confusion (see linked thread above).

Published


not updating values

running an mosquitto broker on an RP3b with OH gives me the following output:
'''
[16:55:08] openhabian@smarthome:~$ python3 /opt/miflora-mqtt-daemon/miflora-mqtt-daemon.py
Xiaomi Mi Flora Plant Sensor MQTT Client/Daemon
Source: https://github.com/ThomDietrich/miflora-mqtt-daemon
[2018-11-16 16:55:13] Connecting to MQTT broker ...
[2018-11-16 16:55:13] MQTT connection established
Adding sensor to device list and testing connection ...
Name:          "FlowerCare"
Internal name: "FlowerCare"
Device name:   "Flower care"
MAC address:   C4:7C:8D:XX:XX:XX
Firmware:      2.7.0
[2018-11-16 16:55:17] Initial connection to Mi Flora sensor "FlowerCare" (C4:7C:8D:XX:XX:XX) successful
[2018-11-16 16:55:17] Announcing Mi Flora devices to MQTT broker for auto-discovery ...
[2018-11-16 16:55:18] Retrieving data from sensor "FlowerCare" ...
[2018-11-16 16:55:18] Result: {"conductivity": 3, "battery": 100, "light": 324, "moisture": 0, "temperature": 19.0}
[2018-11-16 16:55:18] Publishing to MQTT topic "miflora/FlowerCare"
[2018-11-16 16:55:19] Sleeping (300 seconds) ...
'''
But OpenHAB isn't updating any channels. I'm using those:
mqtt="<[broker:miflora/FlowerCare:state:JSONPATH($.anyValue)]"

Had errors in my mqtt configuration, everythings good.
Closing this one.


Meaningful file status

Description

Currently the files under ingestion can have the following status:
INIT = Initializing a file ingestion
IN_INGESTION = Currently under ingestion
ARCHIVED = File moved to Archive
COMPLETED = File verified in Archive
READY = File ingested, ready for download

Proposed solution

Simpler and more precise semantics should be utilised, for example:
UPLOADED -> INGESTED -> VERIFIED -> ARCHIVED

we need to redo the schema and see how to migrate the db.

we will have a new db schema and this will be solved there.


bigint as input value

modularPower throws an error on bigint input. It would be nice to be able to use it with bigints.

@skaunov Send the code that didn't work

I wrapped one of the integers into BigInt() and it returned the result! Just "TypeError: Expected an integer, got bigint" wasn't verbal enough for me to check that there is an integer among the arguments, thought the function doesn't accept bigint.

Ok, gonna replace the single agnostic function with seperate functions for handling numbers and bigints.


Do you have a minimal environment YAML file?

I am unable to recreate your environment using your zsd_environment.yml file, there are various dependency errors I'm getting.
The environment file looks like an export of your current environment, and so there are probably lots of packages, e.g. Anaconda default installs, that aren't needed, and so cleaning up the environment file might help resolve the issue.
In other words, do you have an environment file that lists just the absolute minimum set of packages that are needed?

I don't think I can get this repo to work b/c of differences in CUDA environments and other unresolvable issues, so I'm closing this out


ReactPHP or similar non-blocking I/O implementation

I have been considering logging services including GetSentry and BugSnag. However, I can't come to terms with the default monolog implementation sending synchronous requests. If I am doing offline development, every innocuous error will cause Monolog to timeout while trying to report to the service which can be an annoyance.
Error reporting can be a nasty performance bottleneck, especially if I am dealing with high volume servers where milliseconds matter and the latency of the error reporting server alone can trump the load times of the entire app.
I am unable to justify using this in production out-of-the-box due to these constraints. Any alternative guidance would be helpful.

There are two ways to do performant logging IMO:

Log to filesystem only, and then use something else (logstash, heka, ..) to read the logs and send them off to an aggregation system in the background.
Log using UDP sockets, so that there is no waiting for ACKs, and no problem if the remote host is down (you're just losing logs, but that's usually better than slowing things down, depends on the app requirements).

All log platforms that require you to use TCP/HTTP to send them logs are completely missing the point IMO. They might be acceptable to log errors because those requests are dead in the water anyway and shouldn't occur too often anyway, but if you do logging of events and such, you really want something that performs.
As for ReactPHP/Async, yes we could but that's quite an intrusive requirement and needs tighter integration with the app, so it might be best researched in an external package than in core monolog IMO.

Was digging around, interested in making local stream-based logging a bit faster: found https://github.com/amphp/log, which matches the OP questions.


Implement toggle switch

Do you plan on adding a toggle switch to the widgets. Quite a handy one for a modern UI. Just like you find in android UI or Gtk :)

??

It can be easy done by creating new style for checkbox
Get drawables from Android, and specify them in theme
Unfortunately, I have no time to implement it right now

Ok. Hoping it will added sometime in the future.

Implemented - SwitchWidget

Great work!


When will the fused version of stable adamw coms out?

Exactly, this is not a issue.
In my experiments, stable adamw gives a much better stability on loss curve. In order to training faster, I want to use a fused version, but it really stumped me because I don't have any knowledge in CUDA programming.
I tried to modify the fused_adam.cu, but its computations are all about element-wise, and its unknown to implementing a tensor-wise RMS.

I spoke to the author of Stable Adam W and he stated that there would be a minimal performance boost from using the fused kernel. He is also not able to release the fused version until a bug is fixed in the Triton repository.

I see, thanks.


[CI] Avoid content-length request in test data download

To avoid CI out-rage due to download limitations which happened today cc @icemelon9
Perhaps we need a smarter way to verify the correctness of the file(e.g. including some local meta data).

cc @yzhliu


Is this a production ready project that is maintained?

some of your wording makes it seem like this is not a good choice for production projects.
Should this wording be updated?  It may be outdated or less applicable, it seems like from the changelog and updates and other stuff it might be a good choice from production projects.
specifically in italics
'Note: This project serves as a training exercise for myself in order to better understand the inner workings of Flutter and learn how to write custom Widgets, Elements and RenderObjects from scratch. As a result, the API might change heavily and I may not know about best practices or optimal solutions for many problems.'
I appreciate the idea that of a disclaimer your solution might not be optimal but perhaps a slight wording change especially if it is more solid of a package than it was at the beginning
Thanks

You can certainly use this package in production as it works just fine in its current state. However, if I have enough time to develop this project any further, I cannot guarantee a stable API, which is often seen as a requirement for the label production ready.
I hope this answers your question. If not, feel free to reopen the issue at any time.


It's possible to type the letter e in number fields

Description
It's possible to type the letter e in number fields.
Expected Behavior
Only numbers should be accepted
Actual Behavior
e is accepted
Steps to Reproduce
Open the preferences form and type e into one of the number fields (eg Target confirmations (fast)`
Your Environment

Zap version: next/0.5-pre
Operating System and version: all

Fixed via https://github.com/LN-Zap/zap-desktop/pull/2494


fix for reading the interface name on Apple Silicon

On M1 the interface object is now reported by IOObjectGetClass() as a IOUSBHostInterface.
And the USB Interface Name property is gone. But there is a kUSBString property with the same content.
Also some cleanup to remove search_for_locationID_in_interfaces since I'm more confident it is rendered useless now (and would need to be fixed for M1 anyway). If I could know why it was done that way to begin with. Maybe there are or were cases on older macOS where the interface and serial objects are not in the same hierarchy.

Updated to look for the interface name outside of the hierarchy on 10.12 and 10.13, using a rewritten scan function.
I don't have a 10.14 laying around but hopefully it's enough in the Venn diagram between 13 and 15 to work without needing special code.
I'll put my notes on the differences in the IO registry (that can be explored with ioreg -l).
10.12 - 10.13

serial ports have each a separate IOUSBInterface parent (as per the string returned by IOObjectGetClass())
that parent has a locationID (one per device) and bInterfaceNumber (one per port) but no name
there is separate list of IOUSBInterface with locationID and bInterfaceNumber, and USB Interface Name

So we match them using both properties in the scan_interfaces()/search_in_interfaces functions.
10.15 - 11.0 intel

serial ports have each a separate IOUSBInterface parent
that parent has a USB Interface Name and a kUSBString containing the name

11.0 M1

serial ports have each a separate IOUSBHostInterface parent (IOUSBInterface deprecated)
that parent has a kUSBString containing the name (USB Interface Name deprecated)


Database Initialization

When I use the firebase instance that you provided, it works.  After setting up my own firebase instance, and putting the security rules in from the readme file, I get an error when I launch the app.
Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op.

Is there anything I should do to initialize the database?

With an empty database, it redirects to the 404 page. I was able to strip out the contents of Posts.jsx, which allowed me to create a user, then make a post.  When I restore Posts.jsx, it functions properly.  That is why I thought that maybe the initial database state may be causing issues with the app.

Thanks for the report! This is now fixed.


ENH: add fetch-busco-db action and update evaluate-busco accordingly

What's new

Adds action fetch-busco-db that fetches busco lineage data and stores it in an artifact.
Updates the evaluate-busco such that it can take a busco database artifact as input.
Closes #122

### Blocked by...
- [ ] depends on qiime2/qiime2#755
- [ ] depends on qiime2/q2-types#312

Run it locally

Assuming you already have a local copy of q2-moshpit that is installed in your virtual environment.

cd q2-moshpit
gh pr checkout 143

Let's get you some data to play with:

cd <download here>
qiime moshpit fetch-busco-db --p-virus False --p-prok True --p-euk False --o-busco-db busco_prok_db.qza --verbose
wget https://polybox.ethz.ch/index.php/s/slGyW1uQbyxmyb9/download -O mags.qza

Test it out!

qiime moshpit evaluate-busco --i-bins mags.qza --i-busco-db busco_prok_db.qza --p-lineage-dataset bacteria_odb10 --o-visualization busco.qzv --verbose --p-cpu 6

qiime tools view busco.qzv

Running the tests
pytest -W ignore -vv --pyargs q2_moshpit

Same here, please update with changes from #148, @Sann5 🙏


Gbf/tenant workspace

Description
This PR includes:

management of inconsistent workspaces, by adding the field inconsistenWorkspaces to the status of the tenant
allows for the automatic unsubscription of a tenant from a workspace when the latter gets deleted
adds automatic generation of the needed Kubernetes resources to allow the correct consumption of templates and instances
renames the admin concept to powerUser
adds a description equal to the pretty name of a workspace for better visualization in keycloak

How Has This Been Tested?
At the current state, this PR doesn't add the proper testing, just the minimum for the test to run properly

Gabriele, this PR looks great!
I would suggest to document a little bit more what you intent with "inconsistent" workspace, which does not look obvious to me.

/merge


wl-gammarelay-rs: 0.4.1 -> 1.0.0

I changed the platforms to unix, because it works on freebsd as well.
It would probably work on any platform with dbus and wayland, but unix seems enough.
The cargo outputHashes on rustbus* didn't change.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353023

x86_64-linux

:white_check_mark: 1 package built:

wl-gammarelay-rs

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4796

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353023

x86_64-linux

:white_check_mark: 1 package built:

wl-gammarelay-rs

aarch64-linux

:white_check_mark: 1 package built:

wl-gammarelay-rs

x86_64-darwin

:x: 1 package failed to build:

wl-gammarelay-rs

aarch64-darwin

:x: 1 package failed to build:

wl-gammarelay-rs


Extend selection in patterns shold extend to comma

In the following code
let _ = (krate, _crate_def_map, module_id);
let _: (krate, _crate_def_map, module_id) = ();

if the cursor is on first _crate_def_map and we press extend selection twice, we get

Note how comma is selected. In the second case, we get

That is, comma selection step is missing. This should be fixed.
See also a similar recent issue for mentoring instructions: https://github.com/rust-analyzer/rust-analyzer/issues/2055

Note, while we are at it, it would be cool to change the implementation to try to select the following comma and whitespace, and on the previous one.

Hello @matklad, I'd wish to take on this issue.
I wish to make sure that I we are consistent per your second comment:
In the case of let _: (krate, _crate_def_map, module_id) = ();, by repeatedly pressing the extend selection on _crate_def_map, the expansion should follow the following expansion:
_crate_def_map -- original
_crate_def_map,  -- press once (with white spaces at the end, but not the one at the front)
, _crate_def_map,  -- press twice
(krate, _crate_def_map, module_id)  -- press three times (with white spaces at the end, but not the one at the front)
Am I interpreting it right?

, _crate_def_map,  -- press twice

This step is extra, once we've selected one list item with a delimiter, we should select the whole list next

Hello @matklad, thank you for your patience.
I have fixed the issue (including making sure the delimiter selection prefers the following one instead of previous one). The test I wrote here should verify the behaviour. Please let me know if you like it!


Mr. Wu ,I want to know the Java classes and methods that handle SW6.

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion

Question

What do you want to know?

Skywalking uses SW6 to concatenate each segment. I want to know where is the way skywalking deals with SW6?
I hope you can tell me in your busy schedule. Thank you very much.

The project is maintained by many people, I am just having more time to respond.
Are you asking for these protocols?
https://github.com/apache/skywalking/blob/master/docs/en/protocols/README.md#language-based-native-agent-protocol
What do you mean class and method?


chore: python packages needs to be installed in latest vs code

Issue #, if available:
Description of changes:
Checklist:

[ ] Have you successfully deployed to an AWS account with your changes?
[ ] Have you written new tests for your core changes, as applicable?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

the changes are already merged in  the PR Fix: Fixed Rstudio ec2-user issue, VS python issues #22


user login based navigation guards for CHAPI get/store pages

block CHAPI pages if user not logged in and show link to user login

closes by #343


React-Native 0.59 upgradation issues

I upgraded my react-native version to "^0.59.10" and react version to "^16.8.6". After installing all node modules, getting the error
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>
I build and install the application and ran into following crash
Error: Exception in HostObject::set: Unable to put on NativeModules: Operation unsupported

    This error is located at:
        in t
        in RCTView
        in RCTView
        in t, stack:

Can you please provide some workaround to resolve this issue.

It seems babel or js runtime mismatch the problem.


libzim: 7.2.0 -> 7.2.2

Description of changes
https://github.com/openzim/libzim/releases/tag/7.2.2

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 176618 run on x86_64-linux 1

1 package built:

libzim


[ie/Radio1Be] Add extractor

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

This PR will add support for radio1.be site. This site is subclassed from VRTBaseIE. The test may be fail because closed connection from remote or something like that (which i don't know is it only occur in my machine or not)
Fixes #8707
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[x] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Could we also add support for articles that contain a video? https://radio1.be/lees/europese-unie-wil-onmiddellijke-humanitaire-pauze-en-duurzaam-staakt-het-vuren-in-gaza?view=web


Template for the GSoC proposals

Template for the GSoC proposal
Added links to easy and help-wanted issues on GitHub for students to look at

That's pretty cool actually, I didn't know about gsocguides. I will ad the link to the gsoc page then 👍

@raphlinus Should I merge it myself? :)

I'll merge it now, but if it's approved by me, cmyr, or vlovich, you can feel free to merge it yourself. Thanks!

Will note for the future, thanks!


A couple of broken links

Hi,

At Contributing to Smartdown (github.io), the Issue Tracker (github.com) link redirects to https://smartdown.github.io/smartdown/doc/.

Similarly, at Contributing to Smartdown (github.com), the Issue Tracker (github.com) link gives a 404 error.

@DoctorBud what is the correct url to link to for these?

Thanks so much for creating this issue @UNIVAC-Colonel-Panic  ... I think I've fixed the broken links with this commit, as well as other link problems I noticed.
https://github.com/smartdown/smartdown/commit/efeedc83ce19300965ca906fafbe902f6ffe2e53


StackOverflowError from standard-controller after long uptime

Describe the bug
A EnMasse 0.30 derivate suffered the following java.lang.StackOverflowError after 3120h uptime.  The error killed the resource checker thread (from which it did not recover), so no further reconciliation were performed until the admin pod was manually restarted.
2020-05-25T17:33:14.194Z INFO  [RouterStatusCache] Collecting status from 2 routers
Exception in thread "resource-checker" java.lang.StackOverflowError
        at java.base/java.util.Collections$UnmodifiableCollection.size(Collections.java:1034)
        at java.base/java.util.Collections$UnmodifiableCollection.size(Collections.java:1034)
        at java.base/java.util.Collections$UnmodifiableCollection.size(Collections.java:1034)
        at java.base/java.util.Collections$UnmodifiableCollection.size(Collections.java:1034)
        at java.base/java.util.Collections$UnmodifiableCollection.size(Collections.java:1034)
        at java.base/java.util.Collections$UnmodifiableCollection.size(Collections.java:1034

To Reproduce
Not known
Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

I've been unable to reproduce this error or the unmodifiable collections wrapping issue that lead to it.  I tried running with a conditional break points in the UnmodifiableCollection class that break if the target collection is already unmodifiable whilst adding/removing addresses.
I did see one instance where the Hibernate Validator created a unmodifiable wrapping unmodifiables but the instance seemed shorted live.  I was unable to reproduce the issue again investigate further.  The stack trace looked similar to this: https://hibernate.atlassian.net/browse/HV-1684

I ran an experiment creating/deleting addresses over a long period of time and comparing heap dumps, without finding any difference in the UnmodifiableCollection objects in the heaps. I think we can close this for now and assume that Hibernate Validator fix will prevent this from happening again.


[resolves #242] Make CamelContextFactory available as Gravia service

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/151/
Test PASSed.


Why "Hadrian"?

Out of pure curiosity, I wonder why it is called "Hadrian", and what is the motivation of changing from "Shaking-Up-GHC" to this one.

Hadrian (https://en.wikipedia.org/wiki/Hadrian) participated in a lot of large-scale construction projects, such as Pantheon in Rome and Hadrian's Wall in the UK. The latter starts in Newcastle, where I live, hence the name :-)

(And "Shaking-up-GHC" was a bit awkward as a project name, Hadrian is shorter and easier to use.)

Thanks for the explanation and it is fantastic :) So now it would be easier to answer when people ask me why that project is named that way. I'll close it for now.

Realizing that my comprehensive nightly CI might be a nontrivial endeavor, I am thinking about renaming it to Hadrian's Wall ;)

Sounds great :-) Let's build this wall out of shiny green bricks!


style: Improve inline modal width

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have added information to docs/ if required so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Before:

After:

How did you test this code?

@matthewelwell I've adjusted this to solve the attached issue I found in the backlog, it notifies people about protected / permanent tags.


minimessage issues

Mini message is great and all but the fact you can no longer use &f &c etc... it breaks lots of things. For example when creating a rank prefix on luckperms you will only see the good colors in chat but in custom menus and tab and many many many many more plugins that doesn't support minimessage you will only see "MY RANK". Please add support to the old format.

I really don't see how ChatChat is related to other plugins at all? Especially Menu and Tab plugins? If you want to use placeholders from other plugins that don't use MiniMessages in ChatChat you just have to use the Kyorify expansion. We're not adding legacy support. The plan from the start was to have a working replacement for DeluxeChat but not at the cost of improvement. And with the DeluxeChat to ChatChat convertor we've achieved that.
For example on how to use the kyorify expansion: %luckperms_prefix% will become %kyorify_luckperms_prefix%.


Rename kevlar find --> kevlar novel

Codecov Report

Merging #74 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   85.51%   85.51%
=======================================
  Files          13       13
  Lines        1125     1125
  Branches      189      189
=======================================
  Hits          962      962
  Misses        123      123
  Partials       40       40

Impacted Files
Coverage Δ

kevlar/seqio.py
92.59% <ø> (ø)
:arrow_up:

kevlar/filter.py
69.38% <ø> (ø)
:arrow_up:

kevlar/collect.py
86.6% <100%> (ø)
:arrow_up:

kevlar/__init__.py
86.95% <100%> (ø)
:arrow_up:

kevlar/novel.py
92.74% <100%> (ø)

kevlar/cli/__init__.py
100% <100%> (ø)
:arrow_up:

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 f6ae65a...d50aa5f. Read the comment docs.


Warnings issued on stderr are not reported

First, thanks for this super-useful package.
I was having some trouble with missing images in a newly created .docx file, but pypandoc was not reporting anything, so I decided to run the same arguments through pandoc directly. Running directly I got useful [WARNING] output, but on stderr not stdout which is why pypandoc does not report it. See the example below:
>>> import sys
>>> import tempfile
>>> import subprocess
>>> from pathlib import Path
>>> import pypandoc
>>> tmpdir = tempfile.TemporaryDirectory()
>>> report_dir = Path(tmpdir.name)
>>> p = report_dir / "input.md"
>>> _ = p.write_text('![Mock](missing.png)\n')
>>> stdout = pypandoc.convert_file(
...     f"{p}",
...     'docx',
...     outputfile=f"{report_dir / 'input.docx'}",
...     extra_args=[f'--resource-path={report_dir}'])
>>> print(stdout)
<BLANKLINE>

>>> args = ['pandoc',
...         '--from=markdown',
...         '--to=docx',
...         f'{p}',
...         f'--output={report_dir / "input.docx"}',
...         '--sandbox',
...         '--resource-path={report_dir}']
>>> creation_flag = 0x08000000 if sys.platform == "win32" else 0
>>> p = subprocess.Popen(args,
...                      stdout=subprocess.PIPE,
...                      stderr=subprocess.PIPE,
...                      creationflags=creation_flag)
>>> stdout, stderr = p.communicate()
>>> print(stderr.decode('utf-8')) # doctest: +NORMALIZE_WHITESPACE
[WARNING] Could not fetch resource missing.png: PandocResourceNotFound "missing.png"
<BLANKLINE>

Would it be agreeable to modify _convert_input to return the output on both stdout and stderr rather than just stdout? Happy to put together a PR, if that would help.
Cheers,
Mat
EDIT: Here is my environment, in case it's useful:
> conda info

     active environment : _snld3d
    active env location : C:\Anaconda3\envs\_snld3d
            shell level : 2
       user config file : C:\Users\Work\.condarc
 populated config files : C:\Users\Work\.condarc
                          C:\Anaconda3\envs\_snld3d\.condarc
          conda version : 4.11.0
    conda-build version : 3.21.6
         python version : 3.9.7.final.0
       virtual packages : __win=0=0
                          __archspec=1=x86_64
       base environment : C:\Anaconda3  (writable)
      conda av data dir : C:\Anaconda3\etc\conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/win-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.anaconda.com/pkgs/main/win-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/win-64
                          https://repo.anaconda.com/pkgs/r/noarch
                          https://repo.anaconda.com/pkgs/msys2/win-64
                          https://repo.anaconda.com/pkgs/msys2/noarch
          package cache : C:\Anaconda3\pkgs
                          C:\Users\Work\.conda\pkgs
                          C:\Users\Work\AppData\Local\conda\conda\pkgs
       envs directories : C:\Anaconda3\envs
                          C:\Users\Work\.conda\envs
                          C:\Users\Work\AppData\Local\conda\conda\envs
               platform : win-64
             user-agent : conda/4.11.0 requests/2.26.0 CPython/3.9.7 Windows/10 Windows/10.0.19043
          administrator : True
             netrc file : None
           offline mode : False

> conda list
...
pandoc                    2.16.2               h8ffe710_0    conda-forge
...
pypandoc                  1.7.2              pyhd8ed1ab_0    conda-forge
...

Perhaps just even printing stderr if it is non-empty, with a quiet mode to suppress it, would match the solution in _ensure_pandoc_path, for instance.

Hi. Thanks for reporting this.
I think the second option you provided, would be the best option here.
If you know how to implement it, feel free to go ahead, if not I'll probably get around to it at the end of the month :)

@H0R5E  would the idial solution here, have to have warnings (stderr) suppressed by default, or have them printed by default?

@NicklasTegner, I think having stderr printed by default is more useful. It looks like (from here, for instance), output from writers is piped to stderr, so if you use the --verbose flag, pypandoc is going to suppress the output, which would be unexpected.


Patch-3 fresh

Squashed version of #8

@vojtasvoboda Thank you. Could you please change commit message to something more readable? This Patch-3 thing is not professional.

Sure, message changed

@vojtasvoboda :smile: I have thought commit message not PR headline message. :)

It should be renamed now


AWSv3 stable is out

http://blogs.aws.amazon.com/php/post/Tx29HZZVCASVU21/Version-3-of-the-AWS-SDK-for-PHP

https://github.com/aws/aws-sdk-php/releases/tag/3.0.0

@shadowhand a stable release is very close to hit the streets.

The it_should_rethrow_non_404_responses_when_fetching_metadata spec is failing (not sure why), but other than that, it looks like my pass through the code a few weeks back covered the required changes.

a stable release is very close to hit the streets.

Yey. This is a blocker for L5.1.

I've just pushed a commit that fixed the broken build.

BTW, what will the version number be of this release? The branch alias is still 1.0-dev?

@GrahamCampbell it'll be the 1.0.1, the 1.0.0 was already released for the beta, which was broken, so there's not point in upping anything since it was all pretty broken from the start.

k :)

@jeremeamia @shadowhand @GrahamCampbell If I'm not mistaken, we're good to tag 1.0.1 which marks the first stable release on the first stable release of the 3rd major version of the aws php sdk. (Couldn't fit more "on/off the" in there, I tried.)

:+1:

:tada:


Add new n98-magerun2 command

Magento 2 has been released more than three years ago as a followup for Magento 1, and the third party CLI tool n98-magerun (currently included in oh-my-zsh) also got a new version n98-magerun2 (https://github.com/netz98/n98-magerun2). This PR adds support for the new CLI tool as well, which is just as popular amongst Magento 2 developers as the original tool for Magento 1.

It seems the bulk of the alias is identical to the n98-magerun plugin, can't you adapt that to support both?

@mcornella Good call. That was a dead easy and I should have done that right away. The PR is now updated.

Thanks for your patience!

Thanks!


feat(llm): add integration for Claude

What does this PR do?
Added integration for Claude!
There's just one problem and that is i am not able to find model_card for claude in the docs as result i have commented that part.
Before submitting

[y] Was this discussed/approved in a Github issue or a discussion? Please add a link to it if that's the case.
[y] You have read the contribution guidelines and followed them in this PR.
[y] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[y] Did you write any new necessary tests?

For reference, this is a follow-up PR of #169

Ok noted 👍.
What other than model validation and model info might be the reasons for checks failing?


[Bug] Dropzone pro field doesn't execute 'fileNamer'

Bug report
What I did
I am executing
CRUD::field('invoice_file')
            ->type('dropzone')
            ->label('Invoice file pdf')
            ->withFiles([
                'disk' => 'invoices', // the disk where file will be stored
                'path' => '/', // the path inside the disk where file will be stored
                'fileNamer' => function($file, $uploader) {
                    Log::debug($file->getClientOriginalExtension());
                    return Str::uuid() . '.' . $file->getClientOriginalExtension();
                },
        ]);

What I expected to happen
I expected to
1 - Log
2 - Be able to add extension to the uuid
What happened
Dropzone fields works. Disk is correctly setted and also path.
But we doesn't see logs and doesn't work the changing of name
What I've already tried to fix it
Actually, nothing
Is it a bug in the latest version of Backpack?
After I run composer update backpack/crud the bug... is it still there?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
### PHP VERSION:
PHP 8.1.2 (cli) (built: Jan 19 2022 10:13:52) (NTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.1.2, Copyright (c) Zend Technologies
### LARAVEL VERSION:
<IP_ADDRESS>
### BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.2.1
backpack/crud: 6.3.0
backpack/generators: v4.0.2
backpack/pro: 2.0.18
backpack/theme-coreuiv4: 1.1.1

Sorry, possible duplicate of: https://github.com/Laravel-Backpack/CRUD/issues/5363

Yup, looks like it's the same thing. Let's move the conversation there please. I've passed it to Pedro but he's away. Might take us up to 1 week to solve this, sorry @realtebo . But we will.


Implement redux actions/reducers using the new architecture for dealing with model upload (core state)

Resolves #416
Definition of Done

[ ] The code does at least what is defined in the ticket and does not affect any other functionality
[ ] The PR has a meaningful title
[ ] There is a link to the issue-id
[ ] There are no 'hacks' or shortcuts refactoring is preferred
[ ] The happy case of the app was tested at least once manually

@jhnns ich hab alle offen issues gefixed. Du kannst nochmal drauf schauen


Use links to express relationships

See https://cloud.google.com/blog/products/application-development/api-design-why-you-should-use-links-not-keys-to-represent-relationships-in-apis

Hydra as an alternative to OpenAPI?


Configure Cloud-Native tracing with Istio 1.1

As a: Operator
I want: service calls between services hosted within the istio mesh to be surfaceable in Jaeger.
So that: I can gain insight into the performance of my microservices and highlight any bottlenecks.
Describe the solution you'd like: Now that the cloud-native stack uses istio 1.1, Configure the common.yaml file in the the cloud-native stack to pass configuration to the istio component to make distributed tracing work for all in-mesh services.
Describe alternatives you've considered:
Additional context:

If this PR gets merged, Jaeger (deployed via istio) will be able to push spans to an existing ES cluster. which should solve some of our issues: https://github.com/istio/istio/pull/13331

https://github.com/timfpark/fabrikate-cloud-native/pull/14

closed: https://github.com/Microsoft/fabrikate-definitions/pull/1


Fix WorkflowInstance#output not being set

After moving the state run logic into floe we no longer were setting the state output.  This was causing issues when the caller needs the output to populate e.g. a dynamic dialog dropdown.
https://github.com/ManageIQ/manageiq-providers-workflows/pull/5/files#diff-80ee5262679b29c698ee75771be7948fbfe8fcea9ff7c05ee83aaa9273d47885L89

Backported to petrosian in commit 66bd179909799766133541e324b262d428bbfdd4.
commit 66bd179909799766133541e324b262d428bbfdd4
Author: Keenan Brock<EMAIL_ADDRESS>Date:   Mon Jul 24 12:26:39 2023 -0400

    Merge pull request #35 from agrare/fix_missing_output

    Fix WorkflowInstance#output not being set

    (cherry picked from commit d4035e61c08f61fb99866606a16a958904ae90ae)


Implement withField and dropField for struct types

What is the problem the feature request solves?
See documenttion for more details:
https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/api/pyspark.sql.Column.withField.html
https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/api/pyspark.sql.Column.dropFields.html
Describe the potential solution
No response
Additional context
No response

take

I believe these are purely used for analysis, and end up just becoming named_struct expressions in the physical plan, so they're probably already supported

My understanding is the same as @Kimahriman
that both of these are implemented in terms of UpdateFields which are replaced in the spark Analyzer by the rule ReplaceUpdateFieldsExpression https://github.com/apache/spark/blob/v3.5.2/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/UpdateFields.scala#L79-L87 and based on https://github.com/apache/spark/blob/v3.5.2/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala#L746-L757 it looks like it is replace by CreateNamedStruct expression and should therefore be supported.


V8: Add ckEditor plugin - Keep TextSelection & syntax highlight to source view

The ckEtidor is close to perfect, but its really hard to read/edit the source code (No highlight, No indentation + when i select a text => and click "source" i need to "ctr+f" to find him in the code).
Lets improve the "source" for devs:
1) HTML Syntax highlight.
Syntax Highlighting for the CKEditor (Source View and Source Dialog) with the CodeMirror Plugin
https://w8tcha.github.io/CKEditor-CodeMirror-Plugin/demoFramed.html
2) Keep TextSelection
The plugin that makes the editor keep it's text-selection when switching between WYSIWYG and Source mode, and scrolls the selection into the viewport.
http://ckeditor.com/addon/textselection
3) Add indent ("non plugins" issue":):
http://docs.ckeditor.com/#!/guide/dev_output_format
** I find more plugins for this issues.

** I dont success to manually install this plugins.

It's pretty trivial to add in more add-ins once you know what you're doing. I added some basic notes over in #3945.
I think this issue really comes down to balancing the features we're bundling in the core with how large of a network footprint we want to impose on the editors of the site.

Thanks. I hope in the future will be docs on "how to" add your own plugins to C5 (and how to change language).
Anyway - i think this is more UI issue. If i click on "source" its more friendly do get readable code (Its not like the option to "add table" or "div" from the editor).

@Siton-Design  I have some dated docs over on the ckeditor package for pre version 8 you could glance at, they're probably still pretty true. https://github.com/ExchangeCore/Concrete5-CKEditor/blob/develop/docs/plugin-guide.md
At some point it would be able to get some official docs up though.


[Question]: How to take the livestream video/audio data and turn it into a RTSP feed?

Ask your question
I have an app that is initiating the livestream and listening for the livestream video and audio data. It then creates a TCP server and feeds the incoming livestream video and audio data to ffmpeg to then be streamed over RTSP using rtsp-simple-server. I can get the video to work, but can't for the life of me get the audio to work. I've tried creating two TCP sources (one for video and one for audio) and providing the sources to ffmpeg and I've tried one TCP source with both video and audio.
Here are some sample ffmpeg commands I've tried:
Single source
ffmpeg -f -i tcp://localhost:3331 -vcodec copy -acodec copy -rtsp_transport tcp -fflags +genpts -f rtsp rtsp://localhost:8554/eufy/stream
Double source
ffmpeg -f H264 -i tcp://localhost:3331 -f AAC -i tcp://localhost:3330 -map 0:v -map 1:a -vcodec copy -acodec copy -rtsp_transport tcp -fflags +genpts -f rtsp rtsp://localhost:8554/eufy/stream
ffmpeg -i tcp://localhost:3331 -i tcp://localhost:3330 -map 0:v -map 1:a -c:v libx264 -c:a aac -f rtsp rtsp://localhost:8554/eufy/stream
Any help or insight would be appreciated.
@fuatakgun I looked at your project for some direction, maybe you could steer me in the right direction.

I remember that @bropat had already implemented this in iobroker component of eufy security. You can see how he managed to use two different inputs and generate one output

@fuatakgun Thank you for the information!

By the way, are you able to manage to get benefit of go2rtc based ffmpeg streaming and just using tcp server inside python? @ema-dev

@fuatakgun This particular project I'm working on is written in Go, not Python.


V3+V2 restore fails when server responds to FindPackagesById() with 204-No content

I have a Visual Studio 2015 windows 10 app project with references to packages from both nuget.org(V3) and a local TeamCity server(V2).
During package restore TeamServer is queried for all packages and responds 204 No content to the ones it doesn't know. This causes restore to fail.

This issue was moved to NuGet/Home#1759


[e2e]: delegator-generic create main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v45.0.2
Run: https://github.com/slsa-framework/example-package/actions/runs/5593989576
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.default.slsa3.yml
Trigger: create
Branch: v45.0.2
Date: Wed Jul 19 01:21:12 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v45.0.3
Run: https://github.com/slsa-framework/example-package/actions/runs/5594004596
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.create.main.default.slsa3.yml
Trigger: create
Branch: v45.0.3
Date: Wed Jul 19 01:29:53 UTC 2023
Tests are passing now. Closing this issue.


Add window function fuzzer

Description
The window operator is currently being tested in aggregation fuzzer with Presto aggregation functions. However, the window-specific functions, such as first_value, last_value, lag, and lead, are not tested yet. Also, aggregation fuzzer doesn't test window operator with random frames.

Some methods in the AggregationFuzzer class can be reused by the window function fuzzer, including
findInputGenerator(),
makeSplits(),
deserialize(),
getFuzzerOptions(),
generateKeys(),
generateSortingKeys(),
pickSignature(),
generateInputData(),
execute(),
computeReferenceResults().
We can extract these methods into a base class AggregationFuzzerBase and let AggregationFuzzer and WindowFuzzer both inherit AggregationFuzzerBase.

@kagamiori : We definitely want to add Window function fuzzer testing. Its something we are taking on next in our development.
I too was putting some thought in how to do this and there is lot of re-use between AggregationFuzzer possible for it. In addition to the functions above we need to do more work for generating fuzzer frames as well.
Are you proposing to do the refactoring you have mentioned above or its at an idea stage ?

@kagamiori : We definitely want to add Window function fuzzer testing. Its something we are taking on next in our development.
I too was putting some thought on how to do this. There is re-use with AggregationFuzzer possible for it. In addition to the functions above we need to do more work for generating fuzzer frames as well.
Are you proposing to do the refactoring mentioned above or its at an idea stage ?

Hi @aditi-pandit, I'm planning to start refactoring AggregationFuzzer today if you haven't started yet, and the next step is to build a WindowFuzzer on top of AggregationFuzzerBase. What's your plan? Maybe we can work together.

@kagamiori : That's great. We can work together. Thanks so much.

As a first step, we can take what was being done in AggregationFuzzer for window in the past and extend it to pick up non-aggregate window functions.  We can start with the ranking functions as many of those don't require parameters.  But lets also do the refactoring you are suggesting with AggregationFuzzerBase to a point that we can invoke the WindowFuzzer separately via its own command line flags.

The next step would be to extend WindowFuzzer for value functions that could use column parameters.

Then we could add fuzzer generated frames.

If you start with the first bullet, then I can work on the more window specific extensions in the second and third bullet.

@kagamiori : That's great. We can work together. Thanks so much.

As a first step, we can take what was being done in AggregationFuzzer for window, and extend it to pick up non-aggregate window functions.  We can start with the ranking functions as many of those don't require parameters.  But lets also do the refactoring you are suggesting with AggregationFuzzerBase to a point that we can invoke the WindowFuzzer separately via its own command line flags.
The next step would be to extend WindowFuzzer for value functions that could use column parameters.
Then we could add fuzzer generated frames.

If you start with the first bullet, then I can work on the more window specific extensions in the second and third bullet.
Feel free to suggest any specific items you want to do in this issue as well. We can collaborate here.

@aditi-pandit: Sounds great. I'll start with the refactoring then.

When comparing fuzzer result against Presto-Java, I noticed that the columns in the partition-by clause cannot be complex-typed columns with NULL elements because Presto-Java doesn't support sorting on such columns. However, since the random data is typically generated for all columns together (unless custom input generator is used), VectorFuzzer has no idea which columns are the partition-by key columns, so all columns are generated with the same VectorFuzzer::Options.
I mitigated this issue by setting VectorFuzzer::Options::containerHasNulls to false for all input columns. We could extend VectorFuzzer API to allow us indicating a subset of columns that shouldn't contain NULL elements in the future if needed.
cc @kgpai @mbasmanova

I noticed that the columns in the partition-by clause cannot be complex-typed columns with NULL elements because Presto-Java doesn't support sorting on such columns.

It is probably something slightly different. Presto Type has orderable property. Types with maps are non-orderable. Other types are. For orderable types, the rules for sorting are complicated and they do allow nulls, but not always. See https://github.com/facebookincubator/velox/issues/7536#issuecomment-1812592270

Closing this issue since the fuzzer has been added.


Add log splitter binary for kubernetes alerts

After some discussion with devops we want to make this part of the
docker image instead of inserting it through configmap later.
Reimplemented in rust because we're a rust code base.
Once this is merged the plan is to change the kubernetes command to sh -c "orderbook | log-splitter".
Test Plan
new unit test

That is a good point. Will make a new repo for it.

Requesting review again because I changed this to pull from the other repository.


Sweep: Write a release note for Search mechanism (✓ Sandbox Passed)

Description
This pull request adds a release note for the new search mechanism feature.
Summary

Added a new release note file RELEASE_NOTE.md to document the introduction and features of the search mechanism.
Updated the AI module (upsonic_on_prem/utils/ai/ai.py) to include details on how it processes user queries for the search mechanism.
Modified the Django view (upsonic_on_prem/dash/app/views.py) to handle search requests and pass the results to the template for display.
Enhanced the user interface with a search form and results display section in the template (upsonic_on_prem/dash/app/templates/search.html).
Included instructions on how to use the search mechanism for efficient information discovery.

Fixes #81.

🎉 Latest improvements to Sweep:

New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To get Sweep to edit this pull request, you can:

Comment below, and Sweep can edit the entire PR
Comment on a file, Sweep will only modify the commented file
Edit the original issue to get Sweep to recreate the PR from scratch

Rollback Files For Sweep

[ ] Rollback changes to  RELEASE_NOTE.md

This is an automated message generated by Sweep AI.


How can I enable multiple crd in helm operator ?

I wan to use operator-sdk to create a helm operator.
And I got multiple crd to watch deploy?
I wonder how should I arrange the helm repos?

operator-sdk add  may be used to add more crds and api.
But as the doc stated, this subcommands will not be supported anymore.
If these two subcommands will not be supported any more, how to add multiple api and crds ?

@dongwangdw To have a helm-operator watch and reconcile two different CRDs for two different charts, there are a few things you need to do:

Copy/move your second chart into the helm-charts directory in your project root.
Add another entry in watches.yaml with the group, version, and kind of your CRD and the chart that should be reconciled. If you're using v0.13.0, the chart path is relative to the project root, so you would use helm-charts/<chart-name>
Run operator-sdk add crd --api-version=<apiVersion> --kind=<kind> (where <apiVersion> is <group>/<version> and <kind> is <kind> from your watches.yaml file).
Update deploy/role.yaml as necessary to give the operator permissions to reconcile the resources that the chart contains.

Got you. Thanks.
Well, got another question.
If I add more CRDs which seems it will complicate the logic. Then there will be a comparison:
multiple CRDs in one operator VS multiple operators each with a single CRD. The later one seems make more sense from the design perspective.  Any comments?

I think the only reason I would consider using multiple CRDs in a single Helm operator would be if there was a parent-child relationship between the charts. For example, you might have a parent chart for an App that deploys an AppWebsite and AppDatabase, where AppWebsite and AppDatabase each have their own separate charts. With this model, you can abstract details of what the website and database are and populate shared config once from the main App CR.
If you have two distinct charts that do not have this sort of relationship, it probably makes more sense to make separate operators for them.

Hi @joelanford, I was wondering if your above comment is still recommended now that 1.0.0 is out. I have users that are limited to their specific namespace, and I'd like to be able to just submit crs for the various components they need and have those components get installed into that namespace. I also have the possible use case of wanting to support multiple versions of those helm charts. I only intend to install these particular helm charts when I am making a user space. Does it make sense to make one mammoth helm operator with possibly 3-4 different(non parent child) charts, with possibly up to 3 supported versions for each chart? If I split them up into individual, would you recommend keeping the versions together in one operator, or make a new operator for each version?
Thanks!

It doesn't really matter from a technical standpoint. One single operator with multiple charts will work the same as several operators with individual charts.
The use case you describe sounds like a managed SaaS kind of thing? In that case, I can see an argument for grouping all of the charts in a single operator since it sounds like they're all always deployed together and it would be nice to be able to test and guarantee the deployment of the group of apps together.

I think the only reason I would consider using multiple CRDs in a single Helm operator would be if there was a parent-child relationship between the charts. For example, you might have a parent chart for an App that deploys an AppWebsite and AppDatabase, where AppWebsite and AppDatabase each have their own separate charts. With this model, you can abstract details of what the website and database are and populate shared config once from the main App CR.
If you have two distinct charts that do not have this sort of relationship, it probably makes more sense to make separate operators for them.

Hi @joelanford , currently I am working on a similar use-case of deploying multiple CR using single helm operator. But I am little confused about establishing parent child relationship between these CR. It would be really great if you could share any project/github repo where this is actually implemented.
Thanks!


docs: update react-native example

Based on typescript types updating documentation for react-native.
https://github.com/breez/breez-sdk/blob/main/libs/sdk-react-native/src/index.ts#L486

Thanks @karliatto
CC: @dangeross @JssDWt Can we also catch compile errors for RN example during CI workflows and/or part of makefile commands on sdk-react-native?


[arrow]Fix build error on Visual Studio 2019.

Related: #6497.
Thanks @treigilbert.

/azp run

Thanks for the fix, @JackBoosY!

Does this need to be fixed upstream in Apache Arrow?

@wesm Yes, but I don't have an account.

Anyone can contribute to this project, see
https://github.com/apache/arrow/blob/master/.github/CONTRIBUTING.md

@kszucs are we testing VS2019 in GitHub Actions now? Do you know whether the problem here has been fixed?

@wesm We do test VS2019, see build log and the build machine's spec.
But, both patches look valid because:

ARROW_PARQUET is not enabled in the C++ VS2019 build (created https://issues.apache.org/jira/browse/ARROW-7524)
we don't have a VS2019 build for pyarrow (created https://issues.apache.org/jira/browse/ARROW-7525)


wordsmithing

Hi @jpsnover, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@jpsnover : Thanks for your contribution to the Azure documentation! The author, @heathl17, has been notified to review your proposed change.

Thanks for the PR @jpsnover.  The Azure Style Guide requires the use of sentence casing for titles.  Can you commit with "architecture" in lower case and I'll get the PR merged?  Thanks!
+@ByronR

I updated the PR.

Thanks @jpsnover!
#sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.


ci: Add test steps for go-sdk and cp-connector

This PR

adds go-sdk and cp-connector to the CI
creates Dockerfiles for both
adds both to codecov file

Related Issues

Closes #7450
Follow-up Tasks

The same should be done in #7079
How to test

You can check this run here https://github.com/keptn/keptn/runs/6406419596?check_suite_focus=true
or push in this PR a change to go-sdk or to cp-connector, check CI pipeline for Build

The go-sdk scope is also missing in validate-sematic-pr.yml


TagsInput behaving odd

I have a multiple select as a TagsInput:
<select multiple data-role="tagsinput" placeholder="Tags (i.e. 'Competition', 'Fun training' etc.) Separate with commas">
</select>

When I start to add tags, it automatically adds another comma and the input is on a new line, like in the picture.

Did you find a fix?