Implement a Kit that contains common utilities

Things like spread/rest/etc. For manipulating lists, objects, and all other common needs that tend to pop up when building graphs.

For now

[ ] Objects

[ ] Spread
[ ] Rest
[ ] Pick (may converge with Rest)
[ ] Omit

[ ] Array

[ ] push
[ ] pop
[ ] splice
[ ] concat

I'd like a merge for objects, equivalent to {...a, ...b}

@dglazkov @Mearman I wrote Kit.wrap exactly for this case (existing system APIs). Can we try and exercise it?
See: https://github.com/breadboard-ai/breadboard/blob/9c394a59c8d9fa174892d7da3b0c4efeeba970c4/packages/breadboard/src/kits/builder.ts#L109
Docs: https://github.com/breadboard-ai/breadboard/blob/9c394a59c8d9fa174892d7da3b0c4efeeba970c4/packages/breadboard/docs/kits/libraries-as-kits.md?plain=1#L36
Tests: https://github.com/breadboard-ai/breadboard/blob/9c394a59c8d9fa174892d7da3b0c4efeeba970c4/packages/breadboard/tests/kits.ts#L26

Ah I completely forgot about 'Wrap'. I shall have a play

Ok, some I'm having to reshape to have sensible input and output ports. @PaulKinlan is there a nice way to wrap functions one-by-one??

Yeah - you should be able to just do it.
const MathKit = KitBuilder.wrap({ url: "test" }, { random: Math.random });

Sorry I missed out, then including it alongside other nodes in an existing kit.
Some of them require a bit of tinkering.

Yeah - you should be able to just do it.
const MathKit = KitBuilder.wrap({ url: "test" }, { random: Math.random });

I also forgot about the .wrap!!!
would be very interesting to compare this to the node defining machinery that @aomarks  built.

This ^ and my indecision about what to include in code nodes vs board mechanics vs kits, as well what I was thinking about regarding the "board signatures" makes me wonder if there is a better way of defining "logical nodes"/boards

makes me wonder if there is a better way of defining "logical nodes"/boards

now you have to tell me more about that 😆

well initially it's spawned from wondering about passing a board that has a certain signature but I think it could extend to the underlying logic of the node. If you remember, ages ago I was wondering about ASTs for encoding board logic. Perhaps a BGL way of encoding an AST...?


Bug: Profile and logo image size in pixels are to large and in weird formats

On the live environment the logo of the company is very much stretched everywhere. Will need some extra investigation.

Maybe a potential fix is adding a rounded div around the image with size e.g. 56, and inside nestle a image with smaller size and remove the rounding

Think this is fixed in #281


Issue 154 add ticket types info for event showings

Brief Description
Please include a brief description of changes. Please also include relavent information gathered while making these changes, as well as the context behind the changes. Finally, list any dependencies that were changed or added.
References Issue
Issue #
Type of change

[ ] Bug fix
[ ] Feature
[ ] Major change (potentially breaking, list anything that is broken below)
[ ] Documentation

Breaking Changes
Briefly list things that were broken as a result of this PR.
Testing done
Describe tests that were used, along with instructions for testing.

[ ] Test1

[ ] Test2
...

[ ] Docker was used.

[ ] Docker was not used and I have listed my configuration below.

System Specifications:

Operating System:
Node Version:
NPM Version:

Checklist:

[ ] My code follows the styling guidelines.
[ ] I have added unit tests and verified that they pass.
[ ] I have used the linter and fixed any linting issues.
[ ] I have commented the code.
[ ] I have adjusted the documentation to match the changed code.
[ ] Prior to submitting the PR, I made sure the latest changes were merged with my branch.
[ ] Existing tests pass locally with changes.

Explanation for unchecked
Provide a brief explanation for why the checklist was not completed.
Additional information:
Provide any additional information that might be useful for this commit.

This can be reviewed and merge to include a fix for the put request on the event instances


Support for native upserts in PG

It would be really great to have an upsert command that uses native features of PG

Will do it

Fantastic :) Let's start with some really basic functionality for PG first and take it from there

https://github.com/rom-rb/rom-sql/pull/66

was added to rom-sql v0.8.0


If there are more than one c++ files having the same file name under different folders, setting break point (gdb backend) in one of them may fail.

VSCode Version: all
OS Version: ubuntu 16.04

Steps to Reproduce:

Attach to a running process (gdb backend).
Set break point by left click on the beginning of a code line.
The expected break point may be disabled (hallow red circle appears).
Another file with the same file name may be affected by this action and it seems that the break point I just set was applied to this file instead.

This is caused by the c++ extension. Can you please file an issue here
https://github.com/Microsoft/vscode-cpptools

Thanks, Isidor! New issue is created. https://github.com/Microsoft/vscode-cpptools/issues/692


 It builds and lints fine without this simp theorem.

!bench

oof, nice catch.
bors merge


How to use it with kogrid

What is the way to use it with kogrid. Can you provide a simple code example for this.  I have included the ko-grid-export.js and css in my page. On my gridOptions i am using the config: 'export-basic-config'. but it is not working. It would be very helpful for users if you can provide a working code for this.

To get started I suggest you take a look at one of the two simple, self contained examples in the examples project and then adjust to your needs.
To add export support to the provided sscces simply add the two extensions "toolbar" and "export" to the config. The relevant snippet should look something like this:
window.ko.bindingHandlers.grid.config['example-config'] = {
    extensions: {
        filtering: {},
        sorting: {},
        toolbar: {},
        export: {}
    }
};

Tip: Use ko-grid-bundle rather than manually including any extensions. All you need to get started are the ko-grid-bundle.js and the ko-grid-bundle.css. (If you're concerned with the size of the resulting JS/CSS you can still build your own reduced bundle once you're ready to deploy.)
Please let me know if there is anything more I can help you with.

I consider this issue closed. Feel free to reopen it, if you need further assistence.


Fixed layout  : menu scroll back to the top when expand sub-menu

I use a fixed layout, I have a lot of menus in my sidebar, so, some menus need to scroll to open them, when I expand the menu after scrolling down the scroll menu back up and I have to scroll down again to press the menu I want

This is an issue from AdminLTE, it will be fixed in v2.4.13.

@REJack What is the status of the update? :)

@lacodimizer I'll add later some little fixes to v2.4.13 and release it tomorrow.

Oh great! So I can merge it in the upcoming release.

@lacodimizer v2.4.13 ist auf GitHub released und auch auf npm/packagist verfügbar 😝
v2.4.13 is released on GitHub and available on npm/packagist too.

@REJack nice, it was merged :)

Thanks @REJack, @lacodimizer


[WIP] Create executable using pyinstaller and tests  on azure pipelines.

Description
During prepare of embedding napari in my application I would like to identify all possible problem connected with napari. So I apply test scheme and pyinstaller build scheme to napari.
From my projects I have bad experience with using pyinstaller on anaconda. Now I prefer to use CPython distribution.

This PR can boost #496 or replace it (napari related things are bring from this PR).
PR is to add information about problem connected with napari on azure pipelines (OpenGL on windows - I need help with this)
Introduce tox.

result of build can be seen here https://dev.azure.com/bokota/open source contrib/_build?definitionId=8&_a=summary
PyInstaller support of PySide2 is worst than PyQt5. I need to change backend on macos and windows, because PySide2 base build fails during time of execution. It looks like it need additional hooks for PySide2. (Here you see why I look for PyQt5 support)
Even If it will not be finally merged to napari please keep it until all problems are fixed:

[ ] OpenGl test on windows
[ ] Icons problem on linux.

Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

References

How has this been tested?

Download result of pipeline on each system and test if it launch.
Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

@sofroniewn can you look on this. Did you know who can help with OpenGl?

Thanks again for all the great PRs @Czaki,
I realize it doesn't look like there is much activity at #496, but I've actually been working on it a lot over the last few days in my branch: https://github.com/napari/napari/compare/master...tlambert03:pr/sofroniewn/496
I have it working with pyside2 in that branch, and I've been trying to come up with a strategy to allow people to extend it with plugins after bundling (see #1074).  It'd be nice if we could keep all of the pyinstaller activity on that thread rather than creating a new one.
are you proposing here that we add azure pipelines to our tests?  or is this just something you're working on on the side for your project?
Regarding OpenGL in virtualized environments... there is a lot to be said there.  I would recommend starting with https://github.com/napari/napari/issues/799#issuecomment-565859004, https://github.com/napari/napari/issues/896#issuecomment-577321885, https://github.com/napari/napari/issues/886, and perhaps this one at Vispy.  I can't claim I fully understand it... but after looking into it for a while it definitely seems like a hard problem due to.  I think  virtualGL might be a potential solution ... but it will take some research.

I realize it doesn't look like there is much activity at #496, but I've actually been working on it a lot over the last few days in my branch: https://github.com/napari/napari/compare/master...tlambert03:pr/sofroniewn/496
I have it working with pyside2 in that branch, and I've been trying to come up with a strategy to allow people to extend it with plugins after bundling (see #1074). It'd be nice if we could keep all of the pyinstaller activity on the #496 thread rather than creating a new one. (It's a major feature that we're all very excited to include, but there's a lot of moving parts)

I agree, but there are few reasons:

I'm not a maintainer of napari, so I only can create PR to PR or leave comment. First solution is weak for this moment because #496 need to be rebased. This PR have few important difference, which may be hard form me to explain in comments so I decide to create PR to show it.
This PR base on CPython, not Anaconda.
I switch to PyQt5 backend which is better supported by PyInstaller.
I force to create napari/resources/_qt_resources_*.py during bundling time (versioner create some problems)

But I will be happy if both cases. If this PR is merged or if strong part of this PR will be moved to #496.

are you proposing here that we add azure pipelines to our tests? or is this just something you're working on on the side for your project?

Hard question.
Running on  azure pipelines allow to use CPython, current solution base on Anaconda. It can make difference.
I would like to have at least information about deal with OpenGL in documentation.
So I try to solve OpenGL problem basing on your links.

@Czaki, now that we've closed #496 and #1289 is pretty much working, is it ok with you if we close this PR?

Briefcase looks promising. I do not understand why its ceeate installer on Windows.
But as i chcek pyinstaller is not compatible with napari plugin system. So close this PR.
If briefcase works well with napari I propably would ask few qestions on zulip because i still think if I could apply it to my software.

I do not understand why its create installer on Windows.

as opposed to something like a portable executable?  I assume they went with the .msi installer as the default bundler because it lets you integrate nicely into the start menu, and adds an uninstaller, etc... but I don't know much about the various windows install methods.  I will point out that the code is pretty nicely organized such that that if you wanted a different type of bundle, you can probably create a new template for it (it would go in this folder)

