Update to golang 1.22

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

Project coverage is 0.76%. Comparing base (b4eafd2) to head (6ec5a31).

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #398      +/-   ##
========================================
- Coverage   0.95%   0.76%   -0.20%
========================================
  Files          4       5       +1
  Lines        209     261      +52
========================================
  Hits           2       2
- Misses       207     259      +52

Flag
Coverage Δ

api
0.76% <ø> (-0.42%)
:arrow_down:

controller
0.00% <ø> (ø)

schema
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 files with indirect coverage changes


Change server_status nas to update statusdb by default

Codecov Report
Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@fa12734). Learn more about missing BASE report.

Files with missing lines
Patch %
Lines

taca/server_status/cli.py
0.00%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #453   +/-   ##
=========================================
  Coverage          ?   28.03%
=========================================
  Files             ?       37
  Lines             ?     5450
  Branches          ?        0
=========================================
  Hits              ?     1528
  Misses            ?     3922
  Partials          ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Need an X button to help user go back.

For Uniformity of all the modals across the app, Instead of cross, cancel Secondary Button Added
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=524-1605&t=jjdLVE2dv9npS8sA-1

#5037

Completed


scaleLine.domian([0, 0]).range([100, 0])

if i input the same value in domain, this scale will return average of the range.
const scale = scaleLine.domian([0, 0]).range([100, 0])
scale(0) // 50
it will return 50,  but I want return 100
What should i do?  thank you.

I suppose there's a back-story to your question that you might want to give explicitly.


feat(esbuild): minify esbuild bundle by default

esbuild bundle should be minified by default. If this behavior is not desired you may use the environment variable NO_MINIFY.
Closes #210

Thank you, I needed this! LGTM!


Android: VTM 0.8.0 OpenGL vector map library

VTM 0.8.0 OpenGL vector map library released!

Cool - thanks and congrats for the release :+1: !


Add imprint text

For rolling out the app in the Google PlayStore, we'd, need an imprint text.

[x] German
[x] English

Looking great!
I took the liberty to wrap the imprint in an ion-grid similar to e.g. the privacy policy – otherwise the content might reach into the safe-area on iOS-devices (= covered by the notch in landscape-orientation).
Another thought: Could the imprint (e.g. above the content) be an appropriate space to include the Simport and BMBF logos? Should they even be included in the app?
→ https://raw.githubusercontent.com/sitcomlab/simport-learning-app/develop/documents/simport_bmbf_logo.png


Conflict between area storage key and Areas window settings key

The Areas tab in the Layers window (os.ui.AreasCtrl) saves its settings under areas.*, because os.ui.slick.AbstractGroupByTreeSearchCtrl#save uses the title property as the base key. This conflicts with os.AREA_STORAGE_KEY, which also defaults to areas. This conflict causes an error when loading saved areas from storage, if the Areas tab has written anything to settings (due to invalid GeoJSON in that key). Resolve this conflict, preferably without losing existing areas in storage.

I haven't been able to reproduce this error locally, but the best solution is likely to change the title in AreasCtrl. Those settings (group by and tree expand/collapse state) would be minimally invasive if lost.


Verify daemon ID on target side before PUT

This errors out object PUT requests to a target which do not provide a valid daemon ID (one that belongs to the cluster) as a query string parameter. Also add a test.

In the case of create local bucket, the target-side handler doesn't accept that action message (it defaults to the invalmsghdlr case. For situations similar to target PUT, we'd probably want to do something similar to that which is done here.


Regresssion: Cannot use images with Avatar

Description
Avatar no longer supports required images
CodeSandbox/Snack link
https://snack.expo.dev/yZSTh2TeX
Steps to reproduce

See snack above the first Avatar doesn't have an image

NativeBase Version
3.4.3
Platform

[X] Android
[ ] CRA
[ ] Expo
[X] iOS
[ ] Next

Other Platform
No response
Additional Information
This was originally fixed in https://github.com/GeekyAnts/NativeBase/commit/bbc2ce4dc08bc92c2c2f908f8a684374e642b4ca
but then looks to have been accidentally reverted in
https://github.com/GeekyAnts/NativeBase/commit/eb99803231c2935f6966cbb98d1c580471dec728#diff-94556544ee52296429d62e0c292822e0aaee76abbf7a08f95e025c59eecc624cL34-R35

Hi @johnf, Thanks for reporting the issue. We'll fix it.

Hi @johnf, This issue has been fixed in v3.4.6


Fixing the ability to provide custom retryMap

https://github.com/google/google-api-php-client/issues/1487

I signed it!

Can someone merge this?

@mattwhisenhunt is this okay to merge?

@ggmartins091 it looks correct to me. $this->config['retry'] is intended to be the configuration array for the Runner, as seen here, not a specific number of retry attempts.

@dwsupplee Oh, my bad. I get it now.
So for me to change the number of retries I would need to do new Google_Client(['retry' => ['retries' => 3]]);
Thats what I was missing. Thank you.


undefined method `get' for #<RSpec::ExampleGroups::

While testing my  controllers I came across this issue. I am using rspec-rails-3.4.2, rails-4.2.0 and mongodb. Here is my rails_helper.rb
`ENV['RAILS_ENV'] ||= 'test'

require File.expand_path('../../config/environment', __FILE__)

abort("The Rails environment is running in production mode!") if Rails.env.production?

require 'spec_helper'
require 'rspec/core'
require 'rspec/rails/extensions'
require 'rspec/rails/view_rendering'
require 'rspec/rails/adapters'
require 'rspec/rails/matchers'
require 'rspec/rails/fixture_support'
require 'rspec/rails/example'
require 'rspec/rails/vendor/capybara'
require 'mongoid-rspec'

RSpec.configure do |config|
  config.before(:suite) do
    DatabaseCleaner.orm = "mongoid"
    DatabaseCleaner.strategy = :truncation
  end
  config.before(:each) do
    DatabaseCleaner.start
  end
  config.after(:each) do
    DatabaseCleaner.clean
  end
end

Please just require rspec-rails as require 'rspec/rails' it's likely thats the cause of your issues

@JonRowe  Thanks for the immediate reply. I tried that too. when I require respec-rails it causes another error
Failure/Error: raise ConnectionNotEstablished, "No connection pool for #{klass}" unless pool

Yep, this is a misconfiguration issue on your end, you need to ensure mongoid is loaded before rspec-rails so it doesn't try to use ActiveRecord.

Make sure you can do RAILS_ENV=test rails console without error before trying RSpec.

For us to help any further you'll have to provide backtraces (via https://gist.github.com)


upgrade jena version for test

