Easily identify entities related to parks

Hello and thanks for the integration.
With the current implementation, you choose a resort to add. Unfortunately, this then includes merges all rides for all parks and are all grouped under a single device for the resort. This makes it very hard to work out which entities are for which parks without manually sorting them (e.g. Disneyland Resort includes rides for Disneyland and California adventure, but I might only be going to one of them)
Therefore it would be good if either

When we set up the integration we select parks not resorts or
When we select a resort, it creates a device per park instead of one for the resort and the entities names are clearly marked and assigned which park they are for instead of with the resort (e.g. sensor.guardians_of_the_galaxy_mission_breakout_disneyland_resort would be sensor.guardians_of_the_galaxy_mission_breakout_disney_california_adventure_park)

Hi @BottlecapDave thanks for the issue.
I was about to come onto here and say that it's out of my hands and it's how the https://themeparks.wiki/ API does it, but I did check first, and the API does have both concepts, and this Integration should do something with the resort and parks.
Probably creating a new device per park makes sense, and then having an attribute on the device that specifies with resort will be a good MVP.
I think that device creation is a bit broken at the moment in the integration anyway, so this does need looking at properly, but having the dropdown select from parks instead of resorts work be the quickest fix.
I don't know anything about to code a migration strategy for existing users, so that would be helpful. There probably aren't that many users of the integration though, so i'd be happy to just swap to picking parks over resorts and call it a new major version.
@BottlecapDave do you want to have a bash at that and file a PR?

Thanks for getting back to me.
Sorry to be a pain, but I ended up creating my own variation of the integration for what I wanted to achieve.
When I created this issue, I was hoping to build an integration that worked with your integration which monitored the standby/show times and notified me when the ride/attraction with the shortest time changed. However even with my suggested changes I quickly discovered it was going to be a lot more work than if I just got the data myself, which is what I've ended up doing.
I've yet to try my idea in the real world, so it might not work out :)
Feel free to close this issue.


[Tracking] IBM IKS distrubition for Kubeflow v1.5

Use this issue to track the progress of the testing for Kubeflow v1.5
kubernetes 1.21

[x] single-user deployment
[x] multi-user deployment
[x] validation of the links on dashboard
[x] e2e-mnist (using TFJob)
[x] nested loop
[x] nested pipeline
[x] pipeline caching
[x] Trusted AI pipeline
[x] katib pipeline (using Job)

finished test against v1.5.0-rc.1. waiting for rc.2.
going to verify these test cases on Kubernetes 1.22 as well. foresee some failures there.

re-ran all test cases again for v1.5-rc2. All passed!


Same acc, recall, precision, f1 when it comes to trainer.evaluate()

trainer.evaluate(eval_dataset=encoded_test_dataset)
output is
{'eval_loss': 0.6165900230407715,
'eval_precision': 0.681265206812652,
'eval_recall': 0.681265206812652,
'eval_f1': 0.681265206812652,
'eval_accuracy': 0.681265206812652,
'eval_runtime': 2.1712,
'eval_samples_per_second': 189.3,
'eval_steps_per_second': 5.988,
'epoch': 18.72}

Hello!
This is certainly possible simply due to how precision and recall are calculated. For example in a binary case if the number of false negative and false positive samples is the same, and if the number of true negative and true positive samples are the same:

Positive
Negative

Positive
TP=68
FP=32

Negative
FN=32
TN=68

In this table, the recall is TP / (TP + FN) = 68 / (68 + 32) = 0.68 and precision is TP / (TP + FP) = 68 / (68 + 32) = 0.68. For accuracy, we get (68 + 68) / (68 + 32 + 68 + 32) = 0.68.
However, it admittedly is a bit of a statistical anomaly. How did you calculate the precision, recall and F1?

Also, are you using SetFit? SetFit doesn't output eval_steps_per_second, eval_runtime, epoch, eval_samples_per_second, etc.

Tom Aarsen


Add documentation for known issue with use of custom codecs on version 2.7.0

Description
Adds known bug related with usage of custom codecs in OpenSearch 2.7.0 for segment replication enabled indices.
Issues Resolved
Fixes https://github.com/opensearch-project/documentation-website/issues/4194
Checklist

[ ] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@hdhalter @Naarcha-AWS : Can you please review this change ?

@Naarcha-AWS : As the bug was impacting 2.7.0 only and was fixed in 2.7.1+, I think we don't need backport to 2.11 ?

@Naarcha-AWS @hdhalter : Can you please merge this change


Support custom message converters

User provided custom message converters gets the precedence over existing message converters.
Module author would need to provide the collection of converters with the bean name customMessageConverters
Refactored message converter configuration

No test...
Is it OK already?..

Sorry, it is definitely not ok without tests. So far I have been running this manually by adding a customMessageConverters bean in the s-c-s module.
@Bean
	public List<AbstractFromMessageConverter> customMessageConverters() {
		List<AbstractFromMessageConverter> converters = new ArrayList<>();
		converters.add(new AbstractFromMessageConverter(targetMimeTypes) {
			@Override
			protected Class<?>[] supportedTargetTypes() {
				return new Class<?>[] { byte[].class };
			}

			@Override
			protected Class<?>[] supportedPayloadTypes() {
				return new Class<?>[] { String.class };
			}

		});
		return converters;
	}

We have a long list of tests that are yet to be ported from XD to here. We will have them addressed soon.

Closing this PR for now as we want to provide a better approach that uses annotation

to specify override the custom message converters over the existing converters
to specify ordering of the converters.

The module author would specify something like this:
@MessageConverters(override= true/false, first=true/false)


[FEATURE] Use authentication user to bind and to query

Actually I can either us the authentication user to bind or BaseDN user to search.
Search implies to deal with ldap credentials which is not always necessary.
I want the authentication user to bind AND to search.
One typical use case:

check whether the the authentication user is generally allowed to access the network -> bind with user
check whether this user is part of a certain ldap group -> fetch user record

Hello, @siaccarino

I want the authentication user to bind AND to search.

This is already the current behavior.
You can read the Operations Mode docs to familiarize yourself with the mode's definitions.

Running in bind mode the user auth and all subsequent validations will be made with the user/password provided
Running in search mode the user/password provided will be validated and all subsequent validations will be made with the BindDN and BindPassword.

This part of the code shows how authentication is performed:
https://github.com/wiltonsr/ldapAuth/blob/ddd7a2660174faa39bc4c8a65360d3b347b02c8d/ldapauth.go#L297-L326
Please let me know if this answers your question.

Closing due to inactivity.
If you need more help you can reopen this issue.


Correction for raw affiliation Montpellier Interdisciplinary Center on Sustainable Agri-Food Systems (MoISA), Univ Montpellier, CIHEAM-IAMM, CIRAD, INRAE, Institut Agro, IRD, 34060 Montpellier, France

Correction needed for raw affiliation Montpellier Interdisciplinary Center on Sustainable Agri-Food Systems (MoISA), Univ Montpellier, CIHEAM-IAMM, CIRAD, INRAE, Institut Agro, IRD, 34060 Montpellier, France
raw_affiliation_name: Montpellier Interdisciplinary Center on Sustainable Agri-Food Systems (MoISA), Univ Montpellier, CIHEAM-IAMM, CIRAD, INRAE, Institut Agro, IRD, 34060 Montpellier, France
new_rors: 003vg9w96;05kpkpg04;051escj72;05q3vnk25;03rnk6m14;0005r2j17;04abkkz03;011xg1225
previous_rors: 003vg9w96;05kpkpg04;051escj72;05q3vnk25;03rnk6m14;0005r2j17;04abkkz03
works_examples: W4391110314
contact: 2dd1a01aa86351d82b4bce167e1f1753:b67e02542a94ad4d @ supagro.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


engine: wire up container build and deployer

Mostly getting this up to get some eyes on it. I wouldn't merge this in yet.
Involved writing some code to wrangle which pod we need to be talking to, which @maiamcc wrote.
Currently updating the container takes more than 5 seconds. =\
Three outstanding issues:

The output currently looks terrible

files changed. rebuilding blorg_frontend. observed 1 changes: [/Users/dan/go/src/github.com/windmilleng/blorg-frontend/main.go]
Deploying to pod: blorg-fe-6f9cddf8cf-ggq9j
Got container ID for pod: 5393f4a83f7d25124249fcfc132ca6f3b01e617d554d04f441f8382ec0dfaebe
Updating container...
Container updated

We only succesfully update one time, after which it throws this error: Fixed!

Updating container...
build failed: Error: No such container:path: :/

Do we want to have a flag for this behavior?

I think it might be a bug with how we're handling the pathMappings?

Will re-organize, write more tests and submit as smaller PRs. Thanks for the feedback @nicks!

closing in favor of #237


Bookmark Icon Not Functioning Correctly for Important Notifications

Describe the bug
Starred icon is visible but non-functional.
When clicking the bookmark icon to tag an important notification, it redirects to the "Log Update" section instead of marking the notification.
To Reproduce
Steps to reproduce the behavior:

Go to 'Notification Tab'
Click on the bookmark icon
Observe that it does not mark the notification as important and navigates to another page instead

Expected behavior
The bookmark icon should be marked in red when a notification is tagged as important.
Record Clip
https://github.com/user-attachments/assets/5a8f2dd4-9585-4ba0-a2ff-90fef7f10de5
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

@rithviknishad @Jacobjeevan I would like to work on this. Could you please assign it to me?

That's not a bookmark icon, but rather patient log update icon (speaking of which, we might have to update the icon there 🤔 ).
As such, clicking on it should take to patient log update page - consultation/consultation:id/log_updates/log:id/critical_care. You can navigate it to by visiting a patient consultation page (click on any patient with consultation) and checking the log updates tab for events (and click on view details).

@Jacobjeevan
Changes Made

Once the User clicks the Bookmark icon it turned out to Red
For persisting the State, used local Storage.

https://github.com/user-attachments/assets/938e28ed-6e33-4c5d-a873-d688b7c1a55b

@Jacobjeevan Please let me know if any further changes are required before PR.

@Jacobjeevan
Changes Made
* Once the User clicks the Bookmark icon it turned out to Red