If briefcase works well with napari I propably would ask few qestions on zulip because i still think if I could apply it to my software.

sure,  happy to share what we've learned.  There's some good discussion in #1074 and #1289, and you can try the latest bundles as an example if you'd like.


refactor: update witnesses descriptions (~25/100)

Current dependencies on/for this PR:

main

PR #48   👈

This stack of pull requests is managed by Graphite.


eslint: fix semicolon and some duplicate key

some ESLint semicolon validations
some duplicate object key

Test plan: grunt test

There are IDE like Visual Studio Code that comes with ESLint by default and every time we open bootstrap folder/project we get this validation errors that mess up with other possible real problems :)
If we decide to ignore this problems, they should be added in .eslintignore file.

Update .eslintignore should be a good way to do this.
@bardiharborow any concern about that ?

Sorry, later edit to be more clear.
On our generated client side JS files, like js/dist/, there are semicolons. I'm not sure, but probably babel do that.
But semicolons are not in js/src, ESLint does not require that because of some specific settings in js/.eslintrc.json.

Guys, do you want me to make another PR with the no-new duplicate key?
And rebase this PR without that and leave it here for further discussions?

PR for this coming soon.

Closing for #22477.

Yes, but still https://github.com/twbs/bootstrap/pull/22388#issuecomment-292885868

Yep, I think maybe just moving the .eslintrc.json to the root of the project will fix that issue (from memory we were just trying avoid adding yet another file there).

Yes, no other files, they are hard to maintain.
I'll try to make a new PR with this, this days.


Handling nullable integers

With a filter of season=2022 where season is a nullable int and exception is thrown
System.InvalidOperationException: The binary operator Equal is not defined for the types 'System.Nullable`1[System.Int32]' and 'System.Int32'.

Hi, I will add a fix for this. Thanks for the feedback!


Fix LocalUTCOffsetTimezone string format for negative timezones

For all timezones to the West of UTC, the current code formats it wrong. For instance, I live in the EST timezone, and the generated offset string is currently --5:00 instead of the proper -05:00. The X-Client-Time header is thus always invalid, which is why #565 was happening to me.

hahaha, that the reason why it failed login for the oAuth?

Yup 😅 Ever since they started requiring X-Client-Time, fanbox never worked for anyone with negative timezones. I'm finally getting a refresh token for the first time ever, yay!


Relationships are not updated in a PATCH response

As the title says, really. For example:
GET /employees/
{
	"data": {
		"type": "employees",
		"id": "<id>",
		"attributes": {
			"first-name": "Scott",
			"last-name": "Jones",
			"full-name": "Scott Jones",
		},
		"relationships": {
			"permissions": {
				"data": []
			}
		},
		"links": {
			"self": "https://api.dev/employees/<id>"
		}
	}
}

PATCH /employees/
Request:
{
	"data": {
	"type": "employees",
	"id": "<id>",
		"attributes": {
			"first-name": "Jim",
			"last-name": "Brown"
		},
		"relationships": {
			"permissions": {
				"data": [
					{ "type": "permissions", "id": "practicegroup-employees" },
					{ "type": "permissions", "id": "practicegroup-practices" }
				]
			}
		}
	}
}

Response:
{
	"data": {
		"type": "employees",
		"id": "<id>",
		"attributes": {
			"first-name": "Jim",
			"last-name": "Brown",
			"full-name": "Jim Brown",
		},
		"relationships": {
			"permissions": {
				"data": []
			}
		},
		"links": {
			"self": "https://api.dev/employees/<id>"
		}
	}
}

GET /employees/
{
	"data": {
		"type": "employees",
		"id": "<id>",
		"attributes": {
			"first-name": "Jim",
			"last-name": "Brown",
			"full-name": "Jim Brown",
		},
		"relationships": {
			"permissions": {
				"data": [
					{ "type": "permissions", "id": "practicegroup-employees" },
					{ "type": "permissions", "id": "practicegroup-practices" }
				]
			}
		},
		"links": {
			"self": "https://api.dev/employees/<id>"
		}
	}
}

Why do the attributes come through updated, but the relationships do not?

This looks like an Eloquent caching issue. Are you doing $model->permissions at any point during the request cycle? That will cause them to be cached against the model as an empty relationship, so when they are encoded an empty relationship is being given back instead of the changed relationship.
If that is the case, ideally the hydrator needs to force the relationship cache to be cleared for the specific relationship it has just modified. But I'm not sure how to do that for a specific relationship off the top of my head, so we might have to do a full refresh of the model.
Can you let me know if that is the problem?

I'm not calling it for the model being updated as far as I can see. I am getting the permissions of the current user, however.

Hmmm... have looked into this. I can only reproduce if I force the model to load the relationship prior to the update occurring. The hydrator doesn't cause it to load the relationship as it always uses the relation method.
Saying that, it probably shouldn't be assuming that the relationship isn't already loaded, as it would be reasonable for it to have been used in some other context, e.g. an authorizer or validator.
So I think the way to solve this for the current release is for the Eloquent controller to refresh the model after committing changes to it.

@danherd
can you try the fix branch that I've pushed and let me know if that fixes the issue for you?
$ composer require "cloudcreativity/laravel-json-api:dev-hotfix/0.11.3"

Yeah this fixes it. Having looked a bit closer, I was actually getting the permissions of the employee being updated in the authoriser class (to check the current user is allowed to modify them).

Great, glad that fixes it.
I'll need to test that hotfix branch in the two production applications I have to check that it's definitely ok before I tag it, but I don't envisage any problems. It'd be good to make the refresh of the model more intelligent - i.e. it only refreshes if it's modified any of the relationships, but will have to do that in a future release.

Released v0.11.3


[docs]Update COMMUNITY.md with PowerOCR and PowerAccent

Summary of the Pull Request
Maintenance: Update Community.md with the PowerOCR and PowerAccent contributions.

I allowed myself to put the linkedin link.
An another thing, here the name is Keyboard Accent. Is it the new name of the tool in PowerAccent? (I thought it was ok for QuickAccent :) )

I allowed myself to put the linkedin link (may be it ask to update the PR).
An another thing, here the name is Keyboard Accent. Is it the new name of the tool in PowerAccent? (I thought it was ok for QuickAccent :) )

As a project, we are trying to merge away from "Power" everything.  Descriptive naming is easy to understand and when appropriate, easy to translate as well.  I showed a few people the new stuff and when i said "PowerAccent" before the demo, they thought it was some type of voice versus keyboard input.


Switch uptime to DEVICE_CLASS_TIMESTAMP

Hi,
can you please consider switching uptime to timestamp ?
All integrations are switching, as this will follow the core rules and will be a lot more performance that using seconds/ string.
Thx,
Simone

Hi,
can you please explain a bit further what you are talking about?
Thanks!

I'm asking to move those properties:

to 1 in timestamp format.
This way you won't update the DB each and every second but only on restart.
Simone

Thanks for the explaination. Do you have an example component that uses this method? Or a link to the developer docs?

Thanks for the explaination. Do you have an example component that uses this method? Or a link to the developer docs?

Here you are: https://github.com/home-assistant/core/blob/040075427040cd60bceb2aa0c589730e3c79e251/homeassistant/components/brother/sensor.py#L83
Simone

Thanks for the link!
This only makes sense if a different sensor is spawned for every attribute ("modelname", "external_ip", "external_ipv6","uptime","str_uptime").
For example: One sensor for uptime, one for status (connected or not), one for external_ip and one for external_ipv6. And additional sensors for the open PR  #64, #19
This would close #93.
What do you think @mammuth?
Could you implement it @chemelli74?

Yep, this issue is basically a duplicate of #93 which we just didn't implement so far. But we should probably really do so, since it's putting load onto the state machine and db.

Closed by latest release.


Feature/refresh org fix

For improvements to issue ga4gh/dockstore#210

Codecov Report

Merging #110 into develop will decrease coverage by 0.11%.
The diff coverage is 47.61%.

@@             Coverage Diff             @@
##           develop     #110      +/-   ##
===========================================
- Coverage    65.27%   65.15%   -0.12%
===========================================
  Files           93       93
  Lines         2508     2514       +6
  Branches       317      319       +2
===========================================
+ Hits          1637     1638       +1
- Misses         702      705       +3
- Partials       169      171       +2

Impacted Files
Coverage Δ

...ization/refresh-workflow-organization.component.ts
57.89% <44.44%> (-6.82%)
:arrow_down:

...rganization/refresh-tool-organization.component.ts
52.38% <50%> (-5.52%)
:arrow_down:

src/app/shared/alert/alert.component.ts
81.81% <50%> (-7.08%)
:arrow_down:

Continue to review full report at Codecov.

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


Redis instrumentation not safe for concurrent use

I have been working on some applications that utilize redis heavily. This week I attempted to instrument the redis commands in order to trace the time they take (for local use in order to not send developemnt profiling data to hosted profiling solutions). However, I ran in an issue that I traced back to redis.Client.
We create one global redis.UniversalClient and pass that around to all goroutines. This works and is safe for concurrent use. The issue arises when you call WrapProcess().  This function will change the internal state of the global redis.UniversalClient causing subsequent calls to keep wrapping over and over (and thus causing issues). When I noticed that WrapProcess() has been removed in favor of redis.Hook, I thought this issue would be solved. However, I found that the same issue exists, just in another form. I looked for examples of other projects using go-redis but all I found were simple examples where the client was only used once. I took a look at different dedicated logging solutions (ie New Relic, Datadog, etc) and since they all use WrapProcess() they all have this issue.
Observed behavior:
Using a global redis.UniversalClient is not safe for concurrent use due to the manipulation of internal state. In this case the (old) process function variable via WrapProcess() and (new) hooks array via AddHook.
Desired behavior:
One possible solution here could be to separate the responsibilities of the various tasks. Each client appears to create its own connection pool and there is no way set the pool or create a new  client from an existing pool. If the connection pools were pulled out into their own structure and no longer embedded inside each client, then there would be the ability to create a new client that would simply take the pool as a parameter (instead of an options struct). At this point, the connection pool, say redis.UniversalPool, would become set once globally and each goroutine could safely create its own client, say redis.Client(ctx, pool),  and alter it all it wants.
Example showing incorrect behavior:
I am including a code example for both v6.15.4 and the current master branch (d19aba07b47683ef19378c4a4d43959672b7cec8) that shows how redis.Client is not safe for concurrent use. If redis.UniversalClient or the instrumentation is being use incorrectly in this example, please let me know, since in its current form getting profiling data for redis across multiple goroutines is not possible via the provided methods.
package main

