Add Directus

https://github.com/directus-community/awesome-directus
List with interesting links related to Directus (application which provides a real-time API and App dashboard for managing SQL database content) which helps users to find and share resources like:

Tools and libraries to integrate Directus into their project
Extensions to full-fill their use-cases

By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions.
You have read and understood the instructions for creating a list.
This pull request has a title in the format Add Name of List.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

Your entry here should include a short description about the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

Your entry should be added at the bottom of the appropriate category.
The title of your entry should be title-cased and the URL to your list should end in #readme.

Example: - [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.

The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it.
Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
The default branch should be named main, not master.
Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

Non-generated Markdown file in a GitHub repo.
The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
Not a duplicate. Please search for existing submissions.
Only has awesome items. Awesome lists are curations of the best, not everything.
Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

Doesn't use hard-wrapping.
Doesn't include a Travis badge.You can still use Travis for list linting, but the badge has no value in the readme.
Doesn't include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Go to the top and read it again.

I've reviewed #2079 and #2076.

There are almost no resources in the list. It's expected that you put more effort into an Awesome list for it to be added here.
App should be app.

This repo is sort of well prepared, for its high-resolution logo image and clean structure. However, I expect to see more contents in the future.

As already stated, it seems to be a WIP PullRequest?
The repo itself looks good :)

This is ready to merged. Congratulations.

The description starts with an uppercase character and ends with a period.

And other things. Please read each item in the guidelines.

The description starts with an uppercase character and ends with a period.

And other things. Please read each item in the guidelines.

This has now been fixed and we're about to enforce the linting check to make sure the formatting will always be right.

Integration Examples with Directus. => Integration examples with Directus.

Tweet: https://twitter.com/awesome__re/status/1532323558063341569


Finished create group channel use case and completed test coverage.

New function works as intended, no errors
Added tests pass, contributing to 100% test coverage


Value injector for RESTEasy

What do you think?

Oh great! I'm going to review this soon!

You should already try to enable the tests that involve the annotations, it's easy, there is just some empty overriding test method to remove normally.

There are also some formatting problems... some tabs crept up :)

I am investigating issues with tests right now.

The only test which fails for me is testInject(). And it only fails when it is being run with other tests. If I launch this single test, then it passes.
I get a weird error message from Undertow: UT005014: Failed to parse HTTP request.
Which happens because this is  exactly(!) what it receives on the wire (I have copied the complete byte buffer from the Undertow request parser):
User-Agent: Java/1.8.0_152-release
Host: localhost:24257
Accept: text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2
Connection: close
Cookie: JSESSIONID=5icpg-Zdu3SOLgo2QXDn_s2X_zzO00y5auNK9hqu

At this moment, I have no clue.
I have added a delay in the very beginning of testInject() for 5000ms and all tests pass now, but this is obviously not a solution, it's more of a hint for us.

Thanks for the exploration.
I suppose it is undertow's closing that is not properly synchronized with the execution of tests or something like that… I say that because I've noticed harmless (until now...) undertow errors in the logs at the end of the tests before.
I will try to find the time to fix this next week-end, but if you feel like it, don't hesitate to continue exploring and maybe finding the solution ;)

I have ignored the failing test so that the build is successful now. I have also updated the documentation.
Would you mind merging this PR in this state and moving the test problem to a separate GitHub issue?

yes, ok, let's do that

Thanks again @yegorius :)
We should do a release soon then! Maybe 2.1.0 @leleuj, what do you think?

Yes, it makes sense. It'a noticeable update. Starting...

@victornoel The version is still 2.0.3, I think it should be 2.1.0 as it's a backward compatible change (not a bug fix), isn't it?

yes, 2.1.0 is a good number! (that's what I was saying in my comment above ; )

Yes, right. I just performed the release, it will show up in the Maven central repository in a few hours...


Fix Capitals in navigation bar

Fixes Category Acronyms Capitals

it seems like a very long MR to just fix the category acronyms capitals... also I don't get what was the bug here? The problem is when you have an acronym in a category?


Get grpc error when I use tensorflow java API to load model.

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS 10.14.2
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.15.0
Python version: 3.7

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I run tensorflow java api to load model in flink cluster, it works fine the first time. But When I run the job the second time in cluster,  it turns out to be an error like below:
[libprotobuf ERROR external/protobuf_archive/src/google/protobuf/descriptor_database.cc:58] File already exists in database: tensorflow/core/protobuf/eager_service.proto
[libprotobuf FATAL external/protobuf_archive/src/google/protobuf/descriptor.cc:1358] CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size):
libc++abi.dylib: terminating with uncaught exception of type google::protobuf::FatalException: CHECK failed: GeneratedDatabase()->Add(encoded_file_descriptor, size):
Describe the expected behavior
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

There is a similar bug issue link in grpc issues
https://github.com/grpc/grpc/issues/13249

The root cause may be similar to https://github.com/tensorflow/tensorflow/issues/20561. Both issues cause error message "File already exists in database..". It likely that Flink and Tensorflow linked different versions of the same library (maybe protobuf) twice.

The root cause may be similar to #20561. Both issues cause error message "File already exists in database..". It is possible that Flink and Tensorflow linked different versions of the same library (e.g. protobuf) twice.

Here is the github where I show how to reproduce the issue.   @lindong28  @sjamesr Thanks!
https://github.com/tywtyw/flink-tensorflow

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.

I encountered the same question, how did you fix it? @tywtyw


Automated cherry pick of #4344: fix: reduce action log for keystone authentication event

Cherry pick of #4344 on release/2.13.
#4344: fix: reduce action log for keystone authentication event

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@c6f04de). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #4345   +/-   ##
==============================================
  Coverage                ?   6.89%
==============================================
  Files                   ?     521
  Lines                   ?   88278
  Branches                ?       0
==============================================
  Hits                    ?    6085
  Misses                  ?   81687
  Partials                ?     506

Flag
Coverage Δ

#aFlag
6.89% <0%> (?)

Impacted Files
Coverage Δ

pkg/keystone/models/users.go
1.75% <0%> (ø)

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 c6f04de...b49b189. Read the comment docs.


Synchronizing self-hosted

Greetings!
Will synchronization work if I put Financier on my server?

It's definitely possible, but there will need to be a few code changes to setup the database.
Here's some examples:

https://www.reddit.com/r/financier/comments/7d9osz/scripts_for_running_financier_fork_on_own_server/
https://www.reddit.com/r/financier/comments/t59jec/could_someone_walk_me_through_setting_this_up_on/
https://hub.docker.com/r/tbi88/financier


Feature Request: Add "override" option in TokenEffectIcon that changes the actual effect icon

so the changed icon shows in the sheet, and more importantly the effect panel, too.

now that i think about it, maybe divorcing it from TokenEffectIcon would be better, because then it's possible to have the effect not show on the token too


Update index.stache (and any other templates) to use call expressions for helpers

https://github.com/donejs/generator-donejs/blob/master/app/templates/src/index.stache

This is in major.


Add gltfupaxis support

Details discussed in https://github.com/CesiumGS/cesium-unreal/issues/315

thanks @javagl, I only reword a little bit on CHANGE.md. This can be merged after the CI pass


mp4 no sound

https://blog.txshore.tech/2017/12/10/GoT-s07e03/#more
the video doesn't have sound, but it has sound when playing locally, so it's no the video itsself's problem, anyone can help!

Your video is blocked by CORS policy, I can't play it.

I've solved the sound problem, that is because the sound format was not AAC
But now I meet problem while adding subtitles in MD document.
While I write {% ****** "subtitle=url.vtt" %} in MD document, the whole player doesn't work, I don't know how to add subtitle code in {% %}

See: https://github.com/MoePlayer/hexo-tag-dplayer#usage


Move the docker registry cache to the manager cluster

#522 deployed a docker registry cache in live-1 to help speed up concourse builds, by caching frequently-used docker images.
Now that concourse is running in the manager cluster, the cache should be deployed there, and removed from live-1.
Alternatively, leave the cache running in live-1 and configure the manager cluster to use it.

dupe of https://github.com/ministryofjustice/cloud-platform/issues/1895


Add Func overloads for MDC/NDC/GDC/event properties

Allows:
MappedDiagnosticsContext.Set("Property", () => Environment.TickCount);

See https://github.com/NLog/NLog/pull/743

[ ] also remove trick from wiki: (PropertyWrapper) : https://github.com/NLog/NLog/wiki/gdc-Layout-Renderer#dynamic-properties

Adding Action delegates is more appropriate. The Func cannot return a value which is required for us to set the value of the key in the Context dictionary.


How to get the value of $key ?

$key = Key::createNewRandomKey() .
I tried to store the encrypted password and the value of $key in to the database , but failed . Just found " private 'key_bytes' => string ......" .
So, how to get the value of $key ?

Why would you want to store the key in the same place as the ciphertext? That's a dangerous decision that totally defeats the security of the encryption.
Instead, you should have one key that isn't stored on the same hardware as the database server, let alone in the database server.
Anyway: https://github.com/defuse/php-encryption/blob/master/docs/classes/Key.md#savetoasciisafestring and https://github.com/defuse/php-encryption/blob/master/docs/classes/Key.md#keyloadfromasciisafestringsaved_key_string-do_not_trim--false are the answers here.

Thank you .


Local malware/AV IaaS service - start with Shielded VMs

Issue for this design item is active security at the IaaS level (not after the effect) - we do however have real time at the network
see
DI 5: IaaS/PaaS/SaaS Application Security
Determine list of services to help enable applicaton security firewalls, vulnerability, OS protections.  SCC Threat detection handles what is going on inside the IaaS/PaaS systems where Armor/IDS handle ingress/egress traffic and Shielded VMs handle IaaS.
Security Command Center Premium
Threat detection works by continuous monitoring of all cloud logs including those generated at the VM/Container OS level - via embedded logging agents and Workspace logs.
https://cloud.google.com/security-command-center/docs/concepts-event-threat-detection-overview
Proactive threat detection also occurs at the perimeter to customer networks via Cloud Armor https://cloud.google.com/armor - Google Cloud's DDos and WAF SaaS.  Detection can be customized by adding rules - the following is in place by default

