sql: Display inherited constraints in SHOW PARTITIONS

PR: https://github.com/cockroachdb/cockroach/pull/40493
From release notes:

Added the zone_id and subzone_id columns to crdb_internal.partitions. These columns form a link to the zone configurations in crdb_internal.zones that apply to the partitions. [#40493][#40493] {% comment %}doc{% endcomment %}

Dup of https://github.com/cockroachdb/docs/issues/5533.


Why needed +1 adapter size to show right count of bullets

You shouldn't have to do that, check the code at
https://github.com/badoualy/stepper-indicator/blob/master/library/src/main/java/com/badoualy/stepperindicator/StepperIndicator.java#L1163
The step count is equal to the number of pages in your adapter.


Binding.node is not a valid Win32 application (windows 10)

Hi everyone,
I was trying to create an electron build for x32 and x64 and when I launch the x32 application electron throws the following message:

What am I doing wrong? Any opinions or suggestions?
Here my machine and project features:

W10 64bits
Node 10..13.0
NPM 6.4.1
electron 4.0.1
electron-builder 20.38.2
electron-packager 12.1.0
electron-rebuild 1.8.2
electron-reload 1.2.5
node-gyp 4.0.0

Thank you in advance.

Friend, is this problem solved?

Hi,
What's was your issue? How do you resolved it?
Thanks!

I have same issue, how to resolved it?
This codes hav'nt update two years, and a lots of issue have no one been fixed?


New design: integrate comments

…integrations. Work still in progess :(

Ok now i can see sub-comment but only for 1 depth. We need to see all depth. You can check this post, the first comment has lot of level of sub comment: https://steemit.com/steemdata/@furion/steemdata-meets-sbds-sql-users-rejoice

When i click on comment counter in top (under the post title) "** Comments", the page should scroll to comment section.

Comment form should not have background, and avatar should have strict width and height: https://user-images.githubusercontent.com/16245250/27983625-914d08d6-63eb-11e7-9190-67933b70de6c.png

Comments should not have overflow, this post https://busy5-pr-561.herokuapp.com/food/@sweetsssj/miss-delicious-69-3-tides-at-the-newly-opened-baha-mar-luxury-resort has few comments with image that go over the comments section width and break layout.

2 more changes:

When there is no comments, the comment form should appear. Now its not the case. (Try page /created to get post without comments)
Can you connect back the action for commenting so we can post comment?

There is some issue while commenting. When comment using the first form, the comment will not have parent_author set, so it will not appear on comment list but will create a root post instead. See example: https://steemd.com/tx/cc99a23e298d613fa6b71f1f0204a2c12d5f45fc
Also when i reply on sub-comment, my comment will appear as root comment.

Commenting seem to be working now but the user experience is really poor. Few thing we should change:

When i submit a new root comment, all actual comments must stay visible until the new comment has been broadcasted. Now if i submit a root comment there will be a loading and comments will not be hidden, it's confusing for user. See:

When submit a reply or sub-reply, the comment textarea and button should be disabled until the comment has been broadcasted.

Avatars should have strict squared size, this is not a square its 37.5x40px:

I found few issues:

When there is no comment on single post, there is a loading that go forever see: https://busy5-pr-561.herokuapp.com/test/@hellosteem/gmvrh-test
If i comment a second time before the minimum interval of 20 seconds, the second comment will fail to broadcast (which is normal) but there should be a notification of the error on the UI. Actually it not showing anything and the the loading stay also forever.
See: https://user-images.githubusercontent.com/16245250/28454550-e0cef5f6-6e25-11e7-9c0f-3b3bbf26173c.gif
Also the avatar still not respect the size
And when i submit a comment, the form should stay visible but with disabled input and button.

[x] When there is no comment on single post, there is a loading that go forever see: https://busy5-pr-561.herokuapp.com/test/@hellosteem/gmvrh-test
[ ] If i comment a second time before the minimum interval of 20 seconds, the second comment will fail to broadcast (which is normal) but there should be a notification of the error on the UI. Actually it not showing anything and the the loading stay also forever.
See: https://user-images.githubusercontent.com/16245250/28454550-e0cef5f6-6e25-11e7-9c0f-3b3bbf26173c.gif
[ ] Also the avatar still not respect the size
[ ] And when i submit a comment, the form should stay visible but with disabled input and button.

When you click Reply, it open the reply form. If you click on other Reply it should close all other form and open the last one you clicked.

@mynameisek we dont need this behavior. I dont see how it's helping user experience.

Ask youtube UX experts ;)
The only main argument for which I agree with the useless UX with this feature is the Steem blockchain limit of 20s between each comment, which makes it more user friendly to be able to draft several comment at the same time.
We can keep it like that for now i guess.

When submit a comment, the button "Comment" have disabled style but dont have disabled parameter. I can still click and submit multiple time. We need the disabled parameter.
When comment fail, the comment form will not be visible anymore. I would like to see it with my comment so i can try submit it again.
If i comment in level 3 i will have this error in the console and the form will stay frozen:
https://user-images.githubusercontent.com/16245250/28509725-3d468804-706e-11e7-99a2-ee39d433c750.png

Few remaining issues:

When i load a single post the comment will appear, but if i switch to another post, comment are not loading anymore.
Error are not handled
There should be loading icon on the button when submit a comment


ZcashMiner::parseJob(): Invalid or unsupported block header version

I am running the command
./nheqminer_cuda_tromp -l zen.suprnova.cc:3618 -u user -p pass -cd  0
==================== www.nicehash.com ====================
	Equihash CPU&GPU Miner for NiceHash v0.4b
Thanks to Zcash developers for providing base of the code.
       Special thanks to tromp, xenoncat, mbevand
             and eXtremal-ik7 for providing
      optimized CPU, CUDA and AMD equihash solvers.
==================== www.nicehash.com ====================

Setting log level to 2
[09:03:03][0x00007f8daa1c0780] Using SSE2: YES
[09:03:03][0x00007f8daa1c0780] Using AVX: YES
[09:03:03][0x00007f8daa1c0780] Using AVX2: YES
[09:03:03][0x00007f8da0c32700] stratum | Starting miner
[09:03:03][0x00007f8d9bfff700] miner#0 | Starting thread #0 (cpu_xenoncat_STUB)
[09:03:03][0x00007f8da0c32700] stratum | Connecting to stratum server zen.suprnova.cc:3618
[09:03:03][0x00007f8d9b7fe700] miner#1 | Starting thread #1 (cpu_xenoncat_STUB)
[09:03:03][0x00007f8d93fff700] miner#2 | Starting thread #2 (cpu_xenoncat_STUB)
[09:03:03][0x00007f8d9affd700] miner#3 | Starting thread #3 (CUDA-TROMP) GeForce GTX 1080 Ti (#0) BLOCKS=196, THREADS=64
[09:03:03][0x00007f8da0c32700] stratum | Connected!
[09:03:03][0x00007f8da0c32700] stratum | Subscribed to stratum server
[09:03:03][0x00007f8da0c32700] miner | Extranonce is daea007000000000000000007000eadb
[09:03:03][0x00007f8da0c32700] stratum | Target set to 000f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f0f
[09:03:03][0x00007f8da0c32700] stratum | ZcashMiner::parseJob(): Invalid or unsupported block header version
[09:03:03][0x00007f8da0c32700] stratum | Reconnecting in 3 seconds...
how could I solve this problem?

@hossbeast I've been using https://github.com/aayanl/nheqminer which seems to have fixed all my zcash woes.

The Equihash miners that everyone uses now are:
EWBF
DSTM
Excavator
Claymore

@aaronsace  Which pools do you connect your dstm miner to? I've been trying to set it up but it doesn't seem like there are a lot of pools that its compatible with.

@aaronsace Those all appear to be closed source / proprietary. I won't be running those.

I think this is related to the mining pool you are using. And I have encountered same issue on most pool, but when I using MPH pool it's working like charm. I don't know how is somebody having experience with MPH, but it's working for me for now.


Project file search is subject to unexpected failures

The plugin tries to find the CodeSonar project file created by the analysis in order to extract the analysis ID.  Presently, the plugin does not know the name of the CodeSonar project file, nor its location.  The plugin does a simplistic search which has the potential to find an incorrect file, and therefore an incorrect analysis ID.

We added a projectFile parameter to the plugin in PR #10.  This allows the specify the project file path on the agent.  If the projectFile parameter is set, then the plugin will not search for a project file.  If projectFile is not set, the old search behavior will be used as a fallback.


feat: add support for podman

Basically podman has the same syntax as docker and podman is being used more and more widely, hopefully support for podman will be added.

hi, @baka-gourd . i am not use podman before.
did you mind create PR for it? i think its will very helpful for us

I looked for the current docker implementation and found it to be quite different from what I had envisioned, and it may take me a while to figure out how to use a go client to access podman

idk but why we have to use go client to access podman?

oh i see, you wanna get podman inspect info which like docker?
its interesting, i will keep this issue here and maybe add support in later(because never use podman before and need time to learn it.)
and welcome to send PR to help us improve it.


Make configurable the merge of json body into data

First of all thanks for making this server.
from HttpParser.parsePosted
public void parsePosted(Buf input, KeyValueRanges headersKV, BufRange rBody, KeyValueRanges posted,
	                        Map<String, List<Upload>> files, RapidoidHelper helper, Map<String, Object> dest) {

		boolean completed = parseBody(input, headersKV, rBody, posted, files, helper);

		posted.toUrlEncodedParams(input, dest);

		if (!completed && !rBody.isEmpty()) {
			try {
				Map<String, Object> jsonData = JSON.parse(input.get(rBody), Map.class);
				if (jsonData != null) {
					dest.putAll(jsonData);
				}
			} catch (Exception e) {
				Log.warn("The attempt to parse the request body as JSON failed. Please specify the correct content type in the request header!", e);
			}
		}
	}

Can we make it configurable if we want to bypass this part, and control how JSON will be parsed. it does not have sense to parse it twice. for example there is the simple case when we want to parse JSON as a custom type.
I'm forced right now to use unknown content type in client request to skip this part (to force completed to be true) which is ugly :)

Hi, currently the API doesn't support such customization, but I am working on it now...

So, in v5.2 it will be easily customizable, and lazy by default.

Oh cool ! thanks again for the great work :)

Sure, I am happy to work on Rapidoid. Thank you!


Request textDocument/codeAction failed since Volar 0.26.2

Hi,
after upgrading to 0.26.x, my project starts to popup the following errors.
[Error - 9:57:06 AM] Request textDocument/codeAction failed.   Message: Request textDocument/codeAction failed with message: Debug Failure. False expression<EMAIL_ADDRESS>linked to nonexistent file<EMAIL_ADDRESS> Code: -32603  [Error - 9:59:01 AM] Request textDocument/documentSymbol failed.   Message: Request textDocument/documentSymbol failed with message: Debug Failure. False expression<EMAIL_ADDRESS>linked to nonexistent file<EMAIL_ADDRESS> Code: -32603

