Ownership selection in VPC tiers and VPC public IPs

Description
This PR adds the "ownership selection" UI fields to VPC tiers and VPC public IPs.
The "vpcAccess" response attribute was added so the UI can know if the caller has access to the VPC without needing to make a whole new request. This is needed to avoid 404 links (since router.resolve is assembled through API permissions) and so the UI can present the correct public IPs to their possible network tiers (e.g. if the User has tiers of two different VPCs, they can't use one's public IP in the other).
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI
[ ] test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[X] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):

New tier possible owners' listing

Tier listing

User accessing their tier of the Admin's VPC

Possible owners for new IP

How Has This Been Tested?
Environment with domains Root (with accounts Root Admin and User), Root/1 (with a Domain Admin), Root/A (with a Domain Admin) and Root/A/B (with a Domain Admin and a User). There is a Project in each domain, owned by the respective admins.

Situation
Test

As Root Admin with VPC owned by self.
Every account and project listed as possible owners in Tier creation

As Root Admin with VPC owned by self and one tier for each account and project.
Every account and project listed as possible owners in acquiring public IP.

Same as previous, but remove some of the tiers.
The deleted tiers' owners also disappeared from IP possible owners' list, except Root Admin itself.

As Root Admin with VPC owned by self, create one tier with a private ACL for the User.
User's access to network page did not present the 'ACL' attribute.

As Root Admin with VPC owned by self, create one tier with a global ACL for the User.
User's access to network page showed  'ACL' attribute.

How did you try to break this feature and the system with this change?

@blueorangutan package

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11164


Textures not properly loading on android

On android devices, using pojav launcher(i use android 13), the rails appear red and the texture appears black. I use 1.19.2 fabric and Xiaomi 11T pro.

Also, i use ANGLE renderer

Mods list:

Since when is Pojav (or any mobile Java launcher) supported?

Nemo did fix some bug for android like when models weren't rendering with GL4ES. Look at the changelog for 0.3.2

🤔 So it still isn't working properly
It's not easy to debug rendering issues on Android :(

Ok, i understand


Migration error

i got below error after i create fresh project "composer create-project aimeos/aimeos myshop"
Creating base tables
Using schema from locale.php
Checking table "mshop_locale_site":
In MySqlSchemaManager.php line 315:
Type error: Argument 1 passed to Doctrine\DBAL\Schema\MySqlSchemaManager::p
arseCreateOptions() must be of the type string, null given, called in /User
s/sufi/Documents/Work/GitHub/myshop/vendor/doctrine/dbal/lib/Doctrine/DBAL/
Schema/MySqlSchemaManager.php on line 307

That's a bug in the current Doctrine DBAL library (2.9.1): https://github.com/doctrine/dbal/issues/3410
We've added a constraint to avoid that version. Can you try again?

Wahh.. so fast reply.. okay i'll try tonight.. thanks alot with this fast
assist.. I'll update if got any other error
On Thu, Dec 20, 2018, 19:56 Aimeos<EMAIL_ADDRESS>wrote:

That's a bug in the current Doctrine DBAL library (2.9.1):
doctrine/dbal#3410 https://github.com/doctrine/dbal/issues/3410
We've added a constraint to avoid that version. Can you try again?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/11#issuecomment-448973917, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQuVOT-GoBshStTshIh852LoBxJes8wTks5u63sDgaJpZM4Zb_PK
.

Hi, i still got the same error

Did you run composer update on your existing installation or did you create a new one with composer create-project aimeos/aimeos myshop?
Can you post the output of composer show?

i create another fresh project

composer ignores the requirement in the latest aimeos-core packages and installs the previous version instead which doesn't limit doctrine/dbal to <2.9.1. The only was to be sure composer uses that requirement was to add it to the composer.json file of the Aimeos package itself.
We've released a new version 2018.10.2 which now enforces the correct DBAL version.

Awesome..it works now..thanks for the assist


Create Groups API - add grouping of entities

FEATURE REQUEST

Is there an open issue addressing this request? If it does, please add a "+1" reaction to the
existing issue, otherwise proceed to step 2.

Describe the feature you are requesting, as well as the possible use case(s) for it.
Following the https://github.com/mainflux/mainflux/issues/931, we are breaking it in 2 issues. This one will handle the groups, and new one will be opened for policies (i.e. we will first merge groups only, and then add policies).
Groups will be part of Auth microservice.

Indicate the importance of this feature to you (must-have, should-have, nice-to-have).
Must-have

Will be done in https://github.com/mainflux/mainflux/issues/1346

@drasko are you sure about closing this one?

Damn, wrong issue :)
I will reopen.


Django 2.2 Support

Tests are not run against Django 2.2.
Has anyone successfully used django-hijack with Django 2.2?

Yes

Django 2.2 is currently work, yet not officially supported. That being said, version support is discussed as part of #209

Django 2.2 is currently work, yet not officially supported. That being said, version support is discussed as part of #209

Django 2.2 support has been merged to master and will be included in the next release.

Django 2.2 support has been merged to master and will be included in the next release.


Support passing options to a rule condition

Appends an optional payload parameter on .can() method and condition callback passed to .allow() method.
It makes it possible to check additional informations when it's time to decide if an action can be performed or not. For instance, on an update action, permissions can now be granted according to updated fields, as shown on the updated readme.

This is a great addition. @vadimdemedes I think you should review this for sure.

Hi @vadimdemedes !
I have been followed all your recommendations in (i hope) this last commit.
Thanks for your time and have a nice day !

Thanks for working on this!


fixed infinite render loop

Issue: #1747
What I did
fixed the issue
How to test
check the below URL repo
https://github.com/mattrothenberg/storybook-test

Please don't use "closes" and "fixes" magic words, we close the issues only after release. When a PR gets merged, we simply mark the issue with merged label: https://github.com/storybooks/storybook/blob/master/CONTRIBUTING.md#closing-issues

sorry, I'll be more careful 🙇

Please don't use "closes" and "fixes" magic words, we close the issues only after release. When a PR gets merged, we simply mark the issue with merged label: https://github.com/storybooks/storybook/blob/master/CONTRIBUTING.md#closing-issues

Something to discuss on the Roadmap meeting.


chore: Bump Go version and add Makefile

Bumps the go version and add a Makefile.
Checklist

[ ] I have done a self-review of the PR.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 87.245%

Files with Coverage Reduction
New Missed Lines
%

notifier.go
3
89.51%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
814

Relevant Lines:
933

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 87.245%

Files with Coverage Reduction
New Missed Lines
%

notifier.go
3
89.51%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
814