ML based layer 7 DDoS attacks
OWASP top 10 for hybrid
Load Balancer attacks
Bot management via reCAPTCHA

In addition for Chrome based clients we have BeyondCorp zero trust capabilities.
Cloud Armor
Cloud Armor is part of both network and IaaS/PaaS security.
IDS - Intrusion Detection System
GCP Intrusion Detection System Service (based on the Palo Alto security appliance) - https://cloud.google.com/intrusion-detection-system handles Malware, Spyware and Command-and-Control attacks
Vendor based
For clients with existing Microsoft Defender Endpoint solutions https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/microsoft-defender-endpoint?view=o365-worldwide - Google Cloud supports the previous Cloud app security (Defender for Cloud Apps) solution https://docs.microsoft.com/en-us/defender-cloud-apps/connect-google-gcp
Fortigate
Shielded VMs for IaaS
Looking into Shielded VMs - looks like a local IaaS version of the PaaS services.  Has malware support, no anti-virus yet TBD.
https://cloud.google.com/shielded-vm

20240406: Closing issue during retrofit/rebase of this TEF V1 based/modified repo to TEF V4 standards
This issue may participate in the LZ refactor after rebase
Query on all issues related to the older V1 version via the tag
https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/labels/2024-pre-tef-v4


[FLINK-25997][state/changelog] Fix materialization boundary for FsSta…

What is the purpose of the change
This pull request modifies the way ChangelogKeyedStateBackend uses the sqn maintained by FsStateChangelogWriter.
Avoid always using sqn.next() to act as "upTo" sqn, which is not correct when there is no change appended.
Brief change log

Replace "lastAppendedSequenceNumber" with "nextAppendedSequenceNumber"

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is already covered by existing tests, such as ChangelogDelegateHashMapTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

@rkhachatryan Could you help to review the pr? Thanks!

Thanks for the PR @masteryhx
I have a couple of high-level remarks:

with materialization enabled (100ms interval), ProcessingTimeWindowCheckpointingITCase#testSlidingProcessingTimeWindow fails locally; could you please check this?
the change essentially replaces the use of lastAppendedSequenceNumber with nextAppendedSequenceNumber (and the former isn't use in production code anymore); so the latter should be covered by tests (e.g. FsStateChangelogWriterSqnTest)

@flinkbot run azure

I've discovered several unrelated issues that cause ProcessingTimeWindowCheckpointingITCase to hang or fail: FLINK-26019 / FLINK-26062, FLINK-26063.
Besides that, I've published an alternative fix to the problem of this PR: #18693. It makes lastAppendedSequenceNumber return Optional. That way seems less invasive and more explicit. But in your version API seems more clear.
WDYT?


[BUG] degraded RWX performance

Longhorn RWX performance very bad
we installed a new rancher cluster with longhorn and tested downloads within 3 containers using one RWX volume.
The bottleneck is obviously the RWX volume and allows for 200KB/s.
In a read-write-one volume we can do the same thing at 100MB/s
This issue happens on clusters on Hetzner Cloud (german hoster) and on a Cluster on AWS
To Reproduce
Steps to reproduce the behavior:

Deploy daemon set with RWX volume
go into volumes mount point on any container
download large iso file
only 200KB/s

Expected behavior
The performance when writing into the RWX volume should be similar to read-write-one, maybe not 100% as fat, but in the same ballpark
Log
no relevant output
You can also attach a Support Bundle here. You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment:

Longhorn version: 1.1.1
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Catalog
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE

Number of management node in the cluster: 3
Number of worker node in the cluster: 3

Node config

OS type and version: ubuntu 20.04
CPU per node: 4
Memory per node: 16GB
Disk type(e.g. SSD/NVMe): i would assume SSD
Network bandwidth between the nodes: 1Gbit

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Hetzner Cloud (probably KVM)
Number of Longhorn volumes in the cluster: one RWX volume

Additional context
Here is a screenshot with "dd" into the disks, where the RWX volume is mounted to "/tmp"

This is a duplicate of #2549. We will tackle this together with #2549.

This is super important to fix imho, I'm sad it has been moved to 1.3.0... Are there any workarounds we can do in the meantime?

@wargamez This appears to be a workaround. Seems a good bit slower than RWO but lots faster than native RWX:
https://github.com/longhorn/longhorn/issues/2549#issuecomment-829265306

Closing the duplicated bug, and we can track #2549.


Use yum as package provider for RedHat

I noticed that with epel enabled, yum can resolve the erlang dependencies of rabbitmq without the help of additional puppet erlang modules on CentOs7. Since epel is needed on both CentOS and RedHat, maybe its safe to use yum as the package provider for them instead of rpm.

@stieizc Spec tests need to be updated to reflect this change.
https://github.com/puppetlabs/puppetlabs-rabbitmq/blob/master/spec/classes/rabbitmq_spec.rb#L1120

Update the spec, and test fails for Puppet 4. Any idea? I will look into the build log when in a few days.

@stieizc Do you mind rebasing off master to pull in my test fixes from #416? That should fix unit tests.

I'm Glad to. And now all checks have passed.

@stieizc Sorry for not requesting this last time, but can you squash the two commits down to 1?

Sure.


update mocha to version 3.4.0

https://greenkeeper.io/

Coverage remained the same at 100.0% when pulling 3a71bc1bcf6350a6c7c1dc170c25282e33dde54b on greenkeeper-mocha-3.4.0 into 0e0b27c9aa762732ee8efd90f248e26cc0e6578f on master.


Enable complete concurrency checking in Foundation

Enable complete concurrency checking in Foundation, and Swift 6 mode for the unit tests.

@swift-ci test


Errors during docker production image build

I am trying Docking for the first time, and I went through the process of installing it through  building since I could not get pass the password section when trying the dev docker image.
I was getting the following error during docker build:
10.34 checking for PHP extension directory... /usr/local/lib/php/extensions/no-debug-non-zts-20220829
10.34 checking for PHP installed headers prefix... /usr/local/include/php
10.34 checking if debug is enabled... no
10.36 checking if zts is enabled... no
10.37 checking for gawk... no
10.37 checking for nawk... nawk
10.37 checking if nawk is broken... no
10.37 checking for PostgreSQL support for PDO... yes, shared
10.37 checking for pg_config... not found
10.37 configure: error: Cannot find libpq-fe.h. Please specify correct PostgreSQL installation path
------
prod.Dockerfile:26
--------------------
  24 |
  25 |     # Install PHP extensions
  26 | >>> RUN docker-php-ext-install pdo pdo_pgsql pdo_mysql mbstring exif pcntl bcmath gd
  27 |
  28 |     # Copy project files
--------------------
ERROR: failed to solve: process "/bin/sh -c docker-php-ext-install pdo pdo_pgsql pdo_mysql mbstring exif pcntl bcmath gd" did not complete successfully: exit code: 1

I used the following as reference: https://github.com/docker-library/php/issues/221
and added the following to the prod.Dockerfile
# Install Postgre PDO
RUN apt-get install -y libpq-dev \
    && docker-php-ext-configure pgsql -with-pgsql=/usr/local/pgsql \
    && docker-php-ext-install pdo pdo_pgsql pgsql

And it was able to build the image successfully. But I am now getting the following:

thanks friend, lemme give it a look

FYA I'm working on this https://github.com/shipsaas/docking/pull/29 and hopefully will merge it soon, so that you and everybody else won't suffer this kind of issue any more

It has been released 💪 I'll close this one, thank you
https://github.com/shipsaas/docking/releases/tag/v1.4.2


iOS: Badge related notifications sound/vibrate issue

I was (seemingly) randomly getting notifications with no sound/vibration or no notification at all. This issue had me puzzled for a few months but I finally figured out the pattern.
When pushing notifications with a custom sound and a badge changing the app badge icon from anything to zero, you won't get any sound or vibration with the notification.
Example test pushes with my custom sound.
// Works fine and gives the app icon a badge of 5
notification: {
  title: 'Message',
  body: 'Test 1',
  sound: 'cash.caf',
  badge: 5
}

// Gives a completely silent notification and removes the badge
notification: {
  title: 'Message',
  body: 'Test 2',
  sound: 'cash.caf',
  badge: 0
}

// Sound works again
notification: {
  title: 'Message',
  body: 'Test 3',
  sound: 'cash.caf',
  badge: 0
}

Same scenario (with default sound)
// Works fine and gives the app icon a badge of 5
notification: {
  title: 'Message',
  body: 'Test 1',
  sound: 'default',
  badge: 5
}

// Works fine and removes the badge
notification: {
  title: 'Message',
  body: 'Test 2',
  sound: 'default',
  badge: 0
}

// Still works fine
notification: {
  title: 'Message',
  body: 'Test 3',
  sound: 'default',
  badge: 0
}

I'm guessing this one has been around for a while since I've given up several times on figuring out what was going on.
Using:
react-native v0.48.3
react-native-fcm v9.1.0

is it local notification or remote?

Ah sorry. They are remote.

looks like it is notification property in remote notif, which is 100% handled by FCM SDK. you need to report an issue to them.

please reopen if you have more question

Got this response from Firebase support:
Thanks Baldur.
What version of iOS have you been testing on? We've tried on iOS 11 and were unable to reproduce it. Also, how the sound gets played should be controlled by iOS and not FCM. According to Apple's docs when a non-default sound can't be played, the default sound gets played instead. Can you try adding a breakpoint when you receive the notification and inspect userInfo dictionary to see what the "sound" value is?

@evollu I am unable to re-open issues unless I close them myself.

still it doesn't sound issue of this repo.
you can try to reproduce it using quickstart-ios from firebase. If it all works fine then we can take a look here.
I guess iOS just mute the notif when you set badge to 0?

No mute doesn't happen if you always use badge = 0, just when the badge changes to 0.
But ok, I'll check this further.
Thanks.


Reshare

-Unable to reshare the recovery key from ios.
-When we click on Reshare it is not showing link/QR modal(Tested on iphone 7)

Unable to reproduce this on upgraded app and also on fresh installed.


[Segment] Attached segment does not have raised variation

Not sure whether or not this is by design.
JSFiddle: http://jsfiddle.net/8px6860p/

It's by design. The raised shadow would overlap whatever content it was attached to.

I've added new group variations for stacked, raised, and piled. However they will only work with plural grouped segments.


Chapter 6 - where is cnn_utils

Trying to follow along with the book - had an interest in CNN so I started with the CIFAR-10.
This code finds no library

import cnn_utils as utils.  It's used at the end of the cifar10 transfer learning based classifier.  I cannot find it in my anaconda distribution - cant find it searching anaconda to add, and does not seem to be in the git for the book.
Where is it?

@PCJimmmy  Thanks for pointing this out. I think this is a custom library\module so you may not find it in Anaconda. We will push the module to the repository soon
@raghavbali would be great if you could push the cnn_utils.py module and update here.

Thanks
Like the book – I bought a bunch over the last couple of months as I venture into Python – yours was my most recent purchase and finding it very helpful.
Jim
Sent from Mail for Windows 10
From: Dipanjan Sarkar
Sent: Tuesday, September 25, 2018 3:30 PM
To: dipanjanS/hands-on-transfer-learning-with-python
Cc: PCJimmmy; Mention
Subject: Re: [dipanjanS/hands-on-transfer-learning-with-python] Chapter 6 -where is cnn_utils (#1)
@PCJimmmy Thanks for pointing this out. I think this is a custom library\module so you may not find it in Anaconda. We will push the module to the repository soon
@raghavbali would be great if you could push the cnn_utils.py module and update here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Updated repo with cnn_utils.py
Thanks for the catch @PCJimmmy , some how missed including the file.
Kindly reopen if there are issues. Cheers!

Added a new issue about get_metrics from the same chapter and verse
– but the utils plot worked fine.
jim
Sent from Mail for Windows 10
From: Raghav Bali
Sent: Thursday, September 27, 2018 10:36 AM
To: dipanjanS/hands-on-transfer-learning-with-python
Cc: PCJimmmy; Mention
Subject: Re: [dipanjanS/hands-on-transfer-learning-with-python] Chapter 6 -where is cnn_utils (#1)
Updated repo with cnn_utils.py
Thanks for the catch @PCJimmmy , some how missed including the file.
Kindly reopen if there are issues. Cheers!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


test: Set CT map sizes

Previously, some ConfigMaps used by Cilium in testing didn't have the CT map sizes specified which defaulted to 1000000, or they were explicitly set to 1000000. At the same time, ConfigMaps from examples/, which were used by tests as well, have the CT map sizes set to < 1000000.
This made the CT maps being recreated due to the property change which lead to existing connections to be broken during Cilium upgrades, thus the SVC migration tests were failing. As an example, Nightly tests were failing due to this.
This PR sets the CT map sizes to the ones from examples/. Also, it removes --enable-legacy-services from the v1.4 yamls, as the v1.4 does not support the param.
Fix #7641

This change is

test-me-please

test-nightly

Coverage increased (+0.03%) to 41.262% when pulling fe48b69acfac23f4a71a207783737b5f07d478b8 on pr/brb/ci-avoid-del-ct-maps-in-nightly into b0d25a8f57fcca978b8286cc9e86d6ddcb3e30d9 on master.

@brb What are the consequences for our users? Will they also need to modify the size of their maps? This might required documentation changes with fat warnings.

What are the consequences for our users?

Sorry, my problem statement was wrong. The CT max entries default value in the Go code is the same for all versions - 1 000 000, while in the DaemonSet files located in examples/ we set the value to 524 288. So, the problem was in the clean-state CMs used by the Update tests we didn't set the CT max entries value, and during the update we used the CM from examples/. I will update the PR description and commit message after the nightly run has finished.

@brb What are the consequences for our users? Will they also need to modify the size of their maps? This might required documentation changes with fat warnings.

@brb @aanm It's a bit weird, the default in 1.3 is also 1 mio for TCP. See this change CTMapEntriesGlobalTCPDefault = 1000000 from 1.3 branch:
https://github.com/cilium/cilium/commit/e824a86bba2155226fdca7ef59d143c53ad43795#diff-b72a2fd7a91a97fb4b234e7919334832
So the default is in both cases on 1 mio.
Only in the yaml files we set it to 512k entries. Is one of the tests not using the example yamls such that we get a 512k vs 1mio mismatch?

@aanm Any idea why there is a difference between the i) CM patches and ii) example yamls? Is it such that i) should be considered when upgrading from version x to y where we assume default settings in the agent (hence the 1 mio)? If so might be good to add a comment in there.

