x/pkgsite: Whole file non runnable examples are incorrectly shown with a run button

What is the URL of the page with the issue?
https://pkg.go.dev/github.com/bufbuild/connect-go#example-package-Handler
Specifically the Example (Handler) section
What is your user agent?
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Screenshot

What did you do?
Created a whole file example in a project here that is not runnable (doesn't have //output comment).

Code
// Copyright 2021-2023 Buf Technologies, Inc.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
//      http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

package connect_test

import (
	"context"
	"net/http"

	"github.com/bufbuild/connect-go"
	pingv1 "github.com/bufbuild/connect-go/internal/gen/connect/ping/v1"
	"github.com/bufbuild/connect-go/internal/gen/connect/ping/v1/pingv1connect"
)

// ExamplePingServer implements some trivial business logic. The Protobuf
// definition for this API is in proto/connect/ping/v1/ping.proto.
type ExamplePingServer struct {
	pingv1connect.UnimplementedPingServiceHandler
}

// Ping implements pingv1connect.PingServiceHandler.
func (*ExamplePingServer) Ping(
	_ context.Context,
	request *connect.Request[pingv1.PingRequest],
) (*connect.Response[pingv1.PingResponse], error) {
	return connect.NewResponse(
		&pingv1.PingResponse{
			Number: request.Msg.Number,
			Text:   request.Msg.Text,
		},
	), nil
}

func Example_handler() {
	// protoc-gen-connect-go generates constructors that return plain net/http
	// Handlers, so they're compatible with most Go HTTP routers and middleware
	// (for example, net/http's StripPrefix). Each handler automatically supports
	// the Connect, gRPC, and gRPC-Web protocols.
	mux := http.NewServeMux()
	mux.Handle(
		pingv1connect.NewPingServiceHandler(
			&ExamplePingServer{}, // our business logic
		),
	)
	// You can serve gRPC's health and server reflection APIs using
	// github.com/bufbuild/connect-grpchealth-go and
	// github.com/bufbuild/connect-grpcreflect-go.
	_ = http.ListenAndServeTLS(
		"localhost:8080",
		"internal/testdata/server.crt",
		"internal/testdata/server.key",
		mux,
	)
	// To serve HTTP/2 requests without TLS (as many gRPC clients expect), import
	// golang.org/x/net/http2/h2c and golang.org/x/net/http2 and change to:
	// _ = http.ListenAndServe(
	// 	"localhost:8080",
	// 	h2c.NewHandler(mux, &http2.Server{}),
	// )
}

What did you expect to see?
Expected for the whole file to be shown without a run button shown.
What did you see instead?
A run button, even though the example is non runnable (long running server process)
Because this example is non runnable it fails with the following error because an internal package is used. Even if it would compile it would probably fail because it's attempting to start a long running service.
go: downloading github.com/bufbuild/connect-go v1.4.1
go: downloading google.golang.org/protobuf v1.28.1
package play.ground
	prog.go:8:2: use of internal package github.com/bufbuild/connect-go/internal/gen/connect/ping/v1 not allowed
package play.ground
	prog.go:9:2: use of internal package github.com/bufbuild/connect-go/internal/gen/connect/ping/v1/pingv1connect not allowed

Duplicate of #9679


Huawei - Kirin processor - unlock bootloader - PotatoNV

I found this projet, but the TestPoint mode is needed.
https://github.com/mashed-potatoes/PotatoNV/wiki/How-to-use-PotatoNV

Yes I know of that, use it if you can.


Hybrid Flow sample did not work for me

Hi
I just followed the doc step by step and when reached "Switching to Hybrid Flow and adding API Access back" (http://docs.identityserver.io/en/dev/quickstarts/5_hybrid_and_api_access.html?highlight=refresh token), when the mvc client cannot access the identityserver and I get the following error :

Here is my configuration in mvc client :
`app.UseOpenIdConnectAuthentication(new OpenIdConnectOptions
{
            AuthenticationScheme = "oidc",
            SignInScheme = "Cookies",

            Authority = "http://localhost:5000",
            RequireHttpsMetadata = false,

            ClientId = "mvc",
            ClientSecret = "secret",

            ResponseType = "code id_token",
            Scope = { "api1", "offline_access" },
            GetClaimsFromUserInfoEndpoint = true,
            SaveTokens = true

        });`

And Startup file also the config in Identityserver app:
`public class Startup
{
// This method gets called by the runtime. Use this method to add services to the container.
// For more information on how to configure your application, visit http://go.microsoft.com/fwlink/?LinkID=398940
public void ConfigureServices(IServiceCollection services)
{
services.AddMvc();
//.AddJsonFormatters()
//.AddAuthorization();
        services.AddDeveloperIdentityServer()
        .AddInMemoryScopes(Config.GetScopes())
        .AddInMemoryUsers(Config.GetUsers())
        .AddInMemoryClients(Config.GetClients());
    }

    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
    {
        loggerFactory.AddConsole();

        if (env.IsDevelopment())
        {
            app.UseDeveloperExceptionPage();
        }

        //app.UseDeveloperExceptionPage();

        app.UseIdentityServer();

        app.UseStaticFiles();
        app.UseMvcWithDefaultRoute();
    }
}`

`new Client
{
ClientId = "mvc",
ClientName = "MVC Client",
AllowedGrantTypes = GrantTypes.HybridAndClientCredentials,
                    RequireConsent = true,

                    ClientSecrets =
                    {
                        new Secret("secret".Sha256())
                    },

                    RedirectUris           = { "http://localhost:5002/signin-oidc" },
                    PostLogoutRedirectUris = { "http://localhost:5002" },

                    AllowedScopes =
                    {
                        StandardScopes.OpenId.Name,
                        StandardScopes.Profile.Name,
                        StandardScopes.OfflineAccess.Name,
                        "api"
                    }
                }

...
return new List
{
                StandardScopes.OpenId,
                StandardScopes.Profile,
                StandardScopes.OfflineAccess,

                new Scope
                {
                    Name = "api1",
                    Description = "My API"
                },

            };

`

Not enough info in this short log snippet.

@brockallen I followed the mentioned sample and all the codes are same. When I run the MVC client it redirects to Identityserver (in this case localhost:5000), and the error appears in console, "Bearer was not authenticated"
If I switch back the client configuration to what described here "http://docs.identityserver.io/en/dev/quickstarts/3_interactive_login.html" it again works.

Did you try the working sample from the samples repo?
https://github.com/IdentityServer/IdentityServer4.Samples/tree/dev/Quickstarts/5_HybridFlowAuthenticationWithApiAccess

@leastprivilege yes it did work with original sample. Now I have problem using postman calling the token endpoint, cause it throws the following error :
....
CorsPolicyService did not allow origin: chrome-extension://fhbjgbiflinjbdggehcddcbncdddomop
....
I tried this in my config :
services.AddCors(options =>
            {
                options.AddPolicy("AllowOrigins", policyBuilder =>
                {
                    policyBuilder
                    .AllowAnyMethod()
                    .AllowAnyOrigin()
                    .AllowAnyHeader();
                });
            });

...
 app.UseIdentityServer();
 app.UseCors("AllowOrigins");

But still get same error.

That's a postman issue I guess - but definitely a new issue. closing this one.

The Postman issue seems to be the same as https://github.com/IdentityServer/IdentityServer4/issues/439


WorldUtils Initialization Bug

The WorldUtils class needs to be the first thing to register so it gets access to the MinecraftServer instance first before anything else tries to use it. Currently, the Dynmap wrapper runs first and causes an error.

The way the DynmapWrapper handles getting the world should also be changed because it still gets its own reference to the server but never uses it.


[dx12] don't create RTVs when not possible

Current DX12 code for view creation checks for the image usage and attempts to create all of DSV/RTV/SRV/UAV based on the original image usage. When the user creates a view over multiple levels, neither of DSV/RTV are possible, and we currently panic. We should be handling that gracefully and just accept the fact there is no associated DSV/RTV if it's not possible.
This affects wgpu-rs mipmap example.

Fixed by #2922


Create design for Chisquad 002 Reflections blog on featured posts.

👋 Welcome!
Welcome to Chimoney's open-source community! This issue is specifically designed for first-time contributors, so if you're new to open source or haven't contributed to our codebase before, you're in the right place. We're here to help you take your first steps and guide you through the process of making your first contribution.
What You'll Need to Know
You don't need any prior knowledge or experience to work on this issue. We're here to support you every step of the way, so don't hesitate to ask questions if you're unsure about anything.
Problem
We don't have designs for each of the thumbnails in our featured posts section. We would like to have a design for the A Complete Guide to Integrating the Chimoney GitHub Bot blog

📋 Step by Step

Claim this issue: Comment below to claim this issue. If someone else has already claimed it, feel free to ask if they've started working on it or if they need any help.

Create the design: Create a thumbnail image with dimensions as 426 x 359 px that is appropriate for the A Complete Guide to Integrating the Chimoney GitHub Bot blog

Commit your changes: Once you've made the necessary updates, commit your changes and then submit a pull request so we can review your changes and merge them into the main codebase.

💬 Keep Us Updated
We encourage you to keep us updated on your progress. If you encounter any challenges or need assistance, don't hesitate to ask for help. We're here to support you throughout the process.
👥 Collaboration
If you see someone else working on this issue or if you'd like to collaborate with others, feel free to reach out and offer your assistance. Collaboration is key to our success as a community!
❓ Questions?
If you have any questions or need clarification on anything, please leave a comment below. We're here to help!

@phyleria , Please assign me this issue

@phyleria , Take a look at this

This is nice @callaghan210-coder !
Just a few suggestions:

Can you remove the </>2024</> tag
Have the blog title in sentence case
Remove featured posts
Resize the design to 426 x 359 px

Thanks.

Alright @phyleria , on it

Hi @callaghan210-coder
Kindly do away with the both of them.

I hope this is now okay

Yeah this is ok.
Thanks @callaghan210-coder !

Your welcome, I am happy to contribute to this @phyleria


[Button] Don't open the link in the same page instead of a new page

Functionality of
Go to home page

instead of
Go to home page

when a button is clicked
e.g.
sac.ButtonsItem(label='link', icon='share-fill', href='/', target='_self'),

Related https://github.com/streamlit/streamlit/issues/7464


rhoaieng 1094

use context to create new loggers
remove auto add issue workflow

@KPostOffice Can you please adjust the PR description to be more descriptive?

Hey @KPostOffice :)
This makes sense to remove, however, maybe updating the codeflare docs here could also go alongside this issue.

Hey @KPostOffice :) This makes sense to remove, however, maybe updating the codeflare docs here could also go alongside this issue.

