Remove note left by author

Remove this note left by the author of the page (3rd paragraph):
jgao: list the versions for supporting Bicep parameters file. You can compile Bicep parameters files into JSON parameters files to deploy with a Bicep file.

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

ID: cdf73225-d2e6-b23e-b355-fbb9d3fcdca3
Version Independent ID: fa832878-9821-f222-5f15-ad8646737a04
Content: Create parameters files for Bicep deployment - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/parameter-files.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@regismartel - thank you for your comments.  I have submitted a PR to correct the content.
#please-close


Feature Request - Making it a PWA

Why don't we make it a PWA by introducing service workers ?

I'm unclear what you want us to make into a PWA?


Petset initialization

Incorporates feed back from https://github.com/kubernetes/kubernetes.github.io/pull/698, and a new doc on bootstrapping.

@bprashanth FYI I am OOO for the next 2 weeks.

@johndmulhausen ptal

@smarterclayton fyi

Merging; marking as "to-tech-review" in bug #784

Retroactively looks good to me.
On Wed, Jul 6, 2016 at 7:59 PM, johndmulhausen<EMAIL_ADDRESS>wrote:

Merged #757 https://github.com/kubernetes/kubernetes.github.io/pull/757.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes.github.io/pull/757#event-715088721,
or mute the thread
https://github.com/notifications/unsubscribe/ABG_p7jl5r7CC_hlp5MCrRD21FF_gvhPks5qTEFRgaJpZM4JBqOd
.


Fedora Server has no Legacy Python

Indeed it doesn't! Thanks for the catch!

Unfortunately, most of the spins do. But this should fix it: https://pagure.io/fedora-comps/pull-request/394


Bolt12/p2p master docs merge

Merge Connection Manager docs with Network-Spec docs