OK, cleaned the mess in the CT map sizes and updated the PR description. PTAL.

test-me-please

test-missed-k8s

test-nightly

test-me-please

test-missed-k8s

test-nightly

#7859 needs to be merged first to make the failed tests pass.

test-me-please

test-me-please

test-nightly

test-missed-k8s

test-me-please

PTAL @aanm

Hmm, after merging #7859 this PR became obsolete (minus the test: Do not set bpf-ct-global-any-max commit): #7859 was backported neither to v1.3 nor to v1.4, therefore when upgrading Cilium from Stable -> Development -> Stable (Stable = v1.3 or v1.4), the Stable has CT TCP map size of 1000000. To avoid removing the map, we need to set Development CT TCP map size to 1000000 too.

Hmm, after merging #7859 this PR became obsolete: #7859 was backported neither to v1.3 nor to v1.4, therefore when upgrading Cilium from Stable -> Development -> Stable (Stable = v1.3 or v1.4), the Stable has CT TCP map size of 1000000. To avoid removing the map, we need to keep the Development CT TCP map size set to 1000000.


UnclaimedSwapsRequest

@Valentine1898 saw this has been in progress for a few weeks. Everything ok with this task?

Yeah, I'll probably finish it today. I've had more days off the last few weeks


Add support for arca.live

Arca.live is an internet forum.
Like this post, there are posts that consist of multiple images, but there isn't a useful image viewer to browse through them.
Could you please add support for this site?

I saw you comitted for my request!
I tested v4.7.6 on chrome and other chromium-based browser called naver whale and it works flawlessly!
Thank you for the support of this site.

At certain times, the script couldn't retrieve images, so I fixed this issue.
Since I'm not very familiar with this forum, there might still be some minor problems.

I'll try to fix some if there are some and make pull request about it.
Thank you!


Expose all CLI options to the npm library

Currently only the json flag can be set in the options for runFta. Adding more or all options could be helpful.
Adding options currently exposed to the CLI should be straightforward. For other config options I think #63 should be implemented before implementing this issue. Otherwise, I don't see a way of executing runFta without creating a fta.json for every run.
Additionally, I think changing the exported functions could be beneficial. Currently runFta always returns a string, either containing a formatted table or an array of AnalyzedFile as a string. Automatically converting the array would be helpful. Also, exposing two different functions could help create a clearer interface. Otherwise, every call of runFta would require a check or an explicit cast, even though the expected type is likely obvious from the context.
Other ideas are of course welcome!

I took a deeper look into the project code and WebAssembly. What is the reason for creating several different executables for the CLI instead of using WebAssembly?
There is even fta-wasm at the moment that exposes analyze_file_wasm. Perhaps fta-wasm could be used for the npm package? This could not only address the issue of creating a good interface for the npm package, but it would probably also significantly reduce the package size.
I quickly created a small test on my fork of the project.

Hi @EcoFreshKase. In general, distributing separate binaries was a pratical way to ship the tool to multiple operating systems. It's generally designed to be used as a CLI tool.
That said, it also occurred to me that I could output a wasm build and allow it to run on the web, which works. However, I wasn't envisioning fta being used like a library.
I'm definitely open to perhaps another npm package coexisting that is a library version of fta -- at least, I think that's what you're getting at here?

@sgb-io yes, I would love to have fta as a library, because I want to use it to create a tool in the future.
How would you like to have this implement. Should we creat a npm package fta that has all the library functionality or should this be added to fta-cli? I think both approaches would be feasible. What do you think?