@VanillaSpoon https://github.com/project-codeflare/.github/pull/17


[terra-embedded-content-consumer] Enable content scrolling within the the frame

Summary

Allow content within the iframe to scroll
What was changed:
Adding a scrolling prop with default to true to the iframe so that when more content presented not fit in the view port, content can be scrolled
Why it was changed:
Allow content within the iframe to be scrolled when the content doesn't have any interactable element
Testing

This change was tested using:

[x] WDIO
[x] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[x] Accessibility review
[x] Functional review

Additional Details

This PR resolves:
UXPLATFORM-9420

Thank you for contributing to Terra.
@cerner/terra

@sycombs  - addressed the comment, please take another look

Based off of https://github.com/cerner/terra-framework/pull/1777#issuecomment-1717851691 , looks like there needs to be some additional discussion around this change.

We need to enable scrolling by default so that the embedded content can be scrolled. With scrolling enabled, we to provide an option to turn that off (if needed by consumers).

For a11y, if we're explicitly supporting scrolling in iframes, there should be a visible focus indicator when it has keyboard focus consistent with other focus indicators for the respective themes.
cc: @scottwilmarth

For a11y, if we're explicitly supporting scrolling in iframes, there should be a visible focus indicator when it has keyboard focus consistent with other focus indicators for the respective themes.
cc: @scottwilmarth

@eawww - yeah, we have another ticket to add support for visible focus indicator.

Since this is a documentation update, this will no longer need functional review.


Cannot send .testTx

When I try to send a test transaction with .testTX, the following message appears and hangs:
⠋ Sending test transaction to network...(node:10685) UnhandledPromiseRejectionWarning: ReferenceError: conf is not defined
    at REPLServer.action (/home/master/eac/cli-master/src/TimeNode/repl.js:136:7)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:118:7)
(node:10685) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:10685) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Using Version 4.2.5
npm version 6.2.0
Ubuntu 17.10

Closed by #76


ssl=true and version is not defined, gives an error

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: all
Ruby: -
Distribution: all
Module version: v8.0.2

How to reproduce (e.g Puppet code you use)
class { 'elasticsearch' :
  ssl                  => true,
  ca_certificate       => '/path/to/ca.pem',
  certificate          => '/path/to/cert.pem',
  private_key          => '/path/to/key.pem',
  keystore_password    => 'keystorepassword',
}

What are you seeing
When you want SSL enabled and use the default packages from the OS (so not setting a version), there is an error in getting the catalog.
What behaviour did you expect instead
When no version is defined, expect a version of 7 or higher and add the tls_config from that one.
Output log
 Evaluation Error: Error while evaluating a Function Call, 'versioncmp' parameter 'a' expects a String value, got Boolean (file: /etc/puppetlabs/code/environments/xxx/modules/elasticsearch/manifests/config.pp, line: 96, column: 11)
Any additional information you'd like to impart

Hi, thanks this also fixes https://github.com/voxpupuli/puppet-elasticsearch/issues/1174
At least partly.. shouldn't be the if case not be:
if ($elasticsearch::version == false and versioncmp($elasticsearch::version, '7') >= 0)

cause rn, when i set the version to false, it rolls out the tls config for ela6 and not for ela7
see also here:

What behaviour did you expect instead
When no version is defined, expect a version of 7 or higher and add the tls_config from that one


SeeQR 8.0 Issues

MySQL
[ ] - Unable to import files into SeeQR directly

--MySQL error prevents import without direct install of brew of all MySQL.
[ ] -No export option available.
--Need to fine-tune the query for MySQL.
[ ] Unable to duplicate databases
--Need Query adjustment and fix of copy query.
[ ] Drop database after switching to postgres will sometimes cause issue where unable to delete selected database.
--Most likely has to do with current and last database being reassigned incorrectly.
[ ] MySQL execution query doesn’t function consistently.
-- Need to see how data is being returned from query to correctly translate it to the frontend.

ERD

[x] - Easier viewing options. Pan to scroll or click to move options.
--Difficult to navigate around the ERD when the database is large.
[ ] -MySQL tables don’t populate in the correct location when being imported.
--Top table node doesn’t connect to corresponding columns causing tables to look separated.
--- Need to see how data is being returned from query and translated to frontend.
[x] - MiniMap receives greater functionality and allows for faster movement.
--Needs click to navigate feature and pan to scroll for fast movement.
--Zoomed very far out with MySQL and difficult to see tables.
[ ] Optimized query to load database quicker.
--Large databases will take awhile to load. Can benefit from optimized query.
[x] Saving ERD when using MySQL is inconsistent.
--Additional tables fail to save after multiple tables have been added and saved.

Table View

[x] - DummyData button doesn’t work
Query doesn’t provide dummy data.
Needs to be functional with MySQL
[x] - Table viewing from the table view needs dropdown view
--Difficult to navigate around when there are a lot of tables

Quality of life

[x] - Delete button doesn’t ask for confirmation
--- Seen in both column and sidebar view.
[ ] Export icon next to duplicate and trash icon
[ ] Dependency issues when doing NPM install for Mac.
--Need to move on from deprecated MUI.
[ ]  Scrollbar for sidebar when lots of databases are connected.

Unable to checkmark the first section of the issues mentioned. See below for an update of the resolved issues:
[X] - Unable to import files into SeeQR directly
--MySQL error prevents import without direct install of brew of all MySQL.
[ ] -No export option available.
--Need to fine-tune the query for MySQL.
[X] Unable to duplicate databases
--Need Query adjustment and fix of copy query.
[ ] Drop database after switching to postgres will sometimes cause issue where unable to delete selected database.
--Most likely has to do with current and last database being reassigned incorrectly.
[ ] MySQL execution query doesn’t function consistently.
-- Need to see how data is being returned from query to correctly translate it to the frontend.


Automate tracking of the community pull requests

Background
Almost every morning, @MisRob will make a cup of :tea:, and opens the list of Kolibri pull requests, Studio pull requests, and KDS pull requests and finds all new/updated/closed pull requests authored by external volunteers.
Then she logs them in Google Sheet spreadsheet saved on Learning Equality's Google Drive.

The spreadsheet is used for multiple purposes, for example to pull out user names of contributors for a particular month so we can send them gratitudes, to nudge core team members to follow-up on reviews, to have an overall overview of activity, etc.
Summary
The goal of this task is to automate the aforementioned workflow. This will help us save some time that could be used for interacting with the community instead, and will also prevent from missing some pull requests.
On a high-level it would work:

A new row is inserted to the spreadsheet when a new pull request is open in Kolibri, Studio, and KDS repositories by an author who's not a member of Learning Equality GitHub organization
An existing row is updated when a corresponding pull request is updated (merged, closed, title changed, reviewer changed, assignee changed...)

Spreadsheet columns:

Merged: date when a PR was merged, in yyyy-mm-dd format, example  2024-03-25
Link: PR link, example https://github.com/learningequality/kolibri/pull/12382
Author: Author's GitHub username, example MisRob
Title: PR title, example Add initial version of Bloom Renderer
Repository: Repository name, example kolibri-design-system
Updated: Date when a spreadsheet row was updated the last time, in yyyy-mm-dd format, example  2024-03-25
Reviewers: GitHub usernames of "Reviewers", separated by comma, example rtibbles,akolson,MisRob
Assignees: GitHub usernames of "Assignees", separated by comma, example rtibbles,akolson,MisRob

All information above should be updated in the spreadsheet whenever there is a related change on a pull request.
Guidance

For the first version, it may be best to start two test repositories and a new Google Sheet. As soon as the implementation is stable, we will move to Learning Equality repositories and configure the action with access tokens.
For complex tasks, it is rather hard for our team to maintain shell-based implementations so

Using Python or JavaScript whenever possible is recommended. Here's some good guidance from @rtibbles https://github.com/learningequality/kolibri-design-system/pull/547#issuecomment-2083044718.
Please pay attention to readability and maintainability of implementation (to allow GH actions newbie such as @MisRob to add a new spreadsheet column with a bit of study in a reasonable time frame :)

As per @rtibbles's guidance, whenever possible we try to avoid 3rd party actions, except the official ones (security and upgrade related issues)
It'd be welcome if we didn't need to have three copies of the same action in aforementioned repositories, and also adding a new repository would be ideally straightforward

Comments
Not necessary but if it'd be interesting, we'd welcome exploration of implementing a layer that we could use as a basis for common tasks for many actions. For example, it could have a form of a file with helper functions get_pull_request_author, get_pull_request_title, is_member_of_le, etc. Perhaps this task could lay down some foundations.

@MisRob Could you please assign this issue to me? 😄

Yes , thank you @GarvitSinghal47

This is now done and will be observed in production. Thanks so much @GarvitSinghal47, this is incredibly helpful for me.


TypeError in Hive vault when switching from version 4.5.2 to 5.1.0

Describe the bug
If you store a String value into a hive vault in version 4.5.2 and want to retrieve it in version 5.1.0 a TypeError occurs when the value is decoded in file 'packages/stash/lib/src/api/store.dart'
To Reproduce

Setup a simple vault in version 4.5.2
Store a String value in that vault
Update stash to newest version 5.1.0 in pubspec.yaml
Run app again and try to retrieve value from vault

Expected behaviour
Expected behaviour is that value is retrieved without errors.
Screenshots

Version
Dart SDK version: 3.3.0 (stable) (Tue Feb 13 10:25:19 2024 +0000) on "macos_arm64"

I opened a pull request #71

This is a bit weird, just checked and this scenario is part of the integration tests, actually I test all the base types so not sure how this could be happening to you. Are you overriding the hive version in some way ?

Fixed in the 5.1.1 of stash and 5.1.2 of stash_hive. Thanks for spotting this, it was a tricky one @stevsimo


helm reliance on burst rate seems less viable than using QPS during discovery in managed environment

