Using banner disables usage & synopsis

Example option block :
opts = Trollop::options do
  version 0.1

  usage "foo|bar"
  synopsis "Reticulate splines"

  banner "Recognized options: "
  opt :dry_run, "Don't do anything", short: "-n"
end

Output when passed --help is :
Recognized options:
  -n, --dry-run    Don't do anything
  -v, --version    Print version and exit
  -h, --help       Show this message

If I comment just the banner line, I get this :
Usage: test foo|bar
Reticulate splines

0.1
Options:
  -n, --dry-run    Don't do anything
  -v, --version    Print version and exit
  -h, --help       Show this message

Additionally, I get warnings (the shebang line is #!/usr/bin/env ruby -W) depending on whether the usage or synopsis lines are comment out :
$gem-path/trollop-2.1.2/lib/trollop.rb:480: warning: instance variable @usage not initialized
$gem-path/trollop-2.1.2/lib/trollop.rb:481: warning: instance variable @synopsis not initialized

@tiennou  I agree that the behavior is not intiutive, however it is documented in the Wiki with some examples:
https://github.com/ManageIQ/trollop/wiki/Banner%2C-Synopsis-and-Version
The warnings should be fixed in PR #80

Is there a suggestion on how it should behave?

I suggest that banner, being an alias of text, shouldn't prevent usage & synopsis from working, so we get to have both nice, automated synopsises as well as free-form text. Would that be acceptable ?
From a backward-compatibility POV, I don't think it would have a big impact, seeing as it's either banner/text OR usage/synopsis, you would have skimmed over the former as soon as you were to use the latter.

I'm mixed on this one.
I had thought that banner is used to define your own header.
If you didn't provide it, the system would create one using version, usage and synopsis
@Fryguy you see any downside to always producing header information even when banner is added.


Inconsistencies between static methods and instances

This is working for me:
let fileURL = Bundle.main.url(forResource: "MyMusic", withExtension: "mp3")!
Sound.play(url: fileURL) // Music plays

While this is not
let fileURL = Bundle.main.url(forResource: "MyMusic", withExtension: "mp3")!
let mySound = Sound(url: fileURL)!
mySound.play()

I'm running both these snippets in the exact same spot in my code, so I have no idea why one would work and the other wouldn't.  May very well be user error, but I think I'm following the documentation correctly.

I fixed the issue by moving the reference to the Sound instance to a higher scope

Same for me. It is not working.
let mySound = Sound(url: fileURL)
mySound.play()

Same for me. It is not working.
let mySound = Sound(url: fileURL)
mySound.play()


[#17] Add GitHub templates

These templates are more or less copied from the Zcash Android SDK with minor modifications (removing reference to screenshots, Android, or UI manual tests)

I'll delete the reference to Architecture.md for now.  That'll require a bit more work to refactor the existing docs, which I'll plan to do in a followup PR.
Yes, agree that we mostly care about the links working when filing issues or opening PRs.  I think the links will be broken until we have the first version of this merged to the master branch.


Implemented a guard against invalid values of getPpiX in AbstractGraphics

Gdx.graphics.getDensity() returns infinity inside VirtualBox #6537
This way, AbstractGraphics' getDensity will always return 1f as a default value (instead of Infinity).
By the way, this is my first Pull Request for LibGDX ✨

Normally I'd mention that AndroidGraphics and IOSGraphics override getDensity(), and so could in theory have different behavior than what's documented -- except they also default to a density of 1 already, making this even better. The only place where the documentation might not be 100% accurate is in LwjglGraphics, because it allows an overrideDensity setting in its configuration, but because that is already documented and you'd have to look for overrideDensity to change it, I think we're fine.
Since this is a small change and fixes an annoying bug, I see no reasons that would prevent merging.

I think the fix should be on the root of the issue that is getPpiX() and getPpiY() returning 0 instead of setting an arbitrary density (not allowing differentiating if it's actually the real value or default). On iOS we already have a mechanism to allow the user to define the strategy to follow when ppix or ppiy are unknown (have a look at IosApplication#guessUnknownPpi()). Once set, any other value depending on ppi is then calculated accordingly. Maybe we should add a similar mechanism to the affected backends or, if it affects most backends, consider handling it in the abstract class.

I agree that would be better, but I think as a first step it is a good change if density defaults to 1 if unknown.


Create tags in this repo

In my opinion it would be nice if this repo gets a tag when a new version of the operator is released.
Now 5.4 is released of the operator there should be a tag / release called 5.4, then if you want to install an older version of the operator you know which CRD's, image versions and value files were provided in that release. Now if you want to install an older version based on these examples you need go back to specific commit.

Hi @PaulVerhoeven1
Thank you for reaching out. The examples repo is designed to allow users to easily take PGO out for a test drive, as explained in our quickstart. As such, we expect users to update these examples or create their own manifests as needed. To read more about release tags, please see our FAQ. Hope this helps!


The verification code type has been changed. Please solve it quickly.

The verification code type has been changed. Please solve it quickly.

If you want a fully comprehensive bot, I can help. Feel free to message me on Discord.:newqne


MinCount Sketch link is broken

I'm getting 403 when try http://www.eecs.harvard.edu/~michaelm/CS222/countmin.pdf, the address associated with "MinCount Sketch" at doc/introduction.txt.

@r-gaia-cs Welcome to the Sprint and thanks for the report. I'll look for a replacement.

Does http://dimacs.rutgers.edu/~graham/pubs/papers/cm-full.pdf work for you? It matches the version on the WayBack machine: https://web.archive.org/web/20120401091555/http://www.eecs.harvard.edu/~michaelm/CS222/countmin.pdf

Does http://dimacs.rutgers.edu/~graham/pubs/papers/cm-full.pdf work for you?

@mr-c Yes.

Is it 'safer' to use the Wayback Machine link (and avoid problems in the future?)

I'm fine with the WayBack machine. The only thing better would be a
http://dx.doi.org/... URL.
On Thu Feb 19 2015 at 2:19:37 PM Luiz Irber<EMAIL_ADDRESS>wrote:

Is it 'safer' to use the Wayback Machine link (and avoid problems in the
future?)
—
Reply to this email directly or view it on GitHub
https://github.com/ged-lab/khmer/issues/778#issuecomment-75117102.

Please use Count-Min Sketch as in the Cormode paper and http://www.ncbi.nlm.nih.gov/pubmed/25062443, and add that as a reference.

Fixed in #782.


[POC] As a login user, I would like to get my notification list

Why
After successfully login, fetch notification from the API https://api.github.com/notifications.

Done in https://github.com/nimblehq/github-notifications-mac/issues/37


Update METADATA.yaml

@lannka for approval

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


Events?

Using videogular with 2fdevs plugins, how do you listen for events when videogular has started to play, paused, stopped, or completed a video?

Hello,
check out the API here.
If you are using cuepoints there are the onLeave, onUpdate and onComplete callbacks.

This is a good source of info: http://www.videogular.com/tutorials/videogular-api/

 has a directive &attribute option called vgUpdateState that can pass $state like so:

vg-update-state="onUpdate($state)

Yes, you can get the state with $state or you can read it directly in the API with API.currentState.
http://www.videogular.com/docs/#/api/com.2fdevs.videogular.controller:vgController


import-module dbachecks fails with offline install

On a 2012 R2 server running SQL 2014 with no internet access, I used the Save-Modules of Pester, dbatools, PSFramework, and dbachecks. Using import-module failed with this error:
PS C:\Windows\system32> import-module PSFramework
PS C:\Windows\system32> import-module dbatools
PS C:\Windows\system32> import-module Pester
PS C:\Windows\system32> import-module dbachecks
Exception calling "InvokeScript" with "4" argument(s): "Method invocation failed because [System.String] does not contain a method named 'foreach'."
At C:\Users\ux0msp02\Documents\WindowsPowerShell\Modules\dbachecks\dbachecks.psm1:11 char:12

else { $ExecutionContext.InvokeCommand.InvokeScript($false, ([scriptblock]:: ...

CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : RuntimeException

Versions are:
Pester 4.3.1
PSFramework <IP_ADDRESS>
dbatools 0.9.336
dbachecks 1.1.132
$PSVersionTable is
PSVersion                      4.0
WSManStackVersion              3.0
SerializationVersion           <IP_ADDRESS>
CLRVersion                     4.0.30319.42000
BuildVersion                   6.3.9600.18968
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0}
PSRemotingProtocolVersion      2.2
Some notes on my process:

The computer I ran Save-Module on was running PS 5.1.14393.2189, Build 10.0.14393.2189, PSRemotingProtocol 2.3, same everything else
I moved the folders to C:\Users\username\Documents\WindowsPowerShell\Modules
I had to move the modules up one folder level from their version number to have PS install them, e.g Modules\dbachecks\1.1.132*.* -> Modules\dbachecks*.*
Security is locked down pretty tight on these servers
Repeating import-module dbachecks returns without error, but fails to complete the installation (I think. Get-DbcCheck fails with not finding "...AppData\Local\dbachecks\checks.json'")

Which version have you saved?
Could you run $error[0] | fl - force and put the expanded error in here.

It was the 1.1.132 version saved from the PowerShell gallery.
Exception
System.Management.Automation.MethodInvocationException: Exception calling "InvokeScript" with "4" argument(s): "Method invocation failed because
[System.String] does not contain a method named 'foreach'." ---> System.Management.Automation.RuntimeException: Method invocation failed because
[System.String] does not contain a method named 'foreach'.
at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
at System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame frame) at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame) at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame) at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame) at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0) at System.Management.Automation.ScriptBlock.InvokeWithPipeImpl(ScriptBlockClauseToInvoke clauseToInvoke, Boolean createLocalScope, Dictionary2 functionsToDefine, List1 variablesToDefine, ErrorHandlingBehavior errorHandlingBehavior, Object dollarUnder, Object input, Object scriptThis, Pipe outputPipe, InvocationInfo invocationInfo, Object[] args) at System.Management.Automation.ScriptBlock.<>c__DisplayClassa.<InvokeWithPipe>b__8() at System.Management.Automation.Runspaces.RunspaceBase.RunActionIfNoRunningPipelinesWithThreadCheck(Action action) at System.Management.Automation.ScriptBlock.InvokeWithPipe(Boolean useLocalScope, Dictionary2 functionsToDefine, List1 variablesToDefine, ErrorHandlingBehavior errorHandlingBehavior, Object dollarUnder, Object input, Object scriptThis, Pipe outputPipe, InvocationInfo invocationInfo, Object[] args) at System.Management.Automation.ScriptBlock.DoInvokeReturnAsIs(Boolean useLocalScope, ErrorHandlingBehavior errorHandlingBehavior, Object dollarUnder, Object input, Object scriptThis, Object[] args) at System.Management.Automation.CommandInvocationIntrinsics.InvokeScript(ScriptBlock sb, Boolean useNewScope, PipelineResultTypes writeToPipeline, IList input, Object[] args) at System.Management.Automation.CommandInvocationIntrinsics.InvokeScript(Boolean useLocalScope, ScriptBlock scriptBlock, IList input, Object[] args) at CallSite.Target(Closure , CallSite , Object , Boolean , Object , Object , Object ) --- End of inner exception stack trace --- at System.Management.Automation.ExceptionHandlingOps.ConvertToMethodInvocationException(Exception exception, Type typeToThrow, String methodName, Int32 numArgs, MemberInfo memberInfo) at CallSite.Target(Closure , CallSite , Object , Boolean , Object , Object , Object ) at System.Management.Automation.Interpreter.DynamicInstruction6.Run(InterpretedFrame frame)
at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
TargetObject          :
CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : RuntimeException
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at Import-ModuleFile, C:\Users\ux0msp02\Documents\WindowsPowerShell\Modules\dbachecks\dbachecks.psm1: line 11
at , C:\Users\ux0msp02\Documents\WindowsPowerShell\Modules\dbachecks\dbachecks.psm1: line 34
at , : line 1
PipelineIterationInfo : {}
PSMessageDetails      :

