Internal:  add TypeScript declaration files

Summary
What changed?
Internal:  add TypeScript declaration files
Why?
Support Gestalt in apps using TypeScript rather than Flow.
Internalize ownership of https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/gestalt/index.d.ts for better maintenance
Previous non working version/ PENDING ITP REVIEW Internal: add script for autogenerated TS declaration files #2723
Links

Jira
[TDD](link to Paper doc)
[Figma](link to Figma file)

Checklist

[ ] Added unit and Flow Tests
[ ] Added documentation + accessibility tests
[ ] Verified accessibility: keyboard & screen reader interaction
[ ] Checked dark mode, responsiveness, and right-to-left support
[ ] Checked stakeholder feedback (e.g. Gestalt designers)

try /alpha release

test /alpha release

@AlbertCarreras - when you get the chance, can you reset this back to your original commit


OTEL sources should create events without any transformations

Is your feature request related to a problem? Please describe.
Currently, all OTEL sources (OTEL trace source, OTEL metric source and OTEL log source) does some transformations while creating events from OTEL data.

In all sources, the keys are created by replacing "." with "@"  (dedotting)
In all sources, the attributes are "flattened" by moving them to the root of the event instead of nesting under "attributes"

The dedotting is done to make the data is compatible with OpenSearch.
Describe the solution you'd like
I think the transformations should be outside of the OTEL sources because sink is not always OpenSearch. Also, the users are not given any option to not do the transformations. We should remove the transformations from the OTEL sources and let users explicitly do this as a processor or OpenSearch sink option
   processor:
      - opensearch_compatibility_transform:
           flatten_attributes: true
           dedotting: true

or Alternatively
   sink:
     - opensearch:
           opensearch_compatibility_transformation: true

Describe alternatives you've considered (Optional)
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

OpenTelemetry usually has a threefold aggregation of its signals: resource > instrumentation scope > logs/metrics/spans. This means a resource can hold a collection of instrumentation scopes which can hold a collection of either logs, metrics or span. Each of these levels have a name and can have a nested map of attributes usually described by the semantic conventions.
The flattening of the data structure was introduced to break down these collections to the smallest part, i.e., a log record, metric data point or span record. This helps generate single documents for each of these parts. If the flattening is removed, the collections will stay together. Note, that these collections might have been built arbitrarily by the batch processor, e.g., implemented in the OpenTelemetry Collector. This seems strange to keep in Data Prepper.
Furthermore, Data Prepper currently does not handle the instrumentation scope attributes correctly. They are added as log/metric/span attributes without any means of distinction from the proper attributes of these signals. This needs to be fixed in any case.
Finally, the OpenSearch data model created by Data Prepper does not align to the schemes and mappings defined by the OpenSearch catalogue: https://github.com/opensearch-project/opensearch-catalog/tree/main/schema/observability. This should be aligned, so that data ingested by Data Prepper can be used with the visualizations from the catalogue.

@KarstenSchnitter by "flattening" I meant flattening of "attributes" only. Which is instead of putting attributes under "attributes" key, they are being put under the root. I have added "flatten_attributes" option to OTEL metrics to skip flattening if config option is provided.

@dlvenable regarding migration, I think in 2.x release, we should keep the default behavior as is but provide config options to skip "flattening" of attributes (already available for OTEL metrics) and also provide an option to skip "dedotting".
We should revisit at the time of 3.x release if we want to remove the default behavior altogether or provide optionally (basically, make default behavior optional and optional behavior default)

