Query remote targets for label values

Signed-off-by: Mark Knapp<EMAIL_ADDRESS>This is hopefully to address https://github.com/prometheus/prometheus/issues/3351 to allow for getting labelvalues from remote read targets.
Seems there's been several attempts at this before, but it seems they were abandoned/unfinished/not ideal. This is my first real-world attempt at using protobuf, largely copied/pasted from other examples in remote read, especially func (c *Client) LabelValues. Refactoring to cut down on repetition may be desired.
The question of what url endpoint to hit was raised in another PR, there they chose to define a new url in addition to the remote read url. To me this seemed redundant, and I feel it's implied that label values should be scraped from the same target as the remote reads, so I chose to hit /api/v1/read/labelvalues
Open to any suggestions, just kinda threw this together :)

The failing tests seems to be happening on other PR's too and seems unrelated to my changes.

so I chose to hit /api/v1/read/labelvalues

Are you sure about that?

so I chose to hit /api/v1/read/labelvalues

Are you sure about that?

I'm not sure I follow what you're asking? Am I sure I'm hitting /api/v1/read/labelvalues? Or am I sure sure that's a good idea?
If the former then I imagine you're hinting at more explicit wording. If a user is setup to remote read from another prometheus using the default pathing, meaning no trickery with reverse proxy's or anything, then yes it'll be hitting /api/v1/read/labelvalues. But more specifically it'll hit client.url.String()+"/labelvalues", with the url being whatever the user defined in the config.

Am I sure I'm hitting /api/v1/read/labelvalues?

Yes, because that's not what's in the PR (the PR does appear to be handling this correctly, your description is incorrect).

I must be missing something here then, how is that not what's happening? I can see the requests metrics in prometheus

prometheus_http_requests_total{code="200",handler="/api/v1/read/labelvalues",instance="localhost:9090",job="prometheus"}
7

https://github.com/prometheus/prometheus/blob/fa1e6f745b46bf173d001c7d2ab105f76d79fc26/web/api/v1/api.go#L294-L296
https://github.com/prometheus/prometheus/blob/fa1e6f745b46bf173d001c7d2ab105f76d79fc26/storage/remote/client.go#L202-L204

That's a bit of a bug then, none of the remote read stuff should be under v1 as it's not a v1 API. v1 APIs are JSON.

Ah ok, is that something I should look into changing with this? Or should that be a separate task?

We can't really change it now as that'd be breaking, but we should probably not make it worse.

Right, that makes sense. Should I instead pursue using grpc like what's used for api/v2 admin functions?

Took a crack at doing a grpc implementation, let me know if this seems like a better approach going forward with remote target operations. Presumably something similar would be done for /api/v1/labels

v2 is dead, and there's no plans to switch anything to grpc.

Ok... could you advise on what should be done then?

We should probably get input from other developers on how to handle this.

@gouthamve @tomwilkie @codesome @simonpasquier Any opinions on this? Looking for a little more feedback

Ok, we had an offline discussion on IRC: https://matrix.to/#/!WaUKIfoqfiyWQhenET:matrix.org/$158641669586465FJBvM:matrix.org?via=matrix.org&via=half-shot.uk
Our question is, why we need to create another endpoint? Looks like such protobuf-based label values would expose exactly the same functionality as our JSON one. So why not define that remote target has to (actually it should optionally expose to not add braking change):

api/v1/read
api/v1/labels/{name}/values
api/v1/labels/names

We can totally do that, document that and write client for those in Prometheus  api/v1/labels/{name}/values api/v1/labels/names to support. Cannot see any blocker for that :hugs:
Thoughts?

Also let's focus on what would be the exact use cases here? (: I think there can be many would be nice to define them clearly.

Our question is, why we need to create another endpoint? Looks like such protobuf-based label values would expose exactly the same functionality as our JSON one. So why not define that remote target has to (actually it should optionally to not add braking change) expose:

api/v1/read
api/v1/labels/{name}/values
api/v1/labels/names

Just to be clear, would the design be then to have a config with all three endpoints defined? Optionally excluding label/{name}/values and /labels? So something like:
remote_read:
- url: http:target:9090/api/v1/read
  labels_url: http:target:9090/api/v1/labels
  label_values_url: http:target:9090/api/v1/labels/...

With labels_url and label_values_url being optional? If so, it isn't really intuitive what the label_values_url should look like.
Alternatively, should the /labels and /labels/{name}/values endpoint be interpreted/manipulated from the given url and instead have optional flags to opt in to gathering label information from remote read targets?
The use case for me is essentially what brian pointed out in the that discussion of having sharded instances because we have a lot of targets, and remote reading from those shards. I know it's not a conventional setup but it seems to be working for us for now. We skirted around this issue in the meantime by reverse proxying those endpoints to a script that coalesces the values from the shards 😬Not the prettiest

I've never particularly been a fan of adding additional fields to the config in the various incarnations of this PR over time, I think manipulating to make it just work (barring the warnings you'll get for non-Prometheus backends that always fail it) with no additional configuration would be good. More intuitive configuration would be nice, but it's probably more something to consider for v3.

Do you still want it to be an opt-in thing? If so, how would you propose allowing that option without making a configuration changes?

I'd design for the long term and not have it configurable, as any configuration field would ultimately be vestigial.

@bwplotka Think you could take a look at this?

Bump

@bwplotka Can you take a look at this?

It looks like this is lacking documentation as well.

@bwplotka about using https://github.com/prometheus/client_golang/blob/master/api/prometheus/v1/api.go#L235 : I think you know that API client much better than I do. (Remember, I “maintain” it mostly by accident.) Superficially, it totally makes sense to me to use it. But I don't have any deeper insights about pros and cons.

Thanks, @beorn7, I would use it. For streamed operation with protobuf we need to do custom things, but here we could reuse solid API we already distribute. What do you think @mtknapp? (:

@bwplotka @beorn7 @mtknapp what is the status of this feature? Can I take over and finish this task? Having the labels and autocomplete in Prometheus set to remote read would be a feature nice to have.

Closing as duplicate of #11877


Add tools page

I added it according to the gatsby implementation.
Two things I would change:

remove table borders (feels very 90s)
add a "Show details" button, which expands the markdown in the table (similar to the help text in the nf-co.re parameter page), it is usually only a short text and would fit nicely into the table without needing to go to an extra page

It's not always a short text though (Picard being the main offender: https://main--multiqc.netlify.app/modules/picard )
Also it'll get longer for all modules because I want to add in the search patterns from the config.
Plus having a dedicated URL to link to will help, I point to these docs a lot in issues etc.


[WIP] Added webhook support for commentMonitor

Signed-off-by: Hrishikesh Barman<EMAIL_ADDRESS>

cc @krasi-georgiev


Proposal: List/pull debug logs

So, I want to start working on adding a new feature: The ability to list debug logs in the org and pull one into a file. This is something the cli supports and the VSCode extension as well.
Currently, the cli allows pulling a list of all debug logs in the org using sf apex list log --json. This prints a json to stdout with a list of all debug logs and properties for them in it. Using the information in that, one can then sf apex get log --id=<id> using the id found in the previous json to pull a log into a file.
I'm thinking, since logs can change quite rapidly (one would usually want to pull a very recently created log), there's little incentive to save the log list file on disk, and then use that to list logs to pull. Instead, we could have a single command Sf.pull_log() that would:

Call sf apex list log --json to get all currently available logs
Put that json into a table and into fzf-lua, and when one is selected:
Pull that log using sf apex get log --id= and save it to file, then open that file

I volunteer to work on this, just have one blocker I'm not sure what the best way to solve is: We'd need to run a command, and get the stdout output and use that next to create the fzf-lua table. I think all our current job_call methods at most run a callback function with no arguments, but none can await a command then read the stdout from that command.
What would be the best way to approach this?

Good initiative :)!
In util.lua there is M.silent_job_call using vim.fn.jobstart with on_stdout should be working. Pseudocode as below.
However, vim.fn.jobstart is vimscript based and old. If you have interests, I'd recommend to explore :h vim.loop and :h vim.system providing more idiomatic Lua bindings and are often more flexible. This is an item in my to-do :).
vim.fn.jobstart(cmd, {
  stdout_buffered = true,
  on_stdout = function(_, data)
    if data then
      -- Join the stdout lines and parse the JSON like the existing features in sf.nvim
      if err then
        vim.notify("Failed to parse JSON: " .. err, vim.log.levels.ERROR)
      else
        -- Do something with the parsed JSON result
      end
    end
  end,


implement create or update many users

Is your feature request related to a problem? Please describe.
Implement create or update many users method
Describe the solution you'd like
Implement create or update many users method

Tracked as #47 and #96


Debug Failure. Did not expect ObjectLiteralExpression to have an Identifier in its trivia

🔎 Search Terms
identifier unicode es5 es2015 trivia object literal expression failure
🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This changed in commit or PR _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link
https://www.typescriptlang.org/play?target=1&ts=5.5.0-dev.20240401#code/PTAEAEBcEMCcHMCmkBcpEGcCsAoAbnKAOKgC8oA3qIKwbgILtoBEAZAIIBmsA3A6AL6c4gA
💻 Code
// @target: es5
var G = { 𝔄: "𝔄" };

🙁 Actual behavior
TS server crashes when hovering over this property
🙂 Expected behavior
it shouldn't crash
Additional information about the issue
repro based on https://github.com/microsoft/TypeScript/issues/58018#issuecomment-2028929069

Obviously TS shouldn't crash, but is this legal JS?

It is, yes. That's a valid identifier.

We haven't updated the unicode tables in a while, which may be the problem, but, I haven't tested yet.

Target es5 is crucial here. You have 2 unicode tables there - it doesnt crash with newer targets

Likely this is failing due to the fact that the scanner/parser always pass in the script target, while much of services always asks for the latest.
But, I think this setup is not quite right. If the target is ES5, then we use the older table when parsing and doing stuff. But that doesn't feel right. Our inputs should be permissive, allowing anything, then erroring later if we can't downlevel it. In this case, there actually is a downleveling that works, and esbuild uses it unconditionally (even though it only does ES6+):
var G = { "\u{1D504}": "𝔄" }

Unfortunately just removing the ES5 table is not sufficient as without the scanner/parser to complain, we get no errors. A better change would need to preserve the table but only for emit errors. Or, in the parser, always parse identifiers with Latest then error on them if the script target is ES5 and the scanned text doesn't pass ES5 (which would at least stop the crashing, I think).

I thought \u{1D504} was ES6+ syntax and doesn't work in ES5.  Am I misremembering?

Could be, I'm no historian 😅


[Feature Request] Always use a format that supports chapters

Checklist

[x] I'm reporting a feature request
[x] I've verified that I'm running yt-dlp version 2021.08.10
[x] I've searched the bugtracker for similar feature requests including closed ones

Description

The manual says:
--embed-metadata                 Embed metadata including chapter markers
                                 (if supported by the format) to the video
                                 file (Alias: --add-metadata)

The key part to me being if supported by the format. I assume this refers to the container format, like MP4 or MKV. How can I get yt-dlp to always choose a container that supports chapter embedding? For my use case of archival, I really don't care if the container is always MKV or something, I just want to ensure that chapters are there.
This could be another flag, but I think choosing a good container should be the default for --embed-metadata.
Thanks a lot for working on yt-dlp, it's a great piece of software and you guys have added a lot of great stuff on top of youtube-dl.

I am not entirely sure which formats support it. I know both mp4 and mkv does

For my use case of archival, I really don't care if the container is always MKV or something, I just want to ensure that chapters are there.

You can use --remux-video mkv --merge-output-format mkv

I can confirm that webm also supports chapters and yt-dlp will add them.

You can use --remux-video mkv --merge-output-format mkv

Thanks. But will that option ensure that chapters are there? Or will it just download into whatever container it was going to use anyway, without chapters, and then remux it. Like I assume --merge-output-format mkv is not used if only a single file is downloaded, but you still might want chapters in that situation.
It looks like not a big issue for downloading video though, because MP4, MKV, and WebM all support chapters. But I think this feature could still be useful for downloading audio, which has formats that don't support chapters, like MP3. As well as rarer video formats that don't support them, like AVI.

Remuxing is done before adding chapters

Ok good. But what about formats where remuxing isn't needed, but chapters are still desired?

I was not saying it a full solution. If so, I wouldnt have labeled this issue as enhancement.
I was replying specifically to this part of your post

For my use case of archival, I really don't care if the container is always MKV or something, I just want to ensure that chapters are there.

If you dont care about the format, it is fine to always remux to mkv since mkv can handle all codecs and attachments

Oh ok, go it! Thanks for your work.

We only ever merge videos into mkv,mp4,webm. All of these support chapters.
It is possible that you are downloading some other format from a site. But in this case, since we are not modifying the video file before embedding metadata, it doesn't make sense for yt-dlp to automatically remux it to another format without the user explicitly asking for it. So I'm closing this issue


Make resourceType of Query public

Similar to https://github.com/jsonapi-ios/Spine/pull/1/
Not sure if it makes sense to expose any other properties in Query the same way, this was the only one that we required. Let me know!

Ping @MonsieurDart @pauljeannot 🎈

Sorry for the delay.
Indeed, we will see when needs will evolve! :)
Thank you for contributing!


Smaller screen size than actual screen size

My actual screen size is 1920x1080, and Windows 10 confirms this in settings. However, RobotJS is claiming that my screen size is 1280x720. I've tried 1280x720 in the Windows settings, and there is a visible difference while also RobotJS then claiming that the screen size is 854x480. When running a loop function to get the mouse coordinates at every possible moment, I get the same results, though I don't see how it would have made any other difference. Just something to keep in mind.
Expected Behavior

RobotJS should display my screen size as 1920 for the x value and 1080 for the y value.
Current Behavior

RobotJS shows a smaller size for the screen, being 1280x720 with 1920x1080 and 854x480 with the actual size 1280x720.
Possible Solution

Possibly check in Windows settings for the screen size, unless that's already done.
Steps to Reproduce (for bugs)

Confirm your actual screen size in settings.
Create a script that gets the screen size with RobotJS.
Run the script.

Context

I am trying to accurately make the cursor draw a circle that is circling the middle of the screen in order to play a specific game. I only discovered this yesterday when attempting to figure out the bounds of what I had to work with and needed more specifics.
Your Environment

RobotJS version: 0.6.0
Node.js version: 12.16.1
npm version: 6.13.4
Operating System: Windows 10 (Pro)

This should be caused by the zoom in the win10 display settings，RobotJS may have ignored this.


[Polymetis] Remove time-based tests

Description
Previously we had some tests that time certain operations, and results would heavily depend on capabilities of the hardware the tests were being run on.
Removed the tests so as to not cause flakiness in CI.
Type of change
Please check the options that are relevant.

[ ] Bug fix (non-breaking change that fixes an issue)
[x] Proposes a change (non-breaking change that isn't necessarily a bug)
[ ] Refactor
[ ] New feature (non-breaking change that adds a new functionality)
[ ] Breaking change (fix or feature that would break some existing functionality downstream)
[ ] This is a unit test
[ ] Documentation only change
[ ] Datasets Release
[ ] Models Release

Type of requested review

[x] I want a thorough review of the implementation.
[ ] I want a high level review.
[ ] I want a deep design review.

Before and After
If this PR introduces a change or fixes a bug, please add before and after screenshots (if this is causes a UX change) or before and after examples(this could be plain text, videos etc ) to demonstrate the change.
If this PR introduces a new feature, please also add examples or screenshots demonstrating the feature.
Testing
Please describe the tests that you ran to verify your changes and how we can reproduce.
Checklist:

[ ] I have performed manual end-to-end testing of the feature in my environment.
[ ] I have added Docstrings and comments to the code.
[ ] I have made changes to existing documentation where needed.
[ ] I have added tests that show that the PR is functional.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have added relevant collaborators to review the PR before merge.
[ ] [Polymetis only] I ran on hardware (1) that all scripts in tests/scripts, (2) asv benchmarks.

Codecov Report

Merging #955 (2e53c5d) into main (9e20f08) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #955      +/-   ##
==========================================
- Coverage   32.27%   32.13%   -0.15%
==========================================
  Files         347      346       -1
  Lines       34213    33959     -254
==========================================
- Hits        11043    10912     -131
+ Misses      23170    23047     -123

Impacted Files
Coverage Δ

interpreter/craftassist/tasks.py
74.45% <0.00%> (-0.55%)
:arrow_down:

agents/droidlet_agent.py

agents/craftassist/tests/fake_agent.py
73.67% <0.00%> (+0.69%)
:arrow_up:


Unit Test does not work for Windows App SDK 1.2

Describe the bug
Tried to add a Unit Test project to an existing WinAppSDK project, following the steps in this blog. Could not build the Unit Test project.
Steps to reproduce the bug

Create a new Blank App, Packaged (WinUI 3 in Desktop) called App1.
Follow steps in blog with the latest MSTest (v3.0.0) and Microsoft.TestPlatform.TestHost (v17.4.0) packages. Created new project called App1.Test.
Tried building and running test in App1.Test. It works!
Add Project Reference of App1 to App1.Test.
Tried to build App1.Test.  Failed with the following errors:

Error		PRI175: 0x80073b0f - Processing Resources failed with error: Duplicate Entry.	App1.Test	C:\VS\WinAppSDKUnitTest\App1\App1.Test\WINAPPSDKGENERATEPROJECTPRIFILE	1
Error		PRI277: 0x80073b0f - Conflicting values for resource 'Files/MainWindow.xbf'	App1.Test	C:\VS\WinAppSDKUnitTest\App1\App1.Test\WINAPPSDKGENERATEPROJECTPRIFILE	1

Expected behavior
Test project should build and run without problems.
Screenshots
See sample project attached: WinAppSDKUnitTest.zip
NuGet package version
Windows App SDK 1.2.1: 1.2.221116.1
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 22H2 (22621, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

@ritt-app  in the linked blog post they are not adding a project for the tests but running them in place (i think cause of msix)
i tried and its seems to be working fine, i'm not sure about the side effect of having tests inside the "wasdk" project

Closing this issue as we already have documentation for testing. @ritt-app, please refer to the following links if necessary:

https://learn.microsoft.com/en-us/windows/apps/winui/winui3/testing/
https://learn.microsoft.com/en-us/windows/apps/winui/winui3/testing/create-winui-unit-test-project
Thank you!


esModuleInterop: true  with class static method

Bug Report

🔎 Search Terms

class static import
🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link

https://www.typescriptlang.org/play?module=1#code/JYWwDg9gTgLgBAYQJIBEA0cDedgGdkpwC+cAZlBCHAOQDGwAJrtQNwCwAUJwKYAeksMgFcAdrRjAIIuGACGMABYAVCAkYAKegwBccXDCjARAczgAfRKgCUu-YZOdMnAJDBScdQQB0eApsZWVi5OHM7OUNwwQlDSWl4wEADKBkbG6kGhRJwuEVExcHERYAA2srTc6gD0AHoAOpXAYADaIqQAurj1lRjU1BlZXBx8AvCkouKS0nKKKmoMAEz+OnopJuaWKDYr9saOLm4evqhLgcEu4ZHRsYzxSatpGc4DOZf5hdwlZRU19Y0t7Z1Kt0aH1OM8ONNlKoNL0MpDZoxFrDOEA
💻 Code

import CID, { isCID } from 'cids';

export function pathToCid(cid: string | CID): string
{
	if (CID.isCID(cid))
	{
		return cid.toString()
	}

	return cid.replace(/^\/ip[nf]s\//, '')
}

export function pathToCid2(cid: string | CID): string
{
	if (isCID(cid))
	{
		return cid.toString()
	}

	return cid.replace(/^\/ip[nf]s\//, '')
}

pathToCid('')
pathToCid2('')

🙁 Actual behavior

TypeError: (0 , cids_1.isCID) is not a function

🙂 Expected behavior

no error on output js file
or show typescript error for warn this

Normally I would say this should be considered an error as a native ES module wouldn't normally provide the entire module as the default export. However, that's exactly what NodeJS does for its built-ins like fs when you use import in a native module.
It is odd that the following works:
import CID from 'cids'; // passes through __importDefault
import { isCID } from 'cids'; // directly accesses module object

while the combined version doesn't:
import CID, { isCID } from 'cids'; // passes through __importStar, doesn't adjust for default import

@andrewbranch, @weswigham: Any thoughts on this? I'm leaning towards trying to make it work under --esModuleInterop rather than reporting an error, given that the individual declarations work in this mode.

Fixed by #60262


Checkpoint files for Conceptual Captioning

Are there pretrained weights for conceptual captions trained on the topdown model?
In the googletalk.json file you provided, the vocabulary is of the size 16,303. However, the first embedding layer in the topdown/model-best.json is of size 9488 (https://drive.google.com/drive/folders/0B7fNdx_jAqhtT3pBQlR6emJXM1U). It seems like the pretrained weights provided are for coco but not for google conceptual captions.

https://drive.google.com/open?id=18bRlDMt589kC5e6dtY-LQJovPYjlNHqV

Thanks for quick response.
I couldn't exactly make from the name which one is topdown model. I looked at infos and found trans_* are transformer models and best_smoothing is att2in. Is there a topdown based model?

there is no topdown model.

Hi! Are all models trained on google conceptual captions (gcc) using bottom-up features? Is it possible to evaluate on own images using the gcc models? Evaluation on own images seems to require resnet101, though to my understanding gcc models are trained with bottom-up features. Could you please clarify this? Thanks!

Look at docker image ruotianluo/conceptual_ens3.
It has the ability to run on new images.


Refactor bindings

This PR refactors the codegen-related binding types into a single type responsible for both the code generation and JSON serialization for function.js.
Additionally, the binding type implementations have been macro-ized so that the implementations are generated based on the types of the fields in the binding structures.
This will make it easier to implement new bindings in the future without having to duplicate error-prone code.
Closes #180.

@rylev don't feel compelled to review this as there's a lot here and it doesn't change any functionality, instead it eliminates a lot of copy-pasted binding-related code in favor of using a procedural macro to generate it.
In total, about a thousand lines were removed.  However, many missing tests were added to the codegen binding types so it ended up a net positive in lines.


[Feature] Implements D-Adaptation optimizers

Problem (Why?)
new optimizers!
Solution (What/How?)

[x] implement DAdaptAdaGrad optimizer
[x] implement DAdaptAdam optimizer
[x] implement DAdaptSGD optimizer
Shampoo Optimizer (related to #100)

[x] supports no_preconditioning_for_layers_with_dim_gt
[x] refactor matrix_power()
[x] improve a speed of power_iter()

Other changes (bug fixes, small refactors)

[x] docs for D-Adaptation optimizers

Notes
bump version into v2.4.0

Codecov Report

Merging #101 (566c0bd) into main (44c423a) will decrease coverage by 0.38%.
The diff coverage is 96.99%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   99.65%   99.28%   -0.38%
==========================================
  Files          37       38       +1
  Lines        2618     2930     +312
==========================================
+ Hits         2609     2909     +300
- Misses          9       21      +12

Impacted Files
Coverage Δ

pytorch_optimizer/optimizer/dadapt.py
96.58% <96.58%> (ø)

pytorch_optimizer/optimizer/shampoo_utils.py
99.52% <98.63%> (-0.48%)
:arrow_down:

pytorch_optimizer/__init__.py
100.00% <100.00%> (ø)

pytorch_optimizer/optimizer/shampoo.py
100.00% <100.00%> (ø)

pytorch_optimizer/optimizer/utils.py
96.26% <100.00%> (-1.50%)
:arrow_down:

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


Added a podspec

I've added a podspec since I will be using this with cocoapods. I have not pushed it to the repo that manages pods just yet since the current specification is not complete. Cocoapods requires that a version be tagged in order to work properly and be able to distinguish between versions. I have done this in my own fork and used my fork as the source for the pod but I'd like it to point to the original source.
Once you tag a version, I will update this line: s.source = { :git => "https://github.com/petester42/ios-charts.git", :tag => "0.0.1" } to point to here and then you can merge.
I also tried making sure carthage support works but the way the current project is setup makes it so that the schemes are not there when running carthage.

Please clean this pull request first

all cleaned up

Ok man now you can update the spec to this:
Pod::Spec.new do |s|
  s.name         = "Charts"
  s.version      = "2.0.9"
  s.summary      = "ios-charts is a powerful & easy to use charting library for iOS"
  s.homepage     = "https://github.com/danielgindi/ios-charts"
  s.license      = { :type => "Apache License, Version 2.0", :file => "LICENSE" }
  s.authors      = "Daniel Cohen Gindi", "Philipp Jahoda"
  s.platform     = :ios
  s.ios.deployment_target = '7.0'
  s.source       = { :git => "https://github.com/danielgindi/ios-charts.git", :tag => "v2.0.9" }
  s.source_files = "Classes", "Charts/Classes/**/*.swift"
  s.frameworks = 'Foundation', 'UIKit', 'CoreGraphics'
end

Squash the commits together and I'll pull it.

Sadly iOS 7 is not supported for swift frameworks. It gives the following error:
 -> Charts (2.0.9)     - ERROR | [iOS] Swift support uses dynamic frameworks and is therefore only supported on iOS > 8.
It is also detailed on the cocoa pods blog under the availability section of this article http://blog.cocoapods.org/Pod-Authors-Guide-to-CocoaPods-Frameworks/
I will make the other changes but it has to stay iOS 8 for the pod.

http://stackoverflow.com/questions/24001778/do-swift-based-applications-work-on-os-x-10-9-ios-7-and-lower
Are you sure it's not supported?

It's not a swift issue. Swift will work fine on iOS 7. The problem is the framework. Dynamic frameworks are only supported as of iOS 8. This builds a dynamic framework so no iOS 7 love :(. If someone want to use this on iOS 7 they will need to copy the files to their project and not use the framework.

Ok I see!
Thanks!
Merged.

I've pushed the pod to trunk. If you want to be added as an owner of the pod just register with cocoapods pod trunk register email 'name' and i'll add you as an owner.


Adding verilator support to SVunit

Reference:

https://github.com/svunit/svunit/discussions/222
https://github.com/svenka3/svunit/tree/srini_verilator

@tudortimi can you help in reviewing this pull request ?

I'm happy to merge a PR which adds support for Verilator, but it would need to be a bit more granular. All changes in one commit  is too much. They would need to be grouped into separate commits, which explain why they are needed.

I decided to work on this myself, because it's also a good chance to report issues to the Verilator team.


Update bug_report.md to remove default priority label

As discussed, removes the default priority label for bug reports.

Removed the aspect: Will make another tiny PR for the feature request template.


xt/examples-compilation.t doc/Type/Signature.pod6 failing

Signature.pod6 has this section:
To accept one type but coerce it automatically to another, use the
accepted type as an argument to the target type. If the accepted type is
C<Any> it can be omitted.

    sub f(Int(Str) $want-int, Str() $want-str) {
        say $want-int.^name ~ ' ' ~ $want-str.^name
    }
    f '10', 10;
    # OUTPUT: «Int Str␤»

    use MONKEY;
    augment class Str { method Date() { Date.new(self) } };
    sub foo(Date(Str) $d) { say $d.^name; say $d };
    foo "2016-12-01";
    # OUTPUT: «Date␤2016-12-01␤»

the example test is now dying with
Package 'Str' already has a method 'Date' (did you mean to declare a multi-method?)
at /Volumes/BUKKIT/Users/coke/sandbox/raku-doc/a:8

But this test was added in 2016, and bisectable indicates this method has existed for a while.  Not sure why it's failing now. (I only run the test if the file changed, so it doesn't mean it's been working consistently since it was added.)

Thanks to @AlexDaniel, we see this behavior changed here:
https://github.com/rakudo/rakudo/commit/d95d218f3192b06f0bb52bceb90e43d198747f09

The augment is unrelated to the signature example here, so we can just remove it and rely on Rakudo's built in Str.Date method. (but we should mention that's what's being called)

This is the last failure referred to in #3391


OCPBUGS-43698: featuregate: enable CPOV2

What this PR does / why we need it:
More featuregates should be enabled but CPOV2 is required when the
OpenStack feature is enabled otherwise cluster dump will fail since it
needs to find resources for controlplanecomponents.

/test e2e-openstack

/jira refresh

/approve

/hold for CI to finish first

I checked the logs and it fixed the bug.
/hold cancel

/test e2e-aks e2e-openstack

/lgtm
/hold for CI

/test e2e-openstack
feel free to unhold if the error looks unrelated to you

/lgtm


Copy of coco.tar

The Copy of coco.tar given by Google cloud disk is the coco native data set or processed

The Copy of coco.tar given by Google cloud disk is the coco native data set or processed

Please use the data link in our new repo. Thanks.


Cloudflare Deployment API issue

Hey!
I decided to deploy my own FxTwitter to not put load on fxtwitter.com and for spike usages, i deployed it successfully and tweets ger redirected however if i try to use the API
api.mydomain.com/username/status/id
i get the following SSL error
ERR_SSL_VERSION_OR_CIPHER_MISMATCH
and i haven't been able to understand how to fix it.
Thanks

My bad, it was an issue at the original domain register, wasted 2 days on it

Glad you figured it out :D


Fix nested fragment bug

Missing 'schema' parameter inside recursive, this cause error:

Cannot read property 'getType' of undefined

Can you please add a unit test?

done, I just modify the schema test a little bit


Fix to keep single "0" alive

String "0" should be treated exceptionally when empty() check.

Merged, thanks!


Add a way to create testing doubles for C# calls.

@jskeet, would this suffice to make the client side code testable the way you need it?

I added a TestCalls into a new nuget package Grpc.Core.Testing (as suggested by Jon).
@apolcyn, can you PTAL?

need to re-generate projects?
in sanity seeing:
2c2
<   "version": "1.1.0-dev",
---
>   "version": "1.2.0-dev",
24c24
<     "Grpc.Core": "1.1.0-dev"
---
>     "Grpc.Core": "1.2.0-dev"

Projects regenerated.

Known failures:

#9499


Read shapefile

Closed by https://github.com/geoarrow/geoarrow-rs/pull/823 and https://github.com/geoarrow/geoarrow-rs/pull/759


4-turbo mode: diffs where folder names have spaces in them are not parsed correctly

Great tool!
I just tried aider on a project where the folder names have spaces, and it has an issue parsing the diffs.
For example:
--- PRAW and URS Reddit Scraping/Classify.py
+++ PRAW and URS Reddit Scraping/Classify.py
@@ ... @@
 def classify(filename:str, model: str = 'gpt-3.5-turbo-1106'):
     d = json.load(open(filename))
     for post in d['fetched_posts']:
         messages = copy(MESSAGE_TEMPLATE_CLASSIFY)
...

Gives:
Allow creation of new file PRAW? y
Which is not correct, a new file is not being created here.

Thanks for trying aider and filing this issue.
I just pushed a fix. The change is available in the main branch. You can get it by installing the latest version from github:
python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Trouble installing modin on WSL: No matching distribution found for pandas==0.25.3

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 with WSL (Windows Subsystem for Linux)
Modin installed from (source or binary):
Modin version: Newest
Python version: Anaconda Python 3.7 (Newest)
Exact command to reproduce: pip install modin[ray]

I installed WSL (Windows Subsystem for Linux) and I am trying to install Modin on it. I am using Anaconda 3.7 within WSL and when I type 'pip install modin[ray]' I receive the following error message:
Could not find a version that satisfies the requirement pandas==0.25.3 (from modin) (from versions: 0.1, 0.2b0, 0.2b1, 0.2, 0.3.0b0, 0.3.0b2, 0.3.0, 0.4.0, 0.4.1, 0.4.2, 0.4.3, 0.5.0, 0.6.0, 0.6.1, 0.7.0rc1, 0.7.0, 0.7.1, 0.7.2, 0.7.3, 0.8.0rc1, 0.8.0rc2, 0.8.0, 0.8.1, 0.9.0, 0.9.1, 0.10.0, 0.10.1, 0.11.0, 0.12.0, 0.13.0, 0.13.1, 0.14.0, 0.14.1, 0.15.0, 0.15.1, 0.15.2, 0.16.0, 0.16.1, 0.16.2, 0.17.0, 0.17.1, 0.18.0, 0.18.1, 0.19.0rc1, 0.19.0, 0.19.1, 0.19.2, 0.20.0rc1, 0.20.0, 0.20.1, 0.20.2, 0.20.3, 0.21.0rc1, 0.21.0, 0.21.1, 0.22.0, 0.23.0rc2, 0.23.0, 0.23.1, 0.23.2, 0.23.3, 0.23.4, 0.24.0rc1, 0.24.0, 0.24.1, 0.24.2)
No matching distribution found for pandas==0.25.3 (from modin)
Same error message occurs if I type 'pip install modin' or 'pip install modin'. I tried first installing pandas and ray and then modin, then tried first installing modin. Same result always.
Keep in mind I am running everything within WSL so its not Windows.
Any advice how to fix this?

Hi @NeoXx123, thanks for posting!
Does the same thing happen when you type python3 -m pip install modin[ray]?
The pip in your path looks like it is the python2 version, looking at the pandas distributions available. pandas dropped python2 in 0.25, which would explain why it's not in that list.
Let me know if this helps! I use WSL on a daily basis for development and testing so the issue shouldn't be related to that.

Hi @NeoXx123, I am going to close this in favor of tracking with #888, which is actionable from our side. Please feel free to open a new issue if you still have problems installing.


X-Forwarded-Prefix and URL rewrite support

Issue
x-forwarded-prefix is supported by Spring  — see ForwardedHeaderFilter
Why gorouter doesn't support this header now? I guess it's very useful
Motivation
In many cases, it is a good option. For example — spring-data-rest.
internet-> [LB:443/custom_path/users] -> [gorouter] -> [some_application:8080/users]
                                                                             ^
                                                                             |
                                                                       rewrite url
            add x-forwarded-prefix with the original path

If gorouter support rewrite and x-forwarded-prefix, I will deploy an application without additional configuration. It can simplify deploy process. Otherwise, we need to configure right SERVER_SERVLET_CONTEXT_PATH in spring-boot app, but in my opinion, it's wrong, because an application mustn't known about a public environment - it's infrastructure responsibility.
Also, with x-forwarded-prefix and spring ForwardedHeaderFilter we can use right url in HAL API (with prefix) without additional configuration in Application level
What do you think about it?

I dont sure about issue type. It's not a bug, i gues.. Sounds like a feature :)

Hi @lavcraft, it is unlikely we can prioritize adding this feature since our efforts our currently focused on integrating Istio/Encoy, but I am notifying @shalako for awareness of this feature. Also, we will gladly review a PR!

@lavcraft - would you like to move this over to https://github.com/cloudfoundry/istio-release and we can prioritize getting it integrated as part of the istio track of work @KauzClay mentioned?


[GitHub Actions] Refactor Docker build process to use ghcr.io for build, and then copy to docker.io once build completes

References

Fixes #10092 (or at least, it should improve things significantly)

Description
This PR refactors our GitHub Actions to build our Docker images in a different manner to avoid aggressive rate limiting of DockerHub:

First, the Docker images are built & pushed to GitHub Container Registry (ghcr.io).

The build process now uses images in ghcr.io for the build (by leveraging the changes in #10080), thus avoiding interaction with DockerHub.
The pushed ghcr.io images are private but access is given to GitHub Actions in both DSpace/DSpace and DSpace/dspace-angular.  (We could consider making these public in the future.)

Once the built image exists in ghcr.io, it is copied to DockerHub (docker.io) in a single command.  This minimizes our requests to DockerHub.
Finally, after the image is in DockerHub, (if necessary) we deploy it to Demo/Sandbox sites.

Instructions for Reviewers

This has been tested by pushing the code to a temporary branch in DSpace/DSpace.  The entire build process succeeded, including pushing images to ghcr.io and docker.io.  See https://github.com/DSpace/DSpace/actions/runs/12357249080/job/34487955090

Because the images pushed were temporary, I will be deleting their tags from ghcr.io and docker.io.  But, I've already verified that they pushed successfully.

This PR is just here to verify that PR builds are still succeeding properly.  If it passes all tests, it can be merged.

This will need to be ported to 7.x and 8.x for those branches.

Merging as this is now working both for PRs & non-PRs.  This will need to be ported to 7.x and 8.x


Split brotli package into two

Fix https://github.com/emscripten-forge/recipes/issues/782

Fix #782

To actually fix #782, looks like we'll probably need to remove https://github.com/emscripten-forge/recipes/tree/main/recipes/recipes_emscripten/requests-wasm-polyfill as well?

Let's go with this for now. At some point we may want to use a multi-outputs recipe (and make sure the update bot supports it)

OK thanks for looking into this!


#45 fix copy task generalization code

Fixes #45

@chrispugmire heya, sorry for the delay, i was kinda stuck while vacationing in kashmir (under curfew now).
Care to review the changes? Its been a while since I've worked on this so kinda unsure if the fixes are legit according to the paper and all.

The code runs now, but if I'm reading it right, then it's not doing anything sensible.  The code seems to be summing the inputs.  But its the copy task not the adding_task, so summing the output doesn't produce a meaningful test.  It should display the input, target and output sequences in some format to see if the data is being coppied.   I used some code like this but it's not 100% compatible with your sample I'm sorry.
def to_text(slice):
#print("slice dimensions ",slice.shape,"slice is",slice)
slice = slice.cpu().detach().numpy()
res = []
for seq in range(slice.shape[0]):
w = 0
col = 1
for b in range(slice.shape[1]):
if slice[seq,b]>.5:
w += col
col = col * 2
res.append(w)
return res
def test_now():
for i in range(2):
rnn.eval()
random_length = np.random.randint(2, sequence_max_length + 1)
random_length = 6
input_data, target_output = generate_data(args.batch_size,random_length, args.input_size,args.cuda) # args.batch_size
output = rnn(input_data, reset_experience=True, pass_through_memory=True)
rnn.detach_hidden()
print("input ",to_text(input_data[0]))
print("target ",to_text(target_output[0]))
print("output ",to_text(output[0]))

Closing this cause rewriting the whole thing


[Event Request] "Price Helper - V16" Procedure DeletePrices -> Make it better

Describe the request
Hi,
regarding the old issue https://github.com/microsoft/ALAppExtensions/issues/25838 it would be helpful to get the handled pattern as requested. Your addition of the publishers are a great help but the price list header will still be deleted anyway so we can't handle this. The code in this section is still wrong and a price list header should never be deleted if there are lines that may have other conditions than the header.
So please add a check and don't delete the header or give us at least the option to handle it.
OnBeforeDeletePrices(SourceType, SourceNo, ParentSourceNo, isHandled);  // add isHandled
if not isHandled then begin  // new
PriceListHeader.SetRange("Source Type", SourceType);
PriceListHeader.SetRange("Parent Source No.", ParentSourceNo);
PriceListHeader.SetRange("Source No.", SourceNo);
if not PriceListHeader.IsEmpty() then
PriceListHeader.DeleteAll();
end; // new
Thanks and kind regards,
Christian
Additional context
DO NOT delete price list headers which still have line data
Internal work item: AB#500781

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 17376.


Space side diff

The space is now placed next to the diff view. When you click on the name of a block, you scroll to the corresponding diff on the right.
You can best test this PR with https://localhost:5200/projects/Polymer/Polymer/pulls/3680/diff

Fixes #168
Fixes #178
Fixes #184

As you already said, this doesn't work on small screens.

It should be a lot better now on your screen :)

When there is one chunk, and I click the Link button the linking-mode cannot be canceled, so I'll always see an arrow following the mouse pointer

You can now hit escape and the button text changes to indicate you can reclick to stop the creation of a connection :tada:

It still does not work on my 13.3 inch screen.

Oops that was actually regressed while I was fixing the width calculations. It should be fixed now!

Can the space and diff views be centered?

When one navigation-list is shown, it slides over the space view. Is this intended?

Scrolling with space-view open in this PR is broken: https://localhost:5200/projects/Polymer/Polymer/pulls/3680/diff

We'll do more fixes in another PR.


Events that are "all day" show up in CalendarFolder.findAppointments() on next day

For example, I create an all day event for 2015-07-01 and call it "July 1st". If I look for appointments on 2015-07-02, that event shows up.
From what I can see, the all day event is defined as starting at 2015-07-01 00:00 and ending at 2015-07-02 00:00, and isAllDay() returns true for this event.
My expectation is that such an event would only show up for 2015-07-01.
here's the basic call I'm using
CalendarFolder folder = CalendarFolder.bind(service, WellKnownFolderName.Calendar);
FindItemsResults<Appointment> results = folder.findAppointments(new CalendarView(start, etnd));

Server is 2010 SP2

you can do for all day:
CalendarFolder calendar = CalendarFolder.bind(service, WellKnownFolderName.Calendar);
SimpleDateFormat formatter = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
Date startDate = formatter.parse("2015-07-01 00:00:00");
Date endDate = formatter.parse("2015-07-01 23:59:59");
FindItemsResults appointments = calendar.findAppointments(new CalendarView(startDate, endDate));

Might be a duplicate of #328.

The suggestion above for all day would still pick up the appointment for 2015-07-02, and the underlying use case here is to display a calendar in a portal - looking up one day at a time would require up to 31 calls to the exchange server instead of just one.

I'm not sure about this being a duplicate of #328 - it very well may be - I did see that issue and thought it pertained more to time zone differences. That issue, however, was also reported for ExchangeVersion.Exchange2010_SP2
In my case, the Appointment object does correctly report that the event is an all day event.
For an interim workaround, I'll perform an extra check on events that are all day.


OpenCV 3.2.0 Comparability: loading dnn models

I am trying to load a pre trained Caffe image recognition net but the functions in dnn.hpp and dnn.cpp files  required are not supported in dnn.hpp and dnn.cpp in openCV 3.1.
I am running on windows so the .dll files for OpenCV are pre compiled. Any advice on how to compile the OpenCV 3.2 files I need into .dll without breaking anything else?
Thanks so much

Sorry for the late reply and inconvenience.
To update to OpenCV 3.2, you should

replace src/3rdparty/opencv
replace bin/vs20xx/opencv_**.dll
rename opencv_**310.lib to opencv_**320.lib in the additional dependencies option

I'm sorry I don't have AR.Drone now and have less time to update CV Drone.


Generating Constructor Inheritance

example
public class Color32TransitionState : TransitionStateCollection<Color32>.TransitionState
{
	public Color32TransitionState(string name, Color32 stateObject)
	{
	}
	}

public Color32Transitions(string[] stateNames)
{
}

must be
public class Color32TransitionState : TransitionStateCollection<Color32>.TransitionState
{
	public Color32TransitionState(string name, Color32 stateObject) : base(name, stateObject)
	{
	}
	}

public Color32Transitions(string[] stateNames) : base(stateNames)
{
}

I tried to extract Therabytes.BetterUI.

No promises, but you can try my AssetRipper fork.


chore: Add license and copyright header to the charts

WHAT
Add license and copyright header to the charts
WHY
License and copyright header were missing on charts
FURTHER NOTES
Closes #284

LGTM, are there issues with the business tests, or are they just flaky @tuncaytunc-zf?

Indeed there was an issue with business-tests, the certificates we are using for tests were expired, I have recreated them, the tests are running now but GitGuardian is complaining now we could ignore it as the secrets were already there but just expired :-)

There is issue #278 where we need to delete hard-coded keys from our code base. Once we generate the certs/keys "on-the-fly" we should never have this problem again.


Made management objects serializable and removed duplicate naming for…

… stats
I had to update management model objects to make them properly deserializable, plus improved some little things like removing duplicate naming for stats and renamed PERCENT to RATIO.
Depends on https://github.com/Terracotta-OSS/terracotta-management/pull/19
@anthonydahanne : once PR #19 will be reviewed / merged / released, you'll be able to review and merge this one :-)

@chrisdennis : I do not know if it is normal but I couldn't set @anthonydahanne as the assignee ;-)

@anthonydahanne : this requires this PR also: https://github.com/Terracotta-OSS/terracotta-management/pull/20

test this please

LGTM


Add static method from_optional

Adding issue #19 opened by rksouthee.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0679497) 100.00% compared to head (ea8fb4d) 100.00%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #20   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           13        13
  Lines          962       973   +11
=========================================
+ Hits           962       973   +11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


SOAP feature compression

Hi,
So I've added two feature compression schemes for SOAP features, together with an optional element-specific weighting feature that can be used with one of these feature compression schemes. The class constructor for SOAP now has the following options:
compression="off" -- can also be set to compression="agnostic" or compression="m1n1". The former is element-agnostic, i.e. the features are generated in an element-agnostic way (in which case the user may also want to specify element-specific weighting). The latter corresponds to the mu=1, nu=1 compression scheme from Darby et al., in which case the number of features for a given center will scale linearly rather than quadratically with the number of possible species / elements.
species_weighting=None -- can also be a dictionary (in which case it must contain one element for each species / element with an element-specific weight). If the user does supply a dictionary, these weights are used to weight the elements (in addition to any distance-based weighting they have specified). I can't imagine anyone would want to use this unless they're also using the compression="agnostic" option, but the code does not currently prohibit them from using it in any scenario where they want to do so. The code does not currently check to make sure that species weights are not set to numbers that might cause problems (e.g. 0, complex numbers, etc.) since presumably the user is unlikely to do this.
To implement these changes, we've made changes to the following files;
descriptors/soap.py
descriptors/descriptor_local.py
ext/SOAPGTO.cpp
ext/SOAPGeneral.cpp
ext/SOAP.cpp
ext/SOAPGTO.h
ext/SOAPGeneral.h
ext/SOAP.h
ext/ext.cpp
ext/descriptor.cpp
and of course to the docs and the tests, specifically test_soap.py and conftest.py. We examine how well these compressed features perform in a quick-and-dirty test on the QM9 repo shown here:
https://github.com/jlparkI/dscribe_qm9_testing
Code has been run through the linter -- if there are any problems with the changes I've implemented, or if you don't think the dscribe repo is the right place for these just let me know. Thanks again!

Hi @jlparkI, thanks for the PR. I'm currently on vacation, but I will come back to this.

Hi @jlparkI!
Thanks a lot for all your hard work. This is definitely a very nice feature which will definitely be merged. I do have a few suggestions that we could discuss here:

I think compression could be a dictionary, in the same style as e.g. weighting is. Although currently only the element-agnostic compression takes any additional arguments, there might be other compression schemes in the future that take further arguments and thus a dictionary will be more future-proof IMO. This way we can add e.g. compression parameters etc. in the same argument. This makes it also easier for the user to understand their connection. We can also provide the option or compression being only a string, but then compression arguments need some defaults.
species_weighting should then be in the compression dictionary.
We currently have a parameter called 'crossover', which is actually a type of compression. It should be made into a compression option.

So all in all I would propose having the following compression options (I took inspiration for the names from Table 1 in the article, alternative name suggestions are welcome):
# No compression. Default value. Scales as 1/2*NS(NS+1)(L+1).
compression = None
compression = "nu2"
compression = {
    mode: "nu2"
}

# Compression scheme mu=1, nu=1 explained in https://doi.org/10.1038/s41524-022-00847-y. Scales as N²S(L + 1)
compression = "mu1nu1"
compression = {
    mode: "mu1nu1"
}

# Element-agnostic mu=2 explained in https://doi.org/10.1038/s41524-022-00847-y. Scales as 1/2*N(N+1)(L+1)
compression = "mu2"
compression = {
    mode: "mu2",
    species_weighting: {'H': 0.5, "O": 1}
}

# Compression where cross-species densities are not calculated at all. Corresponds to current setup of crossover=False. Scales as 1/2*NS(N+1)(L+1).
compression = "crossover"
compression = {
    mode: "crossover"
}

@jlparkI: You have done a very nice work with this! Once we figure out the minor details I think this will be good to go :rocket:

Hi Lauri,
Thanks -- I think setting this up so that compression is a dictionary makes a lot of sense and I am fine with those naming suggestions. Let me set that up, I'll use the scheme you have in your comment, and then I can submit a new pull request with those additional commits -- probably later this week or early next. For now I will go ahead and close this pull request. Thanks again!


DrawerLayoutAndroid:- undefined is not an object (evaluating DrawerLayoutAndroid.positions )

I am using DrawerLayoutAndroid's example code.. Some how it's giving me above mentioned error...
var React = require('react-native');
var {
        AppRegistry,
        Text,
        DrawerLayoutAndroid,
        }= React

var AwesomeProject = React.createClass({
    render: function () {
        var navigationView = (
            <Text style = {{margin: 10, fontSize: 15, textAlign: 'left'}}>I'm in the Drawer!</Text> );
        return (<DrawerLayoutAndroid drawerWidth = {300} drawerPosition = {DrawerLayoutAndroid.positions.Left}
                                     renderNavigationView = {() => navigationView}> <Text
            style = {{margin: 10, fontSize: 15, textAlign: 'right'}}>Hello</Text> <Text
            style = {{margin: 10, fontSize: 15, textAlign: 'right'}}>World!</Text> </DrawerLayoutAndroid> );
    },
});
AppRegistry.registerComponent('AwesomeProject', () => AwesomeProject);

Try using
DrawerLayoutAndroid = require('DrawerLayoutAndroid');
Most Android modules have to be imported this way. I'm not sure why

@NishanthShankar Thanks. That worked!

You shouldn't use var DrawerLayoutAndroid = require('DrawerLayoutAndroid'); because that is an un

Confirmed that this is not happening: https://rnplay.org/apps/XDP9Tg
Please make sure you're using DrawerLayoutAndroid correctly :)

Oh.. But the examples use it as var AndoridComponent = require(AndoridComponent)
https://github.com/facebook/react-native/blob/master/Examples/UIExplorer/ToolbarAndroidExample.android.js
https://github.com/facebook/react-native/blob/master/Examples/UIExplorer/ProgressBarAndroidExample.android.js

Encountering the same issues. Only var DrawerLayoutAndroid = require('DrawerLayoutAndroid'); works.

@NishanthShankar - those are inside of the repo itself, that's a little bit confusing indeed.. Notice that throughout all of the codebase require('X') rather than require('../SomeDirectory/X') are used - this is because of how Facebook uses @providesModule internally.
@chourobin - if you copy the exact code from the above rnplay link I provided and run that locally, do you also get errors?

@brentvatne thanks for clarifying.
I got it solved! My problem was that the packager was bombing out -- my project folder path had spaces in it. I was manually starting the server in my project directory using npm start.
Fixing the folder name got the package launcher working properly and everything else worked.
Thanks!

@brentvatne sorry I spoke too soon, I do run into issues with the copy pasted code in a fresh project.

@chourobin - with the exact code from the example? That's weird! Can you upload a repo with it and include the node_modules directory too? Thanks!

@brentvatne - Hey Brent, sure here it is: https://github.com/chourobin/AndroidDrawer. Fresh project, copy pasted code & renamed SampleApp -> AndroidDrawer. Unfortunately it only works when I explicitly include 'DrawerLayoutAndroid'.

Also having this issue from AwesomeProject, only works if drawer is required.

I can't duplicate it on that playground but I'm not sure if I'm seeing it either. Maybe this is a version issue. Used default nexus5 device config.
CPU/ABI: Intel Atom (x86)
Target: Android 6.0 (API level 23)

@brentvatne What information do you need?

What's your react-native version? I'm also getting such errors with 0.11.2 when I try to use any Android native components like BackAndroid, TouchableNativeFeedback etc. Downgrading to 0.11.2 seems to resolve the issue.
I checked https://github.com/facebook/react-native/blob/master/Libraries/react-native/react-native.js in 0.11.2 and the Android modules were absent.
I did a diff between the current master version of the file and the 0.11.2 version, and this is the output.
--- react-native.js.a	2015-09-28 04:44:07.000000000 +0530
+++ react-native.js.b	2015-09-25 00:13:56.000000000 +0530
@@ -21,7 +21,6 @@
   // Components
   ActivityIndicatorIOS: require('ActivityIndicatorIOS'),
   DatePickerIOS: require('DatePickerIOS'),
-  DrawerLayoutAndroid: require('DrawerLayoutAndroid'),
   Image: require('Image'),
   ListView: require('ListView'),
   MapView: require('MapView'),
@@ -29,20 +28,15 @@
   Navigator: require('Navigator'),
   NavigatorIOS: require('NavigatorIOS'),
   PickerIOS: require('PickerIOS'),
-  ProgressBarAndroid: require('ProgressBarAndroid'),
   ProgressViewIOS: require('ProgressViewIOS'),
   ScrollView: require('ScrollView'),
   SegmentedControlIOS: require('SegmentedControlIOS'),
   SliderIOS: require('SliderIOS'),
-  SwitchAndroid: require('SwitchAndroid'),
   SwitchIOS: require('SwitchIOS'),
   TabBarIOS: require('TabBarIOS'),
   Text: require('Text'),
   TextInput: require('TextInput'),
-  ToastAndroid: require('ToastAndroid'),
-  ToolbarAndroid: require('ToolbarAndroid'),
   TouchableHighlight: require('TouchableHighlight'),
-  TouchableNativeFeedback: require('TouchableNativeFeedback'),
   TouchableOpacity: require('TouchableOpacity'),
   TouchableWithoutFeedback: require('TouchableWithoutFeedback'),
   View: require('View'),
@@ -56,7 +50,6 @@
   AppRegistry: require('AppRegistry'),
   AppStateIOS: require('AppStateIOS'),
   AsyncStorage: require('AsyncStorage'),
-  BackAndroid: require('BackAndroid'),
   CameraRoll: require('CameraRoll'),
   Dimensions: require('Dimensions'),
   Easing: require('Easing'),
@@ -78,7 +71,6 @@
   NativeAppEventEmitter: require('RCTNativeAppEventEmitter'),
   NativeModules: require('NativeModules'),
   Platform: require('Platform'),
-  processColor: require('processColor'),
   requireNativeComponent: require('requireNativeComponent'),

   // Prop Types

As you can see, all the Android modules are absent.
Also, I noticed that 0.11.0.tgz is 33M, 0.11.1.tgz is 622K and 0.11.2.tgz is 715K. Weird.

Yes, it works if I revert back to 0.11.0. Don't you use deployment tests?

Android is currently not tested in the open-source repo.

Android is tested now, this should work!


Fix Qt thread id comparison

@dev-gh please review and try on your machine

@pestOO @dev-gh updated

@anosach-luxoft
I've used the same approach here https://github.com/anosach-luxoft/sdl_core/pull/19
Reviewed


fiixed the elem.focus type error

bro I simply added an undefined check by adding a "?" sign before ref.current statements.

Goto the file where you pasted the multi selector code.
search using Command + F key and type focus() in the search bar.
then you will find 3 inputRef.current?.focus(); statements .
simply change them to inputRef?.current?.focus();

related to #127


vcenter_ip bosh director variable confusingly named

Feature Request
Detailed Description
the vcenter_ip variable will accept a DNS hostname, which seems like a more robust solution than hardcoding an ip address for the VCSA appliance.  Neither the variable name or the documentation mention this.
https://bosh.io/docs/init-vsphere/
Configure vcenter_ip (e.g. '<IP_ADDRESS>') with the IP of the vCenter.

Context
CF Toolsmiths recently had to reconfigure the ip address on a VCSA appliance and multiple teams running bosh directors on the cluster subsequently were unable to deploy.
Error: Unknown CPI error 'Unknown' with message 'Please make sure the CPI has proper network access to vSphere. (HTTPClient::ConnectTimeoutError: execution expired)' in 'create_vm' CPI method

The docs and variable name led users to believe they should hard code the ip address of the VCSA appliance managing the cluster.  If the deployment had used a DNS hostname instead, the transition would have been transparent.
Possible Implementation
Updating the docs at a minimum, renaming the variable if there is a elegant way to do that.
Complexity

[X] Low - Simple enhancement or bug fix, no architectural changes or refactoring

closing and reopening to create a tracker story

Closing as fixed.


updated README

changed the Google Environmental name in the code block to the correct reference, e.g. working code.
Fixes # .
Changes proposed in this pull request:

envrironmental variable name

@davidverweij Thanks for catching this!


Tourney Grounds

https://thronesdb.com/card/03042
The two keywords are not separated by a space, so searching by keyword doesn't show this card.

thanks


amda ace imf column names

Speasy version:1.2.7
Python version:3.10.xxx
Operating System:mac

Description
Can't reproduce a plot of the documentation
What I Did
import matplotlib.pyplot as plt
from datetime import datetime
%matplotlib widget

plt.figure()
ace_mag = spz.get_data(spz.inventories.data_tree.amda.Parameters.ACE.MFI.ace_imf_all.imf, datetime(2016,6,2), datetime(2016,6,5))
ace_mag.plot()
plt.tight_layout()
plt.show()

I get :

Column names are ['bx', 'by', 'bz'] whereas they are imf[0] etc. on documentation.

tested the documentation page on Binder, gives me bx, by bz too.
AMDA web GUI seems to propose also bx, by bz
Weird thing that pushed me to write this issue is that Vincent G. has imf[0] etc:
spz.get_data('amda/imf', "2016-6-2", "2016-6-5",disable_cache=True, disable_proxy=True ).columns
Out[83]: ['imf[0]', 'imf[1]', 'imf[2]']

automagically "solved" with pip install —user --upgrade --force-reinstall speasy
weird.

strangely @nicolasaunai and myself yesterday had spz.version = 1.1.2
after reinstall I have spz.version = 1.2.7

yeah well I think I made a typo, I had 1.2.2 :/
So the reinstall did indeed solved your "issue" (which really was not an issue but just that you had an older version than 1.2.0 installed).


Type handling

In GitLab by @zebedee.nicholls on Apr 6, 2023, 15:53
The problem
I was using the PRIMAP bookshelf the other day, and something not ideal happened. In short, the data appears to be saved or read by Pandas (not sure where exactly this happens) as mixed type, which means the categories aren't handled correctly. See the code excerpt below
>>> shelf = bookshelf.BookShelf()
>>> primap_book = shelf.load("primap-hist", version="v2.4.2")
>>> emissions = primap_book.timeseries("by_country")
>>> emissions.get_unique_meta("category")
# 2 and 1 appear as both ints and strings, comes with a DtypeWarning
# from pandas
[0,
 1,
 '1',
 '1.A',
 '1.B',
 '1.B.1',
 '1.B.2',
 '1.B.3',
 '2',
 2,
 '2.A',
 '2.B',
 '2.C',
 '2.D',
 '2.G',
 '2.H',
 '3',
 '3.A',
 '4',
 '5',
 'M.0.EL',
 'M.AG',
 'M.AG.ELV',
 'M.LULUCF']

Definition of "done"
The mixed types are removed. For this particular example, I think everything should be a string. There may be more general principles that can be applied having seen this use case though.
Additional context
The workaround is relatively easy, emissions["category"] = emissions["category"].astype(str).
This test fails (but feels like the wrong way to test for this bug as it is quite slow and a very high-level integration test for what seems like a low-level issue)
def test_primap_book(local_bookshelf):
    shelf = BookShelf()
    primap_book = shelf.load("primap-hist", version="v2.4.2")

    emissions = primap_book.timeseries("by_country")
    assert emissions.get_unique_meta("category") == [
        "0",
        "1",
        "1.A",
        "1.B",
        "1.B.1",
        "1.B.2",
        "1.B.3",
        "2",
        "2.A",
        "2.B",
        "2.C",
        "2.D",
        "2.G",
        "2.H",
        "3",
        "3.A",
        "4",
        "5",
        "M.0.EL",
        "M.AG",
        "M.AG.ELV",
        "M.LULUCF",
    ]

primap-hist repo (and test) has been moved to https://github.com/climate-resource/bookshelf-primap-hist


Motion Sickness Teleport System

Describe the problem
Hi, with the current teleport system provided by the mixed reality toolkit, I get motion sickness when moving back or turning.
Describe the solution you'd like
Similar to the solution provided in the mixed reality portal (black shadows appear on the edge when turning or moving back):
https://user-images.githubusercontent.com/61323363/136593913-0c1fe757-16aa-47f9-aa5d-bbcb11d14087.mp4

Here are some locomotion systems that might be useful to look to get ideas of what kind of locomotion to implement for VR.
Locomotion Vault
Database, created by Microsoft, of over 100 locomotion techniques, but no implementations.
GitHub: https://locomotionvault.github.io/
Paper: https://www.microsoft.com/en-us/research/publication/locomotion-vault-the-extra-mile-in-analyzing-vr-locomotion-techniques/
An Evaluation Testbed for Locomotion in Virtual Reality
Locomotion methods implemented: Arm swinging, Walking-in-place (WIP), Cyberith’s Virtualizer, Joystick.
GitHub: https://github.com/VRatPolito/LET-VR
Paper: https://dx.doi.org/10.1109/TVCG.2020.3032440
Video of Locomotion methods: https://www.dropbox.com/sh/82fw7i277oo0fef/AACl-FxRi6imX7NUpbYIpEhCa/Training?dl=0&subfolder_nav_tracking=1
Scenarios (tasks): https://www.dropbox.com/sh/82fw7i277oo0fef/AABWOcbEH2ReGD4JK8ZeHNmAa/Scenarios?dl=0&subfolder_nav_tracking=1
Redirected Walking Locomotion Techniques Toolkit
GitHub: https://github.com/USC-ICT-MxR/RDWT
Paper: https://ieeexplore.ieee.org/document/7859537
XinReality - Locomotion
A database of Locomotion techniques, similar to Locomotion Vault, but there are some links to source code of the locomotion techniques.
Website: https://xinreality.com/wiki/Locomotion
Travrsal
GitHub: https://github.com/WetzoldStudios/traVRsal-sdk
Website: https://www.travrsal.com/
I hope this helps.

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all-new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visit https://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Fix minor goof on ArrayBuffer

It's ArrayBuffer[Symbol.species], not someBuffer[Symbol.species].

Could you link some docs that support this change?

In a node shell, try
ArrayBuffer[Symbol.species] === ArrayBuffer

If that's inadequate, then check in the ES6 spec under 24.1.3 (Properties of the ArrayBuffer Constructor) for <IP_ADDRESS> (get ArrayBuffer [ @@species ]). It lists a single step: "Return the this value." Since it's attached to the constructor, the this value is the constructor itself.


bf: ZENKO 790 redis limit return

Redis suddenly decided to return key values as strings, so new put sizes were being concatenated instead of added to the existing location value

Hello dora-korpar,
My role is to assist you with the merge of this
pull request. Please type @bert-e help to get
information on this process.
Status report is not available.

Integration data created
I have created the integration data for the additional destination branches.

this pull request will merge bugfix/ZENKO-790-location-limit into
development/8.0
w/8.1/bugfix/ZENKO-790-location-limit will be merged into development/8.1 (pull request #1403)

The following branches will NOT be impacted:

development/6.4
development/7.4

Follow integration pull requests if you would like to be notified of
build statuses by email.

Waiting for approval
The following approvals are needed before I can proceed with the merge:

2 peers

Peer approvals must include at least 2 approvals from the following list:

@RachedBenMustapha

@rahulreddy

@dora-korpar

Are there no unit/functional tests for this method?

There are unit tests, but since the value in question came from redis, it would require integration tests to test, of which there are none currently

We have Redis running as part of S3 CI, we should be able to put something together to test this independently without Integration. I am approving this PR keeping in mind the time constraint but please follow up with a functional test for this.

In the queue
The changeset has received all authorizations and has been added to the
relevant queue(s). The queue(s) will be merged in the target development
branch(es) as soon as builds have passed.
The changeset will be merged in:

:heavy_check_mark: development/8.0

:heavy_check_mark: development/8.1

The following branches will NOT be impacted:

development/6.4
development/7.4

There is no action required on your side. You will be notified here once
the changeset has been merged. In the unlikely event that the changeset
fails permanently on the queue, a member of Release Engineering will
contact you to help resolve the matter.
IMPORTANT
Please do not attempt to modify this pull request.

Any commit you add on the source branch will trigger a new cycle after the
current queue is merged.
Any commit you add on one of the integration branches will be lost.

If you need this pull request to be removed from the queue, please contact a
member of Release Engineering now.

I have successfully merged the changeset of this pull request
into targetted development branches:

:heavy_check_mark: development/8.0

:heavy_check_mark: development/8.1

The following branches have NOT changed:

development/6.4
development/7.4

Please check the status of the associated issue ZENKO-790.
Goodbye dora-korpar.


Initial Memoria Misery Ex timeline and triggers

I have created timeline and triggers files for Memoria Misery Ex, but since I have not many logs which records phase 5 (the phase when Fortius starts). I can only assume it works fine until Gunshield appears.
and there were some tricky hack for phase 3 (after the Susano-like dps check, Varis would randomly use 3 groups of skills), I have to determine which "Magitek" skill would be used with "Gunshield" skill and wait for it charged. Also, I am not familiar with the timeline format, I got a trouble when judge which random skills would be used in timeline, if you have a better method, please let me know.

Thank you for this! It looks like there's a conflicting change for triggers in #1320.  I like all the translations you have here and you have some triggers that that doesn't.  I think to resolve the conflicts, I'd like to land #1320 first, and then merge your changes in on top of that.
If you want to put up a separate change for the timeline (and renamed it to be varis-ex.txt), that could get merged cleanly right now, since #1320 doesn't add a timeline.

Thanks for taking the time to put all that together!  Once the other pull request lands, quisquous or I will go through your triggers with you and see what might need to be done there. The timeline looks really nice, and I hope to see that merged soon.

OK, as @quisquous said, I would put only timeline file so that it can be merged.
And fix those timeline bugs following what @JLGarber said.
I found that I use the master branch into Pull Request, that is not right. I wish to close this PR and I re-pull it as another branch once.

I have open a new Pull Request #1326  which is not master branch anymore, so I have to close this PR. Please merge my new PR.


Releases

Many thanks for the mediarepo <3
Any possibility to have releases, to do a versioned docker tag to pull? Using this in production it's a bit scary to always pull latest.
Thanks!

The idea was that I didn't expect anyone to seriously use this while it was beta or pre-1.0. There's a milestone for things before 1.0 is tagged: https://github.com/turt2live/matrix-media-repo/milestone/1
The release process for this also sucks because of multi-platform builds for binaries (or maybe I just don't publish them?). Either way, something for v1.0 to solve.

RC1 is out: https://github.com/turt2live/matrix-media-repo/releases/tag/v1.0.0-rc.1


"Did not get any data blocks" kills the entire list download.

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2019.11.05
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

I've searched and found people talking about the "did not get any data blocks" error. Says it's something to do with Youtube's CDN not having that format and to try a different format. That's great, I'll do that. But my problem is that reaching this error kills the entire playlist download. I'm trying to download a Youtube playlist of 120 videos. It gets the first 23 fine, then hits the error on video 24. But then it doesn't continue with video 25, the whole thing just dies and no more videos are downloaded.
Workaround is to add "--playlist-start 25" to the command. But then when the error happens again on video 49, it dies again. It's really annoying having to restart the list every time this error pops up.

-i, --ignore-errors              Continue on download errors, for example to
                                 skip unavailable videos in a playlist

Ah, that worked great! I feel this should be default behavior but oh well, as long as there's an option. I'll just add it to my config. Thanks!


Testing symbol presence: also take note of small objects

The S symbol class wasn't checked.
Notified by Sebastian Andrzej Siewior

+1

Thanks.  Merged.


Upload completed FXGL games for public access

Sample game demos have been published to https://fxgl.itch.io/


Springcloud gateway + BFF

I'm looking for a way to introduce BFF(backend for frontend) into BFF. Basically my idea is implementing some web controllers in springcloud gateway, and making all requests go through global/customized filters, then and then enter my own web controllers(a uri like 'forward:/api/hello/gulu').
For example my routes is defined as below:
@Bean
public RouteLocator customRouteLocator(RouteLocatorBuilder builder) {
    return builder.routes().route("hello", r -> {
            logger.debug("Hello, there...{}", r);
            return r.path("/**").filters(f -> {
                logger.debug("add request header...");
                return f.addRequestHeader("header1", "header-value-1");
            }).uri("https://httpbin.org/get");
        }).build();
}

and a global filter:
public class CustomGlobalFilter implements GlobalFilter, Ordered {

    @Override
    public Mono<Void> filter(ServerWebExchange exchange, GatewayFilterChain chain) {
        logger.info("custom global filter:{}", exchange.getRequest().getURI());
        return chain.filter(exchange);
    }

    @Override
    public int getOrder() {
        return -1;
    }
}

and a customized web controller:
@RestController
@RequestMapping("/api")
public class HelloController {

    private Logger logger = LoggerFactory.getLogger(InsuranceQueryController.class);

    @GetMapping("/hello/{name}")
    public String hello(@PathVariable String name) {
        logger.debug("hello, {}", name);
        return "hello," + name;
    }
}

My expectation is that when request url 'http://localhost:18006/api/hello/ramon', it shout intercepted by route 'hello', and then at least filter by my CustomGlobalFilter, and finally enter HelloController. By the actual result is the request handled by RequestMappingHandlerMapping, just like a simple pure web controller, no any route or filters triggered.
I am wondering whether it is right to introduce BFF into springcloud gateway, or there is some elegant way to implement this?

Your controller matches /api first, it never hits the gateway

@spencergibb that is exactly what i want, make the request hit gateway first. I there any means to implement it?

Change the controller mapping to something else and move /api to a gateway route

yes, but if client access the web controller's URL directly, it will bypass gateway. That is some kind of risk, all my filters, something like rate limiter, will be ignored.

Move the controllers out of gateway I guess.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Please invite me to the GitHub Community Organization

Name
Ehmad Saeed
Discord Username (if applicable)
justehmadsaeed#3168
Additional Context
I wanna join it to contribute to the FOSS community and be part of a greater cause.

It's great having you contribute to this project
If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Invitation sent for the GitHub Organisation. Welcome to the community 🎉#
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/jaidevstudio/people
Like this

Tips for practising:
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Please check your email or organisation page 🙏


Make an option to have OSL isconnected() return 1 for userdata parameters

This adds a new ShadingSystem attribute, "userdata_isconnected". The
default of 0 is the old/usual behavior that most applications want.
However, setting this attribute to 1 will cause the OSL built-in
function isconnected(paramname) to return 1 not only for connected
parameters, but also for parameters that may be bound to userdata
(i.e., those that have lockgeom=0).
The use case for this involves a renderer implementation that can mix
C++ and OSL nodes within a shading group. The outputs exported by the
C++ nodes propagate their values to the downstream OSL node via
get_userdata(), hijacking the mechanism that is generally used geometric
primitive variable interpolation. But from the point of view of the
downstream node, they should semantically appear to be a connection.

LGTM


[Pipeline] The gitlab member has parent group owner permissions, but cannot fetch repo under the subgroup.

Steps to reproduce
Add members to gitlab subgroup or project and set the permissions to the maintainer or owner.
The pipeline can fetch the repo under the subgroup.
But add members in the gitlab parent group, and set the permission to the owner.
The pipeline cannot fetch the repo under the subgroup.
rancher/rancher:master

This is pending on upstream gitlab issues:
https://gitlab.com/gitlab-org/gitlab-ce/issues/43684
https://gitlab.com/gitlab-org/gitlab-ce/issues/46510
This project API is not showing users' group permission in this case. Need to add the user to the subgroup or the project member list explicitly.


46 linesandplanes normalequationline

Applet should be done just need to change the starting position of the camera

Can we close this PR untill the final changes (described in the issue) are made? code looks good tho


Double download when browser only has partial srcset support.

While trying to implement a partial responsive image solution for WordPress, we've run across an issue where picturefill is causing a double download on browsers with only partial support for srcset (read: Safari and possibly the in development Spartan browser).
The markup pattern we're using is srcset + sizes, or roughly this:
<img src="path/to/image.jpg"
    srcset="path/to/image.jpg 300w, path/to/image-600.jpg 600w, path/to/image-1200.jpg 1200w"
    sizes="(max-width: 600px) 100vw, 600px"
    width="600" height="400" alt="alt text"
/>

I've created a test case to show an example of the markup and a possible no-src solution being used by The Guardian and others to work around this issue here: http://codepen.io/joemcgill/full/JoVVBv/
Currently, Safari on iOS will trigger a double download either way and Safari 8.02 will trigger the double download if a src attribute is present. I know there is no way to stop the browser from triggering that download, but it would be nice if there was a way to test for partial srcset support and optionally bail on the second request if we know the first is going to be loading anyway.

This is a tough one.
But I don't really get your indentation or I don't understand what you want to achieve. You still want to use an initial src attribute? And picturefill should still polyfill and create a double request, if a src attribute is used, while we should not polyfill, if the Safari iOS bug is detected? Why should we handle these cases different?
In combination with picturefill, I would suggest to not use a src attribute. Additionally see also [this markup pattern for iOS](https://github.com/scottjehl/picturefill/issues/438#issuecomment-75647795
In general a bug detect has to happen async and would look something like this:
(function(){
	var loadBuggyWidth = false;
	var srcset = 'data:image/gif;base64,R0lGODlhAQABAAAAACH5BAEKAAEALAAAAAABAAEAAAICTAEAOw== 9w';
	var img = document.createElement('img');
	var load = function(){
		loadBuggyWidth = true;
	};

	if(('srcset' in img) && !('sizes' in img)){
		img.srcset = srcset;
		img.onload = load;
	}
})();

I asked @joemcgill to file this one so we could take a second look at src support. I want to research the gap between browsers with BG-breaking prefetching and browsers with native respimg support, because I’m wondering if we couldn’t bring back a little Just Use JavaScript™ to cover some cases with src.
Chrome “broke” our original responsive images script on the Boston Globe site when it ramped up prefetching; same with Firefox shortly after. But I think both were in “evergreen” turf by that point, and both will have native picture landing soon. Depending, we might be able to bring back an approach like the one we’re using on BG—which still works in older and less preparse-y browsers—and only risk the double download in the evergreens between “BG’s responsive images break” and “native picture support.”

ping @wilto and @aFarkas to checkup on if we want to try and handle double downloads.

I spent a little time tinkering and thinking this over, and landed back where I started. I’m gonna close this one out for now, unfortunately.


min/max on typed lists extremely slower than on python lists

Description
Calling min/max on Numba's typed lists is extremely slower (>10x) compared to when operating on native Python lists.
How to reporduce
import numpy as np
from numba.typed import List
x = np.random.rand(10000).tolist()
xt = List()
xt.extend(x)

%timeit min(xt)
22.6 ms ± 272 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

%timeit min(x)
166 µs ± 5.23 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

@ma-sadeghi thanks for reporting this, I presume this is a result of having to unbox the elements of the typed list into Python objects during min and max.

Thanks for the report. I think this is expected behaviour, the typed.List uses native memory storage and accessing that through Python calls (as would happen in CPython's min) requires pulling the data out of the storage and converting it to a CPython internal representation for use. If however one does a min call inside a @numba.jit compiled function following the same pattern, the native memory storage can be used directly and the performance is much better.
In [10]: %paste
import numpy as np
from numba.typed import List
x = np.random.rand(10000).tolist()
xt = List()
xt.extend(x)

## -- End pasted text --

In [11]: %timeit min(xt)
34.6 ms ± 327 µs per loop (mean ± std. dev. of 7 runs, 1 loop each)

In [12]: %timeit min(x)
221 µs ± 244 ns per loop (mean ± std. dev. of 7 runs, 1000 loops each)

In [13]: %timeit min(xt)
34.1 ms ± 191 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

In [14]: from numba import njit

In [15]: @njit
    ...: def jmin(x):
    ...:     return min(x)
    ...:

In [16]: %timeit jmin(xt)
46.1 µs ± 8.48 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

I was looking for __min__ and __max__ contracts, but they don't seem to exist?

After some more thought, I believe we could implement min() and max() as methods on the List class, would that help.

@stuartarchibald Thank you for looking into this, actually I'm using min/max inside a Numba jitted function, so probably I'm not being affected. @esc Thank you, looking forward to it.

Closing this issue as it seems to be resolved. If this is not the case please re-open with a comment about any item that appears to be unresolved. Many thanks.


game crashing while cutting down trees, can't re enter world without crashing

Describe the bug
game crashed while cutting down trees, save will not reopen
To Reproduce
not sure, i was in the middle of clearing a forest and it just crashed and my save will not open anymore without crashing.
Rendering Glitches
optifine installed
here's my crashlog >>
https://pastebin.com/7ZyJbd0H

Duplicate of #546. It was determined to be Building Gadget's problem, not ours. I suggest you make sure you're on their latest version and if not look into making a report on their end.


Emoji Support

Can someone help me with sending emoji's along with the message. I tried using unicode encoding but was not successful with that. Thank you.

You can try the emoij package: https://pypi.org/project/emoji/

Yes, i tried this package too. There's no error but it shows different characters in the message instead of the emoji.

do have the following first line in your script: # -*- coding: UTF-8 -*-

Im using emoji without error using python 3.5.2 thats uses by default the UTF-8 encoding. Maybe you're using python 2.X and you need to set the encoding like @kevinkit said.


Email alerts for All England

This PR adds a sign-up form on the All England dashboard, plus a new management command send_all_england_alerts which does just that.
The emails themselves will no doubt be subject to many changes (they're currently just a copy of what's already on the dashboard) but they're not quite "user facing" until the point we actually decide to send them out so it might be acceptable to merge and start collecting signups before iterating on the email content.
Relates to #1553 and closes #1503
Example alert email:

Looks great!
I'd just suggest:

where we say on various measures we should point out that they cover safety, efficacy and cost-effectivenss,
and then after if all CCGs had prescribed as least as well as the median, add "on the cost-saving measures":

This is great and another new feature for a new user group.

Suggest Title Tweak: OpenPrescribing All England Prescribing Dashboard: Monthly Update
Before settling on this tweak I wrote 3 versions that had cost/financial in the title. My first reading of the update was that it is exclusively financial. Having read again and written down titles I don't think it is a good idea to use cost/financial in the title, mainly as things like Low Priority aren't just about cost.

Tweak the feedback invite
This is the first iteration of our new experimental service and we are keen for feedback. Please respond to this e-mail with thoughts and ideas for features you would like to see.

Re:launch discuss at DataLab day tomorrow? I'd be keen to ship and start getting feedback. I think  the type of things we would like to do will take a bit of time. Presume when matrix fully implemented, new analyse form etc. might be a better time to return to it.

Shall we set a date in the diary of when to return to this?

@HelenCEBM Thanks. I've also applied those changes to the All England dashboard itself, from where the original text was lifted.
@brianmackenna Thanks. I've incorporated both those changes. Would definitely make sense to discuss launch date tomorrow and set a date to revisit with further improvements. You had loads of great ideas in  the Google Doc.

Agreed yesterday to :shipit:
And revisit in 2 or 3 months


Correct handling of Directive controllers

Controllers for Directives must be handled differently than normal controllers:
for a controller defined on a directive angular creates a empty object which is then passed as thisto the controller constructor; every directive defined on a tag then may add members to the passed in this.
The current (0.1) implementation does not handle this correctly (see discussion on on issue #28).

Solution: use controller function and define controller as a facade trait instead of a Scala class;
Remodelled Directive API:
trait MyDirectiveCtrl {
  var foo: String = js.native
}

class MyDirective {
  override type ControllerType = MyDirectiveCtrl

  override def controller(ctrl: ControllerType, scope: Scope, elem: JQLite, attrs: Attributes) : Unit = {
    ctrl.foo = "bar"
  }

  override def postLink(scope: Scope, element: JQLite, attrs: Attributes, ctrl: ControllerType): Unit = {
    /* ... */
  }
}


[TEST] - This is a test issue to be deleted.

This is a test issue only, testing a workflow and some notifications.

@samjustus Did you get this?

@caroline-suse-rancher Did you get this?

Yep

@samjustus - Again

OK @caroline-suse-rancher and @samjustus - this concludes our testing. I will be closing this issue now.
Also pinging @MKlimuszka and @Sahota1225 just to see their experience with pings as well.


Azure OpenAI Integration

Azure OpenAI is a common enterprise option for a lot of companies integrating LLMs into their environment. It's our only option for genAI, so any integration would be great!

@thickmn - thanks - we are working on it!

@thinkmn - we have merged a proposed fix into the main branch - could you check it out and confirm if this will meet your needs in terms of configuration of the AzureOpenAI client - please check out the example - appreciate your help and support to test and make sure it meets your needs!

@doberst - thanks for the quick response here! My first go at testing looks like it tries to call openai with a user managed api key and can't successfully set the azure with the environment variables in the example. I'm going to spend more time on this tomorrow, but just as an FYI, when using langchain's integration, I had to pass:
OPENAI_API_TYPE=azure
OPEN_API_BASE
OPEN_API_KEY
OPEN_API_VERSION
DEPLOYMENT_NAME

Hi, I had the same problem as thickmn,
I also have to use the GPT throught azure, I try to use the sample in the link, but It doesn't work ...
My GPT is the 3.5-turbo
However in the requisition I use
AZURE_OPENAI_ENDPOINT
AZURE_OPENAI_API_KEY
api_version = "2023-07-01-preview"
and to call my model is test:
response = client.chat.completions.create(
model = "test",
messages =....
@doberst can you help me, please?

@Clima2024 - happy to help with this - and sorry that you have run into an issue ... without sharing any confidential information, could you share the key 2-3 line code snippet, including the OpenAI configs...    We will also test from our Azure account in parallel....

Thanks @doberst  .. no worries, so in my local env what I do is the following:
import os
from openai import AzureOpenAI
with the var registered in my environment, I do:
client = AzureOpenAI(
azure_endpoint = os.getenv("AZURE_OPENAI_ENDPOINT"),
api_key=os.getenv("AZURE_OPENAI_API_KEY"),
api_version="2023-07-01-preview"
)
#then I call GPT as:
response = client.chat.completions.create(
model = "test",
messages=[
{"role": "system", "content": "You are a helpful research assistant."},
{"role": "user", "content": "Just say hi"}]
)
print(response.choices[0].message.content)
#This model name "test" is the name of the instalation but we call one gpt-35-turbo (in azure they don't use the dot)

Hi @doberst
I am also trying to use the embeding (text-embedding-3-small) at my local azure openai account, and it failed.
Not sure why my infra named the models, but here a use:
model="test-embeddings"
to do the embeding. As long as you are working with it, could you check the embeding calling for azure too? (if it is not ask too much.)


Per-frame metadata is not enabled at the OS level! But, I want to get the time-stamp of image grab.

Required Info

Camera Model
D455

Firmware Version
V2.50

Operating System & Version
Ubuntu1804

Kernel Version (Linux Only)
4.9.201-tegra )

Platform
NVIDIA Jetson Xavier NX

SDK Version
2.50 }

Language
C/C++ }

Segment
Robot }

Issue Description

Per-frame metadata is not enabled at the OS level! But, I want to get the time-stamp of image grab.

Hi @ping28198   What method did you use to install the RealSense SDK on your Jetson, please?  If you built it from packages or built from source code with CMake using the RSUSB backend method then support for hardware metadata should be included in the SDK build.
If you built from source code but did not use the RSUSB backend method (i.e you built with the V4L backend) then you need to apply a kernel patch in order for hardware metadata to be available, as described in the link below.
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_jetson.md#building-from-source-using-native-backend

Hi @ping28198  Do you require further assistance with this case, please?  Thanks!

Thanks very much @ping28198 for the update!