Editorial pass on cmdlet ref

Editorial pass on cmdlet ref
I have lots of notes and bugs to report as we review this PR.

@sdwheeler if you have permission to merge, feel free to merge when you're done, otherwise just tag us and we can merge!  Thanks again!

Also wanted to add that I am merging a PR that adds ValueFromPipelineByPropertyName = true to Set-PSResource.  Could you add this to the docs too @sdwheeler?

@alerickson I don't have merge rights. I am done with my edits for now. It is ready to merge when the build checks are complete.


ensure cache_dir exists

Currently if you install pip-tools on a machine for the first time and use the --rebuild flag the very first time you run it, you get the following exception:
root@ff179494a044:/# pip-compile --rebuild requirements.in
Traceback (most recent call last):
  File "/usr/local/bin/pip-compile", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/piptools/scripts/compile.py", line 145, in cli
    results = resolver.resolve()
  File "/usr/local/lib/python2.7/site-packages/piptools/resolver.py", line 62, in resolve
    self.dependency_cache.clear()
  File "/usr/local/lib/python2.7/site-packages/piptools/cache.py", line 107, in clear
    self.write_cache()
  File "/usr/local/lib/python2.7/site-packages/piptools/cache.py", line 102, in write_cache
    with open(self._cache_file, 'w') as f:
IOError: [Errno 2] No such file or directory: u'/root/.cache/pip-tools/depcache-py2.7.json'

Thanks, @scottbelden !


HealthCheckRoutes Provider, so user can add it to their own server

Created an extension provider for HealthCheckRoutes, so users can get the routes and add it to their own server

@shakeeb-upstart would you be able to add a unit test for regression purposes?


[Docs] Need a thorough API reference for all user-facing parts

Is your report related to a problem? Please describe.
Between not being strict about which functions/fields are exported vs. not, lack of JSDocs on exported properties, and complex types due to usage of Zod and its lack of type referencing, we should have a generated API reference in addition to the procedural guides we have today to give an idea of what's possible and configurable at a glance.

https://github.com/firebase/genkit/pull/123


Intro controlled terms + term suggestion for keywords

Intro controlled terms + term suggestion for keywords to further increase data integration
adding other taxons (postponed to later point); but in controlledTerms BiologicalOrder etc, will be added as study targets

@olinux remaining update for changing the property name in FileBundle (groupedBy -> groupingType) is already done in v4 without the PR

@olinux could you review and merge if everything is fine?


Laboratories revamp

All changes including the OpenSouthCode ones.
Added Ansible 2.8 and GCE Dyn Inventory changes

I don't know why this PR shows so many commits... most of them were merged in #2. Closing in favour of another (upcoming) PR that will contain just new additions since #2


Modal shifts navbar-toggler on fixed navbar

Demo: https://codepen.io/anon/pen/WdBvpB
I've seen a similar issue before, but I could not find it...

Closed thanks to: #25745


[Feature request] Add missing classes for viewport-based sizing

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
.max-vw-100 { max-width:  100vw !important; }
.max-vh-100 { max-height: 100vh !important; }

Motivation and context
Bootstrap already has these:
.min-vw-100 { min-width:  100vw !important; }
.min-vh-100 { min-height: 100vh !important; }

.vw-100 { width:  100vw !important; }
.vh-100 { height: 100vh !important; }

So the missing classes would complete the viewport-based sizing feature. I found a need for them in layouts.
Similar to #28965 which was closed by OP.

Closing as something that can be added easily with the utility API.


Incorrect AMPLITUDE_API_HOST, current one is deperecated

Instead of:
export const AMPLITUDE_API_HOST = 'api.amplitude.com'
it should be export const AMPLITUDE_API_HOST = 'api2.amplitude.com'
Right now this throws an error:
[Amplitude|Error] Event is not submitted. { Error: certificate has expired
at TLSSocket.onConnectSecure (_tls_wrap.js:1041:34)
at TLSSocket.emit (events.js:160:13)
at TLSSocket._finishInit (_tls_wrap.js:638:8) code: 'CERT_HAS_EXPIRED' }

For now, as a temporary solution.
https://github.com/amplitude/Amplitude-Node/releases/tag/v0.3.3
Thanks @timsu

Going to keep this issue open until we have a complete solution with our CA Root Certificate issue.

Ok, api.amplitude.com endpoint should be ok to use now.


Store just SshKey and only generate YAML at account approval

This caused problems when the yaml schema changed. (see #231) The db tooling for yaml is just not up to par, leaving us with no easy way to fix bad data.
The yaml can still be saved to history for reference, if that's why we originally started saving it in the SshKey field.
The fix should be based on the dev branch, since there are already migrations there touching the Accounts table.

Addressed by #243


[BUG] During signup(without google) , the user can select both student and counsellor simultaneously

Describe the bug
In the user signup page , it is possible for the user to checkmark both student and counsellor button simultaneously.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://counsellor-startup.netlify.app/'
Click on 'Sign Up'
Click on any one of the radio buttons in "Are you a Student or a Teacher"
Then click on the other radio button

Expected behavior
Under the section "Are you a Student or Counsellor", the user should be able to select only 1 of the options either Student or Counsellor. Selecting both the buttons makes it ambiguous.
Screenshots
https://github.com/Counselllor/Counsellor-Web/assets/127319225/7bec325f-24e6-4b17-be28-234ab598ce8e
Desktop (please complete the following information):

OS: Windows 11
Browser : Brave, chrome
Version : 2H22

Please assign me this issue, I can fix this issue .
Thank You,
Sayantan Choudhury
GSSOC'23 Contributor

Its Done


Determinate progress-bar broken in @angular rc1 #430 [Still not working]

Bug, feature request, or proposal:
Bug (https://github.com/angular/material2/issues/430)
I did everything like you said before closing the issue 430, but it is still not working (cf my last response)
I think you don't see comment notifications on closed issues, so I open a new one.
What is the expected behavior?
Progress bar to be filled with correct percentage
What is the current behavior?

What are the steps to reproduce?
import {MdToolbar} from '@angular2-material/toolbar';
import {MdButton} from '@angular2-material/button';
import {MdCard} from '@angular2-material/card';
import {MdProgressBar} from '@angular2-material/progress-bar';

@Component({
  selector: 'home',
  templateUrl: 'components/home/home.component.html',
  styleUrls: ['components/home/home.component.css'],
  providers: [],
  directives: [MdToolbar, MdButton, MdCard, MdProgressBar]
})
export class HomeComponent implements OnInit{

     PID completion {{h.pid.percentage}}%{{h.pid.warning}}
    <md-progress-bar mode="determinate" [value]="h.pid.percentage"></md-progress-bar>

Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/o077B6uEiiIgkC0S06dd
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, browsers are affected?
    "@angular/common": "^2.0.0-rc.1",
    "@angular/compiler": "^2.0.0-rc.1",
    "@angular/core": "^2.0.0-rc.1",
    "@angular/http": "^2.0.0-rc.1",
    "@angular/platform-browser": "^2.0.0-rc.1",
    "@angular/platform-browser-dynamic": "^2.0.0-rc.1",
    "@angular/router": "^2.0.0-rc.1",
    "@angular/router-deprecated": "^2.0.0-rc.1",
    "@angular2-material/button": "^2.0.0-alpha.4",
    "@angular2-material/card": "^2.0.0-alpha.4",
    "@angular2-material/checkbox": "^2.0.0-alpha.4",
    "@angular2-material/core": "^2.0.0-alpha.4",
    "@angular2-material/icon": "^2.0.0-alpha.4",
    "@angular2-material/input": "^2.0.0-alpha.4",
    "@angular2-material/progress-bar": "^2.0.0-alpha.4",
    "@angular2-material/progress-circle": "^2.0.0-alpha.4",
    "@angular2-material/radio": "^2.0.0-alpha.4",
    "@angular2-material/sidenav": "^2.0.0-alpha.4",
    "@angular2-material/toolbar": "^2.0.0-alpha.4",

Is there anything else we should know?
Working with Chrome Version 50.0.2661.94 (64-bit) on Mac OSX
Please tell me if you need anything else.

@Pixcell This works fine for me.

http://plnkr.co/edit/6dWmU5t84PLLs7MZKa5q?p=preview

Can you modify the Plunker, to represent your issue?

@josephperrott I can reproduce this in my material2 sample app; we can take a look next time you're around

@DevVersion, I tried a lot of things (even getting values from a webservice, http://plnkr.co/edit/6dWmU5t84PLLs7MZKa5q?p=preview), but I can't reproduce the issue inside the plunker, sorry.
I'm somewhat glad that @jelbourn managed to reproduce it inside his demo app (Because otherwise I would feel very stupid)
I would really appreciate if you could keep me in the loop about this issue, and tell me what was the cause when you find it.
Just let me know if I can help in anyway.

I think the class .md-progress-bar-fill is not properly setting scalex value

Thanks a lot, it was fixed with release of alpha.5-2 !


fluent-bit: cherrypick msgpack to json bugfix

Description
This change picks up this chage from Fluent Bit:
https://github.com/fluent/fluent-bit/pull/9193
Related issue
b/359600424
How has this been tested?

Checklist:

Unit tests

[ ] Unit tests do not apply.
[x] Unit tests have been added/modified and passed for this PR.

Integration tests

[ ] Integration tests do not apply.
[x] Integration tests have been added/modified and passed for this PR.

Documentation

[x] This PR introduces no user visible changes.
[ ] This PR introduces user visible changes and the corresponding documentation change has been made.

Minor version bump

[x] This PR introduces no new features.
[ ] This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
[ ] This PR bumps the version.

We can't (easily) point to commits outside the upstream fluent-bit repo. I forget what we did last time to get around that. Maybe we can point to https://github.com/fluent/fluent-bit/commit/fa46ed93581b289aa7e56fa5e2b610042962b64c instead? I'll try to also see what we did last time...

Ah, we did something like this in the progression #1533 #1534 #1535 -- we'd need to make a branch upstream with the cherry-pick that we want (I think only @igorpeshansky @qingling128 @braydonk have permissions?) or point to an existing upstream commit like the one I linked.

Right I tried my fork but that wouldn't work, it would need to be a branch in fluent bits repo :(


Fix slow loading on homepage

This PR fixes the slow loading on the homepage. It now takes only a few seconds to load the API locally, instead of previously it would take minutes, if it finished at all.
The main fix was using selected_related to prefetch some proposal data, otherwise an exponential number of queries would be run.
To help debug this issue, I created a custom management command that runs the API response to produce the dashboard and logs how many queries were executed.

Hurrah!!


Moar updates for 115

Its all about more, so here is some.

Argh!
Forgot the header catchup in my branch... :sob:
I wonder if it is possible to write a test ti check it. It should be doable I think :grimacing:


Display emergency banner

Because the DWP has remained down for 4 days...
Show a warning to users, this will need to be removed manually...
Or we could write a checker to see if it should be displayed automatically!

Looks MVP.


Why the sampling_offset bias don't get updates during training

in class MSDeformAttn reset_parameters line 69
        with torch.no_grad():
            self.sampling_offsets.bias = nn.Parameter(grid_init.view(-1))

Why the sampling_offset bias don't get updated during training?
I saw that in some other implementations, this parameter would be updated.
Thanks!

Hi @Levishery
I asked myself the same question.
I found a reason : it give to each ref point a favored direction and a favored radius :

Refs points in head h (h in range(nb_head)) : preferred direction h *(2*Pi / nb_head)
Then each point (4 by default ) has a preferred radio : k+1 for k in range(nb_ref_points)

I mean "favored" because it's independent of the data, it's sampling_offsets.bias.
But based on the data and sampling_offsets.weight the real sampling offset might not respect the favored directions and radius.


What does "updgrade" do? I'm scared to click it.

Do we need it?

When a project is created, it takes a snapshot of the blueprint. So if the blueprint is updated, it doesn't affect any existing projects. If you want a project to use the updated blueprint, click upgrade.
The button needs to go somewhere, maybe theres a better place for it.

Is this a thing that we'd expect editors/reporters to use? Or should it be a thing that engineers or PM runs on a case-by-case?

Probably not. We should hide it from editorial.


Temperature doc apparently wrong and untested behaviour

It looks like this is wrong: https://units.docs.pyansys.com/version/stable/user_guide/temperature.html
It states that two temperatures can be added, but we have prevented that and only allow T + dT; otherwise we raise.
I tried to find a relevant unit test, but none is there.

Yes, we should avoid temperature addition.

Yes, I think it was untested due to being unsupported (which is obviously not the right approach :) )

We have test to check addition between temp and temp-diff but we are not checking addition of 2 temp.

@seanpearsonuk We are already raising an exception for temperature addition.

We have test to check addition between temp and temp-diff but we are not checking addition of 2 temp.

Yes, exactly the point.


Conda Activate env, integrated terminal doesn't use the correct python

Environment data

VS Code version: 1.38.0
OS and version: macOS Catalina 10.15
Python version (& distribution if applicable, e.g. Anaconda): Anaconda 3
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): Conda env 3.7

Expected behaviour
Setting conda activate env within the integrated terminal should change the python interpreter to that specific env python.
Actual behaviour
The env being used by integrated terminal, is still the default python of osx and not that of the env activated.
Steps to reproduce:

Install vs code
2- Install Anaconda
3- Create a new anaconda env
4- Run vscode
5- Start integrated terminal (no need for creating a project)
6- Do a Conda activate env. (where env is the name of the created env)
7- Do python --version
8- The python will still be the default osx python and not that of the env.

Hello @Quisher 👋 Thank you for reaching out.
Which version of the extension do you have? And does conda activate env work in a separate terminal window?

hi @kimadeline :)
Version: 1.38.0
Commit: 3db7e09f3b61f915d03bbfa58e258d6eee843f35
Date: 2019-09-03T21:47:00.464Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 19.0.0

which extension are we talking about?
Conda activate env works fine in the default terminal.
Image attached for reference:

Can you try using bash as your integrated terminal, or zsh in an external terminal?

default is bash.
was just checking if maybe it will work in other terminals so activated zsh. But sadly it doesn't work in bash or zsh or others.

which extension are we talking about?

This is the python extension repo, so this one :) However, you don't have any Python files open, correct? So this happens without the Python extension being activated? If you were to uninstall the Python extension, would the problem still happen?

yup it happens even if I have no projects open.

Thank you for the clarification 👍  It is probably not an extension issue then, would you mind opening an issue referncing this one on the vscode repo?

This issue looks like a dup of https://github.com/microsoft/vscode-python/issues/6343. Quoting the workaround,

This is a known issue in VS Code and we're in the process of resolving this.
More info can be found here https://code.visualstudio.com/updates/v1_36#_launch-terminals-with-clean-environments

As a temporary solution please change the setting terminal.integrated.inheritEnv to false as outlined in https://code.visualstudio.com/updates/v1_36#_launch-terminals-with-clean-environments
Please let us know how this goes

Just wanted to say... As referenced

https://github.com/microsoft/vscode-python/issues/6343#issuecomment-509319313

Turning it to false, fixing the issue in macOS


CI: add back shim v1 test

We are going to deprecate shim v1 (#4365), but it is still early to disable the tests for them

Failed test is due to using v2 runc opts.

Agreed it's probably best to keep running tests against io.containerd.runtime.v1.linux

CI green


Get "https://xbarapp.com/docs/plugins/categories.json": context deadline exceeded

@PapyFlexer thanks for reporting this, does it happen a lot for you?

Closing for now, this shouldn't happen very often. Please reopen if you see it again.


Add all sizes to wallet outline

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Community] Add Hanselman's blog post.

 Issue by patridge
Monday Dec 12, 2016 at 21:38 GMT
Originally opened as https://github.com/Wyamio/Wyam.Web/pull/32

Just thought I'd offer an addition of the blog post from Hanselman that got me to check out Wyam today. I was guessing things were reverse chronological and threw it on the top of the list, but I can adjust as needed.
I was using this edit as an excuse to get Wyam running. Unfortunately, I couldn't give it to work locally yet. (I'm still debugging the issue, but I ran into an problem running the build.cmd when it went looking for Wyam.sln in a sibling directory instead of hunting in the submodule path.)

patridge included the following code: https://github.com/Wyamio/Wyam.Web/pull/32/commits

 Comment by daveaglick
Monday Dec 12, 2016 at 21:59 GMT

Thanks! The site is currently undergoing a ton of changes in prep for a new release soon (you can see the in-progress work in the docs-recipe branch. The current build command is optimized for debugging the site on my machine :smile:. The new version has a more universal build command that actually downloads the source from GitHub on build since I know a lot of folks have been looking at it as an example.


Determien the format and mechanism for storing spec metadata

Description
By current plans, spec metadata will be stored as structured metadata associated with the latest release containign the spec version.  Need to verify the gitub APIs that would allow creating and reading releases.  If these do not pan out, the data could be stores:

In a metadata file alongside the spec
in swagger extensions inside the spec

Verifyu that the APIsthat would enable creatign and reading structured release comments exits, and decide on the precise format to eb sued

Closing Guys


Appending additional files to the fat JAR

There should be easier way to append to assembledMappings.
If there is one, document it.

+1. assembledMappings is listed as one of the keys for the assembly task on the README.md but googling for it I only found older examples that did not compile in sbt 0.13.13. Any current examples? I am trying to add a file that was generated by the sbt editsource plugin (http://software.clapper.org/sbt-editsource/) to my fat jar.

Example:
I was using the EditSource plugin to templatize version, project name (based on directory) and buildDate using the following settings where the template file is in module/templates/application.conf for each subproject:
lazy val editSourcePluginSettings = Seq(
  compile in Compile := ((compile in Compile) dependsOn (edit in EditSource)).value,
  flatten in EditSource := true,
  //use <module>/templates/application.conf
  sources in EditSource := Seq(
    baseDirectory.value / "templates/application.conf"
  ),
  targetDirectory in EditSource := baseDirectory.value / "target",
  variables in EditSource ++= Seq(
    ("version", version.value),
    ("projectDirName", baseDirectory.value.name),
    ("buildDate", dateFormat.format(new java.util.Date()))
  )
)

This put my application.conf under module/target/application.conf. I then used the following assembledMappings to get that application.conf into the root of my fat jar:
lazy val assemblyPluginSettings = Seq(
...
  assembledMappings in assembly += {
    sbtassembly.MappingSet(None, Vector(
      (baseDirectory.value / "target" / "application.conf") -> "application.conf"
    ))
  }
...
)

In this case the file did not come from a jar file so the first argument in MappingSet is None, otherwise Some(new File(jarFilePath))? I have not tried that out but according to the MappingSet comments in AssemblyKeys.scala:
...
// Keep track of the source package of mappings that come from a jar, so we can
// sha1 the jar instead of the unpacked packages when determining whether to rebuild
case class MappingSet(sourcePackage : Option[File], mappings : Vector[(File, String)]) {
...


Automatic error report

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>

@Eisukeeeee, it seems you're using a very old Spyder version. Please update or install a new version to fix the error you're having by following the instructions in our documentation:
http://docs.spyder-ide.org/current/installation.html


Refactor kv.Store to interface

Make kv.Store as interface
We may replace to any other kv storage or distributed/remote kv for https://github.com/genjidb/genji/issues/48

Hey @morlay thanks for submitting this PR and for putting so much effort!
Unfortunately, I can't accept this change because of the following reasons:

Genji just went from having multiple store implementations (see v0.14, we had 3 different implementations) to a single one. Maintaining multiple implementations is a LOT of work, I have been trying it for years and I believe is not worth it. Latest release focuses on a single backend for exactly this reason.
Most kv-stores come with their own transactions, whereas Genji implements its own transaction mechanism which relies a lot on Pebble primitives, which are hard to find in other kv-stores.
Even if we decide to merge this PR, you will quickly realize how hard it is to implement another kv-store: Most of them don't allow you to pass your own custom Compare function. Keys in Genji are not lexicographically sorted, but require to be in a certain order when we iterate on them. Most kv-stores use bytes.Compare to determine which key is greater than the other, and this won't work with the binary representation of keys.

Basically, focusing on a single kv-store makes everything much easier to maintain, optimize and control.
They are plenty of things to do in Genji, and I would be more than happy to discuss about ideas in issues, so please don't hesitate to open one if you want any change.

@asdine
I just want to extends Genji scalable like the issue metioned https://github.com/genjidb/genji/issues/48.
kv.Store as interface could reach this goal, with implememting something like raft pebble.
Then Genji will be a Progressive Storage, from embed to distributed 😄 .

@morlay I don't think this is the right way to make Genji distributed. This is a too big of a change to be done without proper discussion and planning sorry.
At the moment, the focus is mainly on adding SQL features and reaching stability. Even if we added raft to Genji, it wouldn't be very useful as it doesn't yet have enough SQL features.
I'm closing this PR but I appreciate the time you spent on this.


Html diff fails if a source file disappears

If a source file disappears between two Html report generations, the attempt to find the now-deleted files fails and causes the new report to not be created.

Working on a fix.

Fixed by #384 (why did autoclose fail?)

Autoclose only triggers on merge to master (not sure if I can change that in settings)

Ah. I think you need to set the default branch to develop to fix that.


Update the changelog from 0.26 to 0.28

Make sure these boxes are checked before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if any
[X] I have reproduced the issue with at least the latest released version of superset
[X] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.28
Expected results
Have a changelog up to date
Actual results
Last changelog entry is 0.26.0 (2018/07/03 14:13 +00:00)
Steps to reproduce

Changelog is back


Feature Request: Network Alpha Setting.

Would it be possible to have the ability to set the Network Alpha from within the UI please? I tried editing the scripts directly to edit it, but I either did it wrong or else it's more than just the 1 script that needs editing. I ask because from what I've read and been told the Alpha should be double the Rank/Dimm, and while we can change the Rank/Dimm the Alpha appears to be locked at 1.
Thanks for the great tool either way!

Solved as of pull #91  thanks!


feat: remove duplicate key in worksituation

remove cdd and cdi key from workSituationKeys
replace cdd by cdd_temps_plein and cdi by cdi_temps_plein
add a test case for migration

fix #889

:tada: This PR is included in version 1.124.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Adjust types to match exported functions

The selectWithResolver type was missing from the type definitions. As I was looking, I noticed a few others things might be wrong. I think this should cover everything. I'll try it in the xml-crypto project and report back when that migration is done. Until then, I'd like to get feedback.

Thank you for submitting this.
I'm not all that knowledgeable about .d.ts files and wasn't aware that they could contain functions. Would you mind explaining how one makes use of those functions?
One thing that I notice is that you are using instanceof Node and properties on Node in these functions, but to the best of my knowledge, the global Node object doesn't exist in the node.js environment, only in browsers. Does that mean these functions would fail if someone attempted to use them in node.js? Or does TypeScript take care of that somehow?

Indeed, duck typing does work better for this use-case. in xml-crypto, there are more libraries available, so we could get away with what you saw here. I've updated this code to reflect what is actually working in xml-crypto. Does this look better to you?
BTW, since the TS refactor of xml-crypto depends on this, this PR is a blocker to that getting released.

Thank you for this update. I've sent you a pull request with some proposed changes and also made some comments.
We should also have unit tests for the new functions you're adding. Can you include those in your PR?

I've made the changes you've discussed.
Side-note: I'd like to format your code, but I don't see a npm script to do so. What formatting tool do you use?

Thank you. I've requested a few more changes to the unit tests, and there's a re-activated comment about the return types in the XPathSelect interface.
Until now, I've been careful to leave the whitespace in xpath.js undisturbed so as to not have a huge diff in the history, but I was thinking that it's about time that I added a prettify process. I can do that once this PR is merged.

I've requested a few more changes to the unit tests

I'm sorry, I don't see these.
Otherwise, I've made all the changes that I think you've requested. Thanks for your review!

I added a prettify process

IIRC, eslint can format code and it is highly customizable, so you can just pick a style that most closely matches what you have now to minimize changes.

@JLRishe , I can't easily tell from reading the code; can select(expression: string, node: Node): SelectReturnType; also be select(expression: string, node: string): SelectReturnType;. I'm checking types in xml-crypto and I see that we are passing a string to select() and it seems to be working. I want to make sure the types are accurate here.

I'm sorry, I don't see these.

I'm referring to these comments:

can select(expression: string, node: Node): SelectReturnType; also be select(expression: string, node: string): SelectReturnType;

I don't see any way that would work.
I think what you're seeing is that xml-crypto has a select(node, xpath) export that calls off to the select(xpath, node) function from the xpath package (note the reversal in parameter order):
module.exports.xpath = function (node, xpath) {
  return select(xpath, node);
};

And xml-crypto's README.md and example.js are importing and calling that function that has the reversed parameters. Could you double check that and tell me if what you're referring to is somewhere else?

I'm referring to these comments:

Since they are "Pending", I don't see them. You have to complete the review for them to be visible to me :)

That's weird. I thought I sent them the same way I sent all the previous comments. Hopefully they're visible now.

Hopefully they're visible now.

They are. I'll resolve these as I push changes related to them. https://github.com/node-saml/xml-crypto/pull/325 is now importing this PR at its HEAD so that I can make sure 100% of these changes are working in real code. I'll convert this to a Draft until it is ready to merge. Thanks again for your attention.

@JLRishe , one of the tests is failing, and it isn't because of any code I wrote. It appears that ProcessingInstruction doesn't have a nodeName property. Since it inherits from Node, and Node has a nodeName, this is a bug in this library.
    it('should correctly identify a Node of type ProcessingInstruction', () => {
            var doc = parseXml('<book />');
            var pi = doc.createProcessingInstruction('xml-stylesheet', 'href="mycss.css" type="text/css"');

            assert.ok(xpath.isNodeLike(pi));
            assert.ok(xpath.isProcessingInstruction(pi));
            assert.ok(!xpath.isComment(pi));
        });

Should have a nodeName of "xml-stylesheet", but it is currently undefined.

It appears that ProcessingInstruction doesn't have a nodeName property. Since it inherits from Node, and Node has a nodeName, this is a bug in this library.

The processing instruction in your test is being created by the @xmldom/xmldom package. The problem you're describing has nothing to do with this xpath package.
It does indeed appear that the DOM 3 standard indicates that processing instructions should have a nodeName property with the same value as its target property, so I would encourage filing an issue on their repo, but I imagine it would be a long time before they addressed it, and having these new functions work with that package is ideal, so please find a suitable workaround. I think removing the nodeName check from the isNodeLike function would be ok. (typeof value.nodeName === "string" || typeof value.target === "string") would also be ok.

filing an issue on their repo

Done: https://github.com/xmldom/xmldom/issues/505

I've put up a PR to get this fixed in xmldom: https://github.com/xmldom/xmldom/pull/509

This is now no longer needed since @xmldom/xmldom pushed a new release. I've reverted the work-arounds and updated the dependency.
FWIW, I note that engine in package.json specified a minimum version of 0.6. I tried installing that, and couldn't.
I installed 0.12 and the tests failed because that version of Node doesn't support const.
I installed v4 and that failed because it didn't support destructuring.
I installed v6 and that failed because yargs-parser didn't support anything less than v10.
I installed v10 and the tests passed.
All that to say, the package-lock.json was changed to "lockfileVersion": 1 from "lockfileVersion": 2 in order to support the older version of Node. I personally recommend doing all development on the oldest-supported version of Node to prevent overlooking these kinds of things.

Please revert the change to the "engines" setting, and the lock file format.

Done. I generated package-lock.json using Node@LTS. Since there are no dependencies, only devDependencies, this shouldn't be a problem for anyone. My apologies for creating extra work for you.

which has gone far beyond its originally stated purpose

Is there something you'd like to see broken out into multiple PRs?

Is there something you'd like to see broken out into multiple PRs?

I would have preferred to have the node type test functions in a separate PR. If this were just the .d.ts changes, it could have been merged a lot sooner. But it's ok. Thank you for sticking through this PR.

I'm not worried about the delay as the PR over at xml-crypto needs all of this for correct operation. Once this gets released then we can unblock that PR. I appreciate your sticking with me on this mess too; I know it was a lot.

The new version is now published to npm. Thanks!

that is strange to change the select return type to Array<Node> | SelectedValue, and this change break my codes.
I think the return type of select(expression: string, node: Node, single: false) should be Array<SelectedValue> | undefined | null, it should not return a single value type as result.

if it selected somethings, (if there is just one selected "Value", the result should be transform to Array<SelectedValue>)
if it selected nothing, statement as null or undefined

BTW, if you make a breaking change, you should public a MAJOR number version.

@iahu I am sorry that this update is causing trouble for you. In this update, the type definitions were modified to be consistent with the actual behavior of the package's implementation. The actual behavior of the package's exported functions has not changed. The type definitions were previously incorrect and have been corrected.
Per the XPath 1.0 spec, an XPath expression can evaluate to:

A node set (0 or more nodes)
A string
A number
A boolean

As you can see, it cannot evaluate to a set of strings, or to a null value. This package's select() function is consistent with this, so Array<SelectedValue> | undefined | null would not be correct.
The author of this PR has added several utility functions that can help to converge the result of select() to the correct type, though it looks like there is currently an issue with the isArrayOfNodes function: https://github.com/goto100/xpath/pull/121
Personally, I think select() isn't very well suited for TypeScript, because of the ambiguity of its return type. For that reason, I strongly suggest using parsed expressions which allow evaluating XPaths without such ambiguity.

after read a bit of the source code. I agree that, the current type declare fit the select function's results.
as my above comment, I want get the facts straight. I suggested to change the select function's results to be "Array of SelectedValue", which like the querySelectorAll DOM API, and as the same time the select1 just like the querySelector API.

Note: I think there is still a little room for correction in the type definitions since they currently imply that select(_, _, false) could return a single Node or null, which is not true.

I'm reviewing this to make adjustments where needed. Are you hereby saying that selectWithResolver(_, _, _, false) will never return Node, null, or undefined? I think I can see that from the code, but I want to double-check. I'll put up a PR with the other adjustments you've requested so you can comment.


samples: nrf52: Ignore messages with prohibited parameters

This fixes an issue of sending Status message as a response
to Set command with prohibited parameters.
Status message shall not be sent in this case according to
Mesh Model Spec 1.3.3.
Signed-off-by: Mariusz Skamra<EMAIL_ADDRESS>

@vikrant8051 FYI

@jhedberg Do not merge this yet. I have to fix the rest of the models

@MariuszSkamra Kindly don't do anything as of now. It will create lot of confusion.
If you required please create separate app.
@jhedberg Please look into this.

@vikrant8051 what confusion do you mean? This PR seems like a fairly straight-forward fix to the app, and this app needs to be fixed no matter what.

Hi Johan,
Kindly wait till I merged from my side.
Thank You !!
On Fri, Jul 6, 2018 at 3:46 PM, Johan Hedberg<EMAIL_ADDRESS>wrote:

@vikrant8051 https://github.com/vikrant8051 what confusion do you mean?
This PR seems like a fairly straight-forward fix to the app, and this app
needs to be fixed no matter what.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zephyrproject-rtos/zephyr/pull/8779#issuecomment-402992759,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFs-ReQa72ERpUUW8wO2bUa0NczpoYFGks5uDzkIgaJpZM4VFNLh
.

@vikrant8051
You are kidding me. I have a lot of work to do, sorry I haven't been responsive. But I found free slot to test your patches and provide a fix. You want me to test your solutions only?? Who am I to you? Someone is paying for my time and you want to throw my work away?

@MariuszSkamra I respect your contribution. Without your help it was impossible to go ahead. It was my humble request only. Kindly don't take it in wrong way.

@MariuszSkamra It was my reflexive reaction. And I think I was wrong & sorry for that.

Codecov Report

Merging #8779 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8779   +/-   ##
=======================================
  Coverage   52.36%   52.36%
=======================================
  Files         195      195
  Lines       24713    24713
  Branches     5136     5136
=======================================
  Hits        12940    12940
  Misses       9699     9699
  Partials     2074     2074

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 f5ec567...bef6451. Read the comment docs.


[VTOL] Offboard attitude control not working

Describe the bug
Based on discussion 13159.
VTOLs don't follow the offboard attitude setpoints, instead they keep the last attitude setpoint from before switching to offboard mode.
For more details (and other issues encountered), see the above-mentioned post on discuss.px4.io.
To Reproduce
Steps to reproduce the behavior:
Run the MAVSDK offboard example.
I have tested it on SITL and HIL on the standard VTOL model (with a slightly modified version of the MAVSDK example), in both simulations there are issues. Haven't dared to try it on a real drone, pretty sure it also doesn't work.
Expected behavior
Drone should follow the attitude setpoints that are passed via MAVLink.
Log Files and Screenshots
See post on Discuss.
Drone (please complete the following information):

Standard VTOL (Gazebo)

Additional context
Main findings so far:

The problem can be solved for SITL by settting the correct ORB ID (mc_virtual_attitude_setpoint) for _att_sp_pub in mavlink_receiver.h (need to set it so that it still works for non-VTOLs)
The SITL solution doesn't work for HIL (tested with Pixhawk 4 as hardware). (not tested on real vehicles)
Additionally, the correct frametype has to be included here for the thrust setpoint.

Is this on master or on a stable build? Note that there was recently quite a lot of work done on offboard mode for non-copter targets by @Jaeyoung-Lim (Jae - FYI).

@hamishwillee  On master and on v1.9.2

@ThomasRigi Could you verify what typemask you are sending and if it is multicopter mode you are observing this behavior?

@Jaeyoung-Lim The typemask is 0b00000111 (Ignore body roll, pitch and yaw rates), sent using MAVSDK function Offboard::set_attitude(Offboard::Attitude attitude). And yes, it is Multicopter mode

@ThomasRigi It seems like this is specific to attitude setpoints. I can fly with body rate setpoints, which uses the same mavlink message (SET_ATTITUDE_TARGET)[https://mavlink.io/en/messages/common.html#SET_ATTITUDE_TARGET]. here is the log.

@Jaeyoung-Lim Thanks for sharing the log! It makes sense that the rate control works, as in that case you don't have to pass through the virtual attitude setpoints in vtol_att_control_main.cpp.
I will shortly create a PR in which SITL works (but HIL doesn't), but I know that @dagar doesn't like creating extra ORB publications.
I failed to figure out where this difference SITL <--> HIL comes from. It would be great if someone finds it (or dares to test the new solution on a real drone :grimacing:)

#13141 is now tackling this problem. Many thanks to @irsdkv !
I'm crossing my fingers that HIL is going to work on his branch :crossed_fingers:

This has been fixed in #13141


CASMTRIAGE-6992

Summary and Scope
The newest ca-certificates rpm uses/requires the statedir environment variable to be set. Under normal circumstances the update-ca-certificates script sets the variable and executes all of the *.run scripts in /usr/lib/ca-certificates/update.d/.  Since we don't use the update-ca-certificates script, we have to set the statedir variable. The changes in this PR are backwards compatible.
This statedir variable change was introduced in ca-certificates-2+git20240416.98ae794-1503<IP_ADDRESS>
We are currently running ca-certificates-2+git20210309.21162a6-2.1
Issues and Related PRs
https://jira-pro.it.hpe.com:8443/browse/CASMTRIAGE-6992
Testing
Tested on:

gamora

Test description:
The ca-cert ansible role was modified with the same changes in this PR. The play was successful.
Risks and Mitigations
None known.
Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] Testing is appropriate and complete, if applicable

/backport release/1.16


chore: fix Lean 3 names in comments, related to presheaves

Rename

presheaf -> Presheaf
opens -> Opens
supr -> iSup
pairwise -> pairwise
and a few more

Nice!
bors merge


SSL configuration is Running - [BRIEF DESCRIPTION]

App Id: d2s8z7iw0zcj4i
Region: us-east-1
Step: SSL configuration
Status: Running

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
When trying to add my domain, it fails and provides the error message below. I have removed all CNAMEs that would be relevant to this domain as it was recently setup in cloudfront/s3 prior to attempting to move to amplify
Error message
Next to useless error message: Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Additional information
Please add any other relevant information. Please feel free to include screenshots.

Refer to our documentation to troubleshoot domain setup.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html#i-get-a-cnamealreadyexistsexception-error
Let us know if you're still facing issues.

@alexcameron1992, Is this still an issue?

@nimacks no longer an issue but the error messages could still be much better in that scenario to prevent wasted time.

@alexcamerondev Can you share some insights so we can make this experience better? What problems did you run into?


Dynamic Fields and Custom Measures

Is there a way to add Custom Measures to the explore? Like if I want to show Average Sales by Month, how do I show it on the explore as a visualisation?

This is something that we will be close to fully supporting in the next version release coming soon. As for now, you'll need to give the Explore Assistant example url's with sample custom dim/measure or table calc creations.


ENH: Change COO reshape to not overflow

PR for #8912.

Test?

I discovered that the approach laid out here is fundamentally flawed. It only protects from overflow in some instances, but not others. In the main thread, I proposed simply elevating the intermediate indexes to int64 when needed. I'll update this PR when I get the time to implement that.

Let's go with a new PR when you find time to update it (so that we get less inactive PRs in the list).

Can this PR be reopened? I have committed a new solution and would like to see how it looks on CI.

It says this branch is force-pushed or recreated and doesn't allow for reopening.


[Feature]: Start paused app instead of redeploying

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
When I pause an app I can't restart it again. Only the Deploy button is available. I would like to just click new button Start.
Why should this be worked on?
Sometimes I have to wait ~20min until the build is complete even though I haven't changed any settings that would need rebuilding the app.
It could be, for instance, really painful when the app is degraded and I can't just restart it.

I'm closing this issue, because tracking features will be moved to GitHub discussions. Github issues will be only for bugs.
Version 4 is in full focus. v3 related issues will be closed, if it is not a critical bug.
Thanks for your understanding.


[Snyk] Upgrade @reduxjs/toolkit from 2.2.4 to 2.2.5

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade @reduxjs/toolkit from 2.2.4 to 2.2.5.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 22 days ago.

Release notes

Package name: @reduxjs/toolkit

2.2.5 - 2024-05-16This bugfix release fixes an issue in the recent createEntityAdapter sorting perf improvements that could (in specific cases) cause Immer to throw an error when trying to read a plain JS value instead of a proxy-wrapped value.
What's Changed

Fix missed spot where use of current may fail if the value is not a draft by @ markerikson in #4412

Full Changelog: v2.2.4...v2.2.5

2.2.4 - 2024-05-09This bugfix release improves sorting performance in createEntityAdapter, shrinks the code size in matcher utilities, fixes assorted issues with query hooks, and makes several TS tweaks.
Changelog
Entity Adapter Sorting Performance
Users reported in #4252 that the sorting performance of createEntityAdapter seemed abnormally bad - the provided comparison functions were being called far more times than expected.
Upon investigation, we had a couple of problems. We were always starting from an array that was in insertion order, not the existing sorted order, and that would always require significant effort to re-sort even if there weren't any actual changes to the sorted results. Also, the sorting checks required frequent access to Immer's Proxy-wrapped values, even in cases where all we needed was the plain state values for comparison purposes.
We've reworked the internal sorting logic to always start from the existing sorted array, do reads against a plain value to avoid the Proxy getter overhead where possible, and optimized inserts into existing sorted arrays.  This should significantly speed up sorted entity adapter behavior.
Matcher Code Size Optimization
We've reworked the internals of the thunk-related matchers to deduplicate some of the logic, shaving a few bytes off the final bundle size.
RTK Query Hook Updates
defaultSerializeQueryArgs can now handle BigInt values safely.
The isLoading flag logic was improved to handle errors when a query hook tries to subscribe.
TS Updates
create.asyncThunk's types were improved to avoid cases where it might infer any.
We've made several internal types changes to work correctly with React 19's upcoming types.
The retryCondition method now receives unknown as an argument, instead of always assuming the user is using fetchBaseQuery.
Other Changes
The Reselect dep has been bumped to 5.1.0 to match the expected internal usage of createSelector.withTypes().
What's Changed

Update remaining Context references to match the new nullable Context by @ aryaemami59 in #4336
types: export GetThunkAPI type by @ shrouxm in #4289
Inline the deprecated React.ReactChild type by @ aryaemami59 in #4382
Replace usage of deprecated JSX global namespace with React.JSX by @ aryaemami59 in #4381
Fix useRef usages to be called with an explicit argument of undefined. by @ aryaemami59 in #4380
Bump reselect dependency to 5.1.0 to resolve #4200 by @ aryaemami59 in #4400
Improve sorted entity adapter sorting performance by @ markerikson in #4361
Prevent inference of any in create.asyncThunk by @ EskiMojo14 in #4389
try out simplifying matcher code by @ EskiMojo14 in #4386
Handle bigint in useQuerySubscription by @ rkofman in #4315
fix: ensure hook subscription failures do not reset isLoading state by @ smacpherson64 in #4364
Improve usability of the initiate() action for mutations. by @ jared-ca in #4337
Make retryCondition receive unknown error, as the user may not be using fetchBaseQuery by @ EskiMojo14 in #3957

Full Changelog: v2.2.3...v2.2.4

    from @reduxjs/toolkit GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

new pr related to this one #856


Font Change

How can I globally change the font on a website, including the footer, navbar, and sidebar?

The master location for fonts is in _sass/_variables and the lines that govern the fonts look like this:
$global-font-family         : $sans-serif;
$header-font-family         : $sans-serif;
$caption-font-family        : $serif;

By default the $sans-serif variable is used which uses the following default and fail overs:
$sans-serif                 : -apple-system, ".SFNSText-Regular", "San Francisco", "Roboto", "Segoe UI", "Helvetica Neue", "Lucida Grande", Arial, sans-serif;

If you are going to change the template font is recommended that you define a new font variable that includes fail overs like the $sans-serif variable and then assign that variable to the primary variables listed above.

How to use external font,  I installed "Lucida Sans" on my system, and replace in the place of "-apple-system", it worked locally, but not on deployment, why so?


Implement Missing NSCalendar APIs Utilizing CoreFoundation Calendar

NSCalendar requires matching functionality, much of which can be implemented in terms of CFCalendar.
Verify using new unit tests for these new APIs.

This issue was addressed in Pull Request 656


Best practices to organise/nest data for d3.js graph output.

Hi all,
I'm looking for some advice on how to properly use data with d3.js. Lets say for instance, I have this data set taken from a raw .csv file (converted from excel);
EA
Jan_2016,Feb_2016,Mar_2016
Netherlands,11.7999,15.0526,13.2411
Belgium,25.7713,24.1374
France,27.6033,23.6186,20.2142

EB
Jan_2016,Feb_2016,Mar_2016
Netherlands,1.9024,2.9456,4.0728
Belgium,-,6.5699,7.8894
France,5.3284,4.8213,1.471

EC
Jan_2016,Feb_2016,Mar_2016
Netherlands,3.1499,3.1139,3.3284
Belgium,3.0781,4.8349,5.1596
France,16.3458,12.6975,11.6196

Using csv would the best way to represent this data be something like;
Org,Country,Month,Score
EA,Netherlands,Jan,11.7999
EA,Belgium,Jan,27.6033
EA,France,Jan,20.2142
EA,Netherlands,Feb,15.0526
EA,Belgium,Feb,25.9374
EA,France,Feb,23.6186
EA,Netherlands,Mar,13.2411
EA,Belgium,Mar,24.1374
EA,France,Mar,20.2142

This seems very long winded to me, and would use up a lot of time. I was wondering if there was an easier way to do this?
From what I can think of, I assume that JSON may be the more logical choice?
And for context of what kind of chart this could go into, I would be looking to put them in a pie chart and update the data depending on the country/month and comparing the three organisations scores depending on the country/month selected.
(plnk to visualise)
http://plnkr.co/edit/P3loEGu4jMRpsvTOgCMM?p=preview
Thanks for any advice, I'm a bit lost here.

Yes, the two most popular formats are DSV (typically CSV or TSV) and JSON.
As a hierarchical format JSON may eliminate some redundancies in hierarchical data, but on the other hand, JSON is self-describing and so tends to introduce other redundant information. For example, in this case, you might say something like:
[
  {
    "name": "EA",
    "countries": [
      {
        "name": "Netherlands",
        "dates": [
          {"date": "Jan_2016", "value": 11.7999},
          {"date": "Feb_2016", "value": 15.0526},
          {"date", "Mar_2016", "value": 13.2411}
        ]
      },
      {
        "name": "Belgium",
        "dates": [
          {"date": "Jan_2016", "value": 25.7713},
          {"date": "Feb_2016", "value": 24.1374}
        ]
      },
      {
        "name": "France",
        "dates": [
          {"date": "Jan_2016", "value": 27.6033},
          {"date": "Feb_2016", "value": 23.6186},
          {"date", "Mar_2016", "value": 20.2142}
        ]
      }
    ]
  },
{
    "name": "EB",
    "countries": [
      {
        "name": "Netherlands",
        "dates": [
          {"date": "Jan_2016", "value": 11.7999},
          {"date": "Feb_2016", "value": 15.0526},
          {"date", "Mar_2016", "value": 13.2411}
        ]
      },
      {
        "name": "Belgium",
        "dates": [
          {"date": "Jan_2016", "value": 25.7713},
          {"date": "Feb_2016", "value": 24.1374}
        ]
      },
      {
        "name": "France",
        "dates": [
          {"date": "Jan_2016", "value": 27.6033},
          {"date": "Feb_2016", "value": 23.6186},
          {"date", "Mar_2016", "value": 20.2142}
        ]
      }
    ]
  },
  …
]

I didn’t copy the data exactly or completely because I’m lazy, but you can see there’s a lot of redundancy here, even if you remove the whitespace. You could design a more concise format, say by requiring that the dates for all the time series are the same, and using null to represent missing data:
{
  "dates": ["Jan_2016", "Feb_2016", "Mar_2016"],
  "organizations": [
    {
      "name": "EA",
      "countries": [
        {
          "name": "Netherlands",
          "values": [11.7999, 15.0526, 13.2411]
        },
        {
          "name": "Belgium",
          "values": [25.7713, 24.1374, null]
        },
        {
          "name": "France",
          "values": [27.6033, 23.6186, 20.2142]
        }
      ]
    },
  {
      "name": "EB",
      "countries": [
        {
          "name": "Netherlands",
          "values": [11.7999, 15.0526, 13.2411]
        },
        {
          "name": "Belgium",
          "values": [25.7713, 24.1374, null]
        },
        {
          "name": "France",
          "values": [27.6033, 23.6186, 20.2142]
        }
      ]
    },
    …
  ]
}

You could also use nested maps rather than arrays to eliminate some redundancy in the property names:
{
  "dates": ["Jan_2016", "Feb_2016", "Mar_2016"],
  "organizations": {
    "EA": {
      "Netherlands": [11.7999, 15.0526, 13.2411],
      "Belgium": [25.7713, 24.1374, null],
      "France": [27.6033, 23.6186, 20.2142]
    },
    "EB": {
      "Netherlands": [11.7999, 15.0526, 13.2411],
      "Belgium": [25.7713, 24.1374, null],
      "France": [27.6033, 23.6186, 20.2142]
    },
    …
  }
}

Then you’d want to use d3.entries or similar to iterate over your data. (You might also be careful about relying on the order of iteration, since strictly speaking it’s not defined, although in practice when parsing JSON I believe most browsers will return it in-order.)
The previous format is pretty concise, only replicating the country names. If you really wanted to eliminate that, you could have a separate array of countries, and then require a two-dimensional array of values, similar to the trick used to avoid redundantly specifying the dates. In fact, you could just generalize it to an n-dimensional matrix of organizations, countries and dates:
{
  "organizations": ["EA", "EB", "EC"],
  "countries": ["Netherlands", "Belgium", "France"],
  "dates": ["Jan_2016", "Feb_2016", "Mar_2016"],
  "values": [
    [
      [11.7999, 15.0526, 13.2411],
      [25.7713, 24.1374, null],
      [27.6033, 23.6186, 20.2142]
    ],
    [
      [11.7999, 15.0526, 13.2411],
      [25.7713, 24.1374, null],
      [27.6033, 23.6186, 20.2142]
    ]
  ]
}

That would be the most concise, I expect, but you’d have to remember the dimension order when accessing the data:
var organization = "EA",
    country = "Netherlands",
    date = "Jan_2016",
    value = data.values[organization][country][date]; // 11.7999

Anyway. The point is, it’s entirely up to you what format you prefer, and there are tradeoffs to consider in terms of performance and convenience: the most convenient format may not be very efficient, and the most efficient format may not be very convenient. It’s the same when evaluating whether to use CSV or JSON.
Also keep in mind that most datasets are gzip-compressed when transmitted over the internet, so some redundancy in the data often isn’t a performance-killer, and attempts to design the most efficient format can sometimes produce counterintuitive results due to compression.
One of the nice things about CSV (or more generally DSV) is that tabular formats can easily constructed into different hierarchies on-the-fly, so you aren’t committing to a single, fixed hierarchy (as in this example of organization / country / date). You can use d3.nest to nest tabular data.
All that said… (and I hope it was helpful)… please use Stack Overflow tag d3.js or the d3-js Google group to ask for help. These issues are for feature requests and bug reports only. Thank you!

maybe you should have an question tag for issues like these

@pokeball99 There’s no question tag because I don’t have the resources to answer all questions, and questions are fundamentally different in nature to bug reports and feature requests; the appropriate forum for questions is Stack Overflow or the d3-js Google group, not here.

@mbostock of course, but the ones you do awnser that are noteworthy enough to be useful to others should warrant such an tag

@pokeball99 Fair point! Thank you.


[Enhancement] Default class labels for semantic segmentation

I'm not sure why the tests are failing here


go.mod, vendor: update cloud SDK modules

The cloud SDK modules are no longer updated automatically by dependabot
due to the frequency of thes updates requiring developer time for review
and also the frequent need to rebase these PRs. We now manually update
them all at once on a monthly basis. See commit 4762a4abae5a
("dependabot: disable cloud provider SDK updates") for details.
Update the cloud SDK modules now using the command:
go get <module>@latest && go mod tidy && go mod vendor

/ci-aks

/ci-eks

/ci-awscni


Security issues

Creating this issue to hopefully unify my posts in discord:

Key stealing attacks

R commitment isn't validated (documented in #1)
R value isn't randomly generated each time, so if an attacker retries a mix they can anticipate the R value and effectively bypass the commitment https://github.com/unyieldinggrace/nanofusion/blob/9102e40983e605e59db886342eba6a4eb0746592/client/src/model/Cryptography/BlockSigner.js#L173
R value can be requested before the commitments are released https://github.com/unyieldinggrace/nanofusion/blob/e4adbd1c6ceb594dba9cfe22821520d98483229e/client/src/model/Client/JointAccountClient.js#L30

DoS attacks

The exit path Receive a -> send back to a, in case b doesn't send, should have at the end of it receive b -> send back to b (the full path being recv a, send a, recv b, send b). This is in case a publishes the exit path even though b did send the funds https://github.com/unyieldinggrace/nanofusion#solving-the-refund-problem

Identification attacks

The server knows everybody right now, and even if the immediate problem was fixed there would be other issues like linking through IPs, MITMing keys, and so forth. Ideally this would use something like a Monero transaction with a ring sig over commitments, and send it via a separate Tor route.

Maybe others, I haven't read through even most of the code yet

@PlasmaPower thanks for the code-review and feedback points. I'm taking a couple of days off from coding to give myself rest, but will certainly get on them. There are some bits that I consider as belonging in the "won't fix" pile (or at least "not my problem" pile). Masking IP addresses is a great idea, but I think it's something that realistically needs to be left to the end user, not baked into NanoFusion. I'm prepared to be persuaded otherwise, but that's my thinking for now.
I hadn't quite worked out the point of the zValue. The scheme "works" if zValues are all empty, which is why I thought it was fine to calculate deterministically (also helped me with debugging, because output was consistent each time I ran it). I see now that it's meant to be a random variable to prevent key re-use attacks. Thanks for pointing that out.

The issues with R commitments not being validated should now be fixed in the mixing context. The PhaseTracker should prevent moving on to signing until all commitments are checked.

Thanks, I'm liking the new phase architecture a lot better :)


Bump future versions on F-Droid

I think you can now bump the 1.0 version on future releases of databag and probably delete the older ones leaving the last one since a lot of things got fixed since 1 f-droid release.

Good suggestion. I deleted the oldest releases and next fdroid release wil tag as 1.1.

Thanks for reminding me, I forgot about this. I just published a release for 1.1 with some minor translation fixes.
Do you find the updated UI to be an improvement? Is the translation useful and/or correct?


Update the commands descriptions and titles

To update the description and titles in the command data file.

This fixes #10 descriptions issue on b874073e2157c4fde2dac3e043ba4ddcab4ad542 commit. Upload my version cause I have troubles with my git terminal.

Sorry cant seem anything that I can merge, can you fork it and make a PR?


chore: jwamaze to 0.0.1

chore: Promote jwamaze to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


Upgrade to Ember Octane

blocked on https://github.com/lytics/ember-data-model-fragments/issues/345

@balajimanoharan does this block Octane or just ED 3.14?

@balajimanoharan does this block Octane or just ED 3.14?

yes, we could keep ember-data fixed at 3.12 until the issue is resolved.


load children inheriting published status

Issue
Child posts with inherit status are not loaded with post.children
Solution
Add post_status=inherit to get_children query.
Impact
None
Usage
Attachments with post.number are now correctly loaded
Considerations
post.children returns also the thumbnail making post.children cumbersome it in twig templates. So, what do you think about excluding it from the method it self?
Testing
Done

Coverage increased (+0.003%) to 96.109% when pulling 68bb4ae176f61f0fdad341b79126c782631fea68 on motia:update_post_children into 2a0b07c46df8c980cf393b5e2f5d56f1e3d14cad on timber:master.

Current coverage is 96.33% (diff: 100%)

Merging #1255 into master will increase coverage by 0.01%

@@             master      #1255   diff @@
==========================================
  Files            44         44
  Lines          3173       3190    +17
  Methods         424        425     +1
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3056       3073    +17
  Misses          117        117
  Partials          0          0

Powered by Codecov. Last update 2a0b07c...68bb4ae

Coverage increased (+0.003%) to 96.109% when pulling 79aabd98c28c1e19cfd0dfb23bcd8fecf6bab7a6 on motia:update_post_children into 2a0b07c46df8c980cf393b5e2f5d56f1e3d14cad on timber:master.

Thanks @motia! I'm not quite sure of which thumbnail issue with post.children you're referring to though? If the concern is about thumbnails loading, those are only called once the actual {{ post.thumbnail }} is called from the templates (it's not a value loaded at init, or at least it shouldn't be)

Also, bonus points for the TESTS!!!! woohooo!!

@jarednova , I meant to say that post.children contains the post thumbnail (not what we want in our templates), I think it should be removed, is not it?


Handle aliases in selection.

When fields have aliases, like id in Mongoid, or when a field is defined as:
field :name, type: String, as: "full_name"

Origin should convert the aliases to the names of the fields in the database.
See:
https://github.com/mongoid/mongoid/issues/1682
https://github.com/mongoid/mongoid/issues/1569

+1 :)

+1


Convert top level api classes to Kotlin

japicmp report there: https://storage.googleapis.com/martin-playground.appspot.com/japicmp.html. I put a few comments inline in the code as the discussion will be easier with some context.
Some more global remarks:

Class file format changed from 50 (java 6) to 51 (java 7). I think this is fine ?
A lot of classes/methods became final, which is a good thing in my opinion.
java.lang.Boolean is now the primitive non-boxed boolean, which I think is also a good thing.
data classes: I'm afraid using data classes broadly will not only add unecessary bytecode but also might convey the wrong intent. When I'm seeing a data class, I'm looking for a usage of .copy() or .equals() and it doesn't seem we need it at the moment ?
@JvmField: I would drop the @JvmField annotation right now. Java can use .getField() instead of .field() and we won't have to maintain @JvmField in the long run.

Overall looks great :). One small bonus I'd like to have is the ability to keep the git history accross file rename. I'm not sure how doable that is but I'll look into it.

I tried to do the "2 steps" commit thing (move first, modify then) to try and keep file history but it's not working great either. See https://github.com/apollographql/apollo-android/compare/master...martinbonnin:convert-to-kt-keep-history.
Both intelliJ and git do not follow accross file renames by default. It works on the commandline if one adds the --follow option but if manual input is needed, we might as well lookup the java file directly.

Class file format changed from 50 (java 6) to 51 (java 7). I think this is fine ?

Yeah, we should be fine

data classes: I'm afraid using data classes broadly will not only add unecessary bytecode but also might convey the wrong intent. When I'm seeing a data class, I'm looking for a usage of .copy() or .equals() and it doesn't seem we need it at the moment ?

Agree, will clean up

@JvmField: I would drop the @JvmField annotation right now. Java can use .getField() instead of .field() and we won't have to maintain @JvmField in the long run.

Was trying to make it backward compatible with Java but I guess went too far, will remove those that not required.


Backends should have a way to view and insert the url

The url generated by the backend editor should be copy-able (and view-able) by the user.
Also, the user should be able to paste in a previously saved url.

So it's fairly easy to just add the field where you can add a URL but it would overwrite all of what has been added to the current backup would we be happy with that or should the user be presented with a dialog where they can accept before moving on?

I would say that entering a URL is a pretty advanced action. If a user is advanced enough to insert a URL we should assume they understand the risks. If the UI looks messed up they can always choose not to save it.


TypeError: Type 'NoneType' cannot be serialized (in etree)

Describe the bug
There is periodical issue while using nornir_scrapli. Script connects to devices and get summary information by custom tasks.
SOME random devices are found in .failed_hosts. Every run I see different devices there. There are 3 custom tasks and everytime random task fails.
Other devices return expected result.
I have no problems while connecting to single device by the same code. Never.
To Reproduce
Steps to reproduce the behavior:

Your script
junos.txt
What you're connecting to (vendor, platform, version)
JunOS 20.2R2 or 17.3R3. (~40 devices)
Anything else relevant
Last func def in junos.txt is called like that:

res = nr.run(task=junos.get_junos_summary_information)

nornir is configured like that:
runner:
    plugin: threaded
    options:
        num_workers: 20

Expected behavior
res.failed_hosts is empty.
Stack Trace
Traceback (most recent call last):
  File "/home/ubuntu/nbox/venv/lib/python3.8/site-packages/nornir/core/task.py", line 99, in start
    r = self.task(self, **self.params)
  File "/home/ubuntu/nbox/venv/lib/python3.8/site-packages/nornir_scrapli/tasks/netconf_rpc.py", line 26, in netconf_rpc
    scrapli_response = scrapli_conn.rpc(filter_=filter_)
  File "/home/ubuntu/nbox/venv/lib/python3.8/site-packages/scrapli_netconf/driver/sync_driver.py", line 305, in rpc
    response.record_response(raw_response)
  File "/home/ubuntu/nbox/venv/lib/python3.8/site-packages/scrapli_netconf/response.py", line 109, in record_response
    self._record_response_netconf_1_0()
  File "/home/ubuntu/nbox/venv/lib/python3.8/site-packages/scrapli_netconf/response.py", line 143, in _record_response_netconf_1_0
    self.result = etree.tostring(self.xml_result, pretty_print=True).decode()
  File "src/lxml/etree.pyx", line 3445, in lxml.etree.tostring
TypeError: Type 'NoneType' cannot be serialized.

OS (please complete the following information):

OS: Ubuntu 18.04
scrapli version: 2021.1.30
scrapli_netconf version: 2021.1.30
nornir_scrapli version: 2021.1.30
nornir version: 3.1.1

Additional context
Add any other context about the problem here.

Well when I decreased num_workers to 5, that begun to work without problems.

Hey @lastorel
Interesting! I'm not really seeing anything obvious -- from what I can tell basically what would have to happen to have that exception is that etree.fromstring would have to return None in here. which... I dont think I've ever seen it do... but clearly we are somewhere/somehow setting self.xml_result to None somewhere/somehow.
Would you be able to either get logs from when it fails, or perhaps simpler, put a print statement in scrapli_netconf/response.py at line 131 and 138 -- the first one we would probably want to print out self.raw_result to see what we are working with, then the second (line 138) would need to print self.xml_result to see what it got set to.
This may be a tricky one to figure out!
Thanks for opening this, hopefully we can get to the bottom of it soon!
Carl

OK.
Thats it:
        raw_result = self.raw_result
        self.xml_result = etree.fromstring(
            self.raw_result.replace(b"]]>]]>", b"").replace(
                b'<?xml version="1.0" encoding="UTF-8"?>', b""
            ),
            parser=PARSER,
        )
        xml_result = self.xml_result

        if self.strip_namespaces:
            self.xml_result = remove_namespaces(self.xml_result)
            self.result = etree.tostring(self.xml_result, pretty_print=True).decode()
        else:
            try:
                self.result = etree.tostring(self.xml_result, pretty_print=True).decode()
            except TypeError as e:
                with open("scrapli_netconf.log", "a") as fs:
                    fs.write(
                        "\nraw_result:\n" + \
                        str(raw_result) + \
                        "\nxml_result:\n" + \
                        str(xml_result)
                    )
                raise TypeError(e)

I think there is some problems of timeout/buffer operation:
scrapli_netconf.log

Yeah that looks like it'll do it... what I don't understand yet is how we even get there. From poking around it looks like from string should always raise an exception (that we wouldn't be catching anywhere that I am aware of) if it cannot parse the raw result.
So that is upsetting for two reasons:

what the heck is going on w/ that fromstring!
more importantly -- why are we "losing" the first part of the result... this is the real problem that we'll need to track down.

I'm assuming from the nornir config that you are using the system transport -- could you try this with paramiko or ssh2 and see if that changes things? Could you also snag full logs (preferably with system transport and also one of the other two so I can compare). If you would rather email me logs so they aren't posted here feel free: carl.r.montanari at gmail.
At the moment I am focusing on looking here and here but I cant reproduce this with my test vsrx so its a bit tough to really see whats going on.
I don't think that there is a timeout issue since we get the end of the message properly... we're just missing parts at the beginning. Thats leading me to think I've got something messed up with stripping/splitting out the channel input from the response... though the fact that it is seemingly random does not help with that theory.
Thanks for the help with this, I'm sure we'll figure it out... eventually 😁
Carl

I get transport types here:
https://carlmontanari.github.io/scrapli/user_guide/project_details/
My results with 20 nornir threads on 46 QFX/EX devices (3 custom tasks):
system: 2 attempts, 3-9 failed devices, Executing time: 0:00:12.804934 ( + sometimes there are empty success task results)
ssh2-python: 2 attempts, 0 failed devices, Executing time: 0:00:20.937015
paramiko: 2 attempts, 0 failed devices, Executing time: 0:00:47.183018
Sorry for my inexperience I don't know how i can dive deeper and get low level transport logs without your help

ssh2python goes brrrrr 🤣
Thanks so much for testing that @lastorel ! Super helpful and sadly confirms my suspicion that there may be an issue with system transport. Apologies for not mentioning how to snag the logs -- the easiest way is to use the enable_basic_logging function -- you might need to run this function after creating your nornir (typically nr) object since I'm not sure off hand what nornir is doing with logging. You can do this like:
from scrapli.logging import enable_basic_logging

< whatever in your normal code >

enable_basic_logging(file=True, level="debug")

That will spit out reasonably pretty looking logs to a file scrapli.log (if you prefer you can pass a string to a file path instead of True to the file argument). Probably with the logs I should be able to figure out some further troubleshooting steps and/or recreate it in some way so I can work on it.
Thanks a bunch for sticking with me on this!
Carl

Done. I sent logs to carl.r.montanari

I think this may be a regression related to #10 ... feels suspiciously the same... it looks like #30 is where the hopefully "fix" got pulled back out. Since you are just having issues with juniper we can can test this out by just quickly patching the "bare" rpc method.
If you can remove the \n character from here that might do the trick...
You can quickly find where that file is like python -c "from scrapli_netconf.driver import base_driver; print(base_driver.__file__)"
🤞 hopefully that is our issue... I think it may be but I'm not 100% sure since the logs seem to just end after sending the return character like we see here:
899   | 2021-06-22 04:07:41,342 | DEBUG    | <IP_ADDRESS>:22          | write: '<?xml version=\'1.0\' encoding=\'utf-8\'?>\n<rpc xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" message-id="103"><get-vlan-information><detail/></get-vlan-information></rpc>\n]]>]]>'
910   | 2021-06-22 04:07:41,390 | INFO     | <IP_ADDRESS>:22          | Read: b'<?xml version=\'1.0\' encoding=\'utf-8\'?>\n<rpc xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" message-id="103"><get-vlan-information><detail/></get-vlan-information></rpc>\n]]>]]><rpc-reply xmlns:junos="http://xml.juniper.net/junos/17.3R3/junos" << SNIP >>'
911   | 2021-06-22 04:07:41,391 | DEBUG    | <IP_ADDRESS>:22          | write: '\n'

Normally I would expect/hope to see something else after this since we should be reading until we see the prompt but I am thinking/hoping that the return we send, in this case on log line 911, is causing the junos box to close the connection since it is not expecting a return to be sent at that time. I tested removing that new line character on the xrv9k, iosxe (csr1000v), and vsrx that I normally use to test and it looks like it is working fine. I made the change here which is a little different since there has been some clean up in the current develop branch, but should have the same impact as the other place I mentioned (that should actually exist in the version you are using).
Thanks for getting those logs and all the effort helping hunt this down!
Carl

That works. After removing \n in _pre_rpc there is no fails (system transport):
DEBUG: Executing time: 0:00:13.307756
such wow

also ssh2 transport works without issue too

Yes!!! Thank you so much for bearing with me and troubleshooting this.... can't believe this happened again haha... I will get this fixed merged to develop probably tomorrow... need to add some tests to make sure this doesn't regress again. I'll leave this open till I get some further testing done, write a real test or two, and get stuff merged.
Thanks again for your help, couldn't have got it figured out without your efforts!
Carl

I will continue to recomend your nice soft!
Thanks so much

Going to close this out now -- this is in develop and there is a test to keep me from breaking this again in the future!
Carl


Amazon Anti-automation challenge

Trying to get this to work on Amazon, but it keeps running into an anti-automation challenge when trying to log in. See failures attached.
AntiAutomationFailure.zip

Are you using a VPN or a different network than you would normally log in with?
This can also happen if you try running debbit too many times in a row.

Nope, normal network. I got it the first time I ran debbit I think. It definitely didn't do any transactions before I was getting that. I had just changed my password and added a phone number to Amazon tho. I'll try to run it again later.

I intend to add the code to handle the anti-automation challenge.
Also, I've got an in-progress branch that keeps the debbit Firefox window open for multiple purchases. This should help prevent this issue since Amazon won't think it's a new browser logging into your account every time.

Also, I've got an in-progress branch that keeps the debbit Firefox window open for multiple purchases. This should help prevent this issue since Amazon won't think it's a new browser logging into your account every time.

Is "close_browser: no" in yet?

@Marksman79 Yes but only if you download and run the source code. I'm trying to get the next release out soon, but it might be a couple weeks.
If you download the latest code on the master branch you can use it now. I deleted the "close_browser: no" feature and replaced it with a different solution that uses cookies. Instructions for the current code on the master branch are found at http://jakehilborn.github.io/debbit

Much thanks for the status update! I'm more than happy to wait until the full release -- take your time :)

@Marksman79 Debbit 2.0 has been released which supports cookie persistence. This should prevent the anti-automation challenges causing failures.

v2.0 Worked perfectly, had the automation challenge and was able to enter the code and the OTP and then press enter in Debbit and it proceeded with no problem!


Expose duration on Modal/ActionSheet - Allow for custom values

It would be nice if the Modal and ActionSheet respectively had a openDuration/closeDuration prop. Maybe the current hardcoded values could be the defaults, and you could pass in custom values for slower or faster animations? In my case, I'd love to be able to slow them down a bit.

See this PR as an example.

Hi @thispastwinter, Thanks for reporting this issue. We will look into it soon 😄 .

Hi @thispastwinter, We have introduced new internal props. _fade, _slide, _backdropFade lets you control animation on these components.

@Viraj-10 awesome! This is great. Thanks!


add magma-nvim to plugins

magma-nvim  allows you to execute python files as if they were Jupyter notebooks right from inside Neovim. I have added all global options as well, with documentation. I am using the magama-nvim-goose fork until their upstream merges their #69 which allows for external kernels.
(unrelated, this is my first open source pull request, so am a little excited)

Whoops, ignore that approval I don't know how to github


fix single-chat throw no conversation id exception

Wechaty Contributors Nomination
Link to https://github.com/wechaty/PMC/issues/16

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


SLURM - Automatic testing are failing on HBv2 job

Job stay queued and node is in draining mode even after several hours
Cycle version 8.2.1, cyclecloud-slurm 2.5.0, SLURM 20.11.7
HBv2 is configured to be in Spot Allocation, and the machine was evicted.
The node remains ready in Cycle.
        "             JOBID PARTITION     NAME     USER    STATE       TIME TIME_LIMI  NODES NODELIST(REASON)",
        "                 4   hb120v2   c213de  hpcuser COMPLETI       0:00     20:00      1 hb120v2-2"

 "PARTITION AVAIL  TIMELIMIT   JOB_SIZE ROOT OVERSUBS     GROUPS  NODES       STATE NODELIST",
        "execute*     up   infinite 1-infinite   no       NO        all    512       idle~ execute-[1-512]",
        "hb120v2      up   infinite 1-infinite   no       NO        all     23       idle~ hb120v2-[1,3-24]",
        "hb120v2      up   infinite 1-infinite   no       NO        all      1    draining hb120v2-2",

After the eviction a new node has been added (the one in draining mode), and is still running in the VMSS.

The node appears to have less memory than configured.
This is the log in /var/log/slurmctld/slurmctld.log
[2021-12-14T09:52:21.414] debug:  Node hb120v2-2 has low real_memory size (435161 < 443596)
[2021-12-14T09:52:21.414] error: _slurm_rpc_node_registration node=hb120v2-2: Invalid argument


[FEATURE] Call answering status

FEATURE

Add CALL_ACCEPTED status

Hello I'm also interested in this awesome feature. Please let me know if this will be implemented

is this feature available, or would it be available soon?


[BUG] - TraceBlockFetchClient doesn't work

Interna
Summary
TraceBlockFetchClient doesn't work.
Steps to reproduce
Steps to reproduce the behavior:

Enable the tracer by adding "TraceBlockFetchClient": true, to the config file.
Run the node
grep for "BlockFetchClient"
Notice the lack of messages.

Expected behavior
Enabling TraceBlockFetchClient should result in the logging of messages related to the block fetch client.
System info (please complete the following information):

Node version 1.18.0

these messages are traced using Debug severity, so minSeverity filter must be set to Debug.

https://github.com/input-output-hk/cardano-node/blob/7d68626f1f9a56ab318fda7be4cb43122b31dad6/cardano-node/src/Cardano/Tracing/OrphanInstances/Network.hs#L111
To me it looks like they should be traced at a minSeverity of Info, but someone is overwriting it.

hi @karknu please can you check if this is still relevant?

Yes, the tracer is still broken.

diff --git a/cardano-node/src/Cardano/Tracing/OrphanInstances/Network.hs b/cardano-node/src/Cardano/Tracing/OrphanInstances/Network.hs
index 518eda7c..7006e430 100644
--- a/cardano-node/src/Cardano/Tracing/OrphanInstances/Network.hs
+++ b/cardano-node/src/Cardano/Tracing/OrphanInstances/Network.hs
@@ -85,7 +85,8 @@ instance HasSeverityAnnotation (TraceSendRecv a) where

 instance HasPrivacyAnnotation a => HasPrivacyAnnotation (TraceLabelPeer peer a)
-instance HasSeverityAnnotation a => HasSeverityAnnotation (TraceLabelPeer peer a)
+instance HasSeverityAnnotation a => HasSeverityAnnotation (TraceLabelPeer peer a) where
+  getSeverityAnnotation (TraceLabelPeer _p a) = getSeverityAnnotation a

Fixed in 6157e1c3f0ee9484f6cfce390b81ed97dd4c0025


Consider suggesting using tempfile classes as context managers

Just as in:
f = Path.open('x')  # SIM115 Use context handler for opening files
f.close()

Ruff nudges the user to use context managers, Ruff should do the same for:
from tempfile import NamedTemporaryFile
f = NamedTemporaryFile()  # Should nudge SIM115
f.close()

and similarly for the other classes in tempfile.

Sounds good to me!

Reopening the issue for now so we remember to stabilise this behaviour as part of Ruff 0.7


Bad Request / Error invalid boolean - V4

Hello, I have errors at the beginning of the script. The repositories information goes well in Influxdb.
I only have information about users that are in error.
HTTP/1.1 400 Bad Request
Content-Type: application/json
Request-Id: 9aeb2e11-d70a-11ea-89ec-00155dba5719
X-Influxdb-Build: OSS
X-Influxdb-Error: unable to parse 'veeam_office365_organization,veeamOrgName=My Name licensedUsers=4,newUsers=0': invalid field format
X-Influxdb-Version: 1.8.1
X-Request-Id: 9aeb2e11-d70a-11ea-89ec-00155dba5719
Date: Wed, 05 Aug 2020 10:58:33 GMT
Content-Length: 136

{"error":"unable to parse 'veeam_office365_organization,veeamOrgName=My Name licensedUsers=4,newUsers=0': invalid field format"}
HTTP/1.1 400 Bad Request
Content-Type: application/json
Request-Id: a1b58705-d70a-11ea-89ef-00155dba5719
X-Influxdb-Build: OSS
X-Influxdb-Error: unable to parse 'veeam_office365_overview_OD,veeamOrgName=My  Name,veeamUserName=admin@myname.onmicrosoft.com,veeamUserODName=0 protectedUser=2,typeUser=1': invalid boolean
X-Influxdb-Version: 1.8.1
X-Request-Id: a1b58705-d70a-11ea-89ef-00155dba5719
Date: Wed, 05 Aug 2020 10:58:45 GMT
Content-Length: 198

Hello, are you using the new script? It should parse your veeamorgname

I had the script from: 10/01/2021
I just tried the last script dated 02/06/2021 and I have the same error.

Hello,
Try the new script and let me know 👍

Is good ! I have no more mistakes but the number of licenses indicated is incorrect.
In Veeam 365, I have 13 licenses in use. And in Grafana I have 6 licenses used.

I have change SELECT field (licensedUsers) mean() by SELECT field (licensedUsers) sum()


scipy.signal.sigtools._convolve2d: ComplexWarning: Casting complex values to real discards the imaginary part

How to reproduce:
call
scipy.signal.sigtools._convolve2d([[2, 2, 2], [1, 2, 3], [0, 1, 0]], [[0.5], [0.5]], 1, 0, 0, 0)
flip=1 is necessary, with flip=0 warning doesn't appear.

that warning no longer appears, so seems fixed.


Does angular-ui team plan to support Angular2?

Angular2 by typescript1.5beta is in alpha preview.

Keep an eye on the meeting minutes in the wiki. Yes eventually we will support AngularJS 2, no, dont expect anything any time soon.


Blog on Hashnode

I want to crate a Complete blog of project and publish it on hashnode

It will have complete project details A to Z. The link of blog of hashnode will be inserted in readme file

@Vikash-8090-Yadav

which type of project ?

which type of project ?

Web3.0 world sir explaining all things in the webpage like a complete blog on hashnode

which type of project ?

Web3.0 world sir explaining all things in the webpage like a complete blog on hashnode

Sure , start working on this

thanks sir

please add the IWOC and  medium or hard tag sir

which type of project ?

Web3.0 world sir explaining all things in the webpage like a complete blog on hashnode

Sure , start working on this

Sir please be active because iwoc is ending tomorrow..so your activeness will be very good as I have to complete this issue ..
So please sir

please add the IWOC and medium or hard tag sir

thanks sir


[Release] sdk/resourcemanager/securityinsights/armsecurityinsights/2.0.0-beta.2

https://github.com/Azure/sdk-release-request/issues/3190

Wait for codegen change.


s/conflicting packages/other packages/

The term "conflicting packages" is inaccurate, because with the alternatives system, multiple packages can provide the same man page without conflicting with each other.

You’re right in that the packages don’t conflict with each other at the Debian package level, but from a logical perspective (detached from any technical details), there still is a conflict: there are multiple manual pages for the same name.
I don’t mind changing the terminology to be more descriptive, but at the same time, I’m not convinced that switching from “conflicting” to “other” is an improvement: “other” isn’t describing anything at all. As a reader, I can ask “sure, it’s another package, but what’s the relationship?”
I’m inclined to leave it as “conflicting”