We ae planning to migrate from Elasticsearch and Splunk to Opensearch and the feedback what we are getting from our users is that this @ notation is not user friendly. It is very confusing for them while writing PPL.
We don't have any issue with nested fields and we are fine with it. Also these renaming is not happening in http input. It is happening only with otel input.
I can see how this dedotting is going to be helpful in preventing mapping conflicts and will reduce field depth, but if you can control your incoming event format then this is not really helpful.
All we are asking is to provide an option to disable this feature. You can have the default behavior of replacing dot with @. Just give user a choice to opt out of it. We can add an option to otel_logs_source.  Something like skip_dedotwith default asfalse`. People can use this option  to skip dedot.
  source:
    otel_logs_source:
      skip_dedot: true

People are used to separate fields with dot and they will make mistake if it is a mix of dot and @.
"resource.attributes.k8s@pod@name" doesn't look as good as "resource.attributes.k8s.pod.name". Now you are saying that people has to remember to give @ instead of dot starting from second field after attributes. This is not at all user friendly.


Lightweight wallet module

The wallet must not consume more than 20MB of RAM at any point. It's not entirely clear to me how we would measure this.
The goal is to keep all of siad below 500MB of RAM consumption. The renter is realistically going to need 300MB of RAM to do all of the erasure coding.
RAM Budget:

gateway - 20 MB
consensus - 100 MB
transaction pool - 50 MB (mempool is big!)
wallet - 20MB
miner - 5MB (doubtful that it needs that much)
host - 5MB when not hosting
renter - 300MB (erasure coding)

The host in heavy use has a generous RAM budget as well. Like 64 MB per TB of data or something. I forget exactly, but it's reasonable to expect 1PB hosts to have 64 GB of RAM, and for 1 TB hosts to have 1 GB of RAM.

we could measure it by comparing memory usage of siad+oldwallet to siad+newwallet. pprof might also help.
The gateway might need more than 20MB if it has lots of peers and is exchanging data with all of them.

Data exchanges in the form of blocks or transactions would count as consensus or tpool weight.


[border-agent] advertises more meshcop service TXT entries

This commit refactors the Border Agent module to advertise almost the full list of
TXT entries of the _meshcop._udp mDNS service. The event emitter is removed
and we rely on the otThreadStateChanged event to notify changes of extended
PAN ID, network name and backbone router state...
resolves https://github.com/openthread/ot-br-posix/issues/702.

Removing event emitter is not directly related to publishing mDNS service and should be done in a separate PR?
Furthermore, I am not sure if it's a good idea to remove event emitter.
The event emitter separate OTBR code from direct OT API calls. For example, if we are to support NCP mode in otbr-agent, we can't call OT API calls directly in OTBR code. I think it's better to keep OTBR code not directly dependent on the current RCP model to allow potential future enhancements.
Thoughts? @wgtdkp @bukepo


Add missing 'Debug's

Fixes #issue
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:
[ ] rustfmt run on changed code

This change is

As usual :)
But I can't build dx and vulkan.
So would be cool if someone will fix those

@omni-viral wait, what? You can't build Vulkan?

@kvark I can't build gfx-backend-vulkan or gfx-backend-dx12

You moved on MacOS? I thought you like Vulkan :)

I would happily accept a PR and publish 0.24.5. The current version of ash also doesn't implement Debug.
What do you want to see in the debug output? The addresses of all the function pointers?


Path property is not working using react and lottie-react

Overview
Path property is not working using react and lottie-react + lottie-interactivity, here's an example
https://codesandbox.io/s/busy-grass-fzxr45
Any suggestions will be highly appreciated, thanks!

Will look in to this and get back to you @drakyone. Seems like a problem with the library.

Any update on this?

hi @shuheen haven't had time to work on this yet. If you or @drakyone would feel comfortable enough taking a crack at it feel free and I would happily review. Will keep you posted, the earliest I can start on this is the week after next.
Cheers

@samuelOsborne hey, I fixed it but couldn't push branch because of the permissions, could you check this please

it's failed because stop method set null to player on the line above


Support DeepL free API

The only difference between free and pro is url with suffix "-free": https://api-free.deepl.com

check version >= 1.4.3


[Feature] Live debugging Java App running in Kubernetes Cluster

So far live debug can be achieved for a pod through port forwarding, refer to 	http://blog.christianposta.com/kubernetes/java-remote-debug-for-applications-running-in-kubernetes/  and https://github.com/solo-io/squash.
This feature includes

How to provide user friendly E2E dev experience to turn on/configure debug port?
Does it perform well?
Any other solution?

@akaroml will drive this feature.

@kyliel where are we here? I think I can do this, but with some degree of latency. You?

I thought we split it to 2 issues - one for local and the other for remote. Local is done (still have some discussions about how to deal with secretes). Then this one is about remote debugging.

Local debug issue is here https://github.com/Azure/vscode-kubernetes-tools/issues/41.


Scheduled jobs to delete the projects and associated objects after the cool-off period (30 days)

Tested project was deleted


Latest version 2.0.0 doesn't work with latest PostCSS 5.1.2

It parses everything ok, just doesnt replace @imports.
Here is a testcase
http://take.ms/qUK5L
if i just do 'npm i gulp postcss gulp-postcss postcss-partial-import'
which installs latest versions, imports will not work.
Works ok with PostCSS 5.0.10

+1
now, I used v1.3.0, but it could't automatic recompile on changes to an imported file when using something like webpack’s watch mode.

I agree, it didn't work at all. So sad, dir option is very useful.

Are you still having this issue?

With version 2.1.0 it now works, but if I try to include file, that did not exists, it just ignore this include and leave it as is, when in version 1.3.0 it throws an error “no such file or directory”.

postcss-partial import is now a layer of sugar atop the new postcss-import, so this should be resolved.


Pex won't do anything.

My permissionsex is red in /plugins. No clue why

That's always because there are errors in the permissions file.

Hence why I closed it...

And since you didn't have the common courtesy to mention that you figured out what the problem was, I had to clarify it for all future generations who will Google "PEX is red" and end up at this thread.


Fix #127: Don't ignore removeDuplicates option when using officialSorting

Pull Request Name
Description
The check for removing duplicates is completely separate from the sorting, even when {officialSorting: false}. Therefore, it shouldn't harm to also execute this check when {officialSorting: true}.
Fixes #127
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] This change requires a documentation update

How Has This Been Tested?

[x] I've added the following test:
Using official sorting, with duplicates
 with config {officialSorting: true} (removeDuplicates is true by default).

Test Configuration:

OS + version: Arch Linux (last upgraded today)
NPM version: 8.5.5
Node version: 16.15.1

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

👋 Hi @mpsijm ,
thank you for your contributions.
Both remaining PR looks fine and I'll merge them in the following days, just to make sure nobody complains between releases.
Our team at my main job has been reduced so I have less opportunities to spend time on the plugin but I hope this is temporary 😉

👋 @mpsijm,
I'll include your changes inside the previous PR #132  which will:

Support classnames generated by plugin(s)
Deprecate officialSorting and prependCustom, always considered to be true (required in order to support plugins)
FIX multiples bugs listed in the PR #132
and also keep removeDuplicates option

I'll publish a beta version soon and collect feedback from our users.
Thanks again for your contributions 🙏

I just published a beta version with the fixes listed on #132
https://github.com/francoismassart/eslint-plugin-tailwindcss/tree/fix/no-custom-classname-plugin-support#readme
npm i<EMAIL_ADDRESS>Please give it a try and provide feedback and or issues 👍
Thank you


how can I send http request with url scheme?

Description:
how can I send http request with url scheme?
I want to send a request "https://portal.tencentciam.com/logout?client_id=YjBiYzNU1YTI&logout_redirect_uri=ciam://oauth-callback/Ciam" , and if success the app will redirect to ciam://oauth-callback/Ciam and it is url scheme
I used oauthswift.client to send request but I failed ,what can I do?
Thanks for you reading,any suggestion appreciated.
OAuth Provider? (Twitter, Github, ..):
OAuth Version:

[ ] Version 1
[✔ ] Version 2

OS (Please fill the version) :

[✔] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[✔] CocoaPods
[ ] Swift Package Manager
[ ] Manually

Library version:

[x] head
[ ] v2.1.0
[ ] v2.0.0
[ ] v1.4.1
[ ] other: (Please fill in the version you are using.)

Xcode version:

[ ] 11.4 (Swift 5.2)
[ ] 11.x (Swift 5.1)
[ ] 10.x (Swift 5.0)
[✔] other: (Please fill in the version you are using.)
13.2
[ ] objective c

Did you ever figure this out? I have a similar problem. 😅


[5.7] Rename method without breaking changes

This reverts commit 2e01692
@taylorotwell , I have returned the ensureOutputIsBeingCapturedForEmail method, since it can be overloading within some app.
I have proposed to delete the method in the 5.8

I think this is overkill tbh.

@GrahamCampbell , I can agree that  it is overkill, but without this we can break some applications ;(


d3m - config file

Allow admin entry/upload of a basic config file

Assess whether files indicated are reachable
Use variation of TA2 config specs below

Pass config variables to TwoRavens UI via url params

Configuration variable
Description
Input/Output

dataset_schema
Path to the current dataset schema
Input

problem_schema
Path to the current problem schema
Input

training_data_root
Path to the root directory of the dataset described by dataset_schema
Input

pipeline_list
Path at which performers should write the pipeline list, output described in Section 4.1.3
Output

executables_root
Directory in which to write the Test Executables.
Output

temp_storage_root
Temporary storage root for performers to use.
N/A

Configuration variable
Description
Input/Output

test_data_root
Path to the root directory of the test data.
Input

solution
Path at which performers should write the Test Executable output, i.e. the solution to the challenge problem given the test data.
Output

temp_storage_root
Temporary storage root for performers to use.
N/A

[ ] config branch models
[ ] config branch logic to get look for/retrieve files
[ ] pass info to rook either by direct url (to start) or with callback info

So far:
The master branch has the ability to retrieve config information from django.  if you pull the master, run the following afterwards:
(assuming you're in a Terminal and have already done workon 2ravens)

fab init_db

adds new table with the D3M search config attributes here (as of 9/18): https://datadrivendiscovery.org/wiki/display/gov/TA2+Configuration+file+syntax

fab make_d3m_config

creates config settings with fully qualified paths based on your local system for the o_196 and o_4550 test data

fab run
go to: http://<IP_ADDRESS>:8080/config/d3m-config/list

if the above worked, you will see info on the config settings available in the database

outcome:

this url:   http://<IP_ADDRESS>:8080/config/d3m-config/json/latest
should give the following but with your local paths and a different id:

{
  "id": 6,
  "name": "o_196seed",
  "is_default": true,
  "dataset_schema": "/Users/yourname/Documents/github/TwoRavens/data/d3m/o_196seed/data/dataSchema.json",
  "problem_schema": "/Users/yourname/Documents/github/TwoRavens/data/d3m/o_196seed/problemSchema.json",
  "training_data_root": "/Users/yourname/Documents/github/TwoRavens/data/d3m/o_196seed/data",
  "executables_root": "/Users/yourname/Documents/github/TwoRavens/test_setup_local/d3m_output_o_196seed/temp",
  "pipeline_logs_root": "/Users/yourname/Documents/github/TwoRavens/test_setup_local/d3m_output_o_196seed/temp",
  "temp_storage_root": "/Users/yourname/Documents/github/TwoRavens/test_setup_local/d3m_output_o_196seed/temp",
  "created": "2017-09-19 01:50:11.257608+00:00",
  "modified": "2017-09-19 01:50:11.258682+00:00",
  "config_url": "/config/d3m-config/details/json/6"
}


Facebook prompting verification when downloading videos

Checklist

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

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.facebook.com/groups/2180265525569106/permalink/2838450249750627', '--verbose', '-u', 'PRIVATE', '-p', 'PRIVATE']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.04.26
[debug] Python version 3.8.0 (CPython) - Linux-3.18.19-15382149-armv7l-with-libc
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1
[debug] Proxy map: {}
[facebook] Downloading login page
[facebook] Logging in
[facebook]<PHONE_NUMBER>750627: Downloading webpage
[facebook]<PHONE_NUMBER>750627: Downloading webpage
ERROR: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/data/data/com.termux/files/usr/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 806, in wrapper
    return func(self, *args, **kwargs)
  File "/data/data/com.termux/files/usr/lib/python3.8/site-packages/youtube_dl/YoutubeDL.py", line 827, in __extract_info
    ie_result = ie.extract(url)
  File "/data/data/com.termux/files/usr/lib/python3.8/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/data/data/com.termux/files/usr/lib/python3.8/site-packages/youtube_dl/extractor/facebook.py", line 677, in _real_extract
    return self._extract_from_url(real_url, video_id)
  File "/data/data/com.termux/files/usr/lib/python3.8/site-packages/youtube_dl/extractor/facebook.py", line 591, in _extract_from_url
    raise ExtractorError('Cannot parse data')
youtube_dl.utils.ExtractorError: Cannot parse data; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

When trying to download a video from a private group, I would need atleast to login with my password and username for me to access those video. When trying to download those video (here on Termux app) I get an error from youtube-dl and facebook would prompt a verification on google chrome.

I have the same exact problem.

A couple of weeks ago I had a similar, but not exactly the same, issue with Facebook. I resolved it as follows. (I don't know if this will work for you, and I don't understand why the coders here haven't commented on this issue.)
Using the yt-dl  --username --password option with Facebook caused serious problems for the Facebook account I was using. Once you get that suspicious activity block from FB, you have to go through the steps to remove it before FB will work on any device. You have to prove you are not suspicious using one of the ways they suggest.
The better method for downloading private or other videos that seem to want a login on yt-dl is to use the --cookies option. This has worked for me and has never caused FB to start blocking my login even when --cookies hasn't worked. Others here have cautioned about using --username --password. I would advise against using it with Facebook.
To use --cookies, you have to make a text file from the cookies stored in the browser in which you have recently (within a few hours) logged in successfully on FB. Then put --cookies PathToCookie-text-file\name-of-cookie-text-file in your yt-dl command.
There are many little addons or programs that make a text file from cookies in various browsers.
I use an addon in Firefox that is very simple and works well: cookies.txt 0.1 by Lennon Hill https://github.com/lennonhill/cookies-txt. This places a button on Firefox that when clicked makes a cookies.txt file wherever you want to save it to.
I also use Termux on Android. I make the cookies.txt file on a desktop and copy it to the tablet root directory and then, on the mobile device, copy the file into Termux's file system. It works, but I don't know if mobile Firefox can use the addon to make the cookie file.

A couple of weeks ago I had a similar, but not exactly the same, issue with Facebook. I resolved it as follows. (I don't know if this will work for you, and I don't understand why the coders here haven't commented on this issue.)
Using the yt-dl --username --password option with Facebook caused serious problems for the Facebook account I was using. Once you get that suspicious activity block from FB, you have to go through the steps to remove it before FB will work on any device. You have to prove you are not suspicious using one of the ways they suggest.
The better method for downloading private or other videos that seem to want a login on yt-dl is to use the --cookies option. This has worked for me and has never caused FB to start blocking my login even when --cookies hasn't worked. Others here have cautioned about using --username --password. I would advise against using it with Facebook.
To use --cookies, you have to make a text file from the cookies stored in the browser in which you have recently (within a few hours) logged in successfully on FB. Then put --cookies PathToCookie-text-file\name-of-cookie-text-file in your yt-dl command.
There are many little addons or programs that make a text file from cookies in various browsers.
I use an addon in Firefox that is very simple and works well: cookies.txt 0.1 by Lennon Hill https://github.com/lennonhill/cookies-txt. This places a button on Firefox that when clicked makes a cookies.txt file wherever you want to save it to.
I also use Termux on Android. I make the cookies.txt file on a desktop and copy it to the tablet root directory and then, on the mobile device, copy the file into Termux's file system. It works, but I don't know if mobile Firefox can use the addon to make the cookie file.

Sounds like more extra step instead of a simple one.


Can't filter by date

Laravel-mongodb Version: 3.9.5
PHP Version: 8.2

Description:
I have a date array and I want to retrieve those that are an interval.
For example: Retrieve me those with a date between 2023-03-01 and 2023-03-31
Steps to reproduce
My collection is presented :
"departures": [
			"2022-10-23T23:30:00.000+00:00", // Date object
                         "2023-02-02T23:30:00.000+00:00",
                         "2023-11-10T18:00:00.000+00:00"
		],
		"price": 310,

$beginDate = Carbon::createFromFormat("Y-m-d", "2023-03-01");
$endDate = clone $beginDate;
$endDate->endOfMonth();

$search->whereBetween('departures', [$beginDate->toDateTime(), $endDate->toDateTime()])->get()->toArray();

Expected behaviour
It must return me an empty array.
Actual behaviour
It returns me all the table.

Hi you can use
protected $dates = ['startDate', 'endDate']; $d = WorkPeriod::where('startDate', '>=', Carbon::parseFromLocale($this->start))->where('endDate','<=',Carbon::parseFromLocale($this->end));
$this->start = can be string no problem with parseFromLocale

Thanks for the correct answer.
$dates is now replaced by $casts.


Data Modelling for Repository Recommendation System

Created data_clean.py script to use when importing new data from the GitHub API : This script deletes stopwords and special characters, as well as deleting unnecessary columns for all possible models ('open_issues_count', and 'allow_forking'). The rest are left to handle for each model type -> Use the eda.ipynb file to carry out feature importance on your target variable of choice.

Created a working repository recommender model based on cosine similarities 'cosine_modelling_repos' : Read the comments at the top of the file for more information.

Created an experimentation repository recommendation model using Pytorch : Read the comments at the top of the file for more information.

Cleaned and inserted final data into modelling pipeline : Over 55,000 of cleaned, unique repositories.

Structured the repo for experimentation -> created folders and subfolders to separate clean from original data.

Very well done Mr. GG @felixggj


Cannot run the MemCached unit tests

See https://github.com/decebals/pippo/pull/334#issuecomment-267345437 for details.

Probably the fast solution is to comment the MemCached unit tests.

@decebals, Is it possible to release a minor version like 1.0.1 by commenting the MemCached unit tests??
Releasing a minor version would be really helpful for me since i require gzip support and using snapshot in production is not a good idea.

@munendrasn Sure, no problem.

@munendrasn Version 1.1.0 is available on Maven Central Repository.


Nested async templates - JS not firing/loading

The parent async templates are working great (and love the gem, btw)!
Started to have the need for nested async templates and followed the instructions to the T, including the <%= content_for :render_async %> inside the parent async template. JS inspection confirms that the nested asyncs are generated but it's not firing the URL.
Can't help but wonder if it's because the JS is dynamically added by the parent and then not firing because it has been added after page load?
Any insights are greatly appreciated.

Hey @chadwtaylor, thanks for submitting the issue!

Can't help but wonder if it's because the JS is dynamically added by the parent and then not firing because it has been added after page load?

I think it is, I'll try to reproduce this myself and come up with a fix for it :)

Now that I'm looking into it, seems like the problem is in the Vanilla JS part. script tag that is returned from parent render_async response doesn't get evaluated in the HTML, thus is not being called.
It's related to this issue https://github.com/renderedtext/render_async/issues/30
As a quick fix, you can turn on the jQuery in your config if you're using it in your project.
I'll try to make this work for Vanilla JS hopefully.

Hi.
Same problem here.
I tryed some versions.
No Probrem:
render_async: v1.2.0
render_async: v1.3.0
Has Probrem:
render_async: v1.4.0
render_async: v2.0.0

Thanks for commeting @filipewl! You explained the situation really well for loading script tags to the DOM.
To sum up again, if you want to have your nested


Bump clap to v4

Description
Add fixes on top of the previous PR to bump clap to v4.
How was this PR tested?

Updated tests
Tested all the quickwit index subcommands manually

@fmassot, PTAL


Roll engine 6a0460c33ff7..df707c6f1f1a (1 commits)

https://github.com/flutter/engine/compare/6a0460c33ff7...df707c6f1f1a
git log 6a0460c33ff7e4f18b809285301337698de9ab38..df707c6f1f1ac87b02d4d65195d171bdd861da01 --no-merges --oneline
df707c6f1 switch to newer APIs for shaders and filters (flutter/engine#8511)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5684603233763328 , https://cirrus-ci.com/task/5966078210473984

Commit queue failed; closing this roll.


[WIP][MondeDiploBridge] Use https and fetch full content

This currently works. I will try remove some parts of the fetched content to make it more clean.
This is my first contribution attempt to this project, let me know if you have any feedback.
@pitchoule you may want to take a look.

Well there have been a few commits after this PR which fixed things. Also op deleted their fork. Closing. Feel free to submit a new PR!


Odd behavior and errors with large number of links

I have ~1350 links, and I seem to have hit some sort of limit. If I add even one more, I get an error message at the top of many of my pages "A PCRE internal error occured. This might be caused by a faulty plugin", and even manually-configured (not Autolink) links stop working correctly.

This is a known limitation in DokuWiki that I couldn't easily get around. I might revisit it someday, but it's a very big project, which would either require me to make a custom page parser in my plugin, or update Doku itself. Here's an explanation of the issue: https://stackoverflow.com/questions/8268624/php-preg-match-all-limit
The only thing you can do in the meantime is try to reduce the number of autolink matches you're using. The best thing to look for is multiple matches that go to the same page. You can find ways to combine these. For example, if you want all of these to go to doku:mypage:
mypage
Mypage
MyPage
mypages
Mypages
MyPages
you can create a single rule instead of six:
[mM]y[pP]ages?
If you can't figure out how to combine certain rules, feel free to send me a couple examples.


certigo: 1.15.0 -> 1.15.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/certigo/versions.
meta.description for certigo is: "A utility to examine and validate certificates in a variety of formats"
meta.homepage for certigo is: "https://github.com/square/certigo"
meta.changelog for certigo is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/01bqkxaddlzh6s0f68rs227rxycvz3kf-certigo-1.15.1/bin/certigo had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.15.1 with grep in /nix/store/01bqkxaddlzh6s0f68rs227rxycvz3kf-certigo-1.15.1
found 1.15.1 in filename of file in /nix/store/01bqkxaddlzh6s0f68rs227rxycvz3kf-certigo-1.15.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
certigo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/01bqkxaddlzh6s0f68rs227rxycvz3kf-certigo-1.15.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A certigo https://github.com/r-ryantm/nixpkgs/archive/4f85997362d9f97fe5282d18956e07cafa44b82c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/01bqkxaddlzh6s0f68rs227rxycvz3kf-certigo-1.15.1
ls -la /nix/store/01bqkxaddlzh6s0f68rs227rxycvz3kf-certigo-1.15.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

certigo

Maintainer pings
cc @marsam @kalbasit for testing.

@kalbasit would it be okay if i remove you from https://github.com/NixOS/nixpkgs/blob/836171e89a50c9b80c3b342d32c1bc165a211409/pkgs/development/go-modules/generic/default.nix#L264-L270


Support full range of U64

Right now we're limited by I64

Added in https://github.com/bearcove/merde/commit/052fdef6fbd272c2caff4f2e04156106897f9218 — which is a turbo-breaking change


I have completed the JSON part. I have fetched the JSON String from t…

…he server and added the comments to the page

I see that formatter suggests you some changes:
https://github.com/AlexandraBulaceanu/step-intern-week2-bulaceanu/pull/5/checks?check_run_id=898388943
for example: changing import order and one line break. It doesn't fix it itself, but I guess you can copy paste the changes it suggests.

For me it seems like the check only "checks" everything:
if ! java -jar ./google-java-format-1.8-all-deps.jar $file | diff $file -; then
EXIT=1
fi
And you should set up command line shortcuts for reformatting code yourself (using this .jar)


update(build): bump libs to 0.9.2 for 0.33.1

Signed-off-by: Luca Guerra<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind release

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

update(falco): bump libs to 0.9.2

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign mstemm for approval by writing /assign @mstemm in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 9cf627c574149e39816ff10ccc73a436cd853336

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.33.1

@LucaGuerra: You must be a member of the falcosecurity/falco-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your maintainers of falcosecurity/falco and have them propose you as an additional delegate for this responsibility.

In response to this:

/milestone 0.33.1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/milestone 0.33.1

@FedeDP: The provided milestone is not valid for this repository. Milestones in this repository: [0.34.0, 1.0.0]
Use /milestone clear to clear the milestone.

In response to this:

/milestone 0.33.1

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Manually done! :laughing:


move overwrite_existing to function api

overwrite_existing is now defined in the input file for open_esm_metadatastore.
This should be a separate function argument.

Do you mean something like passing this argument to this function https://github.com/NCAR/intake-esm/blob/master/intake_esm/core.py#L96 and/or some other function involved in collection build?

Yes. The collection input file specifies the information necessary to generate a collection, but  overwrite_existing is more of a programatic control option, not really an attribute of the collection.

@matt-long, now that you suggest moving overwrite_existing to function API, should we do the same thing for include_cache_dir?

Yes, I think so. I could see this include_cache_dir going away in the case where we have an automated gather_data method.


create-react-app with --typescript doesn't work

Is this a bug report?
Yes

Did you try recovering your dependencies?

Yes
Environment

  System:
    OS: macOS 10.14.2
    CPU: x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
  Binaries:
    Node: 8.12.0 - /usr/local/bin/node
    npm: 6.7.0 - ~/.npm-packages/bin/npm
  Browsers:
    Chrome: 72.0.3626.81
    Safari: 12.0.2
  npmPackages:
    react: ^16.7.0 => 16.7.0
    react-dom: ^16.7.0 => 16.7.0
    react-scripts: 2.1.3 => 2.1.3
  npmGlobalPackages:
    create-react-app: 2.0.2

Steps to Reproduce

When I create a new project using npx create-react-app project-geo-semantic-react --typescript, the generate app in a Javascript App and not a Typescript App.
For example, the package.json doesn't contains typescript dependencies :
  "dependencies": {
    "react": "^16.7.0",
    "react-dom": "^16.7.0",
    "react-scripts": "2.1.3"
  },
  "scripts": {
    "start": "react-scripts start",
    "build": "react-scripts build",
    "test": "react-scripts test",
    "eject": "react-scripts eject"
  },

Expected Behavior

Using --typescript parameters, a configured typescript project is generated.
Actual Behavior

Using --typescript parameters, the generated project is not configured for use typescript.
I need to manually add typescript support on my project.

Hello @ZeJ0hn, this is common problem, and I think we've updated our docs recently to solve this.
First, I'd recommend not having a globally installed create-react-app package. But, if you must, you need to update it for this to work.
If you can remove it, do so, and then running yarn create react-app --typescript or npx create-react-app --typescript will work for you.
Sorry for the inconvenience.


Update for new pyvesynv_v2 library and vesync switch support

Description:
Although the guidelines say to not make trivial edits, I overhauled the older vesync library to support the new outlets and wifi wall switches.  I am just updating the pip package used for the vesync switch component.  I originally tried to submit a PR for the original vesync library to streamline the changes without impacting HA but the owner hasn't responded in several months.  I created a new pip package 'pyvesync_v2' with the same functionality as the original vesync so it would integrate with minimal changes to the component.
**Related issue (if applicable):
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8746 -
Example entry for configuration.yaml (if applicable):
{Same as original vesync switch component)
switch:
  - platform: vesync
     username:
     password:

Checklist:

[X ] The code change is tested and works locally.
[X ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ X] There is no commented out code in this PR.

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

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

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

[NA] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[X] New dependencies are only imported inside functions that use them ([example][ex-import]).
[NA] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[NA] New files were added to .coveragerc.

If the code does not interact with devices:

[NA] Tests have been added to verify that the new code works.

Hi @webdjoe,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


conhost advertises ccc Tektronix color support incorrectly

From @mqudsi on October 15, 2017 15:57
Running 1709/17017.1000:
$TERM is set to xterm-256-color but in the termcap file that ships with WSL's Ubuntu copy, xterm-256-color includes ccc support:
#	Reconstructed via infocmp from file: /lib/terminfo/x/xterm-256color
xterm-256color|xterm with 256 colors,
	am, bce, ccc, km, mc5i, mir, msgr, npc, xenl,
	colors#256, cols#80, it#8, lines#24, pairs#32767,
	acsc=``aaffggiijjkkllmmnnooppqqrrssttuuvvwwxxyyzz{{||}}~~,
	bel=^G, blink=\E[5m, bold=\E[1m, cbt=\E[Z, civis=\E[?25l,
	clear=\E[H\E[2J, cnorm=\E[?12l\E[?25h, cr=^M,
	csr=\E[%i%p1%d;%p2%dr, cub=\E[%p1%dD, cub1=^H,
	cud=\E[%p1%dB, cud1=^J, cuf=\E[%p1%dC, cuf1=\E[C,
	cup=\E[%i%p1%d;%p2%dH, cuu=\E[%p1%dA, cuu1=\E[A,
	cvvis=\E[?12;25h, dch=\E[%p1%dP, dch1=\E[P, dim=\E[2m,
	dl=\E[%p1%dM, dl1=\E[M, ech=\E[%p1%dX, ed=\E[J, el=\E[K,
	el1=\E[1K, flash=\E[?5h$<100/>\E[?5l, home=\E[H,
	hpa=\E[%i%p1%dG, ht=^I, hts=\EH, ich=\E[%p1%d@,
	il=\E[%p1%dL, il1=\E[L, ind=^J, indn=\E[%p1%dS,
	initc=\E]4;%p1%d;rgb\:%p2%{255}%*%{1000}%/%2.2X/%p3%{255}%*%{1000}%/%2.2X/%p4%{255}%*%{1000}%/%2.2X\E\\,
	invis=\E[8m, is2=\E[!p\E[?3;4l\E[4l\E>, kDC=\E[3;2~,
	kEND=\E[1;2F, kHOM=\E[1;2H, kIC=\E[2;2~, kLFT=\E[1;2D,
	kNXT=\E[6;2~, kPRV=\E[5;2~, kRIT=\E[1;2C, kb2=\EOE,
	kbs=\177, kcbt=\E[Z, kcub1=\EOD, kcud1=\EOB, kcuf1=\EOC,
	kcuu1=\EOA, kdch1=\E[3~, kend=\EOF, kent=\EOM, kf1=\EOP,
	kf10=\E[21~, kf11=\E[23~, kf12=\E[24~, kf13=\E[1;2P,
	kf14=\E[1;2Q, kf15=\E[1;2R, kf16=\E[1;2S, kf17=\E[15;2~,
	kf18=\E[17;2~, kf19=\E[18;2~, kf2=\EOQ, kf20=\E[19;2~,
	kf21=\E[20;2~, kf22=\E[21;2~, kf23=\E[23;2~,
	kf24=\E[24;2~, kf25=\E[1;5P, kf26=\E[1;5Q, kf27=\E[1;5R,
	kf28=\E[1;5S, kf29=\E[15;5~, kf3=\EOR, kf30=\E[17;5~,
	kf31=\E[18;5~, kf32=\E[19;5~, kf33=\E[20;5~,
	kf34=\E[21;5~, kf35=\E[23;5~, kf36=\E[24;5~,
	kf37=\E[1;6P, kf38=\E[1;6Q, kf39=\E[1;6R, kf4=\EOS,
	kf40=\E[1;6S, kf41=\E[15;6~, kf42=\E[17;6~,
	kf43=\E[18;6~, kf44=\E[19;6~, kf45=\E[20;6~,
	kf46=\E[21;6~, kf47=\E[23;6~, kf48=\E[24;6~,
	kf49=\E[1;3P, kf5=\E[15~, kf50=\E[1;3Q, kf51=\E[1;3R,
	kf52=\E[1;3S, kf53=\E[15;3~, kf54=\E[17;3~,
	kf55=\E[18;3~, kf56=\E[19;3~, kf57=\E[20;3~,
	kf58=\E[21;3~, kf59=\E[23;3~, kf6=\E[17~, kf60=\E[24;3~,
	kf61=\E[1;4P, kf62=\E[1;4Q, kf63=\E[1;4R, kf7=\E[18~,
	kf8=\E[19~, kf9=\E[20~, khome=\EOH, kich1=\E[2~,
	kind=\E[1;2B, kmous=\E[M, knp=\E[6~, kpp=\E[5~,
	kri=\E[1;2A, mc0=\E[i, mc4=\E[4i, mc5=\E[5i, meml=\El,
	memu=\Em, op=\E[39;49m, rc=\E8, rev=\E[7m, ri=\EM,
	rin=\E[%p1%dT, ritm=\E[23m, rmacs=\E(B, rmam=\E[?7l,
	rmcup=\E[?1049l, rmir=\E[4l, rmkx=\E[?1l\E>, rmso=\E[27m,
	rmul=\E[24m, rs1=\Ec, rs2=\E[!p\E[?3;4l\E[4l\E>, sc=\E7,
	setab=\E[%?%p1%{8}%<%t4%p1%d%e%p1%{16}%<%t10%p1%{8}%-%d%e48;5;%p1%d%;m,
	setaf=\E[%?%p1%{8}%<%t3%p1%d%e%p1%{16}%<%t9%p1%{8}%-%d%e38;5;%p1%d%;m,
	sgr=%?%p9%t\E(0%e\E(B%;\E[0%?%p6%t;1%;%?%p5%t;2%;%?%p2%t;4%;%?%p1%p3%|%t;7%;%?%p4%t;5%;%?%p7%t;8%;m,
	sgr0=\E(B\E[m, sitm=\E[3m, smacs=\E(0, smam=\E[?7h,
	smcup=\E[?1049h, smir=\E[4h, smkx=\E[?1h\E=, smso=\E[7m,
	smul=\E[4m, tbc=\E[3g, u6=\E[%i%d;%dR, u7=\E[6n,
	u8=\E[?1;2c, u9=\E[c, vpa=\E[%i%p1%dd,

As such, the terminal is advertising support for ccc, but it does not seem to support it in practice. Per https://linux.die.net/man/5/terminfo:

On a Tektronix-like terminal, the capability ccc may be present to indicate that colors can be modified. If so, the initc capability will take a color number (0 to colors - 1)and three more parameters which describe the color. These three parameters default to being interpreted as RGB (Red, Green, Blue) values. If the boolean capability hls is present, they are instead as HLS (Hue, Lightness, Saturation) indices. The ranges are terminal-dependent.

As far as I can tell, conhost only supports the HP-like color API:

On an HP-like terminal, initp may give a capability for changing a color-pair value. It will take seven parameters; a color-pair number (0 to max_pairs - 1), and two triples describing first background and then foreground colors. These parameters must be (Red, Green, Blue) or (Hue, Lightness, Saturation) depending on hls.

WSL should probably ship with a patched termcap file that drops ccc from xterm-256-color.
Copied from original issue: Microsoft/WSL#2566

From @zadjii-msft on October 16, 2017 20:10
... Or we could just add support for it :)
That particular sequence is one that's been in my personal feature request backlog for a couple releases now. I'm hoping I can get to it in the next release.  It would let me implement colortool in a way that would actually work with WSL too. It's definitely higher on the backlog.

If you do decide to implement this, is there any chance it could be made an option? Using the SGR bold (increased intensity) parameter to generate 16 colors has been a de facto standard (at least in the DOS world) for what must be decades now. And I think the default Linux console might work that way too actually. It would be sad to lose that functionality if you change it to bold instead.
I realise there are there are now other ways of getting bright colors, but that's obviously going to be a breaking change for old code, not to mention all that glorious ANSI artwork from the BBS era.


Runtime: Core BPF Migration: Struct for loading and checking source BPF program accounts

This is chunk 4/6 of the broken-up PR #79.
Note: Based on #304.
Problem
When the runtime desires to migrate a builtin program to Core BPF, it will do
so by replacing the builtin program's program account with that of a BPF
upgradeable program's program account, which will contain a pointer to its
corresponding program-data account.
Currently, there exists no reliable method in the bank or runtime more broadly
for loading a BPF upgradeable program's accounts and performing various
checks to ensure both program and program-data account are configured as
expected.
Being able to run these checks before fetching the program's accounts is
extremely important for ensuring a smooth migration to Core BPF.
Summary of Changes
Within the builtins submodule, add a module specifically for core_bpf_migration,
and inside of it create a new structure for loading a source BPF upgradeable program's
program and program-data account and running checks.

Codecov Report
Attention: Patch coverage is 85.93750% with 27 lines in your changes are missing coverage. Please review.

Project coverage is 81.9%. Comparing base (cc3afa5) to head (70c605b).
Report is 15 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master     #332    +/-   ##
========================================
  Coverage    81.9%    81.9%
========================================
  Files         837      839     +2
  Lines      226823   227033   +210
========================================
+ Hits       185828   186033   +205
- Misses      40995    41000     +5

Note this can go in separate from #318.


Add close_fds for initdb command and add close fd test

This PR adds a test for https://github.com/neondatabase/neon/pull/1834.
It also fixes the error in https://app.circleci.com/pipelines/github/neondatabase/neon/7753/workflows/94d1b796-10a3-4989-b23c-4c1eb4a49cf5/jobs/79586, which happens because pageserver.pid is held by initdb command on restart.

lgtm, thanks for adjusting those.


Start reorganisation of Methylc seq stuffs

Hi,
I started the reorganisation of MethylC-Seq directory.
To do:

[ ] Move the input data to Zenodo
[ ] Move the Docker image to a dedicated flavour
[ ] Build the Docker image
[ ] Add slides
[ ] Test the tutorial with the Docker image
[ ] Add a Galaxy tour

Bérénice

It must be ready to be merged

It is ready, just in time for the Codefest tomorrow. I will go by cookies soon :)


usePublicUrl is not being used on big files?

It seems like on big files weed upload ignores -usePublicUrl option and tries to upload using internal volume's IP address.
# ls -lahrt
total 837M
drwxr-xr-x    3 root     root        4.0K Sep 24 11:46 ..
-rw-r--r--    1 node     root      255.1M Sep 25 04:37 a736d941-b1a7-4b7a-a28e-98f94e2830d0.mp4
-rw-r--r--    1 root     root        3.4M Sep 25 08:25 c626cdcc-3e50-4314-a507-8a4220bd496e.mp4
# weed upload -usePublicUrl -master=storage.poc.loc:9333 c626cdcc-3e50-4314-a507-8a4220bd496e.mp4
[{"fileName":"c626cdcc-3e50-4314-a507-8a4220bd496e.mp4","fileUrl":"storage.poc.loc:8081/32,4be726f19ba3","fid":"32,4be726f19ba3","size":3564365}]
# weed upload -usePublicUrl -master=storage.poc.loc:9333 a736d941-b1a7-4b7a-a28e-98f94e2830d0.mp4
E0925 08:30:48   441 upload_content.go:224] upload to http://<IP_ADDRESS>:8080/25,4bf5e1f1ef53: Post "http://<IP_ADDRESS>:8080/25,4bf5e1f1ef53": dial tcp <IP_ADDRESS>:8080: connect: connection timed out
W0925 08:30:48   441 upload_content.go:93] uploading to http://<IP_ADDRESS>:8080/25,4bf5e1f1ef53: upload to http://<IP_ADDRESS>:8080/25,4bf5e1f1ef53: Post "http://<IP_ADDRESS>:8080/25,4bf5e1f1ef53": dial tcp <IP_ADDRESS>:8080: connect: connection timed out

Is it the case?

thanks for reporting! added a fix.

Fixed! Thanks!


[website] Add product core page

Preview: https://deploy-preview-27952--material-ui.netlify.app/branding/core/

[x] I have followed (at least) the PR section of the contributing guide.

Looks great, what I could spot:

Wrong product name:

Do we need the component function? It feels like we could simplify the demos to be as the inline previews in the documentation.

When the tooltip is selected, should we have one of the 4 tooltips always open? It would avoid having to move the mouse to hover to see the UI. The 3 other tooltips would allow testing the UX.
There is an alignment issue with the information icon:

"Take care of your design game" I don't understand this third idea. It seems to duplicate with the first one. How about we use it to demo the responsive API instead?
The heading structure feels wrong:

First, the most important message seems to be the bigger text, not the smaller one. If it was the case, why would the most important information be with the smaller font-size?
Second, isn't it confusing to have multiple h1 on the page? How do you find the most important piece of information? https://ahrefs.com/blog/h1-tag/

Use only one H1 per page

https://stripe.com/ might have multiples, it doesn't mean it's right.

There is a rogue h6

@oliviertassinari

fixed
agree, updated.
agree, updated.
nice catch, fixed.
good point, change to responsive instead @danilo-leal what do you think about it?
changed to h2, h3 instead but for which one should be h2, I am not sure.
fixed
@danilo-leal any component you want to replace the DatePicker? or should we go with DatePicker for now?

@siriwatknp @oliviertassinari
Just pushed a bunch of changes. Regarding bullet nº10, I've removed the date picker and added/tweaked some of the components there. I focused on interactivity instead of a simple display of components, so most of them you can interact and play a bit.
Great work, Jun, looking awesome! The interactive responsive slider on the styling section is just too good, very nice touch!

changed to h2, h3 instead but for which one should be h2, I am not sure.

As far as I know, headers are supposed to be continuous h3 + h2 would break this rule. h2 + h3 sounds OK.


iOS version missing documentation

The android repository has documentation but the ios repository does not.  Is this planned to be addressed in the near future?

Yes, we do plan to address the lack of documentation for the iOS repository. We have one more product launch event planned for the remainder of this year, so we haven't had the time to release the 90% completed documentation just yet. In the meantime, the structure of the iOS version of SesameSDK is very very very similar to that of the Android version. I would recommend using the Android version's documentation as a temporary guide to familiarize yourself with the iOS version of SesameSDK.


Fixes restore script AWS configuration

This change removes environmental inference from the restore script, restoring the usual credential precedence established in aws-sdk. The default region (Sydney) is preserved for back-compatibility.
Thanks for your consideration!

@dylanlingelbach, any thoughts on this? Happy to address any suggestions (or concerns) stemming from the change.

@rjz sorry for the delay - this looks good to me.

Although I seem to have lost my write access to this repo. I'll figure out what is going on and get this merged.

Thanks, @dylanlingelbach! Much appreciated.

Bump on this PR. Restore won't work on Lambda without this fix.

LGTM

@dylanlingelbach, any word on this? Or any leads on who in markitx we might reach out to?

@rjz sorry, no updates yet. I've been slammed at work the past few weeks. Will follow up and let you know. Worst case I will fork to my personal repo and move the npm package there

@dylanlingelbach any updates on this or other PRs being merged?

@rafaelmagu still working on regaining write access. Sorry for the delay

Hey guys,
Am set out to work on this issue #52
I saw this pr and wondering why the support for env variables is removed
@rjz mentioned that it is to

restoring the usual credential precedence established in aws-sdk.

Can some one guide me to what is the the credential precedence for node sdk
I searched the documentation, but could only find the one for java (look for Using the Default Credential Provider Chain)
Thanks 🙂

@cg-cnu I think this link is what you are looking for

@dylanlingelbach I guess its mostly talking about the ways in which you can load the credentials but not the order in which node SDK will work / prefer to work.
Is providing secrets using the environment variables considered insecure ?


Uncontrolled component?

It appears this component cannot be used in uncontrolled fashion, and requires that you specify value= and control it from outside state. I'm curious if this has to be the case, or if you'd be open to changes to using it in an uncontrolled fashion?
Given that the state is also stored inside the form, I'm happy to have my code submit that form, or extract data from that form directly.
In particular, I'm noticing my webapp being really laggy on my mobile device, and am exploring causes. One potential cause being the overhead on each onChange event (that causes my form to re-render just to pass down a new value=).
I saw this was briefly brought up in https://github.com/reactjs/react-autocomplete/issues/113 , but was then quickly discarded.

Hi Mike! In the very beginning this component was in fact uncontrolled. The main problem we had at that point was that it was in effect impossible to update the component's value without directly referencing the (undocumented) internal ref state. We decided to make the component controlled with the logic that if it's controlled, at least you have the option of wrapping it with another component to make it uncontrolled.
That's typically the advice I've given to people who need an uncontrolled component: create another component which stores value in its own state, then expose defaultValue and whichever other prop you need to get it bootstrapped.
Now I realise that that's slightly more work, and people are going to end up writing very similar code to achieve this. In my experience I almost always end up writing a bespoke wrapper which takes care of app-specific logic, layout, and styling, so adding the code to handle value is only 3-4 extra lines.
When it comes to performance: Even if we maintained value within Autocomplete, chances are we would have to re-render the entire component whenever value changes (due to the nature of how the items are filtered and displayed), so I don't know if there would be much of a gain by doing this. I would rather try to see if you can wrap Autocomplete in PureComponent to see if you can avoid unnecessary tree creation and diffing.
This feature request keeps coming up every few months, and it's definitely something we have in the back of our minds. It will probably be on the v2 roadmap somewhere, if the grand design allows it.

Very good point about performance, that even in uncontrolled mode, it stills requires a re-render each time the value changes anyway. Doh!
I suspect it will be a bit faster since only renderMenu needs to be re-rendered, not the <input>, and not the surrounding React component that might otherwise manage state. And it avoids the need to do additional callbacks. But not too much.
(Naively, my state was stored in a FormComponent, which caused multiple AutoCompletes to needlessly re-renderer. I've since split up my state into multiple components, though PureComponent would have worked too.)
However, I don't think it's possible to wrap this as an uncontrolled component. Adding that support, would also be a valid solution that would work for me.
To do this, react-autocomplete would need to:

Support passing defaultValue instead of value to <input .../>. This is now possible as of a week ago in 1.6 with renderInput (I didn't realize it at the time I filed this, since I was running 1.5.x), with this:

          renderInput={args => {
            const { value, ...otherArgs } = args;
            return <input defaultValue={this.props.value} {...otherArgs} />;
          }}

Not depend on props.value. Replacing such references with an accessor that returns props.value !== undefined ? props.value || this.refs.input.value would work better. (Or maybe an props.valueAccessorFunction I can override, that does the same thing?)

I have also checked a second iPhone, and noticed that my newer iPhone appears to have serious performance issues overall. And so it might be unfair for my to judge the performance here off my possibly-broken device.

Wether or not the component should be allowed uncontrolled or not aside. The documentation does suggest this is the case because it states that the value attribute is optional.
When I do omit the value attribute te component does not render and throws an error.


multi-account

Please update README.md to explicitly state that this is a single account solution, or is already multi-account
Add multi-account support leveraging assume-role

(2) would include logic that possibly reads your credentials file and assumes every role to get all the data.

Also, is there a roadmap for the use of IAM Role?

@princechennai you can already use roles, just need to first get credentials from a role using STS.


Request for an Hands on session on git and github

Please provide the following information

Is this event for college students or professionals?
  This Event is for college students.

How many folks do you expect will attend this event?
 Around 200 students.

When do you want this event?
11th October 2021

Language of the event?
English/Hindi.

[ ] fill this google form.

@vikassethi09 will reach out to you soon , hopefully you have filled the form

Thanks, yup I have filled in gform.


[Play] Thumbnail decoding failure

Report

Bei mir zeigt die App überhaupt keine Vorschau Bilder an sondern es steht nur die Dateien Endung an.

Play console
Device: -
Device language: German
App version code: 58
App version name: 1.5.4
Android version: Android 9 (SDK 28)
Aves
Aves version: 1.5.4 (Build 58)
Flutter version: 2.5.3 (Channel stable)
Android version: 9 (SDK 28)
Device: CUBOT X19
Google Play services: ready
Screenshot

Still in Aves v1.5.5 (Build 59)
Logs
#119 aves-logs-20211109_120445.txt

Faulty media content IDs are weird: in the range [1683768239,<PHONE_NUMBER>], some consecutive, looks like timestamp but they're not, dangerously close to max int<PHONE_NUMBER>.
Collection contains 6425 items, of which 400 are found to have metadata, so not all entries are missing/unreadable. Broken items are likely obsolete items in the Media Store, deleted by a misbehaving app.
Should:

give different visual feedback for missing v. undecodable items,
provide suggestion to clean items when obsolete items are detected.


Tinyproxy doesn't start

For a reason unknown tinyproxy doesn't start when I run the docker image.
docker exec openvpn-client3 ps aux shows that it runs the run-http-proxy.sh script, but tinyproxy itself isn't running. There's also no open port visible when doing docker exec openvpn-client3 netstat -anp.
When I do  docker exec openvpn-client3 tinyproxy (after editing the tinyproxy config to accept all connections and use the right port), it does run tinyproxy and the VPN connection works.
Any idea how to figure out what's wrong? I have no experience with docker.

Looks like the run-http-proxy.sh script hangs somewhere. Maybe the part where it tries to get tun0, but the network interface is actually tun1

Hi @Miraculix200 ,
A quick question, did this problem happen on the first run of the container or after a connection was dropped?
I'm asking because I'm finalizing a PR and one thing I've done is refactored the openvpn tear down phase.
If the problem manifested itself on the first run, I can expose a variable to remap tun0.

In the meantime, I've added some instrumentation around the check-code so there's information in the log about what's going on.
Below is an example:
2023-02-02 09:03:24 [entry.sh] Backgrounding HTTP proxy ...
2023-02-02 09:03:24 [entry.sh] Backgrounding SOCKS proxy ...
2023-02-02 09:03:24 [entry.sh] Backgrounding openvpn ...
2023-02-02 09:03:24 [run-socks-proxy.sh] Waiting for tun0 to become available.
2023-02-02 09:03:24 [run-http-proxy.sh] Waiting for tun0 to become available.
2023-02-02 09:03:24 OpenVPN 2.5.6 x86_64-alpine-linux-musl [SSL (OpenSSL)] [LZO] [LZ4] [EPOLL] [MH/PKTINFO] [AEAD] built on Apr 17 2022
...
2023-02-02 09:03:25 [run-http-proxy.sh] Ready.  Continuing.
2023-02-02 09:03:25 [run-socks-proxy.sh] Ready.  Continuing.
2023-02-02 09:03:25 [run-socks-proxy.sh] info: starting socks proxy without credentials
2023-02-02 09:03:25 [run-http-proxy.sh] info: starting http proxy without credentials


CMake: bump version to 3.5

DuckDB requires 3.5 already, 2.8.12...3.5 syntax removes a warning, but should not actually change anything.
We could probably bump even further something like 2.8.12...3.17 or something like that.

I would just match duckdb's tbh


locator degree does not match number of roots

I am getting below error, and not able to receive message.
com.google.zxing.common.reedsolomon.ReedSolomonException: Error locator degree does not match number of roots
Please let me know what i need to do to get success response

Hey, thanks for checking out the repo.
Just to check, have you made any changes from the repo in your branch? It sounds like the packet data you're sending may have exceeded what is possible to represent using the 5-bit polynomial. Are you trying to send characters that lie outside of the alphabet?


Multi gpu ffts

Hello.
I hope this is the right forum.
I have a question or three regarding ffts.

Is it still true that the clffts cannot be used to farm out
different batches of ffts to separate gpus?

If in the above the answer is yes, is there any chance that multi-gpu
clffts will be appearing in the next few months?

Is there  cl code that I can copy-n- use/just-study that does batched 1D r2c and c2r
ffts in double precision and for factors 2, 3 & 5 and is suitable for the amd gcn? It
now seems as if I will have to write my own ffts for many-gpus.

Thanks.

Here are my answers:
q1: Yes, the library cannot do multi-GPU by itself. BUT, an application can divide the problem and use multiple GPUs. The sub-problems can be solved with clFFT. People have successfully used clFFT this way in multi-GPU environments.
q2: Not sure about future plans, we can provide an update when the plans are solid.
q3: You can take a look at the examples or client and also read documentation.
https://github.com/clMathLibraries/clFFT/tree/master/src/examples
https://github.com/clMathLibraries/clFFT/tree/master/src/client
http://clmathlibraries.github.io/clFFT/index.html
http://clmathlibraries.github.io/clFFT/index.html#RealFFT


[Enhancement] Method calls equals on an enum instance [EventMeshTcpMessageDispatcher]

Search before asking

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

Enhancement Request

located at:
eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/tcp/client/EventMeshTcpMessageDispatcher.java line 86 92
analysis and explanation:
This method calls the equals(Object) method on an enum instance. Since enums values are singletons, you can use == to safely compare two enum values. In fact, the implementation for Enum.equals does just that.
Describe the solution you'd like
Change to use "==".
Are you willing to submit PR?

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

I want to fix it.


Async component teleport is not rendered in SSR

Version
3.2.6
Reproduction link
https://github.com/frandiox/vue-teleport-ssr-issue
Steps to reproduce

yarn && yarn dev
Open localhost:3000

What is expected?
AsyncComponent teleport element should be rendered and included in ctx.teleports.
What is actually happening?
It is not included in ctx.teleports, but appears in ctx.__teleportBuffers (check server logs).

I was trying to support teleports in vite-ssr library and I found this issue when using async routes in vue-router. Turns out any async component triggers it.
If you remove the async setup from AsyncComponent, it starts working.

This is an issue since August. There is a provided fix since end of December. Is there any chance that the PR will be merged soon?

closed by #5187


client doesn't pass the keys to the connection pool

The connection pool's API is:
def get_connection(self, command_name, *keys, **options):

But the client never passes *keys to the connection pool. Thus, when I wanted to create a sharded connection pool, I wasn't able to do it without inheriting from the client class and passing it. I only passed the first key:
def execute_command(self, *args, **options):

        command_name = args[0]
        key = args[1] if len(args) >1 else ''

        connection = self.connection_pool.get_connection(command_name, key, **options)

I'm not sure this is always correct - please advise. If not passing the key is intentional I will continue with my inherited client, if not I'll gladly provide a patch for this.

Hey Andy. I just remembered this old thing :)
Is it still relevant? Or did you implement it yourself somehow since then?

@andymccurdy Ain't this another issue where the cluster solution would take presidence over a hashring or non clustered solution as was talked in the issue above? If the cluster solution would, then this could probably be closed as no longer valid.

Agreed, cluster solves this much better.


onkey press (enter) not working for some reason

here my code for some reason when i press enter it dont do anything.

$("#msg").emojioneArea({
placeholder: "Enter message",
maxCount: 15,
search: false,
spellcheck: true,
standalone: false,
shortnames: false,
shortcuts: false,
autocomplete: true,
autocorrect: false,
autocapitalize: false,
hideSource: false,
recentEmojis: false,
useInternalCDN: true,
autocompleteTones: false,
filtersPosition: "top",
searchPosition: "bottom",
tones: false,
inline: true,
tonesStyle: "bullet",
hidePickerOnBlur: false,
pickerPosition: "top",
events: {keypress: function (editor, event) {
if (event.which === 13) {
document.getElementById("sendButton").click();
event.preventDefault();
}
}
}
});`