* For persisting the State, used local Storage.

👇

That's not a bookmark icon, but rather patient log update icon (speaking of which, we might have to update the icon there 🤔 ).
As such, clicking on it should take to patient log update page - consultation/consultation:id/log_updates/log:id/critical_care. You can navigate it to by visiting a patient consultation page (click on any patient with consultation) and checking the log updates tab for events (and click on view details).

@Jacobjeevan
Sorry for the misunderstanding. For confirmation, after clicking the "log update" icon, it should navigate to this page, right? But when we click the log update icon for each person in notifications tab, how do we know which person's details it corresponds to?

@Jacobjeevan Sorry for the misunderstanding. For confirmation, after clicking the "log update" icon, it should navigate to this page, right? But when we click the log update icon for each person in notifications tab, how do we know which person's details it corresponds to?

No, it should navigate to the log updates detail page; clicking on the view details button (check the screenshot I attached above) takes you to the relevant page. I looked into it, and it's a BE issue (the URL is technically accurate, however the log update Id stored and retrieved from BE is incorrect); so went ahead and opened a Backend PR.


Celo All-Core Devs Call 9

Celo All-Core Devs Call 9

Event Details
When: Thursday, August 26th, 2021, 12 PM EST/9 AM PST, 60 minutes max
Where: Celo Zoom Event
Meeting ID: 914 1268 3393
Calendar Event: Add to your calendar here
Summary
The Celo All-Core Dev Call is the "All Hands" call for all Celo Core Developers to coordinate technical specifications and proposals and discussions to the Celo Platform.
Agenda

Geth Vuln
CIP-39: Mnemonic phrases with error correcting codes
CIP-40: Extension to ODIS for Password Hashing
CIP-43: Block Context
CIP-47: Increased round change timer for early rounds in IBFT
CIP-45: Relax canPayFee condition for non-native tokens
CIP-48: Modifications to EIP-2929
CIP-41: "E" Hardfork Meta CIP Finalization

Please post all relevant proposals below that should be discussed and evaluated

Video of the call https://youtu.be/nLz_ZWopJag


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/771 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit acfcce8:
Pending

Docs Build status updates of commit acfcce8:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-02-02-08-44-20 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[StarRocks On ES] support ssl-secured ES

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes # https://github.com/StarRocks/starrocks/issues/7160
Problem Summary(Required) ：

When creating an ES external table, add a parameter es.net.ssl: "true" to identify whether the ES cluster has HTTPS enabled
CREATE EXTERNAL TABLE `wyf6` (
  `k1` bigint(20) NULL COMMENT "",
  `k2` varchar(20) NULL COMMENT "",
  `k3` varchar(20) NULL COMMENT "",
  `k4` bigint(20) NULL COMMENT ""
) ENGINE=ELASTICSEARCH
COMMENT "ELASTICSEARCH"
PROPERTIES (
"hosts" = "https://127.0.01:9200",
"user" = "",
"password" = "",
"index" = "test",
"type" = "_doc",
"es.net.ssl" = "true"
);