It does not build for me, there's no figure/ConnectionManagerInteractions.pdf checked in git; is there a way to generate it from the *.drawio file?
! Package pdftex.def Error: File `figure/ConnectionManagerInteractions.pdf' not
 found: using draft setting.

See the pdftex.def package documentation for explanation.
Type  H <return>  for immediate help.
 ...

l.288 ...figure/ConnectionManagerInteractions.pdf}

If we can generate it, then we'll also need to update nix script to build it.

We can't generate it, I must have forgotten to add it, let me fix that


Update the CLI install script release source

Expected Behavior

Release versions should be retrieved successfully.
Actual Behavior

Users may hit the 60 requests per hour limit (per IP) when calling the github api unauthenticated.
Steps to Reproduce the Problem

Call any github api endpoint 60 times, run the install script which fails to find a release version for the CLI and the invalid version is not handled.
Proposed Solution
A "version" branch is set up and maintained by a workflow which is triggered by dispatch (primarily intended for the release job to trigger this workflow on completion).
The version branch will contain a docs folder with a single file containing the latest version tag for the CLI and will be served by gh-pages or an azure static site (with preference for gh-pages).
The install scripts will no longer hit the API for the latest tag.
Release Note

RELEASE NOTE: FIX Dapr CLI install/update release version retrieval

/keep-alive


navigation2: 1.1.13-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository navigation2 to 1.1.13-1:

upstream repository: https://github.com/ros-planning/navigation2.git
release repository: https://github.com/SteveMacenski/navigation2-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 1.1.12-1

Holding for Humble sync


LG Heat Pump - Which Value is shown?

Hey there,
I am using the Integration with my heat pump LG Therma V R290 and it works pretty good.
But I don't understand which value is shown in the auto menu.
In the picture you can see the heat pump shows the value of 17 degress.
But where does it take this value from. It isn't the outdoor temperature and not a value from the heating curve.

i think its the target temperature for the outlet which equals to zero in the thinq app
if you increase it in the thinq app , homeassistent will also update to 21 (if you add 5 in the thinqapp)  or 12 if you lower it
(i have the dutch version , here its called doeltemperatuur uitlaat, which i translated)

Yeah I think thats true but pretty bad shown then. 17 means zero offset and 12 is -5 and 22 is +5.
I am missing the real water target and outdoor unit temperatur. So I can't really understand what is going on.
Maybe with the Modbus Version it is better?

i have seen some yaml code where they offset the values , i tried to do that by going to to developer tools - states
i have changed the state attributes of that entity ,
hvac_modes:

"off"
auto
heat
min_temp: -5
max_temp: 5
target_temp_step: 1
current_temperature: 34
temperature: 0
friendly_name: Warmtepomp therma v
supported_features: 385

hower that does nothing and it keeps referting back. so obviously thats not wheren you can set the offset (or i am doing it wrong , no judgment please, i just started fiddling with it today ;) )

Thanks, I will integrate the Modbus solution.

hmm i have been poking around,  and found this part in the diagnostic file
"airState.ess.solarPowerExt": 0.0,
"airState.tempState.current": 24.5,
"airState.tempState.hotWaterTempMax": 55.0,
"airState.miscFuncState.awhpTempSwitch": 1.0,
"airState.miscFuncState.extraOp": 0.0,
"airState.2nd.tempState.airTempCoolMin": 16.0,
"airState.2nd.tempState.target": 17.0,
"airState.2nd.tempState.waterTempCoolMin": 5.0,
"airState.tempState.airTempHeatMax": 30.0,
no clue yet how to change it the entity though

They set the pump incorrectly. Very classical of installer nowadays. If your pump is entirely for heating floor, you need to specify that the measure of temperature is WATER, and not AIR.
Its in the settings somewhere under USER of FUNCTION. I will take a look at home when I arrive.


Removes exsessive casting in unit test.

Changes in this pull request
Removes exsessive casting in unit test.
Pull request checklist

[x] All tests pass. Demo project builds and runs.
[ ] I added tests, an experiment, or detailed why my change isn't tested.
[x] I have reviewed the contributing guide

<3

Coverage remained the same at 86.297% when pulling 4bcfff59442299d3f1e05e56aeea87f49a79cc3c on valeriyvan:test into 158d04ebe0a5bdabf9ec072da54b4e77e6d18ede on Instagram:master.


[Bug/Glitch] Game freaks out when using a Switch Pro controller

Legacy4J Version
1.21
Mod Loader Version
Fabric
Bug/Glitch Occurrence Context
As soon as I start to use a Switch Pro controller (The only one I have), the game completely goes crazy and I can't do anything from how sporadic everything is.
Crash Report
No response
What happened wrong?
[Flashing Lights Warning]
https://github.com/user-attachments/assets/5377e15a-3a3d-4b69-937d-170db09128ab

Are u with Steam open?


unit testing and coverage

not expecting to work out the gate because of codecov token, lack of tests, but it's a start. #5 #20 #4 #6

this is currently missing packaging (setup.cfg, setup.py), as well as .coveragerc and requirements.txt
here are the ones from my other repo, without necessary changes applied.
Should be able to sed -i these and replace the instances of mud with cafelytics, but it'd be neat to be able to import it as import cafe instead, though PyPi wont let us publish under this name (shouldn't impact how you import).
setup.cfg
# This file is used to configure your project.
# Read more about the various options under:
# http://setuptools.readthedocs.io/en/latest/setuptools.html#configuring-setup-using-setup-cfg-files

[metadata]
name = mud
description = Maximal Updated Density equations for Data-Consistent Inversion
author = Mathematical Michael
author-email =<EMAIL_ADDRESS>license = mit
long-description = file: README.md
long-description-content-type = text/markdown; charset=UTF-8; variant=GFM
url = https://github.com/mathematicalmichael/mud.git
project-urls =
    Documentation = https://pyscaffold.org/
# Change if running only on Windows, Mac or Linux (comma-separated)
platforms = any
# Add here all kinds of additional classifiers as defined under
# https://pypi.python.org/pypi?%3Aaction=list_classifiers
classifiers =
    Development Status :: 4 - Beta
    Programming Language :: Python

[options]
zip_safe = False
packages = find:
include_package_data = True
package_dir =
    =src
# DON'T CHANGE THE FOLLOWING LINE! IT WILL BE UPDATED BY PYSCAFFOLD!
setup_requires = pyscaffold>=3.2a0,<3.3a0
# Add here dependencies of your project (semicolon/line-separated), e.g.
install_requires = numpy
                   matplotlib
# The usage of test_requires is discouraged, see `Dependency Management` docs
# tests_require = pytest; pytest-cov
# Require a specific Python version, e.g. Python 2.7 or >= 3.4
python_requires = >=3.5

[options.packages.find]
where = src
exclude =
    tests

[options.extras_require]
# Add here additional requirements for extra features, to install with:
# `pip install mud[PDF]` like:
# PDF = ReportLab; RXP
# Add here test requirements (semicolon/line-separated)
testing =
    pytest
    pytest-cov

[options.entry_points]
# Add here console scripts like:
# console_scripts =
#     script_name = mud.module:function
# For example:
# console_scripts =
#     fibonacci = mud.skeleton:run
# And any other entry points, for example:
# pyscaffold.cli =
#     awesome = pyscaffoldext.awesome.extension:AwesomeExtension

[test]
# py.test options when running `python setup.py test`
# addopts = --verbose
extras = True

[tool:pytest]
# Options for py.test:
# Specify command line options as you would do when invoking py.test directly.
# e.g. --cov-report html (or xml) for html/xml output or --junitxml junit.xml
# in order to write a coverage file that can be read by Jenkins.
addopts =
    --cov mud --cov-report term-missing
    --verbose
norecursedirs =
    dist
    build
    .tox
testpaths = tests

[aliases]
dists = bdist_wheel

[bdist_wheel]
# Use this option if your package is pure-python
universal = 1

[build_sphinx]
source_dir = docs
build_dir = build/sphinx

[devpi:upload]
# Options for the devpi: PyPI server and packaging tool
# VCS export must be deactivated since we are using setuptools-scm
no-vcs = 1
formats = bdist_wheel

[flake8]
# Some sane defaults for the code style checker flake8
exclude =
    .tox
    build
    dist
    .eggs
    docs/conf.py

[pyscaffold]
# PyScaffold's parameters when the project was created.
# This will be used when updating. Do not change!
version = 3.2.3
package = mud
extensions =
    markdown

setup.py
# -*- coding: utf-8 -*-
"""
    Setup file for mud.
    Use setup.cfg to configure your project.

    This file was generated with PyScaffold 3.2.3.
    PyScaffold helps you to put up the scaffold of your new Python project.
    Learn more under: https://pyscaffold.org/
"""
import sys

from pkg_resources import VersionConflict, require
from setuptools import setup

try:
    require('setuptools>=38.3')
except VersionConflict:
    print("Error: version of setuptools is too old (<38.3)!")
    sys.exit(1)

if __name__ == "__main__":
    setup(use_pyscaffold=True)

.coveragerc
# .coveragerc to control coverage.py
[run]
branch = True
source = mud
# omit = bad_file.py

[paths]
source =
    src/
    */site-packages/

[report]
# Regexes for lines to exclude from consideration
exclude_lines =
    # Have to re-enable the standard pragma
    pragma: no cover

    # Don't complain about missing debug-only code:
    def __repr__
    if self\.debug

    # Don't complain if tests don't hit defensive assertion code:
    raise AssertionError
    raise NotImplementedError

    # Don't complain if non-runnable code isn't run:
    if 0:
    if __name__ == .__main__.:

installations are now working but the way I set up the tests was to fail unless we could import cafe, because I want to know what changes need to be made to have the package name mismatch the import name (much like pip install scikit-learn would be imported as import sklearn)

commit https://github.com/mathematicalmichael/cafelytics/pull/20/commits/09813329e01e1fb9323c33b5b487e6d8be0f0ca4 also moves the package into src/cafe

coverage and such now are picking up the right package name, which is great. However, tests failing for other reasons. For now I'm just getting rid of the existing test.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@65b2318). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master     #20   +/-   ##
========================================
  Coverage          ?   0.00%
========================================
  Files             ?       1
  Lines             ?       9
  Branches          ?       0
========================================
  Hits              ?       0
  Misses            ?       9
  Partials          ?       0

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 65b2318...e6a4c07. Read the comment docs.

more to-do's:
github workflow that checks we can run the commands in the README.


nativebridge load changed to process type detection (instead of O/S)

fixed Issue #49

Makes perfect sense, the bitness of the OS doesn't matter of course, it's the bitness of the process.
One question, would a better solution be to use the property Environment.Is64BitProcess which makes it much more clear at a glance what we're checking for? If so, can you change the PR and I will merge it then?

Resolved with a40e66b.


Unify static linking for all operating systems and improve README

This commit unifies the code for static linking under all operating systems, with a focus on fixing problems for Windows:

All functionality for pkg-config is now enabled for Windows as well. For example, a vcpkg-built pkgconf works great.  Therefore, it seems there is no reason to prevent users from using pkg-config on Windows if they want to use it.

When not using pkg-config on Windows, and statically linking against FFmpeg libraries, many system libraries will also need to be linked. A list of known system libraries will now be linked in such situations.  It's best to use pkg-config, but this will at least reduce the probability of linker errors.  Note that we also have to do this for vcpkg as well.

Code duplication between Windows and other operating systems has been eliminated.

The statik flag is now correctly passed to pkg_config when in the static_linking function.

The GitHub Actions workflow is updated to test the new possibilities on Windows.  The example program is also run, which revealed that the existing vcpkg-based way of finding FFmpeg was missing several system libraries for linking.

Make significant improvements to the README.  More detailed information about environment variables, features, how the build script works, vcpkg triples, and more is included to help users work with the crate.

Improving the functionality of dynamic linking is an obvious next step, but is out of scope of this commit.  It's likely that a single, large function can eventually be used for both static and dynamic linking.

I tested this and it fixes the currently broken linking with OpenCV (#130)


Pretrained model with wav2vec-u for Romanian language

🚀 Feature Request
Pretrained model with wav2vec-u for Romanian language
Motivation
There are no good/opensource models for this language. Since the last wav2vec-u method does not need to have labeled data, I think it can be pretty doable to create such a model and then let other developers to fine tune it
Pitch
Model which transcribes Romanian audio speech into text format
Alternatives
Since Romanian is based on latin letters, special characters could be mapped to that latin letters(like ă->a, î>i and so on)

There is a wav2vec2 model

There is a wav2vec2 model

Yes, but the test performance is only 25% and by testing it myself with pretty simple and clear samples, it is not quite accurate. That's why I was hoping that this new wav2vec which doesn't require labeled data maybe will be able to perform a bit better

There is a wav2vec2 model

Yes, but the test performance is only 25% and by testing it myself with pretty simple and clear samples, it is not quite accurate. That's why I was hoping that this new wav2vec-u which doesn't require labeled data maybe will be able to perform a bit better

To the best of my knowledge, when training the unsupervised model in the first step, the actual wav2vec2 representations were frozen. Those layers weren't updated until the second self-training stage with pseudo transcriptions from an HMM, where the HMM itself is trained on pseudo transcriptions from the unsupervised model in the first step. There is no pre-training involved, at least in the original wav2vec sense.

if you already have a model which you have already fine-tuned to a semi-ok performance, you can likely significantly improve it by self-training on some corpus of similar-domain unlabeled audio data. you can use e.g. w2v -> HMM -> finetune w2v pipeline, or just w2v -> finetune w2v (the delta wont be too huge) and see what you get after a couple of rounds


Can I index/search for nested attributes?

If I have
const products = [
    { id: 1, name: "Apple", price: 5, details: { category: "Fruit" } },
    { id: 2, name: "Banana", price: 10, details: { category: "Fruit" } },
    { id: 3, name: "Grapes", price: 6, details: { category: "Fruit" } },
    { id: 4, name: "Orange", price: 8, details: { category: "Fruit" } },
    { id: 5, name: "Potato", price: 18, details: { category: "Vegetable" } },
    { id: 6, name: "Milk", price: 7, details: { category: "Dairy" } },
    // ...
]

can I search for details.category?

currently no. but I'll add in my todo.


Fix for optional Enums in InstanceForm (#4817)

Pull request opened by the merge tool on behalf of #4817

Merged in<PHONE_NUMBER>746d8b53c618f2e402b7d493ae4b03


Update host for Hesan.He3 1.3.13

#163422

Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels - Intentionally didn't update manifest version


ng new mistakenly thinks it's in a project

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Windows 8.1
Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

@angular/cli: 1.0.0-beta.31
node: 6.9.4
os: win32 x64
@angular/cli: 1.0.0-beta.31

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

mkdir sample
cd sample
yarn add @nagular/cli
node_modules/.bin/ng new sample -dir=.

The log given by the failure.

Normally this include a stack trace and some more information.

You cannot use the new command inside an Angular CLI project.

Mention any other details that might be useful.

This is because the command checks for whether it's in project by looking for the package @angular/cli under node_modules
I think it should be looking for config file .angular-cli.json instead
This is important as ng init / ng update is now gone. It allows me to use the CLI without global installation, earlier I used ng init for this
I'm NOT asking for this to work in existing CLI project. This is NOT a request to make new act as init. The scenario I'm asking to support is when the package exists, but no other files like package.json or .angular-cli.json are present, that is, no Angular CLI project in the folder

Thanks! We'll be in touch soon.

Happy to look at implementation if we agree we want to make that change, which is detecting project presence based on config file .angular-cli.json not presence of node_modules/@angular/cli/.

You have an node_modules folder somewhere in your path (a parent directory or something). Look for a package.json file somewhere, or a node_modules directory. Delete those and you'll be fine.
Most of the time those are in your HOME folder.

@hansl did you check the repro steps?

@hansl  I double checked - I don't.
But anyway, are you suggesting that the mentioned steps above are not supposed to bring the mentioned message?

OH okay. No I didn't. I just see those and copy-paste the message lol

The Angular CLI does use node_modules to detect if it's in a project. This is by design and unlikely to change (it's Ember-CLI specific logic).

I can see the check is made here https://github.com/angular/angular-cli/blob/master/packages/%40angular/cli/ember-cli/lib/models/project.js#L117
It's part of the Angular CLI project, and we modified the file a few times already. If we agree on the principle that the check should be based on presence of Angular CLI config file, that'd be awesome. We have diverged from the ember CLI anyway (Ember still has an init command for example, although again, this is NOT aimed at making new like init). This will enable #4768 for example.

Thanks a lot @hansl and @filipesilva 💯

I'm using the v.1.0.0-rc1 and still having the same issue. In order to initialize a project with local cli by
running:
npm init
npm install @angular/cli --save-dev
modifying package.json adding script: "ng": "node_modules\.bin\ng"
and running:
npm run ng -- new my-project -dir=.
I still get: "You cannot use the new command inside an Angular CLI project."
There are tons of practical reasons why this approach is for us better than having cli globally installed

@machucarro I just tried this myself and it works. See below for a log of what I did.
Maybe you have a CLI config file somewhere up in the directory tree?
kamik@T460p MINGW64 /D/sandbox
$ mkdir local-proj

kamik@T460p MINGW64 /D/sandbox
$ cd local-proj/

kamik@T460p MINGW64 /D/sandbox/local-proj
$ npm install @angular/cli
# ...

kamik@T460p MINGW64 /D/sandbox/local-proj
$ $(npm bin)/ng new local-proj --dir=./ --skip-install
installing ng
  create .editorconfig
  create README.md
  create src\app\app.component.css
  create src\app\app.component.html
  create src\app\app.component.spec.ts
  create src\app\app.component.ts
  create src\app\app.module.ts
  create src\assets\.gitkeep
  create src\environments\environment.prod.ts
  create src\environments\environment.ts
  create src\favicon.ico
  create src\index.html
  create src\main.ts
  create src\polyfills.ts
  create src\styles.css
  create src\test.ts
  create src\tsconfig.app.json
  create src\tsconfig.spec.json
  create src\typings.d.ts
  create .angular-cli.json
  create e2e\app.e2e-spec.ts
  create e2e\app.po.ts
  create e2e\tsconfig.e2e.json
  create .gitignore
  create karma.conf.js
  create package.json
  create protractor.conf.js
  create tsconfig.json
  create tslint.json
Successfully initialized git.
You can `ng set --global packageManager=yarn`.
Project 'local-proj' successfully created.


docs(tracer): snippets split and linting

Issue number: #1260 (older PR)
Summary

NOTE: Accidentally hit merge in previous PR after agreeing on approach. Reusing previous body.

Changes

Please provide a summary of what's being changed

This PR splits code snippets into a top-level examples folder. Formatting and linting (make lint) were adjusted to include examples folder. We also introduce cfn-lint within pre-commit hooks to selectively lint changed infra files within examples folder.
Convention
We'll keep all examples under examples folder. Each feature will have its own source code that match what's in the documentation, followed by one or more infrastructure as code customers could use in the future.
Example for Tracer
examples
└── tracer
    ├── cdk
    │   └── all_cdk_project_files, dependencies, etc.
    ├── sam
    │   └── template.yaml
    ├── src
    │   └── capture_lambda_handler.py
    └── terraform

Caveat
We need to ensure Lambda Powertools dependency/layer is used. In the case of Layers, we will need to upgrade the Layer version within examples as part of each release - @am29d your input is crucial here on the best way forward. Until Alex completes his Layer pipeline refactoring, we can build a workflow action to find/replace Layer ARN Version.
Another caveat is that cfn-lint dropped support for 3.6. We're using an official cfn-lint GitHub Actions as a workaround.
User experience

Please share what the user experience looks like before and after this change

Example of adding an invalid SAM template for a code snippet

Code quality example using our set of rules

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of my this change
[x] Changes are tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

View rendered docs/core/tracer.md

Tracer finalized. Here's how the structure looks like for other examples - names match features in docs to ease discovery. Examples are now fully typed and improved too.
 tree examples
examples
└── tracer
    ├── sam
    │   └── template.yaml
    ├── src
    │   ├── capture_lambda_handler.py
    │   ├── capture_method.py
    │   ├── capture_method_async.py
    │   ├── capture_method_async_concurrency.py
    │   ├── capture_method_context_manager.py
    │   ├── capture_method_generators.py
    │   ├── disable_capture_error.py
    │   ├── disable_capture_response.py
    │   ├── disable_capture_response_streaming_body.py
    │   ├── ignore_endpoints.py
    │   ├── patch_modules.py
    │   ├── put_trace_annotations.py
    │   ├── put_trace_metadata.py
    │   ├── sdk_escape_hatch.py
    │   ├── tracer_reuse.py
    │   ├── tracer_reuse_payment.py
    │   └── tracing_aiohttp.py
    └── template.yaml


chore: fix-ci

💻 变更类型 | Change Type

[ ] feat
[x] fix
[ ] style
[ ] chore
[ ] docs

🔀 变更说明 | Description of Change

📝 补充信息 | Additional Information

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Log issue

Seems the plugin is writing huge log files.
I have only critical errors set to show in the event log.

V0.1.2

I need to figure out how to set the log level for the plugin specific log in addition to the main indigo log.

Is this still happening?  I'm not seeing it on my system.

Might be fixed in 2022.0.1.

Fixed.  Plugin log matches the log level of the Indigo log.

I had quit using it due to the log file issues i was having, installed the new version this morning and so far so good!!
—
Korey Sherwin
SherwinSound.com
702-239-1195

On May 25, 2022, at 5:26 AM, Joe Keenan @.***> wrote:
Might be fixed in 2022.0.1.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


groupby UTC timestamp aggregation

Hi all,
I've found an inconsistency between pandas 0.17 and 0.16.2 when aggregating on UTC timestamps. Here is a snippet to reproduce the problem:
import numpy as np
import pandas as pd

np.random.seed(42)

data = pd.DataFrame({
	'factor': np.random.randint(0, 3, size=60),
	'time': pd.date_range('01/01/2000 00:00', periods=60, freq='s', tz='UTC')
})

gp = data.groupby('factor')

print(gp['time'].min())
print(gp['time'].max())

On 0.16.2 the output seems correct, i.e it returns timestamps:
In [1]: %run bug_pandas.py
factor
0    2000-01-01 00:00:01+00:00
1    2000-01-01 00:00:07+00:00
2    2000-01-01 00:00:00+00:00
Name: time, dtype: object
factor
0    2000-01-01 00:00:57+00:00
1    2000-01-01 00:00:54+00:00
2    2000-01-01 00:00:59+00:00
Name: time, dtype: object

However on 0.17 it returns timestamps as integers:
In [1]: %run bug_pandas.py
factor
0<PHONE_NUMBER>00000000
1<PHONE_NUMBER>00000000
2<PHONE_NUMBER>00000000
Name: time, dtype: int64
factor
0<PHONE_NUMBER>00000000
1<PHONE_NUMBER>00000000
2<PHONE_NUMBER>00000000
Name: time, dtype: int64

It should be noted that the problem doesn't appear with tz=None.
Thanks for your help,
Alexandre

so this works w/o the selection. I'll mark it, and if you'd like to dig in, pull-requests are welcome!
In [16]: data.groupby('factor').max()
Out[16]:
                            time
factor
0      2000-01-01 00:00:59+00:00
1      2000-01-01 00:00:54+00:00
2      2000-01-01 00:00:58+00:00

In [17]: data.groupby('factor')['time'].max()
Out[17]:
factor
0<PHONE_NUMBER>00000000
1<PHONE_NUMBER>00000000
2<PHONE_NUMBER>00000000
Name: time, dtype: int64

closed by #11672


fix RowTappedCommand gesture recognizers from blocking UI

Fixes #175
Also referenced in the comments on #170

@akgulebubekir I think a new hotfix release is warranted because of this


Just like opt out players list, can we get a opt in players list

I can list all of the players I want to sign.

EA api does no support searching multiple players once,you can create multiple filters for each player

@chithakumar13 I wanted to add a list of players, if they are on the list then bid for them. Similar to the unwanted player list, can we have a wanted players list?


Interactive Playground - IntelliSense not working as expected.

VSCode Version: 1.21.0-insiders and 1.20 (a little)
OS Version: Windows 10 Home

Steps to Reproduce:

Open Interactive Playground
Go to IntelliSense section.

Does this issue occur when all extensions are disabled?: Yes
The issue is that in the IntelliSense section, it states "In the below example, position the text cursor in front of the error underline," but in neither version is there an error underline.

With the the Insiders version, ctrl-space doesn't seem to bring up autocomplete candidates.

Compared to v1.20 which does.

Upon further testing with the other examples, I think it has to do with the interactive editor each example is running in.
If I take the exact same code in the interactive editor and move it to a file, it works exactly as expected. The same result is achieved in the "Refactoring" section. It won't work in the interactive editor, but it works when moved to a file on its own.

Reproduces on Windows. Works fine on Mac.

I work with TypeScript (2.7.1) on various react apps and have the same problem everywhere, on all my machines (windows). This isn't only a problem in the interactive playground.

@Xceno Is this with 1.20.1 or with 1.21-insiders? /cc @mattbierner

@chrmarti This is happening with 1.20.1.
Version 1.20.1
Commit f88bbf9137d24d36d968ea6b2911786bfe103002
Date 2018-02-13T15:34:36.336Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64

Sidenote: I've forgotten that my VSCode is synced on every PC and got quite a lot of extensions installed. So maybe it's an extension Problem. I'll try disabling them one by one and report back here.

Update: Okay so I've disabled all extensions and I still got the issue. I noticed however, that this is mainly happening when writing JSX / TSX.

@Xceno Could you open a new issue? This seems to be a different problem.

/duplicate of #44172


No dev-certs reference

Why is there no reference for the dev-certs command?

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

ID: 0ba6c9b5-64cb-1dc6-21d0-ad8f0a1185fe
Version Independent ID: 0f6c658c-b477-2d2c-2a9a-d74f6ae38d3f
Content: dotnet command - .NET CLI
Content Source: docs/core/tools/dotnet.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

dev-certs is documented in the Additional tools section of the Table of Contents -- https://docs.microsoft.com/dotnet/core/additional-tools/self-signed-certificates-guide -- we can add reference documentation for the tool's options in that article.

On further investigation, it looks to me like dev-certs should be documented in the ASP.NET Core doc set, as this command is specific to ASP.NET Core. We should consider moving the Self-signed-certificates guide to the ASP.NET Core doc set too, alongside Enforce SSL
@Rick-Anderson  what do you think?

On further investigation, it looks to me like dev-certs should be documented in the ASP.NET Core doc set, as this command is specific to ASP.NET Core. We should consider moving the Self-signed-certificates guide to the ASP.NET Core doc set too, alongside Enforce SSL
@Rick-Anderson what do you think?

I agree but we should check with @javiercn

Sounds fine to me


ReactiveKit

Hi Everyone,
Much of you started asking for features that Bond is not really meant to support. Bond was primarily made for UIKit bindings, but with help from the community it grow beyond that, adding AppKit support and taking direction into FRP. There were requests for watchOS, error handling, better KVO and more.
All that would make Bond too heavy because it doesn't scale properly. I'm, however, on same page as you are so I did that, but differently. With the experience of Bond and knowing its shortcomings I started a new project - ReactiveKit. It has what Bond has plus much more. It scales well because UIKit extensions are now separate framework. It supports FRP and error handling for asynchronous tasks through Operation type. Observable is still there. ObservableArray has become ObservableCollection and it now supports arrays, sets and dictionaries. It has much better performance. KVO is now easier and safer. Threading is also enhanced.
Currently there is no AppKit support for OS X. That is planned as a separate framework. There are no batch updates of ObservableCollections yet and no support for multiple sections in table/collection view bindings. On every other aspect - its better. Initial version is on CocoaPods so if you're ready for an adventure, check it out - https://github.com/ReactiveKit/ReactiveKit.
What does this mean for Bond? Well, not much will change. I'll try to keep it simple and focused on UI bindings. It will not go deep into FRP.
Thanks!

I'm happy to help with the AppKit bindings when that project gets going.

That's much appreciated @tonyarnold. I'll setup the project within next few days, porting what we currently have, so we have something to start with.

This means that if I need ObservableCollections, which seems it should naturally be a part of bond, I need to buy the whole heavy weight of ReactiveKit, including the promises kit (AKA Operations). Sad.

@ramirez, actually, ReactiveKit has less lines of code then Bond and at the same time more features and better performance. So, hmm, no heavy weight.

Framework
Lines of code approx. (CocoaPods stats)

ReactiveKit
1900

Bond
2300

RxSwift
7800

ReactiveCocoa
10400

I've spent significant amount of time trying to make it as lightweight as possible while at the same time solving deficiencies of Bond.
Doing ObservableCollection (among other things) for Bond would introduce significant backward incompatible changes I don't want to introduce so that people can continue using Bond if it's enough for them. Who needs additional staff is welcome to upgrade to ReactiveKit.

Hmm... Ok. I'll try it. I'm also happy to contribute, if needed.
L.

@ramirez thanks, that's appreciated.

@srdanrasic I'll be glad to help with ReactiveKit too. CI / Coverage / Tests as usual :) If you add me to the organization, I'll set it all up.

@srdanrasic I would love to contribute. Where can I find the roadmap/TODOs?
Also, I think it would be a good idea to document the shortcomings that you have realised in SwiftBond, and how you have/will overcome them in ReactiveKit.

Thanks everyone! TODOs are listed in issues as enhancements - https://github.com/ReactiveKit/ReactiveKit/issues. Feel free to add your own.

Would it be possible to outline a migration guide from Bond to ReactiveKit?


Apache Pulsar Connector integration

Add connector for ingesting from Apache Pulsar

I would like to work on this if no one is already started working on it.

Please take it @SandishKumarHN . Thanks!

Sure, let me assign this to you.

@SandishKumarHN let me know if I can help with this one.

@HansOspina you take up this one if you like, currently occupied with different tasks.

@SandishKumarHN can we at least tackle it together? not sure I have the pinot experience to actually do it all by myself.

@HansOspina let's connect over pinot slack.

Hello, any news on this integration?
Thank you

@xiangfu0 Picking this up.


Translator Error When Using Fluent Functions With DisableMethodInvocation

Receiving an error when using fluent functions in CQL and DisableMethodInvocation is used.
Translator release: 1.5.4
Library:
library ALibrary version '1.0.0'

using FHIR version '4.0.1'

include FHIRHelpers version '4.0.001' called FHIRHelpers
include MATGlobalCommonFunctionsFHIR4 version '6.1.000' called Global

valueset "Diabetes Mellitus": 'http://cts.nlm.nih.gov/fhir/ValueSet/2.16.840.1.113883.3.3157.1329'

parameter "Measurement Period" Interval<DateTime>

context Patient

define fluent function "confirmed"(Conditions List<Condition>):
  Conditions C where C.verificationStatus ~ Global."confirmed"

define fluent function "active"(Conditions List<Condition>):
  Conditions C where C.clinicalStatus ~ Global."active"
    and C.abatement is null

define fluent function "activeOrRecurring"(Conditions List<Condition>):
  Conditions C
    where C.clinicalStatus ~ Global."active"
      or C.clinicalStatus ~ Global."recurrence"
      or C.clinicalStatus ~ Global."relapse"

define "Diabetes Conditions":
  ["Condition": "Diabetes Mellitus"]

define "Confirmed and Active or Recurring Diabetes Conditions":
  "Diabetes Conditions".confirmed().activeOrRecurring()

Error received:
Invalid invocation target: org.hl7.elm.r1.ExprressionRef
Per QMIG STU 2, "This option [DisableMethodInvocation ] should be used with Measures to ensure method-style invocation cannot be used within eCQMs."

The error message here could be better, so I'm going to mark this as an enhancement to improve the error message when accessing a fluent function with the disable method invocation option set.


update logging choices

turn on full path report
turn on full error message report
remove art throttle since it is already set elsewhere

Hi @rlcee,
You have proposed changes to files in these packages:

JobConfig

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for 97aeaa984a5c2a91dbb4d9fc23eec3e7ea8e0db7: build  (Build queue has 1 jobs)
About FNALbuild. Code review on Mu2e/Offline.


Testing removal of 'skim'

Temporary, might be closed.

@tgriesser This function is 4 years old and hasn't been changed. This is a long shot in the dark, but do you happen to remember the reasoning behind it? 😄
No tests are failing after removing it. Still not sure if that's reliable enough, tho.
Currently this is a progressive performance bottleneck as described in https://github.com/tgriesser/knex/issues/2518#issuecomment-369145795

Maybe it converts random classes etc. returned from drivers to plain objects?

Anyways I'm pretty if it is wanted then it would be faster to do with plain javascript loop. @koskimas has done lots of micro optimizations to obejction, he probably remembers from top of the head if there is something really inefficient in that implementation.

Maybe it converts random classes etc. returned from drivers to plain objects?

Exactly it. I forget which drivers it was now though

That makes sense.
Pressumably the only reason it affects so many dialects is because copy-paste when adding new dialects. I imagine mssql can be one such case.

Looks like this could be used also to copy object own properties to plain object https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign
If I didn't do something badly wrong with object assign skimming is ~20x faster https://jsperf.com/knex-skim (on chrome at least)

@elhigu I was about to suggest the same. V8 team has spent some time optimizing Object.assign recently. It will probably get even faster with node 10.

I think you could even skip the whole thing if obj.constructor === Object

@tgriesser @elhigu I'm comfortable removing this for both MySQL (from Tim's comment) and Postgres. I can't speak for other dialects, tho.
Anything we can do to speed up resultset is a win-win situation imo.
If you're ok with this I can update the PR.

@wubzz I believe this is good to go, just need to remember write to changelog about that returned row types might not be plain js objects anymore on those dialects, which were skimmed earlier.


Can't pass string as parameter to query

When using query function and passing &[&my_string] as params, I get compile error:
&[&prm]
  ^^^^ the trait `tiberius::prelude::ToSql` is not implemented for `std::string::String