Relevant Lines:
933

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 8 (75.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 87.245%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/bugsnag/main.go
0
2
0.0%

Files with Coverage Reduction
New Missed Lines
%

notifier.go
3
89.51%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
814

Relevant Lines:
933

💛  - Coveralls


Test notebooks on CI

See: #575

This will be much easier after #610, because the notebooks can then automatically download and load the datasets.


Find a node editor that can take the behavior tree and convert into a tree/graph

egui_node_graph looks like the best option.
Potential flow:

Create behavior tree
Convert behavior tree to a immutable node grapha
Could use mpsc channels to tell which node is active

We could also have that the behavior tree sets up a websocket service, that writes which node is active. Then a second service will read that and compile the tree. The second service would need to read a json to know how the tree is composed such that it can be visualized.

A possible option is to compile the behavior tree to a petgraph, then from a petgraph to a graphviz.dot format for visualization


Update to 1.19.4

Bump version to 1.2.0
Draft pull request as there might be a better way of doing this, I simply copied the code from MathHelper, as multiplyColors got removed in the 1.19.4 version.
Build in case anyone needs it:
https://github.com/TheMrEngMan/Colorful-Subtitles/releases/tag/1.2.0

This is also what i did to update to 1.19.4; i don't think of any other good way really

Updated to Minecraft 1.20.4
Bump version to 1.3.1
Build in case anyone needs it:
https://github.com/TheMrEngMan/Colorful-Subtitles/releases/tag/1.3.1


Interests and Topic feature modules should be merged

Previously we had an authors module so the module structure was:

interests

authors
topics

Now that we no longer have authors, the nested structure adds complexity for no benefit, making it especially difficult to understand the navigation graph. Suggest merging :topics into :interests (whilst keeping the Topic model).
This would enable InterestsListDetailScreen which is currently in the :app module to be moved into the feature module.

The merged two modules, seemed too big to me.
What about just moving ``InterestListDetailScreenandInterests2PaneViewModelinto:interests`?


Not Working on Live Server

Hello!
App is running fine locally but when put on live server its not working (different stun/turn servers used). In console I get following Error:
Uncaught SyntaxError: Illegal return statement
room.js:10 Uncaught ReferenceError: Meeting is not defined
at HTMLDocument. (room.js:10)
at j (jquery.min.js:2)
at Object.fireWith [as resolveWith] (jquery.min.js:2)
at Function.ready (jquery.min.js:2)
at HTMLDocument.J (jquery.min.js:2)

Hi
On the same network its working fine when deployed on server with HTTPS but its not working for different network. Can you please help me with this issue as I have provided the working and paid stun and turn servers but still no luck.


openai ChatCompletionRequest missing tools field?

What is the issue?
when i use ChatOpenAI client in langchain, ollama server receive:
{
	"messages": [{
		"content": "what is the weather in Boston?",
		"role": "user"
	}],
	"model": "llama3:8b-instruct-q4_0",
	"n": 1,
	"stream": false,
	"temperature": 0.0,
	"tools": [{
		"type": "function",
		"function": {
			"name": "tavily_search_results_json",
			"description": "A search engine optimized for comprehensive, accurate, and trusted results. Useful for when you need to answer questions about current events. Input should be a search query.",
			"parameters": {
				"type": "object",
				"properties": {
					"location": {
						"description": "The city and state, e.g. San Francisco, CA",
						"type": "string"
					}
				},
				"required": ["location"]
			}
		}
	}]
}

but in Middleware() funtion, it transform to ChatCompletionRequest, and ChatCompletionRequest have no tools field
type ChatCompletionRequest struct {
	Model            string          `json:"model"`
	Messages         []Message       `json:"messages"`
	Stream           bool            `json:"stream"`
	MaxTokens        *int            `json:"max_tokens"`
	Seed             *int            `json:"seed"`
	Stop             any             `json:"stop"`
	Temperature      *float64        `json:"temperature"`
	FrequencyPenalty *float64        `json:"frequency_penalty"`
	PresencePenalty  *float64        `json:"presence_penalty_penalty"`
	TopP             *float64        `json:"top_p"`
	ResponseFormat   *ResponseFormat `json:"response_format"`
}

func Middleware() gin.HandlerFunc {
	return func(c *gin.Context) {
		var req ChatCompletionRequest
		err := c.ShouldBindJSON(&req)
                 ......}

so i think it's not support function calling？
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
v0.1.46

This has been added recently.  Please upgrade to the latest version and the tools field is now present.


Link to headings and blocks

How can I link to headings and blocks from Excalidraw ?
Moreover, it would be nice if Excalidraw could suggest links instead of using insert link to file.
Thank you,
Missionary

btw. you can link to headings and blocks just as you would normally [[page#^blockref]] or [[page#headingref]] or ![[page#^blockref]], etc.
Yes, it would be nice to have the link suggester... for now you can switch over to markdown view and edit text elements in markdown there. Not ideal, but works.


Cant Edit DNS Hole Widgets (or other widgets too?)

Environment
Docker
Version
No response
Describe the problem
I am using Homarr on unraid.
Everything works fine, instead of Widgets.
if I add a DNS Hole widget, i cant access its settings to enter e.g. Adguard credentials.
The only options on the gead icon is remove and change position.
Where i can find these settings?
Many thanks in advance!
Logs
No response
Context
No response
Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

Hi, you do not need to specify your credentials in the widget, but in the app instead.
This is documented on https://homarr.dev and intended behaviour.

could you please specify it a bit more?
I can´t find the app configuration in the app nor the documentation...
The widget is empty...

Ah sorry, it was not clear for me yet, that you already configured the app.
Can you try to clear the cache once in the settings?

Sure, i have. but nothing changed...


RDK-35564: Support Semantic Versioning for RDKServices

update chanelog and api version failed because I didn't modify CHANELOG.md here i add it. This web-hook check if file was modified not added.


Remove ember-cli-test-loader from bower.json

I'm on Ember 2.7, and I get a deprecation warning after installing ember-cli-mocha:
DEPRECATION: ember-cli-test-loader should now be included as an NPM module with version 1.1.0 or greater.

It'd be nice to remove this line, but I'm not sure how that might affect people using versions of Ember prior to 2.7.

@nucleartide Previous versions of Ember CLI already included the test loader in bower.json anyway (see https://github.com/ember-cli/ember-cli/blob/v2.6.3/blueprints/app/files/bower.json#L6), so I'm not sure why it was added here too. IMHO we can remove that line. Could you open a PR that removes it?

actually, just removing it might not be such a great idea. we should try to mimic what has been done in https://github.com/ember-cli/ember-cli-qunit/commit/115d56bff12ad01ef08c34a1ae9f9011e7e6904b instead.

If I understand correctly, we should be able to add ember-cli-test-loader as a dependency of ember-cli-mocha and then expose it using the shouldIncludeChildAddon hook instead, allowing us to avoid adding the dependency to the consuming application entirely.


Regenerate CRDs with<EMAIL_ADDRESS>

LGTM
Context for this was we need to get rid of the empty status field on the custom resource definitions since they confuse tooling like kapp-controller (it tries to converge back to an empty status state whenever the status is updated).
and TIL about x-kubernetes-map-type:
// x-kubernetes-map-type annotates an object to further describe its topology.
// This extension must only be used when type is object and may have 2 possible values:
//
// 1) `granular`:
//      These maps are actual maps (key-value pairs) and each fields are independent
//      from each other (they can each be manipulated by separate actors). This is
//      the default behaviour for all maps.
// 2) `atomic`: the list is treated as a single entity, like a scalar.
//      Atomic maps will be entirely replaced when updated.
// +optional


Significance of

When we use below in startup.cs file we can able to authenticate from Azure AD.
services.AddAuthentication(AzureADDefaults.AuthenticationScheme)
.AddAzureAD(options => Configuration.Bind("AzureAd", options));
then what is the significance of the below code. If we don't add below code that does mean that out app is not compliant with v2.0 end point. Can any one please explain it.
services.Configure(AzureADDefaults.OpenIdScheme, options =>
{
options.Authority = options.Authority + "/v2.0/";
options.TokenValidationParameters.ValidateIssuer = false;

});

@AmitavaHazra , Could you please provide the document you were referring to for better assistance?

@SubhashVasarapu-MSFT,
Please find doc reference.
https://docs.microsoft.com/en-us/azure/active-directory/develop/quickstart-v2-aspnet-core-webapp


need help

hi i dont understand whats the part of the tor bundle, does it make you anonymous like a vpn or what? is this bot can make an ip ban? does this bot also have anaccount maker like the python one?

i think (take it with a grain of salt) its just to pass the bot network like normal network, since spamming pixels with 100 accounts in a single home ip might get you banned

this bot can not make an ip ban but it may ban the bot account
this doesnt have an account maker

do i need to do anything to start the tor? does it start itself? does it run with the bot?

also whats the difference between developer accounts than the bots account? what do i need to enter? can i create these accounts manually?

just read the readme


[New] Strapi - Change of the project corrdinator in Front Office and its impact in Strapi

I assigned CIRAD as the coordinator of the MRV4SOC project.
However, in the Front Office, if I created another organization (GMV) and set it as the coordinator for MRV4SOC: it overwrited it in Strapi, even before I have validated the organization in Strapi.
This should at least wait for my validation in Strapi.

I was able to reproduce this behavior and identify the root cause. Permanently fixing it will require substantial changes and some time. Since the current implementation may lead to data corruption, we'll be temporarily disabling this feature on the FE. Once the fix is in place, we'll reenable it

Thank you Tiago
What feature will you be disabling in the FE?

ok

JIRA #ORC-409


set recolor "false"

First of all thank you for this repo.
Second, why would you set recolor default to "false" ? it took me a couple of minutes to figure out why the colorscheme was not working after copy and pasting the code into my rc file..

Hi,
In my opinion, recolor "true" is a very opinionated setting. That's why I decided to set it to "false" and keep it in the theme so that users can change it. Maybe I could add a note in the Readme (stating that recolor is set to false) or make a second template with recolor set to "true". What do you think ?

as far as i understood setting recolor to "true" is necessary for the colorscheme take effect? it's not clear to me why anyone would copy the colorscheme and then not set it to true.. but i guess one additional line in the Readme will do just fine.

The recolor option only affect the look of the document. Even if recolor is set to false, the UI (status bar, input bar, background color, ...) still follows the colorscheme.

a fair bit of people that use zathura use tiling window managers

Hello there. I'm in the opinion that we shouldn't set recolor at all.
The default option on zathura is recolor=false, so in effect, the template is redudant, apart from a single, fairly relevant case:
When we include the colorscheme on zathurarc (assuming multiple files), it needs to be included before your custom 'recolor'.
On the other hand if we include it afterwards (i.e. bottom of config), it overwrites user-defined configuration.
My suggestion is to remove the opinion altogether from the template and remove the recolor settings.
Only possible issue, I can think of is that we would be relying on Zathura defaults, which might change over time.
Personally, I think that it's a pretty sane thing to do, in this case.
Cheers

hi,
maybe I should add a second template with set recolor=True to this repository so that both options are available. (and remove the recolor option from the current one as you mentioned)

Yeah, no problem. Will add a PR.


ESM build and refactoring

Add support for ESM .coffee files in coffee-esm.mjs (autodetection)
Build new ESM version dist/main.mjs, which Vite seems to need in #125 (I've tested this in code I haven't yet pushed). (If I just told esbuild to convert the CJS code to ESM, you wouldn't be able to do import {compile} from '@danielx/civet'; only the default export gets preserved.)
CoffeeScript code refactored to use ESM (but some things like Hera parser are still CJS)
Note that default exports no longer have everything; need to import *

Oh, in package.json, I'm not sure whether "exports"/"." should point to the .js or .mjs file... Vite was unhappy with the former, but it was fine when I explicitly imported ".../main.mjs".

I think using import and require conditions is a good idea. My main goal is to have Civet "just work" whatever type of project people npm install into. And that probably means adding both ESM and CJS builds to the default dist with the proper package.json config.


Opt: Implement Code Sinking?

During recent comparison of SPIR-V to DX assembly, it seemed that fxc was performing an optimization that I will call "code sinking". The process of sinking is to move an instruction as deep as possible into conditional control flow. The reason to do this is that, for uniform control flow, execution of code under control flow may be avoided, thus potentially optimizing execution time of the shader.
Analysis suggests that code sinking in spirv-opt could have performance benefit. And if code is not replicated, such sinking should generally do no harm to performance.
Evidence
Using renderdoc profiling of a SPIR-V shader in a real life Vulkan capture on a prominent graphics platform, I performed several different experiments.
First, I did sinking on the SPIR-V as much as possible manually. Even though the opportunities were not huge, about a 1% improvement was achieved in the shader execution time.
Then I reversed the optimization. I pulled as much code out of conditional control flow as I could and placed it just before the control flow divergence. The time for the shader increased by 20%.
These two results suggest two things: the platform I was using was performance sensitive to the placement of code relative to conditional control flow, and the driver's shader compiler was not doing this optimization.
Initial Restrictions
I think we can probably come up with a set of restrictions for this optimization that capture the lion's share of the benefit and keep the algorithm fairly simple. The algorithm can be made more sophisticated as is needed.
One almost obvious restriction is to not push code into loops. Most shader loops execute at least once so at best this will be a wash and at worst it will increase execution time.
Another restriction is to only push combinator operations (add, mul, etc) and non-indexed loads which shader compilers can hoist back out if they wish. In general pushing sample instructions is safe if the control flow is uniform, but it does require that analysis. For the several cases I looked at, most benefit could still be achieved without pushing samples. Removal of this restriction can be done at a later time as it is needed.
Finally, only push instructions whose results are used in just a single block. This trivially avoids duplication. This may mean we miss some opportunities, but it makes the algorithm much simpler. For the several cases I looked at, most benefit could still be achieved with this restriction. Removal of this restriction can be done at a later time as it is needed.
Other Algorithmic Details
A phi function use will count as a use for the block with which the value is associated through the phi function.
Other Optimization Implications
This optimization may reduce opportunities for if-conversion. But while this may slightly increase the shader size comparatively, its performance should be no worse and likely better as more code is under conditional execution. It is therefore recommended that this be done before if-conversion.
Theoretically this could undo opportunities for common subexpression elimination and have negative size and performance implications. It may be desirable to execute CSE first.
IHV Implications
Since fxc does this optimization as well, it is likely that implementing this optimization in spirv-opt will not do harm, at least relative to DX.

I will claim this issue for the moment.

I would like to be able to move loads from global resources.  When we have a function call with a value parameter that is inlined, we will usually end up with a load at the "start" of the inlinined function even if it used much later.
The other way this optimization can help is in trying to reduce the live range of a value, which should help reduce register pressure.  I think we should consider being fairly aggressive with this for experimentation.

The algorithm has been implemented.  For any further improvements, we should open a new issue.


Set up a webhook for GitHub repository

As a developer
I need to configure a WebHook on the GitHub repository
So that commits to the master branch trigger the Tekton pipeline
Details and Assumptions

A team member with admin access will configure the webhook.
The webhook will trigger the EventListener to start the pipeline.

Acceptance Criteria
Given a webhook is configured on the GitHub repository
When a commit is pushed to the `master` branch
Then the webhook should trigger the Tekton pipeline to execute

@ahighmoon Hi Yifei, the instruction starts around the 55m in the video and only the admin is able to do that. So I assign to you for now since you volunteer to demo :)


"We cannot execute …java" on Mac

I have tried a few rounds of removing and reinstalling Java with reboots along the way, but my attempts to launch Browsermob keep invariably failing with the message

Error: JAVA_HOME is not defined correctly.
We cannot execute /System/Library/Frameworks/JavaVM.framework/Versions/CurrentJDK/Home/bin/java

At the same time:

macbook:bin romank604$ whereis java
/usr/bin/java
macbook:bin romank604$ which java
/usr/bin/java
macbook:bin romank604$ echo $JAVA_HOME // in ~/.bash_profile
/Library/Java/JavaVirtualMachines/jdk1.8.0_112.jdk/Contents/Home
macbook:bin romank604$ java -version
java version "1.8.0_112"
Java(TM) SE Runtime Environment (build 1.8.0_112-b16)
Java HotSpot(TM) 64-Bit Server VM (build 25.112-b16, mixed mode)

On Windows 10 I have no problem launching Browsermob, but I must use a Mac and I am new to MacOS. I did whatever I could find (and understand) via Google and StackOverflow - to no avail. Would appreciate any help, thank you.

The startup script is in dire need of simplification. I've got that down for the next major release. In the meantime, you can start BMP on OS X by running java -jar browsermob-dist-2.1.3.jar from within the lib directory.

I did just simplify the startup script, it should work pretty consistently on OS X and Linux now.


Update FAQ #ramp_down_stores: Remove second sentence

This PR removes the sentence: "Whether the CWA will still be available and/or downloadable on other platforms is not known by us and is the responsibility of the respective operators of these platforms or software distribution channels." or "Ob die CWA danach noch auf anderen Plattformen heruntergeladen werden kann, ist uns nicht bekannt und liegt in der Verantwortung der jeweiligen Plattform-Betreiber." from the #ramp_down_stores FAQ entry

@larswmh

This PR removes the sentence: "The CWA will be available in the app stores of Apple and Google until May 31, 2023." or "Die Corona-Warn-App wird noch bis einschließlich 31. Mai 2023 in den Appstores von Apple und Google verfügbar sein." from the #ramp_down_stores FAQ entry

It seems that the PR leaves only that sentence and removes the following text :
"Whether the CWA will still be available and/or downloadable on other platforms is not known by us and is the responsibility of the respective operators of these platforms or software distribution channels."
So it looks to me like the change is correct although the description does not match the change.

@MikeMcC399

It seems that the PR leaves only that sentence and removes the following text :
"Whether the CWA will still be available and/or downloadable on other platforms is not known by us and is the responsibility of the respective operators of these platforms or software distribution channels."
So it looks to me like the change is correct although the description does not match the change.

Thanks for the information. The description should now be correct.


Handle new source element

Since Angular 4.0 there is a source element in xliff and xmb files.
It contains the template file and position, where the text is contained.
There shoUldale be an api to access it and it should be handled correctly.
Tiny translator should even show it.

closed with 0.4


[5.0] Use custom page name for Paginator

Resolving issue #8000

I think it's ok now.

Please squash to one commit when you're done.

Oh sorry, I did not read about the squash commit. :(

What is the use case for using a custom name for this?

It's necessary when you have to paginate two or more results in the same page, for example, on a site I'm building, I need to show two list of products, "Featured Products" and "Related Products", each list has their own pagination links, and of course they need a different "page" variable to avoid conflicts.
When I was looking for a solution, I found many other developers need this advantage, so I decide to make the pull request.
The issue is here #8000 but was closed without solution, and here is "the possible bug" https://github.com/illuminate/pagination/blob/master/PaginationServiceProvider.php#L21

Can you rebase and fix merge conflicts.

I think it's ready.

That's not correct. You've rebased against the wrong branch. We can't merge this.

You need to actually REBASE and force push. There MUST be only one commit in here.

OMG... I'm confused with "rebasing" things...

5.0, then force push. There must be exactly one commit here, not loads of them.

It's now correct?

Nearly, just squash and force push and this should be good. :)

@sahibalejandro No need for you to do anything, I'll get this sorted for you. :)

Merged into 5.1. Thank you @sahibalejandro.

Nice, thank you @GrahamCampbell, and sorry for the little mess.

I don't know, why I changed all the codes you have commit , but two paginations in one page didn't work, and what should i do to make it works?


Serious Bug: Crash when catching error in main.

This should not crash!
function f() {
	throw 42;
}

function main() {
	f();
}
except(e) {
	std::println(e);
}

Fixed! ```cpp
void thr0w(VmCore & core) {
const auto except = core.stack.pop();
if(auto addr = unwind(core.stack)) {
core.pc = *addr;
const auto jumpback = core.stack.pop_frame();
// These 2 were in the wrong order
core.stack.push(except);
core.stack.push_frame(jumpback, 1);
}
else {
throw std::runtime_error{"Unhandled Exception: " + cast::to_string(except, core.heap)};
}
}


Grammatical Corrections

There are minor grammatical errors that I have corrected to help with readability.
Line 31 - in this context, the preposition would be 'apply to' not 'apply for'.
Line 73 - on this line, the word necessary was missing an article, so I added 'the'. It is now consistent with the rest of the page.
Line 81 - the preposition was corrected, so 'in' was replaced with 'on'. I also added a comma after the word 'visit'.
when the English grammar is corrected, I found that web-based translators help keep the true meaning better.

Hey is everything okay with this change :)

We currently have a backlog. We will review these and cross check with our style guides and our copy editors, but it may take some time. Thank you for your patience.

Perfect, Thanks for clarifying this with me. I just wanted to make sure that I wasn't doing anything wrong 😊.


Something wrong with subscription_id in Invoice table

Hi!
In my project I'm using Free subscription plan with $0.00/mon. In pinax_stripe_invoice table I have a record like: id=1, amount_due=0.00, customer_id=1, subscription_id=1. If user change subscription plan for another one with fee, in this table I have same subscription_id for all records for this customer:
id=1, amount_due=0.00, customer_id=1, subscription_id=2
id=2, amount_due=9.99, customer_id=1, subscription_id=2
Same case in pinax_stripe_invoiceitem table. In payment history list I have one description for all Invoices. I think it must be different, in my case for id=1 subscription_id must be 1. I do nothing with this table in my project, just show it like a list, so no any chanegs on my side.

Same here.  The history of changes is lost. Does it make sense to have a new table with that info?


pinned packaging requirement may be outdated

pynetbox version
v7.3.3
NetBox version
v4.0.3
Python version
3.11
Steps to Reproduce
When using an automated dependency updater, requirement that pynetbox v7.3.3 not installing with packaging v24+ is causing problems resolving (and thus upgrading to v7.3.3). Is there an opportunity to remove the pin to below 24?
There is already a PR that  indicates success.
Expected Behavior
pynetbox should use the latest packaging.
Observed Behavior
pynetbox requires packaging<24.

Merged a fix for this


Tests

Are any available for this?

Only as examples at http://andreypopp.github.io/react-textarea-autosize/ not automated.

Feel free to contribute them, thing like huxley could be useful here.


Other dataset

@snatch59 I want to apply the cnn-svm-classifier to only extract features and saved to disk as matrix or excel sheet then feed it to another classifier than svm. My dataset have 8 classes with 25479 images. How can i pass this dataset to the code to extract features?.

@snatch59 I want to apply the cnn-svm-classifier to only extract features and saved to disk as matrix or excel sheet then feed it to another classifier than svm. My dataset have 8 classes with 25479 images. How can i pass this dataset to the code to extract features?.

我也有同样的疑问，你现在解决了吗？


Running on a standard CPU, is 1 frame per minute normal?!

Im trying to call call the rosservice with the topic image from gazebo, so "rosservice call /darknet_node/objects_in_scene "{}" ". This responds after about a minute.
I thought simple forwardpasses in Yolo2 would be about 30fps?