OK, so all the only ForEach methods I can see are the ones in New-Json. Could you replace line 49 in New-Json
@($configs).foreach{$config += "$_ "}
and see if that works for you and if it does I will update

Okay that fixed that one. Once that was unblocked I started getting this:
System.Management.Automation.MethodInvocationException: Exception calling "InvokeScript" with "4" argument(s): "Invalid array passed in, ']' expected.
(1): [" ---> System.Management.Automation.CmdletInvocationException: Invalid array passed in, ']' expected. (1): [ ---> System.ArgumentException: Invalid array passed in, ']' expected. (1): [
at System.Web.Script.Serialization.JavaScriptObjectDeserializer.DeserializeList(Int32 depth)
at System.Web.Script.Serialization.JavaScriptObjectDeserializer.DeserializeInternal(Int32 depth)
at System.Web.Script.Serialization.JavaScriptObjectDeserializer.BasicDeserialize(String input, Int32 depthLimit, JavaScriptSerializer serializer)
at System.Web.Script.Serialization.JavaScriptSerializer.Deserialize(JavaScriptSerializer serializer, String input, Type type, Int32 depthLimit)
at Microsoft.PowerShell.Commands.JsonObject.ConvertFromJson(String input, ErrorRecord& error)
at Microsoft.PowerShell.Commands.ConvertFromJsonCommand.ProcessRecord()
at System.Management.Automation.CommandProcessor.ProcessRecord()
But I traced that to New-Json.ps1 line:
$Descriptions = Get-Content $ModuleRoot\internal\configurations\DbcCheckDescriptions.json | ConvertFrom-Json
I think the $ModuleRoot was giving it issue. I replaced it with:
".\internal\configurations\DbcCheckDescriptions.json"
and everything installed with out error.

Commenting to confirm that I was having the same issue and resolved by following these steps.
Made the following changes to New-Json.ps1.

Replace line 49 with @($configs).foreach{$config += "$_ "}
Replace line 87 with $Descriptions = Get-Content '.\internal\configurations\DbcCheckDescriptions.json' | ConvertFrom-Json

@kylejdoyle // @jeffjacobs0
I'm not at a computer to test but you should be able to update to the following on line 87:
"$script:ModuleRoot\internal\configurations\DbcCheckDescriptions.json' | ConvertFrom-Json
We set $ModuleRoot as a global variable and it looks like it just got missed referencing it properly. It is referenced as a global variable in this manner everywhere else.


Discovery and Alpaca Client Enhancements to ASCOM.Standard

Key changes:

Add the Alpaca clients project to the solution
Adds several new members to the response classes
Moved COM error numbers into their own class, away from Alpaca error numbers
Add components to retrieve and present Alpaca and ASCOM device information from devices discovered by Finder.
All packages up-leveled to alpha19

Overall this looks good to me. I would say merge when ready.


Custom Selectors sometimes generate REALLY unexpected output

I don't really know to explain this one except by example. It may be related to #177, which I just submitted, but it also might be unrelated.
Input:
@custom-selector :--buttons button, input[type="button"];
@custom-selector :--hovers :hover, .hover;

:--buttons:--hovers {
    background: color(#ff9800 shade(20%));
}

button:--hovers,
input[type="button"]:--hover {
    background: color(#ff9800 shade(20%));
}

Output:
:hover:--hovers,
.hover:--hovers {
    background: rgb(204, 122, 0);
}

:hover,
.hover
input[type="button"]:hover,

input[type="button"].hover {
    background: rgb(204, 122, 0);
}

Duplicate of #163 and postcss/postcss-custom-selectors#19 and #177

@topaxi Since this ticket (and #177) do seem related to the postcss issue, do we keep them open until that task is completed and we can test that these examples work as expected? Or do we close these as duplicates?

@awayken that is up to the maintainers of cssnext i guess :)

@topaxi All right. I'll leave my tickets as submitted for now, and I'll subscribe to the postcss ticket to get updates on progress.

Please follow #163.


Major code redesign

This PR is a major code redesign.
When trying to understand why some experiments were not being executed I ended up bouncing back and forth between the functions that had no documentation, no tests and no practical way of investigating what is actually being done.
In an effort to ensure the long term viability of the code, in consultation with an NMR expert we made dramatic changes to the code base making sure the output produced by the code was kept intact.
The major changes include:

Creation of a new way to represent the NMR experiments; previously each experiment had its own dedicated function
Redesigned how experiments are executed in a more generalist way; this opens the path for the easy addition of new experiments and even custom ones
Changed the input arguments from command-line to an input-file; there were too many arguments to be passed via the command line and some of them had inter-dependencies that were not explicit, doing this via an input file makes the parameters easier to understand and change when needed, besides being a reproducible.
To ensure the previous experiments were preserved, they were described (with the assistance of the NMR expert) in the newly created catalog.toml
Added much needed unittests and linting, also with actions via github actions.

With this new redesigned function we should now move forward to add integration tests (and ensure all functionalities are preserved) and user documentation - to be handled in a different PR.

Codecov Report

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

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage        ?   93.95%
=======================================
  Files           ?       13
  Lines           ?      513
  Branches        ?        0
=======================================
  Hits            ?      482
  Misses          ?       31
  Partials        ?        0

Continue to review full report at Codecov.

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


Add application configuration.

There should be a way to define switches, flags and default arguments via some kind of config files.

The config file will be called ref.config.yml or .ref.config.yml.
The config file will be loaded from the user's home directory first then from the project root.
Command line input will override any config settings.
Needs to be unit tested.

Started working on this here #2 . Needs to be tested and integrated into the system.


Change usage of followSurface to arcType. Add CZML property for it.

Change the existing usages followSurface to arcType.
CZML requires a new property to be added, which this PR also adds.
Fixes #7572

Thanks for the pull request @shehzan10!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@mramato Would you mind looking at this?

I would like to get this merged before the release.
cc @shunter, we need to add this new CZML property to czml-writer

This looks good to me! Thanks @shehzan10
@shunter let me know if there's anything else we need to do

FYI looks like this change causes a crash when loading any CZML file which uses followSurface.  I guess there are no tests whatsoever for that codepath?

oh no, thanks @shunter.  looking into that right now


GUACAMOLE-1794: Update KSM SDK to 16.5.3 to add support for "passkey" field type.

Looks good except that the ksm-sdk-16.5.2 directory needs to be renamed to match the version number.

Oops! fixed.


Reference to Block ID from the same file doesn't work

Reference to Block ID from the same file doesn't work. Seems to be related to #8.
Obsidian Version: 1.7.7
Strange New Worlds Version: 2.1.5

This should now be addressed with update 2.2.0. Thank you for reporting this issue.


TypeScript Error: Not a module

When importing TinySlider like the installation guide says with import { TinySlider } from "svelte-tiny-slider" I get the following TypeScript error message in VS Code:

"File '/Users/roythuis/Developer/Websites/frank-van-esser/node_modules/svelte-tiny-slider/dist/index.d.ts' is not a module.ts(2306)"

What version of Svelte are you using?

Hi Mitchel,
I just upgraded to Svelte 4 before installing this package. Here are my devDependencies:
"devDependencies": {
		"@sveltejs/adapter-auto": "^2.0.0",
		"@sveltejs/kit": "^1.20.4",
		"@typescript-eslint/eslint-plugin": "^5.45.0",
		"@typescript-eslint/parser": "^5.45.0",
		"eslint": "^8.28.0",
		"eslint-plugin-svelte": "^2.30.0",
		"sass": "^1.62.1",
		"svelte": "^4.0.0",
		"svelte-check": "^3.4.3",
		"svelte-preprocess": "^5.0.3",
		"tslib": "^2.4.1",
		"typescript": "^5.0.0",
		"vite": "^4.3.0"
	},

This should be fixed with 1.1.4. Thanks!


Add normative text to verification and processing section to verify public key

This PR does the following:

make verification that the public key of the Issuer-signed JWT belongs to the Issuer mandatory
introduce methods to perform this verification based on JWT Issuer Metadata, X.509 Certificates and DIDs

Potentially fixes #168, #143. Partially #134
Out-of-scope:

Renaming and moving JWT Issuer Metadata Section

Addressed all suggestions, please review again @danielfett


Test the GREL post processing script in Specify test installation

What is the issue ?
We need to make sure that the post processing works as intended and preceding this the Digi App output.
Detailed description of the issue.
The question is: Can the Digi App data export be transformed sufficiently to be successfully imported into Specify via workbench?
Why is it needed/relevant ?
This will be a measure of the post processing maturity.
Estimate level of effort required.
easy
What is the expected acceptable result.
A full validation of the post processing output in Specify workbench.
How to approach it?

[x] Create a dataset mockup in the Digi App
[x] Run it through post processing in OpenRefine
[ ] Load this to workbench
[ ] Validate in workbench and import
[ ] Check the import in Specify

What could be the challenges ?
There are two versions of the GREL script now and the correct version must be employed.

A number of issues came up this week:
#475
#474
Further testing of the GREL post processing script at this point makes little sense until these bugs are fixed.


Getting grid like pattern after image stitch many tiff files.

I have many microscope images. i have stitched all the images but getting line between the two images and one image overlap with other due to which my stitched images seems to be blured. i am attaching the image for your reference.

can you suggest me what i am missing. thanks in advance for helping.

I think you need the --flip-y option which flips the tiles before stitching.

I am again getting the same results as previous one. i am attaching my code.`
from ashlar import fileseries, thumbnail, reg
import numpy as np
from tifffile import imread, imwrite, TiffFile
readers = []
for i in range(3):
readers.append(fileseries.FileSeriesReader(
'/home/kuldeep/Downloads/Inferencing_with_Button/Input/',
pattern='image_{series:3}.jpg',
overlap=0.1,
width=11,
height=1,
layout='snake',
direction='horizontal', )
)
print(len(readers))
aligner0 = reg.EdgeAligner(readers[0], channel=0, filter_sigma=15, verbose=True)
aligner0.run()
mosaic_args = {}
mosaic_args['verbose'] = True
mosaic_args['flip_mosaic_y'] = True
aligners = []
aligners.append(aligner0)
mosaics = []
for j in range(1, 2):
aligners.append(
reg.LayerAligner(readers[j], aligners[0], channel=j, filter_sigma=15, verbose=True)
)
aligners[j].run()
print("aligners[0].mosaic_shape", aligners[0].mosaic_shape, aligners[0])
mosaic = reg.Mosaic(
aligners[j], aligners[0].mosaic_shape, None, **mosaic_args
)
mosaics.append(mosaic)
print(type(mosaic))
writer = reg.PyramidWriter(mosaics, r"/home/kuldeep/Downloads/Inferencing_with_Button/16*16.ome.tif",
verbose=True)
writer.run()
`

Sorry, I meant --flip-y on the command line, not --flip-mosaic-y which is what you used there -- they are subtly different. To implement flip-y in your script, add these lines after constructing the readers list:
for r in readers:
  r.metadata._positions = r.metadata.positions * [-1, 1]

(note this assigns back to _positions not positions)
You do not need to use mosaic_args['flip_mosaic_y'] = True at all.
However a bigger issue is that your example image appears to show a 17x17 grid but your FileSeriesReader arguments say width=11, height=1. It seems like you should actually be passing width=17, height=17 but perhaps there are other parts of your processing pipeline not shown here.
Would you mind sharing your images so I could take a closer look?

One other tip: to paste multiple lines of code in a comment here, wrap it in three back-quotes on each end, not one. Single back-quotes are for highlighting an inline code statement and do not work for multiple lines.

Thanks for the reply. Actually in the code it was 17*17. and it stitched well except grid like structure and second thing at stitched part pic is more blurred.
i am attaching the picture for your reference. due to size limit not able to attach 17x17. so just attached 17x3.
Input (17x3).zip
Thanks for your continue support.


w2sidebar menu sometimes shows scrollbars

When the context menu is brought up in a w2sidebar close to the frame edge, there seems to be an edge case where the menu will not move above the item and instead show scrollbars.
see http://jsfiddle.net/mtnsutso/1/
It takes some testing to repeat because it's based on window height, sidebar height, and current item position. Here is a screenshot showing an instance of this occuring in that jsfiddle.

Thanks.

There was a fix recently that would not show unnecessary horizontal scroll. But you are right. There is still an issue. I will keep it open for a while.

Thanks, vitmalina.  I had a look a while back but got pushed on to something else.  I'm hoping to have a look again, I'll let you know if I find something.

This issue seems to be fixed in 1.5
Updated fiddle: http://jsfiddle.net/mtnsutso/4/
I'd say, this issue can be closed.

I'm seeing an unwanted scrollbar in the pop-up menu in the sidebar. I've tried overriding the css for div.menu but that doesn't help. I think this css is being added by the javascript on menu creation or something like that.
How can I get rid of this vertical scrollbar?

@camainc  Can you please provide a Fiddle/Bin/Pen to demonstrate the issue?

@mpf82
Hello, I have the same problem on a demo page.
Tested on Chrome 90.0.4430.93 and Opera 75.0.3969.243.

Is browser zoom an 100%?

At least for me, this seems indeed zoom related.
I don't see a scrollbar in the http://w2ui.com/web/demos/#!toolbar/toolbar/2 demo for zoom 100%
However, for zoom 90% or 150% the scrollbar is visible, while it looks fine for most other zoom level.
Tested in Chrome 88.0.4324.150
In my Firefox 88.0 only zoom level 130%, 200%, and 230% did show scrollbars, all other zoom levels (at least those that you can switch through with CTRL+wheel) did look fine.

@vitmalina No zoom in browser, but OS has 200% zoom, because of 4K screen.
Yes, if I zoom out in browser (to 90%), scrollbars disappear.


EventBus ports, should all be in VertxOptions and should be documented how they work

For EventBus, you have the VertxOptions.setClusterPort() and setClusterHost() but it is not clearly documented how a port is selected without these (random port, what range, ...?)  And for overriding the public published host/port for the cluster it changes to be system properties instead of VertxOptions
So, add setClusterPublicPort and setClusterPublicHost to VertxOptions and document these settings for how they work.  Knowing about vertx.cluster.public.port and vertx.cluster.public.host doesn't appear in documentation for vert-x3.  And that they override the other setClusterPort() and setClusterhost()
This is important for things like Docker where your internal port/host may not be the same as the public.  We are doing a Docker+EC2 discovery (ECS) and it was not obvious where/how to find these properties and how they work.

Hi Jason:

For EventBus, you have the VertxOptions.setClusterPort() and setClusterHost() but it is not clearly documented how a port is selected without these (random port, what range, ...?)

The default values for cluster host and cluster port are documented in the javadoc for VertxOptions:
http://vertx.io/docs/apidocs/io/vertx/core/VertxOptions.html

And for overriding the public published host/port for the cluster it changes to be system properties instead of VertxOptions

VertxOptions doesn't know anything about system properties. Are you referring to running from the command line? You should use -clusterHost and -clusterPort to specify cluster host and cluster port when doing this.

@purplefox referring to VertxImpl line 159..166
                // Obtain system configured public host/port
                int publicPort = Integer.getInteger("vertx.cluster.public.port", -1);
                String publicHost = System.getProperty("vertx.cluster.public.host", null);

                // If using a wilcard port (0) then we ask the server for the actual port:
                int serverPort = publicPort == -1 ? server.actualPort() : publicPort;
                String serverHost = publicHost == null ? options.getClusterHost() : publicHost;
                ServerID serverID = new ServerID(serverPort, serverHost);

Not referring to running from the command-line, referring to any time that you need different public port/host than VertxOptions setClusterPort/setClusterHost.

Hmm.. I actually don't know what "vertx.cluster.public.port" and "vertx.cluster.public.host" are supposed to do. I'm not sure how this code got added.
Will investigate.

I agree however this should be in VertxOptions (once I've figured out why it's here in the first place!).

They are needed for cases like this, AWS, etc. where the port+host you want to listen on, are not the same as other cluster nodes require to communicate with you.  (private  vs. public address.  docker host vs. docker container address/port)

On documentation, it is helpful for people that have containers or firewalls to know how ports are allocated.  So in the place it describes event bus, clustering, etc.  A quick description of how ports are assigned is handy, before you have to find that VertxOptions has the special setting you are looking for, which doesn't describe if ports are chosen in a reliable way (i.e. ElasticSearch starts at 9200 and increments, 9300 and increments) or some random other method.


Use getpass module to get the user

Using the getpass module instead which is an OS Agnostic approach to getting the running user.
https://docs.python.org/3/library/getpass.html

Fixes https://github.com/timmo001/system-bridge/issues/2994

If you push a new system-bridge deb package i can test it here to verify the fix works.

Ruff looks like temp issue with the github api to me. Maybe a retry will fix it. And the lint check can be fixed with fix parameter.


BUG: find_peaks does not detect maximal peak if it's at the start of array

Hello,
I noticed a bug in Scipy while using the find_peaks function.
Namely, if I have a signal/array where I want to detect peaks higher than X % of the maximum, the returned peaks sometimes lack the actual maximum. This happens when the maximum is at the very start of the array -- e.g., index 0.
The function call looks something like the code below
I think it's because the peak picking logic tries to check that neighbouring peaks are of lower value/amplitude, but this is not possible for the start/end positions in the array. I would suggest counting a value at the start/end of an array as a peak if the values in the respective one-sided (instead of two-sided) neighbourhood are lower.
Best
Reproducing Code Example

currentPeakIndices = scipy.signal.find_peaks(currentSignal, height=percentageThresshold*max(currentSignal))[0]

Error message
N/A

SciPy/NumPy/Python version information
1.6.2 1.20.1 sys.version_info(major=3, minor=8, micro=8, releaselevel='final', serial=0)

Hi, thanks for the report. This is known bug already reported here #13710. Closing this issue in favour of the old one which had some activity. Feel free to come over there and comment.


Validation issue with<PHONE_NUMBER> (AS)

The result given by libphone is
PhoneNumberOfflineGeocoder Result sLocation American Samoa
PhoneNumberToTimeZonesMapper Results Time zone(s)   [America/Chicago]
The time zone should have been Samoa Time Zone (UTC-11:00) but it is giving America/Chicago (UTC-06:00) as timezone. Is this issue fixed. How can I get the correct timezone.
Please read the "guidelines for contributing" (linked above) and fill in the template below.
Country/region affected (e.g., "US"): US
Example number(s) affected ("+1<PHONE_NUMBER>"):<PHONE_NUMBER>
The phone number range(s) to which the issue applies ("+1 555 555-XXXX"):
The type of the number(s) ("fixed-line", "mobile", "short code", etc.):
The cost, if applicable ("toll-free", "premium rate", "shared cost"):
Supporting evidence (for example, national numbering plan, announcement from mobile carrier, news article): IMPORTANT - anything posted here is made public. Read the guidelines first!
link to demo

The affected region is AS, not US.
Someone from the team should look at this shortly.

Sorry no one responded to this!
We update timezones semi-automatically and only once in a while, sorry for the delay.
We're mapping 1684 to Pacific/Pago_Pago and this should go out in the next release. Thanks for your patience!

Fixed in release 8.3.1.


change Password failed : [LDAP: error code 50 - Insufficient Access Rights]

I'm using https://github.com/darabi/docker-fusiondirectory-ldap with fusion directory but when i'm tryin to change a user attribute from an application usion my admin user I got nsufficient Access Rights.
dn: cn=admin,ou=aclroles,dc=,dc=*,dc=com
cn: admin
description: Give all rights on all objects
objectClass: top
objectClass: gosaRole
gosaAclTemplate: 0:all;cmdrw
any help !...thx!

SOLVED-by adding acl role (editowninfo to all my users), so no need the admin to modify the an specific user attribute. but still don't understand why the admin could not


WIP: Support scrolling (Overflow::Scroll) in bevy_ui

Objective
Allow users to create scrollable regions using bevy_ui.
Fixes #8074
Prior Art
https://github.com/Piefayth/bevy_ui_scroll
Solution
This is adapted from the scrolling implementation in the examples/ui/ui.rs example, but is improved in that it does not use the style system in order to apply offsets (which requires a relayout each time the scroll position changes, and interferes with users wanting to use that same style):

A new Overflow::Scroll variant has been added to the Overflow enum to mark nodes that should be scrollable
A system has been added that responds to mousewheel events and updates the scroll position
Nodes are laid out like usual (layout using Taffy is not affected)
The position of nodes is offset by the scroll position of their parent after layout while performing transform change detection

Requirements from Taffy:

[x] https://github.com/DioxusLabs/taffy/pull/424 (Correctly lay out Overflow::Scroll/Overflow::Hidden nodes)
[x] https://github.com/DioxusLabs/taffy/pull/446 (Support reserving space for scrollbars for Overflow::Scroll nodes)
[x] https://github.com/DioxusLabs/taffy/issues/470
[ ] Track which nodes' scroll heights have changed in a given layout computation (for efficiently updating scroll_height in bevy on relayout only when actually necessary)
[ ] Release new version

Tasks

[x] Add Overflow::Scroll variant to Overflow enum
[x] Add ScrollPosition component
[x] Add update_scroll_position system
[x] Only scroll the item under the cursor
[x] Horizontal scrolling
[x] Separate Overflow in each axis (https://github.com/bevyengine/bevy/pull/8095)
[ ] [Taffy] Support for scrollview container that have multiple children (currently you need a dummy wrapper node)
[ ] Update scrolling on relayout (height of node or contents may have changed)
[ ] Make ScrollPosition component optional for ui nodes to avoid checking every node on scroll
[ ] AccessKit (need to export scroll_x/scroll_y and scroll_x_max/scroll_y_max to accessibility tree)
[ ] Touch-based scrolling for mobile platforms
[ ] [Taffy] Scrollbars
[ ] Nested scrollviews

Changelog

Add support for Overflow::Scroll

@nicoburns Question I have is, how will games control the appearance of the scrollbars? On the web, the ability to customize the look of scrollbars is quite limited (more so in some browsers than others), and the web community has generally punted on this issue by converging on a fairly common (and boring) visual design for scrollbars which nearly every UI toolkit adheres to. However, games want novel and fancy UI designs, and I think scrollbars are no exception. Granted, not many game HUDs have scrollbars, but there are exceptions:

Chat logs in multiplayer games
Quest and Lore pages
Character bios in RPGs
Help pages
etc.

Editors and other tools built on Bevy UI are of course going to have lots of scrollbars, but their need for visual novelty is not quite as acute.
A related question is the API for controlling scroll position. "Scroll selected item into view" is probably the most common of these. If scrolling is just mucking around with the node transform, then user code which explicitly sets the translation is probably going to mess things up. We really ought to define an API to let user code do the standard things (scroll to top, scroll to bottom, show selected, etc.) without having to directly touch the node transforms.
As someone in the process of writing a UI framework, I'm perfectly prepared to do my own "scrollview" widget implementation if that's what it takes. I know that on the web, scrolling is a style property that is inherent in every element, but that is not the case in desktop toolkits such as Java Swing or Qt, where scrolling is a specific widget type. I actually think that model may be more appropriate, given the general rarity of scrolling views in games, but the question then becomes where do we draw the line between the responsibilities of bevy_ui and the responsibilities of the toolkit?

Is there anything new to announce?

Closing as adopted <3


Question: Where can I get a dist for script tag use, perhaps a CDN version / self hosted version

Hi
This looks awesome. Where can I get a single dist file that I can just use on the script tag?
For example nedb (one you forked) had files in the out folder named nedb.js that one can use simply.
Thanks you so much

Hello @Mashiane .
There is also such a folder in this fork. You should be able to use the file /browser-version/out/nedb.min.js from any CDN.
(this file is present in the NPM package, not in the GIT repo, as it is bad practice to publish built files to GIT).


"uv venv" doesn't find python (Linux)

When trying out uv, I wanted to try install a package to the global space. Which isn't possible apparently, but ok, let's make a virtual environment like the command suggested to me: uv venv
Result:
    × Neither `python` nor `python3` are in `PATH`. Is Python installed?

Yes, Python is installed. Running python3 --version works. which python3 returns /usr/bin/python3. echo $PATH returns ...:/usr/bin/:... so python definitly is in my path. I don't really know what's going wrong here.
I'm on Zorin OS 17 (derivative of Ubuntu), current python version is 3.10.12.
Is there some more information I can give?

Thanks for the report! I'll be looking into this. We've never seen this bug so I'll have to do some digging...

Can you share the output of uv venv -v?

Fixed in #1351

Downloaded the new version and it works, awesome!


promote: Allow providing app-interface checkout directory

Add argument to pass in app-interface checkout directory
Use default location if none is passed in
Don't silently fail if osdctl is called outside of a git repository

Jira: OSD-20520

/lgtm

/retest

/lgtm


Clone spec mapping for common tools

Hello!
The iReceptor team is hoping to load some clones data, so we've been reviewing the clones spec.
I've created a mapping file with my best guess at mapping clones output from MiXCR, Change-o, 10X, and ImmuneDB.
clone.mapping_with_counts_examples-2021-10-29.xlsx
Hoping for a review from the community to see if we are in agreement. Some specific questions that came out of this exercise:

It seems that most tools do not include v/d/j alignment start/end positions in the output. Am I missing something here - maybe a data processing step or a command-line argument?

For single-cell data, it became clear we'll need to resolve this issue: extend clones to the single-cell context. Specifically for 10X data, clone_id comprises multiple loci, so we should decide if/how to split this out. We suggest modifying clone_id as follows (example is for BCR data):

change clone_id = “CLONEA” to three clones with clone_id = “CLONEA_IGH”, clone_id = “CLONEA_IGL”, and clone_id = “CLONEA_IGK”.

In the same vein, we'll need to consider how to count clones derived from single-cell data. I came up with 3 options; option 1 (aggregating clone counts across loci) isn't very useful; option 2 (locus-specific clone count) makes more sense and can be implemented fairly easily; option 3 (cell-specific clone count) is most informative but might be trickier to implement right now, as it links to the cell spec. Any thoughts on how best to proceed?

Thanks!

Would it be so difficult to link to the Cell object? The splitting (points 2+3) does not appear to be too meaningful to me. For example, assume you have (single-cell defined) clones in a mouse that use the same IGL rearrangement. If you split those you will either have two IGL "clones" with identical chains, which in a non-single-cell context would appear as one, or a single merged IGL "clone" of which you know that it does not exist. In either case, the cell-based grouping would be lost and require re-processing of the data.

change clone_id = “CLONEA” to three clones with clone_id = “CLONEA_IGH”, clone_id = “CLONEA_IGL”, and clone_id = “CLONEA_IGK”.

This would be an incorrect definition of clone_id, because the "clone" has two chains. This should just be "CLONEA" if you have paired heavy and light chain data. Also, if you were to annotate the chain in the clone_id column like this, I think using _VH and _VL would be more meaningful - for the most part, you shouldn't have both a kappa and lambda chain.
I agree with @bussec on splitting 2+3.  The answer shouldn't differ in cases where you have the heavy+light chain and have resolved doublets / allelic inclusion.
For Change-O/SCOPer specifically, you can count clones by just skipping the light chain row (IGK or IGL) for the clone; ie,  count only on the IGH data. The algorithm uses both the heavy and light chain to define clones, but the heavy chain is required.

Would it be so difficult to link to the Cell object?

No, but this requires us to change the spec such that the Clone object to link to the Cell object (#317) and that has become dormant (last comment over a year ago). So maybe this is the driver to make that change...
The suggestions @kira-neller made were to map clone annotation tools to the AIRR Clone spec. Nothing like trying to use a spec to poke holes in it 8-)

change clone_id = “CLONEA” to three clones with clone_id = “CLONEA_IGH”, clone_id = “CLONEA_IGL”, and clone_id = “CLONEA_IGK”.

This would be an incorrect definition of clone_id, because the "clone" has two chains. So "CLONEA" if you have paired heavy and light chain data. Also, if you were to annotate the chain in the clone_id column like this, I think using _VH and _VL would be more meaningful - for the most part, you shouldn't have both a kappa and lambda chain.
I agree with @bussec on splitting 2+3. The answer shouldn't differ in cases where you have the heavy+light chain and have resolved doublets / allelic inclusion.
For Change-O/SCOPer specifically, you can count clones by just skipping the light chain row (IGK or IGL) for the clone; ie, count only on the IGH data. The algorithm uses both the heavy and light chain to define clones, but the heavy chain is required.

So if I understand correctly, one should use the clone_id as assigned by the single cell tools. This means that for a specific clone, if a tool does single cell, there are both VH and VL rearrangements and they will have the same clone_id (and presumably the same cell_id), and that is good and we don't want to break that.
The problem then becomes counting and we need to come up with a mechanism for counting clones for each tool. I think that because our Clone object is not well enough defined in relation to Cells this is ambiguous at the moment. Which is exactly why @kira-neller posted the above... So is this true:
clone.sequence_count = sum(rearrangement.clone_id)  - this would include VH and VL sequences...
clone_abundance is then somewhat more complicated based on how the tool calculates clones.

For Change-O/SCOPer clone.clone_abundance = sum(rearrangement.clone_id where rearrangement.locus == IGH)

We would need a rule for each tool in this case?

We would need a rule for each tool in this case?

I don't have a good sense of what the various new single-cell tools that have cropped up recently are doing, so without reviewing the state of the field, I'm not sure, but...
If we're not using the Cell object, and just using Rearrangement, then I assume the general rule of one clone_id per cell_id would hold and you can use that across tools. Ie, count the number of unique cell_id per clone_id.

This means that for a specific clone, if a tool does single cell, there are both VH and VL rearrangements and they will have the same clone_id (and presumably the same cell_id), [...]

Agreed. I cannot think of a situation in with a clone_id and cell_id would diverge. Also -- for any given clone inference -- a cell can only be a member of zero or one clone. (I hope such a deterministic view does not cause a Bayesian outcry :wink: )

So is this true:
clone.sequence_count = sum(rearrangement.clone_id) - this would include VH and VL sequences...

Looks ok to me. However, the usefulness of clone.sequence_count might be reduced in a single-cell context, if tools perform consensus builds and you end up with few unique sequences. In that case clone.clone_abundance would be the more useful parameter.
One more question: Would any of these clone statistics exclude out-of-frame rearrangements?

One more question: Would any of these clone statistics exclude out-of-frame rearrangements?

I think most tools exclude non-productive rearrangements prior to assigning clones, right?

We would need a rule for each tool in this case?

I hope not. I don't have a good sense of what the various new single-cell tools that have cropped up recently are doing, so without reviewing the state of the field, I'm not sure, but...
If we're not using the Cell object, and just using Rearrangement, then I assume the general rule of one clone_id per cell_id would hold and you can use that across tools. Ie, count the number of unique cell_id per clone_id.

But we aren't guaranteed that every tool that produces a clone_id will produce a cell_id are we? I could see how that might work for a single cell pipeline, but what about the bulk AIRR seq clone tools that don't know anything about cells (e.g. MiXCR, ImmuneDB, and even the Change-O non paired pipeline)? They simply assign a clone_id, no?

But we aren't guaranteed that every tool that produces a clone_id will produce a cell_id are we? I could see how that might work for a single cell pipeline, but what about the bulk AIRR seq clone tools that don't know anything about cells (e.g. MiXCR, ImmuneDB, and even the Change-O non paired pipeline)? They simply assign a clone_id, no?

Yeah, I meant in the single-cell context. For bulk, there isn't going to be a single right answer for how to count clones -reasonable approximations are going to differ with the technology too (mRNA vs gDNA, with vs without replicates).

 I have a study with VH and VL data, not paired (for example two different sample processings, with different PCR targets)
My understanding is that one would often run a clone tool (Change-O, MIXCR) on the VH data.
Presumably one could run a clone tool on the VL data. Would one want to do that?
That would give separate clone_id's than that of the VH data, and there is no way to link the VH and VL data. So you would have two independent sets of clones, one from the VH and VL data. Does that make sense and/or is it useful? I assume this is a case that we might run across so would need to support.

Presumably one could run a clone tool on the VL data. Would one want to do that?

Mostly, I would say "no". Light chains don't have as much information, so while you could certainly cluster the VLs for some purposes, it's not going to be specific enough to reliably infer decent from a common ancestor. The V:J gene pairing is going to drive the clustering.
Maybe it's workable in a repertoire with a lot of SHM in the VL, but it's not something I've spent a lot of time on... Dunno if anyone has dug into it using single-cell data.  This is sort of on topic, but not exactly:
https://www.jimmunol.org/content/203/7/1687

Thanks all for your comments. I've modified my mapping file to include new fields showing example counts for sequence_count and clone_abundance for 10X, Change-o, and MiXCR. These are the fields highlighted in yellow in the respective tabs. Based on what's been discussed, are these values correct?
clone.mapping_with_counts_examples.xlsx
Note that for the 10X example, I've included both clone_count and number of unique cells (10X refers to this as "frequency"). It seems like frequency is most useful, but not sure if/how we want to include this in the clones specification.

        sequence_count:
            type: integer
            description: Number of Rearrangement records (sequences) included in this clone.
        clone_abundance:
            type: integer
            description: Non-normalized absolute count of the number of members (immune cells) in this clone.

From the call:

sequence_count to be removed from the Clone schema
Add umi_count (sum of umi_count from Rearrangement), which will map to
** Change-O = sum(umi_count for each clone_id)
** 10X = sum(duplicate_count for each clone_id)
Add duplicate_count (sum of duplicate_count from Rearrangement), which will map to
** MiXCR = count(readID for each cloneID)
Rename clone_abundance to clone_count for naming consistency, mappings the same as @bcorrie listed above
** Corresponds to # of cells for single cell protocols
** Corresponds to # of unique variants for bulk protocols

@scharch about to implement this for 10X, and I am on the verge of being confused again 8-) If I understand correctly we now should have:

umi_count:
** Change-O = sum(umi_count for each clone_id)
** 10X = sum(duplicate_count for each clone_id)
clone_count:
** Corresponds to # of cells for single cell protocols
** Corresponds to # of unique variants for bulk protocols
duplicate_count:
** ???

I am not sure what duplicate_count should be, in particular in the 10X case, when umi_count = sum(duplicate_count for each clone_id)

To be VERY specific for 10X, clone_count = count(unique cell_id for each clone_id)
E.g. For clonotype1 in a 10X airr_rearrangements.tsv
awk -F '\t' '/clonotype1\t/{print $0}' airr_rearrangement.tsv | awk -F '\t' '{print $1}' | sort -u | wc -l

@bcorrie I think  the actual answer is that duplicate_count doesn't apply to 10x data. But damned if I can figure out what I was in my head 3 months ago from the notes here and at #161...

I think the actual answer is that duplicate_count doesn't apply to 10x data. But damned if I can figure out what I was in my head 3 months ago from the notes here and at #161...

And I think you are correct. :)
When you load in the cellranger data, rename duplicate_count to umi_count. Then don't assign duplicate_count because cellranger doesn't remove duplicate sequences. Then things should be consistent.
Quoting @scharch:

add a umi_count field.
revise the documentation of duplicate_count to deprecate its use in favor or umi_count where possible. (duplicate_count would remain available for non-umi contexts.)

Ie, we've used duplicate_count for UMI counts from bulk UMI protocols historically. That's not worked out well with single-cell data. So, preferably, the definition of duplicate_count gets narrowed to copy number of identical sequences (still working the same for bulk UMI protocols). You could still have a duplicate_count in 10x data, but you'd have to go through the calculation and it wouldn't have anything to do with UMIs.
The break is that for bulk sequencing, UMI ~ transcript in any cells; for single-cell, UMI ~ transcript within a given cell. So UMIs were (badly) approximating cell count in bulks, whereas they directly reflect cellular expression in single-cell.


Text case transformation

Description of the new feature / enhancement
Hi there,
When it comes to type text sometimes we spend such a time retyping in the good case (upper or lower).
It would be so time-saving having a shortcut doing it for you.

Great thing is, pressing for instance shift+F3 would transform selected text to lowcase or vice-versa.
What do you thing of this so cool so time-saving feature ?
You're IN ?
If any questions or any exchange about it, feel more than welcome !
Scenario when this would be used?
During training ERP lessons I see eachday, people facing this situation :
After typing a hole text, figuring out that IT'S TYPED UPPERCASE....
Using Microsoft Word, we are so used to case transformation button or shortcut.
First case, all selection is converted uppercase.
Second time use of the shortcut, then text is transformed to the opposite, e.g. lowercase.
Third time pressing shortcut could be first letter of each word in uppercase.
Supporting information
No response

/dup #907


Upgrade zinc to 1.4.0

Builds are failing: https://travis-ci.org/github/higherkindness/rules_scala/builds/739988853
The status doesn't on the PR update cause of #288... I'll reach out to James to see if we can get that taken care of

Version 1.4.0 has breaking changes:

VirtualFile and VirtualFileRef instead of Path;
compiler-bridge.jar must contain META-INF/services/xsbti.compile.CompilerInterface2 or it will throw NPE because of switching to legacy xsbti.CompilerInterface which not available in classpath;
No ScalaPB for sbt internal messages! Only messages from shaded google protobuf package.

I have a fork (https://github.com/TimothyKlim/rules_scala/pull/1/files) which I'm using to upgrade zinc to 1.4.2, drop old scala versions and add scala 3.0.

Awesome. @TimothyKlim do you have plans to merge your fork back here once it's working?

Now I can build and test my project with this commit.
You can cherry-pick changes that you need to continue backwards compatibility, I have my fork to have ability to move fast and break things until I'll port rest things to get rid of sbt.


Duplicate Request and Response templates

The Request and Response templates must be shared between Operazioni Avvio and Categorie

Fixed in commit ff35f09da0b90e8bef0d145ffbdfe69373dc90c6


Evaluate Current Board Revision

In the current board revision, Tyler has kindly added a 4k-ohm pull-up resistor.  This has improved stability, and I've observed that we now spontaneously recover from bus stuckness sometimes!  It's not yet clear whether this is "always," "device flakiness outside of I2C," or "sometimes," pending further testing.
We should evaluate:

Whether this is sufficient.  I am still seeing some bus flakiness, but it seems to recover most of the time.  https://forums.adafruit.com/viewtopic.php?t=187016 suggests that we could even use a 1k-ohm resistor if we wish to be more aggressive.
Whether we wish to add a MOSFET to GPIO pin 17 and the 5v supply to GPS and I2C so that we can soft-reset the devices on the bus if desired.  I want to try a software update before we go down this road, as I have seen the original Board + GPS + BMP + SEN5X sensors run for months at a time with no issue.
The voltage regulator gets quite hot, hot enough that you can't hold your finger on it for long.  We should add a heatsink.

@bensonk I have measured the capacitance of the I2C bus on Dr. Hood's EPA device.  400pF is I2C's design limit.
This is a cheap Chinese LC meter, so take each of these readings with a grain of salt.  They might be correct in an order-of-magnitude sense and do appear to be repeatable at least.  Readings are for SDA-GND, and the value from SCL-GND when tested is similar enough that I'm going to call it identical.
23 pF:  Probes only
1.77 nF:  BMP390 only (on its face, this is already higher than the allowable 400pF on a 100kHz standard I2C bus!)  I suspecct it may contain an active pull-up on the board.  See comment below.
3.61 nF:  Board only.  No devices attached.
3.57 nF:  Board + GPS.  The GPS shouldn't affect this reading, but I am adding it as a control.
5.30 nF:  Board + GPS + BMP390
5.41 nF:  Board + GPS + BMP390 + SEN5X -- This suggests that the Sen5X has a capacitance of 110pF, well within spec for I2C.
7.15 nF:  Board + GPS + BMP390 + SEN5X + 1 DFRobot Gravity MultiGas
8.88 nF:  Board + GPS + BMP390 + SEN5X + 2 DFRobot Gravity MultiGas
10.61 nF:  Board + GPS + BMP390 + SEN5X + 3 DFRobot Gravity MultiGas
So, either my meter is nuts or I'm missing something.  On its face, even just the BMP390 alone should prevent I2C from working, but we've seen it work for 8 months straight with no hiccups.  If you used the official guide, we need an 0.11 ohm pull-up, below the allowable minimum size.  In other words this isn't even supposed to be possible on I2C.
What I'm taking from that document is that we could actually use a much more aggressive pull-up resistor, even in the hundreds of ohms, to try to stabilize the bus.

Update:  I tried the capacitance of a different BMP390 and got a very similar value.
Another thing we could try:  Active pull-ups are apparently able to boost the max allowable capacitance into the nF range.

I have confirmed that we are able to deliberately cause bus stuckness by repeatedly crashing the SimpleAQ service without gracefully closing the I2C connections.

I'm increasingly pessimistic that this is going to be sufficiently stable with the 4k pull-up, but I'm still making a lot of changes to the software to improve things on that side.  Tyler is working on switching us over to an LTC4311-based active pull-up: https://www.analog.com/media/en/technical-documentation/data-sheets/4311fa.pdf

Ok, so I just saw a device on my desk boot-loop for 10 minutes and I couldn't figure out why.
I think it's because the voltage regulator on there has an automatic thermal shutdown, and it was getting too hot.
I put a heatsink on there and that stopped the issue, but even this heatsink gets hot.
We have two options:

A heatsink to disperse heat, or
Reducing the voltage of our power supply from 9V to, say, 7V.  This will give us less power but also dissipate less heat, or
Tyler has suggested that we could use USB-C power and let the wall wart figure it out.

So, I haven't ordered or tried Tyler's new board revision yet, but I'm evaluating the older revisions of the board.
One thing I notice is that the bus issues always  unfold the same way:

OS Errno 5 (Input/output) error when reading data from a DFRobot Multi-Gas device
Hang forever when publishing BMP3XX data.

I found this, indicating that the BCM2835 chip has a bug with no workaround or fix except "don't use any devices that utilize clock stretching," which will cause bus stuckness when clock stretching:  https://www.advamation.com/knowhow/raspberrypi/rpi-i2c-bug.html
There is an option.  We could switch to software i2c, which would use different pins.
Or, we could experiment with a Raspberry Pi Zero 2W, which uses a BCM2710A1.

So, I'm noticing a few more weird things:
All of SimpleAQ service seems to hang on one of these lines during bus stuckness:

https://github.com/adafruit/Adafruit_CircuitPython_BusDevice/blob/main/adafruit_bus_device/i2c_device.py#L172
https://github.com/adafruit/Adafruit_CircuitPython_BusDevice/blob/main/adafruit_bus_device/i2c_device.py#L153
https://github.com/adafruit/Adafruit_CircuitPython_BMP3XX/blob/main/adafruit_bmp3xx.py#L147

So far we've been stable for 7 hours with this build:  https://github.com/simpleairquality/SimpleAQ-Device/actions/runs/12496783549
But, I noticed something:  in this particular case, we publish the BMP3XX data before the DFRobot devices in the order of publish.  I wonder if this matters.

Ok, and moving BMP3XX to the middle of the devices seems to have caused an infinite loop at "waiting for read_byte in adafruit bmp3xx" almost immediately.


Cassandra's replication factor not reported by Elasticsearch's number_of_replicas

I'm using Elassandra <IP_ADDRESS>.
I have created a cluster of 2 nodes and created an index via ES' APIs, like this:
PUT myindex
{
    "mappings": {
      "mytype": {
        "properties": {
          "f1": {
            "type": "text",
            "cql_collection": "singleton"
          },
          "f2": {
            "type": "keyword",
            "cql_collection": "singleton"
          },
          "f3": {
            "type": "date",
            "cql_collection": "singleton",
            "cql_primary_key_order": 1
          },
          "f4": {
            "type": "keyword",
            "cql_collection": "singleton",
            "cql_partition_key": true,
            "cql_primary_key_order": 0
          }
        }
      }
    }
}

and the resulting cql schema:
cqlsh> desc myindex;

CREATE KEYSPACE myindex WITH replication = {'class': 'NetworkTopologyStrategy', 'DC1': '1'}  AND durable_writes = true;
   ......
   .....

after that I have altered the repliation factor via CQLsh
cqlsh> alter keyspace myindex WITH replication = {'class': 'NetworkTopologyStrategy', 'DC1': 2};
but in ES I cannot see my replication factor updated:
GET _cluster/state
{
  "cluster_name": "Elassandra Cluster",
  "compressed_size_in_bytes": 711,
  "version": 10,
  "state_uuid": "QaG4pfvCQQailsnL0iUu4Q",
  "master_node": "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa",
  "blocks": {},
  "nodes": {
    "2ea605c3-32ff-4db8-b5bf-3c1da893e029": {
      "name": "<IP_ADDRESS>",
      "status": "ALIVE",
      "ephemeral_id": "2ea605c3-32ff-4db8-b5bf-3c1da893e029",
      "transport_address": "<IP_ADDRESS>:9300",
      "attributes": {
        "rack": "r1",
        "dc": "DC1"
      }
    },
    "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa": {
      "name": "<IP_ADDRESS>",
      "status": "ALIVE",
      "ephemeral_id": "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa",
      "transport_address": "<IP_ADDRESS>:9300",
      "attributes": {
        "rack": "r1",
        "dc": "DC1"
      }
    }
  },
  "metadata": {
    "version": 1,
    "cluster_uuid": "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa",
    "templates": {},
    "indices": {
      "myindex": {
        "state": "open",
        "settings": {
          "index": {
            "creation_date": "1530611759437",
            "number_of_shards": "2",
            "number_of_replicas": "0",
            "uuid": "VaL7xyIjTFKiRkjOM_45Qw",
            "version": {
              "created": "6020399"
            },
            "provided_name": "myindex"
          }
        },
        "mappings": {
               ....................

I read that 'number_of_replicas' provided by ES should be Cassandra's 'replication_factor' - 1.
It seems that the Cassandra's replication factor update was not propagated to ES.
does 'number_of_replicas' provide some functionality or it's not a real problem to keep it to 0 ?
any ideas?
PS: I have also rebuilt the keyspace in cassandra without success

Yes, CQL schema update does not trigger elasticsearch mapping update.
Number of replica is updated when your index mapping is updated, so, for exemple, you can close and open your index.

Le 3 juil. 2018 à 12:39, Gappa88<EMAIL_ADDRESS>a écrit :
I'm using Elassandra <IP_ADDRESS>.
I have created a cluster of 2 nodes and created an index via ES' APIs, like this:
PUT myindex
{
"mappings": {
"mytype": {
"properties": {
"f1": {
"type": "text",
"cql_collection": "singleton"
},
"f2": {
"type": "keyword",
"cql_collection": "singleton"
},
"f3": {
"type": "date",
"cql_collection": "singleton",
"cql_primary_key_order": 1
},
"f4": {
"type": "keyword",
"cql_collection": "singleton",
"cql_partition_key": true,
"cql_primary_key_order": 0
}
}
}
}
}
and the resulting cql schema:
cqlsh> desc myindex;
CREATE KEYSPACE myindex WITH replication = {'class': 'NetworkTopologyStrategy', 'DC1': '1'}  AND durable_writes = true;
......
.....
after that I have altered the repliation factor via CQLsh
cqlsh> alter keyspace myindex WITH replication = {'class': 'NetworkTopologyStrategy', 'DC1': 2};
but in ES I cannot see my replication factor updated:
GET _cluster/state
{
"cluster_name": "Elassandra Cluster",
"compressed_size_in_bytes": 711,
"version": 10,
"state_uuid": "QaG4pfvCQQailsnL0iUu4Q",
"master_node": "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa",
"blocks": {},
"nodes": {
"2ea605c3-32ff-4db8-b5bf-3c1da893e029": {
"name": "<IP_ADDRESS>",
"status": "ALIVE",
"ephemeral_id": "2ea605c3-32ff-4db8-b5bf-3c1da893e029",
"transport_address": "<IP_ADDRESS>:9300",
"attributes": {
"rack": "r1",
"dc": "DC1"
}
},
"4599072d-b1e7-47d4-b5ad-87ec6d2b85aa": {
"name": "<IP_ADDRESS>",
"status": "ALIVE",
"ephemeral_id": "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa",
"transport_address": "<IP_ADDRESS>:9300",
"attributes": {
"rack": "r1",
"dc": "DC1"
}
}
},
"metadata": {
"version": 1,
"cluster_uuid": "4599072d-b1e7-47d4-b5ad-87ec6d2b85aa",
"templates": {},
"indices": {
"myindex": {
"state": "open",
"settings": {
"index": {
"creation_date": "1530611759437",
"number_of_shards": "2",
"number_of_replicas": "0",
"uuid": "VaL7xyIjTFKiRkjOM_45Qw",
"version": {
"created": "6020399"
},
"provided_name": "myindex"
}
},
"mappings": {
....................
I read that 'number_of_replicas' provided by ES should be Cassandra's 'replication_factor' - 1.
It seems that the Cassandra's replication factor update was not propagated to ES.
does 'number_of_replicas' provide some functionality or it's not a real problem to keep it to 0 ?
any ideas?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/strapdata/elassandra/issues/209, or mute the thread https://github.com/notifications/unsubscribe-auth/AJzHmUqOr0ZR_uoD0mqnYfNbmBLHVetbks5uC0nDgaJpZM4VAs_r.

that's worked nicely!
{
"myindex": {
   "settings": {
     "index": {
       "creation_date": "1530611759437",
       "number_of_shards": "2",
       "number_of_replicas": "1",
       "uuid": "VaL7xyIjTFKiRkjOM_45Qw",
       "version": {
         "created": "6020399"
       },
       "provided_name": "myindex"
     }
   }
 }
}

"number_of_replicas": "1", that is exactly Cassandra's replication factor -1
thanks again!


Issue of NodeOp aggregate_w

Hi, thanks for implementing it in tensorflow and open-sourcing it!
I'm not familiar with tensorflow yet, but I think I've spotted a single mistake here.
At custom_layer.py#L25, self.aggregate_w should pass through sigmoid layer to ensure they are positive. (ref: footnote 3 of paper)

Yes I forgot to add sigmoid, thanks!


Documentation misleading on Private endpoint Public IP access

The table comparing Public endpoint and service endpoint indicates that a Storage Account with a Private endpoint can disable it's Public IP (which means a Storage account no longer is accessible via it's Public IP). However part of our Private endpoint documentation mentions the opposite.
"Private endpoints provide a privately accessible IP address for the Azure service, but do not necessarily restrict public network access to it."
Reference : https://learn.microsoft.com/en-us/azure/private-link/private-endpoint-overview#network-security-of-private-endpoints
How should these two be interpreted as one of main drivers of adopting Private endpoint is to ensure the service isn't accessible using it's Private IP.
P.S. Adopting Private endpoint on Storage & Databases has cost implications which quickly escalate with Data Lake scenario's.

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

ID: 78520cab-232c-47ad-f6f2-4279c8a92527
Version Independent ID: 6867fd52-f291-007a-92aa-d4c4247cc18a
Content: Virtual network integration of Azure services for network isolation
Content Source: articles/virtual-network/vnet-integration-for-azure-services.md
Service: virtual-network
GitHub Login: @asudbring
Microsoft Alias: allensu

@ameetkonnur  I see you are refering to the below:

What this table says is that for

Service EndPoint to work, the service(PaaS reosurce) should not disable Public Access. (cannot)
Private EndPoint to work, the service(PaaS reosurce) may or may not disable Public Access. (can, but not necessarily)

The inference is that Private EndPoints do not have any impact from whether or not the PaaS service is allowing or blocking traffic via Public EndPoint.
Hope this clears things.

@KapilAnanth-MSFT thanks for the response.
The documentation currently is ambiguous and indicates that this is the feature of Private endpoint.  From what I know none of our current services allow to disable public endpoint once private endpoint is enabled ?
The only option is through Azure Policy at every service level and is dependent on the service supporting it and not private endpoint.

@ameetkonnur
I don't think so that's the case.
Wrt, "From what I know none of our current services allow to disable public endpoint once private endpoint is enabled"

This is incorrect
For a service using Private EndPoint, Public access will still work as long the PaaS service's network configuration allows Public access.
It could be the case that your environment is configured to use only Private access method (PE). However, I can guarantee you that enabling PE on a PaaS service does not disable it's Public access.

The statement here states that "Service can disable its public IP address", not necessarily meaning that the Service will disable its public IP address

@KapilAnanth-MSFT I understand your point; referring to the Networking Option on our services that allow to Disable Public Access.
My understanding it doesn't disable / remove the Public IP but merely restricts all Traffic to the Public IP. This nuance is a significant for customer's who are regulated who understand Private Endpoint as No Public IP.
Happy to be corrected if I misunderstood this.

@ameetkonnur
"My understanding it doesn't disable / remove the Public IP but merely restricts all Traffic to the Public IP."

I am afraid this statement is incorrect.
No, Azure Private EndPoint does not "restrict" access to Public IP
In fact, it has no control over the Public network access configuration of the service or Public IP assigned to the service
A private endPoint adds a Private IP to a PaaS service in a VNET which allows resources in that VNET to access it via this Private IP.
Now the behavior of the Public IP access is entirely up to the PaaS service's configuration.

Again, the document never specifies that Private Endpoint as No Public IP.
Hope this helps

@ameetkonnur we are closing this thread as there are no active discussion going on.
Should you require any further information, please tag me and we shall reopen the issue and work with you


7/12 Publish

@Taojunshen,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

ci_scripts/generate_xrefmap.py
:white_check_mark:Succeeded

xrefmap.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix glsl errors

Rename common.glsl to comp_common.glsl -Otherwise IntelliJ complains about reserved word
Add default return statement to count_prio_offset - Otherwise intellij complains
Move return statement to end of priority_map -IntelliJ complains about no return statement when it is in switch { default } case.

Signed-off-by: Tomas Slusny<EMAIL_ADDRESS>

I don't think any of these fixes are actual problems.

That is why I labeled them as enhancement and not bug, its just so intellij stops complaining. Even though, honestly, just looking at count_prio_offset it makes no sense, unless GLSL has implicit returns.

Priority must be 0-17

Yeah, but the function returns nothing unless priority is 0

As far as I can tell, none of these are actual errors.

As I stated before, IntelliJ is complaining about it


dockered agent:7 has some troubles with dockered harbor 1.10.2

sorry, wrong issue. delete it please

misconfiguration


Fix KV.contents bug with occasional invalid consumer name

Using the nuid function to create a unique consumer name sometimes created invalid consumer names that included a / character. Fixing this by using a random integer instead.

I think that the better way to handle this is modify the Util.nuid/1 function to use Base.url_encode64/2 instead of Base64.encode64/2 which changes the / sign to _ which is allowed in Jetstream's consumer names.


Propagate completion?

startWork.Completion.ContinueWith(delegate { completeWork.Complete(); });

Shouldn't we prefer to do:
startWork.LinkTo(completeWork, new DataflowLinkOptions { PropagateCompletion = true })

?

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

ID: 1e2c4aa6-899b-7cf1-c2c1-cb362ee1243a
Version Independent ID: d6a8cfcf-ccaa-320c-d039-e514e42c2a07
Content: How to: Cancel a Dataflow Block
Content Source: docs/standard/parallel-programming/how-to-cancel-a-dataflow-block.md
Service: unspecified
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

@ishepherd I agree, so I have created a Pull Request that makes this change: https://github.com/dotnet/samples/pull/48.


Project: VisuaFind: AI-Powered Image Search for E-Commerce

Project Name
VisuaFind: AI-Powered Image Search for E-Commerce
Description
This project is an image search engine designed specifically for e-commerce websites, offering an unparalleled user experience. Imagine the convenience of simply describing the product or clothing item you're looking for in a search bar, and instantly receiving relevant results. Our solution leverages FastAPI and React.js as the primary frameworks, alongside PostgreSQL integrated with pgvector for efficient vector-based search capabilities. Plus, we used the advanced RAG architecture and our source of images is from kaggle public database.
Technology & Languages

[x] JavaScript
[ ] Java
[ ] .NET
[X] Python
[ ] AI Studio
[X] AI Search
[X] PostgreSQL
[ ] Cosmos DB
[ ] Azure SQL

Project Repository URL
https://github.com/TNAHOM/Image-Search-Engine
Deployed Endpoint URL
No response
Project Video
https://drive.google.com/drive/folders/1xnrlcy-p88D4apVxjOTHFNBSXailxxc7?usp=sharing
Team Members
https://github.com/TNAHOM, https://github.com/temesgen5335

Hello @TNAHOM, @temesgen5335, thank you for participating in RAG Hack!
The team is working hard to distribute badges. Please have each team member fill out this form:
aka.ms/raghack/badge-dist
Thank you!


Don't override credentials when they are not specified in provider specific config

Description
Don't override credentials when they are not specified in provider specific config.
Type of change

[x] Bug fix

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

Please upload report for BASE (main@11391a3). Learn more about missing BASE report.
Report is 30 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1264   +/-   ##
=======================================
  Coverage        ?   94.75%
=======================================
  Files           ?       57
  Lines           ?     2441
  Branches        ?        0
=======================================
  Hits            ?     2313
  Misses          ?      128
  Partials        ?        0

Files
Coverage Δ

ols/src/llms/providers/azure_openai.py
100.00% <100.00%> (ø)

ols/src/llms/providers/bam.py
100.00% <100.00%> (ø)

ols/src/llms/providers/openai.py
100.00% <100.00%> (ø)

ols/src/llms/providers/watsonx.py
97.05% <100.00%> (ø)

/approve

/lgtm

/override "Red Hat Konflux / ols-e2e-tests-rosa / lightspeed-service"


Added Jaeger gRPC receiver

Closes #127 by implementing a gRPC receiver based on the protobuf specification from Jaeger's own repository.
Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>

Things to discuss:

[ ] Should we provide a TLSConfig for the gRPC server? If so, is it better to be part of a follow-up PR?

Travis is failing due to a test failure (see review comment).

This PR has been updated with the results of the first review.
There are still a couple of points to clarify before I can change this PR from its current "draft" status, namely:

Mismatch between Jaeger and OpenCensus/OpenTelemetry when it comes to Links: in Jaeger, we have only a SpanReference on the child span, pointing to its parent. Apparently, OpenTelemetry expects the parent to have a Link to all its children.
Whether span.kind should be removed from the attributes, or kept there duplicating what's already in the property Kind. Other translators seem to duplicate this data, but that doesn't seem appropriate to me.

Codecov Report

Merging #197 into master will increase coverage by 0.48%.
The diff coverage is 85.77%.

@@            Coverage Diff             @@
##           master     #197      +/-   ##
==========================================
+ Coverage   72.75%   73.24%   +0.48%
==========================================
  Files         107      108       +1
  Lines        6108     6320     +212
==========================================
+ Hits         4444     4629     +185
- Misses       1438     1458      +20
- Partials      226      233       +7

Impacted Files
Coverage Δ

receiver/jaegerreceiver/factory.go
96.77% <100%> (+0.94%)
:arrow_up:

translator/trace/jaeger/status_code.go
67.74% <61.53%> (-4.49%)
:arrow_down:

receiver/jaegerreceiver/trace_receiver.go
78.17% <65%> (-1.35%)
:arrow_down:

...ranslator/trace/jaeger/jaegerproto_to_protospan.go
91.55% <91.55%> (ø)

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 5f88de9...88f8ca9. Read the comment docs.

This is now ready for the final review/merge.

Should we provide a TLSConfig for the gRPC server? If so, is it better to be part of a follow-up PR?

We have added TLSConfig to OC gRPC receiver that said I haven't used but others may have.
Is TLS commonly used by Jaeger gRPC? If not I will leave it out (at least in the short run). Anyway, a separate PR will be better for it. (TLS for OC related issue #170)

Is TLS commonly used by Jaeger gRPC?

I don't think it's that widespread yet, but we did hear requests about it in the past. In fact, we added support for TLSConfig because of user requests. That said, I think it does make sense to deal that in a broader way, as the same cert would probably be used for all TLS-capable ports.

@jpkrohling there is one test failure remaining:
--- FAIL: TestCreateWithoutThrift (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x10768dd]

goroutine 27 [running]:
testing.tRunner.func1(0xc000301200)
	/home/travis/.gimme/versions/go1.12.7.linux.amd64/src/testing/testing.go:830 +0x69d
panic(0x117e640, 0x1bef840)
	/home/travis/.gimme/versions/go1.12.7.linux.amd64/src/runtime/panic.go:522 +0x1b5
github.com/open-telemetry/opentelemetry-service/receiver/jaegerreceiver.(*Factory).CreateTraceReceiver(0xc000068ed8, 0x1420580, 0xc00003a1d8, 0xc000068f08, 0x1423900, 0xc00029f6e0, 0x0, 0x0, 0xc0002e14a0, 0x58150b, ...)
	/home/travis/gopath/src/github.com/open-telemetry/opentelemetry-service/receiver/jaegerreceiver/factory.go:137 +0x4dd
github.com/open-telemetry/opentelemetry-service/receiver/jaegerreceiver.TestCreateWithoutThrift(0xc000301200)
	/home/travis/gopath/src/github.com/open-telemetry/opentelemetry-service/receiver/jaegerreceiver/factory_test.go:152 +0x248
testing.tRunner(0xc000301200, 0x12cc228)
	/home/travis/.gimme/versions/go1.12.7.linux.amd64/src/testing/testing.go:865 +0x164
created by testing.(*T).Run
	/home/travis/.gimme/versions/go1.12.7.linux.amd64/src/testing/testing.go:916 +0x65b
FAIL	github.com/open-telemetry/opentelemetry-service/receiver/jaegerreceiver	0.054s

there is one test failure remaining

Sorry, this was caused by the check I added in a previous commit, validating that the ip:ports are unique. I'm removing this feature, as this will fail soon after that anyway, and the only value-add we are offering is a better error message.


Chain selection shouldn't always redirect to landing

I'd like to see better user experience when it comes to switching between BSX<->KSM, current implementation is always redirecting back to landing page (suboptimal). Issue is labeled as research as this could introduce bunch of edge cases, but here's what I can see working:

When user browses either of Explore tabs (collections/gallery), changing chain shouldn't redirect to landing, but rather reload explore
When user looks at item detail, changing chain may redirect back to explore (if user previously got to item detail through explore)
what happens when the user clicks on back button when they are on another chain?
....?

what happens when the user clicks on back button when they are on another chain?

what happens when the user clicks on back button when they are on another chain?

expanded by your comment :)

This will be obsoleted in

#4379


Attempt to index field 'upstream' (a nil value) (Kong v0.10.0rc1)

When setting an invalid upstream URL (e.g. http://mockbincom - I accidentally forgot the "." when I discovered this bug), Kong responded with an internal error and the following message in the error log:
2017/01/06 13:50:03 [error] 89#0: *1842 lua entry thread aborted: runtime error: /usr/local/share/lua/5.1/kong/core/handler.lua:119: attempt to index field 'upstream' (a nil value)
stack traceback:
coroutine 0:
	/usr/local/share/lua/5.1/kong/core/handler.lua: in function 'before'
	/usr/local/share/lua/5.1/kong.lua:240: in function 'access'
	access_by_lua(nginx-kong.conf:80):2: in function <access_by_lua(nginx-kong.conf:80):1>, client: <IP_ADDRESS>, server: kong, request: "GET /my/test/uri HTTP/1.1", host: "my.test.host"

Looking at the file core/handler.lua, I believe that balancer_address.upstream.host should be balancer_address.host.
Could anyone have a look at this issue?
Thanks,
Sigurd

@sisp thx! fixed in the new-router branch, waiting for that to get merged into next


Improve logging of AVC check plugin and its test

Log more selinux-related packages, auditd status, and system journal.
Pull Request Checklist

[x] implement the feature

/packit test --identifier full


get 0 AP

I trained my own data set, but got 0 AP.  Is anybody know what is the problem?

im_detect: 688/691 1.004s 0.001s
im_detect: 689/691 1.008s 0.001s
im_detect: 690/691 1.007s 0.001s
im_detect: 691/691 1.006s 0.001s
Evaluating detections
Writing aeroplane VOC results file
Writing bicycle VOC results file
Writing bird VOC results file
Writing boat VOC results file
Writing bottle VOC results file
Writing bus VOC results file
Writing car VOC results file
Writing cat VOC results file
Writing chair VOC results file
Writing cow VOC results file
Writing diningtable VOC results file
Writing dog VOC results file
Writing horse VOC results file
Writing motorbike VOC results file
Writing person VOC results file
Writing pottedplant VOC results file
Writing sheep VOC results file
Writing sofa VOC results file
Writing train VOC results file
Writing tvmonitor VOC results file
VOC07 metric? Yes
Reading annotation for 1/691
Reading annotation for 101/691
Reading annotation for 201/691
Reading annotation for 301/691
Reading annotation for 401/691
Reading annotation for 501/691
Reading annotation for 601/691
Saving cached annotations to /home2/sdin7632/shanshan/tf-faster-rcnn-r1.0/data/VOCdevkit2007/annotations_cache/annots.pkl
AP for aeroplane = 0.0000
AP for bicycle = 0.0000
AP for bird = 0.0000
AP for boat = 0.0000
AP for bottle = 0.0000
AP for bus = 0.0000
AP for car = 0.0000
AP for cat = 0.0000
AP for chair = 0.0000
AP for cow = 0.0000
AP for diningtable = 0.0000
AP for dog = 0.0000
AP for horse = 0.0000
AP for motorbike = 0.0000
AP for person = 0.0000
AP for pottedplant = 0.0000
AP for sheep = 0.0000
AP for sofa = 0.0000
AP for train = 0.0000
AP for tvmonitor = 0.0000
Mean AP = 0.0000
Results:
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000
0.000

Results computed with the unofficial Python eval code.
Results should be very close to the official MATLAB eval code.
Recompute with ./tools/reval.py --matlab ... for your paper.
-- Thanks, The Management

@zdm123  Hi~ Have you solve this problem?

@zdm123 @weisq2691 I got the same problem. Finally, I found it was because my data and label were not loaded properly. My data file is modified from pascal_voc.py, "cls = self._class_to_ind[obj.find('name').text.lower().strip()" makes the label being matched in lower case. I set the labels in "self._classes" in lower case but my labels in the annotation file are in upper case. In the evaluation stage, it cannot be matched.
I solve this problem by setting "self._classes" in upper case and change  "cls = self._class_to_ind[obj.find('name').text.lower().strip()" to  "cls = self._class_to_ind[obj.find('name').text.strip()".

cls = self._class_to_ind[obj.find('name').text.strip()

I got this problem for 3 days and now you come and save my days!!!!! Thank you so much!!! moazz moazz


Update rector.php

Update Rector config
Description
Rector use PHPStan to detect types
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you @WinterSilence


[T2A6][1]Wong Kai Min Herman

Work in Progress (PostalCode's Regex returning null)

@InspiredAlpaca

Please create a new branch for each activity.
Some comments added. Please close the PR after reading comments.


Added background subtraction

All the code to compute the energy test with background subtraction is there. Just need to set defaults values and options so that it defaults to the code without background subtraction

I solved the inconsistency and committed the code that now lgtm. Will, when you have time have a look and if you are happy merge the pull request.

Superseded by branch tests

I changed --signal to float and added some tests in #3 if you happy accept that merge request. For the Tis I am not sure there is much you can do more, maybe print also the Tis for the background so one can add them with a negative weight in an histogram? Anyway I would start merging what we have and leave it for the future

Sounds good.


Protobuffer support

Love the start of the library!
We use Dynatrace as part of our monitoring stack and presently they only support HTTP & Protobuffer as a transport mechanism for span traces and the such - https://www.dynatrace.com/support/help/shortlink/otel-getstarted-otlpexport
Any chance you would add protobuffer support at all?
Thanks!

I had proto support in https://github.com/RichiCoder1/opentelemetry-sdk-workers so I know it works. Only issue I had with it was size and complexity.
Shouldn't be too difficult to add to this lib since it's more compliant with the source libs.

Theoretically it should be possible to use the OpenTelemetry-JS protobuf exporter? I haven’t tried it myself, but it uses the same interface for that reason..
So if you could give that a try I would greatly appreciate it so we can document that better for when the next person comes along.
Also let us know if you run into any problems ofc and we’ll see what we can come up with 😄

I've had a go at this but turns out the OTEL exporter library uses XMLHttpRequest to send the data.
There is an MR to rework this but not clear how far off it is to landing https://github.com/open-telemetry/opentelemetry-js/pull/3542

Then we might need to look at adapting @RichiCoder1's code and porting it over to here.. Would you be able/have time to have a look at what would need to be done for that @RichiCoder1?

I'm happy to advise, but I'm afraid I can't make any promises.
There's a few complexities with pulling my code:

It uses a custom Cloudflare-tailored Exporter base
It uses a submodule to pull in the OTEL protos and does custom generation off of that: https://github.com/RichiCoder1/opentelemetry-sdk-workers/blob/main/packages/opentelemetry-sdk-workers/package.json#L44

That said, there's no reason it couldn't/wouldn't work verbatim with this package, just would be pulling over a lot of junk. Might be easier to the OTEL JS PR merged, though one might have to ping the slack/WG to get it accelerated.

I know you were very strapped for time, but this is exactly what I was hoping for.. an idea of what would be involved so we can figure out how to best move forward.
Getting a PR merged sounds like it would be by far the best option. In the interim I guess we could fork, merge the PR ourselves and release that.. but that sounds a bit messy?

So I've managed to get a branch of this all working and there's a few things that will likely need fixing on this end.
At present when the exporter calls fetch it resolves to the proxy which then causes it to start a new span and export it again resulting in a never ending loop of spans that keep sending till the Worker exhausts it's request limits.
Not sure how I can unwrap it within the exporter or get the Proxy to detect that it's the exporter that's using fetch vs the worker.

That is awesome news 😄
the unwrapping happens with the unwrap method. Here is an example: https://github.com/evanderkoogh/otel-cf-workers/blob/2d370cb9098a9066da3ba2b3102d6f29e5c99c31/src/exporter.ts#L54
does that help?


Support selector-based overrides

What would you like to be added:
Ability to define a clusterSelector on all FederatedOverride objects.
Why is this needed:
Re-opening this as issues with this keep coming up.
https://github.com/kubernetes-sigs/kubefed/issues/581
https://github.com/kubernetes-sigs/kubefed/issues/643
I had commented on the ticket a year ago but wasn't able to reactivate it.
The original concerns on this was that we didn't have a way to handle conflicting overrides, but I think the solution to this is just make the order the priority. Last override in the list wins.

/kind feature

/remove-lifecycle stale

That's what I plan to do then. But it do need some concern on conflicts and propagated version calculating issues.
/assign

Project is to be archived - closing all issues and PRs
see https://github.com/kubernetes/org/issues/4122 and https://groups.google.com/d/msgid/kubernetes-sig-multicluster/9f8d81d1-07d1-4985-a7bf-d76197deb971n%40googlegroups.com
for details
/close


Could not load compiled classes for build file from cache in 8.2.2

What went wrong:
A problem occurred configuring project ':fluttertoast'.

Could not load compiled classes for build file 'C:\Users\User\AppData\Local\Pub\Cache\hosted\pub.dev\fluttertoast-8.2.2\android\build.gradle' from cache.
Failed to notify project evaluation listener.
Could not get unknown property 'android' for project ':fluttertoast' of type org.gradle.api.Project.
Could not get unknown property 'android' for project ':fluttertoast' of type org.gradle.api.Project.

facing same issue


[🐛 BUG] On_submission_change - submission is completed twice

What went wrong? 🤔
The callback is called twice sometimes with the same parameters. It leads to notifying twice the user that the scenario is completed.
How to replicate
from taipy import Config
import taipy as tp
from taipy.gui import Gui, notify

# Normal function used by Taipy
def double(nb):
    return nb * 2

# Configuration of Data Nodes
input_cfg = Config.configure_data_node("input_dn", default_data=21)
output_cfg = Config.configure_data_node("output_dn")

# Configuration of tasks
first_task_cfg = Config.configure_task("double",
                                       double,
                                       input_cfg,
                                       output_cfg)

# Configuration of scenario
scenario_cfg = Config.configure_scenario(id="my_scenario",
                                         task_configs=[first_task_cfg],
                                         name="my_scenario")

def notify_from_submissions(state, submittable, details):
    submission_status = details.get('submission_status')

    if submission_status == 'COMPLETED':
        print("COMPLETED")
        print(submittable)
        print(details)
        notify(state, 'success', 'Completed!')
        # Add additional actions here, like updating the GUI or logging the completion.

    elif submission_status == 'FAILED':
        print("FAILED")
        notify(state, 'error', 'FAILED!')
        # Handle failure, like sending notifications or logging the error.

if __name__=="__main__":
    tp.Orchestrator().run()
    scenario_1 = tp.create_scenario(scenario_cfg)

    scenario_md = """
<|{scenario_1}|scenario|on_submission_change=notify_from_submissions|>
"""
    Gui(scenario_md).run()

This is what I receive sometimes. The callback is called twice:
[2024-10-25 11:50:04.385][Taipy][INFO] job JOB_double_bd8e666f-5ce5-40ad-8694-af7d02a3faec is completed.
COMPLETED
COMPLETED
<taipy.core.submission.submission.Submission object at 0x000001BAC6CCCE90>
<taipy.core.submission.submission.Submission object at 0x000001BAC6CCD310>
{'submission_status': 'COMPLETED', 'submittable_entity': my_scenario, 'origin_entity_id': 'SCENARIO_my_scenario_6c3d70af-a896-46f8-a0dc-4f17628b29d4', 'origin_entity_type': 'SCENARIO', 'origin_entity_config_id': 'my_scenario', 'creation_date': datetime.datetime(2024, 10, 25, 11, 50, 4, 207191), 'version': '5395d6e1-e5b7-4c2e-a391-7f71d04e8047', 'job_triggered_submission_status_changed': 'JOB_double_bd8e666f-5ce5-40ad-8694-af7d02a3faec'}
{'submission_status': 'COMPLETED', 'submittable_entity': my_scenario}

Expected behavior
This callback should be called once with these parameters
Version of Taipy

release/4.0
develop

Acceptance Criteria

[ ] A unit test reproducing the bug is added.
[ ] Any new code is covered by a unit tested.
[ ] Check code coverage is at least 90%.
[ ] The bug reporter validated the fix.
[ ] Related issue(s) in taipy-doc are created for documentation and Release Notes are updated.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

Hmmm I'm running it on the develop branch and it doesn't seem to have this issue

But the problem does happen on 4.0!

I can see it is closed.
It should be backported to release/4.0. Did I miss the PR for the backport, or is it planned for the future?
@toan-quach @FredLL-Avaiga

I guess @FabienLelaquais has backported this ?

All right. Thx!

It doesn't seem to be fixed in 4.0.2.dev1.
[2024-12-18 14:47:46.742][Taipy][INFO] Updating configuration with command-line arguments...
[2024-12-18 14:47:46.743][Taipy][INFO] Managing application's version...
[2024-12-18 14:47:46.744][Taipy][INFO] Development mode: Clean all entities of version 1b849301-77bb-43bd-b4c4-c4f0f33d15b5
[2024-12-18 14:47:46.793][Taipy][INFO] Checking application's version...
[2024-12-18 14:47:46.794][Taipy][INFO] Blocking configuration update...
[2024-12-18 14:47:46.795][Taipy][INFO] Starting job dispatcher...
[2024-12-18 14:47:46.798][Taipy][INFO] Orchestrator service has been started.
[2024-12-18 14:47:48.220][Taipy][INFO]  * Server starting on http://<IP_ADDRESS>:5000
[2024-12-18 14:47:52.302][Taipy][INFO] job JOB_double_2312965a-c72d-4649-aa26-53e9a2c74e68 is completed.
COMPLETED
<taipy.core.submission.submission.Submission object at 0x000002B99723BC10>
COMPLETED
<taipy.core.submission.submission.Submission object at 0x000002B9972600D0>
{'submission_status': 'COMPLETED', 'submittable_entity': my_scenario}
{'submission_status': 'COMPLETED', 'submittable_entity': my_scenario, 'origin_entity_id': 'SCENARIO_my_scenario_069b8ca0-845d-4929-a1bc-b59341925b2c', 'origin_entity_type': 'SCENARIO', 'origin_entity_config_id': 'my_scenario', 'creation_date': datetime.datetime(2024, 12, 18, 14, 47, 52, 88398), 'version': '1b849301-77bb-43bd-b4c4-c4f0f33d15b5', 'job_triggered_submission_status_changed': 'JOB_double_2312965a-c72d-4649-aa26-53e9a2c74e68'}

on it

Fixed in 4.0.2.dev2!