MIAD-261 backend-managed bulk alerts session

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

Project coverage is 91.28%. Comparing base (ce12aab) to head (efdbb8c).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

...s/journeys/bulk-alerts/check-answers/controller.ts
85.00%
3 Missing :warning:

...urneys/bulk-alerts/upload-prisoner-list/schemas.ts
66.66%
3 Missing :warning:

server/utils/utils.ts
33.33%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
- Coverage   91.71%   91.28%   -0.43%
==========================================
  Files         109      109
  Lines        2268     2283      +15
  Branches      356      350       -6
==========================================
+ Hits         2080     2084       +4
- Misses        184      195      +11
  Partials        4        4

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


Allow single-used load/store to be rematerializable.

An expensive load was considered not rematerializable, as reloading data wasn't going to be profitable, iiuc. However, if the load has a single user, and somehow the user wants to stick with another layout, loading the data in that layout can be profitable. This manifested as a performance issue for the following-like kernel   (Q*K + B) * V, where the load of B in distributed layout was causing Q*K to go from mma layout to distributed and then back to dot operand layout. The overhead can be saved If B can be loaded in mma layout in the first place.

This may not be helpful at this point, but at the risk of over-communicating, I want to add that "single-use load" is probably never the right criterion to use for this sort of thing.
That's because we could have
    %x = load ptr
    %y = fmul %x, 2  // only use of x
    // now use %y multiple times

If there's some optimization you would not want to do if %x were used multiple times, you almost surely also do not want to do the optimization if %y is used multiple times.  Because the load is so much more expensive than the fmul, our analysis of the code can basically consider any uses of the fmul to be "morally equivalent" to uses of the load.

This may not be helpful at this point, but at the risk of over-communicating, I want to add that "single-use load" is probably never the right criterion to use for this sort of thing.
That's because we could have
    %x = load ptr
    %y = fmul %x, 2  // only use of x
    // now use %y multiple times

If there's some optimization you would not want to do if %x were used multiple times, you almost surely also do not want to do the optimization if %y is used multiple times. Because the load is so much more expensive than the fmul, our analysis of the code can basically consider any uses of the fmul to be "morally equivalent" to uses of the load.

I see. We definitely do not want to duplicate load when y is duplicated. I wonder if the current analysis support changing the load's layout in place instead of rematerializing it.

This may not be helpful at this point, but at the risk of over-communicating, I want to add that "single-use load" is probably never the right criterion to use for this sort of thing.
That's because we could have
    %x = load ptr
    %y = fmul %x, 2  // only use of x
    // now use %y multiple times

If there's some optimization you would not want to do if %x were used multiple times, you almost surely also do not want to do the optimization if %y is used multiple times. Because the load is so much more expensive than the fmul, our analysis of the code can basically consider any uses of the fmul to be "morally equivalent" to uses of the load.

I see. We definitely do not want to duplicate the load when y is duplicated. I wonder if the current analysis support changing the load's layout in place instead of rematerializing it. For example, if all nodes in the backward rematerialization chain have single use, than we can just change the layout in place?

I think the hasOneUse was to make sure we don't duplicate ops as we may rematerialize them multiple times which makes sense. However combine a load with single use + convert into a load in the new layout is not always better.

I wonder if the current analysis support changing the load's layout in place instead of rematerializing it. For example, if all nodes in the backward rematerialization chain have single use, than we can just change the layout in place?

Wouldn't the result be the same?

However combine a load with single use + convert into a load in the new layout is not always better.

Yeah I agree and I was thinking about a cost model, mostly from vectorization POV. I double checked our kernel, which is very specific. The B part of Q*K+B comes from a chain of computation and involves multiple indirect loads, which cannot be vectorized. So converting the loads to mma layout wouldn't lose any performance. I can think of a very naive cost model is to just check if the continuity of those loads is 1.

Wouldn't the result be the same?

Oh yeah, we can reply on cleanup passes to remove the original copies.

Sending out a new iteration to make the previous heuristics more robust. Two improvements:

Checking if the load and its transitive users can only used in the rematerializable slice, instead of checking if it only has one use.
Checking if the original load is vectorizable or not. If not vectorizable, rematerialzing it shouldn't be more expensive.


Feature Request: Support for Claude

Hello,
Claude 3.5 sonnet is one of the leading llms for coding related tasks.
as per https://lmarena.ai/
Would be cool to see its inclusion. I would be happy to work on this.

Go for it.


fix dto's validation and write error message's

based on latest issue for todo template

@MohammadJavadTabari
tnx a lot for your PR, but there are several things that are wrong with your changes.
please read this article first about contributing to our open source projects (do not create pull requests directly from your fork's develop branch):
https://www.dataschool.io/how-to-contribute-on-github/
and also we need to have a specific issue submitted for each PR. please review other PRs to grasp the general idea.

ok . so I create new branch on my fork and call it "editForms_validation"  . then I should  make changes . then push it on my fork . then create PR  in upstream  repo for "editForms_validation" . Am I right?

@MohammadJavadTabari
what is the issue you're working on? (what is its number?)
did you check other PRs and see what we need to have in a valid PR? (for example, how to name a branch, how the PR's title should be, what should be in a PR's description)

seems the issue is resolved by some one else .
yeah I see other PR's and find out what to do and how is naming a PR there and again yes I also see branch naming .
sorry for this mistake . I do my best in other issue's .

since this PR is not relevant anymore, I'm closing it.


Agents and agent clusters

It seems like this wants to exclude a substantial amount of host behavior, it would probably be good to be upfront about that.
I also wonder what that means for the theoretical model as a lot of these things a host might want to configure once for certain types of agents or even agent clusters. I suppose that doesn't matter all that much, but it might be worth thinking about and at least be explicit about it in the readme.

@annevk I do not think the goal of this proposal is to completely abstract all parts of hosts as things like ShadowRealms (I think that is the most current moniker [Realms prev.]) are taking on other parts of host abstraction. Is the goal here to try and state the scope of which host behaviors are being virtualized or more a concern that it doesn't include all possible host hook variations when we add that to the explainer.

I don't think all host behavior needs to be expressible per se, but I do think that should be acknowledged. I.e., clearly scope it a subset of host hooks that can affect the code executing in a realm.
I worry a little bit about whether the current layering of host behavior (in ECMAScript, not in this proposal) is optimal given agents and agent clusters, but I suspect that shouldn't matter too much for what is being proposed here, except for possible refactoring down the road.

Agreed. We need to be more specific about host hooks and host behaviors. Our general requirement is that all host behaviors that are observable from within a compartment need to have hooks, but very few host behaviors need to be observable. We plan to start with module loader hooks only. Every other host behavior should at least initially be obscured, like locale observability. That gives us everything we need all the way out to determinism and a clear story for how to inject other host behaviors explicitly going forward.

@annevk I got a reminder from @mhofman about how we intend to frame the Compartment proposal, which changes the texture of my last comment.
We intend to create a proposal for a feature called Lockdown that would interact with ShadowRealm, Compartment, Function, and eval.
While developing the ShadowRealm proposal, we learned that we needed to distinguish ShadowRealm from ShadowRealm after Lockdown. The proposal you know about is the former and we have not proposed the latter. But you can see a hint that we intend it because we stood firm on the single constraint that all properties of a new ShadowRealm must be configurable. That allows for the possibility of a ShadowRealm after Lockdown shim that can delete anything it must.
Our position on Compartment is the same. We intend to propose Compartment as the missing module loader API and its focus will be virtualizing the module system and nothing else. We would then defer the issue of Compartment after Lockdown to the Lockdown proposal proper. The SES Shim already makes this distinction. After initialization, you get a global Compartment and after Lockdown, that gets replaced with Compartment-after-Lockdown).
My prior comment applies to Compartment-after-Lockdown and this proposal has not been updated since we arrived at that consensus. Please pardon the dust!

@annevk We recently landed a significant reduction in scope. The compartments proposal expressly only touches module loader host virtualization, and doesn’t seek to exclude any other host behaviors expressly or implicitly.
I’ll try to recall your suggestion when we frame a Lockdown proposal. There, we do intend to expressly hide host behaviors, to the extent that is possible by omitting certain intrinsics from the shared intrinsics between locked-down compartments / loaders.


Update Getting Started/Overview page content update for April release

As an IBM.com Library adopter,

I need to see the latest information about the adoption routes and adoption resources (adopter starter kit)

so that I know how to adapt properly and use the correct resource.

Additional information
Since we are open the website content to all IBMers, we need to take a look at the Getting Started/Overview page content and make the proper update.
Tasks

[ ] The First audit is done by the leadership team (Linda, Wonil, and Jeff)
[ ] Meet with Jenny to align on the changes/to-dos
[ ] Create updates content for the live website update

Acceptance criteria

[ ] Content is updated on the live website before April 24th.

@wonilsuhibm Task issue created as discussed.

Issue moved to carbon-design-system/ibm-dotcom-library-website #396 via ZenHub


Best way to load/store a mask register?

Does here any better way to load / store a mask register without changing vl and vtype?
li t0, -1
vsetvli x0, t0, e32 # Set VL to VLMAX
vle.v  v0, (t1) # Load masking to v0

Compiler need that operation to spill a mask register.

I think that’s it. Or, in prologue/epilogue code, you spill/fill it as part
of a larger block of registers (e.g. spill all of v0-v7).
On many machines, the extra vsetvli won’t matter too much, and spilling
mask registers shouldn’t be super common, so I think this is OK.
On Mon, Nov 4, 2019 at 12:57 AM Kito Cheng<EMAIL_ADDRESS>wrote:

Does here any better way to load / store a mask register without changing
vl and vtype?
li t0, -1
vsetvli x0, t0, e32 # Set VL to VLMAX
vle.v  v0, (t1) # Load masking to v0
Compiler need that operation to spill a mask register.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-v-spec/issues/315?email_source=notifications&email_token=AAH3XQQ67X7PCW7WUYDO2VTQR7PXDA5CNFSM4JIQSJU2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HWRDUWA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH3XQSI4URAJNWXTHGOCOLQR7PXDANCNFSM4JIQSJUQ
.


fix: JSObject AST parsing logic

Description
As we set the jsObjectVariableName in the jsCode which we parse for AST,

we could only focus on particular variable name instead of all variableDeclarator.

Hence adding node.id.name === jsObjectVariableName.
It worked without it as jsObject was the last node to be called in variableDeclarator when parsing AST.
Fixes #15009
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/ok-to-test sha=e9c34b0

/ok-to-test sha=f977337