is this a knowing issue as i seen a few posts about it, thanks.

ok it seams to be when i click on the area to send text and type message it and press enter it dont send but if i unclick the area and then reclick the area it will then send the message maybe a highlight bug ???


Using the toast in an Effect causes infinite loop

I am trying to throw a toast when certain data constellations occur. This is handled in an effect.  Unfortunately, calling the toast functionality from within an effect causes infinite loops in the effect.
See this fork:
https://stackblitz.com/edit/vitejs-vite-yy5ned?file=src%2FApp.tsx

I'm not really sure why a toast needs to be created inside an effect 😅 Solid toast updates a solid store when a new toast is created and using that inside an effect creates a cyclic loop. You can opt out of reactive tracking of the toast store by using the untrack keyword. Example shown here https://stackblitz.com/edit/vitejs-vite-nrvxuy?file=src%2FApp.tsx

Alternatively, the on function can be used here to explicitly track your dependencies

Please let me know if you run into any issues other and I would be glad to help out 😁

Thanks for your reply, I tried the untrack approach and it works fine.
Nevertheless, I really wonder why you say calling from an effect is unusual way of doing it. I haven't seen a declarative way to create and style the toast - that's why I have to do it in the effect.
Please look at the following:
 <Toaster position="top-center" >
   <Toast condition={condition1() && condition2()} style="error" configuration={ duration: Infinity }>