import (
	"context"
	"fmt"
	"time"

	"github.com/go-redis/redis"
)

const (
	handlerNameContextKey = "handlerName"
)

var (
	redisClient redis.UniversalClient
)

/*
// version = "6.15.4"
//
// What it prints:
//
// Setting WrapProcess from handler1
// [handler1] starting processing: <get handler1: >
// [handler1] finished processing: <get handler1: handler1>
// [handler1] value: handler1
// [handler1] starting processing: <set handler2 handler2: >
// [handler1] finished processing: <set handler2 handler2: OK>
// Setting WrapProcess from handler2
// [handler2] starting processing: <get handler2: >
// [handler1] starting processing: <get handler2: >
// [handler1] finished processing: <get handler2: handler2>
// [handler2] finished processing: <get handler2: handler2>
// [handler2] value: handler2
//
// What I expect (want to happen) instead:
//
// Setting WrapProcess from handler1
// [handler1] starting processing: <get handler1: >
// [handler1] finished processing: <get handler1: handler1>
// [handler1] value: handler1
// Setting WrapProcess from handler2
// [handler2] starting processing: <get handler2: >
// [handler2] finished processing: <get handler2: handler2>
// [handler2] value: handler2
//
func handler(handlerName string) {
	redisClient.Set(handlerName, handlerName, 0)

	// Instrument redis (using example code).
	// https://godoc.org/gopkg.in/go-redis/redis.v6
	redisClient.WrapProcess(func(old func(cmd redis.Cmder) error) func(cmd redis.Cmder) error {
		fmt.Println("Setting WrapProcess from " + handlerName)
		return func(cmd redis.Cmder) error {
			fmt.Printf("[%s] starting processing: <%s>\n", handlerName, cmd)
			err := old(cmd)
			fmt.Printf("[%s] finished processing: <%s>\n", handlerName, cmd)
			return err
		}
	})

	value, err := redisClient.Get(handlerName).Result()
	if err != nil {
		fmt.Printf("[%s] error: %s\n", handlerName, err)
	}
	fmt.Printf("[%s] value: %s\n", handlerName, value)
}

*/

func profile(name string) func() {
	start := time.Now().UTC()

	return func() {
		fmt.Printf("[%s] runtime: %v\n", name, time.Now().UTC().Sub(start))
	}
}

type profiler struct {
	context.Context

	timer func()
}

func (self *profiler) start() {
	self.timer = profile(self.Context.Value(handlerNameContextKey).(string))
}
func (self *profiler) stop() {
	self.timer()
}

func (self *profiler) BeforeProcess(ctx context.Context, cmd redis.Cmder) (context.Context, error) {
	self.start()
	return ctx, nil
}
func (self *profiler) AfterProcess(ctx context.Context, cmd redis.Cmder) error {
	self.stop()
	return nil
}
func (self *profiler) BeforeProcessPipeline(ctx context.Context, cmds []redis.Cmder) (context.Context, error) {
	self.start()
	return ctx, nil
}
func (self *profiler) AfterProcessPipeline(ctx context.Context, cmds []redis.Cmder) error {
	self.stop()
	return nil
}

// master@d19aba07b47683ef19378c4a4d43959672b7cec8
//
// What it prints:
//
// [handler1] starting
// [handler1] runtime: 0s
// [handler1] value: handler1
// [handler2] starting
// [handler1] runtime: 0s
// [handler1] runtime: 0s
// [handler2] runtime: 1.0263ms
// [handler2] value: handler2
//
// What I expect (want to happen) instead:
//
// [handler1] starting
// [handler1] runtime: 0s
// [handler1] value: handler1
// [handler2] starting
// [handler2] runtime: 1.0263ms
// [handler2] value: handler2
//
func handler(handlerName string) {
	fmt.Printf("[%s] starting\n", handlerName)

	ctx := context.WithValue(context.Background(), handlerNameContextKey, handlerName)

	redisClient.Set(handlerName, handlerName, 0)

	// Instrument redis (using example code).
	// https://godoc.org/github.com/go-redis/redis#ex-package--Instrumentation
	redisClient.AddHook(&profiler{
		Context: ctx,
	})

	value, err := redisClient.Get(handlerName).Result()
	if err != nil {
		fmt.Printf("[%s] error: %s\n", handlerName, err)
	}
	fmt.Printf("[%s] value: %s\n", handlerName, value)
}

func main() {
	// Create one client that has a connection pool and set as the global redis client.
	redisClient = redis.NewClient(&redis.Options{
		Addr:     "localhost:6379",
		Password: "", // no password set
		DB:       0,  // use default DB
	})

	// No goroutines used.
	// Demonstrates that the underlying client is indeed not safe for concurrent use.
	handler("handler1")
	handler("handler2")
}

Well, that is how it works. Normally you should add exactly 1 hook and pass request/task specific data via context.

I would really appreciate an example of instrumentation usage where there is a context passed during redis command invocation per request like you said. The only place I see to add a context is during creation of the hook. What other method is available to set a context outside of a hook? If the intention is that you are supposed to add a hook during each request (ie at the beginning in some middleware) then this that is exactly the issue I am seeing. Those hooks will be applied to all requests, not just the current one because the client is global and the hook alters the client's internal state.
Could you provide a working example that demonstrates how to properly use hooks across requests?

Something like
package main

import (
	"context"
	"fmt"

	"github.com/go-redis/redis/v7"
)

type redisHook struct{}

var _ redis.Hook = redisHook{}

func (redisHook) BeforeProcess(ctx context.Context, cmd redis.Cmder) (context.Context, error) {
	fmt.Printf("starting processing: <%s>\n", cmd)
	fmt.Println("context.foo is", ctx.Value("foo"))
	return ctx, nil
}

func (redisHook) AfterProcess(ctx context.Context, cmd redis.Cmder) error {
	fmt.Printf("finished processing: <%s>\n", cmd)
	return nil
}

func (redisHook) BeforeProcessPipeline(ctx context.Context, cmds []redis.Cmder) (context.Context, error) {
	fmt.Printf("pipeline starting processing: %v\n", cmds)
	return ctx, nil
}

func (redisHook) AfterProcessPipeline(ctx context.Context, cmds []redis.Cmder) error {
	fmt.Printf("pipeline finished processing: %v\n", cmds)
	return nil
}

func main() {
	rdb := redis.NewClient(&redis.Options{
		Addr: ":6379",
	})
	rdb.AddHook(redisHook{})

	ctx := context.Background()
	ctx1 := context.WithValue(ctx, "foo", "bar1")
	ctx2 := context.WithValue(ctx, "foo", "bar2")
	for _, ctx := range []context.Context{ctx1, ctx2} {
		err := rdb.WithContext(ctx).Ping().Err()
		if err != nil {
			panic(err)
		}
	}
}

The output is:
starting processing: <ping: >
context.foo is bar1
finished processing: <ping: PONG>
starting processing: <ping: >
context.foo is bar2
finished processing: <ping: PONG>

The only part I don't like is rdb.WithContext API to pass context, but it is not too bad and I hope that in v8 all commads will accept context.Context as first argument.

Thank you! WithContext() appears to be the critical missing piece when using Client as a global across goroutines since it appears that it solves the thread safety issue. I think it would be a good idea to update example_instrumentation_test.go to include this call.
My personal opinion, it may also be prudent to point out somewhere that obtaining a Client instance without calling WithContext on it first is not thread safe. The comment on Client that says It's safe for concurrent use by multiple goroutines. is misleading. I believe it is referring mainly to the underlying connection pool (which is thread safe), but makes developers think it means the actual Client struct usage, which is not thread safe without being careful.

@wspowell, +1

In v8 context.Context is accepted as first argument.


net/http: Last-Modified header did not correctly check the time

Go version
go version go1.22.2 windows/amd64
Output of go env in your module/workspace:
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\bddjr\AppData\Local\go-build
set GOENV=C:\Users\bddjr\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\bddjr\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\bddjr\go
set GOPRIVATE=
set GOPROXY=https://goproxy.cn
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLCHAIN=auto
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.22.2
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=0
set GOMOD=D:\bddjr\Desktop\code\golang\BCSPanel\go.mod
set GOWORK=D:\bddjr\Desktop\code\golang\BCSPanel\go.work
set CGO_CFLAGS=-O2 -g
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-O2 -g
set CGO_FFLAGS=-O2 -g
set CGO_LDFLAGS=-O2 -g
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -fno-caret-diagnostics -Qunused-arguments -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=C:\Users\bddjr\AppData\Local\Temp\go-build2942921968=/tmp/go-build -gno-record-gcc-switches

What did you do?
When I mount a new file and then re mount it as an old file, the server still returns 304, causing the client to not update.
What did you see happen?
304 Not Modified

What did you expect to see?
200 OK
Just change this place to == :
https://github.com/golang/go/blob/master/src/net/http/fs.go#L480C36-L480C38

This doesn't look right, If-Modified-Since is for checking a range of time https://datatracker.ietf.org/doc/html/rfc7232#section-3.3 , not just a single time instance.


Does not worl very well

$string = "The text you want to filter goes here. http://google.com,https://www.youtube.com/watch?v=K_m7NEDMrV0,https://instagram.com/hellow/";
return
Array
(
[0] => http://google.com
[1] => https://www.youtube.com/watch?v=K_m7NEDMrV0,https://instagram.com/hellow/
)

should return
Array
(
[0] => http://google.com
[1] => https://www.youtube.com/watch?v=K_m7NEDMrV0
[2] => https://instagram.com/hellow/
)

Hi Petar @consigliere23 👋,
Thank you for your interest in the Url highlight library.
This is expected behaviour. Comma is a valid character of the URL query component. Therefore https://www.youtube.com/watch?v=K_m7NEDMrV0,https://instagram.com/hellow/ - is a valid URL. Where attribute v contains data K_m7NEDMrV0,https://instagram.com/hellow/.
Library always assumes that character is a part of the URL if it’s a valid character for that component.
While it was correctly separating the first and second URLs http://google.com,https://www.youtube.com/watch?v=K_m7NEDMrV0:

http://google.com
https://www.youtube.com/watch?v=K_m7NEDMrV0

Because comma is not a valid character for the host component (google.com) nor for the scheme (https).
I don't see a way to correctly identify if a comma (or any other valid character) is a part of the URL or a separator to separate URLs. If you have any ideas how to improve this - please post it here. Also you are welcome to post a PR with suggestions.

Hi @vstelmakh,
Well yes it makes sense.
Just in my particular case I needed 3 URLs instead of  2 :/
What if add a option to pass to function if is comma "," is before regular URL (https or http) to extract as separated?