This is more of a general issue with helm that I noticed by virtue that it cascades down to the helm Terraform provider which adopts the same discovery approach for the config built.
In GKE, if you use Anthos Config Management and a range of other products you will end up with a very large number of CRDS.
We have observations of GKE installations carrying 341 CRDs, largely because of Anthos Config Connector which has CRDs for almost all GCP resources as it is used to provision those resources.
All clusters are minimum 1.26 K8S and come with rate limiting and QPS enable and also come with Google's 'Flow Control Guardrail' validating webhook. (more on that in a minute).
Where we use helm is via a Terraform provider, however, this problem would present with helm in it's more normal form and the provider inherits the problem due to the lack of QPS support as referenced by @phil9909 in https://github.com/helm/helm/issues/11280
We observe the helm deploy fail because of the discovery.NewDiscoveryCkuebtFirConfigOrDie() call without QPS settings in the REST config. Burst is set as a workaround here but burst really only allows a certain burst per second and rate limiting is applied even on burst the fair scheduling mechanism has some defences which means that it is very easy to exceed what burst will give you with 2 helm deployments occurring quite close together.
We have tried tuning the FlowSchema associated with the service account being used to apply the helm configuration, however, there are more issues around the Guard Rail webhook that you probably should be aware of.
The Guard Rail webhook explicitly blocks you from using one of the system predefined PriorityLevelConfigurations such as workload-high, or to avoid the issue completely the 'exempt' policy in a FlowSchema. The same Webhook also blocks you from defining a new policy with exempt.
In order to continue using GKE as a service we try to avoid customisation as far as possible to Google defined controls like this and probably other customers will feel the same.
Hence, it appears given the way things are going with these service offerings helm's use of burst to avoid rate limiting issues doesn't seem viable and in this scenario we are forced to look at the lesser change with is to avoid using the specific helm chart (Istio Gateway) and render it out for delivery avoiding the discovery call that lacks the WithDiscoveryQPS setting.
Cloud Provider/Platform (AKS, GKE, Minikube etc.): GKE

That would be great, I think kpt looking at the implementation simply used a fairly restricted limit (IIRC around 60) to overcome the problem and that was hard coded.

Would you be interested in making a PR for it?

Sure, I can take a look at it :)

btw, my original interest was helm terraform provider and I mentioned (but typo'd) what they do for burst. This was much easier to implement than I imagined because the config is passed directly to build a client with the burst/qps and other parameters.
I'll raise another PR with Hashicorp for the QPS parameter for their provider depending upon the outcome of this as I suspect they are more likely to be amenable to the proposed change if it has gone into helm upstream.


Rdstls

@joe-koral-imprivata ok, fixed a few issues detected from the log.
still, this looks like the same error code (something not ok with credentials or similar), but that would require some server side logging to find out what it is.
as for linux code stepping: check out qt-creator, nemiver or any other gdb/lldb frontend ;)

@joe-koral-imprivata hmm, looking at it, maybe they filter the user agent?
we set it in arm_handle_request so maybe try changing that to something mstsc or webview2/windows are sending?

@joe-koral-imprivata also, a dump of a request JSON from a working client could help (just the keys, not the values ;))

merging this as it improves the situation (although not fixing for the reporter)
@joe-koral-imprivata if you find out any more details, please open an issue with what you discovered, I´ll address that then.


how can i use all cpu with  face_encodings only this method

Hello sir,
how can i use all cpu with face_encodings only this method that's have a only one example for this repositry and someone is help to me

Python3
from multiprocessing import Pool
import face_recognition as fr

image_list = ['image1.jpg', 'image2.png', ...]
with Pool() as pool:
    pool.map(fr.face_encodings, image_list)


Force color option

It is really impossible to see the actual..expected diff when the color mode is turned off (for example when piping output to less). It would be nice if we could force the formatter to output colors when otherwise it would be turned off, for example with an --color option.

Optimally the diff should never be inline without the color coding, i.e. it should fall back to the normal:
+ actual
- expected

+ "Hello world"
- "ALL YOUR BASE ARE BELONG TO US!!!"


How could we help?

Not any issue at all, more of a question to our creator! :)
How can we help maintain this repo? The goal should be to have this lightweight framework up to date with its simplicity and not expanding functionality.
Please let us help! :)

@picturepan2 I saw you created an organization for Spectre here on GitHub, would you mind moving this repository there and adding some collaborators, if you don't have the time to maintain this awesome CSS lib yourself anymore?

I've created a Sublime text snippet autocomplete package for Spectre.
You can find it on SpectreCSS-Sublime-Snippets
Thought of leaving a word here 🤠

Spectre CSS is the best frontend framework I've ever seen. You guys have to continue it. Please ;)

100% agree, I really like SpectreCSS because of the simplicity, clean design and what it can do with CSS only.
Would be really great to keep this framework alive! Keen to know what your plans are @picturepan2 and how we could help..


How to query activity logs in a Storage account?

Hi,
If you send activity logs to a log analytics workspace you can use kusto queries to analyse the logs. Are there any tools that will help you with queries in storage accounts?
Regards Peter

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

ID: 8350b5be-45d4-cd5b-48eb-a65da3fa596b
Version Independent ID: 4ab6dc81-aa30-2a02-be89-1569632e850a
Content: Archive Azure resource logs to storage account - Azure Monitor
Content Source: articles/azure-monitor/platform/resource-logs-collect-storage.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@theheatDK Thank you for your query, we will look into it and get back to you at the earliest.

@theheatDK Please see if you can leverage Azure Storage explorer  to view the logs.  However to have a similar experience of querying it via log analytics, we don't have out of box functionality at this time to send storage logs to Log analytics but you can use custom scripts to sends logs to Log Analytics.
Additional reference.
Hope the above information helps, please let us know if you have any further queries. Thanks

Thanks @BharathNimmala-MSFT


Write Explanation: Update and beautify Galaxy landing page

Changed to how-to guide, closed by 5ae48c0


Peek - Escape Support

Description of the new feature / enhancement
It should be possible to close the Windows opened by Peek with just a press of the escape Key.
Scenario when this would be used?
This would greatly improve efficiency and the general workflow
Supporting information
No response

Duplicate of https://github.com/microsoft/PowerToys/issues/26137


[WIP]Refactor crossChainTest and improve crossChain-multiToken-tests

close #1906, close #1000, close #797