</Toaster>

If this were possible, I wouldn't have to imperatively call toast.error(...) in my effect (condition1 and condition2 are true after data has been loaded).
I hope I have been able to clarify the issue.

Haha I apologize for this. I think you're right. Usually, toasts should be called from within event handlers or after a mutation/query completes. But that should not prevent users from using it within an effect too. I have pushed solid-toast v0.4.0 that should fix the infinite loop issue and you wouldn't need to use unwrap now. Let me know if you face any other issues. Hope you have a happy holiday

Here is the example for reference https://stackblitz.com/edit/vitejs-vite-gewldq?file=src%2FApp.tsx


Issue #135: refer Bytebase doc to dev-rel-docs

ref #135

I suggest we get bytebase approved before merging this.

It was already approved by @nmeraihi in the past (Which was the reason behind this PR that got lost through more urgent matters) and we are already self hosting our own instance of the tool.

Please, fix the markdown-check errors.

I did, however,

This will keep on happening until #135 on dev-rel-docs is merged. The rest of the tests are passing!


Error on 12.3 Monterey

AttributeError: 'NoneType' object has no attribute 'group'

@learn2reid Have you tried v3.0.0 of the workflow? It should fix the macOS Monterrey error:
https://github.com/caleb531/open-conference-url/releases/tag/v3.0.0