Well it expects &ToSql which is only implemented by &str.
So something like &prm.as_str() should work (maybe even without the &).

Worked with & as_str(), thx.


(JS-0128) Should not have unused variables

Description
Found variables that are declared but not used anywhere.
Occurrences
There are 2 occurrences of this issue in the repository.
See all occurrences on DeepSource → deepsource.io/gh/guidroid/recruiter.tech/issue/JS-0128/occurrences/

@carolineignr is it under hackoctoberfest than I can do it!


Sync style with live version cmp-safety

Add all the context layers, make standalone

@chunhochow could you review and approve?


Add parameter to factory method for setting handler options

closes https://github.com/microsoft/kiota-http-dotnet/issues/233

Also, did you start working on unit tests for that? Did you run into any issues with those?

Added tests to the PR. As mentioned in #233 , I cannot run the tests locally unless I remove the analyzer reference from the .csproj file.

No need to expedite. Thanks tho.


Adds arrowDirection prop

This adds the ability to declare where the ToolTip displays

Thanks for the PR, can you update the doc too?

@jrichardlai Updated the documentation, PropType declaration and set the defaultProp value for arrowDirection

Thanks!


JS SDK v0.6.0 adds ts-morph devDependency

Upgrading repo from v0.5 to v0.6 threw the following error:

This was addressed by adding ts-morph to the project devDependencies. Is this intended behavior? Is it not a little forced to require developers to add another dependency they do not expect when using this library?

Good finding! I think ts-morph should be a dependency of near-sdk-js instead of a dev dependency of near-sdk-js. Let's fix it!

@idea404 since you already fixed it, can you please crate the PR?


Show Full Record

Long messages that have "Show Full Record" when Clicking show full record, the message pops up but then does not load.
Expected behavior
Expected the rest of the text to load
Environment

OS: iOS
Browser: Chrome
Argilla Version: 1.0.1

@splevine thank you for the feedback! Could you share any screenshots and such?

https://user-images.githubusercontent.com/4715331/202925955-aa6ede07-981d-488d-82ed-ecdf2b99bf4c.mov

Hi @splevine, thank you, we are doing some tests to simulate the behavior and it will be useful to have this extra information.

Does this erroneous behavior always happens on all records or only sometimes?
If pagination is changed to a single record per page, does this still happen?
Is there an error in Chrome's Debug console or weird console log?

Thank you again for your feedback

Hi @splevine, have you had a look and get a little more information? Thanks!


Dolphin 8.0.5 (8.0.x ?) does not start at all here under Windows 10, Germany

On my machine I downloaded release 7.1.27, installed it and it ran out of the box.
Then I downloaded release 8.0.5 and this version does not work at all. It mentioned, that the following libraries are not available:
msvcp140d.dll, vcruntime140d.dll and ucrtbased.dll
I assume, that the "d" means Germany and the newest runtimes from Microsoft do not deliver these language specific libraries any more ? So, what I have on my machine (system32) are:
msvcp140.dll, vcruntime140.dll and ucrtbase.dll

msvcp140d.dll, vcruntime140d.dll and ucrtbased.dll
I assume, that the "d" means Germany

No, it means Debug. You would need to have VS2022 to have those installed, but for normal use it isn't a good idea to run debug Dolphin binaries anyway. That release was an experiment that should have been deleted. I might generate a usable release soon, but for now to use 8 in beta form it is necessary to follow the instructions to clone the repo and build using a free copy of VS2022 and a few minutes of effort.
I've deleted the release, so I'm closing the bug.


Add SSDT Workload to Visual Studio 2019

Description
Improvment
SSDT tools now should be installed as workload for Visual Studio 2019. More information can be obtained here: https://docs.microsoft.com/en-us/sql/ssdt/download-sql-server-data-tools-ssdt?view=sql-server-ver15#ssdt-for-vs-2017-standalone-installer
Size: 500 Mb
Related issue:
https://github.com/actions/virtual-environments/issues/702
Check list

[+] Related issue / work item is attached
[ ] Changes are tested and related VM images are successfully generated

Could you please fill initial PR template to provide size of this workload?

"

Could you please fill initial PR template to provide size of this workload?

Thank you. I have added the info in the initial message.


[Feature request] multiple harpoons

Similar to Mylyn for eclipse, having multiple mark lists, depending on the task at hand. Right now you just have one great list of marks, would be nice if you could group them depending on the task at hand, so you can easily switch context.

I'm curious, what use case would this be for? Currently your marks already depend on what folder you launch nvim from, which already provides a lot of flexibility imo.
By "task at hand" in a projects do you mean, for example, in big projects with dozens of files you want to group files in a mark-lists by tasks, and have multiple lists to quick switch what you are working on?

Well it's very simple, harpoon gives you a single list. That list will mostly be of files "of interest to a certain task/area".  I couldn't wait for primeagen to implement it so I just made my own:
https://github.com/mlp1802/periscope.nvim


[BUG] New-PnPMicrosoft365Group -CreateTeam does not create a Team

Reporting an Issue or Missing Feature
Issue
Expected behavior
A new MS 365 Group is created, along with a corresponding Team
Actual behavior
A new MS 365 group is created, but no team. The Groups' .HasTeam property returns False
Steps to reproduce behavior
PS /home/user> New-PnpMicrosoft365Group -DisplayName "TeamTest" -Description "Foo" -MailNickname "Bar" -Owners "YOUROWNEREMAIL" -CreateTeam
DisplayName Id                                   HasTeam SiteUrl
----------- --                                   ------- -------
TeamTest    fb3c0767-a84f-4136-a357-f45558c23356 False   https://xxxxxxxxxxx.sharepoint.com/sites/Bar

What is the version of the Cmdlet module you are running?
1.10.0
Which operating system/environment are you running PnP PowerShell on?

[X] Windows
[ ] Linux
[ ] MacOS
[X] Azure Cloud Shell
[X] Azure Functions
[ ] Other : please specify

Resolving as issue not related to PNP Powershell.


How can we make RestartSession work?

Hi,
Great job on the project! I'm using it to play videos in VR and it works very well. One issue that I'm running into is restarting the View after hiding/showing the browser window. I tried uncommenting the RestartSession method, but, as the TODO comment states, the texture is not being updated after restart. Do you know what could be causing that or any ways to get around it?
Thanks!

Why do you want to restart the view?

I have a Unity Plane using your plugin for GeckoView. I want to be able to create/destroy that Unity Plane object, or at least hide and show the plane. Thanks!

Why don't you just set scale to 0 to hide the plane?

As far as I understand, this approach has some undesired side effects. What if we switch Unity scenes, and then the Plane object is destroyed? What if there is audio playing from the webview window and we "hide" it? GeckoView itself supports Play/Pause functionality since this is a standard behavior of Android views.
I haven't dug deep enough into this yet, but it seams the issue stems from restart implementation of OVROverlay. What do you think? I'm happy to try and debug this some more.

Check out DeactivateGeckoSession() and its reciprocal in BrowserView.cs and keep the browser gameobject alive through scenes