What if you use the php function explode (https://www.php.net/explode) before calling the library?

And what's a benefit of it? Explode with comma as separator?
To call getUrls for each line?
Explode can mess it up, for example: "http://123.com?c=1234,&id=1,http://456.com/blog
Right?
Just would be good to have option for above example to return 2 urls.
Does not not necessarily need be comma only, maybe can pass any string and than exclude.

No. Explode, then join. Or change the commas to spaces.
On Tue, Jul 2, 2024, at 7:25 PM, Petar wrote:

And what's a benefit of it? Explode with comma as separator?
To call getUrls for each line?
Explode can mess it up, for example: "http://123.com?c=1234,&id=1,http://456.com/blog
Right?
Just would be good to have option for above example to return 2 urls.
Does not not necessarily need be comma only, maybe can pass any string and than exclude.
—
Reply to this email directly, view it on GitHub https://github.com/vstelmakh/url-highlight/issues/47#issuecomment-2204676113, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMNCRGDJLKLF3IW4PR6IETZKMZHVAVCNFSM6AAAAABKFW5OLSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBUGY3TMMJRGM.
You are receiving this because you commented.Message ID: @.***>

--
Dan Langille
@.***

Will not help in this case "http://123.com?c=1234,**&**id=1,http://456.com/blog""
See in 123.com there is comma in url.

I understand the limitations.
I'm trying to offer you ideas. Over to you...

It looks like, it could be solved by adding some kind of a wrapper around library, to additionally process the output. Something like this:
<?php // This code requires PHP 7.4 or above

declare(strict_types=1);

require __DIR__ . '/vendor/autoload.php';

use VStelmakh\UrlHighlight\UrlHighlight;

$result = getUrls('Some string http://google.com,https://www.youtube.com/watch?v=K_m7NEDMrV0,https://instagram.com/hellow/?a=1,https://example.com/test/ which ends here.');
var_dump($result);

/**
 * @return array<string>
 */
function getUrls(string $input): array
{
    $urlHighlight = new UrlHighlight();
    $urls = $urlHighlight->getUrls($input);

    $result = [];
    foreach ($urls as $url) {
        $subUrls = getSubUrls($url);
        $result = [...$result, ...$subUrls];
    }

    return $result;
}

/**
 * @return array<string>
 */
function getSubUrls(string $url): array
{
    $pattern = '/
        (?<=^|,)           # start of string or comma
        https?:\/\/        # scheme
        .+                 # rest of the url
        (?=$|,https?:\/\/) # end of string or comma + scheme
    /xuU';
    preg_match_all($pattern, $url, $matches, PREG_SET_ORDER);
    return array_map(fn ($match) => $match[0], $matches);
}

The result is:
array(4) {
  [0] =>
  string(17) "http://google.com"
  [1] =>
  string(43) "https://www.youtube.com/watch?v=K_m7NEDMrV0"
  [2] =>
  string(33) "https://instagram.com/hellow/?a=1"
  [3] =>
  string(25) "https://example.com/test/"
}

I don't think this should be a part of the library. As it has some rather specific use case. Also it's not perfect and in edge cases (when URL contains comma) may produce invalid result.

Thanks @vstelmakh

Closing this issue as resolved.


Update README - Vertex Task only supports Models with Predict Method

I was playing with vertex-ai-task sample in Australia region, and realised that it works only with Text Models eg text-bison, text-unicorn. More details https://cloud.google.com/vertex-ai/generative-ai/docs/model-reference/text
Error Scenarios

When I tried with Gemini, it returned "Code 400, Gemini cannot be accessed through Vertex Predict/RawPredict API."
When I tried with code-bison, it returned "Code 400, Invalid Argument, Instance Prefix cannot be empty"

I was playing with vertex-ai-task sample in Australia region, and realised that it works only with Text Models eg text-bison, text-unicorn. More details https://cloud.google.com/vertex-ai/generative-ai/docs/model-reference/text
Error Scenarios

When I tried with Gemini, it returned "Code 400, Gemini cannot be accessed through Vertex Predict/RawPredict API."
When I tried with code-bison, it returned "Code 400, Invalid Argument, Instance Prefix cannot be empty"

Resoled error scenario 2 by ensuring right request schema is used within the integration flow. For example - with text-bison input request require field "prompt" vs code-bison input schema require field "prefix".


Opt-out of cloneElement for clickableNode

This function restricts the method by which the clickable node is injected. A wrapping DOM element is a small but necessary compromise.

We could try and deprecate the collapsibles prop entirely, and allow the component to accept instances of Collapsible directly -- which might improves composition for the end user. The cloned props just need to take less precedence than the inject props inbound from the CollapsibleGroup

NOTE: I am not sure if what I about to say is what the issue title is saying exactly so I am just looking for clarification on the issue pulled here.
Recently when working on a project - I needed to nest the clickable node, but with the current api I was not able to accomplish this as a <Collapsible /> wraps the header and body into one.
Example:

Title Text

        // the only element that should trigger a collapse

the collapsible content

I was thinking - a <Collapsible /> could be a HOC that passes props down, one of them being a ref that could be placed onto an element and only that element would be responsible for handling the trigger, instead of an entire clickable node.
Would probably need to pass a ref down for the content as well, to know what to open/close.
When implemented, it would look something like below:
<Collapsible>
  {(collapsibleProps) => {
    return (

Title Text

              {/* the only element that should trigger a collapse */}

the content to collapse.

    );
  }}
</Collapsible>

Would love to discuss!


feat: add auto-generation of account names for create command

Generate default name while using account create
Closes #2709
Introduced changes
Makes --name flag optional for account create command, matching behavior of account import.
When name is not provided, automatically generates incremental names (account-1, account-2, etc.).
Improves UX by reducing required flags while maintaining backwards compatibility.
Checklist

[x] Linked relevant issue
[ ] Updated relevant documentation
[ ] Added relevant tests
[x] Performed self-review of the code
[ ] Added changes to CHANGELOG.md

Hi @petersdt, how's the progress?

@petersdt are you still working on this?

@petersdt are you still working on this?

yes pls be done soon

Hello @petersdt , is there any update for this?

Closing PR due to inactivity.


⚠ Bofrost.at has degraded performance

In cc29ea0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 855 ms

Resolved: Bofrost.at performance has improved in 34cdaec after 24 minutes.


Issue with evaluate script.

Hi,
I get the following error while running evaluation script:
seq2seq-attn-master$ th evaluate.lua -model demo-model_final.t7 -src_file data/src-val.txt -output_file pred.txt -src_dict data/demo.src.dict -targ_dict data/demo.targ.dict
loading demo-model_final.t7...
done!
/Users/gupta/torch/install/bin/lua: ./s2sa/beam.lua:417: table index is nil
stack traceback:
./s2sa/beam.lua:417: in function 'idx2key'
./s2sa/beam.lua:650: in function 'init'
evaluate.lua:4: in function 'main'
evaluate.lua:30: in main chunk
[C]: in function 'dofile'
...upta/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: in ?
Any suggestions ?
regards,
Amit

Did you manually edit the generated dictionaries data/demo.src.dict and data/demo.targ.dict?
You can restart the preprocessing on the same dataset to make sure your dictionaries are valid.

No, I used the exact same commands as mentioned in the readme file. I am using Mac OS X sierra. I also tried running preprocess again, as well as running the entire training again.
I am using only 3 epochs for now to quickly see initial results.  Could that make a difference ?

I found the issue. It's on line no 415 in beam.lua:
for w in line:gmatch'([^%s]+)' do
the above gmatch command fails when the first character of token is not ascii.
That was causing the issue.


Add License

It would be nice to have a license on this project.
It would automatically update on this project: https://github.com/meichthys/christian_foss
Keep up the great work!

Cheers @meichthys , feel free to open a PR for adding one. It's been awhile since I've been able to work on this project.


First stab at updating the readme

Open to any comments. @anishathalye any thoughts you have too would be appreciated :)

Changes look pretty good to me.
Would a screenshot be good to add? Or even a gif or something?

Change the wget to a curl and I'll merge it.
Appropriate flags are -f (for don't return an html error), -s (for silent running), -S (for showing errors even though we are silent). I don't think -L is required, but it does not need to be excluded if you like the idea (following redirects to new places. Perhaps github is finicky about this).

Fixed to use curl instead of wget but I don't think we can do | sh, since doing so breaks the script (requires user input). See: http://stackoverflow.com/questions/2740868/difference-between-piping-a-file-to-sh-and-calling-a-shell-file
Unless anyone knows a workaround? Command is still copy pasteable. Should we cleanup the file after install?

@anishathalye screenshot could be cool. @Aviator45003 what do you think?
We can add it to the repo or make github be the CDN via: http://solutionoptimist.com/2013/12/28/awesome-github-tricks/

A screenshot or gif sounds great. Let's use git as the CDN (nobody who clones wants to see the image in the repo). I am partial to a gif.

Merged. as of this point.

Oh, no screenshot in this commit?

cool :)
Let's wait until after the name change, but I would prefer someone else does it. Gif would be cooler but also more to maintain if the script changes..

Do people find a project more approachable if there is a screenshot?
I can do it, as well as the gif.
On Thu, Jun 11, 2015 at 1:32 PM, Joshua Blum<EMAIL_ADDRESS>wrote:

cool :)
Let's wait until after the name change, but I would prefer someone else
does it. Gif would be cooler but also more to maintain if the script
changes..
—
Reply to this email directly or view it on GitHub
https://github.com/Aviator45003/easy-dotfiles/pull/5#issuecomment-111215732
.

@anishathalye, it's a working copy. The screenshot is extra. Right now, the README as merged is better than the old one. That means it's acceptable. We can add a screenshot later.

I think they would, its a nice way to see if the script is what you want
without downloading/running anything.
On Thu, Jun 11, 2015 at 8:42 PM, Toyam Cox<EMAIL_ADDRESS>wrote:

Do people find a project more approachable if there is a screenshot?
I can do it, as well as the gif.
On Thu, Jun 11, 2015 at 1:32 PM, Joshua Blum<EMAIL_ADDRESS>wrote:

cool :)
Let's wait until after the name change, but I would prefer someone else
does it. Gif would be cooler but also more to maintain if the script
changes..
—
Reply to this email directly or view it on GitHub
<
https://github.com/Aviator45003/easy-dotfiles/pull/5#issuecomment-111215732

.

—
Reply to this email directly or view it on GitHub
https://github.com/Aviator45003/easy-dotfiles/pull/5#issuecomment-111218777
.

