Exporting best model

Hi, I am serving a  BERT model. My constraint is that I want to export the model with the best accuracy on the evaluation dataset. I want to do this automatically (using the BestExporter class from TF). However, the problem is that this repository uses the TPUEstimator API, which does not allow me to pass in this BestExporter object.
I tried to work around by passing in the estimator object in the BestExporter call to export(), but this somehow does not work. There is no error, but I get no output in the export_dir path that I give to my BestExporter object.
Can someone please help?

Check out issue #817

Thanks, I actually looked at @IvanUkhov's wrapper around BestExporter class, and called the export function on this separately (after the estimator.eval() call). This works well, and I can export the best model pretty easily. The default is to compare loss, which threw me off balance since I was expecting it to be accuracy 😬.
I guess the issue is closed for my use-case. It would be nice to have an official example or a documentation line regarding the inter-operability of the Exporter API with the Estimator API. Specifically when one does not use the train_and_eval call.

can you please share the changes you have made in the code to get the best model.


[WIP] Kill Fabritone

Switch from the custom fabritone to my custom baritone.
This causes many compile errors to be sorted out.

This has been done with in latest commit! Thanks a ton for helping me out with this.


Add hive-minio-availability healthcheck

Closes/fixes/resolves issue(s)?

closes #78

What was added/changed/fixed?

[rm] previous implementation of healthcheck (it was duplicate of hive-ready healthcheck)
[added] new inplementation
[updated] chagelog

Related issue(s)? [Optional]
Others [Optional]
Checklist (after created PR)

[X] Added reviewers
[X] Added assignee(s)
[X] Added label(s)
[ ] Added to project
[X] Added to milestone

works fine on the local machine.


Fixed some console errors and fixed the Header

Fixed some console errors and fixed the Header not changing color issue when dark mode is on.

@danishzayan can you review this pr and merge it?

@danishzayan can you review this pr and merge it?

Ok after reviewing i will merge your code


Plugin hapi-swagger (v11.1.0) missing dependency vision when running in typescript

Issue
Hello,
I am trying to set up hapi-swagger v11.1.0 in my typescript project where I get the following error
 err: {
      "type": "Error",
      "message": "Plugin hapi-swagger missing dependency vision",
      "stack":
          Error: Plugin hapi-swagger missing dependency vision
              at new module.exports ( project/node_modules/@hapi/hoek/lib/error.js:23:19)
              at Object.module.exports [as assert] ( project/node_modules/@hapi/hoek/lib/assert.js:20:11)
              at module.exports.internals.Core._validateDeps ( project/node_modules/@hapi/hapi/lib/core.js:348:22)
              at module.exports.internals.Core._initialize ( project/node_modules/@hapi/hapi/lib/core.js:320:14)
              at module.exports.internals.Core._start ( project/node_modules/@hapi/hapi/lib/core.js:240:24)
              at internals.Server.start ( project/node_modules/@hapi/hapi/lib/server.js:523:27)
              at Server.start ( project/dist/lib/server/server.js:40:31)
              at processTicksAndRejections (internal/process/task_queues.js:94:5)
              at async startServer ( project/dist/lib/index.js:12:5)
    }

Environment
v11.1.0

Package.json
"dependencies": {
    "@hapi/boom": "^7.4.3",
    "@hapi/hapi": "^18.3.2",
    "@hapi/inert": "^5.2.2",
    "hapi-swagger": "11.1.0",
    "@hapi/joi": "16.0.0",
    "@hapi/vision": "^6.0.0",
  },
  "devDependencies": {
    "@hapi/code": "^7.0.0",
    "@hapi/lab": "^21.0.0",
    "@types/chance": "^1.0.7",
    "@types/core-js": "^2.5.2",
    "@types/hapi__boom": "^7.4.1",
    "@types/hapi__code": "^5.3.0",
    "@types/hapi__hapi": "^18.2.6",
    "@types/hapi__inert": "^5.2.0",
    "@types/hapi__joi": "^15.0.4",
    "@types/hapi__lab": "^18.1.0",
    "@types/hapi__vision": "^5.5.1",
    "@types/node": "^13.7.0",
    "@types/node-fetch": "^2.5.0",
    "tslint": "^5.19.0",
    "typescript": "^3.6.4"
  }

Steps to Reproduce
Plugin registration

await server.register([
      {
        plugin: Inert
      },
      {
        plugin: Vision
      },
      {
        plugin: HapiSwagger,
        options: {}
      }
    ]);
  }

This doesn't appear to be related to hapi-swagger. If you could provide a sample project with reproducible demo I can take a look.

closing due to inactivity.


Lag

The game about 2 months ago was fine, but I don't know if it was an update or what but the game now I can't play it, 5 fps stable, I tried graphics settings but it doesn't change anything.

Likely something external to the game. Either way, this isn't a great place to report issues. Please open a discussion here https://github.com/ppy/osu/discussions if you can't figure your linux.


Handle cases where site creation fails in New-PnPSite

Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
Improves #4082
What is in this Pull Request ?
Handle cases where site creation fails in New-PnPSite by throwing a clearer exception.
The previous code would try to dereference returnedContext even when null, throwing a Object reference not set to an instance of an object. exception. This happens because Framework.Sites.SiteCollection.Create() returns null in some cases, like if the site url was not retrieved after enough tries.
I'm not sure if the message should be changed. At this point of the callstack we have no idea what went wrong during the creation and if the site was fully created and configured or not. It might be that there was an issue only when retrieving the SPO ClientContext but everything else worked fine.
In any case, anything should be better than an unhandled null exception 😃

hi @jackpoz , am going to close this PR , have refactored the teamsite creation code quite a lot which fixes quite a lot of issues that we have had related to some errors users have faced. It should cover most of the scenarios.


OSE Workshop Reworked

#rebuild

!rebuild

#rebuild

#rebuild

I've sent a message to Roverdude about Firespitter

This is waiting on Firespitter:
OSEWorkShopReworked 1.2.9 requires FirespitterCore but it is not listed in the index, or not available for your version of KSP.
I've sent messages to Roverdude about it

#6280

#rebuild

#rebuild

#rebuild


[Java] fmpp-maven-plugin generates files directly under target/generated-sources

Describe the bug, including details regarding any error messages, version, and platform.
fmpp-plugin-maven is used in arrow-vector module to generate source files before the compilation phase.  Those files are generated directly under target/generated-sources where they conflict with the target/generated-sources/annotations directory created by javac. Per convention each plugin generates files under its own directory to prevent risks of conflicts.
Although it doesn't cause a direct issue with the build, it may also confuse some IDEs (Eclipse and VSCode notably) which are detecting overlapping source directories
Component(s)
Java

This also tends to confuse IntelliJ which expects a subdirectory, and so it doesn't see the generated sources -> IDE complains about undefined symbols

Issue resolved by pull request 41788
https://github.com/apache/arrow/pull/41788


Error when running train.py file