/ok-to-test sha=07bd9ce

Testing for this PR is skipped as this changes doesn’t change any functionality and just add a logical check in code.

/ok-to-test sha=893c4f1

/ok-to-test sha=5cd3ed2

/ok-to-test sha=b8c5b9d


Native support for PEP 393 (new unicode object structure)

Py3.3 is expected to implement PEP 393, the flexible Unicode string representation.
http://www.python.org/dev/peps/pep-0393/
http://wiki.python.org/moin/SummerOfCode/2011/PEP393
Cython should adapt its optimisations for the unicode type accordingly, especially the for-loop.
Migrated from http://trac.cython.org/ticket/721

See also #1655


Unhandled promise rejection in get value()

Hi there! 👋
First of all, thanks for making this!
I ran into an unhandled promise rejection when using your library. In this case, my environment is missing the AWS region however I think this issue would arise for any failure of the request.
node:29164) UnhandledPromiseRejectionWarning: ConfigError: Missing region in config
    at Request.VALIDATE_REGION (<redacted>/node_modules/aws-sdk/lib/event_listeners.js:92:45)
    at Request.callListeners (<redacted>/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at callNextListener (<redacted>/node_modules/aws-sdk/lib/sequential_executor.js:96:12)
    at <redacted>/node_modules/aws-sdk/lib/event_listeners.js:86:9
    at finish (<redacted>/node_modules/aws-sdk/lib/config.js:379:7)
    at <redacted>/node_modules/aws-sdk/lib/config.js:397:9
    at SharedIniFileCredentials.get (<redacted>/node_modules/aws-sdk/lib/credentials.js:127:7)
    at getAsyncCredentials (<redacted>/node_modules/aws-sdk/lib/config.js:391:24)
    at Config.getCredentials (<redacted>/node_modules/aws-sdk/lib/config.js:411:9)
    at Request.VALIDATE_CREDENTIALS (<redacted>/node_modules/aws-sdk/lib/event_listeners.js:81:26)
    at Request.callListeners (<redacted>/node_modules/aws-sdk/lib/sequential_executor.js:102:18)
    at Request.emit (<redacted>/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (<redacted>/node_modules/aws-sdk/lib/request.js:683:14)
    at Request.transition (<redacted>/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (<redacted>/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at Request.runTo (<redacted>/node_modules/aws-sdk/lib/request.js:403:15)
    at <redacted>/node_modules/aws-sdk/lib/request.js:792:12
    at new Promise (<anonymous>)
    at Request.promise (<redacted>/node_modules/aws-sdk/lib/request.js:778:12)
    at Parameter.getParameter (<redacted>/node_modules/aws-parameter-cache/lib/parameter.js:35:52)
    at Parameter.refreshParameter (<redacted>/node_modules/aws-parameter-cache/lib/parameter.js:28:34)
    at Parameter.refresh (<redacted>/node_modules/aws-parameter-cache/lib/refreshable.js:8:14)
    at Parameter.get value [as value] (<redacted>/node_modules/aws-parameter-cache/lib/parameter.js:14:18)

Adding a .catch() to the cachedResult promise should do the trick. Inside that block we can call the wrapping promise's reject. I'd be happy to open a PR, but thought it'd be good to touch base first 😄 Let me know what you think!

@jjdonov Yes we should catch that. You are welcome to create a PR


coprocessor: fix bug caused by use box_try instead of try (#2130)

@BusyJay @andelf @hicqu PTAL

LGTM

LGTM.


R transform tests

TranslationTransform - in R, GetOffset method doesn't seem to be wrapped (invocation causes an error).
Transform (composite transform) - in R, AddTransform method returns Self (shouldn't return it).

@zivy  Thanks for the pull request for this test failure case.

@richardbeare Is the return self from the AddTransform an error in the R code generation or do you think this is coming from the C++ layer?

There's something weird going on - GetOffset is partly wrapped, but isn't recognised as a method by swig. I can't see why. Other transforms seem OK.
The self return should not happen any more - we ought to be returning void instead. I think there is a macro that is meant to be doing that. Hope to check out in more detail soon

I've discovered something very strange about swig/R (at least I presume it is only swig/R) - it seems that methods with "Offset" in the name result in different behaviour - specifically they generate a "$<-" and "[[<-" function pair instead of a "$" function - this means that it is setting up code to do stuff like:
transform$GetOffset() <- x
Which makes no sense. I've not found where in swig it happens. There are about 4 classes that have this problem!

Sounds like you are making progress. What are the 4 classes? We should add some tests!

OK,
I have this sorted out - it is down to a dodgy part of the R binding generator in swig.
First - the classes with problems are:
GridImageSource
FastMarchingUpwindGradientImageFilter
OtsuMultipleThresholdsImageFilter
TranslationTransform

They all have methods ending with "set" -SetGridOffset/GetGridOffset, SetTargetOffset/GetTargetOffset, SetLabelOffset/GetLabelOffset and SetOffset/GetOffset
The symptom is that these methods aren't accessible using the normal $ notation:
tf <-TranslationTransform(3, c(1,2,3))
tf$GetOffset() # error

The problem is that the binding generation has been searching for methods ending in "set" instead of "_set", which is what the comments say it does. Methods ending in "_set" are created for structures and classes so that values can be set. i.e. if we had a publicly visible integer variable x in a class then in R we could do
V <- TheClass()
V$x <- 10

I've submitted a pull request with the change - hopefully it will get through. I'm testing on sitk now
Available at:
github.com/richardbeare/swig/
c90239bce6164a771e07183f208e53d692c2e70f

Resolved in #82

The fix for this issue happened here swig/swig#839


[AIRFLOW-2406] Add Apache2 License Shield to Readme

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2406

Description

[x] Here are some details about my PR, including screenshots of any UI changes: Documentation update

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason: Not needed -- Documentation update.

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[ ] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Awesome! 👍

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3290   +/-   ##
=======================================
  Coverage   75.89%   75.89%
=======================================
  Files         197      197
  Lines       14706    14706
=======================================
  Hits        11161    11161
  Misses       3545     3545

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 9a0c4e6...d090e5e. Read the comment docs.


Rename set/get_indexed's "property" to "property_path"

This parameter's name is oddly misleading. The entire point of set_indexed and get_indexed is to set/get by path, so "property" is not actually just the property name. It's a whole, relative NodePath.
This PR renames the parameter to to "property_path", and touches up the Documentation slightly to make this clearer.
(This minor oddity can easily be backported to 3.x)

Thanks!


[DEVOPS-1357] template compatibility with terraform 0.12

make it work with terraform 0.12

Coverage remained the same at 100.0% when pulling 5fcceed267f40e3a239bf43110571ddc22425d57 on vonq:DEVOPS-1357 into 518879dfe36542b8bb1fbd37281ba4a862a920e4 on dtan4:master.

Coverage remained the same at 100.0% when pulling 5fcceed267f40e3a239bf43110571ddc22425d57 on vonq:DEVOPS-1357 into 518879dfe36542b8bb1fbd37281ba4a862a920e4 on dtan4:master.


Move deposit contract rules to its own Makefile

As part of a series of PRs to clean up the Makefile, this one moves rules associated with the deposit contract to its own Makefile in the solidity_deposit_contract directory. This part of the project has ossified. Also:

I updated dependency versions so tests work now. There was a conflict with eth2spec dependency versions.
I removed CI workflow tasks associated with the deposit contract. The tests were already disabled.

Thank you


YubiKey PIN entry prompt sometimes blocks using browZer

I'll lodge this observation here while I figure out how to reproduce it reliably. Maybe someone else sees the same thing and knows why.
Symptom: sometimes I can't use browZer in Chrome when my YubiKey is plugged in. A PIN prompt appears and steals focus. If I cancel the prompt or enter the wrong PIN it always reappears, blocking input to Chrome.
Workaround: Unplug the YubiKey or enter the correct PIN
My YubiKey is set up with one private key slot that I do use for Ziti, but I've never associated it with browZer in any way. I suspect ZBR's crypto, or perhaps Chrome itself, is querying for HSMs that could potentially provide a key and prompting to unlock any locked slots that are discovered.
❯ pkcs11-tool --module /usr/local/lib/libykcs11.so --list-slots
Available slots:
Slot 0 (0x0): Yubico YubiKey OTP+FIDO+CCID 00 00
  token label        : YubiKey PIV #20608759
  token manufacturer : Yubico (www.yubico.com)
  token model        : YubiKey YK5
  token flags        : login required, rng, token initialized, PIN initialized
  hardware version   : 1.0
  firmware version   : 5.43
  serial num         : 20608759
  pin min/max        : 6/64

This describes the Linux where I'm running Chrome:
❯ cat /etc/*-release
DISTRIB_ID=Pop
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Pop!_OS 22.04 LTS"
NAME="Pop!_OS"
VERSION="22.04 LTS"
ID=pop
ID_LIKE="ubuntu debian"
PRETTY_NAME="Pop!_OS 22.04 LTS"
VERSION_ID="22.04"
HOME_URL="https://pop.system76.com"
SUPPORT_URL="https://support.system76.com"
BUG_REPORT_URL="https://github.com/pop-os/pop/issues"
PRIVACY_POLICY_URL="https://system76.com/privacy"
VERSION_CODENAME=jammy
UBUNTU_CODENAME=jammy
LOGO=distributor-logo-pop-os

Closing because it's stale and I'm not investigating or feeling this pain because I stopped leaving my YubiKey plugged in


PIXI.loaders.Loader Docs

Hey,
I've been playing with pixi.flip and sprite sheets yesterday, and I'd like to suggest a minor update for the PIXI.loaders.Loader docs (docs).
Reading the API docs right now suggests that the Loader is used for loading images. It might be a problem with my imagination, but it actually took quite a while and a bit of reading of pixi's source code to understand that the Loader is actually conveniently contextual:
Loader._pixiMiddleware = [
    // parse any blob into more usable objects (e.g. Image)
    ResourceLoader.middleware.parsing.blob,
    // parse any Image objects into textures
    textureParser,
    // parse any spritesheet data into multiple textures
    spritesheetParser,
    // parse any spritesheet data into multiple textures
    bitmapFontParser
];

Also the comment in the source is doubled wrongly ;P

The loader can load anything. Then for certain things we have parsers to make it handle certain things more gracefully. JSON files try to get parsed for spritesheets and automatically download the image file, XML files try to get parsed for bitmap fonts and automatically download the image file, etc.
Unfortunately there is no parser for video and fonts, yet but that doesn't stop you from being able to use the loader for them.
What was the hard part to wrap your head around? What comment in which source is wrong?

@englercj sorry man, been a bit busy. Here are the answers right away:

Wrong comments in the source:

// parse any spritesheet data into multiple textures
spritesheetParser,
// parse any spritesheet data into multiple
bitmapFontParser

Second comment should be more like // parse any bitmap font data. But that's minor.

The reason for that issue was the docs itself. These docs: http://pixijs.github.io/docs/PIXI.loaders.Loader.html . They say nothing about the contextual nature of the Loader. From the docs I can read as much as that I can load images via the Loader.

Would be nice to add there that it is also possible to load spritesheets and fonts with that Loaders ;)


TypeSpec-Project-Sync.ps1 isn't thread safe in running in parallel

https://github.com/Azure/azure-sdk-for-net/blob/main/eng/common/scripts/TypeSpec-Project-Sync.ps1#L11-L12
. $PSScriptRoot/Helpers/PSModule-Helpers.ps1
Install-ModuleIfNotInstalled "powershell-yaml" "0.4.1" | Import-Module

The steps that install these PS modules isn't thread safe so if multiple projects are running in parallel there is a race condition which might break.
Currently the HealthInsights service directory has 2 libraries and when doing the code gen in parallel it hits sometimes hits issues. See https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2671115&view=logs&j=3facec84-71ec-57ab-8937-e7da3fbb0ee2&t=3cf78e05-2c34-599d-eb6a-be47097ec912 as some examples for the failures.
FYI @asaflevi-ms
@m-nash @chunyu3 @pshao25 this is going to end up being trouble similar to the npm install issue we have. We can solve this either by doing a pre-install of this module before we start the parrallel work or somehow need to break the dependency on the powershell-yaml module in our sync script.

I think this should move to azure-sdk-tools as its a common script for all languages.
I think you mean cross process safe vs thread safe?  If I understand correctly there are multiple processes here not multiple threadss?
We could break dependency on this and do our own yaml parsing, but that seems odd it would be required.  Is there a different way to call this powershell install module in a way that is cross process safe?

This is just FYI: https://github.com/NuGet/Home/issues/2628
I was thinking why we have so many projects installing nuget package in parellel but we never met the process race condition. In the above issue, nuget fixed same problem by adding file lock.
Of course, to not make such complicated, we could do the similar to npm problem:  https://github.com/Azure/azure-sdk-tools/pull/5690#discussion_r1152824394

Cross process or thread doesn't matter much here it mostly boils down to installing modules at the same time causes issues as the modules are user wide.
I'm attempting to help with this particular error in the anyalze job by adding some PS module caching in https://github.com/Azure/azure-sdk-for-net/pull/35339.


Fix "Error: Cannot find module './CoordinatedClient" in new test server

Error!: Cannot find module './CoordinatedClient'
Stack: [{"_fileName":"module.js","_functionName":"$w._oldRes","_lineNumber":"802","_columnNumber":"9","_msg":"    at $w._oldRes@module.js:802:9"},{"_fileName":"module.js","_functionName":"$w._resolveFilename","_lineNumber":"1771","_columnNumber":"1","_msg":"    at $w._resolveFilename@module.js:1771:1"},{"_fileName":"module.js","_functionName":"$w._load","_lineNumber":"362","_columnNumber":"4","_msg":"    at $w._load@module.js:362:4"},{"_fileName":"module.js","_functionName":"$w.prototype.require","_lineNumber":"477","_columnNumber":"8","_msg":"    at $w.prototype.require@module.js:477:8"},{"_fileName":"module.js","_functionName":"require","_lineNumber":"495","_columnNumber":"8","_msg":"    at require@module.js:495:8"},{"_fileName":"/data/data/com.test.thalitest/files/www/jxcore/lib/CoordinatedTape.js","_functionName":"","_lineNumber":"12","_columnNumber":"25","_msg":"    at @/data/data/com.test.thalitest/files/www/jxcore/lib/CoordinatedTape.js:12:25"}]
09-14 18:16:51.282 16527-16612/com.test.thalitest I/jxcore-log: ERROR testUtils: uncaught exception, error: 'Error: Cannot find module './CoordinatedClient'', stack: 'Error: Cannot find module './CoordinatedClient'
at $w._oldRes@module.js:802:9
at $w._resolveFilename@module.js:1771:1
at $w._load@module.js:362:4
at $w.prototype.require@module.js:477:8
at require@module.js:495:8
at @/data/data/com.test.thalitest/files/www/jxcore/lib/CoordinatedTape.js:12:25'
09-14 18:16:51.282 16527-16612/com.test.thalitest I/jxcore-log: ERROR testUtils: TEST_LOGGER:[PROCESS_ON_EXIT_FAILED]
09-14 18:16:51.282 16527-16612/com.test.thalitest E/jxcore-log: Error:
09-14 18:16:51.282 16527-16612/com.test.thalitest E/jxcore-log: Cannot find module './CoordinatedClient'
09-14 18:16:51.282 16527-16612/com.test.thalitest E/jxcore-log:  (module.js 802:9)

There is a problem that files in the same directory can't see each other. We get
ERROR thaliTape: uncaught exception, error: 'Error: Cannot find module './CoordinatedClient'', stack: 'Error: Cannot find module './CoordinatedClient' at $w._oldRes@module.js:802:9 at $w._resolveFilename@module.js:1771:1 at @/data/data/com.test.thalitest/files/www/jxcore/lib/CoordinatedTape.js:14:1 at $w.prototype._compile@module.js:621:8 at $w._extensions[".js"]@module.js:651:1 at $w.prototype.load@module.js:442:1'
while running the app.
These files presented into a builded apk but on device files/www/jxcore folder is epmty. This folder was empty in previous versions also so it's default behaviour.
I'm going to investigate what cordova makes with assets during building process.

Needed files are in data/app/com.test.thalitest-1/base.apk. I installed our app on emulator, grabbed the apk and looked inside the apk.

Still fighting with this problem. There are a lot of error if I change order of require modules. Also I found out that we use old jxcore in our build script. I'm going to change that and check with 0.1.4

We have a work around for this so we will put it into backlog for now.

The work around didn't work. Currently it seems like we can't have require chains that are more than 4 deep. We need to fix this ASAP.

The problem doesn't show up on desktop but that is expected since in thaliTape.js, 'require('./CoordinatedTape');' is only called on mobile:
if (typeof jxcore === 'undefined' || typeof Mobile !== 'undefined') {
  // On mobile, or outside of jxcore (some dev scenarios)
  // we use the server-coordinated thaliTape.
  exports = require('./CoordinatedTape');
  exports.coordinated = true;
} else {
  // On desktop we just use simple non-coordinated tape.
  exports = require('./SimpleTape');
  exports.coordinated = false;
}

Running the same code on Android using the standalone jxcore binary requires a few 'fixes' here and there, but overall it works and loads the CoordinatedClient without errors.
root@generic_x86:/data/local/tmp/thali/test # ../jx UnitTest_app.js
CoordinatedClient loaded
CoordinatedTape loaded
...

We can run jx runCoordinatedTests.js with CoordinatedTape on desktop.

@andrew-aladev
is it possible to run the Thali test app as a cordova app on desktop? Because running jx runCoordinatedTests.js is not what I need in the sense that even on Android I can run it and it doesn't fail to load the CoordinatedClient module.

@enricogior So are the tests actually passing when you run them on Android? How many phones do you have?

At the current point of investigation, this seems a bug in the SpiderMonkey engine when running on ARM.
The bug only shows up when using this code:
if (!Array.prototype.shuffle) {
  Array.prototype.shuffle = function () {...};
}

Replacing existing Array.prototype methods works, adding a new one has the side effect of causing the 'require()' bug.

This is obviously awful but we don't currently have a good way of debugging on ARM what exactly is going on so for the short term we are going to remove prototypes on Array objects (and other system objects) but eventually we have to run this to ground.

See the comment in thaliproject/Thali_CordovaPlugin#1123  about js-polyfill. But @enricogior, if the ponyfill work around works then we should still keep this bug on the backlog but we can come back to it later and now with a test case (js-polyfill) that will apparently blow up on the desktop.

For now we have a work around so we'll have to put it into icebox.


Add new role for managing /etc/hosts entries

We need correct updates in /etc/hosts several times. So it is valuable to make a separate role out of it.
Example
- name: ensure /etc/hosts is updated
  include_role: sap_update_etc_host
  var:
        sap_update_etc_hosts_name: item.name
        sap_update_etc_hosts_domain: item.domain
        sap_update_etc_host_ip: item.ip
        sap_update_etc_hosts_add_alias: ""
        sap_update_etc_hosts_keep_old_alias: default(true )
        sap_update_etc_hosts_check_dns: default(true)
        sap_update_etc_hosts_dns_error_fatal: false
        sap_update_etc_hosts_name_length: 13
  loop:
    - { ip: <IP_ADDRESS>, name: node1 , domain: example.com }
    - name: node2
      domain: example.com
      ip: <IP_ADDRESS>

Tasks: will be extracted from sap_general_preconfigure

As per applicable SAP notes (e.g. 2772999), the /etc/hosts entries must be:
ip $(hostname -f) $(hostname -s) [aliases]
So this is something the role must support (just as in sap_general_preconfigure), meaning that we also need to define (or get from ansible) the DNS domain name of the system(s).
I also like the idea of having one variable which contains a list of dicts, but we'll see what Markus comes up with.

We should avoid duplicating definitions.
Also, in the HSR role I have changed the quite rough solutions I found in other code to a check for conflicts and update only if it's safe. Got experience with wild /etc/hosts in real environments. ;)
https://github.com/sap-linuxlab/community.sap_install/blob/main/roles/sap_ha_install_hana_hsr/tasks/update_etchosts.yml

I agree. The above can then easily be called like:
include_role: update_etc_hosts
vars:
   - update_etc_hosts_nodes: "{{ sap_hana_cluster_nodes }}"

Janine, Bernd and I are through many possible misconfigurations and the code is basically there.  So the idea is to consolidate all these calls with a single role

What final name do you vote for this role:

update_etc_hosts
os_update_etc_hosts
sap_update_etc_hosts
anything else?

+1 for a simple "update_etc_hosts"
"os_update...something" could be misleading to be somehow related to OS updates.
It is not sap special from a functional perspective.

I vote for 'etc_hosts' or 'os_etc_hosts'
Because role should be idempotent and not update the records, but make sure correct records are in the /etc/hosts file, also role should remove records from the hosts file sometimes.  For instance like this
- name: ensure /etc/hosts is updated
  include_role: sap_update_etc_hosts
  var:
        sap_update_etc_hosts:
                - ip: <IP_ADDRESS>
                   hostname: host1
                   aliases:
                    - alias1
                    - anotheralias2
                   comment: "Here comes text after hashsign"
                   state: present
                - ip: <IP_ADDRESS>
                   hostname: host2
                   state: absent

I've added state parameter compared to my initial suggestion.

All roles should ideally be idempotent, but still are called "configure..." because that is the intention if things are not already correctly in place. :)
I don't think an /etc/hosts update run by our SAP deployment roles should delete anything conflicting from the file.
Maybe we could make it optional for the user to explicitly accept conflicts to be solved such that new variable values will overwrite existing different entries?

Agree with @kksat regarding idempotency and overall "KISS principle" naming convention of Ansible Role (e.g. use os_ prefix rather than sap_).
However I am confused what this Ansible Role does in comparison with:
community.sap_operations -- /roles/os_etchosts
While update of /etc/hosts is undoubtably performed during an installation, it is also a maintenance activity. Same as maintaining ~/.ssh/known_hosts (covered by community.sap_operations -- /roles/os_knownhosts)

I have named the role os_update_hosts, the input parameters are compatible with the cluster setup parameters and can hence used immediately (see. README) without rewriting

@rhmk Can you please address previous comment regarding the correct Ansible Collection for update_etc_hosts to belong to? And comparison to existing (incomplete) Ansible Role?
https://github.com/sap-linuxlab/community.sap_install/issues/171#issuecomment-1241756005

Solved in #495.


Adding SkiaSharp to .NET Standard projects produces many duplicate DLLs

Originally in: https://bugzilla.xamarin.com/show_bug.cgi?id=60353
When I add SkiaSharp to a .NET standard project, the project tree gets overrun with DLLs, and DYLIBs (SO files are missing, where could they be?). Please see attached project and screenshots.
There are several bugs here:

Libraries shouldn't display their native file dependencies. I don't see Android support libs, Xamarin iOS, sqlite-raw, or any other native library wrappers showing their dependencies in the tree.

There's duplication in the tree: the same DLL shows up multiple times (along with its config files).

When compiling, there are duplicate output DLLs. libSkiaSharp.dll shows up no less than 3 times in the bin directory. Once at root, and twice on x86 and x64. 64-bit Intel has been mainstream for 10 years now - maybe we can drop at least the 32-bit version?

I understand you are shipping at least 2 * 3 platforms here (32-bit/64-bit, Linux/Mac/Windows) - but they shouldn't appear in the project tree. And please fix the naming in the bin directory so I actually know what to ship. As it is, I have no idea which of these DLLs actually needs to be deployed with my app.
If you need examples on how to package better, please look at Eric Sink's sqlite-raw. He is covering even more complex scenarios than you and never spills out these implementation details.
TestSkia.zip

This should not happen. I tested on VS2017 and VS4Mac 7.2 and this does not happen:

I also downloaded your project, and this is not the case:

I updated my VSM and it now shows that structure @mattleibow
Yes this is much better.
But I'm still getting duplicate dlls. I have 3 copies of libSkiaSharp.dll

Maybe remove the nuget and re-add. delete the obj/bin. When I build, I don't get any native bits:

I'm sorry, I left off an important detail: I'm consuming the library from an additional classic project (so that unit tests work in VS). That project is getting the 3 copies.
You're right, the standard project is free of native libs. You can close this since I won't waste your time supporting classic .NET projects.

classic .net project will copy all the dlls because there is no way to determine what platforms the app will run on. you should be able to take the output of a classic library/app and run in on ANY windows and ANY mac (with mono) without changes. The output folder is the final product that works everywhere - write once, run anywhere

also, the new vs2017 update adds support for unit tests in .net core apps/test libraries (yay)

I'm on VS for Mac and it only seems to work with Classic NUnit 2.x. :-(

ah. that bug
There is a fix in master so the test runner can understand


Splash Screen Implementation

Description:
Implements the splash screen for both iOS and Android using the flutter_native_splash library.
Screenshots

Couple of bullet points which haven't been addressed yet, hope to speak about them in an upcoming meeting.

If you look at the builds, it failed the Flutter one and it says:
"18s
Run dart analyze
Analyzing quack_app...
warning - analysis_options.yaml:10:10 - The include file 'package:flutter_lints/flutter.yaml' in '/Users/runner/work/quack/quack/quack_app/analysis_options.yaml' can't be found when analyzing '/Users/runner/work/quack/quack/quack_app'. - include_file_not_found
1 issue found.
Error: Process completed with exit code 2."
If you could look into this and fix this issue? It seems like you just have something in your files that isn't supposed to be there maybe?


can u pls add fakegato support?

would be nice to have a temp history
and a switch history

Will definitely take a look at fakegato support.


Upgrade nodemon to remove vulnerability

npm install was failing because nodemon was using the 3.3.6 version of event-stream. The new version of nodemon no longer uses this.
Fixes #239

Just realized https://github.com/GoogleChrome/rendertron/pull/241 was opened and also fixes this issue. Feel free to close this PR if that one gets merged.

Thanks for this PR! Sorry that I didn't get around to this earlier. #306 should supersede this.


Add high scale network mode data to contextsetter

subnetIds and SgIds will be populated if the network mode is high scale, return this information to the titusui using the context setter

@tbak should be your primary reviewer here


SAMPLES: Photo editor and viewer samples doesn't work

Is your suggestion related to a missing or misleading document? Please describe.
The mentioned samples which are given in the doc doesn't work. War file successfully gets deployed to Tomcat Server but it doesn't start and it gives an error "FAIL - Application at context path [/photo-edit (2)] could not be started"
Doc: https://is.docs.wso2.com/en/5.11.0/learn/deploying-the-sample-app/#deploying-the-photo-editor-and-photo-viewer-webapps
This issue affects the following doc as well
https://is.docs.wso2.com/en/5.11.0/learn/configuring-access-delegation-with-uma/
Describe the improvement
Please fix the issue in the given sample applications and test it against Apache Tomcat 8
Testing Environment
Ubuntu 18
apache-tomcat-8.5.59

Could see the below logs in tomcat server.
org.apache.catalina.core.StandardContext.startInternal One or more listeners failed to start. Full details will be found in the appropriate container log file+ error in ubuntu
Caused by: javax.net.ssl.SSLHandshakeException: sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
		at sun.security.ssl.Alerts.getSSLException(Alerts.java:192)
		at sun.security.ssl.SSLSocketImpl.fatal(SSLSocketImpl.java:1946)
		at sun.security.ssl.Handshaker.fatalSE(Handshaker.java:316)
		at sun.security.ssl.Handshaker.fatalSE(Handshaker.java:310)
		at sun.security.ssl.ClientHandshaker.serverCertificate(ClientHandshaker.java:1639)
		at sun.security.ssl.ClientHandshaker.processMessage(ClientHandshaker.java:223)
		at sun.security.ssl.Handshaker.processLoop(Handshaker.java:1037)
		at sun.security.ssl.Handshaker.process_record(Handshaker.java:965)
		at sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:1064)
		at sun.security.ssl.SSLSocketImpl.performInitialHandshake(SSLSocketImpl.java:1367)
		at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1395)
		at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1379)
		at sun.net.www.protocol.https.HttpsClient.afterConnect(HttpsClient.java:559)
		at sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.connect(AbstractDelegateHttpsURLConnection.java:185)
		at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1564)
		at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1492)
		at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)
		at sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:347)
		at org.wso2.photo.view.DCRUtility.performDCR(DCRUtility.java:47)

This was due to some permission issue in the photo-edit folder when the files having some permissions.


panic: runtime error: invalid memory address or nil pointer dereference

Client version v0.6.9
Kafka Version: kafka 2.6
Connection Auth: MTLS connection
Auto commit disabled
Our Kafka dev cluster can be under high load, so no idea if its related
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x7d381f]

goroutine 10119 [running]:
github.com/twmb/franz-go/pkg/kgo.(*topicPartitions).load(...)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/topics_and_partitions.go:73
github.com/twmb/franz-go/pkg/kgo.(*consumer).assignPartitions(0xc0007326a0, 0xc0030022d0, 0xc00084d500)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer.go:357 +0x47f
github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).fetchOffsets(0xc00084d4a0, 0xfc34a8, 0xc002bcce40, 0xc0024914a0, 0x0, 0x0)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:1201 +0x4d7
github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).setupAssignedAndHeartbeat.func3(0xc002bd7080, 0xc002554060, 0xc00084d4a0, 0xc0024914a0, 0xfc34a8, 0xc002bcce40)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:750 +0x145
created by github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).setupAssignedAndHeartbeat
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:746 +0x387

looks to be a sync error with getting topic metadata and the assigning heartbeats

Thanks for the report, and apologies for the panic. I’ll be able to look into this later in the day my time.
I checked a few things quickly, and the code causing this panic comments that the caller must be non nil at that point, so that comment is clearly based on old information. My offhand guess is that something got lost / moved / added during refactorings.
How reliably are you able to trigger this panic?

At the moment every app run its happening consuming from 200+topics indvididly. Our Kafka server looks to be at 80%+ load.
I was in the middle of some debugging to add some extra info:
but its like the topic metadata is not full populated before the topics_and_partitions.go:73 is run. I only see one topic listed in the metadata list and not the topic we should be subscribe for heartbeats for
code using
	client, err := kgo.NewClient(kgo.SeedBrokers(c.brokers...),
		kgo.Dialer(tlsDialer.DialContext),
	if err != nil {
		return err
	}
	defer client.Close()

	client.AssignGroup(c.id, kgo.GroupTopics(c.topic), kgo.DisableAutoCommit())

we create 200 clients in the app one for each topic at moment as we currently testing this library as a drop in replacement and would be optimised in that respect later. I don't see this being the problem as the library looks not to contain any sheared state.

Cool thanks for the update—doesn’t seem like an isolation case is a thing.
I was planning to fully audit the consumer group code soon, and I’ll push up that timeline based off this issue (this weekend or next week).
I do see something that I suspect and can validate later when I’m on a laptop: the code does not currently handle the case where a client is assigned a topic it actually did not express interest in. I’ll also look for a case where it is assigned something but the internal topic partitions struct has not been created for some reason.
What group balancer are you using here?

I don't override the default which this client provides

Thanks! And lastly, are all consumers in the group using this client?

All consumer groups with the same consumer id yes. There is only one
instance of this app running so there should be no other clients connecting
with the same id
On Wed, 3 Mar 2021, 6:05 pm Travis Bischel<EMAIL_ADDRESS>wrote:

Thanks! And lastly, are all consumers in the group using this client?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twmb/franz-go/issues/24#issuecomment-789942544, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AALHS7NUG7VVY5RRRGQ6RIDTBZ26JANCNFSM4YQ7J3KQ
.

Interesting, so this happening in the cooperative case. If it's easy, can you try using the StickyBalancer? And, if that causes the issue still, can you try the RangeBalancer? These can help give clues as to areas of my code to look into.

Here's what I'm currently looking at:

A consumer group member indicates only topics it is interested in when joining a group. In the client, these interests are either static (if you use GroupTopics) or dynamic (if you use GroupTopicsRegex). The topics added to the JoinGroup request are only added if they have been discovered from a Metadata response, and they are stored in the client after discovery before tracking that they are usable.
The sticky balancer has at least a few tests that ensure topics/partitions are assigned correctly, even when some consumers are interested in more topics than others.
Given that, a client should only be assigned topics that it has already stored internally.

I can still fix the panic itself by fixing up fetchOffsets, but given that you said all consumers are using this client, there's a bug somewhere else.
Can you also confirm that each consumer is using its own dedicated client? That is, no consumer is sharing a client?
I may add some info level logging around what a client expresses interest in when joining a group, and then what that client is assigned.

We do get some  connection timeouts on this cluster if that's any help, this happens a lot more frequently in other go libs, not sure about the python ones we use.

[unable to open connection to broker addr *:17189 id 51 err dial tcp *:17189: i/o timeout]

Using different balancers
Sticky
client.AssignGroup(c.id, kgo.GroupTopics(c.topic), kgo.DisableAutoCommit(), kgo.Balancers(kgo.StickyBalancer()))

Not crashing a lot of topics are not getting any data, like they are stuck. There is a lot of lag on these topics. Only 3 topics look to have got data from our metrics.
Ran for 10min
Range
client.AssignGroup(c.id, kgo.GroupTopics(c.topic), kgo.DisableAutoCommit(), kgo.Balancers(kgo.RangeBalancer()))

Crashing
panic: runtime error: index out of range [1] with length 1

goroutine 1895 [running]:
github.com/twmb/franz-go/pkg/kgo.(*rangeBalancer).balance(0x152eab8, 0xc00262c000, 0xf4, 0xf4, 0xc00342e3c0, 0xf4)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/group_balancer.go:306 +0x766
github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).balanceGroup(0xc00062edc0, 0xc003227530, 0x5, 0xc0004f7000, 0xf4, 0xf4, 0xa, 0xc0016b4ed8, 0x7f6f401ae8a0)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/group_balancer.go:122 +0x2ab
github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).handleJoinResp(0xc00062edc0, 0xc0012a0180, 0x0, 0x0, 0x2, 0xc0000b6001, 0x1, 0x0)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:1064 +0x73b
github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).joinAndSync(0xc00062edc0, 0xc0006eff03, 0xebfc80)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:973 +0x3d3
github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).manage(0xc00062edc0)
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:385 +0xc5
created by github.com/twmb/franz-go/pkg/kgo.(*groupConsumer).findNewAssignments
        /go/pkg/mod/github.com/twmb/franz-go@v0.6.9/pkg/kgo/consumer_group.go:1316 +0x70a

Could  it be related to the number of topics we have on our kafka instance, we are only consuming from a subset
Total topics is around 350~
We want to consume 200~
Yes no consumer is shearing a client as we create a whole new client per topic
something like this what happens internally.
topics := []string{} // Would contain around 200 topics

wg := &sync.Waitroup{}

for _, topic := range topics {
      topic := topic
       go func() {

       wg.Add(1)
       defer wg.Done()
	client, err := kgo.NewClient(kgo.SeedBrokers(c.brokers...),
		kgo.Dialer(tlsDialer.DialContext),
	if err != nil {
		return err
	}
	defer client.Close()

	client.AssignGroup("consumerID", kgo.GroupTopics(topic), kgo.DisableAutoCommit())
         for {
		fetches := client.PollFetches(ctx)

		iter := fetches.RecordIter()

		for _, fetchErr := range fetches.Errors() {
			// Handle errors
		}

		for !iter.Done() {
			msg := iter.Next()
                        // Do something with message
                }
         }
    }()
}

wg.Wait()

Thanks for testing. If it's not too much, could you try the RoundRobin balancer too?
I think the dialing is unrelated, that's at a lower level than this library.

Also, I see that you're disabling autocommitting -- can you try with autocommitting enabled?
Also, if you're open to joining the Discord channel, I can ask some higher bandwidth questions, and I really appreciate your assisted debugging so far. The surface area of where this could be occurring is so far quite large, so I'm trying to narrow down at least a few things about it.

Ok round robin no crash just stuck.
The reason  auto committing is disabled is because we want to only commit once we know that the msg has processed correctly.  but happily disable for a quick test.
Yeah should be able to join discord, but will most likely be tomorrow.

Thanks!
Here's my current plan:

the panic in the range balancer looks to be from an obvious mistake, I can fix this and add test cases for it
the original reported panic is still an unknown--I can fix it from occurring, but it would only occur on a bad assignment
I'm going to add some verbose logs to the consumer that is assigned the leader, these logs will print what topics every consumer has expressed interest in, and then the resulting plan, and for each consumer, I will add logs printing their assignment.

When I get those fixes / logs in, if possible I'd like you to re-run the cooperative sticky test (original balancer), the sticky test, and the range test. My hope is that the balancing logs there can give some more concrete things to look at. I can have these small commits pushed today.

auto commit same results. some topics working with messages. At this point I am just assuming its pot luck that some topics work the topics which work sometimes as they are different topics if they do work.

Thanks!
Here's my current plan:

the panic in the range balancer looks to be from an obvious mistake, I can fix this and add test cases for it
the original reported panic is still an unknown--I can fix it from occurring, but it would only occur on a bad assignment
I'm going to add some verbose logs to the consumer that is assigned the leader, these logs will print what topics every consumer has expressed interest in, and then the resulting plan, and for each consumer, I will add logs printing their assignment.

When I get those fixes / logs in, if possible I'd like you to re-run the cooperative sticky test (original balancer), the sticky test, and the range test. My hope is that the balancing logs there can give some more concrete things to look at. I can have these small commits pushed today.

No problem

I've pushed two commits, can you try those tests with go get github.com/twmb/franz-go@8178f2cc5883304d409f653ff7e5d42163cf3e8e?

Ok here is the log output for
client.AssignGroup(c.id, kgo.GroupTopics(c.topic), kgo.DisableAutoCommit())

group-balcner-logs.log

client.AssignGroup(c.id, kgo.GroupTopics(c.topic), kgo.DisableAutoCommit(), kgo.Balancers(kgo.StickyBalancer()))

sticky-balancer.log

client.AssignGroup(c.id, kgo.GroupTopics(c.topic), kgo.DisableAutoCommit(), kgo.Balancers(kgo.RangeBalancer()))

range-balencer.log

@owenhaynes I added the new logs at the Debug level, and I think the default is Info -- is it possible to up the level to Debug? I see some logs that are at the info level but none of the new ones I added.
Also, how many members are in the group?

I thought I had set it logging to debug  will double check.
Just one member as only have 1 pod running

Here are the fixed debug logs
defualt_balancer.log
range_balancer.log
sticky_balancer.log

FYI we upgraded to kafka 2.7 today, issue still exists so not related to using Kafka 2.6

For posterity, this was moved to discord and fixed in the following three commits:

https://github.com/twmb/franz-go/commit/e038916 fixes range balancer
https://github.com/twmb/franz-go/commit/e168855 fixes an edge case in the sticky balancer (what caused this panic originally)
https://github.com/twmb/franz-go/commit/a670bc7 allows the leader to balance topics that the leader itself does not know of (the stuck problem from this issue)
https://github.com/twmb/franz-go/commit/938651e ensures that the panic will not happen even if a different leader is bugged


View activity on groups of repositories

It would be useful to either show all activity on a group of repositories in devarchy, or build a link to https://github.com/search/advanced with all of the repositories prefilled, like:
https://github.com/search?utf8=✓&q=repo%3Ashastajs%2Fshasta+repo%3Atonyhb%2Fredux-ui+repo%3Aphilpl%2Ffluorine&type=Repositories&ref=advsearch

Interesting idea, thanks
Future readers: upvote OP if you want this feature.


Support ragged output in TextVectorization layer

System information

TensorFlow version (you are using): 2.2.0
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state.
Currently TextVectorization supports ragged input, but always outputs dense tensor padded with 0.
It would be cool if in INT mode it will output ragged tensor, so caller may call downstream layers that supports ragged tensors and convert to dense when he needed.
Will this change the current api? How?
No api changes
Who will benefit with this feature?
Users of ragged tensors

@shkarupa-alex
Do you have any use case that requires the feature you are interested in? Please feel free to submit a PR if you have use cases that supports that feature.Thanks!

I think this one is being refered to : https://www.tensorflow.org/guide/ragged_tensor

@shkarupa-alex You can  pass a RaggedTensor into StringLookup, which will return a RaggedTensor. The rest of the TextVectorization preprocessing can be done via standard TF ops.
Closing  this issue since String Lookup  layer is now available in TF 2.3.  Thanks!


chore(deps): prayge fixing codecov

Trying different way of setting up webpack config to maybe fix codecov?

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

Project coverage is 8.27%. Comparing base (f508aee) to head (bd79575).
Report is 1 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #793   +/-   ##
======================================
  Coverage    8.27%   8.27%
======================================
  Files          68      68
  Lines        3952    3952
  Branches      864     864
======================================
  Hits          327     327
  Misses       3625    3625

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


Red ring should not be displayed around the Select field like other form fields

Issue Description

The latest version of terra-form removed the red ring to be displayed around all the required form components. But it was not removed in the Select component.

Issue Type

[ ] New Feature
[ ] Enhancement
[x] Bug
[ ] Other

Expected Behavior

The Select field should work like other Form fields and shouldn't display the red ring even if it is required.
Current Behavior

The Select field displays a red shadow now if it is required.

Steps to Reproduce

Open in FireFox and go to Form SelectField component (required field).
Try submitting the form.
Red ring gets displayed around SelectField.

Environment

Component Version: 1.3.0
Browser Name and Version: Firefox
Operating System and version (desktop or mobile): MacOS

Fixed in #719

This has been released in<EMAIL_ADDRESS>


port configuration during setup

https://www.semi.technology/developers/weaviate/current/getting-started/installation.html has quite few choices. Unfortunately the choice of the port for the service is currently not possible. My Jenkins Continuous Integration for e.g. https://github.com/WolfgangFahl/pyWeaviateTest currently fails since port 8080 is in use by jenkins itself.

Given the complexity that the configurator already has (#1680), I'm hesitant to add yet another question. Maybe we could add a link to a page on how to further customize the file further, so that users could manually edit the file to change the port? There are also a few environment variable settings that aren't included in the generator, but the idea is that if someone wants to go this deep, it's ok to manually edit the files. (cc @laura-ham)

IMHO it's just one more field in a dialog:

see http://diagrams.bitplan.com/render/png/0x4e49a1f0.png
source text at http://diagrams.bitplan.com/render/txt/0x4e49a1f0.txt
editable at https://diagrams.bitplan.com using "plantuml" for rendering

What's more is that there should be an option for the configurator's REST API so that no special handling is necessary. Currently i am tentatively thinking about adding code for getting and setting the port in the .yml file for the automatic CI handling in my own python project. It seems awkward to have to do this.

Are you retrieving the configurations dynamically on CI? Please note that there are no stability guarantees on that API. It's single purpose is to serve the UI on our website. For example, if a new field is added in the future and it is not set in old links, they will break.
If we added named configs / preset as outlined in #1680, we could guarantee that those are stable, but current links are not.

Currently i have no other option than to try to keep up to date with the changes semi technologies makes to the configuration process and the versions of the docker core products and the python library. I have not been able to integrate weaviate into any of my projects yet due to this "moving target" nature. Any time i find a few hours to work on things the hours are eaten up by getting up-to-date with the changes in configuration and versions. E.g. lately i had to do a 1 hour download three times failing due to limited docker space before i could continue. That's why added #1682 which points to the issue of the size of downloads necessary which makes local installations awkward for the time needed for download if there is no high-speed internet connection available.

I still can't test weaviate in my CI since i don't even know how to run weaviate on a different port than 8080. How could i change the .yml file to get this effect? There are four 8080 references at this time in the setup i am using:
---
version: '3.4'
services:
  weaviate:
    command:
    - --host
    - <IP_ADDRESS>
    - --port
    - '8080'
    - --scheme
    - http
    image: semitechnologies/weaviate:1.7.0
    ports:
    - 8080:8080
    restart: on-failure:0
    environment:
      TRANSFORMERS_INFERENCE_API: 'http://t2v-transformers:8080'
      QUERY_DEFAULTS_LIMIT: 25
      AUTHENTICATION_ANONYMOUS_ACCESS_ENABLED: 'true'
      PERSISTENCE_DATA_PATH: '/var/lib/weaviate'
      DEFAULT_VECTORIZER_MODULE: 'text2vec-transformers'
      ENABLE_MODULES: 'text2vec-transformers'
  t2v-transformers:
    image: semitechnologies/transformers-inference:sentence-transformers-multi-qa-MiniLM-L6-cos-v1
    environment:
      ENABLE_CUDA: '0'
...

according to https://stackoverflow.com/questions/40553265/changing-port-in-docker-compose-yml it should be sufficient to change the lines:
   ports:
    - 8080:8080

to
   ports:
    - 8090:8080

to e.g. use port 8090

according to https://stackoverflow.com/questions/40553265/changing-port-in-docker-compose-yml it should be sufficient to change the lines:
   ports:
    - 8080:8080

to
   ports:
    - 8090:8080

to e.g. use port 8090

This is indeed correct. The ports string <PortA>:<PortB> essentially means <Port-on-host-machine> should map to <Port-of-container>, so if you change the value before the : you can control how Docker exposes Weaviate to your host machine.

as a workaround i am changing the port with an awk script and checking it with python reading the yaml file.

still at complicated:

I have not been informed about any progress on this.


Training code

Can you share the training codes for training our own datasets? We want to evaluate your method on the complex datasets?

Hi,
I plan to share code for training as well in the (near) future. However, please keep in mind that for training on your own dataset some modifications would probably have to be applied anyway.

OK. @iro-cp  Thanks a lot. I am keeping in mind for your project. Good Luck~

Thanks!


build.py API documentation

Right now the API usable within the build.py is not fully documented. We would need a new documentation page for this kind of thing, as it doesn't fit in the plugins documentation (which is overloaded anyway).

[ ] Document project.* methods (like project.install_file
[ ] Document module variables (like version and name).
[ ] Document the VCS revision stuff
[ ] Document build.py bootstrapping
[ ] Document use_bldsup

@locolupo what do you think? Will the new documentation structure on read the docs allow us to do this in an intuitive fashion? Would you add these things to the plugin documentation (e.G. document project.depends_on() in the install_dependencies plugin) or with a more problem-oriented approach (I want to build with distutils, so here's the distutils plugin and here's project.install_file() and project.depends_on())


Added Questions in Python

Assign me.

I am an experienced coder in Python. I can work on it and provide easy to understand solutions.
I am a 3star at Codechef and 5 star in Hackerrank
So Please Assign me

Go Ahead @pranita736 @Sakalya100


error sending request for url (http://localhost:8899/): error trying to connect: tcp connect error: Connection refused (os error 61)

Hi guys,
I following tutorial on metaplex documentations, but when i try upload, terminal returne me this error:
`[1/4] 🗂  Loading assets
Found 10 asset pair(s), uploading files:
+--------------------+
| images    |     10 |
| metadata  |     10 |
+--------------------+
[2/4] 🖥  Initializing upload
🛑 Error running command (re-run needed): error sending request for url (http://localhost:8899/): error trying to connect: tcp connect error: Connection refused (os error 61)
`
this is configuration file:
{   "price": 1000.0,   "number": 10,   "gatekeeper": null,   "creators": [     {       "address": "9***********************************K",       "share": 100     }   ],   "solTreasuryAccount": "9***************************K",   "splTokenAccount": null,   "splToken": null,   "goLiveDate": "2022-09-25T04:10:29.072976+00:00",   "endSettings": null,   "whitelistMintSettings": null,   "hiddenSettings": null,   "freezeTime": null,   "uploadMethod": "bundlr",   "retainAuthority": true,   "isMutable": true,   "symbol": "CRAB",   "sellerFeeBasisPoints": 500,   "awsConfig": null,   "nftStorageAuthToken": null,   "shdwStorageAccount": null }
any cam help me ?

You have to specify the rpc url as devnet or mainnet.
Run:
# for mainnet:
solana config set --url mainnet-beta

# for mainnet
solana config set --url devnet

Perfect! now work!
I thought that i must set "devent" in sugar command.. but Ok very Thnk

Great to hear it's working now.
In the future, please try to use the discord channel for help unless you feel there is a bug in sugar itself.
Also, can you please close the issue to avoid confusion.


migrate 'utils/dragster' to typescript

Summary
Migrate the dragster utility to Typescript, update it's imports,
and migrate its tests to Typescript.
Ticket Link
Completes https://mattermost.atlassian.net/browse/MM-18909
Additional Comments
I'm not a fan of the null checks in the tests but was unsure what a better method to satisfy the compiler would be so I'm open to suggestions.

Thanks for helping us with the migration, @mbluemer. I'll bug @devinbinnie to make sure that he sees this. He submitted a PR to update the ESLint rules, so that should hopefully be in master relatively soon.


QG 4 checks (Release 23.12)

QG checks
Please keep this issue open until QG is concluded and will be managed by the Issue Creator!
We will inform you about finding and proposals in separated issues, this issue here is for the Overview of the Checks!
Please keep this issue open until QG is concluded!
Product Owner: @jjeroch
Dev SPOC:
Helm Chart Version: 1.7.0
App Version: 1.7.0
Release Managemnet Reference Issue: eclipse-tractusx/sig-release#106
Check of Tractus-X Release Guidelines

Currently implemented automatic checks can be found under your product on our Release Guidelines Checks Board
This QG Check is depending on the mandatory information from our current Release Guidelines

TRG 1 Documentation

[x] TRG 1.01 appropriate README.md
[x] TRG 1.02 appropriate install instructions either INSTALL.md or in README.md
[x] TRG 1.03 appropriate CHANGELOG.md

TRG 2 Git

[x] TRG 2.01 default branch is named main
[x] TRG 2.03 repository structure
[x] TRG 2.04 leading product repository
[x] TRG 2.05 .tractusx metafile in a proper format

TRG 3 Kubernetes

[x] TRG 3.02 persistent volume and persistent volume claim is used when needed

TRG 4 Container

[x] #358
[x] #359
[x] TRG 4.03 image has USER command and Non Root Container
[x] TRG 4.05 released image must be placed in DockerHub, remove GHCR references
[x] TRG 4.06 separate notice file for DockerHub has all necessary information

TRG 5 Helm

[ ] TRG 5.01 Helm chart must be released
[ ] TRG 5.02 Helm chart location in /charts directory and correct structure
[ ] TRG 5.04 CPU / MEM resource requests and limits and are properly set
[ ] TRG 5.06 Application must be configurable through the Helm chart
[ ] TRG 5.07 Dependencies are present and properly configured in the Chart.yaml
[ ] TRG 5.08 Product has a single deployable helm chart that contains all components
[ ] TRG 5.09 Helm Test running properly
[ ] TRG 5.10 Products need to support 3 versions at a time
[ ] TRG 5.11 Upgradeability

TRG 6 Released Helm Chart

[ ] TRG 6.01 Released Helm Chart

TRG 7 Open Source Governance

[x] TRG 7.01 Legal Documentation
[x] TRG 7.02 License and copyright header
[x] TRG 7.03 IP checks for project content
[x] TRG 7.04 IP checks for 3rd party content
[x] #360
[x] TRG 7.06 Legal information for end user content
[x] TRG 7.07 Legal notice for documentation

Hints
Information Sharing
Skipping TRG 5 + 6 as Helm Charts are part of repo portal-cd

Checks finished for portal-backtend, based on 1.7.0-rc4. I've opened following issues:

#358
#360

Please take care, @evegufy

@carslen thanks for the check! could you please close the issue?


Logout

Includes the isDashboard hack from #12 in order to get the login to redirect properly.

I'm going to merge this but the implementation of logoutAction will likely change when we switch to db sessions.


fix certificate renewl docker image letsencrypt

the container doesnt work properly since the environment variables are not available in the cron job in the container
e.g. see links https://ypereirareis.github.io/blog/2016/02/29/docker-crontab-environment-variables/
also the container needs to be changed. it must become more flexible since we aquire 2 certificates now. one for the nginx and another one for the wonnode. => file mapping path must be adjusted
also add "matchat.org" to the alternative certifacte names for owner (in addition to www.matchat.org)
check if nginx.conf (s) need to be adapted
right now letsencrypt certificates for nginx and wonnode have been created manually with standard passwords

=> postponed since the implementation effort currently to high => certificate renew will be done manually for now ...

enabled simple manual renewal process of letsencrypt certificate
described here: https://github.com/researchstudio-sat/webofneeds/wiki/Cryptographic-Keys-and-Certificates


error: Build input file cannot be found:

Hello,
During the build in react-native on iOS it gives me an error that made me confuse. The error said:
error: Build input file cannot be found: 'node_modules/react-native-svg/ios/Utils/RNSVGPercentageConverter.m'
I uninstalled react-native-svg and installed again, and again give me that error.
Please someone help me to solve this.
Thank you in advance!

Facing the same problem since yesterday.

Try running pod install

I solve it. I uninstalled and installed again, and restarted PC.
Thank you guys :)

In my case, 'npm install react-native-svg' is not uploading RNSVGPercentageConverter.m and RNSVGPercentageConverter.h, and pod install doesn't help either.

The file not used / does not exist anymore. Not sure why you'd get any errors. Try removing the dependency and adding it again.

I can see the file in the v9 branch on github.

I can see the reference of the file here https://github.com/react-native-community/react-native-svg/blob/6bd27dfeba3422482b43b15bef287b72d1bf7d54/ios/RNSVG.xcodeproj/project.pbxproj#L23

I fixed it.

react-native unlink react-native-svg
rm -rf node_modules
npm install
react-native link react-native-svg

I created a new PR to fix this issue => https://github.com/react-native-community/react-native-svg/pull/1128

@fabioplaka can you reopen this issue in the meantime? :)

Build works with run-android
Build is fine in xcode.
Build fails with run-ios only - this is what I get.
Tried 9.10.1 and 9.10.2.
Any idea anyone? I have unlinked, resinstalled, etc...

warning: The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 7.0, but the range of supported deployment target versions is 8.0 to 13.0.99. (in target 'AirMaps' from project 'AirMaps')

error Failed to build iOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening flittermobile.xcodeproj

2019-10-02 23:51:49.081 xcodebuild[22635:4710280]  DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:49.113 xcodebuild[22635:4710272]  DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:49.331 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:49.340 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:50.112 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:50.115 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:50.126 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:50.128 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:50.149 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
2019-10-02 23:51:50.152 xcodebuild[22635:4710130] [MT] DTDeviceKit: deviceType from 00008006-0004ED2E1153002E was NULL
** BUILD FAILED **

The following build commands failed:
	CompileC /GitHub/mobile/ios/build/flittermobile/Build/Intermediates.noindex/RNSVG.build/Debug-iphonesimulator/RNSVG.build/Objects-normal/x86_64/RNSVGPercentageConverter.o /GitHub/mobile/node_modules/react-native-svg/ios/Utils/RNSVGPercentageConverter.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
. Run CLI with --verbose flag for more details.```

Have you tried cleaning the build?
Alternatively, you might have a reference to the RNSVGPercentageConverter files which don't exist anymore.

I just spotted that this seems to be an issue only if I link the library. If I do link it, I cannot build with run-ios... If I don't link it, I can build with run-ios successfully. Verified on 9.10.2. Anyone else?

Cleaned all properly and now can build on with run-ios. Thanks for your help!

I was able to build it by removing yarn.lock and doing yarn install again

Looking at my Pods-folder, I could see some old dependencies there. Deleting that folder and running pod install solved my issue.

I was facing same issue and uninstalled the folder, I tried to npx-pod-install then it worked.


Multi component: PackageImports do not work

The module finding code is hard-wired to look in only external packages if you use package imports.
It looks like it might be possible to trick GHC by injecting some entries into the ModuleToPkgConfAll map.
This is something @fendor will need to fix upstream this summer.
Test case: generic-lens.

Branch with a fix is wip/package-imports

Now in hls branch.


set_notification_error, could not locate CCCD descriptor for characteristic

Could not locate CCCD descriptor for characteristic
When tried to execute await characteristic.setNotifyValue(true); will cause an error.
Error:
E/flutter (28172): [ERROR:flutter/lib/ui/ui_dart_state.cc(186)] Unhandled Exception: PlatformException(set_notification_error, could not locate CCCD descriptor for characteristic: 00002a00-0000-1000-8000-00805f9b34fb, null, null)
by the way, it's my first time to right issue in github

Same problem.

no solution?

Have the same problem, found this article: https://www.programmersought.com/article/17006030580/
Note: my app works great on iOS, this only happens on Android

I fixed this issue by modifying the plugin source code as below.
flutter_blue-0.8.0/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java
line start at 496
 //BluetoothGattDescriptor cccDescriptor;
                try {
                    gattServer = locateGatt(request.getRemoteId());
                    characteristic = locateCharacteristic(gattServer, request.getServiceUuid(), request.getSecondaryServiceUuid(), request.getCharacteristicUuid());
                   /* cccDescriptor = characteristic.getDescriptor(CCCD_ID);
                    if(cccDescriptor == null) {
                        throw new Exception("could not locate CCCD descriptor for characteristic: " +characteristic.getUuid().toString());
                    }*/
                } catch(Exception e) {
                    result.error("set_notification_error", e.getMessage(), null);
                    return;
                }

line start at 570
/* if(!cccDescriptor.setValue(value)) {
                    result.error("set_notification_error", "error when setting the descriptor value to: " + value, null);
                    return;
                }

                if(!gattServer.writeDescriptor(cccDescriptor)) {
                    result.error("set_notification_error", "error when writing the descriptor", null);
                    return;
                }*/

Any news on this topic?

same problem,  i tried running the app on two devices, and one got the app running and the other stucked here

same problem,  i tried running the app on two devices, and one got the app running and the other stucked here

Try to use package flutter_blue_plus. Seems to fix some of the errors in this package.


Configurable MetricConfig defaults

Description
Lets users change globals in whylogs/core/configs.py to set the default values for MetricConfig.
clickup
Changes

MetricConfig::__inti__() now takes all Optional arguments and uses whylogs.core.configs variables for None passed in. That was tricky, since it's a frozen data class...

[ ] I have reviewed the Guidelines for Contributing and the Code of Conduct.

#1257 is better


now: command not found

I did $ npm install -g now I am on Ubuntu 16.04
I Got this Code:
`/home/parjanya/.npm-global/bin/now -> /home/parjanya/.npm-global/lib/node_modules/now/download/dist/now

><EMAIL_ADDRESS>postinstall /home/parjanya/.npm-global/lib/node_modules/now
> node download/install.js

> For the source code, check out: https://github.com/zeit/now-cli

> Downloading Now CLI 8.3.11 [====================] 100%

/home/parjanya/.npm-global/lib
`--<EMAIL_ADDRESS>`

now I did  $ls
I got: index.js  node_modules  package.json
then i did: $ now
But it doesnt work, gives me this:
No command 'now' found, did you mean:
 Command 'nw' from package 'netrw' (universe)
 Command 'new' from package 'nmh' (universe)
 Command 'sow' from package 'ruby-hoe' (universe)
 Command 'nop' from package 'graphviz' (main)
 Command 'cow' from package 'fl-cow' (universe)
now: command not found

Can you please check if other globally installed npm packages are accessible?
E.g. - try installing serve: npm install -g serve
If this doesn't work, npm's binary directory is not included in your path or the binaries aren't being linked to your binary directory correctly. In both cases, you need to adjust your npm config.

Oh! i tried the command you said, I got this:
No command 'serve:' found, did you mean:
 Command 'server' from package 'rsplib-legacy-wrappers' (universe)
serve:: command not found

I have no idea how to config npm to run now
Can you point me to help please @leo

@thinkwalnut Firstly, try installing Node.js again - it will set up the right paths. Try clicking the right green button here. Once the installation is completed, try installing serve or now again.

I am having the same problem.


"Caporal Alexa" it is a village in "SÂNTANA" locality, "ARAD" county

Steps 1
Acces to the link https://peviitor-ro.github.io/ui.orase/
Expected result
The page is opens

Steps 2
Click on the search bar and write "Caporal Alexa "
Expected result
In the drop down menu you find "Caporal Alexa, ARAD(SÂNTANA) "
Test Data
Caporal Alexa


Added Orvibo SM11 Door & Window Sensor

added Orvibo SM11 zigbee Door & Window Sensor
https://zigbee.blakadder.com/Orvibo_SM11.html

Thanks


Fix crash when showing error about missing profile

Description
Before:
/opt/chefdk/embedded/lib/ruby/gems/2.6.0/gems/chef-config-15.1.36/lib/chef-config/mixin/credentials.rb:92:in `load_credentials': undefined local variable or method `credentials_file' for #<ChefConfig::WorkstationConfigLoader:0x00007fe01c8321e8> (NameError)

After:
ERROR: CONFIGURATION ERROR:Profile xyz doesn't exist. Please add it to /Users/developer/.chef/credentials.

It looks like this was missed as part of the refactoring in
d730505bb0d53d14c7b005c756ed4993b95fdf94.
Signed-off-by: Andrew Neitsch<EMAIL_ADDRESS>Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Nice catch. Thanks


Cannot discard cards in discard phase

Hi :)
recently i encountered an issue in the discard phase where i cannot put my cards from my hand to the discard pile. When this issue occours I'm unable to pick cards and move them to any play areas. Im using firefox at the latest version.

Hi!
Firefox has a bug where it occasionally likes to forget how to drag/drop.
Due to the relatively small number of people using firefox I haven't explored this too much, but I will eventually.
Suggestion for the mean time is to use another browser

This should no longer be an issue as the reserve prompt now asks you to pick cards, rather than drag them


Pyperclip v1.8.1: KeyError: 'XDG_SESSION_TYPE'

pyperclip v1.8.1 throw KeyError: 'XDG_SESSION_TYPE' error when used with CMD2 v1.3.0 and above. (pyperclip v1.8.0 is working perfect.)
Traceback (most recent call last):
  File "/usr/local/bin/expliot", line 5, in <module>
    from expliot.expliot import EfCli
  File "/usr/local/lib/python3.8/dist-packages/expliot/expliot.py", line 4, in <module>
    from expliot.core.ui.cli import Cli
  File "/usr/local/lib/python3.8/dist-packages/expliot/core/ui/cli/__init__.py", line 5, in <module>
    from cmd2 import Cmd, with_argument_list
  File "/usr/local/lib/python3.8/dist-packages/cmd2/__init__.py", line 30, in <module>
    from .cmd2 import Cmd
  File "/usr/local/lib/python3.8/dist-packages/cmd2/cmd2.py", line 73, in <module>
    from .clipboard import (
  File "/usr/local/lib/python3.8/dist-packages/cmd2/clipboard.py", line 14, in <module>
    _ = pyperclip.paste()
  File "/usr/local/lib/python3.8/dist-packages/pyperclip/__init__.py", line 680, in lazy_load_stub_paste
    copy, paste = determine_clipboard()
  File "/usr/local/lib/python3.8/dist-packages/pyperclip/__init__.py", line 568, in determine_clipboard
    os.environ["XDG_SESSION_TYPE"] == "wayland" and
  File "/usr/lib/python3.8/os.py", line 675, in __getitem__
    raise KeyError(key) from None
KeyError: 'XDG_SESSION_TYPE'

This issue was fixed by #177 and the release of pyperclip version 1.8.2

pyperclip version 1.8.2 resolved the issue with ubuntu 20.04 and python 3.8.5


Create new eth-holesky namespace

holesky gets released on 15th Sept 2023 and will replace goerli testnet.

waiting for new erigon v to fix https://github.com/ledgerwatch/erigon/issues/8210

This has been postponed for at least another 2 weeks - https://www.coindesk.com/tech/2023/09/17/ethereums-holesky-testnet-fails-to-launch-in-rare-tech-misstep-for-the-blockchain/

erigon processing new payloads, nimbus up to date with slots and proxyd handling requests well


Coerce types only in uri parser

This PR moves all type coercing into the URI parsers and makes sure it's only done once for each code path.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

28 of 28 (100.0%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.009%) to 90.14%

Files with Coverage Reduction
New Missed Lines
%

connexion/exceptions.py
2
93.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
3099

Relevant Lines:
3438

💛  - Coveralls


Updating CDK to Use CDK2

Description
Took original PR: #13 and updated the code to use CDK2. Would like some feedback first so submitting as a draft.
Issues Resolved
#13
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I'm closing this PR. We should use https://github.com/opensearch-project/opensearch-cluster-cdk to build a cluster using CDK.


[4.2] IRGen: Make sure we compare equal units in type size comparison

This is in some code to initialize variables for the debugger.
From the original commit
"Zero-initialize uninitialized Dictionary variables for the debugger.
To avoid the debugger displaying garbage or having expressions crash
when inspecting an uninitialized dictionary variable, zero-initialize
the first word of the alloca at -Onone.
"
Unfortunately, we are comparing different quantities in that code.
rdar://40043512

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 93ba990e2dba167718116653821062c478cde4b0


Removed agent, documented how to use it as sidecar

Similar to jaegertracing/jaeger-openshift#35

Thanks!


Updates StrawberryField documentation warning to include stable fields

Description
Updates the stability warning on the field extensions documentation page to include stable features. Confirmed with @patrick91 that the type, python_name, and arguments features are stable (discord thread).
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[x] Documentation

Issues Fixed or Closed by This PR

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Thanks for contributing to Strawberry! 🎉 You've been invited to join
the Strawberry GraphQL organisation 😊
You can also request a free sticker by filling this form: https://forms.gle/dmnfQUPoY5gZbVT67
And don't forget to join our discord server: https://strawberry.rocks/discord 🔥


Google-bigquery: Enable sqlCrudOperations test once Camel version is 18.3

Google-bigquery tests require a fix https://github.com/apache/camel/pull/8162 to work completely.
Until Camel 18.3 is adopted, test sqlCrudOperations has to stay disabled.

The sqlCrudOperations test fails with Camel 3.18.3.
Maybe the tests need adapting. I see some commits relating to google-bigquery are not present in 3.18.x. For now I'm going to disable sqlCrudOperations on the 2.13.x branch.

Fixed in the 2.13.2 release from https://github.com/apache/camel-quarkus/pull/4290.


No network resource in context if not specified in its JSON configuration

The user might provide his/her own libvirt network and attach the domain to it via its XML configuration.
In such cases, the Context object will have the network resource still set to None. This is due to the fact that the Context Factory is not aware of XML specific configurations.
This lead to bugs in the network based Plugins as they assume the presence of a libvirt network but they cannot reach it.
Possible solutions are:

If no network is specified in the Context's JSON configuration, parse the XML and try to understand if the domain is attached to any network. This is a bit controversial as we try to be as much "libvirt config agnostic" as possible.
Once the domain is created, try to lookup to which network is attached. This solution is simpler but requires libvirt APIs support. Investigation required.

Something to start from:
http://libvirt.org/html/libvirt-libvirt-domain.html#virDomainInterfaceAddresses

Fix merged.


[predict_modality] add Simple MLP from Neurips 2021

Describe your changes
Issue ticket number and link
Closes #xxxx (Replace xxxx with the GitHub issue number)
Checklist before requesting a review

[ ] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[ ] Minor changes
[ ] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md

[ ] CI Tests succeed and look good!

limitations:

Unable to test due to GPU requirements.
For now too many code changes are required to make it working.

Superseded by #441


Add test cases to verify closed record to map assignment

Purpose
The issue #17892 is fixed in PR #19262. Therefore here, some test cases are added.
Fixes #17892
Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #20083 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #20083   +/-   ##
================================================
  Coverage            20.51%   20.51%
================================================
  Files                   52       52
  Lines                 1394     1394
  Branches               216      216
================================================
  Hits                   286      286
  Misses                1095     1095
  Partials                13       13

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 b671c7f...559ad77. Read the comment docs.


freetds: fix broken url

@jagajaga
Build tested on nixos 14.12.732.bc4f185 (Caterpillar)

Looks good, and php still builds.


Enhancement for Syntax highlighting

Description
Actual there is no syntax highlighting for the commands #START/#LOCK/#UNLOCK and for the V.I-Variables supported. This should be enhanced in the extension.

Closed with https://github.com/isg-stuttgart/vscode-isg-cnc/pull/153


Excluding some point

Hello, Thanks for this library!
I'm rendering various points on my map for some points of interest but I'm also rendering the user location.
I don't want the user location point to be clustered nor counted in clusters…
I would like to exclude it from clustering, how can I proceed?
Maybe we could expect a nonClusteredMarkers property?
Thanks :)

https://github.com/venits/react-native-map-clustering/blob/master/lib/helpers.js#L10
Just add cluster={false}

https://github.com/venits/react-native-map-clustering/blob/master/lib/helpers.js#L10
Just add cluster={false}

Was not documented, thanks.
I switched to https://github.com/JiriHoffmann/react-native-clusterer which is more performant and easier for this use case


Evaluate you model failed

Hi @Randl ,Thanks for your work.
When I evaluate the model in results folder using following command:
python imagenet.py --dataroot ~/data/ImageNet/ -e --resume results/mobilenet_v2_1.0_224/model_best.pth.tar
Output is this:
......
number of parameters: 3505960
FLOPs: 312339328
=> loading checkpoint 'results/mobilenet_v2_1.0_224/model_best.pth.tar'
Traceback (most recent call last):
File "imagenet.py", line 160, in
main()
File "imagenet.py", line 147, in main
model.load_state_dict(checkpoint['state_dict'])
File "/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 769, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for MobileNet2:
Missing key(s) in state_dict: "conv1.weight", "bn1.running_var", "bn1.bias", "bn1.weight", "bn1.running_mean", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.conv1.weight", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.bn1.running_var", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.bn1.bias",..........
Please tell me what's going wrong?Thanks.

in cpu mode got above error.
set --gpus get accuracy:
number of parameters: 3505960
FLOPs: 312339328
=> loading checkpoint 'results/mobilenet_v2_1.0_224/model_best.pth.tar'
=> loaded checkpoint 'results/mobilenet_v2_1.0_224/model_best.pth.tar' (epoch 401)
100%|██████████| 782/782 [10:13<00:00,  2.01it/s]
Test set: Average loss: 1.1362, Top1: 36043/50000 (72.09%), Top5: 45257/50000 (90.51%)
A little different with readme.md: top1 72.10%, top5 90.48%

@xiaozhuka Hi, I have the same problem,Have you solved it?

Hi @Randl ,Thanks for your work. When I evaluate the model in results folder using following command:
python imagenet.py --dataroot ~/data/ImageNet/ -e --resume results/mobilenet_v2_1.0_224/model_best.pth.tar
Output is this:
......
number of parameters: 3505960
FLOPs: 312339328
=> loading checkpoint 'results/mobilenet_v2_1.0_224/model_best.pth.tar'
Traceback (most recent call last):
File "imagenet.py", line 160, in
main()
File "imagenet.py", line 147, in main
model.load_state_dict(checkpoint['state_dict'])
File "/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 769, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for MobileNet2:
Missing key(s) in state_dict: "conv1.weight", "bn1.running_var", "bn1.bias", "bn1.weight", "bn1.running_mean", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.conv1.weight", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.bn1.running_var", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.bn1.bias",..........
Please tell me what's going wrong?Thanks.

@xiaozhuka Hi, I have the same problem,Have you solved it?

Hi @Randl ,Thanks for your work. When I evaluate the model in results folder using following command:
python imagenet.py --dataroot ~/data/ImageNet/ -e --resume results/mobilenet_v2_1.0_224/model_best.pth.tar
Output is this:
......
number of parameters: 3505960
FLOPs: 312339328
=> loading checkpoint 'results/mobilenet_v2_1.0_224/model_best.pth.tar'
Traceback (most recent call last):
File "imagenet.py", line 160, in
main()
File "imagenet.py", line 147, in main
model.load_state_dict(checkpoint['state_dict'])
File "/.local/lib/python2.7/site-packages/torch/nn/modules/module.py", line 769, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for MobileNet2:
Missing key(s) in state_dict: "conv1.weight", "bn1.running_var", "bn1.bias", "bn1.weight", "bn1.running_mean", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.conv1.weight", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.bn1.running_var", "bottlenecks.Bottlenecks_0.LinearBottleneck0_0.bn1.bias",..........
Please tell me what's going wrong?Thanks.


Update README file

This change is

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

Comments from Reviewable

Coverage remained the same at 48.352% when pulling 861d1e9538ee90c6e2fdfed727614bf047bd05bb on fix/update_readme_cont into d2fdece28faea9f1f17c4a03fdd5ad8d55ee6be2 on dev.