Good point Toyam. The new readme is definitely an improvement. We can add the screenshot when we have time.
Also +1 about the name change.

On Jun 11, 2015, at 10:44 AM, Joshua Blum<EMAIL_ADDRESS>wrote:
I think they would, its a nice way to see if the script is what you want
without downloading/running anything.
On Thu, Jun 11, 2015 at 8:42 PM, Toyam Cox<EMAIL_ADDRESS>wrote:

Do people find a project more approachable if there is a screenshot?
I can do it, as well as the gif.
On Thu, Jun 11, 2015 at 1:32 PM, Joshua Blum<EMAIL_ADDRESS>wrote:

cool :)
Let's wait until after the name change, but I would prefer someone else
does it. Gif would be cooler but also more to maintain if the script
changes..
—
Reply to this email directly or view it on GitHub
<
https://github.com/Aviator45003/easy-dotfiles/pull/5#issuecomment-111215732

.

—
Reply to this email directly or view it on GitHub
https://github.com/Aviator45003/easy-dotfiles/pull/5#issuecomment-111218777
.

—
Reply to this email directly or view it on GitHub.

https://github.com/chjj/ttystudio might be useful for making a gif of the install instead of a screenshot

Right now I'm dealing with the fact that I don't have a good terminal emulator font (it can't show 'm', for example). If I could take a gif of the console, that would be great.

I don't think the font looks too bad on this..
Alternatively:
https://showterm.io/
https://github.com/icholy/ttygif


Create an initial release.

Hi, I found this repository via searching on crates.io under the plotting keyword, following the one with max recent downloads criterion-plot, then followed the discussion to arrive here.
I'd like to play with this code as an unstable dependency, but there are no releases on crates.io. Can we make that happen? How can I help?

The project was pretty much abandoned back in February, so there are currently no plans to add a release on crates.io. If you still want to use it, you can use a git or local path dependency.
https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-git-repositories
If you like the project, feel free to fork it and do some work on it. I'll probably be willing to take it up again if you do so, but I wont work on it alone.

@nathan-at-least, if it is still relevant, you can find the crate at https://crates.io/crates/ploteria. Thank you for your comment, and we are still looking forward to your feedback and/or contribution!

Someone wishes to make the releases on places liek reddit & u.r.l.o?

What does u.r.l.o stand for? 🤨
I think we should mention the release in the original forum thread as well as on reddit. 👍

ah my bad :D
i meant the users discourse forum


libssh2 fails to build with gcc 4.4.7/centos 6 - unrecognized command line option -std=gnu90

The error printed is about some git repo, but I think this is the actual error:
cc1: error: unrecognized command line option "-std=gnu90"
rustc version 1.24.0
cargo version 0.25.0
gcc version 4.4.7
I tried downloading the libssh2 1.8.0 sources and building them by hand on this machine - they ./configure and build ok, so it is not an issue with libssh2 not being buildable with this compiler.
[root@centos65 Foo]# cargo build -vv
    Updating registry `https://github.com/rust-lang/crates.io-index`
   Compiling libc v0.2.37
   Compiling cc v1.0.4
   Compiling pkg-config v0.3.9
     Running `rustc --crate-name libc /root/.cargo/registry/src/github.com-1ecc6299db9ec823/libc-0.2.37/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 --cfg 'feature="default"' --cfg 'feature="use_std"' -C metadata=59e008817578ee87 -C extra-filename=-59e008817578ee87 --out-dir /tmp/Foo/target/debug/deps -L dependency=/tmp/Foo/target/debug/deps --cap-lints warn`
     Running `rustc --crate-name cc /root/.cargo/registry/src/github.com-1ecc6299db9ec823/cc-1.0.4/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=01fd3f704fb11f2e -C extra-filename=-01fd3f704fb11f2e --out-dir /tmp/Foo/target/debug/deps -L dependency=/tmp/Foo/target/debug/deps --cap-lints warn`
     Running `rustc --crate-name pkg_config /root/.cargo/registry/src/github.com-1ecc6299db9ec823/pkg-config-0.3.9/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=896b6b9906494675 -C extra-filename=-896b6b9906494675 --out-dir /tmp/Foo/target/debug/deps -L dependency=/tmp/Foo/target/debug/deps --cap-lints warn`
warning: unused import: `std::ascii::AsciiExt`
  --> /root/.cargo/registry/src/github.com-1ecc6299db9ec823/pkg-config-0.3.9/src/lib.rs:67:5
   |
67 | use std::ascii::AsciiExt;
   |     ^^^^^^^^^^^^^^^^^^^^
   |
   = note: #[warn(unused_imports)] on by default

   Compiling libz-sys v1.0.18
   Compiling cmake v0.1.29
   Compiling openssl-sys v0.9.26
     Running `rustc --crate-name build_script_build /root/.cargo/registry/src/github.com-1ecc6299db9ec823/libz-sys-1.0.18/build.rs --crate-type bin --emit=dep-info,link -C debuginfo=2 -C metadata=e9d102d11f5e3635 -C extra-filename=-e9d102d11f5e3635 --out-dir /tmp/Foo/target/debug/build/libz-sys-e9d102d11f5e3635 -L dependency=/tmp/Foo/target/debug/deps --extern pkg_config=/tmp/Foo/target/debug/deps/libpkg_config-896b6b9906494675.rlib --extern cc=/tmp/Foo/target/debug/deps/libcc-01fd3f704fb11f2e.rlib --cap-lints warn`
     Running `rustc --crate-name cmake /root/.cargo/registry/src/github.com-1ecc6299db9ec823/cmake-0.1.29/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=846d77c8f85f537e -C extra-filename=-846d77c8f85f537e --out-dir /tmp/Foo/target/debug/deps -L dependency=/tmp/Foo/target/debug/deps --extern cc=/tmp/Foo/target/debug/deps/libcc-01fd3f704fb11f2e.rlib --cap-lints warn`
     Running `rustc --crate-name build_script_build /root/.cargo/registry/src/github.com-1ecc6299db9ec823/openssl-sys-0.9.26/build.rs --crate-type bin --emit=dep-info,link -C debuginfo=2 -C metadata=6bc9b54748dee815 -C extra-filename=-6bc9b54748dee815 --out-dir /tmp/Foo/target/debug/build/openssl-sys-6bc9b54748dee815 -L dependency=/tmp/Foo/target/debug/deps --extern pkg_config=/tmp/Foo/target/debug/deps/libpkg_config-896b6b9906494675.rlib --extern cc=/tmp/Foo/target/debug/deps/libcc-01fd3f704fb11f2e.rlib --cap-lints warn`
   Compiling libssh2-sys v0.2.6
     Running `rustc --crate-name build_script_build /root/.cargo/registry/src/github.com-1ecc6299db9ec823/libssh2-sys-0.2.6/build.rs --crate-type bin --emit=dep-info,link -C debuginfo=2 -C metadata=8ed7de87dd81e267 -C extra-filename=-8ed7de87dd81e267 --out-dir /tmp/Foo/target/debug/build/libssh2-sys-8ed7de87dd81e267 -L dependency=/tmp/Foo/target/debug/deps --extern pkg_config=/tmp/Foo/target/debug/deps/libpkg_config-896b6b9906494675.rlib --extern cmake=/tmp/Foo/target/debug/deps/libcmake-846d77c8f85f537e.rlib --cap-lints warn`
     Running `/tmp/Foo/target/debug/build/openssl-sys-6bc9b54748dee815/build-script-build`
cargo:rerun-if-env-changed=X86_64_UNKNOWN_LINUX_GNU_OPENSSL_LIB_DIR
cargo:rerun-if-env-changed=OPENSSL_LIB_DIR
cargo:rerun-if-env-changed=X86_64_UNKNOWN_LINUX_GNU_OPENSSL_INCLUDE_DIR
cargo:rerun-if-env-changed=OPENSSL_INCLUDE_DIR
cargo:rerun-if-env-changed=X86_64_UNKNOWN_LINUX_GNU_OPENSSL_DIR
cargo:rerun-if-env-changed=OPENSSL_DIR
cargo:rustc-link-lib=ssl
cargo:rustc-link-lib=crypto
OPT_LEVEL = Some("0")
TARGET = Some("x86_64-unknown-linux-gnu")
HOST = Some("x86_64-unknown-linux-gnu")
TARGET = Some("x86_64-unknown-linux-gnu")
TARGET = Some("x86_64-unknown-linux-gnu")
HOST = Some("x86_64-unknown-linux-gnu")
CC_x86_64-unknown-linux-gnu = None
CC_x86_64_unknown_linux_gnu = None
HOST_CC = None
CC = None
HOST = Some("x86_64-unknown-linux-gnu")
TARGET = Some("x86_64-unknown-linux-gnu")
HOST = Some("x86_64-unknown-linux-gnu")
CFLAGS_x86_64-unknown-linux-gnu = None
CFLAGS_x86_64_unknown_linux_gnu = None
HOST_CFLAGS = None
CFLAGS = None
DEBUG = Some("true")
running: "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-g" "-m64" "-Wall" "-Wextra" "-E" "/tmp/Foo/target/debug/build/openssl-sys-edebd9703df55829/out/expando.c"
exit code: 0
cargo:rustc-cfg=osslconf="OPENSSL_NO_EC2M"
cargo:rustc-cfg=osslconf="OPENSSL_NO_SRP"
cargo:conf=OPENSSL_NO_EC2M,OPENSSL_NO_SRP
cargo:rustc-cfg=ossl101
cargo:version=101
     Running `rustc --crate-name openssl_sys /root/.cargo/registry/src/github.com-1ecc6299db9ec823/openssl-sys-0.9.26/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=2ba57442a9da3d97 -C extra-filename=-2ba57442a9da3d97 --out-dir /tmp/Foo/target/debug/deps -L dependency=/tmp/Foo/target/debug/deps --extern libc=/tmp/Foo/target/debug/deps/liblibc-59e008817578ee87.rlib --cap-lints warn --cfg 'osslconf="OPENSSL_NO_EC2M"' --cfg 'osslconf="OPENSSL_NO_SRP"' --cfg ossl101 -l ssl -l crypto`
     Running `/tmp/Foo/target/debug/build/libz-sys-e9d102d11f5e3635/build-script-build`
cargo:rustc-link-search=native=/usr/lib64
cargo:rustc-link-lib=z
     Running `rustc --crate-name libz_sys /root/.cargo/registry/src/github.com-1ecc6299db9ec823/libz-sys-1.0.18/src/lib.rs --crate-type lib --emit=dep-info,link -C debuginfo=2 -C metadata=665843ece33ec450 -C extra-filename=-665843ece33ec450 --out-dir /tmp/Foo/target/debug/deps -L dependency=/tmp/Foo/target/debug/deps --extern libc=/tmp/Foo/target/debug/deps/liblibc-59e008817578ee87.rlib --cap-lints warn -L native=/usr/lib64 -l z`
     Running `/tmp/Foo/target/debug/build/libssh2-sys-8ed7de87dd81e267/build-script-build`
fatal: Not a git repository (or any of the parent directories): .git
running: "cmake" "/root/.cargo/registry/src/github.com-1ecc6299db9ec823/libssh2-sys-0.2.6/libssh2" "-DCRYPTO_BACKEND=OpenSSL" "-DBUILD_SHARED_LIBS=OFF" "-DENABLE_ZLIB_COMPRESSION=ON" "-DCMAKE_INSTALL_LIBDIR=lib" "-DBUILD_EXAMPLES=OFF" "-DBUILD_TESTING=OFF" "-DCMAKE_INSTALL_PREFIX=/tmp/Foo/target/debug/build/libssh2-sys-cc3d6a502d9c7d34/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=c++" "-DCMAKE_BUILD_TYPE=Debug"
-- The C compiler identification is GNU 4.4.7
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Found OpenSSL: /usr/lib64/libssl.so;/usr/lib64/libcrypto.so (found version "1.0.1e")
-- Looking for EVP_aes_128_ctr
-- Looking for EVP_aes_128_ctr - not found
-- Found ZLIB: /usr/lib64/libz.so (found version "1.2.3")
-- Looking for include file unistd.h
-- Looking for include file unistd.h - not found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - not found
-- Looking for include file stdlib.h
-- Looking for include file stdlib.h - not found
-- Looking for include file sys/select.h
-- Looking for include file sys/select.h - not found
-- Looking for include file sys/uio.h
-- Looking for include file sys/uio.h - not found
-- Looking for include file sys/socket.h
-- Looking for include file sys/socket.h - not found
-- Looking for include file sys/ioctl.h
-- Looking for include file sys/ioctl.h - not found
-- Looking for include file sys/time.h
-- Looking for include file sys/time.h - not found
-- Looking for include file sys/un.h
-- Looking for include file sys/un.h - not found
-- Looking for include file windows.h
-- Looking for include file windows.h - not found
-- Looking for include file ws2tcpip.h
-- Looking for include file ws2tcpip.h - not found
-- Looking for include file winsock2.h
-- Looking for include file winsock2.h - not found
-- Looking for sys/types.h
-- Looking for sys/types.h - not found
-- Looking for stdint.h
-- Looking for stdint.h - not found
-- Looking for stddef.h
-- Looking for stddef.h - not found
-- Check size of long long
-- Check size of long long - failed
-- Looking for gettimeofday
-- Looking for gettimeofday - not found
-- Looking for strtoll
-- Looking for strtoll - not found
-- Looking for _strtoi64
-- Looking for _strtoi64 - not found
-- Looking for snprintf
-- Looking for snprintf - not found
-- Looking for poll
-- Looking for poll - not found
-- Looking for socket
-- Looking for socket - not found
-- Looking for socket in socket
-- Looking for socket in socket - not found
-- Looking for socket in ws2_32
-- Looking for socket in ws2_32 - not found
-- Looking for select
-- Looking for select - not found
-- Looking for select in ws2_32
-- Looking for select in ws2_32 - not found
-- Looking for inet_addr
-- Looking for inet_addr - not found
-- Looking for inet_addr in nsl
-- Looking for inet_addr in nsl - not found
-- Looking for inet_addr in ws2_32
-- Looking for inet_addr in ws2_32 - not found
-- Performing Test HAVE_O_NONBLOCK
-- Performing Test HAVE_O_NONBLOCK - Failed
-- Performing Test HAVE_FIONBIO
-- Performing Test HAVE_FIONBIO - Failed
-- Performing Test HAVE_IOCTLSOCKET
-- Performing Test HAVE_IOCTLSOCKET - Failed
-- Performing Test HAVE_IOCTLSOCKET_CASE
-- Performing Test HAVE_IOCTLSOCKET_CASE - Failed
-- Performing Test HAVE_SO_NONBLOCK
-- Performing Test HAVE_SO_NONBLOCK - Failed
--
-- The following features have been enabled:

 * Compression , using zlib for compression
 * diffie-hellman-group-exchange-sha1 , "new" diffie-hellman-group-exchange-sha1 method
 * Logging , Logging of execution with debug trace

-- The following REQUIRED packages have been found:

 * OpenSSL
 * ZLIB

-- The following features have been disabled:

 * Shared library , creating libssh2 as a shared library (.so/.dll)
 * "none" cipher
 * "none" MAC

-- Configuring done
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_CXX_COMPILER

-- Build files have been written to: /tmp/Foo/target/debug/build/libssh2-sys-cc3d6a502d9c7d34/out/build
running: "cmake" "--build" "." "--target" "install" "--config" "Debug" "--"
Scanning dependencies of target libssh2
[  9%] [  9%] [ 14%] [ 19%] Building C object src/CMakeFiles/libssh2.dir/openssl.c.o
Building C object src/CMakeFiles/libssh2.dir/agent.c.o
Building C object src/CMakeFiles/libssh2.dir/channel.c.o
Building C object src/CMakeFiles/libssh2.dir/comp.c.o
cc1: error: unrecognized command line option "-std=gnu90"
cc1: error: unrecognized command line option "-std=gnu90"
cc1: error: unrecognized command line option "-std=gnu90"
gmake[2]: *** [src/CMakeFiles/libssh2.dir/openssl.c.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....
cc1: error: unrecognized command line option "-std=gnu90"
gmake[2]: *** [src/CMakeFiles/libssh2.dir/channel.c.o] Error 1
gmake[2]: *** [src/CMakeFiles/libssh2.dir/comp.c.o] Error 1
gmake[2]: *** [src/CMakeFiles/libssh2.dir/agent.c.o] Error 1
gmake[1]: *** [src/CMakeFiles/libssh2.dir/all] Error 2
gmake: *** [all] Error 2
thread 'main' panicked at '
command did not execute successfully, got: exit code: 2

build script failed, must exit now', /root/.cargo/registry/src/github.com-1ecc6299db9ec823/cmake-0.1.29/src/lib.rs:632:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.
error: failed to run custom build command for `libssh2-sys v0.2.6`
process didn't exit successfully: `/tmp/Foo/target/debug/build/libssh2-sys-8ed7de87dd81e267/build-script-build` (exit code: 101)
--- stdout
running: "cmake" "/root/.cargo/registry/src/github.com-1ecc6299db9ec823/libssh2-sys-0.2.6/libssh2" "-DCRYPTO_BACKEND=OpenSSL" "-DBUILD_SHARED_LIBS=OFF" "-DENABLE_ZLIB_COMPRESSION=ON" "-DCMAKE_INSTALL_LIBDIR=lib" "-DBUILD_EXAMPLES=OFF" "-DBUILD_TESTING=OFF" "-DCMAKE_INSTALL_PREFIX=/tmp/Foo/target/debug/build/libssh2-sys-cc3d6a502d9c7d34/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=c++" "-DCMAKE_BUILD_TYPE=Debug"
-- The C compiler identification is GNU 4.4.7
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Found OpenSSL: /usr/lib64/libssl.so;/usr/lib64/libcrypto.so (found version "1.0.1e")
-- Looking for EVP_aes_128_ctr
-- Looking for EVP_aes_128_ctr - not found
-- Found ZLIB: /usr/lib64/libz.so (found version "1.2.3")
-- Looking for include file unistd.h
-- Looking for include file unistd.h - not found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - not found
-- Looking for include file stdlib.h
-- Looking for include file stdlib.h - not found
-- Looking for include file sys/select.h
-- Looking for include file sys/select.h - not found
-- Looking for include file sys/uio.h
-- Looking for include file sys/uio.h - not found
-- Looking for include file sys/socket.h
-- Looking for include file sys/socket.h - not found
-- Looking for include file sys/ioctl.h
-- Looking for include file sys/ioctl.h - not found
-- Looking for include file sys/time.h
-- Looking for include file sys/time.h - not found
-- Looking for include file sys/un.h
-- Looking for include file sys/un.h - not found
-- Looking for include file windows.h
-- Looking for include file windows.h - not found
-- Looking for include file ws2tcpip.h
-- Looking for include file ws2tcpip.h - not found
-- Looking for include file winsock2.h
-- Looking for include file winsock2.h - not found
-- Looking for sys/types.h
-- Looking for sys/types.h - not found
-- Looking for stdint.h
-- Looking for stdint.h - not found
-- Looking for stddef.h
-- Looking for stddef.h - not found
-- Check size of long long
-- Check size of long long - failed
-- Looking for gettimeofday
-- Looking for gettimeofday - not found
-- Looking for strtoll
-- Looking for strtoll - not found
-- Looking for _strtoi64
-- Looking for _strtoi64 - not found
-- Looking for snprintf
-- Looking for snprintf - not found
-- Looking for poll
-- Looking for poll - not found
-- Looking for socket
-- Looking for socket - not found
-- Looking for socket in socket
-- Looking for socket in socket - not found
-- Looking for socket in ws2_32
-- Looking for socket in ws2_32 - not found
-- Looking for select
-- Looking for select - not found
-- Looking for select in ws2_32
-- Looking for select in ws2_32 - not found
-- Looking for inet_addr
-- Looking for inet_addr - not found
-- Looking for inet_addr in nsl
-- Looking for inet_addr in nsl - not found
-- Looking for inet_addr in ws2_32
-- Looking for inet_addr in ws2_32 - not found
-- Performing Test HAVE_O_NONBLOCK
-- Performing Test HAVE_O_NONBLOCK - Failed
-- Performing Test HAVE_FIONBIO
-- Performing Test HAVE_FIONBIO - Failed
-- Performing Test HAVE_IOCTLSOCKET
-- Performing Test HAVE_IOCTLSOCKET - Failed
-- Performing Test HAVE_IOCTLSOCKET_CASE
-- Performing Test HAVE_IOCTLSOCKET_CASE - Failed
-- Performing Test HAVE_SO_NONBLOCK
-- Performing Test HAVE_SO_NONBLOCK - Failed
--
-- The following features have been enabled:

 * Compression , using zlib for compression
 * diffie-hellman-group-exchange-sha1 , "new" diffie-hellman-group-exchange-sha1 method
 * Logging , Logging of execution with debug trace

-- The following REQUIRED packages have been found:

 * OpenSSL
 * ZLIB

-- The following features have been disabled:

 * Shared library , creating libssh2 as a shared library (.so/.dll)
 * "none" cipher
 * "none" MAC

-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/Foo/target/debug/build/libssh2-sys-cc3d6a502d9c7d34/out/build
running: "cmake" "--build" "." "--target" "install" "--config" "Debug" "--"
Scanning dependencies of target libssh2
[  9%] [  9%] [ 14%] [ 19%] Building C object src/CMakeFiles/libssh2.dir/openssl.c.o
Building C object src/CMakeFiles/libssh2.dir/agent.c.o
Building C object src/CMakeFiles/libssh2.dir/channel.c.o
Building C object src/CMakeFiles/libssh2.dir/comp.c.o
cc1: error: unrecognized command line option "-std=gnu90"
cc1: error: unrecognized command line option "-std=gnu90"
cc1: error: unrecognized command line option "-std=gnu90"
gmake[2]: *** [src/CMakeFiles/libssh2.dir/openssl.c.o] Error 1
gmake[2]: *** Waiting for unfinished jobs....
cc1: error: unrecognized command line option "-std=gnu90"
gmake[2]: *** [src/CMakeFiles/libssh2.dir/channel.c.o] Error 1
gmake[2]: *** [src/CMakeFiles/libssh2.dir/comp.c.o] Error 1
gmake[2]: *** [src/CMakeFiles/libssh2.dir/agent.c.o] Error 1
gmake[1]: *** [src/CMakeFiles/libssh2.dir/all] Error 2
gmake: *** [all] Error 2

--- stderr
fatal: Not a git repository (or any of the parent directories): .git
CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_CXX_COMPILER

thread 'main' panicked at '
command did not execute successfully, got: exit code: 2

build script failed, must exit now', /root/.cargo/registry/src/github.com-1ecc6299db9ec823/cmake-0.1.29/src/lib.rs:632:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

[root@centos65 Foo]# gcc -v
Using built-in specs.
Target: x86_64-redhat-linux
Configured with: ../configure --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-bootstrap --enable-shared --enable-threads=posix --enable-checking=release --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-gnu-unique-object --enable-languages=c,c++,objc,obj-c++,java,fortran,ada --enable-java-awt=gtk --disable-dssi --with-java-home=/usr/lib/jvm/java-1.5.0-gcj-<IP_ADDRESS>/jre --enable-libgcj-multifile --enable-java-maintainer-mode --with-ecj-jar=/usr/share/java/eclipse-ecj.jar --disable-libjava-multilib --with-ppl --with-cloog --with-tune=generic --with-arch_32=i686 --build=x86_64-redhat-linux
Thread model: posix
gcc version 4.4.7 20120313 (Red Hat 4.4.7-18) (GCC)

I think this may be a bug for upstream libssh2? I don't think that flag is explicitly passed here

I've removed cmake in #94, so this should in theory be fixed now.


Help Tooltip > Allow opener text to inherit cursor from ancestor, but always keep tooltip contents as text cursor

Related US
Necessary for backport into Grades.
Small PR. Contains changes decided from discussion with Jeff and Nicole here.
In particular to note: this  and this.

Nit: both the PR title and the commit message are pretty long. A typical convention would be something like:
fix: Adjust cursor styling for help tooltip
You could give that name to the PR, and then when squash merging, it'll use that name for the commit.

Haha looks like you merged just before my comment - all good, I see you got fix:  in the commit message at least 👍.


API spec-compliance fixes

Resolves https://github.com/single-cell-data/TileDB-SOMA/issues/834
Resolves https://github.com/single-cell-data/SOMA/issues/133

As above

Sorry, I'm not following what change is being requested by this.

@thetorpedodog @mlin Let me know if you intend to review these spec changes, in which case I'll hold off on merging. Ty!


v-model on select elements does not honor number attribute

Changing the value on the select element sets the model value as a string even when "number" attribute is specified on the select element.
Example on JSFiddle is here:
http://jsfiddle.net/5sH6A/78/

I think there is no problem here, value is correct & html always handle strings.

@boussou is correct, but Vue does provide a way of automatically casting to
number: see here
On 11/01/2015 11:55 AM, "boussou"<EMAIL_ADDRESS>wrote:

I think there is no problem here, value is correct & html always handle
strings.
—
Reply to this email directly or view it on GitHub
https://github.com/yyx990803/vue/issues/670#issuecomment-69478949.

I think this is actually a bug because it works for <input> elements (http://jsfiddle.net/3vmvmqdp/), but it doesn't look like it works for <select>s.

yep, you right. it could be a problem in case of arithemics on the value like "this.n+1"

Looks like this has been fixed in dev branch for #642. I just hope it works for selects that allow multiple values. Does the _.toNumber function support arrays?
https://github.com/yyx990803/vue/blob/dev/src/directives/model/select.js

Now it does :)


defaultProps JS Warning

Getting this warning in Google Chrome console when viewing BlueKit component in the browser:
Warning: Setting defaultProps as an instance property on Select is not supported and will be ignored. Instead, define defaultProps as a static property on Select.

Here's my package.json
{
  "dependencies": {
    "@rails/webpacker": "^3.0.2",
    "babel-preset-react": "^6.24.1",
    "classnames": "^2.2.5",
    "node-persist": "^2.1.0",
    "prop-types": "^15.6.0",
    "react": "^16.0.0",
    "react-bluekit": "^0.4.3",
    "react-dom": "^16.0.0",
    "react-redux": "^5.0.6",
    "react-router-dom": "^4.2.2",
    "redux": "^3.7.2"
  },
  "devDependencies": {
    "eslint": "4.9.0",
    "eslint-config-airbnb": "16.1.0",
    "eslint-plugin-import": "2.8.0",
    "eslint-plugin-jsx-a11y": "6.0.2",
    "eslint-plugin-react": "7.4.0",
    "webpack-dev-server": "^2.9.3"
  },
  "scripts": {
    "bluekit": "bluekit --baseDir ./app/webpack --paths . --exclude \"((?!.*/components/)|.*/components/container/).*\""
  }
}

Thank you for finding inconsistency in React Bluekit code. I prepared with fix of this warning.

Resolved in #126

@JamesPlayer New version, 0.4.4, is released to npm. Check it out, please.

Works perfectly, warnings are gone thanks!


ci: switch to macOS 12

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
It looks like that GitHub is not able to acquire mac OS 11 machines, let's give 12 a try.
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

@kubernetes-sigs/cri-tools-maintainers PTAL

/lgtm


Use simplepush module, enable event, and allow encrypted communication

Description:
Use simplepush module, enable event, and allow encrypted communication.
Related issue (if applicable): fixes 27893 and 6630
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3433
Example entry for configuration.yaml (if applicable):
notify:
  - name: simplepush
    platform: simplepush
    device_key: j4dFd4
    event: home
    password: test123
    salt: e48desftq

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

@MartinHjelmare, thanks.


An extension file is corrupted

Steps to Reproduce:
curl
❯ curl -L -o /tmp/rozbo.papercolor-vscode-0.4.0.vsix https://marketplace.visualstudio.com/_apis/public/gallery/publishers/rozbo/vsextensions/papercolor-vscode/0.4.0/vspackage
code --install-extension /tmp/rozbo.papercolor-vscode-0.4.0.vsix

Installing extensions...
Corrupt ZIP: end of central directory record signature not found
Server returned 500

❯ md5sum /tmp/rozbo.papercolor-vscode-0.4.0.vsix
5428a9691f682c3c0e53fe47bb5b7870  /tmp/rozbo.papercolor-vscode-0.4.0.vsix

browser download
❯ code --install-extension Downloads/rozbo.papercolor-vscode-0.4.0.vsix
Installing extensions...
(node:159216) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
Extension 'rozbo.papercolor-vscode-0.4.0.vsix' was successfully installed.
Server returned 500

❯ md5sum Downloads/rozbo.papercolor-vscode-0.4.0.vsix
73a930ec2f3815f39802bc328cabaf2d  Downloads/rozbo.papercolor-vscode-0.4.0.vsix

Both files are 618k but have the different hash.

Do you have a proxy or something else that might be getting in the way?

I got the same, but I can install the extension through the vscode UI, so I assume there is something invalid about your download command

curl -L -o /tmp/rozbo.papercolor-vscode-0.4.0.vsix https://marketplace.visualstudio.com/_apis/public/gallery/publishers/rozbo/vsextensions/papercolor-vscode/0.4.0/vspackage

We do not own this command. Seems it is working correctly when you download from browser or from UI. It could be an issue with your command or with Marketplace. So please reach out to Marketplace.

 ❯ wget --content-disposition -E https://marketplace.visualstudio.com/_apis/public/gallery/publishers/rozbo/vsextensions/papercolor-vscode/0.4.0/vspackage
...
HTTP request sent, awaiting response... 200 OK
Length: 618049 (604K) [application/vsix]
Saving to: ‘rozbo.papercolor-vscode-0.4.0.vsix.gz’
...

Using wget, it turns out the extension is .gz. It was a coincidence that both are 618k.
 ❯ unar rozbo.papercolor-vscode-0.4.0.vsix.gz
rozbo.papercolor-vscode-0.4.0.vsix.gz: Gzip
  rozbo.papercolor-vscode-0.4.0.vsix... OK.
Successfully extracted to "./rozbo.papercolor-vscode-0.4.0.vsix".

❯ ls rozbo*
.rw-r--r-- 618k  rozbo.papercolor-vscode-0.4.0.vsix
.rw-r--r-- 618k  rozbo.papercolor-vscode-0.4.0.vsix.gz

-rw-r--r-- 1  618510  rozbo.papercolor-vscode-0.4.0.vsix
-rw-r--r-- 1  618049  rozbo.papercolor-vscode-0.4.0.vsix.gz


DecodeTiffMeta err

I use imgproxy to handle a NEF pic.
It decode use tiff.go, the size is 160x120
tiff -> DecodeTiffMeta:  160 0
tiff -> DecodeTiffMeta:  160 120

but the pic size is 7380 × 4928
the pic is https://bos-media.sxtvs.com.cn/smc-test/75c046979256455eaef05eb9bbf3204b.NEF
the test param is /resizing_type:fit/enlarge:0/width:500/watermark:0.0/gravity:sm/

Hey @crystal-lee!
imgproxy doesn't support NEF format. However, it is based on the TIFF format. Specification says that NEF stores a 160x120 thumbnail as the main image, so imgproxy sees NEF files as 160x120 TIFF images.

thanks, i get it. and i use magick to handle it.


[SPARK-48174][INFRA] Merge connect back to the original test pipeline

What changes were proposed in this pull request?
This PR aims to merge connect back to the original test pipeline to reduce the maximum concurrency of GitHub Action by one.

https://infra.apache.org/github-actions-policy.html

All workflows SHOULD have a job concurrency level less than or equal to 15.

Why are the changes needed?
This is a partial recover from the following.

#45107

We stabilized the root cause of #45107 via the following PRs. In addition we will disable a flaky test case if exists.

#46395
#46396
#46425

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this INFRA PR, @cloud-fan ?

Could you review this PR, @viirya ?

Looks good to me.

Thank you, @viirya !
Merged to master.

late LGTM!

Thank you, @cloud-fan !


allow for null/NoneType values during type conversion

to: @austinbyers
cc: @airbnb/streamalert-maintainers
size: small
Background
Empty values should be allowed during type conversion.
Changes

Allowing for null/NoneType values during type conversion in classification.

Testing
N/A

Coverage decreased (-0.02%) to 97.535% when pulling 0ec4e0473943d22946541b6d42741d820ca6b192 on ryandeivert-allow-null-values into b79e82408f5ec32298c2e52459dd3e779a42e527 on master.

PR 777! ✈


Random crashing

Hi,
I am on version 1.5.4, and notice daily crashes. Node-red runs perfectly, then stops, requiring a manual restart (i am using home assistant).
Disabling alpha ess fixes the issue.
Any help / logs required please let me know.
Great app!
Thanks.

@vantech14 please update to v1.5.5 and check if the issue persists…

Possibly a duplicate of issue #2.

@dehsgr this version is not showing in Node-red, there is no option to update.
Thanks.

@vantech14 please try again. Flows weren‘t updated with last publish.

Hi,
I can see the update but get a new error which refuses to let me update the node. Error in this post relates to the email node, but the same error applies to the alpha node.
https://github.com/hassio-addons/addon-node-red/issues/1424

@vantech14 I assume the update error is related to your system/Node-RED install. There would no way for me to fix this.
Nevertheless you could post the whole update log. I‘ll try to help you further…

Hi,
Fixed the issue and I've upgraded. Unfortunately the integration keeps dropping are remotes 0 values. Disabling the node and re-enabling the integration fixes the issue for several hours.
Thanks.

Can you clarify, what you mean with "Unfortunately the integration keeps dropping are remotes 0 values"? Do you get any errors? Do you have a stable connection? I'm not aware of such an issue...

Here is a snapshot:

The values are coming back as zero.

@dehsgr interestingly if you reboot node red it doesn't fix the issue. You need to disable the node, save, re-enable and then it works again

Can you show your settings to me too? You might anonymize relevant parts. And I would like you to provide a log (best: verbose) when the issue occurs.

Im running NR in Home Assistant so unsure where the logs are configured and stored?

I cannot answer this. Maybe the HA documentation helps here…

@dehsgr
Here you go:
18 Aug 18:34:55 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error fetching realtime data for AL7001021060537: Malformed or empty response!
18 Aug 18:34:55 - [error] [ha-entity:Battery State of Charge] State must be defined.
18 Aug 18:34:59 - [info] [alphaess-monitoring:Alpha ESS Smile B3] Renewing token...
18 Aug 18:35:00 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error fetching realtime data for AL700XXXXXXX: Malformed or empty response!
18 Aug 18:35:00 - [error] [ha-entity:Battery State of Charge] State must be defined.
18 Aug 18:35:00 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error during login operation into Alpha ESS monitoring portal: TypeError: Cannot read properties of null (reading 'AccessToken')
We got the following unprocessable body:
{"code":102,"info":"Token is Expire","data":null}
18 Aug 18:35:04 - [info] [alphaess-monitoring:Alpha ESS Smile B3] Renewing token...
18 Aug 18:35:05 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error during login operation into Alpha ESS monitoring portal: TypeError: Cannot read properties of null (reading 'AccessToken')
I am using the same username and password i always have, with the serial number of the battery itself. These settings have worked flawlessly for months.

@vantech14 okay. So there is some issue while the authentication token is renewed. I suggest that you completely remove the alpha node once. Afterwards you safe/publish your flow. Now try to drop a new node instance with a fresh configuration on your flow. I assume there could be some kind of wrong (old) data which are used for token renewal. But I‘m not sure. Token renewal works flawlessly here…

@dehsgr deleted the node, now when I re-add it, populate all fields, it throws a configuration error saying that "some nodes are not configured properly" are you sure you want to deploy?

@dehsgr however the node is working, and pulling data. The error remains.

I would like you to try to tick and untuck the backup fetch checkbox for getting rid of the configuration error. I‘ll try to fix this issue as soon as possible.

@dehsgr woke this morning and found the integration was having token issues again. This was after a fresh reinstall of the alpha node.
Thanks.

@vantech14 token renewal failure seems to be some specific on your site. No other people are reporting this. As already discussed I would like you to set node-red logging to verbose level and provide me such a log.
The "invalid property issue" is fixed now.

@dehsgr here is the log in debug.
26 Aug 19:05:15 - [info] [alphaess-monitoring:Alpha ESS Smile B3] Renewing token...
26 Aug 19:05:15 - [debug] [alphaess-monitoring:Alpha ESS Smile B3] Fetching realtime data...
26 Aug 19:05:16 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error during login operation into Alpha ESS monitoring portal: TypeError: Cannot read properties of null (reading 'AccessToken')
We got the following unprocessable body:
{"code":102,"info":"Token is Expire","data":null}
26 Aug 19:05:16 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error fetching realtime data for AL700xxxxxx: Malformed or empty response!
26 Aug 19:05:16 - [debug] [alphaess-monitoring:Alpha ESS Smile B3] Processing data...
26 Aug 19:05:16 - [debug] [alphaess-monitoring:Alpha ESS Smile B3] Fetching hourly statistics...
26 Aug 19:05:16 - [debug] [alphaess-monitoring:Alpha ESS Smile B3] Fetching daily statistics...
26 Aug 19:05:16 - [error] [ha-entity:Battery State of Charge] State must be defined.
26 Aug 19:05:20 - [info] [alphaess-monitoring:Alpha ESS Smile B3] Renewing token...
26 Aug 19:05:20 - [debug] [alphaess-monitoring:Alpha ESS Smile B3] Fetching realtime data...
26 Aug 19:05:21 - [warn] [alphaess-monitoring:Alpha ESS Smile B3] There was an error during login operation into Alpha ESS monitoring portal: TypeError: Cannot read properties of null (reading 'AccessToken')
Thanks.

@vantech14 I just commited a possible fix for your issue. So I would like you to manueally change your alphaess-monitoring.js with the current one from this repository. Afterwards I would like you to provide feedback if the isse persits or not. If this would fix your issue I will publish this fix to NPM. Otherwise I would like you to place an additional log.

@dehsgr done, will revert back in a few days with an update. Thanks!

@dehsgr the integration is still up and working fine. Your latest fix worked. Thanks again.

@dehsgr are you using this API / have you seen this?
https://community.openenergymonitor.org/uploads/short-url/2qysTQ2mAaEEctdEIHXuaaofYIQ.pdf

@vantech14 thanks for your feedback. I‘ll release the new package later. I‘ll use another API, not the one you linked. Your API requires a secret offered by AlphaESS…

Published updated package now. Thanks for your input to for being able to fix this issue!


add legends

Start a separate issue regarding adding legends.  See #2.

@ignacio82 I got a really, really crude prototype of a legend ripped out from the treemap using gridSVG if you'd like to try it out.

It works :-).
I'm still not sure about how is handling the colors.
This is the static tree with a small data set (I'm generating some fake data to play with this)

this is the d3tree

The color for a 'school' should be the average color of its students, right?
This is a link to my fake data, a small DF with 200 students and a 'big' one with 30k.
Thanks for the great (and incredibly fast) work!


Fix flake8 complaint in group_models_by_index

This change fixes flake8 message that is thrown while running "make lint":
F812 list comprehension redefines 'models' from line 13

Thanks!


Synchronize FeedForward and Receding Horizon combination broken

If a simulation run a synchronize FF and receding horizon in the recipient stage the simulation can break.

closed via #529


bug: source .env.example fails

Bug Report
Kakarot RPC version:
HEAD ->  b4e148fc72c5a414199fe4bec3d0e3c3726d5fee
Current behavior:
cp .env.example .env && source .env

The above will result in the following error:
source .env.example
.env.example:2: command not found: STARKNET_RPC_URL
.env.example:3: command not found: RUST_LOG
.env.example:4: command not found: KAKAROT_HTTP_RPC_ADDRESS
.env.example:5: command not found: KAKAROT_ADDRESS
.env.example:6: command not found: PROXY_ACCOUNT_CLASS_HASH

Expected behavior:
The .env.example should be in the correct format, so that when copied into .env and then used with source,  it should work fine.
Steps to reproduce:
cp .env.example .env && source .env

Anyone working on this? I would be grateful to take it


Bug 367544

TA installation issues fixes:
If there is a pending windows update. We provide a proper error message and fail the task
if there is a pending restart we restart and retry ta installation once.
Testing done: manual validation.

approved with suggestions


[release/8.2] Enable signing and use stable packages

Description
Enabling signing and stabilizing package versions for the release.
Checklist

Is this feature complete?

[x] Yes. Ready to ship.
[ ] No. Follow-up changes expected.

Are you including unit tests for the changes and scenario tests if relevant?

[ ] Yes
[x] No

Did you add public API?

[ ] Yes

If yes, did you have an API Review for it?

[ ] Yes
[ ] No

Did you add <remarks /> and <code /> elements on your triple slash comments?

[ ] Yes
[ ] No

[x] No

Does the change make any security assumptions or guarantees?

[ ] Yes

If yes, have you done a threat model and had a security review?

[ ] Yes
[ ] No

[x] No

Does the change require an update in our Aspire docs?

[ ] Yes

Link to aspire-docs issue:

[x] No

Microsoft Reviewers: Open in CodeFlow

/azp run dotnet.aspire

Validated in an internal build that this is working as expected.


preview

adding preview

There were a couple of issues. The name of the preview image didn't match the .json. Also the .json file was mislabeled. It has to be "sheet.json" spelled exactly that way. I changed it for you.


Personal and New tabs are shown, despite being disabled

From @jeroenpraat on March 29, 2014 19:56
Under 'Settings' > 'Additional features' > 'Network and Stream Filtering', 'Network Personal Tab' and 'Network New Tab' are disabled (by default). But both tabs still show up.
Copied from original issue: friendica/red#384

Thanks, this works again.