@EcoFreshKase I would suggest that fta-cli stays as a CLI wrapper, and a new (non-CLI-named) npm package could be used for the library concept.
Note that you can't use fta as it's too short (NPM don't allow this for security reasons). It could be namespaced like @fta/core or @fta/metrics. Could also be a totally different name like fta-node.
If you can help propose what the library should contain/expose/do I'd be happy to get involved with this

@sgb-io I think it would make sense to group all the fta packages under a common namespace. For the library package I think @fta/core would be fine. In that case I would also propose to move fta-cli to @fta/cli and fta-wasm could be deprecated as this would propably be moved to @fta/core.
As far as I understand scoped packages at the moment, we would need to create a organization in npm. Otherwise, we wouldn't have the @fta prefix for the packages. But I didn't test it yet.
As for what should be contained in the library. I would suggest at least functions to analyze single files and whole projects and a possibility to get the metrics that are used internaly. Maybe even some util functionality to e.g. calculate the fta_score while providing all the metrics. I will try to find time in the coming days to create a whole scheme for what I think should be exposed.

@sgb-io sorry for the wait I was sick the last week.
I didn't want to create a whole docs and leave room for the implementation so I decided to just write a short sentence to everything I think should be contained in the library. This should cover everything I think or do you have any other ideas?
You said you wanted to get involved in this. How do you think should we split the work for this?
scheme
analyzeFile
should take a path to a file that should be analyzed by fta and options to configure the analysis.
analyzeProject
should take a path to the root of whole project that should be analyzed by fta and options to configure the analysis.
Options
Type that contains all configurations for the analysis
AnalyzedFile
fta results for one file
AnalyzedProject
fta results for a whole project
getScore
takes an AnalyzedFile without fta_score and returns the fta_score


Think about uses of redshift

Before we talk to Chris

Matt suggests writing a page on redshift and taking it to Chris

Solved by fa71e43. I would like to show this description to Chris and get his feedback.


ajaxSpider does not correctly work with scrapeOne

If you specify scrapeOne in the ajaxSpider params, it attempts to use values in params.scrape to configure the method. This leads to the error Uncaught TypeError: Cannot read property 'iterator' of undefined. I'm sorry I haven't written a PR but I'm not confident in the quality of my solution.
https://github.com/medialab/artoo/blob/facc765f9ec015e26d570680756140c5c3650915/src/methods/artoo.methods.ajaxSpider.js#L61

Hello @GarySiu. I remember tackling this issue but something was amiss. Can you submit your PR on the repo so we can work this out together?

Created #265. Should probably write a test to cover it but not sure where.


[Concurrent Segment Search] Concurrent search index setting default value is false but is not evaluated when not explicitly set

Is your feature request related to a problem? Please describe
The concurrent segment search index setting has a default value of false, however the setting itself is not used unless it is explicitly set. This can be confusing as checking an index setting with the include_defaults flag will show the concurrent search index setting as false even though concurrent search may still be enabled for a given index depending on the cluster setting.
Describe the solution you'd like
Using the include_defaults flag when getting index settings should give a clearer picture of whether or not concurrent segment search is enabled.
Related component
Search:Performance
Describe alternatives you've considered
No response
Additional context
No response

Going to close this out as I don't think it makes sense for the include_defaults value to show anything other than the default specified in code.


Error when doing Install-Package Thinktecture.IdentityModel.WebApi

When I try and install via Nuget I get the following error: Failed to add reference to 'System.IdentityModel'. Please make sure that it is in the Global Assembly Cache.
I'm running on .NET 4.5.1 so I'm not sure why I am getting the error? Any help is appreciated. Thanks

I just tried and it worked for me. Perhaps enable verbose logging in the nuget window and see if that helps track down the issue.

Hi,
I upgraded to .NET 4.5.2 and it seemed to solve the problem. Thanks.


Use $inject to manually identify your dependencies for Angular components

According to this documentation, for injection, you should use $inject instead passing array of dependencies and controller method.
ng-classify created this script instead of this.

Hi @dlinx,
John Papa's style guide, which is terrific, refers to best practices during development.
ng-classify is the tool you've chosen (thanks) to use during development.
Using ng-classify you get minification-safe code and avoid long dependency lists, since you only need to including them once in the constructor.
The following, taken from John Papa's style guide, are all achieved with ng-classify.

Why?: This technique mirrors the technique used by ng-annotate, which I recommend for automating the creation of minification safe dependencies. If ng-annotate detects injection has already been made, it will not duplicate it.

Why?: This safeguards your dependencies from being vulnerable to minification issues when parameters may be mangled. For example, common and dataservice may become a or b and not be found by Angular.

Why?: Avoid creating in-line dependencies as long lists can be difficult to read in the array. Also it can be confusing that the array is a series of strings while the last item is the component's function.

Hope this clarifies things.

Feel free to re-open if needed.


Add generateOriginalAndDiff method and test class

The generateOriginalAndDiff method is added to compare the original file with the reference file, get the diff, and insert the diff into the corresponding location of the original file.
You can see all the differences and unmodified places from the original file.
In addition, it is very easy and useful for making side-by-side comparison display applications,
For example, if you use something like diff2html（ https://github.com/rtfpessoa/diff2html#usage )
Such a tool displays your differences on the html page. You only need to insert the return value of the method into your js code, and you can get a beautiful html comparison page.

Now, I have modified the related question you mentioned above

Correct the building problems.

Build issues have been fixed.

Thx for your contribution.


minor fix

thanks @zq2209 i'll merge it after you updated all the relevant instances like this in the gene_annotation codes.

@gaow I have made a minor fix on genotype_phenotype_sample_overlap.


Cover URL preview case with property name with no value

Cover case with property name with no value, eg:
<meta data-preact-helmet property="og:title" content="Some fancy title">
Result:

#1084

Thanks david! I just modified a bit to add the value-less attribute to the list as well here: https://github.com/vitorpamplona/amethyst/commit/28da06ef010782a402c52f51b1ffd1603e64a821

That's a good idea. Might be something useful in the value less property 👍


Enable the Manager to set pricing on different categories of repairs

How to Demo:
Pricing should change after the manager sets a new price
Exceptions should be thrown if a manager accidentally submits an invalid price

1 hr 20min


-bash: expo: command not found

Dear Evan,
After cloned and installed npm when I run the command expo start --web in Terminal, I am getting error like expo command not found.
In Package.json the scripts are like this,
"scripts": {
"start": "expo start",
"android": "expo start --android",
"ios": "expo start --ios",
"web": "expo start --web",
"build": "expo build:web",
"eject": "expo eject",
"test": "jest --watchAll",
"update-webpack-config": "cp -r ../../expo-cli/packages/webpack-config/ node_modules/\@expo/webpack-config/",
"deploy": "netlify deploy --prod --dir web-build"
},
What I'm doing wrong here? Pls suggest. Thank you in Advance!

Sorry it was a Permission issue.
Issue resolved by adding,
mkdir ~/.npm-global
npm config set prefix '~/.npm-global'
export PATH=~/.npm-global/bin:$PATH
Thank you :)


Add service for Istio CA health check probe.

With the service "istio-ca", Istio CA health check probe is able to call the Istio CA gRPC service though the service name "istio-ca".

/lgtm
please fix the ingress issue as it will eventually hide a bug
(test simple ingress failure #3757)

@ldemailly we still need https://github.com/istio/istio/blob/master/install/kubernetes/mesh-expansion.yaml#L65 for the mesh expansion. this PR adds a CA internal service (not external service), for health check probe to be able to find the service.


Referencing the typeof mixin class or inside a mixin will compile to any in .d.ts files

🔎 Search Terms
mixin .d.ts
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about Classes

⏯ Playground Link
https://www.typescriptlang.org/play?ts=5.4.0-dev.20240107&ssl=15&ssc=26&pln=12&pc=1#code/MYGwhgzhAEBCkFMDC4rQN7QL4CgcBcBPABwTkRUgiXwHsAnaAXmgDsEB3aACgDp+w9AOYQAXNDCtCAbQC6ASmYA+chGSoIeBAA9iDfNABmAV1bB8AS1qtoAWQvaLrADwAVaDvwJWAExjw1SigaBiVuAIRxV0V0HGh46HoEfGN6G1AqOwcnD20vX39EDDiE0ohaAFsEADFWbnlxe0cbWNK2hKSUtOh8AAsLTXb43FLcXBxgawgDABEEegsANwQfZizm8IoNeQmpgx95pZW19i45heWfepwDi5Xecqra+ofKmrr5IA
💻 Code
// main.ts
class BaseClass { }

type BaseClassCtor = new (...args: any[]) => BaseClass

export function Mixin<T extends BaseClassCtor>(Base: T) {
    return class Mixin extends Base {
        someFn(): Mixin {
            return this
        }
    }
}

the above code will compile to
// main.d.ts
declare class BaseClass {
}
type BaseClassCtor = new (...args: any[]) => BaseClass;
export declare function Mixin<T extends BaseClassCtor>(Base: T): {
    new (...args: any[]): {
        someFn(): any;
    };
} & T;
export {};

🙁 Actual behavior
in the ts codebase it works as expected
const Derived = Mixin(BaseClass)
const derived = new Derived()

// someFn() returns Derived
derived.someFn().someFn()

but in a js codebase using the .d.ts types it doesn't
const Derived = Mixin(BaseClass)
const derived = new Derived()

// someFn() returns any
derived.someFn().someFn()

🙂 Expected behavior
using the type declaration to work correctly
Additional information about the issue
No response

Sounds like a duplicate of #51206. Used search terms: mixin any in:title


milvus无法指定向量库的database，上传切分好的向量

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[X] Chat Knowledge
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
GPU
A6000 48G
Models information
baichuan 13b
embedding：m3e base
What happened
正常连接milvus是可以的，但无法指定特定的向量库的database，尝试过  pilot/vector_store/milvus_store.py init_schema_and_load连接时添加dbname，但还是无法添加到
What you expected to happen
希望改善
How to reproduce

Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Search before asking

[x]  I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information

=3.11

DB-GPT version
main
Related scenes

[ ]  Chat Data
[ ]  Chat Excel
[ ]  Chat DB
[x]  Chat Knowledge
[ ]  Dashboard
[ ]  Plugins

Installation Information

[x]  Installation From Source
[ ]  Docker Installation
[ ]  Docker Compose Installation
[ ]  Cluster Installation
[ ]  AutoDL Image
[ ]  Other

Device information
GPU A6000 48G
Models information
baichuan 13b embedding：m3e base
What happened
正常连接milvus是可以的，但无法指定特定的向量库的database，尝试过 pilot/vector_store/milvus_store.py init_schema_and_load连接时添加dbname，但还是无法添加到
What you expected to happen
希望改善
How to reproduce
Additional context
No response
Are you willing to submit PR?

[x]  Yes I am willing to submit a PR!

默认是和向量空间名绑定的，也就是vector_name绑定的


Pseudo-huber loss metric added

Fixes #5479 .
@trivialfis

You can use XGBoostParameter for defining the delta, see objective/multiclass_obj.cu::SoftmaxMultiClassParam for an example.

Added the registration in regression_obj.cu similar to squared errror. Not sure if that's it?

The parameter is way over my head of understanding C/ CUDA code - I can see how you register a parameter in regression_obj.cu, but wouldn't there have to be a class similar to PoissonRegression in the same file?

@LionOrCatThatIsTheQuestion Don't worry.  I'm happy to merge the PR provided you can add tests in /tests/cpp/objective/test_regression_obj.cc and similar place for metric.  Also I need to go through the derivation later today.
But if you want to give it another go for doing the parameter.  It's a factory method, which is also available in Python and is heavily used by libraries like pytorch and dask.  I found a very long explanation for Python: https://realpython.com/factory-method-python/  But you can simply search for Python factor and there are lots of introduction.  It works similarly in C++, just the macros hiding many details so the code seems a  little bit ridiculous.

The tests are added, I computed the reference values with Wolfram Alpha - really hope there aren't any more typos, I'll check back tomorrow :).