IOS Build Failed: OSError: [Errno 7] Argument list too long

Hi:
Does anyone have this problem when compiling ios on mac?
Failed to execute link command in link_static_library() function。

Error log:
-----------------------------
Traceback (most recent call last):
  File "./tools/build.py", line 2054, in <module>
    sys.exit(main(sys.argv[1:]))
  File "./tools/build.py", line 2044, in main
    build.package_target()
  File "./tools/build.py", line 1614, in package_target
    libs.append(build.link_static_library())
  File "./tools/build.py", line 1685, in link_static_library
    self.execute(command)
  File "./tools/build.py", line 721, in execute
    res = self.execute_with_error(command, env=env, cwd=cwd)
  File "./tools/build.py", line 715, in execute_with_error
    job = subprocess.Popen(command, env=env, cwd=cwd)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 710, in __init__
    errread, errwrite)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/subprocess.py", line 1335, in _execute_child
    raise child_exception
OSError: [Errno 7] Argument list too long
------------------------
Thanks!

Hello. Can I know build script on your terminal?
That error message was arise when you have typo in build command.
Thanks.

I am using the stellite build.py script to compile the latest version of the chromium code.
Maybe I have found the problem, because the ARG_MAX of the mac system is too small, I try to modify the cammand of the IOSBuild(): link_static_library() function seems to be fine.
The previous cammand is:
libtool -static  *.o  -arch_only armv7 -o /outdir/out_ios_arm/libxxx_arm.a
Change to:
libtool -static -arch_only armv7 -o /outdir/out_ios_arm/libxxx_arm.a -filelist /outdir/libtest.a.rsp
(Key point: Write all *.o files to a file first)
Thanks!