Hi @oliverzy, can this problem reproduce in https://github.com/johnsoncodehk/volar-starter? And what are the steps to reproduce? Thank you.

It cannot be reproducible by volar-starter. I can solve this by adding node_modules/@xhs/yam-beer/cmp in 'include' section  of tsconfig.json.
But before 0.26, volar doesn't complain this. Ummm.

Please try to make a repro case when you free, otherwise I can't check this.

I have trying to find the commit that related to this one. The version that starts to crash is 0.26.2, 0.26.1 is fine.
The issue happened when my project is consuming a vue file (no compiled) that comes from other packages.

Hi Johnson,
The project is too big. It's pretty hard to create a repro case.
anyway, I cloned the volar project and debug locally.
Finally I find the commit that causes issue is https://github.com/johnsoncodehk/volar/commit/f6480e4543990eadddf86a8293b6ad0537ff9584
Could you see any problem on this change?

Useful, thanks. :) I think you need to config allowJs option after the fix.
But don't use 0.26.0 ~ 0.26.1 please it has serious performance issue and https://github.com/johnsoncodehk/volar/commit/f6480e4543990eadddf86a8293b6ad0537ff9584 fixed it.

Hi Johnson,
I pull the latest code, and allowJS is turned on, the issue still persists unless I make the following change.

Thanks to check it, I think this change will cause false positives. Can you share your tsconfig?

Sorry, Johnson.
The issue should have been fixed. It looks like some caching problem on my side, I cannot reproduce it anymore.
Thanks again.


Import & render Ladle inside the application

Hey, first and foremost, thanks for the great library. It's really refreshing to get up and running in mere minutes compared to the hours (difficulties aside) I wasted with Storybook. 🙏
Is your feature request related to a problem? Please describe.
We have a repository that's housing the shared components for all other related projects. Right now, when developing, we're running a webpack dev server so that we can modify the shared components and reflect the changes in connected apps via HMR. The "problem" is that running the dev server does not make much sense, we only need to expose the shared components federation via network, but it would be great if we could somehow import the Ladle into the root component of our shared components app, and render it there as the main app while shared components dev server is running.
Describe the solution you'd like
Overly simplistic (I guess), but something like this would be nice:
// index.tsx
import ReactDOM from 'react-dom/client';
import Ladle from '@ladle/react';

import App from './App';

const root = ReactDOM.createRoot(document.getElementById('root') as HTMLElement);

root.render(
	<React.StrictMode>
		<App>
			<Ladle config={/* optional config here */} />
		</App>
	</React.StrictMode>
);

Describe alternatives you've considered
Storybook doesn't have something like this, not sure if there are some other "story" libraries with the functionality I'm asking for.
Additional context
Nothing right now.

I don't really understand what you are trying to achieve.
root.render(
	<React.StrictMode>
		<App>
			<Ladle config={/* optional config here */} />
		</App>
	</React.StrictMode>
);

This is not how Ladle works. The vast majority of its functionality is a custom vite plugin, vite itself and CLI. The UI is just a small piece of it.

@tajo Sorry, I guess I wasn't clear, basically, I want to render the Ladle in the application itself. For example:

start the application with yarn start which is using webpack under the hub
application opens up in the browser
Ladle navigation and stories are visible on the screen

Alternative would be to replace the last step from above by running the ladle separately. It's not a "problem", per see., but I would like, if possible, to avoid having another script running, and run the ladle directly in my app, with a webpack.

run the ladle directly in my app, with a webpack.

That's not possible. Vite is a direct replacement of Webpack. And Ladle is tightly coupled with Vite. So you have to start a Vite dev server for Ladle to function.


Readme update tasks list

Here is a list of changes to make to the README file: (Add more to comments)

Setting up database using initialize script initialize_database.sql
Setting up env vars using .env

Fixed with #3


Modify Rule.send_to_targets() to handle event_bus_name as ARN

Fixes #3803 by converting event_bus_name (which could be an ARN) to a simple name when sending an event to a Rule's targets. This is verified through a new test that constructs an Archive, sends an event where EventBusName is an ARN, and confrims that the event is routed to the Archive.

Codecov Report

Merging #3804 (457118e) into master (9f9716e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3804   +/-   ##
=======================================
  Coverage   94.81%   94.81%
=======================================
  Files         442      442
  Lines       48440    48441    +1
=======================================
+ Hits        45927    45928    +1
  Misses       2513     2513

Flag
Coverage Δ

servertests
55.85% <0.00%> (-0.01%)
:arrow_down:

unittests
94.67% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/events/models.py
93.41% <100.00%> (+0.01%)
:arrow_up:

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 9f9716e...457118e. Read the comment docs.


Rmarkdown Background Process Failure

System: Fedora 35 (but also failed with Fedora 34)
What happens: Knitting a file in the current R terminal (radian) works fine. When I try to use the extension to preview the resulting file (or if I use the background process i.e., smart knitting) to knit my rmarkdown file, I instead get the following error
Error in cat(lim, eval(parse(text = knit_command)), lim, sep = "") :
  object 'lim' not found

Which I deduce from  vscode-R/R/rmarkdown/knit.R has to do with the line
lim <- Sys.getenv("VSCR_LIM")

The same also occurs with a "fresh" install of VSCode (deleting all config / extensions, loading the vscode-R extension only). Maybe some explanation of the logic in the knit.R file can assist in debugging.
EDIT: The extension was previously working fine for me. Possibly this recent PR #784 broke functionality for me. Confirmed that reverting to extension version 2.2.0 fixed this functionality regression for me.

That PR shouldn't have affected the knitting delimiter, so my gut feeling is that it may have been a coincidence. Are you still running into this issue?

Hi Elian, thank you for responding to my issue. My bisect could be wrong, but I have just confirmed that I am still running into this issue with the latest version 2.3.2, and that downgrading to 2.2.0 restores previous functionality.

No worries! Hmm, it's very weird given that 'VSCR_LIM' is explicitly set prior to launching the knitting process. Just to be clear, when using the knit button, are you using the background knit functionality, or is it sending a knit command to the R terminal? IIRC, background knitting wasn't added until 2.3.0