Traceback (most recent call last):
File "train.py", line 104, in
train(args)
File "train.py", line 78, in train
model.fit(
File "/home/coplin/anaconda3/envs/tf/lib/python3.8/site-packages/keras/utils/traceback_utils.py", line 67, in error_handler
raise e.with_traceback(filtered_tb) from None
File "/home/coplin/anaconda3/envs/tf/lib/python3.8/site-packages/keras/engine/training.py", line 1420, in fit
raise ValueError('Unexpected result of train_function '
ValueError: Unexpected result of train_function (Empty logs). Please use Model.compile(..., run_eagerly=True), or tf.config.run_functions_eagerly(True) for more information of where went wrong, or file a issue/bug to tf.keras.

Can you please provide more information about your setup. Otherwise, it is not possible to resolve this problem.
Are you using the docker setup ? Which Tensorflow Version are you using ? Which command did you use for running train.py

I encountered the same issue. I used Docker to run the setup. Below are the environment configurations and commands used:
Package                      Version

absl-py                      1.0.0
astunparse                   1.6.3
cachetools                   5.1.0
certifi                      2019.11.28
chardet                      3.0.4
cloudpickle                  3.0.0
dbus-python                  1.2.16
decorator                    5.1.1
dm-tree                      0.1.8
easydict                     1.13
flatbuffers                  1.12
gast                         0.4.0
google-auth                  2.6.6
google-auth-oauthlib         0.4.6
google-pasta                 0.2.0
grpcio                       1.46.1
h5py                         3.6.0
idna                         2.8
importlib-metadata           4.11.3
keras                        2.9.0
Keras-Preprocessing          1.1.2
libclang                     14.0.1
Markdown                     3.3.7
numpy                        1.22.3
oauthlib                     3.2.0
opt-einsum                   3.3.0
packaging                    21.3
pip                          20.2.4
protobuf                     3.20.1
pyasn1                       0.4.8
pyasn1-modules               0.2.8
PyGObject                    3.36.0
pyparsing                    3.0.9
python-apt                   2.0.0+ubuntu<IP_ADDRESS>
requests                     2.22.0
requests-oauthlib            1.3.1
requests-unixsocket          0.2.0
rsa                          4.8
scipy                        1.10.1
setuptools                   62.2.0
six                          1.14.0
tensorboard                  2.9.0
tensorboard-data-server      0.6.1
tensorboard-plugin-wit       1.8.1
tensorflow                   2.9.0
tensorflow-compression       2.9.0
tensorflow-estimator         2.9.0
tensorflow-io-gcs-filesystem 0.25.0
tensorflow-probability       0.17.0
termcolor                    1.1.0
typing-extensions            4.2.0
urllib3                      1.25.8
Werkzeug                     2.1.2
wheel                        0.34.2
wrapt                        1.14.1
zipp                         3.8.0
python3 ./train.py
--train_data_dir="demo_samples/training"
--val_data_dir="demo_samples/validation"
--train_output_dir="output"
--model=additive_lstm_demo

Can you double check that the dataset loaded correctly and placed at the correct location? And double check that all pathes are correct. The error usually arises when the dataloader is empty and not training is conducted.

Are you solved this problem?

Insufficient training data may be the cause of this error


Transparent NavBar Color after scroll down

I have the transparent navbar currently selected and when I scroll down it turns into an orange navbar. I'd like the navbar to totally disappear after you scroll past 600px. Please help.

Hello @nyimht, thank you for using our products, you have to create your own style to hide the navbar on scroll; please check out this example and try to reproduce it and integrate into your project.
Best,
Stefan


[tool_crash] ShaderCompilerException

Command
flutter run

Steps to Reproduce

Changeto Flutter dev channel
Create Flutter project for Linux
Flutter run, option chrome

Logs
ShaderCompilerException
#0      ShaderCompiler.compileShader (package:flutter_tools/src/build_system/targets/shader_compiler.dart:189:9)
<asynchronous suspension>
#1      writeBundle.<anonymous closure> (package:flutter_tools/src/bundle_builder.dart:185:25)
<asynchronous suspension>
#2      Future.wait.<anonymous closure> (dart:async/future.dart:522:21)
<asynchronous suspension>
#3      writeBundle (package:flutter_tools/src/bundle_builder.dart:163:3)
<asynchronous suspension>
#4      WebDevFS.update (package:flutter_tools/src/isolated/devfs_web.dart:866:9)
<asynchronous suspension>
#5      ResidentWebRunner._updateDevFS (package:flutter_tools/src/isolated/resident_web_runner.dart:489:35)
<asynchronous suspension>
#6      ResidentWebRunner.run.<anonymous closure> (package:flutter_tools/src/isolated/resident_web_runner.dart:283:41)
<asynchronous suspension>
#7      asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:24)
<asynchronous suspension>

[✓] Flutter (Channel dev, 3.4.0-17.2.pre, on Kali GNU/Linux Rolling <IP_ADDRESS>-microsoft-standard-WSL2, locale en_US.UTF-8)
    • Flutter version 3.4.0-17.2.pre on channel dev at /home/admin/Documents/flutter/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d6260f127f (7 days ago), 2022-09-21 13:33:49 -0500
    • Engine revision 3950c6140a
    • Dart version 2.19.0 (build 2.19.0-146.2.beta)
    • DevTools version 2.16.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /home/admin/Android/Sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /usr/local/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Debian clang version 14.0.6-2
    • cmake version 3.24.1
    • ninja version 1.11.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2021.3)
    • Android Studio at /usr/local/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Kali GNU/Linux Rolling <IP_ADDRESS>-microsoft-standard-WSL2
    • Chrome (web)    • chrome • web-javascript • Google Chrome 106.0.5249.61

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: e3c29ec00c9c825c891d75054c63fcc46454dca1
Plugins
path_provider_linux-2.1.7
path_provider_windows-2.1.3
shared_preferences-2.0.15
shared_preferences_android-2.0.13
shared_preferences_ios-2.1.1
shared_preferences_linux-2.1.1
shared_preferences_macos-2.0.4
shared_preferences_web-2.0.4
shared_preferences_windows-2.1.1

Closing as duplicate of https://github.com/flutter/flutter/issues/107590


Having a Custom Module Emit a Signal

I'm finishing up my HTTPS module and would like for the class to emit a signal once the program has processed the HTTP response. How exactly does one go about using Godot's emit signal system in a custom module?

Great to hear about your work on the module, but I think with this kind of question you should better ask at Godot's forum or developers IRC chanel  #godotengine-devel at freenode.net
Github is used for reporting bugs and feature requests.

@freemanfromgodotengine Thanks for pointing that out. I'm not sure why I thought the forums got removed.....

more reason to work on an asset sharing site for 2.1, so everyone can find
your work more easily
On Tue, Feb 2, 2016 at 1:29 PM, Freeman<EMAIL_ADDRESS>wrote:

Great to hear about your work on the module, but I think with this kind of
question you should better ask at Godot's forum
http://godotengine.org/projects/godot-engine/boards or developers IRC
chanel #godotengine-devel at freenode.net
Github is used for reporting bugs and feature requests.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3554#issuecomment-178672646.


How to contribute  python SDK / C++ SDK

as title

as title

If you have one, I want to make test it. Thank you

as title

If you have one, I want to make test it. Thank you

https://github.com/syp25815/bpx-api-py


allow select/copy of emails from "Most received from" BarChart

Hi,
It's currently impossible to select/copy emails from "Most received from" BarChart.
I have an old and huge mailbox I am cleaning up and I am going through biggest count emails first.
ThirdStats is great to identify those emails but I have to type them by hand every time in the search field.
Thanks.

Hi there,
first: when giving a bug report, please use the corresponding issue template. It's there to prevent basic questions to narrow down the problem.
By default, ThirdStats uses a cache to quickly display already processed data. So after making changes, you need to reprocess the data with the refresh button. There is (at the moment) no incremental update to the stats data.

Haha yes, makes sense now. That's why it's so important to use the issue templates here, totally misinterpreted your description.
I'm using Chart.js to generate the charts, I don't know a way to copy text from the generated canvas, but I'll take a look at possible workarounds. I'm sure I can find something.


Add list of tested model numbers

Add list of tested Synology models this script is tested to run on.

Tested models:
DS212j-j
RS18017xs+


Update locate docs

Closes #192 and #190

[x] update README.md

[x] add locate section with image
[x] add pulp to requirements
[x] add links to example notebooks

[x] docs/

[x] index.rst
[x] _static/references.bib (add entry from all citation in notebook)

[ ] add Examples sections to docstrings where appropriate

Yay, I'll do that very soon.


nz-popover Unclear deprecated message

Reproduction link
https://stackblitz.com/edit/angular-rf7vbl
Steps to reproduce
Use nzVisible on nz-popover
<button nz-button nz-popover nzType="primary" nzPopoverTitle="Title" nzPopoverContent="Content" [nzVisible]="true">
  Hover me
</button>

What is expected?
The deprecated message could be more clear. Either it should detect that it's nz-popover that's being used and suggest nzPopoverVisible or the message could simple say:

Please use 'nzTooltipVisible', 'nzPopoverVisible' or 'nzPopconfirmVisible' instead.

What is actually happening?
Will give this deprecation warning

deprecated: 'nzVisible' of 'nz-tooltip' is deprecated and will be removed in 10.0.0.
Please use 'nzTooltipVisible' instead. The same with 'nz-popover' and 'nz-popconfirm'.

Following the instructions in the deprecated message will result in code that isn't working.

Environment
Info

ng-zorro-antd
9.3.0

Browser
latest

In version 10.0 we have removed all the deprecation info of Popover (Tooltip & Popconfirm as well). So I am going to close this issue. Thanks for concerning. :hug:


spring-cloud-sleuth-dependencies:1.2.3.RELEASE refers to SNAPSHOT parent

POM of spring-cloud-sleuth-dependencies:1.2.3.RELEASE contains the following (unexpected) parent declaration:
<parent>
  <artifactId>spring-cloud-dependencies-parent</artifactId>
  <groupId>org.springframework.cloud</groupId>
  <version>1.3.3.BUILD-SNAPSHOT</version>
  <relativePath/>
</parent>

Yeah, we had a small glitch ;) check this version https://repo.maven.apache.org/maven2/org/springframework/cloud/spring-cloud-sleuth-dependencies/1.2.4.RELEASE/spring-cloud-sleuth-dependencies-1.2.4.RELEASE.pom

Oh great!
Sorry, did not notice version 1.2.4.RELEASE already exists :(

No problem, thanks for keeping your eyes on these things :)


unix,fs: use uv__load_relaxed and uv__store_relaxed

Rebased to resolve the conflicts. Could this please get a review? Thanks.

cc @bnoordhuis


Adoption Request: Silicon Economy EDC

Adoption Request
General information
Hello,
within the Silicon Economy project, we are using the EDC to securely connect platform services.
Currently, our EDC development is in the early stages. However, further progress is being planned as we integrate the EDC into more projects.
Our repository contains run-configurations and deployment examples for:

separated control and data plane
postgreSQL-based persistence
HTTP data exchange
docker compose
helm charts (OpenShift)
postman collection for all API interaction (provider and consumer)

Title
Description
Contact
Links

Silicon Economy EDC
The Silicon Economy EDC is a configured version from the Connector of the Eclipse Dataspace Components (EDC). It is used and specialized to easily integrate Silicon Economy components with the IDS.
<EMAIL_ADDRESS>Link to repository

Adoption level
Next, please tell us what level of adoption you intend to provide. (pick only one)

[X] Reference a feature as "friend"
[ ] Incorporate a feature as core EDC component

Add new row to known_friends.md.

Sorry for the huge delay!


Updated graph-cli and graph-ts versions

Description
Updated to graph-cli version to 0.87.0 and graph-ts version to 0.35.1
Your ENS/address: siddhantk08.eth

Thanks @siddhant-k08! Maybe a good idea might be creating a PR to create-eth-extensions repo, since we are no longer maintaining this branch and npx create-eth@latest with subgraph gives latest changes.

Hey @technophile-04 , sorry I found that out after already creating a PR. But I've already created a similar PR to the create-eth-extensions branch too


README: Clarify and make more readable information about the path at which binaries can be found.

/cc @liminzhu

Looks good :D


update pre-commit config

Codecov Report

Merging #11 (e87b604) into master (5a9ef7d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #11   +/-   ##
=======================================
  Coverage   83.47%   83.47%
=======================================
  Files           4        4
  Lines         115      115
=======================================
  Hits           96       96
  Misses         19       19

Flag
Coverage Δ

py3.6
?

py3.7
?

py3.8
83.47% <ø> (ø)

py3.9
?

pypy3
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Bump to NVML version 11.7

There is another issue in that go tool cgo -godefs does not handle anonymous nested structs and one was introduced in nvml.h for this version.

Updated to handle addition of anonymous structs in nvml.h


How can I style the text of a specific node rather than the whole text of a flowchart?

How can I style the text of a specific node rather than the whole text of a flowchart?  Thank you in advance.

You can add a class like this:
https://mermaidjs.github.io/mermaid-live-editor/#/edit/eyJjb2RlIjoiZ3JhcGggVERcbkFbQ2hyaXN0bWFzXSAtLT58R2V0IG1vbmV5fCBCKEdvIHNob3BwaW5nKVxuQiAtLT4gQ3tMZXQgbWUgdGhpbmt9XG5DIC0tPnxPbmV8IERbTGFwdG9wXVxuQyAtLT58VHdvfCBFW2lQaG9uZV1cbkMgLS0-fFRocmVlfCBGW2ZhOmZhLWNhciBDYXJdXG5jbGFzc0RlZiB0ZXN0IGZpbGw6bGlnaHRibHVlLGNvbG9yOmdyZWVuLHN0cm9rZTojRkY5RTJDXG5jbGFzcyBBIHRlc3QiLCJtZXJtYWlkIjp7InRoZW1lIjoiZGVmYXVsdCJ9fQ
Here you can find the man

You can add a class like this:
https://mermaidjs.github.io/mermaid-live-editor/#/edit/eyJjb2RlIjoiZ3JhcGggVERcbkFbQ2hyaXN0bWFzXSAtLT58R2V0IG1vbmV5fCBCKEdvIHNob3BwaW5nKVxuQiAtLT4gQ3tMZXQgbWUgdGhpbmt9XG5DIC0tPnxPbmV8IERbTGFwdG9wXVxuQyAtLT58VHdvfCBFW2lQaG9uZV1cbkMgLS0-fFRocmVlfCBGW2ZhOmZhLWNhciBDYXJdXG5jbGFzc0RlZiB0ZXN0IGZpbGw6bGlnaHRibHVlLGNvbG9yOmdyZWVuLHN0cm9rZTojRkY5RTJDXG5jbGFzcyBBIHRlc3QiLCJtZXJtYWlkIjp7InRoZW1lIjoiZGVmYXVsdCJ9fQ
Here you can find the man

Thank you for the reply.  But your sample shows how to style a specific node rather than the text of that node, such as the text font or text size.  I'd like to find a way to style the text of a node specially which will have different font, text size, text color from the text from other nodes.