To answer to the parameter part, I took a short look at the factory methods and if I have time left at the end of my current project, I'll come back to your explanantion and try again with a new PR :).

@LionOrCatThatIsTheQuestion  Just checked the derivation.  Good work!  Now one last piece, please add documentation on doc/parameter.rst objective section and eval_metric section.

Documentation done as well, thank you very much for guiding me through the code :).

@LionOrCatThatIsTheQuestion Thanks!

@LionOrCatThatIsTheQuestion  Excellent feature implemented to xgboost. Is it possible to select \delta paramater or it´s 1 ?   =)

It's just 1 for now, but should be easily extended.  I'm not sure how urgent it's to get different slope for this function.

It's just 1 for now, but should be easily extended. I'm not sure how urgent it's to get different slope for this function.

Hi, @trivialfis thanks for support this feature! Based on sklearn's suggestion, it seems that delta is better at 1.35 to achieve 95% statistical efficiency.
https://scikit-learn.org/stable/modules/linear_model.html#huber-regression

@LionOrCatThatIsTheQuestion Would you like me to carry it out?  :-)

@trivialfis @LionOrCatThatIsTheQuestion After reviewing the code, it seems to me that implemented delta of 1 actually depends on variance of dependent variable y, which is similar to SGDRegressor with huber loss. By comparision, HuberRegressor in sklearn says that its delta is scaling invariant. Reference: https://scikit-learn.org/stable/modules/linear_model.html#notes
Therefore, users should standardize y to ensure variance as 1. Is my understanding correct? Thanks. :)

@LionOrCatThatIsTheQuestion @trivialfis Hi, I review the XGboost GPU supported metrics documents and it seems that this huber loss is current not compatible with GPU training. Am I correct? https://xgboost.readthedocs.io/en/latest/gpu/index.html#objective-functions

@QuantHao The list should be updated, since the pseudo-Huber loss is supported by the GPU algorithm.

I filed a pull request to update the list: https://github.com/dmlc/xgboost/pull/5909

@QuantHao The list should be updated, since the pseudo-Huber loss is supported by the GPU algorithm.

Thanks so much for your confirmation.

@hcho3 Also, it would be great to update "Supported parameters" section for GPU support documentation. It seems that some parameters are missing such as colsample_bynode, alpha, lambda, etc. Thanks.
https://xgboost.readthedocs.io/en/latest/gpu/index.html#supported-parameters


TopologyAwareScheduling enabled by default for Kueue v0.9.1

This PR enables TopologyAwareScheduling by default for Kueue v0.9.1 installation.
It is already enabled for its v0.9.0.

Can you please expand the PR description with:

What are the consequences of enabling TAS by default?
Why do we need to this enabled by default?
How have we verified this change is safe?


Read the rate from cache if available

Before it was not read due to failing to create a DataTime from 'time_next_update_utc'. Cache is now written to disk only if parsing is successful. Refactor code by splitting into logical methods - reading, writing the rates from cache, getting rates from the service and parsing json are now in their own methods.

There's really no reason for this as all these methods would only be called once by the main GetConversionRates method
Following that logic we shouldn't have GetConversionRates since it is only called once by ConvertAsync ;). But seriously, I'm only adding a few lines (excluding doc comments and better logging) and now we have methods without side effects and we can easily change them separately.

Also, like I wrote in the commit message, cache is now written to disk only if parsing conversion rates is successful. Before it was cached even if it is invalid - for example due to some temporary issue with the service. The app will then be unable to get the rates after the service is fixed until the next day, since it will be finding a cache file. This is not hypothetical problem - I actually wrote an app that shows conversion rates with gir.core and I had several different temporary issues with the service (https://github.com/fawazahmed0/currency-api#readme), so when I saw in TWIG that you have the same functionality I decided to check what you use. Here is my app:

Another issue is that if writing the cache file failed you didn't show conversion results, and I think it is more important to show rates, than to cache them.
I'm also checking that cache file exists only once :)

Following that logic we shouldn't have GetConversionRates since it is only called once by ConvertAsync ;). But seriously, I'm only adding a few lines (excluding doc comments and better logging) and now we have methods without side effects and we can easily change them separately.

GetConversionRates() is called not just by ConvertAsync but in many places throughout controllers and views within the application.
Method calls take up resources. We try to keep it at a minimum.
And thank you for implementing all those check and writing only when needed..like i said just keep it all in that one method and we will merge :)

Method calls take up resources. We try to keep it at a minimum.

It is not called in a loop so 2 additional method calls are nothing. Compared with better code readability it is worth it.
Anyway I created another merge request - #702 - with everything inside one method. You can compare the result and pick what you like.


Bitrate menu doesn't work anymore.

On Chrome Control+Shift+alt+S doesn't work anymore to show bitrate menu.

The latest release fixes this behavior (the key combo was changed by Netflix to CTRL-ALT-SHIFT-B)


Drop the 'v' from tag names

Using v tags seems to be problematic for Swift Playgrounds/SPM... I need to check if its a problem for regular SPM, and if it is then we should drop them. If not, i should file a feedback with Apple.

~In the meantime, i'm going to also create a 0.0.2 tag.~

Maybe this is a bug in Swift Playgrounds iPad app.. It works fine in Xcode. I raised FB9812401 for it.


Fix Slide width calculation error

Fix #221 and could also fix (or try to) #94

The changes have been implemented in commit: e10e4fb
Thanks for the PR though.


fix(setting): link color for different themes

Remove custom color for link in setting component and use system color

Thank you!


Error creating appsync resources with  ConcurrentModificationException: Schema is currently being altered

Hi,
I am trying to create resources using the module and I get the following error sometimes ,
Error: error creating Appsync API Key: ConcurrentModificationException: Schema is currently being altered, please wait until that is complete.

This error looks intermittent and the issue doesnt appear when we trigger the same command after 30 - 60 seconds.
I did try adding depends_on constraint but that also did not solve the issue.
Has any one faced this issue before ?
.I tried adding support for functions and there were intermittent issues for functions too with similar error
Error: Error creating AppSync Function: ConcurrentModificationException: Schema is currently being altered, please wait until that is complete.

There is a mention of this error in the document https://docs.aws.amazon.com/appsync/latest/APIReference/API_UpdateGraphqlApi.html but no clear instruction on how to get around it ?
Terraform code to reproduce this error is similar to https://github.com/terraform-aws-modules/terraform-aws-appsync/tree/master/examples/complete
Add more datasources, resolvers in the file.
Has anyone faced this issue before ?

Hi @cshivashankar !
Thanks for opening this issue. I don't think I personally saw this error with my projects or when I use the example code so I believe the workaround is to look into the simplification of data sources and resolvers configurations.
Could you share your complete configuration code for this module with replaced sensitive values? I want to see if I can do anything in the code to make this work.
As a more permanent fix, I would recommend opening an issue in Terraform AWS Provider. There are several similar issues in other resources, for eg, https://github.com/hashicorp/terraform-provider-aws/issues/14541 .

I have just rerun the same example 5 times and just once I got the error ConcurrentModificationException. I am pretty sure it is a bug in Terraform AWS provider.

Yes it is intermittent error , I think a bug needs to be created with terraform aws provider

@antonbabenko I have raised an issue with provider , https://github.com/hashicorp/terraform-provider-aws/issues/16211
I think this issue should be solvable by reducing the parallelism in execution , but its more of  a workaround rather than actual fix .
I doubt if there there is a retry option in terraform which is also a good workaround for this problem.
Regards,
Chetan

I agree, let's see what kind of feedback will be on that one. Thanks for opening it.


exception on conversion

./gwhisper <IP_ADDRESS> examples.NestedTypeRpcs echoNestedMaps simple_map_string=::key=8 value=ad::
terminate called after throwing an instance of 'std::invalid_argument'
what():  stoul
Aborted (core dumped)

If user input "key=8 value=0xad::", the as which could be intercepted as a hex number(0XAD), without "0x" the stoul() will throw the exception.
Hence here only the value starts with "0x|0X", likes "value=0xFF::" is allowed, or "value=FF::" is allowed as well? Is default number base decimal?

only hex numbers starting with 0x or 0X are allowed
others should be interpreted as decimal.
core dump here is a bug.
We should either return a meaningful error message, or implement better regex in the grammar parser to only allow valid values to be parsed


"in the last months" sounds grammatically weird; should include number of months, or reword

As noted in #1566, there's a bot nag that went a bit haywire this morning and started over-posting "The bug assignee didn't login in Bugzilla in the last months [...]"
e.g. in https://bugzilla.mozilla.org/show_bug.cgi?id=1769830#c9
I'm filing this issue just about the wording there -- "in the last months" sounds grammatically weird. We should include a count (e.g. "in the last 2 months"), or if we're intentionally being vague, we should reword to a different phrase like "...in recent months".

