include 'Hello World' Skill in samples listing

adds "Hello World" example to the skills listing in readme
I also switched the invocation name to "Hello World" instead of "Greeter", since single-word invocation names are no longer allowed: https://developer.amazon.com/public/community/post/Tx1I0GCST7POI00/Tips-for-Choosing-an-Invocation-Name-for-Your-Alexa-Custom-Skill (see item 2 on the "New Invocation Name Requirements" list)

Hi mutexkid,
It looks like there are some merge conflicts, could you resolve them and resubmit?

@deegles re-upped - thanks!


[Bug fix] Return parameter data with same type as before applying scale or offset

At the moment, on applying scale or offset, parameter data return type changes to tuple if the original data type was any iterable data type. Desired behaviour is to return data with same type as before applying scale or offset.
This PR fixes it.
Changes proposed in this pull request:

Update scaling and offset logic

@jenshnielsen @astafan8

@astafan8 I have added a test for setting numpy valued param. Please have a look when time allows.


Is there a way to remove a "boat" icon after the time has completed for it showing only the line or track points?

I'm testing out your library, but what I have found is if I put a few years worth of data into it the older "complete" arrays of lat and lng still show an icon like they are still moving.
Is there anyway that if the leaflet program has passed the last "time" of the array of the individual item the animated marker can go away?
Example:
	var data = [[ {lat:30, lng:-117, time:1500000000, dir:0, info:[]},  											  {lat:31, lng:-116, time:1700000000, dir:0, info:[]}], 											[ {lat:30, lng:-118, time:1600000000, dir:0, info:[]},  											  {lat:31, lng:-116, time:1800000000, dir:0, info:[]}]]	;
For this example how do I make the icon for the first array of data go away (but keep trackLine and/or trackPoint) at time:<PHONE_NUMBER> ?

Any followup on this one?


doc: Copy full contents of doc folder

Fixes #9128

cc @ru-fu

Codecov Report

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

@@           Coverage Diff           @@
##           master    #9307   +/-   ##
=======================================
  Coverage   52.48%   52.48%
=======================================
  Files         201      201
  Lines       25310    25310
  Branches     5285     5285
=======================================
  Hits        13284    13284
  Misses       9941     9941
  Partials     2085     2085

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 3e27187...601c051. Read the comment docs.


[next] CLI options cannot be passed with new command

statinamic start --devPort=4000 throw an error. commander try to validate all option, but we already use minimist, later in the process.
We probably need to ditch commander.

statinamic start -- --devPort=4000 ?

No -- is a npm specitic ting. Commander try to validate all option, and I only registered --config. Will need a good cleanup between commands and API.

yargs is a good tools for this job.

We already use minimst

How about add all available options to commander ?

I will move options definition in an file so we can use in commander and in minimist too (if minimist is still relevant).

Btw, I will add this https://github.com/kentcdodds/webpack-validator


Get MentalCommand Data

Hey,

I want to use the mental commands for my application using Unity... I subscribe to sys  and com data streams and then I load my profile.... But how am I able to retrieve the current mental commands? I can't find a  method in DataStreamManager.... I know that the current command is printed in OnMentalCommandReceived but how can I actually get to the commands?

The commands that I get seem very random.... Usually when I start the EmotivBCI  App, it sometimes asks me to relax (I guess to get my current neutral mental command) with my eyes open and then with my eyes closed... Is there a way in Unity to do the same? And could it be that when I get the mental commands in unity that it doesn't actually use the profile as its basis?

Hi @floriangamillscheg ,
Thanks for your patience.

Yes, after subscribing the data and load profile. You can see the current mental commands. But you must load a training profile that contains training data for at least one action. Please reference the "Getting live actions" flow at link
In the OnMentalCommandReceived  function, we have parsing the data . You can uncomment the lines
// TODO: emit event to other modules MentalCommandReceived(this, comEvent);
To emit the event to BCITraining example. Currently, In BCITraining.cs we haven't example for live modes.  But you can do similar to other event. We will update the example for training later
Actually we do not need to add eye open and eye closed in your application. But i should relax before each training, it might help the training result better.


Replace CompositePropertySource with one PropertySource per file

See https://github.com/spring-projects/spring-boot/issues/18174 for background and a sample application.
Raising an issue here as requested by @spencergibb

Any chance of someone looking into this soon? I could take a stab at it if not but haven't worked in this code base before so it could take some time to get my bearings.

@ewiegs4  if you want to take a shot at fixing the issue that would be great!  @TYsewyn can help if you have any questions.  Since the problem was in Greenwich as well, make sure you submit any PRs against the 2.1.x branch.

@philwebb can you point us to how this behavior is implemented in Boot?  We have multiple property sources due to the profile being enabled.  We would want to combine them into a composite, but how come CompositeProperty source does not implement the expected list behavior (overriding the list property)?

The IndexedElementsBinder class is the main area to look. Specifically, the bindIndexed method limits itself to one source at a time and returns as soon as binding works.
In Spring Boot 1.x we found that binding lists with profile specific files was quite confusing because it wasn't always clear when elements would be overridden (see this comment for an example).
In ConfigFileApplicationListener we register one PropertySource per file (see the addLoadedPropertySources method) which means they are adapted directly to individual ConfigurationPropertySource instances used by the Binder.
The Binder doesn't treat the CompositePropertySource in any special way, it's just a regular PropertySource as far as it's concerned. Because of this, the indexed binder will consider it as a single source and the early exit rule isn't being applied.

I have confirmed that the same problem happens with just a normal config client (as opposed to using spring.cloud.config.server.bootstrap=true).
I am not really sure how we can solve this problem without introducing a breaking API change.  The problem I believe lies in the PropertSourceLocator interface, who's locate method returns a single PropertySource.  Since we may have multiple sources (ie multiple configuration files) we have to return a CompositePropertySource in our implementations.
I guess ideally we have a method that returns a List of PropertySources and we can add each one separately to the environment.  We could add a default method to the API which does this and just have the default implementation call the existing locate method.  Then override the default implementation in our PropertySourceLocator implementations.

Can you unwrap any composite results in PropertySourceBootstrapConfiguration?

Possibly, but that is just adding them to another CompositePropertySource, so whether it is a composite or a flat composite I don't think it will make a difference in the end.

Maybe instead of taking in a composite in insertPropertySources we could again take in a collection of property sources and insert them?


The installation script is stuck.

I am trying to install Tapa on an Ubuntu 20 server according to the documentation, but the process gets stuck when executing the "bash install.sh" script.
Setting up xrt (2.14.354) ...
Unloading old XRT Linux kernel modules
Unregistering old XRT Linux kernel module sources 2.14.354 from dkms

-------- Uninstall Beginning --------
Module:  xrt
Version: 2.14.354
Kernel:  5.4.0 (x86_64)
-------------------------------------

Status: Before uninstall, this module version was ACTIVE on this kernel.

xocl.ko:
 - Uninstallation
   - Deleting from: /lib/modules/5.4.0/updates/dkms/
 - Original module
   - No original module was found for this module on this kernel.
   - Use the dkms install command to reinstall any previous module version.

xclmgmt.ko:
 - Uninstallation
   - Deleting from: /lib/modules/5.4.0/updates/dkms/
 - Original module
   - No original module was found for this module on this kernel.
   - Use the dkms install command to reinstall any previous module version.

depmod...

DKMS: uninstall completed.

-------- Uninstall Beginning --------
Module:  xrt
Version: 2.14.354
Kernel:  5.4.195 (x86_64)
-------------------------------------

Status: Before uninstall, this module version was ACTIVE on this kernel.

xocl.ko:
 - Uninstallation
   - Deleting from: /lib/modules/5.4.195/updates/dkms/
 - Original module
   - No original module was found for this module on this kernel.
   - Use the dkms install command to reinstall any previous module version.

xclmgmt.ko:
 - Uninstallation
   - Deleting from: /lib/modules/5.4.195/updates/dkms/
 - Original module
   - No original module was found for this module on this kernel.
   - Use the dkms install command to reinstall any previous module version.

depmod...

DKMS: uninstall completed.

------------------------------
Deleting module version: 2.14.354
completely from the DKMS tree.
------------------------------
Done.
Invoking DKMS common.postinst for xrt
Loading new xrt-2.14.354 DKMS files...
Building for 5.4.0 5.4.195
Building initial module for 5.4.0
Secure Boot not enabled on this system.
Done.

xocl.ko:
Running module version sanity check.
 - Original module
   - No original module exists within this kernel
 - Installation
   - Installing to /lib/modules/5.4.0/updates/dkms/

xclmgmt.ko:
Running module version sanity check.
 - Original module
   - No original module exists within this kernel
 - Installation
   - Installing to /lib/modules/5.4.0/updates/dkms/

depmod...

DKMS: install completed.
Building initial module for 5.4.195
Secure Boot not enabled on this system.
Done.

xocl.ko:
Running module version sanity check.
 - Original module
   - No original module exists within this kernel
 - Installation
   - Installing to /lib/modules/5.4.195/updates/dkms/

xclmgmt.ko:
Running module version sanity check.
 - Original module
   - No original module exists within this kernel
 - Installation
   - Installing to /lib/modules/5.4.195/updates/dkms/

depmod...

DKMS: install completed.
Finished DKMS common.postinst
Loading new XRT Linux kernel modules

Can someone provide some help?

@lifeformg Can you check if you can run sudo apt-get install -y xrt --no-install-recommends successfully? The log you show is from this command.

@lifeformg Can you check if you can run sudo apt-get install -y xrt --no-install-recommends successfully? The log you show is from this command.

Yes, I also discovered later that it was due to a stuck installation process of XRT. Running sudo apt-get install -y xrt --no-install-recommends also gets stuck at Loading new XRT Linux kernel modules. It seems that this is not a problem with TAPA (so I will close this issue later). But do you have any leads on the issue with XRT installation?

But do you have any leads on the issue with XRT installation?

No idea. I've never seen similar issues before 😅


x/net/trace: export http.Handlers

What version of Go are you using (go version)?
go version go1.8.1 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/peter"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.8.1/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.8.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/m1/g2lm29h90y1dw2gllcb2mk200000gn/T/go-build364343606=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