Thanks Ian! DeactivateGeckoSession works. Actually, OVROverlay has a 'hidden' property which we use instead of setting the scale to 0.


Persist editorstate

Summary
Below two utilities added;
convertFromRawToEditorState(editorStateRaw)
convertEditorStateToRaw(editorState)

What's the context of this PR? Also worth noting, fromJS and toJS exist in src/model/immutable/EditorState.js.

What's the context of this PR? Also worth noting, fromJS and toJS exist in src/model/immutable/EditorState.js.

Hi,
This is for the issue
https://github.com/facebook/draft-js/issues/2381

@claudiopro @mrkev gentlemen, couldn't produce those travis ci build errors on my dev environment.
Using vscode 1.45.1 on windows 10 machine.
Version: 1.45.1 (system setup)
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-14T08:27:35.169Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.14393

Installed Flow Language Support but it suffers Flow version 0.115.0  since we have ^0.125.0 in .flowconfig
npm run build never results with errors!
What should I check?

@claudiopro @mrkev gentlemen, couldn't produce those travis ci build errors on my dev environment.
Using vscode 1.45.1 on windows 10 machine.
Version: 1.45.1 (system setup)
Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
Date: 2020-05-14T08:27:35.169Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.14393

Installed Flow Language Support but it suffers Flow version 0.115.0 since we have ^0.125.0 in .flowconfig
npm run build never results with errors!
What should I check?

yarn run test-ci  is doing exactly that 👍   learning...

Actually I stand corrected, @mrkev is right. The static methods EditorState.fromJS() and .toJS() already exist and do a lot more than what you need. Let us know if you have further questions!

The fromJS() is a static method of EditorState, so this is how you'd use it: https://codesandbox.io/s/draft-js-tofromjs-08w8c?file=/src/App.js
However, it looks like the EditorState.fromJS() API is really new and didn't make it to an npm release yet. I will publish one in the coming days unless @mrkev was planning to do so already.

@claudiopro @mrkev
I've tested fromJS. Works fine. Looks like we can achive editor persistency requirements this way.
Wish I knew it at the very beginning :-)
Anyway, if there's anything I can help, I'd be glad to do it. You can assign me what you consider appropriate.
Thanks again


v0.4.3

Coverage remained the same at 100.0% when pulling ad0fa270616b87b7716dde84c5effa159a29c3ca on develop into 944a980d1cc1cc2a17b8089d16ae9251b80ed9c5 on master.


Mist accessing input file from S3 bucket

Getting error in accessing input file to query from S3 bucket.What i may be doing wrong?

@arshanvit
We cannot know what you’re doing wrong without any details :)
Anyway, if you want to make s3 reading/writing work, you have to do 2 things:

Add run-options = "--packages com.amazonaws:aws-java-sdk:1.7.4,org.apache.hadoop:hadoop-common:2.7.3,org.apache.hadoop:hadoop-aws:2.7.3” into your context config.
Add AWS credentials into your job (or into the config):

scala:
context.hadoopConfiguration.set("fs.s3a.access.key", “<AWS ACCESS KEY>”)
context.hadoopConfiguration.set("fs.s3a.secret.key", “<AWS SECRET KEY>”)

python:
self.context._jsc.hadoopConfiguration().set("fs.s3a.access.key", “<AWS ACCESS KEY>”)
self.context._jsc.hadoopConfiguration().set("fs.s3a.secret.key", “<AWS SECRET KEY>”)

context config:
spark-conf {
	spark.hadoop.fs.s3a.access.key = <AWS ACCESS KEY>
	spark.hadoop.fs.s3a.secret.key = <AWS SECRET KEY>
}

Note: this won't work if the bucket is in Frankfurt

Thanks :)


Should depend on the glibc version of iconv in Linux and no local package should be installed

GNU libc has iconv in-built. If available in the platform that should be used instead of installing a local libiconv. Some of the discussion is available in the issue #16.

You said on the other issue:

