[Solved] Error install flow-typed on Mac OS 10.11.6

$ npm i<EMAIL_ADDRESS>-g
npm WARN deprecated<EMAIL_ADDRESS>This package is deprecated. Use Object.assign.

><EMAIL_ADDRESS>preinstall /Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b
> node lifecycleScripts/preinstall

[nodegit] Running pre-install script
[nodegit] Configuring libssh2.
{ Error: Command failed: /Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/libssh2/configure --with-libssl-prefix=/Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/openssl/openssl
configure: error: cannot find OpenSSL or Libgcrypt,
try --with-libssl-prefix=PATH or --with-libgcrypt-prefix=PATH

    at ChildProcess.exithandler (child_process.js:206:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:877:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
  killed: false,
  code: 1,
  signal: null,
  cmd: '/Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/libssh2/configure --with-libssl-prefix=/Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/openssl/openssl' }
configure: error: cannot find OpenSSL or Libgcrypt,
try --with-libssl-prefix=PATH or --with-libgcrypt-prefix=PATH

[nodegit] ERROR - Could not finish preinstall
{ Error: Command failed: /Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/libssh2/configure --with-libssl-prefix=/Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/openssl/openssl
configure: error: cannot find OpenSSL or Libgcrypt,
try --with-libssl-prefix=PATH or --with-libgcrypt-prefix=PATH

    at ChildProcess.exithandler (child_process.js:206:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:877:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
  killed: false,
  code: 1,
  signal: null,
  cmd: '/Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/libssh2/configure --with-libssl-prefix=/Users/sean/.nvm/versions/node/v6.6.0/lib/node_modules/.staging/nodegit-af44277b/vendor/openssl/openssl' }
/Users/sean/.nvm/versions/node/v6.6.0/lib
└── (empty)

I tried to brew install openssl, but...
This formula is keg-only, which means it was not symlinked into /usr/local.

Apple has deprecated use of OpenSSL in favor of its own TLS and crypto libraries

Does work after running
brew install libgcrypt

Leaving this up here for anyone else.

Thanks, @ctrlplusb. I think this would be a good thing to go into an FAQs section of the README. I'll add this there and then close this out

Ok, I've added an FAQs section over here with this issue listed.
Thanks again for the report!


Add GoCoin Integration

GoCoin (https://www.gocoin.com) integration

I'll close this since it has been more than a year without activity


Add SSL_CTX_set_session_id_context

As in the title, it'd be nice to have a binding to SSL_CTX_set_session_id_context too. Without it, it's impossible to enable server session resumption with client certificates.
Would you accept a PR that adds a contextSetSessionIdContext :: SSLContext -> ByteString -> IO () function?

Sure, I will accept a PR and publish new version on Hackage.


Progress the Scrum of Scrum app board to a Beta version

As a product owner I need to be able to see an aggregated representation of my teams progress so that I can determine if they are going to be able to make their release dates or see if any teams are blocked so that I can help resolve any problems.
Success criteria

Complete the prototype configured by users and demonstrated to their respective project executive.
Determine and estimate the extent of work that is necessary to provide initial working version to product group.

@watkinspd I want to issue this as a P4P can you please provide me with authorization to do so?

@lmullane Can you work with me to prepare this as a P4P? Thx

@paulroberts68 Yes, I can.


Feature Request: Flexible kubernetes client authentication methods

The current method of authenticating with the kubernetes API via the secret mounted on /var/run/secrets/kubernetes.io/serviceaccount/token makes it very hard to test, or use the kubernetes client low level client outside of a k8s environment. The fabric8 client has a flexible way to provide the token, the ca mount taking the least precedence:
https://github.com/fabric8io/kubernetes-client#configuring-the-client
Would be nice to have a similar way to have a default of reading the file, but allowing an environment variable or use kube config to pull the token.

Closing this issue as it is already fixed by using official K8s java sdk client.


alternative way to download the data?

Hi,
Thanks for sharing your code and data. However, downloading the data from BaiduNetDisk is extremely slow for me. Could you consider adding alternative links, e.g. Google drive? Thanks!

Thanks for your interest! Unfortunately, I don't have enough space on Google Drive. But I have also uploaded them to OneDrive. Hope that helps.

I updated the links in readme. The new Cloud.THU links should work.


Update ne-d3d11_4-d3d11_video_decoder_histogram_component_flags.md

@v-rajagt : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@stevewhims : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fallback to old version of port compiler

Fix compilation problems in quick and dirty way.
We need a ticket upstream
Somewhere above version 1.6.0 of pc (port_compiler) the environment variables are passed down to pc. This causes error in compilation stating that -o flag cannot have multiple out files.

CI workflow on first commit of this PR had passed https://circleci.com/workflow-run/b86a7134-5304-4543-b2a2-6e144f1493fb
I do not see how a comment in rebar.config could break CI so I am considering merging. I see two approvals.

(For future reference, reviews can be dismissed to make the checks pass).


Logger json

Problem
We want to incorporate JSON formatted logging
Solution
Implement the Alephant-Logger-Json gem

:dragon_face:

@Integralist added some comments, some are more important than others :+1:


Psr7 update issue

my php version is 8.0 and psr version is 7
but this shows me this error
Problem 1
- Root composer.json requires slim/psr7 ^0.5 -> satisfiable by slim/psr7[0.5.0].
- slim/psr7 0.5.0 requires php ^7.1 -> your php version (8.0.0) does not satisfy that requirement.
Problem 2
- phpunit/phpunit[7.5.0, ..., 7.5.20] require php ^7.1 -> your php version (8.0.0) does not satisfy that requirement.
- Root composer.json requires phpunit/phpunit ^7.5 -> satisfiable by phpunit/phpunit[7.5.0, ..., 7.5.20].

You need to manually update to version 1.3.0, composer will not bump to a major version for you. In your composer.json file change the dependency to:
"slim/psr7": "^1.3.0"

Remove your composer.lock file and run composer update


Esv6 series supports Premium Storage and Premiumm Storage Caching

It is mentioned in the docs that Esv6 series does not support Premium Storage or Premium Storage Caching. It is inaccurate. Creating this pull request to correct this.

@mattmcinnes -
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[C++] Parquet RecordReader Skip performance is bad

Describe the bug, including details regarding any error messages, version, and platform.

I did a perf and found most cpu time spent on alloc and dealloc memory. The issue looks like relates to here https://github.com/apache/arrow/blob/9e5c5b5274928c92a642657e73e8cf84f17bd7b7/cpp/src/parquet/column_reader.cc#L1494C18-L1494C32
Component(s)
C++

Would you mind try to set these record "cached" like valid_bits_ and Resize without thrink to fit?
I think this might help in some cases.


Jets doesn't seem to create lambdas and/or log groups for routes created by gems/middleware

Checklist

[X] Upgrade Jets: Are you using the latest version of Jets? This allows Jets to fix issues fast. There's a jets upgrade command that makes this a simple task. There's also an Upgrading Guide: http://rubyonjets.com/docs/upgrading/
[X] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.rubyonjets.com
[X] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System
Ubuntu 20.04

Jets
3.0.9

Ruby
2.7.2

Expected Behaviour
Any routing/endpoints defined by middleware (like omniauth) should run in one of the deployed lambdas, and should have an associated log group.
Current Behavior
As far as we can tell, routes/endpoints that are defined by middleware (like omniauth) do not get mapped to a particular lambda or in the very least aren't traceable in logs. I know this because we are getting a 500 on an omniauth endpoint when we hit it, but we can't find any logs corresponding with this 500 (or corresponding with any middleware-created endpoints).
Step-by-step reproduction instructions
Set up a blank jets project that includes a gem that defines some sort of route. In our case we are using omniauth via Auth0, however you could probably reproduce this with Sinatra for example.
Code Sample
We have a minimal setup reproducing this bug located here https://github.com/rachaelghorbani/auth0-test
Solution Suggestion
My guess is Jets is only aware of routes that are defined in routes.rb. Perhaps allow defining routes in routes.rb that should be handled by middleware, and then wire these into API Gateway properly. We don't mind having to manually annotate these routes as long as the middleware can still handle them.

Thanks for putting together the test repo. Deployed it and the controller lambda functions for the auth0_controller.rb are being created, at least when I tested. 🧐

Haven't really dig into omniauth in quite a while. Believe the workflow is that:

You try to log into a protected page
Omniauth will redirect you to a 3rd party page that handles auth/login
You login and that 3rd party login provider is configured so that it knows to call your auth0_controller.rb as a callback/hook. The 3rd party provider passes auth information, which the omniauth provider stores in the request env via the middleware. This is made available with something like request.env['omniauth.auth']  So you can store this session info and consider the user logged in.

The logs from the middleware will log in the same process as the calling lambda function. In this case, the lambda functions are the ones in the screenshots above. IE: Auth0Test-dev-auth0_controller-callback
It looks like the omniauth default logger is STDOUT, which should work. https://github.com/omniauth/omniauth/blob/a62d36b3f847e0e55b077790112e96950c35085a/lib/omniauth.rb#L28
The jets default logger writes to stderr which also shows up in CloudWatch: https://github.com/boltops-tools/jets/blob/b83e7606e7009037cef69be27ed784b31f3dbf32/lib/jets/application/defaults.rb#L53
You might want to configure the Omniauth logger to the same logger as Jets:
OmniAuth.config.logger = Jets.logger

Though omniauth logging to stdout should still show up just find in the specific lambda function CloudWatch logs.
Haven't been able to dug into omniauth much yet, so there might be other caveats. Unsure if there's much more here. Hopefully, the explanation helps though.

Thanks a bunch. I did some more digging based on what you said and I found the error. It looks like API Gateway doesn't like something about how omniauth is formatting its lambda response:
Thu Jul 01 07:05:04 UTC 2021 : Endpoint response body before transformations: {"statusCode":302,"body":["302 Moved"],"isBase64Encoded":false,"headers":{"Location":"/auth/failure?message=missing_client_id&strategy=auth0","Set-Cookie":"rack.session=truncated; path=/; HttpOnly","X-Runtime":"0.001454","x-jets-call-count":"20","x-jets-prewarm-count":"1"}}
Thu Jul 01 07:05:04 UTC 2021 : Execution failed due to configuration error: Malformed Lambda proxy response
Thu Jul 01 07:05:04 UTC 2021 : Method completed with status: 502

I'm suspicious that it might be something like this: https://stackoverflow.com/a/50263560/4245513

Dug into this a bit. Havent messed with omniauth-like workflows in a while so was unable to set it up to the point where omniauth reproduces the 302 Moved redirect.
Nevertheless mimicked it with a piece of middleware:
app/models/my_middleware.rb
class MyMiddleware
  def initialize(app)
    @app = app
  end

  def call(env)
    Jets.logger.info("middleware test")
    status, headers, body = @app.call(env)
    body = ["302 Moved"] # mimic the issue
    [status, headers, body]
  end
end

And configure it in
config/application.rb
require "#{Jets.root}/app/models/my_middleware"
Jets.application.configure do
  # ...
   config.middleware.use MyMiddleware
end

This reproduces the API Gateway error {"message": "Internal server error"}. So think you're right, the ["302 moved"] causes API Gateway to error. API Gateway is pickier about the response body. The standard is that with 302 there should be no body or guessing an empty string.
Think either have to:

Dig into the omniauth and edit it so that it does not add the  ["302 moved"]. Unsure where that is and that patch may not be accepted by omniauth 😞
Possibly add another piece of middleware after omniauth that replaces ["302 moved"] with ""

Think would give #2 a try. Hoping the middleware above is a starter example. Hope that helps. 🤞
FWIW, auth needs some love for Jets. Some more thoughts here: https://community.boltops.com/t/current-state-of-building-authentication-and-authorization-in-jets/691

Thanks @tongueroo this is great and in line with my thinking as well. I will try monkey-patching and report back here, but thanks so much for taking the time to reproduce the issue!!

This seems to work:
class OmniauthFix
  def initialize(app)
    @app = app
  end

  def call(env)
    status, headers, body = @app.call(env)
    body = body.join(', ') if [301, 302].include?(status) && body.is_a?(Array)
    [status, headers, body]
  end
end

Thanks again!


Styles

This tool has a lot of filters. I was wondering if there are presets of filter chains like in imagepipe or snapseed.
If this is not present in imagetoolbox it might be a nice addition in the future.

I can add this, vut i don't know which filters i should yse to achieve effects on the picture, you can provide it, if you can

Thanks for the fast response. I made a mistake. Imagepipe is not able to apply filters. I confused it with another tool.
Photo Editor (MIT licence) has filters. Here is a link to the code likely invoking the filters.
https://github.com/burhanrashid52/PhotoEditor/blob/master/photoeditor/src/main/java/ja/burhanrashid52/photoeditor/ImageFilterView.kt

No, i don't need such libraries, i just want you to combine my app existing filters to create some presets :) if i add that i would also add ability to create custom presets

Yes, that is of course correct. It was my assumption that you can do exactly that with the existing ~ 60 filters.
I had searched online for cookbooks that describe something like this, but unfortunately, it was not very successful.
That's why the question came up whether you can use existing libraries to do something like this with fast presets. In the end, the filters could still be processed with the existing filters.
I have tried different things, but the creation of the vignette effect, for example, actually from everything else, I have not been able to do it.
In the end, this is exactly what happened to my initial request.

Vignette works pretty well, can you then tell some names of presets you want to exist initially?

Sorry, for the confusion. I didn't mean that the Vignette effect in ImageToolbox does not work. It works very well. My apologies. I need to be more precise with comments.
What I want to say is that I cannot tell you how others do that. It appears that there are typical image processing workflows to get filters like “oil painting”, “Fine art”, “Accentuate”, “Posh”, “Grunsh”, “Vintage”, “Faded Glow”, “Silhouette” … “Morning”. For example, “Morning” from Snapseed seems to add a gradient of a yellowish tone (bottom) to transparent (top) as double exposure. I looked on the internet and there are just numerous apps doing that. But they don't disclose how they do it exactly.
Maybe in future, somebody with more artistic insight will be able to answer this.
Thanks for considering it!

Okay thanks for the idea

The latest commits (entering 2.6.1 alpha) exceed by far my enhancement request. Well done!

I also added a small preview at the start of each filter to easily understand the approximate effect👀

Alpha 02 will contain more than 44 predefined filters with no arguments

So, i think this can be closed

The previews are great and very useful.
This is definitely closed!

The previews are great and very useful.
This is definitely closed!


fix: step description, seeds and styles

fix: step test based on description
fix: suggested css styling
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #53033

Hi @RGHANILOO, thanks again for all of your help with this.
Unfortunately it looks like your last commit changed some other lines that shouldn't be touched. If I had to guess, it looks like Prettier was run against the seeds for the files that you manually updated. If you have a Prettier extension installed, I'm not sure why the .prettierignore file was, well, ignored.
Would you be able to revert those changes? It may be easiest to copy the seeds over from the main branch, then make your changes manually with Prettier disabled.
Alternatively, you could close this PR and open up a new one.


Add note on historical naming in CircuitInstruction

Summary
There was some confusion about why CircuitInstruction.operation is often an instance of a class called Instruction.  This is generally useful information to know for both users and developers, as it charts some of the history and evolution of Qiskit.

Details and comments
Fix #8711.
The output looks like this:

cc: @kevinsung, @kdk.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.303%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
57815

Relevant Lines:
68580

💛  - Coveralls


Should the CLI say 'ok' after it has executed a line, like gforth?

It seems redundant. We already display errors, why do more? Is the user going to be confused if we don't show anything?

Perhaps a command line should specify the required behavior. This is more complex than it looks because of deciding where to put the ok string (row, column) in the face of a variable output. Perhaps an interesting puzzle for someone to pick up.


handle markdown for testfiles

Hopefully a useful tool to generate markdown files for testfiles.
Usage:
------
To generate markdown files:
    python3 test_main.py --generate-markdown

To combine markdown generation with testing for a specific rule code:
    python3 test_main.py --generate-markdown grf001

Explanation:
------------
The '--generate-markdown' flag tells the script to create markdown files based on the tests.

The 'grf001' in the second example represents a specific rule code. You can replace it with any valid rule code to test.

The markdown generation process will take into account the specified rule code, if provided.

For SYS001 it generated some weird formatting (due to multiline errors?)

https://github.com/buildingSMART/ifc-gherkin-rules/commit/1311f58bc5fe3e7fbc1ae7411e1cf11dc738cc9b

Answering @aothms

And for the pass results it generated a "Rule passed" message.

I followed the conventions in  https://github.com/buildingSMART/ifc-gherkin-rules/tree/main/features#readme .
It makes somse sense to include the 'pass' files in, as that way it's still possible to add a description (for instance with 'ALB002').

I tested your script on the GRP001 PR [...]  I didn't get any content in the error column for the fail files. Is that intended?

For SYS001 it generated some weird formatting (due to multiline errors?)

I think for both GRP001 and SYS001 the error was due to multiline errors. The handling of multiple lines in one markdown cell seems to be problematic in github
https://github.com/markdown-it/markdown-it/issues/406
https://stackoverflow.com/questions/47324785/how-can-i-add-a-table-with-multi-row-cells-to-a-readme-in-vsts/47335483#47335483
There are some workarounds, but the most straightforward and often used I think is just using html. This should be rendered well in Github. In case of multiple results pertaining to one rule, the table will have additional rows to accommodate each result. Furthermore, descriptions are adjustable even after generating a markdown file.
The old markdown table will still be parsed in the script (for instance, when updating a description after 'Error')
After modifying the script (see also on the top of the PR).
SYS001

GRP001


libssl1.1 has reached EOL

https://www.openssl.org/blog/blog/2023/06/15/1.1.1-EOL-Reminder/
libssl1.1.1 has reached EOL and most distro's are swapping to libssl3.

Was this meant to be filed here?

rust-openssl supports libssl3 https://github.com/sfackler/rust-openssl/pull/1694
https://github.com/sfackler/rust-native-tls/issues/240 rust-native-tls already supports libssl3 as well. 0.10.29 is just the minimum supported version
The readme says
## Requirements

On Linux:

- OpenSSL 1.0.1, 1.0.2, 1.1.0, or 1.1.1 with headers (see https://github.com/sfackler/rust-openssl)

On Windows and macOS:

- Nothing.

Reqwest uses [rust-native-tls](https://github.com/sfackler/rust-native-tls),
which will use the operating system TLS framework if available, meaning Windows
and macOS. On Linux, it will use OpenSSL 1.1.

We should be able to use libssl3 instead of libssl1.1 now


tab-completing formula names is very slow

Please follow the general troubleshooting steps first:

[*] Ran brew update and retried your prior step?
[*] Ran brew doctor, fixed as many issues as possible and retried your prior step?

Bug reports:
tab-completion of formula names is very slow (many seconds), after updating to brew 0.9.9. I only have the core tap installed.
I'm actually unsure if these completions were already slow before, as I hadn't triggered one for a while before this latest update.

They were already unusably slow but I think maybe it got even worse?

Oh, it's scanning Library/Formulas which is a symlink to the core tap now, and then it scans all the taps, so it's ~twice as bad as it used to be.

also i'm pretty sure the algorithm used for taps is quadratic in the number of formulas

This was very fast before all core formulae were moved to a separate core tap. Now it's very slow. Just compare to brew-cask completion, which takes just a moment to complete formula name.

@pavelrad Casks are also in a tap so there's a bug here we'll need help fixing.

Why this issue is closed? It's definitely not fixed yet.

@pavelrad Because it's less slow than it was.

In my subjective experience it's now much faster than it was before the split! I don't know why we're seeing different things.

No, it is not fixed at all.
It used to take a couple hundred milliseconds.
It now takes 3-5 seconds of one of my 3.2GHz Xeon cores maxed out at 100% utilization.  Performing tab completion in brew actually makes my fans spin up.
I am not sure what the exact mechanism is yet, but the issue is homebrew core being a tap.  That's what is causing the unacceptably poor tab completion performance.
It's not the aliases, nor is it having to look through stuff twice (Formula, AND the tap).  It's just the fact that is there at all.  If you remove the Formula alias in /usr/local/Library/, as well as the Aliases alias, tap completion performance is not improved even slightly.  It's still dog slow.
If you move the Formula directory out of the homebrew tap and anywhere else, then create a new symlink to it in the /usr/local/Library directory, homebrew's old 200ms tab completion performance immediately returns.  Or just putting it back in its old spot works too.  Whatever is going on, it has nothing to do with the number of formula, and everything to do with simply being in the taps folder.

I figured out the problem.  We're getting differing results because some of us have an old completion script for homebrew that is getting loaded in addition to the new one.
Everyone, check your $(brew --prefix)/etc/bash_completion.d/ directory for any other files starting with 'brew'.  There should be exactly one file, which is called simply 'brew'.  I found I had a 'brew_bash_completion.sh' and 'brew_bash_completion.sh.default' in that directory.  They're leftovers from brew-legacy.
Remove them and the speed issues immediately go away, but completion works without issue.

Great! Removing 'brew_bash_completion.sh' really helped with this issue, thank you, @metacollin.

@pavelrad @metacollin Glad to hear it. We could/should perhaps have Homebrew remove them for you.

Yes, that would be nice to have some automatic cleanup of these old scripts.

Thirded.

We could/should perhaps have Homebrew remove them for you.

Any idea where to put this? I could imagine this being a (very lightweight) new check in brew doctor, but in this case we could only suggest the removal and not perform the removal automatically. (Since we're always asking for brew doctor output for troubleshooting, this might be sufficient.)

@UniqMartin Feels like the sort of cleanup update-report.rb does?

Feels like the sort of cleanup update-report.rb does?

Didn't think of it, but this sounds like the right place. I'm still a bit hesitant to automatically delete stuff in etc, even if we're pretty sure the files are created by Homebrew. I think so far we've refrained from doing something like this. (There might be a tiny minority of users who have customized etc/bash_completion.d/brew_bash_completion.sh and would be unhappy about it being silently wiped from their disk. Just wanted to mention this; I'm personally not very concerned that's a significant issue—unlike the tab completion slowness.)

I think so far we've refrained from doing something like this.

I think we've actually taken more extreme measures previously: https://github.com/Homebrew/brew/blob/0123e04faed98c06dc395a2ec0d7fa0de2f56d5d/Library/brew.rb#L81-L84

There's also the mv foo foo.bak approach. 👍  for auto-cleanup one way or another.
Probably paranoid, but would we want to replace it with a zero-length file in case someone's shell profile sources it without checking for existence so we don't break their terminal?

There's also the mv foo foo.bak approach. 👍 for auto-cleanup one way or another.

We'd have to move it out of the directory, but I generally like the idea of moving files instead of wiping them. The question here would be where to move them to? Do we want to introduce the equivalent of lost+found for Homebrew and where would that live?

Probably paranoid, but would we want to replace it with a zero-length file in case someone's shell profile sources it without checking for existence so we don't break their terminal?

I think in this case it's better to break the profile, so that it can be fixed. The error message will be sufficiently descriptive and if people come to us with that error, we'll know what to do. If too many issues are created after doing this (I hope not), we can still create a dummy file (with a comment that it's obsoleted by brew) there to avoid being swamped.

It's worth noting that uninstalling bash completion (either the actual package or from within homebrew) removes a large number of scripts from bash_completion.d/ and overwrites modifications etc. upon installation.  bash_completion's documentation explicitly states that user modifications to installed completion scripts in that directory will not persist over upgrades, installations, or uninstalls.  Brew's expected behavior would be to overwrite any files in that folder that it created (even if it was quite a while ago) upon an update if needed.  There is a separate folder for user modified or user created completions, $(brew --prefix)/etc/profile.d/.
I think is actually a valid concern, I've seen a lot of people who do some pretty terrible things in the name of 'ricing' their shell, like aliasing cp to call rsync, or having `which python` return a bash script that would return a path instead of a path, and I'm sure there is at least one unfortunate bastard out there who is sourcing this homebrew file, and in his system-wide bash profile, not just his user account.
But, noting what I said in the first paragraph, we could simply over write the file with an empty one, thus solving the problem without causing Mr. Unfortunate Bastard a very bad day.
An even safer option might be to only do that if it is unchanged from the original homebrew-legacy script (this would need to be a manual check I think - but it could just pull it in from github or a gist to do the check), and if there are changes, move it  elsewhere, symlink it into bash_completion.d/, track it with git in its new home (since a .git folder in bash_completions.d/ would be no bueño), and emit the familiar 'git stash pop' warning about user changes.
A solution I am not nearly as big a fan of, but is probably the simplest, safest, and also least effective, is to just add a brew doctor check for the file that recommends the user remove it manually.
Just some ideas. I have no feeling for what they think is 'ok' or not for homebrew to do, but I tried to think up things that, while involved, would minimize the invasiveness.  shrug

Am out & about, so will give more feedback later, but one quick thing to be aware of: "bash completion" can mean two things in Homebrew land. There's brew's own support for bash completion, and there's the bash-completion formula, which is a separate project that provides completions for many commands. These two things are independent. "Uninstalling bash completion" and the $(brew --prefix)/etc/profile.d sounds like you're referring to the bash-completion formula stuff. Brew's own bash completion support gets installed directly to /usr/local/etc/bash_completion.d/brew, should be independent of the bash-completion formula, and there is no support for uninstallation of it.
(There's a similar issue with zsh-completion vs "zsh completion".)
The stuff about leaving user modifications alone to "bash_completion" might be from bash-completion's own doco. AFAIK, brew's bash completion does not support user modification at all. And, as a separate issue, the way we install bash-completion might be messing with their support for unmolested user modifications, but I don't think that's the problem this GitHub issue is addressing.
Will give this a close reading and feedback later tonight.

This has been improved as much as it will be for now. If this is still broken for you: we'll accept PRs for this but we're not actively working on it at this time.


Fix incorrect file names associated with rules

I believe I have identified 2 rules with incorrect file names. This PR makes no changes to the rules or the file contents, only corrections to the file names.
Please let me know if there are any issues with this PR.
Required items, please complete
Change(s):

Rename ExcessiveNetworkFailuresFromSource.yaml to ExcessiveHTTPFailuresFromSource.yaml

Rename ExcessiveHTTPFailuresFromSource.yaml to ExcessiveNetworkFailuresFromSource.yaml
Reason for Change(s):

Current files are incorrectly named with each other's associated name

Version Updated:

No - There are no changes to any file contents, I am unsure if updating the version is appropriate for these changes. Please inform me if you would like the changes to be made, or feel free to update the version.

Testing Completed:

Yes, rules ran without any issues (there is no change to the file contents so the deployed rules are identical to previous version)

Checked that the validations are passing and have addressed any issues that are present:

Some validations are failing on duplicate names caused by the file name changes. I am unsure how to resolve. Perhaps making both file name changes in one commit could resolve this?

@microsoft-github-policy-service agree company="Department of the Premier and Cabinet"

@devikamehra : I am fine with the name change, but don't know what's the impact on the rule that is part of your solutoin, so letting you approve.

Hi @ryan-aus, can you please fix the validations?

@ryan-aus Can you please make the same changes in here as well?

@ryan-aus Can you please make the same changes in here as well? https://github.com/Azure/Azure-Sentinel/blob/master/Detections/ASimNetworkSession/ExcessiveHTTPFailuresFromSource.yaml

By changing the file names as suggested in my PR the URL in the ASimNetworkSession/ExcessiveHTTPFailuresFromSource.yaml should point to the correct rule so I don't see a need to make the changes here, unless I am misunderstanding something?

Hi @ryan-aus, can you please fix the validations

Hi @ryan-aus, waiting your reply on @devikamehra comments. Thanks

@ryan-aus Can you please make the same changes in here as well? https://github.com/Azure/Azure-Sentinel/blob/master/Detections/ASimNetworkSession/ExcessiveHTTPFailuresFromSource.yaml

edit: I understand what you are asking now. I'll get onto it :)

Hi @ryan-aus, please provide some update on this PR

Hi @ryan-aus, can you please act upon Devika's comment?

Hi @ryan-aus, kindly provide some update.

Hi @ryan-aus, can you please provide some update?

We wanted to check on the status of PR #7695. PR is pending for more than 30 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation

Since we have not received a response in the last 7 days, we are closing your PR #7695 per our standard operating procedures. If you still need support for this issue, you can re-open the PR at any time.
If you do re-open, we simply request that you ensure the PR has response to the last request. Thank you for your cooperation.


AutomaticSettingsEnum and RawRepresentable

I think considering that enums should be Codable we don't have to make them RawRepresentable in AutomaticSettingsEnum protocol. We can make them CustomStringConvertible for representing their value in UI, although that's not covering all possible cases either, so it would be more extensible if it would be a custom AutomaticSettingsPresentable protocol that returns any view.
The use case I have is to use AutomaticSettings to configure SwiftUI views from their set of props and often these props can't be represented just by a string. For example it can be a color and then we can have a color picker to choose from it (or any custom view client wants to use for that, it shouldn't be part of the library), or it can be an image then client might want to show image picker.
Another use case is to have nested settings represented as enum with associated values, they can't have raw value, but can be Codable and can be CustomStringConvertible or implement proposed AutomaticSettingsPresentable.
Here is what I put together so far, here I had to flatten enum that could have used nested enum for "avatar" prop:
https://user-images.githubusercontent.com/68597557/109497254-aad31300-7a89-11eb-81b0-094088866b8f.mp4

@IlyaPuchkaTW how'd you imagine AutomaticSettingsPresentable protocol? because what I do right now is I leverage the function DSL I build here to extend the library in my projects for custom types, e.g.
extension AutomaticSettingsViewDSL {
    @_disfavoredOverload
    func setting(_ name: String, keyPath: WritableKeyPath<Settings, Version?>, requiresRestart: Bool = false, sideEffect: (() -> Void)? = nil, uniqueIdentifier: String) -> some View

I didn't look deep into the implementation details but I guess I can try to use this attribute to come up with something that will fit my use case.


How to convert datasets.arrow_dataset.Dataset to torch.utils.data.Dataset

Hi,
I'm trying to pretraine deep-speed model using HF arxiv dataset like:
train_ds = nlp.load_dataset('scientific_papers', 'arxiv')
train_ds.set_format(
        type="torch",
        columns=["input_ids", "attention_mask", "global_attention_mask", "labels"],
    )
engine, _, _, _ = deepspeed.initialize(
    args=args,
    model=model,
    model_parameters=[p for p in model.parameters() if p.requires_grad],
    training_data=train_ds)

but deepspeed.initialize accepts torch.utils.data.Dataset only. How can I convert HF-style dataset to torch-style dataset?

Hi,
a HF dataset can be converted to a Torch Dataset with a simple wrapper as follows:
from torch.utils.data import Dataset

class HFDataset(Dataset):
    def __init__(self, dset):
        self.dset = dset

    def __getitem__(self, idx):
        return self.dset[idx]

    def __len__(self):
        return len(self.dset)

train_ds = HFDataset(train_ds)

@lhoestq Since the Arrow Dataset already provides __getitem__ and __len__, I think we could use the virtual subclass mechanism from the abc module to elegantly solve this issue. This mechanism would allow the Arrow Dataset to be used in place of the Torch Dataset because the isinstance(instance of Arrow Dataset, TorchDataset) check would return True (DeepSpeed has this check here).
And it requires a minimal change in the arrow_dataset.py file:
if config.TORCH_AVAILABLE:
    from torch.utils.data import Dataset as TorchDataset
    TorchDataset.register(Dataset)

Interesting ! Thanks for sharing this @mariosasko . I like the idea
This looks like something we should add IMO

@mariosasko
Thx for your code!
It perfectly works with a small modification for HF NLP dataset:
original_ds = nlp.load_dataset('scientific_papers', 'arxiv')
train_ds = HFDataset(train_ds['train']) # needs splitting

@lhoestq Sadly, from Python 3.7 onwards torch.utils.data.Dataset doesn't support the virtual subclass mechanism due to typing.Generic type no longer having abc.ABCMeta as its metaclass.
With that in mind, another option is to remove a direct type check (isinstance(dataset, torch.utils.data.Dataset)) in deepspeed.initalize and to rewrite the checks in a manner similar to torch.utils.data.DataLoader (link). This is exactly why the DataLoader works with arbitrary objects that provide __getitem__ and __len__ (and in our case, the ArrowDataset). By doing so, their code wouldn't be any stricter in comparison to the DataLoader.
So if you agree, I can open an issue in their repo and fix this if they like the idea.

That makes sense ! Feel free to open an issue on their repo and discuss this idea

@y-rokutan Hi, now if you install deepspeed from master (this feature will be available in the next official release), the code should work without subclassing. Let us know if you still have any issues.

Worth mentioning that any function that expects a torch..Dataset (like torch..DataLoader) will fail a mypy-esque typecheck if a datasets.Dataset is passed, even though it implements the interface correctly (I think). The virtual subclass idea was a good one- I wonder if there's another workaround given the Generic issue. What we're really talking about is something similar to the structural subtyping semantics that typing.Protocol defines. If torch..DataLoader accepted anything that supports __getitem__ and __len__ methods this would be much easier. Not sure if there's a way to do this without the wrapper from the perspective of datasets.


Redstone Integrator does not output redstone signal

Descripe
The Redstone Integrator does not output any signal, despite explicitly turning it on in the lua REPL. Redstone dust is not activated, and redstone lamps do not light up.
Please note that I've selected Version 1.16.5-0.7.7r only because 1.16.5-<IP_ADDRESS>r isn't available to be picked. I am using the very latest version for Minecraft 1.16.5.
Steps to reproduce

Attach a Redstone Integrator to an Advanced Computer using wired modems.
Place a redstone lamp, connected to any side of the Redstone Integrator with redstone dust.
In the lua REPL on the computer, run the following code:rsi = peripheral.wrap("redstoneIntegrator_0")
rsi.setOutput(<insert redstone dust side here>, true)

Observe that the lamp does not turn on.

Multiplayer?
No
Version
1.16.5-<IP_ADDRESS>r
Minecraft, Forge and maybe other related mods versions
Minecraft 1.16.5 Forge 36.2.26 CC: Tweaked 1.100.2 JEI <IP_ADDRESS>
Screenshots or Videos
No response
Crashlog/log
No response

0.7.7r should not even work due to the reason that it does not contain any class files.
Are you sure you play with this version?

As I stated near the top of my message:

Please note that I've selected Version 1.16.5-0.7.7r only because 1.16.5-<IP_ADDRESS>r isn't available to be picked. I am using the very latest version for Minecraft 1.16.5.

I wasn't able to pick <IP_ADDRESS>r when filling out the form.

Sorry, I misunderstood that

I tested it a bit. I can approve that the redstone integrator does not work in 1.16. But it works in 1.18.

If you want, you can use this version.
advancedperipherals-1.16.5-<IP_ADDRESS>r.zip
Unzip the file before using.

To confirm, despite having the same version number, the code is different?

yes it's
I just didn't changed the version in the properties file(I only do that when I do a update)

I've finally had a chance to try your patched version, and can confirm that it works 👍

I've finally had a chance to try your patched version, and can confirm that it works 👍

I finally had a chance to prepare the release 😓
Releasing later today

Fixed in 0.7.8r


fix: fix linked service on cog service base (#1685)

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

Codecov Report

Merging #1745 (a153540) into spark3.1 (7abf839) will decrease coverage by 81.15%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           spark3.1   #1745       +/-   ##
============================================
- Coverage     85.83%   4.68%   -81.16%
============================================
  Files           272     272
  Lines         14230   14230
  Branches        767     767
============================================
- Hits          12215     667    -11548
- Misses         2015   13563    +11548

Impacted Files
Coverage Δ

.../azure/synapse/ml/cognitive/FormRecognizerV3.scala
0.00% <ø> (-88.00%)
:arrow_down:

...oft/azure/synapse/ml/cognitive/TextAnalytics.scala
0.00% <ø> (-86.51%)
:arrow_down:

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/com/microsoft/azure/synapse/ml/stages/udfs.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/microsoft/azure/synapse/ml/vw/VectorZipper.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...rg/apache/spark/injections/BlockManagerUtils.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/microsoft/azure/synapse/ml/codegen/CodeGen.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../microsoft/azure/synapse/ml/io/binary/Binary.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../microsoft/azure/synapse/ml/param/ModelParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../microsoft/azure/synapse/ml/param/UDPyFParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 255 more

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

/azp run


Feature Request: Ability to replace unknown types with error state dummies

Hi team,
I'm working on a project which has multiple distributed software components, which will at some times have different versions and I need to make the configuration type binding highly error tolerant.
If configuration needs to be updated for a new code base say, I want to make sure older versions of the software can gracefully still load up, and present to the user information about the object that could not be loaded.

In the error handler callback, I would like the ability to access the parsed data for the unknown type so i can make a dummy
OR
in the binder, I would like to be able to know the nominal type

so if a property is defined as an IShape and it has a value with an unknown $type: Square, if the binder could know it was an IShape i could make a dummy IShape and it could still be parsed correctly, putting unknown data into the JsonExtensionData

With the current functionality, I am so close to being able to do what I would like to do but I just can't quite do it other than having to do all of the binding manually with my own code, or make custom converters for all the levels
I think the binder option would probably be the better one because then there will be no exceptions flying all over the place
This sort of functionality would make version updates so much more graceful
Thank you very much for the great project!!!
Thoughts?

For now, what I've managed to do is use the Error delegate to absorb the error, and basically skip the un-bindable types.
The only problem with this is I can't deserialize the unknown with a dummy type and present it to the user in the originals place.


IT security

Highlight that IT security needs to be contacted about site access like GitHub, published pages in GitHub, Slack, Netlify. People often get permissions that change as security is built upon and IT helpdesk may not necessarily understand the implication where IT security staff would.

https://github.com/best-practice-and-impact/govcookiecutter/issues/17


error on decrypt process

hey bro,  i got error during decrypt hudson.secret.key:
Traceback (most recent call last):
File "C:\decryptor\decrypt.py", line 50, in
main()
File "C:\decryptor\decrypt.py", line 33, in main
x = o.decrypt(hudson_secret_key)
File "build\bdist.win32\egg\Crypto\Cipher\blockalgo.py", line 295, in decrypt
ValueError: Input strings must be a multiple of 16 in length
thanks for help!

Hey,
It seems like you have the wrong size for the file hudson.util.Secret. Could you confirm it is 272 bytes long?
Otherwise, the format may have changed since I wrote that script. Feel free to follow thiébaud.fr/jenkins_credentials.html, you may notice some difference with your version.
Oh, and please: http://imgur.com/ybsOOsX

hey,
yes it's 272 bytes.

and sorry about bro :)

Hum... have you modified the script? Because line 33 (where the error is triggered) should be 'for password in passwords:' and not that decrypt call.
Could you add a 'print  hexlify(hudson_secret_key)' just before the decrypt call to see the exact value used?
Also, your credentials.xml should contains a  tag (with a lower case). I do think you are using a different format that may not be compatible with the version I tested. Did you edit the credentials.xml by hand?

hi,
i changed the script just with some print commands.
i'm attaching the output of 'print hexlify(hudson_secret_key) as you suggested.
and credentials.xml, decrypt.py files

Ah I think I know, the Python file handling on Windows requires a special flag for binary files. Try to replace the
hudson_secret_key = open(sys.argv[2]).read()

with
hudson_secret_key = open(sys.argv[2], 'rb').read()

If that works, could you patch your fork and send a pull request? Thanks

well, it's working now,
thanks!


desktop_webview_window in macos don't run

Version problem, request help！！！

this plugin minimum depolyment target required macOS 10.12
Maybe you need upgrade you flutter app min depolyment target on macOS.

this plugin minimum depolyment target required macOS 10.12
Maybe you need upgrade you flutter app min depolyment target on macOS.


Amwg1 speedup

This branch provides the speedup for generating the table.  It is based on how it was done in metadiags, namely a separate execution of diags for each entry in the table specification.  There is final testing on edison All tests passed.

All tests pass.


Footer - text to the right jump up when mouse hover over the "Get our awesome emails" button

Platform / OS
Windows 10 with Chrome
Description
When "Get our awesome email" button is hovered over, the text just below it jump up.
Steps to Reproduce

Open https://www.sleepbus.org
Scroll down to bottom of the page
Move mouse over the "Get our awesome email" button

Expected Behavior
Only button should highlight without making any change to text around it.
Additional Info
None

NVM, looks like repository is outdated.

@mohsinrasool, yes, but thanks anyway, I'll forward to Sleepbus people.


Fix SFTP setting in OpenSSH config on Debian/Ubuntu

I found that SFTP wouldn't work on my Ubuntu master node - this was because the SFTP subsystem setting in /etc/ssh/sshd_config was RedHat specific.
On Debian/Ubuntu systems, the appropriate line is:
Subsystem sftp internal-sftp
This patch (should) fix this in the Chef recipe.

Thanks, we're just working on getting some big updates out. Is it possible for you to make this PR against the develop branch?

@dougalb Whoops - that was my intention. New PR here: https://github.com/awslabs/cfncluster-cookbook/pull/17


System.IndexOutOfRangeException: Index was outside the bounds of the array. - Spatial4n.Util.GeohashUtils.DecodeBoundary

Sometimes under heavy load I see this in our logs:
 System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at Spatial4n.Util.GeohashUtils.DecodeBoundary(String geohash, SpatialContext ctx)
   at Spatial4n.Util.GeohashUtils.Decode(String geohash, SpatialContext ctx)
   at Lucene.Net.Spatial.Util.ShapeFieldCacheProvider`1.<>c_DisplayClass5_0.<GetCache>b_1()
   at System.Lazy`1.CreateValue()
   at System.Lazy`1.LazyInitValue()
   at Lucene.Net.Spatial.Util.ShapeFieldCacheDistanceValueSource.CachedDistanceFunctionValue..ctor(ShapeFieldCacheDistanceValueSource outerInstance, AtomicReader reader)
   at Lucene.Net.Spatial.Util.ShapeFieldCacheDistanceValueSource.GetValues(IDictionary context, AtomicReaderContext readerContext)
   at Lucene.Net.Queries.Function.FunctionQuery.AllScorer..ctor(FunctionQuery outerInstance, AtomicReaderContext context, IBits acceptDocs, FunctionWeight w, Single qWeight)
   at Lucene.Net.Queries.Function.FunctionQuery.FunctionWeight.GetScorer(AtomicReaderContext ctx, IBits acceptDocs)
   at Lucene.Net.Queries.CustomScoreQuery.CustomWeight.GetScorer(AtomicReaderContext context, IBits acceptDocs)
   at Lucene.Net.Search.Weight.GetBulkScorer(AtomicReaderContext context, Boolean scoreDocsInOrder, IBits acceptDocs)
   at Lucene.Net.Search.IndexSearcher.Search(IList`1 leaves, Weight weight, ICollector collector)
   at Lucene.Net.Search.IndexSearcher.Search(IList`1 leaves, Weight weight, ScoreDoc after, Int32 nDocs)
   at Lucene.Net.Search.IndexSearcher.Search(Weight weight, ScoreDoc after, Int32 nDocs)

Looking up to GeohashUtils.cs I found a comment "//TODO check successful?" at https://github.com/Spatial4n/Spatial4n/blob/b5dea48324d6e6423bfc2a3e2551778a4b1bbcf7/src/Spatial4n/Util/GeohashUtils.cs#L186
I am curious if you can elaborate on the comment. Could this be related to my issue?

TODO is not owned by us, that is something that was added upstream in spatial4j. They may also be able to assist you with this issue.


use thread pool to do commit log segment sync parallel

This does not have a JIRA ticket in Apache JIRA. Would you mind to create it? Is this still relevant?


Installation fails on Cuda12

Hi,
I was wondering if there is a new wheel for cuda12 installation? I tried this on a debian 12 system with cuda12, however the setup fails to build the wheel. I also tried with cuda11.3, where it fails because debian12 has a g++ version 12 and it requires a g++ version <=10.
Thanks

Unfortunately, the strict CUDA version requirement is a result of the OpenFold dependency. There may be a way to work around it with a modified OpenFold fork, but at the moment I would suggest trying to install a compatible g++ with cuda11.3.

In theory, you should also be able to install a g++ version compatible with your cuda inside the conda environment. Although we also had cuda11.6 installed, and when I encountered the same problem (g++ version mismatch), I just did the installation with cuda11.6 and it was done without a problem

That is what I ended up doing, installing a g++10 on my debian12 setup within a venv and LMOD setup. For now please feel free to close the issue. Thank you for the help


Kanban setup

Github project link: https://github.com/chumaachike/reactproject/projects/1
There are two people in this team namely: Achike Chuma and Morris Emeka, we have assigned specific tasks to individuals

Hello Team👋,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉 🎉 🎉 🎉 🎉
Cheers and Happy coding!👏👏👏

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Build Test Runner

This issue is part of the migration to v3. You can read full details about the various changes here.
In Exercism v3, one of the biggest changes is that we'll automatically check if a submitted solution passes all the tests.
We'll check this via a new, track-specific tool: the Test Runner. Each test runner is track-specific. When a new solution is submitted, we run the track's test runner, which outputs a JSON file that describes the test results.
The test runner must be able to run the tests suites of both Concept Exercises and Practice Exercises. Depending on the test runner implementation, this could mean having to update the Practice Exercises to the format expected by the test runner.
Goal
Build a test runner for your track according to the spec.
If you are building a test runner from scratch, we have a starting guide and a generic test runner that can be used as the base for the new test runner.
If a test runner has already been built for this track, please check if it works on both Concept Exercises and Practice Exercises.
It can be very useful to check how other tracks have implemented their test runner.
Tracking
https://github.com/exercism/v3-launch/issues/4

The Cpp test runner is https://github.com/exercism/cpp-test-runner

@patricksjackson It was last updated five months ago, so it might be work checking to see if it works on http://exercism.lol/

This issue can be closed as the test runner is built and working.


Change timeframe

Describe the bug
Iv'e made a selectbox that sends a new timeframe to my server, fetch new data for that timeframe and send back new candles to my chart. When new data is received it updates the chart but its really buggy to draw the new candles as they overlap or gets thin "yRange" not re-calculating correctly.
Iv'e tried alot of different things and made sure i get new data before update but the problem is still there.
My recent code is to really make sure i get the new data before making the update.
export let bar = "15m";
    let prevBar = bar;
    $: {
        if (bar !== prevBar) {
            console.log("bar changed to", bar);
            update();
            prevBar = bar;
        }
    }
const update = () => {
        setTimeout(() => {
            chart.update("full", { resetRange: true });
        }, 200)
    }

Ive also tried with setTimeout and wait 2 seconds before the update and still have the same problem.  it work 50/50.
example of error when changed from 15m to 1m chart
Also tried with update("data"), update(), update("legend") etc etc.
Any suggestions on how i should handle timeframe change in the data?.
Best / F

Reproduction

Steps to reproduce
Live data from server
afterUpdate(async () => {
        await tick();
        if (chart?.data?.panes?.length > 0) {
            chart.data.panes[0].overlays[0].data = candlesticks;
            chart.data.panes[0].overlays[1].data = indicators.sma;
            chart.data.panes[0].overlays[2].data = orderbook;
            chart.data.panes[0].overlays[3].data = positions;
            // chart.data.panes[0].overlays[4].data = candlesticks;
            chart.data.panes[0].overlays[4].data = orders_history.reverse();
            chart.data.panes[1].overlays[0].data = indicators.rsi;
        }
        chart.update("data");
    });

Javascript Framework
no-framework (vanilla-js)
Logs
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a NightVision issue and not a framework-specific issue.
[X] The provided reproduction is a minimal reproducible example of the bug.

Found an issue, my candlestick data has  ohlc = [...candlesticks, ...candlesticks_history]
The history candles did not update as fast as the candlesticks chunk and the data had two different timeframes for a few ms.
Removed my history candles and it works alot better.  Will load the history as in the chart example: https://codesandbox.io/s/playing-around-10-0zh0jh?from-embed
Im still interested in a better solution to change timeframe :)

I would just create a separate component that access the timeframe as a prop, then when changing the timeframe just reinit the component

Live data from server
afterUpdate(async () => {
        await tick();
        if (chart?.data?.panes?.length > 0) {
            chart.data.panes[0].overlays[0].data = candlesticks;
            chart.data.panes[0].overlays[1].data = indicators.sma;
            chart.data.panes[0].overlays[2].data = orderbook;
            chart.data.panes[0].overlays[3].data = positions;
            // chart.data.panes[0].overlays[4].data = candlesticks;
            chart.data.panes[0].overlays[4].data = orders_history.reverse();
            chart.data.panes[1].overlays[0].data = indicators.rsi;

And how to get the data using the built-in indicators? I can't figure it out. I don't want to do this on the server somehow, but apparently I will have to.


Add missing enum variants for FFmpeg 6.1.0

I noticed that there are some new variants in 6.1.1, but this should fix 6.1.0 compilation.
Preferably merged after #15 so this is verified, it's a bit difficult to test with different FFmpeg versions locally.

I guess the docker images already use 6.1.1...

Note that FFmpeg 6.1 now passes CI (<5.0 failures are something else). PR is ready


yAgents vs yWorkflows

Wondering if yWorkflows are still relevant in the yAgents application.
incredible work so far and this project needs more attention. when an app on top of this goes viral, people will realise the potential. I've seen some no-code ai app builders offering ui tools and promising something like yAgents under the hood. but its just demos mostly. checkout: imagica.ai and brancher.ai .. may be you could work on UI as well to monetise this tech better. since one of your goals is to help yAgents to be monetised, ai apps with cool interfaces built on top of yAgents are actually monetisable.

yWorkflows will be discontinued, as it is now being replaced by yAgents. There have been several refactors and changes that make yAgents much more powerful.


Compatibility with Python 3.5 (and earlier)

Hi!
The latest version is not compatible with python 3.5 because is uses, in one instance, the string format f"this is the string {variable name}".
This formatting is nicer, but I think it is not worth to give up compatibility for one single line. Moreover, since I need python 3.5 for other reasons, I can't use the new version without making changes so I hope you will accept this changes.
I run all tests and they passed smoothly.
Thank you!
Giovanni

Hi @Giovanni-Alzetta, Thank you for the pull request. It looks good to me.
After merging, I will set up tox to ensure it works on all Python 3.x and release soon


[Question] Following Semantic Versioning

From what I know about Semantic Versioning, I think that we aren't following it strictly.
Shouldn't the version number be bumped according to:

Major number: When the API changes in a way that introduces breaking changes.
Minor number: When new features are introduced.
Patch number: When bugs get fixed or enhancements are made to the code base.

Don't get me wrong here @robertohuertasm, you are in charge and how you bump the version number is up to you. I'm just struggling with my O.C.D. 😄

You're right.

To specify what I mean with "new features", I believe that every time a new folder or file gets supported/added the minor number should get bumped.

@JimiC, you're right, I'm not following it strictly. Only for major versions until now. But I have no problem in adopting it so the community can be happier and your O.C.D stops haunting you 😉


ft mutation seems to think it succeeds when it doen't

$ echo 'open the pod bay doords please hal' > foo; radamsa -m ft -p od < foo > bar; diff foo bar
1c1
< open the pod bay doords please hal

open the pod bay doords please hay doords please hal
$ echo 'open the pod bay doords please hal' > foo; radamsa -m ft -p od < foo > bar; diff foo bar
1c1
< open the pod bay doords please hal

open the pod bay doords please hadoords please hal
$ echo 'open the pod bay doords please hal' > foo; radamsa -m ft -p od < foo > bar; diff foo bar
$ echo 'open the pod bay doords please hal' > foo; radamsa -m ft -p od < foo > bar; diff foo bar
1c1
< open the pod bay doords please hal

opelease hal
$ echo 'open the pod bay doords please hal' > foo; radamsa -m ft -p od < foo > bar; diff foo bar
$

$ I="open the pod bay doords please hal" echo "$I" | radamsa -m ft -p od -n 100 | grep "$I" | wc -l
0
$


Recursive endpoint for retrieving details about an inscription

This should be useful for a variety of use cases.

I was thinking about making this endpoint and I would have done the ones you didn't include haha.
What I have wanted for months are
Address
Sat
Output
We should add output so we can eventually do a output recursive endpoint.  This would have tons of uses. You could do dynamic content like a website that you can update based on coin control. Having an inscription aware of what output its on is how you would do this.
Address would also play into this and all those combined gets us very close to making a PSBT marketplace all on inscriptions.

I wanted to keep the response as minimal as possible, as we can always add stuff. But probably never remove. Happy to add whatever we want, if there's consensus. Output sounds good to me!
Re: sat, I think that'd require sat index, no? That's why I didn't add it.

Re: sat, I think that'd require sat index, no? That's why I didn't add it.
Yeah but were already crossing that bridge with sat endpoint. So it would return null probably if no sat index.

if there's consensus. Output sounds good to me!

@rot13maxi and I were talking on the sat endpoint spaces the other day about how we really want to get to the point where we can start making crazy cool inscriptions that get us closer to calling data on the bitcoin node. Been wanting to see if we can get openordex onto an inscription since it came out. Knowing an inscriptions output is invaluable for this kinda stuff.

I'll work on adding output to JSON.
Still not convinced about sat. My gut feeling is that we should refrain from adding recursive endpoints that rely on a full sat index. The risk is creating two classes of inscriptions, especially as the index grows larger and slower with every block.
We can achieve most of the things with /children recursive endpoint that would be made possible with sat endpoint. Agree that sat endpoint would make it much easier as there are no sit ups involved in figuring out the inscription id. But it also comes at a cost and maybe something we can punt on in favor of /children.

Would be awesome to add the Sat Endpoint. IMO the Sat Endpoint is the most important endpoint for Ordinals right now. With the Sat Endpoint, you can futureproof your collection so that you can access all of the endpoints that come out in the future, and then you can lock down your work by burning the sat. This would probably also give the dev some breathing room from people wanting stuff right away because they can just add the Sat Endpoint and then get it when it comes out.

I'm curious to know what thing is possible with sat endpoint that's not possible with a children endpoint?

@elocremarc Added satpoint to response JSON.

I'm curious to know what thing is possible with sat endpoint that's not possible with a children endpoint?

Sat is more controlled by creator and child can be collector basically. However you could make the sat endpoint work for a collector if you reference the sat that an inscription is being inscribed to. With sat you can know what the sat is prior to inscribing. However if you want to know the sat in some future child its good to have it here in this endpoint because it might not be known ahead of time.

@elocremarc @casey I am planning to work on this again this week. Let me know and I can add whatever other keys to JSON.

Been making some onchain inscription viewers I think having parent would also be super helpful here. Then you can traverse up and down the family tree.
example: if you are on some inscription then you can see the siblings with the /children endpoint. Or find the ancestor.

Let me know and I can add whatever other keys to JSON.

My vote adding the sat_number and parent to the JSON

@casey don't we wanna add the sat number in this endpoint? This is very useful combined with the child endpoint or doing the sat endpoint on children you might not know the sat number for prior to inscribing like with batch inscriptions.

YES PLEASE. Right now we have to inscribe the sat number itself, either in the individual inscription metadata or in a lookup table afterwards. Both are hacky solutions where truth is what the dev wrote and not the blockchain. Biggest gap for us right now.

@owenstrevor Yeah, planning to work on this today!

Is this ready for review?

@raphjaph Going to working on the requested changes today!

@raphjaph Updated the branch with the suggested changes. Few open questions:

I think we should make it clear that this recursive endpoint may add new keys in the future. For example, we may want to add delegate etc.
I added sat to the JSON. The request will fail with 404 if sat index isn't found. Can you please review that bit?

I think we would also want to add things like charms

@raphjaph I think I addressed all of the comments, except address one.

Added charms and output
Made sat optional

Still holding out some hope for including address. It'll unlock so many use cases! I am curious if there's some way without having to compromise on address reuse. Maybe the address could be hashed?

What use cases does address unlock?

@raphjaph The primary one I'm interested in is using address for randomization seed for generative art, where rarity would be tied to an address. Moving inscription to a new address would generate a new output. But moving it back to the previous address would then generate the output you had before.
Additionally, you can also do soulbound'esq inscriptions where the inscription behaves different when it's held by a specific owner.

@raphjaph Removed the address

@devords Thank you for raising this. I've been thinking about such endpoints since mid-2023 and how much this kind of self-data would benefit the inscription ecosystem. I didn't realise that it was already in progress until @gmart7t2 pointed me here. I would like inscriptions be able to reference their own:

Inscription ID
Sat number
UTXO location
Address location
Metadata

It looks as though some of these are being pushed through soon? Could you clarify which ones are currently looking likely to be included and then I may respond with some arguments for including the others? :)

@raphjaph The primary one I'm interested in is using address for randomization seed for generative art, where rarity would be tied to an address. Moving inscription to a new address would generate a new output. But moving it back to the previous address would then generate the output you had before.
Additionally, you can also do soulbound'esq inscriptions where the inscription behaves different when it's held by a specific owner.

Yes 100% this!

@raphjaph I think I addressed all of the comments, except address one.
* Added charms and output

* Made sat optional

Still holding out some hope for including address. It'll unlock so many use cases! I am curious if there's some way without having to compromise on address reuse. Maybe the address could be hashed?

Is this just a general "reusing bitcoin addresses is bad" argument, or is there something more specific to ordinals/inscriptions? Thanks

I also have Gen art that depends on the art changing based on the address that holds the inscription.Since a majority use wallets like xverse, unisat, etc. Which uses a constant static receiving address, we need to open the possibility to take advantage of this for creative features such as dynamic art based on the address of the holder that owns it.Sent via the Samsung Galaxy Note20 Ultra 5G, an AT&T 5G smartphone
-------- Original message --------From: ZedZeroth @.> Date: 2/18/24  04:46  (GMT-08:00) To: ordinals/ord @.> Cc: Subscribed @.***> Subject: Re: [ordinals/ord] Recursive endpoint for retrieving details about an
inscription (PR #2628)
@raphjaph The primary one I'm interested in is using address for randomization seed for generative art, where rarity would be tied to an address. Moving inscription to a new address would generate a new output. But moving it back to the previous address would then generate the output you had before.
Additionally, you can also do soulbound'esq inscriptions where the inscription behaves different when it's held by a specific owner.
Yes 100% this!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

This will be a fun one! Thank you @casey and @raphjaph 🙏

Thanks for this. I think I am going to map out what inscriptions can currently access, and what they can't, and then push for "full inter-ordinal operability" where possible. It looks like a sat/inscription's current address is the only major one missing though? I have some further ideas but they may be more technically challenging/prohibitive to include. Huge progress with this has been made in the last six months though, it's fantastic for building :)


Is there an example of using a tcomb generated reducer and combineReducers?

I'm attempting to utilize the automatically generated reducer with combineReducers using the latest version of redux-tcomb and redux. When calling redux's combineReducers, I'm getting an error. From looking over the issues for this project, it appears that this should be possible based on a PR that was done for redux. I get an error thrown that says path.join is not a function which originates from Struct in tcomb (our state is a Struct). Is there an example of how to do this?

Hi,
Could you please post a minimal example reproducing the error?

@gcanti thanks for the quick response. I was able to make it past the issue I was having with the path.join error by changing the way I was calling combineReducers. However, now I've run into an issue where I cannot publish actions that are not specified by the action union.
Minimal example located here: https://tonicdev.com/zpratt/redux-tcomb-example
My real world use case is that I want to use redux-tcomb with the redux-simple-router and I get errors with a I dispatch pushPath since it publishes a redux-simple-router specific action and tcomb doesn't have a corresponding action defined in the union.
I think it would be useful to have an example of how to use combineReducers here. Once we figure this out, I'll provide a PR to capture how to wire it all up.

The point of redux-tcomb is to ensure type safety, hence the raised error is expected: the other action is not recognized so redux-tcomb thinks it's a development error. I see 2 solution:

Maintaining strong type safety (suggested): add the other action to Actions

var Other = t.struct({
    wizards: t.list(t.String)
});

Other.prototype.patch = function (state) {
  return state;
};

var Actions = createUnion({
    AddName: AddName,
    DeleteNames: DeleteNames,
    other: Other
});

....

Decrease type safety: ignore not recognized actions

var typeCheckedActions = {
    AddName: AddName,
    DeleteNames: DeleteNames
};

var Actions = createUnion(typeCheckedActions);

// dummy action
var Dummy = t.struct({});
// patch is just the identity
Dummy.prototype.patch = function (state) { return state; };

// override the default union type dispatch
var defaultDispatch = Actions.dispatch;
Actions.dispatch = function (action) {
  return typeCheckedActions.hasOwnProperty(action.type) ? defaultDispatch(action) : Dummy;
};

...

I think I've got my answers for my initial topic. I propose that we close this issue an open another issue to discuss how to integrate 3rd party actions (like the pushPath action that comes with the redux-simple-router. Thoughts?


Could this be made to support NicoNico too?

NicoNico, or better known as Nico Nico Douga or NND is a Japanese video website but it do have English interface support to some extend.
Main website : http://www.nicovideo.jp/
Normal video pages URL are usually in format of
http://www.nicovideo.jp/watch/ <video ID>
Embed video pages URL are usually in format of
http://embed.nicovideo.jp/watch/ <video ID>
Embed video doesn't need login information to play.
I'm asking this because some of original official video uploaders only upload to NND and their video reprints on YouTube always get taken down. If NND could be supported then I'll link the source directly to the original videos on NND instead of those reprints on YouTube.
I'm currently using mediaelement to play videos from YouTube with custom subtitle containing video transcriptions in our language in VTT format. Most of the video uploaders doesn't respond to our request of adding subtitle in other languages directly to the video captions even if we've made it for them so we're using this as a workaround to display subtitles while still playing the original official videos off YouTube.

@MuhdNurHidayat The problem is that, if the NND doesn't provide an API to interact with the videos, MEJS cannot work with it. For what I've seen, they just allow you to upload videos but that's it

do you actually mean something like these though?

from the website itself, but only in Japanese: http://dic.nicovideo.jp/a/ニコニコ動画api
some code found online:

https://codepen.io/hayu/pen/wJWdBw
https://github.com/ra-gg/node-nicovideo-api
https://github.com/mo-san/niconico-tools
https://www.rubydoc.info/gems/niconico/1.6.0/Niconico
https://rubygems.org/gems/nicovideo/versions/0.1.8
https://pypi.org/project/nicopy/
https://www59.atwiki.jp/nicoapi/pages/31.html
http://embed.ly/docs/explore/oembed?url=www.nicovideo.jp%2Fwatch%2Fsm11252319

From normal user view, for example the video here http://www.nicovideo.jp/watch/sm11252319
The video ID is sm11252319
The normal embed code (which apparently is a JavaScript) as shown on video page is
<script type="application/javascript" src="https://embed.nicovideo.jp/watch/sm11252319/script?w=640&h=360"></script>
When we open the JavaScript in new tab, we could see the actual embed is https://embed.nicovideo.jp/watch/sm11252319?oldScript=1&referer=&from=0&allowProgrammaticFullScreen=1
When we get to that embed page, the actual video is in html > body > div#ext-player.someID1 > div#rootElementId.someID2 > div.someID2 > div.someID3 > div#video.someID4 > video and this way it's MP4
If we on the other hand try to load http://flapi.nicovideo.jp/api/getflv?v= <video ID> , the URL field in the respond itself is a percent-encoded link to the video, but in this way it's FLV.


Not updated in the marktplace

I See that has a change 3 weeks ago, that allow compatible with the 2023.2, but it still doesn't appears on the marketplace, anyway can help with it?

Hello Everyone,
My apologies for the delay, and thank you for your patience. 🙏
I'm excited to share that the update has now been published and is under review. 🚀

Please feel free to try it out and share any feedback or issues you may encounter here. 🐞📝
Thank you for your support and understanding during this time.
Best Regards,
Francisco Marin


ci: upgrade Windows runners to windows-2022

#666  👈 (View in Graphite)
main

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


x/pkgsite: package removal request for github.com/ruralcoder/gotribe

What is the path of the package that you would like to have removed?
github.com/ruralcoder/gotribe

Are you the owner of this package?
Yes

What is the reason that you could not retract this package instead?
Moved it under github.com/hackemy/gotribe

Package has been removed!


WIP Issue/add solidity #2047

WIP to add language from https://github.com/highlightjs/highlightjs-solidity
I have manually tested but having issues running npm test getting: Error: Cannot find module '../../build'

Solidity should stay in it’s own repo :) we are splitting out the languages.

Solidity should stay in it’s own repo :) we are splitting out the languages.

@marcoscaceres if languages are being split into their own repos how can languages be included/configured?
I want to get Solidity highlighting in a hosted Discourse forum and their recommendation was to get Solidity added to highlightjs: https://meta.discourse.org/t/how-to-get-solidity-syntax-highlighting/117774

@abcoathup  I think @marcoscaceres  means it should live in a repo under: https://github.com/highlightjs ?
@marcominetti  we now figure out what options we have for languages by simply grabbing highlight.js from npm, should we amend our process to walk through all the repos owned by highlightjs? is there a central registry listing supported syntaxes you are looking to maintain?


dropEffect should be set when dragging and dropping

Hi,
we noticed an issue with the drag and drop of emails coming from for example Outlook. When dropping an email into the upload field, the mail gets removed from the mailbox.
This can be fixed by setting
event.dataTransfer.dropEffect = 'copy';
in the dragenter, dragover, drop events. Unfortunately we have no access to them using ng-flow.js
Maybe this can be a configuration parameter in the future?

Pull requests are welcome


Display task

User can look at a task.
acceptance criteria:

Task can be displayed
view for task which contains all components

Moved to Sprint 2 because of time management/similarities to update task issue #4
Should a fragment be used or dialog


opt  merge each input block into the build block

Proposed changes
In non-spill
the past approach was to accumulate blocks and only merge them into one build block when get eos.
This caused the build sink side of the pipeline to be fast before get eos, resulting waiting for upstream data.
                     HASH_JOIN_SINK_OPERATOR  (id=4  ,  nereids_id=342):
                            -  CloseTime:  avg  0ns,  max  0ns,  min  0ns
                            -  ExecTime:  avg  17s635ms,  max  17s635ms,  min  17s635ms
                            -  InitTime:  avg  0ns,  max  0ns,  min  0ns
                            -  InputRows:  sum  10.0M  (10000000),  avg  10.0M  (10000000),  max  10.0M  (10000000),  min  10.0M  (10000000)
                            -  MemoryUsage:  sum  ,  avg  ,  max  ,  min
                                -  BuildBlocks:  sum  10.29  GB,  avg  10.29  GB,  max  10.29  GB,  min  10.29  GB
                                -  BuildKeyArena:  sum  10.37  GB,  avg  10.37  GB,  max  10.37  GB,  min  10.37  GB
                                -  HashTable:  sum  102.15  MB,  avg  102.15  MB,  max  102.15  MB,  min  102.15  MB
                                -  PeakMemoryUsage:  sum  0.00  ,  avg  0.00  ,  max  0.00  ,  min  0.00
                            -  OpenTime:  avg  7.802us,  max  7.802us,  min  7.802us
                            -  WaitForDependency[HASH_JOIN_SINK_OPERATOR_DEPENDENCY]Time:  avg  0ns,  max  0ns,  min  0ns
                          EXCHANGE_OPERATOR  (id=1):
                                -  BlocksProduced:  sum  2.499K  (2499),  avg  2.499K  (2499),  max  2.499K  (2499),  min  2.499K  (2499)
                                -  CloseTime:  avg  7.900us,  max  7.900us,  min  7.900us
                                -  ExecTime:  avg  3.7ms,  max  3.7ms,  min  3.7ms
                                -  InitTime:  avg  7.253us,  max  7.253us,  min  7.253us
                                -  MemoryUsage:  sum  ,  avg  ,  max  ,  min
                                    -  PeakMemoryUsage:  sum  16.04  MB,  avg  16.04  MB,  max  16.04  MB,  min  16.04  MB
                                -  OpenTime:  avg  675ns,  max  675ns,  min  675ns
                                -  ProjectionTime:  avg  0ns,  max  0ns,  min  0ns
                                -  RowsProduced:  sum  10.0M  (10000000),  avg  10.0M  (10000000),  max  10.0M  (10000000),  min  10.0M  (10000000)
                                -  WaitForDependencyTime:  avg  0ns,  max  0ns,  min  0ns
                                    -  WaitForData0:  avg  5s838ms,  max  5s838ms,  min  5s838ms

Therefore, the approach has been changed to merge each input block into the build block immediately.
In spills, it is necessary to avoid merging blocks multiple times.

run buildall


Feature/wgpu

Description
Adds the WebGPU API into the framework.

WebGPU window.
WebGPU Renderer:

Uses instanced drawing (the OpenGL renderer will use this design as well soon),
Support for multiple render passes.

WebGPU rendering system + component:

Rendering and Material component will be merged with the OpenGL ones soon.

Related issues

Closes #27
Closes #18

WGPU examples work on my end 🎉

giving my review comments, even though the PR was closed:
OpenGL
example00: great!
example01:  while rotating faces don't always show
example02: great!
example03 / example04 / example05:
- great!
- scaling down camera too much: gives nan and does not recover values
- adding a light component on monkeh: changes the light parameters of the pistol. Probably due to all entities using the same shaders.
WebGPU
example00: great!
example01: pistol and monkeh all good, but with keys 1 and 2, e1 and e2 should be selected, but no entities with this name
example02: after multiple times of pressing space to change the scene the program crashes with error: Vertex buffer index 16 is greater than the device's requested max_vertex_buffers limit 16

giving my review comments, even though the PR was closed:
OpenGL example00: great! example01: while rotating faces don't always show example02: great! example03 / example04 / example05: - great! - scaling down camera too much: gives nan and does not recover values - adding a light component on monkeh: changes the light parameters of the pistol. Probably due to all entities using the same shaders.
WebGPU example00: great! example01: pistol and monkeh all good, but with keys 1 and 2, e1 and e2 should be selected, but no entities with this name example02: after multiple times of pressing space to change the scene the program crashes with error: Vertex buffer index 16 is greater than the device's requested max_vertex_buffers limit 16

Added an issue for this (https://github.com/papagiannakis/pyGandalf/issues/37), Thanks!


Fix chained proxy disconnect hang

I noticed a few of the unit tests were taking over 30s to complete. I discovered that chained proxy connection failures were not properly transitioning the server's connection state to DISCONNECTED, so any subsequent writes from the client would hang for 30s waiting for the server to connect.
This PR also fixes a related issue where encrypted upstream proxy connection failures did not invoke the proxyToServerConnectionFailed() filter method.
I also added 500ms sleeps in the HttpFilterTests to allow the filters to fire after requests are sent.
As a result of these fixes, unit tests finish about 1m faster and are more reliable.

No veto. I've seen such effects too. It's great if it's fixed now. I'm not fully understand the connection flow, but it's tested well. I've cherry-picked it to my fork and tested with mitm and my proxy.

Sounds good. I'll leave it open for a few more days, in case anybody else wants to review/comment.
ConnectionFlow is definitely a beast. I'm looking forward to improving it, perhaps when we start to add HTTP/2 support.


some about js

The request is rejected because it's content might offend other contributors.
Still thank You for contributing attempt )

The request is rejected because it's content might offend other contributors.
Still thank You for contributing attempt )


Update gateway docs to account for environments with loadbalancers

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
Assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

@cmluciano can you discuss with @frankbu on the best way to reflect instructions for node port on ICCS?

@cmluciano: PR needs rebase.

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: cmluciano
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
Assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

@cmluciano could you fix the circleci errors?

/lgtm

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

Needs approval from an approver in each of these files:

OWNERS [linsun]

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

Closing in favor of https://github.com/istio/istio.github.io/pull/1341


cherry-pick change char function behavior same with mysql (#30034)

cherry-pick from master https://github.com/apache/doris/pull/30034
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


stdin error in docker

I'm having issues when trying to run this in docker (kubernetes to be more specific but this should not be different) which does not use tty by default
maybe you should check for tty
with something like this
if (process.stdin.isTTY){
        process.stdin.setRawMode(true);
      }

[h265ize]: [verbose] Handling keypresses...
[h265ize]:
        - message: process.stdin.setRawMode is not a function
        - stack: TypeError: process.stdin.setRawMode is not a function
    at /usr/local/lib/node_modules/h265ize/h265ize:485:23
    at h265ize.handleKeypress (/usr/local/lib/node_modules/h265ize/h265ize:453:12)
    at runCallback (timers.js:574:20)
    at tryOnImmediate (timers.js:554:5)
    at processImmediate [as _immediateCallback] (timers.js:533:5)
From previous event:
    at Object.h265ize.runCli (/usr/local/lib/node_modules/h265ize/h265ize:47:10)
    at Object.<anonymous> (/usr/local/lib/node_modules/h265ize/h265ize:1701:13)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.runMain (module.js:590:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:149:9)
    at bootstrap_node.js:509:3
        - __stackCleaned__: true
[h265ize]: [verbose] Cleaning up temp files...
[h265ize]: Process exited.

After adding the condition for isTTY does it run perfectly?

no
I believe all the keypress event won't work
I used a workaround with this command
script --return -c "[executable string]" /dev/null
this seem to work


Evaluate default viewport for mobile

Feedback from Tony:

I think the default viewport could be zoomed out

I'm doing a big revamp of the site to be mobile-first, so this won't be relevant.


SPI: one of the half-duplex modes is not supported

I can't actually tell whether the half-duplex mode we support is 3 or 4-wire single-bit. Our data modes are defined as:
#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
#[cfg_attr(feature = "defmt", derive(defmt::Format))]
pub enum SpiDataMode {
    /// `Single` Data Mode - 1 bit, 2 wires.
    Single,
    /// `Dual` Data Mode - 2 bit, 2 wires
    Dual,
    /// `Quad` Data Mode - 4 bit, 4 wires
    Quad,
    #[cfg(spi_octal)]
    /// `Octal` Data Mode - 8 bit, 8 wires
    Octal,
}

This suggests 4-wire half-duplex mode (MOSI, MISO both transfer data). This is essentially full-duplex with state machine control.
The ESP32 TRM explicitly calls out a different single-bit half-duplex mode:

The three-line half-duplex communication differs from four-line half-duplex communication in that the reception and transmission shares one signal bus

The other TRMs call out three-line half-duplex mode in a table, e.g. esp32-c6 v1.0 table 28-4.

This is actually more general than the hardware supports (i.e. it's possible to generate invalid configuration). The set of options is roughly:
enum TransferMode {
    ThreeWireHalfDuplexSpi,
    FourWireHalfDuplexSpi,
    FullDuplexSpi,
    Qpi, // 4 bit command, address, data
    Opi, // 8 bit command, address, data
    WideSpiNarrowAddress(2/4/8 data bits), // 1 bit command, 1 bit address
    WideSpiWideAddress(2/4/8 data bits),   // 1 bit command, N bit address
}

That being said, the S2 TRM explicitly says as an example, that this is supported:

😩


Meli's attempt at completing the Beginning Git video course

Hi Sam,
This PR represents all the work I did to complete the raywenderlich.com Beginning Git video course. It was an awesome experience. Thank you :)
Melina

Hey @sicardimelina—congrats on submitting a PR at the end of the git course. I hope you enjoyed the course, and that you learnt a lot.
If you weren't already using git for your code, hopefully this has inspired you to do so. I hope that you enjoy the "Mastering Git" video course as well—you'll soon be able to get git to bend to your will :)
You can now wear this #GitWarrior badge with pride—and Tweet your historic victory!

sam


player_justification.py

Guys, checkout the GUiI and let me know if there are any changes to be done

New request being raised


Using estimators created by tf.keras.estimator.model_to_estimator in tf.estimator.train_and_evaluate causes a memory leak of sorts

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes, see provided gist
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Debian Jessie
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): ('v1.8.0-0-g93bc2e2072', '1.8.0')
Python version: 2.7.9
Bazel version (if compiling from source): n/a
GCC/Compiler version (if compiling from source): n/a
CUDA/cuDNN version: n/a
GPU model and memory: n/a
Exact command to reproduce: Run provided gist

Describe the problem
When using keras models with train_and_evaluate, you need to convert them into estimators first. The best way to do this is tf.keras.estimator.model_to_estimator. However, when train_and_evaluate is called with those estimators, the default graphs that are created end up referenced in _GRAPH_LEARNING_PHASES and never deallocated (so sort of a memory leak). This is due to a call to keras.backend.set_learning_phase that's done as part of the created model_fn, which uses the default graph as a key into _GRAPH_LEARNING_PHASES. Since that graph changes every so often in train_and_evaluate, that key changes so there's a new key inserted without the old ones being removed.
Or at leas, I'm 80% certain that's what's going on. You can see in the provided code a very simple Keras model turned into an estimator and then run with a hook that prints out memory usage and the size of _GRAPH_LEARNING_PHASES. It increases over time!
While the leak is small in the provided example, it has caused me great strife in real usage.
Source code / logs
The source code and a sample run are provided in this gist. It should be easy to run the python script yourself however you'll need to have numpy and psutil installed.

I found that adding :
class MemoryLogHook(tf.train.SessionRunHook):
    def begin(self):
        K._GRAPH_LEARNING_PHASES = {}
        K._GRAPH_UID_DICTS = {}

fixes the issue. (will stabilize around 1.35e+9) We probably need to clear those before calling the model_fn.

Hey thanks! This workaround works for me, although I imagine it's probably not the best solution :)

Also I think that this workaround will not work with mode_to_estimator cause in that case the model_fn it is not explicitly built.

ping @fchollet

Nagging Assignee @rohan100jain: It has been 19 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 34 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 49 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 64 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 89 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Any updates on this?

@zmjjmz We are checking to see if you still need help on this issue, as you are using an older version of tensorflow(1.x) which is officially considered as end of life. We recommend that you upgrade to 2.6 which is latest stable version of TF and let us know if the issue still persists in newer versions. Thanks!


Update stackrox/stackrox dependency

Our dependency was a bit dated, which, from a functionality standpoint, is ok, but this will also resolve vulnerabilities which have since been patched in dependencies.

Verified if we were to update the version of scanner in stackrox's go.mod there will be no dependency issues.

@misberner I added exclude github.com/stackrox/scanner v0.0.0-20220426001230-9ab6777c9581 to my go.mod just now, and I saw no change to the go.sum. I also searched the go.sum file for any reference to scanner, but I did not find anything, so it looks like adding the exclude doesn't do much


Header with user logged in takes up too much vertical space

When logged in, the header for the site looks like this:

It should be all on the same line.

I'll take a look at this

Looks a bit better.