All you have to do is to set the style for text as already shown above.  See here:
https://mermaidjs.github.io/mermaid-live-editor/#/edit/eyJjb2RlIjoiZ3JhcGggVERcbkFbQ2hyaXN0bWFzXSAtLT58R2V0IG1vbmV5fCBCKEdvIHNob3BwaW5nKVxuQiAtLT4gQ3tMZXQgbWUgdGhpbmt9XG5DIC0tPnxPbmV8IERbTGFwdG9wXVxuQyAtLT58VHdvfCBFW2lQaG9uZV1cbkMgLS0-fFRocmVlfCBGW2ZhOmZhLWNhciBDYXJdXG5cbmNsYXNzRGVmIHRlc3QgY29sb3I6Z3JlZW4sZm9udC1mYW1pbHk6YXJpYWwsZm9udC1zaXplOjEwcHhcbmNsYXNzIEEgdGVzdCIsIm1lcm1haWQiOnsidGhlbWUiOiJkZWZhdWx0In19

Thank you jgreywolf for you reply！


dev-example-python-agent

A fix for the example python agent. Creation of an account in the Dockerfile was not automatically creating the associated home directory, so a step has been added to create it manually.

Fix has been tested by @jb2197 and confirmed as working.


Remove all global log invocations

Fixes #21

Is putting a logger field into most structs, and adding an extra arg to the initialisation methods really the best way to go about this?
Most things have access to router.Ourself, so could just use the logger in that.
Though ironically with my changes in #33 and the changes in here, I think LocalPeer isn't actually doing any logging itself anymore - it only needs a logger in order to feed it into the LocalConnection constructor.

I elected to make the minimum changes necessary. I am happy to refactor further. May I delete methods like remoteConnection.log, or is that prefix useful/necessary?

#33 gets rid of remoteConnection.log.

It would be easier to integrate this with other schemes if it took an interface rather than a concrete *log.Logger.

Agreed. https://github.com/weaveworks/mesh/issues/37


Impington Lane street in missing Impington "location"

We've received this:

I live in Impington Lane (there’s only one in the UK, as far as I am aware) - which the site gives, correctly, as either CB24 9NJ or CB24 9LT.
However, what doesn’t get picked up is the settlement/location as Impington - it’s just given as Cambridge (which is the correct post town). OS Locator (which you indicate as a source) gives settlement/locality information - but you appear to be discarding this.
Doesn’t matter for Impington Lane, but for other local roads such as New Road, High Street (that’s in Histon), Station Road etc the loss of settlement/locality appears to be a critical omission.

I've tested the following address in Impington Lane's neighbour against our Sorting Office:
79 Impington Lane
Impington
CAMBRIDGE
CB24 9NJ

... and I get what looks like a correct parse of the address:
SAON:
PAON: 79
Street: Impington Lane
Locality: Impington
Town: Cambridge
Postcode: CB24 9NJ

I believe I need to know the exact address the user started from to investigate further. Anybody can offer a real address in Impington Lane that fails parsing?


Candlestick Chart

Wrapper to create can candlestick charts

use Plotly fill-to parameter

added with https://github.com/plotly/python-api/commit/cd0dff83db7a2e3ece86414a19db3ab4135f93a8


Layer count mismatch