:white_check_mark: Build AElf <IP_ADDRESS>48 completed (commit https://github.com/AElfProject/AElf/commit/7fd94d700d by @xiangxingxing)

:white_check_mark: Build AElf <IP_ADDRESS>10 completed (commit https://github.com/AElfProject/AElf/commit/8e38022c54 by @xiangxingxing)

:white_check_mark: Build AElf <IP_ADDRESS>73 completed (commit https://github.com/AElfProject/AElf/commit/0ef319f2a3 by @xiangxingxing)

:white_check_mark: Build AElf <IP_ADDRESS>64 completed (commit https://github.com/AElfProject/AElf/commit/988c4cebaa by @EanCuznaivy)

:x: Build AElf <IP_ADDRESS>73 failed (commit https://github.com/AElfProject/AElf/commit/abdd642e97 by @rosona)

:x: Build AElf <IP_ADDRESS>79 failed (commit https://github.com/AElfProject/AElf/commit/5ddd07f4b9 by @EanCuznaivy)


Flying made easier

What feature do you want to get added on the base engine?
im sorry if this has been asked, pls dont ban me
on the character maker there should be a flying selection, to make a character fly vertically or horizontally
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

alright


React structure

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Created a folder structure for react app
Checklist

[x] I've read the contribution guidelines.
[x] This PR meets all the requirements like Clean Code, proper README and Documentation

Related Issues or Pull Requests
Fixes: #76

@BALaka-18 resolved the conflicts. Kindly review.

Are you working on this @Asim-2000 ?
Your PR is being reviewed here

Well, I just saw the review. Will do that ASAP.


Move pedal event logic to controlsd

Makes it so we only have to read the disengage on gas param once, and the car interfaces probably don't need to know about the toggle

I believe the diffs are just order based, the final alert is whatever is added first if the priority and start frame is the same

One of the specific routes with differences:

Looks okay to me. The active event is the first added event, so now we do the pedalPressed event check first and on rising edge of gas it's active for one frame.


Cranelift: hit unreachable case while lowering fcvt_to_uint

.clif Test Case
function %f13(f32x4) -> i32x4 {
block0(v0: f32x4):
  v1 = fcvt_to_uint.i32x4 v0
  return v1
}

Steps to Reproduce
Compiling this causes an unreachable case to be reached: https://github.com/bytecodealliance/wasmtime/blob/380db48ce6b7a754025e33bde02a8d14897376f3/cranelift/codegen/src/isa/x64/lower.rs#L800-L804
Expected Results
Either an error or a successful compilation.
Actual Results
Panic
Versions and Environment
Cranelift version or commit: 0c2e0494bdc63518962ad2773abc13d7d1d1dd8a
Operating system: linux
Architecture: x86_64

Hi @elliottt . Does this unreachable case get hit before patch https://github.com/bytecodealliance/wasmtime/pull/4684 ?
I have not tried, but the comments here say this unreachable is guarded by both the Opcode::FcvtToUnit and Opcode::FcvtToSint. It looks like 4864 removed one of those branches and if so I wondering if that patch (removing that code) introduced the possibility of this fall through?

I hit the panic at c4fd6a95da582a4a21802d981062c9202746ef19, which is the direct parent of #4684. This panic is from fcvt_to_uint, vs fcvt_from_uint which was ported to ISLE in #4684.

Hi @elliottt. I went to try and reproduce this issue and noticed that I was seeing the unreachable code but not where this original bug filing notes it. Looks like code has changed with https://github.com/bytecodealliance/wasmtime/pull/4704. Was that patch supposed to resolve this issue? I can reproduce still with this file:

test run
target x86_64
function %f13(f32x4) -> i32x4 {
block0(v0: f32x4):
v1 = fcvt_to_uint.i32x4 v0
return v1
}
; run: %f13([0x1.0 0.0 0x1.0 0x75bcd18.0]) == [1 0 1 123456789])

run with:
RUST_BACKTRACE=1 ./target/debug/clif-util test cranelift/filetests/filetests/runtests/test.clif


[QUESTION] Can "garbage" files also be removed?

First of all: thank you very, very much for providing this plugin! I am really glad, I found it!
I would like to ask a question, if this is ok here (if not, please tell me, where I should do so; I tend to stay away from stackoverflow, but if it is a must... well... then I will abide...)
Not including js files that webpack generates because of css files, works great. Nevertheless those "garbage" files live on in the public or dist folder and need a manual cleanup.
Do you know of a way how to remove those empty js files automatically, too?
Thank you!

Unfortunately, plugins which allow webpack to remove certain output files seem not to last very long. The last one I knew of (https://github.com/AnujRNair/webpack-extraneous-file-cleanup-plugin) has some people reporting that it no longer works as of webpack 4.
However, since they are empty files, they tend not to take up much space. As far as clean up, I'd recommend either using rimraf with a prebiuld step to cleanup the dist folder on each build, or a webpack plugin like the clean-webpack-plugin to nuke the output entirely before each build.

Thank you! But I only want to delete the empty files, which are actually generated usually after a prebuild or a "nuke" as you put it :-). They will still be upoaded to a server, where there was no "nuke" and mess up the file structure.
I will think about another solution, when I use the plugin again.
Have a good night!


Add pre-commit hook to prevent accidentally committing to master

Something like this:
- repo: https://github.com/pre-commit/pre-commit-hooks
  rev: v4.5.0
  hooks:
    - id: no-commit-to-branch
      args: ['--branch', 'main']
    - id: check-added-large-files
    - id: check-merge-conflict

Whatever you commit to master won't be authorized to be pushed without PR so I don't really see the use of it. Maybe I am missing something?

Yeah, the branch protection will prevent you from pushing local changes. This would just be in addition to that, so you can't end up with local changes to master in the first place.


Fix cmake drogonctl crosscompile

This one is a big one :smile:: it includes pregenerated template files and adds a new CMake option for cross compilation targets or for hosts that can't run the compiled executables for some reason (missing dynamic libraries).
It would be nice if a script (preferably not a shell script) could regenerate them, though I wasn't sure if I should include them.
The generated templates are the latest.

@ivanka2012 Thanks so much for your patch, but I think we should not include the generated souce code from the csp templates.
@rbugajewski

I thought about that, but if someone doesn't have drogon_ctl installed for the host, then the template files have to go from somewhere, in this case from the pregenerated files. We can rewrite the template files, but that would be a bit too much work for such a minor issue.
Or we can do it Canadian Cross style and build drogon_ctl twice, with 2 compiler toolchains.

Or we can do it Canadian Cross style and build drogon_ctl twice, with 2 compiler toolchains.

Could you elaborate on that? How would two toolchains solve the issue?

In this case, drogon is being cross compiled here. Currently, drogon_ctl requires to be ran to generate template files, but if it's cross compiled, then it can't run on the host.

Or we can do it Canadian Cross style and build drogon_ctl twice, with 2 compiler toolchains.

Could you elaborate on that? How would two toolchains solve the issue?

In this case, drogon is being cross compiled here. Currently, drogon_ctl requires to be ran to generate template files, but if it's cross compiled, then it can't run on the host. If drogon_ctl was also compiled for the host beforehand, then there's no problem.

Note that we only need the '_drogon_ctl'(a simple version of drogon_ctl) to generate template files. so we can force cmake always compile _drogon_ctl to host version.

Or we can do it Canadian Cross style and build drogon_ctl twice, with 2 compiler toolchains.

Could you elaborate on that? How would two toolchains solve the issue?

In this case, drogon is being cross compiled here. Currently, drogon_ctl requires to be ran to generate template files, but if it's cross compiled, then it can't run on the host. If drogon_ctl was also compiled for the host beforehand, then there's no problem.

Note that we only need the '_drogon_ctl'(a simple version of drogon_ctl) to generate template files. so we can force cmake always compile _drogon_ctl to host version.

That's an interesting thought. I've found this email that suggests that ExternalProject_Add could be used in this case, should I use that instead?

Or we can do it Canadian Cross style and build drogon_ctl twice, with 2 compiler toolchains.

Could you elaborate on that? How would two toolchains solve the issue?

In this case, drogon is being cross compiled here. Currently, drogon_ctl requires to be ran to generate template files, but if it's cross compiled, then it can't run on the host. If drogon_ctl was also compiled for the host beforehand, then there's no problem.

Note that we only need the '_drogon_ctl'(a simple version of drogon_ctl) to generate template files. so we can force cmake always compile _drogon_ctl to host version.

That's an interesting thought. I've found this email that suggests that ExternalProject_Add could be used in this case, should I try that instead?

Actually we don't need deploy the drogon_ctl to the target system, so this problem can be resolved by disabling the BUILD_CTL and the BUILD_EXAMPLES options.

@an-tao I think _drogon_ctl is the way to go then.

So can we merge this PR? @ivanka2012 @rbugajewski

So can we merge this PR? @ivanka2012 @rbugajewski

Yeah, it should be OK to merge


Serious issues when generating speeches for German language

Several issues appear when generating speeches for the German language:

In German texts, numbers and dates will ALWAYS be spoken in English
In German texts, English names and English words, that actually should be spoken in English and pronounced that way, were spoken in German, which sounds weird
when too many non-german words occur within a sentence, like French names, the speaker turns into English dialect
the same happens the longer a text is

It would be necessary to provide the target language as a specific setting to "tell" the models which language to be expected. Auto classification of the underlying language doesn't work at the moment and probably can't work in the long term thanks to the integration of English words and names naturally in other native languages (anglicisms).

@itinance Elevenlabs now supports specifying target language, please reopen this issue if that no longer works for your usecase!


Support for custom form xsl

I am working to uplift cht-core to use the latest Enketo libraries and we would like to replace our "roll-your-own" transform logic with enketo-transform.  However, we have some customizations in our form xml that are not compatible with the openrosa2html5form.xsl that is used by the transformer.  There does not seem to be a "recommended" way to use the transformer with a custom form xsl (maybe there is and I have just missed it somewhere?).
Would you be open to a PR to add support for custom xsl instead of just hard-coding it in the transformer?

To give a little more details about our particular use case, we have a number of custom xml_type values that we need to support.  In the current implementation of the openrosa2html5form.xsl file, these custom types will result in an error because of this code:
            <xsl:otherwise>
                <xsl:value-of select="$error"/>
                <xsl:message terminate="no">ERROR: Unsupported data type '<xsl:value-of select="$xml_type"/>' found.</xsl:message>
            </xsl:otherwise>

Our current custom xsl replaces this section with this code:
      <xsl:when test="$xml_type = 'tel'">tel</xsl:when>
      <xsl:otherwise>text</xsl:otherwise>

cc @garethbowen

Hi @jkuester thanks for filing this! I discussed this with @lognaturel and we think a good solution to this might be to parameterize it in the transform call, e.g. something like:
const survey = await transform({
    xform,
    custom: {
        dataTypes: {
            tel: {
                // Ideally you'd map the model data type to an existing XForm type
                model: 'string',
                view: 'tel',
            },
        },
    },
});

This would deviate a bit from the current behavior where the view type is mapped directly from the model type (i.e. string here would typically map to text), but there's no reason that has to be set in stone. This would also be a good opportunity to extract this particular behavior out of the XSLT, which will make it easier to reason about/maintain, and may improve performance a bit for larger forms[^1].
[^1]: The type mappings, and the binding-attributes template generally, are a surprisingly expensive part of the transform.

To be explicit, after the latest round of work on Transformer, we very much want to treat the fact that it uses XSL as an implementation detail so that it can be replaced over time.

Thanks @eyelidlessness and @lognaturel for the updates!  Just wanted to note here that I believe your proposed solution would work for our use cases!  The tricky part was that we have been using <xsl:otherwise>text</xsl:otherwise> to basically support an indeterminate amount of custom types in forms.  However, that functionality is now considered deprecated and we have replaced it with a separate approach that only uses string types.  We should be able to use this proposed approach to configure any required determinate custom types and then drop support for the indeterminate ones....


Crash on start up with other addon: Rocket Log

Describe the Bug
I had a running and working save with quick item search. I added another mod called Rocket Log (https://mods.factorio.com/mod/rocket-log) and after that Quick Item Search started to give these errors at startup (directly at load of save)
To Reproduce

Install Quick Item Search
Install Rockert Log
Launch the game

Save file & Username
Username: bykoow
File: https://www.swisstransfer.com/d/adb42c82-f482-4867-a3d1-afd7287a9ceb
Log file
factorio-current.log
Error
https://cdn.discordapp.com/attachments/708269007098085448/969717826963267615/unknown.png

Sourcehut ticket: https://todo.sr.ht/~raiguard/factorio-mods/11

Was fixed in 44002cc.


How to use it programmatically?

Hello,
Is there a way to trigger to recording audio from JS code without pressing button by user?
Thank you.

Hi, you can use refs to access the component and then call the exposed methods such as: start, stop, pause, resume

Hello,
I try .. but error
<VueRecordAudio ref="voicerecord" @result="onResult" :mode="recMode" />
this.$refs.voicerecord.start()
VM4033:37 [Vue warn]: Error in v-on handler: "TypeError: this.$refs.voicerecord.start is not a function"
but I use this and it works
<VueRecordAudio id="voicerecord" @result="onResult" :mode="recMode" />
document.getElementById('voicerecord').dispatchEvent(new Event('mouseup'))

Hi, this is really strange because the methods are defined, checkout this screenshot:
<vue-record-audio ref="Recorder" />

export default {
  mounted() {
    console.log(this.$refs.Recorder.start)
  }
}

Probably is something else in your code, If you copy n paste an extract of it, I try to help

Hello, ok. it is work now.
It does not work because I have multiple of  components.
So, when I make it to have only 1 component, start() and stop() now work!
Thank you so much

It does not work because I have multiple of components.
So, when I make it to have only 1 component, start() and stop() now work!

multiple vue-record components? if so, it sound like a bug to me, maybe I should check..
can you paste some of the code, i'm a little bit curious


Add broadcast syntax for dividing by a scalar

This adds the syntax A ./= c and B .= A ./ c.
Also, it adds a generic fallback error for broadcast operations that are not implemented yet, for example:
julia> i = Index(2)
(dim=2|id=484)

julia> A = randomITensor(i,i')
ITensor ord=2 (dim=2|id=484) (dim=2|id=484)'
ITensors.NDTensors.Dense{Float64,Array{Float64,1}}

julia> A .= 2 ./ A
ERROR: The broadcasting operation you are attempting is not yet implemented for ITensors, please raise an issue if you would like it to be supported.

The error message part is really helpful


[Resources] Template specs track 2 SDK

All SDK Contribution checklist:
We would like to onboard track 2 SDK for template specs to unblock our Bicep work https://github.com/Azure/bicep/issues/3162.
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

hi @shenglol as synced in teams channel, is it ok for you to directly release new Track2 SDK later? That will save lots of migration effort from customers.

Replaced by https://github.com/Azure/azure-sdk-for-net/pull/23799 and it will be released this week.


Made CSS compatible with Twitter bootstrap

Will be nice, if somebody (or author) adjust CSS so input box looks same way as Bootstrap (http://twitter.github.com/bootstrap) input boxes (same size, same look & feel, etc).

P.S. or maybe reuse chosen CSS from http://harvesthq.github.com/chosen some way?
I.e. they "Multiple Select" looks PERFECT :)

Theming is pretty easy, just need to do it. Thanks for the suggestion! I'll add either or both in the next release.

Sure, just make sure you also update your "Builder" (http://textextjs.com/download.html), so user can select which "theme" to use before he minimize to single js file :) Because today I spend some time to change styles directly in minified js file ;-), which is bad practice for sure :)

@evereq have you made any attempts at theming already?

ah, no I did only slight changes, see http://jsfiddle.net/g6Gxb/2/ :)
I use it for now as prototype, so will probably wait you or somebody to do it better than me :)
Btw - I think if you want to made plugin popular, make sure you add support of theming from jQuery UI... I stop usage of jQuery UI after Bootstrap out, however many developers still use it :)

I haven't looked at how jQuery theming works. Do you have any experience with it?

I beg you not to use jQueryUI or at least not make it mandatory as MANY people do not use jQueryUI

@alexgorbatchev - I have experience with jQuery UI themes only as "customer", i.e. just utilize existed CSS framework. I think in your case, it should be same way - just utilize some jQuery UI CSS framework (http://jqueryui.com/docs/Theming/API)  CSS classes and you will automatically get look & feel of jQuery UI and be compatible with themes switcher, etc (even button to open drop down may look different according to selected theme). Again, like @jasondavis it should be not "mandatory" but just an option. However such option can be very attractive for users as still a LOT of people actually do use jQuery UI :) (see for example small list of plugins which use it here http://jqueryui.com/docs/Theming/ThemeRollerReady) and work to made your library to utilize jQuery UI themes looks relatively simple for me :)


feat(evolutions): add links to the pokemon

(੭ु˙꒳˙)੭ु⁾⁾
fixes #95
idk if i was supposed to add the cursor: pointer or just add an empty href ¯\_(ツ)_/¯ if the href is preferred, i can change it.
also, idk if we want an active state or something.
another thing to note is the fact that i can still click on a selected pokemon and it will "reload". should we disable that or is it not a big deal? i dont really see any harm in it. what do you think?

lgtm! i don't think either of those is necessary (adding an active state or disabling the pokemon you're already on). maybe a hover state, but since there's the cursor, i think it's good enough.


Can keras-maskrcnn training/inference work without the mask image annotation?

I am interested in running the keras-maskrcnn with keras-retinanet mainly to detect bounding box. papers reported that retinanet with roialign get better MAP (~1-2% performance improvement).
In the readme, for the csv based training, it needs the mask image for the annotated object, can this part be optional? then it will be exactly what I mentioned.
Thanks and best regards,
Liming

Retinanet is a single-stage object detector, introducing ROI Align will make it a two-stage object detector. Basically, that is what FasterRCNN does. Which paper reports that Retinanet with ROIAlign gets better mAP?
If you're only interested in bounding boxes, check keras-retinanet.

Retinanet is a single-stage object detector, introducing ROI Align will make it a two-stage object detector. Basically, that is what FasterRCNN does. Which paper reports that Retinanet with ROIAlign gets better mAP?
If you're only interested in bounding boxes, check keras-retinanet.
This blog post reported:
The ROI align is reported to have ~3 points improvement in AP in trainval35k.
https://medium.com/@steve101777/roi-pooling-and-roi-align-dd13bfece1df

Thanks a lot.
Liming


CCL & DCL are not aggregated

Hi!
I have 2 Systems running, and on both (both 3xMultiplus5000, one system with 2 Basengreen batteries, one with 3 Basengreen batteries) CCL & DCL are not aggregated? Any idea?

dbus-spy

Aggregator:

1st battery:

Check Io/AllowToCharge on each battery, if it is zero/off then its CCL is ignored (it should probably be taken to be zero, but I can fix that if this is the case).
Also, check the log
cat /var/log/BatteryAggregator/current

Can this be changed?
here´s the log:

If we are talking about a battery then the logic in dbus-serialbattery is
    self._dbusservice["/Io/AllowToCharge"] = (
        1
        if self.battery.charge_fet
        and self.battery.control_allow_charge
        and self.block_because_disconnect is False
        else 0
    )

It looks like these variables are not supported by lifepower bms hence it is 0. I might try to contact dbus-serialbattery author to try to understand why this is the case.

Has the latest version helped?

Hi! Yes! It works great! I thank you very very much!!! Robert
-------- Ursprüngliche Nachricht --------Von: pulquero @.> Datum: 14.03.24  17:31  (GMT+01:00) An: pulquero/BatteryAggregator @.> Cc: johnboyi @.>, Author @.> Betreff: Re: [pulquero/BatteryAggregator] CCL & DCL are not aggregated (Issue
#25)
Has the latest version helped?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Now it works! Thank you!


Call and re-render only if value changes, not on any set

Is this not the default? I may actually just be doing something wrong so I wanted to check. If I set an observed value, but it's the same value (eg: obj.x = 1; obj.x = 1), will it render twice?
Sorry to spam here. My google search didn't turn anything clear up, so I wanted to make a question for poor readers like myself.

No. But you can simply test these kind of tests by trying?
Op do 19 apr. 2018 01:24 schreef Nate Wienert<EMAIL_ADDRESS>
Is this not the default? I may actually just be doing something wrong so I
wanted to check. If I set an observed value, but it's the same value (eg: obj.x
= 1; obj.x = 1), will it render twice?
Sorry to spam here. My google search didn't turn anything clear up, so I
wanted to make a question for poor readers like myself.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/458, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhLa87ARoOxLnQ7Y_fDQgXTzMBZ07ks5tp8sxgaJpZM4Ta68B
.

Ah thought I had seen this happening but didn't investigate enough... apologies.


api key

Is there a way to enter in our  itunes/apple api key?
Thanks

iTunes Search API do not required API key to use it. Let me know if that helps, thanks!

I understand that no API key is needed but
If we wanted to add our API key where in the code could we add it ?
Thanks so much for your help

On May 2, 2016 at 10:29:13 PM, Leo Hsieh<EMAIL_ADDRESS>wrote:
iTunes Search API do not required API key to use it. Let me know if that helps, thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I see, for this one it was not designed for OAuth required API at first place, so it might not be easy to use this codebase to extend for that. You need to see how your API integrated with its keys and it may vary based on different API structure.
I had another app used Facebook API with app id, maybe that one can help you, thanks.

Ok… I see
Can you let me know in which file(s) I can find the url string that is passed to iTunes ?
Thanks !!!

On May 3, 2016 at 8:50:09 AM, Leo Hsieh<EMAIL_ADDRESS>wrote:
I see, for this one it was not designed for OAuth required API at first place, so it might not be easy to use this codebase to extend for that. You need to see how your API integrated with its keys and it may vary based on different API structure.
I had another app used Facebook API with app id, maybe that one can help you, thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

It is in the first table of iTunes Search API references.

Thanks !!!

On May 3, 2016 at 9:08:26 AM, Leo Hsieh<EMAIL_ADDRESS>wrote:
It is in the first table of iTunes Search API references.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

No problem!


Manually generate auto-generated Terraform code when using terraform.tfvars module source.

Recently, I needed to help a Gruntwork customer debug an issue with why a Terraform template could no longer apply. They're using Terragrunt and shared with me the following output:
aws_iam_policy.manage_ebs_volume: Refreshing state... (ID: arn:aws:iam::111111111111:policy/jenkins-manage-ebs-volume-vol-0f9999999999999)
aws_iam_policy_attachment.attach_manage_ebs_volume: Refreshing state... (ID: attach-manage-ebs-volume)
Error refreshing state: 1 error(s) occurred:

* aws_lambda_function.scheduled_lambda_job_not_in_vpc: RequestError: send request failed
caused by: Get https://lambda.us-west-2.amazonaws.com/2015-03-31/functions/jenkins-backup: dial tcp: lookup lambda.us-west-2.amazonaws.com on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:58188-><IP_ADDRESS>:53: i/o timeout

It was a confusing error and we couldn't resolve it prima facie, so the next step is to keep simplifying the setup until we can explain why a simpler failure occurred. That, in turn, required that we use a native Terraform configuration that mimics the automated functionality of terragrunt apply in combination with a terraform.tfvars that copies a module to a temp directory.
But explaining how such a template should be structued to the client was still non-trivial, so it'd be nice if we had a terragrunt generate command (or something better named) that automatically outputs the Terraform code that Terragrunt internally generates for debugging situations like this one.

I'm not sure I fully understand what the generate command would do?
BTW, is there a reason you can't do the local testing using the --terragrunt-source parameter?

Is there a reason you can't do the local testing using the --terragrunt-source parameter?

That mostly solves the problem, but for a true reproduction, it'd be nice if Terragrunt could "generate" the native Terraform equivalent of using terragrunt.terraform.source in terraform.tfvars.

Isn't that exactly what it does when you run, say, terragrunt plan? The contents of the tmp folder are that output...

Ah, I didn't make the connection that you could access that tmp folder even after terragrunt plan is run, and I suppose it's in the logs, too. Do you know if tmp files get deleted immediately after the terragrunt run finishes? If not, manually accessing the tmp folder solves the problem.

They do not get deleted, so you should be able to cd into that folder and use it as you wish.

Great, closing this issue. Thanks.


According to https://go.dev/wiki/MinimumRequirements, Golang supports kernel version 2.6.32 or later, but os.checkPidfd() unconditionally calls pidfd_open(2), which was introduced in 5.3.
os.checkPidfd() should check availability without calling potentially unavailable system calls. Alternatively, allow to disable the use of pidfd by GODEBUG.
Related: #62654
CC @kolyshkin

Related Issues and Documentation

cmd/compile: SIGILL when building golang.org/x/sys/unix on openbsd/arm64 #42517
affected/package: exec linux bad system call using Termux (Android) #60125 (closed)
x/sys/cpu: Crash on android arm64 device in release build #36622 (closed)
runtime: unexpected signal on Termux android/arm64 #29623
cmd/go: get in docker buildx gets SIGSEGV error #40969 (closed)
runtime: panic 'exitsyscall: syscall frame is no longer valid' in go tool #24656 (closed)
runtime: fatal error: checkdead: runnable g #40368 (closed)
cmd/go/internal/list: segmentation fault on darwin #67113 (closed)
os,syscall: double close pidfd if caller uses pidfd updated by os.StartProcess  #68984
cmd/compile: when cmd/compile is compiled with `-gcflags = "all=-N -l"`, the output binary panic on start #54412 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I find it weird that this crashes the process with SYGSYS. The way pidfd_open is implemented in linux it should catch the errno (ENOSYS):
https://github.com/golang/go/blob/6885bad7dd86880be6929c02085e5c7a67ff2887/src/internal/syscall/unix/pidfd_linux.go#L18-L21
The stacktrace shows it is crashing in runtime_entersyscall. Perhaps there is a seccomp(2) filter in place causing the process to receive a SIGSYS?

Yes, it would be extremely unfortunate if every unrecognized system call triggered a SIGSYS signal.  That would make it impossible to write programs that run on both older and newer kernel versions.  We need to understand what is causing that SIGSYS kernel.  I don't think it is the kernel.
That said, I see that this is android.  We may need to skip the pidfd calls on Android.  CC @golang/android

Additionally, would be good to see a strace output to aid with debugging, i.e strace go env.

I realized the true culpit is seccomp.
$ grep Seccomp: /proc/self/status
Seccomp:        2
$ strace -fqq --signal=SIGSYS --trace=none go env
[pid 12717] --- SIGSYS {si_signo=SIGSYS, si_code=SYS_SECCOMP, si_call_addr=0x5a46509700, si_syscall=__NR_pidfd_open, si_arch=AUDIT_ARCH_AARCH64} ---

So we should check if seccomp is enabled?

Checking if seccomp is enabled won't really help us, because we won't know the policy.
I think we should just skip the pidfd calls if GOOS == "android".

I think the safest approach is to disable pidfd on Android.

We can probably just make android use pidfd_other.go https://github.com/golang/go/blob/master/src/os/pidfd_other.go#L5

Alternatively, check kernel version?
https://github.com/golang/go/blob/master/src/internal/syscall/unix/kernel_version_linux.go

It seems to me that the kernel version isn't going to tell us anything about the seccomp policy.

Not to check the seccomp policy, but to check if the kernel version  supports pidfd.
Since Android with newer kernel would not have the problem, I don't think disabling pidfd for GOOS=android is a good idea.

Is there reason to believe that the seccomp policy matches the kernel version?
In normal Linux use we don't have to check the kernel version, because the system call with fail with an ENOSYS error.  In this case the above discussion suggests that it is the Android seccomp policy that is sending the SIGSYS signal.  But the kernel might have been updated without updating the seccomp policy.  Is there a way that we can find out?

https://github.com/termux/termux-packages/issues/21265
Users reported go works fine on newer Android

Thanks.  I still don't know how to tell whether pidfd_open is supported or not.  I'm certainly happy to accept a patch that has been tested on multiple versions of Android.

Any chance this was a bug with android? I found this https://github.com/aosp-mirror/platform_bionic/commit/3de19151e508e14654a2d3204d9981c514f1c93a but it points to a private issue.

https://android-review.googlesource.com/c/platform/bionic/+/1208625
https://cs.android.com/android/_/android/platform/bionic/+/refs/tags/android-11.0.0_r1:libc/SECCOMP_WHITELIST_COMMON.TXT;l=76
pidfd_open was added to seccomp allow list since Android 11
And
https://source.android.com/docs/core/architecture/kernel/generic-kernel-image#inhibits-platform-upgrades

Android 10 supports 3.18, 4.4, 4.9, 4.14, and 4.19 kernels

So, checking if the kernel version is 5.3 or newer before calling pidfd_open should work on all Android devices.

Oops, pidfd_send_signal was not allowed in Android 11, but it fixed in 12
Since Android 11 supports only 4.19 and 5.4 kernels (https://source.android.com/docs/core/architecture/kernel/android-common), change to check against 5.5 rather than 5.3

So, it's not the Android kernel but its seccomp policy which results in a process being killed (instead of something like returning ENOSYS). Apparently, this was fixed in Android 12, we can add a kludge to do a one-time runtime check for Android >= 12 and disable pidfd entirely if this requirement is not met, just to avoid being killed.
Alas I can't find any code that checks Android version in this repository.

This also means that there's no CI in place to test Android 11, or this would have been caught earlier.

You're right, we should check Android version >= 12 rather than kernel version.
Here is the way to get Android version in C (sorry I'm not familiar with Cgo)
https://gist.github.com/cions/07fa5f11e38945fa96916888b7e88d0c

Thanks.  If we have to we can add a call to __system_property_get in runtime/cgo/gcc_android.c.
But honestly I think it would be simpler to just skip the pidfd calls on Android.  Any patch to use them should be written by an Android developer who is able to test on various Android releases.

@gopherbot Please open a backport to 1.23.
This issue causes Go programs to be unable to exec other programs on earlier versions of Android.  There is no workaround.


Get-DbaDbColumn and Get-DbaDbPrimaryKey

Summary of new feature
Being able to get column names from a table and more importantly the PrimaryKey(s) of a table.
Latest version of dbatools as of writing
1.0.153

Please provide the latest released version of dbatools module.
1.0.153

Using Get-DbaDbTable you can do this:
$table = Get-DbaDbTable -SqlInstance SRV1\SQL2017 -Database AdventureWorks -Table 'Person.Person'
$table.Columns | Format-Table -Property Name, InPrimaryKey, Datatype

What else would you like to have?

I also found a way to get the data needed in the end.
I was suggested to request this as a new command in the dbatools slack channel.
But I guess it's not that important if you can achieve your results with the current commands.
Thanks!


serialize_parameters and deserialize_parameters use inconsistent types for parameter values

Tech Debt
As discussed in https://github.com/ni/measurement-plugin-python/pull/931#discussion_r1775380078, serialize_parameters uses a Sequence[Any] for the parameter values and deserialize_parameters uses a Dict[int, Any] for the parameter values.
This is a result of how they were originally used on the server:

deserialize_parameters builds a dict of parameter values, indexed by field id, then it is converted to a dict indexed by variable name in order to use the **kwargs operator.
The service's measure() call returns a list/tuple of outputs, and serialize_parameters takes that list/tuple and serializes it.

In the measurement client, we end up doing a little bit of extra work to flatten the parameters in one case but not the other.
AB#2867307

https://github.com/ni/measurement-plugin-python/blob/main/packages/service/ni_measurement_plugin_sdk_service/measurement/client_support.py now defines wrapper functions that have a consistent signature.


Base currency fluid version

removed hard coded $ and replaced with printing PAIR_WITH name in places.
added detection for Binance supported fiat currencies and a changer for 2 decimal places for fiat currencies and 8 decimal places for crypto.
reworded buying / selling commands, a bit clearer.
changed MAX_COINS to TRADE_SLOTS to help newbies understand a bit easier.
added VERBOSE_MODE in config to allow users to easily toggle whether to see bot actions in more detail (to be adjusted later to include more functions).
fixed the buy/sell fee's - these are now independent and calculated via price of buy and sell
fixed returned profit amounts

I think its too invasive on current codebase. You can continue your version in your fork.

I am closing this now, as the code is drifting more and more away from main. Feel free to continue the good work in your own fork.
Thanks!

absolutely understood, coolio! 👍


Give schools some attributes

Per https://github.com/leereilly/swot/pull/37#issuecomment-29682530, I think it would be helpful to include a few pieces of information (if possible) about each school:

Name
Homepage
Level (high-school/community-college/etc.)

Is this hard to standardize internationally?

Accredited (true/false)

May be inferrable from level?

public/private
Location

How to handle schools w/ multiple campuses?

Having this information will broadly increase the applicability of this dataset, e.g. I'd love to collaborate w/ Code.org on merging their listing. /cc @brandonbloom @tobyk100
@leereilly is the reason for having the flat files (instead of a a big file) that you don't have to load all the information in memory?  If we want to stick with that structure, it might make sense to make each entry under https://github.com/leereilly/swot/tree/master/lib/domains its own little YAML/JSON file with the attributes above.

Maybe this structure will stick?https://github.com/kiler129/SwotNG-database/blob/master/STRUCTURE.md ;)


⚠ AtmoVantage Appointment Booking has degraded performance

In 7d175da, AtmoVantage Appointment Booking (https://booking.atmovantage.com/#/customer/schedule) experienced degraded performance:

HTTP code: 500
Response time: 10494 ms

Resolved: AtmoVantage Appointment Booking performance has improved in 546a8d9.


WEB-4049: Adding support for images with more than one . in their name

This was causing the slide generator to fail with @1.5x.png files

Gonna go ahead and merge this without review.


"powershell.exe" renamed to "pwsh.exe"

The windows-2019 image no longer has "powershell.exe": https://github.com/actions/runner-images/issues/7516

Looking at your job logs it looks like you replace the PATH env var, maybe that's the cause?
We could try to get rid of powershell all together though, as there is no reason we couldn't do the same task with plain javascript.

you replace the PATH env var

You're right! Fixed by appending new paths to $GITHUB_PATH instead of trying to use $GITHUB_ENV.
Thank you!


Feature Request: OLED

We've been working with the FlashTool and it seems to work really well.
Something that I think would be really nice to add is a some text on the OLED that would give feedback to students.

Display the xChips Wifi address while the chip is being configured and then  "Connection Established"
When we were trying to connect a bunch of chips at the same time we had to do them one at a time so that the wifi addresses didn't get mixed up. This took a long time.

Name/index name when it's sending data for easy identification.

And any debugging messages would also be excellent to have.

Hi @EBUJOLD we currently adding to a new revision and improving the stability of the current systems watch this space for a new release.

Thank you for the quick response.
We thought these would be useful new features for teachers.
What is the best place to request features and bugs for future reference?

@EBUJOLD I set up this website for ideas. I never tried it, but I thought your idea was great, so lets give it a try: https://xinabox.ideascale.com
Once you guys and we have played with it for a while, we will let everybody else know about it.

@EBUJOLD New release available: https://github.com/xinabox/FlashTool/releases/tag/v1.1-beta.3 please check out the OLED information.


Proposal for multi-variant testing on Voice and Tone

To determine voice and tone of our brand, and what guidelines to include in the design system - we should conduct user tests to determine the tone of writing that resonates best with users and best promotes comprehension and prevents errors.
The best way to do this would be multi-variant testing writing similar content but using different tones, and evaluating user's reactions, comprehension and feedback on each variation.

Proposal done and is found here: https://gcdocsp.tbssct.local/gcdocs/llisapi.dll?func=ll&objaction=overview&objid=33576843


Occasional graphical artifacts and/or blank screen only on "google.com"

OS/Platform
Windows
Installed
https://ungoogled-software.github.io/ungoogled-chromium-binaries/
Version
Version 104.0.5112.102 (Official Build, ungoogled-chromium)
Tested upstream?

[ ] I have tried reproducing this issue in Chrome and it could not be reproduced there
[ ] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there

Description
Occasionally, searching on Google results in graphical artifacts and/or blank screen displayed instead of the actual content.
How to Reproduce?
I can't reproduce this at will. I use Google as my default search engine. Normally, when I search for something using the address bar, the Google website loads up and displays the content with no problems. However, from time to time, the search results come up either unreadable and crumbled with graphical artifacts, or the content is almost completely blank.
Actual behaviour
This is what the site looks like when the problem happens.

Expected behaviour
The Google website with the search results should be displayed normally.
Relevant log output
No response
Additional context
I've encountered this problem on at least two completely different devices. The only similarity is that they run Windows 10. Otherwise, the hardware has nothing in common — one is a tablet with Intel CPU and GPU, the other a desktop with AMD CPU and GPU, which is why I don't really suspect a GPU failure here.
I believe this started sometime after Ungoogled Chromium v100, so it wasn't that long ago. In the past, I  don't remember having this kind of a problem. I'm curious whether I'm the only one to experience the issue. Also, what's peculiar is that it seems to only affect "google.com" and no other websites.

Have you tried the same in a new empty profile?

It's difficult to reproduce at will, unfortunately. It just seems to happen randomly, so even if I do create a clean profile and have no problems there, there's no guarantee that the culprit was something specific to the other profile, or rather I was just lucky.
The profiles on the two devices, where I've encountered the problem, are completely separate though.
I will try to do more testing nonetheless. I was just curious whether I was the only one affected by this, or maybe someone else's experienced this problem too.

I remember having a blank page very-very rarely on Linux at some point, don't know if it is related to your issue, but it was likewise totally random. For me it got gradually better. And I already do not remember when was the last time that I got an empty screen instead of a webpage. I never had artifacts in browser though. 🤷
I would suggest trying Chromium and see if you could reproduce it there.

@tomasz1986 I think by now you can guess what my question would be? :)

@tomasz1986 Maybe this is your issue or at least related to yours: https://bugs.chromium.org/p/chromium/issues/detail?id=1442633
Try the workaround from there.

Thank you for the investigation! The issue and the screenshots do indeed look very similar to what I've been experiencing. I must say though that it's been happening very rarely in the recent times.
I'm now going to try running the browser with the --disable-gpu-driver-bug-workarounds flag and see whether there is any difference.


Provide comibnators using uniform

It exists for backward compatibility, and using uniform we get a much more useful api.
For example, it can automatically figure out uniform tuples and it expects the underlying monad to carry around the seed.
I ran into this issue today. Now I just worked around it by running the random function by hand twice and passing the stdgen by hand around.
Checkout the docs, they recommend using uniform as well:
https://hackage.haskell.org/package/random-1.2.0/docs/System-Random.html#t:Random

Yeah, I have been wanting to move over to Uniform. I wouldn't be opposed to a PR support both the Uniform/Random variants for a short term (using CPP).

Now I just worked around it by running the random function by hand twice and passing the stdgen by hand around.

Note that you can currently use setNonDeterministic function now to get a similar behavior: https://hackage.haskell.org/package/fakedata-0.7.0/docs/Faker.html#v:setNonDeterministic

In this case it's really about the type level machinery I'm interested in, not so much the non-deterministic behavior (in fact it was already set).

Yeah, I have been wanting to move over to Uniform. I wouldn't be opposed to a PR support both the Uniform/Random variants for a short term (using CPP).

I was thinking of keeping the old functions in place but adding other ones that use Uniform instead. Allowing users to choose whatever type class they like.
After all, random isn't deprecated or anything

@jappeace I looked into this today and it seems I need to only introduce two more function: https://hackage.haskell.org/package/fakedata-0.7.0/docs/Faker-Combinators.html
I was surprised that only two functions requires the Random constraint. I'm just planning to add the suffix Uniform to those functions. What do you think ? Any other better names you have in mind ?

Adding fromRangeUnfiform and pickAnyUniform would be a great help indeed. An excellent solution!

I tried implementing this and got stuck because of dependency issues as it seems the ecosystem hasn't yet adopted to random-1.2.0 yet. I think I would wait till this issue get's closed: https://github.com/commercialhaskell/stackage/issues/5474


SDK does not handle transcription packets / they never come through OnDataPacket

SDK is missing support for transcription packets in handleDataPacket

I created PR for handling transcription event at https://github.com/livekit/server-sdk-go/pull/571


Supporting new Litecoin addresses with M prefix

New Litecoin addresses can have 'M' prefix, but the validator returns false for these addresses. For example MEeHd1yYTgZUjC1sAL1X1BvUK15rqsUDbi is a valid Litecoin address.

Sorry, it seems to be fixed in the recent versions.


Ontology Legend

The legend at https://ternaustralia.github.io/ontology_tern/ lists the same acronym for ‘Data Properties’ and ‘Annotation Properties’, differentiated only by colour of font of acronym.
Gail Wittich - Gaia Resources

I can't see what you are referring to here.

The issue is that the superscript character indicator for both "Data Properties" and "Annotation Properties" is "dp". The second should be "ap".
This is a pyLODE issue which I have reported here:

https://github.com/RDFLib/pyLODE/issues/156

Also, this particular ontology is really documented at https://linkeddata-dev.tern.org.au/tern-ontology, and one presumes eventually, https://linkeddata.tern.org.au/tern-ontology, rather than https://ternaustralia.github.io/ontology_tern/. This point is itself the subject of Issue #41, also raised by @gwittich
(@gwittich: you don't need to add your details to Issues as you post them, your GitHub account will tell us that!)

Closing as it is related to pyLODE and not the information model.


REST API Auth for event based retention

Document details the use of basic auth for REST API. Mentions "complianceuser" and "compliancepassword" with no insight into where these fields are set. Are we using azure AD? What permissions need to be assigned? What about MFA?

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

ID: 1ad33cc1-dc5f-d9f9-ee8e-b103839945ef
Version Independent ID: 1bc3101d-afe8-2056-3cda-6a86a209c460
Content: Start retention when an event occurs - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/event-driven-retention.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

Hi @stevensarginson
The authentication fields are available when you select "Show advanced options" in the connector and are the equivalent to signing into Security & Compliance PowerShell from a PowerShell session. If you have permissions to create events in the compliance portal, that's all the permissions you need but if you need a specific reference: https://docs.microsoft.com/en-us/microsoft-365/compliance/get-started-with-records-management#permissions
MFA isn't supported, and better security is one of the reasons why we're switching to Graph API.
If you haven't yet configured this with the REST APIs, I recommend you don't but wait for the Graph APIs, as documented in the Note now added to the top of this section. In the meantime, consider using the PowerShell cmdlets if you need automation?

@stevensarginson Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Hello, When I use the 'Debug' function, it encounters "typeerror: runfile( got an unexpected keyword argument current__namespace".So how should I deal with this issue?

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
What steps reproduce the problem?

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version:
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Thanks a lot.In fact,I have installed the "Spyder_64bit_full.exe"(5.3.0).But,I cannot find the corresponding Start Icon. How sholud I start  Spyder?

Thank you very much. I have open  Spyder.But how can I configure the GPU version of tensorflow for Spyder5.3.0?

Please see our documentation about that:
https://docs.spyder-ide.org/5/faq.html#using-packages-installer

Ok,thank you very much.


AttributeError in Chroma: 'LocalAPI' object has no attribute 'get_or_create_collection'

While using the LangChain package to create a Vectorstore index, I encountered an AttributeError in the chroma.py file. The traceback indicates that the 'LocalAPI' object does not have the attribute 'get_or_create_collection'. This issue occurred when trying to create an index using the following code:
index = VectorstoreIndexCreator().from_loaders([loader])
Here's the full traceback:
Traceback (most recent call last):
File "main.py", line 29, in <module>
   index = VectorstoreIndexCreator().from_loaders([loader])
 File "/home/runner/VioletWarpedDeclarations/venv/lib/python3.10/site-packages/langchain/indexes/vectorstore.py", line 71, in from_loaders
   vectorstore = self.vectorstore_cls.from_documents(
 File "/home/runner/VioletWarpedDeclarations/venv/lib/python3.10/site-packages/langchain/vectorstores/chroma.py", line 268, in from_documents
   return cls.from_texts(
 File "/home/runner/VioletWarpedDeclarations/venv/lib/python3.10/site-packages/langchain/vectorstores/chroma.py", line 231, in from_texts
   chroma_collection = cls(
 File "/home/runner/VioletWarpedDeclarations/venv/lib/python3.10/site-packages/langchain/vectorstores/chroma.py", line 81, in __init__
   self._collection = self._client.get_or_create_collection(
AttributeError: 'LocalAPI' object has no attribute 'get_or_create_collection'. Did you mean: 'create_collection'?

It seems like the 'LocalAPI' object is missing the 'get_or_create_collection' method or the method's name might have been changed in the recent updates.

I was facing this problem. Downgraded the package version to 0.3.15 and it worked. Some issue on latest version it seems!


Prove "getbalance" RPC method

What current issue(s) does this address, or what feature is it adding?
This PR is a result of a discussion with @Igoris on the #python channel in Discord where the functionality of the "getbalance" RPC method was questioned.
The original tests did not verify an expected balance was returned
How did you solve this problem?
Open wallets from WalletTestCase and verified that the RPC method returned the expected balances.
How did you make sure your solution works?
make test
Are there any special changes in the code that we should be aware of?
No
Please check the following, if applicable:

[x] Did you add any tests?
[x] Did you run make lint?
[x] Did you run make test?
[x] Are you making a PR to a feature branch or development rather than master?
[x] Did you add an entry to CHANGELOG.rst? (if not, please do)

Coverage increased (+0.01%) to 86.404% when pulling a5e5645b38137265286eb4382898f7367e096e14 on jseagrave21:prove-getbalance into 97e14255161be81aa59af797237b1d95c8911a57 on CityOfZion:development.

@jseagrave21 Good job 👍 I am Igoris by the way ;)

thanks!


OpenRefine Governance Model by Aayushi Chauhan

o	Name: Aayushi Chauhan
o	Project Assigned: OpenRefine (https://openrefine.org/)
o	Link to the project governance model: https://github.com/matrix-msu/Enslaved-OpenRefine/blob/e5ff1107b703ab6df3e1a970d871d79e08f8017c/GOVERNANCE.md
o	Governance Model:
OpenRefine is a powerful open source that focus on working on messy data, this involves cleaning the data, transforming it from one form to another, matching the data with external databases via reconciliation services, and fixing inconsistencies by merging the similar values in data. It also takes proper care of the privacy of the data; the processes take place in our own device and not on some cloud.
Project Roles:
a.	Users: These are the members who use the project, these users give the project a purpose.
b.	Contributors: These are the members who contribute to the project, either one time or multiple times.
c.	Committers: These are the members who have a deep understanding on the project strategy and have worked closely with users and contributors.
Decision Making Process:
OpenRefine uses the Apache Decision making to decide the future of the project. This involves:
a.	Lazy Consensus: This means that one does not need an explicit approval from users to proceed, silence is also treated as a consent. This is done to avoid unnecessary discussions but one needs to be prepared if someone objects.
b.	Consensus Building: This is done to get the feedback of the users before introducing any new feature. There are notations like:
•	+1: This means that I agree with the project and will help to make it happen.
•	+0: This means that I agree with the project but will not make it happen.
•	-0: This means that I do not agree with the project but I am offering no alternative.
•	-1: This means that I do not agree with the project but I am offering an alternative.
c.	Voting: In this, not just consensus plays a part but a proper voting takes place to get the numbers.
Decision making is done in a transparent, open manner. No decisions about the project’s bug fixes or features is done without community involvement and participation. Discussions must begin at the earliest possible point, the community’s participation is vital during the entire decision-making process.
The Governance Model of OpenRefine was easy to comprehend and while reading about it I got familiar with some decision making processes and how an open source software works, how open source contribution happens and much more. Overall, it was a wonderful and interesting task, any views are welcome.

@arliss-NF

Hello, @Aayushi-10. Your contribution is closed. I think you have to open it so it can be as visible as the rest of ours.
How do you do that? Scroll to the last message box in this thread and click on the visible reopen issue box. All the best.

You are welcome

@Aayushi-10 - so glad you enjoyed the first task.  Understanding how an open-source project works is very important to finding the best way to make contributions and to set expectations.


Attachments bug with previous apps

http://esrith-demo.maps.arcgis.com/apps/GeoForm/index.html?appid=f153c1d625e54199ac9c95614738f88a&webmap=f3f8de5fd91640a5b4d561e14affd7d4
 TypeError: Cannot read property 'enableAttachments' of undefined {stack: (...), message: "Cannot read property 'enableAttachments' of undefined"} "TypeError: Cannot read property 'enableAttachments' of undefined
    at declare._createForm (http://tpl.maps.arcgis.com/apps/GeoForm/js/main.js:639:97)
    at null.<anonymous> (http://tpl.maps.arcgis.com/apps/GeoForm/js/main.js:1638:28)
    at g.hitch (http://js.arcgis.com/3.13/init.js:177:296)
    at c (http://js.arcgis.com/3.13/init.js:76:221)
    at d (http://js.arcgis.com/3.13/init.js:76:10)
    at b.Deferred.resolve.callback (http://js.arcgis.com/3.13/init.js:77:350)
    at ia (http://js.arcgis.com/3.13/init.js:505:92)
    at Pa (http://js.arcgis.com/3.13/init.js:507:60)
    at oa (http://js.arcgis.com/3.13/init.js:506:397)
    at http://js.arcgis.com/3.13/init.js:515:482"

Here's one that also is not loading fields.
?appid=9e314aa25fd84314a7f9071912cac095

@CTDisasterManagement I can fix some with the above checked in. However, some old forms are not loading the fields they are configured with. Can you look into it?
Thanks

@driskull
These forms are not working because of the changes we have made for issue #210 in V4.6
Now we are reading for 'name' and 'alias' properties in configured fields whereas these old forms have 'fieldName' and 'fieldLabel' instead.
We can fix this specific problem by coding for 'name' and 'alias' in main.js but there are lot of config changes we have done since then which will get updated only when these forms are reconfigured via builder mode.
Thanks!

Ok. I dont think we need to do anything then.

Hello,
I have tested with the current version of the application & have found that if the feature service contains  attachments then Geoform will not load in the downloaded version. It will work as expected if created through the builder in ArcGIS Online.
Here is my sample: http://julia.esri.com/geoformbug/
As you can see, the application spins and throws the same error: TypeError: Cannot read property 'enableAttachments' of undefined "TypeError: Cannot read property 'enableAttachments' of undefined
Web Map in use: http://www.arcgis.com/sharing/rest/content/items/ae52c97213ff40a2b29cc7ca8a74c0cb/data?f=pjson
Basic hosted feature service with attachments. If the attachments are removed, then the application works as expected.
Created a hosted version of the application and copied the format of the defaults.js file and it still does not resolve the issue.
According to the help, you only need to update the attachment labels and help text; the help does not include information on how to configure the "attachmentInfo" parameter.
I work in technical support and will log a software bug for the issue there as well, but I wanted to check with you to see if there is another workaround for this user that wants to run the app from his local web server and use attachments...?
Thanks,
Julia

hey @jguard
Sorry about that.
I've pushed the fix for that into the master branch. We patched AGOL with that fix already. You should be able to grab it from the releases or master download code.
Let me know if that helps

hey @driskull
I tested with both the development branch and the master...still the same issue. Maybe I am doing something wrong. With that development branch configured the same way, I can open the application and see attachments but I can not create new ones...
Development branch:
http://julia.esri.com/geoformdevelopment/
Master branch:
http://julia.esri.com/geoformbug/

hey @jguard , It looks like you don't have the latest version of code from the master branch. main.js is different.

@driskull
Sorry, I sent you the wrong app. It should be: http://julia.esri.com/geoform/
Is the new version of the code available on the public site?
I copied and pasted the RAW main.js from the master branch into my app.
Just compared by hand the change in line 639 of main.js as documented here:
https://github.com/Esri/geoform-template-js/commit/1cfbfbb752b46e6a0f6c32a990aae27444279ebf
My code shows the new change:
if (this._formLayer.hasAttachments && this.config.attachmentInfo[this._formLayer.id] && this.config.attachmentInfo[this._formLayer.id].enableAttachments)
However, I am still unable to add new attachments.
I also tried downloading the master branch again and reconfiguring and I still can't get the attachments button to display

hey @jguard
The attachment option names changed.
Here is an example configuration of how it looks:
http://www.arcgis.com/sharing/rest/content/items/be338760de9249f8b15df22a8e4ee586/data?f=pjson
I'm updating the readme to put in the latest info.
"attachmentInfo": {
      "GeoForm_Sample_3731": {
        "enableAttachments": false,
        "attachmentIsRequired": false,
        "attachmentLabel": "",
        "attachmentHelpText": ""
      },
      "Test_secured_geoform_9625": {
        "enableAttachments": true,
        "attachmentIsRequired": false,
        "attachmentLabel": "",
        "attachmentHelpText": ""
      }
    },

Hi,
I downloaded the code on the first week of May 2017.
My feature service (in the web map) has attachments enabled but there is no option to add an attachment with the form.
the code looks like this (in defaults.js):
"attachmentInfo": {
"SeaUrchin_PublicReports_4978":{
"enableAttachments": true,
"attachmentIsRequired": false,
"attachmentLabel": "הוספת תמונה",
"attachmentHelpText": "הוספת תמונה"
}
},
this is my geoform: http://mapateva.org.il/Apps/SeaUrchins/index.html
any idea why it doesn't work?
Dikla.

Hi @deshegis , I'm not able to see the URL. Did it move somewhere else?

sorry, try again now - you will see it.

Hey @deshegis, I'm seeing a 404 error now for that URL.


Update assignments for 2018

[x] New body of assignment
[x] Update LG/SC

Completed updates on LG/SC for the assignments


Can't get a value from ConfigMap

I have a very simple sample that should retrieve a value from a ConfigMap but I can't get it working.
Code, pom and logs are on this gist: https://gist.github.com/lburgazzoli/c8cb1db8f57eeaeca57d8d5b53d61b1e
If I replace the home method with a direct call to a KubernetesClient, I'm able to get the desired value from the config map :
@RestController
@SpringBootApplication
public class MyApplication {
    private static final Logger LOGGER = LoggerFactory.getLogger(MyApplication.class);

    @Autowired
    public KubernetesClient client;

    @RequestMapping("/")
    String home() {
        return client.configMaps().withName("demo").get().getData().get("mydata");
    }

    public static void main(String[] args) throws Exception {
        SpringApplication.run(MyApplication.class, args);
    }
}

It looks like the ConfigMap PropertySource is not taken into account.

@lburgazzoli Are you on OpenShift by chance? Or vanilla Kubernetes? so by default only the default service account is attached to a pod. this service account may not have privileges to read from the kubernetes API.
If on open shift oadm policy add-role-to-user <role> <username>
see here https://docs.openshift.com/enterprise/3.2/admin_guide/manage_authorization_policy.html#viewing-cluster-policy
try adding 'edit' role to see if it works; then we can back it down from there

I'm on minikube and using the client with autowire in my app works as expected so it looks like something inhibits the PropertySource to load

@lburgazzoli what happens if you include this dependency?
<dependency>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-context</artifactId>
</dependency>

@christian-posta yep, that was the missing piece, thx.
So, should spring-cloud-context be declared non optional ?

yep @lburgazzoli looks like @iocanel fixed it on master. thanks for reporting this issue!