What did you do?
My program exposes an admin HTTP API, for a variety of internal routes, as well as pprof and expvar. For this it uses its own http.ServeMux (actually a gorilla/mux.Router, but it's irrelevant) into which I register routes explicitly:
r := mux.NewRouter()
r.Handle("/debug/vars", expvar.Handler())
r.HandleFunc("/debug/pprof/", pprof.Index)
r.HandleFunc("/debug/pprof/cmdline", pprof.Cmdline)
r.HandleFunc("/debug/pprof/profile", pprof.Profile)
r.HandleFunc("/debug/pprof/symbol", pprof.Symbol)
r.HandleFunc("/debug/pprof/trace", pprof.Trace)
r.Handle("/debug/pprof/block", pprof.Handler("block"))
r.Handle("/debug/pprof/goroutine", pprof.Handler("goroutine"))
r.Handle("/debug/pprof/heap", pprof.Handler("heap"))
r.Handle("/debug/pprof/threadcreate", pprof.Handler("threadcreate"))
myFoo.Register(r.PathPrefix("/admin/foo/").Subrouter())
myBar.Register(r.PathPrefix("/admin/bar/").Subrouter())
myBaz.Register(r.PathPrefix("/admin/baz/").Subrouter())
r.Handle("/", rootHandler(r))
return http.Serve(adminListener, r)

I'm now wanting to expose the x/net/trace handlers (specifically /debug/events, for go-conntrack).
What did you expect to see?
I expected to be able to do something like
r.HandleFunc("/debug/requests", trace.Requests)
r.HandleFunc("/debug/events", trace.Events)

What did you see instead?
No such Handler functions are exposed by the trace package. Instead, the trace package assumes its users will always want it to register its routes into the http.DefaultServeMux, and performs those registrations as an import side effect, as part of package init.
Please expose the code in func init as individual handlers, so users who don't use the DefaultServeMux can still have access to x/net/trace functionality.

@rakyll @hyangah Seems fine to me, thoughts?  Are there implications on other /debug/ handlers?  net/http/pprof seems to allow this.

Furthermore, having init() register handlers causes multiple registration issues (panic below) when some of the code uses x/net/trace from $GOPATH/src/golang.org/x/net/trace, and other parts of the code use it from vendor/golang.org/x/net/trace. Both imports run their init, and since they are separate packages due to their import path, the program panics:
panic: http: multiple registrations for /debug/requests

goroutine 1 [running]:
net/http.(*ServeMux).Handle(0x18a7780, 0x15d8e89, 0xf, 0x185a8e0, 0x15f0948)
	/usr/local/Cellar/go/1.8.3/libexec/src/net/http/server.go:2254 +0x610
net/http.(*ServeMux).HandleFunc(0x18a7780, 0x15d8e89, 0xf, 0x15f0948)
	/usr/local/Cellar/go/1.8.3/libexec/src/net/http/server.go:2286 +0x55
net/http.HandleFunc(0x15d8e89, 0xf, 0x15f0948)
	/usr/local/Cellar/go/1.8.3/libexec/src/net/http/server.go:2298 +0x4b
github.com/ahmetb/gcb-dashboard/vendor/golang.org/x/net/trace.init.1()
	/Users/ahmetb/workspace/gopath-gcb-dashboard/src/github.com/ahmetb/gcb-dashboard/vendor/golang.org/x/net/trace/trace.go:121 +0x42
github.com/ahmetb/gcb-dashboard/vendor/golang.org/x/net/trace.init()
	/Users/ahmetb/workspace/gopath-gcb-dashboard/src/github.com/ahmetb/gcb-dashboard/vendor/golang.org/x/net/trace/trace_go17.go:22 +0x1cd
github.com/ahmetb/gcb-dashboard/vendor/google.golang.org/grpc.init()
...

@zombiezen, I'd like to survey all global registrations and export handlers where we don't. I am strongly +1ing to export handlers from x/net/trace.
@ahmetb, are you vendoring in libraries? Unfortunately, libraries vendoring their dependencies is not a properly supported feature, we suggest only the main packages should vendor dependencies. I'd love to avoid global registration on the default mux, but these inits have been put in place in the past and I am not sure if we can break the behavior -- especially the standard library's.

Note that this bug is for golang.org/x/net/trace handlers, not the net/http/pprof ones.  No strict compatibility promise, but silent breakage would be bad.

Closed via https://github.com/golang/net/commit/1816238e220c6657276d3bc6118dc95b3eb21e71.

considering a popular golang library like this; it's most likely to get imported by multiple different packages.
so, i think it should allow us to handle errors safely.
i just can't understand the point of making panicable calls in the very init func?
please, at least, consider wrapping it up with sync once or something-else fits the idiomatic way of go.


[Feature Request]: text extraction from diagrams/graphs

Feature Description
Examples:

https://huggingface.co/google/deplot (only tabular data)
https://huggingface.co/google/pix2struct-infographics-vqa-base

Larger workflow: Text extraction pipeline for non-textual documents e.g.

image using ViT (blip/blip2): https://llama-hub-ui.vercel.app/l/file-image_blip2

Does this feature belong to llama hub instead of llama_index?
Next levels - how can we integrate this into a PDF document extraction / website extraction workflow? (text + img)
Caveat

Before further work, let's test out if Blip2 can handle technical diagrams or graphs (my guess is no)
This work will not cover direct VQA. In the direct VQA pipeline, there may need to exist a refactor or separate layer to handle various types of mixed-media retrieval. From a feasibility standpoint, existing text captions and autocaptioning of diagrams/graphs can serve as the text-based retrieval embedding for mixed media. The retrieved media can then be fed into the downstream VQA/multi-modal LLM pipeline. It is an open question whether descriptive or tabular data extraction from diagrams/graphs/natural images is sufficient or whether direct VQA is necessary for good performance on task. I believe this depends on the use-case.

Reason
No response
Value of Feature
Increase the amount of information extracted from mixed-media documents.
I think they (and Blip2 models) should be part of the PDF/website workflows with automatic dispatch, preprocessing and model selection/ensembling layer, but some coordination and maintenance on the best performing models would be valuable.
Some filtering layer to filter out low-quality outputs from the vision models could be helpful as well.

good points all around. Some quick responses to your questions:

RE where to put this: probably fits better in llama_hub right now. Would be amazing if you want to make a contribution there.
RE how to integrate into PDF/website extraction: this takes some refactoring right now, current workflow is parse PDF/website in to plain text documents, then parse documents into Nodes. We are moving towards keeping more raw information, and then parsing document into multi-media Nodes (e.g. image node, table node, etc)
RE beyond text based retrieval: we are moving towards supporting joint image/text embeddings (e.g. Google Palm API) to do native cross-modal retrieval.
RE value of feature: 100% agreed. If you are interested in help making this happen, let's chat more!

Actually, I think what I did was not very useful.
In the typical use-case, an image caption is equivalent to the image content. In the graph case, the description of the graph is what we want for the embedding, but the content should be the graph data.

Another interesting direction is extraction from diagrams graphs -> inserting into SQL database.
We may expect varied results here, especially since the source attribution is lost. Not my favourite pathway but combining the SQL database with original sources seems decent solution for some kinds of queries.

Definitely believe that to effectively make use of the charts/diagrams, the multi-step query engine would be key.


Merge branch v1.9-branch of github.com:opendatahub-io/kubeflow into sync-ds

Inspired by @jiridanek in #125 I created this by this link:
https://github.com/red-hat-data-services/kubeflow/compare/master...opendatahub-io:kubeflow:v1.9-branch

/lgtm


[Caching] Disable some tests on linux as they failed on aarch64-linux

The error is known due to  rdar://119964830

@swift-ci please smoke test


Try running PHP unit tests in PHP 8

Summary
Try running PHP unit tests in PHP 8
Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

Fixes #

Just noting that this still runs PHP 8 tests with PHPUnit 5. It would make sense to run tests on PHP 8 with a newer PHPUnit version and then patching that version, instead of trying to patch PHPUnit 5.
This approach worked well in the AMP plugin: https://github.com/ampproject/amp-wp/pull/5637/files as well as in WordPress core: https://github.com/WordPress/wordpress-develop/commit/c31a5512732647412d8df03f43c1e0768db281ea#diff-b5d0ee8c97c7abd7e3fa29b9a27d1780R21-R33

@swissspidy Can you wait until I request another review. I am actively working on this PR.


Detect more def* forms in Clojure syntax

It’s a convention in Clojure to use macros whose name starts with def to define something. Libraries often define their own custom macros for this:

defroutes (Compojure)
defc (Rum)
etc

This is why whitelisting only def macros from clojure.core isn’t working: it will detect standard vars and functions but will fail to detect custom def*-like macros.
This PR adds regular expression that treats all forms whose name starts with def (with optional namespace) similarly.
This convention is very well established so it shouldn’t hurt anyone.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for looking into this.
Our clojure grammar actually comes from https://github.com/atom/language-clojure . Could you please resubmit this PR against that repo. That way any improvements will benefit other editors and tools as well. Once the PR is merged into language-clojure, back in the vscode Clojure extension, simply run npm run update-grammar to pick up the changes

done https://github.com/atom/language-clojure/pull/65

Thanks! I've pulled in these changes from language/clojure with 0ddd558b297a67348ca5ec9b87ee357ed51d0bd0 They should be in VSCode 1.13 insiders once those start up again
Closing but marking as part of the May milestone so that it is clear this change was picked up, just not through this PR

awesome! Thank you!


Enchant issues

Good afternoon,
I have your mod in my modpack and I'm really hoping for the ability to enable/disable enchants from the mod. It seems that your Beheading enchant won't drop vanilla skulls and so it's basically useless. Also, the autosmelt and life leach enchants are duplicates in the pack, so I'd love the ability to turn them off so I only have the one.
Thanks!

"Beheading enchant won't drop vanilla skulls"
False.  I just double checked by testing this on a zombie, i got the item "minecraft:skull" with metadata 2.  I got a vanilla wither skull too.  These are identical to the skulls in the creative inventory
For monsters that dont even have a vanilla skull, i THEN use the built in vanilla naming system to give it custom texture.
"I'd love the ability to turn them off so I only have the one." no problem at all.  I make a point of having everything in the config file.   All enchants be disabled inside of the "content" category in the config file.

I just tested disabling enchantments just to be sure that it works. 👍  hopefully it works the same for you

I'm not sure what's going on, then. I have a diamond sword with only Beheading and Looting III and when I kill Wither Skeletons, it doesn't drop the vanilla Wither Skelly head, it drops what looks like a wither skelly head but is called MHF_WSkeleton's Head 397:3.
Also, thanks about the enchant. Coulda swore I looked through the entire config for that, dunno how I missed it.

Oh what mod version are you using? I only tested the latest 1.12.1 build, are you on an old one? @orionSquared  .
And yeah the config doesnt have the best layout, theres no enchantment category its all just in content

Ah, I'm utilizing the latest 1.11.2 version. Mod version 1.5.11

Ah ic ic I will look into the 1.11.2 version tomorrow for sure.  I must have fixed it at some point in 1.12 but not ported it back.

Sorry but there is so much activity in 1.12 it seems silly to backport a small thing like this to 1.11.
I will fix 1.10 or 1.11 issues if they are game breaking, or exploits, or crashing things, but for new features and minor bugs there isnt much point, even the big modpacks and streamers are on 1.12 now.


Running multiple instances at once (npm start)

Correct me if I am doing something wrong, but it looks like the current setup doesn't allow for a user to run multiple instances of the seed at once using npm start  When I do it, I get this error:
Finished 'serve.dev' after 31 s
Error: listen EADDRINUSE :::5555
Obviously the reason is because I already have an instance running on this port.  I have seen other examples solve this by incrementing the port if the one it wants to use is already being used.  So the next time I run npm start it should go for 5556.
Here is the use case:
I am working on multiple projects at the same time that are using the seeder.  Instead of stopping and running each one, it would be easier/faster to just being able to run them at same time.

What you can make is:
npm start -- --port 5556


Add /acl/policy.. + /acl/token.. endpoints support into ACL library

I took a first stab at writing some tests for this PR, you can merge them from consul-acls branch if you want

Thank you very much! I'll add a description to the Readme and deal with the tests. I will also add the processing of additional keys from the documentation and try to test it on a live server.
I wanted to ask right away, if you are aware, do I understand correctly that in aсl.legacy there is now an older format for communicating with the API for managing tokens and rules? What I suggested in the PR will be relevant, in terms of the description in the Consul documentation?
To be honest, I'm a little confused about the documentation and changes between versions.

Yes, there is a legacy ACL policies and tokens API, which you can see here: https://developer.hashicorp.com/consul/api-docs/v1.14.x/acl/legacy.
It looks like it's not documented in v1.15 anymore, so maybe they fully removed it in the latest version. I figure we should keep the legacy methods around until the versions of consul that include it are no longer supported, which I assume could be a while.
In regards to the API we can just refer to the endpoints you implemented as the standard versions and we don't need to add any type of modifier to when documenting them (e.g. we don't need to say new Token API or anything like that).
Also, when documenting/implementing the API endpoints I liberally copy/paste from https://developer.hashicorp.com/consul/api-docs (the NOTICE includes appropriate attribution/license for using that text).