tried the example in notebook folder
(20, 3)
is_human                                              heavy                                              light
0         1  QVQ-LVQSGA-EVNKPGASVKVSCKASGYTF---------TGYVVH...  EIV-LTQSPVTLSLSPGERATLSCRASQSV-----------SFYLA...
1         1  QVQ-LVQSGA-EVKKPGASVKVSCKASGYTF---------TGYYMH...  SYE-LTQPPS-VSVSPGQTARITCSGDALP-----------KQYAY...
Traceback (most recent call last):
File "C:\Users\work\antibody\Humatch-master\testClassify.py", line 25, in
cnn_heavy = load_cnn(os.path.join(weights_dir, "heavy.weights.h5"), "heavy")
File "C:\Users\work\antibody\Humatch-master\Humatch\model.py", line 130, in load_cnn
CNN.load_weights(weights)
File "C:\Users\miniconda3\envs\gpu9\lib\site-packages\keras\utils\traceback_utils.py", line 70, in error_handler
raise e.with_traceback(filtered_tb) from None
File "C:\Users\miniconda3\envs\gpu9\lib\site-packages\keras\saving\hdf5_format.py", line 817, in load_weights_from_hdf5_group
raise ValueError(
ValueError: Layer count mismatch when loading weights from file. Model expected 3 layers, found 0 saved layers.

Hi, I just tried deleting everything, reinstalling, and running the notebooks and it seemed to work OK for me (the weights download automatically). Could you please provide a bit more information:

Which notebook did you try to run?
Which cell did the notebook fail at?
How did you set up your environment? (It looks like you're perhaps using conda and not venv)
How are you running your notebook? (I am using VSCode and running on Linux Fedora 38)
Did you create the testClassify.py file and what does this look like? (It seems like this is being called too)

If you do need to use a different set-up to that described in the README (especially using conda and not venv) then you could try manually downloading the weights as described in the README and trying again

I had the same error and fixed it by upgrading Tensorflow to 2.17. Then I got a different error, something like:
TypeError: generator yielded an element that did not match the expected structure. The expected structure was (tf.float64,), but the yielded element was [[[-0.47  0.24  0.07 ...
This I fixed by changing the way the data generator is used inside the classify script, for the predict_from_list_of_seq_strs function.
Instead of
return model.predict(test_generator, verbose=CNN_verbose)
I used:
return model.predict(next(iter(test_generator)), batch_size=batch_size, verbose=CNN_verbose)
(Remember to "pip install ." after changing code)
I other words, I parse the data itself, not the generator. I seems like the custom generator isn't compatible with Tensorflow 2.17 at least. Hope it makes sense. So far it works, but might not be the case always. It took me a long time to figure out.
@lewis-chinery Can you please share the exact package versions you have?

Closing after fix suggested in issue TypeError: generator yielded an element that did not match the expected structure. #3 solved the above.


cvpr2021 model training error

Hello,
With similar to the https://github.com/autonomousvision/transfuser/issues/158#issue-1679710037 issue, I am trying to run the model in HPC and facing the error in loading .npy file for various models,
Transfuser model: (in Town03)

Geometric fusion: (in Town03)

cilrs: (in Town04)

I am using Minimal Dataset to train the model. I also changed the root_dir variable to a local directory in config.py for all the models.
Please help me to resolve the issue.
Thanks.

This generally occurs if the file you are trying to open and load is empty, can you check if the file exists and try to load it independently


Add notification for a weekly recap

Kind of like how spotify does a yearly "spotify wrapped" of all the music you've been listening to, let's build a weekly recap of the projects you've been working on. Let's have the notification take them to a page for this weekly recap information.
The goal for the notification is to help the user get value out of the work they are tracking. The app works so frictionlessly, that the individual can forget they are getting all this good information in about themselves. The notification provides a great point for them to re-engage with the value they are creating for themselves

Blocked until #236 is completed


There is not hippiehug >= 0.1.1 in pypi, required in setup.py

It indeed is not pypi, it has to be installed manually from Github. Have you tried the instructions from README, or they did not work.
Right now I am working on fixing this by moving to a reimplementation of hippehug.

Ah, right, with the version in the requirements/base.txt (https://github.com/gdanezis/rousseau-chain@dc3cf26bc0281e9b01352316da2e655b0ed84f45&subdirectory=hippiehug-package), it can be installed, though not with the version in setup.py (hippiehug >= 0.1.1)

azul and me just released hippiehug-0.1.3 and also there is now a claimchain-0.3.1 which requires it.


Misc fixes

don't ship the bytecode in git and made the Ratmen stop shooting Rouge bolts

Ohhhh, thank you!


How can I show SVG file on React Native?

I want to show svg files (i have bunch of svg images) but the thing i couldn't find the way to show. I tried to use Image and Use components of react-native-svg but they don't work with that. And i tried to do that with native way but it's really hard work to show just svg image.
Example code:
import Svg, {
  Use,
  Image,
} from 'react-native-svg';

<View>
  <Svg width="80" height="80">
     <Image href={require('./svg/1f604.svg')} />
  </SvgRn>
</View>

i know react native doesn't support svg basically but i think someone fix this problem with tricky way (with/without react-native-svg)
Also i created the topic on stack overflow.
http://stackoverflow.com/questions/38830568/how-can-i-show-svg-file-on-react-native

I also pretty much have the same question. Can I break the question in two though:
If I have the SVG as a string <svg><circle>..., can I render that using this library?
Then how to require an SVG. (I think this is quite hard, and just straight doesn't work in React Native right now).

@richardgill i can help for 2nd question 😄
If you add the images (or svgs) to Images.xcassets (on xcode) or drawable folder (on android), you can access to images that you want easily.

Hello, i made this library react-native-svg-uri.
You can render SVG Images from an URL or a static file.

You could also just simply convert the svg to json, then require the json through react native.

@Introvertuous Can you show example on how to get SVG file to you react native app by converting svg to json? :)

The idea is just to break down your svg into its individual paths and then each path would be a property within the json object.
{
"path":"M41.55-23.70Q41.55-18.18 40.17-13.62Q38.78-9.07 36.16-5.78Q33.54-2.50 29.97-0.77Q26.40 0.95 22.04 0.95Q16.10 0.95 11.76-2.34Q7.42-5.63 5.12-11.25Q2.81-16.88 2.81-23.48Q2.81-29.95 5.22-35.67Q7.63-41.38 12.20-44.84Q16.77-48.30 22.68-48.30Q28.13-48.30 32.43-45.32Q36.74-42.33 39.15-36.70Q41.55-31.08 41.55-23.70M22.04-2.53Q26.05-2.53 28.65-5.77Q31.25-9 32.43-14.26Q33.61-19.51 33.61-25.49Q33.61-29.88 32.92-33.43Q32.24-36.98 30.90-39.52Q29.57-42.05 27.54-43.45Q25.52-44.86 22.89-44.86Q19.27-44.86 16.79-42.56Q14.31-40.25 12.99-36.63Q11.67-33.01 11.20-29.30Q10.72-25.59 10.72-22.11Q10.72-17.75 11.71-13.31Q12.69-8.86 15.22-5.70Q17.75-2.53 22.04-2.53Z"
}

Then you can simply import the json into your app and use react-native-svg to render the svg like this:
  import json from './json'

  render() {
    return (
      <Svg height={200} width={200} >
        <Path d={json['path']} />
      </Svg>
    );
  }

@Introvertuous Then you should check this project out https://github.com/matc4/react-native-svg-uri because it looks like it is doing same kind of stuff like what you are suggesting. Maybe you should join that project? :)

Yeah that is definitely a better solution than what I was proposing.

Just as another option, you can create icon set from your svgs using IcoMoon, and then use it through https://github.com/oblador/react-native-vector-icons

@Sly777 iOS doesn't support SVG assets.

Watch out if you target Android, from the README for react-native-svg-uri:

[ANDROID] There is a problem with static SVG file on Android, Works OK in debug mode but fails to load the file in release mode. At the moment the only workaround is to pass the svg content in the svgXmlData prop.

What I do is I use svg-to-react-cli and then tweak the output, saves some time, but it's still not optimal.

@matc4 can I use this lib with Expo? Expo has react-native-svg inside, and exports Svg from expo. Will it work?

Use can use https://github.com/target/react-native-svg-parser instead.

use this one ,is very simple and works fine https://github.com/seekshiva/react-native-remote-svg

I created a tool to do this, if you have a bit of patience and don't mind running a Python script:
https://github.com/SM-Wistful/react-native-svg
It's mostly for personal use, but let me know if it seems helpful?

@SM-Wistful Thank you for your script, but in my case, it's not enough, a lot of SVG attributes are not supported and my svg files are not displayed correctly... Too bad svg imports are not supported by react-native...

@lucasriondel Check here: https://github.com/react-native-community/react-native-svg/issues/753

You could use IcoMoon to get JSON with path from your SVG. And then display it with the react-native-svg:
    <Svg width={32} height={32} viewBox="0 0 1024 1024">
      <Svg.Path fill="black" d={path} />
    </Svg>

Release mode in android requires a bit of extra work:
App.js
import * as React from 'react';
import SvgUri from 'react-native-svg-uri';
import svgXmlData from './test.svg';

export default () => (
  <SvgUri
    width="200"
    height="200"
    svgXmlData={svgXmlData}
  />
);

.babelrc
{
  "presets": ["module:metro-react-native-babel-preset"],
  "plugins": [
    ["babel-plugin-inline-import", {
      "extensions": [
        ".svg"
      ]
    }]
  ]
}

@matc4 Perhaps should update the documentation?

it's not working well with mine now.

As getting this issue for now

This is IOS Simulator screen

This is How I implement your solution
import imgSrc from '../../assets/images/background.svg';

<SvgUri
      svgXmlData={imgSrc}
    />

This is How I create .babelrc file like you make
{
  "presets": ["module:metro-react-native-babel-preset"],
  "plugins": [
    ["babel-plugin-inline-import", {
      "extensions": [
        ".svg"
      ]
    }]
  ]
}

Updated

Installed metro-react-native-babel-preset already.

Let's advise

@lunvjp What versions are you using?

 React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (4) x64 Intel(R) Core(TM) i7-5557U CPU @ 3.10GHz
      Memory: 2.46 GB / 16.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 10.15.0 - /usr/local/bin/node
      Yarn: 1.12.3 - /usr/local/bin/yarn
      npm: 6.5.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 24, 25, 26, 27, 28
        Build Tools: 23.0.1, 23.0.3, 25.0.2, 25.0.3, 26.0.2, 26.0.3, 27.0.3, 28.0.2, 28.0.3
        System Images: android-22 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom, android-24 | Google APIs Intel x86 Atom, android-24 | Google Play Intel x86 Atom, android-26 | Google APIs Intel x86 Atom, android-27 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom
    IDEs:
      Android Studio: 3.1 AI-173.4907809
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.58.6 => 0.58.6
    npmGlobalPackages:
      create-react-native-app: 1.0.0
      react-native-cli: 2.0.1
      react-native-scripts: 2.0.1

This is my info.
And all is good with me until now. I used this package react-native-svg-transformer and it worked!

Please refer to my Medium article on a step by step guide on how to add SVG icon to your Expo app.
https://medium.com/@henni1989younes/add-custom-svg-icons-to-your-expo-app-279b492f6a15

@lunvjp i have the same problem, and i figure it out, the config in metro.config.js has an item called inlineRequires, it is false, set it to true and done.
module.exports = {
  transformer: {
    getTransformOptions: async () => ({
      transform: {
        experimentalImportSupport: false,
        inlineRequires: true,
      },
    }),
  },
};

Docs have been updated, react-native-svg includes this functionality now: Use with content loaded from uri
import * as React from 'react';
import { SvgUri } from 'react-native-svg';

export default () => (
  <SvgUri
    width="100%"
    height="100%"
    uri="http://thenewcode.com/assets/images/thumbnails/homer-simpson.svg"
  />
);

Use with xml strings
import * as React from 'react';
import { SvgXml } from 'react-native-svg';

const xml = `
  <svg width="32" height="32" viewBox="0 0 32 32">
    <path
      fill-rule="evenodd"
      clip-rule="evenodd"
      fill="url(#gradient)"
      d="M4 0C1.79086 0 0 1.79086 0 4V28C0 30.2091 1.79086 32 4 32H28C30.2091 32 32 30.2091 32 28V4C32 1.79086 30.2091 0 28 0H4ZM17 6C17 5.44772 17.4477 5 18 5H20C20.5523 5 21 5.44772 21 6V25C21 25.5523 20.5523 26 20 26H18C17.4477 26 17 25.5523 17 25V6ZM12 11C11.4477 11 11 11.4477 11 12V25C11 25.5523 11.4477 26 12 26H14C14.5523 26 15 25.5523 15 25V12C15 11.4477 14.5523 11 14 11H12ZM6 18C5.44772 18 5 18.4477 5 19V25C5 25.5523 5.44772 26 6 26H8C8.55228 26 9 25.5523 9 25V19C9 18.4477 8.55228 18 8 18H6ZM24 14C23.4477 14 23 14.4477 23 15V25C23 25.5523 23.4477 26 24 26H26C26.5523 26 27 25.5523 27 25V15C27 14.4477 26.5523 14 26 14H24Z"
    />
    <defs>
      <linearGradient
        id="gradient"
        x1="0"
        y1="0"
        x2="8.46631"
        y2="37.3364"
        gradient-units="userSpaceOnUse">
        <stop offset="0" stop-color="#FEA267" />
        <stop offset="1" stop-color="#E75A4C" />
      </linearGradient>
    </defs>
  </svg>
`;

export default () => <SvgXml xml={xml} width="100%" height="100%" />;

Also, released today in v9.13.0, SvgCss and SvgCssUri, same as SvgXml and SvgUri but with support for css style elements, with handling of the cascade, media queries, pseudo selectors, specificities, priorities, complex selectors etc:
import React from 'react';
import {SvgCss} from 'react-native-svg';

const xml = `<svg xmlns="http://www.w3.org/2000/svg"
  width="100%"
  height="100%"
  viewBox="0 0 1000 500"
>
  <defs>
    <style type="text/css">
      rect {
        fill: lightgreen;
        stroke: blue;
        stroke-width: 5;
      }
    </style>
  </defs>
  <rect x="200" y="100" width="600" height="300" />
</svg>
`;

const App = () => <SvgCss xml={xml} />;

export default App;

https://github.com/react-native-community/react-native-svg/blob/8c1a1737b052000569662d6f2539ffe3c47564ba/__tests__/css.test.tsx#L1-L54

import {SvgCss} from 'react-native-svg';

const xml = (obj) =>
  `<svg
    xmlns="http://www.w3.org/2000/svg"
    width="24"
    height="24"
    viewBox="0 0 24 24"
    fill="none"
    stroke="${obj.color}"
    stroke-width="2"
    stroke-linecap="round"
    stroke-linejoin="round"
    class="feather feather-film">
    <rect x="2" y="2" width="20" height="20" rx="2.18" ry="2.18"></rect>
    <line x1="7" y1="2" x2="7" y2="22"></line>
    <line x1="17" y1="2" x2="17" y2="22"></line>
    <line x1="2" y1="12" x2="22" y2="12"></line>
    <line x1="2" y1="7" x2="7" y2="7"></line>
    <line x1="2" y1="17" x2="7" y2="17"></line>
    <line x1="17" y1="17" x2="22" y2="17"></line>
    <line x1="17" y1="7" x2="22" y2="7"></line>
  </svg>
`
const App = () => {
    <SvgCss
         xml={xml( {color: 'tomato'} )}
     />
}


remove tests for v0

v0 is now deprecated and no longer being mantained. We can spare the maintenance of v0 tests if delete them.

Closed via https://github.com/authzed/authzed-go/pull/102/


Now, unstable feature works again !

The issue remains that the unstable features and rust_1_57 pose problems when used together...
Look at the file boxed.rs line 66: fn alloc(layout: Layout) -> Result<NonNull, TryReserveError> { ... } What do you put in the cfg if you want to add rust_1_57? No matter how I look at the problem, no solution seems logical... rust_1_57 relies on a partial use of stabilization...
So I've decided to ban the combined use of these two features.

Cool :)
Envoyé avec la messagerie sécurisée Proton Mail.
------- Original Message -------
Le mardi 13 juin 2023 à 1:30 PM, vcombey @.***> a écrit :

Merged #38 into master.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Session tags page uses non-standard ASCII dashes

On the page session tags, some of the tag CLI references use non-standard ASCII dashes which cause an error when copying and pasting locally.
For example in this block from the page;
aws sts assume-role \
--role-arn arn:aws:iam::123456789012:role/Role1 \
--role-session-name Session1 \
–-tags Key=Star,Value=1 Key=Heart,Value=1 \
--transitive-tag-keys Star Heart

The first dash on –-tags is actually ASCII character U+00E2 LATIN SMALL LETTER A WITH CIRCUMFLEX character (â) instead of U+002D HYPHEN-MINUS character (-).
When pasted into CLI this results in the following error:
usage: aws [options] <command> <subcommand> [<subcommand> ...] [parameters]
To see help text, you can run:

  aws help
  aws <command> help
  aws <command> <subcommand> help

Unknown options: Key=Star,Value=1, –-tags

Obviously ideally a user should retype commands for security but on occasion copy and paste is faster and easier.

Thanks for reporting this. I will fix this and publish an update as soon as possible. ETA week of 7/20 due to publishing cadence.

Thanks again for reporting this, the fix is live in the documentation.


Create AIF logo

AIF logo after voting: https://s3.amazonaws.com/aif2/logo/Screen+Shot+2015-01-02+at+5.38.02+PM.png


Avoid operator creation in transpiler

Summary
This removes very nearly all of the use of DAGOpNode.op in the default transpiler paths.  The sole exception is in InverseCancellation, which currently would involve some quite awkward gymnastics for little near-term benefit. The pass should move fully to Rust soon, making it not worth the effort.
Most of the tricks here involve using the knowledge that most operations will involve only Rust-space standard gates, and that these cannot be control-flow operations.

Details and comments
This always includes a reno for a couple of the new methods added to DAGOpNode and CircuitInstruction.
I measured this as being ~25% faster for some toy workloads at transpilation levels 0 and 1, probably mostly driven by the new methods letting us avoid a lot of isinstance checks in most cases.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

108 of 114 (94.74%)  changed or added relevant lines in 15 files are covered.
93 unchanged lines in 17 files lost coverage.
Overall coverage increased (+0.06%) to 89.814%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/synthesis/high_level_synthesis.py
25
26
96.15%

crates/circuit/src/circuit_instruction.rs
11
16
68.75%

Files with Coverage Reduction
New Missed Lines
%

qiskit/primitives/backend_sampler.py
1
98.86%

qiskit/primitives/base/base_sampler.py
1
96.97%

crates/accelerate/src/two_qubit_decompose.rs
1
90.61%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
2
88.39%

qiskit/primitives/base/base_estimator.py
2
94.59%

crates/qasm2/src/lex.rs
2
93.13%

qiskit/primitives/sampler.py
3
95.89%

qiskit/primitives/estimator.py
3
96.25%

qiskit/providers/fake_provider/generic_backend_v2.py
5
95.02%

qiskit/providers/fake_provider/fake_backend.py
7
87.32%

Totals

Change from base Build<PHONE_NUMBER>2:
0.06%

Covered Lines:
67037

Relevant Lines:
74640

💛  - Coveralls


MGraph Authentication Issue

I was trying MSGraph Plugin and ran into the following error.
In the device DeviceCodeCallback is the following message.
To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code BLZHA9C6L to authenticate.
Also i noticed is that chatCompletionService.GetStreamingChatMessageContentsAsync already return befor the above authentication is completed

I see the issue, you are invoking the prompt manual, weel i am using AutoInvoke. Can it work with auto Invoke?
using string prompt = "What is my next meeting?";
var results = kernel.InvokePromptStreamingAsync(prompt, new KernelArguments(settings));
await foreach (var message in results)
{
Console.Write(message);
}

Hello @nssidhu  I truly apologize but I totally missed the notification on this issue. Since it's very old, can you let me know if you're still blocked?


[Bug] UI/UX - Irrelevant field for family when the user is a Company

(UI/UX) Irrelevant field if the user register as a Company in the Support Form
To Reproduce 5 of 5 times
Steps to reproduce the behavior:

Go to dev.podkrepi.bg/support
Click on "Компания"
Click on "Напред" button
Choose either "Компания Дарител" or "Компания Доброволец"

Expected behavior
To see the field for Company name, not
Needed field for "Family"
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Windows 10, OS build 19042.1348
Pc Acer
Mobile device
Huawei Y7 2019

OS: Android
Browser [e.g. chrome)
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

From my perspective, a human being is creating the submission no matter they sign up for company supporter.
We need to have the first and last name of the person that created the submission.
I agree we might add another new field so they enter "Company name", beside the person names.


[performance_benchmark] needs reworking

https://docs.hydradx.io/performance_benchmark
is out of date

Now even more so

hi guys, can you please indicate what changed?

@vgantchev  the bench-wizard should be autoinstalled now, the output is different than what's displayed.
So best would be to re-run the bench and replace everything


Conflicting Reports (local_variable_names & no_prefixes)

As is visible we have conflicting rules OOTB:

Local variable name does not match pattern ^L._.+$: path

Avoid hungarian notation

This is based on the standard abaplint.json delivered with the VS Code extension.
{
"local_variable_names": {
      "exclude": [],
      "severity": "Error",
      "patternKind": "required",
      "ignoreNames": [],
      "ignorePatterns": [],
      "expectedData": "^L._.+$",
      "expectedConstant": "^LC_.+$",
      "expectedFS": "^<L._.+>$"
    },
    "no_prefixes": {
      "exclude": [],
      "severity": "Error",
      "data": "^[LGM].?_",
      "statics": "^S.?_",
      "fieldSymbols": "^<[LGM].?_",
      "constants": "^[LGM]?C.?_",
      "types": "^TY_",
      "methodParameters": "^[ICER].?_",
      "allowIsPrefixBoolean": true
    }
}

in the default configuration, everything is enabled to allow easy discovery, it does not reflect recommended best practices
in your team/company/project you have to decide what is good and bad, and enable/disable the corresponding rules

I would expect the default config to give a green light to (some standard of) good code but OK...

alternatively, take the configs from https://github.com/abaplint/abaplint/blob/main/docs/getting_started.md and enable rules instead of disabling


Add dev docs for folks contributing to the cnf project

some docs to get started / debug

[x] add build info for tools
[x] add some examples of working with ansible inside of the tool container
[x] add curl command for packet api to view a device

What else do we need?

k8s and openstack items

K8s and openstack

[ ] Add stages to deploy k8s cluster
[ ] Add tips and debugging info for k8s cluster deployment
[ ] Add stages to deploy openstack cluster
[ ] Add tips and debugging info for openstack cluster deployment

Add note about facility with more resources for our test case


Inconsistent counter values in multi-threaded vector access using thread_pool

Hello,
I am encountering an issue with inconsistent counter values in a multi-threaded environment using your thread pool implementation. The expected behavior is for the counter to reach a consistent value of 204 after each complete run of the sim() function, which performs multiple enqueues to the thread pool. However, the actual counter values vary.
#include <iostream>
#include <vector>
#include "thread_pool.h"
using namespace std;
dp::thread_pool pool(24);

class Vect
{
public:
    std::vector<std::vector<double>> data;
    int id = -1;
    Vect(int id) : data(30, std::vector<double>()) { this->id = id; }
    std::mutex g_i_mutexx;
    int counter = 0;

    void addDataVec(int pos, double value)
    {
        std::lock_guard<std::mutex> lock(g_i_mutexx);
        data[pos].push_back(value);
        counter++;
    }

    std::vector<std::vector<double>> returnVectorSim()
    {
        return (data);
    }

    void clearVector(int num_vec)
    {
        data[num_vec].clear();
    }
};
void sim()
{

    Vect vect(1);

    for (size_t var1 = 0; var1 < 17; var1++)
    {
        for (int var2 = 0; var2 < 12; var2++)
        {
            pool.enqueue([&vect, var1, var2]() {
                vect.addDataVec(var1, var2);

                });
        }
    }
    pool.wait_for_tasks();

    std::cout << vect.counter << std::endl;
}

int main()
{
    for (size_t var = 0; var < 17; var++)
    {
        sim();
    }
    return 0;
}

Observed Counter Values:

The values observed were not consistent and varied across different runs (e.g., 204, 203, 202, 201).

Expected Behavior:

The counter should consistently reach 204 after each execution of sim().

Could someone help me understand what might be causing these inconsistencies?
Thank you!

This should be fixed now in #68


Link to Github in package.json

This will create a link from https://www.npmjs.com/package/steamworks.js to the Github repo.

Note that we do have to publish a new version to make the link appear.


HSEARCH-3058 Search 6 groundwork - Add generic type parameters to PropertyHandle

https://hibernate.atlassian.net/browse/HSEARCH-3058
Actually I'm not sure whether it is a good idea to make the method PojoPropertyModel#getHandle return a PropertyHandle<? super T> instead of just PropertyHandle<T>.
This was my first choice, since the field GenericContextAwarePojoPropertyMode#rawPropertyModel has a type PojoPropertyModel<? super T>. On the other hand, I'm not sure about the solution I applied.

Pull Request Test Coverage Report for Build 1

20 of 20 (100.0%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 88.596%

Totals

Change from base Build 204:
0.005%

Covered Lines:
16696

Relevant Lines:
18845

💛  - Coveralls

@yrodiere thanks for the nice explanation.
If I got it well, the 'rawTypeModel' instance cannot be any PojoRawTypeModel<? super T>, it must have the same raw type of the generic type model.
The only difference can reside in the generic of the generic 'T' type.
I think I've addressed your corrections here.
Thank you for the review too.

I fixed an oversight and renamed a class that had a typo in its name. Waiting for CI then I'll merge.

Thanks

Merged, thanks!


Character Head Bobbing

What is your question?
Does anyone know how to make the character's head beat faster without making the BPM higher than it needs to be?

Change setProperty('dad.danceEveryNumBeats',1) default is 2

Change setProperty('dad.danceEveryNumBeats',1) default is 2

thank u

I recomend make this in the onCreatePost()

I recomend make this in the onCreatePost()

ok


Multi threaded program is not debuggable properly

I was trying to debug the attached script. This script is written for practicing multi-threading. To view each & every details of how threading is being worked I wrote this script and started debugging. But the debugger is not been able to start all the thread. Before starting all the threads it starts executing the 1st thread's target function which it should not supposed to do
To reproduce the problem please re-run the following python script and put debugger as per the screenshot provided
Please note that PyCharm Community Edition is doing this debugging properly.
Program Details: Microsoft Visual Studio Community Edition 2019
Version 16.0.0
Python 16.0.19074.1
Python Base Version 3.7.2
==========================================================
import threading
import time

thread_lock = threading.Lock()
COUNT = 20
global_variable = 0

def inc_by_1():
    global global_variable
    with thread_lock:
        for i in range(COUNT):
            global_variable += 1

def inc_by_4():
    global global_variable
    with thread_lock:
        for i in range(COUNT):
            global_variable += 4

def dec_by_1():
    global global_variable
    with thread_lock:
        for i in range(COUNT):
            global_variable -= 1

def dec_by_2():
    global global_variable
    with thread_lock:
        for i in range(COUNT):
            global_variable -= 2

def dec_by_2_again():
    global global_variable
    with thread_lock:
        for i in range(COUNT):
            global_variable -= 2

t1 = threading.Thread(target=inc_by_1)
t2 = threading.Thread(target=inc_by_4)
t3 = threading.Thread(target=dec_by_1)
t4 = threading.Thread(target=dec_by_2)
t5 = threading.Thread(target=dec_by_2_again)

t1.start()
t2.start()
t3.start()
t4.start()
t5.start()

print("Calling from main thread::\nCurrent value of global_variable is::: {0}".
      format(global_variable))

=======================ooooooooo============================
I'm sorry for writing code in this way as default code integrator was not working for my code.

Hi Sajeeb
Thank you for the detailed report! It helped a lot.
In short, everything works as expected. Here's a couple things I want to mention:

This program has race conditions because the threads are never joining, so they could still be running when the print statement is executed. You can resolve this by adding "t1.join() t2.join()…" before the print statement to ensure that all the threads have finished executing before the print statement is executed.

PyCharm and PTVS use two different methods of debugging.

PyCharm debugs one thread at a time. It continues debugging a thread until it has finished executing and then moves onto another next thread and so forth.
VS debugs all the threads concurrently. The debugger switches between threads as they are executed and, every time a breakpoint is hit, all the threads are paused and the specific thread where a breakpoint was hit is debugged. When the debugger is continued, all threads begin and debugging continues concurrently another until breakpoint is hit, and so forth. This allows VS to implement true concurrent debugging which better reflects non-debugging programming execution. You can view which thread is currently being debugged by going to "Debug->Windows->Threads"

In PyCharm, you see the debugger executing the main thread, so you're able to see execution of "t1.start() t2start()…print()...end thread" and then it switches to another thread and so forth. In VS, the debugger hits the breakpoint at "t1.start()" in main thread and then switches to "inc_by_1" in a different thread ID (4 for example). Since this is multithreading, there's no guarantee that this order will be preserved, so it could hit breakpoint at "inc_by_4" before "inc_by_1".
Let me know if you have any other questions.

I'm extremely sorry for replying late.
Thank you very much for your kind attention. I understood your point of view. But I think I was unable to express my problem to the point. I think I should try again to express my problem.
The main problem is in VS2019 I have no control over the child threads.  Now I will select a child thread from thread queue and the target function listening to that thread will start executing. But here the problem started. The debugger freeze here and stop responding. To make is responsive again I need to pause the debugging and then resume that.
Where PyCharm does not freezes. When I select a thread function listening to that thread start executing while it was in the middle of execution of another function.
Is it happening due to differences in VS and PyCharm debugger?

Ah, okay. I see what you mean. This is indeed a bug.
Whenever the thread that is being debugged hits a case such as "thread_lock" and unable to proceed, VS will pause debugging on that thread. A user should be able to switch threads and continue debugging, but PTVS is not allowing the user to switch threads.
I'll look into fixing this. The current workaround is to press the pause button which resumes debugging the previous thread.
Thanks for reporting it!

You are most welcome. Again thanks for your support.


Content inconsistent from the compose-builder folder make build result

After running make build from the compose-builder folder, the generated content is inconsistent with the current docker-compose files.  In this case, people need to manually modify the files if they want to open PRs.
I create a draft PR to show the differences from my environment:  https://github.com/edgexfoundry/edgex-compose/pull/179/files
This issue might be caused by the different version of docker-compose
I am using
$ docker-compose -v
docker-compose version 1.27.4, build 40524192
$ docker version
Client: Docker Engine - Community
Version:           20.10.8
API version:       1.41
Go version:        go1.16.6
Git commit:        3967b7d
Built:             Fri Jul 30 19:54:27 2021
OS/Arch:           linux/amd64
Context:           default
Experimental:      true
Server: Docker Engine - Community
Engine:
Version:          20.10.8
API version:      1.41 (minimum version 1.12)
Go version:       go1.16.6
Git commit:       75249d8
Built:            Fri Jul 30 19:52:33 2021
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.4.9
GitCommit:        e25210fe30a0a703442421b0f60afac609f950a3
runc:
Version:          1.0.1
GitCommit:        v1.0.1-0-g4144b63
docker-init:
Version:          0.19.0
GitCommit:        de40ad0

@cloudxxx8 Yes, this is a know issue with the latest docker-compose. I have mentioned previously that we must use docker-compose 1.27.2 which generates the depends-on sections correctly. Based on my research they don't plan to fix this issue.

@lenny-intel should we update the compose-builder in Readme to indicate the recommended docker-compose version?

@lenny-intel should we update the compose-builder in Readme to indicate the recommended docker-compose version?

@cloudxxx8 , Agreeded.

@cloudxxx8 , README has been updated. Can we now close this issue and the related PR?

Sure, thanks


hyphenation support

Some Python static site generators (Acrylamid, Nikola,...) have support for hyphenating posts which can look really nicely - see e.g. http://blog.posativ.org/2014/gentoo/. The Acrylamid docs is here: http://posativ.org/acrylamid/filters.html?highlight=hyphen#hyphenate and e.g Nikola is using http://pyphen.org/.
I'm not aware what is the situation in Golang, but probably there is some decent hyphenation support as well.
Sincerely,
Gour

@natefinch I'd rather have it slow (ala Pygments), than no feature. ;)

Note/Update: This issue is marked as stale, and I may have said something earlier about "opening a thread on the discussion forum". Please don't.
If this is a bug and you can still reproduce this error on the latest release or the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still relevant and valuable, please tell us why.

This is a feature request and I still think it is relevant and valuable. As an example, please see how missing hyphenation makes the title of my blog post look rather ugly. (Please let me know, if I'm just missing the correct option somewhere, and sorry for the noise then.)

i second the idea that hyphenation support is still a relevant and valuable feature request. especially narrow(-ish) columns on small screens would benefit.
The main problem seems to be that there doesn't exist up-to-date offline soft hyphen &shy generation support (for go) - in particular, no currently maintained hunspell bindings exist.
but (for people arriving here and looking for a possible solution), browsers have caught up: have a look into the hyphens and the overflow-wrap CSS properties.


WIP - Metadata

Changes:

Moved to sourcing information from metadata service
No longer requires API Access Key + Secret
Simplified codebase to match the simpler metadata service data
Moved to constmetrics, dropped locking
Moved to multi-stage builds

Still to do:

Add back in health status for services & stacks, requires changes to metadata service
Debug logging, fix handling and passing around the go-common logging
Add back in state for hosts, requires changes to metadata service
Improve how we handle the rancher-metadata URL and version
Resolve 'built-in' metrics through the go-common router. Currently initialised but not appearing.
If possible, obtain possible states from rancher directly or from a standard library

@flock3 Added you as a reviewer, not yet finished but would appreciate any feedback


CardboardGUI refresh when loading a new scene

Hello,
I don't know if it is really a bug but in doubt, I post it. I have two scenes in my project, both are using the CardboardMain game object with a CardboardGUI attached to this object.
In my CardboardGUI script on each scene, I've registered the OnGUI callback to the CardboardGUI class. When the player is loading the second scene, the GUI rendered on the first scene is not cleaned. If I want to remove the first scene GUI, I have to manually remove the OnGUI callback from the CardboardGUI class before the Application.LoadLevel call. Is it normal ?

Yes, that is what you are supposed to do.  See the Cardboard Plugin Reference:
Code Changes to Scripts using OnGUI()
void Start() {
    CardboardGUI.onGUICallback += this.OnGUI;
    // ...
}

void OnDestroy() {
    CardboardGUI.onGUICallback -= this.OnGUI;
    // ...
}

void OnGUI() {
    if (!CardboardGUI.OKToDraw(this)) return;
    // ...
}

The reason it can't be automatic is that CardboardGUI does not know if your object will be used across level loads.  See:
http://docs.unity3d.com/ScriptReference/Object.DontDestroyOnLoad.html

Okay, I understand ! Sorry, I had not seen these lines in the configuration yet.
Thank you for your answer.


Docker for testing (second version)

This PR adds Docker support for testing. It supersedes PR #832.
The first commit adds Docker, and a Rake task for running the RSpec unit tests inside the container.
The second commit enables integration tests to run inside the container, adds the supporting code for testing generated backup archives, and provides one integration test (in integration/acceptance/archive_spec.rb).

I have rebased this PR on the current master branch, and updated it.
It provides Rake tasks and support files for running the RSpec unit tests and a skeleton integration suite on a development workstation. The integration test suite is in the integration/ directory. There willl be separate PRs to port additional integration tests from vagrant/ and enable Travis CI to use Docker.
Tested with a host system using macOS 10.12, Ruby 2.4.2 and Docker 17.09-ce-mac35.


WIP: haproxy to microservice

https://github.com/rancher/rancher/issues/2179

Would you mind giving encuraging UI options for https://github.com/rancher/rancher/issues/2179#issuecomment-203178448 ?

-1 from me.. The point of this is to make it possible to put in whatever config you want, while preserving 1st-class options for the common use cases.  None of those are really commonly used (attaining arbitrary scores on SSL test notwithstanding), and most of these could/should be set by the application.  HSTS and HPKP would be the most relevant and I could see an argument for those since one of the 1st class features is SSL termination.. but they are also particularly dangerous and can lock users out en-masse for max-age if misconfigured.

That's indeed a pack of diligent arguments.

I wanted to draw another use case, which is more closely related to the rancher stack (or might be in future): A lot of Catalogue stacks out there have poorly secured api endpoints or web client interfaces. Example: Graylog input api endpoint (isn't yet a catalogue item, I'm on it) needs to be exposed publicly for docker daemons being able to talk to them and there is not one grain of protection on it. Vendor suggests reverse proxy. My imagination is just about to go wild what would be possible with rancher's user management exposed to the load balancers or any kind of client side certificate integrations... :smile:

I'm not familiar with graylog in detail but I don't see why you'd have to publish it to a host port.. Any port the container is listening on is available on the 10.42.x.y address/DNS name of the container from inside the managed network, they don't need to be published to a host port to be used.
Using rancher's authentication to authorize a HTTP request would probably better suited to a nginx-based balancer where you can do lua scripting to talk to the Rancher API directly.  I don't think you can do much useful with what haproxy has built-in (an essentially hardcoded userlist with plaintext passwords, or hashed with algorithms we don't use for local auth.  In either case have no way to create that for any of the other external forms of auth).

I think the point here is that it is the docker daemon's logging driver which talks, and it just failed on that locally, so I assume there is no way to make the docker daemon itself (!) join the managed network.
I'm eagerly looking forward to witness advances in this field as they unfold. This PR is very exciting! As I understood it from here putting nginx in place was an afterthaught. :+1:

Have you any stance on GORB so far?
As I figured out, IoT applications are stuck on Rancher due to the lack of UDP support as for now.
https://github.com/kobolog/gorb
https://blog.codeship.com/kernel-load-balancing-for-docker-containers-using-ipvs/


Remote Provider User Preferences

Description
This PR adds a new struct MeshMapPreferences which aids the users to persist their meshmap preferences.
This PR fixes #
Notes for Reviewers
For testing please check other supporting PR on meshmap
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #5204 (39b960f) into master (0ef53a9) will decrease coverage by 7.77%.
The diff coverage is n/a.

:exclamation: Current head 39b960f differs from pull request most recent head ccaa6ee. Consider uploading reports for the commit ccaa6ee to get more accurate results

@@            Coverage Diff             @@
##           master    #5204      +/-   ##
==========================================
- Coverage   30.93%   23.15%   -7.78%
==========================================
  Files          63       63
  Lines        5441     5441
==========================================
- Hits         1683     1260     -423
- Misses       3339     3907     +568
+ Partials      419      274     -145

Flag
Coverage Δ

gointegrationtests
?

unittests
23.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-58.95%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.14% <0.00%> (-55.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.91% <0.00%> (-54.55%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.92% <0.00%> (-48.08%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.15% <0.00%> (-44.61%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.26% <0.00%> (-36.96%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.03% <0.00%> (-23.03%)
:arrow_down:

mesheryctl/internal/cli/root/system/dashboard.go
1.70% <0.00%> (-9.41%)
:arrow_down:

mesheryctl/pkg/utils/healthcheck.go
8.82% <0.00%> (-4.42%)
:arrow_down:

Continue to review full report at Codecov.

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

Closing this PR as these changes are not required anymore for persisting remote provider users' preferences. As of now we can use the existing extension point interface


Hook up mapped code edits experiment for TS

For https://github.com/microsoft/TypeScript/pull/55406
Requires using TS 5.4+ and also enabling a hidden setting: typescript.experimental.mappedCodeEdits.enabled
/cc @zkat @ulugbekna @jrieken

this seems in line with everything, yep!


Concept Insights: fix corpus id in createCorpus

Concept Insights API requires corpus id to be of the form /corpora/{account_id}/{corpus}. createCorpus() takes in a Corpus object, where an id is required. createCorpus() creates a URL for the request by concatenating /v2/corpora/{account_id} and the id from the Corpus object; currently this concatenation results in /v2/corpora/{account_id}/corpora/{account_id}/corpus.
Proposed solution: since Concept Insights API does not require an id for the corpus, allow Corpus.id to be null in createCorpus(). Add a name field to Corpus and use that when calling createCorpusIdPath() from createCorpus().

@alseddnm

@alseddnm is working on this.

@alseddnm did some work on this e447b8ccdb50f65f13e64253dbd79aa74e13e30b

fixed in the next release


Update release.yaml

use release branch for releases

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Client with a long uptime doesn't give the correct values

Issue Context
I have a machine that I rarely shut down (normally place into a sleep state). The up-time is around 7-9 weeks.
Cause of the issue
Because Environment.TickCount is an int, it is not capable of functioning after ~48-49 days. It is giving me a negative value for the days due to my up-time.
Solution
A functional alternative can be found here. It is a method that is very similar to the methods we use to obtain other information of the client's machine.

While on the topic of system information, I just wanted to point out this: perhaps the client should append each antivirus name that is not null or empty to a StringBuilder (using a newline after each valid name) before returning. Might be good to know if a client has multiple anti-viruses installed.

I'm aware of this. For me it's not even returning the correct value when I have my Win8.1 in VM running for 1h. This needs to be fixed asap.

Alrighty. :)

Can you tell me if it's working with your long-uptime system? My longest uptime is just 35 days.^^


How do I exclude namespaced classes that should very much never be in the main nav, from the main nav?

I have an Errors module which exports about 30 class Something extends Error that lead to the worst documentation pollution if they're included in the "classes" section in of the nav:

These classes should only ever show up on the Errrors namespace page, so: what (combination of) keywords make that happen?
I've been trying the most ridiculous combinations just in the hopes that it'll trick jsdoc into doing the right thing, but nothing seems to work, and have the docs be a hot mess of LOTS_OF_ERROR_CLASS_NAMES that make it impossible to see which actual classes exist is not great.

Have you tried using the @private tag? According to the documentation:

The @private tag marks a symbol as private, or not meant for general use. Private members are not shown in the generated output unless JSDoc is run with the -p/--private command-line option.

The problem is that these are symbols that are meant for general use - they are exported as constants for use in error comparisons, so I want there to be jsdocs for them, but I also want them captured in, for instance, a heading like "errors" rather than having every single one listed in the main nav.

My apologies I misinterpreted the outcome that you desired, you still need them documented but you only want them to be associated with the Error namespace. I would try the memberOf keyword:

In the following example, the hammer function would normally be documented as a global function. That's because, in fact, it is a global function, but it is also a member of the Tools namespace, and that's how you wish to document it. The solution is to add a memberof tag:

https://jsdoc.app/tags-memberof.html

that did the trick, thanks!


CUPS error log should include timeout's in default state.

It should be easier to diagnose time outs from spooling application to CUPS.
Ran Adobe Lightroom to produce a large print.  Due to machine's age, etc., job took a long time (5.5 minutes). CUPS Timeout setting at default is 300 seconds.  CUPS error log says

E [17/Apr/2019:21:50:17 -0700] [Job 3] Aborting job because it has no files.
E [17/Apr/2019:21:50:40 -0700] [Client 44] Returning IPP client-error-not-possible for Set-Job-Attributes (ipp://localhost/jobs/3) from localhost.
E [17/Apr/2019:21:57:10 -0700] [Job 4] Aborting job because it has no files.
E [17/Apr/2019:21:57:41 -0700] [Client 59] Returning IPP client-error-not-possible for Set-Job-Attributes (ipp://localhost/jobs/4) from localhost.

which gives no hint that there was a timeout.
Doing this on MacOS, I could use cupsctl to set Loglevel to debug, and then I saw the timeout.  Changed Timeout to better value & things work as expected. As a user, I shouldn't have to go that deep to diagnose things. Log level for timeout message should be changed.

I think the basic root cause in this issue here is same as in
https://github.com/apple/cups/issues/5559
In both cases by default CUPS does not provide sufficient information
to the user that something "unusual" happened or is still going on.

@jsmeix This issue is different - the application submitting the print job is taking too long to prepare the PDF file for submission, and the timeout isn't getting logged.

[master d6395c6f1] Increase the MultipleOperationTimeOut default to 15 minutes, log an error if the job submission times out (Issue #5570)
[branch-2.2 c1b133728] Increase the MultipleOperationTimeOut default to 15 minutes, log an error if the job submission times out (Issue #5570)

For what it's worth, the case in question was NOT a multiple file job, I don't believe.  It was a single page (ergo single PDF) that had 25 little images on it (as in a photographer's contact sheet).  That's why it took so long to produce.
The "fix" (from my side) was to increase the Timeout parameter, not the MultipleOperationTimeOut parameter.  You all probably understand more than I about CUPS (I hope so), but I just want to ensure that the right thing is happening.

@fhcarter There are two timeouts at play here - one between the Create-Job and Send-Document requests, and one during the Send-Document request. I'll double-check that the timeout has been increased for both, but the intent is to log an error whenever a timeout occurs...

@michaelrsweet Thanks.  Just wanted to make the situation clear.  The code changes looked like it changed only MultipleOperationTimeOut.  It specifically didn't change DEFAULT_TIMEOUT, instead changing the initializer from DEFAULT_TIMEOUT to 900.

@fhcarter Right, DEFAULT_TIMEOUT is used in a bunch of places for the normal networking timeout value, but in this case MultipleOperationTimeOut is king because it applies to the submission process (how long it takes the document(s) for the job to be submitted) and not to how long cupsd waits for a client to send something over the network.
After reviewing the code in scheduler/job.c, it looks like I needed to move things around a bit for the logging to only happen for a timed out job (vs. the hold on the job expiring normally).

[master 6dac1130b] Update the hold expiration logging - one case is an error, the other is normal.
[branch-2.2 a2eba0431] Update the hold expiration logging - one case is an error, the other is normal (Issue #5570)

@michaelrsweet OK, thanks.  At home tonight, I'll try updating my CUPS config, changing MultipleOperationTimeOut instead of Timeout (setting Timeout back to its default) and see if that (also) solve the issue.  It's easy enough to test -- just not from here at work.

@michaelrsweet I tried making the changes you appeared to make via cupsctl -- specifically setting Timeout back to 300 and setting MultipleOperationTimeout to 900.  This put me back to the job being canceled due to no files:

[01/May/2019:20:25:56 -0700] Scheduler shutting down due to program error.
E [01/May/2019:20:31:16 -0700] [Job 18] Aborting job because it has no files.
E [01/May/2019:20:32:40 -0700] [Client 16] Returning IPP client-error-not-possible for Set-Job-Attributes (ipp://localhost/jobs/18) from localhost.
E [01/May/2019:20:33:05 -0700] Scheduler shutting down due to program error.
E [01/May/2019:20:40:20 -0700] [Job 19] Aborting job because it has no files.
E [01/May/2019:20:40:51 -0700] Scheduler shutting down due to program error.
E [01/May/2019:20:47:35 -0700] [Job 20] Aborting job because it has no files.
E [01/May/2019:20:48:18 -0700] Scheduler shutting down due to program error.

(I tried 3 times.)  However, setting Timeout to 900 (and returning MultiOperationTimeout back to its default) works fine.
Mojave 10.14.4, CUPS 2.2.9
Attached copy of error log from running in this configuration
cupsd.conf:

_debug_logging=1
_remote_admin=0
_remote_any=0
_share_printers=0
_user_cancel_any=0
BrowseLocalProtocols=dnssd
DefaultAuthType=Basic
JobPrivateAccess=default
JobPrivateValues=default
MultipleOperationTimeOut=900
PageLogFormat=
SubscriptionPrivateAccess=default
SubscriptionPrivateValues=default
Timeout=300
WebInterface=true

cups.log
Hopefully, this helps with diagnostics

The simplest change here is to just bump the default Timeout value to 900 seconds; the new logging code will still let you know why the timeout occurred for the job, and the increased timeout value will hopefully make these issues non-existent.
I looked at tying the in-process IPP request to MultipleOperationTimeout and all other HTTP requests to Timeout, but that seemed too likely to be abused...
[master f49af670a] Bump the default Timeout to 15 minutes as well (Issue #5570)
[branch-2.2 9d6f1bb6c] Bump the default Timeout to 15 minutes as well (Issue #5570)

@fhcarter
for the fun of it a side note reagding your
https://github.com/apple/cups/issues/5570#issuecomment-488374265
a single page ... that had 25 little images on it

I guess each of that little images was perhaps several megabytes big
(e.g. just what one gets from a modern smartphone camera ;-)

@jsmeix -- Indeed.  Approximately half were iPhone 7's, the other half either Sony NEX-7 (24MP) or Sony a7r iii (40+MP) :-).  It takes a while.
@michaelrsweet Thanks -- that seems like a reasonable fix.  With the error message, it will be (more) possible to find the cause and deal with it.


Reindex embeddings if they are stale

TODO: BFG version upgrade after https://github.com/sourcegraph/bfg-private/pull/195 is merged.
Test plan
Tested locally - I opened a repo that I haven't worked on in a while, created a commit and saw that embeddings were reindexed for the repo.
Changelog

Embeddings index is periodically rebuilt if it is stale.

Do we have telemetry that measures when an a codebase is re-indexed?

@chenkc805 No, but it's a good idea, let me add it 👍

@olafurpg I followed the procedure you wrote down in #5140 and it seems everything's in order - we only download the binary if the users is PLG and local embeddings are enabled.


Fix ipv6 netmask

Related issues (optional)
Fixes #825

@shubheksha , can you please also fix the image, as  Alessandro suggests? Thanks!

@shubheksha - You're almost there! See @aboch's comment

Fixed it :D

Please squash the commits into one.
Besides that, looks good to me. Thanks.


WARN[0000] The "t" variable is not set. Defaulting to a blank string.

do you have more information/details about this warning?

IIRC, it was happening every docker-compose command, the same when a variable in the compose.yaml file is not found in the environment or in the .env file. Although it wasn't preventing the building, docker-compose spammed in the shell this error WARN[0000] The "t" variable is not set. Defaulting to a blank string. every command


Feature/keyword index

Added keywords created in supernote tablet to newly generated pdf as annotations. Tested on Chauvet 2.15.29 version.
Please, let me know if you have any questions or suggestions to improve the code. Its was written under specific use case so source code might be not efficient

Hi @jya-dev , Thanks for feedback  - Keyword Annotation feature has been disabled by default


API upgrade to v19

Upgrade the API peer version to v19. This will deprecate support for the obsolete peer interface on appliances.

In the release notes we should document the supported API versions.

We should probably also consider printing a warning if the appliance version is lower than supported when a command is executed.


Millon of close syscalls on lxd container when run a simple python scrip

Required information

Distribution:=Ubuntu
Distribution version:="Ubuntu 16.04.3 LTS"
The output of "

config:
  storage.zfs_pool_name: debtool
api_extensions:
- id_map
api_status: stable
api_version: "1.0"
auth: trusted
public: false
environment:
  addresses: []
  architectures:
  - x86_64
  - i686
  driver: lxc
  driver_version: 2.0.7
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.4.0-96-generic
  server: lxd
  server_pid: 17398
  server_version: 2.0.10
  storage: zfs
  storage_version: "5"

LXC and LXD versions:

$ apt-cache policy lxd
lxd:
  Installed: 2.0.10-0ubuntu1~16.04.2

$ apt-cache policy lxc
lxc:
  Installed: 2.0.7-0ubuntu1~16.04.1

Storage backend in use: zfs

Issue description
I noticed a very different performance running commands on lxd container so, simple as sh.nice()
Host
    $ time python -c "import sh; sh.nice()"
    real	0m0.077s
    user	0m0.052s
    sys	0m0.012s

Container
    $ time python -c "import sh; sh.nice()"
    real	0m0.215s
    user	0m0.088s
    sys	0m0.120s

My next step was use strace which says that the container version call 1048796 times the syscall close!! The majority of the times it returns EBADF (Bad file descriptor).
I do not know if it is a sh module issue or not but it is happening with lxd and not with lxc
Steps to reproduce
    $ uname -a
    Linux cmp-1 4.4.0-96-generic #119-Ubuntu SMP Tue Sep 12 14:59:54 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
    $ lxc launch ubuntu:precise new-precise-sh
    $ lxc exec new-precise-sh -- bash
    $ apt-get install python-pip
    $ pip install sh
    $ strace -f -e close python -c "import sh; sh.nice()" 2>&1 | wc -l
    1048796

Information to attach
Running strace with output in different files. There is one too big strace.2618
    $ strace -ff -o strace python -c "import sh; sh.nice()"
    $ ls -la
    total 75276
    drwxrwxr-x  2 user user     4096 Sep 27 16:58 .
    drwxr-xr-x 41 user user     4096 Sep 27 16:45 ..
    -rw-r--r--  1 root   root     121780 Sep 27 16:33 strace.2615
    -rw-r--r--  1 root   root       3995 Sep 27 16:33 strace.2616
    -rw-r--r--  1 root   root       6108 Sep 27 16:33 strace.2617
    -rw-r--r--  1 root   root   76558803 Sep 27 16:33 strace.2618
    -rw-r--r--  1 root   root     362363 Sep 27 16:33 strace.2619
    -rw-r--r--  1 root   root      10748 Sep 27 16:33 strace.2620

The content of the file something like this:
    $ cat strace.2618
    ...................
    ...................
    getrlimit(RLIMIT_NOFILE, {rlim_cur=1024*1024, rlim_max=1024*1024}) = 0
    close(3)                                = 0
    close(4)                                = 0
    close(5)                                = -1 EBADF (Bad file descriptor)
    close(6)                                = -1 EBADF (Bad file descriptor)
    close(7)                                = 0
    close(8)                                = 0
    close(9)                                = -1 EBADF (Bad file descriptor)
    close(10)                               = 0
    close(11)                               = -1 EBADF (Bad file     descriptor)
    ...................
    ...................
    close(33)                               = -1 EBADF (Bad file descriptor)
    close(34)                               = -1 EBADF (Bad file descriptor)
    close(35)                               = -1 EBADF (Bad file descriptor)
    close(36)                               = -1 EBADF (Bad file descriptor)
    close(37)                               = -1 EBADF (Bad file descriptor)
    close(38)                               = -1 EBADF (Bad file descriptor)
    ...................
    ...................
    close(1048568)                          = -1 EBADF (Bad file descriptor)
    close(1048569)                          = -1 EBADF (Bad file descriptor)
    close(1048570)                          = -1 EBADF (Bad file descriptor)
    close(1048571)                          = -1 EBADF (Bad file descriptor)
    close(1048572)                          = -1 EBADF (Bad file descriptor)
    close(1048573)                          = -1 EBADF (Bad file descriptor)
    close(1048574)                          = -1 EBADF (Bad file descriptor)
    close(1048575)                          = -1 EBADF (Bad file descriptor)
    ioctl(1, SNDCTL_TMR_TIMEBASE or TCGETS, {B38400 -opost -isig -icanon -echo ...}) = 0

Can you try running the same test but under a "script /dev/null bash" session?
So:
lxc exec new-precise-sh -- script /dev/null -c bash

This will cause your shell to use container-allocated pts rather than host allocated. I suspect this may be the source of the confusion in python or the python module you're using.

It did not work, same think
strace -f -e close python -c "import sh; sh.nice()" 2>&1 | wc -l
1048798

Ok, so the problem is just that this piece of python code is crap.
                # don't inherit file descriptors
                max_fd = resource.getrlimit(resource.RLIMIT_NOFILE)[0]
                os.closerange(3, max_fd)

This iterates over every single fd number up to RLIMIT_NOFILE, closing them all one by one, regardless of whether the fd actually does exist.
On a normal system, this will be capped at 1024 as the most comment NOFILE limit. In LXD containers, we have it bumped to a much higher value, causing it to take longer.
I'd strongly recommend you tell the upstream to look at /proc/self/fd/ and just close those fds rather than iterate through the entire possible range.
Closing this issue as it's not a LXD bug, it's just bad python code which would cause the same problem on a system that's got an increased NOFILE limit.

Thanks!!


Reverse the code

First of all, is it safe to use the script, or does allowing WOL-Packages to go through open some kind of security risk?
And also - how would I revert the process?
Thanks for your feedback!

Sorry I should perhaps clarify: I'm talking about this script of yours.

To revert it you can re-run the script with any "=0" replaced with "=1" (and vice versa), although I don't use this setup anymore and there is a suggestion in this issue that it no longer works - https://github.com/dhutchison/dhutchison.github.io/issues/9 .
I'm not aware of any way this could be a security risk, but ultimately you need to decide what your risk/reward threshold is.


Add lightbulb emoji

Related to https://app.clubhouse.io/glitch/story/1735/finish-remix-this-postcard-for-onboarding-user-dashboard
add-lightbulb-emoji.glitch.me

:rocket: PR was released in v0.19.0 :rocket:


ARC-0001: Add support for a message per group

algorandfoundation/ARCs#1

This is a great addition 👍

Considering having NOTE field which already exists with the same identical mission.


Reduce vtgatev2_test flakiness.

@alainjobart @sougou

LGTM Let's try this I guess... Not sure what it means.


Try to update pyqt to 5.15.2 using qt-main and qt-webengine

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hey @SrNetoChan, thanks for your help with this.
@andfoy is working on it too and already has working recipes. But he's blocked at the moment because his latest builds for qt-webengine have not been uploaded yet.

I will close this one then! Thank you.


Update travis to build doc also for Python 3.x (up to 3.6)

I tried for Python 3.6 in branch travis-python-3.6. @HubLot could you have a look?

Yes, Travis is experimenting issues due to the recent AWS outage.
Refresh regularly and it should come up.
The  travis-python-3.6 branch failed in Travis CI because of a wrong declaration in the table of content while generating the doc (the code itself is OK). Nothing big I guess.

The error is due to the fact jupyter-nbconvertcannot convert notebooks into markdown because it cannot find python2 kernel. Indeed, the notebooks where created with a python2 kernel, so the tool needs this kernel to convert the notebook.
Since it's a python3 environment, there is no python2...
I'm not sure there is simple solution to that. Maybe give up on building the doc with python3 ?

apparently could can have a Python2 kernel within Python3 jupyter:
http://stackoverflow.com/a/37857536
Could you have a look?

Thanks. It worked see my .travis.ymlmodification (https://github.com/HubLot/PBxplore/commit/743594b9ac349d15851517368812d7298a5c1447) and Travis result (https://travis-ci.org/HubLot/PBxplore/jobs/207400221)

Very nice. Could you propagate this for all Python 3.x versions we are currently tested? And then create a PR?

Thanks @HubLot


Integrate BoringSSL test suite (BoGo)

See https://github.com/google/boringssl/blob/master/ssl/test/PORTING.md and https://github.com/rustls/rustls/blob/main/rustls/examples/internal/bogo_shim.rs.
Issues

[ ] Implement alerts
[x] Detect missing key shares (MissingKeyShare-Server-TLS13)
[ ] Detect duplicate key shares (DuplicateKeyShares-TLS13)
[ ] Server doesn't correctly returns session ID (EmptySessionID-TLS13, Server-ShortSessionID-TLS13, Server-FullSessionID-TLS13)
[ ] Server doesn't correctly handle too long session ID (Server-TooLongSessionID-TLS13)
[ ] Server should accept any record version for compatibility reasons (LooseInitialRecordVersion-TLS13)
[ ] decrypt_handshake should ignore user_canceled warning alert (SendUserCanceledAlerts-TLS13)

[ ] decrypt_handshake doesn't handle (post-handshake) alerts (SendBogusAlertType, EmptyCertificateList-TLS13)
[ ] decrypt_handshake must handle bad records/ciphertexts (TLSFatalBadPackets, TLS13-EmptyRecords, TLS13-OnlyPadding, SkipEarlyData-OmitEarlyDataExtension-TLS13, SkipEarlyData-Interleaved-TLS13)
[ ] decrypt_handshake must handle invalid certificates (GarbageCertificate-Client-TLS13)

[ ] handle key update (TooManyKeyUpdates, KeyUpdate-ToClient, KeyUpdate-ToServer, KeyUpdate-InvalidRequestMode, KeyUpdate-RequestACK)
[ ] finish digest not checked (BadFinished-Client-TLS13, BadFinished-Server-TLS13)
[ ] correctly handle HRR (UnnecessaryHelloRetryRequest-TLS13, SecondHelloRetryRequest-TLS13, HelloRetryRequest)
[ ] check key usage bit (ECDSAKeyUsage-Client-TLS13)

I'm currently working on this.

I noticed that a large number of BoGo test cases currently fail because the BoGo runner and the Bertie shim are unable to agree on a cipher suite. The reason is, that Bertie is temporarily unable to do RSA, while the BoGo runner defaults to RSA for the TLS1.3 cipher suites and by default loads a RSA certificate for most test cases.
See:
https://github.com/google/boringssl/blob/ad57528d2c978543106f9b115bd0eb658f3ebdd2/ssl/test/runner/runner.go#L769
https://github.com/google/boringssl/blob/ad57528d2c978543106f9b115bd0eb658f3ebdd2/ssl/test/runner/cipher_suites.go#L56
Bertie needs to support RSA again for this large number of test cases to pass, or at least to reveal their actual failure reasons. I have verified that some test cases advance further and give different failure reasons if I force the BoGo runner to load an ECDSA certificate instead.


Feature/fix header stacking

Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged

Thanks for the PR! I see what this is doing, and it is a fix, however, it does also impact the location of the mobile menu button. I made a PR which addresses this and the knock-on effects of adjusting the mobile menu button position whilst correcting the location of the service title here so going to close this in favour of this: https://github.com/UKHomeOffice/govuk-react/pull/411