es.net.ssl` is true to indicate that HTTPS is enabled for the ES cluster accessed; false indicates that HTTPS is not enabled, and ordinary HTTP requests are used
At this time, Starrocks both FE and BE  ignores all certificates and hostnames for now

[FE PR Coverage check]
:disappointed: fail : 3 / 39 (07.69%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/planner/EsScanNode.java
0
1
00.00%

:large_blue_circle:
com/starrocks/external/elasticsearch/EsRepository.java
0
1
00.00%

:large_blue_circle:
com/starrocks/external/elasticsearch/EsRestClient.java
0
25
00.00%

:large_blue_circle:
com/starrocks/catalog/EsTable.java
3
12
25.00%


chore(main): release 1.2.0

:robot: I have created a release beep boop
1.2.0 (2023-08-18)
Features

删除 labeled 以外的事件 (2a3ca05)
删除 synchronize 事件 (86fac72)
删除 unlabeled 事件 (ce17eec)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/lwsgh/release-please-example/releases/tag/v1.2.0 :sunflower:


Vulnerability roundup 101: libpano13-2.9.19: 1 advisory [9.8]

search, files

[ ] CVE-2021-20307 CVSSv3=9.8 (nixos-unstable)

Scanned versions: nixos-unstable: f5e8bdd07d1.

Fixed in 21.11+


Missing code of the introduction chapter

Hello,
The following code is not present at the end of the FINDING KEY CONNECTORS section of https://github.com/joelgrus/data-science-from-scratch/blob/master/code/introduction.py:
num_friends_by_id = [(user["id"], number_of_friends(user)) for user in users]
sorted(num_friends_by_id,
       key= lambda (user_id, num_friends): num_friends,
       reverse=True)

For the Python 3 version, I propose the following solution to take the PEP 3113 into account:
num_friends_by_id = [(user["id"], number_of_friends(user)) for user in users]
sorted(num_friends_by_id,
       key= lambda userid_numfriends: userid_numfriends[1],
       reverse=True)

What exactly does the "key= lambda userid_numfriends: userid_numfriends[1]" section of the sorted code line do?  It generates the correct output (according to the sample output in the text) but I'm lost as to how it computes it?
Thanks!

It works perfectly for me!

This is due to the versions of python, the code in book is in version 2. The code mentioned by @JBPressac for version 3 works perfectly.

muito obrigado pela  ajuda!

What exactly does the "key= lambda userid_numfriends: userid_numfriends[1]" section of the sorted code line do? It generates the correct output (according to the sample output in the text) but I'm lost as to how it computes it? Thanks!

The sorted function asks how to sort the list! Well we wanna sort the list which each element is a tuple (each element of the list is a tuple of two values).
The key parameter wants to know by which value of the tuple the sorted function should work, so with the lambda expression we say that each item of the list is a tuple (x is a tuple so we can sort it with x[0] or x[1]) and sort it with the second value of the tuple.


feat: add Klarna to confirmPayment payment methods

Summary
Add the bindings for users to pay with Klarna programmatically
Motivation
Klarna was supported on paymentsheet, but there were no bindings so folks couldn't use it programatically
Testing

[x] I tested this manually
[x] I added automated tests

Documentation
Looks like there used to be a Klarna docs page, but it was removed? I'm not sure why. Typescript will auto-doc that confirmPayment now accepts Klarna

requesting reviews from android/ios run cc @csabol-stripe @skyler-stripe whenever you get a chance to take a peak 😄

Created tickets for both of those points, merging this now though


Add luajit

"Crate for interfacing with LuaJIT from Rust, for running high-performance Lua code that can integrate with native-code written in rust."
LuaJIT is better suited for game development due to it's increases in performance.

Thanks


Containerd (still?) can't run on arm64

Running on an arm64 odroid c2.
Quite odd, I compiled against containerd 0.2.1 and am still getting this error. Compiling against go 1.5.3.
Am I doing something wrong?
WARN[0000] containerd: low RLIMIT_NOFILE changing to max  current=1024 m[ 3577.734671] docker-containe[1251]: syscall 1069
[ 3577.740213] Code: f94023e4 f94027e5 f9400fe8 d4000001 (b13ffc1f)
[ 3577.746179] CPU: 1 PID: 1251 Comm: docker-containe Not tainted 3.14.29 #1
[ 3577.752934] task: ffffffc072626c00 ti: ffffffc072120000 task.ti: ffffffc072120000
[ 3577.760351] PC is at 0x4e8db8
[ 3577.763252] LR is at 0x4e8d98
[ 3577.766185] pc : [<00000000004e8db8>] lr : [<00000000004e8d98>] pstate: 60000000
[ 3577.773538] sp :<PHONE_NUMBER>2077c0
[ 3577.776817] x29: 0000007fb2151800 x28:<PHONE_NUMBER>001e00
[ 3577.783911] x27:<PHONE_NUMBER>e16f28 x26:<PHONE_NUMBER>000000
[ 3577.789191] x25: 0000007fb2952780 x24:<PHONE_NUMBER>8470a8
[ 3577.794436] x23:<PHONE_NUMBER>e39000 x22: 0000007fb295148f
[ 3577.799730] x21:<PHONE_NUMBER>000000 x20:<PHONE_NUMBER>46a7a0
[ 3577.804960] x19:<PHONE_NUMBER>438ab0 x18:<PHONE_NUMBER>000000
[ 3577.810236] x17:<PHONE_NUMBER>be3650 x16:<PHONE_NUMBER>207eb0
[ 3577.815445] x15:<PHONE_NUMBER>000180 x14:<PHONE_NUMBER>000000
[ 3577.820681] x13: 0000007fb2951650 x12: 0000007fb2152090
[ 3577.825904] x11:<PHONE_NUMBER>3d0f00 x10:<PHONE_NUMBER>848a78
[ 3577.831124] x9 :<PHONE_NUMBER>207fa0 x8 :<PHONE_NUMBER>00042d
[ 3577.836321] x7 :<PHONE_NUMBER>10ca80 x6 :<PHONE_NUMBER>532020
[ 3577.841515] x5 :<PHONE_NUMBER>000000 x4 :<PHONE_NUMBER>000000
[ 3577.846813] x3 : ffffffffffffffff x2 :<PHONE_NUMBER>000080
[ 3577.851999] x1 :<PHONE_NUMBER>2078b8 x0 :<PHONE_NUMBER>000004
ax=4096
DEBU[0000][ 3577.857234]
 containerd: read past events                  count=0
DEBU[0000] containerd: supervisor running                cpus=4 memory=1719 runtime=runc runtimeArgs=[] stateDir=/run/containerd
DEBU[0000] containerd: grpc api on /run/containerd/containerd.sock
FATA[0000] containerd: epoll wait                        error=function not implemented

Epoll support is enabled in the kernel.
Maybe I need to bump go to v1.6.2.
https://github.com/paralin/SkiffOS/issues/1

Can you try with this patch: https://github.com/docker/containerd/pull/226 ?

@paralin since #226 was merged, could you try with the master version?

I'll give it a try, sorry for the delay.

Fixed, I cherry picked that commit, thanks


Remove --output flag

Following my discussion with @nojb on https://github.com/dbuenzli/cmdliner/pull/17 and some thought, I think that introducing --output was a bad idea:

Shell redirection exists for a reason. It is trivial to write: refmt --parse ml --print re file.ml > file.re.
I introduced --in-place because writing to the file that we read from is a special case, and requires some shell shenanigans to work (see https://github.com/facebook/reason/issues/886). But at the same time introduced --output, which is wholly unnecessary and also brings with it some annoying command-line logic (mutual exclusion of --output, --in-place, etc).

I have removed --output in this PR that (at the moment) builds upon https://github.com/facebook/reason/pull/995.
/cc @chenglou @yunxing

I think your diff contains content from the other PR?

@yunxing fixed.

Looks good. Thanks.


2.3.0 is throwing exceptions while retrieving metadata using /wsfed/metadata

While trying to get the meta data using the latest version (2.3.0), it is throwing the below error. However it works fine with the version 2.0.5. Below is the error.
Error (using 2.3.0)

Success (using 2.0.5)

Have you tried updating identity server itself?
—
cheers
Dominick Baier
On 25 April 2016 at 02:19:24, Vinay Kasireddy<EMAIL_ADDRESS>wrote:

While trying to get the meta data using the latest version (2.3.0), it is
throwing the below error. However it works fine with the version 2.0.5.
Below is the error.
Error (using 2.3.0)
[image: image]
https://cloud.githubusercontent.com/assets/2632995/14771450/e680be34-0abd-11e6-8fa0-832b15778ca3.png
Success (using 2.0.5)
[image: image]
https://cloud.githubusercontent.com/assets/2632995/14771465/635624e4-0abe-11e6-8cae-7d184b1b1a44.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/IdentityServer/IdentityServer3.WsFederation/issues/65


Improve performance of roll command

The roll command is very inefficient for large inputs (for example (500d2)d1000). It should be re-worked to improve the stability of the Apollo bot.

When running a large roll command, the whole bot hangs. It's not so much an "inefficient" thing as a "too much work to do feasibly" thing. The bot locks up because currrently the roll work is being done on a separate thread, rather than a process, so it gets stuck with the GIL.
Using a process instead would stop Apollo from hanging (I've tested this) but still runs into the issue of consuming system resources by performing a large comptuation, which doesn't terminate quickly.
Using a future for the work and raising the reduce methods to async would introduce coroutine points for the dice roll evaluator to be interrupted by something like asyncio.wait_for, but it isn't trivial. Without anything actually asynchronous, the program never has anywhere to actually yield control and have the CancelledError to be raised by the loop. Since the code inside the reduce tree is all synchronous, it would have to be modified manually to support this.
It may be easiest to instead create a subprocess to run the work and manually kill it after some duration rather than trying to modify the evaluation code to support this. I'll investigate this now.


swiper width: auto not working.

I want slider width to be auto. But when i put width: auto; Slider not working
Here is the codepen

I would love to see some functionality as well. The slides I show all have a fixed with and their own breakpoints and I do not want Swiper handle that, especially because they can vary in width.

what I need is Test slider. the width should be auto. But it is taking full width.

I have found a workaround by just using width: auto !important on the slide item element. This works for our case.

Thank you width: auto !important worked

Thank you width: auto !important worked

Would be nice with a proper way of doing this though. I'm trying to avoid patching my stylesheets with !important altogether.


refactor(semantic, transformer): simplify FxIndexMap type aliases

Pure refactor. Simplify FxIndexMap type aliases by using rustc_hash's FxBuildHasher, instead of longhand BuildHasherDefault<FxHasher>.

#7524  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Nov 28, 6:43 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Nov 28, 6:43 AM EST: A user added this pull request to the Graphite merge queue.


Parsing dates with pattern DMYYYY does not work

Hello,
I have a problem with following code:
moment('551960', 'DMYYYY', true); // invalid-date
//Object { _isAMomentObject: true, _i: "551960", _f: "DMYYYY", _strict: true, _isUTC: false, _pf: Object, _locale: Object, _d: Invalid Date, _isValid: false }
moment('05051960', 'DMYYYY', true); // valid-date
//Object { _isAMomentObject: true, _i: "05051960", _f: "DMYYYY", _strict: true, _isUTC: false, _pf: Object, _locale: Object, _d: Date 1960-05-04T22:00:00.000Z, _isValid: true }

What's the reason that string '551960' cannot be parsed?
This is my output:
Tue Oct 24 2017 08:55:36 GMT+0200 (CEST)
24.10.2017, 08:55:36
-120
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:56.0) Gecko/20100101 Firefox/56.0
Thank you :smile:

Given an input like '1111960', would you expect that to be parsed as 1st Nov 1960, or 11th January 1960?
Why?

Oh, yes that's right. For '1111960' I would expect an invalid date, because there is a ambiguity. My format says there are four digits at the end which reflect the year. So there are three remaining digits, which are ambiguous. '551960' on the other hand is unambiguous. I just wanted to know if this is expected.

I can't say it's expected behaviour, but I wouldn't want to 'fix' it to support this format either: there are so many ambiguous combinations it would be an incredibly unreliable format to parse.

That's right. Thank you for the clarification. I think the issue can be closed.

@ashsearle I am also facing the same issue. When format information is missing, Only then question of ambiguous combinations come into picture. So, as long as format is provided, momentjs should parse correctly.

Also I think , going by discussion above, fixing this might not be as easy as said. It seems momentjs  first try to parse  input string as date, and then apply formatting on top of that.

@manusinghal19 Please re-read https://github.com/moment/moment/issues/4265#issuecomment-338938105 and tell me which result you'd expect from parsing "1111960"

@ashsearle
moment('1111960'); // invalid-date moment('1111960', 'DMMYYYY', true); // valid-date  interpreted as 1st Nov 1960 moment('1111960', 'DDMYYYY', true); // valid-date  interpreted as 11th Jan 1960

I agree with @manusinghal19. I think this is a common scenario.

@manusinghal19 This issue is specifically about DMYYYY format, not DMMYYYY or DDMYYYY.
With DMYYYY you have to choose: should moment('1111960', 'DMYYYY', true) be interpreted as 1st Nov 1960 or 11th Jan 1960?  Why?
I've never seen anyone use DMYYYY format for formatting dates for display or data-interchange format - probably because it's error-prone, ambiguous and broken.
e.g. if you make the arbitrary choice that 1111960 represents 1st November 1960 instead of 11th January 1960, that means you have no way to pass 11th January (of any year) to another system.
It's a rubbish date format.  Don't use it!

Yeah, @ashsearle is completely right. If your format is ambiguous, the parser has to guess and I don't see how it could reliably guess right.
re: DMMYYYY: there is a valid complaint about Moment's greediness as a parser: in principle you do get one degree of "freedom", i.e. with DMMYYYY it ought to be able to decide how many digits to allocate to D. But Moment's parser will eat up as many digits with D and never backtrack when it can't fulfill YYYY. There's a ticket somewhere about that, but a) that does not apply DMYYYY, which doesn't even work in principle and b) we're not going to fix DMMYYYY until we get rid of non-strict parsing and rewrite the whole parser.


Clear command

I hated it when the scala repl couldn't clear the terminal for me, it still can't. Would it be a good idea to have clear command work from Ammonite Repl?
I know Ctrl + L works simply in both repl, but would it be good to have clear work as well?

Ctrl-L will do for now


Benchmark

It might be useful to start gathering benchmarking data at this point now that most of the systems are in place.

Benchmarking framework has been added to #55 and being used as a means to aid in optimizing functions there. Once in main, Hex should get some benchmarking data as well.


Created the social_media.html file for the community page #442

Issues Identification
Creation of new file
Closes: #442
Description
The social media has been created . It has same header and footer as resource.html . The links you provided are added to create this file.
Summary
By clicking the Respective links it takes the user to the respective links.
Details
The page where created as reference as same as resources.html file and the links where added . The screenshots are provided for the reference .
Types of Changes
Please check the boxes that apply

[ ] Bugfix (non-breaking change that fixes an issue)
[  ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[  ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Please check the boxes that apply

[  ] My code follows the style guidelines of this project
[  ] I have performed a self-review of my own code
[  ] I have commented my code, particularly in hard-to-understand areas
[  ] I have made corresponding changes to the documentation
[  ] My changes generate no new warnings
[  ] My changes do not break the current system and pass all existing test cases
[  ] I have added tests that prove my fix is effective or that my feature works
[  ] New and existing unit tests pass locally with my changes

Screenshots
in light mode :

in dark mode;

Additional Information
@vivekv2810 can you assign the level 2 or level 3 as creating a complete page.

@vivekvardhan2810 others are providing level 2 or level 3 for creating the new page

Sorry @Lekkhasri i will update the level.

@vivekvardhan2810 ok sir


Fix repository structure

GitHub Issue: #

Proposed Changes

What is the current behavior?

What is the new behavior?

Checklist
Please check if your PR fulfills the following requirements:

[ ] Documentation has been added/updated
[ ] Automated Unit / Integration tests for the changes have been added/updated
[x] Contains NO breaking changes
[ ] Updated the Changelog
[ ] Associated with an issue

Other information

The correct folder should be "src", not "Src".


Higher level function for lapack pstrf and pstf2

Now that https://github.com/scipy/scipy/pull/10537 added ?pstrf and pstf2, I am interested in starting discussion on potential higher level functions. It appears that this question was left open ended on that PR, and I haven't found the discussion continued elsewhere. I guess the basic question would be:

Would pivoting be an option to existing functions, something like cholesky(..., pivot=True), or entirely new functions? (This might also apply to how higher level functions for https://github.com/scipy/scipy/issues/10788 are ultimately written as well)
For me, at least, it would be helpful to have this in for the 1.4 release :) is there still enough time to get this in? I'd be happy to contribute.

For lu_factor, I wanted to add a complete_pivoting parameter (default False). When True, it would use ?getc2 and return a tuple with three elements (the usual plus column permutation indices). lu_solve could simply check the number of elements in the input tuple to decide which LAPACK routine to call, so it wouldn't need an extra argument.

Makes sense. If that ends up happening, then it might make sense to coordinate the naming of this complete_pivoting parameter with cho_factor, and make cho_solve mirror lu_solve.

I fully acknowledge the need for this. However I am more inclined to get rid of the ?_factor and ?_solve stuff and actually make this more streamlined inside lu and cholesky by somehow returning the factorization if needed and/or using Cython and accepting nD arrays for the left hand side. I don't have a concrete plan yet though and it requires some API design touching some backward compatibility pain points.

My reasoning is that ?_solve already accepts the low-level LAPACK representation and hence you can directly call the original LAPACK routine with what ?_factor returns. Hence the user actually saves time by skipping these thin wrappers and directly calling the routines themselves via scipy.linalg.lapack namespace.

Certainly it is faster to skip ?_solve and ?_factor but they are a bit friendlier, if only because of the names, than the LAPACK wrappers themselves. We can certainly add this functionality to lu and cholesky, but why not also add it to ?_solve and ?_factor (unless you intend to deprecate them)?

@ilayn does this mean that ?_solve and ?_factor will be deprecated soon? I do slightly prefer lu and cholesky, and I would definitely be in favor of them accepting nD arrays. I usually find myself using numpy's cholesky for this very reason, but if scipy had both a pivot option and could broadcast over dimensions like numpy, then scipy would be the clear winner IMO.

I wouldn't say they are going to be deprecated because that's quite often used out in the wild and would be quite an annoyance for not much of a return. Especially not without offering a better alternative.
Instead we can try to find a way to inject more functionality to the regular lu, qr, cholesky without breaking anything. But designing a better API is not so straightforward and needs some discussion on the list I guess. nD array inclusion is something I am also longing for a lot for linalg.solve but unfortunately I am sparing much less time than I wished these days.


[Enhancement] treat enough replica as normal case when adding replica

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #7924
Problem Summary(Required) ：

// Enough replica causing replica add failed should be treated as a normal
// case to avoid too many useless warning logs. This will happen when a clone task
// finished for decommission or balance, and the redundant replica has been deleted
// from some BE, but the BE's tablet report doesn't see this deletion and still report
// the deleted tablet info to FE.

run starrocks_fe_unittest

[FE PR Coverage check]
:disappointed: fail : 1 / 6 (16.67%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
0
1
00.00%

:large_blue_circle:
com/starrocks/master/ReportHandler.java
0
4
00.00%

:large_blue_circle:
com/starrocks/common/InternalErrorCode.java
1
1
100.00%


Item equip types from cache

Need to compare but the BodyParts EquipType data can be found in item defs
624 - hats
625 - hair

No, It's not in the item definitions natively.

It is in higher revisions, see ItemDecoder718 and ItemDefinitionsParamConverter although it wouldn't make sense why it has that information in there twice. Anyway, this ticket is a todo; Check/Explore

It is in higher revisions, see ItemDecoder718 and ItemDefinitionsParamConverter although it wouldn't make sense why it has that information in there twice. Anyway, this ticket is a todo; Check/Explore

Since void is made with a 634 client build, you'll have to store newer item equip type data in a local repository then.

Since void is made with a 634 client build, you'll have to store newer item equip type data in a local repository then.

ItemDefinitionsParamConverter packs the data from one cache into another, that's the whole idea behind #110; easily add and modify cache data rather than having a load of extra config files to load on startup.

Only looks like these two params.
They don't line up with current data of what is a hat/needs hair removing.
A way of identifying a sub-set of hats and masks.
Doesn't seem to be useful.


Failure to extract player url for a youtube video

Checklist

[ x] I'm reporting a broken site support
[ x] I've verified that I'm running youtube-dl version 2021.12.17
[ 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 issues including closed ones

Verbose log
mini-me@virtucon ~/h/l/cinema> youtube-dl --verbose 'https://www.youtube.com/watch?v=r0Ogt-q956I'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=r0Ogt-q956I']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Single file build
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-6.2.0-1010-nvidia-x86_64-with-glibc2.35 - OpenSSL 3.0.2 15 Mar 2022 - glibc 2.35
[debug] exe versions: ffmpeg 4.4.2, ffprobe 4.4.2
[debug] Proxy map: {}
[youtube] r0Ogt-q956I: Downloading webpage
[youtube] r0Ogt-q956I: Downloading API JSON
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
WARNING: unable to extract player URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/mini-me/home/hut/youtube-dl/youtube_dl/YoutubeDL.py", line 859, in wrapper
    return func(self, *args, **kwargs)
  File "/home/mini-me/home/hut/youtube-dl/youtube_dl/YoutubeDL.py", line 955, in __extract_info
    ie_result = ie.extract(url)
  File "/home/mini-me/home/hut/youtube-dl/youtube_dl/extractor/common.py", line 565, in extract
    ie_result = self._real_extract(url)
  File "/home/mini-me/home/hut/youtube-dl/youtube_dl/extractor/youtube.py", line 2128, in _real_extract
    self._sort_formats(formats)
  File "/home/mini-me/home/hut/youtube-dl/youtube_dl/extractor/common.py", line 1444, in _sort_formats
    raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description
This situation occurs for youtube-dl built from source revision 00ef748cc0e35ee60efd0f7a00e373ab8d1af86b.
As far as I could hunt down things, youtube-dl attempts to download https://www.youtube.com/watch?v=r0Ogt-q956I&bpctr=9999999999&has_verified=1 from the function _request_webpage which is invoked (with some levels of indirection) from _real_extract. In extractor/youtube.py, line 1602, in function _extract_player_url,  there is the r'"(?:PLAYER_JS_URL|jsUrl)"\s*:\s*"([^"]+)"' regexp that is matched against the webpage whose url is given above, and nothing is found. It may therefore be a change in the contents of that webpage that induces this behaviour. I can unfortunately not go into debugging that without more background knowledge of what youtube precisely does and how it may or may not have changed recently. The _extract_player_url function is called in a loop on line 2034 of extractor/youtube.py and, because of this parsing failure, that loop is exited early, and the formats variable is never populated on line 2076 with the video formats. Hence the reported error.

[x] I've searched the bugtracker for similar issues including closed ones

Actually doing that would have saved you the debugging. Review #30839.

I've spent a whole morning looking for a similar issue. Debugging happened faster and more methodically than searching for a needle in a haystack. And I still do not know how to solve that issue. I'll spend another morning tomorrow. Or in a week.

Duplicate from this one, perhaps ? https://github.com/ytdl-org/youtube-dl/issues/32499
It would perhaps make sense to attempt to link such issues with this error message in the youtube-dl cli app itself.

The yt-dlp repo gets around this issue.
https://github.com/yt-dlp/yt-dlp


[Typescript] Default shims-vue.d.ts prevents missing files from being detected

Version
4.5.11
Environment info

  System:
    OS: Linux 5.11 Arch Linux
    CPU: (8) x64 Intel(R) Core(TM) i7-10510U CPU @ 1.80GHz
  Binaries:
    Node: 15.10.0 - /usr/bin/node
    Yarn: 1.22.10 - /usr/bin/yarn
    npm: 6.14.11 - /usr/bin/npm
  Browsers:
    Chrome: Not Found
    Firefox: 86.0
  npmGlobalPackages:
    @vue/cli: 4.5.11

Steps to reproduce

vue create project  # enable typescript
cd project
Modify App.vue:

import HelloWorld from './components/HelloWorldddd.vue';

The following does not report any errors:

tsc -p tsconfig.json --noEmit

What is expected?
I expected Typescript to complain about a missing import. Instead, the error goes by undetected.
What is actually happening?
No errors detected.

Note that this also means IDEs won't report errors anymore.

here is the erre in vue installement :"npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself."
what shoull i do with this ?


Add / subtract workdays

Hey,
I needed a way to step over weekends and created a small helper method to achieve this.
Tried to make it into a pull request, but found the code a bit intimidating on the first sight, so thought I should ask first if this is something that has its place in the library?
Ideally I guess it should be available as moment.add(2, 'workdays') or so.
Let me know if this is useful or maybe already covered somehow?
JSfiddle demo: http://jsfiddle.net/dain/5xrr79h0/
  moment.fn.addWorkdays = function (days) {
    var increment = days / Math.abs(days);
    var date = this.clone().add(Math.floor(Math.abs(days) / 5) * 7 * increment, 'days');
    var remaining = days % 5;
    while(remaining != 0) {
      date.add(increment, 'days');
      if(date.isoWeekday() !== 6 && date.isoWeekday() !== 7)
        remaining -= increment;
    }
    return date;
  };

The algorithm for adding/subtracting (without holidays) should be pretty straight forward and work in constant time, the one outlined here is taking O(N).

I worked on this last night. My solution is as follows:

Finding the number of workdays, or weekends, is a composition of a more fundamental problem
The fundamental problem is "How many of a particular day exist between two days?"
More abstractly, the fundamental problem is "How many solutions of a periodic function exist within some interval?"

In light of that, finding the weekends (in the USA) is solved by:
"How many Saturdays are in this interval AND how many Sundays are in this interval?"
The week days, excluding Holidays, is the inverse operation.
This problem could be generalized to any other regular system, such as cultures with a different set of weekend days.
Not sure if this would help anyone, but it's the solution I'm going with in my app. I should have it abstracted into a library in the next day or two.

@ichernev is this the correct fixture data for add/subtract?
https://gist.github.com/jmeas/0a713e0b9901a25a32f1

Thanks @daaain. Based on your solution I've created this gist with add/subtract(x, 'workdays')and isWorkday(). Uses moment.easter (included in gist). Polish National Holidays are included, but you can easily modify script to add your own.

@loostro, your add/subtract algorithms will be much slower than the constant-time solutions in moment-business when adding large numbers of days.

@jmeas in my current use case, thats not a problem, as i need to "add" or "subtract" 2 workdays only, but it would be nice to see a better solution

as i need to "add" or "subtract" 2 workdays only,

Ah, okay.

it would be nice to see a better solution

I linked to one in the above comment :stuck_out_tongue:

@jmeas you mean moment.fn.addWeekDays ? or the libraries from previous posts?

you mean moment.fn.addWeekDays ? or the libraries from previous posts?

Both. moment.fn.addWeekDays is from a library in a previous post, moment-business.

ah.. i see :) i'll study your solution and fix my gist for Polish workdays to be more efficient, thanks!

This is how a similar problem (getting difference in week days) is solved in the Carbon library (PHP):
diffFiltered() method: link
diffInWeekdays link (basically a wrapper over diffFiltered)
This interface allows a developer to specify a custom filter function - thus enabling to use the library with arbitrary weekends/holidays

Here's a working constant time solution to this problem for western work weeks:
moment-business
atm it's hard coded to support the western work week, but the underlying algorithms should be generic enough to be tweaked to support any weekend, so long as it's periodic (repeats week to week) and simple (every week is structured the same in terms of what's a week and what's not a week; it doesn't go back and forth between two patterns, for instance)

Instead of hard coded western work week and weekend, wouldn't it be easier for moment to incorporate if easily configurable workdays and non-workdays, and easy to set work time hh:mm

Most likely, which is one of the reasons why I made the underlying algo's in moment-business accept any periodic pattern of "holes" (weekends) along an interval (a week). This should make it fairly simple to build an API to work with configurable work days and non-work days. I didn't put in that effort because I only needed western work week support for my use case.
I never made a PR into moment directly because I'd prefer there be some confirmation from the core team that the PR wouldn't just get closed, but I never heard back.

@jmeas seems like it makes the most sense as a plugin, just as you have it. We'd be happy to list it under the plugins section of the docs you make a PR to that effect on the docs repository.

To me, this is a permanent plugin. We don't have the ability to determine work days for every locale, especially including holidays. I'm going to close this one.

Thanks for the updates @icambron & @maggiepint !

FYI, @daaain 's method is also O(1) - it may look as though it iterates over all days between the start and end, but it does not. It only iterates over the modulo which is bounded < 5.


[Logitech Litra] Allow managing Litra devices' temperature and brightness using presets

With this update, you can now set the brightness and temperature of your Litra devices from Raycast - thanks to @zalewskigrzegorz for the suggestion.
It isn't possible to get the current brightness or temperature, so we can't have a nice "increase by 10%" or "decrease by 10%" user experience. Instead, you can configure presets using the new "Manage Temperature Presets" and "Manage Brightness Presets" commands, and then those presets are available for each device in the "Manage Devices" command.
Closes https://github.com/raycast/extensions/issues/5101.

FYI @zalewskigrzegorz


All errors show up as *errors.withStack

I am using github.com/pkg/errors and all errors show up in Sentry with the title *errors.WithStack. Is there any way of customizing this message so that it's more informative? The full error message is shown below but it's hard to differentiate issues when they all have the same title.
Using Sentry 9.1.2 on the server side.

Don't think this question belongs here per se.
But, when composing a Sentry event you can assert the error and get an underlying error (if available) through .Cause(err)
e.g.,
ev := sentry.NewEvent()

// ..

if _, ok := err.(interface{ Cause() error }); ok {
	e := errors.Cause(err)

	ev.Exception[0].Type = e.Error()
	ev.Exception[0].Value = reflect.TypeOf(e).String()
}

In sentry this will show up like so, here is an example where we retrieve an underlying *pq.Error which was wrapped many layers down.

Hope this helps.

Hey, thanks for the patience. I've scheduled some time for patches/features and releasing a new version of sentry-go this and next week. I'll make sure to fix all the remaining issues till the EOD next week. Cheers!


How to use the the paid version of the icons?

My team has a key of hugeicons and i've already added in the NPM_TOKEN, but still has only access to the free ones.

Please check this link for available packages and how to install and use them:
https://docs.hugeicons.com/

@xtawfik that has nothing to do with what the @gabrielbuzziv asked.

@xtawfik
At the docs has an example <Home01Icon variant="solid" />
But this is not available on this library.
The only icon props available is size

@hugeicons/react is not available on NPM


Brand

[ ] Placeholder images
[ ] Logo
[ ] Social media channels (FB, TW, LinkedIn)
[ ] Powerpoint template
[ ] 1 pager
[ ] Business cards
[ ] Letterhead
[ ] Dataverse

see #364


Rename this repository react-cli which is the name used in npm

It is a bit confusing when we have two names in npm and github.
Developer community is well adjusted to use the name cli for scaffolding tools

I guess we don't have that name on npm right, it's already taken. If we are to publish it with the name 'react-cli' we need to publish as a scoped package :thinking:

ok, that I forgot and agree with your point. Any good name suggestions going align to -cli ?

if we need a new name "react-factory" and "react-machine" are available @rajikaimal  @charithsoori

@charithsoori Any comments on this :thinking: ?

Like the name ract-factory
On Wed, Aug 23, 2017 at 8:06 PM Rajika Imal<EMAIL_ADDRESS>wrote:

@charithsoori https://github.com/charithsoori Any comments on this 🤔 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/99xt/react-scaffolder/issues/27#issuecomment-324354825,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAebGDDxR5hjuctkcygWf36sXMHGc1INks5sbDjdgaJpZM4Np9TD
.

--
Sent from IPhone


Test draft PR for github action

DO NOT MERGE!
just a test

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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

Project coverage is 99.85%. Comparing base (1a50508) to head (a1cdded).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #85   +/-   ##
========================================
  Coverage    99.85%   99.85%
========================================
  Files           23       23
  Lines         1350     1350
========================================
  Hits          1348     1348
  Misses           2        2

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Fix creating empty commit on PR

Fix https://github.com/huggingface/huggingface_hub/issues/2411 cc @narugo1992
When trying to create an empty commit, the commit call is not done. Instead we retrieve the information from the last commit on the revision. On a PR, the revision is quoted twice leading to a HTTP 404 error. This PR fixes it.
I also added a regression test for it.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


using bdd: to describe the git revision of a graph

I've opened a few small tasks on the project, i just use it as a todo list, but maybe it will prove more useful than that, we'll see... In the mean time, I think this issue will need some discussion:
We currently do not use the bdd: prefix at all, maybe we could start with the git revision of a graph transferred to fuseki. For instance if a file W123.ttl is at revision "abc" in git, maybe we could have
bdd:W123 adm:hasGitRevision "abc"

? The other obvious case would be status, as it has to do with the metadata and not the data. But it's more tricky, as perhaps it would mean replacing (in git):
bdr:W123 a :Work ;
    :status bdr:statusReleased ;
    ...

with
bdd:W123 :status bdr:statusReleased .

bdr:W123 a :Work ;
    ...

?

this issue has been addressed via bda: and rfc011 reworking


Add 6 more provider codes to the Apply button

Context
These are :new: providers that are being on-boarded with Apply.
Changes proposed in this pull request
Update the conditional and add more cases.
Guidance to review
Obviously there are better ways to do this! If necessary please leave exact-ish description of in which controller I could hoist an array with the provider code whitelist, or what works better.
Feel free to also push new commits to this branch.
I have not tested this locally.
https://trello.com/c/XFn2UPjD/617-add-december-providers-to-support-console

It looks like it works on the review app!


Raspberry Pi problem

I have read on the site that there is a support of Raspberry pi but really it is not tested in real life and I tried to do it but without any success. I used the latest version of library webcam-capture-0.3.10.jar, bridj-0.7-20140918.jar and fatal error occured, log is bellow:
[error occurred during error reporting (printing native stack), id 0xb]
Java frames: (J=compiled Java code, j=interpreted, Vv=VM code)
j  com.github.sarxos.webcam.ds.buildin.natives.OpenIMAJGrabber.startSession(IIDLorg/bridj/Pointer;)Z+0
j  com.github.sarxos.webcam.ds.buildin.WebcamDefaultDevice.open()V+175
j  com.github.sarxos.webcam.ds.cgt.WebcamOpenTask.handle()V+55
j  com.github.sarxos.webcam.WebcamProcessor$AtomicProcessor.run()V+14
j  java.util.concurrent.ThreadPoolExecutor.runWorker(Ljava/util/concurrent/ThreadPoolExecutor$Worker;)V+95
j  java.util.concurrent.ThreadPoolExecutor$Worker.run()V+5
j  java.lang.Thread.run()V+11
v  ~StubRoutines::call_stub
CPU:total 1 (ARMv6), vfp
/proc/cpuinfo:
processor	: 0
model name	: ARMv6-compatible processor rev 7 (v6l)
Features	: swp half thumb fastmult vfp edsp java tls
CPU implementer	: 0x41
CPU architecture: 7
CPU variant	: 0x0
CPU part	: 0xb76
CPU revision	: 7
Hardware	: BCM2708
Revision	: 000d
Serial		:<PHONE_NUMBER>f4454f
Also I tested on Linux 32/64 successfully

Hi @maxx79,
You may try downgrading BridJ to 0.6.3-SNAPSHOT to check if it runs well on RasPi.
The other option is to use webcam-capture-driver-v4l4j instead of a default one. It also has some issues, but at least does not crash (usually). There is still much work to be done to stabilize Webcam Capture API on RaspberryPi.
And the last one, if you do not need high FPS is to use webcam-capture-driver-fswebcam.

Thank your for the links, actually I tried to use the library 0.6.3-SNAPSHOT and the effect is the same - crashing of JVM. I will try the recommended jar files and will write about result.


PDF.js does not acknowledge page labels

For instance, if the PDF has its initial pages designated by roman numerals and the rest by decimal arabic numerals, PDF.js's page numbering ignores that and only displays the page index.
This would be very helpful to fix when navigating large documents such as textbooks, where the page numbers and the page index are offset.
More Information:
PDF17: Specifying consistent page numbering for PDF documents
PDF 1.7 (ISO 32000-1) Section 12.4.2 (Page Labels)

Closing as duplicate of issue #3165.


Sphinx-Needs and parallel Sphinx build

I do not have a concrete requirement for this yet, but do you think sphinx-needs could at some point support sphinx parallel builds sphinx-build -j? This means the read-phase when the needs model is created can be parallel and the extension sets parallel_read_safe to True
https://www.sphinx-doc.org/en/master/extdev/index.html

The parallel-read-safe extension must satisfy the following conditions:

The core logic of the extension is parallely executable during the reading phase.
It has event handlers for env-merge-info and env-purge-doc events if it stores dataa to the build environment object (env) during the reading phase.

I would like to support this, but the problem is (afaik) that there is no data exchange between the different workers of a parallel sphinx-needs build.
Which means the important need-dictionary is not in sync and every worker is using a different one.
I made some tests in the past and I got quite fast errors like "need-id not found", "key error in ...".
technical details
Sphinx parallel build starts for x documents a single worker in the writing event.
Due to the lack of an event just before the writing starts (maybe this has changed in the while),  Sphinx-Needs uses the first document event to do a lot of calculations on all needs (e.g. dynamic functions, back link, ...) and then sets a global flag, which indicates that this step does not need to be redone on the other document events.
So when Sphinx starts parallel workers, only worker 0 would have the finished need-dict.
worker 1-x would still have the old version.
own thoughts
I'm also not sure if the parallel option is a very good supported feature in the sphinx-extension world.
You need only one plugin, which does not support it and the benefit is gone.
I also have read some issue comments from Sphinx maintainers, that this feature is quite old, not very maintained and not used by them (status ~2 years ago).
But I would be happy to be wrong and I thing spending again some hours to figure out what needs to be done to get it running is a good idea.

First changes and tests are looking good and docs with needs and needtables only get already build.
Performance boost is not so high as expected (10% -15%).
But maybe the benefit is showing, when documents are getting more complex.


codecov badge

@h1alexbel thank you for your Pull Request. I'll assign someone to review it soon.
If this PR solves a todo from the code, please don't forget to remove it.

@h1alexbel please review this Pull Request. Deadline (when it should be merged or closed) is 2023-04-15T16:10:14.191478.
You should check if the requirements have been implemented (partially or in full), if there are unit tests covering the changes and if the CI build passes. Feel free to reject the PR or ask for changes if it's too big or not clear enough.
Estimation here is 30 minutes, that's how much you will be paid. You will be paid even if this PR gets rejected.

@rultor merge

Codecov Report
Patch and project coverage have no change.

Comparison is base (5a23782) 100.00% compared to head (bf7c8c2) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #12   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            4         4
=========================================
  Hits             4         4

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

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

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 1min)

@h1alexbel thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Version upgrade Kibana 6.2.3

latest build is not installaable for kibana 6.2.3

Download 5.6.5 logtrail plugin from : https://github.com/sivasamyk/logtrail/releases/download/v0.1.23/logtrail-5.6.5-0.1.23.zip
Unzip logtrail-5.6.5-0.1.23.zip
Update the kibana.version to 6.2.3 in package.json file
zip it back.
Install it using following command
kibana-plugin.bat install file://<path-to-local-logtrail.zip file>

is this the right solution ?

https://github.com/sivasamyk/logtrail/blob/master/docs/how_to.md#1-hostname-field-need-to-be-of-type-keyword  this is the right solution . thanks .

You can download Logtrail for 6.2.3 from https://github.com/sivasamyk/logtrail/releases/download/v0.1.27/logtrail-6.2.3-0.1.27.zip


🎨 Remove bluebird Promise.each from db.utils

https://github.com/TryGhost/Ghost/issues/14882
I have used the in-house sequence function for refactoring Bluebird's Promise.each
Taken some reference from here: https://github.com/TryGhost/Ghost/pull/15626

Hey @vershwal - would you be able to rebase and remove the Bluebird Promise import at the top? 🙂


KAFKA-3043: Replace request.required.acks with acks in docs.

In Kafka 0.9, request.required.acks=-1 which configration of producer is replaced by acks=all,
but this old config is remained in docs.

thanks for the contribution. Just a minor correction.

Thank you for review, @gwenshap.
I revised it.

LGTM. Thanks for contributing!


Nerf +Insist

This is meant to be added BEFORE abilities project is done

That will cause conflicts, also you could of just changed the branch back instead of closing the PR.

Meh this works so Ill keep it simple

Closed because no longer needed.


fix for "No Docker client strategy found" despite not invoked

fix for https://github.com/testcontainers/testcontainers-node/issues/367

@calvintwr seems the PR is failing the tests

@cristianrgreco yea let me check it out.

@cristianrgreco can I get some help?

@cristianrgreco great got it thanks. i think in your suggestion, given that getDockerClient will be invoked, it might still result in the same issue. i'll see if I can come up with something, if not there might be practical fix for this. And honestly it doesn't require a fix. It just stands in the way when one tries to create abstractions.

i think in your suggestion, given that getDockerClient will be invoked, it might still result in the same issue

That shouldn't be the case, getDockerClient is only invoked when dockerClient() is called, whereas before it was invoked when imported, so unless you call testcontainers code it shouldn't invoke, or am I missing something?


SSO with angular-oauth2-oidc and keycloak

Hello,
I'm trying to integrate the SSO solution in my Angular applications, I've a Portal application as the main entry point which displays to the users the available applications once logged in using Keycloak as the OIDC provider.
keycloak uses the sessions for the logged users, I've configured my Portal application as follows:
export const authCodeFlowConfig: AuthConfig = {
  // Url of the Identity Provider
  issuer: 'http://<IP_ADDRESS>:8080/auth/realms/my-portal',
  // URL of the SPA to redirect the user to after login
  redirectUri: window.location.origin + '/',
  postLogoutRedirectUri: window.location.origin + '/login',
  // The SPA's id. The SPA is registerd with this id at the auth-server
  // clientId: 'server.code',
  clientId: 'app-portal',
  // Just needed if your auth server demands a secret. In general, this
  // is a sign that the auth server is not configured with SPAs in mind
  // and it might not enforce further best practices vital for security
  // such applications.
  // dummyClientSecret: 'secret',
  responseType: 'code',
  // set the scope for the permissions the client should request
  // The first four are defined by OIDC.
  // Important: Request offline_access to get a refresh token
  // The api scope is a usecase specific one
  scope: 'openid profile email offline_access',
  showDebugInformation: true,
  requireHttps: false,
  disableAtHashCheck: true,
  sessionCheckIntervall: 60000,
  sessionChecksEnabled: true,
  clearHashAfterLogin: false,
  silentRefreshTimeout: 5000, // For faster testing
  // Not recommented:
  // disablePKCI: true,
};

And my other apps as follows (below the configuration of one app):
export const authCodeFlowConfig: AuthConfig = {
  // Url of the Identity Provider
  issuer: 'http://<IP_ADDRESS>:8080/auth/realms/my-portal',
  // URL of the SPA to redirect the user to after login
  redirectUri: window.location.origin + '/',
  postLogoutRedirectUri: window.location.origin + '/login',
  // The SPA's id. The SPA is registerd with this id at the auth-server
  // clientId: 'server.code',
  clientId: 'app1',
  // Just needed if your auth server demands a secret. In general, this
  // is a sign that the auth server is not configured with SPAs in mind
  // and it might not enforce further best practices vital for security
  // such applications.
  // dummyClientSecret: 'secret',
  responseType: 'code',
  // set the scope for the permissions the client should request
  // The first four are defined by OIDC.
  // Important: Request offline_access to get a refresh token
  // The api scope is a usecase specific one
  scope: 'openid profile email offline_access',
  showDebugInformation: true,
  requireHttps: false,
  disableAtHashCheck: true,
  sessionChecksEnabled: true,
  sessionCheckIntervall: 60000,
  clearHashAfterLogin: false,
  silentRefreshTimeout: 60000, // For faster testing
  // Not recommented:
  // disablePKCI: true,
};

When I access my Portal application, I'm redirected to keycloak server for authentication as well and be back then to my home page to display the links to my other apps, when I try to access my app1 (in a new tab) there is a check of the session state done by default by the module angular-oidc2 and I'll be automatically logged, the SSO works well in this case.
My problem is the "Logout", when I try to logout from the App1, the Portal application is still accessible and when I do refresh the tokens are well received because the cookie and tokens are valid in the Portal's tab. So how can I run the logout from all my applications?
Below my service implemented with the great help of @jeroenheijmans and his shared application:
  public init() {
    // This is tricky, as it might cause race conditions (where access_token is set in another
    // tab before everything is said and done there.
    // TODO: Improve this setup.
    window.addEventListener("storage", event => {
      // The `key` is `null` if the event was caused by `.clear()`
      if (event.key !== "access_token" && event.key !== null) {
        return;
      }

      console.warn(
        "Noticed changes to access_token (most likely from another tab), updating isAuthenticated"
      );
        this.isAuthenticatedSubject$.next(this.isAuthenticated());

      if (!this.isAuthenticated()) {
        this.router.navigateByUrl("/login");
      }
    });
    this.oauthService.events
      .pipe(filter(e => ['session_terminated', 'session_error'].includes(e.type)))
      .subscribe(e => {
        this.oauthService.logOut()
      });
      this.oauthService.checkSession()
    this.oauthService.configure(environment.authCodeFlowConfig);
    this.oauthService
      .loadDiscoveryDocument()
      .then(() =>
        this.oauthService.hasValidIdToken()
          ? this.resolveInitialization()
          : this.oauthService.tryLogin()
      )
      .then(() => {
        return this.oauthService
          .refreshToken()
          .then(() => {
            return this.resolveInitialization();
          })
          .catch(result => {
            this.initializedSubject$.next(true);
            console.warn("User not logged in");
            return Promise.resolve(result);
          });
      })
      .catch(result => {
        this.initializedSubject$.next(true);
        console.warn("User not logged in");
        return Promise.resolve(result);
      });
  }

Many thanks for your help

Glad to hear my sample was useful.
You mention in your question:

My problem is the "Logout", when I try to logout from the App1, the Portal application is still accessible and when I do refresh the tokens are well received because the cookie and tokens are valid

Then you probably need to get some Keycloak support instead, and see how you can have the logout call from an SPA cause the user to be signed out (cookies removed, refresh tokens revoked).
If you'd start up multiple instances of my sample app, you should see this thing already, as that server is configured to do so. Once you log out from app1, the other app's will see that and no longer can refresh or re-login without entering a password on the IDS.
Hope that helps.

Many thanks @jeroenheijmans I'll see that with Keycloak server.


fix(template): Make PR template more task oriented

<!-- Thanks for contributing! -->
The purpose of this PR is:

To create a more task oriented list to help ppl remember to update the docs/changelog
Creating more meaning full standardized title

This is what had to change:
The tasks that shows up in the list of PR shows tasks.
It's often 1-2 out of 4 all the time. or 50% completed

I think we do not use them the "correct way" so I changed it.
I also think this made the template less bloated, ppl don't like to read or follow the template so often
( Less to read/follow === more they will write a better PR )
This is what like reviewers to know:
This PR uses this new template and this is what it looks like. (kind of)

<!-- TASKMASTER: Mark what you have done, Remove unnecessary ones. Add new tasks that may fit -->

[x] I prefixed the PR-title with docs: , fix(area): , feat(area):  or breaking(area):
[ ] I updated ./docs/CHANGELOG.md with a link to this PR or Issue
[ ] I updated ./docs/v3-UPGRADE-GUIDE
[ ] I updated the README.md
[ ] I Added unit test(s)

<!-- Add a `- fix #_NUMBER_` line for every Issue this PR solves. Do not comma separate them -->

fix https://github.com/node-fetch/node-fetch/pull/825#issuecomment-643457324

I like this structure, but I have a question about the Taskmaster and some of the tasks:  Is the Taskmaster an admin/reviewer?  If so, I think updating the README and adding unit tests should be the responsibility of the submitter.

The taskmaster is the person who makes the PR (submitter).
Just took the title from the british TV show Taskmaster and thought it sound a bit cliché
There is a Swedish equivulant called "Bäst i test"
if it's confusing then i think we should just remove it. (And i'm about to do it now)

The taskmaster is the person who makes the PR (submitter).

In that case, instead of Remove unnecessary ones, could we do or indicate why the task is not applicable?

or indicate why the task is not applicable?

Just in this particular case it's pretty clear that it dose not need any of the other task other than "I prefixed the PR-title with..."
Normally I (the submitter) would just remove all the others but wanted to show kind of what it would look like. I do not want to have to explain why i removed some of them.

Add a - [ ] I have searched for similar PR/Issues?
or is it too much?
They will probably search for it anyway if they compliment the this pr fix 001, 002 etc" list

or is it too much?

It isn't too much to ask since this has already happened


Deploy mongodb replicaset using petsets.

https://github.com/kubernetes/kubernetes/issues/30289
Initial support for mongodb replicasets.