We generally see about 10-15 fps with YOLOv2 on a GPU, and 0.25 - 1 fps on CPU. Unfortunately, we've never seen performance on the order of 30 fps, but perhaps that might be a function of copying data from ROS messages into the detector and back? We haven't actually debugged the problem.
1 frame per minute does seem exceedingly slow, but if I had to guess, I might attribute the problem to the Gazebo simulation. Do you know how much CPU / Memory your simulation itself is consuming?
If you do have a performance improvements, we're happy to accept a PR. However, we will be deprecating our darknet detector going forward to future ROS releases (such as Melodic); we plan on continuing with the DRFCN detector though. If you're still interested in using darknet for future releases, I'd suggest checking out: https://github.com/leggedrobotics/darknet_ros

Thanks for the quick response to such a dumb question. I was just certain I'd heard 30fps would be easily achievable using anything.

No problem. Good luck!


TABLET - Android Samsung - mapping issues

When I open the link and zoom on the map using fingers on screen the map zooms then jams. I can not scroll across the page. If I hit the minimise button it does nothing. This was using Chrome and IE browsers.
When I then pick a route to view it will not change screen to that view, if I choose a location it too will not change.

Issue is a bug with pinch zoom on all tablet devices. Have resolved the issue to https://github.com/CanterburyRegionalCouncil/MetroInfoMaps/issues/33


Implement LevelLineChart

Ideally, we want the props to be somewhat consistent (or easy to map to) Metric.thresholds and MetricLevel so we can easily implement the metric-aware version of this chart.
The chart should assert that the number of levels is compatible with the number of thresholds (there should be one fewer threshold than level)
const levels: MetricLevel[] = [
  { id: "low", color: "green" },
  { id: "medium", color: "yellow" },
  { id: "high", color: "red" },
];

const thresholds = [10, 50];

<LineChartLevels
  timeseries={timeseries}
  xScale={xScale}
  yScale={yScale}
  levels={levels}
  thresholds={thresholds}
/>

Note: The non-metric aware version ended up being too simple, so we are going to close this and use #190 as reference for the metric-aware version.


audiobookshelf: 2.6.0 -> 2.7.0

Description of changes
https://github.com/advplyr/audiobookshelf/releases/tag/v2.7.0
Diff: https://github.com/advplyr/audiobookshelf/compare/v2.6.0...v2.7.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Upgraded my instance without problem.


Speed between spark.randomforset and mml.lightgbm

Speed between spark.randomforset and mml.lightgbm，which is faster?

@RainFung the doc states:
https://github.com/Azure/mmlspark/blob/master/docs/lightgbm.md
LightGBM on Spark is 10-30% faster than SparkML on the Higgs dataset, and achieves a 15% increase in AUC

It varies a lot from cluster to cluster and based on parameters though, and I haven't done any performance comparisons recently.
In general, I think a better comparison would be between lightgbm and xgboost (which also has a spark implementation).  I would try all 3 and see which one is best on your dataset, as performance could vary greatly on different datasets.
I've put the azure databricks and HDInsight notebooks here for the Higgs dataset in my branch (note I will not be checking these in):
https://github.com/imatiach-msft/mmlspark/tree/ilmat/higgs-notebooks/notebooks/higgs
You can get the Higgs csv dataset here:
https://github.com/guolinke/boosting_tree_benchmarks/tree/master/data
Specifically this link:
https://archive.ics.uci.edu/ml/datasets/HIGGS

Also note that Random Forests and Gradient Boosted Trees/Machines are different classes of algorithms. Another better comparison would be between mml.LightGBM and spark.GradientBoostedTrees.
Note that Gradient Boosted Trees are more expressive than Random Forests.
From the databricks blog:

Random Forests: Since each tree in a Random Forest is trained independently, multiple trees can be trained in parallel (in addition to the parallelization for single trees). MLlib does exactly that: A variable number of sub-trees are trained in parallel, where the number is optimized on each iteration based on memory constraints.
GBTs: Since GBTs must train one tree at a time, training is only parallelized at the single tree level.

and

Comparing the two methods, Random Forests are faster to train, but they often require deeper trees than GBTs to achieve the same error. GBTs can further reduce the error with each iteration, but they can begin to overfit (increase test error) after too many iterations. Random Forests do not overfit as easily, but their test error plateaus.

Thanks I'll try.


Fix runc

We need to use a propagation flag in order to have skydive seeing bind mounts made by the hosts
Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

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

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@matrohon is this still a Draft? Cna you please rebase and ensure it works with and without spire? I think we should consider merging it.

matrohon/skydive:0.16 is working fine with kind and runc, but I'm having troubles pulling this image with helm charts.
I'm using this script to deploy skydive:
#!/bin/bash

set -x

docker tag matrohon/skydive:nsm.0.16 networkservicemesh/skydive:master
docker save -o build/images/skydive.tar networkservicemesh/skydive:master
make $CLUSTER_RULES_PREFIX-skydive-load-images
make helm-delete-skydive
until ! $(kubectl --namespace nsm-system get pods | grep -q ^skydive* ); do echo "Wait for skydive to terminate"; sleep 5; done
make helm-install-skydive
set +x

@ivanayov , you might be interested.


docs: add tag v2.0.0 as pinned version

:tada: This PR is included in version 2.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Listing blocks in duplicate pages doesn't load properly

Describe the bug
when you 'duplicate' content with blocks, the blocks are duplicated but keep their original id.
When the page loads, calls to plone are made to querystringsearch with subrequests, whose subrequest id is the block id.
If I jump to the other page, where the block ids are the same, the call to plone for the other page's block fails because in the redux state there is already a subrequest with that id, so it gets that result .
To Reproduce

I create a page A containing a list
I duplicate the entire page, calling it B, with the content mode copy/paste function
I modify the list of page B by modifying the filters
I display page A by refreshing
I see page B... it has the same contents as A
I refresh the correct contents to appear
Returning to page A now the list of B appears (it is fixed by refreshing again)

@giuliaghisini I see how your solution should make sure that the subrequest ids are unique even if block ids are duplicated on different pages.
But, it also has me thinking: if I navigate from page A to B, wait 10 minutes, and then navigate back to A, wouldn't it be reasonable for me to expect the query to be fetched again? But if the old subrequest is still in the redux state, that won't happen, right? Your solution doesn't help here. So maybe a better solution would be to clear subrequests from the state when the user navigates to a different page. (Or maybe I'm missing something and that won't work or is too difficult.)

@davisagli what you think is correct, but.. there are situations in which having the redux state populated avoids unnecessary calls to the backend when you navigate from one page to another.. for example we have many sites where header and footer data are requested at the backend.. in these cases it is convenient to make a request to the backend only once, and not on every page change..

@davisagli afaict the query is still fetched everytime, see also the video below. I would still see possible benefits in clearing the subrequests, but maybe (depending on how the code is written) it is using old results while waiting for new results and that could have benefits too. I'm not sure about this still, but anyway I think this matter is out of the scope of this bugfix.
Listing-1.webm

@pnicolli oh yeah, that makes sense. it wasn't preventing fetching the request, it was just causing a flash of old results while it loads


GH-41871: [Python][C++] Fix pyarrow not working with -fvisibility=hidden

Rationale for this change

This change fixes an import error in the Python runtime when linked against an Apache Arrow installation that used -fvisibility=hidden.
By allowing use of -fvisibility=hidden there is a potential to make the required components for Python smaller
What changes are included in this PR?

Added explicit export for the two offending symbols
Are these changes tested?

Built locally and imported pyarrow without error
Are there any user-facing changes?

No

GitHub Issue: #41871

I'm surprised, are those really the only changes required?

For me locally these were the only two that prevented me from being able to import pyarrow. Maybe it is worth setting up in CI to target more platforms?

Assuming there is a real use of -fvisibility-hidden, we could add a Crossbow task for it, but first we should find out about the potential benefits.

Sounds good - I claim no expertise in this particular flag but am going off of the benefits outlined here:
https://gcc.gnu.org/wiki/Visibility
From what I saw locally, the libarrow.so size reduction dropped from 19 MB down 18, inline with the supposed 5-20% savings from that article.

@WillAyd Does this PR need updating or rebasing?

Just updated - from a local test still seems to work. I don't think the CI failures are related
Do you still think it is worth adding a Crossbow task for this? Not familiar with that setup but happy to look if that is the next step

If we want to support it, we probably want to test it somewhere on CI (do you aim to use it for Python wheels perhaps? cc @raulcd @jorisvandenbossche ).

I think it would be good to add for Python wheels. It does make the wheel a bit smaller, supposedly has runtime performance benefits, and I see SciPy / NumPy use hidden symbol visibility, at least with GNU compilers.
@rgommers might have some more insights

Hmm, then we can perhaps try adding those flags to ci/scripts/python_wheel_manylinux_build.sh?

I think it would be good to add for Python wheels. It does make the wheel a bit smaller, supposedly has runtime performance benefits, and I see SciPy / NumPy use hidden symbol visibility, at least with GNU compilers.

