[WIP] Coverage Docs v2

Reworked Coverage Docs
Restructuring of docs generation

Templates hold the style and html structure
Data templates (json-files) contain the content, that will be used and rendered when using the templates

script create_data_coverage.py creates the data-templates using the information from

implemented coverage (community/pro)
raw test metrics (community/pro; and other test sources)

The MD are generated with basic information (MD template is in the script)

another json-data file service_display_name.json was added with some additional service information

long-name/display name, short-name; link to api
used for the MD templates to display the full name (and make the search better)
currently static (needs to be adapted for new services) but nothing breaks if service is missing

Section Overview

grid view with simple search

Update workflow:

latest run: https://github.com/localstack/docs/pull/515
will update the data-templates (json files), and add any MD that is still missing (if new services are added)

TODOs

[x] Coverage Docs moved to references, but added as second item, right below "Configuration" (one option that was discussed with @HarshCasper)
[x] update links from "feature coverage"
[ ] add links for each service to the coverage page -> will be part of https://github.com/localstack/docs/pull/415
[x] remove old coverage docs

Future Nice-to-haves

pinned header for the table
sortable table
analytics events for "show tests"
more information about tests

maybe include snapshot skipped paths
maybe origin information (internal/external operation call)

logo/images for services + improved section overview

This is great!
Instead of having a dedicated section for coverage, any thoughts on moving the coverage pages as a children of the service doc page? Something like /user-guide/aws/<service>/coverage, and then have link in service doc to the coverage page.

Instead of having a dedicated section for coverage, any thoughts on moving the coverage pages as a children of the service doc page? Something like /user-guide/aws/<service>/coverage, and then have link in service doc to the coverage page.

@viren-nadkarni: majority of feedback was a preference to a separate section where all the coverage-information is collected for the services. But Harsh and me will still discuss this.

It would also be nice to define these terms:
Internal Test Suite | External Test Suite | AWS Validated | Snapshot Tested | Details

There is actually a section "Terminology" under the table and it is linked from the meta-header Testing*. But I guess it's not that obvious 😅

I did some final changes including:

removing the "Coverage" from the title -> it now uses the "real service name" as title

The title includes the abbreviated and the long-name of the service
The subtitle includes the api name
Generally: the md files are created with the script create_data_coverage.py using the service-name information from service_display_name.json (this file needs to be maintained manually). If the service is not found in the json, we use the default, e.g. the api-name.

changed the naming of version (indicating pro/community) in the table to edition

moved the "LocalStack Coverage" into the References section, and added it as #2 right below Configuration.

As the coverage contain subpages, by default the title in the menu is bold (if not selected!).
I am not sure if this is something we want to keep (I think it actually fits). But if not, I would need some helps to adapt the style here 🙏
See screenshot here:

In theory the PR would be ready (after a formal review of the included script), not sure if we want to do this before the official release or not 🙂

/cc @thrau @HarshCasper


Windows command line interprets package-scripts as launching package-scripts.js

Problem description:
Due to a little-known feature, at least to me, on windows, executing package-scripts from the cmd prompt or from npm scripts (even in Cygwin) causes the package-scripts.js file to be opened in the default application for .js files. see here.
Suggested solution:

Update documentation to only suggest using p-s. e.g. This solution.
Document this issue.

Possible additional solutions:

Remove the package-scripts bin.
Rename package-scripts.js.

Interesting! I like the idea of renaming package-scripts.js. could you make a PR?

What do you suggest as a new name?

So it just needs to have .js removed right? So package-scripts should work right? Can we do this is a way that's not breaking?

Just need the script base name to not collide with the command.  I guess this is why they use gulfile.js and gruntfile.js.  We could use p-s-file.js or similar.  Could add the new filename as an alternative.
The fact that this has never come up before suggests not many people have this issue.  I was surprised.  Simply suggesting p-s over package-scripts may be enough.

Oooohhh, I see... Hmmm... What do think? I like the idea of an alternative for now and maybe a deprecation warning to make it not breaking.
How about: package.scripts.js?

I'm thinking about renaming the who project to nps. So, the binary would be nps with no variants, and the file would be npsfile.js
Thoughts?

What do you think of #55?

I'm thinking of either doing:

rename package-scripts.js to package-scripts.config.js
rename package-scripts.js to package.scripts.js
remove the package-scripts alias

Thoughts?

Astestically speaking I don't like the dot seperators.  I'd personally psfile.js or p-sfile.js. But we should ensure this isn't conflict with other tools.
Deprecating the package-scripts alias should be good.  Since the npm name is p-s I assume that is what most people are using.  I think I was using the long version because that is what is shown here:  https://www.npmjs.com/package/p-s#this-solution

I like the package-scripts.config.js solution as mentioned in #55 along with the nps alias. Kudos for @Hypercubed noticing the issue on Windows 👍

I was thinking though... It's not really a config is it? Hmmm... I'm just not feeling great about any solution... I'm leaving toward removing the package-scripts alias. That'd be the most straightforward change I think...

Good point @kentcdodds... it isn't a config, they are scripts! Removing the alias sounds legit. That'd be a breaking change since some might be using the alias already, but I think that is okay.

Yeah, this'll push us to 3.0.0
Anyone want to file that PR? Should be pretty simple

How about deprecating for a while?  I imagine most users are using p-s and/or not on windows.  Just update the docs to remove package-scripts references and add a warning that package-scripts is going away.  Not a breaking change.
Of course, if they see the warning that means they are not impacted by this issue!

@kentcdodds I'm up for it, but I'm about to goto bed and am preoccupied tomorrow night. If anyone wants to take it before then feel free... otherwise I'll pick it up around Monday.

I'll do it now if y'all are alright with that :)

So I guess this is not an issue anymore?

Yes! As of 3.0.0, this is no longer an issue. Thanks for bringing it up :)
On Sun, Sep 18, 2016 at 4:37 PM Sorin Muntean<EMAIL_ADDRESS>wrote:

So I guess this is not an issue anymore?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kentcdodds/p-s/issues/54#issuecomment-247882429, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABbmDM0-Z-EH14spYWlQdTmRUF8yBFMpks5qra81gaJpZM4J0q1S
.

--

Kent C. Dodds https://twitter.com/kentcdodds

great! :dancer:


[Bug] Dubbo 2.7.23 Consumer not consuming properly when provider custom extends Dispatcher

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

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

Apache Dubbo Component
Java SPI Extensions (apache/dubbo-spi-extensions)
Dubbo Version
Dubbo Java 2.7.23,openJdk 1.11
Steps to reproduce this issue
1 CustomDispatcher implements Dispatcher and implements the related methods.
2 In the META-INF/dubbo directory, create a new file with the name org.apache.dubbo.remote.Dispatcher, with the content custom=com.xx.xx.CustomDispatcher
3 Call ProtocolConfig's setDispatcher method to set it to custom: protocolConfig.setDispatcher("custom")
4 start Provider sucessfully
5 When starting the consumer, it encounters the no provider error, and when tracing it through the debug, I found that when creating the NettyClient object, it calls org.apache.dubbo.remote.transport.dispatcher.ChannelHandlers#. wrapInternal -> ExtensionLoader.getExtensionLoader(Dispatcher.class)
.getAdaptiveExtension() can't find the class CustomDispatcher and gives error
PS: This error is not encountered when the Dispatcher's implementation class is changed to the built-in configuration.
What you expected to happen
The client-side Dispatcher should not be bound strictly and consistently to the server-side. If this is the case, there is no way to optimise the server-side Dispatcher schema when the client-side code is unable to make a task change that needs to be done
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You'd better not to customize dispatcher

thanks,I accept your suggestion.  I just customize a dispatcher that extends MessageOnlyDispatcher and makes a few changes about RejectedExecutionException. It does't affect the main process of MessageOnlyDispatcher. So,how should I do? @AlbumenJ


rm redundant lines in travis.yml

Why is this redundant?

@terrytangyuan duplication:
https://github.com/dmlc/xgboost/blob/master/.travis.yml#L5-L7
https://github.com/dmlc/xgboost/blob/master/.travis.yml#L24-L26

Oh I see. It wasn't fully loaded when I was viewing on my phone

can't rerun failed travis build :(

@terrytangyuan Can you look into the travis test failure? Seems it is introduced by some recent commits.


LSTM layer occasionally outputs junk

I am using ngimel's fork of torch-rnn to add cuDNN support. Every once in a while when first starting training, my loss function returns NaN, presumably from the LSTM layer (the only one from cuDNN) outputting junk. When I stop and restart it begins returning normal values (initial losses are in the 3-4 range). Is this a bug with cuDNN or something else? I am running CUDA 8 RC and cuDNN 5.0 on a GTX 960.

There's several cudnn 5.0 version. Can you specify the exact one?
From torch you can print cudnn.version

Version is 5005

Error appears to have been in my code, not CUDNN, as it appears I would occasionally pass junk data to the layer. Fixing my code appeared to stop the NaNs.


[Feature request] Allow choosing the acq_function during ask().

Currently, skopt requires the acquisition function and parameters to be given as an Optimizer object is created. However, it would be nice if this was specifiable for each opt.ask() call.
In particular, I'd like to use LCB with different kappa values to output a final table over the best points. I might also like to use acq-optimizer=samplilng with a large n_points once in a while to output "best so far" stats.

Can be done now with the new update_next function, will be included in release 0.7.2
acq_func_kwargs = {"kappa": 0}
opt = Optimizer([(-2.0, 2.0)], "GP", n_initial_points=1,
                acq_func="LCB", acq_optimizer="sampling",
                acq_func_kwargs=acq_func_kwargs)
opt.run(objective, n_iter=20)

acq_func_kwargs = {"kappa": 100000}
opt.acq_func_kwargs = acq_func_kwargs
opt.update_next()
opt.run(objective, n_iter=20)


The normal node in the p2p network cannot connect.

Some p2p nodes can't connect to any node. You must delete the node.json that rejects your peer and restart the peer node before you can link yourself to the peer. Regularly deleting node.json and restarting p2p can solve this problem, but it is rather rude. The scoring system in p2p rejects the system, and from time to time it will isolate some good nodes and never connect.