perf(button-effect): remove button-effect from DOM

DO NOT MERGE

@manucorporat please close this pull request if you don't want merge this.

@lweb20 you know that he and I are ionic team member? ^^ you don't have to comment like this

IMHO If not expect comments, these should not be enabled.
Pd:
Do not misunderstand my words, they are translated xD

@danielsogl  what happened?

@manucorporat He wrote you shouldn't create PR's if you don't merge them :D Nothing offensive

@danielsogl hahaha funny, well, most of the DONT MERGE mean:

higher priority issues appear, and the PR is not finished
Needs a meeting in order to

We have this policy in other for other member of the team, to

@danielsogl @lweb20 hahah no worries, I was just curious :D
Sometimes they are non-trivial PRs that require a lot of time, creating a PR is good way to great to get feedback from other members of the team or the community itself.
Sometimes, it is just not the right time to merge it, since it does not match with the mid-term roadmap or there are higher priority issues, but creating a PR is a good way to don't forget it, this PR is a good example.

@manucorporat I made a comment about the commit but @danielsogl not expected the strange comment. I'm new in this and I don't know who are Ionic-team members.

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


Add char-based language model training process for aishell.

Add prepare_char_lm_training_data.py in local/ for char-based lm train data preparation process.
Add steps 9,10,11 in prepare.sh for lm data preparation and RNN lm training.
https://github.com/k2-fsa/icefall/issues/920

Thanks for your contribution!
Do you have any WERs to report with LM fusion? It would be great to have some numbers here.

LGTM!
Merging it now.

good job! Can we use SF in sherpa-onnx now? I found kaldi and  wenet already support LM in C++  runtime, but in sherpa-onnx or sherpa-ncnn  i don't know how to use extern LM.

I would recommend using RNNLM rescoring in #1002 in Sherpa as it is much faster than shallow fusion while yielding similar WERs. This is on our to-do list.

RNNLM

Thanks, but the decoding code in icefall is not suitable for use in the production environment.I think sherp-onnx maybe one option, but unfortunately, it is not supported LM.

RNNLM

Thanks, but the decoding code in icefall is not suitable for use in the production environment.I think sherp-onnx maybe one option, but unfortunately, it is not supported LM.

@minmin0 Can you help to test it in sherpa-onnx if it is supported in sherpa-onnx?

onnx

yes

RNNLM

Thanks, but the decoding code in icefall is not suitable for use in the production environment.I think sherp-onnx maybe one option, but unfortunately, it is not supported LM.

@minmin0 Can you help to test it in sherpa-onnx if it is supported in sherpa-onnx?

yes， I am willing to do  that


Epic8 #44 faffy

This branch updates some styling:

sticky header
sticky hashtag-title (in seperate component)
some margins/paddings at page hashtag

This just needs linting

Outdated, doing this in a newer branch


Edit page is not loading with HasConditionalContainer

I updated to Nova  v3.12.0  and Conditional Container v1.2. When I did this my edit pages for the pages containing ConditionalContainer stopped loading.  It only happens after I added Use HasConditionalContainer;.  Without HasConditionalContainer I get the error of TypeError: this.field.expressionsMap is undefined but with it nothing shows up on the page.
It does work as expected on Create, but once I save and try to edit the item it breaks and becomes hidden with no errors that I can see.

Did you manage to resolve this?

can you try this PR will fix this? https://github.com/dcasia/conditional-container/pull/43

I publish the fix by @bolechen on v1.2.1

Erm, I have composer updated to get v1.2.1 and no Nova won't load it just timesout.
When I roll back to v1.2.0 it loads but you can't edit.
When using ConditionalContainer and JsonWrapper on the resource should you use: HasConditionalContainer or HasJsonWrapper? You can't use them together as they clash.

I am unsure if it's a clash with another package but the second I updated (without changing anything else) I now can't access Nova. It simply spins and times-out. I remove the package and it works fine.
The only other field I have that isn't a Nova one is JsonWrapper.

Hmm, thanks for looking into this.
I know find when using your package that my nova instance just times-out (504). The second I remove the package it works beautifully.
Will have to investigate.

It's a bit weird, but on Nova v3.12.1 version 1.2.0 worked.
Now with v1.2.1 the edit page is just like @Slgoetz said.
Rolling back to v1.2.0 the edit page works perfectly.


Make it possible to specify which Pixart model to load (or all)