Add integration tests(e2e)

For integration testing it's good to cover several points:

Docker image
RPC calls to node
Contract deploy and invocation
Node connection to other node (p2p)

Thoughts:

Integration tests should be run as a separate command https://stackoverflow.com/questions/25965584/separating-unit-tests-and-integration-tests-in-go
I found approach with naming all integration tests like 'server_integration_test' and having them run with special tag 'integration'.
For running other nodes and checking docker image testcontainers library could be used

I think we have everything from this list now covered either by unit tests or neo-bench (which does a lot of things now).


Navigation component

Create a Navigation component as described in #1873

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test


chore: bump dagger patch version from 0.2.11 to 0.2.19 in docs

Signed-off-by: Jeremy Adams<EMAIL_ADDRESS>

SGTM
I have a comment about:
./pkg/pkg.go
34:		DaggerModule:   gv.Must(gv.NewVersion("0.2.11")),
35-		UniverseModule: gv.Must(gv.NewVersion("0.2.9")),

Did we do any breaking changes since 0.2.9/0.2.11 that would push us to update these values?

I asked same thing here and was told that we didn't need a change: https://discord.com/channels/707636530424053791/796905486145683506/986793776251019284

SGTM
I have a comment about:
./pkg/pkg.go
34:		DaggerModule:   gv.Must(gv.NewVersion("0.2.11")),
35-		UniverseModule: gv.Must(gv.NewVersion("0.2.9")),