I am seeing the same issue as well.

This issue has been open a long time ago when the network was known to have lots of issues.
Since the networking looks stable now, and since this issues doesn't really provide much information, I'm going to close this.


Register SlashCommandGroups

As you describing it in your README, you can create a slashgroup. But how is it possible to register these groups as they are not a SlashComandModule.

I got it to work :)

@tygore587 Glad to hear that you could get it to work. However, it would be great if you could share the solution here. I stumbled upon the same issue and can't figure out what I'm doing wrong. Any guidance?
[SlashCommandGroup("notes", "Manages notes for users")]
public class NotesModule : SlashCommandModule
{
    [SlashCommand("add", "Adds a new note to the user")]
    public async Task Add(InteractionContext ctx, [Option("User", "The user")] DiscordUser user, [Option("Text", "The actual note"), RemainingText] string text)
    {

    }

    [SlashCommand("list", "Lists all notes of a user")]
    [Aliases("ls")]
    public Task List(InteractionContext ctx, [Option("User", "The user")] DiscordUser user)
    {

    }
}

I updated the documentation with a pr. See here
You need to have a container which inherits from SlashCommandModule and in there you need to create Slashcommandgroups which also inherits from SlachCommandModule.


Translations update from Weblate

Translations update from Weblate for FriendsOfShopware/FroshPlatformBunnycdnMediaStorage.
Current translation status:

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

Project coverage is 5.81%.  Consider uploading reports for the commit db77d03 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##              main     #73   +/-   ##
=======================================
  Coverage     5.81%   5.81%
  Complexity      99      99
=======================================
  Files           10      10
  Lines          275     275
=======================================
  Hits            16      16
  Misses         259     259


Add hardware for onboard voltage sensing.

Currently the controller uses the voltage sensed at the ECU and transmitted via CAN to adjust the heater PID.
Chassis wiring and ground points could mean reasonable differences between the battery voltage at the module and the ECU.
Suggest incorporating an ADC channel and appropriate divider on the module to measure "local" battery voltage for heater compensation.

Another argument for same would be having 042 design self-contained.

https://rusefi.com/forum/viewtopic.php?p=47055#p47055

Another option https://github.com/mck1117/wideband/issues/194

hardware done 6831f5a1ae87e985e1e40e1fd2b6fe2107d40a72


Update composer to version 1.6.5 from 1.5.2

Update composer binary to the latest version

@maximest-pierre was this tested locally? Does it need to be reviewed?

@tzarebczan this is just update binary for composer to the newest version. and same with jquery before.

It was tested locally. To test it you only need to clone the repo in a temp directory and run the dev.sh.

LGTM, pushing.


Added sample for label-specific Github Actions

What type of PR is this?
enhancement
Which issue does this PR fix:
#507
What does this PR do / Why do we need it:
This PR contains a sample Github Actions workflow that only runs when a specific label is present
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

It seems that label-specific workflows as I used them in this commit  only run when the label is added. I will look into the possibility of adding label-specific execution paths to the existing workflows.

Pending items:

[ ] investigate missing coverage
[ ] add mandatory check for - minimal-check / Build and test ZOT - minimal edition (pull_request)
[ ] add mandatory check for - complete-check / Build and test ZOT (linux, amd64) (pull_request)
[ ] remove mandatory check for - Build and test ZOT
@rchincha please confirm the check related changes.

Let's close this for now.


Problems with links and state

I am experimenting inconsistencies, when I try to set the path to certain states i.e. signUpPath  = '/register' corresponds to STATES.SIGN_UP and loginPath='/login' corresponds to STATES.SIGN_IN.
e.g. when I am in /password-reset and click to Sign In which redirects me to /login the state for the form is STATES.SIGN_UP instead of STATES.SIGN_IN.
Anyone experimenting the same?

Are you using Flow Router or React Router?

I am using FlowRouter.

oki, yeah, I could have guessed looking at your other issues. 👍

Closing due to inactivity


Extend dashes forwarding test

Closes https://github.com/rust-lang/cargo/issues/2213
Looks like this was fixed when we've transitioned to clap!

I've also verified that the test indeed fails pre-clap
@bors r+

:pushpin: Commit 30e52ca has been approved by matklad

:hourglass: Testing commit 30e52ca906dbb5b9c15345beecfea23c12dff895 with merge 19962e21c0bf893a5cda9e251ddd4fd8acc3ad5e...

:sunny: Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 19962e21c0bf893a5cda9e251ddd4fd8acc3ad5e to master...


GH-37021 [Java][arrow-jdbc] Pluggable getConsumer

Rationale for this change
This was discussed in this thread. The getConsumer implementation depends on the implementation for getArrowTypeFromJdbcType, especially for timestamp objects. Since we can provide a different implementation for JdbcToArrowTypeConverter it follows we should be able to provide a different implementation for JdbcConsumerGetter
What changes are included in this PR?
Adds a way to configure an alternate JdbcToArrowUtils.getConsumer function through JdbcToArrowConfigBuilder.setJdbcConsumerGetter.
It also throws a more helpful exception from the default implementations when a provided type is not mapped.
Are these changes tested?
No, the default behavior remains unchanged.

Related: #37021
Closes: #37021

Hmm, it's not clear why, but CI is failing - does the test suite pass for you?

Hmm no... it is hanging on TestFlightSql.testGetTablesResultFilteredWithSchema, trying to track it down.

ahh tracked it down to an uncaught exception with unmapped JDBC types


Remove Block Personalization code

Remove the block change and slider functions and remove them from the visual elements from the welcome screen.

Resized the blocks that make up the program on the Building Screen so they're bigger, independent of the UI slider that was removed in this issue.
To do this, two lines of code had to be removed from MainMenuViewController.swift:
after line 27 print("block size " , blockSize):
myDestination.blockWidth = blockSize
myDestination.blockHeight = blockSize
Saving these deleted lines here in case we need then later.


Release deploys slightly different artifacts to libs-release-local and maven central

Current steps are:
$ ./mvnw deploy

# later

$ ./mvnw deploy -Pcentral

See slack discussion
https://pivotal.slack.com/archives/spring-cloud/p1487362868006845
Possible workaround:
http://stackoverflow.com/questions/635154/multiple-deployments-in-maven#answer-1259799

IIRC, metadata (times?) changes, therefore jar change, therefore signatures change.

This blog article illustrates what would be quite ugly: http://techtraits.com/build management/maven/2012/06/24/Maven-Deploying-multiple-artifacts-from-one-build.html
Essentially, if you manually specify EVERY file (signature files included), then you might be able to upload everything where you wish.

./mvnw deploy depends on install, which effectively rebuilds the jar. While this should in theory detect no changes due to the first deploy step, sometimes the plugin decides to recompile the sources, even if they're not changed - e.g. courtesy of this bug https://issues.apache.org/jira/browse/MCOMPILER-205 . There's no intrinsic guarantee that a recompilation and JAR rebuild will not happen.
It shouldn't be necessary to deploy artifacts in two places, though.
One proposal I've made in the past (and I believe @rwinch and @trevormarshall were experimenting with that as well, independently) would be to deploy releases directly to Maven Central and mirror them in Spring repositories. Snapshots and milestones would only be deployed directly to Spring snapshot and milestone repos.

@mbogoevici thanks for details I couldn't remember.

deploy releases directly to Maven Central and mirror them in Spring repositories

would be great.

How about taking the Spring Framework route? Publishing artifacts to repo.spring.io, promote artifacts in a second step to Maven Central: https://github.com/spring-projects/spring-framework/wiki/Gradle-build-and-release-FAQ#what-about-publishing-artifacts-to-maven-central
I think this approach requires the release to be built with a CI with the according plugin to support artifact/build promotion.

That is what we were avoiding


+ node should use convolve from Distributions.jl

