rework append_if_no_line to use file provider subresource

there's a ton of the guts of the file provider missing if you don't
do this.

          2 Errors

:no_entry_sign:
Please add labels to this Pull Request

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger

one more bit incoming

@damacus okay shippable

This is awesome thank you.
If I had given myself some headspace Im pretty sure I would have just nicked the code from another project I had (oh the irony) that I wrote.
I'll have a proper look over this, this weekend and merge both pull requests.

^ i came up with a non-slurpy version of append_if_no_line but its pretty ugly -- without bug reports i don't think it'd be worth it.


Added validation in login and sign-up page

Related Issue

Closes: #133
Description of Changes
I have added validation for every inputs in both sing-up and log-in pages. Now user have to enter real data for logging into the website
Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

@sourabhsikarwar Hello
Have you checked my PR :) ???

Thanks @sourabhsikarwar
Please update the lables under gssoc 😀


Wrong usage in example.

At readme.md
In example there is --fbpass when your script takes --fbpassword as argument.

I made a pr bout that. :)

hey thanks for the PR. I didn't expect that the script still works :grin:


Proxy method

-x, --proxy [protocol://]host[:port] Use this proxy
     --proxy-anyauth Pick any proxy authentication method
     --proxy-basic   Use Basic authentication on the proxy
     --proxy-cacert <file> CA certificate to verify peer against for proxy
     --proxy-capath <dir> CA directory to verify peer against for proxy
     --proxy-cert <cert[:passwd]> Set client certificate for proxy
     --proxy-cert-type <type> Client certificate type for HTTPS proxy
     --proxy-ciphers <list> SSL ciphers to use for proxy
     --proxy-crlfile <file> Set a CRL list for proxy
     --proxy-digest  Use Digest authentication on the proxy
     --proxy-header <header/@file> Pass custom header(s) to proxy
     --proxy-insecure Do HTTPS proxy connections without verifying the proxy
     --proxy-key <key> Private key for HTTPS proxy
     --proxy-key-type <type> Private key file type for proxy
     --proxy-negotiate Use HTTP Negotiate (SPNEGO) authentication on the proxy
     --proxy-ntlm    Use NTLM authentication on the proxy
     --proxy-pass <phrase> Pass phrase for the private key for HTTPS proxy
     --proxy-pinnedpubkey <hashes> FILE/HASHES public key to verify proxy with
     --proxy-service-name <name> SPNEGO proxy service name
     --proxy-ssl-allow-beast Allow security flaw for interop for HTTPS proxy
     --proxy-tls13-ciphers <ciphersuite list> TLS 1.3 proxy cipher suites
     --proxy-tlsauthtype <type> TLS authentication type for HTTPS proxy
     --proxy-tlspassword <string> TLS password for HTTPS proxy
     --proxy-tlsuser <name> TLS username for HTTPS proxy
     --proxy-tlsv1   Use TLSv1 for HTTPS proxy
 -U, --proxy-user <user:password> Proxy user and password
     --proxy1.0 <host[:port]> Use HTTP/1.0 proxy on given port
 -p, --proxytunnel   Operate through an HTTP proxy tunnel (using CONNECT)

Hi @hmert. I have an idea for implementing proxy commands to more clear. I wanted to get approval from you. proxy.go file is starting to be more complicated and messy for future commands. I was writing code for --proxy-digest flag and I had to modify proxyUserCmd by adding a new parameter. But, adding a new parameter affects to proxy_test.go file and proxyUserCredentials, err := proxyUserCmd(proxyUser) line. I think this problem will grow and be a problem. Maybe, refactoring will be necessary for each flag. That's why I think we can define a struct as follows:
type GurlProxy struct {
	proxyURL  string
	proxyUser string
	username  string
	password  string
	basic     bool
	digest    bool
	ntlm      bool
	negotiate bool
}

This struct may grow in the future, but I think, in this way, we suffer less pain. I am going to store necessary information as a struct and I'll convert functions to methods like func (g *GurlProxy) proxyCmd() (string, error). What do you think about that?


Invalid column state seems improperly handled in setColumnState

Hi!
In the following snippet of code, extracted from dist/ag-grid.js lines 3239 to 3254, the variable oldColumn is sometimes undefined, and oldColumn.setVisible(…) raises an exception:
ColumnController.prototype.setColumnState = function (columnState) {
    var oldColumnList = this.allColumns;
    ...
    if (columnState) {
        columnState.forEach(function (stateItem) {
        var oldColumn = utils_1.Utils.find(oldColumnList, 'colId', stateItem.colId);
        if (!oldColumn) {
            console.warn('ag-grid: column ' + stateItem.colId + ' not found');
            success = false;
        }
        oldColumn.setVisible(!stateItem.hide); // EXCEPTION HERE!
        ...

It looks like it is not the desired behavior, since it might simply cut off the grid generation (and pollutes the console with big flashy red exceptions just after the more tactful console.warn).
Is it the expected behavior?
Thanks in advance

code above is wrong, it's actually:
                if (!oldColumn) {
                    console.warn('ag-grid: column ' + stateItem.colId + ' not found');
                    success = false;
                    return;
                }

it you are missing the return statement. maybe you are on an older version, what you describe doesn't happen in the latest.

Haven’t you added that return statement yesterday?

i released yesterday. i can't remember when i actually added it!!


🐞 codegen: Handling lists of objects

Overview
The Go generated client executes the query when a scalar or list is requested.
https://github.com/dagger/dagger/blob/fdbf190307776b4cfb8acaa39cc82956d104e310/codegen/generator/templates/src/object.go.tmpl#L40-L44
Problem
When it's a list of objects you need to select the fields.
I looked the API for a problematic case:
type Container {
  envVariables: [EnvVariable!]!
}

type EnvVariable {
  name: String!
  value: String!
}

In fact when attempting to call it:
diff --git a/sdk/go/dagger/client_test.go b/sdk/go/dagger/client_test.go
index bd5fbefd..74263ae0 100644
--- a/sdk/go/dagger/client_test.go
+++ b/sdk/go/dagger/client_test.go
@@ -101,3 +101,23 @@ func TestContainer(t *testing.T) {
        require.NoError(t, err)
        require.Equal(t, "3.16.2\n", contents)
 }
+
+func TestList(t *testing.T) {
+       t.Parallel()
+
+       ctx := context.Background()
+
+       c, err := Connect(ctx)
+       require.NoError(t, err)
+       defer c.Close()
+
+       alpine := c.
+               Core().
+               Container().
+               From("alpine:3.16.2")
+
+       envs, err := alpine.EnvVariables(ctx)
+
+       require.NoError(t, err)
+       require.Equal(t, "PATH", envs[0].Name)
+}

I get an error:
Error: Received unexpected error:
       input:1: Field "envVariables" of type "[EnvVariable!]!" must have a sub selection.

/cc @aluzzardi @dolanor

@sipsma's suggestion in #3860

RE: how we can fix this. @aluzzardi mentioned not allowing objects in lists in Discord (at least that was my understanding, correct me if wrong).

Yeah, basically a workaround so it stops hurting. Of course with extensions the pain will come back at some point.

If we are willing to trade more complexity for having the ability to return objects in lists maybe we could figure something out around autoselecting all trivial scalars for those cases, though I don't know if we'll have to apply that fix on a case-by-case basis (ugly) or if there's a way of cleanly generalizing it. Vague idea, might be a deadend, but in general it would be nice to support these return types.

Agreed. This might be the cleanest solution, although I'm not sure either about edge cases.
Also creates a disparity between Object (need to call Object.Field(ctx) -- it's lazy) and [Object!]! (need to call Object.Field -- it's not lazy). We could have both APIs be the same, but the behavior would be different (lazy vs non-lazy) which can be confusing. Implementation would also need to be different (return "pre-fetched" field vs query the API).
There's also a half-baked option which is auto-selecting ID to make the query work, keeping leaf resolvers as usual. The problem though is how to "resume the query" (when calling .Field(ctx)). Either we re-run the full query with the extra field (but there's no way in GraphQL to select the Nth element, so we'd be calling field on ALL elements) OR we find a different way to resume the query (top-level selector ... but that's very manual)

A similar issue came up when I was implementing Stat (#3764). The GraphQL interface is something like Directory { stat(path: String): FileInfo! } with FileInfo being a type sort of like fs.FileInfo. But right now you actually have to select each individual field like size, err := dir.Stat().Size(ctx).
It's not the exact same problem, but maybe our solution here could address both.

Somewhat related:

https://github.com/dagger/dagger/issues/3577

Reopen because it's not implemented for NodeSDK and Python for now

What should happen next with this? Were any PRs opened for this?

It's been fixed in Go, needs fixes for Node.js and Python.
This proposal suggests a different implementation:

#4920

But in the meantime I'll do the same that was done in Go, maybe @TomChv can work on the Node.js fix.

Python fix has been merged. Only Node.js remaining.

OK!
We want to get a new release out this week, which leaves us with today.
Would delaying it by a few more hours make a difference for Node.js?

That depends on @TomChv, but probably not enough time.

Not a problem, it's OK to leave it for the next release. 👍 @TomChv

As early as @TomChv can do it. 👍


sourceMappingUrl in not minified file breaks Rails 4

On the bottom of dist/angular-flash.js there is a
131 //# sourceMappingURL=angular-flash.js.map
sitting, which blows up my Rails app :)
This should probably go to dist/angular-flash.min.js so the mapping can be found when You are using the minimified version.

Thank you for letting me know. I will take a look as soon as possible.

It is actually there on purpose as the original file is ES6 and it maps to that. It is also a JavaScript comment so it shouldn't break too much. Could you explain a little how it blows up your Rails app? Is it the asset precompilation and if so what is the error?
There should also be map for the .min.js so I added one there now.

I get:
ERROR ActionController::RoutingError: No route matches [GET]
"/assets/angular-flash.js.map"
This does look like something that can be resolved on my side but that's
the first time I see a library behave like that. Other libraries I have do
not include the map file for not minified version of js, so I assumed
something might be wrong on library's side.
Pozdrawiam,
Michał Orynicz
2016-04-24 10:09 GMT+02:00 Roope Hakulinen<EMAIL_ADDRESS>
It is actually there on purpose as the original file is ES6 and it maps to
that. It is also a JavaScript comment so it shouldn't break too much. Could
you explain a little how it blows up your Rails app? Is it the asset
precompilation and if so what is the error?
There should also be map for the .min.js so I added one there now.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/sachinchoolur/angular-flash/issues/35#issuecomment-213911064

Ok, It seems that problem lies in rails sprockets: https://github.com/rails/sprockets/issues/284 , https://github.com/rails/sprockets/issues/157
So not much to do on angular-flash side.

Thank you for getting back on this. I'll close the ticket for now. Please let me know if there is something that I can help you with still.


🐞 fix: error: has an unfree license (‘unfree’)

fix error: not allow Unfree software and add some missing brackets in exp  flake NixOS(line 638)

Looks good to me, thanks!


Next.js app which only has simple pages/index.tsx deploys with no error, but returns 502 error

Describe the bug

I made a simple Next.js App, which uses Yarn 4.0.2
It neither has pages/api folder nor getServerSideProps() now. I am planning to add those, but not now. Now it has only one React component in pages/index.tsx which show some lines of simple text.
I didn't forgot to add output: 'standalone' in next.config.js.
Then I depolyed it to my Azure SWA via Azure Pipelines.
After some tries, it did deploy successfully. But when I access to my app by web browser, it shows this:

And when I open console by pressing F12, it shows Failed to load resource: the server responded with a status of 502 ().
To Reproduce
Steps to reproduce the behavior:

Make a simple Next.js app which uses Yarn ver 2. or later (aka Yarn berry)
Deploy it to SWA by Azure Pipelines
Access it via any web browser

This is my Azure Pipelines workflow YAML file:
name: Azure Static Web Apps CI/CD

pr:
  branches:
    include:
      - main
trigger:
  branches:
    include:
      - main

jobs:
- job: build_and_deploy_job
  displayName: Build and Deploy Job
  condition: or(eq(variables['Build.Reason'], 'Manual'),or(eq(variables['Build.Reason'], 'PullRequest'),eq(variables['Build.Reason'], 'IndividualCI')))
  pool:
    vmImage: ubuntu-latest
  variables:
  - group: Azure-Static-Web-Apps-jolly-flower-0b2849210-variable-group
  steps:
  - checkout: self
    submodules: true
  - task: AzureStaticWebApp@0
    inputs:
      azure_static_web_apps_api_token: $(AZURE_STATIC_WEB_APPS_API_TOKEN_JOLLY_FLOWER_0B2849210)
###### Repository/Build Configurations - These values can be configured to match your app requirements. ######
# For more information regarding Static Web App workflow configurations, please visit: https://aka.ms/swaworkflowconfig
      app_location: "/" # App source code path
      api_location: "" # Api source code path - optional
      skip_api_build: true
      output_location: "" # Built app content directory - optional
###### End of Repository/Build Configurations ######

Expected behavior
Just show my text website.
Screenshots
Added above
Device info (if applicable):

OS: Every
Browser: Every
Version: Every

Additional context
None

It seems to be Yarn berry isn't supported. you folks should notice it in official docs.


options[:class] on form level is ignored

Hello,
CSS class applied on form level is getting ignored but it needs to be added along with Rails default generated CSS classes.
<%= bootstrap_form_for @user, class: "custom-form-class" do |form| %>
  <%= form.text_field :name %>
<% end %>

This is generating below HTML:
<form class="new_user" role="form" novalidate="novalidate" id="new_user" action="/users" accept-charset="UTF-8" method="post">

    Name
    <input class="form-control" aria-required="true" required="required" type="text" name="user[name]" id="user_name">

</form>

Hello This can be achived using html: {class: ""}, your form helper syntax is wrong.
Thank you.


[api-minor] Remove the image-related error message prefixes

Other custom errors, based on BaseException, do not use such a format.

/botio test

Thanks; using the actual exceptions rather than a message prefix overall seems much nicer!


Error when Run simulator on ubuntu 20.04 libcurl: (60) SSL: couldn't get X509-issuer name!

I got this isssue when i run :
roslaunch sim_pkg map_with_all_objects.launch
Got:
libcurl: (60) SSL: couldn't get X509-issuer name!
Segmentation fault (core dumped)
[gazebo_gui-3] process has died [pid 50261, exit code 139, cmd /opt/ros/noetic/lib/gazebo_ros/gzclient __name:=gazebo_gui __log:=/home/bblab7490/.ros/log/12b34770-0506-11ee-88e3-3b8ed2e5f3df/gazebo_gui-3.log].
log file: /home/bblab7490/.ros/log/12b34770-0506-11ee-88e3-3b8ed2e5f3df/gazebo_gui-3*.log
Does anyone know ? Thank you

Fixed overload: Edit file :  Simulator/src/models_pkg/track/materials/scripts/bfmc_track.material
texture_unit
{
texture 2021_Big.png
}
Change to :
texture_unit
{
texture 2021_VerySmall.png
}

Written also in the official documentation.


🚀 Fetch all fields from API instead of hardcoding

What problem does this solve?
The list of fields that is returned by custom report is hardcoded in the FieldCollection class.
Because of this, a custom report when passed :all will not include custom fields, but only the hardcoded ones.
Describe the solution you'd like
BambooHR returns the list of all fields via api, which is accessible using client.meta.fields method. Can we populate that while initializing a client and generate the list of fields from that information instead of hardcoding it?
Possible alternatives
Support something like client.report.custom([:all, customField1, customField2], 'JSON') which will let users specify the additional fields they like (in addition to the ones returned by :all).

The alternative mentioned above seems easy to implement. For example, I think this implementation supports the following scenarios

client.report.custom(:all, 'JSON')
client.report.custom([:all, 'customField1', 'customField2'], 'JSON')
client.report.custom(':all, customField1, customField2', 'JSON')

--- a/lib/bamboozled/api/field_collection.rb
+++ b/lib/bamboozled/api/field_collection.rb
@@ -2,8 +2,19 @@ module Bamboozled
   module API
     class FieldCollection
       def self.wrap(fields)
-        fields = all_names if fields == :all
-        fields = fields.split(",") if fields.is_a?(String)
+        if fields == :all
+          fields = all_names
+        elsif fields.is_a?(String)
+          fields = fields.split(",").map { |item| item.strip }
+        end
+
+        if fields.include?(:all) || fields.include?(":all")
+          fields.delete(:all)
+          fields.delete(":all")
+          fields += all_names
+          fields.uniq!
+        end
+
         new(fields)
       end

@splybon Does this make sense? :)

Hi @balasankarc thanks for the issue!  Yes that makes total sense.  I will play around with this myself as well to try and investigate some other methods, but I think I like your alternative solution.
My only hesitation with the client.meta.fields would be that it would make another network call to initialize the client.  I like the flexibility of your alternative solution
In the mean time a workaround for you could be something like
fields = (Bamboozled::API::FieldCollection.all_names + client.meta.fields.map {|f| f['alias']}).compact.uniq
client.report.custom(fields, "JSON")

@splybon and I discussed a documentation based and an API based approach to this issue, but haven't decided which is our preferred method of moving forward.
Possible solutions:
Documentation
Don't add any code to the project to address this, but instead just update the documentation with this example to show how users can include custom fields in addition to the default hardcoded fields.

pros: doesn't add any new features or change the API
cons: doesn't fix the issue

API
This is a code based solution that should abstract away some of those calls so that you can write something like the following:
fields = (client.meta.defaultFields + client.meta.fields)
client.report.custom(fields, "JSON")

pros: a more expressive API
cons: requires new feature work. adds complexity. requires research of the custom field alias (why is it sometimes nil?)


Update setup.cfg to 1.3.3

Codecov Report

Merging #250 (cdeb736) into main (c84d2b1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #250   +/-   ##
=======================================
  Coverage   91.24%   91.24%
=======================================
  Files          15       15
  Lines        1633     1633
=======================================
  Hits         1490     1490
  Misses        143      143

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


transform it category in example to array

Codecov Report

Merging #100 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #100   +/-   ##
=======================================
  Coverage   97.86%   97.86%
=======================================
  Files          13       13
  Lines         468      468
  Branches       82       82
=======================================
  Hits          458      458
  Misses          4        4
  Partials        6        6

Continue to review full report at Codecov.

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


ci: Print hint to fix auto-generated file checks

Description
ci(gha): Print hint to fix generated file check
This commit is to print steps on how to fix github action check for
generated files. This will surely help new contributors to have
better experience and reduce feedback loop.
Suggested by: Kornilios Kourtis<EMAIL_ADDRESS>Signed-off-by: Tam Mach<EMAIL_ADDRESS>Testing
Testing was done by one dummy change in one of the file https://github.com/cilium/cilium/pull/13718/checks?check_run_id=1296840070
@@ -690,7 +690,3 @@ spec:
       - configMap:
           name: cilium-config
         name: cilium-config-path
-
-
-
please run 'make -C install/kubernetes quick-install' and submit your changes
Error: Process completed with exit code 1.

ci: Print hint to fix auto-generated file checks

Mark ready to merge as changes are only for github action (e.g. full ci is not required).


Add support of CodeCoverage history

Several options to implement:

Always store results in repository and just deploy them to Pages
cons:

requires commit to ongoing PR (with probably additional squashing)
that leads to changes that initially was not a part of a PR

Before running tests, download current CodeCoverage results


Always assume default YT cluster will be used in YQL query

Changelog entry
Adding DefaultCluster to the list of used clusters
YQL can't give us all clusters that will be used during execution, but we need this information in order to create and pass yt tokens
YQL team said that they can't give us something like GetClustersForWhichYQLWillUseTokens because YQL can add new clusters during execution
Changelog category

Bugfix

Additional information
In a simple example with python udf
$script = @@#py
def foo():
    """
       ()->String

       foo function description
    """
    return b"bar"
@@;

$udf = Python3::foo($script);
SELECT $udf(); -- bar

We see an error 'Client is missing credentials'
Because YQL did not return default cluster as used one, but used it during execution

@vitstn Could you please review and merge it?


Allow non integer numbers

Allow numbers written in words to be parsed too.

added this feauture for up to 999.999


Determine JavaScript libraries and modules to include in framework

jQuery?
Carousel?
Modal/lightbox
Animation

Let's break this into smaller, more manageable issues: #10, #11, #12, #13


jwdemo to 0.0.5

Promote jwdemo to version 0.0.5

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@muirp-owner: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

promotion-build
990fbf7dea858312f28e6ee64e3521c41631d9b3
link
/test this

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


Content Script cannot use React

Hi, I have the content script that will create a div element and use React render to that div then append it into the document.body, but  when content script loaded it show this error Cannot use import statement outside a module

My content script code:

hello @dim0147
I created a pull request to fix the issue.

https://github.com/Jonghakseo/chrome-extension-boilerplate-react-vite/pull/9

Please check if it helps you solve your problem.
Thanks for raising a useful issue.

Thank you

@all-contributors please add @dim0147 for bug

@all-contributors please add @dim0147 for bug

@all-contributors please add @dim0147 for bug


type2ans.json

I am working on VQA-RAD evaluation with evaluate_vqa_rad_beam_scale.sh.
there is a error:
File "/infodev1/home/m253461/BiomedGPT/tasks/pretrain_tasks/unify_task.py", line 97, in init
self.type2ans_dict = json.load(open(os.path.join(self.cfg.neg_sample_dir, 'type2ans.json')))
FileNotFoundError: [Errno 2] No such file or directory: '/data/omnimed/pretrain_data/negative_sample/type2ans.json'
How to get this json file?
Thanks,
George

done.thanks.

Hi @George-WG-Cao do you know how to fix this issue

Sorry, I dont know.
I only found this package is very hard to use.
please contact authors.
All best,
George
On Sun, Dec 8, 2024 at 7:05 AM Duong Dang @.***> wrote:

Hi @George-WG-Cao https://github.com/George-WG-Cao do you know how to
fix this issue
—
Reply to this email directly, view it on GitHub
https://github.com/taokz/BiomedGPT/issues/62#issuecomment-2525684539,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVASLNMQLHKFTAAZILXR45T2EQYYFAVCNFSM6AAAAABSDXCLWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRVGY4DINJTHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Adding AYCD AutoSolver

Can I suggest u to add AYCD AutoSolver support on captcha blocks? it is one of the best solver and it would be useful.
thanks
If u need account
please contact me<EMAIL_ADDRESS>

Any update on this?

Any update on this?

not yet

not yet

Please send me an email ruri(at)openbullet[dot]dev

details sent

Please confirm that you received the email

Yes I got the email, I hope I will be able to look into this soon.

i did not notice that my subscription has ended.
i renewed it with captchapass pack.
please check it.
thanks

Yes I got the email, I hope I will be able to look into this soon.

Any update??

I tried implementing this, but when I fetch tasks (after creating them) I just get an empty list. I followed the docs, and the task is being created successfully, but nothing is returned, even on multiple calls and after waiting for some time. Maybe I will try contacting the dev to see if the API has issues.

I tried implementing this, but when I fetch tasks (after creating them) I just get an empty list. I followed the docs, and the task is being created successfully, but nothing is returned, even on multiple calls and after waiting for some time. Maybe I will try contacting the dev to see if the API has issues.

It is so sad to hear that. hope you will find a way to add this soon.
thanks


result of generate_3D_obj_GT.py look weird

Does this look what you have for ground truth or could you please share the ground truth for this image?

the image is:

HICO_train2015_00004909.jpg

GT here means using GT 2D h-o boxes, not the real 3D h-o. Because we can just use 2D clues to reconstruct the 3D h-o from the single RGB. Sometimes the reconstruction would not be accurate for hard cases.


Missing two "}"

[It looks that "}" are missing in this section:
using System;
namespace CosmosTableSamples
{
using System.Threading.Tasks;
using Microsoft.Azure.Cosmos.Table;
using Microsoft.Azure.Documents;
public class Common
{
    public static CloudStorageAccount CreateStorageAccountFromConnectionString(string storageConnectionString)
    {
        CloudStorageAccount storageAccount;
        try
        {
            storageAccount = CloudStorageAccount.Parse(storageConnectionString);
        }
        catch (FormatException)
        {
            Console.WriteLine("Invalid storage account information provided. Please confirm the AccountName and AccountKey are valid in the app.config file - then restart the application.");
            throw;
        }
        catch (ArgumentException)
        {
            Console.WriteLine("Invalid storage account information provided. Please confirm the AccountName and AccountKey are valid in the app.config file - then restart the sample.");
            Console.ReadLine();
            throw;
        }

        return storageAccount;
    }

]

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

ID: ef1295f9-eab5-8918-ab99-3a20b8723805
Version Independent ID: 0e551ce1-761a-d379-7b6c-4caf49c5bbb2
Content: Azure Cosmos DB Table API using .NET Standard SDK
Content Source: articles/cosmos-db/tutorial-develop-table-dotnet.md
Service: cosmos-db
Sub-service: cosmosdb-table
GitHub Login: @sakash279
Microsoft Alias: akshanka

Hello @elvirk , thanks for the question! We are taking a look into this and will get back to you soon.

@elvirk Thanks for bringing this to our notice and I observed the similar behavior. Actually the article was written in such a way that one method is added after the other and entire code repository is referenced from the below GitHub code sample:
https://github.com/Azure-Samples/azure-cosmos-table-dotnet-core-getting-started/blob/main/CosmosTableSamples/Common.cs
So while putting the first method CreateStorageAccountFromConnectionString , just the code till the end of that method is shown and rightly it missed the parenthesis. I would suggest you to check the above link provided which has entire code.
Please let us know if that helps or I can reach out to content author and mention the same.

@elvirk Awaiting your response to take this further based on above comment.

Please close this issue, I will not be able to try this until the weekend. If I have additional questions I'll open new issue. Thank you.


'maxResults' isnt a valid L.esri.Geocoding.geosearch() constructor option

based on my testing, you have to supply the constructor option to the arcgisOnlineProvider itself.  this seems reasonable to me, so we just need to update the documentation

closed via 104ea30


Adds an end user friendly span customiser

As per discussions with @adriancole, we don't want to expose the need to deal with tracer.currentSpan and its possibility of being null to end users. This adds a class that nicely wraps that up, which can be made into a spring bean that end users can wire in.
@adriancole, a few things:

not sure about whether we should just have a reference to Tracer or whether we should always do Tracing.tracer().etc ?
You mentioned returning a Noop span if there isn't one current, however you cannot create one without a TraceContext so it seemed simpler to just not do the operation if there isn't one and then return customiser rather than the real span.

Re: static final SpanCustomizer NOOP = new SpanCustomizer() ...
Its probably more work to return the NOOP than the current approach so I'll leave it as is?

yeah agreed NOOP would be more work in hindsight.

added docs https://github.com/openzipkin/brave/commit/a3f5f0aeb518b6d7fd024ae02826e09f09a9ccd0


KubeVirt fix ci (DC)

Signed-off-by: Helene Durand<EMAIL_ADDRESS>What this PR does / why we need it:
Fix Ci/
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind failing-test
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

NONE

Documentation:

NONE

/test pull-machine-controller-e2e-kubevirt


fix(scripts/test): ensure the script fails once the first failure happens

#100

Thanks a lot Lee for the great work! :+1:


Add r2 HTTP client

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆
https://github.com/mikeal/r2
Should be included because it's the successor to request, is nicer to use, and is seeing adoption.

Passing on this as it's not actively maintained anymore.


Build issues

Due to the merged of https://github.com/twbs/bootstrap/pull/22045
~they are few errors, on Travis see :~
~- https://travis-ci.org/twbs/bootstrap/jobs/223876338#L312~
~- https://travis-ci.org/twbs/bootstrap/jobs/223876309#L303~
And on my computer (Win10) js-compile-bundle don't work because it failed here :
shx sed 's/^(import|export).*//'
I changed this part of this task into to make it works :
shx sed \"s/^(import|export).*//\"
/CC @bardiharborow

Regarding Travis, the main branch is passing. Your branch may need to be rebased though. Thanks for testing on Windows, I haven't really done that yet. I'll get that issue patched tonight.

But my branch is up to date @bardiharborow , so I don't understand how my branch can generate this build issue

@Johann-S, I just re-shrinkwrapped your branch, and it's all green now.

Yes I saw it thank you 👍 but I don't understand why I didn't have the good shrinkwrap... I rebased my branch this morning

@Johann-S, you'd overridden the shrinkwrap to include popper.js, and rebasing didn't pick up some last minute changes that I made to my PR before merging it.

Fixed in #22477.

It seems they are build issues with this error : sh: 1: grunt: not found
https://travis-ci.org/twbs/bootstrap/builds/225503458
It's strange because sometimes this error doesn't happen like here : https://travis-ci.org/twbs/bootstrap/jobs/225375395
/CC @bardiharborow


Added property to disable default animation on input error

I just added boolean xml attribute to disable the default 'shake' animation on the card validation error.

Thanks!


Odd redirect from https://www.visualstudio.com/vsts/integrate

This redirects to https://www.visualstudio.com/wp-content/uploads/2016/04/Integrate.png
Odd that it also appears to be a Wordpress content folder.

@nschonni I have reported this one to the Visual Studio team to get it fixed. Thanks for letting us know.


Allowed custom BatchSamplers when instantiated in *_dataloader hook

What does this PR do?
Allowed custom BatchSamplers when instantiated in *_dataloader hook
Fixes #13007, #11807
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
cc @borda @justusschock @awaelchli @ninginthecloud @rohitgr7 @otaj @akihironitta

Thank you for the review! I addressed all your concerns :+1:
Btw, regarding the complexity, it surprisingly wasn't too bad for me, since it's basically just logic from dataloader handling made generic

@otaj There's a failing test. Also, @awaelchli would you kindly review?

@awaelchli thank you very much for the review! I hope I addressed everything

@otaj There's a failing test. Also, @awaelchli would you kindly review?

@carmocca that should be taken care of now. It was a simple issue of changing the text of exception raised not matching the pytest catching

IPU failure looks flaky. I've rerun twice, going for thrice

IPU failure looks flaky. I've rerun twice, going for thrice

And it failed again. Can we maybe mark the test as flaky, or set a bit lower threshold?

I ran it twice too. Does not look flaky to me. The environment must have changed, we may start seeing it on other PRs too. There is also the possibility that it's related to the PR. Note that in the IPU strategy we use the _get_dataloader_init_args_and_kwargs that was modified here. Just pointing it out as a possibility.

Note that in the IPU strategy we use the _get_dataloader_init_args_and_kwargs that was modified here. Just pointing it out as a possibility.

You're absolutely right regarding this, however the code should be functionally the same for normal dataloaders. But, to be fair, it doesn't seem to fail on other PRs, so it's quite likely that it is indeed a regression in this PR.
Let's try to re-run once or twice more and if it is still failing, let's investigate more

There is something fishy going on. If I try to modify the test_optimization to force it on cpu (change accelerator, remove RunIf and remove dm.trainer checks), I pass the test every single time, and comfortably. So it really seems the issue is introduced in this PR. :facepalm:

@carmocca, @awaelchli I finally managed to properly debug the cause of the IPU regression. The problem was, that I was passing all default kwargs from torch.utils.data.DataLoader to poptorch.DataLoader which didn't play nicely with it. I don't think there's a substantial change from when you reviewed it (just split __pl_saved_kwargs into __pl_saved_kwargs and __pl_saved_default_kwargs).

Hey @otaj I am confused on how to update my sampler to fit this API.
We are dealing with audio data and we want to batch samples by sequence length. In order to do this, we have shape files that are CSVs mapping between each audio sample and its length. Our BatchSampler reads these files, sorts utterances by length, and then constructs fixed batches of similarly long utterances such that all batches are similar in size.
I am not sure what to move to a Sampler here. We could move the loading of this shapes file to the sampler backed by a new dataset, but we do not sample from the shapes file to build batches, we consume the entire file at once.
I think part of my doubt is a lack of understanding on what exactly you do to the Sampler passed into the BatchSampler. I could move the loading of the shapes file inside a Dataset, pass in a Sampler to this dataset as a hacky way of giving the dataset to the BatchSampler, and totally ignore the Sampler in my BatchSampler logic while just reading from its member Dataset, but I imagine this will not solve the problem I'm having.

Hi @psridhar-asapp. First things first, this doesn't really add new API. It keeps the functionality as before, just know it allows your custom BatchSampler to have also more arguments, that don't appear in a signature of regular BatchSampler.
I remember we spoke on this topic in some other issue (I think it was #11807). I don't think we have a good resolution to it yet, honestly.
We don't really do much to your sampler. Everything important regarding Sampler and DataLoader is happening in this method (and of course, all the other methods it calls). https://github.com/Lightning-AI/lightning/blob/caaf35689c585fba1bb33b70243e7ddd2cf5d13c/src/pytorch_lightning/trainer/connectors/data_connector.py#L238
Currently, it sets shuffling if it was set to None, and we're training, wraps it in a distributed wrapper and does a bit more of a magic in _update_dataloader, which is just juggling arguments. The most important bit of call tree in _update_dataloader is honestly this function https://github.com/Lightning-AI/lightning/blob/caaf35689c585fba1bb33b70243e7ddd2cf5d13c/src/pytorch_lightning/utilities/data.py#L315
That is where we wrap either BatchSampler or Sampler in structures used for prediction or fault tolerance. I think (and think is the operative word here) it might be possible to remove the Sampler from being passed around and you could be happy with your BatchSampler only solution, but I don't really have a clear vision about that yet and it will require non-trivial engineering work.
If you don't care about either of these things, you can just expose argument sampler in your custom BatchSampler, (which needs to be a proper subclass of it and instantiated in *_dataloader hook) and don't use that sampler at all. I'm sorry, I don't have a better solution for you


Inject git commit into binary

.. which shows up in pod logs.
{"level":"info","ts":1614286509.002165,"logger":"cmd","msg":"Git Commit: 89b04f64e2774876870de93fbf046b617de2c0cf"}
{"level":"info","ts":1614286509.002208,"logger":"cmd","msg":"Operator Version: 0.0.1"}
{"level":"info","ts":1614286509.0022101,"logger":"cmd","msg":"Go Version: go1.15.5"}
{"level":"info","ts":1614286509.002214,"logger":"cmd","msg":"Go OS/Arch: darwin/amd64"}
{"level":"info","ts":1614286509.0022159,"logger":"cmd","msg":"Version of operator-sdk: v0.18.2"}
{"level":"info","ts":1614286509.0096781,"logger":"leader","msg":"Trying to become the leader."}
{"level":"info","ts":1614286509.009747,"logger":"leader","msg":"Skipping leader election; not running in a cluster."}

replaced by https://github.com/redhat-developer/gitops-operator/pull/65


Request: Use MySqlConnector for async I/O

The MySqlDbHelper code uses ExecuteReaderAsync, etc.. It's a long-standing bug in MySql.Data (bug 70111) that async I/O is not implemented correctly; thus this method will actually run synchronously. (See also here and here on Stack Overflow.)
To fix this, you could switch to https://github.com/mysql-net/MySqlConnector, an OSS replacement for MySql.Data that supports true asynchronous I/O.
If you're interested in this, I'd be happy to open a PR.

Would you be able to contribute here RepoDb.MySqlConnector? I plan to introduce this one soon. No hurry for now.

Was that last comment directed to me? Are you asking for RepoDb.MySql to be duplicated as RepoDb.MySqlConnector?

Yes, I can initially create the solutions and will eventually ask you if you could help me with that moving forward to have it work with MySqlConnector.

@bgrainger - this is next on line for me. Any news on your side here?

Sorry, I lost track of this. 😬
I'm happy to contribute a PR if copy/pasting the "RepoDb.MySql" as "RepoDb.MySqlConnector" (then making appropriate changes) is still the preferred approach.

@bgrainger - I am also thinking to combine your library inside the RepoDb.Mysql and create the dedicated integration tests for MySqlConnector. Same goea to the RepoDb.SqlServer in which both MDS and SDS is present at one package. Your thoughts?

This should be possible, but may be complicated by the fact that both libraries (MySql.Data and MySqlConnector) use the same namespace: https://github.com/mysql-net/MySqlConnector/issues/189. You'd have to use extern aliases to reference both libraries.
I'm not sure if this would cause any problems for consumers of the library, or if it would end up "just working" based on the specific MySQL library they referenced.

I pushed the changes here: https://github.com/bgrainger/RepoDb/commit/e2bc3426f08ad9ba824b28fc4c7dbd583a94117d
They're simple enough, but unfortunately it breaks all the tests with hundreds of errors:
1>C:\Code\RepoDb\RepoDb.MySql\RepoDb.MySql.IntegrationTests\Operations\BatchQueryTest.cs(37,41,37,56): error CS0433: The type 'MySqlConnection' exists in both 'MySql.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=c5687fc88969c44d' and 'MySqlConnector, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=d33d3e53aa5f8c92'

Even though the test project still only references MySql.Data 8.0.20, the transitive reference causes both libraries to be in scope, and the compiler can't decide which one to pick.
This would be painful to have to remedy (with extern aliases) in all consumers, so I think it rules out this approach until the namespace is changed.
Since a lot of the classes (e.g., the ones that have to end up referencing a specific MySqlDbType instance) ended up getting copied anyway, it might be best to go with the original plan of copying the entire library?

Make sense! Thanks for exercising it. Then, I would be very happy if you do the entire copy and make a separate "RepoDb.MySqlConnector" folder as part of the top folder (pushed by you). Then, let us work together to pass all the things there.
Would it be possible for you to initiate the copy of the entire folder of "RepoDb.MySql" and create a new folder for "RepoDb.MySqlConnector"?

Yes, I can do that.

Nice if you can make your own branch then I will also push my updates there until we finalize the solution and merge it back to the master. I will add you as collaborator on the project.
Also, I will put you as owner of RepoDb.MySqlConnector Nuget Package if you are fine with that.

I've pushed a new branch, mysqlconnector. To avoid it just being a wall of green newly-added code, I split it into two commits. The first just duplicated all the RepoDb.MySql files as-is; the second shows the actual changes: https://github.com/mikependon/RepoDb/commit/fa13702718ba8ace68a3c99505fdcaa5525bf73a

I had reviewed all your changes. It is awesome and well-refined! :) I will revisits your branch on Saturday and will ensure running all the Unit/Integration Tests on my local DEV environment. Once they're all passing, then let us merge it right away.
P.S: Just do all the changes if you think you still have something before Saturday so I can include those on my testing and validation activities.
Big thanks for this "change" mate, you did all my work by integrating your library to RepoDb.

@bgrainger - I made a reviews today on your solution. Also, I had run the Integration Tests locally and it is all green. Thanks and cheers for this branch.
Would you be able to issue a PR anytime soon? I will then create the dedicated AppVeyor CI builds for this project and relate it to its README.
Integration and Unit Tests result:

If you're happy with the mysqlconnector branch (looks like you pushed a few updates to it), I can open a PR from it: https://github.com/mikependon/RepoDb/pull/446

I made a thorough review today CET. Yes, you can now issue a PR so I can finalize the CI builds and the docs this weekend.

Opened here: https://github.com/mikependon/RepoDb/pull/446

@bgrainger - done. Feel free to add your library on the Credits section of the Main README of the project.

Thanks for merging it! Done: c8d0f77ba34ec7d0c7d92d8207986baa6f11c178.

Planning to release the first version. I think I am OK on RepoDb side, unless you have something. Are you OK as well?

I made an additional series of code-checks and local tests execution (also with App Veyor CI Builds) today.
The initial release can be found here.
The documentation has been updated to the latest.


Allow encoder to accept shapely objects directly

This came up when optimizing my tile generation code. When sending wkt to .encode(), there was significant time being spent in wkb.loads just to find out that the argument was not wkb. Furthermore, because I'm working with the objects before sending to .encode, this change allows me to skip the object -> wkt/wkb -> object transformation and just pass in an object directly.
Thanks!

Thanks! I made a couple of minor updates before merging.
Would a new release be convenient for you?

Thanks! I copied this project into my project repo for the time being, so
no rush. But I'd be happy to use a new release and get that code out of my
project repo.
-Evan
On Wed, Jul 15, 2015 at 1:36 PM, Robert Marianski<EMAIL_ADDRESS>wrote:

Thanks! I made a couple of minor updates before merging.
Would a new release be convenient for you?
—
Reply to this email directly or view it on GitHub
https://github.com/mapzen/mapbox-vector-tile/pull/16#issuecomment-121739785
.

I created a 0.0.10 release and pushed it over to pypi.

Many thanks!


Fix for Issue #143.  GoogleMaps Heatmap not zooming and panning corre…

Thank you, this has been added and will be in the next tagged version


chore: Ensure cleanup script is generated if RG is created

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Ensure the cluster cleanup script is generated after the resource group is created. This is done to avoid leaking empty resources groups if cluster creation fails during Github action execution.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build 855122477

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 53.919%

Totals

Change from base Build 854194019:
0.2%

Covered Lines:
2215

Relevant Lines:
4108

💛  - Coveralls


PEAR 1322 - Mutations button, survival plot in Mutations tab not plotting for a mutation

Description
pear-1322

Oops - I didn't see that this had more commits.
This isn't working as expected.

use a cohort with bronchus and lung primary site
in Mutation Frequency, Genes tab, click on the # Mutations button for TP53
it switches to the Mutations tab, applies the search, but it's not attempting to plot for the topmost mutation

it should plot topmost now afterwards; was missing a filter
https://github.com/NCI-GDC/gdc-frontend-framework/assets/90918464/852dd142-7a36-48cc-b781-55ec0a8ad154

working as expected. 👍

I noticed one oddity that @wteouchicago might want to confirm is an issue.
MF App: All GDC Cohort, default filtering

Select a Mutation in the table: PTEN
Tab switches to Mutations shows the top Mutation as PTEN R130Q; search is filled in with ENSG00000171862
Clear search
The top mutation table changes to BRAF 640B; however, the survival plot still shows PTEN R130Q

I noticed one oddity that @wteouchicago might want to confirm is an issue.
MF App: All GDC Cohort, default filtering

Select a Mutation in the table: PTEN
Tab switches to Mutations shows the top Mutation as PTEN R130Q; search is filled in with ENSG00000171862
Clear search
The top mutation table changes to BRAF 640B; however, the survival plot still shows PTEN R130Q

It's a bit weird -- ideally it would plot the top mutation, but I think we can live with it unless it's a super easy fix. I can justify it if necessary and/or we can fix it later.
I found some weirdness with top mutation plotting recently: https://gdc-ctds.atlassian.net/browse/PEAR-1621 -- maybe they are related and can be fixed with PEAR-1621? 🤷♀

I noticed one oddity that @wteouchicago might want to confirm is an issue.
MF App: All GDC Cohort, default filtering

Select a Mutation in the table: PTEN
Tab switches to Mutations shows the top Mutation as PTEN R130Q; search is filled in with ENSG00000171862
Clear search
The top mutation table changes to BRAF 640B; however, the survival plot still shows PTEN R130Q

It's a bit weird -- ideally it would plot the top mutation, but I think we can live with it unless it's a super easy fix. I can justify it if necessary and/or we can fix it later. I found some weirdness with top mutation plotting recently: https://gdc-ctds.atlassian.net/browse/PEAR-1621 -- maybe they are related and can be fixed with PEAR-1621? 🤷♀

tried to resolve, but attempting to implement solution to plot the topmost after clearing the search led to unintended side effects (plotting/unplotting whenever the search term changes and then no longer retaining the default behavior of plotting topmost when button is unselected)

tried to resolve, but attempting to implement solution to plot the topmost after clearing the search led to unintended side effects (plotting/unplotting whenever the search term changes and then no longer retaining the default behavior of plotting topmost when button is unselected)

that's ok -- we can attempt to fix later

still looks good for 1322 and also checked that PEAR-1621 is fixed by this. 👍

I see 2 linter warnings regarding dependency array. Can we fix those? Suppressing might not be an ideal solution.

I see 2 linter warnings regarding dependency array. Can we fix those? Suppressing might not be an ideal solution.

thank you, i agree. its fixed now. updated dependency arrays for 2 warning sources & still functions as before


Update aws-core, http-client-spi, ... to 2.17.133

Updates

software.amazon.awssdk:aws-core
software.amazon.awssdk:http-client-spi
software.amazon.awssdk:netty-nio-client

from 2.17.104 to 2.17.133.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #68.


detect anonymous permissions

Context
After creating a clusterrolebinding for the user system: anonymous, I didn't see it in the list of Risky report.
See the details :
kubectl create clusterrolebinding anonymous-xdemo --clusterrole=view --user=system: anonymous

Given it some thought and it’s a bit tricky to implement as rbac-police is built to alert on risky permissions rather than risky users & groups. Might still be doable though.
Btw it’s possible to run with ‘—violations user,group’ and to search for users & groups with risky permissions, including the anonymous user / group


Brand New Install Error in log

Brand New Install Error in log
Any Ideas?
Error while setting up tuya_local platform for climate
File "/config/custom_components/tuya_local/climate.py", line 62, in async_setup_entry
File "/config/custom_components/tuya_local/climate.py", line 50, in async_setup_platform

What device are you trying to set up here?

A Kogan power monitoring plug. ???

You'll need to untick "Include device as a climate entity" for that, and tick "Include device as a switch entity" at the bottom.  the errors are coming from the climate entity because the device does not support it.

Thankyou. I was trying to set it up using yaml. The Integration worked. Just not getting the power monitoring. Will disconnect overnight and see how we go in the morning as per your suggestion.
Thanks Again!

I have the newer model Kogan Power monitor. I needed to change DP's as follows and is reporting all attributes now.
Model: KASPEMHUSBA
from homeassistant.components.switch import ATTR_CURRENT_POWER_W

ATTR_SWITCH = "switch"
ATTR_TIMER = "timer"
ATTR_CURRENT_A = "current_a"
ATTR_VOLTAGE_V = "voltage_v"

PROPERTY_TO_DPS_ID = {
    ATTR_SWITCH: "1",
    ATTR_TIMER: "2",
    ATTR_CURRENT_A: "18",
    ATTR_CURRENT_POWER_W: "19",
    ATTR_VOLTAGE_V: "20",
}

Thanks for the info.  I see the string "USB" embedded in the model number, does this plug come with USB as well?  Could it be separately switchable?  Do you have the full list of DPS reported by the device?

I think the new version I released today should support this device out of the box.

Thanks for the info. I see the string "USB" embedded in the model number, does this plug come with USB as well? Could it be separately switchable? Do you have the full list of DPS reported by the device?

read out when off and no load
'1' : false,
'9' : 0,
'18' : 0,
'19' : 0,
'20' : 2261,
'21' : 1,
'22' : 590,
'23' : 27459,
'24' : 15242,
'25' : 1368

read out when on and with load
'1' : ftrue,
'9' : 0,
'18' : 5094,
'19' : 11198,
'20' : 2257,
'21' : 1,
'22' : 590,
'23' : 27459,
'24' : 15242,
'25' : 1368

The other way to overcome the delays is to flash alternate firmware (Tasmota, ESPHome etc), where you can use your own MQTT server or direct connection to Home Assistant. But I don't know if the new devices are still ESP8266 devices, as Tuya seem to be switching some products to Realtek lately.

Yes they can be flashed, all mine run tasmota via tuya convert.  My old man has some and wants to keep them stock so I am doing this to set them up for him.


Loss Explosion Issue When Training Model from Scratch

Checks

[X] This template is only for usage issues encountered.
[X] I have thoroughly reviewed the project documentation but couldn't find information to solve my problem.
[X] I have searched for existing issues, including closed ones, and couldn't find a solution.
[X] I confirm that I am using English to submit this report in order to facilitate communication.

Environment Details
ubuntu 22.0.4
torch  2.3.1
8 x H800 80GB
Steps to Reproduce
Hello, thanks for the inspiring project you've shared.
I have attempted to train a small model from scratch using the WeNet4TTS Premium dataset.
The hyperparameters are as outlined in your paper:
model_cls = DiT
model_cfg = dict(dim=768, depth=18, heads=12, ff_mult=2, text_dim=512, conv_layers=4)

Additionally, I have set the epochs=1000 , as I observed that the default epochs=11 resulted in the learning rate decreasing too rapidly.
However, I've encountered the loss begins to grow and eventually explodes during training.

Have you experienced this issue? I would greatly appreciate any advice or suggestions on how to address this problem.
Thank you in advance for your time and assistance.
Best regards!
✔ Expected Behavior
No response
❌ Actual Behavior
No response

We haven't encountered this in our experiments (we trained small models with half the batchsize as base model, and only 800k updates)


Bluetooth: Controller: ISO Sync payload number and timestamp

Add implementation to send ISO Sync payload number and
timestamp from LLL to HCI/ISOAL layer.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

LGTM. I assume that the footer information will not be needed in the handling of NODE_RX_TYPE_SYNC_ISO_PDU.

rx_ftr, rx_iso_meta and rx_pdu_meta are union. For now, rx_iso_meta is sufficient.


MediaPipe hand detector empty results in web demo

Issue Description
Testing the MediaPipe hand detector (handdetect.json) in the web demo results in empty hand results.
Checking logs I see for every detection human shows a cached model msg for both detector and skeleton models.
Hand config:
hand: {
    enabled: true,
    rotation: true,
    skipFrames: 99,
    skipTime: 1000,
    minConfidence: 0.5,
    iouThreshold: 0.2,
    maxDetected: 2,
    landmarks: true,
    detector: {
      modelPath: "handdetect.json",
    },
    skeleton: {
      modelPath: "handlandmark-lite.json",
    },
  },

Logs:

sorry this somehow completely fell under the radar.
fixed now, will publish on github and npmjs later today.


chore: release main

:robot: I have created a release beep boop
accesskit: 0.8.1
0.8.1 (2023-03-22)
⚠ BREAKING CHANGES

Move thread synchronization into platform adapters; drop parking_lot (#212)
Make Node opaque and optimize it for size (#205)

Features

Add C bindings (18f702d)

Bug Fixes

Set appropriate representations on all public types that will be exposed via FFI (54e82f6)

Documentation

Fix outdated documentation for TreeUpdate (#182) (dd658c7)

Code Refactoring

Make Node opaque and optimize it for size (#205) (4811152)
Move thread synchronization into platform adapters; drop parking_lot (#212) (5df52e5)

accesskit_c: 0.1.0
0.1.0 (2023-03-22)
Features

Add C bindings (18f702d)

Dependencies

The following workspace dependencies were updated

dependencies

accesskit bumped from 0.10.1 to 0.8.1
accesskit_windows bumped from 0.13.2 to 0.14.0
accesskit_macos bumped from 0.6.2 to 0.7.0
accesskit_unix bumped from 0.3.2 to 0.4.0

accesskit_consumer: 0.15.0
0.15.0 (2023-03-22)
⚠ BREAKING CHANGES

Move thread synchronization into platform adapters; drop parking_lot (#212)
Make Node opaque and optimize it for size (#205)

Features

Automatically get button and link labels from descendants (#184) (ec5c38e)
Basic Unix platform adapter (#198) (1cea32e)
Text support on macOS (#191) (3a35dbe)

Bug Fixes

Make Node::filtered_parent recursive as it was meant to be (#203) (d2faef5)
More reliable handling of the edge case for wrapped lines (#192) (c626d2c)

Code Refactoring

Make Node opaque and optimize it for size (#205) (4811152)
Move thread synchronization into platform adapters; drop parking_lot (#212) (5df52e5)

Dependencies

The following workspace dependencies were updated

dependencies

accesskit bumped from 0.10.1 to 0.8.1

accesskit_macos: 0.7.0
0.7.0 (2023-03-22)
⚠ BREAKING CHANGES

Move thread synchronization into platform adapters; drop parking_lot (#212)
Make Node opaque and optimize it for size (#205)
Move lazy initialization from the core platform adapter to the caller (#179)

Features

Live regions on macOS (#196) (47d8d9f)
Text support on macOS (#191) (3a35dbe)

Bug Fixes

Correctly apply the DPI scale factor to coordinates (#185) (d263938)
Don't expose the window title in our root element on macOS (#187) (9739b74)
Expose static text as the value rather than the title on macOS (#186) (e3720c8)
Expose which accessibility selectors are actually allowed for a particular node (#181) (c4cbb23)
Fix macOS leaks (e8537fc)
Handle views with flipped coordinates (#174) (d14484c)
Make VoiceOver move through nodes in logical order (#176) (f060be4)
More reliable handling of the edge case for wrapped lines (#192) (c626d2c)
Pin objc2 dependency to 0.3.0-beta.3 (#201) (0adfed1)
Re-export types from objc2 (#172) (1ac67ad)

Code Refactoring

Make Node opaque and optimize it for size (#205) (4811152)
Move lazy initialization from the core platform adapter to the caller (#179) (f35c941)
Move thread synchronization into platform adapters; drop parking_lot (#212) (5df52e5)

Dependencies

The following workspace dependencies were updated

dependencies

accesskit bumped from 0.10.1 to 0.8.1
accesskit_consumer bumped from 0.14.1 to 0.15.0

accesskit_unix: 0.4.0
0.4.0 (2023-03-22)
⚠ BREAKING CHANGES

Move thread synchronization into platform adapters; drop parking_lot (#212)
Make Node opaque and optimize it for size (#205)

Features

Basic Unix platform adapter (#198) (1cea32e)

Bug Fixes

Update atspi dependency (#217) (93f2dc9)

Code Refactoring

Make Node opaque and optimize it for size (#205) (4811152)
Move thread synchronization into platform adapters; drop parking_lot (#212) (5df52e5)

Dependencies

The following workspace dependencies were updated

dependencies

accesskit bumped from 0.10.1 to 0.8.1
accesskit_consumer bumped from 0.14.1 to 0.15.0

accesskit_windows: 0.14.0
0.14.0 (2023-03-22)
⚠ BREAKING CHANGES

Move thread synchronization into platform adapters; drop parking_lot (#212)
Make Node opaque and optimize it for size (#205)
Update winit to 0.28 (#207)
Move lazy initialization from the core platform adapter to the caller (#179)

Bug Fixes

Correct broken UIA method implementation that was incompatible with Windows 11 ATs (#193) (3c527c7)
Fix Windows 32-bit build errors (#223) (41f28b6)
More reliable handling of the edge case for wrapped lines (#192) (c626d2c)
Provide fallback property implementations for the window root (#194) (f3d30b9)
Update windows-rs to 0.44 (#220) (a6b0a12)

Miscellaneous Chores

Update winit to 0.28 (#207) (3ff0cf5)

Code Refactoring

Make Node opaque and optimize it for size (#205) (4811152)
Move lazy initialization from the core platform adapter to the caller (#179) (f35c941)
Move thread synchronization into platform adapters; drop parking_lot (#212) (5df52e5)

Dependencies

The following workspace dependencies were updated

dependencies

accesskit bumped from 0.10.1 to 0.8.1
accesskit_consumer bumped from 0.14.1 to 0.15.0

accesskit_winit: 0.13.0
0.13.0 (2023-03-22)
⚠ BREAKING CHANGES

Move thread synchronization into platform adapters; drop parking_lot (#212)
Make Node opaque and optimize it for size (#205)
Update winit to 0.28 (#207)
Move lazy initialization from the core platform adapter to the caller (#179)

Features

Basic Unix platform adapter (#198) (1cea32e)
Feature-gate the Unix adapter in accesskit_winit (#214) (be95807)

Bug Fixes

Don't force winit's X11 and Wayland features to be enabled (#209) (a3ed357)
Reduce the winit version requirement to match egui (#170) (1d27482)

Miscellaneous Chores

Update winit to 0.28 (#207) (3ff0cf5)

Code Refactoring

Make Node opaque and optimize it for size (#205) (4811152)
Move lazy initialization from the core platform adapter to the caller (#179) (f35c941)
Move thread synchronization into platform adapters; drop parking_lot (#212) (5df52e5)

Dependencies

The following workspace dependencies were updated

dependencies

accesskit bumped from 0.10.1 to 0.8.1
accesskit_windows bumped from 0.13.2 to 0.14.0
accesskit_macos bumped from 0.6.2 to 0.7.0
accesskit_unix bumped from 0.3.2 to 0.4.0

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit-v0.8.1 :sunflower:

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit_consumer-v0.15.0 :sunflower:

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit_macos-v0.7.0 :sunflower:

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit_unix-v0.4.0 :sunflower:

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit_windows-v0.14.0 :sunflower:

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit_winit-v0.13.0 :sunflower:

:robot: Release is at https://github.com/DataTriny/accesskit/releases/tag/accesskit_c-v0.1.0 :sunflower:


db せっけー

https://docs.google.com/spreadsheets/d/1wUyqap2oTYuHRk79X3OVJFz5n3xfs_xgOG7R2kdWCsM/edit#gid=0


APT whitelist request for httpd

I am writing this RESTful api: https://github.com/cerndb/storage-api . It uses mod_wsgi compiled for python3. In order to be able to do so it needs some tools from httpd-devel.

I should use apache2 package.


Milestone 1: Create a minimally playable game

For our first milestone, we would like to create a minimally playable version of our music game with the following requirements.
As a user:

[x] I am presented with a welcome screen when I navigate to the app's webpage
[x] I can read the game's instructions
[x] For each question:

[x] I can hear two tones played
[x] I can replay the tones as many times as I want
[x] I can guess the interval between the tones
[x] I can change my answer as many times as I want before submission
[x] I can skip the question

[x] I can see my current score
[x] I can see the current question number I'm on
[x] At the end of the game, I can see how many questions I've answered correctly
[x] I can replay the game

In addition, our app should be deployed, and it should look somewhat nice 🙂

DONE 🎉


Comment Fix and Function Name Changed

[x] Adjust comment on showSlide function
[x] Adjust function name from showSlides to showSlide

Is there a reason you created a issue for this? I think it wasn't necessary to create this issue as it was addressed in the pull request. Right?


AWS PowerShell NetCore module does not respect $env:AWS_PROFILE

Expected Behavior
Cmdlets in the AWSPowerShell.NetCore module should use the profile specified in $env:AWS_PROFILE.
Current Behavior
Cmdlets in the AWSPowerShell.NetCore module do not respect the $env:AWS_PROFILE variable.
Possible Solution
When the variable is specified, its value should be used as the value of "ProfileName"
Steps to Reproduce (for bugs)

Create a named profile in ~/.aws/credentials
Run a Cmdlet which allows specifying a profile via the -ProfileName parameter, but do not specify a value for that parameter.

Context

Writing documentation which targets both PowerShell and bash environments is more cumbersome because credentials are specified differently.
Explaining the use of the module to peers can be problematic because some environment variables are supported by the Cmdlets and some are not.
Tooling which indicates what profile is currently in use is significantly harder to write; the value needs to be maintained and managed as separate state rather than assuming that this variable is authoritative.
To my knowledge, it is not possible to check the currently active profile after running Set-AWSCredential

Your Environment

AWSPowerShell.NetCore version used: 3.3.365.0
Service assembly and version used: n/a
Operating System and version: Windows 10 1703
Visual Studio version: n/a
Targeted .NET platform: n/a

I have the same issue with .NET Framework version.
> $env:AWS_PROFILE = 'csr'
> Get-S3Bucket
Get-S3Bucket : No credentials specified or obtained from persisted/shell defaults.

> Set-AWSCredential -ProfileName $env:AWS_PROFILE
> Get-S3Bucket
CreationDate           BucketName
------------           ----------
[...]

> Get-Module AWSPowerShell | select -Property Version
Version
-------
3.3.390.0

I would be happy to help with getting this fixed. Is anyone able to point me in the direction of the code that I would need to change?


python310Packages.dulwich: add optional-dependencies

Description of changes

add optional-dependencies
clean-up inputs

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

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

2 packages marked as broken and skipped:

python310Packages.optuna
python39Packages.optuna

2 packages failed to build:

python310Packages.functorch
python39Packages.functorch

119 packages built:

breezy (python310Packages.breezy)
cabal2nix
commitizen
dep2nix
devpi-client
dvc
dvc-with-remotes
errbot
flatpak-builder
glanceclient (python310Packages.python-glanceclient)
haskellPackages.cabal2nix-unstable
heatclient (python310Packages.python-heatclient)
home-assistant-component-tests.airnow
home-assistant-component-tests.eafm
home-assistant-component-tests.google
hydra_unstable
ironicclient (python310Packages.python-ironicclient)
jenkins-job-builder (python310Packages.jenkins-job-builder)
klaus (python310Packages.klaus)
manilaclient (python310Packages.python-manilaclient)
nix-prefetch-bzr
nix-prefetch-scripts
nix-update-source
openstackclient (python310Packages.python-openstackclient)
poetry2conda
python310Packages.aioeafm
python310Packages.aria2p
python310Packages.check-manifest
python310Packages.cliff
python310Packages.dulwich
python310Packages.elegy
python310Packages.expecttest
python310Packages.gcal-sync
python310Packages.hacking
python310Packages.hg-git
python310Packages.hypothesis-auto
python310Packages.ical
python310Packages.jedi-language-server
python310Packages.keystoneauth1
python310Packages.openstackdocstheme
python310Packages.openstacksdk
python310Packages.osc-lib
python310Packages.oslo-concurrency
python310Packages.oslo-config
python310Packages.oslo-context
python310Packages.oslo-db
python310Packages.oslo-i18n
python310Packages.oslo-log
python310Packages.oslo-serialization
python310Packages.oslo-utils
python310Packages.pipenv-poetry-migrate
python310Packages.poetry
python310Packages.pyairnow
python310Packages.pytest-golden
python310Packages.python-cinderclient
python310Packages.python-jenkins
python310Packages.python-keystoneclient
python310Packages.python-novaclient
swiftclient (python310Packages.python-swiftclient)
python310Packages.scmrepo
python310Packages.stestr
python310Packages.strawberry-graphql
python310Packages.subunit2sql
python310Packages.tempest
python39Packages.aioeafm
python39Packages.aria2p
python39Packages.breezy
python39Packages.check-manifest
python39Packages.cliff
python39Packages.dulwich
python39Packages.elegy
python39Packages.expecttest
python39Packages.gcal-sync
python39Packages.hacking
python39Packages.hg-git
python39Packages.hypothesis-auto
python39Packages.ical
python39Packages.jedi-language-server
python39Packages.jenkins-job-builder
python39Packages.keystoneauth1
python39Packages.klaus
python39Packages.openstackdocstheme
python39Packages.openstacksdk
python39Packages.osc-lib
python39Packages.oslo-concurrency
python39Packages.oslo-config
python39Packages.oslo-context
python39Packages.oslo-db
python39Packages.oslo-i18n
python39Packages.oslo-log
python39Packages.oslo-serialization
python39Packages.oslo-utils
python39Packages.pipenv-poetry-migrate
python39Packages.poetry
python39Packages.pyairnow
python39Packages.pytest-golden
python39Packages.python-cinderclient
python39Packages.python-glanceclient
python39Packages.python-heatclient
python39Packages.python-ironicclient
python39Packages.python-jenkins
python39Packages.python-keystoneclient
python39Packages.python-manilaclient
python39Packages.python-novaclient
python39Packages.python-openstackclient
python39Packages.python-swiftclient
python39Packages.scmrepo
python39Packages.stestr
python39Packages.strawberry-graphql
python39Packages.subunit2sql
python39Packages.tempest
rabbitvcs
reno
streamdeck-ui
vcstool
vimPlugins.YouCompleteMe
vscode-extensions.ms-python.python
xandikos
ycmd


Ppp

Codecov Report
Base: 43.00% // Head: 43.00% // No change to project coverage :thumbsup:

Coverage data is based on head (80b2d95) compared to base (0a9246b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #196   +/-   ##
=========================================
  Coverage     43.00%   43.00%
  Complexity      442      442
=========================================
  Files           111      111
  Lines          2351     2351
  Branches        271      271
=========================================
  Hits           1011     1011
  Misses         1283     1283
  Partials         57       57

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add shuffle metrics for parallel indexing

Description
Part of https://github.com/apache/druid/issues/10352. This PR adds these metrics for middleManagers. These metrics have the supervisorTaskId as their dimension.

ingest/shuffle/bytes: Number of bytes shuffled per emissionPeriod.
ingest/shuffle/requests: Number of shuffle requests per emissionPeriod.

I haven't updated document yet, will add them with missing shuffle configurations together in a follow-up PR.

This PR has:

[x] been self-reviewed.

[x] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

Looks like the analyzeDependencies job is failing
[WARNING] Unused declared dependencies found:
[WARNING]    org.checkerframework:checker-qual:jar:2.5.7:compile

Thanks, fixed now.


Automated cherry pick of #9480

Cherry pick of #9480 on release-6.3.
/cc  @cwarnermm
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 025cbcfa02fa67578aad012b23c8bbe130d5eeaf


Coexistence with Base Line Policy and Custom Policies and Price Related question

Checking the documentation on this link :
https://docs.microsoft.com/en-us/azure/active-directory/conditional-access/concept-baseline-protection#
I see that activating Baseline policy for End Users is "Free" . That's so good, but, i need to clarify what happens if we need to create a specific custom policy in order to Exclude some PowerUsers. Reading this documentation we need to add a AD P1 to those "Powerusers" in order to apply a custom Policy. But we want to maintain the Baseline Policy for all other users ( because is free ). How about this combination of policies ? If a PowerUser has applied both policies ( Baseline and Custom ) the Custom policy will win always ? also, it's a supported scenario to have the Base Line policy applied and use several Custom policies managed a AD P1 for those users ?
It will be interesting to add this scenario to the Documentation, as we have several clients asking for this :).
Thanks !

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

ID: 53ea4e70-2931-8691-e9cc-13c72b514d9c
Version Independent ID: c1aa4a8f-16b2-79ac-edb1-8a87f51a3f7b
Content: Conditional Access baseline protection policies - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-baseline-protection.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Aniseto Thank you for the feedback. We are actively investigating and will get back to you soon.

@MicrosoftGuyJFlo Could you please review the above scenario if it can be added to the documentation.

Conditional Access is a premium feature and requires a license. Azure AD is licensed per user. https://azure.microsoft.com/pricing/details/active-directory/

#please-close


Preventing select2:opening event throws an error

Preventing the select2:opening event throws "args is undefined" error. Here is a fiddle for troubleshooting: http://jsfiddle.net/r56zh872/5/

I was able to avoid this error by adding a line to Select2.prototype.trigger (which is in core.js):
actualTrigger.call(this, preTriggerName, preTriggerArgs);
args = args || {};  //   <------ added this line
if (preTriggerArgs.prevented) {

I don't know if there are any side effects to this, but this fix seems to work fine.

This should have been fixed in https://github.com/select2/select2/commit/71669359582ab04571f13bce8ff5c99318739c7b and is a duplicate of https://github.com/select2/select2/issues/3431.


WIP: some testing of Model catalogue functions

Codecov Report

Merging #360 (9622670) into master (01f6ab1) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #360      +/-   ##
==========================================
+ Coverage   72.29%   72.35%   +0.05%
==========================================
  Files         157      157
  Lines       28119    28088      -31
==========================================
- Hits        20330    20322       -8
+ Misses       7789     7766      -23

Impacted Files
Coverage Δ

resqpy/olio/simple_lines.py
40.16% <0.00%> (-0.49%)
:arrow_down:

resqpy/model/_xml.py
85.86% <0.00%> (-0.28%)
:arrow_down:

resqpy/olio/wellspec_keywords.py
75.34% <0.00%> (-0.23%)
:arrow_down:

resqpy/olio/factors.py
97.05% <0.00%> (-0.09%)
:arrow_down:

resqpy/fault/_gcs_functions.py
89.05% <0.00%> (-0.08%)
:arrow_down:

resqpy/well/well_utils.py
95.06% <0.00%> (-0.07%)
:arrow_down:

resqpy/olio/triangulation.py
92.62% <0.00%> (-0.04%)
:arrow_down:

resqpy/lines/_polyline.py
90.24% <0.00%> (-0.03%)
:arrow_down:

resqpy/olio/transmission.py
84.26% <0.00%> (-0.03%)
:arrow_down:

resqpy/grid_surface.py
81.06% <0.00%> (-0.03%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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

I think this batch of tests is worth merging in, even though the Model._forestry module is not covered yet.


Bugfix request info

Issues associated with this PR
Your PR should present related issues you want to solve.
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

This value is set when a problem with the request, and is configured according to the log format of istio when it obtained.


Add support for 'add' operation without position.

Spec (https://tools.ietf.org/html/rfc6902#section-4.1) requires this operation to replace an existing target field or add a new one, both of which are covered by the same $set mongodb operation.

This PR should be merged as it fixes the issues with add. Any news ?

Any update?

+1 I am waiting for this resolve ;)


Fans don't spin using manual mode on G15 AMD

Every time I try to set my fans to spin according to the setting, they literally go to the lowest RPM possible and don't spin. Also, every time it tries to set the boost, my audio pops and there's a frame skip on my screen, as if it's trying to change the entire power mode.
I've tried to use manual mode on this laptop since I got it, and it had BIOS 1.8.0, and it's now on 1.16.1, none of the BIOS I've tested ever worked. Also, I've tried pretty much every version of AlienFX Tools since 8.3.0.

As the images below suggest, the boost dot alternates between red and green. Using the command "alienfan-cli.exe unlock" gets me exactly the same result. But, if I put a really high polling rate, the program is able to set the speed as it's intended to, yet it spins down to zero as soon as the time defined in ms passes and then goes up again depending on the temperature. For example, I've tried setting a different curve here:

Here's the system ID detected by the program:

I'm totally open to create any dump of this system if needed.
My system:
Dell G15 5525, (R7 6800H with RTX 3060)
Windows 11 23H2 stable
BIOS 1.16.1

I've tried downloading AlienFX 8.1.4 and somehow, the fan control works perfectly on this version. It would be very appreciated if this problem is investigated. Also, from 8.2.0 onwards, it absolutely stops working on my laptop.

Both GPU and CPU fans are working now accordingly to the curves and absolutely no audio pops of frame skips on my machine.

Thanks. I'll check it then have a little time...

Issue fixed as of latest version (<IP_ADDRESS>)


Dim Level Status Effect

On the notification manager, is there any way to allow an option to have the dim level shown as one of the effect options?

Could you clarify what you mean? Currently there is an option for changing the brightness level and an option for choosing the desired effect.

My use case here is that when I have an automation running, I change the LED color so others know the device is running an automation however, when I use this to change the LED color, it appears to override the LED dimmer level indicator (how far up the bar goes) and just applies the selected effect.
To resolve this my request would be to add an effect that only changes the LED color but keeps the dimmer indicator level unchanged.  Hopefully that makes sense.
Thanks!

Ah gotcha, makes sense! I could be wrong, but in all of my experience the notifications use the entirety of the LED bar when active and don’t take into account the current dimmer level of the switch. One option that might work (only for solid effects) is to use the LED manager node to change the LED color of the bar to still have a visual reference to your automation running, and this would honor the current dim level of the switch.

I did consider that however the drawback there is that I can't simple use the clear notification option on the notification manager to remove it, which is slick!    Not the end of the world I guess to have a separate flow to change it back to the previous color though.   Figured I'd check to see if it could be done.


docs: adr on native binaries

Addressing https://github.com/algorandfoundation/algokit-cli/issues/383
Proposed Changes

Initial draft on evaluating switch to new distribution model with native binaries

@achidlow @daniel-makerx @robdmoore (cc @joe-p) please note this is still a draft WIP but feel free to leave some initial thoughts opinions on this


Some float value would map to int in enum values e.g 1.0 -> 1

See the example for FloatEnum which has values 1.0, 2.0, 3.0 would be mapped to 1, 2, 3. I think this is incorrect.
enum FloatEnum {
  One: 1.0,
  Two: 2.0,
  Three: 3.0,
}

swagger spec:
FloatEnum:
      type: number
      enum:
        - 1
        - 2
        - 3

Another thing is I notice that the docs guides me following cases but it doesn't work, could you update the docs?
enum Hour {
  Zero = 0,
  Quarter = 0.25,
  Half = 0.5,
  ThreeQuarter = 0.75,
}

Seems like we have a typo in the docs, the example above show the correct syntax with : instead of =

@timotheeguerin Does that mean every emitter needs to check by themselves if they have requirement to distinguish 1.0 and 1?
model Result {
  prop: 1.0   // Here emitter may need to chech if the devired type is int or float
}

yeah.. which is not ideal. Or we need to have at least an helper of the sort getNumericPrecision(numericType) // => int32, float64, etc.)

fix the above doc

Doc fixed in https://github.com/microsoft/typespec/pull/2245, this can remain as a design discussion

I opened this issue: https://github.com/microsoft/typespec/issues/2240
This would be an issue in csharp because adding a new value will cause breaking changes in C#
for example, in one version we have:
enum Foo {
A: 1.0;
B: 2.0;
}

and everything will generate as integers. When they add a real float number, everything suddenly changed to generate as float:
enum Foo {
A: 1.0;
B: 2.0;
C: 3.3;
}


Create miq provision virt workflow

Adding a middle layer for other workflows to inherit from (MiqProvisionConfiguredSystemWorkflow and eventually MiqProvisionHostWorkflow)

@gmcculloug Please review

This is just a rename of MiqProvisionWorkflow to MiqProvisionVirtWorkflow and creating a new blank MiqProvisionWorkflow

Wow this aligns amazingly well with my EMS refactoring for the Foreman workers.
:+1: on the idea.  I still have to review.  I would say ignore the rubocop stuff on the rename for now.

@brandondunne it says so much was changed, could you summarize what was actually changed?

TODO:

[x] Spec recursion on SUBCLASSES and test encrypted fields
[x] Update ws callers
[x] Add TODO: validate_sysprep_file is a UI validation helper, not a part of the workflow


Bad Sad thrown when enabling an unused import

Description:
Open parser.bal and try to enable and disable ballerina/io import in line 2. Then try to compile it.

Steps to reproduce:
Project is attached
bal-order-processing.zip
Affected Versions:
2.0.0-SLA3

As per the gif attached, crashing occurs at Desugar.  Also tired parsing the bal file no issue saw related parser.

This is fixed in Ballerina 2201.0.0 (Swan Lake), hence closing.


BUGFIX: Reconciled text color fix

Github Issue (if applicable): #470
Trello Link (if applicable): n/a
Forum Link (if applicable): n/a
Explanation of Bugfix/Feature/Enhancement:
Fix for #470. Corrects an issue where the reconciled row's styling could be lost after it was checked and then unchecked.
Recommended Release Notes: n/a (minor issue)

@joshmadewell - Any chance you could glance at this and double check I haven't accidentally introduced any bugs? I tweaked your code slightly, and while everything seems to be working correctly, it would be nice to have a second set of eyes on this in case you know of an edge case that I'm not aware of.

This looks bueno to me! Nice job -- easier fix than I thought it'd be ;)

Awesome!


Add a header level offset?

+++
title = "Test"
+++

## First

will generate a h1 tag for "Test" and a h2 tag for "First" but it is invalid Markdown (First header should be a top level header [MD002] from mdl).
Would it be feasible to have header level offset configuration that lets you start the # First level markdown at header level 2 (i.e. h2)?

I don't believe this theme is the only one in hugo that suffers from this issue. To me the simplest solution would be to disable the lint check for your particular project/website as it isn't applicable to the markdown you are writing.


Song detune as a slider

The song detune modulator is a modulator that controls the pitch of the entire song (except things with SD-Affected ticked off). What is strange is that there is no actual song detune setting, just the modulator. Why not make one?

Now that I think about it, this might require some changes to the modulator as well. Not sure how big/small these changes would be but I know they'll exist.
As for where to put it, perhaps between song key and song key octave.
Another thing. I think there may be like, around 500 options to put on the slider assuming it follows the modulator stepping (1 = 4) so perhaps each slider option could be in steps of 50, then an input box can be used for those in-betweens.


Simulation freezes on attempt for second connection.

Dear Visit-Developers; The first time connection to an instrumented simulation (2 parallel processes) code works fine.When I try to reconnect after a disconnect the simulation freezes at VisitAttemptToCompleteConnection:CreateEngineAndConnectToViewer Calling visit_engine Calling visit_initialize: argvvisit v 2.4 engine debug 5 noloopback host login2 port 18204 key 519e5a6a6c2831a08ec1 sshtunnelingfor Process 0 and VisItAttemptToCompleteConnection:VerifySecurityKeys desc=13539016 BroadcastInt value=1607671674, sender=0 BroadcastInt return 0 BroadcastString str=d1e7a5906389b49d, len=1607671675, sender=0 BroadcastString return 0 BroadcastInt value=10, sender=0 BroadcastInt return 0 BroadcastString str=, len=11, sender=0 BroadcastString return 0VerifySecurityKeys return 0VisItAttemptToCompleteConnection VerifySecurityKeys failed. return 0for Process 1. What am I missing? Attached find a modified version of pjacobi.py (mpi4py) which reproduces the problem. Yours sincerely,Christoph Statz
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1065
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Simulation freezes on attempt for second connection.
Assigned to: -
Category: -
Target version: -
Author: Brad Whitlock
Start: 05/16/2012
Due date:
% Done: 0%
Estimated time:
Created: 05/16/2012 01:44 pm
Updated: 07/21/2014 03:44 pm
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: 2.4.2
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Dear Visit-Developers; The first time connection to an instrumented simulation (2 parallel processes) code works fine.When I try to reconnect after a disconnect the simulation freezes at VisitAttemptToCompleteConnection:CreateEngineAndConnectToViewer Calling visit_engine Calling visit_initialize: argvvisit v 2.4 engine debug 5 noloopback host login2 port 18204 key 519e5a6a6c2831a08ec1 sshtunnelingfor Process 0 and VisItAttemptToCompleteConnection:VerifySecurityKeys desc=13539016 BroadcastInt value=1607671674, sender=0 BroadcastInt return 0 BroadcastString str=d1e7a5906389b49d, len=1607671675, sender=0 BroadcastString return 0 BroadcastInt value=10, sender=0 BroadcastInt return 0 BroadcastString str=, len=11, sender=0 BroadcastString return 0VerifySecurityKeys return 0VisItAttemptToCompleteConnection VerifySecurityKeys failed. return 0for Process 1. What am I missing? Attached find a modified version of pjacobi.py (mpi4py) which reproduces the problem. Yours sincerely,Christoph Statz
Comments:
The problem seems to persist in visit 2.5.0. Maybe this helps someone trying to use the python libsim with mpi4py or even fixing the underlying problem:Where do the broadcast function handles get lost and why obviously only in the python version? Bug/Behavior persists in 2.6.1 Are you sure this only concerns Pyton? I'm seemingly having the same issue with Libsim and C in VisIt 2.7.3.

1073_pjacobi_mpi4py.py.zip


feat(TableV2): new component

Summary
Type

Feature

Summarise concisely:
Add a new component TableV2. The endgoal is to merge Table (v1), List (v1) variant table & List (v1) variant explorer in this new component.
Table (V1) didn't supported ordering & selectable rows

Juste quick note on checkbox component, it doesn't listen to keydown Enter to change it's state (but not related to TableV2) 🚀

Juste quick note on checkbox component, it doesn't listen to keydown Enter to change it's state (but not related to TableV2) 🚀

@ModuloM This might seems strange but for changing state with accessibility you should use space bar instead of enter. If you check on native checkbox that's how you change it and enter doesn't work 😛


Add translation for useState hook

Glossary document says there is no need translation of the state word.
so should i not touch this [state variable] ?

Glossary document says there is no need translation of the state word. so should i not touch this [state variable] ?

Hi @thecodermehedi
Really appreciate your initiative and also great that you read the glossary. To answer your question, Yes, You don't need to translate the word into Bengali.
The reason behind this is, State is a very common word in the realm of React. So we want to keep it as it is.