(Note: it's possible we're already intending to include a count, and we were just ending up with null count being inserted into the string here, so In the last <nothing> months was printed as a result of that.  If so, then there's nothing special to be done here and feel free to just close this.)


Dependency conflict when running setup.py develop

When installing a development checkout per the documentation, there is a dependency conflict:
$ $VENV/bin/python setup.py develop
running develop
<snip>
Installed /tmp/google-cloud-python/lib/python2.7/site-packages/oauth2client-2.0.0-py2.7.egg
error: oauth2client 2.0.0 is installed but oauth2client<2.0.0,>=1.4.11 is required by set(['gapic-google-logging-v2', 'gapic-google-pubsub-v1'])

Thanks for reporting @sebbacon!
I just tested this and it happened to me as well, except it pulled in oauth2client 3.0.0.
error: oauth2client 3.0.0 is installed but oauth2client<2.0.0,>=1.4.11 is required by
set(['gapic-google-logging-v2', 'gapic-google-pubsub-v1'])

@dhermes @bjwatson, we have 'oauth2client >= 2.0.1' as a requirement.  Can gapic bump it's oauth2client versions?

Definitely need to upgrade @bjwatson, as we're very quickly approaching 4.0 in oauth2client (finally getting rid of some really bad old stuff, e.g. httplib2). I'm happy to help upgrade.
See the discussion in #2316, essentially using development / editable mode won't really work any more with namespace packages. We should just remove that from CONTRIBUTING. @tseaver WDYT?

I ran into this also, and removed the upper-bound from our oauth2client dependencies. We'll review later and bound it to >=3.0.0, <4.0.0dev.
This issue should be fixed for now.

Thanks @bjwatson!
I'm seeing a different but related issue now.
error: google-gax 0.14.1 is installed but google-gax<0.14.0,>=0.13.0
is required by set(['gapic-google-logging-v2', 'gapic-google-pubsub-v1'])

@bjwatson, @dhermes just pointed out to me that pip install -e . doesn't throw this google-gax error. I'm not sure what the difference in handling between setup.py and pip are though.

I was actually trying to point out that using -e should no longer be done. But that also seems true.

@dhermes sorry yeah I was getting this for python setup.py install as well.

@daspecster @dhermes This is due to inconsistent bounds in https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/setup.py#L58
gapic-google-* 0.9.x packages are intended to work with google-gax 0.13.x. For example, see setup.py in gapic-google-pubsub-v1 0.9.3.

FYI, we are finalizing the 0.10.x packages which will depend on google-gax 0.14.x: https://github.com/googleapis/api-client-staging/pull/24.

Thanks @bjwatson. It'll be important for us to keep from cutting a release with the breakage. We will probably have 15 releases next week, of which 3 (google-cloud-pubsub, google-cloud-logging and google-cloud) will be impacted by this inconsistency.

@dhermes What can we do to help with this? google-gax and gapic-google-* are tightly coupled, so the version bounds make sense (to avoid surprises like we have had before with things spontaneously breaking), but I understand that you need to know what the pairing is. Is looking at the setup.py in any gapic-google-* of a particular 0.x version good enough to see what its matching version of google-gax is?

@bjwatson It's been resolved as of the iam push to PyPI this morning.

Ok, great!


Defines by option

Needs options to define text macros, like:
listDefine = ["ASIC", "CELLDEF"]

v.0.1.3
New option preDefined is introduced.
Usage:
"svlog.server.preDefined": [
["ASIC", null],
["DEADBEEF", 10]
],


DDAU hooks

I'm setting onSelect to (action 'dateSelected') but it's never firing that action. When you said: you can extend the pikaday-input component and overwrite certain callbacks, are you saying we should change your addon code directly?
Sorry and thanks!
Edit: Oh i got it doing onPikadaySelect instead. I think the docs need to be updated? I'll make a small pull request

@misteroh I had the same issue I'll try onPikadaySelect=(action 'dateSelected')

@pixelhandler, @misteroh This is what I did to get DDAU working. Notice the hooks are not in the actions hash:
javascript
import Ember from 'ember';
import PikadayInputComponent from 'ember-pikaday/components/pikaday-input';
const { get, run } = Ember;
export default PikadayInputComponent.extend({
onPikadayOpen() {
this.attrs.onOpen();
},
onPikadayClose() {
this.attrs.onClose();
},
onPikadayRedraw() {
const pikaday = get(this, 'pikaday', pikaday);
run.later(() => {
  return pikaday.adjustPosition();
});

}
});

{{modify-ember-pickaday
firstDay=0
value=startDate
placeholder='Date'
class='form-control'
format="MM/DD/YYYY"
yearRange="currentYear,2010"
onOpen=(action 'myFunc1')
onClose=(action 'myFunc2')
}}

I think the new beta version 2.0.0-beta.1 addresses this issue. Please reopen and elaborate if that's not the case, thanks!


ko.po

I would like to proceed with the Korean language.
Do you mind if I proceed with the work?

@jhwon0812 sure, please go ahead, thanks for offering help!
Quick reminder, the way localization is done is not directly through editing file on git, but instead please do that in
https://hosted.weblate.org/projects/xenko/
More details: https://github.com/xenko3d/xenko/blob/master/docs/localization.md

@xen2
I've translated it into a separate file.
To who put links to create files po ko pull request for.
Please check the full request.
Also, if you have any problems with him, I would appreciate it if you could tell me.


Add tests to language part 2

Add tests for parsing, international, and script help.

Hi @bingbing8, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Yanbing Wang). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

:clock1:

close to re-trigger the build and tests

Hi @bingbing8, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Yanbing Wang). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Assigning the PR to @JamesWTruher to continue the review. Once I have his sign-off, I will merge it.

:shipit:


[Dialog] Id attributes for dialog actions

It would simplify Selenium testing if we were able to add id attributes to the actions. This would allow Selenium to easily find the buttons using css selectors.
For example:
//Standard Actions
let standardActions = [
  { id: 'cancelButton', text: 'Cancel' },
  { id: 'submitButton', text: 'Submit', onTouchTap: this._onDialogSubmit, ref: 'submit' }
];

<Dialog actions={standardActions} />

Using actions as a JSON object is now deprecated. You can simply pass an array of FlatButton components to the actions property.


fix: Popups and prevent panic on unknown event type

Description

Popups have been fixed in release mode

They were broken in release because of the base-href setting being relative and not absolute

Remove panic on an unknown event type and log instead

Previously if there was a panic it would stop the komorebi polling thread and completely stop responding to events.

Testing
Clicked on the clock to open the calendar to make sure it works and I'm now able to switch workspaces and see when komorebi is offline (previously it never received these events because of dead polling thread)

Looks good! Thank you for the contribution


more fleet framework migrations

add a fleen gen transform to de-pointer *[]sometype and *map[string]sometype fields
migrate resource fleet_agent_policy
migrate resource fleet_integration
migrate data_source fleet_integration
migrate resource fleet_server_host

I was having problems earlier testing those, I think it has to do with on
read, not knowing if they should be false or null. I'll try that bit again
with a computerd/default false, and on read set false if not set.
Regarding the x-pointers flag, I can revert that. Will just need to change
up the helpers to do pointers instead. As you said, it doesn't change much.
As you've brought up manipulation of the published spec, whats your take
then on all of the nested anonymous structs? Fleet isn't too bad, but
Kibana's is a disaster for generation at the moment.
On Thu, Sep 26, 2024, 11:26 PM Toby Brain @.***> wrote:

@.**** commented on this pull request.
In internal/fleet/agent_policy/resource_test.go
https://github.com/elastic/terraform-provider-elasticstack/pull/785#discussion_r1777962451
:

@@ -57,7 +57,7 @@ func TestAccResourceAgentPolicy(t *testing.T) {
ResourceName:            "elasticstack_fleet_agent_policy.test_policy",
ImportState:             true,
ImportStateVerify:       true,

		ImportStateVerifyIgnore: []string{"skip_destroy"},

		ImportStateVerifyIgnore: []string{"monitor_metrics", "monitor_logs", "skip_destroy"},

What's happening here? This PR populates these fields on Read, why are we
not setting them to computed?
—
Reply to this email directly, view it on GitHub
https://github.com/elastic/terraform-provider-elasticstack/pull/785#pullrequestreview-2332679631,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMTBCAIQCRDCIZZRAEOD3ZYTGABAVCNFSM6AAAAABOR4KYFKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGMZSGY3TSNRTGE
.
You are receiving this because you authored the thread.Message ID:
@.***
.com>

ok.

x-go-pointer flag is dropped.
fixed agent policy monitoring test ignores finally
shared integration.create with update
added FromSDK tests to all 3 migrated resources

whats your take then on all of the nested anonymous structs?

 Ultimately we need to fix these in the source though so anyone using the specs can benefit.

And here I couldn't sleep this morning. Looks like I'll be rebasing the
other pr now.
On Sun, Sep 29, 2024, 6:36 AM Toby Brain @.***> wrote:

Merged #785
https://github.com/elastic/terraform-provider-elasticstack/pull/785
into main.
—
Reply to this email directly, view it on GitHub
https://github.com/elastic/terraform-provider-elasticstack/pull/785#event-14444932539,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMTBBUUAZ3CAOFV6TBUXDZY7J2VAVCNFSM6AAAAABOR4KYFKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGQ2DIOJTGI2TGOI
.
You are receiving this because you authored the thread.Message ID:
<elastic/terraform-provider-elasticstack/pull/785/issue_event/14444932539@
github.com>


Memory Preallocation: Add support for pre-allocation

Add support for pre-allocating all the RAM for the VM. This will reserve the RAM at VM startup.
Also add support for realtime and mlock.
Enabling prealloc, realtime and mlock gives the lowest latecy.
Disabling prealloc, enabling realtime and disabling mlock gives highest density as the memory will be swapped.
This PR depends on vendoring the updated qemu package. https://github.com/01org/ciao/pull/1428 and
https://github.com/01org/ciao/pull/1430
Signed-off-by: Manohar Castelino<EMAIL_ADDRESS>

Great - I'm curious what some of the performance impacts for this will be (ie, start time for qemu pending the size of the backed memory).  This looks good to me, though your commit message could use a space...

@egernst this is disabled by default. This is just an useful know to have available specially when trying to root cause performance issues. This helps eliminate some of the variability.

@mcastelino -- yeah, understood.  Still am curious though!  Only I see is need to s/pre-allocatingall/pre-allocating all/

LGTM

Coverage decreased (-0.04%) to 65.949% when pulling 0a7e444775e6dbb135515d439df8523e6a14e66a on mcastelino:topic/prealloc into 3ac2a1dffae04d80929e3eeaef7be039c524db1a on containers:master.


x/net/http2/hpack: lookup maps for the dynamic table leak despite evictions

What version of Go are you using (go version)?
go version go1.8 darwin/amd64

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/chrisroche/go"
GORACE=""
GOROOT="/Users/chrisroche/.gimme/versions/go1.8.darwin.amd64"
GOTOOLDIR="/Users/chrisroche/.gimme/versions/go1.8.darwin.amd64/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/7s/kv_f_g112gq5gl7zt_lyt2zr0000gp/T/go-build079652742=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?
Adding the following test to the current master of x/net/http2/hpack/tables_test.go:
func TestHeaderFieldTable_LookupMapEviction(t *testing.T) {
	table := &headerFieldTable{}
	table.init()
	table.addEntry(pair("key1", "value1-1"))
	table.addEntry(pair("key2", "value2-1"))
	table.addEntry(pair("key1", "value1-2"))
	table.addEntry(pair("key3", "value3-1"))
	table.addEntry(pair("key4", "value4-1"))
	table.addEntry(pair("key2", "value2-2"))

	// evict all pairs
	table.evictOldest(table.len())

	if l := table.len(); l > 0 {
		t.Errorf("table.len() = %d, want 0", l)
	}

	if l := len(table.byName); l > 0 {
		t.Errorf("len(table.byName) = %d, want 0", l)
	}

	if l := len(table.byNameValue); l > 0 {
		t.Errorf("len(table.byNameValue) = %d, want 0", l)
	}
}

And running go test -v -run TestHeaderFieldTable_LookupMapEviction from that directory
What did you expect to see?
PASS

What did you see instead?
=== RUN   TestHeaderFieldTable_LookupMapEviction
--- FAIL: TestHeaderFieldTable_LookupMapEviction (0.00s)
	tables_test.go:110: len(table.byName) = 4, want 0
	tables_test.go:114: len(table.byNameValue) = 6, want 0
FAIL
exit status 1
FAIL	golang.org/x/net/http2/hpack	0.006s

More context
When the load to a gRPC service was recently ramped up, we were alerted to a memory leak which took approximately 24hrs to consume all the RAM and die.  Unfortunately, it doesn't appear that the map entries are evicted, but instead set to zero:
if t.byName[f.Name] == id {
	t.byName[f.Name] = 0
}
if p := (pairNameValue{f.Name, f.Value}); t.byNameValue[p] == id {
	t.byNameValue[p] = 0
}

This eventually (but somewhat slowly) grows huge for variable headers like request IDs, timings, etc. Rolling back the package to the prior SHA immediately rectified the problem. I'm working on a patch to replace the above to use delete() instead. Curious if there is a reason to not?

@rodaine, thanks for the debugging!
If you want to send a fix, please do. Otherwise @tombergan or I can handle it too. Let us know.

No sweat! Pushing the patch momentarily :)