see http://archive.apache.org/dist/jena/binaries/
[apache-jena-fuseki-4.9.0.tar.gz](http://archive.apache.org/dist/jena/binaries/apache-jena-fuseki-4.9.0.tar.gz)                       2023-07-04 17:46   36M
[TXT] [apache-jena-fuseki-4.9.0.tar.gz.asc](http://archive.apache.org/dist/jena/binaries/apache-jena-fuseki-4.9.0.tar.gz.asc)                   2023-07-04 17:46  833
[TXT] [apache-jena-fuseki-4.9.0.tar.gz.sha512](http://archive.apache.org/dist/jena/binaries/apache-jena-fuseki-4.9.0.tar.gz.sha512)                2023-07-04 17:46  162
[   ] [apache-jena-fuseki-4.9.0.zip](http://archive.apache.org/dist/jena/binaries/apache-jena-fuseki-4.9.0.zip)                          2023-07-04 17:46   36M
[TXT] [apache-jena-fuseki-4.9.0.zip.asc](http://archive.apache.org/dist/jena/binaries/apache-jena-fuseki-4.9.0.zip.asc)                      2023-07-04 17:46  833
[TXT] [apache-jena-fuseki-4.9.0.zip.sha512](http://archive.apache.org/dist/jena/binaries/apache-jena-fuseki-4.9.0.zip.sha512)                   2023-07-04 17:46  159

see https://stackoverflow.com/questions/77898345/insert-command-returning-a-415-response-after-upgrading-from-jena-3-17-0-to-4-9


Upgrade components

Reference Issues or PRs
Following have been upgraded now:

JupyterHub to 4.x
JupyterLab to 4.x
conda-store to 2024.1.1

Moved to postgresql from mysql
didn't bothered adding redis yet, doesn't seems to be required for now

batchspawner 1.2 → latest main from GitHub

Note: keycloak is already on a later version (16.1.0) than Nebari's keycloak (15.0.2), both of them should be upgraded soon, but not the priority right now.
I have NOT added jhub-apps yet as that would require more work and time to figure out:

support for batchspawner in jhub-apps
using conda-store API to fetch environments, like we do in Nebari

TODO for later:

Need to make sure condarc is automatically created so that people can select their created environment in the jupyterlab via nb_conda_kernels.

JupyterLab

JupyterHub

Conda Store

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

I supose the theming and extra customizations were now moved to its own yml files correct?


Data disk storage account type change

Looks like VMSS "data" disk storage account type can't be changed after VMSS creation,  just like OS disk type. If this is the case, can we please edit the document and add the restriction for "data" disk as well.
Properties with restrictions on modification
Managed OS disk storage account type ==> Probably add OS and Data rather than just OS Disk

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

ID: 706ceb71-6add-c678-bdb5-ee0d9386488b
Version Independent ID: bf4ab279-85fa-6a7a-8756-24af74fe1444
Content: Modify an Azure virtual machine scale set - Azure Virtual Machine Scale Sets
Content Source: articles/virtual-machine-scale-sets/virtual-machine-scale-sets-upgrade-scale-set.md
Service: virtual-machine-scale-sets
GitHub Login: @mayanknayar
Microsoft Alias: manayar

@RishiMaini Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Sorry for the delay on this. Updated the doc
https://github.com/MicrosoftDocs/azure-docs-pr/pull/107114
Once the PR merges the changes will go live after a few hours.


IRGen: Fix a map entry use-after-free issue in LargeLoadableHeuristic

rdar://139398409
Should fix the failure in  https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-macos/7219
=================================================================
==36519==ERROR: AddressSanitizer: heap-use-after-free on address 0x61b000759fd9 at pc 0x00010d30c053 bp 0x7ff7b46c3670 sp 0x7ff7b46c3668
WRITE of size 1 at 0x61b000759fd9 thread T0
    #0 0x10d30c052 in runPeepholesAndReg2Mem
    ```

@swift-ci smoke test

https://ci.swift.org/job/oss-swift-incremental-ASAN-RA-macos/7223/ has the fix and hopefully is green again


Array<"foo"> inferred to ReadonlyArray gives T=string, not T="foo"

TypeScript Version:  2.3.4
Code
declare function id<T extends string>(list: Array<T>): Array<T>;
declare function readonlyId<T extends string>(list: ReadonlyArray<T>): ReadonlyArray<T>;

type LiteralType = "foo" | "bar";

const array: Array<LiteralType> = [];
const readonlyArray: ReadonlyArray<LiteralType> = [];

const foo: ReadonlyArray<LiteralType> = id(array);
const bar: ReadonlyArray<LiteralType> = readonlyId(array);
//    ~~~
// Type 'ReadonlyArray<string>' is not assignable to type 'ReadonlyArray<LiteralType>'.
//  Type 'string' is not assignable to type 'LiteralType'.const bar: ReadonlyArray<LiteralType>
const baz: ReadonlyArray<LiteralType> = readonlyId(readonlyArray);

Expected behavior:
readonlyId(array) returns ReadonlyArray<LiteralType>
Actual behavior:
readonlyId(array) returns ReadonlyArray<string>

Definitely seems questionable.

Just ran into this with string enums.
function first<T>(arr: ReadonlyArray<T>): T { return arr[0]; }
const enum E { A = "A" }
const es: E[] = [E.A];
const e: E = first(es); // Error, string not assignable to E

This is a consequence of the weird type that concat has in Javascript, and how Typescript tries to infer type parameters for it. The type constraint T extends string obscures the problem because of an obscure fallback in the case that the inferred type isn't assignable to the type constraint.
Here's a smaller repro that shows the exact problem without the type constraint:
type A<T> = { concat(items: T | A<T>): void; }
type B<T> = { concat(items: T | B<T>): void; }
declare function deat<T>(at: A<T>): T;
type LiteralType = "foo" | "bar";
declare var blt: B<LiteralType>;

let foo: LiteralType = deat(blt);
//  ~~~
//  Type '"foo" | "bar" | B<LiteralType>' is not assignable to type 'LiteralType'.

Previously, since "foo" | "bar" | B<LiteralType> isn't assignable to string, type inference just used string instead since T extends string. Now we can see that inference got T="foo" | "bar" | B<LiteralType>.
Inference proceeds like this:

Look for candidates B<LiteralType> ==> A<T>. You might think this would trivially find T=LiteralType, but instead the inference proceeds structurally.
Look for inferences in the methods concat(items: LiteralType | B<LiteralType>): void ==> concat(items: T | A<T>)
Look for inferences in the parameters LiteralType | B<LiteralType> ==> T | A<T>
Look for inferences in the first half of the union: LiteralType | B<LiteralType> ==> T. This finds T = LiteralType | B<LiteralType>. :cry:
Look for inferences in the second half of the union: LiteralType | B<LiteralType> ==> A<T>. But no inferences result from this.

Done. We got the incorrect inference "foo" | "bar" | B<"foo" | "bar"> because of the way unions are inferred.
I'll think about whether pairwise comparison would be possible, since it's what most people expect to happen, but I'm not confident of success.

After poking around about, @ahejlsberg and I found that my repro oversimplifies the problem. Because the real concat in both Array and ReadonlyArray take arrays, step 5 is able to find some inferences (also, step 5 happens before step 4, but this makes no difference). Problem is, they're bad ones.
The structural comparison of "foo" to ReadonlyArray finds a lot of methods that match, but instead of having parameters of type T, they have parameters of type string. Since string is a superclass of both "foo" and "bar", the resulting inference is T=string.
We think that structural inference can be improved by skipping inference for any source type that doesn't have all the members of the target type. This source wouldn't be assignable anyway, so it's probably a low-quality source of inferences. This would rule out String as an inference source for ReadonlyArray specifically.
I'll try it out tomorrow and see if it breaks anything else.
Note that this would fix the repros posted originally but not the small one I just posted.

Is this not at least in part related to problems you found with Promise as well?

@DanielRosenwasser which problems? The ones in definitely typed with T | Promise<T>?
Yes, specifically my repro highlights this problem. But the fix that @ahejlsberg and I came up with only fixes the original repro since it fails because of structural inference of String ==> ReadonlyArray<T>, not because of inference of "foo" | "bar" | B<"foo" | "bar"> ==> T.
It might fix the promise problems too, but I'm going to run the fix on RWC/DT to find out.

This fix is now in master. I merged manually after we reviewed the change offline.


python array for GPDB 4.2 and Postgres 9.1

make sure arrays are passed into python as strings.

That should work too.


Figwheel Client and Server have different versions!!

Error message:
Figwheel Client Version 0.5.9 is not equal to Figwheel Sidecar Version 0.5.13. Shutting down Websocket Connection!
To fix try:
Reload this page and make sure you are not getting a cached version of the client.
You may have to clean (delete compiled assets) and rebuild to make sure that the new client code is being used.
Also, make sure you have consistent Figwheel dependencies.

@gadfly361
Wow, it worked! Didn't know about hard refresh (Ctrl + F5).
Thank you so much.


Int boxing bug

I started working on official Scala.JS support for Scalaz.
One of the tests MapTest fails thus:
[info] ! .==>> map:mapKeys sound: Exception raised on property evaluation.
[info] > ARG_0: Bin(1740834887,1268128331,Bin(-1421249690,1876982179,Tip,Tip),Bin(2147483647,-2147483648,Tip,Tip))
[info] > Exception: scala.scalajs.runtime.UndefinedBehaviorError: An undefined behavior was detected: 0 is not an instance of java.lang.Integer

It seems that the insert method of Map (aka ==>>) gets an Order[java.lang.Integer] and tries to feed it an Int. This only happens under Scala.JS so I think it's a Scala.JS bug.
You can reproduce by checking out my fork: https://github.com/japgolly/scalaz/tree/scalajs
Ensure you're on the scalajs branch, then run sbt 'testsJS/test-only scalaz.MapTest'.
(If it passes, run it again; the default settings only perform 33 tests per property.)

(Also the tests don't run in parallel like my other projects with μTest. I assumed it was the Scala.JS SBT plugin that forked multiple instances of node/phantomjs. Sequentially is super slow - where do I look next to fix this? Settings? ScalaCheck? SJS?)

Can you please point out some lines number to look at?
An Ordering is created only when a Sorted data structure is used and it is not a general case...
Probably if you can identify the exact data structure it will be really helpful!

Sure. I already pointed out the data structure, test and method though, is that not what you mean? What I say the insert method of Map and MapTest, those are files in the Scalaz repo, I don't mean anything about the Map  in stdlib. Maybe I wasn't clear about that.
Let me provide some links. The failing test is:
https://github.com/japgolly/scalaz/blob/scalajs/tests/shared/src/test/scala/scalaz/MapTest.scala#L591-L599
The method that crashes:
https://github.com/japgolly/scalaz/blob/scalajs/core/shared/src/main/scala/scalaz/Map.scala#L32-L46
Specifically, I think this is the exact line:
https://github.com/japgolly/scalaz/blob/scalajs/core/shared/src/main/scala/scalaz/Map.scala#L38

I will have a look at this next week. This week I'm completely overwhelmed with academic stuff.

@sjrd No worries. Thanks for letting me know :)

I think this is a duplicate of #1984, which is fixed in master. Guessing that from the % in the test.

I cannot reproduce the bug on your branch. I have tried running it ~15 times and it didn't fail.
If you can still reproduce it with 0.6.5, could you try with a locally-published 0.6.6-SNAPSHOT?

That's weird, I ran it another 10 times just now and it only passed once.
I also tried it with a local 0.6.6-SNAPSHOT and it continually passes now. It must've been the % bug like you suggested.

OK, thanks for trying. I'll close this as a duplicate of #1984, then.


Fix lint on serverless-helpers

Le linter est cassé sur main pour le package serverless-helpers:

Thanks for reporting this? I can't reproduce the error on main though 🤔
Are you on node16? We have made the switch a few days ago

Seems good with node 16


libtensorflow: 1.9.0 -> 1.13.1

Motivation for this change
Update libtensorflow to latest stable version 1.13.1.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Ping @basvandijk, please review

I will still need to test if all dependencies are in place when built with CUDA support (later today). So far, I noticed that when built with GPU support, ldd shows that some of the dependencies cannot be found in cuda* space. Not sure if that is how it was before.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/63208


The use of .scss format

Han.css is developed using .sass format in most cases for its conciseness. From v3.1.0, where we started using Gulp for development, the main files (src/sass/han.scss and index.scss) switched to .scss format since the module gulp-sass fails to compile .sass files correctly. Will switch back once the bug is fixed.

Doesn't writing Han.css in SCSS better than writing it in SASS? Because SCSS is closer to CSS, doesn't this make new contributors unfamiliar with SASS or SCSS easier to contribute?

My opinion is that there are hundreds of languages using indentation instead of curly brackets to delimit ranges. I don’t see why we shouldn’t use the Sass format. It is cleaner to look at and easier to write. That’s just a matter of preference. :smile:
FYI, the SCSS format is used in some cases in the project, such as listing font-family. https://github.com/ethantw/Han/blob/master/src/sass/typography/ff/_biaodian.scss


Doubt in MicroMasters program

Hi sir,
I saw your post on LinkedIn saying adding MOOC Certificates to your resume these days don't have much value. While I was searching courses in edx I noticed several reputed universities offering Micromasters degree online. What do you think about an undergrad student taking these courses? Whether taking this course online is valuable and offers a student a good career and job.
I tried to reach you on Instagram about this issue because I am planning to do Micromasters in statistics and data science offered by the Massachusetts Institute of Technology. waiting for your opinion.

In the post previous to that I have said that unless you showcase your work, course is not going to help.


Run a check_submodule_init script during CMake configure

Checking if git submodules have been initialized and printing warning messages.
Added a python script in scripts/git/check_submodule_init.py which will run inside the CMake: CMakeLists.txt

The script won't check for the status of the submodules if not a git repository
This will only run if python/python3 is available

Fixes #6434

@googlebot I signed it!

I tested this out and pushed some changes to your branch.

execute_process(
  COMMAND !(python --version) || python scripts/git/check_submodule_init.py

didn't work how I wanted, so I replaced it with
find_package(Python3 COMPONENTS Interpreter QUIET)
if(Python3_FOUND)
  execute_process(
    COMMAND python scripts/git/check_submodule_init.py

The git status check and inspecting the output is a bit of a roundabout way to check if in a git repo, and sys.exit(1) is not a graceful way to exit (the 1 is an error). I replaced that block of code with
  # No-op if we're not in a git repository.
  try:
    subprocess.check_call(['git', 'rev-parse', '--is-inside-work-tree'],
                          stdout=subprocess.DEVNULL,
                          stderr=subprocess.DEVNULL)
  except:
    return

Thanks @ScottTodd , I was also thinking of using subprocess, but then left as we need to another module :)
Btw some checks are failing, any reason?

Thanks @ScottTodd , I was also thinking of using subprocess, but then left as we need to another module :)
Btw some checks are failing, any reason?

Yeah, I'm poking the bots until they are happy :P
logs from https://source.cloud.google.com/results/invocations/22410ba2-350d-44a3-b83f-59ad0ad061c3/targets/iree%2Fgcp_ubuntu%2Fcmake%2Flinux%2Fx86-swiftshader%2Fpresubmit/log:
-- Found Python3: /usr/bin/python3.6 (found version "3.6.9") found components:  Interpreter
  File "scripts/git/check_submodule_init.py", line 27
    f"The git submodule '{submodule.split()[1]}' is not initialized. Please run `git submodule update --init`"
                                                                                ^
SyntaxError: invalid syntax
CMake Error at CMakeLists.txt:337 (message):
  check_submodule_init.py failed, see the logs above

That was using python (which could be python2) instead of python3 to run the script. Might be fixed now 🤞

found version "3.6.9"

Oh, sorry for my f-string. But the log says found version "3.6.9"

found version "3.6.9"

Oh, sorry for my f-string. But the log says found version "3.6.9"

Yeah, that's from the CMake find_package, but isn't necessarily what is used when you run
  execute_process(
    COMMAND python

We could actually use ${Python3_EXECUTABLE} to make it match. Pushed another commit 😅

found version "3.6.9"

Oh, sorry for my f-string. But the log says found version "3.6.9"

Yeah, that's from the CMake find_package, but isn't necessarily what is used when you run
  execute_process(
    COMMAND python

We could actually use ${Python3_EXECUTABLE} to make it match. Pushed another commit 😅

Hopefully, it will work now 😅


Updated GenBank parser based on latest scikit-bio dev

Thanks to @RNAer 's effort, now we can take advantage of the latest scikit-bio from his repo. I updated the code accordingly.

@ekopylova @RNAer I tried to update the GenBank parser, but... maybe we will have to get #43 merged first before this PR.

I updated all GenBank format parsing codes based on @RNAer 's latest scikit-bio, which can be installed using the command: pip install git+https://github.com/rnaer/scikit-bio/@mixin.

Coverage decreased (-0.2%) to 92.114% when pulling cade4d074d66f3255f8c4af42ba9a0c2570d89e7 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage decreased (-0.2%) to 92.114% when pulling cade4d074d66f3255f8c4af42ba9a0c2570d89e7 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage decreased (-0.08%) to 92.282% when pulling bf924d3ed7d0078bf2bdbfab117eb3c0ed0f7c6c on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage decreased (-0.08%) to 92.282% when pulling bf924d3ed7d0078bf2bdbfab117eb3c0ed0f7c6c on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage increased (+0.09%) to 92.45% when pulling 309fbd73ed8a923c5e29d08d5bfd15adc514d269 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage decreased (-0.08%) to 92.282% when pulling 309fbd73ed8a923c5e29d08d5bfd15adc514d269 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage decreased (-0.08%) to 92.282% when pulling 3c16f6e44d6a260aced313e636cfa84e28304b55 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage increased (+0.09%) to 92.45% when pulling 3c16f6e44d6a260aced313e636cfa84e28304b55 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage decreased (-0.08%) to 92.282% when pulling d4588a4ae521f4a2ef240ed4ac24e1a7ac7ad0ea on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.

Coverage increased (+0.3%) to 92.63% when pulling 207ba0b2348bae91c73b74bc7d9a894311866036 on qiyunzhu:gm2 into e38826e4229c2aa51d034fec5fe6edbc5619a62c on biocore:master.


MySQL server has gone

{
    "output": "Connecting to DSN 'mysql:host=***.vpn2keboola.com;port=33028;dbname=***;charset=utf8'
Exporting to in.c-ex-mysql-***
Exporting to in.c-ex-mysql-***
Exporting to in.c-ex-mysql-***
Empty row packet body. Retrying... [1x]
Exporting to in.c-ex-mysql-***
Connecting to DSN 'mysql:host=***.vpn2keboola.com;port=33028;dbname=***;charset=utf8'
PDOStatement::execute(): MySQL server has gone away. Retrying... [1x]",
    "errorOutput": "PDO::query(): MySQL server has gone away",
    "container": {
        "id": "398259974-398259951.398259975--0-keboola-ex-db-mysql",
        "image": "147946154733.dkr.ecr.us-east-1.amazonaws.com\/developer-portal-v2\/keboola.ex-db-mysql:3.1.0"
    }
}

https://connection.keboola.com/admin/utils/logs?file=2018/05/28/01/2018-05-28-01-17-10-5b0b3c7688506-exception
https://connection.keboola.com/admin/utils/logs?file=2018/05/28/01/2018-05-28-01-17-10-5b0b3c768c68e-log

Mal by to byt user error zrejme.

nj, myslim ze to duplikat tohle: https://github.com/keboola/db-extractor-mysql/issues/46, ale stejne je to blby


DNS search domain

Running on docker ubuntu. everything works fine, but I wanted to migrate from old windows  emby. But now I need to enter ipaddress? andyway to get it to read hostname with my serchdomain from host?
Eg_ mediaserver instead of mediaserver.domain.com

Hi, what do you mean by "now I need to enter ipaddress"? Where exactly are you referring to?

Sorry i found it out. I was thinking uncle path.. hehe Microsoft head doing linux config.. I mapped the folder on the host. Now it works.
Sent from Ninehttp://www.9folders.com/

Fra: Luke<EMAIL_ADDRESS>Sendt: søndag 10. mars 2019 22:30
Til: MediaBrowser/Emby.Build
Kopi: Stian Hill; Author
Emne: Re: [MediaBrowser/Emby.Build] DNS search domain (#80)
Hi, what do you mean by "now I need to enter ipaddress"? Where exactly are you referring to?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MediaBrowser/Emby.Build/issues/80#issuecomment-471355776, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AnVWFAyb3jcrAn3c8z_urQd71Qd2ne9Sks5vVXlngaJpZM4bngyP.


Embedly support

https://embed.ly/providers/new
Would make videos work on Reddit, among other places. Context:
https://www.reddit.com/r/lbry/comments/gdo200/will_lbrytv_be_added_as_an_embedly_provider_to/

Please get with Josh to setup the developer account when this work begins.


Old data printed

I'm used sunmiInnerPrinter.printBitmap to print my image.
But it printed old data instead of new data.
I tried printing text and images but the results were similar.
For example:

1st cmd: printOrginalText('first') -> printer not work
2nd cmd: printOrginalText('second') -> print 'first'
3rd cmd: printOrginalText('third') -> print 'second'

@phidoi247 Are you still having this problem?

@phidoi247 Are you still having this problem?

Yes, I am still having this problem
I am using Sunmi T2

@phidoi247 Sorry. This repository is applicable only for SUNMI V1 or V2
You can refer to my previous post here: https://github.com/labibramadhan/cordova-sunmi-inner-printer/issues/14


The software assumes that the robot begins on the Fixed Frame

I have had this problem:

As you can see the self collisions shape are spawn on 0,0,0 of the common_world frame. Unfortunately, I have not been able to find the part of the code that creates the shapes for the self-collisions.
In my robot configuration, all the collision surfaces are expressed in the world frame which is my fixed frame. I am not able to configure the software to accommodate my situation.
I think this can be fixed by adding another parameter to the config file start_here.py

I tried to include the world joint in the robot chain. It adapts the cylinders to the robot but the rest is retransformed weirdly.

I have solved the problem for myself
ptA = np.matmul(orient[0:3,0:3], ptA) + base_frame
ptB = np.matmul(orient[0:3,0:3], ptB) + base_frame
If I get some support from the authors I can help you to generalise the case and contribute:)

Thanks for the comment!  Yeah, this type of setup is not the easiest to set up, sorry about that.  That being said, the "official" way to do this would be to make sure the base_frame offset is included as a fixed joint in the urdf, and make sure that joint is included in the joint list in the start_here / info_file.  It sounds like you tried that, but my guess is that the offset joint was incorrect in the urdf.  Once the fixed joint is correctly added and specified from the urdf, it should be doing the exact same computation that you manually included in the post above

Hi djrakita,
I guessed those could have been official solutions and I tried these and others. However, I was Either getting a wrong positioning of the robot (but matching collision shapes) or a wrong base position of the collision shapes.
This unofficial solution seemed to be the only working solution for me as the problem is only in the collision base_frame which is assumed [[0,0,0], [0,0,0,1]] of common_world (That is what I understood).
One could separate the world frame and the base of the chains in the start_here.py configuration.

Gotcha, thanks for the additional info!  Actually, I've been working on a new robot library that will include a newly implemented version of relaxedIK and other motion planners and whatnot that will handle tricky configurations like this much, much better.  It should be available over the next couple of months.  Glad you came up with a workable solution in the short term, and hopefully the new library will afford an even more solid and robust solution for this kind of thing in the long term

Hi @djrakita,
I am working on the same robot as @Raziel90, but I want to use the new CollisionIK solver, can I use his trained model or do I need to train a new one? In this case, have you solved the problem reported in this issue or do I need to manually manage them?
Thanks for your support


jsonable_encoder() vs dict() to obtain a dict from a BaseModel object

Apparently there are two ways to convert a BaseModel object to a dict format :
the native BaseModel dict() function and the fastapi jsonable_encoder() that used internally by the framework .
which one should I use ?
is there a difference in the performance ?

Both produce dictionaries, but dict might produce objects like datetime or Enum, while jsonable_encoder produces serialized fields (it would convert those objects into str, for example).


⚠ RPC server - Astar has degraded performance

In b21c5a7, RPC server - Astar ($ASTAR_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Astar performance has improved in ef6ea0d after 29 minutes.


Project 1: Task 22: Typo in First Sentence of Task Description (Minor)

In Project 1, Task 22, the opening sentence of the task description reads as "The graphic for India shows a According to the data,..." Based on my understanding of what the intended sentence should read as, the bolded upper case 'A' and the space between it and the lower case 'a' should be deleted, so that it reads as "The graphic for India shows according to the data, ..."

I'll Replace

The graphic for India shows a According to the data, the number of people living in extreme poverty has been increasing in Nigeria and the United States, the massive decreases in China and India have shaped the overall trend that extreme poverty is decreasing worldwide, both in percentage and in absolute number.

with

According to the data, the number of people living in extreme poverty has been increasing in Nigeria and the United States, and the massive decreases in China and India have shaped the overall trend that extreme poverty is decreasing worldwide, both in percentage and in absolute number.

Steps

[ ] Make the change
[ ] Rebuild the notebooks


Dependency not publishing for NPM repo

Hello Team,
I am using jfrog cli to publish artifact for a NPM repo. I am able to publish the artifacts while the dependency is not getting published. Could you please advice.
below pasted is the command I am using.
./jfrog rt c --url=${repoUrl} --apikey=${this.dsl.env.ApiToken}
mkdir .jfrog
cd .jfrog
mkdir projects
cd projects
cat > npm.yaml <<EOL
version: 1
type: npm
resolver:
repo: npm-int
serverid: Default-Server
deployer:
repo: npm-int
serverid: Default-Server
EOL
cd ../..
npm -version
npm install npm@latest -g
./jfrog rt npmi common --build-name=${buildName} --build-number=${buildVersion}
./jfrog rt bag ${buildName} ${buildVersion}
./jfrog rt bce ${buildName} ${buildVersion}
./jfrog rt npmp --build-name=${buildName} --build-number=${buildVersion}
./jfrog rt bp ${buildName} ${buildVersion}

@Arun1111,
JFrog CLI does not upload the dependencies to Artifactory.
The dependencies are already there in the remote repository cache. You should see the path for each one of them in the build-info page.
For more information see Building npm packages.
If you have more questions, please ask.

@Arun1111,
I'm closing this issue for now, but please let us know if you have follow up questions.


LVGL-rs on Raspberry Pi

Will this library work on RPi CM3+ with or without X?
I'm trying to work on an "embedded" GUI, but Linux and it's framebuffer isn't mentioned in the list of "supported displays", so I'm unsure. Lvgl itself can be run on Raspberry Pi, but I'm wondering what's the status of it in lvgl-rs.

I didn't try to run in Raspberry Pi, but I see no reason why it wouldn't work. As long as you have access to the framebuffer, you can create your own embedded-graphics driver for it. I run lvgl-rs on nRF52832 platform which are way less powerful than a Raspberry Pi.

Oh, so framebuffer driver needs to be implemented before we can use it.
Then that's great news, I'll look into doing that and make it work.
The question wasn't about performance, but about ability to use framebuffers and draw onto screens detected by OS and not baremetal.
Thanks!

Cool, let me know how it goes. 🙂

Once #64 gets merged native framebuffer support should be added if you're still interested, @MGlolenstine

I'm not in the immediate need of this, as I've had to switch projects due to unrelated issues, but I'd love to have it accessible if I return to the project someday :)


”Todirești” is listed in the drop-down menu

Description:
"Todirești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Todirești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Todirești, IAȘI".


[Feature]: Mayam calendar

https://github.com/loathers/TourGuide/blob/tes-apriling-helm/Source/relay/TourGuide/Items of the Month/2024/Mayam Calendar

Closed by #221


fix(deps): Updating p2wdb and minimal-slp-wallet

:tada: This PR is included in version 3.1.10 :tada:
Your semantic-release bot :package::rocket:


Ability to hide the sidebar?

@Athou! :-)
What a fantastically simplistic feed reader. Thank you so much for the development of it.
I have one suggestion. I prefer to always zoom in and make the text larger.
In that regard, the sidebar stays in my way and I don't need to see it anyway as I prefer reading all my subscriptions.
Would you find it useful, too, to add a setting by which the user can hide (or make smaller?) the sidebar that holds the subscriptions to give some more space to the content when zoomed in? :-)
Thank you!

You can hide the sidebar by pressing F.
This is temporary though, if you want to hide the sidebar forever you can add this to your custom css (in your preferences):
.sidebar-nav-fixed { display: none }
.main-content { margin-left: 0 }

That's right.

I guess I am going for the single manual F button press then every time I open the feed.
Thanks though!
:-)

Actually, just to add a small note here. I used Stylish Chrome extension now to create a style that hides the sidebar. Quite neat. I can simply disable the style and back is the sidebar. Just FYI. :-)

Glad you found something that works for you :)


Node 0.12: Not a valid kExternalUnsignedByteArray

Hi,
I get a  TypeError: Illegal source: Not a valid kExternalUnsignedByteArray error when using node 0.12?
var memcpy = require("memcpy");
var ab = new ArrayBuffer(8);
var buffer = new Buffer(ab.byteLength);
memcpy(buffer, ab);

No solution to this?

I haven't found one. I use Buffers now directly so I just replaced the
memcp dependency.
Am 15.04.2015 um 19:10 schrieb Mateo Gianolio
<EMAIL_ADDRESS>
No solution to this?
—
Reply to this email directly or view it on GitHub.

Currently investigating, but it seems that the version of v8 used in node 0.12 doesn't expose the backing data anymore without externalizing it (which we don't want). There seem to be some workarounds using v8::Utils, but I haven't managed to get it to work, yet, as this is not exposed publicly.
If you have an idea, let me know.


M68K addx doesn't handle carry properly

The current definition is
{ local tmp =zext(XF)+Ty; addflags(tmp,Tx); Tx=Tx+tmp; extendedResultFlags(Tx); }
If Ty = 0xffff_ffff, Tx = 0 and XF = 1 zext(XF)+Ty will overflow but the carry will be discarded causing the operation to fail to  properly set the CF/XF.
It looks like the ADDC in ia.sinc handles this properly.

Good find. This should be updated to reflect the correct carry semantics.


minor cleanups

@SlyngDK  please have a look here: the build is failing about java imports not being sorted. But in the diff there are no changes in the imports!

Have you tried running mvn impsort:sort formatter:format to fix it. It is good to run before committing.
Maybe an unused import triggering the issue.
You can try the same on #122

@SlyngDK  both pull requests failed with:
Error:  Failed to execute goal org.sonarsource.scanner.maven:sonar-maven-plugin:<IP_ADDRESS>5:sonar (default-cli) on project quarkus-logging-json-parent: You're not authorized to run analysis. Please contact the project administrator. -> [Help 1]

@dfa1 Yes, there is a permission problem against SonarCloud, normally when I rerun manually it's works but not today.
@gastaldi I will be happy if you will fix the integration against SonarCloud, so it works with pull requests.

@SlyngDK unless the SONAR_TOKEN is fixed in the build script, this is a limitation on how Sonarcloud reads the SONAR_TOKEN as described in https://sonarcloud.io/project/configuration?analysisMode=GitHubActions&id=quarkiverse_quarkus-logging-json

Maybe something like https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ could help (eg. Triggering the PR build only when a safe to test comment is added to the PR)


Solidity Vscode Intellisense only working if i specify a contract

Hello folks!
I'm struggling severely to configure my Vscode working properly.
I have imported contract of openzeppelin - ERC721.sol
Now I expected its function setApprovalForAll() be available in intellisense when i start typing it in my contract, but here is what i see if I start typing it and then click Ctrl+Space:

In the same time if i explicitly type the contract name - then i can see intellisense autocompletion:

Please give me a hint - what am I doing wrong?

I realised that it is working this way because ERC721.sol is in one of node_modules.
Once i put my contract file into the openzeppelin module folder, next to the ERC721.sol - autocompletion works fine.
But isn't that possible to make it so that I could use autocompletion for modules that are in package.json?

Hi @markokhman in settings you can set the value of the dependencies forlder, that is the defaulted to node_modules as many contracts are distributed using npm, you can change the folder if you want (relative to the root of the project)
We have support also for the "remappings" file, check the readme for instructions on that (and the above) to get bigger detail.
https://github.com/juanfranblanco/vscode-solidity#project-structure-and-remappings

@juanfranblanco thank you very much for getting into this issue for me! Really appreciate that.
I was able to successfully import a contract (ex. ERC271) that is provided by @openzeppelin and stored in node_modules, so I guess this means that my VSCode is configured properly. Also i get autocompletion if i reference directly from contract:

I face an issue if I try to access the methods of this imported contract directly:

I was able to find out that if I put my custom contract next to the ERC721.sol - then the autocomplete magically works!

But isn't it supposed to work the same way once I import the ERC721 from the @openzeppelin?

Seems ok here:

if the parser fails, try this.

The issue was with vscode, as there was not possible to set the label (null) the autocomplete was breaking, this was not related to the parser.

Hi @juanfranblanco thanks for your awesome plugin.
If I may there is still a small issue in 0.0.137, not sure if this is helping but it's quite confusing. When the contract can not be parsed I guess with a syntax error (not all though...), completion goes away. Not sure if this is normal behavior or expected.

Cheers !


Introduce inset-block usage in the viewer CSS

Given that the viewer always set the dir-attribute, to either LTR or RTL, we should be able to use this logical CSS property to (very slightly) reduce the size of the CSS; please see https://developer.mozilla.org/en-US/docs/Web/CSS/inset-block

/botio-linux preview

There are likely other potential simplifications here:
https://phabricator.services.mozilla.com/D170496
(for the context Itiel is a RTL user and doesn't know that much how github works hence they just made a m-c patch).

There are likely other potential simplifications here: https://phabricator.services.mozilla.com/D170496

I've only just skimmed that, but it looks like a huge patch with lots of more-or-less unrelated changes. And from the inline comments, it seems that it was at least partially written based on code-inspection rather than actual testing?

This PR happened because I found inset while browsing MDN articles, please see PR #16173, which also made me notice the inset-block property.
As I've done previously when modernizing our CSS, I usually try to limit the size/scope of the changes to make testing simpler since these sort of changes generally require manual testing.

There are likely other potential simplifications here: https://phabricator.services.mozilla.com/D170496

I've only just skimmed that, but it looks like a huge patch with lots of more-or-less unrelated changes.

Correct.

And from the inline comments, it seems that it was at least partially written based on code-inspection rather than actual testing?

Some of the changes there don't require actual testing, like top: 0 and bottom: 0 becoming inset-block: 0.
But I've tested the vast majority of the patch in both RTL and LTR and didn't spot any issues.


Use Varints for persistent serialization format to unlimit the key/value lengths.

What would you think about using Varints (http://golang.org/pkg/encoding/binary/) to encode all numerical values in gkvlite's datafile?
Advantages:

would allow support for keys and values up to 2**63-2 without bloating databases which don't use this functionality (2**63-2 because the header needs to encode len(key)+len(value)+len(priority)).

[though if the priority were encoded as a fixed size, it's size could be implied].

would save a bunch of bytes / item for databases with many small records

Disadvantages:

would change the binary format of the database, so old DB files wouldn't be loadable.
would change the StoreCallbacks.ItemAlloc interface
could add up-to ~8 bytes per record for databases with large key or value records (e.g. records over 268,435,455 bytes would take more that 4 bytes to encode the length header), or with large Priority values (over the same quantity). From examining the code it looks like there are 4 of these integer quantities per item, and MaxVarintLen64 is 10 bytes (2 more than the fixed width encoding).
variable-size integers are a bit trickier to deal with in the data file format (though looking at the code I don't think it would be too hard).

I also realize that this project hasn't really changed much in 2 years, so maybe this question will be unseen :). However, would you be interested in a pull request that would do this? I think the biggest hurdle would be the datafile incompatibility. If that's a hard-stop for accepting the pull request, then I probably wouldn't work on it for quite a while.
For now, I've forked and simply bumped the key size up to 32 bits (luci/gkvlite@cf7fa95b097418138e80474e217d865e2eb853ac) (I needed up-to 2MB keys :/).

Hi - an interesting idea.
Not generally opposed to this, but a few more questions/ideas to riff off yours....
Especially, there's also a persisted version number, so that one possibility is to just check that and keep the old read/write() methods around for backwards compat.  Might not be worth the backwards compat effort, though.
One thing I'm unclear about is if you'd just want to bump the key/val length types up to uint32/uint64, or uint64/uint64, or actually use Varint/Varint for even the in-memory representations.  I'd prefer the first, if your app needs could be met that way.
btw, if you don't mind me asking, wondering what kind of app has keys that might be up 2**63-ish long?  That's a long key!
Finally, I don't think there's that many users of gkvlite, but probably throwing a big "major version number" increment, a'la semantic versioning, would also be needed if backwards compat wasn't supported.

Ah, yeah after I posted this question, I noticed the VERSION const. I think bumping it would be definitely required, not sure about keeping both sets of read/write logic around. If I did that I'd basically have 2 sets of everything, which sounds like pretty high maintenance overhead (and since gkvlite seems to be pretty quiet feature-wise, it would probably be best for downstream users to just pin the version of the library they're using anyway :)).
I was only planning on changing the serialized format to use varints; in-memory objects would keep normal {,u}int{32,64} types. Keeping varints around in memory is a hassle (especially without the possibility of operator overloading).
Oh, yeah, I don't actually want 2**63 long keys (which would be completely insane) :D. I specifically need 2MB keys (which is also pretty huge), to conform to this (https://cloud.google.com/appengine/docs/go/datastore/#Go_Quotas_and_limits, see the bottom limit re: composite index entry size). I'm writing a go-appengine testing framework, and I'd like to use gkvlite to emulate the datastore's semantics (particularly; gkvlite is native-go and supports multiple goroutine-safe Snapshots and allows database mutations without invalidating the snapshots, unlike, say, boltdb which uses a reader/writer lock over the entire database, or leveldb which has the right semantics but isn't native-go).
I was also just thinking that varints would be a handy way to improve storage size for gkvlite (on-average) at the same time as effectively unlimiting the size of the key/vals that it contains :)

Cool, makes sense.
In memory of (u)int{32,64} for key and val lengths makes sense to me instead of in-memory varints.
Varints for persistence is also ok by me, even if incompatible.

(Just so you know, I do plan on actually doing this... I just got pulled into a couple other directions in the meantime :disappointed:)


Signing of assemblies with a strong name

Are there any plans on when you will release signed assemblies?
I would need to have signed assemblies in place for my release builds, but do not really want to sign them manually all the time, I get a fresh nightly build.

Just following up with the Microsoft.Framework.* package owners as EF is dependent on these being strong named before we can be. I've started a follow up on https://github.com/aspnet/EntityFramework/issues/999 so we can continue discussion on timeframe etc. there.

Is there any progress?
See https://github.com/aspnet/EntityFramework/issues/999#issuecomment-143674933


item.priceCurrency is missing in eth-master listings

In listings, we're expecting non-cryptocurrency listings to have a item.priceCurrency object.
Currently, that field appears to be missing if you load a current master v4 listing into the eth-master branch and view it in the desktop client.
Below is an example output from GET http://localhost:4002/ob/listing/QmPFvqyMp13EQHP94AZhFWQWQtmUQteeCoS75wEfyKu2Wb/magnetic-tortoise
{
    "listing": {
        "slug": "magnetic-tortoise",
        "vendorID": {
            "peerID": "QmPFvqyMp13EQHP94AZhFWQWQtmUQteeCoS75wEfyKu2Wb",
            "handle": "",
            "pubkeys": {
                "identity": "CAESIHJ0a33yfPM5cLv4gn1jxEEYrm01rwgqhNzY9nncQMrR",
                "bitcoin": "A8EfU5B/jyUYqOBWQk4LFEhfby1osoBBVD/7/QQabCUI"
            },
            "bitcoinSig": "MEQCIF6kAZVbGQqdX9tF42n4Vgi1l5zLQClTVGOp2cAERaVGAiAFO+dNZkmoXldANwtnnJ7yNR8EsyqNK4SjfAhh55hdLA=="
        },
        "metadata": {
            "version": 4,
            "contractType": "PHYSICAL_GOOD",
            "format": "FIXED_PRICE",
            "expiry": "2037-12-30T23:00:00.000Z",
            "acceptedCurrencies": [
                "BTC"
            ],
            "pricingCurrency": "EUR",
            "language": "",
            "escrowTimeoutHours": 1080,
            "coinType": "",
            "coinDivisibility": 0,
            "priceModifier": 0
        },
        "item": {
            "title": "MAGNETIC TORTOISE",
            "description": "
WOODEN INLAY - PUZZLE -

REMOVABLE MAGNETIC PIECES

MEASURE: CM 27 X 25 X 2

",
            "processingTime": "",
            "price": 100000,
            "nsfw": false,
            "tags": [
                "art",
                "wooden-art",
                "geometric-art",
                "precious-wood",
                "inlay",
                "wooden-inlay",
                "modern-art",
                "puzzle",
                "games",
                "magnetic"
            ],
            "images": [
                {
                    "filename": "027_MAGNETIC TORTOISE.png",
                    "original": "zdj7WVUJZkTBbmPzDo9RYzg25dhetQDQX6GBgdMNs1fGog3Lz",
                    "large": "zb2rhoNdi62Hztjdgq4cofqgYVCZxQg5RCwomxYPB3ge59ngD",
                    "medium": "zb2rhYyZhL5zNuWWuKsgekgHHDDHzrdwtMB7qP8us9vtRBFVi",
                    "small": "zb2rhoiugNoZWN6Lbk7PDNRqthGBC2d6cwqEGDFpvQ3Gw9kvu",
                    "tiny": "zb2rhf3Fm6gsyhbe6CBsecFkxno7oqJv9LUtnxVMxg7zrWnc2"
                },
                {
                    "filename": "TANGRAM TORTOISE.png",
                    "original": "zdj7Wk5QqhUS2ufPdPs2LNEKA4FjF8Y7TwnqFPfMMRFqWaGfK",
                    "large": "zb2rhb8tZBamuhdDy3QMg9fRPYCAX7j9Wkv1H83rQtNrckrbj",
                    "medium": "zb2rhbMUF37mtDeFhtBrWAfyHrfPtxq1Yun5K3fUcxpXcvAwD",
                    "small": "zb2rhjKyW4nAGQr2oasfoH2YZQxK8oBcLMo9TGnoKknBYwcyc",
                    "tiny": "zb2rhYFeYUcPVuxB1wH6hRY27q5fXAsAAXREH97PHnfVF4ZxG"
                }
            ],
            "categories": [
                "art",
                "games"
            ],
            "grams": 0,
            "condition": "NEW",
            "options": [
            ],
            "skus": [
                {
                    "productID": "",
                    "surcharge": 0,
                    "quantity": 0,
                    "bigSurcharge": "",
                    "bigQuantity": ""
                }
            ],
            "priceModifier": 0,
            "bigPrice": ""
        },
        "shippingOptions": [
            {
                "name": "DHL EUROPE",
                "type": "FIXED_PRICE",
                "regions": [
                    "AUSTRIA",
                    "BELGIUM",
                    "BULGARIA",
                    "CROATIA",
                    "CYPRUS",
                    "CZECH_REPUBLIC",
                    "DENMARK",
                    "ESTONIA",
                    "FINLAND",
                    "FRANCE",
                    "GERMANY",
                    "GREECE",
                    "HUNGARY",
                    "IRELAND",
                    "ITALY",
                    "LATVIA",
                    "LITHUANIA",
                    "LUXEMBOURG",
                    "MALTA",
                    "NETHERLANDS",
                    "POLAND",
                    "PORTUGAL",
                    "ROMANIA",
                    "SLOVAKIA",
                    "SLOVENIA",
                    "SPAIN",
                    "SWEDEN",
                    "UNITED_KINGDOM"
                ],
                "services": [
                    {
                        "name": "DHL",
                        "price": 10000,
                        "estimatedDelivery": "7-10 DAYS",
                        "additionalItemPrice": 0,
                        "bigPrice": "",
                        "bigAdditionalItemPrice": ""
                    }
                ]
            },
            {
                "name": "DHL WORLDWIDE EXPRESS",
                "type": "FIXED_PRICE",
                "regions": [
                    "ALL"
                ],
                "services": [
                    {
                        "name": "DHL",
                        "price": 15000,
                        "estimatedDelivery": "7-12 DAYS",
                        "additionalItemPrice": 0,
                        "bigPrice": "",
                        "bigAdditionalItemPrice": ""
                    }
                ]
            }
        ],
        "coupons": [
            {
                "title": "20% FOREVER COUPON",
                "hash": "QmTWmRKs7jqPHER5YPeERR33WkUP2A8QdveBYRD8o21G1r",
                "percentDiscount": 20,
                "priceDiscount": 0,
                "bigPriceDiscount": ""
            }
        ],
        "moderators": [
            "QmYAkueJM36r7qtLLQu5rensaDdAYAP5W5Fv11vsSppzdM",
            "QmeRw4K4s5HvWmYZhTdDt8mdr5rf6MCNZHW51T8ZmKKF9g",
            "QmboYfRzSH8n4U8DbDfw1bt754XHy2onQLRsienEpEBXGy",
            "QmaeNaBvbxEy5KJhQaW64MGL3XgJGXWKqpJuFSPe1WXQ2A",
            "QmPu4xhmjTCQ7YpnRxXCqn9AEmrXvesdK9wRgJ8YYEEMPX",
            "QmUoakQaUw1MMJeQAPToRnxkz65bmHr1juG1amsBuvyrdR",
            "QmZt6CsjnWjGAXDks1Ws4PQz6asUVMfqHddiYDUnoZetwv",
            "QmX6s5DNJZ5uVA1im1BfxrmTUfYoZTZxFL2zzfjFgDUhXN",
            "QmNiSpE5tCiupkvsumnfxwBjBkEVRseaRiUt2JFdVmSh1n",
            "Qmeh5qqTAjrGyypnXdM4bgF7w4AwYpTG2rQ4kkXhv5bMLG",
            "QmUkQuY5ChYhutKuy4qmgqg6HPzqYvG4xSJtPHY7RCAcMQ",
            "QmZ8pAsnSoLNJkMVGoLrtG4qq9GeTEWBFHkwyjugPtbH4y",
            "QmTZYbccDY9hz1YLFnbV2c2XqottSyoRPay1n7VmwLtezT",
            "QmcscQDiCuTSGxBeMD9qyXwRMcbLU5m9P1kupojYJdFdoh"
        ],
        "termsAndConditions": "ALL MY WORKS ARE SINGLE PIECES AND COMPLETELY HANDMADE WITH PRECIOUS WOOD.\nIF YOU LIKE MY INLAY AND YOU BUY IT AT THE PRICE SHOWED ON SITE I WILL SEND IT TO YOU AS SOON AS POSSIBLE.",
        "refundPolicy": "NO RETURN, IF YOU LIKE IT BUY IT"
    },
    "hash": "Qmd6yDPSLkANoiKZMosD1qtMFe6XzM1tEbdnJnbjdetyn4",
    "signature": "5ddZ3nk67N99WKxZKtXJLmQZSLcWgZWe+4sGNiMLRYK4qvf0QdKW0nqd9BwTE9ooMEwXFxyVX4owAixe+i/lBw=="
}

While working through this issue, we found that normalizing the data would cause the GET /ob/listing/... call (which normally returns a signature) to make the signature a lie (should someone consuming the API want to independently verify the listing signature). It seems the signature from this call is not used/consumed by the desktop client (according to discussion with @jjeffryes and @rmisio) and unclear if the mobile client uses it.
After discussion with the team, we decided this behavior existed already (where the signature does not represent the provided listing, but the original (un-normalized) listing. We should regard the listing signature valid for the original listing and still useful in this case for the time being.
In all cases, the signature would not be for the provided listing body but would remain in the response if anyone wanted to calculate the listing's sig independently. (The signature comes from signing the jsonpb's JSON representation of the listing as stored on IPFS and on the local disk of the vendor node who owns the listing.)


Add documentation to Readme for mobile support (iOS/Android)

Resolves #433

lgtm. Is the BoltDB wrapper so that gomobile can generate Android/iOS wrapper functions? I haven't used the tool before.
Can you squash the commits and then I'll merge it in.

The BoltDB wrapper is what iOS and Android can bind to. Instead of exposing bolt and all of it's methods to java or objc, the logic should all be done in the go layer. The BoltDB struct would most likely be named something else in a real app and then handle all of the storing and retrieving of data specific to the app.
I did not really explore exposing just the base DB type. Hypothetically this might be possible (and do all bolt interaction from java directly) but gobind does have limited type support, and the example I wrote would be the cleaner implementation.
Thanks for looking at the PR, I'll condense the commits when I get home.


Does 9P file server largely solve slow Windows Network Share access sluggishness?

Does adding the 9P file server for cross-file access between Linux and Windows, as mentioned in your recent blog posting, largely help resolve the widely reported I/O sluggishness when accessing Windows Network Shares from wsl?  Just listing network shares can take over 100x as long from the LInux side as it does from File Explorer.  This is the main reason we might have to abandon WSL when everything else seems to work pretty well..
Would running Linux under a VM using Samba for Windows file access resolve this issue?
Thanks.

I do not know why this was added, I've been using WSL for two years and have C: and two network drives mapped using devfs and it's working great, please do not add anything into WSL for newbee's that would slow down IO acess

On Feb 19, 2019, at 2:58 PM, Lewis L. Donofrio<EMAIL_ADDRESS>wrote:
I do not know why this was added, I've been using WSL for two years and have C: and two network drives mapped using devfs and it's working great, please do not add anything into WSL for newbee's that would slow down IO acess

Umfortunately, you seem to be in the minority with dozens if not hundreds of separate messages detailing extremely slow I/O under WSL, Microsoft acknowledging it, saying it is a hard problem, working on it for years but not yet announcing an solid fix.
Any word from the Microsoft staff working on this.  I would like to continue using WSL with Debian but it is presently excruciatingly slow with remote shares on an i7 laptop from 2018 with 24GB RAM and a fast SSD under Windows 10, something like Oct 2018 release.

Ran course. I couldn't follow the question enough to add anything constructive.


Workspace names truncated after whitespace

In i3 I have my workspace names including whitespace:
bindsym $mod+1 workspace 1: web
In sway that give the workspace the name "1:"

Add quotes:
bindsym $mod+1 workspace "1: web"


[FLINK-13504][table] Fixed shading issues in table modules.

What is the purpose of the change
Moved the common calcite specific dependencies configuration to
flink-table in order to have them unified across flink-table-planner,
flink-table-planner-blink, flink-table-runtime-blink & flink-sql-parser.
Shading configuration is also done in the table parent pom so that all
modules have common relocation policies.
Enabled shading for flink-sql-parser.
Verifying this change

For verification of sql-parser module e.g. run the query attached to jira issue.
Verify contents of flink-table-planner, flink-table-planner-blink, flink-table-runtime-blink, flink-sql-parser, flink-table-uber, flink-table-uber-blink do not contain unwanted classes.

Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@twalthr @zentol I would appreciate if you could have a look.

This looks correct to me, but when I run it locally the shade-plugin fails with an IllegalArgumentException when building the blink uber jar. Currently checking what's going on,

For clarification: all non-uber modules appear to be working fine.

nvm, my issue is due to using different JDKs. +1 if Travis passes.

Unfortunately it does not :( There are some convergence problems in flink-json (sic!) Checking them out.

I have run a simple check and the issue described in the parent ticket is gone (maven version 3.2.5):
the SqlCreateTable uses proper shaded SqlIdentifier.names field.

Unfortunately this approach does not work.
The problem is that when we relocate flink-sql-parser it uses the relocated guava classes. flink-table-planner* have direct dependency on calcite which pulls the unshaded versions of the same classes in tests (as tests are executed before shading). Therefore we cannot move the shading configuration to the flink-table module.
I will close this PR in favor of a new approach (#9313 )


Problem with games

Before opening this kind of issue, please ensure:

Are you using a compatible console version? Yes, European Scph-7502
Are you using CD-R media, and not CD-RW? Yes, I've checked.
Did you copy all the files in the expected format to the memory card? Sure, Tonyhax boots normally.

tonyhax version: Which version are you running?  The lastest one (1.1.2)
Installation method: How did you get tonyhax on the memory card? By using Mc Annihilator.
Entry point game: Which game are you using to launch tonyhax?
Console model: "SCPH-7502"
Integrity check: If it boots, does the built-in integrity check succeed? Yes, it does.
BIOS version: If you can get to boot, which version of the BIOS does it report? I'll check it out later. I don't remember right now.
Target game: Medievil 2 (SCUS-94564) and Time Crisis (SLUS-00405)
Bug explanation:
Medievil 2 boots to the title loading page and stays there. Time Crisis boots to a black screen.
I have tried another game (Looney Toons Ship Raider (NTSC-U) and it works perfectly.

MediEvil 2 contains anti-piracy checks so that makes sense. Kind of surprised that the anti-piracy 'prohibited' screen does not show though. Medievil 2 is not yet supported in Tonyhax International but it can be, I'll look into it.
Now Time Crisis I don't believe contains any anti-piracy checks. Does that game work with the original Tonyhax v1.4.3?

MediEvil 2 contains anti-piracy checks so that makes sense. Kind of surprised that the anti-piracy 'prohibited' screen does not show though. Medievil 2 is not yet supported in Tonyhax International but it can be, I'll look into it.
Now Time Crisis I don't believe contains any anti-piracy checks. Does that game work with the original Tonyhax v1.4.3 or UniRom for reference?

Yes, Time Crisis works both with original Tonyhax 1.4.3 loaded from Crash Bandicoot 3 (eu) and with Gameshark cartridge swap method.

MediEvil 2 contains anti-piracy checks so that makes sense. Kind of surprised that the anti-piracy 'prohibited' screen does not show though. Medievil 2 is not yet supported in Tonyhax International but it can be, I'll look into it.
Now Time Crisis I don't believe contains any anti-piracy checks. Does that game work with the original Tonyhax v1.4.3 or UniRom for reference?

Yes, Time Crisis works both with original Tonyhax 1.4.3 loaded from Crash Bandicoot 3 (eu) and with Gameshark cartridge swap method.

Alright, so I just tested Time Crisis USA on my stock SCPH-1001 and it worked with the Tonyhax International v1.1.2 Boot CD via CD Player Swap Trick. I'm going to try specifically booting it with Crash 3 EU to see if that could possibly be related.

I tested your boot method like so with my stock SCPH-1001:

Burned Crash 3 Europe to a CD-R.
Copied over the latest 'HAX' and 'BISCPS-1007300000000' files to an authentic PS1 memory card.
Booted the Tonyhax International v1.1.2 Boot CD with the CD Player Swap Trick.
Booted Crash 3 Europe from the Tonyhax International Boot CD.
Loaded the exploit save game file and started tonyhax international v1.1.2 from memory card.
Finally, put in Time Crisis USA. It booted correctly and I am able to play it on my console.

So I am unable to reproduce this Time Crisis boot issue.
For further debugging, are you using the FreePSXBoot images or the save game exploit? Are you sure you are using the correct up to date save game files (both 'HAX' and 'BISCPS-1007300000000'?) from the latest v1.1.2?
I do not have a light gun, do you have a light gun plugged in during booting of the game by any chance?
You say that the game boots fine with Unirom and Tonyhax v1.4.3 (original). The same exact burned CD-R in the same console reproduces the boot issue for you only in International v1.1.2?

Do you think different boot methods may affect the effectivity of tonyhax international? I was under the assumption that different boot methods just offer more possibilities to the users to access the same program (although I'm thankful you took the time to reproduce the sequence).
Next time I plug my ps1, I'll send you a video with all the steps for further clarification.
I'll rule out any Cd related problem since the games not only work fine with UniRom and OG but also on pc emulator.

Do you think different boot methods may affect the effectivity of tonyhax international? I was under the assumption that different boot methods just offer more possibilities to the users to access the same program (although I'm thankful you took the time to reproduce the sequence). Next time I plug my ps1, I'll send you a video with all the steps for further clarification. I'll rule out any Cd related problem since the games not only work fine with UniRom and OG but also on pc emulator.

It should make 0 difference how you boot into Tonyhax International, I haven't come across any examples of i.e. one boot method working while the other does not. I was just trying anything to reproduce this issue.
I expect MediEvil 2 to work with Tonyhax OG v1.4.3 since it uses a different anti-piracy bypass system. I expect it to fail in Tonyhax International v1.1.2 because I have not added explicit support for it (yet). This issue will be easy to solve.
Now Time Crisis is a different beast, I don't really have an answer for why it does not work for you. I don't have any PAL consoles, but I have a ton of different Japanese and USA ones. It's possible something is explicitly going only with PAL consoles.
Are you indeed using the FreePSXBoot method with Tonyhax International? Is there a light gun connected during all of that? I'm trying to rule out this: https://github.com/brad-lin/FreePSXBoot#troubleshooting
It is recommended to use a normal game pad when running the exploit. Special game pads may interfere with the exploit and prevent it to run.

Can you reproduce this issue on your PAL console @M4x1mumReZ ?

@alex-free I'll try to reproduce the issue and I'll get back to ya.

I have written two discs using my Plextor at 4x speed - one being a Verbatim 700MB CD-R and a RiTEK thermal printable 700MB CD-R, both games worked flawlessly without any problems using both cheat cartridge, disc swap with Tonyhax boot CD, and FreePSXBoot. My assumption is that the discs you are using is of poor quality or that you haven't burned them correctly.

@alex-free @daro-glez Okay. I have burned the games using my Plextor on two CD-Rs - one being a Verbatim 700MB CD-R ("MediEvil II") and a RiTEK thermal printable 700MB CD-R ("Time Crisis"). Both games worked just fine - "Time Crisis" didn't lock up on a black screen and I was able to play it, the same goes to "MediEvil II". MediEvil II's anti-piracy doesn't activate on the loading screen, but only activates if you reach either the 'Whitechapel' or 'Demon' levels according to Redump.

@alex-free @daro-glez Okay. I have burned the games using my Plextor on two CD-Rs - one being a Verbatim 700MB CD-R ("MediEvil II") and a RiTEK thermal printable 700MB CD-R ("Time Crisis"). Both games worked just fine - "Time Crisis" didn't lock up on a black screen and I was able to play it, the same goes to "MediEvil II". MediEvil II's anti-piracy doesn't activate on the loading screen, but only activates if you reach either the 'Whitechapel' or 'Demon' levels according to Redump, as if I have time for that.

What model is your console again?

@alex-free SCPH-7502 - no mods installed.

@alex-free SCPH-7502 - no mods installed.

This is sounding more and more like @daro-glez has an iffy CD drive. @daro-glez can you re test? Maybe test with your console orientated vertical.

Sounds more like @daro-glez is using either poor quality CD-Rs, has burned them incorrectly, or has a CD drive that needs the laser retuning.

Sounds more like @daro-glez is using either poor quality CD-Rs, has burned them incorrectly, or has a CD drive that needs the laser retuning.

I was having issues like @daro-glez until I figured out how to refurbish CD drives https://alex-free.github.io/unofficial-ps1-cd-drive-service-manual

Yeah, I saw your guide which is how I got to know you.

This is great for when I'm going to pick up an SCPH-1002 for re-testing, probably will find one that can't read discs for cheap and then I'll follow your guide.

This is great for when I'm going to pick up an SCPH-1002 for re-testing, probably will find one that can't read discs for cheap and then I'll follow your guide.

I've literally bought broken PS1s that can't read discs and just done the refurb as shown in my guide and it worked out great. I say probably an 80% success rate with it. Sometimes the spindle motor literally just needs some light clock oil and it works/sounds like new. Sometimes it doesn't work out, but you'd be surprised.
You can almost always just buy a PSone and the console model you actually want, and transfer the PSone drive into the target console model you actually want to use as well if the drive refurb really doesn't work out. But if you do the refurb method a few times (sometimes you need to keep relubricating until it stays working) usually it works out. I have a launch SCPH-1000 console (VERY EARLY SERIAL NUMBER, definitely launch unit or close to it from 94) with the original drive, works pretty well once it's lubed up.

Thanks for the tips. 👍🏻

Oh yes, I have seen AliExpress sell those dodgy PS1 replacement lasers - I don't trust them AT ALL on my life as I've heard many bad things about them from other people. The mechanism falls to pieces very easily because its so cheap and fragile, and it doesn't read discs correctly once you have installed them in.

Oh yes, I have seen AliExpress sell those dodgy PS1 replacement lasers - I don't trust them AT ALL on my life as I've heard many bad things about them from other people. The mechanism falls to pieces very easily because its so cheap and fragile, and it doesn't read discs correctly once you have installed them in.

 All of it is documented and belive it or not the CD drive of a PS1 is quite close to a Sony wakman from the mid 90s. I'd pay $100 for something that isn't a scam.

@alex-free SCPH-7502 - no mods installed.

This is sounding more and more like @daro-glez has an iffy CD drive. @daro-glez can you re test? Maybe test with your console orientated vertical. Also, what kind of CD-Rs are you using?

I re-tested Time Crisis and it didn't boot regardless of the method I employed. Curiously enough, Unirom can read the audio files stored inside but the game still refuse to start.
Maybe my cd lens is on its last legs or maybe the cd I used is scratched/bad quality.
Anyway, thanks to @alex-free and @M4x1mumReZ for helping me. Much appreciated.

@alex-free SCPH-7502 - no mods installed.

This is sounding more and more like @daro-glez has an iffy CD drive. @daro-glez can you re test? Maybe test with your console orientated vertical. Also, what kind of CD-Rs are you using?

I re-tested Time Crisis and it didn't boot regardless of the method I employed. Curiously enough, Unirom can read the audio files stored inside but the game still refuse to start.
Maybe my cd lens is on its last legs or maybe the cd I used is scratched/bad quality.
Anyway, thanks to @alex-free and @M4x1mumReZ for helping me. Much appreciated.

Awesome. We got it all debugged from different parts of the globe :)
Sorry about your laser. I recommend refurbishing your CD drive as I explain here: https://alex-free.github.io/unofficial-ps1-cd-drive-service-manual .
Alternatively, you could also buy a PSone and swap that CD drive to your SCPH-7502 (or just use the PSone).
Have you tried tilting the whole console vertical (test tilting it to the left and to the right) and then booting the game? Sometimes it works. If it does, even more reason to try to refurbish the drive.
I recommend using Verbatim Data Life Plus CD-Rs and nothing else if you want to try different CD-R media. It can make a big difference.
I'm going to close this issue since it is not directly related to the loader itself.


http: abort filter chain after reset stream was called

Commit Message: http: abort filter chain after reset stream was called
Additional Description:
To close https://github.com/envoyproxy/envoy/issues/26994. The #26994 is caused by that the filter chain still continue and try to send reply after the whole stream is closed. We should abort the filter chain to avoid any possible further logic after the stream is reset.
Also see https://github.com/envoyproxy/envoy/pull/30835 for more context.
Risk Level: low.
Testing: integration.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

cc @kyessenov @soulxu @vikaschoudhary16 @phlax

cc @phlax @RyanTheOptimist @yanavlasov for release context

It take too much time to run these tests on my PC. Let the ci check it. orz

hmmm, I  just realized it's possible we need to do the same here for the 2 other resetStream calls in the filter_manager so have
https://github.com/envoyproxy/envoy/blob/main/source/common/http/filter_manager.cc#L966
and
https://github.com/envoyproxy/envoy/blob/main/source/common/http/filter_manager.cc#L1007
call your newly enhanced resetStream function rather than call the filter manager callbacks directly.
You want to fix or shall I?
h/t @fredyw for making me look at this code today =P

@alyssawilk I knew that code when I doing the fix.
The change is optional because the sendLocalReply self will handle the status correctly.
Basically, I treat the resetStream and sendLocalReply of FilterManager as two independent methods and will handle the status and reset behaviour by itself.


feat: adding RequiredLogin component

This PR adds the RequiredLogin component built by @evert into react-ketting. I have made a couple of minor adjustments, and have a couple of questions as well as to how this should be included.
Questions:

Should all the console.logs be removed?
Should the cookie name be a variable?
Is it wise to use useHistory and import react-router-dom or should that maybe be passed up to the parent component to handle?
What else do I need to consider to improve the quality of this PR/addition?

Should all the console.logs be removed?

I think it's OK to keep in this case. They are fairly useful for debuggin. Maybe it's nice to prefix all logs with something like [ketting] ?

Should the cookie name be a variable?

This might be nice if someone needs in the future, but also not something required for an initial PR.

Is it wise to use useHistory and import react-router-dom or should that maybe be passed up to the parent component to handle?

This is a good question and my gut says that it's probably bad to expect that to exist! Not everyone uses it

update: replaced use of useHistory to redirect back to url with a successfulAuthentication function, which needs to be passed in from the parent component and consequences of successful authentication are now handled on the parent.
This would break current implementations of this component if migrating to this version.

Well done!


[Enhancement] allow corral packages to use other package's private and public keys

This would help support air gap nodes created after it's bastion has been created.

@igomez06 Can this be closed with #103 ?


Policheck PR check in azure-rest-api-specs is failing on custom-words.txt "whitelist"

The Policheck PR check in azure-rest-api-specs is failing because there are words containing "whitelist" in the custom-words.txt file.
The words being flagged are already merged to main:
https://github.com/Azure/azure-rest-api-specs/blob/main/custom-words.txt#L2390
but Polichek flags them in the PR checks for PR #18372.

Suppression mechanism is added and this work will be added to the suppression list. The change is pending to be released.
//CC: @tianxchen-ms

@mikekistler, new version of policheck tool was released to PROD pipeline and some terms were added to suppression list.


Suppress warning when adding valid tag_id filter search field to the index

I've been getting this warning, but everything seems to be configured correctly as far as I can tell. I have posted some info about my setup at this stack overflow question.
https://stackoverflow.com/questions/58758924/wagtail-postgresql-search-backend-shows-warning-mymodel-search-fields-contains
This PR just makes the warning go away, based on this particular case. I'm sure there is probably a better way to handle this, so if anyone has any suggestions would be awesome.
Thanks!

@benpetty - thanks for submitting this PR. I have and a look and I think there needs to be some more investigation as to the root cause. Adding a specific case to ignore a warning for this by its field name is not ideal.
Would you be ok to raise an issue for this (instead of a PR) and maybe attempt to work out and document the minimum steps to reproduce. E.g. is it postgres specific or maybe field name specific? Maybe the code for generating the warning does not correctly handle some relationship types.
For now I will close this PR as it may suppress legitimate warnings.
I think the stack overflow question you posted should be added as a reference to the issue raised.


Create a SECURITY_CONTACTS file.

As per the email sent to kubernetes-dev[1], please create a SECURITY_CONTACTS
file.
The template for the file can be found in the kubernetes-template repository[2].
A description for the file is in the steering-committee docs[3], you might need
to search that page for "Security Contacts".
Please feel free to ping me on the PR when you make it, otherwise I will see when
you close this issue. :)
Thanks so much, let me know if you have any questions.
(This issue was generated from a tool, apologies for any weirdness.)
[1] https://groups.google.com/forum/#!topic/kubernetes-dev/codeiIoQ6QE
[2] https://github.com/kubernetes/kubernetes-template-project/blob/master/SECURITY_CONTACTS
[3] https://github.com/kubernetes/community/blob/master/committee-steering/governance/sig-governance-template-short.md

Required SECURITY_CONTACTS file still does not exist. Please resolve as soon as possible.

kubernetes/perf-tests still needs a SECURITY_OWNERS file.
/assign @krzysied @k8s-ci-robot @gmarek @shyamjvs @wojtek-t @bowei

LOL assigning @k8s-ci-robot 😆

https://github.com/kubernetes/perf-tests/pull/358 is already out.


feat(protocol): Span Batch Type

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#197 👈
#196
#195
#194
#193
#192
#191
#190
main

This comment was automatically generated by st.


Log LLM tool call for streamed response

The current LLM streamed response parsing only collects the message content/text. This PR generalizes this to allow collecting any state needed to generate the response_data. Streamed OpenAI chat completions now include the tool calls in the Logfire UI. Behaviour for Anthropic and for OpenAI completions (non-chat) remains the same.
Fixes https://github.com/pydantic/logfire/issues/542

cc @willbakst in case you want to take a look

Thanks! Are you happy for me to merge?

Yes, happy to merge! Thanks


controller not working anymore after i add the patch

hi, after I add the patch it fixes my FPS
but my controller not working anymore, inside the game at the controller section I try to move to Xbox 360 compatible, it not working I cant move to this section to enable the controller
any help, please

Make sure the controller is connected and Steam recognises it before launching the game, and make sure the game is completely exited before opening a new instance.
FPS should never go higher than 30 anyway, since collision, animations, and more break above 30, and if you do use 30 make sure enable VSync to utilise proper frame-pacing.
Controller works fine on any FPS regardless though; but still, good to use one that doesn't break the games physics ;)

hi bro thanks for the reply
I don't have a problem with steam
the controller is connected with steam, this problem starts when I download this file and I add it to the game file  https://github.com/CookiePLMonster/SilentPatchBully/releases
now when I open the game I go to the controller, there 2 sections, when I try to go to "xbox 360 controller section" to enable the controller I cant move to this section I don't know why that happen

I understand, but I explained how to fix it.
Please close the game, disconnect and reconnect the controller and make sure Steam recognises it has connected.  Then boot up the game, and use the keyboard keys to navigate to the section and turn it on. The patch doesn't change anything regarding controls to the game so it has nothing to do with that. This is a generalised Bully issue.


Theming2 touchups

🎯 Goal
Few small changes to make the UI look more consistent. Mostly affecting the thread header.

:tada: This PR is included in version 3.0.0-theming2.1 :tada:
The release is available on:

npm package (@theming2 dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: