[WIP][Stdlib][SR-1052] Added @discardableResult attr to mutating functions

What's in this pull request?
Part 2 for SR-1052: Add @discardableResult attr to mutating functions in stdlib that return results
Resolved bug number: Part 2 of (SR-1052)
I tried to find all of the mutating functions with non-Void return values and that were not marked with @warn_unused_result. I then marked them with the new @discardableResult attribute. These will stay around after the @warn_unused_result attribute is removed and warning on unused results becomes the default.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

Did you audit the underscore-land?  I see a few cases in stdlib/public/core/Runtime.swift.gyb.

@gribozavr, I was looking at mutating functions in particular as they have side-effects and the result may be unneeded. I don't see any mutating functions in stdlib/public/core/Runtime.swift.gyb, and most of the functions there have @warn_unused_result or return Void.
Do you have an example of where it should be added in that file?

@tanadeau I was thinking about _swift_stdlib_atomicFetch${operation}, in C these functions are typically used just for side-effects.  But I see what you mean -- they have fetch in their name, which sounds like the primary operation, and the result should not be discarded.  We can add functions without a result that don't have fetch in the name and don't have a result to make semantics more clear.
But there seem to be other cases -- for example what do you think about _StringBuffer.grow()?

I believe I've addressed all the comments with the exception of Dmitri's dealing with Runtime.swift.gyb.

I made all of the pop* methods have @warn_unused_result for consistency and correctness (to force checking of failure).
I removed the @discardableResult attr from the with* methods that take closures that can return something.
I removed the @discardableResult attr from the native* and _custom methods that are implementation details.
I removed the @discardableResult attr from a removeFirst that returned Void.

I added @discardableResult to _StringBuffer.grow(). That method appears to be used once with no use of the return value. Also the meaning of the return value is not documented.

Your patch LGTM to the extent of the changes that you made, but I'm still a little concerned about errors of omission.

@gribozavr, understood. One thing to keep in mind with this PR is that the @discardableResult attribute doesn't actually do anything yet. As part of the switch to make warning on unused results the default and enabling the new attribute, I wouldn't be surprised if some new warnings show up, but that would be taken care of in that future PR.

@tanadeau OK.  If you are happy with the current state of the patch, we can merge it.

I'm happy if you're happy :smile:

@swift-ci Please test and merge

I think I fixed the test failure that CI found. The expected output for the serialized XML version of the remove(_:) call was missing the new @discardableResult attribute.
I'm not sure why this test doesn't run for me. I'm testing in an Ubuntu VM using build-script -R -T.

@tanadeau Thank you!  Could you resolve merge conflicts?

@gribozavr I rebased and fixed the conflicts. I believe this is ready for a CI test-and-merge.

@swift-ci Please test and merge

@gribozavr The OS X build passed. The Linux build failed due to a mismatch between a change in swift-corelibs-xctest and swift-integration-tests.
/tmp/swift-package-tests/test-xctest-package/Output/test-xctest-package.py.tmp.dir/tool/main.swift:9:17: error: missing argument for parameter 'name' in call
let x = MyXCTest()
                ^
<unknown>:0: error: build had 1 command failures

The change at https://github.com/apple/swift-corelibs-xctest/commit/ce992f550b0bf2e10c66103c74a77576c63fb146 added a new required initializer to XCTest, but swift-integration-tests wasn't updated for subclasses of XCTest.


Asking for advices

Hello ! i hope you're doing well ! i need to be oriented a little bit ! i want to become a web developer . I learnt Python, Html/Css, Php, Mysql, and some JQuery ! and now im learning Bootstrap ! i don't know if this is enough to start applying what i've learnt and build real world websites or should i learn more ? I mean is this enough to start building projects ?

Please post your question on my website sonarlearning.co.uk


TypeError: React.createClass is not a function

./node_modules/react-videojs/dist/index.js
node_modules/react-videojs/dist/index.js:8
   5 | var blacklist = require('blacklist');
   6 | var React = require('react');
   7 |
>  8 | module.exports = React.createClass({
   9 |   displayName: 'VideoJS',
  10 |
  11 |   componentDidMount: function componentDidMount() {

I think this is due to the changes in react. Any plans to handle this?

+1
This, and any components importing it, are unusable with the newer version(s) of react, causing a hard-crash on load.

Yes, I had to downgrade to React 15 to get my work done.

I have the same problem. Does the author not maintain it?


Unable to load KYNode

I grabbed a workflow from a Youtube video using this node.  I loaded the workflow and then loaded all the missing nodes.
This one won't load.  I have run updates on the UI and the python dependencies to no avail.
The reason it won't load is that you introduced openai_VLM.py which requires dotenv - but this appears not to be a requirement for or loaded up with ComfyUI as standard - I am receiving the error:
File "F:\10 AI Images\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-KYNode\openai_VLM.py", line 7, in <module>     from dotenv import load_dotenv ModuleNotFoundError: No module named 'dotenv'
This is not an error I receive for any other module or why running ComfyUI.
I suspect I can fix it but you may want to either add it as a requirement for ComfyUI or remove it from the openai_VLM.py.

As a work around if you go to the ComfyUI folder you can use:
.\python_embeded\python.exe -m pip install python-dotenv
Which will install dotenv in the right place so that this node will now work.  Adding it to requirements doesn't help, because it says it is satisfied based on your global - it won't add it here.

I am also having this issue.
I tried installing that and it installed okay, but It's still failing to import.
0.0 seconds (IMPORT FAILED): x:\xx\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-KYNode

requirements.txt updated,
I have removed dotenv reference,


fix(versions): update Activiti/activiti-cloud-full-chart-security versions into master

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.59

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.59


applyBoxFit doesn't properly handle an output size of Size.zero

https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/painting/box_fit.dart#L127
The result is a returned size with Nans in some cases resulting in issue: https://github.com/flutter/flutter/issues/11586

We might want to assert it's non-zero, and have callers handle that case specially. There's rarely any point to do anything if you're outputting to a zero rect.

As mentioned offline, I would expect the returned size(s) to be Size.zero not assert.  There's legitimate use cases where the given output size is Size.zero.


Create README.hi-IN.md

Added Hindi README file, making sure the important English words are is it is.

Please have a look @bitcrazed :)

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.


Inna: Headless-CMS, 3 weeks

my fork of module repository

Learning Objectives

[x] 🥚 Understanding what a content management system is, and why people use it
[x] 🥚 Given an existing CMS installation fill in and publish a new piece of content
[x] 🥚 Perform a simple http call via postman to retrieve a piece of information.
[x] 🐣 Extend and adapt an existing CMS configuration
[x] 🐣 Perform a http call with filtering via Postman
[x] 🐣 Perform a http call with "include" via Postman
[x] 🐣 Perform a http POST call to create new data
[x] 🐣 Perform a http PUT call to update existing data
[x] 🐣 Configuring endpoint permissions (Authentication vs Authorization)
[x] 🐥 Design a data model from scratch based on user requirements

Week 1
I Need Help With:

Not yet

What went well?

CMS installation fill in and publish a new piece of content

What went less well?
Lessons Learned

Understood what a content management system is, and why people use it

Sunday Prep Work

In progress

Week 2
I Need Help With:

Not yet

What went well?
What went less well?
Lessons Learned
Sunday Prep Work

In progress

Week 3
I Need Help With:

Not yet

What went well?
What went less well?
Lessons Learned
Sunday Prep Work

In progress


Week 5 Review Ticket

2.7+/3

[x] JavaScript with Website
[x] Fibonaaci Sequence
[x] Team Planning

Score: 2.7+

Outfit store, nice wireframe, good message presented as well.
Inclusion of APIs, Search, login system
Displays knowledge of extends in classes, while slightly different from Mr. Mort's ideal code I think its good
No super();
Nice javascript table implemented, relates to the project idea as well

Score: 2.7+/3
Comments:

Very good job presenting the idea as well as the purpose behind it - also included ideas for technicals and features
Wireframe ideas look very professional and show their idea for the website styling
Proper use of extends in fibonacci, showing for, while, and recursion loops - missing the super() but not a big deal
Implemented a javascript dress list - repurposed from Mr. M's code, but also reveal plans to modify it around their website

Shreya - Clothing shopping.  Fashion.  Using a lot of API and Pixel finder.  Lots of searching.   Alerts.
Bria/Ellie - Fibo  did not do extends.  Worked on loop in independent print.  Talked about recursive.    Has some learning and review to perform.
Aadya/Shreya - Quicly reviewed FIbo, closer to requirements.  Javascript adapted to project.    Talked about future with javascript and pages.
2.7 for all


Support module script dependencies

This is required for  because it depends
on  component.

This change is

Confirmed that the element-mixin test suite passes locally with this change.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


The TensorFlow library was compiled to use SSE4.1 instructions, but these aren't available on your machine

Install Problem here:
I am running an Ubuntu 18.04 VM, with a Py3.6 environment, on a windows host machine. I got through the install with minimal problems, and compiled Tensorflow as everyone suggested. I am not totally sure what information is even relevant here, but ultimately, I cannot get past the "Check your Install" section. I run
python -m spinup.run ppo --hid "[32,32]" --env LunarLander-v2 --exp_name installtest --gamma 0.999
and I end up getting this as an error message.
2018-12-29 09:09:02.787690: F tensorflow/core/platform/cpu_feature_guard.cc:37] The TensorFlow library was compiled to use SSE4.1 instructions, but these aren't available on your machine. Aborted (core dumped)
Nothing I've found on SO or various other sites has helped. Any advice or guidance you could provide would be greatly appreciated.

Same issue, same OS and environment, also in a VM. Please help!

I have a AMD a4-3400 APU with Radeom(tm) HD Graphics Too and i face the same error

I am using an old AMD, so this error still exist there. Commenting out this line worked for you? For which TF versions (?)


Fix to_s: responds_to?. Fix msg in parser, same.

Seemed unnecessary to bloat specs with test for this.
The to_s problem (testing with to_s, LOL):
pp 1.responds_to? :to_s

The message in the parser:
1.responds_to? to_s

Thanks! Seems almost OK, the symbol might have quotes (search Symbol.needs_quotes? in the project). I'd also like a spec or two for this :-)

What do you think of KISS-approach simply quoting/escaping it always?
Put specs in macro-specs?

I'd prefer to avoid quotes if they are unnecessary. Specs go here.

Cool, I'm on it.

Hope I got everything right with git-squashing and such now.

Perfect, thank you! :-)


Improvement multiply/plus DECIMAL32 and DECIMAL64 safely and not check overflow

Proposed changes
multiply/plus DECIMAL32 and DECIMAL64 is safe and we do not need to check overflow.
I tested this PR on TPCH Q1: ~20s -> ~16s
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall


Feature request for more detailed comments fields.

For example.. I just used this tool to import the "Non-crossed" list.
All of the comments fields are the same "Firebog | Non-crossed".
It would be nice if the ones which are also in the "Ticked" list showed as "Firebog | Ticked" instead, so I can tell them apart.
I'm just thinking that if I ever have an issue and need to whitelist, it will be useful to know if the list containing the site I am working on is Ticked or Non-Crossed.
I think I can probably get what I want by deleting them, and then re-running the tool twice.  Once to add the Ticked list, and then again to add the Non-crossed list.  (I'll give that a try next).

As is, you can get that to happen by importing them smallest to largest since non-crossed is (always?) a super-set of ticked.
I've been talking to the guy who maintains the lists about providing slightly different ways to pull the lists, so I'd like to see where that goes before I do something weird (code-wise) like this.

Makes sense.
As you say, I was able to get what I wanted by deleting them, and then re-running the tool twice. Once to add the Ticked list, and then again to add the Non-crossed list.
So, there is nothing urgent about this.


Multiline command doesn't work?

In Node Repl, I can run the following short script but in loopback console it ends with SyntaxError.
if (1) {
  console.log('Hello');
}

Any config change which I am missing here?

Good find! Thanks!

Thanks. @doublemarked.


(libs) Re-design the hierarchy of standard-library.

FACTS:

There are some basic structures we use in every day programming activities or proof constructions, such as some basic algebraic structures (e.g. semigrp, monoid), higher-order functional vehicles (functor, applicative, monad, arrow and maybe selective)
There are some basic data structures which can be implemented in a straight-forward built-in fashion, made them efficiency and usable.
Combine the above, we can build a minimal scale library which shows the functionalities of functional programming languages.

Lift them to a sperate library Prelude can resolve some pressures and redundancy  dependencies which had appeared in building the base library. And this approach can be considered easier to further maintaining job and documented materials. This is also a chance to solve some not valuable legacy that from Haskell and PureScript. So syntax like applicative (idiom) bracket, bind-notation and pattern-matching as control flows; applicative record update, visitor composition is also a point to throw away some functions.
Design the hierarchy of such essential type classes is up for more discussions with some sophisticated developer. It is becoming even more circuital since we do not have mechanism to do default behaviors or minimal implementations. It is worth doing in the meanwhile.
In addition, some experiential and not essential libraries can be put to contrib, such as testing framework. It doesn't need to support a stable API since the languages is developing itself.

good suggestion.
We are still improving the standard library.
If we have more time, we will consider adding such an experimental library.


Intellisense with require

Issue Type: Bug
Noticed after some months and some updates of not using node.js (at least any external modules) that "require" doesn't offer intellisense even in older projects. Suggests using import, with which intellisense works.
Edit: forgot to add images

Require has a little decoration, because:

Have tried removing all extensions and settings, clean installing latest stable vscode 1.30.2 and 1.29.1, and making a new project in another folder - behaved identically. Also tried suggested tsconfig.json.
I've disabled suggestion to import in settings, to not get suggestions for "import {readFile} from 'fs'".
Usually setup with "npm init" and then "npm i -D @types/node" (tried also without -D). Also tried multiple also dev release of typescript.
Didn't notice any errors in TS log.
VS Code version: Code 1.30.2 (61122f88f0bf01e2ac16bdb9e1bc4571755f5bd8, 2019-01-07T22:54:13.295Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 x 3193)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.93GB (8.26GB free)

Process Argv

Screen Reader
no

VM
0%

@mjbvz Man this is very annoying. Sometime one have to require some function in typescript thats not typed and these is not even function signatues shown in ts file but vscode figures it out in js file.
More over tslint plugin complains even when I put disable comment for some section.

Mentioning that vscode 1.31.0 and TS 3.3.1 selected hasn't impacted this.
import fs = require('fs'); works too, but is still less useful, can't do import {property} = require('module'), can do import {property} from 'module' - getting bit odd js from it, requires whole thing into a temporary variable and adds another line to assign the property to the name I declared. In the same pattern have to do import * as module from 'module' to get regular stuff. Import is just more cumbersome for me.
Side note: better json support would be nice as well... (I think we used to have this...)

This sounds similar to https://github.com/Microsoft/TypeScript/issues/30056
@entr0cks Does intellisense for require never work in your project, or only after you make some edits? Also, do you have a small example project that shows this behavior?

Similar to linked issue.
Never works on the few computers that I have. Just create a new .ts file and require into a var/let/const, the type of that will be any (see my prev comments).


Destroying HotspotDOM

Fix #122
When destroying a HotspotDOM, do not remove the dom but hide it away.
Adapted the events because of bad reference when destroying it.

 All committers have signed the CLA.


NProgressOptions doesn't have any type suggestion with TypeScript

I'm using Next 13.4.6.
It will be great if you integrate the NProgressOptions in this awesome package.

Hello, I will work on this.
I'll keep you informed.

I just updated the plugin to version 1.2.5.
You can now import the NProgressOptions type directly from the plugin like this:
import type { NProgressOptions } from 'next-nprogress-bar';

I just updated the plugin to version 1.2.5.
You can now import the NProgressOptions type directly from the plugin like this:
import type { NProgressOptions } from 'next-nprogress-bar';

Great work!
Thank you for your quick fix.
Now it's working perfectly.


chore: use @hello-pangea/dnd

This PR migrates react-beautiful-dnd to @hello-pangea/dnd
react-beautiful-dnd is no longer maintained and will not support React 18. Myself and another developer have converted the codebase to Typescript and have added support for React 18 and strict mode.

Thanks @Xhale1 , we'll re-open this PR when we look at updating to React 18.


Test not failing when exception is thrown (in v1.9.0)

Description

I was doing some TDD with new Catch version (1.9.0).
When expecting that tests failed, they were passing.
Reducing the problem to minimum, discovered that the
(expected) exception was been thrown. But the tests was passing.
I also tested with release 1.8.2 and it worked fine.
Steps to reproduce
This minimal code can reproduce the error (i hope):
#define CATCH_CONFIG_MAIN
#include "catch.hpp"

TEST_CASE("should fail"){
    throw std::exception();
}

Extra information

Catch version: v1.9.0
Operating System: Linux
Compiler+version: gcc 5.4.0

Well, that was an embarrassing mistake. The underlying cause was that to prevent double reporting of exceptions when using CATCH_CONFIG_FAST_COMPILE, unexpected exceptions had to be ignored under certain conditions. Due to a mistake on my part, when CATCH_CONFIG_FAST_COMPILE was not defined, all exceptions were ignored...
I am going to tag this with a revisit, because I need to take a look at why this didn't show up in Catch's self-test, but 1.9.1 is out and it should be fixed.

I figured out why it didn't show up -- Phil approved the differences caused by the bug by mistake in 250f0ee7fbda0268cd91f2411236a923b44fbe39...

Yup - sorry about that. Thought it was something else.


Privacy mode

Hi, I have a DH-SD-H4C-0400B camera, it has a privacy mode where its lens goes down. When I enable this mode in DMSS, the following entry appears in the camera logs
Name: leLensMask
State: Start
Result: Success

Maybe someone knows if there is any way to manage this locally via cgi or rpc?

Just switched from Amcrest to Dahua integration  and would also love the ability to flip on the Privacy Mode switch for some cameras that I have that support it (model IP4M-1041W if it matters). The official Amcrest integration has it, would it be possible to port it from there maybe?

I've DH-H4C This camera also has a Pivacy mode.


Add provisioning logic

This successfully triggers provisioning.

I rebased this. The latest series of patches allows me to write the image to the VM, and the VM boots. I would like to merge this before working on some of the error handling cases.

I'm fine with this based on existing reviews.  @dhellmann I'll leave the merge to you so you can decide how you want to squash this

I rebased and squashed a couple of the fix-up patches into the main provisioning patch, but I'll leave the others as they are because they're somewhat unrelated changes.


Release v0.2

Merge release-v0.2 branch back to master

I'm not sure which is more exciting--a release-v0.2 tag or a merge back to master!


Add a version of io::BufRead::read_until() with a read limit

So there's an ::std::io::BufRead::read_until(), which is great, except in cases where the input is untrusted (e.g. the context of a server process that listens for connections from the general public): while there's no possibility of an actual buffer overflow, an attacker could trivially cause undesirably high memory usage. Thus, read_until() would have to be avoided, but implementations might end up with what amounts to the same thing, with a limit on the amount of data read.
Given that this is a common usecase (it seems so to me, anyway), I propose adding this variation to BufRead as a provided method:
/// As `read_until()`, except that `buf` will be overwritten, not appended to,
/// and with the additional restriction that not more than `buf.len()` bytes
/// will be read.
fn read_until_full(&mut self, byte: u8, buf: &mut [u8]) -> Result<usize> { ... }

(I'm not particularly attached to the name read_until_full. There's probably a better one for it.)

You can use
bufreader.take(max_bytes).read_until(byte, &mut buf)

…and if you need to reuse bufreader later, you can just do bufreader.by_ref().take(...). Sounds good to me; thanks!

But with take(max_bytes) data after the byte will be lost

That’s what https://github.com/rust-lang/rfcs/issues/1427#issuecomment-216115458 is about.


Missing output tensor resize in Max/Min Ops

Max/Min Ops do not resize the output tensor properly.

duplicate issue: #64
closing it.


Spelling error in log message.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

thanks!


There are guidelines and tools for releasing artifacts to separate suite components

GIVEN I am releasing suite component A
AND the release of suite component A produces an artifact A1
AND the artifact A1 must be uploaded to suite component B on release of suite component A
THEN there are Jenkins pipeline and/or GitHub action utilities that can copy the component A pipeline-built artifact A1 to a folder on a branch of component B
AND there is a Jenkins pipeline and/or GitHub action utility to open a PR to merge the branch including artifact A1 into the main branch of component B
Said plainly, there are tools I can use so that if my project's pipeline produces a ZIP or a brew .rb file, I can use my pipeline to upload the ZIP or .rb file to a separate GitHub repo (like our helm chart repository or homebrew cask) in a new pull request.
In order for this to be useful, the outcome of this card should be some examples and documentation that others may follow.

Note - this may be accomplished by having component B watch for new releases of component A, and then modifying itself as appropriate. A GitHub action would work well to do this for OSS projects.


Fix NIOSSL deprecations

In NIOSSL 2.14.0 a patch landed that deprecates the .forClient way to create TLSConfigurations.
This ticket consists out of two tasks:

[ ] bump required NIOSSL version to 2.14.0
[ ] replace all usages of TLSConfiguration.forClient with TLSConfiguration.makeClientConfiguration

Fixed thanks to @jareyesda!


Consider not offering plain getAll() methods for models like device/tags/etc in favor of the getAllByXYZ()

See: https://balena.zulipchat.com/#narrow/stream/345748-aspect.2Fengineering/topic/Encourage.20friendly.20SDK.20usage

I had the idea of creating a getAll method for the release model, but I came across this issue. I'm synchronizing some data across multiple fleets, including releases, but this doesn't have the getAll method.
Just here to give my opinion, but why remove the getAll method? This is a very powerful method and requires only one call to the API to get just about everything you need.

Hi @bartversluijs,
The reason that we started being against plain getAll methods, is that at this point our platform offers many public resources as well, and we have seen users doing plain getAll requests without any additional $filter statement. This results retrieving way more data that intended, which does consume both user & server bandwidth, and users then have to do the filtering using code.
For example, if at this point we implemented release.getAll() and a user was to make that call without any $filter option provided, that would retrieve 34440 public releases, which include public balenaOS & supervisor releases, as well as releases of open-fleets, apps & blocks.
As a result we decided that it would be beneficial to not offer getAll methods, but provide variants like getAllBy<relation> similarly to how the release model has getAllByApplication.
We do realize that cases like the one you described where the scope is broader exist, but we think that those use cases can be supported by variants like device.getAllByOrganization and adding support for providing an array of items to filter on, rather than a single item like it is today. For example changing release.getAllByApplication also accept an array for application slugs.
Would that work for your use case?

Hi @thgreasi,
Thanks for your detailed explanation and it absolutely makes sense that you've decided not to provide a getAll function for all resources. In a perfect world, everyone using an API would only filter on what's absolutely necessary instead of filtering on the client-side, but it's not a perfect world unfortunately 😅.
But your proposal to add support for arrays instead of single strings only would be perfect. You still need to filter and it requires less requests, which keeps the server happy and, in my case, my background job. Just out of curiosity, is there a limit on how many slugs can be provided in such an array / filter?

Hi @bartversluijs,
It's great to see that you like our proposal.
We currently do not apply any custom limitation on the number of items that can be used in $in $filter expressions. Realistically the first limitation that someone could hit is having the length of the generated query sting exceeding maximum allowed limits of the browser/server/etc.
As an example of query string size, we are confident using 50 device uuids in a $in $filter would generate a request with a query string within the supported range.
On top of that though, for all the methods that we atm offer support for batch operations, the SDK splits the parameters in chunks if needed and does multiple requests, so that it can avoid that limitation and abstract it away from the user.
We atm only support batch operations with arrays of ids as parameters, but I'm already working on adding support for string array parameters in the next major.


Error: tpm attestation: only TPM_ALG_RSA supported

Hi,
I'm using Windows Hello, if I use -7  (ECDSA w/ SHA-256) as the first crypto param which Chrome suggests should always be there, I get the above error from parsePubArea. However if I use [-257, -7] then Chrome is happy and Fido is happy, however I don't know the impact of this?

Shouldn't make any difference other than MS working.
-7 is "ES256" and -257 is "RS256".
Seems like a bug on Windows part.


Code Completion in Oracle

System information:

Windows 10
DBeaver version 7.0.3

Connection specification:

Oracle

Describe the problem you're observing:
Code completion in Oracle suggests SYSDATE() followed by a couple of parenthesis, which result in a syntax error, since SYSDATE in Oracle is a reserved word, not a function
Steps to reproduce, if exist:
Start code completion in an Oracle SQL window
Include any warning/errors/backtraces from the logs

thanks for the bug report

the same applies to postgres with current_time and current_timestamp (possibly more).

still reproducible for postgresql

Yes, I didn’t specify.
I removed from functions these keywords in Oracle: CURRENT_DATE, CURRENT_TIMESTAMP, DBTIMEZONE, SESSIONTIMEZONE, SYSDATE, SYSTIMESTAMP
From PostgreSQL i removed only CURRENT_DATE. Another functions may have precision

https://www.postgresql.org/docs/current/functions-datetime.html

verified


Add performance-move-const-arg and performance-unnecessary-value-param checks to clang-tidy

https://github.com/flutter/engine/pull/36221 fixed a lot of these issues, but there are still performance-move-const-arg failures in Linux Android clang-tidy
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8799825936871750113/+/u/test:_lint_android_debug_arm64/stdout
❌ Failures for clang-tidy on /b/s/w/ir/cache/builder/src/flutter/flow/diff_context.cc:
/b/s/w/ir/cache/builder/src/flutter/flow/diff_context.cc:205:24: error: std::move of the variable 'readback' of the trivially-copyable type 'Readback' has no effect [performance-move-const-arg,-warnings-as-errors]
  readbacks_.push_back(std::move(readback));

All platforms had performance-unnecessary-value-param errors:
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8799824923980615249/+/u/test:_lint_ios_debug_sim/stdout
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8799824923980615265/+/u/test:_lint_host_debug/stdout
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8799824923350402209/+/u/test:_lint_host_debug/stdout
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8799824923350402193/+/u/test:_lint_android_debug_arm64/stdout
Fix the errors, and turn these checks on.  More context: https://github.com/flutter/engine/pull/36885#discussion_r1000064035
cc @gaaclarke

These are done now: https://github.com/flutter/engine/blob/main/.clang-tidy#L28


Fix MarathonResource hash as well

Turns out that Python3 does not allow child classes to inherit hash methods, if you explicitly define eq you also have to explicitly define eq.
MarathonTask -> MarathonResource -> MarathonObject, so for MarathonTasks to be hashible we actually need to define hash on MarathonResource.
Oops.

This should fix #203 for real


Document how to use offline buildpack

As a note, this seems to cause issues with pcfdev. Running into this with 0.15.0:
2016-05-24T21:24:14.78-0400 [APP/0]      OUT 2016-05-25 01:24:14.782 ERROR 13 --- [cloud-foundry-2] o.s.c.d.s.c.CloudFoundryAppDeployer      : Error creating app bar-http
2016-05-24T21:24:14.78-0400 [APP/0]      OUT org.cloudfoundry.client.v2.CloudFoundryException: CF-AppInvalid(100001): The app is invalid: buildpack java_buildpack_offline is not valid public url or a known buildpack name

Added description of optional CF deployment properties.  Merged as 8d52376f6ecc043f26580e55ce8ba09d3cca2d4b


Navigating to a custom screen on Notification Click

I have everything set up and working even on the background. Thanks for the amazing library. So I wanted to know how I can navigate to a screen different from the initial screen, am using autoroute, I am listening on the action stream so its very specific.

Hi! Sorry for taking so long to respond to your issue on GitHub. I was focused on releasing the new 0.7.0 version of awesome_notifications and awesome_notifications_fcm and i didn't had time enough to do both. But now I can answer all your questions.
So now i'm asking you to recreate this topic using the new issue template. There's a lot of missing informations that i need to understand your problem, and all the instructions are already in the new issue template. Also, remember to check beforehand if your issue was posted by another user.
So I will automatically close all previous issues so far.
Sorry for the inconvenience and i will be waiting for your new issue request.
Thank you so much for your support!


how to update update resolv.conf in Cloudfoundry VMS??

When we deploy cloud foundry on AWS ,following entries are updated in resolv.conf
cat /etc/resolv.conf
nameserver x.x.x.x
nameserver x.x.x.x
search x.x.x.x
We would like to add "options rotate timeout:1 retries:1" in resolv.conf during VM creation process .
cat /etc/resolv.conf
options rotate timeout:1 retries:1
nameserver x.x.x.x
nameserver x.x.x.x
search x.x.x.x
What is the best way to achieve this ?
Regards,
Naresh Sharma

Hi @nsharmacovs. Changes to resolv.conf are made by BOSH rather than cf-release, so you'll have more success contacting the BOSH team. You can find them in their Slack channel (https://cloudfoundry.slack.com/messages/C02HPPYQ2) or opening an issue on https://github.com/cloudfoundry/bosh.
Also, cf-release is no longer a supported product. I'd advise you move to cf-deployment as soon as possible.

Thanks for the reply .We are in process to move to CF-deployments .


client: Add LocalVehicle to LocalPlayer vehicle property

Please target the dev branch instead of master since master is only used for qa-dev-tools atm.


add rfl::NamedTuple::apply() for convenient compile-time iteration

Adds rfl::NamedTuple::apply() as proposed in https://github.com/getml/reflect-cpp/issues/58.
I made one small design deviation from @liuzicheng1987's proposal which was to implement rfl::Field::name() by way of its rfl::internal::StringLiteral name, rather than via rfl::Literal, so that it can be a compile-time constant. I think that's important, e.g. the simple test I added wouldn't be possible without it.

I think the compile-time name is a great idea!


Environment variables in docker-compose.yml are not expanded

Describe the bug
Environment variables in docker-compose.yml are not expanded
Steps to reproduce
I have prepared the following docker-compose.yml Everything is based on https://github.com/containerd/nerdctl/blob/main/examples/compose-wordpress/docker-compose.yaml except for one line.
version: '3.7'
services:
  wordpress:
    image: wordpress:5.7
    restart: always
    ports:
      - 8080:80
    environment:
      WORDPRESS_DB_HOST: db
      WORDPRESS_DB_USER: exampleuser
      WORDPRESS_DB_PASSWORD: examplepass
      WORDPRESS_DB_NAME: exampledb
    volumes:
      - wordpress:/var/www/html
      - ${HOME}/hello.txt:/root/hello.txt
  db:
    image: mariadb:10.5
    restart: always
    environment:
      MYSQL_DATABASE: exampledb
      MYSQL_USER: exampleuser
      MYSQL_PASSWORD: examplepass
      MYSQL_RANDOM_ROOT_PASSWORD: '1'
    volumes:
      - db:/var/lib/mysql

volumes:
  wordpress:
  db:

I have created a file to mount the volume.
echo $HOME
/Users/hieki

echo hello > ~/hello.txt
cat ~/hello.txt
hello

I ran the finch compose up command. It failed.
WARN[0000] Ignoring: volume: Bind: [CreateHostPath]
INFO[0000] Ensuring image wordpress:5.7
INFO[0000] Ensuring image mariadb:10.5
INFO[0000] Creating container myapp_wordpress_1
INFO[0000] Re-creating container myapp_db_1
FATA[0000] error while creating container myapp_wordpress_1: exit status 1
FATA[0000] exit status 1

I modify docker-compose.yml and rewrite the files to mount to absolute paths. It works fine.
version: '3.7'
services:
  wordpress:
    image: wordpress:5.7
    restart: always
    ports:
      - 8080:80
    environment:
      WORDPRESS_DB_HOST: db
      WORDPRESS_DB_USER: exampleuser
      WORDPRESS_DB_PASSWORD: examplepass
      WORDPRESS_DB_NAME: exampledb
    volumes:
      - wordpress:/var/www/html
      - /Users/hieki/hello.txt:/root/hello.txt
  db:
    image: mariadb:10.5
    restart: always
    environment:
      MYSQL_DATABASE: exampledb
      MYSQL_USER: exampleuser
      MYSQL_PASSWORD: examplepass
      MYSQL_RANDOM_ROOT_PASSWORD: '1'
    volumes:
      - db:/var/lib/mysql

volumes:
  wordpress:
  db:

finch compose up
WARN[0000] Ignoring: volume: Bind: [CreateHostPath]
INFO[0000] Ensuring image wordpress:5.7
INFO[0000] Ensuring image mariadb:10.5
INFO[0000] Creating container myapp_wordpress_1
INFO[0000] Re-creating container myapp_db_1
INFO[0000] Attaching to logs
wordpress_1 |AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
wordpress_1 |AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
wordpress_1 |[Fri Dec 16 12:35:10.369453 2022] [mpm_prefork:notice] [pid 1] AH00163: Apache/2.4.38 (Debian) PHP/7.4.21 configured -- resuming normal operations
wordpress_1 |[Fri Dec 16 12:35:10.369539 2022] [core:notice] [pid 1] AH00094: Command line: 'apache2 -D FOREGROUND'
db_1        |2022-12-16 12:35:10+00:00 [Note] [Entrypoint]: Entrypoint script for MariaDB Server 1:10.5.18+maria~ubu2004 started.
db_1        |2022-12-16 12:35:10+00:00 [Note] [Entrypoint]: Switching to dedicated user 'mysql'
db_1        |2022-12-16 12:35:10+00:00 [Note] [Entrypoint]: Entrypoint script for MariaDB Server 1:10.5.18+maria~ubu2004 started.
db_1        |2022-12-16 12:35:10+00:00 [Note] [Entrypoint]: MariaDB upgrade not required
db_1        |2022-12-16 12:35:10 0 [Note] mysqld (mysqld 10.5.18-MariaDB-1:10.5.18+maria~ubu2004) starting as process 1 ...
db_1        |2022-12-16 12:35:10 0 [Warning] Could not increase number of max_open_files to more than 1024 (request: 32184)
db_1        |2022-12-16 12:35:10 0 [Warning] Changed limits: max_open_files: 1024  max_connections: 151 (was 151)  table_cache: 421 (was 2000)
db_1        |2022-12-16 12:35:10 0 [Note] InnoDB: Uses event mutexes
db_1        |2022-12-16 12:35:10 0 [Note] InnoDB: Compressed tables use zlib 1.2.11
db_1        |2022-12-16 12:35:10 0 [Note] InnoDB: Number of pools: 1
db_1        |2022-12-16 12:35:10 0 [Note] InnoDB: Using ARMv8 crc32 + pmull instructions
db_1        |2022-12-16 12:35:10 0 [Note] mysqld: O_TMPFILE is not supported on /tmp (disabling future attempts)
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Using Linux native AIO
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Initializing buffer pool, total size = 134217728, chunk size = 134217728
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Completed initialization of buffer pool
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Starting crash recovery from checkpoint LSN=45391,45391
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: 128 rollback segments are active.
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Removed temporary tablespace data file: "ibtmp1"
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Creating shared tablespace for temporary tables
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Setting file './ibtmp1' size to 12 MB. Physically writing the file full; Please wait ...
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: File './ibtmp1' size is now 12 MB.
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: 10.5.18 started; log sequence number 45403; transaction id 20
db_1        |2022-12-16 12:35:11 0 [Note] Plugin 'FEEDBACK' is disabled.
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Loading buffer pool(s) from /var/lib/mysql/ib_buffer_pool
db_1        |2022-12-16 12:35:11 0 [Note] Server socket created on IP: '::'.
db_1        |2022-12-16 12:35:11 0 [Note] InnoDB: Buffer pool(s) load completed at 221216 12:35:11
db_1        |2022-12-16 12:35:11 0 [Note] Reading of all Master_info entries succeeded
db_1        |2022-12-16 12:35:11 0 [Note] Added new Master_info '' to hash table
db_1        |2022-12-16 12:35:11 0 [Note] mysqld: ready for connections.
db_1        |Version: '10.5.18-MariaDB-1:10.5.18+maria~ubu2004'  socket: '/run/mysqld/mysqld.sock'  port: 3306  mariadb.org binary distribution

Expected behavior
Environment variables in docker-compose.yml should be expanded.
Workaround
When using docker compose run, you can mount any file by using the -v option.
However, this is not available with docker compose up.

Hi @hieki, it is an issue existing in upstream. I've fowarded your issue to upstream nerdctl. https://github.com/containerd/nerdctl/issues/1663
I think other environment variables except for $HOME works fine, for example $PWD. And I guess the reason for this error might be that the $HOME is resolved into the home directory of guest instead host from this github issue comment. https://github.com/containerd/nerdctl/issues/996#issuecomment-1284131734

Closed because this is duplicate of an upstream issue https://github.com/containerd/nerdctl/issues/996.


Add cluster-service stats

Add simple stats to cluster, showing number, and type of, requests processed.
Example output:
> show stats
name: cluster
-------------
map_shard_req   map_shard_resp
1               2

@jwilder

@corylanou

+1

+1


update INSTALL.md

Building F* was taking >30 minutes for me with make -C src/ocaml-output -j 6, on a system with an i7 4700 @ 2.4 GHz (8 threads) and 12GB of RAM.
Profiling showed that building F* was maxing out my RAM. make -C src/ocaml-output -j 4 is much faster for me.
Adding a note about this so that others can do faster builds.

That's strange. I routinely run make -C src/ocaml-output -j 24 on an i7 (also 4 cores) with just 8GB of RAM and I never ran into the issue you're experiencing. It completes in about 30 seconds.

My mistake, I got the command wrong - this was with make -j 6 ocaml-fstar-ocaml.
I've had the same issue on other systems before, but I'll test with some other systems with the current build as a sanity check.

That makes more sense, I can seen the memory usage peaking with that target.
It's still unusual. It typically takes around 6 minutes for me on the same machine.

Thanks!


QPy3.6 Terminal: Permission denied

Hi,
I installed QPython and QPy3.6 but when I open the terminal with the Python 3.6 switch checkbox activated, the terminal says:
/data/user/0/org.qpython.qpy/files/bin/qpython3-android5.sh && exit
y/files/bin/qpython3-android5.sh && exit <
/system/bin/sh: /data/user/0/org.qpython.qpy/files/bin/qpython3-android5.sh: can't execute: Permission denied
126|ASUS_Z017D_1:/ $

Could you help me?
Thanks

same issue with me on my galaxy s7.  I can't execute any python like I can from my tablets

If it persist, try clearingthe cache; if it still persists, try zipping your qpython folder, clearing the app files, open the terminal again, and then unzip your qpython folder.


LibIPC: Wait until socket is writable when transferring IPC messages

This fixes a large number of timeouts in WPT tests using workers where a large amount of data is transferred. For example: http://wpt.live/encoding/textdecoder-fatal-single-byte.any.worker.html  (and many others)

Is this meaningfully different from just replacing the sched_yield() with some kind of usleep()? :thonk:

My intent in using poll() rather than sleep() was to ensure that we could continue writing as soon as the socket was no longer blocked.
As previously written, there was no real difference between using usleep(). I believe it now functions as intended.


Delete CNAME

Since the domain is expired, I think. And it's redirecting to a driver(?) site, I think it's best to remove this CNAME to avoid github redirecting to semantic.gs.

I already tried that but nobody cares for now :( https://github.com/tylertate/semantic.gs/pull/115

Yeah, I sent a tweet to him but it was ignored... :(
Em 31 de out de 2017 07:20, "Murat Çorlu"<EMAIL_ADDRESS>escreveu:

I already tried that but nobody cares for now :( #115
https://github.com/tylertate/semantic.gs/pull/115
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tylertate/semantic.gs/pull/116#issuecomment-340719394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbYB1yMbKbqTU-VBIi5Lgs_Gmkdjka84ks5sxvR2gaJpZM4QL6LL
.

Same here :( https://twitter.com/muratcorlu/status/906845106925383680


Make it build with ghc-9.10

Only needs a bump to the base dependency. Would appreciate a Hackage metadata edit as well.
With your ok, I can (as a Hackage Trustee) do the metadata edit for you.

@Gabriella439 Would be nice to get this bumped since Stackage Nightly is about to bump to GHC 9.10 and Servant depends on pipes-safe.

pipes-safe is also a dependency of libsystemd-journal which cannot be fixed until this is fixed.

I also published the matching revision to Hackage


✨ Add fedora support

@andreiborisov Friendly ping :)

@scaryrawr It seems that @andreiborisov is not online, so I want to merge.
Excuse me, could you rewrite the commit messages? (Not the title of the PR, but the message of all commits included in the PR).
Only rebase merge is allowed here.


Update INSTALLATION.md

Made some small change will enhance the quilty of reading

i have made changes

I don't see any new commit. Did I miss something by any chance?

i have made changes but its not reflecting in the same pr do not know why


dynamic shading fps boost

I feel like dynamic shading gives me more fps than no methane light map mode on my potato pc

It's probably because Dynamic Shading cuts even further back on lightmap access, but I don't see how much faster it could be. If you want to, you can try measuring your average FPS with/without Dynamic Shading.


No Achievements Badge

No Achievements Badge
https://github.com/stenowtf/Steam-Deck-Tweak-No-Achievements-Badge
A tweak to remove the Achievements Badge from the game page.
Checklist
Failure to complete this checklist or deleting boxes from the checklist will result in the closing of your pull request unless this is a theme update. Please write any comments regarding this checklist at the bottom of your pull request.
Check every box.

[x] I am the original author of this theme or have permission from the original author to make this pull request.
[x] All copyright of this theme's contents belong to the listed author or is cited in the repository linked above.
[x] This theme's target has been marked appropriately and only styles said target.
[x] This theme works properly on the latest versions of SteamOS for Steam Deck, decky-loader and SDH-CssLoader.
[x] This theme only uses * or !important if absolutely necessary.
[x] This theme is under 4MB in size and uses the least disk space possible.
[x] This theme's preview image does not include text unless it is necessary to describe changes that can be made.
[x] This theme is safe for work and does not contain any sexual, drug-related, or profane content.
[x] This theme prefixes any CSS variables with a unique identifier.

Check one box.

[x] I am not bundling a part of another theme with this theme to encourage mixing and matching themes.
[ ] Themes included with this theme are toggleable using a patch.

Check one box.

[ ] This is a keyboard theme applied to the default keyboard.
[ ] This is a system-wide theme applied to the default keyboard. The keyboard is toggleable using a patch.
[x] This theme does not target the keyboard.

Would you mind re-submitting this on the site? DeckThemes.com

Would you mind re-submitting this on the site? DeckThemes.com

Sure! Done → https://deckthemes.com/taskStatus/view?task=f53e9d90-221d-4cca-9443-7d205388a2dd


slot error

ValueError: Invalid to parse arguments in line ' slot{"is_self": True}'. Failed to decode parametersas a json object. Make sure the eventname is followed by a proper json object. Error: Expecting value: line 1 column 13 (char 12)

did you try slot{"is_self": true}?


Animated weather card

~~https://codepen.io/ste-vg/pen/Gqakbo~~
Better version: https://codepen.io/GooeyIdeas/pen/NRvoQv

hey @ciotlosm
Added the animated weather card, using dark sky instead of yweather (not sure if that's an issue?). Getting the following error
cardConfig is not defined

Looking at the source code for that card type, I can see the JS variable being referenced, but not set. Unless it is set from a parent?

hey @philhawthorne , the card is still in the works. As I mentioned on the show, this is currently being worked on.
It will probably be ready for use around next week as it needs a build system with the different libraries it needs.

Ahhhh right! Clearly I got too excited and want to use it straight away :)

Wow Looking awesome!
Waiting for any news :)

Yeah, this looks awesome. Can't wait!

This is going slower than expected because in order to make this work I'd either need a build system or external libraries. I want to keep custom cards limited to 1 file if possible, so already evaluated StencilJS. One solution is to go with webpack but I want to retain the ability for people to read the source as it helps with contributions and debugging.
I hope I get something working at start of September with an approach that is usable and easy to install for users.

so now we can use it or it still in demo?

@ezbe Unfortunately not. Work stagnated and probably will still go slow for a while until I sort my time schedule better.

I believe this fills the need: https://github.com/bramkragten/custom-ui/tree/master/weather-card


Improve configuration of Private DNS zone

In section Configure the Private DNS Zone it has a manual step of getting the private ip address assigned to the PostgreSQL.
The purpose of using the CLI is to automate this process in a scripts.

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

ID: ba229540-9878-d93e-7004-9a1c48200928
Version Independent ID: 4a5ec508-7e42-6483-a488-6fdcc14c3944
Content: Private Link - Azure CLI - Azure Database for PostgreSQL - Single server
Content Source: articles/postgresql/howto-configure-privatelink-cli.md
Service: postgresql
GitHub Login: @kummanish
Microsoft Alias: manishku

@ealkalai I have provided the content here for getting the private IP. The information is part of the network interface and can be fetched through the solution listed. Let me know if this helps.

My intention was to be able to create these private endpoints via an arm template. but having to do multiple references will not work. So for now i have managed to find the private ip, with powershell cmdlets. It would work similarly with the cli. However, due to the hardcoded array indices it could be prone to errors. Do you think we can add a feature request to get the assigned ip address directly from the endpoint?
$endpoint = (Get-AzPrivateEndpoint -name $privateEndpointName -ResourceGroupName $resourceGroupName)
$endpointNicName = (Get-AzResource -Id $endpoint.NetworkInterfaces[0].Id).Name
$endpointNic = Get-AzNetworkInterface -Name $endpointNicName -ResourceGroupName $resourceGroupName
$endpointIp = $endpointNic.IpConfigurations[0].PrivateIpAddress

@ealkalai Thank you for the additional detail. A good feature request and something to add to the following: Better support for automation of installation in ARM Template / PowerShell
We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


fix: add missing closing bracket when site does not have pages

Just closing the bracket properly.
On a new website that does not have any pages yet, it was causing an error:
❯ hugo server -v
Start building sites …
hugo v0.111.3+extended darwin/arm64 BuildDate=unknown
INFO 2023/05/13 17:42:47 syncing static files to /
INFO 2023/05/13 17:42:47 postcss: use config file <path_to_project>/themes/gruvbox/postcss.config.js
INFO 2023/05/13 17:42:47 postcss: use config file <path_to_project>/themes/gruvbox/postcss.config.js
Error: Error building site: JSBUILD: failed to transform "js/flexsearch.js" (text/javascript): "<path_to_project>/themes/gruvbox/assets/js/flexsearch.js:75:2": Unexpected ";"
Built in 603 ms

Good catch, thanks for the PR!


feat: use nedb uid function for ids

Rather than generating non web safe ids as described in issue #7 we are now using the nedb uid function.
I'm super flexible to how we do this so feel free to request changes.

Looks good! I will publish these changes soon.


MEXC - Unable to select SHIB/USDT as spot trading pair when using AMM Arbitrage strategy

Describe the bug
When the AMM arbitrage strategy is selected and MEXC is one of the exchanges selected, SHIB/USDT is not listed as a spot trading pair.

Steps to reproduce

After starting and connecting, choose strategy with "create">"amm_arb">"mexc"
When the option to select token pair comes out, the list does not include SHIB/USDT

Release version
1.22.0
Type of installation
Source
Attach required files
logs_hummingbot.log
Config.log

@nikspz thank you very much - I missed this. Appreciate you looking out.


Allow deserialization into empty lists from NULL

We should allow deserialization into empty lists from CBOR NULL to avoid the need for the weird Option<Vec<_>> in these cases.

compatibility with go idiosyncrasies 😣


[improve][broker] Recover susbcription creation on the broken schema ledger topic

Motivation
Right now, broker failed to create schema-consumer if the topic's schema ledger is already deleted and broker is not able to recover that ledger. In that case, client will keep receiving below error message and will never be able to consumer on that topic.
2024-04-09T23:17:00,140 - INFO  - [pulsar-client-io-35-3:ConsumerImpl] - [persistent://my-prop/ns1/t1][test] Subscribing to topic on cnx [id: 0x59ed6366, L:/<IP_ADDRESS>:60336 - R:localhost/<IP_ADDRESS>:60330], consumerId 1
2024-04-09T23:17:00,141 - INFO  - [pulsar-io-8-1:ServerCnx] - [[id: 0x92bca72e, L:/<IP_ADDRESS>:60330 - R:/<IP_ADDRESS>:60336]] Subscribing on topic persistent://my-prop/ns1/t1 / test. consumerId: 1
2024-04-09T23:17:00,142 - WARN  - [PulsarTestContext-executor-OrderedExecutor-0-0:ServerCnx] - [/<IP_ADDRESS>:60336][persistent://my-prop/ns1/t1][test] Failed to create consumer: consumerId=1, No such ledger exists on Bookies -  ledger=6 - operation=Failed to open ledger
2024-04-09T23:17:00,143 - WARN  - [pulsar-client-io-35-3:ClientCnx] - [id: 0x59ed6366, L:/<IP_ADDRESS>:60336 - R:localhost/<IP_ADDRESS>:60330] Received error from server: No such ledger exists on Bookies -  ledger=6 - operation=Failed to open ledger
2024-04-09T23:17:00,143 - WARN  - [pulsar-client-io-35-3:ConsumerImpl] - [persistent://my-prop/ns1/t1][test] Failed to subscribe to topic on localhost/<IP_ADDRESS>:60330
2024-04-09T23:17:00,143 - WARN  - [pulsar-client-io-35-3:ConnectionHandler] - [persistent://my-prop/ns1/t1] [test] Error connecting to broker: org.apache.pulsar.client.api.PulsarClientException: {"errorMsg":"No such ledger exists on Bookies -  ledger=6 - operation=Failed to open ledger","reqId":1815857183629359419, "remote":"localhost/<IP_ADDRESS>:60330", "local":"/<IP_ADDRESS>:60336"}
2024-04-09T23:17:00,143 - WARN  - [pulsar-client-io-35-3:ConnectionHandler] - [persistent://my-prop/ns1/t1] [test] Could not get connection to broker: org.apache.pulsar.client.api.PulsarClientException: {"errorMsg":"No such ledger exists on Bookies -  ledger=6 - operation=Failed to open ledger","reqId":1815857183629359419, "remote":"localhost/<IP_ADDRESS>:60330", "local":"/<IP_ADDRESS>:60336"} -- Will try again in 1.583 s
2024-04-09T23:17:01,727 - INFO  - [pulsar-timer-84-1:ConnectionHandler] - [persistent://my-prop/ns1/t1] [test] Reconnecting after connection was closed

Modifications
Schema should be recovered if schema ledger is failing to open due to non-recoverable ledger error.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

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

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@rdhabalia @lhotari @Technoboy-
This pr may cause some problems, please look at the third part of this test. For example, we want to evolve schema from PersonTwo to PersonOne. Data with schema PersonTwo cannot be consumed. And only the schema PersonOne is version 1.
    /**
     * This test validates that consumer/producers should recover on topic whose
     * schema ledgers are not able to open due to non-recoverable error.
     *
     * @throws Exception
     */
    @Test
    public void testDeletedSchemaLedgerRecovery() throws Exception {
        final String tenant = PUBLIC_TENANT;
        final String namespace = "test-namespace-" + randomName(16);
        final String topicOne = "test-multi-version-schema-one";
        final String subName = "test";
        final String topicName = TopicName.get(TopicDomain.persistent.value(), tenant, namespace, topicOne).toString();

        admin.namespaces().createNamespace(tenant + "/" + namespace, Sets.newHashSet(CLUSTER_NAME));

        // (1) create schema
        Producer<Schemas.PersonTwo> producer = pulsarClient
                .newProducer(Schema.AVRO(SchemaDefinition.<Schemas.PersonTwo> builder().withAlwaysAllowNull(false)
                        .withSupportSchemaVersioning(true).withPojo(Schemas.PersonTwo.class).build()))
                .topic(topicName).create();

        Schemas.PersonTwo personTwo = new Schemas.PersonTwo();
        personTwo.setId(1);
        personTwo.setName("Pulsar");

        Consumer<Schemas.PersonTwo> consumer = pulsarClient
                .newConsumer(Schema.AVRO(SchemaDefinition.<Schemas.PersonTwo> builder().withAlwaysAllowNull(false)
                        .withSupportSchemaVersioning(true).withPojo(Schemas.PersonTwo.class).build()))
                .subscriptionName(subName).subscriptionType(SubscriptionType.Shared).topic(topicName).subscribe();

        producer.send(personTwo);
        // producer.close();
        // consumer.close();

        // (2) Delete schema ledger
        MetadataCache<SchemaStorageFormat.SchemaLocator> locatorEntryCache = pulsar.getLocalMetadataStore()
                .getMetadataCache(new MetadataSerde<SchemaStorageFormat.SchemaLocator>() {
                    @Override
                    public byte[] serialize(String path, SchemaStorageFormat.SchemaLocator value) {
                        return value.toByteArray();
                    }

                    @Override
                    public SchemaStorageFormat.SchemaLocator deserialize(String path, byte[] content, Stat stat)
                            throws IOException {
                        return SchemaStorageFormat.SchemaLocator.parseFrom(content);
                    }
                });
        String path = "/schemas/public/" + namespace + "/test-multi-version-schema-one";
        SchemaLocator schema = locatorEntryCache.get(path).get().get();
        long ledgerId = schema.getInfo().getPosition().getLedgerId();
        pulsar.getBookKeeperClient().deleteLedger(ledgerId);

        // (3) Topic should recover from deleted schema and should allow to create consumer and producer?????????
        Consumer<Schemas.PersonOne> consumer1 = pulsarClient
                .newConsumer(Schema.AVRO(SchemaDefinition.<Schemas.PersonOne> builder().withAlwaysAllowNull(false)
                        .withSupportSchemaVersioning(true).withPojo(Schemas.PersonOne.class).build()))
                .subscriptionName(subName).subscriptionType(SubscriptionType.Shared).topic(topicName).subscribe();

        Producer<Schemas.PersonOne> producer1 = pulsarClient
                .newProducer(Schema.AVRO(SchemaDefinition.<Schemas.PersonOne> builder().withAlwaysAllowNull(false)
                        .withSupportSchemaVersioning(true).withPojo(Schemas.PersonOne.class).build()))
                .topic(topicName).create();
        assertNotNull(consumer1);
        assertNotNull(producer1);

        Schemas.PersonOne personOne = new Schemas.PersonOne();
        personOne.setId(1);

        producer1.send(personOne);
        Assert.assertEquals(personOne, consumer1.receive().getValue());

        // Old "version" consumer cannot be consumed.
        producer.send(personTwo);
        Assert.assertEquals(personTwo, consumer.receive().getValue());

        consumer.close();
        producer.close();
        consumer1.close();
        producer1.close();
    }

Compared with #18010, there is no configuration control. The default behavior is to automatically skip when the integrity of the schema is destroyed.
Even after this error occurs, whether to skip or not, there should be a configuration that allows users to configure whether to skip or manually repair when the integrity of the schema is damaged.
Of course, the function of manually repairing lost schema does not exist at present. However, I have a pr trying to repair the missing schema.(https://github.com/apache/pulsar/pull/20415） In the future, I will send an email to discuss how to deal with this lost schema.
The current processing of pr does not seem to be a good way.

Even after this error occurs, whether to skip or not, there should be a configuration that allows users to configure whether to skip or manually repair when the integrity of the schema is damaged.

If schema ledger itself is deleted then it means it's similar as having topic without schema, and in that case, we should not make the topic unavailable. This PR simply handles non-recoverable error instead eating and ignoring the exception, and not allowing client to connect. yes, I also agree, we should allow client to connect and create a new schema ledger if needed.

This pr may cause some problems, please look at the third part of this test. For example, we want to evolve schema from PersonTwo to PersonOne. Data with schema PersonTwo cannot be consumed. And only the schema PersonOne is version 1

It's because schema ledger is already deleted and in that case, topic doesn't have schema means it should allow to create a new version. Schema deletion means data loss and you can not do much to avoid such situation other than allowing to add a new schema and make topic available to the client applications.


Use bool for predicates

Signed-off-by: Davide Bettio<EMAIL_ADDRESS>These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Also term_is_number (can't comment on the line)

This changes make sense from a theoretical point of view, from an empirical point of view they might slow down the generated binary.


Incorrect TOC for page 'Logging (not ASP.NET Core)'

Description
In https://github.com/dotnet/AspNetCore.Docs/pull/28726 the page "Logging (not ASP.NET Core)" has been moved to "Advanced":

In the TOC:
https://github.com/dotnet/AspNetCore.Docs/blob/b7ae8c247e6db5918138541cd7287f1c033480fd/aspnetcore/toc.yml#L1777-L1779
I think this has been a mistake, because this page is about the migration from 2.1 to 2.2 or 3.0. Doesn't this belong under "Migration and Updates"?
Page URL
https://learn.microsoft.com/en-us/aspnet/core/migration/logging-nonaspnetcore?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/migration/Logging-nonaspnetcore.md
Document ID
dbcd23c0-3bca-e480-936b-2cf6f2d62c7e
Article author
@Rick-Anderson

Associated WorkItem - 192423

Yes, you're right, I'll move it.


appdynamics collector

Hi,
is appdynamics collector ready to use or still work in progress? I see "paychex" has started working on that but I cloudn't find enough details about it.
Thanks,
Anjana

Paychex is not working on it. I guess CapitalOne is according to this: https://github.com/capitalone/Hygieia/issues/697

Yes. We are. We have some working version ready. Need some brush up before it goes out.

Any update on this? Would love to integrate an AppD collector into our environment.

@nireeshT @satishc1 showed me the work today - looks like PR will be merged next week?

@tabladrum  anyone from your team will be able to help me with my issue on appD and hygieia integration? i have raised the issue. https://github.com/Hygieia/hygieia-perf-appdynamics-collector/issues/2


[Slider] Fix a11y issues with the handle

Closes #14456

slider role on MDN
WAI-ARIA 1.0: slider: Not explicit about the element that should hold the role
WAI-ARIA Authoring practices: slider: Implement the documented roles
material web components: Match our previous incorrect implementation

Ant Design is using the same approach 👍.


Update clarinet.md

Re-worded sentences for clarity
Added bulleted list

Description
Describe the changes that were made in this pull request. When possible start with the motivations behind the change. Be sure to also include the following information:

Motivation for change?
What was changed?
Link to relevant issues?

Checklist

[ ] Conventional commits were used
[ ] New links to files and images were verified
[ ] For fixes/refactors: all existing references were identified and replaced
[ ] Style guide was reviewed and applied
[ ] Clear code samples were provided
[ ] People were tagged for review

@sabbyanandan: Please review my changes. Thanks.

@sabbyanandan: Can I go ahead and merge these changes?


Search for "captive=false" no longer working

For at least the last 12 hours searches trying to exclude captive/cultivated observations don't work - exactly the same observations are returned when selecting "any" or "no" for the "Captive / cultivated" option.  Selecting the "yes" button does appear to function though.  Here is an example search that should return no observations (as all Ostriches in NZ are captive/cultivated and all observations have been marked as such):
http://naturewatch.org.nz/observations?utf8=✓&q=&search_on=&quality_grade=any&captive=false&place_id=&swlat=-55.956&swlng=160.6&nelat=-25.901&nelng=-171.25&taxon_name=Struthionidae&taxon_id=20523&day=&month=&year=&order_by=observations.id&order=desc&rank=&hrank=&lrank=&site=&tdate=&d1=&d2=&filters_open=true&view=table
I have tested this with a variety of searches through both iNaturalist.org and the naturewatch.org.nz domain.
--Tony

I also note that if you select the "any" radio button, it is shown as cleared again when the results are displayed (the other 'any' radio buttons on the search page don't do this)

Thanks for the report Tony. I just committed a fix for this bug, which will be released later today. I'll let you know when the fix is out so you can test for yourself and let us know if everything is working as you'd expect.

Thanks Patrick, just tested it and seems to be working great.


Export packing outputs as a tiff file

Problem
See #165
Solution

The multi-channel image is represented as a numpy array with dimensions C x X x Y x Z. Each channel represents a single type of packed ingredient or compartment.
For single_sphere ingredients, the ImageWriter class creates a mask based on the position and radius of the sphere, and sets all pixels within this mask to 255. Other ingredient types are currently unsupported.
For compartments represented by meshes, the ImageWriter class creates a hollow or filled mask based on the provided options, and sets all pixels within this mask to 255.
We can also convolve the image with a point-spread function as needed. Currently available options are "gaussian" and "box".
We use the aicsimagio package to export the packing image as an ome.tiff file. The camera is set looking down the -z axis similar to a microscopy assay.

with @meganrm
Type of change

New feature (non-breaking change which adds functionality)

Change summary:

added the ImageWriter class to export packing outputs as tiff images
added options in the packing config to edit the output image properties, including:

the voxel size (in units of grid spacing) OR image size (voxels in each dimension)
hollow or filled mesh representation

Steps to Verify:

conda activate autopack
pack -r cellpack/tests/recipes/v2/test_image_export.json -c cellpack/tests/packing-configs/test_image_config.json

Example packing of peroxisomes in the average nuclear and cell shape:
pack -r examples/recipes/v2/peroxisomes.json -c cellpack/tests/packing-configs/test_image_config.json
Screenshots:

Keyfiles:

cellpack/autopack/writers/ImageWriter.py

Codecov Report
Patch and project coverage have no change.

Comparison is base (e349ce6) 98.72% compared to head (7ce7e19) 98.72%.

:exclamation: Current head 7ce7e19 differs from pull request most recent head d1b3016. Consider uploading reports for the commit d1b3016 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #166   +/-   ##
=======================================
  Coverage   98.72%   98.72%
=======================================
  Files          10       10
  Lines         236      236
=======================================
  Hits          233      233
  Misses          3        3

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

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

[ ] Include a setting to create hollow masks for spheres.
[ ] Check with matheus about creating peroxisome meshes

It's hard to be sure, but looking at the screenshots of the magenta spheres, it seems like the image is flipped

It's hard to be sure, but looking at the screenshots of the magenta spheres, it seems like the image is flipped

Yeah, I changed the dimension order in a later commit. Let me update the screenshots


[EMB-16] Allow user-nodes to be sorted

Purpose
User nodes endpoint need to be able to be sorted.
Changes
Remove hardcoding of user-node sort endpoint and add tests
QA Notes
Attempt sorting of the user-nodes endpoint.

@hmoco Please fix the flake errors.


New Users Do Not Show in Activity Filters

Describe the bug
New users do not show up in the 'Activity Filters' section. Since I added the extension we have added multiple new staff members but when typing their names they do not show up (ie the auto-complete suggestions). I can not type their names either and get their contributions to show.
Environment:

Chrome
Azure DevOps

To Reproduce
Steps to reproduce the behavior:

Install the extension
Add a new user to the organization
Go to Activity Filters and try and type their name
Name does not show up

Expected behavior
Typing the person's name shows their suggested name and can choose the user to view their contributions.

I'm exactly having the same issue. What can I do that other team members appear?
Most of them are registered 9 month, but are not shown.


Refactoring of Mailgun service & Added constants for headers

It's gonna throw an exception earlier then in a method when one or more params of environment variables are empty.
Created constants of headers & urls

Codecov Report

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

@@            Coverage Diff             @@
##             develop     #162   +/-   ##
==========================================
  Coverage           ?   58.16%
==========================================
  Files              ?      199
  Lines              ?     2907
  Branches           ?      122
==========================================
  Hits               ?     1691
  Misses             ?     1192
  Partials           ?       24

Flag
Coverage Δ

web
58.16% <0.00%> (?)

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

Continue to review full report at Codecov.

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


Please add Howard County, MD (to HCPSS and Howard County Board of Education)

(to HCPSS and Howard County Board of Education)
TO<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: [*** INSERT UNIQUE SUBJECT LINE***]
Message (Don't forget to replace the [x]'s with your information!)
Dear Howard County Education Officials,
My name is [YOUR NAME], and I am a resident of Howard County, Maryland.
I write to you as people across the world take a public stand against policing and systemic racism. As demonstrated by the largest protest in Howard County history, the people of our community are also demanding change. It is critical that every elected official condemns police violence and affirms that Black Lives Matter.
But speaking out is not enough, we also demand action.
We demand that Student Resource Officers be removed from Howard County Schools and that those funds go to counselors and other school supports that actually create safer schools. The presence of police in our schools leads to higher rates of suspensions, expulsions and arrests, especially for Black students. Police in schools do not increase safety, but do aid the school-to-prison pipeline that pushes children, especially Black children, into the criminal legal system. Instead of putting our students at risk every day they go to school, we need to invest in proven safety solutions.
As the rest of the country reckons with the inherent injustice of police, Howard County must do the same. Our demands are clear: fund counselors, defund police.
Sincerely,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

Hi @egfrank! Thanks for your interest in adding Howard County, MD to the site. I have a few suggestions to make your message even more compelling & fact-based so that representatives have no grounds to ignore it.

Could you edit the email to provide more detail about the city budget to make a more compelling argument, showing the disparities in numbers? To get your submission published quicker, please provide sources for the data as well.
Could you list the date and location of the protest you mention in the first paragraph?

Please see our email style guide for more info.
Thank you!


AttributeError: module '_sha3' has no attribute 'sha3'

Version: 1.6.0
Python: 2.7/3.4/3.5
OS: osx/linux/win

What was wrong?
running tox pass the tests fine
but running them individually yields this error on some
I guess it's my local install since tox is fime, but couldnt find the root of it
➜  web3.py git:(master) ✗ sudo pip3 uninstall pysha3
Uninstalling pysha3-0.3:
  /usr/local/lib/python3.5/dist-packages/pysha3-0.3-py3.5-linux-x86_64.egg

pysha3 0.3 is install, seems to be the right version
─
➜  web3.py git:(master) ✗ py.test-3 tests/filtering/test_contract_on_event_filtering.py
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/_pytest/config.py", line 283, in _getconftestmodules
    return self._path2confmods[path]
KeyError: local('/home/lotsos/PycharmProjects/web3.py/tests/filtering/test_contract_on_event_filtering.py')

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/_pytest/config.py", line 283, in _getconftestmodules
    return self._path2confmods[path]
KeyError: local('/home/lotsos/PycharmProjects/web3.py/tests/filtering')

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/_pytest/config.py", line 314, in _importconftest
    return self._conftestpath2mod[conftestpath]
KeyError: local('/home/lotsos/PycharmProjects/web3.py/tests/filtering/conftest.py')

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/_pytest/config.py", line 320, in _importconftest
    mod = conftestpath.pyimport()
  File "/usr/local/lib/python3.5/dist-packages/py/_path/local.py", line 650, in pyimport
    __import__(modname)
  File "/home/lotsos/PycharmProjects/web3.py/tests/filtering/conftest.py", line 6, in <module>
    assert sha3_256(b'').hexdigest() == 'c5d2460186f7233c927e7db2dcc703c0e500b653ca82273b7bfad8045d85a470'
  File "/usr/local/lib/python3.5/dist-packages/sha3/__init__.py", line 97, in sha3_256
    return SHA3256(s)
  File "/usr/local/lib/python3.5/dist-packages/sha3/__init__.py", line 30, in __init__
    self._s = _sha3.sha3()
AttributeError: module '_sha3' has no attribute 'sha3'
ERROR: could not load /home/lotsos/PycharmProjects/web3.py/tests/filtering/conftest.py

Cute Animal Picture

put a cute animal picture here.

fixed by uninstalling the wrong lib, gave me headaches
➜  web3.py git:(master) ✗ sudo pip3 uninstall sha3
[sudo] password for lotsos:
Uninstalling sha3-0.2.1:
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/DESCRIPTION.rst
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/INSTALLER
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/METADATA
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/RECORD
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/WHEEL
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/metadata.json
  /usr/local/lib/python3.5/dist-packages/sha3-0.2.1.dist-info/top_level.txt
  /usr/local/lib/python3.5/dist-packages/sha3/__init__.py
  /usr/local/lib/python3.5/dist-packages/sha3/__pycache__/__init__.cpython-35.pyc
Proceed (y/n)? y
  Successfully uninstalled sha3-0.2.1


Please support Fusionjs integration for React components

It looks like the current support for React elements is limited to CRA projects.
Would it be possible to add support for the new CRA alternative https://fusionjs.com/ ?

Hi folks, Fusion.js core developer here. I'd be happy to work with folks to figure out the best way to get storybook & Fusion.js interop working well. Happy to jump on a call if helpful.

Join our slack. I think @ndelangen will be happy to onboard new people =)
Also we are working on the presets ecosystem for storybook, so it will also be possible as a preset in the feature.

@KevinGrandon Sorry for me jumping on this a bit late.
Let's schedule a meeting shall we?
When would be a good time for you?

I'd be happy to, but it sounds like waiting for the presets might be the best way to go here? Any documentation on the plan for those?

When @igor-dv is back from vacation I'm sure he'll add some docs about it.
We'll gradually be opening the presets feature and documenting it better.
To the point that once we have the unified config we'll just have a list of add presets configured.

Sounds good, schedule is just a bit crazy these days, but will see if we can schedule some time to get this done!


listener support Unix Socket

The New Feature
Describe the new feature you want to support clearly.
Your scenes
Describe your use scenes (why need this feature).
Your advice
Describe the advice or solution of this new feature.
Environment

MOSN Version

refer #1336


ci: mirror 'dev' --> 'beta'

Try out this version of the Hiro Wallet - download extension builds.

CI job to mirror dev to beta.
Hoping to do dev releases from beta, so that dev remains void of release commits. See commentary here https://softwareengineering.stackexchange.com/questions/377374/using-gitflow-and-semantic-versioning-how-to-avoid-version-number-conflicts-whe

Note that this only does the mirroring, but from here can set up the next steps.


How to authenticate against Kerberos enabled cluster?

I am trying getting a token successfully
` public string authenticate(string url)
{
string op_literal = "?op=";
string operation = "GETDELEGATIONTOKEN";
string domain = ReadDomain();
string user = ReadUsername();
SecureString pass = ReadPassword();
HttpWebRequest request;
HttpWebResponse response;
//First get the delegation token from the URL
request = (HttpWebRequest)WebRequest.Create(url + op_literal + operation);
     CredentialCache cc = new CredentialCache();
     cc.Add(new Uri(url + op_literal + operation), "Negotiate", (
                 (domain == null || domain == "") ?
                 new System.Net.NetworkCredential(user, pass) :
                 new System.Net.NetworkCredential(user, pass, domain)));

     request.Credentials = cc;
     request.CookieContainer = new CookieContainer();

     response = (HttpWebResponse)request.GetResponse();
     // Display the status.
     Console.WriteLine(((HttpWebResponse)response).StatusDescription);
     // Get the stream containing content returned by the server.
     Stream dataStream = response.GetResponseStream();
     // Open the stream using a StreamReader for easy access.
     StreamReader reader = new StreamReader(dataStream);
     // Read the content.
     string responseFromServer = reader.ReadToEnd();
     // Display the content.
     Console.WriteLine(responseFromServer);
     // Clean up the streams and the response.
     reader.Close();
     response.Close();
     return responseFromServer;
 }`

As second step I am passing this token in the URL after ?delegation=

Is this the right way? I noticed Kerberos support is not available in the master

If this is working then shouldn't have a problem.

Somehow in the second step passing the delegation token in the URL gives me a bad request error. Passing the delegation token is done in GET request, whereas the Upload file happens through PUT.  I can not combine delegation and op=CREATE in one request for example.

So back up a step here, I think you should be able to pass NetworkCredential directly to the constructor for WebHDFS client. Your example code doesn't use the library created here.
Are you using the webhdfs-dotnet library to interact? Kerberos support should be there there just aren't any tests.

I have cleaned up my code and just used the library webhdfs-dotnet. I still ended up getting the error
"Should get a 307. Instead we got: Unauthorized Unauthorized"
I have set this as well in the code(commented out Kerberos, to force it to be used)
`          AuthenticationManager.Unregister("Basic");
        AuthenticationManager.Unregister("NTLM");

        AuthenticationManager.Unregister("Negotiate");

        AuthenticationManager.Unregister("Digest");

        //AuthenticationManager.Unregister("Kerberos");`

hmm interesting. Not sure how the custom handler is different than what the code is doing currently: https://github.com/risdenk/webhdfs-dotnet/blob/master/WebHDFS/WebHDFSClient.cs#L37
Thanks for the followup and glad it is working for you :)


[SR-15280] Remove DiagnosticsEngine and report diagnostics via a callback closure

Previous ID
SR-15280

Radar
rdar://83850489

Original Reporter
@ahoppen

Type
Bug

Additional Detail from JIRA

Votes
0

Component/s
SwiftSyntax

Labels
Bug

Assignee
None

Priority
Medium

md5: 5671cec3472131d2317036b28cf2d9f7

Issue Description:
As discussed here (https://github.com/apple/swift-format/pull/268#issuecomment-933617735):

maybe the whole concept of a diagnostic engine/consumers shouldn't live in SwiftSyntax/SwiftSyntaxParser at all? If the parser is the only user of it, it could just take a callback function instead of a diagnostic engine reference, and pass its own lightweight Diagnostic value containing information relevant to a parse issue. Then, clients of SwiftSyntaxParser would translate those into whatever diagnostic engine they wanted to use (for example, swift-format could start using TSC instead).

Implemented in https://github.com/apple/swift-syntax/pull/325


NSFS | NC | CLI | List config files performance improvement

Explain the changes

Check wide before reading the config files will read the config files only when needed.

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

[ ] Doc added/updated
[ ] Tests added

Closing this PR since NC tests can not pass without the filter changes, these PR changes will be available on https://github.com/noobaa/noobaa-core/pull/7745


user defined metadata - spatial

Demonstration of user customised metadata from XIOS
This includes spatial coordinate reference system metadata definitions.

also
fixes #35


read config files from directory

Codecov Report

Merging #77 (b6d0513) into main (35ca869) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #77   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        11    +1
  Lines          316       345   +29
  Branches        40        46    +6
=========================================
+ Hits           316       345   +29

Impacted Files
Coverage Δ

src/rezide/utils/config_dir.py
100.00% <100.00%> (ø)

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 35ca869...b6d0513. Read the comment docs.


Standardize behavior of optionals + defaults

1. Default values are now implemented in separate ZodDefault class.
Previously this logic was folded into ZodOptional.
2. Standardized behavior of .optional() and .nullable() methods.
Previously calling .optional() on an instance of ZodOptional instance would return the current instance instead of redundantly nesting ZodOptional instances. This seems entirely unnecessary, feels too magical, and introduced complexity into the typings. This complexity makes it harder to write generic functions on top of Zod: https://github.com/colinhacks/zod/issues/355#issuecomment-805130057
3. No undefined defaults
// before this was OK
z.string().optional().default(undefined)

// now it's not
z.string().optional().default('asdf'); // must be string

There's no point in setting undefined value as a  default value, it's a NOOP. This also lets us explicitly eliminate optionals from the inferred output types of ZodDefault:
export class ZodDefault<T extends ZodTypeAny> extends ZodType<
  util.noUndefined<T["_output"]>,
  ZodDefaultDef<T>,
  T["_input"] | undefined
> {
  // ...
}

4. Calls to .optional always return schema with T | undefined
This is a consequence of #2.  Previously calling .optional() on z.string().default() would return the instance unchanged, since defaults values were implemented in the ZodOptional class. Now it introduces a new ZodOptional instance that wraps a ZodDefault instance. As such the type signature is now string | undefined instead of string.
This introduces a nice duality into chained calls to default and optional.
z.string().default('asdf'); // string
z.string().default('asdf').optional(); // string | undefined
z.string().default('asdf').optional().default('asdf'); // string

5. Introduce .nullish()
// equivalent
z.string().nullish()
z.string().optional().nullable()

Default values are now implemented in separate ZodDefault class.

No opinion

Standardized behavior of .optional() and .nullable() methods.

No opinion

No undefined defaults

Seems good as long as you don't have to pass a default() to optional() (but seems like you don't considering 4.)
Are you doing a ?? defaultValue in the code, i.e z.string().optional().nullable().default('x').parse(null) return x? I think it should if not.
Can this also .default() used on complex objects too?

Calls to .optional always return schema with T | undefined

Nice.

Currently defaults are only applied when the schema receives an undefined value. I'd be opening to adding a new subclass (NullishDefault?) or add flag inside ZodDefault to support ??-like behavior.
Regarding the base-class convenience method, this could be introduced as a new method (nullishDefault()?) or as a param on default (.default('asdf', {nullish:true})). Probably gonna do the new method though because I don't like params in general.

@colinhacks If a new method will be added (nullishDefault), then it will be great if a null-only default is provided as well, just to be consistent, and also since T.nullable().default() output type is T | null instead of just T (related #425)

@colinhacks +1 for a nullishDefault.
I'm running into some impedance mismatches between Zod and React Hook Forms with regards to undefined / null handling.
RHF discourages the use of undefined for field states and reserves its use internally - explicitly setting a field to undefined will reset its state back to the form default.
Setting the field as null is the 'proper' way, but that means changing the schema from optional() to nullish(), effectively obviating .default() and causing consumers of the form to receive number | null type.


eos_cli_config_gen - Add LLDP documentation section

Enhancement summary
Add LLDP documentation templates
Which component of AVD is impacted
eos_cli_config_gen
Use case example
Currently we have no documentation templates covering LLDP
Describe the solution you would like
Add LLDP documentation templates covering both global config and interfaces where lldp knobs have been set.
Describe alternatives you have considered
No response
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Can I try this one?

Since the table is under a dedicated LLDP section we don't need LLDP in every column title. Try to keep it short a close to the variable name.
Please write the data model out, so it is easier to understand what you mean.
Also some cli examples would be great to compare.

Suggested minor change (compatible to what we have today):
lldp:
  timer: < transmission_time >
  timer_reinitialization: < re-init_time >
  holdtime: < hold_time_period >
  management_address: < all | ethernetN | loopbackN | managementN | port-channelN | vlanN >
  vrf: < vrf_name >
  receive_packet_tagged_drop: < true | false >
  run: < true | false >

Suggested major change for future versions:
lldp:
  timers:
    lldp_timer: < transmission_time >
    lldp_reinitialization_timer: < re-init_time >
  holdtime: < hold_time_period >
  vrf: < vrf_name >
  management_address:
    - < all | ethernetN | loopbackN | managementN | port-channelN | vlanN >
    - < all | ethernetN | loopbackN | managementN | port-channelN | vlanN >
  receive_packet_tagged_drop: < true | false >
  tlv_transmit:
    link-aggregation: < true | false >
    management-address: < true | false >
    max-frame-size: < true | false >
    med: < true | false >
    port-description: < true | false >
    port-vlan: < true | false >
    power-via-mdi: < true | false >
    system-capabilities: < true | false >
    system-description: < true | false >
    system-name: < true | false >
  run: < true | false >

If no vrf is defined, the default vrf will be used.
We can define multiple IPs per VRF to be included in the LLDPDU, current model doesnt support this.
If only vrf an not specific interface is defined,  "all" should be used ( advertising all ips in the vrf)
TLV_transmit allows us to exclude certain values from the LLDPDU if desired.
=============================
Regarding the current LLDP config per interface.:
According to README, we have:
# Routed Interfaces
ethernet_interfaces:
  <snip>
   lldp:
      transmit: < true | false >
      receive: < true | false >

But I think this should be under "#Switched interfaces" instead, as LLDP is a L2 Protcol. I'll move this?

When I check on EOS 4.26.1F I cannot enter more than one management address (or all).
VRF is set in EOS with lldp management-address vrf ... so if we change the model at some point, it should reflect that.
management-address also supports ipv6 keyword, so it should probably be a dict with interface, vrf, ipv6 keys.
tlv_transmit could be added now without any breaking changes to existing model.
I would not go with your suggestion of adding a timers dictionary. While it makes sense to group these, we try to reflect EOS CLI as closely as possible in eos_cli_config_gen.

For this issue, the main objective is just to add a documentation section for LLDP. It is fine if you add some extra knobs and options under lldp while you work on it, but no breaking changes.

 I'm not sure what I saw yesterday.. however what EOS allows you is to specify a VRF as the source + an Interface/IP that is not on that vrf. e.g.:

sh run | i lldp
lldp management-address Loopback0
lldp management-address vrf management

Is possible, even though Loopback0 is not in vrf management. In this case "lldp management-address Loopback0" is just ignored. Maybe this behaviour has changed in newer EOS versions though. Could you check on 4.26? Maybe this could be reported as something to be fixed in EOS (if its still present)

TLV Transmit: True, I can add this to the code
Regarding the timers: I was thinking wrong here. I interpreted "lldp timer" as a "section" but instead you see "lldp timer timeInS" and "lldp timer reinitialization timeInS" as independent commands

(config)#lldp timer ?
  reinitialization  Configure the delay before LLDP re-initialization is attempted
  <1-32768>         Configure the period at which we transmit LLDPDUs (in seconds)

========================
Minor model change then:
lldp:
  timer: < transmission_time >
  timer_reinitialization: < re-init_time >
  holdtime: < hold_time_period >
  management_address: < all | ethernetN | loopbackN | managementN | port-channelN | vlanN >
  vrf: < vrf_name >
  receive_packet_tagged_drop: < true | false >
  tlv_transmit:
    link-aggregation: < true | false >
    management-address: < true | false >
    max-frame-size: < true | false >
    med: < true | false >
    port-description: < true | false >
    port-vlan: < true | false >
    power-via-mdi: < true | false >
    system-capabilities: < true | false >
    system-description: < true | false >
    system-name: < true | false >
  run: < true | false >

Major model change:
lldp:
  run: < true | false >
  timer: < transmission_time >
  timer_reinitialization: < re-init_time >
  holdtime: < hold_time_period >
  receive_packet_tagged_drop: < true | false >
  tlv_transmit:
    link-aggregation: < true | false >
    management-address: < true | false >
    max-frame-size: < true | false >
    med: < true | false >
    port-description: < true | false >
    port-vlan: < true | false >
    power-via-mdi: < true | false >
    system-capabilities: < true | false >
    system-description: < true | false >
    system-name: < true | false >
  management_address:
    ipv4_interface: < all | ethernetN | loopbackN | managementN | port-channelN | vlanN >
    ipv6_interface: < all | ethernetN | loopbackN | managementN | port-channelN | vlanN >
    vrf: < vrfname >

Let's not worry about the major model change now. We have what we have now, and your additions will be good.
It seems like EOS accepts anything in vrf and interface, but I am not sure whether that is on purpose or not, so let's not address that in AVD. We will just configure whatever the user supplies.


Add TOYOTA Corolla Cross Hybrid Taiwan Fingerprint

stock route : 80e58d853114e409|2021-12-01--12-46-04
op route : 80e58d853114e409|2021-12-01--13-21-54
not have stop and go,

This would revert a lot of changes, can you delete this file and add your changes to selfdrive/car/toyota/values.py?

This would revert a lot of changes, can you delete this file and add your changes to selfdrive/car/toyota/values.py?

Sorry feedback too late,
I used 0.8.14 ver. update on my fork

All your FW versions are already in openpilot under the TSS2 CorollaH