I trust you've discovered https://golang.org/doc/contribute.html ?
(Sorry in advance for the wall of words. It tries to be comprehensive but it might be intimidating. There aren't many steps in the end.)

Yessir!

https://go-review.googlesource.com/38781


[Internal] Versioning: Adds guidance for versioning SDK releases

The following PR adds a versioning guideline that clarifies which are the released versions.
The current SDK release method is to always release version X and X-preview but this causes a wide range of issues with Nuget and chained dependencies (package A depends on SDK version X, if the user wants to use X-preview they can't because Nuget would attempt to use X because X is semantically newer than X-preview).
The goal is to improve the release methodology to address the versioning issues arising in tooling used by our customers.
Preview of the document: https://github.com/Azure/azure-cosmos-dotnet-v3/blob/users/ealsur/versioning/docs/versioning.md
Closes #4169

azp autom-merge

nit: it is applicable after 3.37.0 release. should we mention it somewhere?

nit: it is applicable after 3.37.0 release. should we mention it somewhere?

We can include it in the next changelog


Eq 650 Implement Previous + Next from Schema

What is the context of this PR?
Moved to using the schema for navigation, rather than methods on the questionnaire manager. Added a new class Navigator, which deals with finding block and location paths through a survey from a schema. Refactored all the previous routing behaviour to use new navigator and removed redundant dependencies.
It's been necessary to modify the way a schema is loaded, so it returns a tuple of the parsed schema and original json, so the navigator is able to take the json as an argument.
How to review
Review the approach, confirm that tests pass and test coverage is appropriate.

Codecov's patch is complaining as I've refactored a number of method names in preexisting files. I've managed to bump them quite a lot by mocking out boto, but not enough to get our current target of 90.3%.
I think the patch blocking a PR isn't helpful, given it's difficult to know what the coverage of a patch will be before letting codecov run, so we're forever making updates to a PR to get it to increase.

This is the coverage report: https://codecov.io/gh/ONSdigital/eq-survey-runner/compare/b9b82d14406a662b6c8bede15bc3ee72e18347c4...9f5190fde5b3680b688116dffcf12997fe2a8240
We can either disable blocking the PR or change the codecov setting (i.e. just fix it at 85% rather than always the same or higher than the parent branch...

If I click on the first page I prefer Star Trek, it gives me a 500
block_id_index = next(index for (index, b) in enumerate(blocks) if b["id"] == block_id)
StopIteration

@LJBabbage - The recursive visit_block method was only working with block ids and had no idea interstitial pages could be jumped to. Have added a test to catch it and handled the case.


Support rustls-native-certs as an alternative to webpki-roots

Some people may prefer to use the systems root certificate store with rustls rather than build Mozilla's one into their application (see rustls-native-certs/README.md for possible reasons). This is probably something to do after you've figured out how to reduce the amount of work CI is already doing 😉

The way reqwest handles this seems relevant: https://github.com/seanmonstar/reqwest/pull/1058

The way reqwest handles this seems relevant: https://github.com/seanmonstar/reqwest/pull/1058


Is there a way to ture off autocollect feature?

Can I turn off autocollect feature completely?
Can I turn off autocollect for User information only?

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

ID: d56ecb14-9e66-5313-a381-d2824ec3d1e2
Version Independent ID: ff056ac8-ae76-be10-6895-757333a28e05
Content: Azure Application Insights for JavaScript web apps
Content Source: articles/azure-monitor/app/javascript.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@maple10001 Thank you for the feedback!  We are investigating this and will get back to you shortly.

@maple10001 Using "disableTelemetry" set to true would mean that telemetry is not collected or sent which effectively turns off autocollect.  Additionally, you can configure other fields to limit or disable user information.  Look through Configuration for more information on this:
https://docs.microsoft.com/en-us/azure/azure-monitor/app/javascript#configuration
Additionally, you have the option to use Application Insights Web Basic which performs no autocollection.  This is a lightweight experience that comes with the bare minimum number of features and functionalities and relies on you to build it up as you see fit.
https://docs.microsoft.com/en-us/azure/azure-monitor/app/javascript#application-insights-web-basic

@maple10001 We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Prefer Never to NoReturn in auto-typing

Closes https://github.com/astral-sh/ruff/issues/9212.

@charliermarsh some context please :)

https://github.com/python/typing/issues/1458#issuecomment-1699831429


ValueError: No way to determine width or height from video. Need -s in inputdict. Consult documentation on I/O.

Traceback (most recent call last):
File "human_activity_recognition.py", line 204, in
X_train = reader.read_videos(train_files)
File "/home/user/Documents/ActivityRecognition/Methods/3rd/Human-Activity-Recognition-master/utils.py", line 70, in read_videos
self._read_video(path) for path in tqdm(paths)
File "/home/user/Documents/ActivityRecognition/Methods/3rd/Human-Activity-Recognition-master/utils.py", line 70, in
self._read_video(path) for path in tqdm(paths)
File "/home/user/Documents/ActivityRecognition/Methods/3rd/Human-Activity-Recognition-master/utils.py", line 126, in _read_video
cap = FFmpegReader(filename=path)
File "/home/user/anaconda3/lib/python3.7/site-packages/skvideo/io/ffmpeg.py", line 130, in init
raise ValueError("No way to determine width or height from video. Need -s in inputdict. Consult documentation on I/O.")
ValueError: No way to determine width or height from video. Need -s in inputdict. Consult documentation on I/O.
50%|████████████████████▋                    | 151/300 [02:39<02:37,  1.06s/it]

Chances are, you did not delete one of the corrupted videos present in the dataset. More info in instruction no. 3 in the README.


Re-enable executor_deinit1.swift test case

rdar://77658743

@swift-ci Please test

It passes locally for me.


Can we have examples of testing other triggers and bindings beyond HTTP

Been trying to unit test Storage Queue bindings and it's a bit challenging. Luckily found this stackoverflow post
Specifically,
def main(req: func.HttpRequest,  msg: func.Out[str]) -> func.HttpResponse:
Requires the argument msg which cannot simply be an instance of func.Out
Instead it needs to be an instance of class QueueMessage, with the attribute 'set' assigned:
msg = func.QueueMessage(body=b'')
msg.__setattr__('set', set)

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

ID: d955d169-8fb8-c511-8911-3e49f95c5ce6
Version Independent ID: 8a3d927c-51ce-d7d4-e5d1-acd928826db2
Content: Python developer reference for Azure Functions
Content Source: articles/azure-functions/functions-reference-python.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hey, does these help:
Queue Trigger:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-storage-queue-trigger?tabs=python
Blob Trigger:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-storage-blob-input?tabs=python
Timer Trigger:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-timer?tabs=python
In the menu on the left there are a number of other options.

Hey Samuel,
Thinking about testing here, the examples provided show how to setup the
function but it took me a long while to work out how to setup an actual
test
On Sun, Nov 29, 2020, 11:24 PM Samuel Stenton<EMAIL_ADDRESS>wrote:

Hey, do these help:
Queue Trigger:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-storage-queue-trigger?tabs=python
Blob Trigger:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-storage-blob-input?tabs=python
Timer Trigger:
https://docs.microsoft.com/en-us/azure/azure-functions/functions-bindings-timer?tabs=python
In the menu on the left there are a number of other options.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/66844#issuecomment-735441850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK6BTT7DAHPJNR7U5S6XRYDSSKNXTANCNFSM4UGVTGBA
.

@jordaniza Thank you for your feedback! We will review and update as appropriate.

Hello @jordaniza, Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

This ticket is from 2020, would it be possible to put this higer on the priority list. Writing proper tests for a queue trigger is not easy to figure out. @ggailey777


IPython console breaks after python file run.

Description
What steps will reproduce the problem?

Run Python file (F5)
After (1), any action in the IPython console will trigger this

What is the expected output? What do you see instead?
Output is as expected, except for the error screen (which has a button directing me to this error fom)
Please provide any additional information below
This behaviour started after installing 3.2.3.
File "/home/marc/tf/lib/python3.5/site-packages/qtconsole/base_frontend_mixin.py", line 163, in _dispatch
handler(msg)
File "/home/marc/tf/lib/python3.5/site-packages/spyder/widgets/ipythonconsole/namespacebrowser.py", line 220, in _handle_execute_reply
self.handle_exec_method(msg)
File "/home/marc/tf/lib/python3.5/site-packages/spyder/widgets/ipythonconsole/shell.py", line 351, in handle_exec_method
view = ast.literal_eval(literal)
File "/usr/lib/python3.5/ast.py", line 84, in literal_eval
return _convert(node_or_string)
File "/usr/lib/python3.5/ast.py", line 62, in _convert
in zip(node.keys, node.values))
File "/usr/lib/python3.5/ast.py", line 61, in
return dict((_convert(k), _convert(v)) for k, v
File "/usr/lib/python3.5/ast.py", line 62, in _convert
in zip(node.keys, node.values))
File "/usr/lib/python3.5/ast.py", line 61, in
return dict((_convert(k), _convert(v)) for k, v
File "/usr/lib/python3.5/ast.py", line 83, in _convert
raise ValueError('malformed node or string: ' + repr(node))
ValueError: malformed node or string: <_ast.Call object at 0x7f9064efa978>
Version and main components

Spyder Version: 3.2.3
Python Version: 3.5.2
Qt Versions: 4.8.7, PyQt4 (API v2) 4.11.4 on Linux

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  None (NOK)
numpy >=1.7      :  1.13.0 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.7 (OK)
jedi >=0.9.0     :  0.11.0 (OK)
psutil >=0.3     :  5.3.1 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  None (NOK)
cython >=0.21    :  None (NOK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.2.0 (OK)
pylint >=0.25    :  1.7.2 (OK)

Please post the Python file that you're running so we can reproduce this on our side.

It seems to have to do with the path library (attached, rename .txt to .py).
A minimal script using path that crashes the console is:
from path import path
if name == "main":
logdir = path('~')
path.txt

Thanks a lot for posting this! We'll take a look at it for 3.2.4

Closing as a duplicate of #5636.


No Quick Fixes Available on Svelte files only

For some reason, when I hover over an ESLint issue on a Svelte file, it'll state that it knows there's an issue, but that there's no quick fixes available.

However, if I go to a TypeScript file, I can easily click on Quick Fix.

Weirdly enough, it actually used to work for me, and then all of sudden it stopped working one day. I even tried to completely erase VS Code and start from scratch, but to no avail.
Here are the devDependencies in the package.json for linters I'm using:
"devDependencies": {
  "@typescript-eslint/eslint-plugin": "^5.50.0",
  "eslint-config-prettier": "^8.5.0",
  "eslint-plugin-prettier": "^4.2.1",
  "eslint-plugin-svelte": "^2.15.0",
  "prettier": "^2.8.3",
  "prettier-plugin-svelte": "^2.8.1",
}

I'm using eslint-plugin-svelte, rather than eslint-plugin-svelte3 (the official Svelte one), since eslint-plugin-prettier recommends to use the non-official one.
Here's the pertinent info on the .eslint.cjs file:
extends: [
  'eslint:recommended',
  'plugin:@typescript-eslint/recommended',
  'plugin:svelte/recommended',
  'plugin:prettier/recommended',
],
plugins: ['@typescript-eslint'],
ignorePatterns: ['*.cjs', '*.config.js'],
parser: '@typescript-eslint/parser',
parserOptions: {
  extraFileExtensions: ['.svelte'],
  project: './tsconfig.json',
},
overrides: [
  {
    files: ['*.svelte'],
    parser: 'svelte-eslint-parser',
    // Parse the script tag in Svelte as TypeScript
    parserOptions: {
      parser: '@typescript-eslint/parser',
      project: './tsconfig.json',
    },
  },
],

Here are my user settings on VS Code:
{
  "eslint.validate": [
    "svelte",
    "typescript"
    "javascript",
  ],
  "svelte.enable-ts-plugin": true,
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "[svelte]": {
    "editor.defaultFormatter": "svelte.svelte-vscode"
  },
  "[typescript]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "css.validate": false,
  "scss.validate": false
}

I'm aware of eslint.probe as the "new" way to do it, but it actually doesn't work for me.

@D-Marc1 does fixing work for you correctly on the command line (e.g. when running eslint in the terminal). If yes, can you please provide me with a GitHub repository that I can clone that demos this. This ensures that we are using the same setup.

I'm having the same problem.   I believe it's a similar issue to #1610
When I disable "Disable Rule Comment" in the vscode-eslint it then works and I see the Quick Fix option -- so that's a good temporary workaround to try unless you absolutely need the "disable on the next line/whole file" options.  I tried the suggested fix in the issue above and changed the commentStyle to line but that didn't work
If I go to the Output console tab > Select ESLint in the dropdown, I see this error:
[Error - 5:36:49 PM] Request textDocument/codeAction failed.
  Message: Request textDocument/codeAction failed with message: Invalid regular expression: //** eslint-disable-line/: Nothing to repeat
  Code: -32603

@jamesinvt thanks for pointing this out.

@dbaeumer, this doesn't work when I run this on command line either, with: npx eslint --fix.

@D-Marc1 if it doesn't work on the command line (terminal) then it will not work in the extension. The exension basically uses the eslint npm package internally.
I will close the issue since there is nothing the extension can do if it doesn't work in ESLint itself. All I can recommend is to open an issue with the ESLint npm  package.

@D-Marc1 when you say "it didn't work" when you ran npx eslint --fix do you mean that it produced an error or just did nothing at all?
I ran this in the root folder and there was no output or change to the files.  I ran npx eslint --fix ./src and that did work.
@dbaeumer Sorry for the back-and-forth.  Let me know if you want me to open a new issue but I do believe the Author and I are having the same issue.  I don't think it's an issue with the eslint cli package.
I set up an example repo here that should let you reproduce the error.  No need to run any build or anything:
https://github.com/jamesinvt/vscode-eslint-svelte-test
To reproduce:
In src/App.svelte hover over the console.log and it will show No Quick Fixes Available and in the ESLint output it will give the error:
[Error - 5:36:49 PM] Request textDocument/codeAction failed.
  Message: Request textDocument/codeAction failed with message: Invalid regular expression: //** eslint-disable-line/: Nothing to repeat
  Code: -32603

Go into .vscode/settings.json and under eslint.codeAction.disableRuleComment change enabled to true and the quick fixes option will work.

@jamesinvt You are correct, I forgot to add ./src. I'm getting the exact same situation as you. I see it working to lint on command-line and I see the exact same message in the output.
@dbaeumer Please reopen this issue. I made a mistake by not typing in the full command for eslint. npx eslint --fix ./src does in fact work and fixes the linter issues. This looks like it probably is an issue with VS Code after all.

The regexp problem is tracked here and already fixed in main: https://github.com/microsoft/vscode-eslint/issues/1610 Will be in the new next release which will ship beginning of next week.


Split task log by size.

This PR is the purpose to resolve #1193 and replace PR #700.
This PR splits task log by the configured size.
So long running task can also show the log by digdag log -f.
If server admin configured the size to small one, user can show the task log in near real time.

@muga Thanks for your pointing out.
I guess UPLOAD_THRESHOLD of BufferedRemoteTaskLogger should be also configurable.
So If I will fix it, I try to refactor and improve both code.


game crash when discussing hackathon with Annika (on August 4)

What bug did you encounter?
[code]
I'm sorry, but an uncaught exception occurred.
While running game code:
File "game/script.rpy", line 817, in script call
call ask_hackathon from _call_ask_hackathon
File "game/scripts/labels/npc_question_and_answer.rpy", line 51, in script
$ topics_to_ask.remove('Hackathon')
File "game/scripts/labels/npc_question_and_answer.rpy", line 51, in
$ topics_to_ask.remove('Hackathon')
KeyError: u'Hackathon'
-- Full Traceback ------------------------------------------------------------
Full traceback:
File "game/script.rpy", line 817, in script call
call ask_hackathon from _call_ask_hackathon
File "game/scripts/labels/npc_question_and_answer.rpy", line 51, in script
$ topics_to_ask.remove('Hackathon')
File "/Users/lambert_pierre/Desktop/LearnToCodeRPG.app/Contents/Resources/autorun/renpy/ast.py", line 923, in execute
renpy.python.py_exec_bytecode(self.code.bytecode, self.hide, store=self.store)
File "/Users/lambert_pierre/Desktop/LearnToCodeRPG.app/Contents/Resources/autorun/renpy/python.py", line 2235, in py_exec_bytecode
exec(bytecode, globals, locals)
File "game/scripts/labels/npc_question_and_answer.rpy", line 51, in
$ topics_to_ask.remove('Hackathon')
File "/Users/lambert_pierre/Desktop/LearnToCodeRPG.app/Contents/Resources/autorun/renpy/python.py", line 875, in do_mutation
return method(self, *args, **kwargs)
KeyError: u'Hackathon'
Darwin-19.6.0-x86_64-i386-64bit
Ren'Py <IP_ADDRESS>5
Learn to Code RPG 1.0
Sun Dec 26 23:12:25 2021
[/code]
Steps to Reproduce
on the 4th of August, ask Annika about Hackathon to eventually have the game crashing at some point in the discussion. Unfortunately, it appears you can't really avoid it as the "That's everything I need to know" option is not clickable.
Screenshots
No response
Additional Information
No response

Duplicate of #14. Fixed.
Please feel free to download the updated version from itch. Your saved game should carry over without a problem.