Yes. But the build.jl may need changes so that it does not download the libiconv in case the functionality exists. Seems the current provides directive is not honored as in Linux the debian package manager does not show libiconv as a registered package (it's subsumed into glibc). So one may need to check for a linking symbol like a library version or iconv function call.

The existing build.jl is supposed to check whether libc provides the needed functions first. And indeed on most Linux distributions there's no need to install the GNU libiconv package as glibc is enough. Could you check what happens if you remove validate = validate_iconv? If that works, then the validation of the function's behavior is failing for some reason.

Issue is closed with the merger of #19.


Squiz/ClosingDeclarationComment: add trait support

Description
As suggested by @jrfnl in https://github.com/PHPCSStandards/PHP_CodeSniffer/pull/340#issuecomment-1946387893, this PR changes the Squiz.Commenting.ClosingDeclarationComment sniff to require a closing declaration comment for traits as well.
Suggested changelog entry
Squiz.Commenting.ClosingDeclarationComment: closing declaration comments are now required for traits.
Related issues/external references
Suggested in https://github.com/PHPCSStandards/PHP_CodeSniffer/pull/340#issuecomment-1946387893
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

[ ] This change is only breaking for integrators, not for external standards or end-users.

[ ] Documentation improvement

PR checklist

[x] I have checked there is no other PR open for the same change.
[x] I have read the Contribution Guidelines.
[x] I grant the project the right to include and distribute the code under the BSD-3-Clause license (and I have the right to grant these rights).
[x] I have added tests to cover my changes.
[x] I have verified that the code complies with the projects coding standards.
[ ] [Required for new sniffs] I have added XML documentation for the sniff.

Rebased without changes. Merging once the build passes.


[v1.15] docs: ipsec: remove limitation for native-routing with L7 egress policy

Backport of

[ ] #32906

Once this PR is merged, a GitHub action will update the labels of these PRs:
 32906

/test-backport-1.15


Instructions for simple humans

Hi, I’m a simple human that uses the terminal quite a bit to commit stuff while trying not to break the repo. Tiny Care Terminal seems like a great way to have a prettier and more useful terminal window somewhere on screen. However, after doing the NPM dance, I have no clue how to configure it, where is this .env file you speak of? 🤔
Oh, and thank you for making this 🙌🏽.

Hi!
You can open your .bashrc file (located in your home folder but hidden) and add the following
## This is the tiny-care-terminal configuration variables
# List of accounts to read the last tweet from, comma separated
# The first in the list is read by the party parrot.
export TTC_BOTS='tinycarebot,selfcare_bot,magicrealismbot'

# List of folders to look into for `git` commits, comma separated.
export TTC_REPOS='~/Documents/Github'

# Location/zip code to check the weather for. Both 90210 and "San Francisco, CA"
# should be ok. It's using weather.service.msn.com behind the curtains.
export TTC_WEATHER='Athens'

# Set to false if you're an imperial savage. <3
export TTC_CELSIUS=true

# Unset this if you _don't_ want to use Twitter keys and want to
# use web scraping instead.
export TTC_APIKEYS=true

# Refresh the dashboard every 20 minutes.
export TTC_UPDATE_INTERVAL=10

# Twitter api keys
export CONSUMER_KEY='...'
export CONSUMER_SECRET='...'
export ACCESS_TOKEN='...'
export ACCESS_TOKEN_SECRET='...'

You can check if they are working in the terminal
$ printenv | grep TTC_*
TTC_BOTS=tinycarebot,selfcare_bot,magicrealismbot
TTC_APIKEYS=true
TTC_WEATHER=Athens
TTC_CELSIUS=true
TTC_UPDATE_INTERVAL=10
TTC_REPOS=~/Documents/Github

Then open tiny-care-dashboard
$ tiny-care-dashboard

Hope this helps a bit.

After following this guide and creating the .bashrcfile my self and pasting the configuration lines.. still doesn't work 😔 i mean the app works but cant configure it.
$ printenv | grep TTC_* returns nothing but $ printenv  does return some other configurations.

If you place it in bash_profile it seems to work.

I'll add a couple more instructions where to put these variables. Thanks!
On Wed, Apr 19, 2017 at 9:05 AM Kevin Hellemun<EMAIL_ADDRESS>wrote:

If you place it in bash_profile it seems to work.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/notwaldorf/tiny-care-terminal/issues/21#issuecomment-295324725,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABTkUnW4rT-kmFx6s8MqM60F9Ij8sUoPks5rxjDPgaJpZM4NBg_L
.

Added some instructions in 6a53e22


added wait stream param

Description of the Change
Added configurable timeout value to compensate probable lag or latency between the Twitter stream host and client that may vary depending on the client's circumstance/s e.g. location.
Is this change addressing any open issues?
No.
Is this change backwards compatible for existing users?
Yes.
Does this change require additional Twitter API knowledge?
No.

@HunterLarco can I assist with getting this merged? I'm seeing that very frequently 20 seconds is not enough for the heartbeat due to lag. It is consistently being killed at the 21 to 22 second mark.


Add EDI Secretary

The 20/21 committee was in agreement that adding an EDI Secretary to the committee would help to advance EDI initiatives within CompSoc.

See #32


Prod beta to prod stable

Codecov Report
Patch and project coverage have no change.

Comparison is base (44389b5) 21.41% compared to head (142bc05) 21.41%.

Additional details and impacted files
@@             Coverage Diff              @@
##           prod-stable     #212   +/-   ##
============================================
  Coverage        21.41%   21.41%
============================================
  Files               42       42
  Lines              864      864
  Branches           155      155
============================================
  Hits               185      185
  Misses             632      632
  Partials            47       47

Impacted Files
Coverage Δ

...nents/ServiceNotConfigured/ServiceNotConfigured.js
0.00% <ø> (ø)

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


EmbedMolecule() and EmbedMultipleConfs() very slow for monoatomic species

Description:
Conformer generation for monoatomic species seems to be incredible slow, the process taking multiple orders of magnitude longer than for molecules.
For example, generating a conformer for monoatomic oxygen takes 7-8 seconds while the same operation takes about half a millisecond for dioxygen (see Example).
The issue affects both the rdkit.Chem.rdDistGeom.EmbedMolecule() and rdkit.Chem.rdDistGeom.EmbedMultipleConfs() functions.

RDKit Version: 2019.03.1
Platform: macOS 10.14.5

Example:
>>> from rdkit.Chem import AllChem

>>> mol1 = AllChem.MolFromSmiles('O')
>>> mol2 = AllChem.MolFromSmiles('O=O')

>>> %timeit AllChem.EmbedMolecule(mol1)
7.74 s ± 518 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

>>> %timeit AllChem.EmbedMolecule(mol2)
613 µs ± 56.4 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

Any update on this issue?


System Informer .NET performance bug

Brief description of your issue
I'm a programmer, in C#.
I was monitoring a program in which I made excessive use of RAM and freeing it.
When I view the .NET performance tab I notice that none of the counter values are displayed.
Steps to reproduce (optional)
Debugging the program in question does not update the values of the counters.
I changed the Platform Target from Any CPU to x64, the setting from Debug to Release and it didn't update the counter values either.

Actual behavior (optional)
The counter values do not display any initial value, nor are they updated throughout runtime.

Environment (optional)
System Informer nightly 3.0.10254.5727
Windows 11 Enterprise 22H2 Build 22621.963

When I view the .NET performance tab I notice that none of the counter values are displayed.

Was this application .NET Core or .NET Framework?

Was this application .NET Core or .NET Framework?

Net 7.0 (net7.0)

Net 7.0

7.0 is .NET Core
Microsoft has removed the support for performance counters for all .NET Core based projects. They're now using a custom in-house protocol named EventPipe. The Windows implementation of the protocol has a major design flaw allowing processes to create fake pipes and terminate processes querying the counters and prevent the process from executing and makes the protocol completely unusable for third party projects (and thus hasn't been implemented by anyone including Sysinternals Process Explorer's .NET performance feature).
You have to use their counters tool going forward. Software can block or terminate the counters tool without affecting your task manager: https://learn.microsoft.com/en-us/dotnet/core/diagnostics/dotnet-counters

.NET Core apps profiling is not supported yet?

@rodion-m
No because Microsoft has completely removed all support for counters from .NET Core

.NET Core 3 and higher provides similar information through EventCounter API https://learn.microsoft.com/en-us/dotnet/core/diagnostics/event-counters

@TETYYS
The EventCounter API has a major security flaw where it calls GetNamedPipeClientProcessId and TerminateProcess. It cannot and will not be supported by System Informer.

@TETYYS
The EventCounter API has a major security flaw where it calls GetNamedPipeClientProcessId and TerminateProcess. It cannot and will not be supported by System Informer.

Where can I find information about that security issue, to inform me about it?


Guard against unnecessarily calling dump_graph in logging

resolves #4569
Description

Adds a new Lazy data type that can be reused anywhere in the code base (code quality passes mypy --strict without any ignores)
Includes mashumaro serialization
Has the offending cache events use the lazy type
Tests that the existing fire_event function will work with this new lazy construction
Tests that all relevant subclasses of Cache apply this lazy evaluation strategy to catch similar issues in the future

The Challenge
Mashumaro does not support composing serialization strategies which makes serializing generic types, like Lazy tedious and error prone. Even if the wrapped type T has a mashumaro serialization strategy, it must be explicitly called by the serializer for Lazy itself. To be explicit, this means we would have to manually code mashumaro serialization strategies for all combinations of types that Lazy wraps in our code base e.g. Lazy[str], Lazy[int], Lazy[Dict[str, List[str]]].
I have attempted to dig into Mashumaro's metaprogramming library to see if I can write a truly generic serialization for the Lazy type, but I haven't been able to figure it out.
Questions

We a few paths forward on this PR:

Use this Lazy type to more easily communicate intention, and get good mypy type checking but introduce room for runtime errors when we forget to add a serialization strategy when it's going to be serialized like in the events module.
Ditch Lazy so we don't have this gap to introduce new errors in serialization and instead pass unapplied methods with # type: ignore because mypy doesn't handle them well.
Spend more time looking into mashumaro metaprogramming to potentially get the best of both worlds.

Suggested Review Flow

Look at the two test cases. They should very clearly show the expected behavior.
Read the implementation for lazy behavior which is found in dbt.lazy.py::Lazy.
Find the call sites for the updated cache events to see how it all fits together from beginning to end.

Checklist

[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the CHANGELOG.md and added information about my change

Good call on the README, @emmyoop. I've added a new logging section now.

Hi @nathaniel-may
As the author of mashumaro, I would be happy to help you sort out the problem if you would explain what you want to achieve. If it's not difficult, could you please write a sample code explaining the problem? You can do it here or create a new issue in the mashumaro repo.
As far as I understand, you want to serialize generic types. Can GenericSerializableType help with it?

Hi, @Fatal1ty
That's very kind of you to offer to help here. Mashumaro does have several options for serializing generic types, but I haven't managed to figure out how to defer specifying the serialization for the concrete inner type to mashumaro.
Using the code in this PR, I currently have a SerializationStrategy registered with a config that returns a dict since that's the only concrete type we're using it with at this time:
class LazySerialization1(SerializationStrategy):
    def serialize(self, value) -> Dict[str, List[str]]:
        return value.force()

class Config(MashBaseConfig):
        serialization_strategy = {
            ...
            Lazy[Dict[str, List[str]]]: LazySerialization1()
        }

And this works great, but when I swap out the class to instead use a value of Lazy[str] mashumaro fails to serialize with the following exception: mashumaro.exceptions.UnserializableField: Field "dump" of type Lazy[str] in DumpBeforeAddGraph is not serializable.
What I want is a way for the above to work for any type Lazy[T] when T is either natively serializable (like str), or when T is a serializable type, or there exists a serialization strategy for T without needing to write code to handle serializing T again in the generic type's serializer. If something like the following worked that would be much nicer than what I've come up with:
psudo-code:
class LazySerialization(SerializationStrategy):
    def serialize(self, value):
        return value.force()

class Config(MashBaseConfig):
        serialization_strategy = {
            ...
            Lazy: LazySerialization()
        }

@nathaniel-may thank you for the details!
I came up with the idea of using DataClassDictMixin with SerializableType since it's legit. The code should be written as follow:
from dataclasses import dataclass
from datetime import date
from typing import Generic, TypeVar
from mashumaro import DataClassDictMixin
from mashumaro.types import SerializableType

T = TypeVar("T")

@dataclass
class Lazy(Generic[T], DataClassDictMixin, SerializableType):
    inner: T

    def _serialize(self) -> T:
        return self.to_dict()['inner']

    @classmethod
    def _deserialize(cls, value):
        return cls.from_dict({'inner': value})

@dataclass
class LazyTest(DataClassDictMixin):
    lazy: Lazy[date]

obj = LazyTest.from_dict({'lazy': '2022-01-31'})
assert obj.lazy == Lazy[date](date(2022, 1, 31))
print(obj.to_dict())
assert obj.to_dict() == {'lazy': '2022-01-31'}

But unfortunately it won't work as expected. I think it's a bug that could be fixed. I'll dig deeper and try to fix it.

@gshank suggested Lazy live in dbt.helper_types which makes sense to me so I've moved it there.

@Fatal1ty, since I've closed this PR if you'd like to continue our conversation in an issue or PR in the mashumaro repo, I'm happy to do so.

Nice work here!!
Is this a fix we're going to be able to backport to 1.0.latest, for inclusion in v1.0.2 (given it resolves a significant performance regression)? Or does it intersect with / depend on https://github.com/dbt-labs/dbt-core/pull/4505 in a way that makes backporting too difficult?

@jtcohen6 this definitely depends on #4505. If it's important to backport, I can make a different commit to 1.0.latest but the user-facing behavior won't be exactly the same. If I exclude the dump field from the structured logging output, but keep it in the string message, I could get this fixed over there too without mashumaro.

@nathaniel-may I'd be supportive of fixing the performance regression, even at the cost of losing that information in the structured logging output. To that end:

If I exclude the dump field from the structured logging output, but keep it in the string message, I could get this fixed over there too without mashumaro.

This feels like the right balance of effort + impact

But unfortunately it won't work as expected. I think it's a bug that could be fixed. I'll dig deeper and try to fix it.

@nathaniel-may actually I missed a much simpler solution that works right now. You can wrap a value in the dataclass and use serialization hooks:
from dataclasses import dataclass
from datetime import date
from typing import Generic, TypeVar
from mashumaro import DataClassDictMixin

T = TypeVar("T")

@dataclass
class Lazy(Generic[T], DataClassDictMixin):
    inner: T

    def __post_serialize__(self: T, d):
        return d.pop('inner')

    @classmethod
    def __pre_deserialize__(cls, d):
        return {'inner': d}

    # @classmethod
    # def __post_deserialize__(cls, obj: T):
    #     # if you want to get LazyTest(lazy=datetime.date(2022, 1, 31))
    #     # instead of LazyTest(lazy=Lazy(inner=datetime.date(2022, 1, 31)))
    #     # but serialization will be broken because date doesn't have to_dict
    #     return obj.inner

    class Config:
        serialization_strategy = {
            # ...
        }

@dataclass
class LazyTest(DataClassDictMixin):
    lazy: Lazy[date]

obj = LazyTest.from_dict({'lazy': '2022-01-31'})
print(obj)
assert obj.lazy == Lazy[date](date(2022, 1, 31))
print(obj.to_dict())
assert obj.to_dict() == {'lazy': '2022-01-31'}


Profile needs to include properties for Contributed By and License.

OCP documents need to include the contributor and the license with which the document is contributed.  As a JSON file, the profile needs to have properties to contain these values.

The current Profile format specification does not contain these properties.  Await the Redfish Forum to update the Redfish Profile Specification to include these properties, then the profiles will be updated.

The "License" and "ContributedBy" property was added to Redfish Profile schema in 2021.2.

Fixed in OpenRMC1_1 in https://github.com/opencomputeproject/HWMgmt-OCP-Profiles/commit/b4da82a18ad2ea5368f6d899fe3466c69f284eea

Closed.


Align serialization of references on ResolveInfo

See https://github.com/LIonWeb-org/organization/issues/36 and discussion in PR #6 .

Done in the meanwhile.


doc: release-notes: Add ARM64 and IPC release notes

For Zephyr v3.2.0
Signed-off-by: Carlo Caione<EMAIL_ADDRESS>

Sorry I caused a merge conflict by adding more stub entries, can you rebase? Thanks!


Propose a better flow to handle async result from detectExposure on Android

What?
Calling detectExposure on Android doesn't return result immediately like iOS. Android needs to wait for result from Broadcast Receiver which could take a long time. In some cases, the app is closed before the receiver sends summary back.

Exploration summary:
Existing flow from react-native-background-fetch library
On Andorid:

App is launched the first time.
The app calls BackgroundFetch.configure and then BackgroundFetch.scheduleTask.
The lib will schedule background job to run the app according to the configuration.

On iOS:

App is launched the first time.
The app calls BackgroundFetch.configure and then BackgroundFetch.scheduleTask.
The lib will schedule background job to run the app according to the configuration.
Based on the docs here https://github.com/transistorsoft/react-native-background-fetch#implementation, it says Unlike iOS, the Android implementation can continue to operate after application terminate (stopOnTerminate: false) or device reboot (startOnBoot: true).. This is not true based on what we have been tested. iOS can still run and show notifications after the app is terminated.
Unlike Android, there is no need to register for headlessTask AppRegistry.registerHeadlessTask https://reactnative.dev/docs/headless-js-android.html. It appears that iOS can run the whole app headless by default as long as iOS background job is run. When the whole app is run headless, it will render HomeScreen, hence calling ExposureNotificationService.start, hence updating exposureStatus.

For Option 2:
Would the timeout not apply if we used WorkManager? From the linked GH issue:

The only thing WorkManager will do is simplify the code a bit. It won’t change the actual performance. Under-the-hood, WorkManager is using JobScheduler for devices running >= Lollilop; AlarmManager for everything else.

I agree that increasing the timeout is not a good idea, maybe it takes 30s in a good phone in good conditions, but if for decent hardware we're seeing issues, I'd assume minutes for lower-end phones on poor 3G.
For Option 4
I am not really a react native dev, but I agree that option 4 makes a lot of sense.
Would you be able to split this in say, three milestones?

option 1 first (with the "you're seeing yesterday's notification" bug)
option 1 + 3: local notification in native layer.

could we pass the translation from the JS layer ahead of time so we don't have to port translation into native?

option 4: all of the above I guess?

@eljojo Thanks for the comments.

Regarding option 2: that's what I am thinking. It's not an ideal solution.
I would like to just go straight to option 4 (included option 1). Combining option 3 isn't a good addition as it introduces more fragment in the code.

PR is up https://github.com/cds-snc/covid-shield-mobile/pull/640

Close as done. Tested with @timarney Let's see how it perform.


Fixing is_cumulus_vx condition

Using square bracket for tests doesn't call the function at all, but checking if "is_cumulus_vx" is or not an empty string. This means the install_license function was never invoked, both for Cumulus or Cumulus VX.

Coverage remained the same at 77.245% when pulling 12dbfeaa60305f066db40938145af781a8bbc83a on ale-cumulus-license-fix into c51ae98cbc229bc27838a523f5e6083800ef7b57 on master.

@acarpi there are no Celestica's available to verify that this is now working on physical switches. Have you verified that this resolves the issue?

I tested it on a Celestica and Cumulus VX. You should be able to verify this issue with any Cumulus box, assuming no license is installed prior ZTP.


Running deployctl using --unstable

Hey! I'm trying out Deno Deploy and wanted to connect to a MySQL database, when I important and construct the Client class from the mysql third-party package. I get notifified that certain properties don't exists on the Deno instance and that I might want to use the --unstable flag.
So I figured same old, same old and added --unstable but this threw an error. After I checked out the help text I noticed that there isn't an option for the unstable flag, Does this mean that it's impossible to run packages that require --unstable using Deno Deploy? I figure this will cross of alot of packages from being compatible with the hosting service

More context, in case anyone else wonders:

https://github.com/denoland/deno/discussions/13607
https://github.com/denoland/deploy_feedback/issues/22


Flutter Multiplying double 0.0 by -1.0 gives -0.0 instead of 0.0

Code:
@override
Widget build(BuildContext context) {
double num = 0.0;
num = num * -(1.0);
print('num');
print(num); //  prints -0.0 instead of 0.0
return Scaffold(
appBar: AppBar(
title: Text(widget.title),
),
body: Center(
child: Text(num.toString()) // Shows -0.0 on emulator and real device
));
}

In the future please file bugs about Dart itself in the Dart issue tracker rather than the Flutter issue tracker.
However, this isn't a bug. Dart uses IEEE-754 doubles, which uses signed zeros.


Reduced Classes

I have tried to reduce Labels 20 classes to 3 classes, we can get results predected as class 0.
Do you have any ideas??

Labels: -1 license plate, 0 unlabeled, 1 ego vehicle, 2 rectification border, 3 out of roi, 4 static, 5 dynamic, 6 ground, 7 road, 8 sidewalk, 9 parking, 10 rail track, 11 building, 12 wall, 13 fence, 14 guard rail, 15 bridge, 16 tunnel, 17 pole, 18 polegroup, 19 traffic light, 20 traffic sign, 21 vegetation, 22 terrain, 23 sky, 24 person, 25 rider, 26 car, 27 truck, 28 bus, 29 caravan, 30 trailer, 31 train, 32 motorcycle, 33 bicycle
num_classes = 20
full_to_train = {-1: 19, 0: 19, 1: 19, 2: 19, 3: 19, 4: 19, 5: 19, 6: 19, 7: 0, 8: 1, 9: 19, 10: 19, 11: 2, 12: 3, 13: 4, 14: 19, 15: 19, 16: 19, 17: 5, 18: 19, 19: 6, 20: 7, 21: 8, 22: 9, 23: 10, 24: 11, 25: 12, 26: 13, 27: 14, 28: 15, 29: 19, 30: 19, 31: 16, 32: 17, 33: 18}
train_to_full = {0: 7, 1: 8, 2: 11, 3: 12, 4: 13, 5: 17, 6: 19, 7: 20, 8: 21, 9: 22, 10: 23, 11: 24, 12: 25, 13: 26, 14: 27, 15: 28, 16: 31, 17: 32, 18: 33, 19: 0}
full_to_colour = {0: (0, 0, 0), 7: (128, 64, 128), 8: (244, 35, 232), 11: (70, 70, 70), 12: (102, 102, 156), 13: (190, 153, 153), 17: (153, 153, 153), 19: (250, 170, 30), 20: (220, 220, 0), 21: (107, 142, 35), 22: (152, 251, 152), 23: (70, 130, 180), 24: (220, 20, 60), 25: (255, 0, 0), 26: (0, 0, 142), 27: (0, 0, 70), 28: (0, 60,100), 31: (0, 80, 100), 32: (0, 0, 230), 33: (119, 11, 32)}

Are you trying to change the number of classes for Cityscapes? You will need to change num_classes,full_to_train and train_to_full, but otherwise it should work.

Yes, I want to train  Cityscapes data.
Thank you!!


Added more unit tests

Current coverage is 70.84%

Merging #7 into master will increase coverage by +26.07% as of ae62dbd

@@            master      #7   diff @@
======================================
  Files           27      37    +10
  Stmts          498     950   +452
  Branches         0       0
  Methods                  0
======================================
+ Hit            223     673   +450
  Partial          0       0
- Missed         275     277     +2

Review entire Coverage Diff as of ae62dbd

Powered by Codecov. Updated on successful CI builds.


Fix a bug in TextStringBuilder ensureCapacity() when new capacity >= Integer.MAX_VALUE / 2

In order to show that the old version has a bug that the new one fixes I added a test with the minimum such value:
sb.ensureCapacity(Integer.MAX_VALUE / 2);
It passes locally but throws a Java heap space error after commit

Hello @iliyanzan
Closing:

The PR fails the build
This has been addressed differently in git master or a snapshot build from https://repository.apache.org/content/repositories/snapshots/org/apache/commons/commons-text/
Please review git master and see if your use case is fixed
TY!


Update on font style changes

Fixes #21
Only visible TextEditors get refreshed when you change the settings so this updates on active item change to catch up inactive tabs, and regardless, the change event fires before the presenter updates, so setTimeout 0 is used to push the update to the end of the call stack.

LGTM. Thanks!


Confusing browsing/peek documentation

This page mentions a State property, which doesn't exist on the Message objects that Peek returns using the library that the Peek link references. This page seems to be mixing different libraries (or maybe versions) together without mentioning them, which is confusing.

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

ID: 162ed183-3df1-ca16-ea20-94b7c55136e0
Version Independent ID: cb33f224-d71d-6e0e-4b03-834be7fde9ce
Content: Azure Service Bus - message browsing - Azure Service Bus
Content Source: articles/service-bus-messaging/message-browsing.md
Service: service-bus-messaging
GitHub Login: @axisc
Microsoft Alias: aschhab

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Hello @josh-endries, the PeekBatch/PeekBatchAsync library provided in the document does have a State property in its BrokeredMessage Class. Please let me know if you have any additional questions.

Yes, but the Message object returned by the Peek/PeekAsync doesn't. Additionally, the PeekBatch library is legacy, superseded by the Peek library, so going forward (and currently, really) the typical object doesn't. Maybe it could clarify this in the part that references Status.


Update spring-boot up to 2.1.3.RELEASE

https://spring.io/blog/2019/02/15/spring-boot-2-1-3-available-now

Will be fixed in scope of #24


clients fail with a heartbeat exception despite ongoing traffic from server

I think this is because the server sends traffic and therefore does not need to send a heartbeat.
A workaround is to hack IO.on_read to set the Channel0._last_heartbeat to time.time() if the frame is not for Channel0.

I had another failure after 14 hours of testing multiple apps, so maybe my hack is not complete...
Pivotal responded thus:
On 11 April 2015 at 13:49:06, Laing, Michael<EMAIL_ADDRESS>wrote:

. Once heartbeats have been negotiated, are they always sent
by the server on the interval or are they sent only if other traffic
has not been sent by the server in the interval?

They are always sent at half the timeout intervals but can be paused in some circumstances.
However, both peers must treat any traffic as heartbeats (or well, connection activity).

. Are heartbeats always sent by the server per connection and
on the channel that negotiated the connection (channel 0 in my
case)?

Yes.
Channel 0 is special: connection negotiation happens on it, heartbeats are sent on it.
Applications cannot use it.
Relevant implementation bits:

https://github.com/rabbitmq/rabbitmq-server/blob/master/src/rabbit_reader.erl#L1063-1069
https://github.com/rabbitmq/rabbitmq-server/blob/master/src/rabbit_heartbeat.erl

Let me know if you have more questions.
MK
Staff Software Engineer, Pivotal/RabbitMQ

I'll try and get this addressed this week.

Hi Gavin, I'm still having issues with the head code.
I have a simple publisher publishing every 10 secs to a topic exchange. So no heartbeats are sent as traffic is flowing continually. My consuming client gets a connection reset after 15 mins - rabbitmq logs:
=INFO REPORT==== 27-Jul-2015::11:18:21 ===
accepting AMQP connection <0.26763.0> (<IP_ADDRESS>:34141 -> <IP_ADDRESS>:5672)
=INFO REPORT==== 27-Jul-2015::11:18:23 ===
accepting AMQP connection <0.26779.0> (<IP_ADDRESS>:34142 -> <IP_ADDRESS>:5672)
=ERROR REPORT==== 27-Jul-2015::11:33:21 ===
closing AMQP connection <0.26763.0> (<IP_ADDRESS>:34141 -> <IP_ADDRESS>:5672):
{heartbeat_timeout,running}
Looking through the debug logs I dont see any "base|write_frame|Writing frame: Heartbeat" calls - I guess because there are no heartbeats being sent as traffic is continually flowing.
$ grep -i -e heartbeat -e write_frame do_sub.log
2015-07-27 11:18:21,892|DEBUG|12348|base|write_frame|Writing frame: ProtocolHeader
2015-07-27 11:18:21,943|DEBUG|12348|base|write_frame|Writing frame: Connection.StartOk
2015-07-27 11:18:21,957|DEBUG|12348|base|write_frame|Writing frame: Connection.TuneOk
2015-07-27 11:18:21,960|DEBUG|12348|base|write_frame|Writing frame: Connection.Open
2015-07-27 11:18:22,013|DEBUG|12348|heartbeat|_start_timer|Started a heartbeat timer that will fire in 300 sec
2015-07-27 11:18:22,024|DEBUG|12348|base|write_frame|Writing frame: Channel.Open
2015-07-27 11:18:22,051|DEBUG|12348|base|write_frame|Writing frame: Exchange.Declare
2015-07-27 11:18:22,070|DEBUG|12348|base|write_frame|Writing frame: Queue.Declare
2015-07-27 11:18:22,110|DEBUG|12348|base|write_frame|Writing frame: Queue.Bind
2015-07-27 11:18:22,134|DEBUG|12348|base|write_frame|Writing frame: Basic.Consume
2015-07-27 11:23:22,037|DEBUG|12348|heartbeat|_check|Data has been received, exiting heartbeat check
2015-07-27 11:23:22,039|DEBUG|12348|heartbeat|_start_timer|Started a heartbeat timer that will fire in 300 sec
2015-07-27 11:28:22,044|DEBUG|12348|heartbeat|_check|Data has been received, exiting heartbeat check
2015-07-27 11:28:22,046|DEBUG|12348|heartbeat|_start_timer|Started a heartbeat timer that will fire in 300 sec
2015-07-27 11:33:22,052|DEBUG|12348|heartbeat|_check|Data has been received, exiting heartbeat check
2015-07-27 11:33:22,054|DEBUG|12348|heartbeat|_start_timer|Started a heartbeat timer that will fire in 300 sec
I've rerun the tests using pika and I can see the heartbeat being sent back to rabbitmq every 580 secs when data is flowing constantly.
2015-07-27 10:38:38,713|DEBUG|11591|heartbeat|_send_heartbeat_frame|Sending heartbeat frame
2015-07-27 10:48:18,724|DEBUG|11591|heartbeat|_send_heartbeat_frame|Sending heartbeat frame
2015-07-27 10:57:58,738|DEBUG|11591|heartbeat|_send_heartbeat_frame|Sending heartbeat frame
2015-07-27 11:07:38,750|DEBUG|11591|heartbeat|_send_heartbeat_frame|Sending heartbeat frame
Should the heartbeat.Checker._check() method send a heartbeat frame if data has been received?
Also I think there may be a bit of a race condition if the consumer checker timer triggers before a heartbeat is received if traffic is flowing and then stops - self._last_heartbeat is initialised to 0 and only set to a proper time if a heartbeat is received - the snippet below shows that its still at 0 when this check is done ... let me try to reproduce but I guess maybe the last_heartbeat time should be updated on Data Receive???
2015-07-24 22:57:59,297|DEBUG|11529|heartbeat|_check|Data has been received, exiting heartbeat check
2015-07-24 22:57:59,298|DEBUG|11529|heartbeat|_start_timer|Started a heartbeat timer that will fire in 300 sec
2015-07-24 23:02:59,300|DEBUG|11529|heartbeat|_check|Checking for heartbeat, last:<PHONE_NUMBER> sec ago, threshold: 600
2015-07-24 23:02:59,301|ERROR|11529|heartbeat|_check|Have not received a heartbeat in<PHONE_NUMBER> seconds
Hope this all makes sense - if not let me know and I'll try and clarify what I'm seeing.
Brendan

Heartbeating has been completely changed for 1.0 and this problem should be go away.


Drag not working on mobile

On mobile chrome, android and ios, dragging the slider is not working. Touching on the bar will set the slider correctly though. Also, thank you very much for the plugin.

I noticed this as well, the timeline control is not very mobile friendly.

+1 .. any ideas how to fix this? Would love to use this plugin on a mobile-focused project but not entirely sure if it's capable.


sysflash: pm_sysflash: Fix incorrect include of NSIB image define

A mismatch in defines was made for _image_1_primary_slot_id resulting in some configurations not working. This fixes that the linker variable is exposed for mcuboot so that it knows which slot is running an which slot a bootloader upgrade is to be put into.
Ref. NCSDK-19223

@sigvartmh sdk-nrf manifest update PR needed here.

manifest PR here https://github.com/nrfconnect/sdk-nrf/pull/12726


[BUG] Can't use : in JSONPath queries

Insomnia Version: Version 5.9.6 (<IP_ADDRESS>3)
Operating System: Max OSX 10.12.6

Details
When forming a JSONPath query like below, the parser complains that it is an invalid query:
$._links.['account:details']
I tested the query on here: https://jsonpath.curiousconcept.com and with version 0.3.0 and newer it works.

Hi @kemalenver, this seems to be working for me. It might have been fixed when I updated the JSONPath dependency in a previous release.
As you can see here, the original JSON is on the left, and the filtered JSON is on the right.
Also, you have an extra dot before your first bracket that is not supposed to be there.

I just realized that it was probably the extra dot causing your problems. If you delete that, it should work fine.

Closing this for now as it seems to work.


Extract the render-to-string portion of to_html() into its own public method

Add a .to_inline_html() method to Plot which can then be sent elsewhere for rendering.
For example, when using in Evcxr this turns this construction:
let plotly_file = "/tmp/plot.html";
plot.to_html(plotly_file);
let plotly_contents = fs::read_to_string(plotly_file).unwrap();
println!("EVCXR_BEGIN_CONTENT text/html\n{}\nEVCXR_END_CONTENT", plotly_contents);
fs::remove_file(plotly_file)?;

into this one-liner:
println!("EVCXR_BEGIN_CONTENT text/html\n{}\nEVCXR_END_CONTENT", plot.to_inline_html());

I debated adding a helper method for that wrapping - I'm happy to do so but I feel it would be presumptive without an OK from @igiagkiozis since they might not wish to officially support such usage, and interested parties probably are using DARN anyway which is a better place for it.
Closes #12 as long as people are happy to do their own root div or Evcxr content wrapping.

@Xymist I just found this whilst looking to introduce some improvements for DARN... I want to make some changes to reduce notebook sizes (sometimes 6-7MB) to 10-100KB. I'm torn between forking  igiagkiozis/plotly and making these improvements directly or introducing them as a post-processing step in DARN.

Hi @Xymist, please re-direct the pull request to the dev branch.

@igiagkiozis - Done, thank you.

@Xymist I just found this whilst looking to introduce some improvements for DARN... I want to make some changes to reduce notebook sizes (sometimes 6-7MB) to 10-100KB. I'm torn between forking  igiagkiozis/plotly and making these improvements directly or introducing them as a post-processing step in DARN.

@shahinrostami the size is due to every plot being self contained, which means that plotly.js is copied in every single plot. This is quite inefficient for this use case. I'm currently looking into alternative ways to better support this use case.

@Xymist, @shahinrostami I've pushed an implementation that works with inline html nicely without the extra js dependencies. You can try it out in the latest dev branch; this feature will be released in version 0.5.xx


Indentation problem while going to a new line in fish

When I write the following

and press enter, it indents all lines one block right.

I also just tried this with fish and didn't have the issue.


Add public scratch buckets support and use it for m2lines

Ref https://github.com/2i2c-org/infrastructure/issues/2123
Todo:

[ ] add docs
[ ]

terraform plan output:
Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # google_storage_bucket.user_buckets["persistent-public"] will be created
  + resource "google_storage_bucket" "user_buckets" {
      + force_destroy               = false
      + id                          = (known after apply)
      + location                    = "US-CENTRAL1"
      + name                        = "m2lines-persistent-public"
      + project                     = "m2lines-hub"
      + public_access_prevention    = (known after apply)
      + self_link                   = (known after apply)
      + storage_class               = "STANDARD"
      + uniform_bucket_level_access = (known after apply)
      + url                         = (known after apply)

      + versioning {
          + enabled = (known after apply)
        }

      + website {
          + main_page_suffix = (known after apply)
          + not_found_page   = (known after apply)
        }
    }

  # google_storage_default_object_access_control.public_rule["prod.persistent-public"] will be created
  + resource "google_storage_default_object_access_control" "public_rule" {
      + bucket       = "m2lines-persistent-public"
      + domain       = (known after apply)
      + email        = (known after apply)
      + entity       = "allUsers"
      + entity_id    = (known after apply)
      + generation   = (known after apply)
      + id           = (known after apply)
      + project_team = (known after apply)
      + role         = "READER"
    }

Plan: 2 to add, 0 to change, 0 to destroy.

Changes to Outputs:
  ~ buckets = {
      + persistent-public  = "m2lines-persistent-public"
        # (4 unchanged elements hidden)
    }

@consideRatio, thanks a lot for the feedback. I've just updated this PR with:

made bucket_public_access be its own terraform variable (a list holding the bucket names), instead of having it be a field of hub_cloud_permissions

this simplified the config
and since it isn't linked to a namespace, I think having it separate from the hub cloud permissions is less confusing.

added the public bucket name to the bucket_admin_access list for m2lines
updated the terraform plan output above with the new changes

What do you think about skipping persistent in the naming, and just go with m2lines-public as a name and PUBLIC_BUCKET env var?

Hmm, I think it makes sense to be explicit that the bucket was given a persistent nature, esp since we can also have (if requested) buckets that are not persistent, but public, right>?

Thanks a lot for the feedback @consideRatio and @sgibson91. I'm currently applying the changes to the cluster, then I'll merge this, then report back in the ticket.


Simplify Testing setup with Makefile

All of the prow end to end tests that we have use the same setup where we build the binary then run the script. Example: https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes-sigs/hydrophone/hydrophone-periodic.yaml#L32-L34
We could simplify this by adding a make command called test-e2e in the Makefile that runs build then invokes the hack/run-e2e.sh script.
If anyone wants to take a look at this, please feel free to assign yourself to the issue. And if there are any questions, reach out in this issue or to me on slack at @rjsadow
/good-first-issue

cc @NaveenGokavarapu19 If you're interested please assign yourself!

/assign

Fixed with #122


Exponential Backoff with retries

I just wanted to ask a clarifying question. Is there exponential backoff with the retries? As far as I could tell, it only waits 1 sec before retrying.
Is there a way to implement it?

Yes we can do that:
Update this code part
def retry(self, func, **kwargs) -> Any:
attempts = 0
error = Exception()
while attempts < self.retries:
try:
return func(**kwargs)
except Exception as error:
attempts += 1
error = error
time.sleep(self.retry_sleep)
To instead use @backoff
See https://pypi.org/project/backoff/ would do the trick. Believe one can add number of retry with Backoff as well which is cleaner solution then the current option.
If you do a PR for this I will approve it