Examples:
To download all: python PixArt-alpha/tools/download.py To download just 512x512: `python PixArt-alpha/tools/download.py --model_names "PixArt-XL-2-512x512.pth"

@lawrence-cj then run without args :)

Nice work. Approved.


Ignore unsupported partition fields

Thought I'd give it a try and help resolve the forward-compatible issue for new transforms to the partition spec detailed on issue#32

@rdblue I'm following up on your initial review and basically dropping the original change proposal, that was a bit naive. Since I believe that initially I've greatly underestimated the task at hand I wanted to ask whether you consider the current state of the PR as worth following up on?


Prettier

@piscisaureus has some concerns about prettier still (regarding the formatting of matricies) - will close this for now and do a new patch at some point.
(Thanks for the review @ryanmurakami - I will check this when I do another prettier patch and make sure to address your comments)


Cannot find module '@angular/tsc-wrapped/src/tsc'

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ ] 3.x
I'm submitting a ...  (check one with "x")
[1 ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

E:\Working Project\schoolsDna>ionic cordova run android --prod
? The plugin @ionic/cli-plugin-cordova is not installed. Would you like to install it and continue? Yes

npm install --save-dev --save-exact @ionic/cli-plugin-cordova@latest
√ Running command - done!
[WARN] Error occurred during command execution from a CLI plugin (@ionic/cli-plugin-cordova). Your plugins may be out of       date.
Error: Cannot find module '@angular/tsc-wrapped/src/tsc'

Expected behavior:

Steps to reproduce:

Related code:
insert any relevant code here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

global packages:
@ionic/cli-utils : 1.5.0
Cordova CLI      : 6.5.0
Ionic CLI        : 3.5.0

local packages:
@ionic/app-scripts              : 1.3.12
@ionic/cli-plugin-cordova       : 1.4.1
@ionic/cli-plugin-ionic-angular : 1.3.2
Cordova Platforms               : android 6.1.2
Ionic Framework                 : ionic-angular 3.5.0

System:
Node       : v7.10.0
OS         : Windows 10
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed
npm        : 5.1.0

Duplicate https://github.com/ionic-team/ionic/issues/12065

P.S.: Update npm to last version may most probably gonna help

done this with npm 5.3 and install node modules again. but still facing same issue

sorry didn't saw you weren't using the actual ionic app-scripts :(
then downgrade npm
or upgrade npm and ionic app-scripts and ionic-angular (but pay attention to the change log https://github.com/ionic-team/ionic/blob/master/CHANGELOG.md a line need to be added to index.html)

Hello, all! At this point, this is due to issues in the latest release of npm. Downgrading to npm 5.1 fixes the issue, but we are considering just adding this package to our package.json (even though we technically shouldn't have too) for the meantime until npm fixes their issue. Gonna keep this one open for now as we discuss what we should do.

@jgw96 downgrading ... or upgrading everything might work too I think
about adding the package in our package.json...at one point npm gonna be fine again, no need to rush ;)

Run npm install --save-dev @angular/tsc-wrapped in your shell

Hello all! As it seems it has been a while since there was any activity on this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!

This still occurs with npm > 5.1 correct?
Is it Ionic's direction that we should just add @angular/tsc-wrapped to our package.json, or is that still under consideration?

@Barryrowe - FWIW, I am using npm version 5.3 and do not have this issue. What version are you using?

Node v8.4.0
NPM 5.3.0
I just downgraded to NPM 5.2.0 due to this issue and still get the error with @angular/tsc-wrapped not found

Try npm installing that module, angular/etc-wrapped

@wbhob that's where I am at now, which is where my original question comes from. Is that the official path forward, and the templates for Ionic will include the package in package.json?

It's the only fix I have found, and it's pretty standard practice: if one module depends on another, you have to install it.

@wbhob agreed if you depend directly, just odd that it would be installed as a transitive dependency with npm 4.X and not 5.x. Simple solution, just odd.

Hi all, I ran into this myself and was able to resolve it by removing my package-lock.json and node_modules and installing again, after having upgraded node and npm. There must have been an issue with the older version that stayed broken even after upgrading.

@mlynch Hi which version of Node and NPM are you using?

Node 8 / NPM 5

This can also happen if you lost internet connection while running npm install. @bkarv steps solve the issue.


Error: cannot find account "true"

Possibly related to https://github.com/soywod/himalaya/issues/167
I'm on the most recent version of Himalaya and I'm having trouble getting mailbox preview to work within the most recent version of NeoVim. It's worth noting, however, that I'm a new user of this project, and never used mailbox preview prior to now, so I'm not sure what is causing the issue. Here is an image of the error message that I am seeing when I try to preview my Gmail account's mailboxes on macOS.

I isolated the problem to this section of account_entity.rs. I'd fix it myself but I'm not sure exactly how to do so. Otherwise, I would have submit the pull-request myself.
Any idea what might be causing the problem?

For reference, here is my config file
name = "Austin Traver"
downloads-dir = "~/Downloads"
signature = "redacted"

[gmail]
default = true
email =<EMAIL_ADDRESS>
imap-host = "imap.gmail.com"
imap-port = 993
imap-login =<EMAIL_ADDRESS>imap-passwd-cmd = "redacted"

smtp-host = "smtp.gmail.com"
smtp-port = 465
smtp-login =<EMAIL_ADDRESS>smtp-passwd-cmd = "redacted"

I'm afraid the telescope mailbox picker is broken. I will investigate and let you know. Thank you!

The issue is fixed on development, it will be on master with the next release. I let you know!

Thank you for getting to this. Follow up on this thread when the changes are pushed to master, and I'll confirm the fix 👍🏻

Update: Running v0.5.5 of Himalaya at the moment, and the issue is resolved.
Thank you for taking the time to get to this!


feat(ingestion): pull metabase database, schema names from raw query and api

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[x] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

This is a follow up to https://github.com/datahub-project/datahub/pull/5647 to:

pull the database name from the raw query when possible, see https://github.com/datahub-project/datahub/pull/5647#issuecomment-1242298818
pull the schema name from the API when possible; this is not documented in https://www.metabase.com/docs/latest/api/database but the default schema can be stored in the API response under [].details.schema

Tested locally with acryl-datahub==<IP_ADDRESS>.

Thanks @remisalmon!


Consolidate date macros into timestamps.sql

resolves #5521
related PRs:

https://github.com/dbt-labs/dbt-snowflake/pull/273

Closed but related (to be reopened in part 2):

https://github.com/dbt-labs/dbt-spark/pull/480
https://github.com/dbt-labs/dbt-redshift/pull/191
https://github.com/dbt-labs/dbt-bigquery/pull/323

Description

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

Could you implement a test that confirms that current_timestamp_in_utc yields the expected value as well as the expected data type? I'd suggest something like this.

While we certainly could I do not see a compelling case for doing so as we already have added more testing for this than other macros, if a particular adapter has this as a concern it wouldn't be difficult to add this in the adapter specific implementation.

The type of thing I'm worried about is if a user has their database timezone set to some non-UTC timezone and somehow current_timestamp_in_utc accidentally reports a timezone-naive local time that isn't actually in alignment with UTC.

This is a valid concern for redshift where the established behavior for _utc relies on it's default behavior but adding a test for utc won't actually catch this.

Do you know if the data type (or value) of current_timestamp_in_utc will be different than the implementation within dbt-utils for any adapters? (I didn't check those implementations, so I don't know.)

It will not, I'm specifically checking for that in the adapter level tests.

Received offline feedback concerning the need to include a "backcompat" macro for current_timestamp_in_utc as we may be diverging from dbt-utils versions in a breaking manner.
Digging in there are two adapter specific _utc implementations (that don't just use current_timestamp):

Snowflake
Postgres
Redshift
Looking at the three we aren't changing the datatype (and checking the values)

@colin-rogers-dbt @dbeatty10 Thanks for all the great discussion in here!
I had a chance to chat with Doug earlier today, and hear his perspective on the complexity that he's been going deep on, and simmers under the surface. There's quite a lot!
To simplify matters, here are my proposed next steps:

Split this PR into two.
The first PR should just provide backward compatible implementations for dbt_utils.current_timestamp and dbt_utils.current_timestamp_in_utc — via dbt.current_timestamp_backcompat and dbt.current_timestamp_in_utc_backcompat, respectively — so that we can unblock dbt-utils v1.0. Exit criteria: The return types and values of these back-compat macros match exactly the return types and values of the deprecated dbt_utils macros. (I'd also be happy to see the reorganization / centralization of our existing macros.)
The second PR should add in dbt.convert_timezone + dbt.current_timestamp_in_utc, as net-new macros that do not exist currently. I've opened a new issue outlining the spec for both: https://github.com/dbt-labs/dbt-core/issues/5969
For future (tech debt): https://github.com/dbt-labs/dbt-core/issues/5967

In terms of timing, we only need the first PR for inclusion in v1.3, to unblock dbt-utils v1.0. The second PR could be merged for inclusion in v1.4 instead.

One easy way to test the backwards compatibility is just to compare the new SQL to the old SQL. If they are the same, then we are done.

Good idea, I've added an assertion to our functional tests to validate the compiled SQL

Yes, we won't be implementing all adapter zone tests for all adapters. We did expect that everything in the "basic" directory would be a requirement for all adapters, but tests in the adapter zone outside of that are optional. Not all adapters implement all features.


config jetty version in diff profile

What changes were proposed in this pull request?
To slove #904 , need use org.spark_project.jetty package code not org.eclipse.jetty package code, and  oap dep different spark version need different jetty version.
How was this patch tested?
Manual test

may oap should remove jetty deps, spark alreay dep it.

@zhixingheyi-tian test this ?

@LuciferYang
Currently, OAP'DailyTest tests the branch of Master.
Can you test it locally?

Already test it in local model, I mean, you can verify that.


Distinguish why Build.Repository.Name results in Null and Build.SourceBranch does not result in Null

When declaring
variables:
  BuildRepoName: $[ variables['Build.Repository.Name'] ]
  BuildSourceBranch: $[ variables['Build.SourceBranch'] ]

The documentation states it is an agent scoped variable.  Perhaps there should be more clarification on what that means.

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

ID: 9ffebdc8-521c-44b5-cde4-4abdc1bd6f18
Version Independent ID: dc0effda-6d93-62aa-64c5-4e6f283b0826
Content: Predefined variables - Azure Pipelines
Content Source: docs/pipelines/build/variables.md
Product: devops
Technology: devops-cicd-build
GitHub Login: @juliakm
Microsoft Alias: jukullam

A better enhancement would be to show how to get the value in case it is needed.
How to extract agent-scoped variables for use in [places that do not allow it]
Since agent-scoped variables cannot be used without being run in an agent, the most common cases being conditions in stages, jobs and steps, they need to be set from an agent and set as a variable.
This can be done through [ a link that points to the task.setvariable ] that is executed in the initial stages.  For example
stages:
  - stage: bootstrap
    displayName: Bootstrap build pipeline
    jobs:
      - job: exposeAgentScopedVariables
        displayName: Expose agent-scoped variables
        steps:
          - bash: |
                echo "##vso[task.setvariable variable=X.Build.Respository.Name]$(Build.Repository.Name)"
                echo "##vso[task.setvariable variable=X.Build.Respository.Uri]$(Build.Repository.Uri)"
            displayName: Expose agent-scoped variables

Then in future stages it can be accessed as
condition: and(succeeded(), ne(variables['X.Build.Respository.Name'], 'specialName'))


Migrate emails from postmark to loops.so

https://loops.so/docs/integrations/supabase
Loops.so is a better email platform because it not only offers transactional emails, but also offers marketing in one which makes it really easy to have an all-in-one email platform. The campaigns that you can create in loops.so is really nice and will be super helpful when growing a business especially one with an email list.
I like that it's event based so it allows us to send personalized emails based on events that occur within the app.
Also, it integrates well with stripe as well which can send emails to people that have subscribed!

An alternative i was looking at was Resend, but it's not comprehensive and it doesn't provide a UI to edit the emails which I feel is pretty important because I don't want to be writing code to write emails.


Parsing mixcr files with incomplete amino acid sequences

Hello, I've noticed that while using repLoad on mixcr files some clonotypes with incomplete CDR3 aminoacid sequence are removed (Ex. "CIVRVEL_GANNLFF"); Is there anyway to parse automatically files including those clonotypes?
Thank you.

Hi, @francops1722!
My name is Aleksandr Popov, I am a developer of the Immunarch package. Thank you for using our software!
Could you, please, provide an example of your data that has removed clonotypes? It will help to find out what happened on the parsing stage.
Best regards,
Aleksandr

Thanks


Add new ReCAP location xmr/PG to SubmitCollection

The new location has been created, see https://github.com/pulibrary/bibdata/issues/1963
rare$xmr
internal name: scarcpxm
Customer code: PG
Fulfillment unit: Closed
Remote Storage: ReCAP remote storage
Additional background can be found here:
Project overview
Locations proposal
Locations checklist

It also needs to be Supervised Use in the Use Restriction.

@regineheberlein will update the ticket when the data work is done in Alma.

needs more clarification regarding the location code

This location code is included in the previous issue so it will be added there. #1963

The location was added in bibdata in https://github.com/pulibrary/bibdata/issues/1963


[arm] cleaned up armenian phones

Cleans up the phone lists for arm_e and arm_w. Closes #298.

[x] Updated Unreleased in CHANGELOG.md to reflect the changes in code or data.

@ajmalanoski is this ready, in your opinion? LGTM.

@ajmalanoski is this ready, in your opinion? LGTM.

It looks good to me, as well!

It looks good to me, as well!

Thanks Hossep and congrats, you're now a contributor to WikiPron!

Thanks Hossep and congrats, you're now a contributor to WikiPron!

@jhdeov I'm running a scrape of Armenian right now. I've noticed that [ʏ] occurs very often in the West Armenian data (I didn't count, but there are dozens of instances). I just wanted to confirm than this phone is indeed not found in Western Armenian.

@jhdeov I'm running a scrape of Armenian right now. I've noticed that [ʏ] occurs very often in the West Armenian data (I didn't count, but there are dozens of instances). I just wanted to confirm than this phone is indeed not found in Western Armenian.

Rats. [ʏ] is an allophone of /ju,ju/ and the hy-pron script generates it. Apologies for missing that. The [ʏ] segment is only found in Western Armenian, not Eastern Armenian.
I reread the script and you'll find that apparently they also generate [œ] as a foreign phoneme in Western. I checked my scraped data and this segment is only found once.

Rats. [ʏ] is an allophone of /ju,ju/ and the hy-pron script generates it. Apologies for missing that. The [ʏ] segment is only found in Western Armenian, not Eastern Armenian.
I reread the script and you'll find that apparently they also generate [œ] as a foreign phoneme in Western. I checked my scraped data and this segment is only found once.

Okay, so if I understand correctly, the .phones file for Western Armenian should include [ʏ] and [œ]?

Okay, so if I understand correctly, the .phones file for Western Armenian should include [ʏ] and [œ]?

@ajmalanoski  Correct, with a note like this
ʏ # allohpone that is automatically transcribed from word-medial յու /ju/
œ # rare and a marginal borrowed phoneme

@ajmalanoski  Correct, with a note like this
ʏ # allohpone that is automatically transcribed from word-medial յու /ju/
œ # rare and a marginal borrowed phoneme

Okay, I can take care of the edits. Thanks!

Okay, I can take care of the edits. Thanks!


Update OpenSSL::SSL::Context default ciphers

Version 5.7 adds DHE-RSA-CHACHA20-POLY1305 to CIPHERS_INTERMEDIATE.
Also replaces the JSON::Any with JSON::Serializables.

I would prefer to have two separate PRs for this: A refactor and regenerate with new feature(s).


Staging deploy

bump ruby version (not really necessary, but hey, it helps)
run npm install, cp env file and run npm run build on capistrano deploy
a few more things that are needed to keep capistrano happy

I tried running the specs locally and the build fails on ruby 2.4.2, passes on 2.4.1. I think you haven't changed the version of ruby for travis, which is why CI still passes.

Yeah, I had fixed it, just on another branch. Now it's here, and I also updated travis

great, thanks!

@agnessa ready for review


(PUP-10164) Parse Content-MD5 leniently

Previously, we parsed the Content-MD5 header strictly (RFC 4648), but some
webservers don't include padding, so parse the header leniently (RFC 2045).
Blocked on https://github.com/puppetlabs/puppet/pull/8131

CLA signed by all contributors.

jenkins please test this with servertests

This is ready for a rebase :-D


Expose new custom components env vars to csv-generator and manifest-templator

What this PR does / why we need it:
This is a follow-up for https://github.com/kubevirt/kubevirt/pull/8673. This PR aims to expose the new env vars introduced in the previous PR to csv-generator and manifest-templator.
Release note:

Expose new custom components env vars to csv-generator and manifest-templator

/cc @jean-edouard @enp0s3 @fossedihelm
Can you please pay special attention to https://github.com/kubevirt/kubevirt/pull/8792/commits/25a82b6a703ddb513392b9da6a37511543314aa0?
Some context: currently, for some reason, it is allowed to provide either all of the custom sha env vars or none of them. This is a very old code and I guess this restriction isn't necessary. Am I missing anything?

/retest-required

/retest-required

/cherrypick release-0.58


MultiGPU TensorFlow uses Keras

Updated the multi-GPU TensorFlow example with TF.Keras as this is easier to use

rerun tests

@bschifferer I liked this change, but not sure we ever got it to a point where we could merge it. Should we keep this open and revive it or close and re-submit a version built on main?


Marksman completion add extra [[ when completing a wiki-link

I installed the latest Marksman (1.0.0-7f3b471) using snap and use it with nvim.
When I try to insert a wiki-style link, the completion inserts an extraneous [[. For instance:

when I input [[<C-x><C-o>, the resulting link is [[[[file
when I input [](/<C-x><C-o>, the result is [](/[](/path/to/file).

Have I made a misconfiguration in my configuration, or maybe I am using marksman wrongly?
My nvim version is as follows, and I've also included the minimalist init.vim I use below.
$ nvim --version
NVIM v0.9.2
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/gcc -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -fno-common -Wno-unused-result -Wimplicit-fallthrough -fdiagnostics-color=auto -fstack-protector-strong -DUNIT_TESTING -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -I/usr/include/luajit-2.1 -I/usr/include -I/usr/include/luv -I/builddir/build/BUILD/neovim-0.9.2/redhat-linux-build/src/nvim/auto -I/builddir/build/BUILD/neovim-0.9.2/redhat-linux-build/include -I/builddir/build/BUILD/neovim-0.9.2/redhat-linux-build/cmake.config -I/builddir/build/BUILD/neovim-0.9.2/src -I/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/include -I/usr/include

         fichier vimrc système : "$VIM/sysinit.vim"
               $VIM par défaut : "/usr/share/nvim"

Run :checkhealth for more info

call plug#begin()

Plug 'williamboman/mason.nvim'
Plug 'williamboman/mason-lspconfig.nvim'
Plug 'neovim/nvim-lspconfig'

call plug#end()

" Lua plugins
lua << EOF
require("mason").setup()
require("mason-lspconfig").setup{
  ensure_installed = {"marksman"},
}

require("lspconfig").marksman.setup{}

EOF

It works perfectly in Helix out of the box, but I think it is Helix doing the auto pairs functionality. Please don't make any changes without confirming it still works in Helix, thanks.
I press [ and get [] then [ again and get [[]] with the cursor right in the middle with autocompletion working giving me all the links to the files in the workspace.

@khauchy thanks for reporting this. I can reproduce in nvim. I haven't used omni-completion via C-x C-o before however, so I'm not yet sure why it adds extra braces.
I personally use nvim-cmp and TAB completion -- with this setup I don't have any extra braces.

Not happening to me. Marksman installed with mason and using omni-completion (which is automatically set by nvim-lspconfig). I'm on nvim nightly btw.
marksman-omnicompletion.webm


add possibility to explicitly put DATABASE_URL inside query! and query_as! macro as rust variable

something like that
let database_url = ""; sqlx::query_as!(     SomethingType,     "SELECT * FROM something LIMIT $1",     *limit as i64,     database_url )

This is not possible. The macros cannot read variables or even constants. It has to either be a string literal or something available in the environment at compile time.


[MX-114] Infrastructure for hiding back button on Partner view

#skip_new_tests.

:rocket: PR was released in v1.19.2 :rocket:


"istioctl kube-inject" link returns 404

I found the link on installation guide returns 404.
"istioctl kube-inject" link on istio.github.io/installing-istio.md at master · istio/istio.github.io · GitHub is currently returns 404.
The link address is https://github.com/istio/istio.github.io/blob/master/_docs/tasks/{{home}}/docs/reference/commands/istioctl.html#istioctl-kube-inject
And it should be https://github.com/istio/istio.github.io/blob/master/_docs/reference/commands/istioctl.md#istioctl-kube-inject

the .md in the istio.github.io aren't exactly meant to work out of the box when seen on github, they get processed by jekyll (which understands {{home}}) and pushed to istio.io, for instance
https://istio.io/docs/tasks/installing-istio.html#deploy-your-application
doesn't have any broken links


If an application has no more than one non-tabbed window, Meta-W should close the window

Windows for applications such as Boot Environments, Desktop, Keyboard, Mouse, Print Settings, Screen Settings, Sharing, Shortcut Keys and Sound do not close when expected.
(Windows of Wireless Networks could not be tested in a VirtualBox guest.)
Under /Applications/Utilities I found Calculator similarly non responsive to the keyboard shortcut, so I guess that it's a generic issue.

They currently can be closed with ⌘-Q (Quit) but I agree they should also be able to be closed with ⌘-W (Close).

They currently can be closed with ⌘-Q (Quit) but I agree they should also be able to be closed with ⌘-W (Close).


How to set the context in jsonl file for fine tuning

Describe the feature or improvement you're requesting
I need to fine-tune an ada model.
I have the instruction set which describes the task to perform. The instruction set contains 8-10 commands which are the rules to be followed in the inference time.
Along with these commands, I have some examples to feed the system as a few shots.
So while finetuning I can put the few shot examples in the JSONL file in the following format
{"prompt":<example 1>", "completion":<example1 response>}
{"prompt":<example 2>", "completion":<example2 response>}

But where should i keep the instruction set/rules to follow/command set/prompt while finetuning?
Is it possible to keep this also in the JSONL file?
Additional context
No response

Hey! Please refer to our updated guide on fine-tuning: https://platform.openai.com/docs/guides/fine-tuning/example-format, there is only one way to pass in data for training with the traditional completion pairs, see the example above, everything you want the model to know must be in the prompt completion pair.


Use poetry in the example

Now we are just using requirements.txt but I think it's better to use poetry to pin versions.
Otherwise, a new major version of a certain library may appear and break the sample code.

@maronuu @kitagry Is there any reason why you did not use  the Poetry? If not, I will submit a PR.


Test invoke with AND endorsement for multiple orgs

Currently our network tests verify only invoke with endorsement policies when one of the organization is supposed to endorse (OR ('Org1MSP.member', 'Org2MSP.member')). The case when both organizations need to endorse (AND ('Org1MSP.member', 'Org2MSP.member')) is not covered, since it requires changes in the test scripts for invoke (expect-invoke.sh). In this case we need to explicitly provide --peerAdresses for peer chaincode invoke.

duplicate with #79


Checking something: Endianness of scom addresses

https://github.com/open-power/skiboot/blob/f901fcafae14d38e29f1cc11440086ee678785d0/hw/xscom.c#L108
So I know the p9 can run in either big or little endian mode, but this bit of code leads me to believe that
scom addresses are always big endian, regardless of whether the cpu is running in big or little mode.
Just a bit of clarification would be nice, thank you.

SCOM addresses are just offsets into the XSCOM MMIO area. The SCOMs themselves are big endian MMIO registers so we use the in_be64()  and out_be64() when accessing them.

Oh, so like, recv_status_reg is at 0x603fc00000000 + recv_status_reg on p9 systems?
And I assume that's true for any other scom reg? Neat.

More or less. Historically SCOMs were accessed from outside the chip via a address/data register pair and the "scom address" needs to be translated from a register number to an offset into the XSCOM MMIO area. For P9 all that requires is shifting the address up by three bits, but it was a little more complex on P8 which is why we use xscom_addr() to do the translation.
The other thing is that each chip has its own xscom MMIO range so 0x603fc00000000 + (recv_status_reg << 3) is the address for chip 0 and 0x623fc00000000 + recv_status_reg is the same scom on the 2nd chip. xscom_addr() also handles that.


Syntax error

This is what I get when I try to run the script:
File "download_archive.py", line 93
def download_descriptions(ids: list, descs_dir: str, num_processes: int) -> list:
^
SyntaxError: invalid syntax
macOS Catalina
Python 3.7.3

Hi! :)
I couldn't reproduce your error. I'm working on with a linux os and not macOS, but I don't think that is the cause of difference.
Can you try to provide more details somehow? :)
Are you running a clone of the most-updated master branch?

Having the same problem on Ubuntu Server. The error is pointing to ids.

I think I know what the issue might be.
I think in newer versions of python the type 'list' is deprecated in favor
of 'List' from the typing module.
On Tue, Jun 2, 2020, 19:55 Takeshi Sugiyama<EMAIL_ADDRESS>wrote:

Having the same problem on Ubuntu Server. The error is pointing to ids.
[image: Screen Shot 2020-06-02 at 9 53 26 AM]
https://user-images.githubusercontent.com/58148757/83547438-11091600-a4b7-11ea-9053-c6b374296904.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GalAvineri/ISIC-Archive-Downloader/issues/32#issuecomment-637679060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFUF4HHYJSQHYPBGOUXIEE3RUUVGHANCNFSM4I3PUHOA
.

So fix will be to add
from typing import List
...
def download_description(ids: List, descs_sir: str, ...

Exactly. Thank you :)
I also think there might be other places places where the same replacement
is needed.
On Tue, Jun 9, 2020, 00:09 Jamil<EMAIL_ADDRESS>wrote:

So fix will be to add
from typing import List
...
def download_description(ids: List, descs_sir: str, ...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GalAvineri/ISIC-Archive-Downloader/issues/32#issuecomment-640887991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFUF4HFTM5WHQ4YL4KQMZE3RVVHQTANCNFSM4I3PUHOA
.


A few typos

Awesome stuff! I fixed a few typos and I hereby waive all rights to these changes. Let me know if you need me to sign something to formally do that.
It’s kind of hard to see what the changes are:

Afterall -> After all
evolution strategies-> evolution strategies (ES) Add ES acronym to evolution strategies, as is done with other acronyms in the text
spatial-temporaldetails -> spatial-temporal details

Thanks, Just fixed this!


Fix clippy complaints

This pr addresses all current clippy warnings, in particular adds approx crate to deal with float equality. Using our current benchmarks the float equality change seems to have no effect on performance.

Looks good to me.


fix(KONFLUX-3663): format PipelineRun files and upload SAST results

This update configures the SAST task to upload SARIF results to quay.io for
long-term storage
Related:

https://issues.redhat.com/browse/KONFLUX-3663
https://issues.redhat.com/browse/KONFLUX-2263

/ok-to-test

Fix this PR with https://github.com/stolostron/multicloud-operators-foundation/pull/801


Add Ceph support

Replaces #497 - I don't have permission to push to the original branch.

Hello, are you still interested on supporting ceph? I've been writing a lib on my own to support it before finding this PR. I can help finish this if needed.
Also as a side note for this PR, it could be great to have a way to define which daemons to start using the DEMO_DAEMONS environment variables, users may not need to launch a full ceph demo cluster, only some parts of it like rgw or mds (for cephfs). But this might break the current implementation that uses an HTTP check on the rest_api module to validate the container's creation. (there could be other ways to check that the script has ended like looking into /etc/ceph/I_AM_A_DEMO if possible)
Let me know if I can be of any help here.


Manager31 minion not in configuration -> target systems list

will also port to master

@hustodemon  make rubocop happy first
features/step_definitions/navigation_steps.rb:311:43: C: Prefer to_s over string interpolation.

  find('input[name="filter_string"]').set("#{$minion.full_hostname}")

                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^

features/step_definitions/navigation_steps.rb:312:35: C: Do not use parentheses for method calls with no arguments.

  find('button[value="Go"]').click()

                                  ^

here is the naming convention for secondary features:
https://github.com/SUSE/spacewalk-testsuite-base/blob/master/docs/Guidelines.md#rules-for-features
example
https://github.com/SUSE/spacewalk-testsuite-base/blob/master/run_sets/testsuite.yml#L54

Ufff, sorry for the embarrassment with the franky.feature! :trollface:  Fixed your issues, but didn't add to refhost - is there a reason why this test should run on ref hosts?
guys, let me know if it's ok now. If yes, i'll squash the commits, merge it and backport to master.

@hustodemon  thx ! the ruby code steps are fine.
I would however:

add the feature to an existing secondary feature.
rename the feature name, and explain what is the goal of the test. Why are we expecting to see not target system found? If we change the order of the feature, does still work?
what are the dependency of this test?

In longterm all this seems you contradictory, but we need to make clear th intent of the test and which dependency are implicit, to don"t have a failing test without know why. Just looking in the logic there, we are expecting that there are no system in that page, but ok, what does it mean?

"Explain the goal of the test" : yes, but please do this in the scenario name - NOT in the feature description
The feature description is for explaining which feature OF THE PRODUCT we test, not to describe the test.
I felt I needed to clarify this @MalloZup :smile:

sure , buti it not clear which feature are we testing even. ( a part of that)

Right.

add the feature to an existing secondary feature.

Can you advise to which one? I was looking in the features, but couldn't find any place where this would fit, that's why I created a new one.

rename the feature name

I was looking at a couple of other Features and i thought the name "Systems in "Configuration => Systems => Target Systems" page" was quite ok comparing to them (e.g. "Patches page", "The channels page"). Then OTOH, we have features that are called like "Install a patch on the client via Salt through the UI", or "Install a package to the traditional client", but this only looks like we don't have any convention here at all! What should be then the right name for a feature? Should it be a sentence, or a name?

and explain what is the goal of the test. Why are we expecting to see not target system found?

Salt minions (unlike traditional systems) should never appear in that list. I'll try to be more explicit in the Scenario.

If we change the order of the feature, does still work?  what are the dependency of this test?

It depends on the presence of a salt minion in SUMA. I assumed that the core features ensure that. Don't they?

@hustodemon thank you for the honest feedback, yes. :clap: :clap:
Ok , now we know that  a salt-minion should never appear there.

So the feature we need to adress is here:
https://github.com/SUSE/spacewalk-testsuite-base/blob/master/features/allcli_overview_systems_details.feature

I would personally add this kind of test for all 3 types of Minions , creating for each a scenario.

For the Feature Naming, it was always a mess the naming of Scenarios. for the scenario Names, there are small conventions https://github.com/SUSE/spacewalk-testsuite-base/blob/master/docs/Guidelines.md#rules-for-scenarios.

I completely agree with you, but we are starting to change it, and we want new test to be less messy that in the past.
For me there, the most important it to understand what the test want to do.
This is the most important stuff for maintaining in long term.
I am not so pedant for that

It depends on the presence of a salt minion in SUMA. I assumed that the core features ensure that. Don't they?

Correct! :rocket:

@hustodemon i am closing this pull request because this repo will move inside the spacewalk fs repo.
Feel free to reopen it against SUSE/spacewalk soon. thx


separate solr tasks so it can be deployed to its own server

search_api_solr 8.3 is not quite working, thus this PR is stuck: https://github.com/Islandora-Devops/claw-playbook/pull/123.  Want to get the ansible changes to support indepndent solr server deployement in, thus this PR.  Will circle back to the search_api_solr 8.3 issue later on.
GitHub Issue: (link)
https://github.com/Islandora-CLAW/CLAW/issues/1197
What does this Pull Request do?

Supports installing solr on its own server.

How should this be tested?

Get the PR
vagrant up
Create content
Verify that it gets indexed and returned

In addition, testing the following is helpful.

Get the PR
Modify or create the following playbook:

---

- include: bootstrap.yml
  tags:
    - bootstrap
- include: solr.yml

deploy and check that solr has been correctly installed

Interested parties
@Islandora-CLAW/committers

vagrant up

@Natkeeran
I'm getting this:
TASK [webserver-app : Set default solr server host from hostvars] **************
Thursday 08 August 2019  20:19:31 -0300 (0:00:02.597)       0:41:44.839 *******
fatal: [default]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'ansible.vars.hostvars.HostVarsVars object' has no attribute 'ansible_ssh_host'\n\nThe error appears to be in '/home/daniel/Code/Environments/claw-playbook/roles/internal/webserver-app/tasks/solr.yml': line 7, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Set default solr server host from hostvars\n  ^ here\n"}

I checked my vagrantfile and ansible_ssh_host is not defined.  Is that something we can trigger to get ansible to autocreate it or do we need to add it manually?

@dannylamb
I assume you are mergin with latest and testing.
I've updated the pr, pls try again.


Add license information

Currently, this repository is provided without a license.
I recommend adding one to help clarify what users can (and not) do with the code.

done sorry


Property with reference type becomes an And schema

So, looking at Dns, RecordSet has a property called properties.
https://github.com/Azure/azure-rest-api-specs/blob/f8a99d2a884f76df769e3bd4d5d5d189a051d8f7/specification/dns/resource-manager/Microsoft.Network/stable/2018-05-01/dns.json#L1422-L1451
The "$ref" for the properties property is for RecordSetProperties. RecordSetProperties is just a bunch of properties (nothing too odd with it).
However, when this gets created in the code model as this:

So, it creates an And schema for this. But, why? The property is just a "$ref" to RecordSetProperties. Unless multiple "$ref" is allowed, this should never get created as an And schema. My assumption is that "$ref" is similar to "type" where "type" is a conceptual type (string, array, object, integer, etc.) where "$ref" is a real type (one that is actually defined in the swagger itself). So, representing it as an And is inaccurate if you're simply binding a type reference to that property. Am I wrong? 🤐

Yeah, this doesn't look correct. it should create an and schema only if it has additionalProperties or a parent schema.

Refactored, removed AndSchemas. Assuming this isn't an issue anymore


Can't download specific video from VEVO

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.05.10. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.05.10

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
>youtube-dl -v "http://www.vevo.com/watch/dove-cameron/Better-in-Stereo-%28from-Liv-and-Maddie%29/USWV21322440" -o "Better in Stereo.%(ext)s"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', 'http://www.vevo.com/watch/dove-cameron/Better-in-Stereo-%28from-Liv-and-Maddie%29/USWV21322440', '-o', 'Better in Stereo.%(ext)s']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2016.05.10
[debug] Python version 3.5.1 - Windows-10-10.0.10586-SP0
[debug] exe versions: ffmpeg N-79947-gcd244fa, ffprobe N-79947-gcd244fa
[debug] Proxy map: {'https': 'https://<IP_ADDRESS>:8888', 'http': 'http://<IP_ADDRESS>:8888'}
[Vevo] USWV21322440: Downloading video info
ERROR: Vevo said: Hmm, something went sideways! Please try another video.
Traceback (most recent call last):
  File "c:\program files\python35\lib\site-packages\youtube_dl\YoutubeDL.py", line 676, in extract_info
    ie_result = ie.extract(url)
  File "c:\program files\python35\lib\site-packages\youtube_dl\extractor\common.py", line 341, in extract
    return self._real_extract(url)
  File "c:\program files\python35\lib\site-packages\youtube_dl\extractor\vevo.py", line 225, in _real_extract
    self.IE_NAME, response['statusMessage']), expected=True)
youtube_dl.utils.ExtractorError: Vevo said: Hmm, something went sideways! Please try another video.

Description of your issue, suggested solution and other information
I can stream the video in the browser file and downloading the .m3u8 link I got from the network request inspector directly also works. Other videos can be downloaded normally with youtube-dl. This is the response:
HTTP/1.1 200 OK
Access-Control-Allow-Origin: *
Content-Type: application/json; charset=utf-8
Server: Microsoft-IIS/8.5
X-AspNet-Version: 4.0.30319
X-AspNetMvc-Version: 2.0
X-Powered-By: ASP.NET
X-Vevo-SN: WIN-7A37LTQO4TO
Content-Length: 328
Cache-Control: private, max-age=970
Date: Mon, 16 May 2016 09:56:02 GMT
Connection: close

{"video":null,"errorInfo":{"ytid":null},"isApproved":false,"isMonetizable":false,"statusMessage":"Hmm, something went sideways! Please try another video.","statusDetails":"Insufficient memory within specified address space range to continue the execution of the program.","statusCode":187,"countryCode":null,"languageCode":null}

It's fixed in git (7e8ddca1bb10068356d1ec43cf66e7627b76fce7) but not included in the latest release yet. Please wait for the next release.


Cannot access a disposed object

After rebasing my PR on latest changes I occasionally see this failure:
=== TEST EXECUTION SUMMARY ===
   Microsoft.AspNetCore.SignalR.Client.FunctionalTests  Total: 4, Errors: 0, Failed: 0, Skipped: 0, Time: 1.055s
xUnit.net .NET CLI test runner (64-bit Desktop .NET win10-x64)
  Discovering: Microsoft.AspNetCore.SignalR.Client.FunctionalTests
  Discovered:  Microsoft.AspNetCore.SignalR.Client.FunctionalTests
  Starting:    Microsoft.AspNetCore.SignalR.Client.FunctionalTests
System.ObjectDisposedException: Cannot access a disposed object.
   at System.Threading.TimerQueueTimer.Change(UInt32 dueTime, UInt32 period)
   at Microsoft.AspNetCore.Sockets.ConnectionManager.Scan() in C:\source\SignalR-Core\src\Microsoft.AspNetCore.Sockets\ConnectionManager.cs:line 81
   at Microsoft.AspNetCore.Sockets.ConnectionManager.Scan(Object state) in C:\source\SignalR-Core\src\Microsoft.AspNetCore.Sockets\ConnectionManager.cs:line 75
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Objectstate, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.TimerQueueTimer.CallCallback()
   at System.Threading.TimerQueueTimer.Fire()
   at System.Threading.TimerQueue.FireNextTimers()
SUMMARY: Total: 2 targets, Passed: 1, Failed: 1.

Thanks!


branches with more than 1 slash are truncated

hi,
when i have a branch with 2 or more slashes in its name, the extension truncates the name
For example:
feature/something/something-else/another-thinggets truncated and it shows feature/something

А что это все должно в папки превратиться? Или просто имя длинное с слешами?

Recursion then. Let's see what I can do.

Yeah same issue here. We use multiple slashes in our company as well for the branches names. Any update on this?

I also found the same problem, and I'd like to request a quick fix.


1354 plug externalization

Changed plugin manager plugin attributes to have a dict[str, dict[str, PluginInterface]] structure.

looks great.. thanks for doing the conversion over to dictionary based plugins


Run test cases from dataProvider in sorted order

When cases are provided via a @test:Config dataProvider make them run in a sorted order.

@manuranga can you elaborate on this more?
Currently data driven tests accept two formats of data as mentioned here.

Map of tuple
Array of array

Array of array tests are running in the provided order of the array.


[token-2022] Add support for record program for Transfer and TransferWithFee instructions

Problem

The confidential transfer instruction with split proofs does not yet support proofs from record program.
The new zk-sdk does not have support for the VerifyTransfer and VerifyTransferWithFee proofs any more

Summary of Changes
3dc64b0: I first realized that there are a lot of repetitive logic in the verify_*_proof functions, so I refactored the logic into a verify_and_extract_context function. In the end, this turned out quite nicely with the use of generics and traits.
c4cbfa9: I made each of the individual proofs in the TransferWithSplitProofs instruction to be read either from instruction data, record account, or context state account. Previously, the proofs could only be read from context accounts. In the process (after a bit of wrestling), I ended up removing the custom logic for parallel execution and auto-close. These custom logic does not really make much sense any more with the added flexibility of instruction data and record account and were over-complicating the logic. In this commit, I made some changes in the verification function lifetimes, but these changes are undone in the next commit 🙏 .
e7a08d7: Since zk-sdk does not support VerifyTransfer and VerifyTransferWithFee any more, the original confidential Transfer instruction is to be removed. Instead, now I made the original Transfer instruction the TransferWithSplitProof without fee (but still with the name Transfer) and the TransferWithSplitProof instruction to be TransferWithFee instruction now that these two instructions require different instruction data. In the process, I updated the transfer amount ciphertext validity proof in transfer instructions to be BatchedGroupedCiphertext3HandlesValidityProof instead of for 2 handles.
ca8b1ad: This is a smaller commit. I ended up just removing verify_*_proof functions and using verify_and_extract_context function directly.
46a180b: I updated the client according to the new updates in the program interface. Since the parallel execution logics were removed, I ended up removing the relevant functions in the client.
4c8c53b: Updated the tests to test for all three possible ways to include zk proofs in the instruction (instruction data, record account, context state account).
b9343d9: Updated token-cli for the newly changes interface in the token-client. Right now, the confidential transfer logic uses context state accounts to include proofs. I'll update these to use record program instead in a follow-up PR.

Thanks for the review! I'll address the increment on a follow-up PR as you suggested 🙏


Show weapon name and ammo on player UI

Show weapon name and current ammo on the player UI when the player equips a weapon.

Done in commit d647aa41b44bf8d25f8a0d448f487387007aa3aa.


ja: Make docs/setup/production-environment/tools/kubeadm/self-hosting.md follow v1.17 of the original text

This is a Feature Request

What would you like to be added
Update docs/setup/production-environment/tools/kubeadm/self-hosting.md to follow v1.17 of the original (English) text.
英文のまま更新するだけで十分です。
翻訳は必須ではありませんが、歓迎します。翻訳用にIssueを作成する必要はありません。
Why is this needed
content/ja/docs/setup/production-environment/tools/kubeadm/self-hosting.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.17-ja.3/content/ja/docs/setup/production-environment/tools/kubeadm/self-hosting.md
Original
https://github.com/kubernetes/website/blob/release-1.17/content/en/docs/setup/production-environment/tools/kubeadm/self-hosting.md
diff between translated and v1.17
https://gist.github.com/ffdedd7ce3e3cc44c1414418bb9e52a1
Note
Currently, we use dev-1.17-ja.3 branch. Please open a PR targeting the branch.
ref
How To Contribute
翻訳スタイルガイド
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel

/assign

/close


upscale 2x button actualy refine image?

I'm new to stablediffusion but as I understand in stableswarmui if i click on upscale 2x button next to my generated image it will upscale and refine this image using currently selected model? also how can I stop this process.

Correct, that's what it does. It's kinda just a convenience option that will probably be removed once the Image Editor is complete.
You can stop any generation at any time by clicking the red X button next to the Generate button (ie, the Interrupt button).


Handles errors due to Promise rejections

First off, this a very nice plugin. Thank you!
It wouldn't work for me though, because I use Promises with most of my replies. After a bit of digging I found that response.isDeveloperError doesn't get set because promise rejects are wrapped in Boom objects. You therefore have to also check response.isBoom to capture Promise rejections.
BTW, I change from lab.before to lab.beforeEach in the test file because the test I added would timeout with a node UnhandledPromiseRejectionWarning otherwise.
Thanks again, and keeps those Hapi posts coming.

Hi Tarafi,
thank you for your contribution to handle Promise rejections with hapi-dev-errors. I didn’t know that hapi supports to reply a rejected promise, like reply(Promise.reject(new Error('message'))).
Your code changes look good. Thank you for updating the test and adding a new one for promises!
The only thing I’m wondering is why you’re using response.output.payload.statusCode instead of response.output.statusCode. Boom derives the status code within the payload from the output status code.
I’ll wait for your response and merge it then. Maybe I’m lacking knowledge again 😃
Again, thank you very much for your support! Will definitely merge the PR, great work! ❤

Hey Marcus,
Thanks for responding and considering the PR!
Regarding the statusCode, during testing I thought it was undefined, but I must have imagined that 😄. I'll change it and do a push.
Thanks again!

Thank you for support! I really appreciate it 🤜 🤛
I tagged release 1.3.0 and published it on NPM 😃

Same here. Thank you!


ARROW-9827: [C++][Dataset] Skip parsing RowGroup metadata statistics when there is no filter

@bkietz with some delay, added the suggested comment

Thanks for opening the follow-up issue!


LVC conversion script questions

Edit: questions still remaining
• How should the sxs_catalog.json file be maintained, updated? It's currently on the arXiv entry for our paper [Mike will update black-holes.org so this file is available at some known URL?]
• Is the catalog paper always the right reference to cite? Cite catalog paper first, followed by Bibtex keys in our metadata [Harald is updating BibTeX keys]
• There is one extra time at the beginning of each NRtimes vs what I expect...important to track down why? Important to output one NRtimes entry per mode, when they are all the same? [Just output times for the 22 mode]
Edit: questions resolved
• The current script gives roundoff differences vs Patricia's old script for phases, derived horizon quantities. Are these a problem? [Probably fine, since we're using different numerical method to get these quantities]
• Should license be public? [Yes, public waves will be public]
• Should spline tolerances use relative Linf, not abs Linf? [yes]
• Should spline tolerances be set to 1e-6? [yes]
• Should we round eta < 0.2501 to 0.25? [yes]
• Should we use a bogus negative number for eccentricity when we can't measure it? [yes, but use -1]

Looking through the script, I noticed a few more things which I wasn't sure about:

Currently when computing omega and LN_hat we use forward differences which results in those data sets having 1 less point than say n-hat-vs-time.
The function that looks at comparable cases currently only considers the mass ratio and magnitudes of the spins and not their directions (it also does not take into account eccentricity). So a q=5, chi1=(0.9,0.0,0.0) would be considered close to q=5, chi1=(0.0,0.0,0.9).
Currently when we call romspline to generate the spline we use the default value for the degree of the spline, which is 5. However, in LALSuite, when the data is being reconstructed the interpolation is done using cubic splines. Should we also just use cubic splines?

• I vote for sticking with forward differences, which is just what we did before, for consistency
• I agree it's not optimal, but it's not obvious to me how to properly fold in everything you might want to look at. Is eccentricity more or less important than mass ratios and spins for determining accuracy?
• I vote for using the same degree as before, not cubic splines, for consistency.

Update from May 8 vacuum call:

Leave the finite differencing in place for now
Eccentricity should probably not be very different for the "similar run" used in error estimates, since an eccentricity 0.1 run isn't really the same as a circular run. Add logic to check that the eccentricity be close enough that only quasi-circular runs are used to estimate errors for quasi-circular runs (maybe check that the difference in eccentricity is less than say 0.01 or 0.05?)
Discussion of spline tabled for now while Geoffrey thinks about it


./config.sh: line 29:  6458 Segmentation fault: 11  ./bin/Agent.Listener configure "$@"

Hey I Have problem with configuration tfs on Mac. TFS is running on local server in my company. I try connect with PAT Authentication type.
Enter (Y/N) Accept the Team Explorer Everywhere license agreement now? (press enter for N) > y

Connect:

Enter server URL > https://tfs.company.local
Enter authentication type (press enter for Negotiate) > PAT
Enter personal access token > ****************************************************
Connecting to server ...
./config.sh: line 29:  6458 Segmentation fault: 11  ./bin/Agent.Listener configure "$@"
My Mac OS version = High Sierra 10.13.5

@qubuss Thank you for your feedback. The GitHub issues in this repository are intended to allow readers to submit feedback on our documentation. Can you let me know what topic you were reading when you encountered this issue, so I can route it to the right author? Thank you.

The repo for support of the agents is https://github.com/Microsoft/vsts-agent/issues

I'm closing this issue since it is not associated with a topic. To continue the discussion, please open a new issue that is associated with a topic by browsing to the topic on docs.microsoft.com and choosing Content feedback at the bottom. Thank you for your feedback.


Decoding occasionally fails when streaming large-ish buffers

Bug Report
Version
Running from 1987a40.
Platform
Linux nexus 5.3.1 #1-NixOS SMP Sat Sep 21 05:19:47 UTC 2019 x86_64 GNU/Linux

Crates
Affects tonic
Description
When running these lines I occasionally get an error message from prost informing me that a buffer underflow has occurred:
Error: Status { code: Internal, message: "failed to decode Protobuf message: buffer underflow" }

Sometimes this happens, other times it doesn't. It seems to be related to the number of bytes that I try to push through from the server to the client.
After digging around for quite some time, the failing requests all seem to be missing 3 additional bytes after the 5 byte header. When testing the blob-client with 10000 bytes a request ends up having a total length of 10005. After the buffer is advanced 5 bytes, the length is now 10000. The prost::encoding::bytes::merge function then reads off 3 bytes to get the length of the byte array from the proto, which is 10000 and then checks for underflow.
I'm stumped as to how these bytes are missing in some cases and are present in many others.
This prost issue (https://github.com/danburkert/prost/issues/98) suggests using {encode|decode}_length_delimited, but that didn't seem to have any effect when I swapped those calls in in prost.rs.

After making the following changes I'm now unable to reproduce the issue:
diff --git a/tonic/src/codec/prost.rs b/tonic/src/codec/prost.rs
index 495dc11..5c29a06 100644
--- a/tonic/src/codec/prost.rs
+++ b/tonic/src/codec/prost.rs
@@ -49,12 +49,14 @@ impl<T: Message> Encoder for ProstEncoder<T> {

     fn encode(&mut self, item: Self::Item, buf: &mut BytesMut) -> Result<(), Self::Error> {
         let len = item.encoded_len();
+        let additional = prost::encoding::encoded_len_varint(len as u64);
+        let required = len + additional;

-        if buf.remaining_mut() < len {
-            buf.reserve(len);
+        if buf.remaining_mut() < required {
+            buf.reserve(required);
         }

-        item.encode(buf)
+        item.encode_length_delimited(buf)
             .map_err(|_| unreachable!("Message only errors if not enough space"))
     }
 }
@@ -68,7 +70,7 @@ impl<U: Message + Default> Decoder for ProstDecoder<U> {
     type Error = Status;

     fn decode(&mut self, buf: &mut BytesMut) -> Result<Option<Self::Item>, Self::Error> {
-        Message::decode(buf.take())
+        Message::decode_length_delimited(buf.take())
             .map(Option::Some)
             .map_err(from_decode_error)
     }

However, I'm not exactly sure how to reliably test that this changes solves the problem.

I can reproduce with blobservice if I run the client in a loop.

@seanmonstar would hyper be cutting the body off by chance?

I wouldn't expect it to. Could try wireshark or something to sniff what DATA frames are sent/received.

Cool I'll look at it with Wireshark.

Looking at the logs coming out of h2 it looks like when 10008 bytes get sent from the server (10000 bytes from the bytes field in the message + 5 bytes for the tonic header + 2 bytes for the encoded length (10000) of the byte array + 1 for the proto tag) the first 10005 make it to the client, and the client tries to decode those 10005 bytes before waiting for the final 3 bytes it needs to fully decode the message.
Is there any reason the decoder would not wait for at minimum the number of bytes required to fully decode a message?

@seanmonstar After looking at frames using wireshark, I don't see anything anomalous. 10005 bytes are sent as DATA on stream 1, there are WINDOW_UPDATES on streams 0 and 1, then 3 bytes are sent as DATA on stream 1.
I noticed that between the 10005 bytes being sent the tonic decoding thread gets woken up and tries to decode the message before having all the bytes needed to decode it.

I'm not convinced this is related to threading or h2. I think that the issue most likely lies in tonic.
When receiving data by polling the body, it's possible to end up in a situation where the polling results in a buffer of size N where N >= length, but also that N < 5 + length.
Roughly the flow is like this:

Tonic polls for data, gets N bytes, where length <= N < 5 + length
Tonic sees that N bytes have been read, and if N >= length, advances the buffer by 5 bytes
Advancing the length reduces the buffer length by 5, now we have that length - 5 <= N < length
Since we need length bytes for the proto, prost fails with a buffer underflow


Fix bugs when using permalink /%postname%/

When using permalink /%postname%/
the resolver confuse between page and post.
It always return
array(2) {
  ["page"]=>
  string(0) ""
  ["pagename"]=>
  string(31) "request-uri-here"
}

even it is a post.

Mind tagging the new version for composer?

@tlovett1 A bit late, but done now.


Fix Elixir builds: Use HTTPS for installation url

Elixir builds are currently broken because of https://github.com/hexpm/hex_web/issues/371.
The build errors with:

The command "wget http://repo.hex.pm/builds/elixir/v1.3.0.zip" failed and exited with 5 during .

This is the output of that command:
wget http://repo.hex.pm/builds/elixir/v1.3.0.zip
--2016-06-30 02:28:55--  http://repo.hex.pm/builds/elixir/v1.3.0.zip
Resolving repo.hex.pm (repo.hex.pm)... <IP_ADDRESS>
Connecting to repo.hex.pm (repo.hex.pm)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://s3.hex.pm.s3.amazonaws.com/builds/elixir/v1.3.0.zip [following]
--2016-06-30 02:28:56--  https://s3.hex.pm.s3.amazonaws.com/builds/elixir/v1.3.0.zip
Resolving s3.hex.pm.s3.amazonaws.com (s3.hex.pm.s3.amazonaws.com)... <IP_ADDRESS>
Connecting to s3.hex.pm.s3.amazonaws.com (s3.hex.pm.s3.amazonaws.com)|<IP_ADDRESS>|:443... connected.
ERROR: no certificate subject alternative name matches
	requested host name ‘s3.hex.pm.s3.amazonaws.com’.
To connect to s3.hex.pm.s3.amazonaws.com insecurely, use `--no-check-certificate'.