Did we do any breaking changes since 0.2.9/0.2.11 that would push us to update these values?

These "should" be ok since they're supposed to be modified when someone upgrades packages. When bumping docs, we shouldn't have to worry about package versions.


Chrome omnibox support

The new Kitsu search is great, and a feature I use on heaps of other sites is chrome's omnibox
https://developer.chrome.com/extensions/omnibox
allowing the user to search anime/mange with this would be great.

I definitely support this. Omnibox is awesome!
We'd probably need a full-page search results template for this, though. Unless we want to force people to write anime:..or something similar for deciding which site to land on if they didn't select a search result right away.

yea +1 for search results page, It could also allow refining results by more parameters which is useful in the case of where people cant remember the title.

I'm not a fan of a dedicated search page as we already have full search capability for them on their own browse page with advanced filtering.

It is possible for what the user puts into the omnibox takes the user to the most relevant page, which is what many anime sites like kissanime do

Maybe you could have a url param that just opens the search dialog with your query. Seems weird but better than nothing i guess

possible, or take the user to the kitsu dashboard and instead of using the existing search box, have a modal open with the search bar and results in.

You could also reserve the Anime and Manga keywords and have it work like this:


Add new exercise strain

As explained in #90

@rchavarria if you have time to check these out and merge them that would be great,  if not I can do it.  Thanks!

I'm on the way of merging your recent contributions. I've been busy this weekend.
BTW, thanks a lot for so many new exercises. I'll merge them bit by bit.

No problem.  I had some time and was happy to do it.  Thanks!


Fix multi column joins

Fixes zbrookle/dataframe_sql#36

Codecov Report

Merging #38 into master will decrease coverage by 0.95%.
The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   88.10%   87.14%   -0.96%
==========================================
  Files          10       10
  Lines        1219     1206      -13
  Branches      234      232       -2
==========================================
- Hits         1074     1051      -23
- Misses        104      119      +15
+ Partials       41       36       -5

Impacted Files
Coverage Δ

sql_to_ibis/parsing/transformers.py
84.66% <ø> (-0.09%)
:arrow_down:

sql_to_ibis/sql/sql_value_objects.py
85.21% <64.28%> (-1.55%)
:arrow_down:

sql_to_ibis/parsing/sql_parser.py
86.44% <100.00%> (-2.05%)
:arrow_down:

Continue to review full report at Codecov.

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


added AI Trading Simulator

Description
Added AI Trading Simulator
Fixes #884
Type of change

[x] Added a new machine learning frameworks, libraries or software.
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Please review my pull request @sanjay-kv


"stage 4 / action making the conversation" should probably be "... converting" or "... doing the conversion"

Thank you for pointing it out.
I've fixed the typo in #127


Add support for incrementing and decrementing a gauge

The statsd spec provides for incrementing and decrementing a gauge just as can be done with a counter. This allows independent parts of an application to report the change they are making to the gauge without having to synchronise or coordinate with each other.
The gauge takes either a number (say 123.45) or a delta (such as -1 or +7).

I'm unlikely to merge a PR that adds this.
I don't think gauge deltas make sense because of the ambiguity they introduce: -1 was already a valid value to set a gauge to but with deltas it now means something different. The fact that you can no longer set gauges to negative values is noted in the docs as a limitation. The suggested work around is to set a gauge to zero first. This isn't a particularly satisfying answer because it's perfectly reasonable to send a single metric per UDP packet and UDP packets can be reordered, leading to different values on the server side.
Another reason I'm unlikely to add support for this is because DataDog clients don't appear to support it and some Cadence users use it with DataDog.

Those are great points. I agree that it's likely to cause more trouble than its worth, so I'll close this out. Thanks for taking the time to give such a detailed response! 😊


[UI] Node Templates - Cloned Unusable

What kind of request is this (question/bug/enhancement/feature request): bug
Related: https://github.com/rancher/rancher/issues/26575
Steps to reproduce (least amount of steps as possible):
Unable to Clone a Cloned Template

Clone any existing Node Template
Attempt to use it in any cluster.

Result:
Permission denied or 404 error
Other details that may be helpful:
It appears that with the change to make node templates global or access by multiple users some underlying logic has broken with the use of any clone template.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.3.3
Installation option (single install/HA): HA

It appears that templates prior to the change that moved the templates to the cattle global data are affected.
For example, if you had UserA (not Global Admin) that created Template1 BEFORE the global data migration for node templates, and then UserA clones Template1 to Template2 in 2.3.3 it Template2 will be unavailable to UserA period.
New Templates created by UserA are ok.

When a user goes to clone the nodetemplate it is also copying the annotations over to the new template which is what rancher uses to know whether or not it has made rolebinding for the creator.
Annotations should not be copied, just the non-meta fields of the nodetemplate.

Reproduced the issue in v2.3.3 cloned templates has the annotations copied and trying to use the cloned templated returns an error as described using templates created in v2.3.2
Validated this is no longer the issue in v2.3-head (05/22/2020) commit a621912b1
I will check 2.4-head and master-head next.

Validated the fix also in v2.4-head (05/22/2020) commit d8f101974
Cloned templates after upgrading from v2.3.2 works for creating clusters. Cloned templates with extra annotations, the annotations got removed in the cloned template.
Issue #27227 opened because the UI gets unresponsive after upgrading to master-head


Improve dockerfile / fix file permission issue

Description
Improves dockerfile for the agent backend.
I had issues to open files, which are created by the agent.
The docker image ran as root user,
so it created files with root permission.
Motivation and Context
How has this been tested?

Screenshots (if appropriate)
Types of changes

Changes visible to users:

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

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[ ] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] My change works!

Thanks!


triangle: incorrect test in some tracks

Please check if there's a test that states that a triangle with sides 2, 4, 2 is invalid. The triangle inequality states that for any triangle, the sum of the lengths of any two sides must be greater than or equal to the length of the remaining side. If this doesn't affect this track, go ahead and just close the issue.

def test_triangles_violating_triangle_inequality_are_illegal_2(self):
    self.assertRaises(TriangleError, Triangle, 2, 4, 2)

I'm not totally sure what to do. Should the test case just be removed or should we test it as "isosceles" instead?

Let's test in as isosceles--it's a good edge case to have, I think.

We should wait until this discussion is settled: https://github.com/exercism/xerlang/issues/85

Agreed.

So my understanding of the erlang issue is that while 2, 4, 2 is a special triangle (degenerated - basically a line) we will treat it as not a triangle (has no area) to keep this exercise simple. So maybe the x-common description needs to be updated but the python test case can stay as it is.

There was a bit of confusion with this issue. I've summarized my current understanding here: https://github.com/exercism/x-common/issues/202
I am closing this issue. If necessary I will open a new issue once we have come to an agreement about how to resolve the inconsistencies and what exact steps to take.