help?> convolve
search: convolve ConcurrencyViolationError

  convolve(d1::Distribution, d2::Distribution)

  Convolve two distributions and return the distribution corresponding to the sum of
  independent random variables drawn from the underlying distributions.

  Currently, the function is only defined in cases where the convolution has a closed
  form. More precisely, the function is defined if the distributions of d1 and d2 are
  the same and one of

    •  Bernoulli

    •  Binomial

    •  NegativeBinomial

    •  Geometric

    •  Poisson

    •  Normal

    •  Cauchy

    •  Chisq

    •  Exponential

    •  Gamma

    •  MvNormal

  External links: List of convolutions of probability distributions on Wikipedia
  (https://en.wikipedia.org/wiki/List_of_convolutions_of_probability_distributions)

This probably would also need a specific [compat] version for Distributions.jl

Implemented by a9841d4 and a5f72c8 in #69.
Currently the normal distribution resorts to specialized methods, as the convolve function accesses properties of the distribution, e.g. dist.Σ, whereas this property might not always be available. One way to solve this is to extend the univariate and multivariate normal distributions with properties, similarly as in https://github.com/JuliaLang/julia/blob/3bf9d1773144bc4943232dc2ffaac307a700853d/stdlib/LinearAlgebra/src/cholesky.jl#L497-L509. This might actually be a nice property to have in order to make the package more user-friendly. Just let me know what you think of it.
Not sure about the previous [compat] comment.

Personally I don't like overriding getproperty function as for me it seems too dangerous and "dynamic".

This might actually be a nice property to have in order to make the package more user-friendly.

As I understand this considered to be non-Julian approach (though even Julia Base uses it, but I assume this is just a very old code you shared). We already support cov() function for every Gaussian distribution, so there is no need for extra dist.Σ imo. And this is exactly the problem here, you cannot easily "add" properties to your structure, but you can easily add new methods to existing functions like cov(::MyCustomPrettyGaussian).
We probably should just define Distributions.convolve method for our own distributions, smth like:

import Distributions: convolve

function Distributions.convolve(left::UnivariateNormalDistributionsFamilty, right::UnivariateNormalDistributionsFamily)
    cleft  = convert(NormalMeanVariance, left)
    cright = convert(NormalMeanVariance, right)
    return NormalMeanVariance(mean(cleft) + mean(cright), var(cleft) + var(cright))
end

And in the same way for MultivariateNormalDistributionsFamily. We can put it in distributions/normal.jl. This file already has some "common" methods defined for the whole family.

Not sure about the previous [compat] comment.

We have a [compat] section in our Project.toml with minimal versions of packages required. AFAIK convolve has been added recently (or at least documented recently) - we should double-check that convolve function exists in the [compat]-ible version of Distributions.jl. We can do it later when we will merge your PR. Just something to keep in mind.

@bartvanerp  or instead of convert in my code snippet we could just use mean_var(). I think mean_var() is a better approach.

Another benefit of our custom convolve function would be that we can use it in other places in our code. However I don't recall we ever needed it in any other place....

Fixed in #69


Automatic PR of restapi_auto_jugeorge_master into master

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2319

Broken PR


style(global): apply design

add header(nav-bar); change background color

Having trouble to use native tree icon, so I delete native title and move everything into 'extra'.

Found one problem: copy icon at TABLE tree will have weird borders if clicked. Not sure what went wrong.


feat: display hardforks on startup

Addresses https://github.com/paradigmxyz/reth/issues/1343
Currently the startup output looks like this.
Hardforks:
      Frontier      : Block(0)
      Homestead      : Block(1150000)
      Dao      : Block(1920000)
      Tangerine      : Block(2463000)
      SpuriousDragon      : Block(2675000)
      Byzantium      : Block(4370000)
      Constantinople      : Block(7280000)
      Petersburg      : Block(7280000)
      Istanbul      : Block(9069000)
      MuirGlacier      : Block(9200000)
      Berlin      : Block(12244000)
      London      : Block(12965000)
      ArrowGlacier      : Block(13773000)
      GrayGlacier      : Block(15050000)
      Paris      : TTD { fork_block: None, total_difficulty: 0x000000000000000000000000000000000000000000000c70d808a128d7380000_U256 }
      Shanghai      : Timestamp(1681338455)

I'll explore more on how to align those : on same vertical line
print the TTD type fork to something human readable

Hi @seroze, this PR looked stale so I rebased it and addressed @mattsse's review comments. I hope you don't mind; I retained your original commits in the new PR, so you will be counted as a contributor because of the first stab you took.
I am closing this PR in favor of the new one at #3227
Thanks for contributing ❇


Split out mismi-s3-core.

Bit fed up with dealing with amazonka just for Bucket, Key and Address.
This splits out a core set of data-types and parsers for these and a few other simple mismi-s3 types with minimal dependencies.
This has been done with a backwards compatible shim left in mismi-s3 so when upgrades projects won't know the difference until they want to know (and more importantly will allow a staggered roll-out).
@nhibberd

🚀 from me.

Looks good :facepunch:

:+1: on rebase and :green_apple:


Adding autolabeler workflows to Python templates

Just thought it would be interesting to add an automatic labeler workflow to the Python templates.

Codecov Report

Merging #86 (0fd1fcc) into main (0f49fa1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   88.33%   88.33%
=======================================
  Files           6        6
  Lines         120      120
=======================================
  Hits          106      106
  Misses         14       14

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 0f49fa1...0fd1fcc. Read the comment docs.

Requesting @asbfall review on this as its affects the templates he implemented some months ago.

Looks great to me! I am a real fan of labels... anything related to automating their tagging is perfect! Thanks @jorgepiloto!

Merging this 🚀


Alias issue

Hi, I've got a question about aliases. Our server is having issues for some reason with the automatically hashed aliases on edge selections. Other aliases seem fine.
This is a snippet of the request body. Green works, red doesn't:
- "query": "query {\n  postsquery__19mo3grcmntt4: posts {\n    __typename\n    edgespostConnection__5zdq2eh8km7i: edges {...
+ "query": "query {\n  postsquery__19mo3grcmntt4: posts {\n    __typename\n    edges {...
+ "query": "query {\n  postsquery__19mo3grcmntt4: posts {\n    __typename\n    edges: edges {...

The red aliases case returns an empty array for the edges
{
  "data": {
    "postsquery__19mo3grcmntt4": {
      "__typename": "PostConnection",
      "edgespostConnection__5zdq2eh8km7i": []
    }
  }
}

I'm aware this should be working on our end, and it must be an issue with our server implementation. Supposedly we're using https://gqlgen.com server side, but the team has yet to work out what the issue is.
As a route to resolve this I tested disabling the hashed aliases in a fork by just returning the original name in GraphQLField, and everything seems to be working fine.
public var alias: String? {
    switch self {
    case let .leaf(name, parent, arguments),
        let .composite(name, parent, _, arguments, _):
+        // using aliases that are different from the actual name breaks edges in our graph, disabling for now
+        return name
-        return "\(name.camelCase)\(parent.camelCase)_\(arguments.hash)"
    case .fragment:
        return nil
    }
}

My question is what issues will this cause if any? I assume the aliases are used for some kind of disambiguation. In what scenario would this cause problems?

You are right, I added aliases as a way to deterministically disambiguate between different calls to the same field. GraphQLSpec doesn't restrict that and there's no other "easy" way of doing the disambiguation when you try to conceal GraphQL entirely.
Having used elm-graphql as inspiration, it seemed like a natural thing to do since it helps us cover more cases. Thinking about it now, it might be better to allow it as a flag (e.g. using the hashed variation by default but letting the devs opt-out).
The only real hiccup is that you MUST NOT call the same field twice in a single part of the query (which is obviously a super edge case for most developers).

Did you need to change anything else? It night make sense to extend the client and the builder.

That makes sense. I can't imagine us querying the same field twice though. Would would be the use case of that? And if the hashing wasn't there wouldn't it just merge the duplicate calls?
I didn't need to change anything else to get it working, though I guess ideally it wouldn't even write the alias (which is now the same duplicate name) into the query, but that seemingly required a whole bunch of other code changes.
- field: field {...} // my fix
+ field {...} // ideal

As another note, these hashed aliases would probably make http caching more difficult, especially as the Swift hashing used is non deterministic

@yonaskolb do you know of any good alternative that we could use to deterministically hash fields? Preferably something in the standard library or something with low overhead?

Perhaps a counter by field that increments on duplicate field access? And for the 0 case it wouldn't even use an alias? That would seem to solve both issues, but I don't know the intricacies of the system well enough to know if that would work. I also don't quite understand the need to create seperate aliases for multiple accesses so my picture of the situation is probably off.

Thinking about it now, it might be better to allow it as a flag (e.g. using the hashed variation by default but letting the devs opt-out).

I would also say that it would be great if the API users could decide whether they would like to use aliases or not, for multiple reasons:

Caching, as said earlier
Mocking - when working with mocked JSON responses, it is nearly impossible to decode the response with aliases right now
Testing - Writing unit tests for queries (and decoding) is also really hard with aliases

The only real hiccup is that you MUST NOT call the same field twice in a single part of the query (which is obviously a super edge case for most developers).

I agree that because of this reason it is not a bad idea to allow aliases to be turned off.

@tuxi7x Great points! I personally also agree it would be better if the developer could determine the usage of aliases as well as what the generated query will eventually 'look' like. I'm exploring some new ideas for v6 that are aimed at improving the developer experience. I'll start a Discussion shortly and tag you all there next week, I think that's a better place for these discussions 👍
Thank you also @yonaskolb for raising this 👍


Invalid http upgrade

Hi,
I am new to IOS Development, While implementing a web socket for my app i got an error as invalid http upgrade. It is happening when i send message to the remote backend/Server, it is working in localhost. If i change the request type to https/wss it shows CFNetwork SSLHandshake failed (-9847). Eagerly waiting for the reply

Invalid HTTP Upgrade is a response from the Web Server when trying to upgrade to a WebSocket connection. Sounds like there is a network configuration between your WebSocket client and server, hence it working fine on localhost. I would start by looking there. The SSL issue is being caused either by the network configuration issue just like the standard HTTP or ATS.
https://ste.vn/2015/06/10/configuring-app-transport-security-ios-9-osx-10-11/

Thank you for your Reply,
I added the following keys to my info.plist but still it is not working. Showing the same error as before
NSAppTransportSecurity

NSAllowsArbitraryLoads

Then it probably is a network configuration issue. I would start by doing some network troubleshooting and see what is going on there (check WebSocket server setup, your load balancer or proxy if you have one, or maybe run a packet capture). Doesn't sound like this is an issue with Starscream though and there is not anything I can do to help to narrow it down as it seems specific to your configuration.


Install-TheFucker.ps1 Breaks $Profile when $Profile already exists.

PoShFuck adds "Import-Module posh-fuck" to the $profile script. Only problem is it adds no new-line before doing so, so if you have anything in $profile already, it breaks the $profile.

Thanks bryanwinstead, you're right and it should be fixed now


face_mask rearrange in Autoencoder Forward Pass

Hi Phil,
Happy New Year! 🎉🎉🎉
I encountered a minor issue in the code when I set return_recon_faces to True. During the execution of the autoencoder's forward method, I noticed that face_mask is rearranged.
face_mask = rearrange(face_mask, 'b nf -> b nf 1 1')
recon_faces = recon_faces.masked_fill(~face_mask, float('nan'))

However, after applying masked_fill to recon_faces, it seems that the shape of face_mask isn't restored. This is causing an error when trying to perform a repeat operation on face_mask later in the process.
face_mask = repeat(face_mask, 'b nf -> b (nf r)', r = 9)

Perhaps the shape of face_mask could be restored right after the masked_fill operation.
Best regards,
Xueqi Ma

Thank you for your prompt reply. I hope your holiday season is filled with joy and relaxation.
In the meantime, I'll consider working on a Pull Request to address this issue. Once I have something ready, I'll submit a PR for your review.


Simplify class management and change of density value formats

What type of PR is this?
Add one of the following kinds:

correction
enhancement/feature

What this PR does / why we need it:
Including proposal of solution for:
#56 Mapping of enum and classes in densitydata & simplify class CellPopulationDensityData
#58 Integer value format for population density values
Which issue(s) this PR fixes:

Fixes #56 #58

@bigludo7 please for your review
Consider the change in the schemas to validate if this proposal is ok for you, compared with a alternative:
 propertyName: dataType
        mapping:
          NO_DATA: '#/components/schemas/NoDataPopulationDensityData'
          LOW_DENSITY: '#/components/schemas/LowDensityPopulationDensityData'
          DENSITY_ESTIMATION: '#/components/schemas/DensityEstimationPopulationDensityData'

Hello @jgarciahospital
Work for me !
Thanks


Custom Leaves Issue

Hello and thanks in advance for reading/replying this thread... I wanted to request if it is possible and doesn't mess too much with the mod, that you may remove the custom leaves behavior and make a "Lite" Mod Version that uses vanilla leaves, because is messing in a wierd way with two of my mods ("Physics mod" and "Walljump mod") In the manner of that I can't walljump this leaves because of the realistic leaves behavior your mod gives to the leaves and with the physics mod every time a block is destroyed particles of said block crumbles to the ground leading to mayor fps drops due to a lot of leaves particles falling arround me on the forest  ( I tried disabling all the leaves on the physics mod side but doesn't seem to work because the leaves on the moded trees are custom in some way) Is Ok if you won't do it but it couldn't hurt to ask xD Thanks in advance

Using vanilla leaves isn't an option due to the custom behaviour required for leaves growth (changing the vanilla leaves would cause more incompatibilities than it would fix, I expect).
You can already turn on the vanillaLeavesCollision option in the server config (/saves/[world]/serverconfig) if you'd like, which should fix the wall jump. As for the Physics mod they have a configurable blacklist which you should add the DT leaves blocks to.


Error in assuming access role .. AppRunnerECRAccessRole

Hi
I've selected the option to have App Runner create a role for accessing private ECR but then the error is raised. Any advice?

I went through the same thing.
I received deployment log.
"[AppRunner] Failed to pull your application image. Be sure you configure your service with a valid access role to your ECR repository."
Even I provided App Runner with an access role.

I get this same error message:
"[AppRunner] Failed to pull your application image. Be sure you configure your service with a valid access role to your ECR repository."
The problem:
Service running in us-west-2 cannot pull from image in ECR ap-southeast-2.
Solution:
Create ECR in the same region as App Runner Service.
Not sure what's wrong, but I would expect App Runner to be able to pull from any region's ECR.
I hope this helps.

Hi @kzima, I successfully pulled and run my cross-region (and also cross-account) images on App Runner.
Could you possibly share your policy document for the ECR repository in ap-southeast-2 region? (Make sure that all AWS account IDs and user names are masked before putting it here)

Haven't heard back for a while so going to close out this ticket.


Is there anyway you could make st support arabic or any unicode for that matter? or is that just bloat?

st top
gnome-terminal bottom

It is not bloat and it does not only limit Arabic, Persian and any other Right-To-Left language suffers from the same issue.
It's not about the Unicode neither. As you can see characters are well rendered. It's about Bi-directional processing. Everything is rendered correctly but misplaced and not attached to the other characters.
KDE Konsole does an awesome job in handling bidi texts. If I wanted to work on this I would start by investegating that.
(It's using Qt tho, which is generally good with bidi so maybe it's a dead end. Regardless it stays the best showcase for good bidi rendering)

also look into the harfbuzz C++ library. It is used to correctly render unicode characters in the right form (right to left or left to right, initial, medial, final and isolated, etc) - though you'll have to write and compile it yourself.

I'm not going to look into this. If someone finds an elegant solution and wants to open a PR, they are welcome to.

For people searching for a solution this should work fine since it worked for me.


tracking: s3 source

rfc: https://github.com/risingwavelabs/rfcs/pull/22

Final solution rfc: risingwavelabs/rfcs#31


Added whatsnew notes on netcdf delayed saving and Distributed support.

Here's my suggestion explaining the benefits of delayed saves, and Distributed.
We could consider a new UserGuide section for this, but I think we can still argue that this is a special usage + doesn't belong there.

@bjlittle think you suggested this ?


Wrap as_real_imag

@isuruf I'd like to work on this issue.

@isuruf Can you please give me some pointers so that I can work on this?

Have a look at as_numer_denom whose function signature is the same as as_real_imag


Using VCS and SSH connections together for the same workspace

Feature description 💡
Hello @alfespa17, @jcanizalez;
I would like to briefly summarize what I need. While creating the Workspace, I connected the Terraform repo to Terrakube via the VCS GitHub App. However, in the Terraform repo, the private module pulls my modules by making an SSH connection to the repo. Unfortunately, this is not available in the current Terrakube structure, you have to choose one of the two. However, this is available in Terraform Cloud. Is it possible to add this feature to Terrakube?
For example Terraform Cloud screenshots:

Anything else?
No response

I think I can add this feature without a big change in the code, I will try to add it in the following days

Thank you very much, @alfespa17


Add UI interface with project logo, favicon

Make a proper interface as web template with project logo as WeatherForcasting
And also a favicon to the project same as logo
Don't need to change any inner content of the project.

i would like to work on the issue under JWOC

@Animesh606 I would love to work on this issue under JWOC. Please assign this issue to me

@Animesh606 I would like to work on this issue under JWOC'24.

@Animesh606 I would like to work on this under JWOC

@rakesh-paulraj you were assigned the issue 2 days ago. Have you done with it? Just inform updates.

Just working on my other jwoc issues can ,i haven't started it yet can i get some time please

@rakesh-paulraj you can take one day more as per the rules..
Otherwise the next one got a chance

Thise issue is solved by @rakesh-paulraj with PR #15

Thank you so much


With cookie auth not work

I tried to set up the example: https://github.com/zeit/next.js/tree/canary/examples/with-cookie-auth
Whether in v1 or v2, I can't get my token back at the server side loading.
On the client side, everything is okay, but nothing on the server side... an idea?

hey @Gmulti, could you show me a repo to look at so I can try to replicate?
Also, I'm not sure what you mean about getting your token back at the server, if the client works fine (specifically the profile page) you should be getting the token in the server.

Unfortunately I can't provide you with the repo in public.
Here is an example of what I use :
 static async getInitialProps(ctx) {
        const { store } = ctx

        const accessToken = auth(ctx)
        let projects
        console.log('AT : ', accessToken)
        if (accessToken) {
           // Get Projects
            console.log(success)
            projects = result

            }
        }

        return {
            accessToken
        }
    }

Unfortunately I can't provide you with the repo in public.
Here is an example of what I use :
Function auth

export const auth = ctx => {
  const { accessToken } = nextCookie(ctx)

  if (ctx.req && !accessToken) {
    ctx.res.writeHead(302, { Location: '/login' })
    ctx.res.end()
    return
  }

  // We already checked for server. This should only happen on client.
  if (!accessToken) {
    Router.push('/login')
  }

  return accessToken
}

HOC withAuthSync :

import { Component } from 'react'
import { auth } from '../helpers/auth'
// Gets the display name of a JSX component for dev tools
const getDisplayName = Component =>
Component.displayName || Component.name || 'Component'
export const withAuthSync = WrappedComponent =>
class extends Component {
static displayName = withAuthSync(${getDisplayName(WrappedComponent)})
    static async getInitialProps(ctx) {
        const accessToken = auth(ctx)

        const componentProps =
            WrappedComponent.getInitialProps &&
            (await WrappedComponent.getInitialProps(ctx))

        return { ...componentProps, accessToken }
    }

    render() {
        return <WrappedComponent {...this.props} />
    }
}

export default () => {
return target => withAuthSync(target)
}

My Page :

@withAuthSync()
@Head()
@Page()
class PageHome extends Component {
static async getInitialProps(ctx) {
const { store } = ctx
const accessToken = auth(ctx)
console.log('AT : ', accessToken) // Null on server side / Good on client side
if (accessToken) {
// Get Projects
}
    return {
        accessToken
    }
}

}

In the meantime, I'll still try to give you a quick example in case it doesn't work

Okay, I got it!
In development, I use a server node that already contains isomorphic-fetch.
However, this is not the case during the deployment on v2 and it no longer existed.... ! And no logs to find out.
I added it and it's all good
It's all right, I'm closing the issue

Awesome! I'm glad you figured it out


Update nokogiri

Current version has a vulnerability: https://groups.google.com/forum/#!topic/ruby-security-ann/aSbgDiwb24s

:ship:


I don't use vagrant and i have this issue ?

I can't run dsh command's
dsh st
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 30: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 192: [: ./: unexpected operator
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 128: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 30: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 202: [: unexpected operator
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 128: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 139: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 128: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 139: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 149: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
/home/asoo/Documents/projects/cn/.docker/bin/dsh: 155: /home/asoo/Documents/projects/cn/.docker/bin/dsh: [[: not found
"vagrant" executable not found.

@softeg what kind of shell do you have? echo $SHELL

BTW what is your OS?
dsh has not been much tested on Linux yet (outside of Tiny Core used by boot2docker).

Hi,
I have the same issue into Linux ((K)Ubuntu 14.10).

Can you test on Windows and/or Mac?

@comxd  can you change the script preamble to #!/bin/bash in .docker/bin/dsh and see if that works for you?
I have a workaround available now to fake bash in boot2docker, so there won't be a need to stick with sh standards and limitations.

Oh yes, it working good with "#!/bin/bash"!

But I have this error:
docker-compose pull
Pulling db (blinkreaction/mysql:5.5)...
Couldn't connect to Docker daemon at http+unix://var/run/docker.sock - is it running?

If it's at a non-standard location, specify the URL with the DOCKER_HOST environment variable.

@comxd looks like your docker daemon  is not running.
Does docker version and sudo docker run hello-world work for you?

Sorry, I finally now can run this command without error :/

Doh, same issue here... wouldn't run properly in linux (ubuntu) until I made this change.

/bin/sh is a link to /bin/dash  in Ubuntu since 8.04 /bin/bash is still the user shell. My guess is these scripts are written on OSx which uses bash as sh.

Fixed in v1.0.0-rc1


Enhance exception message for debugging purposes

Code looks good to me. @rdunlop - merge if this meets your needs?


Work on User Page

Put all work for this issue in a branch called issue-36.
For this issue:
-Finish with page format and stylization

Added Accordion to UserPage.jsx


Add Wercker to CI/CD Tools

Wercker is a CI/CD automation platform. We're pretty agnostic as to the end deploy target although we specialise in Docker (Docker-native some might say;) )
You can use Wercker to setup your build, test, deploy pipelines and deploy anywhere although we play especially nicely with Kubernetes.
Here are some resources:
Deploying to Kubernetes: http://blog.wercker.com/deploying-a-microservice-to-kubernetes
Kubernetes round up: http://blog.wercker.com/kubernetes-kubecon-roundup
CloudNativeCon Pancake Breakfast - Kubernetes Takes Us Beyond 'Peak Confusion':https://soundcloud.com/thenewstackanalysts/tns-analysts-show-111-kubecon-pancake-breakfast-kubernetes-takes-us-beyond-peak-confusion
Virtual private pipelines: http://blog.wercker.com/virtual-private-pipelines-has-arrived
Features: http://www.wercker.com/features
:)

Looks great, thanks.
On Mon, Nov 28, 2016 at 7:34 PM, Victoria Domalik<EMAIL_ADDRESS>wrote:

Wercker http://www.wercker.com/ is a CI/CD automation platform. We're
pretty agnostic as to the end deploy target although we specialise in
Docker (Docker-native some might say;) )
You can use Wercker to setup your build, test, deploy pipelines and deploy
anywhere although we play especially nicely with Kubernetes.
Here are some resources:
Deploying to Kubernetes: http://blog.wercker.com/
deploying-a-microservice-to-kubernetes
Kubernetes round up: http://blog.wercker.com/kubernetes-kubecon-roundup
CloudNativeCon Pancake Breakfast - Kubernetes Takes Us Beyond 'Peak
Confusion':https://soundcloud.com/thenewstackanalysts/tns-
analysts-show-111-kubecon-pancake-breakfast-kubernetes-
takes-us-beyond-peak-confusion
Virtual private pipelines: http://blog.wercker.com/
virtual-private-pipelines-has-arrived
Features: http://www.wercker.com/features
:)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/issues/25, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8MBhlenKD6Jxp-7ka-9P3tO64uxc16ks5rC51KgaJpZM4K-h8l
.

--
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://cncf.io/
tel:+1-415-233-1000

Wercker is now included under CI/CD in v0.9.3


Clean up apps and directories.

Cleaning out Siegfrieds HTML directory and the unused blazer app directory.

@mrawls not sure if you have any vested interest in keeping the "blazer" app around or no.

I'd envisioned that being a thing in the future that identifies streaks or does halfway interesting science/statistics on images! It will remain empty for some time, though, so feel free to toss it for simplicity and we can add it when we actually get to it.


Update framerate using frame count and time duration

Instead of keeping fixed 30fps, get current timestamp
on frame count zero and update framerate using full
video duration upon closing file

I had to do this to be able to get a consistent video from a remote WebRTC browser camera. Otherwise the video track file would be out of sync with audio track file. I know this is not optimal as the ideal would be to get this information (together with width and height) from WebRTC track, which I couldn't find in the code


Robot.sublime-build

Hi,
I have noticed that every time the plugin updates, my Robot.sublime-build file is overwritten. Any chance this could be avoided?

You should define the user settings, they are not overwritten by the plugin update.  From Sublime menus: Preferences -> Package Settings -> Robot Framework Assistant -> Settings - User

Okay, I didn't know that the build settings can also be defined here. So I can just add this to my settings:
"cmd": ["python", "-m", "robot.run", "-d", "C:\\tmp\\output", "-L", "TRACE", "$file"],
"selector": "source.robot",
"variants": [

    { "name": "Selects the test cases by tag",
      "cmd": ["python", "-m", "robot.run", "--include", "REPLACE_WITH_TAG", "$file"],
    },

    { "name": "Selects the test cases by name",
      "cmd": ["python", "-m", "robot.run", "--test", "REPLACE_WITH_TEST_NAME", "$file"]
    }
]

That is a build setting? But in practice yes, one can also define also user specific build settings. Just remember to choose the right build settings when starting Robot test.

That was my initial question. Is there a way of not overwriting the build settings on update?

Ah. I did miss read the question, sorry. But same rule applies, define a user build setting in the same folder where other user defined settings goes. The user defined settings are never overwritten by the plugin.

Okay, this worked thanks for your help!

No problem and glad to help


Dynamic Resolution does not seem to be supported for Windows version

When starting a RDP session using Windows remote desktop - the session does not automatically resize with window size. https://superuser.com/questions/98494/windows-rdp-possible-to-resize-resolution-on-the-fly

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

ID: 5d746fc6-0121-c9dc-f552-2d53a902922f
Version Independent ID: a7508e87-b2ac-c474-f168-5af94c250332
Content: Remote Desktop clients feature comparison
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-features.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Update index.html

I dont know how to complete this assignment

re-check

i redid all steps correct but now wont push [#)

you can see everything here
https://labs.cognitiveclass.ai/tools/theiadocker/?md_instructions_url=https://cf-courses-data.s3.us.cloud-object-storage.appdomain.cloud/IBMDeveloperSkillsNetwork-CD0101EN-SkillsNetwork/labs/Project/project-instructions.md&lti=true

my push keeps failing says its because my username n i know the info is correct what do i do to complete this project really getting frustrated


[bug] "building..." hangs and does not show error in the console

With the new anchor version, you now need to directly use ctx.bump instead of using .get(). If your program still uses the .get() method, an error is not generated when attempting to build your program on solana playground. Instead, "building..." just hangs in the console and will never generate the method 'get' not found error.
Screenshot of building issue:

Successful build after removing this error from the code:

It doesn't actually hang, the error message just gets filtered out. This was a false positive with detecting useless errors and was fixed in 6126b424ae6ccdbf6e870ae8db37d15cc01442c3 (but not yet deployed). I've just deployed it so it should work now.
There is a new "Improve build errors" setting to enable/disable this behavior (enabled by default).


Capital letters are not respected in display names

I started tracking as "TheLotto" but now the site says my name is Thelotto.

Yea I'm "standardizing" usernames, it was the safe thing to do at first, but I'd like to improve on that, just gotta make sure it doesn't break anything.


Proxy support for Pipeline and tasks

Feature request
This is probably worth splitting into two issues, but here goes.
As an admin, I want the tekton controllers installed to use users network proxy settings defined for the platform so that both the tekton controllers as well as the Task or Pipeline steps / user workloads can access images, git repos, etc through our organizations proxy and with custom certificates.
Why is this important?
To enable customer installs of the tekton to have network communications using the to use global proxy settings
Note: this is only for tektoncd/pipeline but it does affect other components (tektoncd/triggers, …). It also, most likely, need to be a TEP I guess.
/cc @sbose78 @afrittoli @bobcatfish
Use case
A user sets up tekton in a cluster that uses a global proxy for network communications. It should work : aka tasks should be able to communicate with the outside world through the proxy (if what is in the tasks supports that of course), the different tekton components should go through the proxy too when doing network query ; for tektoncd/pipeline, it might be when querying the oci registry for the image config, …
Parts
There is actually few parts for that issue :

How the user can provide the proxy configuration ? (through a configmap seems the best)
Support for proxy's environment variables inside tektoncd/pipeline code (cmd/controller, cmd/webhook, …)
How the controller/admission webhook can add those at runtime ? (this requires #1606 to work)

/assign

/remove-lifecycle stale

Note we have a start to a TEP about this in https://github.com/tektoncd/community/pull/232


[Bug]: Crash when removing, then re-adding a model

Description of the bug
A reproducible crash can happen when updating a fully spawned model to something else that doesn't exist, then back to the original.
Steps to reproduce the bug

Run the site editor in the demo world
Change a model to trigger a new asset lookup to something that doesn't exist, i.e. a OpenRobotics/AdjTable to OpenRobotics/AdjTable2.
Change the model back to the previous, i.e. OpenRobotics/AdjTable
Panic!

Additional information or screenshots
After spending some time debugging the issue I narrowed it down to a complex interaction between the code to spawn models and the Aabb plugin, specifically:

When the asset source for a model is changed, its entire scene, including entities that will contain Handle<Mesh components is despawned .
The Aabb plugin internally keeps track of what entities have what mesh handle, however it only does cleanup when the Handle<Mesh> component is removed. The query for removed Handle<Mesh> components does not get triggered when an entity is despawned, so the internal hash map falls out of sync with what entities actually exist and contain the mesh handle.
Next time the same handle is added any entity, this for loop will be triggered and because an old entity, that has since been despawned, refers to that handle, a command for an entity that doesn't exist will be triggered here and the application will panic.

This issue is quite painful when spawning / despawning models, however the code was copied from an old PR in bevy. It seems mesh aabb update is still in progress in main so it's not going to land anytime soon and we will need to keep relying on our custom aabb module.

Solved by https://github.com/open-rmf/rmf_site/pull/142


Unclear dependency on simple-bus/doctrine-orm-bridge

Steps to reproduce:

Install a symfony standard distro

Follow steps from symfony-bridge documentation
2.1. Run composer require simple-bus/symfony-bridge
2.2. Add new SimpleBus\SymfonyBridge\SimpleBusCommandBusBundle() to AppKernel
2.3. Add new SimpleBus\SymfonyBridge\SimpleBusEventBusBundle() to AppKernel
2.4. Add new SimpleBus\SymfonyBridge\DoctrineOrmBridgeBundle() to AppKernel

Run app/console cache:clear

An error appears:
PHP Fatal error:  Class 'SimpleBus\DoctrineORMBridge\EventListener\CollectsEventsFromEntities' not found in app/cache/dev/appDevDebugProjectContainer.php

This can be fixed by calling composer require simple-bus/doctrine-orm-bridge after step 2.1.
I think this is related to the simple-bus/doctrine-orm-bundle being "suggested" as a dependency but not required. Either documentation should state that you need to add this dependency manually to start using DoctrineOrmBridgeBundle or the file composer.json should make this dependency a required one.

Maybe an exception could help too?
If SimpleBus\DoctrineORMBridge not found, throw exception and say that the user probably forgot to install the ORM bridge?

simple-bus/doctrine-orm-bridge is only a suggested package so you have to run
composer require simple-bus/doctrine-orm-bridge

@hacfi That's what I'm asking: if it should remain a suggestion dependency, then the documentation needs to make this explicit. But if it's a hard dependency as I understand by this tweet, then it's composer.json who needs to be fixed.

Well simple-bus/doctrine-orm-bridge requires doctrine/orm. If we'd require this package by default it would require doctrine/orm too. So I guess the doc should be changed.

I'll add some lines to the docs soon :)
Op 11 mei 2015 11:18 schreef "Ruud Kamphuis"<EMAIL_ADDRESS>
Well simple-bus/doctrine-orm-bridge requires doctrine/orm. If we'd
require this package by default it would require doctrine/orm too. So I
guess the doc should be changed.
—
Reply to this email directly or view it on GitHub
https://github.com/SimpleBus/SymfonyBridge/issues/12#issuecomment-100829606
.

Nice! The point made by @ruudk is key for this decision: Not every symfony project using simple-bus needs to depend on doctrine/orm. This solves my initial question at twitter. Thanks!

Right, also, maybe it's good to have that exception, warning users about
the extra dependency.
Op 12 mei 2015 19:07 schreef "Berny Cantos"<EMAIL_ADDRESS>
Nice! The point made by @ruudk https://github.com/ruudk is key for this
decision: Not every symfony project using simple-bus needs to depend on
doctrine/orm. This solves my initial question at twitter. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/SimpleBus/SymfonyBridge/issues/12#issuecomment-101351862
.

I added a note about the extra dependency you need to install: http://simplebus.github.io/SymfonyBridge/doc/doctrine_orm_bridge_bundle.html. Maybe if someone encounters this problem again, we can add an exception as well. Would you agree with that?

@matthiasnoback, exception would be better solution IMHO, because it's not clear that simple-bus/doctrine-orm-bridge package should be installed too.


google_accounts_daemon should not change permissions on home directories

Following https://github.com/GoogleCloudPlatform/compute-image-packages/pull/385, google_accounts_daemon is setting home directories to mode 0750 when it installs public keys to ~/.ssh/authorized_keys.  This causes problems if users have intentionally set home directories to have different permissions, e.g. world-readable.  This is an issue with gce-xfstests, for example.  Probably google_accounts_daemon should not change the home directory's mode if it already exists.  (Changing the mode of ~/.ssh to 0700 should still be fine, though.)

I've created a new pull request #394 to remove setting permissions on home directories. We should use the umask permissions automatically during useradd, and shouldn't have to override them.

Nice! I thought about that but was not sure if there was a reason for it.

Our initial reason for setting the permission on the home directory was (if I recall correctly) to ensure the SELinux context was set properly. Based on our current understanding, I don't believe that should be necessary. We'll build some images off of this change and verify the images work properly before we publish the changes. This may take a few weeks to reach production.

New images are released with this fix.


Pull only public boards

Hello:
$boards = $bot->boards->forMe();
$boards = $bot->boards->forUser($me_username);
Is it possible to get a list of ONLY public boards (Not secret)?
If so, please share the code.
Thanks!

Hi, I see that you can create private boards:
// Create a private board
$bot->boards->createPrivate('Name', 'Description');
and pass more options in an update:
$bot->boards->update($boardId, [
'name'        => 'New title',
'description' => 'New description',
'privacy'     => 'secret',
'category'    => 'sports',
]);
But I would like to select (get) boards that are NOT secret only.
Can you please tell me where to update this code?
Or if this is possible?
Please assign the label for this post as a question.
Thank you.

Hi! There is no need to update the code. Just filter the received boards by privacy field.

Hello!
I would like to adjust the code in a plugin that someone created based on your code.
I would only like to pull public boards and randomly pin to one of them.  The plugin is pulling all boards (secret too).  Basically I don't want to pin to secret boards.
Here's a snippet:
$boards = $bot->boards->forMe();
$boards = $boards[array_rand($boards)];
$boardId = $boards['id'];
Any ideas how I can pull (or filter) so my array has only public boards (no secret boards)?
Thanks!

Try this:
$boards = $bot->boards->forMe();
$onlyPublicBoard = array_filter($boards, function (array $board) {
    return $board['privacy'] === 'public';
});


Adjust screening-schema.json to reflect the changes introduced in DPG…

Adjust screening-schema.json to reflect the changes introduced in DPGAlliance/DPG-Standard#25

There seems to be an overlap with these two questions in the Do No Harm question:

Please describe any additional (not described in the questions above) risks and mitigations that this project uses to prevent harm.
Is there any additional information you would like to share about the mechanisms, processes or policies that this project uses to avoid doing harm?

Please clarify the distinction, or remove the second one, and migrate the answers from the projects already screened to make sure we don't lose any information already submitted.

@nathanbaleeta: The screening schema is missing the following in indicator 6:
If yes - is there a mechanism for extracting or importing non PII data from the system
in a non-proprietary format?

Adding it to the schema implies that you have to add it to those projects already screened.

Are you referring to the difference in wording because that section I presume already exists on line 129

There seems to be an overlap with these two questions in the Do No Harm section:

Please describe any additional (not described in the questions above) risks and mitigations that this project uses to prevent harm.
Is there any additional information you would like to share about the mechanisms, processes or policies that this project uses to avoid doing harm?

Please clarify the distinction, or remove the second one, and migrate the answers from the projects already screened to make sure we don't lose any information already submitted.

The questions seem to point to the same answer. Actually several respondents leave one of the two blank, it's a pattern I have observed happening. I'll remove the second one and migrate the answers from the projects already screened to make sure we don't lose any information already submitted. I'll request your review when it's ready.

There seems to be an overlap with these two questions in the Do No Harm section:

Please describe any additional (not described in the questions above) risks and mitigations that this project uses to prevent harm.
Is there any additional information you would like to share about the mechanisms, processes or policies that this project uses to avoid doing harm?

Please clarify the distinction, or remove the second one, and migrate the answers from the projects already screened to make sure we don't lose any information already submitted.

The questions seem to point to the same answer. Actually several respondents leave one of the two blank, it's a pattern I have observed happening. I'll remove the second one and migrate the answers from the projects already screened to make sure we don't lose any information already submitted. I'll request your review when it's ready.

Consolidated the 2 question responses into the former then migrated changes to all the screened projects as documented here

@nathanbaleeta: The screening schema is missing the following in indicator 6:
If yes - is there a mechanism for extracting or importing non PII data from the system
in a non-proprietary format?

Adding it to the schema implies that you have to add it to those projects already screened.

Are you referring to the difference in wording, because that section I presume already exists on line 129

Thanks @nathanbaleeta, you are correct, and I made a small modification on the wording to address the small discrepancy that you pointed out in 180811f
What I meant to say is that the screening schema is missing this other question from the same indicator 6:
If yes - describe the mechanism for extracting or importing non-personally identifiable information
from the system in a non-proprietary format:

So the first question asks is the mechanisms exists and expects a yes/no answer, and the second question asks to describe such mechanism if it exists. The first question is in the schema, the second question is not.

Added checkNonPIIAccessMechanism to screening-schema.json to address missing question  as referenced in this commit


feat: enforce maximum call depth for local api operations

What?
Enforces maximum call depth for Local API operations.
This helps to identify hooks like this:
import type { CollectionConfig } from 'payload'

export const InfinityLoop: CollectionConfig = {
  slug: 'infinity-loop',
  fields: [],
  hooks: {
    afterRead: [
      async ({ req, context, doc }) => {
        // fetch self, doesn't have any condition so infinity loop
        await req.payload.findByID({
          req,
          id: doc.id,
          collection: 'infinity-loop',
        })
      },
    ],
  },
}

And now we throw the ReachedMaxCallDepth error if call depth is more than payload.config.maxCallDepth which by default is 30.
How?
Uses ALS to track current operations call depth.

If you have performance concerns about running an ALS for each operations, I did some benchmarking, await asl.run() doesn't seem different than await Promise.resolve(1).


neutrinordp: avoid pampassword leakage

The leakage does not occur in the most usual use case of xrdp.
It occurs in NeutrinoRDP proxy mode with PAM authentication enabled.
Reported by @TOMATO-ONE

Note: It shouldn't be a problem in the most use case of xrdp and it is not a vulnerability such as remote code execution. So I've raised the fix as a usual public PR.

CC: @aquesnel

Fyi: as I was making the longing changes I tried to omit explicit security sensitive data from the logs, but when logging unknown data I decided that logging with LOG_DEVEL and TRACE it was ok to log sensitive data since these settings should not be used in a production setting.
If you disagree with decision then maybe we should open an issue to revive logging for all unknown fields for potentially sensitive data.

Thanks for the careful work on logging change.

I decided that logging with LOG_DEVEL and TRACE it was ok to log sensitive data since these settings should not be used in a production setting.

I'm happy with this method. It would be much better if we add a note that LOG_DEVEL and TRACE level might dump sensitive datum and should not be used in a production setting.

Merged, thanks @TOMATO-ONE !
The fix will be included in the next release 0.9.16, will be released soon. Also Thank you for the correct method of reporting security issues.


STM32F407 Hardfault on rclc_support_init

Issue template

Hardware description: Custom STM32E407 board
RTOS: FreeRTOS
Installation type: micro_ros_setup
Version or commit hash: foxy

I am trying to run microros on a custom STM32E407 board with a serial transport. I have been able to test gpio and UART successfully but when I try to run the int32_publisher app I get a hardfault at RCCHECK(rclc_support_init(&support, 0, NULL, &allocator));.  Any ideas on what could cause this or how to debug it?

Sorry, it is a STM32F407 not E407.

Hi! I'm working on smt32f407 and MicroROS. How are you compiling micro ros library on the embedded stm32 ? I had hard fault errors when I was compiling with CubeIDE default compiler.

Could you provide details for replicating this issue?

I basically followed the First FreeRTOS application tutorial to generate the firmware. I configured it to use the int32_publisher example app, although I do get the same issue with ping_pong. I made some minor changes to the firmware to toggle the GPIO on my custom board, as well as changing the alternate function GPIO being used by USART3. I build the firmware using ros2 run micro_ros_setup build_firmware.sh and then flash it to the board with dfu-utils.
If I don't run the appMain task, everything works fine. When I run it, the code enters the HardFault_Handler function defined in stm32f4xx_it.c.

Well, this tutorial and micro_ros_setup package (that is being used by this tutorial) builds a firmware for Olimex E407. And probably this is not compatible with your custom board.
I recommend you use micro-ROS utils for STCube and integrate micro-ROS directly in a ST project for your specific board and configuration.
If you don't mind I'm closing this and please feel free to open another issue in the repo I have linked if you have any problem.


Add range selection to Details panel main chart

closes
Prerequisites:

[ ] Unless it is a hotfix it should be merged against the dev branch
[ ] Branch is up-to-date with the branch to be merged with, i.e. dev
[ ] Build is successful
[ ] Code is cleaned up and formatted

Summary
Experimenting with range selection.

How it looks at this time:

How it looks at this time:

I can now compute the difference in values and draw a rule with arrowheads that is colored base on whether the change is an increase or decrease.

I also discovered a UI problem that will be difficult to work around.  Once the interval is selected, mouseover within the interval no longer will show tooltips for any points inside the interval.  This is because the mark drawn for the interval is above everything, by design in Vega.  There is discussion about allowing an option to specify the z-index of marks or layers, which would be useful in other circumstances, but no implementation of this, other than that the order of layers makes some difference, though it does not impact the selection drawing.

I can now compute the difference in values and draw a rule with arrowheads that is colored base on whether the change is an increase or decrease.

I also discovered a UI problem that will be difficult to work around.  Once the interval is selected, mouseover within the interval no longer will show tooltips for any points inside the interval.  This is because the mark drawn for the interval is above everything, by design in Vega.  There is discussion about allowing an option to specify the z-index of marks or layers, which would be useful in other circumstances, but no implementation of this, other than that the order of layers makes some difference, though it does not impact the selection drawing.

Here is what I have as of today:

Here is what I have as of today:

Starting fresh, adding only what I needed, I managed to figure out the weird complication with multiple locations.

To do this, I am filtering the data for showing the area under the line to only include the 0th location.  Doing this filtering also avoids the random highlighting of one location or the other, but it means we would always only show the tooltip for the 0th location.  At least it is consistent.

Starting fresh, adding only what I needed, I managed to figure out the weird complication with multiple locations.

To do this, I am filtering the data for showing the area under the line to only include the 0th location.  Doing this filtering also avoids the random highlighting of one location or the other, but it means we would always only show the tooltip for the 0th location.  At least it is consistent.

I like how the shaded regions work.  The horizontal lines and the percentage indicator/lines.  Is there some place outside the plot area where we could have the start, end, and % difference text?
An alternate method would be to always have the context date (i.e. the red line) be the anchor.  Then when you hover over other points it would show you the % diff instead of having to click and drag to create the selection.
The more I think about it, it feels a bit odd to have the context date separate from a range selection.
One last note, have you played with making the discrete observations use a bar style instead of an interpolated line?

I like how the shaded regions work.  The horizontal lines and the percentage indicator/lines.  Is there some place outside the plot area where we could have the start, end, and % difference text?
An alternate method would be to always have the context date (i.e. the red line) be the anchor.  Then when you hover over other points it would show you the % diff instead of having to click and drag to create the selection.
The more I think about it, it feels a bit odd to have the context date separate from a range selection.
One last note, have you played with making the discrete observations use a bar style instead of an interpolated line?

I like how the shaded regions work. The horizontal lines and the percentage indicator/lines. Is there some place outside the plot area where we could have the start, end, and % difference text?

Yes, we could take some of the large space available to display data about the selected range.  We might do this in a way that is similar to what is being done on the survey results page, for example.  This would allow us to show more data, and avoid collisions with other data and graphics within the chart area.  Annotations within charts are hard to do well because of this.

An alternate method would be to always have the context date (i.e. the red line) be the anchor. Then when you hover over other points it would show you the % diff instead of having to click and drag to create the selection.

Yes, I like this idea, but there are a couple complications.

The current date (aka context date) is not (yet) directly changeable on this detail chart page.  It can be changed by selecting a different date with the date control at the top, or by clicking on a different date in any of the small multiples on the right.  There is no reason I know of that we could not also allow clicking on a date in the detail chart page too.
Hovering over another date would always show this range selection, in addition to the tooltip for the hovered point.  If we do this, the range selection visualization should probably be a bit more subtle or light-weight, and then it would make more sense to show the associated data for the range in a separate location.
There would be no way to preserve the range selection independent of hovering, not unless we add some additional capability, so it would disappear as soon as you stop hovering in the chart.  A modified click, such as shift-click, could make the range selection stick, but when would it go away?
Touch-only support should be considered.

The more I think about it, it feels a bit odd to have the context date separate from a range selection.

Yes, it does seem extraneous.

One last note, have you played with making the discrete observations use a bar style instead of an interpolated line?

Yes, I have tried that, but got a mess with multiple locations that I didn't understand.  Now that I can filter out the data for other locations, it should be doable.
We could also use a "stepped-line" mode, which would be like the tops of the bars, rather than smoothed or straight lines (two alternatives) between points.

I like how the shaded regions work. The horizontal lines and the percentage indicator/lines. Is there some place outside the plot area where we could have the start, end, and % difference text?

Yes, we could take some of the large space available to display data about the selected range.  We might do this in a way that is similar to what is being done on the survey results page, for example.  This would allow us to show more data, and avoid collisions with other data and graphics within the chart area.  Annotations within charts are hard to do well because of this.

An alternate method would be to always have the context date (i.e. the red line) be the anchor. Then when you hover over other points it would show you the % diff instead of having to click and drag to create the selection.

Yes, I like this idea, but there are a couple complications.

The current date (aka context date) is not (yet) directly changeable on this detail chart page.  It can be changed by selecting a different date with the date control at the top, or by clicking on a different date in any of the small multiples on the right.  There is no reason I know of that we could not also allow clicking on a date in the detail chart page too.
Hovering over another date would always show this range selection, in addition to the tooltip for the hovered point.  If we do this, the range selection visualization should probably be a bit more subtle or light-weight, and then it would make more sense to show the associated data for the range in a separate location.
There would be no way to preserve the range selection independent of hovering, not unless we add some additional capability, so it would disappear as soon as you stop hovering in the chart.  A modified click, such as shift-click, could make the range selection stick, but when would it go away?
Touch-only support should be considered.

The more I think about it, it feels a bit odd to have the context date separate from a range selection.

Yes, it does seem extraneous.

One last note, have you played with making the discrete observations use a bar style instead of an interpolated line?

Yes, I have tried that, but got a mess with multiple locations that I didn't understand.  Now that I can filter out the data for other locations, it should be doable.
We could also use a "stepped-line" mode, which would be like the tops of the bars, rather than smoothed or straight lines (two alternatives) between points.

To capture our discussion from last week:

I'd like to see a bar+line combo chart, but limit it to when there is only one location.  If there is more than one location, the chart should show only the separate 7-day average lines.
We could move the tool-tip to follow the y-intercept on the series line instead of where the mouse pointer is.  This seems to be best practice on other sites

The range selection mechanism I think then would be different based on if there are multiple series.  If there's one series, you compare over time, but with multiple series, you compare over series.
Based on these observations, what do you think about closing this issue and starting a new one to focus on the bar+line chart?

To capture our discussion from last week:

I'd like to see a bar+line combo chart, but limit it to when there is only one location.  If there is more than one location, the chart should show only the separate 7-day average lines.
We could move the tool-tip to follow the y-intercept on the series line instead of where the mouse pointer is.  This seems to be best practice on other sites

The range selection mechanism I think then would be different based on if there are multiple series.  If there's one series, you compare over time, but with multiple series, you compare over series.
Based on these observations, what do you think about closing this issue and starting a new one to focus on the bar+line chart?


GetTable("") and GetTable("/") should be equivalent

Currently GetTable("/") results in adding an extra leading slash to any keys.  It should instead return the same thing that GetTable("") returns.

Fixed by #204.


Add 'paused' to a list of available statuses in ST2 CLI

Add "paused" to possible execution status.

Changelog updated!

Thanks for updating changelog :+1:
How about fixing beforementioned pylint warning? https://github.com/StackStorm/st2/pull/4037#discussion_r175122145
TravisCI is not happy

Hopefully we're all good now :)


Support task resource tracking in OpenSearch (#3982)

[Backport 2.x] Support task resource tracking in OpenSearch (#3982)

Support task resource tracking in OpenSearch

Reopens changes from #2639 (reverted in #3046) to add a framework for task resource tracking. Currently, SearchTask and SearchShardTask support resource tracking but it can be extended to any other task.

Fixed a race-condition when Task is unregistered before its threads are stopped

Improved error handling and simplified task resource tracking completion listener

Avoid registering listeners on already completed tasks

Signed-off-by: Ketan Verma<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/1254/
CommitID: 77a5124

Failed in :qa:wildfly:composeUp due to a transient network error when calling Docker registry. Re-triggering the workflows.

Gradle Check (Jenkins) Run Completed with:

RESULT: null ❌
URL: https://build.ci.opensearch.org/job/gradle-check/1255/
CommitID: 930ba7c

The Jenkins' build succeeded, but the GitHub action while trying to get the job status failed.
https://build.ci.opensearch.org/job/gradle-check/1255/

Codecov Report

Merging #4087 (991632b) into 2.x (517b815) will increase coverage by 0.00%.
The diff coverage is 78.13%.

@@            Coverage Diff             @@
##                2.x    #4087    +/-   ##
==========================================
  Coverage     70.61%   70.62%
- Complexity    56628    56687    +59
==========================================
  Files          4543     4545     +2
  Lines        272332   272525   +193
  Branches      40017    40037    +20
==========================================
+ Hits         192307   192461   +154
+ Misses        63902    63865    -37
- Partials      16123    16199    +76

Impacted Files
Coverage Δ

...rg/opensearch/common/settings/ClusterSettings.java
91.89% <ø> (ø)

...org/opensearch/action/support/TransportAction.java
55.42% <50.00%> (-0.28%)
:arrow_down:

...rc/main/java/org/opensearch/tasks/TaskManager.java
67.74% <62.50%> (+0.05%)
:arrow_up:

...ster/node/tasks/list/TransportListTasksAction.java
47.61% <66.66%> (+3.17%)
:arrow_up:

...a/org/opensearch/threadpool/TaskAwareRunnable.java
69.56% <69.56%> (ø)

...erver/src/main/java/org/opensearch/tasks/Task.java
79.43% <75.00%> (+1.66%)
:arrow_up:

.../opensearch/tasks/TaskResourceTrackingService.java
79.76% <79.76%> (ø)

...ensearch/common/util/concurrent/ThreadContext.java
92.96% <80.00%> (+0.76%)
:arrow_up:

...ch/common/util/concurrent/OpenSearchExecutors.java
81.72% <83.33%> (+0.11%)
:arrow_up:

.../org/opensearch/action/search/SearchShardTask.java
75.00% <100.00%> (+8.33%)
:arrow_up:

... and 516 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


Lean login command

When I try to login with the credentials i got from the email and following the steps the login fails. I copy pasted the user id and the even first to notepad to make sure nu extra characters/markup is being copied.
Is this a bug?
Expected Behavior
It will log in
Actual Behavior
Throws the following error:
Error: Credentials are invalid
Visit https://www.lean.io/docs/lean-cli/tutorials/authentication#02-Logging-in for more information

Checklist

[X] I have completely filled out this template
[X] I have confirmed that this issue exists on the current master branch
[X] I have confirmed that this is not a duplicate issue by searching issues
[X] I have provided detailed steps to reproduce the issue

Can you please try it again with lean login --verbose and share the full output? Feel free to remove your user id and the masked API token if you'd like.

user@DESKTOP-1NFF5PV:~/Documents/Private/TDM500$ lean login --verbose
Your user id and API token are needed to make authenticated requests to the QuantConnect API
You can request these credentials on https://www.quantconnect.com/account
Both will be saved in /home/user/.lean/credentials
User id:
API token:
GET https://www.quantconnect.com/api/v2/projects/read with data {}
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.6/site-packages/lean/main.py", line 96, in main
    lean.main(standalone_mode=False)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/user/.local/lib/python3.6/site-packages/lean/click.py", line 83, in invoke
    result = super().invoke(ctx)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/user/.local/lib/python3.6/site-packages/lean/commands/login.py", line 50, in login
    "https://www.lean.io/docs/lean-cli/tutorials/authentication#02-Logging-in")
lean.models.errors.MoreInfoError: Credentials are invalid
Error: Credentials are invalid
Visit https://www.lean.io/docs/lean-cli/tutorials/authentication#02-Logging-in for more information

Hmm, from those logs it does look like invalid credentials are the problem.
Because I have no idea what's wrong here I just released a new version of the CLI with more extensive debug logging around the "are the credentials valid" check. Can you try updating with pip install --upgrade lean and then post the full output of lean login --verbose again?

GET https://www.quantconnect.com/api/v2/projects/read with data {}
Request was not successful, status code 200, body:
{"errors":["Hash doesn't match. UID: <user_id> Hash: <hash> Timestamp: '1625580765'"],"success":false}
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.6/site-packages/lean/components/api/api_client.py", line 96, in is_authenticated
    self.get("projects/read")
  File "/home/user/.local/lib/python3.6/site-packages/lean/components/api/api_client.py", line 75, in get
    return self._request("get", endpoint, {"params": parameters})
  File "/home/user/.local/lib/python3.6/site-packages/lean/components/api/api_client.py", line 137, in _request
    return self._parse_response(response)
  File "/home/user/.local/lib/python3.6/site-packages/lean/components/api/api_client.py", line 156, in _parse_response
    raise AuthenticationError()
lean.models.errors.AuthenticationError: Invalid credentials, please log in using `lean login`
Traceback (most recent call last):
  File "/home/user/.local/lib/python3.6/site-packages/lean/main.py", line 96, in main
    lean.main(standalone_mode=False)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/user/.local/lib/python3.6/site-packages/lean/click.py", line 83, in invoke
    result = super().invoke(ctx)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/user/.local/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/user/.local/lib/python3.6/site-packages/lean/commands/login.py", line 50, in login
    "https://www.lean.io/docs/lean-cli/tutorials/authentication#02-Logging-in")
lean.models.errors.MoreInfoError: Credentials are invalid
Error: Credentials are invalid
Visit https://www.lean.io/docs/lean-cli/tutorials/authentication#02-Logging-in for more information

I am in WSL2 on windows by the way.

Thanks, no you can leave out your UUID and hash, the "Hash doesn't match" part of the message tells me enough. I'll do some debugging on WSL2 myself to see if there are problems with how the password is masked.

Alright, thank you for your effort so far :)
Does an test account/login exists to see if it is my environment?

It's not just your environment, I also experience issues when I copy+paste my token into Windows Terminal + WSL2. Manually typing the password does seem to work here, but that's obviously not optimal. I've just released a new version of the CLI which disables password masking on WSL2, so if you run pip install --upgrade lean one more time you should be able to copy+paste your API token into lean login.

GET https://www.quantconnect.com/api/v2/projects/read with data {}
Successfully logged in

Thanks a lot!
I together with wsl2 I use Windows Terminal Preview. Wouldn't have thought that that was the issue. I use this for my day to day work but never had any issues with it until now..

Nice! Most of the CLI was tested on WSL2, but this password masking bug clearly slipped past. I just re-ran the end-to-end tests on WSL2 no other major features are broken, thanks for reporting the issue and for helping debug it!


Move images from pptx

Just keeping track of image extractions from PPTX files:

[x] Lesson 1
[ ] Lesson 2
[x] Lesson 3
[ ] Lesson 4
[ ] Lesson 5
[ ] Lesson 6
[ ] Lesson 7
[ ] Lesson 8
[ ] Lesson 9
[x] Lesson 10

We're using this utility to remove the files: https://github.com/revan/pptx2md To run it on a non GNU unix system you'll need to replace the call for sort with gsort. On a mac you can get gsort by running brew install coreutils, assuming you have homebrew installed already.

Awesome @jduckles  Works well.

+1 +1 +1 !!!

Final round of images came in with #43


Update dependencies golang/x libs

Codecov Report
Patch and project coverage have no change.

Comparison is base (a2f226f) 40.64% compared to head (3a873a2) 40.64%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1612   +/-   ##
=======================================
  Coverage   40.64%   40.64%
=======================================
  Files         169      169
  Lines       11083    11083
=======================================
  Hits         4505     4505
  Misses       6287     6287
  Partials      291      291

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

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

-> #1621


It seems I can not make

It seems I can not make a call to China.
voice call to China will be block .
my app has a great demand in China, any idea I can enable access.

Hi @lyqflnh
For phone number and geo-permission, please file a Twilio Support ticket and we can help you from there.
Thanks.
-bobie


Your Css is not visible

Issue Description
Assign me this Task. I can solve @aditya-bhaumik
Not visible Css.
I'm contributor under GSSoc^24
Additional Context
No response
Screenshots or Examples (if applicable)

Proposed Solution (optional)
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

it is visible


Update snapshot.js version

https://github.com/snapshot-labs/snapshot-hub/pull/407 is working again


Theme Customizer Sanitization

There is a problem with a required setting with the WordPress theme check plugin, this is the error that i get:
REQUIRED: Found a Customizer setting that did not have a sanitization callback function. Every call to the add_setting() method needs to have a sanitization callback function passed.

I found this, too.

@blackmekemeke @venkatraj @MennStudio I can't replicate this...
Can you please post a code sample of what you're doing that gives you this error?

Embed 'Kirki' to custom theme and check with 'Theme Check' plugin. That throws the error.
New Theme Review Guidelines requires sanitization for each customizer settings.
The custom theme can be as simple as a child theme of Twenty Fifteen.

I just pushed a new commit that should fix this for you...
Can you please check that all your controls are still being saved?
I added sanitization calls and wrote some custom functions where needed.
You can always however write your own sanitization function wherever you want to override the default ones simply by adding
'sanitize_callback' => 'my_custom_function'

to your field definitions.

Thank you so much @aristath :)

@aristath -
Would you mind adding this option to the documentation on kirki.org?

Yup, I'm planning to.
I'm working on some major changes as we speak that will fix a lot of bugs.
As soon as I push the new version up, I'll also update the docs. :)


[New Port Request] VTIL-Core

Library name:
VTIL-Core
Library description:

VTIL Project, standing for Virtual-machine Translation Intermediate Language, is a set of tools designed around an optimizing compiler to be used for binary de-obfuscation and de-virtualization.
The main difference between VTIL and other optimizing compilers such as LLVM is that it has an extremely versatile IL that makes it trivial to lift from any architecture including stack machines. Since it is built for translation, VTIL does not abstract away the native ISA and keeps the concept of the stack, physical registers, and the non-SSA architecture of a general-purpose CPU as is. Native instructions can be emitted in the middle of the IL stream and the physical registers can be addressed from VTIL instructions freely.
VTIL also makes it trivial to emit code back into the native format at any virtual address requested without being constrained to a specific file format.

Source repository URL:
_https://github.com/vtil-project/VTIL-Core_
Project homepage (if different from the source repository):
_https://vtil.org/_
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now, consider making a pull request instead.