Yes indeed. We use it also with Clang on macOS, and with any other compilers that support the -fvisibility=hidden (C) and -fvisibility-inlines-hidden (C++) flags. In addition, we use a linker script to hide all symbols except PyInit_* from extension modules:
https://github.com/scipy/scipy/blob/76bf366486b0583dca789823878b80d610eb571b/meson.build#L131-L143
And there's a script to check the hiding is complete that we run in CI.
We originally started doing that after we got bug reports of symbol clashes for embedding use cases (https://github.com/scipy/scipy/pull/8451), and then expanded it for robustness and binary size reasons after we switched to Meson. Binary size gain is nice, but the robustness is even more important - symbol clashes are rare, but also very painful to resolve (IIRC we had one with GDAL in conda-forge right before we improved the hiding with Meson).

@github-actions crossbow submit -g wheel

Ok, judging by the Crossbow build failures above, it seems more symbols need explicit exporting.
Traceback (most recent call last):
  File "<string>", line 2, in <module>
  File "/Users/runner/work/crossbow/crossbow/test-env/lib/python3.10/site-packages/pyarrow/__init__.py", line 65, in <module>
    import pyarrow.lib as _lib
ImportError: dlopen(/Users/runner/work/crossbow/crossbow/test-env/lib/python3.10/site-packages/pyarrow/lib.cpython-310-darwin.so, 0x0002): symbol not found in flat namespace '__ZN5arrow7compute9IdenticalERKNS0_10ExpressionES3_'

$ c++filt _ZN5arrow7compute9IdenticalERKNS0_10ExpressionES3_
arrow::compute::Identical(arrow::compute::Expression const&, arrow::compute::Expression const&)

Hmm thanks @pitrou - will take a look

Are there any docker images that work with the python-maximal presets but don't require cuda? I'm guessing the symbol issue might be from something in the maximal presets (at least, I cannot reproduce this issue with the standard preset) but am having a hard time getting any type of cmake configuration to work with that

Your best bet is simply to reproduce one of those Docker jobs locally.

Usually I would look for the actual archery docker run ... command line in the build logs, for example:
https://github.com/ursacomputing/crossbow/actions/runs/10387577860/job/28761361168#step:7:1
Here, things are made a bit more complex by the fact that a first Docker run builds the wheel (the "Build wheel" step) and a second different Docker run runs the tests against that wheel (the "Test wheel" step).
(but perhaps it's enough to shell inside the "Build wheel" Docker run and try to Python-import the wheel there)

Was able to get this working with a local crossbow run. From what I could tell, the pattern was that for anything using the ARROW_FRIEND_EXPORT template, the associated definition still needed ARROW_EXPORT

Any chance we can do another crossbow run on this?

@github-actions crossbow submit -g wheel

Thanks @kou . I assume the macos wheel issues are unrelated (at least seeing them on other PRs)

Thanks @kou . I assume the macos wheel issues are unrelated (at least seeing them on other PRs)

Could you rebase? Those wheel jobs haven't failed on main for our nightly jobs for the last days and they shouldn't fail

No problem - just did it again, although I did update the branch yesterday as well...so maybe it is related?
Apologies in advance as I have also been doing merges from main instead of rebases. If the latter is preferred, will keep that in mind for any future PRs

@github-actions crossbow submit wheel-macos-*

Apologies in advance as I have also been doing merges from main instead of rebases. If the latter is preferred, will keep that in mind for any future PRs

I think that's completely fine :) as soon as it is updated. I missed that the branch was updated, sorry, I just triggered the macOS wheels jobs again but those are passing on main at the moment.

OK so I'm guessing it is related now. The pattern I saw before on Linux was that anything with an ARROW_FRIEND_EXPORT in its declaration needed ARROW_EXPORT in the definition.  My current guess is there must be something specific to macOS and what gets imported for parquet reading where that relationship between the macros is not maintained. Will see what I can find


Cmd+R as shortcut for sync

As suggested by @baybatu in #299, let's use Cmd+R as a shortcut for sync.

https://github.com/hackjutsu/Lepton/commit/a0ce21f1c3693012a498eeb501d90a18f00507ad


Open the door to more HW description

Today in the spec we can describe boards (via BSP pack) and devices (via Software Pack).
I think we induce some pack types without really defining them, that's why I still promote this proposal to introduce pack types:
https://github.com/Open-CMSIS-Pack/Open-CMSIS-Pack-Spec/issues/108
Then, in the project YML file we have definitions like:
  target-types:
    - type: B-U585I-IOT02A
      board: B-U585I-IOT02A

The tool picks the board definition from the BSP.
I think we may envision the use of various pack types to describe hardware.
So I think it may make sense to explicitly tell the tool which packs he can use when defining this target.
We may have something like:
  target-types:
    - type: B-U585I-IOT02A
      board: B-U585I-IOT02A
      target_packs:
         - pack: can be a BSP pack for a board definition
         - pack: can be a DFP pack for a device definition
         - pack: can be a pack dedicated to some hardware definitions (to be discussed)

The proposal is really to let the tool know about all the packs that relate to a target.
This should be restricted to some pack types (BSP, DFP, to be discussed).
As hardware is not as versatile as SW it probably makes sense to indicate the links to be considered.

As suggested by Evgueni : maybe we can deal with packs dependencies.

Ok, so based on your clarifications:

If I need another HW pack type then I can create in the spec another element at the same level as board and device
From the BSP pack I can create dependencies on these additional packs

@fred-r yes I suggest to close the ticket for now. I'm sure we need extension over time.

Reopening as the extension is needed now.

Can you explain the requirement next Tuesday?

Sorry it is unclear what is needed now.

Superseded by #144


Using A2DP Sink

I'm trying to set up my computer as a bluetooth speaker. I compiled & installed bluez-alsa and managed to connect my phone to it. Last three lines of bluealsa's output are:
bluealsa: ../../src/transport.c:548: State transition: 1 -> 2
bluealsa: ../../src/transport.c:94: Created new IO thread: A2DP Sink (SBC)
bluealsa: ../../src/io.c:330: Starting IO loop: A2DP Sink (SBC)`

Also hcitop shows constant flow of data - everything looks good.
The problem is that no sound is played back from the headphones connected to the computer (regular audio output e.g. with aplay or mpg123 works just fine).
What should I do to get the audio data from bluealsa played back via computer's sound card?

Figured it out. I needed to do the following:

add ~/.asoundrc file with correct interface, device & profile (just like in the Readme)
when a2dp connection is established "arecord -f cd | aplay" can be used to play back the stream using the default audio output

Yeap, BlueALSA tires to act as an extra audio card. There is no direct link between separate audio devices. One has to use extra program for redirecting audio signal.
For testing purposes I'm using ecasound sound processor, as follows:
$ ecasound -i alsa,phone -o alsa


Update release notes for CPLM005 release

Added improvements & bug fixes that come along with the new CPLM release

@ekremsekerci, could you please give me write access so I can make a few minor language edits?

@ekremsekerci, could you please give me write access so I can make a few minor language edits?

Just added you as a collaborator

Thanks!

@ekremsekerci Was the 2FA link added somewhere in the Metrics app?


Universal_V3?

Hi,
I noticed that there's no Universal_V3 model. Is this because V3 is too small to do well in the generalized case? Or is it possible it does quite well but simply hasn't been tried / trained yet?
Was Universal_V3 likewise trained for 2,500K steps? How long did this take for such a large dataset and was it also done with 2 V100 GPUs?
Couldn't find the info on the paper, so hope @jik876 can provide the answers!

need universal v3 pretrained model too！！！


feat: nexo import support

Adding Nexo support by parsing the CSV file of transactions.
closes #208

Sure, give me one more week ;)
Le mer. 21 août 2024, 14:24, OxyFlax @.***> a écrit :

@lasconic https://github.com/lasconic do you have any time to review
this PR and eventually merge it?
—
Reply to this email directly, view it on GitHub
https://github.com/lasconic/finary_uapi/pull/209#issuecomment-2301924122,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABZGH7VPP3TLWPTGTGDR2DZSSBG7AVCNFSM6AAAAABLTD6QY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBRHEZDIMJSGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

See https://github.com/lasconic/finary_uapi/pull/211

@all-contributors please add @OxyFlax for code


Code reviews

In one of the lectorials, you mentioned that we can request code reviews from you (or other professors). Before requesting this, is there anything we should prepare in particular for the session, apart from a list of currently supported features and how to run the code?

It would be good if the code actually runs on at least one machine in order to demonstrate it ;)
Automated test cases also help to speed up the process and make it more enjoyable: I might still come up with some trickier input on the fly, but it's easier if such a try involves changing a file/source and rerunning the tests and not manually rebuilding the system lib by lib and entering the same information/code on each run.


Keep expanded folder after renaming

Fixes #44230
Can't pinpoint where refresh is ending :( Feel free to close this.

Closing as #44362 is a better implementation.


why was issue #23819 closed?  It is still a problem

https://github.com/ytdl-org/youtube-dl/issues/23819

Script iniciado en 2020-02-01 23:48:35-07:00 [TERM="xterm" TTY="/dev/pts/2" COLUMNS="80" LINES="32"]
]0;stephen@trifid: ~/Music/tools/dlsstephen@trifid:~/Music/tools/dls$ ../grab-Chrome-videos.sh ../junk.htm
QsnZMee16lk
1  hash is QsnZMee16lk
downloading Bob Seger - Shame On The Moon

for x in $@
youtube-dl -x --verbose --audio-format mp3 --audio-quality 0 --no-playlist -o '%(title)s.%(ext)s' 'https://youtube.com/watch?v=QsnZMee16lk'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-x', '--verbose', '--audio-format', 'mp3', '--audio-quality', '0', '--no-playlist', '-o', '%(title)s.%(ext)s', 'https://youtube.com/watch?v=QsnZMee16lk']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.01.01
[debug] Python version 3.7.3 (CPython) - Linux-4.19.0-6-amd64-x86_64-with-debian-10.2
[debug] exe versions: ffmpeg 4.1.4-1, ffprobe 4.1.4-1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] QsnZMee16lk: Downloading webpage
[youtube] QsnZMee16lk: Downloading video info webpage
[youtube] Downloading just video QsnZMee16lk because of --no-playlist
[youtube] {18} signature length 106, html5 player vfl1GpCbm
[youtube] QsnZMee16lk: Downloading player https://www.youtube.com/yts/jsbin/player_ias-vfl1GpCbm/en_US/base.js
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/youtube.py", line 1384, in _decrypt_signature
video_id, player_url, s
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/youtube.py", line 1297, in _extract_signature_function
cache_res = res(test_string)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/youtube.py", line 1360, in
return lambda s: initial_function([s])
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/jsinterp.py", line 258, in resf
res, abort = self.interpret_statement(stmt, local_vars)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/jsinterp.py", line 56, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/jsinterp.py", line 92, in interpret_expression
m.group('expr'), local_vars, allow_recursion - 1)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/jsinterp.py", line 189, in interpret_expression
m.group('x'), local_vars, allow_recursion - 1)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/jsinterp.py", line 56, in interpret_statement
v = self.interpret_expression(expr, local_vars, allow_recursion)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/jsinterp.py", line 211, in interpret_expression
raise ExtractorError('Unsupported JS expression %r' % expr)
youtube_dl.utils.ExtractorError: Unsupported JS expression '[1376500448,'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError("Unsupported JS expression '[1376500448,'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.")); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.


docs: fix typos

🙂

My pleasure! I cloned every repository in the organization and put it in a folder, then from the root I ran cpell on every README, src/**, docs/**, and tests/** file from every repository. This output ~700 suspected typos, of which I manually filtered through to find the real ones. There was 100 here, 1 in nestjs, and a few in both docs and docs-v3 although I learned after that those weren't used anymore.
I was bored :3 Thanks for making such a great orm :))


install error

when i use "yo jhipster-entity-audit", it throw error like "events.js:141
throw er; // Unhandled 'error' event
^
TypeError: jhipsterFunc.copyTemplate is not a function
at null. (/usr/local/lib/node_modules/generator-jhipster-entity-audit/generators/app/index.js:126:24)
at Array.forEach (native)
at copyFiles (/usr/local/lib/node_modules/generator-jhipster-entity-audit/generators/app/index.js:125:15)
at module.exports.yeoman.generators.Base.extend.writing.writeBaseFiles (/usr/local/lib/node_modules/generator-jhipster-entity-audit/generators/app/index.js:146:12)
at /usr/local/lib/node_modules/generator-jhipster-entity-audit/node_modules/yeoman-generator/lib/base.js:430:16
at processImmediate [as _immediateCallback] (timers.js:383:17)"

jhipster version is 2.26.2

It's because the module generator-jhipster-entity-audit requires JHipster v2.26.3 or v2.27.0, which is not out yet. You have 2 solutions :
wait
or
use the latest master branch of JHipster and npm link it(like all JHipster contributors)


Update sbt-bloop to 1.4.3

Updates ch.epfl.scala:sbt-bloop from 1.4.0-RC1-190-ef7d8dba to 1.4.3.
GitHub Release Notes
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-bloop" } ]

labels: sbt-plugin-update, semver-pre-release

Codecov Report

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

@@           Coverage Diff           @@
##           master     #295   +/-   ##
=======================================
  Coverage   41.83%   41.83%
=======================================
  Files          47       47
  Lines        1702     1702
  Branches       54       54
=======================================
  Hits          712      712
  Misses        990      990

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 fe361a4...1ff17cd. Read the comment docs.


Whats the new way of filtering posts by tags?

Hey,
I'm trying to upgrade from wpapi 0.9 to latest.
previously I've been using filters for filtering posts by tags.
I have a use case where I need to query by tags and ".tags()" don't fully fit.
currently, tags return posts with "tag1" or "tag2"
and I need a way to filter as such:
include: "tag1" and "tag2" and ("tag3" or "tag4")
exclude: "tag5"
with the previous API that would look like this:
this.wordpress
.posts()
.filter({
    tag_slug__and,
    tag_slug__in,
    tag__not_in
})

whats the right way to do it now?

The recommended way to filter by taxonomy term slugs is to fetch the terms separately, note their ID, and query with IDs as usual: .tags( id1, id2 ). This is recommended because tag IDs are a more stable indicator than slugs, and all relationships in the API are ID-driven.
The filter query parameter can be reinstated however by installing the plugin at github.com/WP-API/rest-filter; if you use auto-discovery on a site with that plugin enabled then the bound client will provide the old .filter method.
Hope this helps! Closing the issue as it doesn't seem to represent a bug; frustrating though it may be, that's how the core API is designed to work.
(Providing a way to do AND relations, however, is still on the roadmap and is being tracked in the core WP trac ticketing system)

I missed the complexity of the query you want to make; for that usage yes, I think the rest-filter plugin will be required. However for "not in," you should be able to use the tags_exclude parameter

thnx for the quick response.
I've tried your suggestion following the docs but came across an error.
seems like the filter function not implemented or am i doing something wrong?
WPAPI.discover( 'http://mysite.com' )
    .then(function(wp) {
        wp.posts().filter({
            category_name: 'islands',
            tag: [ 'clouds', 'sunset' ]
        })...
    });

this lines of code thorw an error:
"TypeError: wp.posts(...).filter is not a function"

Just to check -- you do have the rest-filter plugin installed and active?

yes, I did but even if it's not installed I think filter supposed to be available and the answer from the server will fail but it, not the case, I get js error that filter function not found.

The filter method is only available if you use auto-discovery: https://github.com/WP-API/node-wpapi/blob/master/README.md#other-filters it's available as a mixin for use with custom endpoint handlers, but the out-of-the-box WP client methods this library provides are based around a current snapshot of the functionality of the core REST API, and unfortunately this does not include filter. I'd be open to making an easier method to add mixins to already-configured route handlers, but I'd want to understand why filter isn't available for you after auto-discovery, first. If you look at the output of yoursite.com/wp-json, do you see "filter" included in the list of parameters on the posts collection?

are you sure filter available when you use auto-discovery? I get an error that filters function not available in auto-discovery and nothing is been sent...

@igoldny it should be available, yes; please see my question above about whether you see it enumerated in the output from the root wp-json output. If it's there, then the fact filter does not work is a bug, if it's not there then the bug is in the rest-filter plugin.

@kadamwhite yes looks like filter is there when i'm using "category_name" instead of category with id
i get this output:
fxapi [17/Jul/2017:12:49:20 +0000] "GET /wp-json/wp/v2/posts?_embed=true&filter%5Bcategory_name%5D=forecasts&page=1&per_page=6 HTTP/1.1" 200 58673 "-" "node-superagent/3.5.2"

but can't test that with auto-discovery coz it's not working here
same query with auto-discovery with not sent any request to wp and i will get error that filter function not working.
wpapi.discover('http://wp.fxempiredev.com')
            .then(function(wp) {
                return wp.posts().filter({
                    category_name: 'news'
                })
            });

TypeError: wp.posts(...).filter is not a function

Hi @kadamwhite. My name is Ariel, I work with @igoldny.
He asked me to take a look at the WordPress instance. To your question, filter doesn't return as a possible parameter of the posts collection.
I'm a newbie when it comes to WordPress code, however I'm happy to try to set up a new instance in a different way. Did you by chance manage to get this plugin running? If so, with which version of WordPress?
Thanks in advance for the help.


Refactor interactive user prompting infrastructure

Context
The PR refactors UserInputHandler so that it could potentially be used as a public API to allow build logic to ask the user questions interactively.
The API groups a series of related questions into a batch, or "interaction", and returns a provider that lazily prompts the user for answers and returns the result of the interaction. Here's an example:
val provider = userInputHandler.askUser { questions ->
    // Lambda can use any of the methods that were previously on `UserInputHandler`
    val value1 = questions.askQuestion("thing?", "default")
    val value2 = questions.selectOption("number?", listOf(1, 2, 3), 2)
    SomeData(value1, value2)
}

The provider can be used wired into a Property or other lazy type.
This PR does not attempt to deal with concurrency (the existing infrastructure did not do this either). It does not require that the usage be declared, but this should probably happen.
Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@tresat thanks for the review

@bot-gradle test and merge

The merge queue build has started. Click here to see all failures if any.


Don't allow platform in cask descriptions.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

All casks are for macOS, so this is useless information.

@reitermarkus in the spirit of this PR, shouldn't macOS be removed from VALID_LOWERCASE_WORDS (at the start of desc_helper.rb)? (Interestingly, it used to be the only item there, before #8333!)

It's still allowed in formula descriptions.


added mouseover functions as suggested by prof during sprint 1 demo

added mouseover functions for review article page. hovering the mouse over the icons on the page now tell what review functions each perform. this was a recommendation from the professor during the sprint 1 demo.

👍


Moving Object to Untracked Sub-Package

When moving an object to an untracked sub-package - the changes are incorrectly tracked.

Create a package PACKAGE_A with sub-package PACKAGE_A_SUB
Add TABLE_1 to PACKAGE_A
Created an online Repo for PACKAGE_A with "Ignore Sub-Packages"
Stage, push, pull to target system
Now, move TABLE_1 to PACKAGE_A_SUB
Do a diff and notice the change
Stage and notice it intends to move TABLE_1 to PACKAGE_A_SUB

Our expectation would be that TABLE_1 is deleted since it is not tracked.

There may be other issues here - I explicitly said "Ignore sub-packages" when creating online Repo for ZPMR_DATE_SRV but a "Pull" is trying to pull in the sub-package itself (DEVC ZPMR_DATE_SRV_DB) as well as objects in the parent package ZPMR_DATE:

Is it possible to make this repo available publicly? If no, can you create a test repo please?


Revert "run_interop_matrix.py speedup"

Reverts grpc/grpc#15629
Looks like this PR introduces a breakage in interop_matrix suite - https://source.cloud.google.com/results/invocations/314e699c-c083-495f-ab09-4c286fb53845/targets
I have a hard time understanding why things are suddenly broken - the PR only changes the way the docker images are downloaded, it doesn't change anything about the way the tests are run.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Maybe because too many jobs running at the same time? How about increasing individual docker job timeout?


Potential typo in "Scaling Graph Neural Networks" notebook

❓ Questions & Help
Hi, in the Scaling Graph Neural Networks when we're iterating over each mini-batch of the data, shouldn't we pass the current batch to the model inside the loop? I.e., "model(sub_data.x, sub_data.edge_index)" instead of "model(data.x, data.edge_index)", because now it seems even though we're iterating over the mini-batches we're still passing the whole dataset to the model each time.

Please let me know if I'm missing something. Thanks!

Indeed. Thanks for reporting :) This is now fixed.


Bug Fixed

Fixed a bug regarding wrong installation command to start monogdb service in READme documentation.
Fixed a bug regarding change of class for active toggled button.

Hi @obliviateandsurrender ... thank you for your PR. But you have not followed the Best practices.

Link your PR to a proper issue.
Add a proper commit message
Squash your commits into one
No proper PR title
No proper PR description

Read through all the links mentioned above please. Especially this.


Adding nimbus as a screenshot tool

Checklist

[x] I made something good today 💐.
[X] I put links in the correct format: [Tool](link) — description.
[X] My description starts with the lower-case letter and ends with the full stop.
[X] I put the tool in the alphabetical order.

Optional

[X] I added the appropriate labels: free, open-source or mac.
[X] I put the tool only to one category.
[x] I followed Contribution Guidelines.

thanks for adding )))


real bbox of text

In Browsers a call to getBBox() returns a box which is never the most tight box around the symbols. Instead they provide a box which would sorround any letter + accent.
The current bbox feature which can be used with getPath().getBoundingBox() however returns a tight box. While this is suitable in many cases, other use cases need the browser behavior instead.
So would it be possible to include a method which returns the bounding box of text like the Browser do?
e.g.
var box = font.getBBox(text, x, y, fontSize)

I found this ressources that talk about min/max values of the font which might be useful:
https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6head.html
https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6hhea.html
https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6hmtx.html
My usecase is the svgdom project which enables svg.js to run on the server. Among other things, this should provide the right results for textLength and bounding boxes. The text bbox is the only missing part. See also this stackoverflow question:  http://stackoverflow.com/questions/43140096/reproduce-bounding-box-of-text-in-browsers
Last thing to say: Thanks for your awesome library!
//EDIT: Actually it would be enough to add the lineGap metric. This way anyone could calculate the height of the bbox as linespace = height = ascent - descent + linegap (taken from https://www.freetype.org/freetype2/docs/glyphs/glyphs-3.html)

+1 for this

There is an entry for lineGap in the hhea table:
font.tables.hhea.lineGap

@swallace21 was it recently added? If not, why does it take 3 years for someone to answer? :D - no offense though :). I went on with fontkit in this particular case


go-ethereum: 1.14.6 -> 1.14.8

Release notes: https://github.com/ethereum/go-ethereum/releases/tag/v1.14.8
Note that cmd/p2psim was removed in https://github.com/ethereum/go-ethereum/pull/30250
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 334768 run on aarch64-darwin 1

3 packages built:

go-ethereum
go-ethereum.clef
go-ethereum.geth


[Vitals] Cannot remove registered vital

[Actual Results]: Nothing hapens
[Expected results]: The registered vital should be removed
[Steps to reproduce]: Vitals >  Search (patient with registered vitals) > Remove

 Resolve Pending Issues/Bugs on POC

Closed in #343.


False Negative on Conventional PR?

I'm using Conventional PR to prevent commits with bad messages to be merged to the main branch. I'm using the default conventional commit pattern
^(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test){1}(\([\w\-\.]+\))?(!)?: ([\w ])+([\s\S]*)
However, it seems commit messages like test commit is still considered to be valid. It seems like the conventional PR doesn't filter commit messages correctly.

Thanks for filing an issue.
The pattern seems fine, can you post the action workflow file here?

Sure, here you go
name: Check PR semantics

on:
  pull_request_target:

jobs:
  cpr:
    runs-on: ubuntu-latest
    steps:
      - name: Check PR semantics
        uses: Namchee/conventional-pr@latest
        with:
          access_token: ${{ secrets.ACCESS_TOKEN }}
          title_pattern: ^(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test){1}(\([\w\-\.]+\))?(!)?: ([\w ])+([\s\S]*)

You're using the wrong option, commit message is controlled by the commit_pattern option, not the title_pattern.

@Namchee, changed the option and it worked like a charm! Thanks!

/add-knowledge

The summary is so bad here
{
    "issue_number": 26,
    "problem": "The user is experiencing a false negative on a conventional PR.",
    "solution": "The user needs to change the option from `title_pattern` to `commit_pattern` in the action workflow file. After making this change, the issue is resolved."
}

The problem part does not explain the detail of the issue.

Direct result from ChatGPT
Problem: The user @Namchee is experiencing an issue with Conventional PR, where commits with messages like "test commit" are not being filtered correctly despite using the default conventional commit pattern.

Solution: @Cristopher identifies the problem as a configuration issue and suggests changing the option from `title_pattern` to `commit_pattern` in the action workflow file. After making this change, the issue is resolved, and the Conventional PR now filters commit messages correctly.

Seems to be good enough. However, it doesn't tell us the core problems, which are misconfigured workflow.

Moreover, the runner seems to be too aggressive. We need to re-think the target events.

WIll try on another issue


Add Wordle Game

Description
Wordle is a word guessing game where the player has to guess a 5 letter word.
Player is given 6 guesses.
How-To-play

Computer generates a random word which the player has to guess.
Player makes a 5 letter word guess.
Then after enter, the square tiles change its color according to the following color coding scheme:
Correct letter with correct position - GREEN
Correct letter with wrong position - YELLOW
Wrong letter - GREY
Keyboard keys also change their color according to same color scheme(after a guess is made).
If the player guesses the correct word, he/she receives a congratulations message.
If the player cannot make the guess and exhausts all the guesses, he/she loses the game.

Please assign me this issue to work on  for GSSoc'23

Hi @ishita80 ,I assigned this issue to you. Please complete the work within a week and go through the readme and contributing.md for successful merge.

i would want to work on this issue. i am a gssoc 23 contributor. please assign this issue to me @panwar8279


Great tutorial on how to import geodataframe to spatialite database

Thanks for this - first time using spatialite and I find this extremely helpful.
I am now running the executemany portion and my tuple length is 230k - it has been running awhile now (just at 1hr mark) - is this normal?
Also noticed ALOT of new triggers, index, views being created - is this the mod at work to enable the spatial queries?
Just trying to understand what is going on under the hood.

I'm afraid I can't help you that much. I've always worked with tiny datasets in SpatiaLite.
If the query takes a lot of time, I would try these 2 things:

work with a smaller dataset, and maybe performing the same query in batches.
work with an in-memory SQLite database.

I am defintely not an expert in SpatiaLite, but I read here that it defines a few triggers (and I guess also views and indexes) in order to compute and verify geometries.
If you are stuck, maybe try reading a few questions on gis.stackexchange.

No problem! Found a solution but thanks anyway! If anyone is wondering in future  when you use this method:
Create an index on some_id that you intend to use in your Update/Set/Where query and you should see significant improvement to your query.
For reference:
Without index - 1000points: 42 secs
With index - 230k points: 3secs


Feature: Information Source Encoding Types

Currently information sources are always represented as special discrete parameters.
This is might be appropriate e.g. for most multi-fidelity use cases.
When information sources are not ordered (e.g. multiple dataset) it might not be desired to encode them as (0, 1, 2, ...). This encoding is used to train the model, but some models might infer dataset 0 is more similar to dataset 1 than to dataset 2.
Therefore this PR proposes to separate encoding and source indices of information source parameters.
Description of changes:

Add ordinal, normal and discrete information source parameters
Separate source encoding and indices in multisource userfunction and multi source optimizer
Fix userfunction, notebook and tests appropriately

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

Codecov Report

Merging #191 into master will decrease coverage by 0.32%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
- Coverage   89.52%   89.19%   -0.33%
==========================================
  Files         105      106       +1
  Lines        2855     2907      +52
  Branches      292      298       +6
==========================================
+ Hits         2556     2593      +37
- Misses        247      259      +12
- Partials       52       55       +3

Impacted Files
Coverage Δ

emukit/core/discrete_parameter.py
86.66% <ø> (-1.22%)
:arrow_down:

emukit/core/optimization/acquisition_optimizer.py
84.84% <0%> (ø)
:arrow_up:

emukit/test_functions/multi_fidelity/park.py
100% <100%> (ø)
:arrow_up:

emukit/test_functions/multi_fidelity/hartmann.py
100% <100%> (ø)
:arrow_up:

...optimization/multi_source_acquisition_optimizer.py
90.9% <100%> (+0.28%)
:arrow_up:

emukit/test_functions/multi_fidelity/currin.py
100% <100%> (ø)
:arrow_up:

emukit/test_functions/multi_fidelity/branin.py
100% <100%> (ø)
:arrow_up:

emukit/core/parameter_space.py
97.26% <100%> (+0.03%)
:arrow_up:

emukit/core/__init__.py
100% <100%> (ø)
:arrow_up:

emukit/test_functions/multi_fidelity/borehole.py
100% <100%> (ø)
:arrow_up:

... and 6 more

Continue to review full report at Codecov.

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

There might be small comments here and there, but we'll bother about that later. For now I'd like to discuss the whole premise here.

Yes, I just posted the version I'm currently working with as a suggestion.
Other designs might fit better to the library.

Do we actually need all these parameter types? That starts to feel very specific, to the point when I am not sure anymore if this is really necessary.

Definitly, I'm also not happy with all these specific classes.
First I thought about a information source wrapper, which mostly leads through the wrapped parameters attributes (via getattr, setattr). I cancelled this approach mainly because of two reasons:

I was not sure how this affects the type testings (especially isinstance). 2. You would have to transform back from the model encoded values to the parameter values, which is currently not available in a generalized api and requires individual code  (see source_indices methods in the code).

The only reason InfoSourceParam exists (to my understanding) is so that we could distinguish it among other parameters in multi-fidelity setting. Perhaps a different approach could be used in the library, when we distinguish these params by name (just for example), and then just use discrete or cateogorical params as info source params as necessary? On one hand, it seems less complex, on the other hand, we might get into dangerous string matching before long with this.

In general I strongly prefer typed solutions against matching some magic-number or magic-string.
A pythonic solution would use duck-typing to determine a objects purpose, I think.
That said, another solution would be to not having any source-type or -name at all, but let the user
specify one or multiple parameters by instance or name when calling a multi fidelity/source function.

@dekuenstle @marpulli @mmahsereci @javiergonzalezh please offer opinions!

Maybe we can just input the encoding into the information source parameter object:
class InformationSourceParam(CategoricalParameter):
    def __init__(self, encoding: Encoding):
        if encoding is None:
            # default the encoding to ordinal - I don't know how it knows the number of information sources here though
        CategoricalParameter.__init__(self, 'source', encoding)

    def source_indices(self, values: np.ndarray) -> np.ndarray:
        """ See InformationSourceParameter.source_indices """
        return np.asarray([self.encoding.get_category(row) for row in self.round(values)])

    @property
    def n_sources(self) -> int:
        """ Number of sources. """
        return self.encodings.shape[0]

Then in something like MultiInformationSourceEntropySearch where you check what type of InformationSourceParameter you have, you could check the type of the encoding object or just assume the target is always to last information source.
This doesn't deal with the DiscreteInformationSourceParameter but I'm not 100% convinced that class is necessary and if it is, could we use a different Encoding class to do the same thing?
Does this make sense?

@marpulli In think almost every parameter type could be used to represent multi-information sources. You could also think about e.g. relative dataset size as a continuous fidelity measurement. I'm not saying emukit has to support all the cases right now, but it would be nice if it's easy to extend.

Sorry for the delay in replying.
I think that continuous vs discrete information sources need different enough treatment that they can be separate classes. For instance, the optimization procedure is likely to be different and evaluating the objective function can be represented as a list of function for discrete information sources but not for continuous.
All the discrete/categorical information source parameters just refer to an index into a set of information sources. With the current implementation, if the model needs this index to be encoded in a particular way, it can be done in the model itself or in the model wrapper. I see the convenience of not having to encode this by hand in the model but it would be nice to have one class that represents all these so that if we add different types of encodings of categorical variables, we don't need to add a new information source parameter class.

Thanks for your answer @marpulli , it helped me sorting thoughts about the topic:
I have cases where not only the model but also the optimizer requires specific encoding.
A viable solution for my case might be to replace the information source in the parameter space with another parameter and write a wrapper for the user function which transforms the other parameter to the information source. Others might only require a wrapper for the model, like you've proposed.
In practice I've realized, that the approach of this PR currently doesn't work completely. E.g. the context handling (using GPyOpt Context Handler) cannot be used for one-hot-parameters without a bunch of specific glue code.
As a conclusion:
Continuous parameters might already be usable with the continuous parameter, if special handling is required it can be done through the parameter index. Anyway continuous and discrete information sources require separater handling. Discrete information sources can be improved by exchangeable encoding. However this requires features which have to be implemented before (e.g. DiscreteParameter uses a Encoding object like CategoricalParameter, Context handling works for all encodings). Therefore I'll close this PR for now.


cgroups: cannot found cgroup mount destination: unknown"

What happened:
Executing:

kubectl apply -f https://k8s.io/examples/application/deployment.yaml
kubectl get pods

nginx-deployment-67594d6bf6-2b2r7   0/1       ContainerCreating   0          24s
nginx-deployment-67594d6bf6-fgrsm   0/1       ContainerCreating   0          24s

It keeps in state "ContainerCreating" and the logs show:
E0921 12:26:15.586713    7538 kuberuntime_manager.go:646] createPodSandbox for pod "nginx-deployment-67594d6bf6-fgrsm_default(f286ef84-bd96-11e8-9acd-12eb73fcb3ee)" failed: rpc error: code = Unknown desc = failed to start sandbox container for pod "nginx-deployment-67594d6bf6-fgrsm": Error response from daemon: cgroups: cannot found cgroup mount destination: unknown
E0921 12:26:15.586811    7538 pod_workers.go:186] Error syncing pod f286ef84-bd96-11e8-9acd-12eb73fcb3ee ("nginx-deployment-67594d6bf6-fgrsm_default(f286ef84-bd96-11e8-9acd-12eb73fcb3ee)"), skipping: failed to "CreatePodSandbox" for "nginx-deployment-67594d6bf6-fgrsm_default(f286ef84-bd96-11e8-9acd-12eb73fcb3ee)" with CreatePodSandboxError: "CreatePodSandbox for pod \"nginx-deployment-67594d6bf6-fgrsm_default(f286ef84-bd96-11e8-9acd-12eb73fcb3ee)\" failed: rpc error: code = Unknown desc = failed to start sandbox container for pod \"nginx-deployment-67594d6bf6-fgrsm\": Error response from daemon: cgroups: cannot found cgroup mount destination: unknown"
W0921 12:26:29.358565    7538 kubelet_getters.go:264] Path "/var/lib/kubelet/pods/f287ca39-bd96-11e8-9acd-12eb73fcb3ee/volumes" does not exist
W0921 12:26:49.869332    7538 kubelet_getters.go:264] Path "/var/lib/kubelet/pods/f287ca39-bd96-11e8-9acd-12eb73fcb3ee/volumes" does not exist"

What you expected to happen:
The pod is succesfully deployed.
How to reproduce it (as minimally and precisely as possible):

Deploy bosh-lite
Deploy v.0.21.0 from kubo
In the worker

mkdir /sys/fs/cgroup/cpu,cpuacct
mount -t cgroup -o cpu,cpuacct none /sys/fs/cgroup/cpu,cpuacct

vim /etc/sysctl.conf
fs.inotify.max_user_watches=1048576
sysctl -p

mkdir /sys/fs/cgroup/systemd
mount -t cgroup -o none,name=systemd cgroup /sys/fs/cgroup/systemd

Try to deploy a pod

Anything else we need to know?:
I have tried https://github.com/voidlinux/void-packages/issues/9811#issuecomment-381917614 in the vm but it does not work.
From the comments it leads to something with the docker version but I can not say.

Thank you for the issue @rauizab.  I'm closing this out due to inactivity.  Please check out the latest v0.24 release if the problem persists.  If it continues after the upgrade please reopen the issue.


lua sandbox raises error

j.builders.runtimes.lua.sandbox(reset=True,zhub_client=zhub_client, flist_create=True,merge_base_flist="tf-autobuilder/threefoldtech-jumpscaleX-development.flist")
  File "/sandbox/lib/jumpscale/Jumpscale/builder/system/BuilderBaseClass.py", line 166, in wrapper_action
    res = self.apply_method(func, kwargs)
  File "/sandbox/lib/jumpscale/Jumpscale/builder/system/BuilderBaseClass.py", line 93, in apply_method
    return func(**kwargs)
  File "/sandbox/lib/jumpscale/Jumpscale/builder/runtimes/BuilderLua.py", line 223, in sandbox
    j.builders.web.openresty.sandbox(reset=reset)
  File "/sandbox/lib/jumpscale/Jumpscale/builder/system/BuilderBaseClass.py", line 166, in wrapper_action
    res = self.apply_method(func, kwargs)
  File "/sandbox/lib/jumpscale/Jumpscale/builder/system/BuilderBaseClass.py", line 93, in apply_method
    return func(**kwargs)
  File "/sandbox/lib/jumpscale/Jumpscale/builder/web/BuilderOpenResty.py", line 127, in sandbox
    self._copy(dir_src, dir_dest)
  File "/sandbox/lib/jumpscale/Jumpscale/builder/system/BuilderBaseClass.py", line 416, in _copy
    createdir=True,
  File "/sandbox/lib/jumpscale/Jumpscale/builder/tools/BuilderTools.py", line 115, in copyTree
    createdir=createdir,
  File "/sandbox/lib/jumpscale/Jumpscale/sal/fs/SystemFS.py", line 265, in copyDirTree
    if ":" not in src and j.sal.fs.isDir(src):
  File "/sandbox/lib/jumpscale/Jumpscale/sal/fs/SystemFSDecorators.py", line 154, in wrapper
    raise ValueError(msg)
ValueError: Argument path in j.sal.fs.isDir expects an existing path value! /sandbox/bin/resty does not exist.

probably part of creating full flist for JSX 10.0.1

needs https://github.com/threefoldtech/jumpscaleX/issues/683

verified  on  development_builders


feat(notifications): Implement POST /subscription V2 API

PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.
What is the current behavior?

Issue Number: #3044
What is the new behavior?
Per https://app.swaggerhub.com/apis-docs/EdgeXFoundry1/support-notifications/2.x#/default/post_subscription, implemented POST /subscription V2 API.
Close #3044
Does this PR introduce a breaking change?

[ ] Yes
[x] No

New Imports

[ ] Yes
[x] No

Specific Instructions
This PR depends on https://github.com/edgexfoundry/go-mod-core-contracts/pull/439
Other information

rebased

@FelixTing , there are compiler errors in the unit tests.

bumped go-mod-core-contracts module from v2.0.0-dev.13 to v2.0.0-dev.17


TypeError: Rdp.registerTabActor(...) is undefined

With WebSocket Monitor 0.6.2 and Nightly 2016-06-06, I get the following:
TypeError: Rdp.registerTabActor(...) is undefined

and the panel does not load.

The fix goes to firebug.sdk
https://github.com/firebug/firebug.sdk/commit/bd43add69503dd8b421f8768b2d048bae9642271
(new module path for ActorRegistryFront)
Will be fixed in WebSocket Monitor 0.6.3
Honza

0.6.3 is on AMO (waiting for review ;-)
Please let me know if the problem is fixed for you now.
Honza

Great, that seems to fix it. Review approved on AMO.

Thanks!


The testcafe/testcafe Docker image contains the old npm version 6.14.15

What is your Scenario?
Running tests in the testcafe/testcafe Docker image.
What is the Current behavior?
The Docker image contains<EMAIL_ADDRESS>which has limited support for the package-lock v2 format.
What is the Expected behavior?
The Docker image contains recent stable version of npm with package-lock v2 support.
What is your public website URL? (or attach your complete example)
N/A
What is your TestCafe test code?
N/A
Your complete configuration file
No response
Your complete test report
No response
Screenshots
No response
Steps to Reproduce

Run command:

docker run --rm -it --entrypoint /bin/sh testcafe/testcafe

Enter command:

$ node --version && npm --version

TestCafe version
2.3.0
Node.js version
No response
Command-line arguments
N/A
Browser name(s) and version(s)
No response
Platform(s) and version(s)
No response
Other
No response

As far as I can tell, the issue is caused by version mismatch between the apk package and npm:
alpine:3.11: npm package 12.22.6-r0<EMAIL_ADDRESS>alpine:edge: npm package 9.3.1-r0<EMAIL_ADDRESS>The Dockerfile references multiple package repositories, including 3.11/main. Thus the package manager installs "the latest" version 12.22.6, which in turn installs outdated npm.