This change is

@bprashanth, PTAL and let me know if this is along the right lines. I'll be updating the README in line with the other examples.

@bprashanth Fixed the issues. PTAL.

@foxish I think logging should not be configured in the config file to not change the behavior of the "standard" mongodb image, as setting the logpath back to stdout via the command line is not possible, AFAIK.

@sunruh That is a good point. Thanks for pointing it out.
I've removed logging from the config file. It should be possible now to override it.

@foxish im gettin a
error validating "mongodb.yaml": error validating data: the server could not find the requested resource (get .apps);
on 1.3.5
any idea?

@eyalse Could you do kubectl version and tell me your client and server versions? Are they both 1.3.5?

@eyalse did you updated to newer version of kubernetes or did you installed the kubernetes on fresh system? If it is the former, try to remove ~/.kube/schema.

@foxish & @ingvagabund heya
im deploying a fresh cluster via gke.
kubectl version returns:
Client Version: version.Info{Major:"1", Minor:"3", GitVersion:"v1.3.3", GitCommit:"c6411395e09da356c608896d3d9725acab821418", GitTreeState:"clean", BuildDate:"2016-07-22T20:29:38Z", GoVersion:"go1.6.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"3", GitVersion:"v1.3.5", GitCommit:"b0deb2eb8f4037421077f77cb163dbb4c0a2a9f5", GitTreeState:"clean", BuildDate:"2016-08-11T20:21:58Z", GoVersion:"go1.6.2", Compiler:"gc", Platform:"linux/amd64"}

i think that i need to enable the apiVersion: "apps/v1alpha1" somehow.

/cc @magicwang-cn @kevin-wangzefeng

maybe we should migrate this to charts, https://github.com/kubernetes/charts/pull/38 has set precedence

Yes! Please bring this on over to kubernetes/charts repository. In the incubator directory for now until PetSet is out of alpha.


Fix potential data inconsistency under heavy ddl operation

What problem does this PR solve?
Issue Number: ref #5032
Problem Summary: Currently we cache decoding schema for decoding raft data if a table schema doesn't change. And we judge whether a table schema has changed based on the table schema version.
But the schema version is not strictly consistent with the actual table schema which can be seen at https://github.com/pingcap/tiflash/blob/master/dbms/src/TiDB/Schema/SchemaBuilder.cpp#L362. More concretely, when applying different schema changes in a diff, the table schema version will be set to the latest schema version after the first schema change is applied. And the later schema changes won't change the schema version any more. So the decoding thread cannot detect schema change in these cases and will use the wrong schema version to decoding the data.
cherry pick of https://github.com/pingcap/tiflash/pull/5044
What is changed and how it works?
Add an internal schema version for DecodingStorageSchemaSnapshot.
Check List
Tests

[ ] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix potential data inconsistency under heavy ddl operation

/run-all-tests

/run-all-tests

/merge


Display Saved Tags using select2-rails

Can someone explain how i am supposed to get and display already selected and saved tags with this new code?
$.fn.select2.amd.require(
['select2/data/array', 'select2/utils'],
function (ArrayData, Utils) {
function CustomData ($element, options) {
CustomData.super.constructor.call(this, $element, options);
var data = {
results: []
};
}
Utils.Extend(CustomData, ArrayData);
CustomData.prototype.query = function (params, callback) {
for (var i = 1; i < 5; i++) {
  var s = "";

  for (var j = 0; j < i; j++) {
    s = s + params.term;
  }

  data.results.push({
    id: params.term + i,
    text: s
  });
}

callback(data);

};
$("#select").select2({
dataAdapter: CustomData
});
}
Reading the Select 2.4.0 documentation, it says query function has been deprecated for this function but I dont know how it is supposed to work. I have a this select code
<%= select("learning_article", :tag_list, Tag.all.map(&:name), {}, {class: "js-example-tags", :"data-tags" => "true", :multiple => "true" }) %>
It saves the tags but then it doesnt display the selected tags.

select2-rails does not currently support 4.0.

So how would it work if it  is in select2 3.5.2?


datadog agent build fails due to missing include paths

Output of the info page (if this is a bug)
(Paste the output of the info page here)

Describe what happened:
I'm trying to build the datadog agent on a Raspberry Pi running 32-bit Ubuntu Server 20.04 (armhf architecture).
I followed the directions for cloning and building per the readme.md. When I run the invoke agent.build command I see the build stops with an error indicating it can't find a header file. Did a little triage work and it appears the problem is the build doesn't have the include paths set right to pick up the file from the directory it's located in.
Here is the build output:
`
ubuntu@ubuntu:/space/datadog-agent$ invoke agent.build --build-exclude=systemd  --python-runtimes 3
-- Configuring done
-- Generating done
-- Build files have been written to: /space/datadog-agent/rtloader/build
make: Entering directory '/space/datadog-agent/rtloader/build'
make[1]: Entering directory '/space/datadog-agent/rtloader/build'
make[2]: Entering directory '/space/datadog-agent/rtloader/build'
make[2]: Leaving directory '/space/datadog-agent/rtloader/build'
[ 25%] Built target datadog-agent-rtloader
make[2]: Entering directory '/space/datadog-agent/rtloader/build'
make[2]: Leaving directory '/space/datadog-agent/rtloader/build'
[100%] Built target datadog-agent-three
make[1]: Leaving directory '/space/datadog-agent/rtloader/build'
make: Leaving directory '/space/datadog-agent/rtloader/build'
make: Entering directory '/space/datadog-agent/rtloader/build'
make[1]: Entering directory '/space/datadog-agent/rtloader/build'
make[2]: Entering directory '/space/datadog-agent/rtloader/build'
make[2]: Leaving directory '/space/datadog-agent/rtloader/build'
[ 25%] Built target datadog-agent-rtloader
make[2]: Entering directory '/space/datadog-agent/rtloader/build'
make[2]: Leaving directory '/space/datadog-agent/rtloader/build'
[100%] Built target datadog-agent-three
make[1]: Leaving directory '/space/datadog-agent/rtloader/build'
Install the project...
-- Install configuration: ""
-- Up-to-date: /space/datadog-agent/dev/lib/libdatadog-agent-three.so
-- Up-to-date: /space/datadog-agent/dev/lib/libdatadog-agent-rtloader.so.0.1.0
-- Up-to-date: /space/datadog-agent/dev/lib/libdatadog-agent-rtloader.so.1
-- Up-to-date: /space/datadog-agent/dev/lib/libdatadog-agent-rtloader.so
-- Up-to-date: /space/datadog-agent/dev/include/datadog_agent_rtloader.h
-- Up-to-date: /space/datadog-agent/dev/include/rtloader_types.h
-- Up-to-date: /space/datadog-agent/dev/include/rtloader_mem.h
make: Leaving directory '/space/datadog-agent/rtloader/build'
templates.go
templates.go
go generate ran successfully
github.com/DataDog/datadog-agent/pkg/collector/python
pkg/collector/python/aggregator.go:18:10: fatal error: datadog_agent_rtloader.h: No such file or directory
18 | #include <datadog_agent_rtloader.h>
|          ^~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
`
I tried editing pkg/collector/python/aggregator.go so that it had a fully qualified path to the file and the build failed with a different include file missing which is why I believe the problem is missing include path setting somewhere.
Describe what you expected:
Expected the build to work.
Steps to reproduce the issue:
Clone the repository from github, follow the directions to build the agent as shown above.
Additional environment details (Operating System, Cloud provider, etc):
Raspberry Pi running 32-bit Ubuntu Server 20.04 (armhf architecture)

I encountered the same problem.
In my case, it seems that CGO_CFLAGS was not set correctly because I cloned the source code into the wrong path without following the documentation. (Described below)
Checkout the repo: git clone https://github.com/DataDog/datadog-agent.git $GOPATH/src/github.com/DataDog/datadog-agent.

Also, the CGO_CFLAGS setting seems to be controlled by the embedded_path / rtloader_root options, so you might want to try changing those values.

Hey @nostaw-kgm, @mokemoko,
This is indeed due to the unexpected checkout path. The project was initially a non-go-module project and most of the build scripts still use the expectation that it is checked out in ~/go/src/github.com/DataDog/datadog-agent I believe. If you want to build the source from a different path, you can change some of those CGO lines in aggregator.py to make it build (the lines that need changing):
-#include <datadog_agent_rtloader.h>
+#include "datadog_agent_rtloader.h"
- #cgo !windows LDFLAGS: -ldatadog-agent-rtloader -ldl
+#cgo !windows LDFLAGS: -L${SRCDIR}/../../../rtloader/build/rtloader -ldatadog-agent-rtloader -ldl
-#cgo windows LDFLAGS: -ldatadog-agent-rtloader -lstdc++ -static
+#cgo windows LDFLAGS: -L${SRCDIR}/../../../rtloader/build/rtloader -ldatadog-agent-rtloader -lstdc++ -static
+#cgo CFLAGS: -I "${SRCDIR}/../../../rtloader/include"  -I "${SRCDIR}/../../../rtloader/common"

I'll close this issue for now as it has been resolved it seems but feel free to reopen if needed.


[WIP] [POC] Experiments with MVT rendering using web workers

THIS FOR DISCUSSIONS, DO NOT MERGE.
You can test with:

npm run serve-examples
chromium-browser http://localhost:8080/mapbox-vector-tiles-custom-worker.html

Make sure to run it in chrome as it currently relies on Offscreen canvas (style uncoupling is a WIP).
Here is the workflow:

mainThread renderer#getTile() IDLE -> LOADING
worker renderer#prepareTileInWorker()
worker renderer#renderTileImage()
mainThread renderer#onWorkerMessageReceived() LOADING -> LOADED
mainThread renderer#renderFrame() (compose the tiles, no change)

By changing the useWorker constant in mapbox-vector-tiles-custom-worker.js you can test the two modes:

with worker;
without worker (same as before).

The POC implements the logics for all the render modes.
Currently, it only works with image render mode.
As soon as style uncoupling is done it will work in hybrid mode.
I compared worker / non-worker and the performance gain is very clear.
The frame rate is always around 60 and it feels very smooth.
I am very satisfied with this finding, it validates we can expect a major gain by using web workers.
It also shows Offscreen canvas is a great feature we can use as an extra performance gain, if it is available.
I will continue working on the POC. Notably, I want to fix/investigate these issues:

[ ] after some time there are unrendered blocks on the screen;
[ ] some tiles lack some symbols (looks like rendering happens before all resources are loaded);
[ ] hybrid render mode;
[ ] browsers without offscreencanvas support (limit to polygon + line instructions creation and copy from worker).

@ahocevar, I have reworked the PR following our discussions:

create a dedicated custom MVT example;
minimize changes in the library;
continue loading on non-hifi tiles.

Could you please have a new look before I invest more time in cleaning the example itself and splitting the changes into smaller commits?
Regarding the library changes I:

added setters to allow overriding exported functions (ES6 exports are immutable);
put custom logics into a CustomVectorTileLayer and a CustomVectorTileLayerRenderer that inherits from their library counterparts; these files are in the examples directory;
kept some small conditions where it made sense.

The code is now clearer but the issue with empty tiles appearing after some time has not been fixed. Do you have other ideas about what could produce this behaviour?

@ahocevar, I finally found out what caused the transparent tiles. I will update the PR with the fix and the changes you asked in comments.

@ahocevar, I implemented your fallback suggestions.
I refactored the code to have a cleaner loadImage function, I will think about your idea of extending an offscreen canvas.


children passed to created group aren't added/removed correctly

Version

Phaser Version: 3.23.0

Description
If you pass children when creating a group, createCallback isn't called for those game objects.
If you destroy any of children, [https://photonstorm.github.io/phaser3-docs/Phaser.GameObjects.Group.html#removeCallback] isn't called and the destroyed game object is not removed from the group.
Example Test Code
https://codepen.io/samme/pen/WNQKgPq
See console. Assertions should pass.
Additional Information
children need to be added after the group is configured.

Thanks for opening this issue, and for submitting a PR to fix it. We have merged your PR into the master branch and attributed the work to you in the Change Log. If you need to tweak the code for whatever reason please submit a new PR.


chore: profile routes

Profile routes get and put

/:id | Read - get
/:id | Update - put

https://trello.com/c/7GHVoWsR/75-backend-routing-profiles

 Backend Routing: Profiles

So i had to edit the .env file because that's what was preventing the authMiddleware from creating a new profile for a new user. Now it works!


fix concurrent safety for package gdb

Codecov Report

Merging #1998 (e78a1f8) into master (3e7e8ba) will decrease coverage by 0.08%.
The diff coverage is 38.88%.

@@            Coverage Diff             @@
##           master    #1998      +/-   ##
==========================================
- Coverage   76.48%   76.39%   -0.09%
==========================================
  Files         512      511       -1
  Lines       46651    46411     -240
==========================================
- Hits        35679    35454     -225
+ Misses       9000     8995       -5
+ Partials     1972     1962      -10

Flag
Coverage Δ

go-1.15-386
76.39% <38.88%> (-0.01%)
:arrow_down:

go-1.15-amd64
?

go-1.16-386
?

go-1.16-amd64
?

go-1.17-386
?

go-1.17-amd64
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

database/gdb/gdb.go
66.83% <38.88%> (+0.67%)
:arrow_up:

net/gtcp/gtcp_pool.go
68.18% <0.00%> (-6.82%)
:arrow_down:

contrib/registry/polaris/polaris_watcher.go
67.85% <0.00%> (-5.36%)
:arrow_down:

os/gfsnotify/gfsnotify_watcher_loop.go
78.15% <0.00%> (-5.05%)
:arrow_down:

os/gcron/gcron_schedule.go
70.42% <0.00%> (-1.17%)
:arrow_down:

container/gmap/gmap_hash_str_any_map.go
97.00% <0.00%> (-1.00%)
:arrow_down:

os/gcmd/gcmd_parser.go
92.19% <0.00%> (-0.71%)
:arrow_down:

container/gtree/gtree_avltree.go
95.40% <0.00%> (-0.58%)
:arrow_down:

contrib/drivers/oracle/oracle.go

Continue to review full report at Codecov.

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


in okhttp3 we are unable to send Json data in side the request..server setup is of  PHP.  server side not getting any request parameter while sending parameter from android

What kind of issue is this?

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use Stack
Overflow. https://stackoverflow.com/questions/tagged/okhttp

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed. Here’s an example: https://gist.github.com/swankjesse/981fcae102f513eb13ed

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

Please ask questions on Stack Overflow, https://stackoverflow.com/questions/tagged/okhttp


Decimal values exceeding scale now properly round

This PR fixes #5288

Opened here for now so I don't lose track of it while we're freezing all PRs until release :)

BUG: digit is passed as unsigned, so it can never be false
(comparison always compares to false warning..)


Adds the ability to install via pip

Fixes #35
Now you can install it via pip install tespeed
NOTE: This has not been uploaded to PyPi yet (Just PyPiTest). Leaving that honor
to the owner :)

Opened wrong pull request. Closing and opening one directly to Janhouse:master


vulkan-headers: revert version back to <IP_ADDRESS>

Motivation for this change
Introduced through #66822
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

#67841 fixes vulkan-tools as well


Unable to get selected value out of dynamically created mat-select without falling back to NativeControl

What is the expected behavior?
Being able to get the selected value out of a dynamically created mat-select control with document.getNativeElement or any other way
WITHOUT USING NATIVECONTROL, as it is being drawn completely different from the mat-select!
Using NativeControl IS NOT THE SOLUTION!
What is the current behavior?
NativeElement created by mat-select does not hold the selected value
What are the steps to reproduce?
<mat-form-field>
<mat-select id={{column.columnName}} placeholder="{{column.columnName|translate|capitalize}}">
<mat-option *ngFor="let foreignRow of foreignDataMap[getForeignTableName(column.columnName)]" [value]="foreignRow.uid">
 {{foreignRow.value}}
</mat-option>
</mat-select>
</mat-form-field>

var element = document.getElementById( column.columnName ) as HTMLInputElement;

element.value <-- does not exist

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All latest.
Is there anything else we should know?
Yes, please STOP CLOSING this issue without providing the bugfix!
Mixing nativeControl with mat controls is unacceptable, UI looks absolutely wrong.

See response https://github.com/angular/material2/issues/15341#issuecomment-469151248 please try and keep all comments on a single issue, even if that issue has been closed. If you make your case well enough the issue will be reopened.

Please stop closing issues without providing the solution and I'll stay in that single issue. :-) 👍

Closing as duplicate of #15334
As noted in the other issues you have opened, this is working as expected.  Because the value is held in Angular's representation of the data structure, it is not stored in the DOM element.
Because our MatSelect is not built on top of the native <select>, the value property does not event exist on the <mat-select> element.  Since the underlying DOM object for <mat-select> element is not HTMLSelectElement (it is instead an HTMLElement), all of the properties that are available to interact with for a native <select> are not present.  This results in what you are looking to do in your description not being possible, it is a limitation caused how renders make rendering determinations with the HTMLSelectElement, preventing us from using it.
Instead you will need to, as noted in #15334, get a reference to the MatSelect instance either with a ViewChild or a template reference.  From there you will be able to get the value for the element.  I understand this is not the answer you are looking for on this issue, however this is the position we find ourselves in with the limitations of browsers.
Please note:  As has been mentioned in the other issues you have opened for this topic, It is not appropriate to continue opening multiple issues for the same request.  If you would like to continue discussion, please do so in the original issue that is opened for a request/issue.  Opening multiple issues for the same thing only works to fragment the discussion and make it more difficult for us to support and provides response/answers/solutions.