Bug is only encountered when using the background knit functionality. Unfortunately, the "Preview Rmd" function seems to ignore the setting (i.e., bug is encountered when trying to preview the Rmd regardless of whether the "use background process" box is checked in the settings).
One possible fix (which sidesteps the breakage of the background knit, but I don't really care for it personally) is to make the preview Rmd function respect that setting.

Right - the preview button ignores the setting as it is intended for the knit button (currently, it is isn't possible to preview documents knit in the R Terminal). The intended behaviour is that if the user wants to opt out of smart knitting/revert to older behaviour, they can toggle that setting.
The main issue is figuring out why the knit variables aren't being set appropriately, causing the preview to crash.

Sorry I wasn't able to look at this again. Is there anything else I can do to help figure out why the knit variables aren't being set appropriately? Am not familiar with how to write VSCode extensions, but happy to provide any logs that are needed (properly anonymized).

No worries at all! There are 2 places with useful error information - the R Markdown output stream (which you've sent some of), and the developer console.
Developer tools can be accessed by the default shortcut Ctrl+Shift+I or the 'Toggle Developer Tools' command in the command palette. If there are any errors in the console there, they may be useful!

As an aside, here is some background on the knit process if it is at all helpful.
Environment values are set in knit.ts:
const lim = '<<<vsc>>>';
const re = new RegExp(`.*${lim}(.*)${lim}.*`, 'gms');
const scriptValues = {
'VSCR_KNIT_DIR': knitWorkingDirText,
'VSCR_LIM': lim,
'VSCR_KNIT_COMMAND': knitCommand
};

These are passed to the knit process during its creation in rmarkdown/manager.ts:
const processOptions: cp.SpawnOptions = {					env: {
...process.env,
...scriptArgs
},
cwd: args.workingDirectory,
};

Which is then accessed via Sys.getenv(...) in R.

HI, i had a similar issue.
Error in cat(lim, rmarkdown::render(file_path, output_format = set_html,  :    object 'lim' not found
Likewise, when I downgraded to 2.2.0, it worked normally. Will it be able to be fixed through an upgrade in the future?

HI, i had a similar issue.
Error in cat(lim, rmarkdown::render(file_path, output_format = set_html,  :    object 'lim' not found
Likewise, when I downgraded to 2.2.0, it worked normally. Will it be able to be fixed through an upgrade in the future?

Hi there! Could you please post your developer tools logs? Developer tools can be accessed by the default shortcut Ctrl+Shift+I or the 'Toggle Developer Tools' command in the command palette.

Hi Elian, thanks for engaging us on this. The 2.2.0 workaround is going to fail for me because I'm shifting towards Quarto (which requires the latest version of this extension) but still have some .Rmd files for historical purposes. I haven't been able to get the developer logs to properly show, but I have identified the issue.
A common practice in scripts is to clear the environment using rm(list = ls()) and it is the inclusion of this line at the start of my .Rmd scripts that creates the issue. Removing it solves the problem. One solution is simply not to clear the environment (or maybe I can detect when the extension is calling the script and only trigger the cleanup if the extension is not the one calling the script) - but I think this is a fairly common use-case (to clear the environment to ensure reproducibility).
Hopefully this will help the team triage the issue and decide whether to fix it. A simple "hello world" Rmd with the rm(list = ls()) command will trigger the error. I can completely understand if the issue is marked WONTFIX (though that would be unfortunate).

Hi Andrew,
Thanks for the information -- super useful! If rm is the issue, I think this is relatively straight forward to implement. Looks like we need to hide away the rmarkdown variables in a new environment so it isn't touched by rm

Hi again @AndrewCSQ  and @Albatrosses-reply, the latest push to the repo should have fixed this issue. If you find you still run into it, please feel free to open!

It took a little bit of searching to figure out how to install the correct version of R extension for vs-code.
It should also be noted that as of latest commit https://github.com/REditorSupport/vscode-R/commit/8f5f78172e4bade335a52778e0759fb319b46aaf, there exists a module interpolation issue that requires fixing prior to being able to install the extension.
The solution below does not have this problem and works for both the preview + knit feature.
git clone https://github.com/REditorSupport/vscode-R.git # or ssh alternatively
git checkout 3ed84833d4deffc381259757e85628aff8c878bd #1117 fix
# npm install -g yarn (if you don't have it already)
yarn install
vsce package
code --install-extension r-2.5.0.vsix


🐛 Bug Report: Unable to change the logo

👟 Reproduction steps
Try to change the logo on a running pingvin instance
👍 Expected behavior
Logo should be changed to the newly uploaded file
👎 Actual Behavior
Logo remains original (uploaded  ~1y ago) file
📜 Logs
Haven't got any logs, cause I don't know where they are, but I already tried:
-waiting multiple hours after uploading
-clearing browser cache
-restarting docker container
-rebuilding docker container
-restarting host vm
-uploading logo.png manually to /data/images/logo.png
-updating pingvin to the newest version

Are the images in /data/images/ the new logo or the default one?

i can't check the image directly, cos I only have shell access, but the file gets the current date in the "last modified" field, everytime I upload a new logo, so I would assume the upload works

Found a way to access the images folder and downloaded the logo.png file and it is indeed the correct "new" one, but the file on the page, still hasn't updated

Okay thanks. Can you share your docker-compose.yml? What if you try it in a private browser window?

My docker-compose file is basically the standard one from the guide without ClamAV and weirdly enough, if I open the site in a private window in works, indicating, the logo is cached in my browser, but disabling the cache doesn't work... I use firefox btw

Yes, that actually fixed the issue...
 Thanks very much for the help XD


Fixes language mistakes in German.strings

This fixes major grammar mistakes in German.strings :-)

Hey @larsschwegmann, thanks for the PR. How does this compare to #162? I'm not really in a position to judge these kinds of changes.

@nickoneill Oh my bad, I didn't check if there was already a PR updating that file. The changes made in #162 are fine and comparable to mine. I guess I'm closing this then. Merging his changes should be ok :)
Thanks for responding so quickly! PermissionScope is pretty awesome 👍


Test results are not merged when running on many nodes

Having tests run in parallel in a pipeline and picking them up under the same path does not merge the results. The results get overwritten and only the last parallel stage to finish gets recorded.
step([$class: 'QuarantinableJUnitResultArchiver',
  testDataPublishers: [[$class: 'ClaimTestDataPublisher'], [$class: 'QuarantineTestDataPublisher']],
  testResults: 'path/to/results.xml'])

I've tried updating the junit plugin to the latest version (1.28 as of this report) as well as bumping the jenkins version dependency to 2.138.4 to match the upstream junit plugin. I also tried updating the jenkins plugin dependency to version 2.23 without any success. Going further than that in version produces compilation errors:
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /home/builder/git/quarantine/src/test/java/org/jenkinsci/plugins/quarantine/QuarantineUiTest.java:[126,44] cannot find symbol
  symbol:   method getHtmlElementsByTagName(java.lang.String)
  location: variable form of type com.gargoylesoftware.htmlunit.html.HtmlForm

Actually this seem to work if you just name each result slice differently. I.E. test-results-1.xml, test-results-2.xml


Epic 5859 : Error validation and display in domain designer

Test failure on teamcity doesn't seem to be related to my recent changes.


v1 API deprecated

Hi,
unfortunately it seems the project is broken due to the v1 API being deprecated and the schedule stop pair endpoint is no longer working. (https://github.com/transitland/transitland/discussions/396#discussioncomment-5651874). It would be great if it could be updated to use the v2 APIs.

I'm trying to update the project to v2, I've made a fork: https://github.com/AlexanderPuckhaber/transitland-processing-animation/tree/api_v2_attempt
Unfortunately, the data provider transit.land deprecated the ScheduleStopPair object and associated endpoint. That appears to be necessary for the motion of the vehicles.
See: https://www.transit.land/news/2019/10/17/tlv2.html#gradual-deprecation-of-tlv1
Maybe there is an alternative way to get the trip information of "when did the vehicle stop at each stop", but I'm not sure.
We would need equivalent functionality to the origin/destination stop/arrival times found in the v1 API
Transitland v2 has Trips: https://www.transit.land/documentation/rest-api/trips
However, I am unable to get the stop_times array in the response because I cannot figure out how to request "a single trip by ID


WIP: Add documentation for new analyzers

Fixes #27366
Internal review URLs:

File Type
File Name
Published Url

content
aspnetcore/diagnostics/code-analysis.md
https://review.learn.microsoft.com/en-us/aspnet/core/diagnostics/code-analysis?view=aspnetcore-7.0&branch=pr-en-us-27727

content
aspnetcore/diagnostics/asp0015.md
https://review.learn.microsoft.com/en-us/aspnet/core/diagnostics/asp0015?view=aspnetcore-7.0&branch=pr-en-us-27727

content
aspnetcore/diagnostics/asp0016.md
https://review.learn.microsoft.com/en-us/aspnet/core/diagnostics/asp0016?view=aspnetcore-7.0&branch=pr-en-us-27727

content
aspnetcore/diagnostics/asp0017.md
https://review.learn.microsoft.com/en-us/aspnet/core/diagnostics/asp0017?view=aspnetcore-7.0&branch=pr-en-us-27727

content
aspnetcore/diagnostics/asp0018.md
https://review.learn.microsoft.com/en-us/aspnet/core/diagnostics/asp0018?view=aspnetcore-7.0&branch=pr-en-us-27727

content
aspnetcore/diagnostics/asp0019.md
https://review.learn.microsoft.com/en-us/aspnet/core/diagnostics/asp0019?view=aspnetcore-7.0&branch=pr-en-us-27727

@captainsafia -- If sample code would be helpful in any of these, please put it in a comment or you can push directly to the tdykstra-analyzers15to19 branch. Also please double-check the "When to suppress" sections as I left them all as "Do not suppress".


test: move unit unit

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Remove to_dict and make Cond object support the dict constructor

What's Changed
With this change, getting a dict from Cond is more easy and beautiful.
c = Cond(...)
dict_of_c = dict(c)

Added

__iter__ to Cond

Removed

to_dict method of Cond

I will make all the calls to all to_dicts to be replaced by dict and then will create another pr


Use mimalloc in rust binarytree

I could not find a good general purpose allocator written in pure rust.
But mimalloc, which is written in c, is really easy to use.
It requires just two line to enable globally.
The check function also takes self by value, for another small improvement.
This helps because the deallocation can already be done inside check.
@hanabi1224 what do you think?

FFI via non-stdlib is typically not accepted, and specially marked with -ffi suffix when they are.  In this case, I don't see how mimalloc represents rust's unique value, other languages can easily use it as a drop-in replacement, that just end up measuring the libary itself, that's exactly why GMP usage via non-stdlib is not accepted here.

Hmm, why is FFI allowed in the stdlib but not in any other lib?
This seems like an arbitrary distinction to me.
In practice nobody cares if functionality is in the stdlib or a different lib as long as they are both easy to use.
This rules seems to just favour large stdlibs for no reason.

stdlib represents the unique value in the language, while arbitrary FFI usage does not.  For all problems, people can build a dynamically linked lib from the fastest C program and use it in most of the languages via FFI, thus the results compare nothing but the FFI overhead, which is not what the problems are designed for

@hanabi1224 would this PR be acceptable if I made two versions?

6-ffi.rs, which uses mimalloc
6.rs, which is pure rust

Oh the irony. Rust devs improving Rust benchmarks by using a library written in C.


How to hide heading sections?

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
I would like to hide the "Shelves" and "Books" headers on the top-right of the webpage and only have these enabled when the user logs in.
I thought of using CSS in the Custom HTML Head Content section in Settings so as a test, I tried to hide the entire heading field to see if it works by using .header {display:none;} but the header section is still displayed.
Is this the right method to hide the "Shelves" and "Books" headers or is there a more efficient method?
Exact BookStack Version
v22.02.3
Log Content
No response
PHP Version
7.4.3
Hosting Environment
Ubuntu 20.04, Bookstack installed using this script: https://github.com/BookStackApp/devops/blob/main/scripts/installation-ubuntu-20.04.sh

The following will hide both of those menu items using JavaScript:
<script>
    window.addEventListener('DOMContentLoaded', function() {
        const isLoggedIn = document.querySelector('header form[action$="/logout"]') !== null;
        if (!isLoggedIn) {
            const linksToHide = document.querySelectorAll('header a[href$="/books"], header a[href$="/shelves"]');
            for (const link of linksToHide) {
                link.style.display = 'none';
            }
        }
    });
</script>

Unfortunately, there will still be a flash of these items before the JavaScript runs, and they will show if JavaScript is disabled. It's not yet possible (In a widely supported way) to achieve this with CSS alone in the current configuration of BookStack.
For better customization, you could override the header bar template using the visual theme system.

Since there's been no follow-up I'm going to close this. If the issue remains and is something you still require to be fixed respond to my previous comment and this can then be re-opened.

Sorry for the late reply. Thanks for your comment, I think the best method for me is just to edit the header-blade.php file directly and comment out the lines which displays the "Shelves" and "Books" headings.

No worries @peteSmith1229,
Editing core app files can mess with update updates though, Using the visual theme system I linked above would allow you to make your modifications without editing original app files.

That's good to know, thanks again!


✨ Improve query analyzer language selection

Purpose
Each field defines its own default search analyzer in its mapping (or defaults to the content analyzer if a specific search query analyzer is not defined). This is appropriate in principle but it
breaks down when we attempt to do multilingual searches: search queries are then analyzed using the field's analyzer or search analyzer, which might be in the wrong language, and give terrible results for the search query.
To prevent this from happening, we decided to enforce an analyzer at query time. If we can find an appropriate analyzer, it can then be used before searching even in indices in other languages.
Proposal
To best find this appropriate analyzer, we let the frontend client pass us a preferred query language, which it can determine by guessing and offering guesses to the user to help them, or other UI elements.
What we suggest:

do not change the base UI at all;
guess query language in the frontend (using the same Java langdetect lib as ported to javascript instead of python);
when our confident guess differs from the current language and we can offer a better analyzer (eg. the language we guessed is supported by the current Richie instance), show a toast to the user offering this: It seems like your search query is in Spanish, would you like to search in Spanish?;
if the user does change the query language, we then display a language selector next to the search input.

This way we can be helpful wrt. query language and irrelevant results, without breaking search behavior due to the inherent unreliability of attempting to guess the language of a very small text sample. We also don't pollute the UI for something that 95-99% of users will never need.
In the absence of a cue by the frontend (which would only be there for less than 5% of queries), we then default to the current UI language.
In any case, we can keep searching in all languages' versions of the fields and be sure we have the best analyzer we could offer.
Alternatives considered
We started out trying to just automatically detect the query language and using that guess to choose the most correct analyzer without any user input besides their text query. We wanted to use langdetect for this.
langdetect is a great library, but it cannot provide reliable guesses in the absence of enough data. The author of the original library (in Java) notes here that it is not reliable for tweets which are too short (at 100-500 characters), and that for automatic detection to be worthwhile, it needs to be very reliable.
They echo this note from Elastic that notes ElasticSearch's own language detection should not be used for search queries as it's unreliable for text shorter than 50 characters.
Additionally, we observed as we attempted it that it fails for common queries attempted in Richie. More importantly, it produced unexpected behavior as the guesses changed when using search-as-the-user-types: if the analyzer changes through different languages as the user types their query, the search results will also flicker between vastly different result sets instead of progressively narrowing to our best results.
TODO

[x] set an analyzer for full-text queries in the backend;
[x] accept a new query string parameter on search API endpoints so the frontend can let us use the user's preferred language (as opposed to the current UI language);
[ ] help the user select an appropriate query language in the frontend;

Fixes #997.

Update
After some consideration, we decided that trying to guess the language in the frontend and adding a query parameter was too much work and "trying too hard" in terms of a UX we have not seen elsewhere.
For now, we can fix what needs fixing by just analyzing queries in the current active language. If we see problems with this approach, those problems will help guide us to the best way to handle language in query analysis later on.


PCIe TPU Compatibility with AXI-PCIe Root Port - MSI supported but not MSI-X

Description
Hello,
I'm trying to use the AXI-PCIe root port bridge offered by Xilinx with the PCIe Edge TPU.
In the root port configuration, it appears the AXI-PCIe root port bridge only supports single MSI, but not MSI-X: https://github.com/Xilinx/linux-xlnx/blob/master/drivers/pci/controller/pcie-xilinx.c#L273
@mbrooksx indicated that the Edge TPU with PCIe requires MSI-X in Issue 122. Are there any Apex driver modifications we can make to use just single MSI, but not MSI-X?
Any advice would be very much appreciated.
Click to expand!
Issue Type
Support
Operating System
Linux
Coral Device
Accelerator Module
Other Devices
No response
Programming Language
No response
Relevant Log Output
No response

Hi, AXI-PCIE root bridge documentation says it supports MSI-X. If its not, please check with them on how to change pcie-xlinux.c to support for MSI-X. Thanks!!


Fix pools name on mapping.ts

can you fix the merge conflicts?

can you fix the conflicts?

The conflicts are the already merged details, I only changed the name of the pools and changed the imageURL for each pool


BUG/ENH: lsq_linear: fixed incorrect lsmr_tol in first call to lsmr; added lsmr_max_iter option and unbounded output

What does this implement/fix?

Bug fix: In the first call to lsmr, the lsmr tolerance option was set to lsq_linear's tol option as opposed to lsmr_tol as it's supposed to. I corrected this (including the cases if lsmr_tol is None or 'auto') and added error checking for the lsmr_tol argument.
New features:

lsmr_max_iter option that gets passed to the lsmr solver; setting this higher than the default value is necessary for poorly conditioned matrices, so this enables the use of lsq_linear with poorly conditioned sparse matrices
x_unbounded attribute added to the OptimizeResult return value to give the user access to the unbounded least squares solution

@tylerjereddy, checking in here. Is there anything I need to do to get this PR reviewed by a scipy.optimize expert or is it just a matter of waiting? I can imagine that given the number of daily PRs, being a reviewer isn't an easy job.

@tylerjereddy any updates? We're hoping to use this feature through scipy as soon as possible.

I put a 1.9.0 milestone on it to prevent it from slipping through the cracks. You could try using git blame to see how has historically maintained the code and ping them. That's probably what I'll try doing when I come back to this, assuming that person is still active.

Great, thanks! It looks like the code was originally developed and maintained by @nmayorov and was most recently updated by @tupui. Could either of you review this PR?

Thank you for your comments @tupui. I've made all the requested changes. If not you, who should I ping to expedite the review process for this PR? I haven't made any changes to the core algorithms.

@eytanadler thank you for the update. You already pinged Nikolay and Tyler added a release milestone. So just needs time for someone to go over.
Quick note, I can see some issues in CircleCI. Usually it means that you should update with main. Could you merge main? Not rebase this is important.

Got it, thanks for the help @tupui!

Thank you @rgommers for your review and small improvements!
Regarding maxiters, you're correct that I was going for consistency, but I'm happy to make the change. I'll push an update with the new name shortly.

CI is passing with 100 % patch diff coverage and Ralf seems pretty happy here + other core devs have had their comments addressed (myself included).

thanks!

Fantastic, thank you all!


unexpected result

I was hoping to use this to validate github URLs for repos and users, but the following returns false:
isGithubUrl('https://github.com/alferov');

which I think a user would expect to be a valid github URL, or is this module only for validating github-git urls? thanks!

Oh, It's definitely should be there as it's a valid GitHub URL. It would be also nice to provide a set of  different rules to give a user control over validation.
I'll implement that in next release.

thanks!


Fundamental Issues with Deployment Process

Describe the bug
There are some fundamental issues with the #current-cloud-backend.zip and amplify-cfn-templates folder in the deployment S3 bucket. Consider the following (which happened to us in production, ouch):

A bad value is put in a cloudformation template (for example, hosting/template.json)
amplify publish fails (as it should), but amplify-cfn-templates/hosting/template.json is still updated with the bad template
hosting/template.json is reset locally, and other changes are added - when amplify status is run, hosting/template.json appears to not have any changes to deploy, but other changes will be deployed
When amplify publish is run, the local hosting/template.json is not uploaded to the S3 deployment bucket (since it matches up with what is in #current-cloud-backend.zip)
The bad template in amplify-cfn-templates/hosting/template.json is deployed as part of the cloudformation change set, and the deployment fails
As long as amplify publish is run with an unmodified (and correct) local hosting/template.json, the deployment will fail
Only once local hosting/template.json is modified will the deployment succeed

Amplify CLI Version
3.17.0 (although this issue has existed for a long time)
To Reproduce
See above
Expected behavior
I expect the local cloudformation templates to be deployed when I run amplify publish every time, regardless of the state of files in the S3 deployment bucket.
Screenshots
N/A
Desktop (please complete the following information):

MacOS (not relevant)
Node Version v10.14.1 (not relevant)

Additional context
I think the best fix for this issue is to upload all local cloudformation templates to the deployment bucket every time amplify publish is run, regardless of whether or not there are diffs with current-cloud-backend

I've had something similar happen to me too!
It would be really nice if Amplify ran something like ValidateTemplate on all the templates it is about to upload/use. Especially in production.
Even better (but likely not happening) would be to use cfn-lint across everything (since ValidateTemplate has various limitations, like 51KB maximum file size). I have a feeling that using a Python dependency in a TypeScript application might not make all of the Amplify userbase very happy. Maybe it could be an optional (if present run check) pre-push/publish hook.

We actually do run cfn-lint on all our templates before uploading them - it doesn't prevent you from uploading a non-existent (but syntactically valid) value, and then you can still end up in this state.

Closing as we have since added cfn-lint to run on push, promoting validation errors as warnings to the deployment process. We do have another item in our backlog to validate the CFN templates and parameters to further ensure invalid templates are not uploaded and fail fast.


tutorial: plugins

Hi,
I'm considering removing the "install abapGit Plugins" from the tutorial, in the future I suggest everything to be supported from the main repo.
Any thoughts?
Also see https://github.com/larshp/abapGit/pull/823

Good idea! However, there are plenty of objects we version by abapGit-plugins - IWPR, AOBJ, ...

Yep. Will do ASAP.

Here is the list. I will create the issues later on.
 AOBJ
 ASFC
 ASIS
 AVAS
 IWMO
 IWPR
 IWSV
 IWVB
 OCAO
 SAJC
 SAJT
 SUSH

CHDO Change Documents

AOBJ
ASFC
ASIS
AVAS
IWMO
IWPR
IWSV
IWVB
OCAO
SAJC
SAJT
SUSH
CHDO

@jfilak do you still use the plugins? I'd like to delete the bridge code


Update google-cloud-pubsub to 1.114.2

Updates com.google.cloud:google-cloud-pubsub from 1.107.0 to 1.114.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

labels: library-update, semver-minor

Superseded by #258.


Add 'single-zero' functionality to RBA

It's available now! Still need to implement some of the split and 3 number bets for Single 0.


return result object instead of boolean in firstOrLastLine

isFirstLine and isLastLine functions expect a results object-literal, not a Boolean.

Huh, thought this is social app? @natejenkins sorry, can revert if you want me to

reverting and re-opening, just to safe

apologies again


when terminal address have Chinese， will be repeated

VSCode Version:1.13.0
OS Version:win10

Steps to Reproduce:

Duplicate of #26038
@baixiaoyu2997 As I understand, this is already fixed and will be released soon.

Thanks @CherryDT, that's right


Allow * atom insertion in Ketcher

Background
It doesn't seem possible to add dummy (*) atoms in Ketcher. We would like to use Ketcher in the new ChEBI database development and this is an important feature since in ChEBI * and R groups, and A (any) atoms have a very defined meaning
Differenciating * from A is quite important for ChEBI since they have a meaning (related to the Ontology):
* = attaches to something (meaning it is a fragment)
R = something attaches here (meaning it is a core structure where a fragment will attach)
A = any single atom
Solution
button allowing the insertion of * atoms like MarvinJS has.

Hi @eloyfelix
Thanks for your request.
Could you please provide some examples in different formats (SMILES, Mol etc) with such atom (please try to provide small molecules) and expected output on the canvas

First example:
https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI:47265
molfile: ChEBI_47265.mol.zip
SMILES: Br*
It should show in canvas (see the selected button in MarvinJS in the image would be the expected functionality):

Second example:
https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI:142521
molfile: ChEBI_142521.mol.zip
SMILES: [C@@]1(O[C@]([C@H](NC(=O)C)[C@H](C1)O)([C@@H]([C@H](O)CO)O)[H])(C(O)=O)O[C@H]2[C@H]([C@H](O[C@H]([C@@H]2O)*)CO)O[C@H]3[C@@H]([C@H]([C@@H](O)[C@H](O3)CO)O)NC(C)=O

Many thanks for looking into it!

Meeting note: Considered as moderate to fix.
Preliminary milestone - 3.1

Many thanks, @AlexeyGirin! I noticed there are no dates for the milestones. Do you have a rough idea of when this might happen? Even something like "sometime in 2025" would be helpful.
Thanks again for considering the feature.

Hello, @eloyfelix!
Thank you again for your interest.
We will be adding the star/dummy atom in release 3.1. You can expect the final version of it in mid-March 2025 (with the beta version available a little sooner than that).


fix: [EP-1993] hide widget footer when no data available

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


Config.js Issue

Here is the error:
`WARNING! Could not validate config file. Starting with default configuration. Please correct syntax errors at or above this line: /home/pi/MagicMirror/config/config.js:164
module: "MMM-GooglePhotos",
^^^^^^^^^^^^^^^^^^
SyntaxError: Unexpected string
at new Script (vm.js:74:7)
at createScript (vm.js:246:10)
at Object.runInThisContext (vm.js:298:10)
at Module._compile (internal/modules/cjs/loader.js:678:28)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:722:10)
at Module.load (internal/modules/cjs/loader.js:620:32)
at tryModuleLoad (internal/modules/cjs/loader.js:559:12)
at Function.Module._load (internal/modules/cjs/loader.js:551:3)
at Module.require (internal/modules/cjs/loader.js:658:17)
at require (internal/modules/cjs/helpers.js:20:18)
`
Here is the part of the config.js file:
`
module: "weatherforecast",
position: "top_right",
header: "Weather Forecast",
config: {
location: "New York",
locationID: "5128581",  //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
appid: "YOUR_OPENWEATHER_API_KEY"
}
},
]

},
{
module: "MMM-GooglePhotos",
position: "top_right",
config: {
albumId: ["AEt1YzuZGt8nJSGfLaXwkg074eaO7PlwTcyEiA5e_waSeE0xCCsMAgW104vwOBtdcVm_deXB4cdN"], // your album id(s) from result of auth_and_test.js
refreshInterval: 100060,
scanInterval: 100060*10, // too many scans might cause API quota limit also.
//note(2018-07-29). It is some weird. API documents said temporal image url would live for 1 hour, but it might be broken shorter. So, per 10 min scanning could prevent dead url.
    sort: "time", //'time', 'reverse', 'random'
    showWidth: "800px", // how large the photo will be shown as. (e.g;'100%' for fullscreen)
    showHeight: "600px",
    originalWidthPx: 800, // original size of loaded image. (related with image quality)
    originalHeightPx: 600, // Bigger size gives you better quality, but can give you network burden.
    mode: "hybrid", // "cover" or "contain" (https://www.w3schools.com/cssref/css3_pr_background-size.asp)
    //ADDED. "hybrid" : if you set as "hybrid" it will change "cover" and "contain" automatically by aspect ratio.
}

};`
I double checked that that is the name of the file. I don't know what it is. I've been having problems with config.

Error says you might have some error(mistypo) in line 164 of config.js. You might be probably missing some comma or bracket over the line 164.


Add proxy pass for test homu

This change is

r? @edunham

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

Comments from Reviewable

@bors-servo r+

:pushpin: Commit 6c0822a has been approved by edunham

:hourglass: Testing commit 6c0822a with merge c743665...

:sunny: Test successful - status-travis


SPACESHIP_PROMPT_FIRST_PREFIX_SHOW must place before theme

Description
SPACESHIP_PROMPT_FIRST_PREFIX_SHOW=true environment must be set before theme.
Screenshot
Place SPACESHIP_PROMPT_FIRST_PREFIX_SHOW=true after theme

Place SPACESHIP_PROMPT_FIRST_PREFIX_SHOW=true before theme

Thank you @trmaphi. But it's not specific to SPACESHIP_PROMPT_FIRST_PREFIX_SHOW. See https://github.com/denysdovhan/spaceship-prompt/issues/478#issuecomment-430762510

Thank you @trmaphi for the PR. But it's not specific to SPACESHIP_PROMPT_FIRST_PREFIX_SHOW. See https://github.com/denysdovhan/spaceship-prompt/issues/478#issuecomment-430762510 and https://github.com/denysdovhan/spaceship-prompt/issues/407#issuecomment-377730733

Oh, I see. Then, I'm gonna close this MR>


Implement progress bar, multithreading

Hi all,
I implemented multithreading in the dimensionality reduction widget, so now napari does not crash during the computation. I've tried also to implement a progress bar (#36). However, my long-running function (umap or tsne) are not generators, therefore, the progress bar that I get is only an indeterminate progress bar (video below).
https://user-images.githubusercontent.com/52177660/152180924-b0d4281a-362a-4b6e-986f-05023f1cdf9c.mp4
Umap itself is also able to provide a tqdm progress bar in the console:

However, since umap is running in a secondary thread I was googling is it possible to somehow make it that the main thread receives this umap tqdm and it is displayed in napari itself not the console. I found that "PyQt widgets cannot be modified from any thread other than the application thread, and since the updates to stdout and stderr are coming from the secondary worker thread it would violate that". This is from this stackoverflow thread, which I thought could help me, but I cannot grasp the code there:
https://stackoverflow.com/questions/21071448/redirecting-stdout-and-stderr-to-a-pyqt4-qtextedit-from-a-secondary-thread
I thought I could somehow get the umap tqdm to output to another widget I create instead of relying on napari's progress bar, which is not able to display actual progress because the worker is a function, not a generator, so nothing is yielded. Does it seem doable to you? Or do you think it's too much struggle and it's enough to have this progress indication as it is now?
I know it is possible to have two-way multithreading, where it is possible to send and receive values to and from the main thread, but then again a generator is needed:
https://github.com/napari/napari/blob/6a3e11aa717e7928a0a5a3c7693577729a466ef1/examples/multithreading_two_way.py
It seems I cannot figure this out on my own. @jo-mueller, I think you are quite an expert with tqdm, maybe you will be able to help me out? @haesleinhuepf, thoughts?
Thanks a lot for any feedback!
Best,
Laura

Hey Laura @lazigu ,

I found that "PyQt widgets cannot be modified from any thread other than the application thread

I think this may be only partially true. You can install signals to your threads which can be "emitted" by the thread and conected to from the outside of the thread. button.clicked is an example of this system. There is a tutorial on the napari page that explains how to do this (see at section: "Adding custom signals".)

and since the updates to stdout and stderr are coming from the secondary worker thread

I played around with that feature in a different context, but, retrospectively, it's more trouble that it's worth because while rerouting stdout (all print statements, tqdms, etc) to a custom output (like a custom-made progressbar) can be nice for visualization, it also mutes the IPython console which can make debugging incredibly difficult :/

the worker is a function, not a generator, so nothing is yielded
This may be the key problem here...do I understand it correctly that the umap function iterates over something (internally), which you would like to display somehow? I fear that it may be super difficult to actually catch and display the progress happening within umap. I think it may be wise to leave it with a sign that shows that the plugin is working and not frozen.

One thing I could think of: If the processing time of umap/other alogorithms scales in a predictable way with time to simply print out an estimate on how long it could take. And maybe a stop button if the result ends up being multiple hours or so ^^

I played around with that feature in a different context, but, retrospectively, it's more trouble that it's worth because while rerouting stdout (all print statements, tqdms, etc) to a custom output (like a custom-made progressbar) can be nice for visualization, it also mutes the IPython console which can make debugging incredibly difficult :/

Ah ok, so it really seems not worth it then.

One thing I could think of: If the processing time of umap/other alogorithms scales in a predictable way with time to simply print out an estimate on how long it could take. And maybe a stop button if the result ends up being multiple hours or so ^^

This is a good idea, I will look into it. Ryan @Cryaaa, by any chance maybe you have encountered estimating how long would umap take during your master's work?
Thanks for the help @jo-mueller!

Codecov Report

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

:exclamation: Current head 1c0901d differs from pull request most recent head 677a3f5. Consider uploading reports for the commit 677a3f5 to get more accurate results

@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage        ?   72.53%
=======================================
  Files           ?       14
  Lines           ?     1074
  Branches        ?        0
=======================================
  Hits            ?      779
  Misses          ?      295
  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 73f060c...677a3f5. Read the comment docs.

This is finished and ready for the review now. I left only napari progress bar, and multithreading was implemented in all widgets except plotting one (possible issue for the future).
This branch contains merged add_clustering_methods branch already, so it should be merged after #65, #66 and #67 are already merged, and released in the separate release in case of possible bugs. I will request review after other PRs are tackled.
It would also close #68.
P.s. Tests on macos-latest py3.9 run over 20 min now, it should be taken a look if it does not finish successfully.

Hey @jo-mueller,
I am getting the same error, but only for the call_to_function test, not test_umap. I was running tests just by typing in pytest:

Or also with coverage:
coverage run --source=napari-clusters-plotter -m pytest -v napari_clusters_plotter && coverage report -m
That´s how it looks like:

test_umap also passes if I run it directly in pycharm.


FFT View: Graphical Performance Improvements

FFTView: Graphical Performance Improvements

Added metal rendering of View
Removed unnecessary calculations on fftData that are not used in visualization
Added temporary array of amplitude floats to swap the amplitude array instead of adjusting element by element (I suspected that this results in less SwiftUI update events, but I am not certain)

Woohoo!

Woohoo!


Updated storage package versions to 1.1.0 for the azure mega-package

The old monolithic package azure-storage is getting installed along with azure, and it is causing problems for users who install the newer storage packages. Please let me know whether this should be targeting dev branch instead. Thanks!

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1978   +/-   ##
=======================================
  Coverage   53.17%   53.17%
=======================================
  Files        4714     4714
  Lines      117148   117148
=======================================
  Hits        62295    62295
  Misses      54853    54853

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 80fc6f0...6d344ed. Read the comment docs.


Appearance

Goliath lacks in Appearance which is present in other applications like WordPress.

Can you elaborate what do you mean by appearance?

Theme options
Customize options
Widgets


Install dependencies issue

I had a problem when install dependencies by running script reload_deps.sh.
How to fix it?
##Deno i'm using
% deno --version
deno 1.13.2 (release, x86_64-apple-darwin)
v8 9.3.345.11
typescript 4.3.5

I had also tried deno 1.11.5, the error is same

##Here is the error:
Check file:///Volumes/Data/work/tokenomic/deno_rest/deps.ts
error: TS2345 [ERROR]: Argument of type '"jwk"' is not assignable to parameter of type '"raw"'.
        "jwk",
        ~~~~~
    at https://deno.land/x/god_crypto@v1.4.10/src/aes/aes_wc.ts:25:9

TS2345 [ERROR]: Argument of type '"jwk"' is not assignable to parameter of type '"raw"'.
    "jwk",
    ~~~~~
    at https://deno.land/x/god_crypto@v1.4.10/src/rsa/rsa_wc.ts:55:5

TS2304 [ERROR]: Cannot find name 'HmacKeyAlgorithm'.
): algorithm is HmacKeyAlgorithm | RsaHashedKeyAlgorithm {
                ~~~~~~~~~~~~~~~~
    at https://deno.land/x/djwt@v2.4/algorithm.ts:24:17

TS2304 [ERROR]: Cannot find name 'RsaHashedKeyAlgorithm'.
): algorithm is HmacKeyAlgorithm | RsaHashedKeyAlgorithm {
                                   ~~~~~~~~~~~~~~~~~~~~~
    at https://deno.land/x/djwt@v2.4/algorithm.ts:24:36

TS2304 [ERROR]: Cannot find name 'EcKeyAlgorithm'.
): algorithm is EcKeyAlgorithm {
                ~~~~~~~~~~~~~~
    at https://deno.land/x/djwt@v2.4/algorithm.ts:30:17

TS2339 [ERROR]: Property 'hash' does not exist on type 'KeyAlgorithm'.
        return keyAlgorithm.hash.name === algAlgorithm.hash.name;
                            ~~~~
    at https://deno.land/x/djwt@v2.4/algorithm.ts:47:29

TS2339 [ERROR]: Property 'namedCurve' does not exist on type 'never'.
        return keyAlgorithm.namedCurve === algAlgorithm.namedCurve;
                            ~~~~~~~~~~
    at https://deno.land/x/djwt@v2.4/algorithm.ts:49:29

Issue was fixed with deno version 1.14


Libp2p testing overhaul

This issue will track the work pipeline regarding libp2p testing. The two main points here are the integration testing ownership and thorough System Testing + Interop Testing
Integration Testing
Currently, we have integration tests for libp2p modules, both on libp2p modules and libp2p itself. Despite causing an obvious testing overlap, this creates a problem on Node.js@15 with mismatch versions when a libp2p peerDependency is installed on a module. See more
The best solution to solve the above problem is to delegate libp2p modules to completely handle their integration tests. Libp2p would focus entirely on testing its API, configuration and core components, while the modules would be responsible for the integration and test the subsystem in depth.
Libp2p interfaces should be expanded to export all integration tests needed per type of module (pubsub, peerDiscovery, peerRouting, ...). With this, libp2p modules can require and use these tests by simply requiring them and providing instances of the module. This enables us to have less replicated code as the boilerplate to test different pubsub/peerDiscovery modules, while keeping a guarantee that these tests will run.
Before releasing a stable release of libp2p, we should have visibility of possible side effects of a new release within the context of each libp2p module, in order to guarantee that no regressions are introduced on the modules before release.  We can achieve this by using Github Actions Worflow actions dispatch. We can leverage dispatch-action to let other repos know of the changes and run their workflow. To run additional dependency versions, we can use cross-env and test them similarly to dev.to/joshx/test-your-npm-package-against-multiple-versions-of-its-peer-dependency-34j4. The easier solution is probably to create nightly releases with commit in master and verify each libp2p module before cutting the final release.
With the above changes in practise, libp2p package json would only include as devDep the essencial modules to test its basic APIs (transports, streamMuxer and connEncryption). On the other side, libp2p modules would install libp2p as a peerDependency and devDependency using semver ranges.
To sum up, the above proposal includes:

libp2p tests its basic API and core component
libp2p-interfaces exports integration tests per module type
libp2p modules are responsible for running their integration tests with libp2p
?libp2p should have CI visibility on side effects affecting its modules

Other ideas:

In the future, we can have a js-libp2p-integration module with the integration tests, if we go with a lerna repository #824
We can do master commit release builds in js-libp2p as a way to test modules.

Thorough System Testing and Interop Testing
With the testground js sdk on its way, we can start to specify what tests plans we would like for a Thorough System Testing.
Other ideas:

Perhaps a module js-libp2p-testing within the Lerna setup, which would include integration tests and test plans.

Related:

#790

@jacobheun let me know your thoughts on this proposal, so that I can iterate on a more actionable plan with tasks + milestones

@jacobheun let me know your thoughts on this proposal, so that I can iterate on a more actionable plan with tasks + milestones

I did a PoC with pubsub for the integration testing proposal. There is also one point in https://github.com/libp2p/js-libp2p-interfaces/pull/81 description that would be good to consider. Shortly, interface tests should receive an instance of libp2p instead of an instance of the module?
@achingbrain @hugomrdias what do you think about this testing structure?

I did a PoC with pubsub for the integration testing proposal. There is also one point in https://github.com/libp2p/js-libp2p-interfaces/pull/81 description that would be good to consider. Shortly, interface tests should receive an instance of libp2p instead of an instance of the module?
@achingbrain @hugomrdias what do you think about this testing structure?

I did a PoC with pubsub for the integration testing proposal. There is also one point in libp2p/js-libp2p-interfaces#81 description that would be good to consider. Shortly, interface tests should receive an instance of libp2p instead of an instance of the module?
@achingbrain @hugomrdias what do you think about this testing structure?

with this setup an instance so that libp2p-interfaces doesnt depend on libp2p, only the package do.
Can we publish nightlies here dispatch to the others repos CI and there receive or fetch the latest version, force a npm install libp2p@version or npm i libp2p@next and avoid the cross-env/env vars part ?

The initial part of this work with pubsub was done. We still need to do the same for the remaining libp2p modules

Closing as completed


Have two simplebars with vue js, one doesn't have scroll bar, any ideas?

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Reproducible example

Additional context
Add any other context about the problem here.
Your environment

Software
Version(s)

SimpleBar

Browser

npm/Yarn

Operating System

Found a work around by using this.$nextTick in vue and remounting when data changed, closing issue since it was an issue with my code.


Fixing animation stop when switching controllers

Use case:

showing indeterminate progress in list at first controller
push second controller
animation is stopped when popped back to first controller

This fixes that behaviour

'3. animation is stopped when popped back to first controller' fix is great and important.
please merge!

Thanks for the fix.
It should be merged.

I was about to open the same PR. Is this going to be merged soon?

I was about to open the same PR. Is this going to be merged soon?

Also wondering


chore: Disable flaky test.

Disables flaky test on CI.
I decided to disable it since commits have been landing to main despite the test failing. It's a known env-specific issue and  I think disabling this is the right move so that merges aren't accidentally made when a different failure happens. This should be uncommented again and addressed as part of https://github.com/googlemaps/android-maps-compose/issues/174.

Looks like the failure is from the test that I just commented out for the pull_request_target run. Not sure why it's not picking up the change in this commit. Note though that the other run-instrumentation-test runs pass so I believe it should be safe to merge this @wangela


Server application start point file is missing

Hi,
After running npm run build many times, the \src\index.js is still missing for me. Is it supposed to be there ? Please see the below screenshot for my build output.
https://drive.google.com/file/d/16PqtDneK0zcFbbFOYScWupOUXeIlh1U9/view?usp=sharing
Thanks,
Phuong

There is no src/index.js file. Look into package.json to see entry file to start.

Hi @nguyentran4896
Thanks for the response, how come in the application structure section, it shows that there is an index.js under /src folder.
.
├── config                   # Project and build configurations
├── dist                     # Distribution folder
├── locales                  # Text files
├── logs                     # Log files
├── public                   # Static public assets and uploads
│   ├── admin                # Dashboard index.html
│   ├── admin-assets         # Dashboard assets
│   └── content              # Store root folder
|
├── scripts                  # Shell scripts for theme install/export
├── src                      # Application source code
│   ├── admin                # Dashboard application
│   │   └── client           # Client side code
│   ├── api                  # REST API
│   │   └── server           # Server side code
│   ├── store                # Store application
│   |   ├── client             # Client side code
│   |   ├── server             # Server side code
│   |   └── shared             # Universal code
│   └── index.js             # Server application start point
├── theme                    # Theme as a local package
└── process.json             # pm2 process file
Also under public/content I have nothing beside an "images" folder and README.md file. Am I missing something. Did you have any issue on first set up, can you help me out please ?

Also under public/content I have nothing beside an "images" folder and README.md file. Am I missing something. Did you have any issue on first set up, can you help me out please ?
You're on the right way. It looks like the document is out of date a lot, including this Readme

Can you show me where the files for the store are located?

Look like they are missing for me? Do you have the original frontend store code?

they're still there https://github.com/cezerin/cezerin/tree/master/theme/src
just clone or fork the master branch

Delete dis folder and rebuild

Hi @dewelloper
I'm having trouble opening the store page, can you help me out please ? Please see screenshot at
https://drive.google.com/file/d/1XlR59mN1aHxJgM6unddklKaK3R90xeIE/view?usp=sharing.
Thanks,

Hi nunygph,
Your build apearing successfull but i think your before builds created (with transpile babel webpack etc..) some different js files under the dist folder. Clear there and rebuild, after run start then you will see localhost:3000

share your "npm start" result screen

Hi @dewelloper
Thank you for the response, I cleared and deleted the dist folder, rebuild. All file inside the "dist" folder got generated again. I did the same thing twice but I"m still having the same issue. I'm able to launch the admin page successfully but not the store, not sure why. Please see my shared screen below.
Build screenshot : https://drive.google.com/file/d/17BAbHHqJt3I2f-NO9BGy8k3hAWapVNGP/view?usp=sharing
Run start scrdeenshot: https://drive.google.com/file/d/11_hzV9oJkR5y3jXWAPzY72Mz-G1N9Csi/view?usp=sharing
Also, what is the "dist" folder for ?
Thanks,

There iş no any error in your code :) get a pull request
12 Eki 2018 Cum 11:56 tarihinde Phuong Nguyen<EMAIL_ADDRESS>şunu yazdı:

Hi @dewelloper https://github.com/dewelloper
Thank you for the response, I cleared and deleted the dist folder,
rebuild. All file inside the "dist" folder got generated again. I did the
same thing twice but I"m still having the same issue. I'm able to launch
the admin page successfully but not the store, not sure why. Please see my
shared screen below.
Build screenshot :
https://drive.google.com/file/d/17BAbHHqJt3I2f-NO9BGy8k3hAWapVNGP/view?usp=sharing
Run start scrdeenshot:
https://drive.google.com/file/d/11_hzV9oJkR5y3jXWAPzY72Mz-G1N9Csi/view?usp=sharing
Also, what is the "dist" folder for ?
Thanks,
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cezerin/cezerin/issues/583#issuecomment-429255287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEyiJeHnh0TdY_hzdJ-2H9Dtdh61-ooOks5ukFlCgaJpZM4W3JkR
.

Yea I did many times but still not working :)

Do you have /src/index.js in your project folder ?

There iş no you can control online codes
12 Eki 2018 Cum 12:02 tarihinde Phuong Nguyen<EMAIL_ADDRESS>şunu yazdı:

Do you have /src/index.js in your project folder ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cezerin/cezerin/issues/583#issuecomment-429257173,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEyiJZeJqOGZr7tmoUEoI0aMxpI_3VxKks5ukFqvgaJpZM4W3JkR
.


Find tf.exe example

To find tf.exe, you can use:
function Find-VisualStudioTf {
    $ErrorActionPreference = 'Stop'

    $path = .\tools\vswhere -latest -products * -requires Microsoft.VisualStudio.TeamExplorer -property installationPath
    $path = join-path $path '\Common7\IDE\CommonExtensions\Microsoft\TeamFoundation\Team Explorer\tf.exe'
    return $path
}

Thank you for the example.

Please see my reply on issue #157. Thank you for thinking of it, though.

For anyone coming here, you can use this simple one-liner:
"vswhere" -latest -products * -requires Microsoft.VisualStudio.TeamExplorer -find "**\Team Explorer\TF.exe"

Which output for me:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Common7\IDE\CommonExtensions\Microsoft\TeamFoundation\Team Explorer\TF.exe

@Indigo744 which version of vswhere are you using?
I'm using the one included in the Azure DevOps build agent vsts-agent-win-x64-2.153.1.zip and I get the following error:
Visual Studio Locator version 1.0.62
Copyright (C) Microsoft Corporation. All rights reserved.

Error 0x57: Unknown parameter: find

Thank you.

Find was introduced in a later version. I recommend pulling it from nuget.org or hosting the binary yourself so you can rely on what functionality was included.


Re-adding all_vms method

Clicking on the number of VMs from the 'All VMs' option will not load the corresponding page. Page will just show the spinner and not load.
backport: also backport https://github.com/ManageIQ/manageiq/pull/21523
BEFORE

AFTER

@miq-bot add-reviewer @NickLaMuro
@miq-bot add-reviewer @kavyanekkalapu
@miq-bot assign @kavyanekkalapu

Backported to morphy in commit 1fea0d7a257e817876dec19aa62023ebe5837e67.
commit 1fea0d7a257e817876dec19aa62023ebe5837e67
Author: Nick LaMuro<EMAIL_ADDRESS>Date:   Wed Oct 27 10:58:53 2021 -0500

    Merge pull request #21522 from MelsHyrule/fix_cluster_all_vms

    Re-adding all_vms method

    (cherry picked from commit 5f7a6f7cebb9a273d92b733dc247d1cb255f9675)


HELP how to edit the frame buffer.

I've been attempting to edit the frame buffer but no matter what I do I can not get a single output from anything it just sits there with yellow words, I'm running this on a physical machine and using UEFI. This is my code hope someone knows what's wrong, or rather what's missing.
#![no_std] // Don't link the Rust standard library.
#![no_main] // Disable all Rust-level entry points.

use core::panic::PanicInfo;
use bootloader::*;
use bootloader::boot_info::*;

// This function is called on panic.
#[panic_handler]
fn panic(_info: &PanicInfo) -> ! {
    loop {}
}

#[no_mangle]
pub extern "C" fn _start(boot_info: &'static mut BootInfo) -> ! {
    let mut frame_buffer:&mut Option<&mut FrameBuffer> = &mut boot_info.framebuffer.as_mut(); // Get a mut reference to the frame buffer.

    //Whatever makes the screen change that I can't seem to figure out.

    loop {}
}

Also sorry if I'm not supposed to put this here but I couldn't think of a place to write a help message since if I, for example, put it on stack exchange then chances are it would get drowned out and even if someone saw it they probably wouldn't be using this.

Well I changed the frame_buffer getter to this.
let mut frame_buffer:&mut FrameBuffer = &mut boot_info.framebuffer.as_mut().unwrap();

Ok so I did this.
`let frame_buffer:&mut FrameBuffer = &mut boot_info.framebuffer.as_mut().unwrap(); // Get a mut reference to the frame buffer.
let frame_buffer_data:&mut [u8] = frame_buffer.buffer_mut();
for i in frame_buffer_data.iter_mut() {
    *i = 0xC0 as u8;
}

loop {}`

But it appears to be grayscale even though the hexadecimal value should be max Red or Max Blue.

The framebuffer probably uses 24bit and not 8bit colors. This means that every group of 3 (or 4 depending on the color format used) bytes represents a single pixel with each byte being a different channel.

But the frame_buffer.buffer_mut() only returns u8, unless I'm misunderstanding and I have to set one u8 then the next to the green channel, and then the next to the blue channel? But that doesn't make sense since if I add a counter that counts how many times it's activated and tell it to stop at 1366 it does a single full-length line, which would mean that each pixel is a single u8 right? I think I am misunderstanding something major.

Ok so I did what you suggested and it almost worked? Only the words changed color. But then I set v2 to 0xFF and then the entire thing changed but it looked odd, I don't know how to describe it.
    let v1:u8 = 0xFF as u8;
    let v2:u8 = 0x00 as u8;
    let v3:u8 = 0x00 as u8;
    let mut counter:u8 = 0;
    for i in frame_buffer_data.iter_mut() {
        match counter {
            0 => {*i = v1},
            1 => {*i = v2},
            2 => {*i = v3},
            _ => {counter = 0},
        }
        counter += 1;
    }

Ok so I did what you suggested and it almost worked? Only the words changed color. But then I set v2 to 0xFF and then the entire thing changed but it looked odd, I don't know how to describe it.
    let v1:u8 = 0xFF as u8;
    let v2:u8 = 0x00 as u8;
    let v3:u8 = 0x00 as u8;
    let mut counter:u8 = 0;
    for i in frame_buffer_data.iter_mut() {
        match counter {
            0 => {*i = v1},
            1 => {*i = v2},
            2 => {*i = v3},
            _ => {counter = 0},
        }
        counter += 1;
    }

This works but now I have a new adventure, make it so that things other than the yellow but now blue words change color.
#[no_mangle]
pub extern "C" fn _start(boot_info: &'static mut BootInfo) -> ! {
    let frame_buffer:&mut FrameBuffer = &mut boot_info.framebuffer.as_mut().unwrap(); // Get a mut reference to the frame buffer.

    let frame_buffer_data:&mut [u8] = frame_buffer.buffer_mut();

    let v1:u8 = 0xFF as u8;
    let v2:u8 = 0x00 as u8;
    let v3:u8 = 0x00 as u8;
    let v4:u8 = 0xFF as u8;
    let mut counter:u8 = 0;
    for i in frame_buffer_data.iter_mut() {
        match counter {
            0 => {*i = v1},
            1 => {*i = v2},
            2 => {*i = v3},
            3 => {*i = v4},
            _ => {counter = 0},
        }
        counter += 1;
    }

    loop {}
}

[...] and I have to set one u8 then the next to the green channel, and then the next to the blue channel?

For the RGB you have to do that. For the BGR the channels are in the opposite order. For the U8 pixel format every byte is a single grayscale pixel.
https://github.com/rust-osdev/bootloader/blob/fa617315a4c01ed31e8593d5848e7d302ebf9fe8/src/boot_info.rs#L204-L224
You can look at the pixel_format field of frame_buffer.info() to know which pixel format to use.

But that doesn't make sense since if I add a counter that counts how many times it's activated and tell it to stop at 1366 it does a single full-length line, which would mean that each pixel is a single u8 right?

Maybe the screen is 455 pixels wide? (1366/3 = 455) Note that resizing the qemu window does not resize the framebuffer.

Ok so I did what you suggested and it almost worked? Only the words changed color. But then I set v2 to 0xFF and then the entire thing changed but it looked odd, I don't know how to describe it.
    let v1:u8 = 0xFF as u8;
    let v2:u8 = 0x00 as u8;
    let v3:u8 = 0x00 as u8;
    let mut counter:u8 = 0;
    for i in frame_buffer_data.iter_mut() {
        match counter {
            0 => {*i = v1},
            1 => {*i = v2},
            2 => {*i = v3},
            _ => {counter = 0},
        }
        counter += 1;
    }

That code skips the 0 => { *i = v1 }, arm due to counter being incremented immediately after setting it to 0. In addition I just read that you have to step bytes_per_pixel bytes every time with all bytes after the first 3 being ignored. Try
// FIXME move next statement before let frame_buffer_data:&mut [u8] = frame_buffer.buffer_mut();
let bytes_per_pixel = frame_buffer.info().bytes_per_pixel;

let v1:u8 = 0xFF as u8;
let v2:u8 = 0x00 as u8;
let v3:u8 = 0x00 as u8;
for pixel in frame_buffer_data.chunks_mut(bytes_per_pixel) {
    *pixel[0] = v1;
    *pixel[1] = v2;
    *pixel[2] = v3;
}

Ok, I'll try that, also I'm not using qemu I have a physical computer.

Well the code worked perfectly, the entire screen is blue, and now I can get on with writing a graphics driver.


[Issue] - Health Checker "Failed to Health Checker against"

Provide Version Number
<IP_ADDRESS>3, <IP_ADDRESS>2
Describe the issue
When attempting to run HealthChecker.ps1 after the Holidays, it requested to update to <IP_ADDRESS>2 from <IP_ADDRESS>1.  That proceeded ok and I re-ran HealthChecker.ps1 to see where our Exchange server was at.  However it only gave me the output:
"Exchange Health Checker version <IP_ADDRESS>2
Failed to Health Checker against EX19"
Since then I waited for an update to EHC and today noticed that we got <IP_ADDRESS>3.  It gives the same results.  I've tried rebooting Exchange, with no change and  I always run EMS elevated.
Setup is fairly straightforward:
Single Forest, Single Domain, all VM's on a single host.  AD & Exchange are separate VMs.  Exchange Server 2019 is a single server deployment with all roles on one VM:
ProductVersion   FileVersion      FileName

15.02.1258.028   15.02.1258.028   C:\Program Files\Microsoft\Exchange Server\V15\bin\ExSetup.exe
We are an MSP and we have 2 confirmed Exchange servers - ours which is detailed above, but also an Exchange 2016 deployment that is doing the same thing.  I am confirming with my team but I think we have 2 others that are similarly affected for a possible total of 4 deployments all exhibiting the same result.
I checked for any other issues related to the "Failed to Health Checker against" type messages and did find two in the Issues section.  They referenced being able to recreate a similar issue with a path problem to "exsetup.exe", however ours seems ok in that regard:
"[PS] >get-command exsetup.exe
CommandType     Name                                               Version    Source

Application     ExSetup.exe                                        15.2.12... C:\Program Files\Microsoft\Exchange Server\V15\bin\ExSetup.exe"
And our path variable has the Exchange Bin path in it:
"PATH=; ;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Windows\System32\OpenSSH;C:\Program Files\Microsoft\Web Platform Installer;C:\Program Files\dotnet;C:\Program Files\WindowsPowerShell\Scripts;C:\Program Files\Microsoft\Exchange Server\V15\bin;C:\Program Files\Microsoft\Exchange Server\V15\Bin\Search\Ceres\Native;C:\Program Files\AzureConnectedMachineAgent;"
Lastly, I restored a copy of HealthChecker.ps1 from December:
".\HealthChecker.ps1
Confirm
Are you sure you want to perform this action?
Performing the operation "Update script to latest version" on target "HealthChecker.ps1".
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"): n
Exchange Health Checker version <IP_ADDRESS>1"
I let <IP_ADDRESS>1 run and it ran correctly and provided a proper report as expected with no "red" errors or correctable items needing to be addressed.
I immediately went back and tried to re-run <IP_ADDRESS>3 and it produced the same error and did not run correctly.
Expected behavior
I expected EHC to run, but it did not, just giving us the below output.
Script Output
Exchange Health Checker version <IP_ADDRESS>3
Failed to Health Checker against EX19
and
Exchange Health Checker version <IP_ADDRESS>2
Failed to Health Checker against EX19
From the last two versions of the script.
<IP_ADDRESS>1 partial log showing correct results:
Exchange Health Checker version <IP_ADDRESS>1
Virtual Machine detected.  Certain settings about the host hardware cannot be detected from the virtual machine.  Verify on the VM Host that:
- There is no more than a 1:1 Physical Core to Virtual CPU ratio (no oversubscribing)
- If Hyper-Threading is enabled do NOT count Hyper-Threaded cores as physical cores
- Do not oversubscribe memory or use dynamic memory allocation

Although Exchange technically supports up to a 2:1 physical core to vCPU ratio, a 1:1 ratio is strongly recommended for performance reasons.  Certain third party Hyper-Visors such as VMWare have their own guidance.
VMWare recommends a 1:1 ratio.  Their guidance can be found at https://aka.ms/HC-VMwareBP2019.
Related specifically to VMWare, if you notice you are experiencing packet loss on your VMXNET3 adapter, you may want to review the following article from VMWare:  https://aka.ms/HC-VMwareLostPackets.
For further details, please review the virtualization recommendations on Microsoft Docs here: https://aka.ms/HC-Virtualization.
Exchange Information
Name: *redacted*
Generation Time: 02/07/2024 18:24:45
Version: Exchange 2019 CU13 Nov23SU
Build Number: 15.02.1258.028
Exchange IU or Security Hotfix Detected:
	Security Update for Exchange Server 2019 Cumulative Update 13 (KB5026261)
	Security Update for Exchange Server 2019 Cumulative Update 13 (KB5029388)
	Security Update for Exchange Server 2019 Cumulative Update 13 (KB5030524)
	Security Update for Exchange Server 2019 Cumulative Update 13 (KB5030877)
	Security Update for Exchange Server 2019 Cumulative Update 13 (KB5032146)
Server Role: Mailbox
DAG Name: Standalone Server
AD Site: Default-First-Site-Name
MRS Proxy Enabled: True

I also just confirmed, with a copy of the restored HealthChecker.ps1 version <IP_ADDRESS>1 - it also runs correctly on the customer's Exchange 2016 mentioned above.

@TCSITom please provide the debug log from the failed version of the script and send it to<EMAIL_ADDRESS>.\HealthChecker.ps1 -SaveDebugLog

@lusassl-msft  and @dpaulson45  - I've just emailed the debug logs from our EX19 and our Customer's EX16 servers.
Thanks!

The inner error that is causing this problem:
Exception calling "Add" with "2" argument(s): "Item has already been added. Key in dictionary: 'Default Web Site/CustomSite/Pages'  Key being added: 'Default Web Site/CustomSite/Pages'"
Inner Exception:    at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
   at System.Management.Automation.Interpreter.ActionCallInstruction`2.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.PSScriptCmdlet.RunClause(Action`1 clause, Object dollarUnderbar, Object inputToProcess)
   at System.Management.Automation.PSScriptCmdlet.DoProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()
Position Message: At C:\HealthChecker.ps1:4211 char:13
+             $ipFilterSettings.Add($appKey, (New-Object System.Collect ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Script Stack: at Get-IPFilterSetting<Process>, C:\HealthChecker.ps1: line 4211
at Invoke-AnalyzerIISInformation, C:\HealthChecker.ps1: line 4763
at Invoke-AnalyzerEngine, C:\HealthChecker.ps1: line 8627
at Get-HealthCheckerData, C:\HealthChecker.ps1: line 15412
at Invoke-HealthCheckerMainReport, C:\HealthChecker.ps1: line 15493
at <ScriptBlock><End>, C:\HealthChecker.ps1: line 16304
at <ScriptBlock>, <No file>: line 1

https://github.com/microsoft/CSS-Exchange/blob/ce4285865f3cb4debb5030afc65585fd9554a65c/Diagnostics/HealthChecker/Analyzer/Get-IPFilterSetting.ps1#L19


Map optimisations

I experimented with some numbers and changing the GROW_FACTOR  to 3 gives a 20% speed up. Other numbers seem to be less optimal. I assume higher numbers become a lot more inefficient when the table is very large due to initialising and copying to huge table. MAP_LOAD_PERCENT seems to be good at 75%.
https://github.com/billyquith/wren/commit/5b75dc301a39228d48373ef4e760e088325e743c
base
GROW_FACTOR 2 MAP_LOAD_PERCENT 75
map_numeric - wren             .......... 2.07s 0.0701 100.73% relative to baseline

GROW_FACTOR 3 MAP_LOAD_PERCENT 75
map_numeric - wren             .......... 1.71s 0.0245 122.05% relative to baseline
map_numeric - wren             .......... 1.72s 0.0452 121.06% relative to baseline

GROW_FACTOR 4 MAP_LOAD_PERCENT 75
map_numeric - wren             .......... 1.76s 0.0962 118.44% relative to baseline

GROW_FACTOR 5 MAP_LOAD_PERCENT 75
map_numeric - wren             .......... 1.88s 0.1039 110.87% relative to baseline

GROW_FACTOR 3 MAP_LOAD_PERCENT 70
map_numeric - wren             .......... 2.20s 0.0563  94.55% relative to baseline

GROW_FACTOR 3 MAP_LOAD_PERCENT 80
map_numeric - wren             .......... 1.74s 0.0461 119.87% relative to baseline

I ran the full benchmarks twice.
api_call - wren                .......... 0.14s 0.0025  98.54% relative to baseline
api_foreign_method - wren      .......... 0.93s 0.0296  97.95% relative to baseline
binary_trees - wren            .......... 0.55s 0.0072 100.55% relative to baseline
binary_trees_gc - wren         .......... 2.55s 0.0361  97.17% relative to baseline
delta_blue - wren              .......... 0.45s 0.0044  93.78% relative to baseline
fib - wren                     .......... 0.65s 0.0296  92.88% relative to baseline
fibers - wren                  .......... 0.13s 0.0017  99.24% relative to baseline
for - wren                     .......... 0.23s 0.0110 100.86% relative to baseline
method_call - wren             .......... 0.33s 0.0044  97.85% relative to baseline
map_numeric - wren             .......... 1.73s 0.0208 120.50% relative to baseline
map_string - wren              .......... 0.19s 0.0017 107.25% relative to baseline
string_equals - wren           .......... 0.40s 0.0127  98.50% relative to baseline

api_call - wren                .......... 0.14s 0.0021 100.00% relative to baseline
api_foreign_method - wren      .......... 0.93s 0.0281  97.85% relative to baseline
binary_trees - wren            .......... 0.55s 0.0038 100.18% relative to baseline
binary_trees_gc - wren         .......... 2.56s 0.0779  96.72% relative to baseline
delta_blue - wren              .......... 0.45s 0.0291  94.20% relative to baseline
fib - wren                     .......... 0.64s 0.0179  93.90% relative to baseline
fibers - wren                  .......... 0.13s 0.0055 100.00% relative to baseline
for - wren                     .......... 0.23s 0.0027 100.43% relative to baseline
method_call - wren             .......... 0.32s 0.0029  99.07% relative to baseline
map_numeric - wren             .......... 1.72s 0.0382 120.92% relative to baseline
map_string - wren              .......... 0.19s 0.0027 107.25% relative to baseline
string_equals - wren           .......... 0.40s 0.0197  99.25% relative to baseline

I wondered if anyone else had experimented with this.

Personal opinion, grow facteur should be configurable.

https://github.com/wren-lang/wren/blob/44d6d205866424c07ab48a499a8cf42fe1bcda92/src/vm/wren_value.c#L14
But the default values could be better?

There is no such thing as better default values. It only depends on
usage/mesure/allocation strategies... The value you want to change only
shows that the default allocation grows doesn't feat the usage. In the real
world you will want to reserve enough to avoid growth.

@billyquith Default values can be better, but you have to take performance as well as memory consumption into account. Your value is better for a simple reason: We have to allocate WAY less, especially on big programs. However, the memory cost is way higher. You could even set it to 20 and you'll only have to reallocate once for most programs, making it very "efficient". But the memory impact would be hard, especially for an embedded language where there might already be constraints .
Tuning these, here, probably mean benchmarking different values against A LOT of code. I've seen a couple projects where the best growth factor was 1.75, and they had taken into account the realloc algorithm, average map load and optimal memory consumption

The cost of resize (larger and smaller) and the associated copies and insertions of the values is significant. If noone has investigated what the best performance/memory trade off is then perhaps the first guess of the best values is not the best. There is a comment in the code to indicate this.
3 might be high, but the cost of resizing appears to be quite high in Wren. Other languages, like Lua, seem to have much more efficient table implementations.
map_numeric - wren             .......... 1.73s 0.0912 120.43% relative to baseline - growth factor 3
map_numeric - lua              .......... 0.22s 0.0057  12.96%  - much faster
map_numeric - python           .......... 1.72s 0.0743  99.46%

Because there is no answer, because there is no ideal generic situation to
test against. In my program 3 might be a waste of memory. But in yours 20
might not be enough. If you really see/need an improvement in your program,
just tune it in your program. Like I said the best growth, is no growth. So
questing for the ideal value is futile and a waste of time.

@billyquith Beware of a comparison to Lua here: map_numeric in the standard C Lua implementation is much faster because the Lua "map" (table) is implemented as a hybrid array/map. Since the keys are added sequentially from 1, the implementation only uses/grows the array portion.
Note the actual hash map portion only grows/rehashes when there are no other slots remaining. Lua tables do not have a load factor.

@benpop, Yes, thanks, I was aware of the dual hashtable implementation in Lua, and you're right the map_string benchmark would be a better comparison for the hash table. Wren works better with higher growth factor, Lua has a faster hash table, and Python 3.8 is slightly slower than Wren.
map_string - wren              .......... 0.18s 0.0047 113.74% relative to baseline
map_string - lua               .......... 0.14s 0.0027  76.37%
map_string - python            .......... 0.17s 0.0063  94.44%

However, the map_numeric benchmark gives an idea for how much better performance can be with a different perspective or specific tuning. Perhaps Wren could benefit from dynamic arrays or a dual hashtable implementation.

@benpop

Note the actual hash map portion only grows/rehashes when there are no other slots remaining. Lua tables do not have a load factor.

That is an interesting observation but the Lua hash table keeps track of the free list, so its insertions are more optimal. Wren has lots of collisions (duplicate indices) whilst inserting, accessing, and resizing, especially in smaller tables. This is likely why aggressively growing the hash table is effective. This also assumes that the start index (modulo hash) has good distribution.
Lua also does more to try and avoid collisions by moving colliding nodes that are in inappropriate slots to available slots. Wren, on the other hand, has to traverse a large part of the entries when there are a lot of collisions, so its behaviour tends towards O(n) (a linear search), rather than O(1), which a hash table should be (ideally).


Release baresip version 0.6.2

We are planning to make a new release of baresip.
Please help with review, testing, documentation etc.

compile testing on supported platforms
basic testing against other SIP endpoints
make should pass with no warnings
make test should pass with no errors

Platforms:

[ ]  Android NDK r19
[ ] CentOS 7.6
[ ]  Debian 8
[ ]  Debian 9
[ ] iOS 11.2
[ ]  Mingw (Debian 8, gcc 4.9.1, wine 1.6.2)
[ ]  OpenBSD 6.4
[ ]  OSX 10.14
[ ] Raspbian 9.6 (arm)
[ ]  Ubuntu 16.04
[ ]  Ubuntu 18.04 LTS
[ ]  Windows 8.1
[ ]  Windows 10

SIP Testing:

[ ]  Linphone 4.1.1 (OSX Desktop)
[ ]  Jitsi
[ ] FreeSwitch
[ ] iptel.org (ser 3.3.0)
[ ] sip.antisip.com (kamailio 5.2.1)

Alfred E. Heggestad writes:

Platforms:

[x]  Android NDK r19
[x]  Debian 9

except this issue: https://github.com/alfredh/baresip/issues/686

baresip 0.6.2 is now released, here is the announcement:
http://lists.creytiv.com/pipermail/re-devel/2019-April/001336.html


Slowdown due to unecessary full Γ-context resolution

Improved with commit #e207d08.
It is necessary to modify the structure of Γ-context and split into 2 parts: fixed-resolution and non-fixed-resolution. In theory, it should be possible to provide constant-time resolution on Γ-contexts.


Start Up Filesystem Diagnostic Test

As a Block Node Operator, I want the Block Node to test the readability/writeability of the mounted root
path at system boot up as a diagnostic check.  If the root path is not readable/writable, log an error
and shutdown with an exit code.

Persistence Service should have similar startup logic.
Side note, we need startup hooks for services and a way for those to receive a "startup" and "shutdown" signal; possibly via messaging?
It might be reasonable to add that as one or more separate stories, however.