update unsintall controller to cleanup share manager crds (#2384)

Proposed Changes
update unsinstall controller to cleanup share manager crds to resolve ticket '[BUG] uninstall-controller does not cleanup share-manager crds' (#2384)

add clean up for share-manager and settings

Types of Changes
Bug
Verification

deploy longhorn
deploy a rxw volume/pod
uninstall using uninstaller from "https://raw.githubusercontent.com/longhorn/longhorn-manager/master/deploy/uninstall/uninstall.yaml"
check log for uninstaller

time="2021-04-01T03:07:55Z" level=warning msg="Volume in use" accessMode=rwx controller=longhorn-uninstall frontend=blockdev migratable=false owner=centos-master-0 shareEndpoint="nfs://<IP_ADDRESS>/pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4" shareState=running state=attached volume=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4
time="2021-04-01T03:07:55Z" level=info msg=Uninstalling... controller=longhorn-uninstall
time="2021-04-01T03:07:55Z" level=info msg="Found 1 share managers remaining" controller=longhorn-uninstall
time="2021-04-01T03:07:55Z" level=info msg="Marked for deletion" controller=longhorn-uninstall owner=centos-master-0 shareManager=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4 state=running volume=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4
time="2021-04-01T03:07:55Z" level=info msg="Found 1 share managers remaining" controller=longhorn-uninstall
time="2021-04-01T03:07:55Z" level=info msg="Marked for deletion" controller=longhorn-uninstall owner=centos-worker-1 shareManager=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4 state=unknown volume=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4
time="2021-04-01T03:07:56Z" level=info msg="Found 1 share managers remaining" controller=longhorn-uninstall
time="2021-04-01T03:07:56Z" level=info msg="Found 1 share managers remaining" controller=longhorn-uninstall
time="2021-04-01T03:07:56Z" level=info msg="Marked for deletion" controller=longhorn-uninstall owner=centos-worker-2 shareManager=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4 state=error volume=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4
time="2021-04-01T03:07:56Z" level=info msg="Found 1 volumes remaining" controller=longhorn-uninstall
time="2021-04-01T03:07:56Z" level=info msg="Marked for deletion" accessMode=rwx controller=longhorn-uninstall frontend=blockdev migratable=false owner=centos-master-0 shareEndpoint= shareState=unknown state=detaching volume=pvc-833a1af2-2a70-4bd7-8723-4ce4c43955b4
time="2021-04-01T03:07:56Z" level=info msg="Found 1 share managers remaining" controller=longhorn-uninstall

Linked Issues
Refs: # https://github.com/longhorn/longhorn/issues/2384
Further Comments
None
Signed-off-by: Clark Hsu<EMAIL_ADDRESS>

I have updated according to the reviews and updated the verification steps.


Human readable repository state

Production root-signing relies on documentation that's manually kept up-to-date to list who the root signers are, what delegations exists, which artifacts the repository contains etc. This has significant downsides and I'd like to avoid this manual work.
It's completely reasonable to expect the tooling to automatically describe important repository details in a way that is always up-to-date and readable in a browser. Some options:

javascript solution that dynamically reads metadata and produces html that describes the repo (this requires no build steps but does require someone with js skills) OR
a doc-build step in tuf-on-ci-build-repository where some static docs (markdown or html) are produced and then published alongside the repository

as previous (node/svelte) work there is https://github.com/DataDog/tuf-explorer
with the markdown approach we could even produce a mermaid diagram with delegations and signers displayed

There is a generic issue about this in theupdateframework/tuf-on-ci#39 but it might be more useful to solve the sigstore use case and then see if it's general enough or no...

I have a rough POC with markdown in
https://github.com/jku/tuf-on-ci/commit/f1374d0a3575df63f7fa98babda67310575d8489. I think I'd prefer HTML but this was so quick to write that I did it. It currently produces the following content for this repository (apologies to signers for unnecessary pings):

Signers

Role
required # of signatures
Signers

root
2
@jku, @kommendorkapten, @joshuagl, @mnm678

timestamp
1
online key

snapshot
1
online key

targets
1
@jku, @kommendorkapten, @joshuagl, @mnm678

registry.npmjs.org
1
@jku

possible downsides:

this will likely require enabling markdown on GitHub Pages (I think adding a _config.yml file to git is enough)
There's no builtin markdown rendering on GCS  so this is really only useful for  https://sigstore.github.io/root-signing-staging/

this will likely require enabling markdown on GitHub Pages (I think adding a _config.yml file to git is enough)

I was not  able to do this. Maybe it's possible but I can't figure it out

It's not clear which step actually runs jekyll: maybe it will only happen for data committed to git (I would rather avoid committing redundant files) and not to data I upload to Pages
it's also possible that I just don't know what the configuration should look like -- there are no example projects

I think GH pages requires pushing to git, but it doesn't need to be your main branch or even a branch with similar contents to main. IIRC default configuration is to have a gh-pages branch, or similar, where the contents for the pages are stored.

I think GH pages requires pushing to git, but it doesn't need to be your main branch or even a branch with similar contents to main. IIRC default configuration is to have a gh-pages branch, or similar, where the contents for the pages are stored.

tuf-on-ci uses requires the "Github Actions" as source instead of "Deploy from branch" (and I won't change that, the branch method is just insane). Your comment did make me try a fresh project from scratch and the proposed template does this before uploading to pages:
        uses: actions/jekyll-build-pages@v1
        with:
          source: ./
          destination: ./_site

that looks promising. Would be nice to get this as documentation instead of having to create new projects to find out but this works...

That actually pretty much worked: https://jku.github.io/tuf-on-ci-sigstore-test/metadata/

The downside of "rendering" on the server is that the description cannot say things like "signing event starts in 2 days" - a javascript solution could.

This is what it currently looks like

> ## Signers
>
> | Role | Next signing | Signers |
> | - | - | - |
> | root | [Starts 2024-05-22](## '2024-05-22 12:37:39 - 2024-06-26 12:37:39') | 2 of @jku, @kommendorkapten, @joshuagl, @mnm678 |
> | timestamp | [Starts 2024-04-28](## '2024-04-28 07:20:45 - 2024-05-02 07:20:45') | 1 of _online key_ |
> | snapshot | [Starts 2033-03-26](## '2033-03-26 11:49:10 - 2034-03-26 11:49:10') | 1 of _online key_ |
> | targets | [Starts 2024-05-22](## '2024-05-22 12:38:57 - 2024-06-26 12:38:57') | 1 of @jku, @kommendorkapten, @joshuagl, @mnm678 |
> | registry.npmjs.org | [Starts 2033-01-06](## '2033-01-06 08:54:04 - 2034-01-06 08:54:04') | 1 of @jku |
>

Initial version is in tuf-on-ci now, we should get it in the next release

First version of this is now live:

preprod (GH pages) staging: https://sigstore.github.io/root-signing-staging/
staging https://tuf-repo-cdn.sigstage.dev/

Two issues remain the actual sigstage repository:

CSS styling is no applied
the json links are incorrect

both have the same underlying reason: jekyll writes links as absolute so changing the "root" location breaks the links.
A workaround for the json links has been merged in tuf-on-ci.

I'll modify the README to link to the description and will close this issue.


created redux for searchbar

Updating the Searchbar component to allow redux to handle the state.

 move retrieval and filtering of mock search data into redux


Refactor theme

Separate theme components, layouts, and hooks into directories
Allow shared to be imported with a relative path in content
Remove unused components and their dependencies
Remove one-off deps in favor of inline solutions
Remove edit links from index pages without any content

9

hard

/


Failed to convert ResponseEntity object

Hello, when invoking a remote method that returns a "responseEntity" object, the following error occurs:
java.io.IOException: Incomplete output stream
	at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1559)
	at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1528)
	at java.base/java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:527)
	at feign.Client$Default.convertResponse(Client.java:108)
	at feign.Client$Default.execute(Client.java:104)
	at org.springframework.cloud.openfeign.loadbalancer.LoadBalancerUtils.executeWithLoadBalancerLifecycleProcessing(LoadBalancerUtils.java:56)
	at org.springframework.cloud.openfeign.loadbalancer.RetryableFeignBlockingLoadBalancerClient.lambda$execute$2(RetryableFeignBlockingLoadBalancerClient.java:156)
	at org.springframework.retry.support.RetryTemplate.doExecute(RetryTemplate.java:329)
	at org.springframework.retry.support.RetryTemplate.execute(RetryTemplate.java:225)
	at org.springframework.cloud.openfeign.loadbalancer.RetryableFeignBlockingLoadBalancerClient.execute(RetryableFeignBlockingLoadBalancerClient.java:103)
	at feign.SynchronousMethodHandler.executeAndDecode(SynchronousMethodHandler.java:119)
	at feign.SynchronousMethodHandler.invoke(SynchronousMethodHandler.java:89)
	at feign.ReflectiveFeign$FeignInvocationHandler.invoke(ReflectiveFeign.java:100)

The spring cloud feign version used is 2020.0.5

Hello, @eduardosilvaneto, thanks for submitting the issue.
Please learn how to properly format code and logs.
It does not seem that the issue is in SC OpenFeign - there's problem processing the input stream with Java methods, the response might be corrupt. In any case, I can take a closer look at this for you, anyway, but in order to get this done, please, upgrade to 2021.0.0 (2020.0.x is no longer supported) and provide a minimal, complete, verifiable example that reproduces the issue.

Thank you very much for your feedback. I really had something corrupting the file, maybe my VPN. Performing the test with the same code worked.

Thank you very much for your feedback. I really had something corrupting the file, maybe my VPN. Performing the test with the same code worked.

Thank you very much for your feedback. I really had something corrupting the file, maybe my VPN. Performing the test with the same code worked.


How to use tableTools Extras

Trying to use tableTools extra, so follosing the How to use Extras instructions, I am declaring
//= require dataTables/extras/dataTables.tableTools

in application.js and
*= require dataTables/extras/dataTables.tableTools

in application.css
However after initializing
jQuery ->
  $('.dataTable').dataTable({
    dom: 'T<"clear">lfrtip',
    tableTools: {sSwfPath: '/swf/copy_csv_xls_pdf.swf'},
    autoWidth: true,
    processing: true,
    pagingType: 'full_numbers'
  })
  return

buttons are available with no functionality. Is the path wrong? Am I missing a declaration? This only works if physically copying the swf files in public/swf, so the point of using the gem is somehow lost.
I'll appreciate your comments on this.

I was able to solve my problem arranging the order of js loading in application.js .
The following order now works for me.
//= require dataTables/jquery.dataTables
//= require dataTables/extras/dataTables.tableTools
//= require dataTables/bootstrap/3/jquery.dataTables.bootstrap


Question: Multiple element queries. What am I doing wrong

I'm figuring out how to use eqcss and I'm having a problem.
In https://gist.github.com/caraya/29754c23543bde2d1c7b08d6b1eee2a5 I created to element queries, one with minwidthpixel and the other one with maxwidthpercents. The first one works as intended but the second one doesn't.
I'm pretty sure I'm doing something wrong but can't figure out what it is.  Can anyone point me out to where the mistake is?
Thanks

Hi Caraya! So what's going on here is actually that the code is running exactly as written, but that may not be what you're looking for!
So in your demo, you've got two div elements, one with class="minwidthpixels" and one with class=maxwidthpercent. Both of these elements are 100% wide (due to div being a block level element) so as long as the page is wider than 500px, the (min-width: 500px) element query will apply.
Now for the (max-width: 50%), that will apply any time the element is less than 50% the width of its container. Since div is block, it's 100% wide all the time (in other words, too wide for the demo. In order for that style to apply as-written, you'll need to constrain the element to less than 50% of its parent container's width. Or else you could flip that over to min-width: 50% and it would apply all the time (since the element is 100% wide.
Also just as a side-note, you can use <script type=text/eqcss> or <style> to write EQCSS-enhanced CSS, but you don't need the type=text/eqcss on <style> tags. You can also include EQCSS in any external CSS stylesheets linked into a page where EQCSS.js is loaded :)
Here's an updated pen with the second div constrained to 49% wide, so the query applies at all times: http://codepen.io/tomhodgins/pen/eZjKva

Got it. Thanks for the clarification

Happy to help! Feel free to ask about anything if you have any more questions or ideas as you use EQCSS 😊