Binderator improvements

Adding some trivial improvements to binderator:

download additional files for bindings improvements
POM files, JavaSources and JavaDocs (for parameter names)

batch operations

tracing

batch operations
binderator would exit after 1st unsatisfied dependency which was productivity killer for large bindings like AndroidX and Google.Play.Services-Firebase-MLKit.
With this improvement binderator continues checking dependencies for all artifacts in config.json file and throws exception for all dependencies:
Example:
Dependency errors : 2
1
	No matching artifact config found for: org.jetbrains.kotlin.kotlin-stdlib:1.4.32 to satisfy dependency of: androidx.window.window:1.0.0-alpha06
2
	No matching artifact config found for: org.jetbrains.kotlinx.kotlinx-coroutines-android:1.4.3 to satisfy dependency of: androidx.window.window:1.0.0-alpha06

   at AndroidBinderator.Engine.BuildProjectModels(BindingConfig config, TemplateConfig template, Dictionary`2 mavenProjects) in /Users/Shared/Projects/d/X/XC/binderator-improvements/Util/Xamarin.AndroidBinderator/Xamarin.AndroidBinderator/Engine.cs:line 386
   at AndroidBinderator.Engine.ProcessConfig(MavenRepository maven, BindingConfig config) in /Users/Shared/Projects/d/X/XC/binderator-improvements/Util/Xamarin.AndroidBinderator/Xamarin.AndroidBinderator/Engine.cs:line 86
   at AndroidBinderator.Engine.BinderateAsync(BindingConfig config) in /Users/Shared/Projects/d/X/XC/binderator-improvements/Util/Xamarin.AndroidBinderator/Xamarin.AndroidBinderator/Engine.cs:line 56
   at Xamarin.AndroidBinderator.Tool.Program.Main(String[] args) in /Users/Shared/Projects/d/X/XC/binderator-improvements/Util/Xamarin.AndroidBinderator/Xamarin.AndroidBinderator.Tool/Program.cs:line 64
   at Xamarin.AndroidBinderator.Tool.Program.<Main>(String[] args)

/azp run


Cannot create endpoints inside a connector in the same package

Description:
For example, consider the following filter connector:
package ballerina.net.http;

public connector TestClient (HttpClient httpClient) {

    endpoint<HttpClient> httpEP {
        httpClient;
    }

    action post (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action head (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action put (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action execute (string httpVerb, string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action patch (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action delete (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action get (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action options (string path, OutRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }

    action forward (string path, InRequest req) (InResponse, HttpConnectorError) {
        return null, null;
    }
}

When this is inside the ballerina.net.http package, a null pointer exception is thrown in the TypeChecker since the actualType is null.
This works when this definition is placed after the HttpClient definition in the natives.bal file.
While in this example the endpoint is at connector level, putting it inside an action doesn't work either.

Closing since this is no longer valid


Update Start-NetEventSession.md

Added Description for As Job and Input object

Docs Build status updates of commit c72aab1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/neteventpacketcapture/Start-NetEventSession.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@get-itips @tseward
Peer review needed for this PR. Thanks.


Add support for Latent Hyperlinks

Today when we generate a page, sometimes GPT-3 will add a link. However those are not supported yet.
To make it work, various solutions:
the simplest way could be to let GPT-3 generate the link with the correct path eg.
Click me

or we could use an arial-label like explained here: https://www.w3.org/WAI/WCAG22/Techniques/aria/ARIA8

I wonder if you could have links re-use the existing page layout by telling it to update the previous code and replace the main content (so header, nav, etc stay in place). Or maybe the initial prompt can tell it to identify a content area that should be changed by links. Love the idea of building a multi-page site just by clicking around.

That's a great idea to prevent the change in styling!
I think it could probably be instructed with something like "clicking on links should change the content section of the page using javascript"
FYI there is already a system in place to generate sub-HTML content (demo), but each subsequent call will change the inner design:
/**
 * An asynchronous function that returns web content that matches a brief
 *
 * It will be injected in a
 somewhere in the page.
It generates valid HTML and JS, without any error and exception.
 * @param {string} brief instructions given in natural language
 * @param {Object} rules - some rules to control the look of the app
 * @param {string} rules.framework - The CSS class framework to use - you must only use this for styling
 * @param {string[]} rules.design - Mandatory rules of design: how to create the HTML layout, design patterns..
 * @param {string[]} rules.logic - Mandatory rules for JS code and app logic: how to manage variables, libraries..
 * @param {string[]} rules.images - Mandatory rules for defining how to write the alt= text of : keywords to use, format..
 * @param {string[]} rules.direction - The general art direction and composition rules of both the web page and the images
 * @returns {Promise<string>} HTML and JS content (a root
 and a


[pallet_collective] set_members Extrinsic Allows Arbitrary 'prime' Argument

Is there an existing issue?

[X] I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

[X] This is not a support question.

Motivation
The motivation behind filing this issue is to address problematic behaviour in the pallet_collective module of the FRAME pallets. The current implementation of the set_members extrinsic allows for an arbitrary value to be passed as the prime argument, despite its intended purpose of representing one of the members in the new_members argument. This behaviour deviates from the expected functionality.
Request
The request is to modify the set_members extrinsic in the pallet_collective module of the FRAME pallets to enforce the intended usage of the prime argument.
Solution
To address this issue, I propose that the prime argument should be restricted to accepting only one of the accounts listed in the new_members argument. This modification will ensure that the set_members extrinsic functions as intended, allowing for correct and secure updates to the collective membership. I would be happy to raise a PR to fix this issue.
Are you willing to help with this request?
Yes!

Yea sounds reasonable. Please go ahead if you like 😄


crop_stages_endpoint

This also contains a package-lock without new packages. Please fix that along with the merge conflicts

sure Niels ... I can undo the package lock commit like I did last time as u said ... should i pull develop again to fix the merge conflicts or it should be the same way like undoing the commits

hmm.. this might be difficult because you have several commits with package-lock changes. I think the easy way would be to completely overwrite the package-lock with the one from develop

and that will fix merge conflicts as well or do we need to some other process

That will removedthe conflict with that file only

Next step is to pull develop and merge develop into your branch

thanx Niels ...

@humayunadilshahzad
You still need to overwrite the package-lock file with the one from develop. Simply copy and paste it in

or
git checkout develop package-lock.json

@humayunadilshahzad can you implement changes and merge?

Approved


Sink

Lot's here. Most importantly this is now a sink.
Also:

Added R# settings.
Converted to .net core and support net standard.
No longer use log context.
Add a method to get log events from the current context.

@MitchBodmer,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Update metadata tables for unpartitioned tables

This removes the partition column from the files and entries metadata tables when the underlying table is not partitioned. For unpartitioned tables, this is an empty struct and cannot be projected in Spark.
In support of suppressing the partition column, this also updates ManifestReader and FilteredManifest to support schema-based projection. Because this is already updating the Filterable API, this fixes #145 and adds case sensitivity methods.

+1 LGTM


Optional connector features

It would be a nice addition if a connector has the ability to implement more than just the PhysicalMemory trait.
Those additional traits could be features like retrieving and setting the CPU state or sending inputs to a target.
It would also be great if certain portions of the code could make use of those optional features in case they are available. For example, the win32 initialization could just read the CR3 instead of scanning for the START_BLOCK.
This however requires it to work via static linking and additionally over the inventory FFI.
One solution could be to introduce a new Trait that exposes down- / upcast features:
trait Connector {
    fn physical_memory(&mut self) -> Option<&mut dyn PhysicalMemory>;
    fn cpu_state(&mut self) -> Option<&mut dyn CpuState>;
    // ...
}

and also a way to cast from a PhysicalMemory or CpuState object back to the underlying Connector (as_connector()...).

This has been implemented via cglue.


Added Interactive Resume

Description
Added Interactive Resume - A Game like resume template
Fixes:  #464
Checklist:

[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings

#SCREEN-SHOTS

@itsmohitnamdeo


Fix stream SSE uploads with S3 encrypt type

When you try to upload files stream based with encryption activated, minio-go
try to upload the parts using the ServerSideEncryption header, but it isn't
supported for uploadPart with encrypt type S3, only is needed for customer
keys. For S3 encryption we simply add the header at the creation, and in the
following uploadPart we don't pass the header.
I'm not sure, but maybe we need to exclude the encrypt KMS type too (I'm not
sure).
Here is an example of it failing before the patch, and working after it.
package main

import (
	"log"
	"os"

	minio "github.com/minio/minio-go"
	"github.com/minio/minio-go/pkg/encrypt"
)

func main() {
	s3Client, err := minio.New("s3.amazonaws.com", "API_KEY", "API_SECRET", true)
	if err != nil {
		log.Fatalln(err)
	}

	object, err := os.Open("test.txt")
	if err != nil {
		log.Fatalln(err)
	}
	defer object.Close()

	options := minio.PutObjectOptions{ServerSideEncryption: encrypt.NewSSE()}

	n, err := s3Client.PutObject("my-bucket", "my-objectname", object, -1, options)
	if err != nil {
		log.Fatalln(err)
	}

	log.Println("Uploaded", "my-objectname", " of size: ", n, "Successfully.")
}

@jespino I think you're also right about KMS - but we should check to make sure...

You are welcome, I'm implementing the encrypt.NewSSEKMS to try it ;) If I realize that is needed for KMS too, I will send another PR, if you want :)

That would be awesome :) @jespino

I have created what I think is needed, but I don't know too much about KMS, so I don't know if the context as an optional string is enough.


Added Channel Group

Description
In Android O,Push Channel also have group to categorize notification channel such as Work and Personal Notifications
Motivation and Context
It can help developer to assign notification channel to a group,easily categorizing it.
Which help user experience better.
Example use cases are chat app between Personal and Business account messages notifications
How Has This Been Tested?
Using Outsystem,I use the Firebase Push Plugin and modified the source code and the wrapper on the plugin on Outsystem itself.Also reference the source code instead of the original author git with the channels related function in the latest version of this plugin.
Also tested on Mi Max on lineageOS 15.1 and Mi A1 on Android 8.0 for creation,listing and deletion of channel.Setting and listing channels with the group
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks for contributing!
Can you update the type definitions to reflect on those new APIs and attributes?

What is it,is it the API.md

It's the https://github.com/phonegap/phonegap-plugin-push/blob/master/types/index.d.ts file. That typescript "interface descriptor" needs to reflect the new APIs you're adding, so that the project compiles for the users that consume our API through a typescript project.

There you have it

Sorry I am not familiar with typescript,sorry to inconvenience you with my new features that I want to add on

Sorry I am not familiar with typescript variable,sorry to inconvenience you with my new features that I want to add on.I thought it seemed like javascript

Not at all an inconvenience! Please pat yourself on the back, you're doing all of us a big favor.
You even walked the extra mile to document/type missing parts that weren't even related to your additions to the API, which is awesome.
The simple fact that you're taking the time to work through suggestions on this PR is another amazing things. So yes, thank you very much for your contribution ;)
I won't bother you with some of the other missing parts I noticed, as they're definitely not related to your PR, and I'll work on them later myself.


Migrate breadcrumbs to Stimulus (RevealController / w-breadcrumb)

Overview

Adds a new generic RevealController that handles the ability for content to be opened/closed while updating the relevant parts of the DOM such as aria-expanded on the toggle.
Register this controller as a generic w-reveal and also the w-breadcrumb (singular) identifiers so that we can differentiate usage via different DOM attributes and events.
This code paves the way to replace the other two JS implementations of open/close panel widgets; Collapsible Panels and the Collapsible sidebar but aims to not implement too much for that upcoming work. e.g. #10168
Closes #10118
Replaces PR #10160

Testing

If manually testing this feature; be sure to check the 'hover to expand' behaviour, this is a bit nuanced but basically allows the 'peek' of the breadcrumbs being shown as open when you hover and they should only close if you move away from the header as a whole. Additionally, clicking the toggle when in this 'peek' mode should mean they stay open.
Breadcrumb usage (JS enabled); Editing a Snippet, Editing a Page.
Breadcrumb usage (no JS should be enabled); Page chooser panel, same UI but not the hover effect and no toggle.

Checklist

[X] Do the tests still pass?
[X] Does the code comply with the style guide?
[X] For Python changes: Have you added tests to cover the new/fixed behaviour?
[X] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

Safari 16.1, Firefox 115, Chrome 117

[X] For new features: Has the documentation been updated accordingly? (added note to upgrade considerations)

Just cleaned up a few things upon re-reading the PR; some duplicate HTML in the unit tests, a better solution for the initial toggle target access that I had to implement a workaround for https://github.com/hotwired/stimulus/issues/711 and additional classes handling.
Finally, the 'ready' event now fires after a setTimeout, this should allow any other non-Stimulus JS on initial load to listen to these events a bit easier. Less chance of a race condition between Stimulus & non-Stimulus code (e.g. minimap listeners / tooltip listeners).

Thanks for this PR! In terms of functionality, it seems to work as expected (I didn't see any change in user-facing behaviour). I can't wait to use it to work on #8767!

Brilliant, thank you for testing.

However, I find it hard to review because the refactoring is done in a single commit that simultaneously handles other cases that aren't actually observed in Wagtail's breadcrumb templates.
I appreciate that you're trying to pave the way for migrating the panels to Stimulus, which I think would make sense to use the same controller if possible....

Agreed, I have removed the ones that are clearly isolated from the core open/close behaviour and class setting/resetting. I can remove the initial class if needed also.
@laymonage - really appreciate the thorough review, let me know if you really disagree with the RevealController/Open/Closed naming setup and I can rebuild this accordingly but this seemed to be the best direction to go in based on lengthy discussions in #10168 and my review of all other implementations of similar functionality.

Thanks for the updates LB! I'm off tomorrow but I'll re-review when I'm back on Monday. Really appreciate the fast response 😃

Thanks @laymonage  - I hope this is in a better shape for the initial implementation.
There is still some code that relates to aria-controls in place but this is stuff that intertwines with other code and hopefully it's accepted as is.
Should be good for another review now, all the initalize and the onbeforematch code has been completely removed.

Thank you @laymonage  - appreciate the feedback. I know it's a bit of a stretch to include some code for 'future', thanks for being OK with some of this being included.


MINOR: Optimization of equals methods on implementations of Commands.Handler in shell.command package

Optimization of  equals  method on  multiple implementations  of Commands.Handler  in shell.command package ,for making it less verbose
Committer Checklist (excluded from commit message)

[x] Verify design and implementation
[x] Verify test coverage and CI build status
[x] Verify documentation (including upgrade notes)

@divijvaidya  @ijuma  Do you think that those changes worths a pull request


launcher: rename to zig-wrapper + passthrough mode

launcher: rename to zig-wrapper + passthrough mode
Also remove ld64.lld, since it was removed from zig upstream back in 2021: https://github.com/ziglang/zig/commit/0e15205521b9a8c95db3c1714dffe3be1df5cda1
Until today the "launcher" could do one thing well: set the environment variables and exec the zig c++ command. However, it was clumsy with other subcommands: ar, ld.lld do not expect the -target. So there was this "special case" for subcommands that are not c++, but, for the sake of consistency, needed to be in the top-level repository.
This change leaves only one sub-command for the target, which truly expects a -target flag: c++. From now:

all other sub-commands required for the toolchain are in top-level.
anything that is not needed for the toolhain can be directly used via zig-wrapper.

This simplifies the launcher quite a bit.

Tried this branch on an Windows host and getting some linker errors trying to use it on my project:
ERROR: C:/users/fabian/_bazel_fabian/xjw2phwq/external/freetype/BUILD.bazel:213:11: Linking external/freetype/libwinfonts.a failed: missing input file 'external/zig_sdk/tools/ar', owner: '@zig_sdk//:tools/ar'

Looks like something with the archiver is broken, I think the compilation itself worked.
I probably won't have the time to investigate in more detail before the weekend but wanted to give you the heads up.

Looks like something with the archiver is broken, I think the compilation itself worked.
I probably won't have the time to investigate in more detail before the weekend but wanted to give you the heads up.

Thanks for suggesting to look into this.
However, given how easily and often I break Windows, I would rather learn how to test it myself and only ask for your final confirmation before cutting a major release, instead of on PRs like this.
I have some issues running Bazel on my Windows VM, I reached out to you directly.

Tried this branch on a Windows host and getting some linker errors trying to use it on my project:
<...>

I am pretty sure it was me forgetting the .exe again. Fixed. I will re-request the review after CI stabilizes.

Windows host support works fine for me now, thanks for fixing 🙂

Thanks for review!


Fix bug: Some tasks were not being set as ready to destroy

Changes

Set runnable as unique pointer
Ensure that all tasks are set as ready to destroy
Kill the thread in runnable destruction
Reset the runnable pointer after setting the task as ready to destroy to force the destruction of the thread

#233


Remove qiskit-terra dependency in favor of qiskit

qiskit-honeywell-provider depends on qiskit-terra (>=0.16.0) https://www.wheelodex.org/projects/qiskit-honeywell-provider/ which is reaching EoL:

[!IMPORTANT]
The package qiskit-terra is not going to be updated after August 15th, 2024. Since Qiskit 0.44 (released on July 27th, 2023), the qiskit meta-package only contains qiskit-terra. In Qiskit 1.0 and beyond, the meta-package architecture is removed.
If you are installing or depending on qiskit-terra, consider changing that to qiskit: Either qiskit>=0.x,<1 (if you did not transition to Qiskit 1.0 yet) or qiskit>=0.x,<2 (to also include Qiskit 1.*).
Read more.

qiskit-quantinuum-provider 0.4.1 (released on Jul 22, 2024) depends on qiskit-terra.
qiskit-terra reached end-of-life and the last update was 0.46.3, on Sep 22, 2024.


Happy birthday @toddmotto

Happy birthday @toddmotto! Thanks for starting this great repo!

Thank you kindly sir!  It's crazy how it's got so popular! 👊🏻
@toddmotto

On 21 Oct 2017, at 23:17, James Brooks<EMAIL_ADDRESS>wrote:
Happy birthday @toddmotto! Thanks for starting this great repo!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Add fix for fold=false with BoseFS

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 89.872%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
4357

Relevant Lines:
4848

💛  - Coveralls


Option to disable or hide the alpha channel slider

Hi,
Is it possible that you might add an option to the configuration to disable or preferably hide the alpha channel slider?
Currently I'm hiding it in a hard coded css way:
https://github.com/H0rn0chse/GifBuilder/blob/main/styles/options.css#L50
Thanks in advance.

Hi,
I'm added use_alpha option in color picker configuration
const picker = new ColorPickerControl({      container: document.body,      use_alpha: false });
In the next version I will add the ability to change this parameter dynamically.

Hi,
I added use_alpha option in color picker configuration
const picker = new ColorPickerControl({ container: document.body, use_alpha: false });
In the next version I will add the ability to change this parameter dynamically.

Hi,
I added use_alpha option in color picker configuration
const picker = new ColorPickerControl({ container: document.body, use_alpha: false });
In the next version I will add the ability to change this parameter dynamically.

Hi,
I added use_alpha option to the configuration.
const picker = new ColorPickerControl({ container: document.body, use_alpha: false });
In the next version I will add the ability to change this parameter dynamically.

Thank you very much
I've tested the new version and it works perfectly for me!


Additional status icons on Devices page

In GitLab by @jasonpagetas on Jul 9, 2024, 04:18
Currently on the devices page there is a status icon showing if the inventory has been conducted. There could be a benefit to having other quick information shown to a user / admin when accessing this page.
Status icons could include:

[ ] Device online
[ ] User logged in
[ ] Security compliant ok
[ ] Updates available (OS or Software)

added 1m of time spent

marked this issue as related to #85

marked this issue as related to #19

@jasonpagetas, this issue was migrated from gitlab to github. During this migration you were not added as the user whom opened this issue. This message will ensure that you receive further updates on this issue as it progresses.
Further information can be found within #215


Prehash selectors for runtime performance

FOR TESTING ONLY
Change for testing runtime perf performance improvement of prehashing selectors.

This change is

superceded by #1699?

@GreatKeeper thanks a bunch.  Closing this PR since @bviglietta merged this through #1699.


🎉 cleanup for new project

setup app.tsx and cleaning/delete files for starting the new project
remove files:
app.module.css
logo
remove folder:
components

remove favicon link in main


Ekans giving 45 Speed EVs.

Leveling in a Savanna using a Mareanie (Slot 1) holding a Lucky Egg and an Aerodactly (Slot 3) holding EXP Share. After about 15 kills of Attack EV Pokemon both got to max Speed EVs. Attempted to replicate with Popplio (Slot 2) holding Lucky Egg and Magikarp (Slot 4) holding EXP Share but did not observe Speed EV gain. Put Lucky egg back on Mareanie and killed an Ekans and Magikarp gained 48 Speed EVs from a battle. Took lucky egg off and killed a Patrat and no EV increase. Killed an Ekans and gained 45 Speed EVs. Switched to Popplio (Lucky Egg)and killed Ekans gaining 45 Speed EVs. Tested in single player to determine if the bug was from the mod or server. Ekans gave no Speed EVs, though I should not they were spawned from a spawner and not natural spawns, in case it may be relevant.

Closing the ticket as it no longer is a issue.
It now gives naturally 1 attack evs.
If it happens again feel free to open the ticket again.


Create New070122.java

CodeScene Delta Analysis Results
View detailed results in CodeScene

Recommended Review Level
Detailed -- Inspect the code that degrades in code health.

Warnings
Low Code Health for new/renamed Modules

New070122.java has a low code health score with 4.21.

Review of New Files
1 new file(s) below the threshold for new code (9).
0 new file(s) where the code health is above the threshold.
Lenia/New070122.java

Bumpy Road: A Bumpy Road is a function that contains multiple chunks of nested conditional logic inside the same function. The deeper the nesting and the more bumps, the lower the code health. Detected Bumpy Roads: onTick, fullRecalc, assemble, toBreakNearPlayer, searchForPlaceables and 2 more functions..
Deep, Nested Complexity: Deeply nested logic in: fullRecalc (depth = 6), searchForPlaceables (depth = 5), recalcNearby (depth = 5), toBreakNearPlayer (depth = 4), possibleToPlace (depth = 4) and 1 more functions.
Many Conditionals: Overall Code Complexity is measured by the mean cyclomatic complexity across all functions in the file. The lower the number, the better.
Complex Method: Complex implementations in: onTick (cc = 43), assemble (cc = 13), toBreakNearPlayer (cc = 13), valid (cc = 11), BuilderProcess.GoalAdjacent.isInGoal (cc = 10) and 4 more functions.. A Complex Method has a high cyclomatic complexity. The recommended threshold for the Java language is  a cyclomatic complexity lower than 9.
Excess Number of Function Arguments: Excess arguments in: possibleToPlace Functions with many arguments indicate either a) low cohesion where the function has too many responsibilities, or b) a missing abstraction that encapsulates those arguments.
Primitive Obsession: Code that uses a high degree of built-in, primitives such as integers, strings, floats, lacks a domain language that encapsulates the validation and semantics of function arguments. Primitive Obsession has several consequences: 1) In a statically typed language, the compiler will detect less erroneous assignments. 2) Security impact since the possible value range of a variable/argument isn't retricted.
Custom Rule: Detect TODOs: You have 6 occurrences of the pattern Detect TODOs in the source code file.

CodeScene Delta Analysis Results
View detailed results in CodeScene

Recommended Review Level
Detailed -- Inspect the code that degrades in code health.

Warnings
Low Code Health for new/renamed Modules

New070122.java has a low code health score with 4.21.

Improvements

Lenia.py improves its Code Health from 2.91 -> 5.63

Review of New Files
1 new file(s) below the threshold for new code (9).
0 new file(s) where the code health is above the threshold.
Lenia/New070122.java

Bumpy Road: A Bumpy Road is a function that contains multiple chunks of nested conditional logic inside the same function. The deeper the nesting and the more bumps, the lower the code health. Detected Bumpy Roads: onTick, fullRecalc, assemble, toBreakNearPlayer, searchForPlaceables and 2 more functions..
Deep, Nested Complexity: Deeply nested logic in: fullRecalc (depth = 6), searchForPlaceables (depth = 5), recalcNearby (depth = 5), toBreakNearPlayer (depth = 4), possibleToPlace (depth = 4) and 1 more functions.
Many Conditionals: Overall Code Complexity is measured by the mean cyclomatic complexity across all functions in the file. The lower the number, the better.
Complex Method: Complex implementations in: onTick (cc = 43), assemble (cc = 13), toBreakNearPlayer (cc = 13), valid (cc = 11), BuilderProcess.GoalAdjacent.isInGoal (cc = 10) and 4 more functions.. A Complex Method has a high cyclomatic complexity. The recommended threshold for the Java language is  a cyclomatic complexity lower than 9.
Excess Number of Function Arguments: Excess arguments in: possibleToPlace Functions with many arguments indicate either a) low cohesion where the function has too many responsibilities, or b) a missing abstraction that encapsulates those arguments.
Primitive Obsession: Code that uses a high degree of built-in, primitives such as integers, strings, floats, lacks a domain language that encapsulates the validation and semantics of function arguments. Primitive Obsession has several consequences: 1) In a statically typed language, the compiler will detect less erroneous assignments. 2) Security impact since the possible value range of a variable/argument isn't retricted.
Custom Rule: Detect TODOs: You have 6 occurrences of the pattern Detect TODOs in the source code file.


Version bump for Google One Tap

Version bump for the framework and OIDC modules

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3179277925

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3179277925
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3180930534

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3180930534
Status: cancelled

Closing the PR since this is updated from the dependency updator. Updated version is 5.11.0[1]
[1] https://github.com/wso2/product-is/blob/master/pom.xml#L2250


Preview geospatial data on a map (t-shirt)

Preview visualization of geospatial data on map - depends on dataset formats

adjusting placeholder based on cards that have been written

One card left to refine - reducing placeholder

Bumping back to 8, now that there are 3 cards to refine

Cards have been refined - closing out the placeholder


CC1: error: unrecognized command line option "- mavx2"

centos 6.3 install  fail OpenBLAS-0.3.7
make error
CC1: error: unrecognized command line option "- mavx2"

With the extra space between the - and the mavx2 ? I do not see this in any of the 0.3.7 makefiles, so perhaps this is in your CFLAGS or some similar environment variable ? (In any case, do you need 0.3.7 specifically, or can you update to 0.3.10 ?)

There are no spaces
Openblas-0.3.10 has the same error
Here is the compiled  error information：
cc -O2 -DMAX_STACK_ALLOC=2048 -DUSE_LOCKING -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DNO_AVX512 -DNO_WARMUP -DMAX_CPU_NUMBER=24 -DMAX_PARALLEL_NUMBER=1 -DVERSION=\"0.3.10\" -mavx2 -UASMNAME -UASMFNAME -UNAME -UCNAME -UCHAR_NAME -UCHAR_CNAME -DASMNAME=ssum -DASMFNAME=ssum_ -DNAME=ssum_ -DCNAME=ssum -DCHAR_NAME=\"ssum_\" -DCHAR_CNAME=\"ssum\" -DNO_AFFINITY -I.. -I. -UDOUBLE  -UCOMPLEX -c sum.c -o ssum.o
CC1: error: unrecognized command line option "-mavx2"

With the extra space between the - and the mavx2 ? I do not see this in any of the 0.3.7 makefiles, so perhaps this is in your CFLAGS or some similar environment variable ? (In any case, do you need 0.3.7 specifically, or can you update to 0.3.10 ?)

There are no spaces
Openblas-0.3.10 has the same error
Here is the compiled error information：
cc -O2 -DMAX_STACK_ALLOC=2048 -DUSE_LOCKING -Wall -m64 -DF_INTERFACE_GFORT -fPIC -DNO_AVX512 -DNO_WARMUP -DMAX_CPU_NUMBER=24 -DMAX_PARALLEL_NUMBER=1 -DVERSION=\"0.3.10\" -mavx2 -UASMNAME -UASMFNAME -UNAME -UCNAME -UCHAR_NAME -UCHAR_CNAME -DASMNAME=ssum -DASMFNAME=ssum_ -DNAME=ssum_ -DCNAME=ssum -DCHAR_NAME=\"ssum_\" -DCHAR_CNAME=\"ssum\" -DNO_AFFINITY -I.. -I. -UDOUBLE  -UCOMPLEX -c sum.c -o ssum.o
CC1: error: unrecognized command line option "-mavx2"

https://www.tuicool.com/articles/JBZz6b7
This might help.

I guess we could add another version test in Makefile.x86_64 - the other spots in kernel/Makefile and cmake/system.cmake already check for gcc >= 4.7 before adding the option.


Migration makes chapter unread

Steps to reproduce

Add manga 'Home Alone Romp with my Childhood Friend' from Nhentai to Library (Nhentai code 350536)
Mark chapter as read
Migrate manga to Nana (English) source

Alternatively

Add comic 'A Sunday Schooling [NLT Media]' from 8Muses to Library
Mark chapter as read
Migrate comic to AllPornComic source

Expected behavior
Migrated chapter should be marked as read (same as original)
Actual behavior
Chapter is unread
Crash logs
No response
Tachiyomi version
483
Android version
Android 12
Device
Galaxy A72
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

I believe its intentional since it cannot find a chapter number to compare the read status to, so it cant do anything to move it over


[REQUEST] Tweak Brush function

Left-click (attack) should set "First point"
Right-click (use)  should set "Second point"
Shift+Left-click should cycle between forms (Circle, Cube)
Shift+Right-click should spawn selected form
I don't think players would like using commands, on most servers commands disabled in a first place.
Also, you should add "Sphere" form. Sphere is HELLA annoyong to build.

I used commands to mimic World Edit. Also commands allow for an easy check for servers to see if they want everyone using the brush or just some. By default, deops can use the brush as well. There is also a config option to use permission check to use it. Since the brush can be laggy I left this option for server owners to control who gets to functionality.
If I were to remove commands, I would just end up making a GUI for the shape and color changing. But for now, I think I'll keep it the way it is.


Added program for bubble sort

Kindly assign above issue under hacktoberfest2022

@Bhavya1912 i am practicing dsa ,so kindly assign me this issue


fix: inclusion of helm sub commands in zarf tools help

Description
Fixing zarf tools return output to include helm when running both zarf tools and zarf tools -h
Related Issue
Fixes #2143
Type of change

[ X] Bug fix (non-breaking change which fixes an issue)

Code is good, looks like you need to fix the unverified commit


e2e poc demo video

Description
MeWe developer that wants to publish content to the chain should utilize this service to easily publish content.
Demo Script

Uploading image/audio/video via client
Show batching using chain transaction
Show in another instance of client the published content.

Here is the demo video
Private Zenhub Video

@aramikm  that was some smooth editing


Field flattens array from DB

I just upgraded to Formtastic to version 3 from V 2.
I have a model that has tags on it (acts as taggable on). To edit these in a form I have .tag_list on the model.
When I use tag_list as a field name it flattens the array and puts spaces between the values.
In version 2 in put a comma seperated list in the values.
To temporarily fix this I am doing this:
def tag_field(form, field, data, title=nil)
    values = form.object.send(field).join(',')
    form.input field, input_html: {value: values, autocomplete: 'off'}, wrapper_html: {'data-tags' => data}, label: title
end

I'm confused. Can you show me what you were trying to do with standard Formtastic methods, what the relevant HTML output was, and what output you expected?
It sounds like .tag_list isn't an array or other collection that Formtastic is expecting, or you're doing something very strange. What does .tag_list.inspect look like?

~/Sites/elf (development) > a.tag_list.inspect
=> "[\"Cricket World Cup\", \"Christchurch rates\"]"

If I do this in the form:
  <%= form.input :tag_list %>

I get this in the field:
Cricket World Cup Christchurch rates

Rolling back to Formtastic v2 I get this, same form code:
Cricket World Cup, Christchurch rates

The helper in the previous comment was just to DRY up the code; it would look like this in the form directly:
form.input :tag_list, input_html: {autocomplete: 'off'}, wrapper_html: {'data-tags' => data}, label: title

No commas to be seen with this either.

+1

@rhulse @felipeclopes
If I follow the method chain, let's see if we can figure this out. Let's start here;
form.input :tag_list, input_html: {autocomplete: 'off'}, wrapper_html: {'data-tags' => data}, label: title
I presume this ends up calling StringInput, though it's not explicit in your example and not 100% clear to me. Let's assume it does, as you seem to want a text field. This ultimately calls StringInput, which hands off most of the heavy lifting to Stringish
        def to_html
          input_wrapping do
            label_html <<
            builder.text_field(method, input_html_options)
          end
        end

As you can see, it calls out to the Rails form builder's text_field helper. input_html_options doesn't do anything to transform your value either:
        def input_html_options
          {
            :id => dom_id,
            :required => required_attribute?,
            :autofocus => autofocus?,
            :readonly => readonly?
          }.merge(options[:input_html] || {})
        end

So, my guess is that while you upgraded Formtastic, you also upgraded Rails, and that something in Rails has changed in the way text_field handles this scenario. Can confirm if you upgraded Formtastic in isolation, or if other gems (like Rails or ActiveAdmin) were involved?
I think the fix for this is to create a tag_list_as_striing accessor on your model and use that in the form, so that you have control over exactly how the tag list is transformed back and forth without fighting against Rails. I can help you with that after we can confirm if this is isolated to Rails or not.

Closing due to lack of activity. Please feel free to re-open with new information!

:+1: I have the same problem

Hi, it seem like Rails behavior changed and it displays now an array separated with spaces. @justinfrench Is there a way of defining a proc in the input definition which transforms the value? There used to be value_method but I think it was removed.
Creating this accessor like you already suggested is a solution but then you also have to write 'setter' method which we don't need here. We just need to transform the value into something else. The rest is handled by rails properly.

How about this as a workaround?
form.input :tag_list, :input_html => {:value => f.object.tag_list.join(',')}

I think this acts_as_taggable_on wiki page confirms this is a rails change rather than something specific to formtastic.

@hingerlime does that work-around work for now?

I tested very quickly, but as far as I can tell, yes. It does work. I'm working together with @joker-777 so I'll wait for him to confirm, since he's been working on this bug on our end.

Yes, it works perfectly. Thanks @gingerlime

Ok great! I'm going to leave this open for now. Would be good if someone could explore a patch that detects the acts_as_taggable column and automatically applies the work-around.

I'm not that familiar with formtastic, but should be fairly straight-forward to detect if the field is mounted by acts_as_taggable_on and apply this (as an aside, it's not usually an actual column, but an external model, which acts_as_taggable_on gem handles for you...).
I think it would be nice feature for acts_as_taggable_on / formtastic users, but this implicit workaround (especially now going against the rails defaults) might be confusing if you don't expect it? Not sure what's the formtastic "philosophy" for doing things automatically this way.

We could try to do something. Could you create a sample project reproducing the issue including a failing test case? That would help tremendously.

I wouldn't implement something which detects acts_as_taggable_on and applies a workaround at all. I think a good help page in the wiki which explains the problem and offers this solution would be totally enough.


[max_by]Result error of char/varchar/boolean/tinyint data type

Steps to reproduce the behavior (Required)

CREATE TABLE '...'

 CREATE TABLE `duplicate_table_with_null_partition` ( `k1` date, `k2` datetime, `k3` char(20), `k4` varchar(20), `k5` boolean, `k6` tinyint, `k7` smallint, `k8` int, `k9` bigint, `k10` largeint, `k11` float, `k12` double, `k13` decimal(27,9) ) ENGINE=OLAP DUPLICATE KEY(`k1`, `k2`, `k3`, `k4`, `k5`) COMMENT "OLAP" PARTITION BY RANGE(`k1`) ( PARTITION `p202006` VALUES LESS THAN ("2020-07-01"), PARTITION `p202007` VALUES LESS THAN ("2020-08-01"), PARTITION `p202008` VALUES LESS THAN ("2020-09-01") ) DISTRIBUTED BY HASH(`k1`, `k2`, `k3`, `k4`, `k5`) BUCKETS 3 PROPERTIES ( "replication_num" = "3", "storage_format" = "v2" );

INSERT INTO '....'

curl --location-trusted -u root: -T /home/disk4/lvchenyang/codebak/StarRocksTest/lib/../common/data/basic_types_data -XPUT -H label:stream_load_1668414293798 http://ip:port/api/test_max_by_d0b15080_63f5_11ed_abc4_00163e0e550b/duplicate_table_with_null_partition/_stream_load

SELECT  '....'

select max_by(k1,k3) from duplicate_table_with_null_partition;
select max_by(k1,k4) from duplicate_table_with_null_partition;
select max_by(k1,k5) from duplicate_table_with_null_partition;
select max_by(k1,k6) from duplicate_table_with_null_partition;
select k3,max(k1) from duplicate_table_with_null_partition group by k3 order by k3 desc limit 10;

Expected behavior (Required)
max_by(k1,k3) 2020-08-09
max_by(k1,k4) 2020-07-24
max_by(k1,k5) 2020-08-25
max_by(k1,k6) 2020-08-25
Real behavior (Required)
max_by(k1,k3) 2020-07-25
max_by(k1,k4) 2020-07-17
max_by(k1,k5) 2020-06-23
max_by(k1,k6) 2020-06-23

StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

+----------------------+
| current_version()    |
+----------------------+
| MAIN-RELEASE e21cd48 |
+----------------------+

max_by just find one k1 with the max value of k3, A more appropriate case should be used.


Error while running the program

I am running into this error (using windows). It is because of the version of my Chrome not being compatible?

Your current driver version is 114. Please pass driver_version=114 to constructor, it should be fine.

I am getting the same error even if I pass in the driver_version=114 to the constructor of ChatGPT_Client.

your email and password is visible on the image. I suggest removing the image and change your password.
you can explicitly pass the chromedriver path, which may solve your issue.

No, Google SSO is not yet supported.
Can you try headless=False, perhaps that can solve the problem.

Can you try headless=new, perhaps that can solve the problem.

Closed due to inactivity


Tag aliases

How would this feature be useful?
Let's say you have multiple sessions tagged with 'format', 'lint', and 'test'. Before you commit, you want to run all of these tagged sessions. There are currently two ways to do this, run nox -t format lint test, or add a new tag to all of the sessions, for example, 'check' and run nox -t check.
The problem with the first approach is its getting verbose and a developer could miss one, or not get the message when another tag gets added to the list. There are other methods of enforcement, but I feel like that's out of scope.
The problem with the second approach is it increases maintenance and it's not clear that 'check' means 'format', 'lint', and 'test'.
Describe the solution you'd like
I'd like to see tag aliases (Or pseudo tags, meta tags, tag groups ... whatever name is the most intuitive).
Tag aliases would correspond to one or more tags so that when the alias was specified, it would be as if referenced tags were specified.
A possible implementation could look like this:
nox.options.tag_aliases = {
    'check': ('format', 'lint', 'test')
}

Then nox -t check becomes equivalent to nox -t format lint test
Describe alternatives you've considered
Currently adding additional tags to sessions. Would really prefer a different approach.
Anything else?
No response

Haven't thought about it much, but quick design thought, what about a decorator? Like:
@nox.alias("check")
@nox.session(...)
def format(...): ...

@nox.alias("check")
@nox.session(...)
def lint(...): ...

? Or maybe adding them as a keyword argument to session, aliases=["check'].
Things like how they would show up in help, etc. would probably guide the design more. And I could see this being related to #631.

The problem with a decorator is the same as adding a new tag to all the sessions, it increases the maintenance burden and the intent is not explicit. I really don't see this as being related to the individual sessions as much as being related to the calling arguments.

I finally got around to creating a tool I'm calling NoxOpt. It's a very early prototype, but I think it might implement something similar to this. It has this concept of automatic tags where, if I name my sessions in a particular way, it will generate some tags based on those names. The following example would generate these tags:

check
check-python
check-javascript

from noxopt import NoxOpt

nox = NoxOpt(auto_tag_depth=2)

@nox.session
def check_python_tests(session: Session) -> None:
    ...

@nox.session
def check_python_format(session: Session) -> None:
    ...

@nox.session
def check_javascript_tests(session: Session) -> None:
    ...

@nox.session
def check_javascript_format(session: Session) -> None:
    ...

As a bonus it also lets you define options for your sessions using type annotations:
from typing import Annotated
from noxopt import NoxOpt, Option, Session

nox = NoxOpt()

@nox.session
def sum_numbers(
    session: Session,
    nums: Annotated[list[int], Option(nargs="*", type=int)],
) -> None:
    session.log(sum(nums))

$ nox -s sum-numbers -- --nums 10 3 26 4
nox > Running session my-session
nox > Creating virtual environment (virtualenv) using python in .nox/my-session
nox > 43
nox > Session my-session was successful.


feat: add "quote" slide template

Description
Add a new "quote" slide template. Example from Keynote:

This template should support a quote (quote text or title) and the author of a quote (smaller text below). Text styles should be applied from a dedicated slide theme.
Also, make sure that the template supports the footer and header based on the slide deck configuration values, and custom background (see existing templates for reference).
Requirements

[ ] Implement a new slide template
[ ] Implement a new theme for the template
[ ] Add documentation where needed
[ ] Update the example app with the new slide template example
[ ] Update the README.md file with the new slide template example
[ ] Prepare the CHANGELOG.md file for an upcoming release

can you assign this to me..

Sure, good luck!

@notpritam Feel free to use https://github.com/mkobuolys/flutter_deck/issues/21 for reference to make the collaboration more efficient.

@notpritam Feel free to check #24 for reference to make the collaboration more efficient.

Okay got it , thanks for the reference...

@notpritam Any progress or we can re-open this issue for someone else?

@notpritam Any progress or we can re-open this issue for someone else?

I am almost done, i am pushing it in few hours

@notpritam Any progress or we can re-open this issue for someone else?

I am almost done, i am pushing it in few hours

I am sorry for being pushy, but I do not even see that you have forked the repository - is it really a matter of hours?

Hi @mkobuolys, could you assign this issue to me?

Hi @mkobuolys, could you assign this issue to me?

Let's wait for the @notpritam response/progress change until tomorrow. If there is none, I will re-assign the issue to you since the Hacktoberfest is going to end soon  👍

Ok @priyavratamohan, assigning this to you - good luck!

@priyavratamohan A friendly reminder that Hacktoberfest is coming to an end.

Implemented with https://github.com/mkobuolys/flutter_deck/pull/37


Unable to update Mail Server configuration without providing a username.

Describe the bug
I am running the Artifactory Mail Server Resource. I configured the Mail Server (with a username and password) and we see the successful set up in the UI. I then update the script so that the mail server resource doesn't include a username and password (as stated in the doc it's optional), we can see in the UI that the username got removed, however we get the below error:
terraform apply
artifactory_mail_server.mymailserver: Refreshing state...

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  ~ update in-place

Terraform will perform the following actions:

  # artifactory_mail_server.mymailserver will be updated in-place
  ~ resource "artifactory_mail_server" "mymailserver" {
      - password        = "test-password" -> null
      - username        = "test-user" -> null
        # (8 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

artifactory_mail_server.mymailserver: Modifying...
╷
│ Error: Provider produced inconsistent result after apply
│
│ When applying changes to artifactory_mail_server.mymailserver, provider "provider[\"registry.terraform.io/jfrog/artifactory\"]" produced an unexpected new value: .username: was null, but now cty.StringVal("").
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.
╵

If we initially create the mail server without a username and password it doesn't result in an error. (edited)

Requirements for and issue
Terraform script:
terraform {
  required_providers {
    artifactory = {
      source  = "jfrog/artifactory"
      version = "12.3.1"
    }
  }
}

#Configure Artifactory
provider "artifactory" {
  url = "http://ARTIFACTORY_URL/artifactory"
  access_token = "TOKEN"
}

resource "artifactory_mail_server" "mymailserver" {
    enabled         = true
    artifactory_url = "http://ARTIFACTORY_URL/artifactory"
    from            =<EMAIL_ADDRESS>    host            = "IP"
 #   username        = "test-user"
 #   password        = "test-password"
    port            = 25
    subject_prefix  = "[Test]"
    use_ssl         = true
    use_tls         = true
}

Artifactory version: 7.90.5
Terraform provider version: 12.3

Expected behavior
We should be able to update the Mail Server configuration to not include a username and password.

@NoffarLevi Thanks for the report! I've added this to our plan to investigate.


[Google Play Game Services] Any tutorial available?

Apart from this link: https://pub.dev/documentation/googleapis/11.4.0/games_v1/games_v1-library.html
Do we have any other instructions on how to use this package?

Sadly, no. Everything thing here is auto-generated.
Does this help: https://developers.google.com/games/services/web/api/rest ?

There are a few different APIs:

Google Play Game Services - games_v1
The Google Play games service allows developers to enhance games with social leaderboards, achievements, game state, sign-in with Google, and more.
Documentation
API details

Google Play Game Services Publishing API - gamesconfiguration_v1configuration
The Google Play Game Services Publishing API allows developers to configure their games in Game Services.
Documentation
API details

Google Play Game Management - gamesmanagement_v1management
The Google Play Game Management API allows developers to manage resources from the Google Play Game service.
Documentation
API details

Game Services API - gameservices_v1
Deploy and manage infrastructure for global multiplayer gaming experiences.
Documentation
API details

Do I need to use all of them?
My app is an online game where you can play solo and get points. These points go to a global ranking and you have some achievements to unlock. No saved games, every time you play you get these points. Just a basic game.
So the first thing is to log in the user. I checked the list but how do I do that?
NOTE: I've got everything setup in Google Play Console and Cloud already

Also found this game made by someone working at that point in the Flutter team: https://github.com/filiph/tictactoe
This is not using this package, it uses this 3rd party package instead: https://pub.dev/packages/games_services

@erperejildo – sorry. I've only used 3-4 of these APIs directly. I have no experience with most of them. I just try to get our bindings up-to-date!

@erperejildo – sorry. I've only used 3-4 of these APIs directly. I have no experience with most of them. I just try to get our bindings up-to-date!

could you point me to someone else maybe?


iso attribute has no effect

Basically adapting the recipe from the docs for low light capturing, a change of picamera.iso has no effect. No matter if put to zero or to 800, the outcome is exactly the same (neglecting noise). Is this meant to be so? Shouldn't the iso affect the appearance?

Refer to the answers on your forum post at https://www.raspberrypi.org/forums/viewtopic.php?f=43&t=160933
ISO still allows exposure time to vary, so look at the settings recorded in the EXIF of a capture. Has it cranked the ISO up but exposure down to achieve the correct scene illumination? If the image is noiser, then probably.
NB Not a PiCamera bug if there is a bug.

Just to echo what @6by9's said: setting iso to 800 but leaving shutter speed to float usually results in similar output (the sensor gain is cranked up by the iso setting, but the camera's firmware rapidly reduces the shutter speed to maintain consistency). Setting both the iso and the shutter speed should result in some fairly significant differences.
If you want to play with this interactively, try the following gist (best run over SSH so you can see the preview on the Pi's display and the controls on your SSH client; set preview alpha if you don't want to do this):
https://gist.github.com/waveform80/7a0ca251d1c352b594c18ffb36db2588

Closing old tickets; do feel free to re-open if you've got a further query on this topic.


Error when enrolling Sentinel in simplified pricing through code.

Hi,
The documentation states to enroll into simplified pricing tiers for Sentinel thorugh code, the sku for Microsoft.OperationsManagement/solutions needs to be changed to "Unified". I have changed the value to "Unified" however I get the following error when running our pipeline: Error converting value "Unified" to type 'Microsoft.Azure.Monitor.LogAnalytics.ControlPlane.DataContracts.Contracts.AzureRm.Workspaces.Default.Sku'
Can someone tell me whats going wrong here?

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

ID: ea5c3718-0460-25f7-bfac-dce8d333a7b7
Version Independent ID: 4b5a9897-b880-4df3-cfc1-e2daf1a638ec
Content: Enroll in a simplified pricing tier for Microsoft Sentinel
Content Source: articles/sentinel/enroll-simplified-pricing-tier.md
Service: microsoft-sentinel
GitHub Login: @austinmccollum
Microsoft Alias: austinmc

@ChonoOG
Thanks for your feedback! We will investigate and update as appropriate.


fix: improve log when using alternative port

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[x] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This no longer logs if an empty array is passed in portsToCheck, and also corrects the log message so it doesn't say that it failed to find a port in the range that did produce a port.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@pi0 There seems to be a flakey test, FYI. Latest commit passes with no code changes compared to previous failure. Maybe we could increase timeout? 🤔


Unhandled Exception Occured

I'm getting the error (first pic) but when I add an M1 (2nd pic, G-code line 5), I don't get the error.
Any ideas on how to solve this?

I have tested with a STM32F756 board (telnet and websocket connection) and I am not able to replicate. I do not have a Teensy available at the moment, awaiting delivery of a new one, so cannot check that.
Does the same happen with a USB and/or a websocket connection (assuming you are using telnet)?

Is that on a Teensy?  If you post the GCode, I will give it a try.

Yes, that's on a Teensy 4.1 with your V2.09 board.
Here is the G-Code.
%
G90 G94 G17 G49 G40 G80
G21
(2D CONTOUR2)
G54
S10000 M03
M08
G00 X-33. Y0.
G00 Z9.
Z1.
G01 Z-11. F500.
X-28. F500.
Y25.
G02 X-25. Y28. I3.
G01 X25.
G02 X28. Y25. J-3.
G01 Y-25.
G02 X25. Y-28. I-3.
G01 X-25.
G02 X-28. Y-25. J3.
G01 Y0.
X-33.
G00 Z9.
M05
(DRILL1)
M09
M01
G55
S5000 M03
M08
G00 X0. Y0.
G00 Z22.
Z2.
G98 G73 X0. Y0. Z-11. R1. Q2. F500.
G80
G0 Z22.
M30
%

OK, tried it.  Ran ioSender 2.0.42.  Teensy 4.1 on a T41U5XBB with a 20230501 build.  USB and Web Socket connections both ran the program and did not get an exception. Note, this is on a board that is not installed in a machine though that probably doesn't matter.
I'm kind of stabbing the dark but... M1 puts the controller into the Hold:0 state and doesn't act as a toggle.  Strange that it would allow your program to continue.  What build level of grblHAL are you running?  (Console Ver statement)

I I'm on ioSender 2.0.42. with Teensy 4.1 on a V2.09 T41U5XBB.
Here is what the console shows
[VER:1.1f.20230501:]
[OPT:VNMSL+,35,1024,4,0]
[AXS:4:XYZA]
[NEWOPT:ENUMS,RT+,ES,REBOOT,EXPR,TC,SED,RTC,ETH]
[FIRMWARE:grblHAL]
[NVS STORAGE:*FLASH]
[DRIVER:iMXRT1062]
[DRIVER VERSION:230331]
[DRIVER OPTIONS:USB.1]
[BOARD:T41U5XBB]
[AUX IO:4,3,0,0]
[IP:<IP_ADDRESS>]
[PLUGIN:MODBUS v0.14]
[G54:0.000,0.000,0.000,0.000]
[G55:0.000,0.000,0.000,0.000]
[G56:0.000,0.000,0.000,0.000]
[G57:0.000,0.000,0.000,0.000]
[G58:0.000,0.000,0.000,0.000]
[G59:0.000,0.000,0.000,0.000]
[G59.1:0.000,0.000,0.000,0.000]
[G59.2:0.000,0.000,0.000,0.000]
[G59.3:0.000,0.000,0.000,0.000]
[G28:0.000,0.000,0.000,0.000]
[G30:0.000,0.000,0.000,0.000]
[G92:0.000,0.000,0.000,0.000]
[TLO:0.000,0.000,0.000,0.000]
[PRB:0.000,0.000,0.000,0.000:0]

I see three differences from the build I used to test:

I set up 3 axes, not 4
In [NEWOPT...] we match up except for EXPR. Not sure which plugin that is. Maybe try with out it?

I've used the web builder to generate the .hex. It seems I was selecting an option that was throwing things out. I'm not sure which one, possibly one in the advanced features.
Thanks for your help.


className mismatch error on refresh

I downloaded the with-styled-components example from the repo and ran yarn install & yarn start. All fine.
After making a code change the HMR works fine and the browser is updated.
But refreshing the browser after a code change, results in an error.
Warning: Prop `className` did not match. Server: "sc-bwzfXH eaLeSp" Client: "sc-bdVaJa igGFgA"

I am not sure what the expected behaviour is after a refresh.

Try it in production mode. As far as I can tell, the mismatch only happens when reloading a page that was updated by HMR. Rarely a serious in dev and not applicable in prod.

Yes, the error occurs only while developing and not in production. Since you don't have HMR there.
But I am wondering what the underlying bug is. Why is there a mismatch after I refresh?
The client bundle that the server uses is probably not updated.
In theory it doesn't have to be because we use HMR and therefore the original script doesn't have to be updated. But why not keep it in sync to avoid the error?

I fought with this for a few hours but gave up because it didn't bother me enough.
I think it's because the class name generator can only be instantiated identically on the first load. On subsequent loads, only the modified module (the M in HMR) seems to be run through the generator and all the other unmodified modules are left out and none of the server is re-run through the generator.  I think I saw this in the integer suffixes of the class names too.
I suspect you might have to feed all the HTML through the same generator again including both server and unmodified client modules in the identical sequence as you do on first load.
Let us know if you make progress on this.

Any progress?

My work around is to hit rs on the server terminal to restart it. Loads after that are perfect until the next HMR.
@jaredpalmer  The rs reload is so fast, it would add little time to the HMR process to just restart after each HMR. Could we add an option for that? I think that resart is what's running all of both client and sever through the same generator and correcting it.

Bump on this issue, seeing the same thing. Definitely seems related to HMR. Though something even using the rs command or restarting completely shows the same issue. I'm shooting in the dark here, razzle is an incredible piece of software but issues like this are really scary as an amateur JS dev.

I've set it up like this and I get zero errors or warnings. Maybe this will help.
client.js
hydrate(
  <ThemeProvider theme={theme}>
    <BrowserRouter>
      <After data={data} routes={routes} />
    </BrowserRouter>
  </ThemeProvider>,
  document.getElementById('root'),
);

server.js
server
  .disable('x-powered-by')
  .use(express.static(process.env.RAZZLE_PUBLIC_DIR))
  .get('/*', async (req, res) => {
    const customRenderer = (node) => {
      const App = (
        <ThemeProvider theme={theme}>
          {node}
        </ThemeProvider>
      );

      return {
        html: renderToString(App),
        serverState,
      };
    };

    try {
      const html = await render({
        req,
        res,
        routes,
        assets,
        document: Document,
        customRenderer,
      });
      res.send(html);
    } catch (error) {
      res.json(error);
    }
  });

Document.js
export default class Document extends React.Component {
  static async getInitialProps({ assets, data, renderPage }) {
    const sheet = new ServerStyleSheet();
    const page = await renderPage(App => props => sheet.collectStyles(<App {...props} />));
    const styleTags = sheet.getStyleElement();

    return { assets, data, ...page, styleTags };
  }

  render() {
    const { helmet, assets, data, styleTags, serverState } = this.props;
    const htmlAttrs = helmet.htmlAttributes.toComponent();
    const bodyAttrs = helmet.bodyAttributes.toComponent();

    return (
      <html {...htmlAttrs}>
        <head>
          <meta httpEquiv="X-UA-Compatible" content="IE=edge" />
          <meta charSet="utf-8" />
          <title>React Prime AfterJS</title>
          <meta name="viewport" content="width=device-width, initial-scale=1" />
          {helmet.title.toComponent()}
          {helmet.meta.toComponent()}
          {helmet.link.toComponent()}
          {styleTags}
        </head>
        <body {...bodyAttrs}>
          <AfterRoot />
          <AfterData data={data} />
          <script
            type="text/javascript"
            src={assets.client.js}
            defer
            crossOrigin="anonymous"
          />
        </body>
      </html>
    );
  }
}

@sandervspl So you don't get a className mismatch even when you modify a component that uses withStyles, HMR reloads, and then you reload the page?


Close blocks if called twice in Go Client

Describe the bug
Calling Client.Close() twice blocks due to the presumably since the underlying CloseAsync blocks.
To Reproduce
Initialize a client and call Close twice successively. The second call will block indefinitely.
Expected behavior
Subsequent calls to Close should be a no-op or at least an error statement the client is already closed.
Screenshots
n/a
Desktop (please complete the following information):

macOS when using the Go client
Linux (the apachepulsar/pulsar Docker image) for the Pulsar server

Additional context
n/a

This appears to be the case for producers as well (and I would presume consumers).

I'd like to take a shot at fixing this

Closed as stale.
The development of the (reimplemented pure) Go client has been permanently moved to https://github.com/apache/pulsar-client-go.


Unbound in version 1.10.1 is not respecting the external maximum ttl.

When using unbound in version 1.10 on my Centos 7 server, the external TTL is not respected, causing some problems in the generation of the server cache.
There are some zones that have a TTL greater than 86400 and this is not respected in Unbound, generating a higher consumption of external requests and some blocks in RBL providers.
Until then I did not explicitly set the cache-max-ttl parameter in the config and with that the external TTL was not being respected.
After I insert the following config in the unbound, the external ttl is respected again
cache-max-ttl: 864000

Hi,
This is not a bug. Unbound specifically set limits on the allowed min and max TTL values as per https://datatracker.ietf.org/doc/html/rfc1035#section-7.3.
Based on what you are trying to achieve you can either use a higher cache-max-ttl as you have done.
Starting from version 1.13.1 you can also use serve-original-ttl: yes. This implicitly disables the min and max limits but it always returns the original TTL (without decrementing). As documented in the man page this is quite a unique option and not fit for normal DNS resolving; mainly for research.


basic routing rules

This PR resolves #18
This PR adds some basic routing functionality to the mavp2p tool. I tried my best to follow the standard as defined in - https://mavlink.io/en/guide/routing.html
I apologies for my Go skills, its my first time using Go!
The following rules are iterated over every connected node, every time a message is received:

If the message is of type COMMAND_LONG, COMMAND_ACK or COMMAND_INT then, read the target_system and target_component from it and mark the routeMsg variable true. Otherwise broadcast it.

if target_system is more than 0 (non-broadcasted) then route it, otherwise broadcast it
        if routeMsg { if targetSystem > 0 { // Route only if it's non-broadcast command

if target_system matches the system id of one of the connected nodes then proceed with routing, otherwise discard
          for remoteNode := range nh.remoteNodes { // Iterates through connected nodes
             if remoteNode.SystemID == targetSystem {

If target_component matches a node or set to broadcast then route, otherwise discard
                if remoteNode.ComponentID == targetComponent || targetComponent < 1 { // Route if compid matches or is a broadcast

Does a check that the message is not addressed to itself, if it is then it discards and prints a warning
                    if remoteNode.Channel != evt.Channel { // Prevents Loops

Finally, if all of these criteria are met then the message is sent to the node
                        node.WriteFrameTo(remoteNode.Channel, evt.Frame)

When the cli arg --print-routes is used we can see the behaviour of the routed frames. Eg, here is the interaction of a GCS connecting to a camera video stream using this camera protocol - https://mavlink.io/en/services/camera.html#video_streaming. The COMMAND_LONGS are replied with COMMAND_ACKS etc. (I censored our public IP here)

Known Issues that should be fixed before merging:

we only check a set of 3 messages if they have a target_system field. I’m not sure right now how to check for every incoming message.
This routing goes against what @aler9 mentioned about:

some nodes are meant to listen on all incoming traffic for saving telemetry, storing logs, broadcasting telemetry, debugging reasons. So there should be a way to keep the actual behavior on these nodes.

So we'd need to allow for some nodes to be marked as loggers or listeners or whatever...

Hello, i realized that the router already performs some filtering by stopping stream requests messages. Therefore, what i wrote about global loggers is not a current feature, and there are no issues in filtering messages by system ID / component ID and merging this PR. Sorry for the time it took but i had to re-immerse myself into the Mavlink world, since i was away from it for a couple of years.
I added tests and merged this PR, thank you very much.
I'll also add two more features in the near futures:

if the target node is not found (because the target node has not sent a heartbeat yet), the message will be broadcasted globally
routing will be extended to all messages with a TargetSystem and TargetComponent field


equal and like LINE 44

LINE 44

sprintf('WHERE "%s" %s ?', $id, (ctype_digit($data) === true) ? '=' : 'LIKE'),

ctype_digit is checking if string = only digits
if true it get a equal otherwise a LIKE

situation

tbl_name, col_name and entry = 00001-Name

so i do a request for 00001

expected

SELECT * FROM tbl_name WHERE col_name LIKE '%00001%'
result: 1

get

SELECT * FROM tbl_name WHERE col_name = '00001'
result: 0

database type

mysql

perhaps you should change the /(#any)/(#any)/(#any) on #Line 39 to /(#any)/(#any)/(#operator)/(#any)

@Midako Were you able to find a fix or solution? I am still looking for it, please help me out if you have got it.


Question: where to place prepopulated database file on Android?

For iOS it's working fine to place the prepulated database within "www" folder and it to the Project, that it will appear also in "Copy bundle resources".
But what are the necessary steps to include the prepopulated database file also for android?
I tried to put the www folder under "android/app/src/main/assets", but it won't work.

That's where it should go.

When obmitting the "www"-Folder and just place the sqlite-file direct under android/app/src/main/assets, everything works as expected 👍

it's like throwing darts...which openDatabase call did you use when you file is located under assets and how does the filename related to dbname?


fish: An error occurred while redirecting file '/Users/koofrank/.rbenv/version'

when I use fisher install plugins, then show:
fish: An error occurred while redirecting file '/Users/koofrank/.rbenv/version'
open: No such file or directory
and I use fisher any command line doesn't work, so I how to uninstall fisher and fix this problem, I use fisher uninstall not work

@phpmaple Thanks, I don't use rbenv so I have never had this problem before, but @daenney, can you share some of your rbenv wisdom with us here?

@phpmaple How do you do the rbenv stuff? With the normal rbenv integration or using the fisherman rbenv plugin?

@daenney When I use fisher install some plugins, Then show this error, and then fisher doesn't work

@phpmaple I think the folder is created when rbenv the utility, not the fisherman plugin, is installed, so if you don't have it installed, then an error will be thrown. In any case, the issue is not fisherman-specific.

@jethrokuan I uninstall  rbenv, then everything is ok

@phpmaple do you confirm that you have https://github.com/rbenv/rbenv installed?

@jethrokuan  I sure

@phpmaple Thanks! Close if your problem is resolved.

It looks like the bobthefish theme is expecting to find the global ruby version in ~/.rbenv/version but it's not there. I had this same problem too (coming from having regular rbenv+fish to fish+fisherman with the rbenv plugin). The relevant lines of fish_prompt.fish are:
   # Don't show global ruby version...
  set -q RBENV_ROOT
    or set -l RBENV_ROOT $HOME/.rbenv

  read -l global_ruby_version <$RBENV_ROOT/version

However, I did not have the version file in my ~/.rbenv folder
For me, the following resolved the issue, though I'm not sure if it's a proper solution:
rbenv version-name > ~/.rbenv/version

@rnorth I have forked, bobthefish here, I don't use it myself, but since it's quite popular I've put it there hoping to fix some of its bugs in the future.

Everyone still having problems:
Replace
# Don't show global ruby version...
set -q RBENV_ROOT
  or set -l RBENV_ROOT $HOME/.rbenv

read -l global_ruby_version <$RBENV_ROOT/version

with
set global_ruby_version (rbenv global)

Per https://github.com/rnorth/bobthefish/commit/ba3b165371ff90ef916ac008a25f60f62f56c84a

The above bgbthefish fix is a package specific fix but to fix your whole RBENV system see the following:
So it looks like after you install RBENV there is no ~/.rbenv/version file. To create it just run rbenv global system . Then that file will be created and everything will work.
To change the version from system just run something like rbenv global 2.4.1 (of course after you've installed version 2.4.1 using rbenv install 2.4.1).


Change denial to tightness

Per the Office of Research, all instances of "inferred denial index" or its short-forms should be changed to "credit tightness index".
This mirrors changes made to the data processing pipeline.
Changes

Renames denial to tightness and other similar names in the block, template, tests, and testing fixtures
Adds data migration

Testing

Check out the branch
Test via tox -e fast

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the development playbook
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged / future todos are captured in comments
[x] Visually tested in supported browsers and devices (see checklist below :point_down:)
[x] Project documentation has been updated
[x] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer 8, 9, 10, and 11
[ ] Edge
[ ] iOS Safari
[ ] Chrome for Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

@chosak Is the included data migration insufficient?

@hillaryj yes, the included auto-generated migration is a schema migration, not a data migration. The schema migration includes only changes to the field schema (the makeup/format of the StreamField blocks). This won't do anything to alter the field content that already exists in the database.
For example, if the database field contains a block content like
[{'type': 'data_snapshot', 'value': {'denial_month': u'2015-01-01', ...}, ...]

and we now want it to be
[{'type': 'data_snapshot', 'value': {'tightness_month': u'2015-01-01', ...}, ...]

we have to write a manual script (the data migration) to alter the content. We have a few examples in the codebase -- here's a recent one. We do have some (possibly out-of-date) documentation on writing these, and I'd be happy to help as well. @Scotchester is also a data migrations expert by this point 😄.

Okay, cool - since the data in the denial_ prefixed fields doesn't matter, is that still a data migration?

Looks good to me pending changes based on Andy's comments

@hillaryj great question. If all of these things are true, then you shouldn't need a data migration:

Existing page denial_ fields are obsolete and can be left behind in the DB as they're not going to be displayed on pages any longer.
New page tightness_ files are all optional, and you are going to manually go in and enter them on any new pages where they need to go.
There aren't any live pages that are currently showing denial_ fields and need to be showing tightness_ fields and would be missing production data between steps 1 and 2.

If live pages exist with denial_ data and you're going to have missing data in production until it is converted to tightness_ (especially if the new fields are required), then a data migration would be needed.

Excellent - we meet all of those criteria! Thanks for the thorough explanation!


Detect existing SW in website and include infoin API response

Keep this changes as a PR since merging to master will launch a deployment in preview environment (pointed to PUBLIC pwabuilder website)

@JuanAr,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


How to implement IP blacklist？

What is your question about gnet?
How to implement IP blacklist？
I want to ban specific IP connections

You can do that via iptables: https://www.e2enetworks.com/help/knowledge-base/how-to-block-ip-address-on-linux-server/
If you must do it in gnet, use c.RemoteAddr() to get the client's address and block it.

I

What is your question about gnet?
How to implement IP blacklist？
I want to ban specific IP connections

I did like this:
var (
        clientBlock   = sync.Map{}
)

func init() {
    clientBlock.Store(`<IP_ADDRESS>`, 0)
}

func (cs *server) OnOpened(c gnet.Conn) (out []byte, action gnet.Action) {
    ip, port, err := net.SplitHostPort(c.RemoteAddr().String())
    if err != nil {
        log.Error(err)
        action = gnet.Close
        return
    }

    num, ok := clientBlock.Load(ip)
    if ok {
        action = gnet.Close
        clientBlock.Store(ip, num.(int)+1)
        return
    }

    // ...
}

So are there any questions?


[CT-2279] [Feature] add adapter response message to terminal LogTestResult

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Tests can be a real money drainer in BQ since the billing is based on bytes processed. We can configure lots of tests without knowing how much they are costing and have a surprise with the bill at the end of the month.
@aezomz added the adapter_response to tests in #6645, making it possible to better monitor bytes processed by tests.
I am proposing to also show the bytes_processed, through the adapter_response message, in the LogTestResult in the terminal, the same way as it is shown for models. In this way, developers can more quickly and easily monitor test costs, and detect if there is an unexpectedly expensive test being run.
So, instead of this

We would have something like this

As this is a change I think must be made in dbt-core, not in dbt-bigquery, I chose to include the full adapter response message, not only the bytes_processed, because other adapters don't have it.
Describe alternatives you've considered
I modified the following files:

proto_types.py
types.py
test.py

To include the adapter response message in the LogTestResult
Who will this benefit?
Mainly for developers using dbt-bigquery.
Are you interested in contributing this feature?
Yes, if this is considered a good issue I can open the PR.
Anything else?
No response

On principle, I'm 100% for it.
If we're to make that help_wanted, I need a sanity check from @jtcohen6 that how we're doing that for models is portable to tests.

@emmyoop weren't you looking at logging recently? Just looking for a quick sanity check before marking that help_wanted and getting @bruno-szdl to take a pass at it. I don't want anybody losing their time here ;)

Hmm. The equivalent to this for other commands is:
https://github.com/dbt-labs/dbt-core/blob/311a57a21e61955904202bdb469c2077202e9073/core/dbt/task/run.py#L186-L204
Specifically this line:
https://github.com/dbt-labs/dbt-core/blob/311a57a21e61955904202bdb469c2077202e9073/core/dbt/task/run.py#L192
Rather than including the full adapter_response in the log output, the adapter can choose what string gets passed into result.message! Here in dbt-bigquery is where the logic for that is defined, including the rules about when/how to include bytes_processed.
I'd prefer to take a consistent approach here, if possible.


Export to and from non-default database isn't possible

System information:

Operating system (distribution) and version win10
DBeaver version 21.2.2 CE

Connection specification:

Database name and version Netezza

Describe the problem you're observing:
Table export to the database that isn't set as a default database in connection properties isn't possible
Steps to reproduce, if exist:

Start to perform export to database that isn't set in connection properties OR from CSV to Database or to csv/xml/json/etc.
Check DDL, only Schema and target table are shown.
Start export.

Actual result
Target table is created, but no data is added due to error (TEST database is set as a default in connection properties)

But export was performed to Customers DB

Target DDL for created table:
CREATE TABLE ADMIN.ARTIST12 (
"ArtistId" INTEGER NOT NULL,
"Name" VARCHAR(120)
);
Include any warning/errors/backtraces from the logs

fixed in the scope of #14057


Bonobo Git Server with Git 2.26

Hi,all
I try to make a Bonobo Git Server (IIS+ASP.NET) with git 2.26. I success full for installl (run on browser, login account admin and create new repositorie, one account. After that i use  git clone with command below:
git clone http://<IP_ADDRESS>/newgit/Test.git or git clone https://mydomain/newgit/Test.git (ssl is configed OK).
C:\Users\Blood Tear>git clone http://<IP_ADDRESS>/newgit/Test.git
Cloning into 'Test'...
Username for 'http://<IP_ADDRESS>': test
Password for<EMAIL_ADDRESS>fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
C:\Users\Blood Tear>
Error on https
C:\Users\Blood Tear>git clone https://mydomaint/newgit/Test.git
Cloning into 'Test'...
Username for 'https:/mydomaintt': test
Password for 'https://test@mydomaint':
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
C:\Users\Blood Tear>
Some information:
Git Server: https://bonobogitserver.com/ (default Git use for server is
D:\GitServer\App_Data\Git>git --version
git version 1.9.5.msysgit.1
D:\GitServer\App_Data\Git>
I try to make a new Git folder with git was installed from (https://git-scm.com/download/win) (bin folder)
C:\Program Files\Git\bin>dir
Volume in drive C is
Volume Serial Number is
Directory of C:\Program Files\Git\bin
03/23/2020  02:23 PM              .
03/23/2020  02:23 PM              ..
02/19/2020  03:25 PM            43,592 bash.exe
02/19/2020  03:25 PM            43,072 git.exe
02/19/2020  03:25 PM            43,592 sh.exe
3 File(s)        130,256 bytes
2 Dir(s)  53,795,102,720 bytes free
C:\Program Files\Git\bin>
Please help me for solved this problem

I met the same problem, I reinstalled the  Bonobo Git Server  to fix it

@willdean Do you know if anything has been done at all regarding upgrading the version of Git in App_Data then?
I'm not that experienced with ASP.NET itself but if it's just fixing stuff that breaks after dropping in a new version I might be able to have a look.

I think there are two separate parts to upgrading Git in Bonobo:

Upgrading the Git tools in app_data, which is probably mostly just a matter of replacing the files with up-to-date versions.
Upgrading LibGit2Sharp

I think you could do '1' on its own, make it work, and then Git would work OK as a Git server, but you might (I don't know) find that the very old version of LibGit2Sharp we're using would have incompatibilities with stuff the new version of the tools did.
I am not sure how much work is involved in upgrading LibGit2Sharp (it wouldn't be very difficult to just upgrade the Nuget packages and see what happened...)

@willdean It's been a while :weary:
I did take a look at this last night just to see whether I could get something working - I did notice though that msysgit is obsolete. The contents of App_Data\Git is flat, whereas as far as I can tell, the Git for Windows download has relative dirs baked in the executable.
I don't mind copying the whole thing in.. however it does feel somewhat messy, especially as I reckon there's probably quite a lot of extra programs that aren't really necessary (the current Git install contains antiword.exe which probably is never used?)
Maybe I will have to play around to see how much stuff is actually needed from Git for Windows once it works?


Use SQLite conn.interrupt() instead of sqlite_timelimit()

Refs #1270, #1268, #1249
Before merging this I need to do some more testing (to make sure that expensive queries really are properly cancelled). I also need to delete a bunch of code relating to the old mechanism of cancelling queries.

I can remove this code too:
https://github.com/simonw/datasette/blob/6f41c8a2bef309a66588b2875c3e24d26adb4850/datasette/database.py#L190-L192

This isn't actually working - the outer code attempts to send an .interrupt() call to the connection object via the connections thread-local, which doesn't work because it's a thread-local so the connection isn't visible to that code.
Need to figure out how to communicate with that thread properly.


how to implement Central to charging request like RequestStartTransaction

how to invoke functions from central to charging point like for making
RequestStartTransactionRequest

I guess you want to send a 'RemoteStartTransaction'. That is explained in both #62 and #81 . Also #7 explains how to send a request from the central system to the charge point.

ok. thanks

I am struck here.. My use case is at any time the CSMS needs to send message to Charge point
how to initiate from central to CP like RemoteStart in the middle
not during connect  as
await asyncio.gather(cp.start(), cp.get_configuration())

I think this question is answered in this comment in #86 .


"Configuring experimental options" is not in Windows Git installation steps

Upcoming PR will suggest to not enable either.


Add info to webpage how to get in touch

is the page title okay? or should i use just "Contribute"?

I think such a page makes sense. Three ideas:

Should we mention that we've more specific documentation at https://voxpupuli.org/docs/ ?
Should we mention how people can contact us (like IRC/mailinglist/slack)?
Mention that every project has a contributing guideline with more specific instructions?

is the page title okay?

A "long" title for the page looks fine to me.  Maybe if we add it as a menu item in https://github.com/voxpupuli/voxpupuli.github.io/blob/0c25777c12f2317b8040cbe467d772a0741efce7/_config.yml#L27 we can use only "Contribute".

Should we mention how people can contact us (like IRC/mailinglist/slack)?

It is in the footer of all pages.  I feel like this is enough but I am probably biased.


Feature: enhancing the readme by adding  a troubleshooting section

Is your feature request related to a problem? Please describe.
Currently, the README lacks a dedicated troubleshooting section, which makes it difficult for users to find solutions to common issues they may encounter while using or setting up the project.
Describe the solution you'd like
Adding a "Troubleshooting" section to the README that lists common issues users may face, along with solutions or steps to resolve them. This could include installation errors, common bugs, and configuration problems, with clear instructions on how to address each.
Describe alternatives you've considered
I considered adding troubleshooting tips scattered throughout the README, but a dedicated section would make it easier for users to navigate and find the help they need quickly.
Additional context
Adding this section could improve user experience and reduce the number of repeated issues reported in the repository. I can contribute by drafting the initial troubleshooting content if needed.

please assign me this issue, thank you!

this is not something we looking for atm


Apparent incompatibility with 6.5.0

I don't have a minimal test case yet, but this babelrc works pre-6.5.0, and throws this error with 6.5.0:
ModuleBuildError: Module build failed: SyntaxError: /project/ui/views/Component.jsx: Decorators are not supported yet in 6.x pending proposal update.

The .babelrc:
{
  "presets": [
    "es2015",
    "stage-0",
    "react",
    "async-to-bluebird"
  ],
  "plugins": [
  ]
}

Edit: Appears at this time that putting the plugin in the babelrc (and not in babel-loader's query) fixes the issue.

+1

I can't seem to reproduce this. Would you be up for putting together a reproducible example repository?
To verify, you have no Babel config in your webpack config beyond the basic enabling of the loader?

https://github.com/STRML/babel-broken-legacy-decorators

I can confirm that I have experienced this issue as well

I've got a fix up in PR: https://github.com/babel/babel/pull/3348

This should now be fixed in<EMAIL_ADDRESS>


[item] Fix indexing work types

Fixes https://jira.sng.sk/browse/WEBUMENIA-1291
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have updated the CHANGELOG
[x] I have requested at least 1 reviewer for this PR
[ ] I have made corresponding changes to the documentation

Oh, i should have thought about this too when discussing it with you.

yes, usually we can split the values, like "maliarstvo, koláž" or "úžitkové umenie, sklo, nádoba"
and keep them separate
but sometimes, you can't do that, at the same level of "hierarchy" you have terms like"maliarstvo, funkčné"  - the order needs should reverted into "funčkné maliarstvo", the comma doesn't imply hierarchy, it's a remainder of an obsolete rule for writing the values in the vocabulary.
in that case, you wont solve this by path hierarchy tokenizer, because it won't tell you when to separate the value by comma and when to keep it as one (or revert the order)

I think we have to discuss this first before releasing it, or postpone. We should also fix it at the source - edit the values in CEDVU.


Support for language parameter

Please add support for the optional language parameter as supported by the Google Place Autocomplete.

language — The language in which to return results. See the supported list of domain languages. Note that we often update supported languages so this list may not be exhaustive. If language is not supplied, the Place Autocomplete service will attempt to use the native language of the domain from which the request is sent.

That param is not supported. This component uses the google.maps.places.AutocompleteService.getPlacePredictions call. See the reference for allowed params.

Thanks for the link, found out from there that the way is to use:
<script src="https://maps.googleapis.com/maps/api/js?libraries=places&language={language}"></script>


implement getuserSyncs

Type of change

[ ] Bugfix
[x] Feature
[ ] New bidder adapter
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Does this change affect user-facing APIs or examples documented on http://prebid.org?
[ ] Other

Description of change
Implemented getUserSyncs

contact email of the adapter’s maintainer<EMAIL_ADDRESS>

@ym-ainiguez Looks good, we can open a PR in prebid.js repository with these changes and once it's merged can close this ticket.

@desidiver lets first make sure there are no more updates left on the adserving side https://yieldmo.atlassian.net/browse/FS-10320?focusedCommentId=124859


Combine DAI transfers in mint()

Scott will be sending us code to accurately calculate cDAI amount minted, which will make this possible.
Will also allow us to use saveDaiPriceInDaiCurrent in mint

From Scott:
Here is the code i mentioned related to rounding.
https://github.com/Keydonix/dai-hrd/blob/master/contracts/DaiHrd.sol#L572
This is from a project I worked on for tokenizing the DSR, and needing to join/exit based on the value of the DSR, which was constantly going up; very similar to cDAI.
It sould be something like this:
function _getCostofAsset(uint256 _amount) internal returns (uint256) {
        // calculate DAI needed to mint _amount of cDAI
        uint256 exchangeRate = cDai.exchangeRateCurrent();
        emit ExchangeRate(exchangeRate);
        // return _amount.mul(exchangeRate).div(10**18);
        return _amount.mul(exchangeRate).add(10**18-1).div(10**18);
    }

Adding one less than the number you are dividing by ensures that, if it is an exact multiple of 1018, it will round that addition right back down. any other non-divisible-by-1018, it will get rounded down from the next number up.

@kseniya292 You want to update mint to return a boolean when addressing this ticket?

@dougiebuckets Yes. I can do all the tickets involving the mint function.


Deduce size_type from size() for SizeIs()

I was trying to use SizeIs) on an object that provides a size() member function but no size_type member type alias.  It looks to me like it would be pretty straightforward to deduce the size_type from the return value of size() by using decltype(std::declval().size()).  This makes the SizeIs matcher a little more flexible.
public:
 typedef internal::StlContainerView<
      GTEST_REMOVE_REFERENCE_AND_CONST_(Container)> ContainerView;

typedef typename ContainerView::type::size_type SizeType;

typedef decltype(std::declval().size()) SizeType;
explicit Impl(const SizeMatcher& size_matcher)
: size_matcher_(MatcherCast(size_matcher)) {}

Fixed with https://github.com/abseil/googletest/commit/3d71ab4c37de5f13da905a4b60c127176160537d


fix: [Do Not Merge]update bf-lu version to resolve app settings missing issue in luis publishing

Description
Upgrade bf-lu package to resolve app settings missing issue in luis publishing
Task Item
Closes #5732
Screenshots

Coverage remained the same at 55.05% when pulling 873e278172c13850d07aa4c751f9210e44a91939 on feich/UpdateBFLULibrary into fef792908912e6f6d1e0a795e45d406026f270b2 on main.

@chrimc62 In this PR we are upgrading bf-lu which in turn upgrades the bf-cli to incorporate the microsoft/botframework-cli#1108.

Hi @srinaath, I have another bf-lu fix that was just merged a few hours ago and the nightly build supposed to be released soon. I will update this PR to that latest bf-lu version once it comes out so that the composer can consume all the latest R12 bf-cli fixes. I will remove [Do Not Merge] in title once it's done. Thanks.

This PR is ready for merging.

@feich-ms can we pick the latest bf-lu and get this PR merged.

@srinaath I have updated the bf-lu package version to the latest 4.12.0-rc0. But the Docker Build Validation and Security Analysis validation failed in CI, do you have any insights how to make them pass.

@feich-ms Lets get it merged. The docker build failure is not because of this PR.

@feich-ms it looks like static-eval 2.0.2 is the comp governance issue. Can u use yarn resolutions to update it? Right hereBotFramework-Composer/package.json at main · microsoft/BotFramework-Composer (github.com)

@srinaath the static-eval 2.0.2 package is in runtime/node folder that is using npm to manage the packages. I tried add the resolution in runtime/node/package.json, but it seems npm doesn't support resolutions. I also tried added it to the resolution of Composer/packagejson, but it yarn install didn't update the package-lock.json in runtime/node folder. Do you have any suggestions?

@srinaath we did much investigation today. It looks like not a good choice to switch from npm to yarn for node runtime package management since that will require users to install yarn additionally if they eject js runtime. To fix the static-eval 2.0.2 package issue completely, we suggest to fix that from the root package that used statuc-eval. Here is the fix PR from js sdk https://github.com/microsoft/botbuilder-js/pull/3331. We already asked @chon219 to cherry-pick the fix to next js sdk rc release. After the fix is out, we can update the packages in runtime/node from composer side to consume the latest js sdk which will not have dependency on static-eval any more.


Apply default black code style

Apply black coding style for consistency - would be more convenient if dev's would apply this themselves before committing.

for the tutorial examples the md files would have to be changed since the code is automatically extracted. We may get back to this followup problem when the demostarter works which is dependend on #498


Minor lint feedback

Misspelled words, missing header, and small optimization to remove inadvertent copies.

Thanks Michael!