https version works fine:
wget https://repo.hex.pm/builds/elixir/v1.3.0.zip
--2016-06-30 02:32:23--  https://repo.hex.pm/builds/elixir/v1.3.0.zip
Resolving repo.hex.pm (repo.hex.pm)... <IP_ADDRESS>
Connecting to repo.hex.pm (repo.hex.pm)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3579706 (3.4M) [application/zip]
Saving to: ‘v1.3.0.zip.3’

100%[====================================================================================================================>] 3,579,706   1.98MB/s   in 1.7s

2016-06-30 02:32:25 (1.98 MB/s) - ‘v1.3.0.zip.3’ saved [3579706/3579706]

I have deployed a workaround that skips the HTTPS redirect for elixir builds. I still recommend merging this PR of course.

Thanks @ericmj, that was very fast.

Ooops. Sorry about that.

This has been deployed.


Disabled:true causes Glyphicon to disappear

When i set a node to disabled:true the glyphicon disappears. The node still remains a hyperlink with mouseover color changes.

I just tested all modern browsers with a disabled node with both font-awesome & glyphicon icons and all worked as expected. I also experimented with image based icons and they too worked fine.
Using a font-based icon set should not cause issues - you may be using an old bootstrap version with image based icons? Could you please detailed information - bootstrap / glyphicon versions, jstree version, browser you are using - I will reopen the issue when further information is provided.
Best regards,
Ivan