Find key themes from Academia interview results

This ticket documents research analysis activities for Academia interview results.
Acceptance Criteria:

[x] Upload notes to Miro
[x] Pull out problems/opportunities (data) from each note document
[x] Compile data from all interviews (colour-coded to identify participant)
[x] Sort data into key themes

Interesting to see there are less theme's coming out of the academic research notes, but no less impactful. Well done @jaimiebutton !


Files 4.0

This change includes a brand new implementation for Files, that keeps most of the public API intact, while modernizing the underlying implementation.
Under the hood, Files now uses value types and protocols, rather than class inheritance - which improves the type safety of the API, and streamlines the underlying code. Error handling
is also improved to include more underlying info, and the docs have been reworked to be much more thorough.
The API has also been fine-tuned and modernized, dropping the FileSystem class in favor of more “Swifty” APIs on Folder, introducing more options for creating new files and folders, and making it possible to easily change properties on file system sequences.
A list of all changes will be posted as part of this version’s release notes.

Hello. It is a nice framework. But where is new docs? How to create a Folder without FileSystem?


Why does MNE use adaptive z-scoring in find_bads_eog?

According to the documentation of find_bads_eog

Detection is based on Pearson correlation between the filtered data and the filtered EOG channel. Thresholding is based on adaptive z-scoring.

Are there any experiments/papers demonstrating the benefits of this? or is it just a traditional practice? (which i believe it isnt as there are many papers still using visual inspection), or is it just a practical recommendation without detailed evidence?
Thank you very much. I am concerning if such computation would be too conservative.

It’s original MNE cuisine :) something that turned out to work. We implemented it based on our own intuition and tested it on multiple datasets. We could add other options or change the default if we can convince ourselves of an alternative.

On 11 Jun 2019, at 08:46, matthewhang<EMAIL_ADDRESS>wrote:
According to the documentation of find_bads_eog
Detection is based on Pearson correlation between the filtered data and the filtered EOG channel. Thresholding is based on adaptive z-scoring.
Are there any experiments/papers demonstrating the benefits of this? or is it just a traditional practice? (which i believe it isnt as there are many papers still using visual inspection), or is it just a practical recommendation without detailed evidence?
Thank you very much. I am concerning if such computation would be too conservative.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

It’s original MNE cuisine :) something that turned out to work. We implemented it based on our own intuition and tested it on multiple datasets. We could add other options or change the default if we can convince ourselves of an alternative.
…
On 11 Jun 2019, at 08:46, matthewhang @.***> wrote: According to the documentation of find_bads_eog Detection is based on Pearson correlation between the filtered data and the filtered EOG channel. Thresholding is based on adaptive z-scoring. Are there any experiments/papers demonstrating the benefits of this? or is it just a traditional practice? (which i believe it isnt as there are many papers still using visual inspection), or is it just a practical recommendation without detailed evidence? Thank you very much. I am concerning if such computation would be too conservative. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

thank you for the answer! I was just testing around the options!


Scheduled Google tasks

Hi there,
First of all, it is a great family planner! well done
Are you planning to add scheduled google tasks ?

Tasks would be a great addition!   Thanks for this awesome integration!

Tasks are a completely different API. There are currently no plans to add this as this would be too big a change. It might be a nice 2.0 (which I'm not planning yet) feature. I will keep this issue open.


Export as SVG

I already use html2canvas in order to export an html div as png, but I would like to do the same to produce a svg file. The documentation is quiete poor on this library and I have hard time understanding how to do this!
There is any way to do it ?
thanks,
Jeroem

Well, you can save the canvas as SVG yourself, I think that is beyond the point of this library. Search online for canvas to svg, or javascript img to svg libraries.


[DOC] Fix broken docstrings of RandomIntervalSpectralForest

Reference Issues/PRs

Fixes #470 (@mloning)
What does this implement/fix? Explain your changes.

This PR fixes the problem that docstrings of the class RandomIntervalSpectralForest from sktime.classification.frequency_based._rise.py are incompatible with Sphinx, including the bib reference and code block.
I have fixed it based on the docstrings of TimeSeriesForestClassifier. Before the modification, the docstring of RandomIntervalSpectralForest is:

Below is the result after running make.bat locally on the modified docstrings:

Does your contribution introduce a new dependency? If yes, which one?
No.

PR checklist

For all contributions

[x] I've added myself to the list of contributors.
[x] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[x] I've added unit tests and made sure they pass locally.

Thanks @AaronX121, I fixed some minor formatting issues, everything looks great! Thanks for the fix!

My pleasure to make this great repo more readable : )
In addition, I found that broken docstrings also exist in other classes, such as KNeighborsTimeSeriesClassifier.(https://www.sktime.org/en/latest/modules/auto_generated/sktime.classification.distance_based.KNeighborsTimeSeriesClassifier.html#sktime.classification.distance_based.KNeighborsTimeSeriesClassifier)

Thanks @AaronX121, feel free to open another PR or separate issue for it! 👍


Adding a sample TypeSpec ARM project created from template for Out-of-box Green CI

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

For help with Step 2 / "ARM review", see https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[ ] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[x] Other, please clarify:

Sample TypeSpec generated ARM project

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

@weshaggard, @mikeharder  the Staging Swagger LintDiff error on PatchPropertiesCorrespondToPutProperties is not correct. Looks like it did not follow the ref link to find complete properties list. I know it is staging, still it would occur for all ARM specs. Who should I file bug to get this fixed?

@weshaggard, @mikeharder the Staging Swagger LintDiff error on PatchPropertiesCorrespondToPutProperties is not correct. Looks like it did not follow the ref link to find complete properties list. I know it is staging, still it would occur for all ARM specs. Who should I file bug to get this fixed?

@AkhilaIlla @rkmanda FYI

@weshaggard, this is ready to merge. At least the out-of-box project has not obvious LintDiff/Model validation errors.

@konrad-jamrozik , what is the PR Summary failure about? I did checked appropriate boxes in the description.

@allenjzhang lets please get this green so we can be sure things are working. For the ARM review requirement we should just apply the approval label to get that green.

@konrad-jamrozik , what is the PR Summary failure about? I did checked appropriate boxes in the description.

@allenjzhang you tripped over this:

https://github.com/Azure/azure-sdk-tools/issues/6673

I.e. automation wants for you to do this:

https://github.com/Azure/azure-rest-api-specs/pull/27502#issuecomment-1906925946

I am attempting to circumvent it by adding label RPaaS. Let's see if it works.

/azp run

@allenjzhang to follow-up to my previous comment here, adding the RPaaS label doesn't unblock the automation. But you should be able to unblock it by adding openapi-subtype: rpaas or openapi-subtype: providerHub to readme.md as e.g. seen here:

@konrad-jamrozik It doesn't show me that RPSaaS check blocks this PR, is that fine?

@visingla-ms I am already working with @allenjzhang to address this. See here: https://github.com/Azure/azure-rest-api-specs/pull/27502#issuecomment-1915427553

@allenjzhang, @konrad-jamrozik - Does this PR actually need review by ARM? I see @visingla-ms left a couple of unaddressed comments. If it doesn't require review, please keep it out of the queue (leave it in ARMChangesRequested state).

@mentat9 sorry for the confusion. We will bypass the ARM review. @allenjzhang to bypass it, we need to ensure that from labels with ARM in them, only ARMReview and ARMSignedOff labels are present.


Introduce a long --config alias for -C

Currently, we provide a -C option for providing the path to configuration for Viceroy (see below). We should add a longer --config option for those who prefer that style.
https://github.com/fastly/Viceroy/blob/main/cli/src/opts.rs#L33
Reported by @peterbourgon. :slightly_smiling_face:

Nice, thank you!! I guess just curiosity, but why -C and not -c?

No strong reasoning there! We used -C rather than -c because ime that seemed more common for specifying a path to a config of some sort, i.e. git. I recall -c being a flag used for things like color, or counting, when I took a look around, and the hope was to keep the -[a-z] space as open as possible.
Most likely, -C (or --config!) will become invisible once we've integrated more closely with the Fastly CLI.

I've got a PR incoming for this!


Load pf-ring kernel module in init container

Remove all kmm references
Load kernel module via init container
Update docs

Open questions:

should we remove from worker codebase all the code related to loading kernel module?

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

Comparison is base (db51e6d) 24.81% compared to head (8a5cf5a) 25.18%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1476      +/-   ##
==========================================
+ Coverage   24.81%   25.18%   +0.36%
==========================================
  Files          14       14
  Lines         552      544       -8
==========================================
  Hits          137      137
+ Misses        415      407       -8

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

@corest, this can be a future feature request that comes in addition to this PR, however, one thing that I'm missing is for Kubeshark to NOT use PF-RING if enabled=false. Today, if PF-RING is loaded, Kubeshark will use it, even if this feature is disabled. It's not clean. Does this make sense? Is this for now? Is this for later?

When I test it, I get this:
 load-pf-ring Loading pf_ring module for kernel 5.10.192-183.736.amzn2.x86_64                                                                                │
│ load-pf-ring insmod: ERROR: could not insert module /opt/lib/modules/5.10.192-183.736.amzn2.x86_64/pf_ring.ko: Invalid module format

In any case, we should fail gracefully to AF_PACKET.
Worker goes into crashloop:

@alongir fixed to just log error and fallback into available methods for packet capture

When running it, worker complained about this: https://github.com/kubeshark/kubeshark/blob/aaeb3ca1eb8d5cfa512fc1f7bc07c90fbb662317/helm-chart/templates/09-worker-daemon-set.yaml#L60 (-no-kernel-module)
When I removed it from the manifest, it didn;t load the PF-RING.


DynamicMapProperty escaping equal sign

DynamicMapProperty extracts key and value from keyValue string splitted by '='.
If value contains '=' char, keyValue string has more than two '='s, therefore after splitting DynamicMapProperty can't extract key&value, then just ignores it.
so, user can't set values that contains '=' char. and user get no error or warning, Key&values gonna just missing.
archaius should provide proper excaping.
I made a change to = can be escaped by \ (in java "\\=")

Closing because v1 is deprecated


How many channels Nano can drive?

I understand there are two options - connects LED serially, or connect them as several channels and drive them separately, even 8 channels with 1 led per each.
I'm thinking out loud here... 1 channel per led approach has a bonus that when installing - I need only one cable visible in the room (no exit cable).
While routing 8 channels requires more initial effort, I think it's easier to maintain/debug if something goes wrong - I can loose one channel/room and the rest will keep working.
The question is whether Auduino Nano can handle 8 channels (enough pins, etc.).
Again, if it's not something straight-forward (soldering/wiring aside), let's skip it for the first version.

Looking at APA102 tutorial, seems like adruino only need to drive clock and data, i.e. 2x8 pins. So Nano should have enough.
Further question: can all led channels share single clock pin? This way I'll only need 1 clock + 8 data pins.

Hah. I just commented on the other issue to that effect. The APA102 should work fine on a shared clock. I'll see if I can test and prove that this weekend, but in principle, it should work.

The Nano has 14 digital pins; it should be possible to run therefore 13 individual LEDs [though we are likely to not want to use TxD/RxD, and perhaps not the SPI pins to make it easier to program and debug]. Still, even with that, you should be good to go for 8 channels.

From what I read, analog pins can work as digital as well (but not vice
versa). Hence it gives us another 8, right?
(Not that I'll need those, but in case me out someone else wants to extend
it in the future)
On Fri., 28 Jun. 2019, 23:58 jim-sokoloff<EMAIL_ADDRESS>wrote:

The Nano has 14 digital pins; it should be possible to run therefore 13
individual LEDs [though we are likely to not want to use TxD/RxD, and
perhaps not the SPI pins to make it easier to program and debug]. Still,
even with that, you should be good to go for 8 channels.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/haizaar/doll-house-lights/issues/2?email_source=notifications&email_token=AAAOGWI4KQ55BORMTCLB2XLP4YKIDA5CNFSM4H3XNTF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY2EZBQ#issuecomment-506743942,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAOGWOVBTJA2G7WSDBB6JDP4YKIDANCNFSM4H3XNTFQ
.

You're right. It appears to be 22 in total perhaps. (I first just googled "Arduino nano how many digital pins" and took the answer of 14 to be "enough for haizaar's use case".)

Looks like this one is clear now. Closing :)


community: Add alazy_load to AsyncHtmlLoader

Also fixes a bug that _scrape was called and was doing a second HTTP request synchronously.
Twitter handle: cbornet_

@cbornet After your changes, _fetch_valid_connection_docs is no longer used, is that a coincidence?

Indeed. It should probably be removed.

Actually, I need functionality like that, so it's easier to skip invalid links:
except aiohttp.InvalidURL:
    if self.ignore_load_errors:
        logger.error(f"Invalid URL: {url}")
        return ""

instead of validating links before passing them to the loader. I will create a PR for this.


Add KubeCon talk as reference to readme

This was my personal favorite of KubeCon NA 2022 talks and I think it deserves a spot in the readme.

@sttts Is this okay to merge?

@jonnylangefeld of course it is. Great idea. Merging.


Websoket frame failes if the frame content is deflated partial EOM (0,0,-1,-1) from Tomcat.

Scenario is when the tomcat communicates to netty with websocket connection. The frmae size is 8192 bytes.
Tomcat sends the compressed websocket data frames to netty.
When tomcat sends the websocket frames to netty, every frame ends with the EOM byte sequence which is 0,0,-1,-1.
There are four conditions now:

The compressed data is small enough that the complete content and the EOM is contained in one frame.

The data is large that it takes multiple frames to complete and the last frame contains some part data and the complete EOM.

The  data is such that the EOM is in the two frames. e.g. the data is 8189 bytes, so the EOM starts in one and spreads over the next one.

The data and the EOM totals the 8192bytes. even in this case a next frame is sent with 0 as the first byte.

The conditions 1and 2 passes but 3 and 4 fails. Becuase the frame that contains only the partial EOM of previous frame is unable to be deflated in the DeflateDecoder
io.netty.handler.codec.compression.JdkZlibDecoder.decode(ChannelHandlerContext, ByteBuf, List)
Expected behavior
The websocket message should be successfully sent.
Actual behavior
The message fils with the below exception
Caused by: io.netty.handler.codec.CodecException: cannot read uncompressed buffer
at io.netty.handler.codec.http.websocketx.extensions.compression.DeflateDecoder.decode(DeflateDecoder.java:89)
at io.netty.handler.codec.http.websocketx.extensions.compression.PerMessageDeflateDecoder.decode(PerMessageDeflateDecoder.java:64)
at io.netty.handler.codec.http.websocketx.extensions.compression.PerMessageDeflateDecoder.decode(PerMessageDeflateDecoder.java:30)
at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:88)
Steps to reproduce
Host a zip file in tomcat and transfer to netty application on a websocket connection.
But it is not reproducible with any zip file, only with the ones where content is not compressible by the java Deflater class.
Minimal yet complete reproducer code (or URL to code)
I have debugged in a complicated software, I am in process of creating a minimal code, but if the issue is crlear can we please get the fix.
Though not in a perfect way, but I was able to fix the code by following changes in DeflateDecoder
DeflateDecoder.txt
:
boolean partialEOMFrame = true;
for (int i = 0; i < msg.content().readableBytes(); i++) {
byte contentAtI = msg.content().readByte();
if (contentAtI != 0 && contentAtI != -1) {
partialEOMFrame = false;
break;
}
}
msg.content().readerIndex(0);
;
;
;
if (!partialEOMFrame && readable && compositeUncompressedContent.numComponents() <= 0) {
compositeUncompressedContent.release();
throw new CodecException("cannot read uncompressed buffer");
}
Netty version
4.1.15
JVM version (e.g. java -version)
1.8
OS version (e.g. uname -a)
windows.

@sakshamverma can you please upgrade to 4.1.43.Final and see if the problem still exists ? @amizurov did some fixes related to this as far as I remember.

Hi @sakshamverma , @normanmaurer, will look as soon as possible, also it will be great to have some reproducer or steps (websocket handshake request and sequence of frames). As well I try to check compression on latest autobahn testsuite.

@sakshamverma I think this is fixed by https://github.com/netty/netty/commit/491b1f428b00bba45a2c51d171cf819704b0fbe6#diff-30ed3b7e954d7e7b8509e71945b53430 . Please upgrade and report back if you still see the issue.

@normanmaurer it was another problem, I checked compression with latest autobahn testsuite and some test not passed, that depends on some last fragmented frames, i'll try to dig deeper and fix it.

Alight let me reopen it then @amizurov


Add GPU IaaS example with Jupyter.

I think this look good, but will wait for @pontus to chime in

No problem with merging, see comments for ssh if you want to regard  them.


Stop console auto scroll

This is not for requesting support for new games/applications
To do this you should go to https://github.com/CubeCoders/AMPTemplates and first attempt to build a configuration yourself - otherwise you can request a template from this repo.
Feature Request
Feature Information:
A feature to disable console auto scroll to bottom. This allows for reading of the console output while the server is starting, without the console forcing a scroll to bottom.
I confirm:

[x] that I have searched for an existing feature request matching the description.

You can achieve this currently simply by selecting some text in the console window


Place flutter_gpu in the package cache.

Resolves https://github.com/flutter/flutter/issues/131711.
Fetch https://storage.googleapis.com/flutter_infra_release/flutter/${engine_version}/flutter_gpu.zip and extract it into the package cache.
Cannot function until https://github.com/flutter/engine/pull/53107 has been merged/rolled into the framework.

Added a test that verifies packageDirs entries are fetched from the expected URL and placed in the expected cache directory.


The automated release is failing 🚨

:rotating_light: The automated release from the dev branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the dev branch. You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

The release 1.3.0 on branch dev cannot be published as it is out of range.
Based on the releases published on other branches, only versions within the range >=1.2.2 <1.3.0 can be published from branch dev.
The following commits are responsible for the invalid release:

Merge branch 'feature/router-profile' into feature/backend (05fa030)
feat(server): new router and schema for skills (c40e8ae)
feat(server): new router and schema for profile skills (9cfddc6)
docs(database): adjust readme (6a9fa10)
ci: remove main on push branches and add release (9fba295)
chore: branchs in package.json release (31607e8)
Merge branch 'feature/refactor' into dev (ae4f035)
feat: new schema for fetch contacts (db88ea9)
refactor: router fetch components (ad3c8ab)
feat: new index for import in other workspaces (f3f48ac)
refactor: env.ts schema for new environment structure (31f81a5)
ci: new model of execution scripts (cd78a54)
chore(workspace): adding zod and dotenv dependencies, chore(server): adding .gitignore (07519bf)
feat: example with structure on environment (8425f25)
docs: new documents for orientations in environment setup (5be1d42)
feat: directory the storage on pnpm and cache directory npm (8a7e20e)
style: new settings.json on focus in minimalist on vscode (9fafb0b)
fix(database): adjusted new connection cluster mongodb atlas (e4239c6)
docs(documentation): new documentation for intrusion for connection in new cluster (3da0acb)
chore: update package.json version [skip ci] (77acb93)

Those commits should be moved to a valid branch with git merge or git cherry-pick and removed from branch dev with git revert or git reset.
A valid branch could be main.
See the workflow configuration documentation for more details.

Good luck with your project ✨
Your semantic-release bot :package::rocket:

:rotating_light: The automated release from the dev branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the dev branch. You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

The release 1.3.0 on branch dev cannot be published as it is out of range.
Based on the releases published on other branches, only versions within the range >=1.2.2 <1.3.0 can be published from branch dev.
The following commits are responsible for the invalid release:

fix: force update (6eeb042)
Merge branch 'feature/router-profile' into feature/backend (05fa030)
feat(server): new router and schema for skills (c40e8ae)
feat(server): new router and schema for profile skills (9cfddc6)
docs(database): adjust readme (6a9fa10)
ci: remove main on push branches and add release (9fba295)
chore: branchs in package.json release (31607e8)
Merge branch 'feature/refactor' into dev (ae4f035)
feat: new schema for fetch contacts (db88ea9)
refactor: router fetch components (ad3c8ab)
feat: new index for import in other workspaces (f3f48ac)
refactor: env.ts schema for new environment structure (31f81a5)
ci: new model of execution scripts (cd78a54)
chore(workspace): adding zod and dotenv dependencies, chore(server): adding .gitignore (07519bf)
feat: example with structure on environment (8425f25)
docs: new documents for orientations in environment setup (5be1d42)
feat: directory the storage on pnpm and cache directory npm (8a7e20e)
style: new settings.json on focus in minimalist on vscode (9fafb0b)
fix(database): adjusted new connection cluster mongodb atlas (e4239c6)
docs(documentation): new documentation for intrusion for connection in new cluster (3da0acb)
chore: update package.json version [skip ci] (77acb93)

Those commits should be moved to a valid branch with git merge or git cherry-pick and removed from branch dev with git revert or git reset.
A valid branch could be main.
See the workflow configuration documentation for more details.

Good luck with your project ✨
Your semantic-release bot :package::rocket:


Long ping times

Hey all,
Saw a similar post with this problem. When I try and install via Helm chart the default Grafana chart showed very long ping times; circa ~3 seconds for some larger clusters, and even a ten node cluster is showing ~1 to ~2 second ping times.
As one user has done, I made a new build and instrumented out the calls a bit and the reported time seems accurate, but when I hit the /ping endpoints directly from pods with other tools like a timed curl the time seems much more reasonable (~a few mills).

Ford thing to check would be your CPU limits - you might be starving them
for CPU time.
On Fri, 7 Oct 2022 at 06:34, Montana Mendy @.***> wrote:

Hey all,
Saw a similar post with this problem. When I try and install via Helm
chart the default Grafana chart showed very long ping times; circa ~3
seconds for some larger clusters, and even a ten node cluster is showing ~1
to ~2 second ping times.
As one user has done, I made a new build and instrumented out the calls a
bit and the reported time seems accurate, but when I hit the /ping
endpoints directly from pods with other tools like a timed curl the time
seems much more reasonable (~a few mills).
—
Reply to this email directly, view it on GitHub
https://github.com/bloomberg/goldpinger/issues/125, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADVIOWUZZH7PSPULB3CAVTWB6Y7VANCNFSM6AAAAAAQ7IF2DI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Best regards,
Mikolaj Pawlikowski
+44<PHONE_NUMBER>


Fix Duplicate Events in Meetup.com Results on Our End

UpstatePHP is showing 3 times.
This may be due to it being posted on a shared HackGreenville Meetup.com account, or maybe it's something else.
Here's the JSON showing the duplicates as of Aug 13, 2019. It's showing in our API on both the tags and non-tags results
  {
    "created_at": "2018-04-03T12:32:29Z",
    "data_as_of": "2019-08-13T23:35:20Z",
    "description": "Project estimation is a source of unending frustration for developers and clients alike. Apply these eight Protips\u2122 to spare your sanity\u2014and maybe even save your job. ----------- Food will be provided by our sponsors, so come hungry! Support our efforts (and help us have food) by donating here: https://opencollective.com/upstatephp",
    "event_name": " Jonathon Hill presents Estimation Protips\u2122",
    "group_name": "HackGreenville Community",
    "nid": "117",
    "rsvp_count": 5,
    "status": "upcoming",
    "tags": "1",
    "time": "2019-08-15T22:00:00Z",
    "url": "https://www.meetup.com/Hack-Greenville/events/262126063/",
    "uuid": "45568434-87a4-4043-8771-f7c7477c45c2",
    "venue": {
      "address": "101 N. Main St 3rd Floor",
      "city": "Greenville",
      "country": "us",
      "lat": 34.851925,
      "lon": -82.39995,
      "name": "OpenWorks",
      "state": "SC",
      "zip": null
    }
  },
  {
    "created_at": "2018-04-03T12:32:29Z",
    "data_as_of": "2019-08-13T23:35:20Z",
    "description": "Project estimation is a source of unending frustration for developers and clients alike. Apply these eight Protips\u2122 to spare your sanity\u2014and maybe even save your job. ----------- Food will be provided by our sponsors, so come hungry! Support our efforts (and help us have food) by donating here: https://opencollective.com/upstatephp",
    "event_name": " Jonathon Hill presents Estimation Protips\u2122",
    "group_name": "HackGreenville Community",
    "nid": "117",
    "rsvp_count": 5,
    "status": "upcoming",
    "tags": "1",
    "time": "2019-08-15T22:00:00Z",
    "url": "https://www.meetup.com/Hack-Greenville/events/262126063/",
    "uuid": "45568434-87a4-4043-8771-f7c7477c45c2",
    "venue": {
      "address": "101 N. Main St 3rd Floor",
      "city": "Greenville",
      "country": "us",
      "lat": 34.851925,
      "lon": -82.39995,
      "name": "OpenWorks",
      "state": "SC",
      "zip": null
    }
  },
  {
    "created_at": "2018-04-03T12:32:29Z",
    "data_as_of": "2019-08-13T23:35:20Z",
    "description": "Project estimation is a source of unending frustration for developers and clients alike. Apply these eight Protips\u2122 to spare your sanity\u2014and maybe even save your job. ----------- Food will be provided by our sponsors, so come hungry! Support our efforts (and help us have food) by donating here: https://opencollective.com/upstatephp",
    "event_name": " Jonathon Hill presents Estimation Protips\u2122",
    "group_name": "HackGreenville Community",
    "nid": "117",
    "rsvp_count": 5,
    "status": "upcoming",
    "tags": "1",
    "time": "2019-08-15T22:00:00Z",
    "url": "https://www.meetup.com/Hack-Greenville/events/262126063/",
    "uuid": "45568434-87a4-4043-8771-f7c7477c45c2",
    "venue": {
      "address": "101 N. Main St 3rd Floor",
      "city": "Greenville",
      "country": "us",
      "lat": 34.851925,
      "lon": -82.39995,
      "name": "OpenWorks",
      "state": "SC",
      "zip": null
    }
  },

I think this was fixed in the course of another fix. Closing, at least for now.


client.moveIndex fails silently when passed incorrect type

Algolia library version: 3.24.12
Node version: v4.8.4
What is the current behavior?
When client.moveIndex() is passed an index as its first parameter instead of the string name of that index, the command completes successfully but nothing is moved.
The minimal version of this issue is below:
var algoliaSearch = require('algoliasearch');
var client = algoliaSearch(APP_ID, APP_KEY);

var tempIndexName = 'temp-index';
var existingIndexName = 'existing-index';
var tempIndex = client.initIndex(tempIndexName);
var indicies = [{}, {}]; //etc
var tempIndex;

client.copyIndex(existingIndexName, tempIndexName).then(() => {
    tempIndex = client.initIndex(tempIndexName);
    return tempIndex.addObjects(indices)
}).then(() => {
    return client.moveIndex(tempIndex, existingIndexName)
}).then(() => {
    console.log('done');
    return;
}).catch(err => {
    console.log('error!');
    return err;
});

What is the expected behavior?
In the above example I would expect to see error! logged, then an typeError returned, but instead we see done logged out. In the Algolia UI, the existing index will say updated 1 minute ago, but none of the indices will have changed.
It seems from the source code like this might just be standard behavior for this library, so maybe it's not really an issue, but it is something that got me hung up for a while, so maybe if nothing else the expected inputs should be documented with the function.

I notice that you wrote var indicies, but use the variable indices. Is it possible that it's related to this (since indices is undefined, which is a valid value).
EDIT: ah I see what you mean here, there should be an error for the input of the object instead of string

So the bug happened because of two reasons:

the index object was used, instead of the string
since we assumed it was a string, encodeURIComponent returned [object Object] ("%5Bobject%20Object%5D")
since that is a valid index name, Algolia (the backend) didn't complain
since the index doesn't exist, the operation is considered void and does nothing

The correct code in this case would've been this:
const algoliaSearch = require('algoliasearch');
const client = algoliaSearch(APP_ID, API_KEY);

const tempIndexName = 'temp-index';
const existingIndexName = 'existing-index';
const tempIndex = client.initIndex(tempIndexName);
const objects = [{}, {}]; // etc

client
  .copyIndex(existingIndexName, tempIndexName)
  .then(() => tempIndex.addObjects(objects))
  .then(() => client.moveIndex(tempIndexName, existingIndexName))
  .then(() => {
    console.log('done');
    return;
  })
  .catch(err => {
    console.log(err);
    return err;
  });

I've let this know to our core engine team to discuss why this didn't throw an error (backend) and the reason is this:
If you do atomic reindexing, you will typically do a sequence like:

Add records to a nonexistent index
Move that index to its destination
If you don’t want to wait after 1, then you are going to issue the move operation before the index actually exists. This is why no error can be thrown for copying a non-existing index

I opened a separate issue for type checking, since it's a bigger issue than just this one method. For now I can't say much more than be careful which method you write with which arguments

Great, thank you!


How to get Frree-Community version?

After submitting the form for the Free-version, nothing came to the mail
ask your question here. be as specific as possible.
https://screenpi.pe/onboarding/free-community

you can build it from source

I compiled it and ran it. Screenpipe.exe listens to my microphone. How do I give it voice commands?

screenpipe.exe is running on CUDA. Listens to the microphone. Displays what I say, like “open calculator”. But it doesn't do anything. How do I get it to work?

@xteam777 can you explain with more details plz?
screenpipe is free now: https://screenpi.pe/


cmd/go: misleading warning on GOROOT == GOPATH in module mode

What version of Go are you using (go version)?

go version go1.17 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/usr/local/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/usr/local/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.17"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build561180864=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I was installing go install<EMAIL_ADDRESS>Inside a fresh system I did:
$ curl -L https://golang.org/dl/go1.17.linux-amd64.tar.gz | tar -C /usr/local -xz
$ export GOPATH=/usr/local/go
$ export PATH="${GOPATH}/bin:${PATH}"
$ go install<EMAIL_ADDRESS>warning: GOPATH set to GOROOT (/usr/local/go) has no effect
go: downloading github.com/golang/protobuf v1.5.2
go: downloading google.golang.org/protobuf v1.26.0
$ ls -al /usr/local/go/bin
total 25120
drwxr-xr-x  2 root root     4096 Aug 22 14:11 .
drwxr-xr-x 10 root root     4096 Aug 16 16:27 ..
-rwxr-xr-x  1 root root 14007666 Aug 16 16:30 go
-rwxr-xr-x  1 root root  3318877 Aug 16 16:30 gofmt
-rwxr-xr-x  1 root root  8384142 Aug 22 14:11 protoc-gen-go
$ ls -al /root/go/bin
ls: cannot access '/root/go/bin': No such file or directory

In another fresh system I did:
$ curl -L https://golang.org/dl/go1.17.linux-amd64.tar.gz | tar -C /usr/local -xz
$ export PATH="/usr/local/go/bin:${PATH}"
$ go install<EMAIL_ADDRESS>go: downloading github.com/golang/protobuf v1.5.2
go: downloading google.golang.org/protobuf v1.26.0
$ ls -al /usr/local/go/bin
total 16932
drwxr-xr-x  2 root root     4096 Aug 16 16:30 .
drwxr-xr-x 10 root root     4096 Aug 16 16:27 ..
-rwxr-xr-x  1 root root 14007666 Aug 16 16:30 go
-rwxr-xr-x  1 root root  3318877 Aug 16 16:30 gofmt
$ ls -al /root/go/bin
total 8196
drwxr-xr-x 2 root root    4096 Aug 22 14:14 .
drwxr-xr-x 4 root root    4096 Aug 22 14:14 ..
-rwxr-xr-x 1 root root 8383846 Aug 22 14:14 protoc-gen-go

What did you expect to see?
As you see, setting GOPATH is critical for getting binary installation into /usr/local/go/bin instead of /root/go/bin. Without it, it does not work. So the warning saying GOPATH set to GOROOT (/usr/local/go) has no effect is misleading. Setting GOPATH has a pretty dramatic effect.
What did you see instead?
I have seen the warning saying GOPATH set to GOROOT (/usr/local/go) has no effect and I assumed I can just remove setting GOPATH but it turns out that is not true. It has an effect.
I think this is important because installation instructions only talk about setting export PATH=$PATH:/usr/local/go/bin, but without setting GOPATH. So installed binaries go into /root/go/bin which is not in PATH based on the installation instructions.

Setting GOPATH to GOROOT isn't a supported way of working as it mixes user code with the standard library. If you need to control where the binaries are installed, use GOBIN.

I think issues with reported here still hold:

Warning is misleading.
Documentation sets PATH only to /usr/local/go but not also sets GOBIN.

Looks like none of the commands in module mode check check this
cc @bcmills @jayconrod @matloob

We should fix this. The install directory should not be set to $GOPATH/bin if the first element of GOPATH is the same as GOROOT.
We should also mention adding $GOPATH/bin to the install instructions. That's covered by #44930.
I'd recommend against installing anything in $GOROOT/bin that isn't usually there. If you're setting GOBIN, it's fine to point it to a directory already in PATH like /usr/local/bin.

Can GOROOT == GOPATH be an error instead of just a warning?

export GOROOT=/usr/local/go
export PATH=$PATH:$GOROOT/bin
export GOPATH=/home/priyambada/golib
export PATH=$PATH:$GOPATH/bin
This eventually works for me without an error
If you installed GoLang in a custom directory other than the default (/usr/local/), you must specify that directory as the value of the GOROOT variable.

@bcmills This is in the 1.18 milestone; time to move to 1.19?  Thanks.

Is this something an external contributor can pick up? I'd like to work on this. Thanks

@danishprakash Of course.

export GOROOT=/usr/local/go export PATH=$PATH:$GOROOT/bin export GOPATH=/home/priyambada/golib export PATH=$PATH:$GOPATH/bin
This eventually works for me without an error If you installed GoLang in a custom directory other than the default (/usr/local/), you must specify that directory as the value of the GOROOT variable.

Thank you. when upgrading from 1.19 to 1.20 it helped

Guys you must seriously consider to produce a comprehensible, complete and well organized documentation and you want to speak also to beginner not only to internal engineering teams
I worked with many languages compiled or intepreted but I cannot believe that GO was born without a decent dependencies managemente tools at the early stages, it sounds like a big oversight or a language was born in 1990.
This could have happened to Java or C++ but not to GO , not in 2009
I read many discussions about GOPATH, go.mod, GOROOT, import path, and bla bla .... those will be recognized by people as battle arenas not confortable place, this could be frustrating for US to see people fighting
Documentation cannot rise to the level of using the word  "ecosystem" before stopping to be a patch work ! believe me, just see their facial expressions when they read the doc
To be clear I am not speaking about the power of the GO language but about the way doc is organized and made understandable by common people like me
I am sure  a beginner cannot find a simple page where GOROOT is explained with its default !!! and worse not to understand that it cannot be the same as GOPATH wasting a lot of time
The simple fact this discussion exists is POC something is not happening as you wanted !
Please. Become aware of this


Add react16 to list of peerDependencies

thank you @WernerLDev , I published v5.0.11 with your fix 👍


pressing special characters will cause 'quicksearch' regex to fail (line 756)

I get this when trying to press special keys such as +, /, ?, etc..
SyntaxError: nothing to repeat
if(new RegExp('^' + chr + '+$').test(word)) {
Traced to line 756.
Although I'm not trying to use any quick search features, it gets picked up anyway and causes issues.
Suggestion to include/use a regexEscape type function on the chr such as:
RegExp.escape = function(text) {
return text.replace(/[-[]{}()*+?.,\^$|#\s]/g, "\$&");
}
jQuery UI also has a similar function to escape the characters.

Thanks for reporting but this was fixed a while ago:
https://github.com/vakata/jstree/blob/master/src/jstree.js#L778
Best regards,
Ivan


Fix broken web link for SGX mitigation tools

Signed-off-by: manju956<EMAIL_ADDRESS>

This PR changes has been included in https://github.com/hyperledger/avalon/pull/719 and hence closing this PR


Ensure openings follow parent line

The DB already follows the format Opening family: variation1, var2, var3... It follows that French Defense: Advance Variation derives from French Defence. We should also ensure that the corresponding pgns also follow this logic: 1. e4 e6 2. d4 d5 3. e5 should be derived from 1. e4 e6 2. d4 d5 and not 1. d4 e6 2. e4 d5. This is already the case for most openings. However, there are still a lot of bad transpositions.
To fix them we should:

Ensure every Opening family has a unique shortest pgn #55 - WIP
Use the fixed families to fix bad var1's, use fixed var1's to fix var2's etc.
If name ends in "...Gambit", ensure "...Gambit Accepted" and "...Gambit Declined" are children of "Gambit"

Roadblocks to the above:
I ran into some problems related to two major opening families for black: The Modern and the Pterodactyl Defense. Study here - https://lichess.org/study/WrOHG3qw
These are well-established openings (40 lines+34 lines in the db) that have multiple sources corroborating that they both start with 1.e4 as well as 1. d4. Often times lines will transpose into each other. The seemingly only way to tell a 1.e4 Modern apart from a 1. d4 line is by its ECO.
I personally do not think that ECO is enough to know the move order. Almost every other opening's move order can be inferred from its name. Not to mention, it does not help when trying to group openings by family, for example for the lichess opening pages that were in development.
In the study, I've outlined a possible solution but it would require breaking precedent from all other chess sites. Even then, it is not perfect. Would really love some feedback/ideas. Or whether it's best to just leave the ambiguity.

Proposals 1-3 make a lot of sense to me, and they could be added to the automated linter.
If we give up on sorting and grouping by ECO, what other ordering seems best, and how should be split the files? Order by moves, and split by first move?

Ordering by moves seems to be the most logical. I'd be wary about trying to add too much logic to it and recreating https://xkcd.com/927/.
Going by first moves alone, there'd be

e4     1888 openings
d4     1078 openings
__ Remaining 426 openings

Ordered by pgn,name and eco (for those who want it). It should be much more readable/accessible especially once bad transpositions are fixed.

Revisiting this and having some doubts ...
Regarding ECO: I was hoping we could remove it entirely, and asked around a bit. For all its flaws, people are still using it to reference other material, so it needs to stay :disappointed:
Regarding 2: I am no longer sure that it's really the case that child variations by name are necessarily child variations by moves (in the canonical or most common move order).
Regarding 3: To clarify, how would we deal with something like the following?
c.tsv:C40       Latvian Gambit: Mason Countergambit     1. e4 e5 2. Nf3 f5 3. d4

Rule 2:

Take for example:

A56 | Benoni Defense | 1. d4 Nf6 2. c4 c5
A43 | Benoni Defense: Old Benoni | 1. d4 c5

A43 and some other Old Benonis are violations of rule 2 because of A56. Simple solution: Rename them to their own
family: Old Benoni Defense. They aren't children by moves, so why should they be children by name?

Undoubtedly there will be some vagueries. I haven't dived in too deep on this (there are 170 mismatches for rule2 by my count for just var1s), This has to be looked at slowly and case by case but from correcting previous openings, I think simple solutions like this exist for most openings.

Rule 3 and Latvian Gambit: Mason Countergambit:

The opening would be checked for rule 2. Since the opening family Latvian Gambit is till ...f5, var1 Mason Countergambit with 3. d4 is a direct continuation. It would pass Rule 2.
I had included rule 3 because Gambits being accepted or denied are treated as their own Families/variations since ...Gambit, ...Gambit Accepted and ...Gambit Denied are not separated by : or , so they would not be caught by rule 2. See https://lichess.org/opening/tree for better visualisation (Benko, King's gambit, etc.)
Rule 3 does not apply here. When would rule 3 apply? When name matches regex 'Gambit\s(Accepted|Denied)$'
Countergambits should ostensibly also be gambit continuations under "gambit declined" but they are named and separated by : or , so there's no need to include them in rule 3 as they will be caught by rule 2.


Disable an active twostate button

Even tho button was active, user could re-click on it, this fix disables the button when it is active to eliminate the confusion.
Fixes https://github.com/ManageIQ/manageiq-ui-classic/issues/491
After fix

LGTM :+1: :)


Create User Model

Sub-Task of User Story 02
Model required for user authentication
Minimum requirements for Model

email (username) - validator required
password

Tests passed, branch merged.


Updating omsagent yaml to use the latest yaml

What this PR does / why we need it:
Updating the yaml since the omsagent daemonset creation with the earlier yaml was failing
Which issue this PR fixes
Updating the yaml since the omsagent daemonset creation with the earlier yaml was failing

/assign @jackfrancis

/lgtm


Only works with password?

Hi,
I was checking this package, and was wondering what will it take to make it work with other login services?
Thanks

If you want to patch for all created users, you could do the following:
// Meteor.isServer
var originalInsertUserDoc = Accounts.insertUserDoc;
Accounts.insertUserDoc = function(options, user){
  var userId = originalInsertUserDoc.call(this, options, user);
  if (userId){
    // call your callbacks here
  }
  return userId;
};


Changing classes names

Hi @iziz, i'm trying to include libPhoneNumber inside a framework i'm helping to build. To avoid crashes, when another app is using this framework and also includes your library, i'm changing the class names by adding a prefix. The problem is that when I change the classes names the library is not formatting anymore. Have any clue why ?

If you don't mind, Can I see that codes ?

I manage to make it work using the latest version of the library. I just have one test failing (testAustriaNationalNumberParsing)
Here's the source:
https://github.com/Urucas/libPhoneNumber-iOS

Sorry for late.
"testAustriaNationalNumberParsing" tets will always fail. (you can ignore this, and it is not exist in latest code)
But, maybe I think it need to update for some warning "PreferredDomesticCarrierCode" -> "preferredDomesticCarrierCode".

Thanks for your feedback @iziz . I'll left the testAustriaNationalNumberParsing failing, and change PreferredDomesticCarrierCode -> preferredDomesticCarrierCode.

So, it works fine ?

It's working fine.

OK  good, thanks :)


Draw a rectangle in two clicks

A large number of areas (building) has rectangel form. It would be nice to have an ability to draw rectangles (areas with right angles) in two clicks.
A current way to draw rectangles is quite difficult: draw an area, transform it to a rectangle with a hotkey or a context menu. Move / rotate the rectangle to make it aligned as it is supposed to be.
Example usage:

Hold "Shift" key;
Draw a diagonal line with an "area tool" (point two corners);
The diagonal is expanded to a rectangular area.

Yeah, I was just going to say:
Got an idea for a quick building addition mode:
Two clicks to define a rectangular building!
(OK, maybe a third click in the same spot to mean "I'm finished".)
That way, say in rural area with lots of rectangular barns,
or communities with lots of rectangular homes,
one could make a lot of buildings in a jiffy!


Add support for standardrb

We use standard along with rubocop in our project. We use the inherit_gem config in .rubocop.yml to add on to standard's config. We also use the require config to require standard along with other gems we use (rubocop-performance, rubocop-rails, etc).
Can a channel be made that requires standard?

It should be possible to just include the standard gem in all regular channels that have compatible rubocop versions, then let people's rubocop config decide whether or not to use it.

So actually @cquinones100 we do have a dedicated standardrb plugin already, we just don't actually have documentation up for it so I 100% understand how you haven't been able to add that in. Right now we only support version 1.1.5 but are in the process of releasing a newer version 1.4.0 in the near future.
To enable standard for your codebase, simply add the following to the plugins section of your .codeclimate.yml file:
plugins:
  standard:
    enabled: true
---the rest of your configuration---

I'm really curious how this will work out for you. Please give it a shot and let us know if that solves your issue, otherwise happy to look into that some more 😄
Also so sorry for not responding sooner, will try and keep an eye out more in the future.

Hi @RubyBrewsday thanks for the response! We don't use standard directly, instead we use it via rubocop[https://github.com/testdouble/standard#usage-via-rubocop], so I don't think we need the standard plugin-- instead i think we need a rubocop plugin channel that includes standard as @BrianHawley mentioned.
Additional context: we use rubocop version 1.39.0

@cquinones100 Hey there, I hope you had a good weekend. Just wanted to give a heads up that I've tested out a fix and we should be pushing that up later on today. I'll let you know when that is ready to go but you should be able to use standard if you set your rubocop channel to rubocop-1-39-0 in your .codeclimate.yml file.

@cquinones100 Okie doke, those changes should have gone out yesterday afternoon. If you could try re-running analysis on your repo and let us know how that works out for you that would be awesome. In case you might not know how to manually refresh your repo, check out this support article and that should show you the easiest way too (besides pushing up new code 😄): https://docs.codeclimate.com/docs/branches-and-comparisons#section-what-triggers-a-branch-analysis

@RubyBrewsday you should also make a PR for channel/beta.

It turns out that the standard gems are not released at the same pace as the corresponding rubocopo gems that they reference. This means that referencing the standard gems at all in this engine prevents us from updating the other rubocop gems when new versions come out. This is currently preventing us from updating rubocop to 1.51.0, for instance.
It might be a good idea to have a separate set of channels for rubocop with the standard gems included, so the regular rubocop channels can keep up to date.
Alternatively, it might be a good idea to have the standard engine be able to handle the case of people running standard through rubocop instead of through its own tools, so the regular rubocop engine doesn't have to reference the standard gems.


Resolves #38: use relative paths as (source) pub.rootPath + (target) pub.url

I didn't explicitly test this on a site but I've been using the changes in the @djencks/antora-lunr pipeline extension and associated lunr-ui extension for some time with no problems.

Thanks David!
Let's use window.antora.basePath and see what happens in the wild :lion:


Intermittently Message Flow on Some Streams Appears to Stop

Within our system sometimes one of our streams stops moving messages from the publication to the image.  We have a single media driver on one machine with 43 processes and ~400 streams running.  I am running with small terms (64K) as this is a resource constrained acceptance test system.  This manifests as one of the channels continually reporting "BACK_PRESSURED (-2)" on offer.  Digging with AeronStat I see the following for stream 669.
834:              112,224 - snd-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
835:              144,992 - pub-lmt:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1078:               47,136 - sub-pos: 86 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1079:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1080:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1081:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1082:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1083:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1084:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1085:               47,136 - sub-pos:<PHONE_NUMBER>07 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1086:              112,224 - rcv-hwm: 1049 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1200:               47,136 - sub-pos: 1170 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1217:               47,136 - sub-pos: 1186 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1325:               47,136 - sub-pos: 1293 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1525:               47,136 - sub-pos: 1468 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1695:               47,136 - sub-pos: 1649 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1799:               47,136 - sub-pos: 1755 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1888:               47,136 - sub-pos: 1855 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1918:               47,136 - sub-pos: 1886 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>
1938:               47,136 - sub-pos: 1921 -436124807 669 aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>

The sender (there is only one is this set up) is published up to 112,224 where as all of the subscriptions have stopped at 47,136.  Looking at the system counters of AeronStat I can see a large number of NAKs and some flow control under flow events.
03:12:02 - Aeron Stat
=========================
  0:            3,668,480 - Bytes Sent
  1:           52,288,224 - Bytes received
  2:                    0 - Failed offers to ReceiverProxy
  3:                    0 - Failed offers to SenderProxy
  4:                    0 - Failed offers to DriverConductorProxy
  5:               41,619 - NAKs sent
  6:               41,615 - NAKs received
  7:              721,434 - Status Messages sent
  8:              721,921 - Status Messages received
  9:            1,399,492 - Heartbeats sent
 10:            1,195,703 - Heartbeats received
 11:               13,923 - Retransmits sent
 12:                  433 - Flow control under runs
 13:                    0 - Flow control over runs
 14:                    0 - Invalid packets
 15:                    0 - Errors
 16:                    0 - Short sends
 17:               29,824 - Client keep-alives
 18:                  425 - Sender flow control limits applied
 19:                    0 - Unblocked Publications
 20:                    0 - Unblocked Control Commands
 21:                    0 - Possible TTL Asymmetry
 22:                    1 - send-channel: aeron:udp?group=<IP_ADDRESS>:10000|interface=<IP_ADDRESS>

It would be useful of some of these counters could be reported against their specific streams (I've raised enhancement #233 to cover this).  If use the LogInspector, I can see that the Image does not have any messages beyond the sub-pos (43,136).
 ======================================================================
 Fri Jun 03 03:21:08 UTC 2016 Inspection dump for /dev/shm/aeron-tradex/images/UDP-ac1d0101-0-efc10001-10000-E6014379-29D-419.logbuffer
 ======================================================================
 Time of last SM: Thu Jan 01 00:00:00 UTC 1970
 Initial term id:<PHONE_NUMBER>
    Active index: 0
     Term length: 65536
      MTU length: 4096

 default Data Header{frame_length=0 version=0 flags=11000000 type=1 frame_length=0 term_offset=0 session_id=-436124807 stream_id=669 term_id=1723865762 reserved_value=0}

 Index 0 Term Meta Data status=CLEAN termOffset=0 termId=0
 Index 1 Term Meta Data status=CLEAN termOffset=0 termId=0
 Index 2 Term Meta Data status=CLEAN termOffset=0 termId=0
 %n======================================================================

...

  Is this reproducible? But maybe different streams when it happens?
Also, what version is this against? Does it change when tried against the previous version(s)?

I think it is fairly apparent here that the SM receiver window combined with the publication term window is too big and causing the cleaning of the publication term from under the receivers trying to NAK.
The publication term window length is set by aeron.publication.term.window.length. This defaults to term buffer length / 2. But if overridden might be an issue without taking the term buffer length into account.
The receiver window (in the SMs) is the min of term length and aeron.rcv.initial.window.length which defaults to 128K.
So, by default, the publication limit (pub-lmt) should only be ahead of a receiver consumption point (the sub-pos) by at most 1.5 term lengths. This seems to be about what is going on from the position counters of pub-lmt, snd-pos, sub-pos, etc.
This seems to be almost assured with 64K term lengths to cause a cleaning to happen like this.
I think what needs to happen is that the SM receiver window should be min of term length / 2 or 128K.
@mikeb01 if you can try setting aeron.rcv.initial.window.length to 64K/2 and let me know how that works for you.
In general, I think this is mostly an artifact of that window being too large wrt small term buffers for certain loss cases.

I can reproduce is fairly reliably in our test environments (running it concurrently on 25+ servers makes finding these issues pretty easy).  I'll try the window length option next week.
The particular topic that experiences this issue gets a burst of traffic just after the start up process completes (priming a bunch of caches with some reference data).
This was on the latest release (Aeron 0.9.8 & Agrona 0.5.1).

BTW, the initial cause of the loss is a question. My guess would be the SO_RCVBUF sizes in comparison to the receiver window. How much buffer are you using?

I've changed the value used for the receiver window to be what I suggested. So, you shouldn't need to change the window length.

The environment where I've been testing is probably running with defaults for buffer sizes, but I will double check when I'm back in the office.

If this setup uses a single multicast channel (or a few channels) and mostly just different streamIds, then loss is probably going to happen normally. Things should progress in any case, but it might be good to increase the buffer sizes anyway.

In some ways it's quite useful to have a bit of packet loss during our acceptance test runs, gives us confidence that the system will run okay in the face of real packet loss in production.

It is immensely useful to be able to inject loss. Which is why Aeron provides the debugging versions of the channel endpoints to simulate loss.

Didn't get a chance to test this week, will test again this Friday hopefully.

Closing.  I'm not seeing over cleaning occurring after setting aeron.rcv.initial.window.length to termLength/2.  I haven't tried the updated build as the configuration seems to be sufficient.


Is it possible to read the value of another namespace in an existing namespace?

Is your feature request related to a problem? Please describe.
import { useTranslations } from 'next-intl';
import MembershipButton from '../membershipButton';
import SignInButton from '../signInButton';

const NavigationButtons = () => {
  const t = useTranslations('NavigationButtons');
  const l = useTranslations('Login');

  return (

      <MembershipButton i18n={{ title: t('membership') }}></MembershipButton>
      <SignInButton i18n={{ signIn: l('title') }} />

  );
};

export default NavigationButtons;

Describe the solution you'd like
Navigation Buttons is what I mainly want to use, but I occasionally need to read data in another namespace, such as Login. At this time, I have to define another variable to read. Is it supported? It is written as follows:
import { useTranslations } from 'next-intl';
import MembershipButton from '../membershipButton';
import SignInButton from '../signInButton';

const NavigationButtons = () => {
  const t = useTranslations('NavigationButtons');

  return (

      <MembershipButton i18n={{ title: t('membership') }}></MembershipButton>
      <SignInButton i18n={{ signIn: t('Login:title') }} />

  );
};

export default NavigationButtons;

Describe alternatives you've considered
not

This is covered in the rendering messages docs:


Allow getting path of a non-textual file

VSCode Version: 1.8.1
OS Version: Win10

Unless I'm missing something it's impossible to get file object (or simply file path as string) in extension if you have a non-textual file open.
The use case here is that you open let's say an image file, or docx file in vscode, and you want to provide a hotkey for opening this file in your default app associated with given extension. This is a case that I wanted to fix in a small extension sandcastle/vscode-open#2.
For text file it's simple because you could simply get vscode.window.activeTextEditor.document, but that's not a case in case of any format unsupported by VSCode.
I tried to get a inspiration using workbench.action.files.copyPathOfActiveFile command, so it took me to GlobalCopyPathAction class, but it uses editorService which  I believe is not exposed to extensions API.

I fear that the proper solution would only be implemented with #2582, but I hope I'm just missing something in your API. :)

Yes - binary files have no representation in the API atm. Closing as dupe of #2582


[action_mask error]

for any game which set the "action_mask" not equal all 1, for example when creating the BaseEnv:
    if not self._continuous:
        action_mask = np.ones(self.discrete_action_num, 'int8')
    else:
        action_mask = None

    # Here I set the action 2 to be invalid:
    action_mask[2] = 0

    obs = {'observation': obs, 'action_mask': action_mask, 'to_play': -1}
    return BaseEnvTimestep(obs, rew, done, info)

Will result in the following error:
Traceback (most recent call last):
File "./zoo/custom/pkgir/config/pjk_disc_gumbel_muzero_config.py", line 93, in
train_muzero([main_config, create_config], seed=0, max_env_step=max_env_step)
File "/home/LightZero-main/lzero/entry/train_muzero.py", line 174, in train_muzero
train_data = replay_buffer.sample(batch_size, policy)
File "/home/LightZero-main/lzero/mcts/buffer/game_buffer_muzero.py", line 76, in sample
batch_target_policies_non_re = self._compute_target_policy_non_reanalyzed(
File "/home/LightZero-main/lzero/mcts/buffer/game_buffer_muzero.py", line 681, in _compute_target_policy_non_reanalyzed
batch_target_policies_non_re = np.asarray(batch_target_policies_non_re)
ValueError: setting an array element with a sequence. The requested array has an inhomogeneous shape after 2 dimensions. The detected shape was (128, 6) + inhomogeneous part.
Exception ignored in: <function MuZeroEvaluator.del at 0x7f8bebff93a0>
After reading the code in game_buffer_muzero around p.661
I found that when
if self._cfg.env_type == 'not_board_games':
The legal_actions isn't processed. But when the case is board game, the legal action is processed.
So I guess the action_mask for not_board_games scenario isn't supported?

Heelo, I understand your concerns. In previous versions, we did not specifically test for scenarios where the action_mask in not_board_games contains zeros. However, theoretically, our handling of variable action spaces should be extendable to not_board_games. Therefore, we have proposed to expand the original env_type into two variables: env_type and action_type.
In our latest PR https://github.com/opendilab/LightZero/pull/160, we have implemented and optimized this adjustment. We warmly invite you to review and test these modifications. Thank you for your valuable feedback, it is greatly appreciated and beneficial for the advancement of LightZero. Best wishes!

Hi I run the following file to test
python3 ./zoo/box2d/lunarlander/config/lunarlander_disc_gumbel_muzero_config.py
I set the polocy dict as:
env_type='not_board_games',
action_type = 'varied_action_space',
and set
action_mask[0] = 0
in env file.
If I didn't set action_type = 'varied_action_space', the error mentioned above still occur. But after setting action_type = 'varied_action_space', the error dissapeared, and the reward does increase through training step increases.
But some weired part remain: the complete value will become -inf throughout the training process as:
[12-04 22:47:17] INFO     collect end:                                                                muzero_collector.py:729
episode_count: 16
envstep_count: 1248
avg_envstep_per_episode: 78.0
avg_envstep_per_sec:<PHONE_NUMBER>4049363
avg_episode_per_sec: 2.8803709569294056
collect_time: 5.554840067217127
reward_mean: -373.18820628837983
reward_std:<PHONE_NUMBER>359075
reward_max: -135.52152484840005
reward_min: -788.0163128857876
total_envstep_count: 1248
total_episode_count: 32
total_duration: 14.115308258408682
visit_entropy: 1.3028649394086207
completed_value: -inf
[12-04 22:47:17] WARNING  NaN or Inf found in input tensor.                                                       x2num.py:14
[12-04 22:47:17] WARNING  NaN or Inf found in input tensor.
Doesn't know whether this is an issue?
By the way I've a question:

Today before you upload the fixed version pull request, I am checking the same place and tried to fix this bug. I did the exactly same thing as you did but I just use the "else" part to run the code.
But the weired thing is :
the else part in the picture I capture, the code inside it is independent to
state_index and current_index
so they just keep on producing same thing? I.e. the variable "target_policies" keeps appending same thing?
Thanks for your reply!!

Thank you for your feedback.

Regarding the issue of encountering completed_value: -inf when running lunarlander_disc_gumbel_muzero_config.py, I would like to confirm, did you only use the default configuration and make no additional modifications? Did this problem arise at the very beginning of the program execution? On my macOS system, I executed 30K environment steps and did not encounter a similar issue. In order to pinpoint the problem more accurately, please provide more detailed information.

About your observation that the code segment does not use state_index and current_index, this is because our goal here is to transform the visit count distribution obtained from MCTS search into target_policies that comply with a specific data format. This is mainly accomplished through distributions = roots_distributions[policy_index] and policy_index += 1. I acknowledge that there is redundancy in this section of the code and there are more efficient implementation methods. We will optimize it in the coming weeks. I greatly appreciate your valuable suggestion.

Best wishes!

Hi I've run
lunarlander_disc_gumbel_muzero_config.py
under the default config with
action_type = 'varied_action_space', added at p.43
and
action_mask[0] = 0 added at p.139 in
LightZero-fix-action-mask/zoo/box2d/lunarlander/envs/lunarlander_env.py
and
completed_value: -inf
occurs.
I guess that you didn't add action_mask[0] = 0  in the env file so all element in action_mask = 1. If I don't put action_mask[0] = 0 into the env file, the error won't occur neither, but this is not what we want right? Since this enhancement is created for some action is masked as 0.
Thanks!

Hello, indeed, after following your modifications, we did encounter this issue. We are currently investigating the cause and searching for a solution. Thank you for your patience and feedback.

Hi, this problem occurs because masked actions were not handled properly in gumbel muzero collecting. Now the error has been solved in #178 . We welcome you to review the change and test if it resolves the problem you were facing. Please let us know if you have any other questions or feedback. Thank you for reporting this issue!


Merge test file directory

This repository currently have the two directories to maintain test resources('test' and 'tests') but this is confusing. Let's merge these to into a single one.

recheck

this PR is not merging the directories, just doing a change in the Makefile, is that expected?

this PR is not merging the directories, just doing a change in the Makefile, is that expected?

Hmm it's wired. It seems git mv was not properly included. Let me fix it now ...

in infra-operator, heat-operator its now /tests directory, in placement, node it is /test . might want to align there as well

in infra-operator, heat-operator its now /tests directory, in placement, nova it is /test (not checked all others). might want to align there as well

Yeah. For now I'm renaming the directory in placement-operator by https://github.com/openstack-k8s-operators/placement-operator/pull/24 (it's a good timing because we can't directly rename kuttl directory once we add the reference to install_yamls Makefile).

recheck


We can’t retrieve the specified metadata object: Audience. Certain metadata types, like Audience are not currently supported by the CLI.

Summary
Short summary of what is going on or to provide context.
Steps To Reproduce:
Repository to reproduce: dreamhouse-lwc
NOTE: If your issue is not reproducable by dreamhouse-lwc, i.e. requires specific metadata or files, we require a link to a simple Salesforce project repository with a script to setup a scratch org that reproduces your problem.

This is step 1.
This is step 2. All steps should start with '1.'

Expected result
Describe what should have happened.
Actual result
Describe what actually happened instead.
Additional information
Feel free to attach a screenshot.
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):
SFDX plugin Version(to find the version of the CLI plugin run sfdx plugins --core)
OS and version:

duplicate of #571


tf.truncatediv does not support float/complex tensor

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.14.0
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Not sure if it is an expected behavior but it conflicts the documentation (https://www.tensorflow.org/api_docs/python/tf/truncatediv).
tf.truncatediv will directly crash when the input is float-related data type of complex-related datatype.
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np
import warnings
warnings.filterwarnings("ignore")
dtype_list = [
    'bfloat16', 'bool', 'complex128', 'complex64',
    'double', 'float16', 'float32',
    'float64', 'half', 'int16', 'int32', 'int64', 'int8',
    'uint16', 'uint32', 'uint64', 'uint8'
]
for dtype in dtype_list:
    x = tf.constant(np.random.rand(0), dtype=dtype)
    y = tf.constant(np.random.randint(0, 100, ()), dtype=dtype)
    try:
        out = tf.truncatediv(x,y)
        print(f"Success on dtype: {dtype}")
    except:
        print(f"Fail on dtype: {dtype}")

Relevant log output
Fail on dtype: bfloat16
Fail on dtype: bool
Fail on dtype: complex128
Fail on dtype: complex64
Fail on dtype: double
Fail on dtype: float16
Fail on dtype: float32
Fail on dtype: float64
Fail on dtype: half
Success on dtype: int16
Success on dtype: int32
Success on dtype: int64
Success on dtype: int8
Success on dtype: uint16
Success on dtype: uint32
Success on dtype: uint64
Success on dtype: uint8

@drewshark I wasn't able to replicate this issue as reported. Please check this gist on the latest TF v2.14.
The result is as follows;
Success on dtype: bfloat16
Fail on dtype: bool
Fail on dtype: complex128
Fail on dtype: complex64
Success on dtype: double
Success on dtype: float16
Success on dtype: float32
Success on dtype: float64
Success on dtype: half
Success on dtype: int16
Success on dtype: int32
Success on dtype: int64
Success on dtype: int8
Success on dtype: uint16
Success on dtype: uint32
Success on dtype: uint64
Success on dtype: uint8

Thank you!

Hi @sushreebarsa ,
I apology for my mistake, my environment is tensorflow 2.10.0 instead of 2.14.0. I confirm your result on tensorflow 2.14.0. It looks like tf.truncatediv still does not support complex tensor.

Hi @drewshark ,
The complex dtype still not supporting for truncatediv Op in tf-nightly also. We will look into this and do needful changes and will update you.
Thanks!

@drewshark,
Complex types are not supported.  Floor/truncation doesn't make much sense for complex inputs. Thank you!

Hi @tilakrayal ,
If this is the case, do you mind update the documentation here https://www.tensorflow.org/api_docs/python/tf/truncatediv to avoid inconsistency?

@drewshark,
Could you please feel free to contribute to the changes required in the documentation with the PR? Thank you!


added filepicker to load rust engine.

Hi Sineaggi,
I'd like to contribute to this project. To make things easier in the future I just added a quick and simple file picker to select the xi-editor core, it stores this in the registry.
In the future the xi-editor would of course be bundled with the application in install and the registry key won't be necessary, however for now it's a good solution to allow multiple people to work on the codebase.
I hope that's okay.

It took quite a few days to merge, are you interested in collaborating on a windows version? I've made a start of going it alone but would be happy to look at moving the code into your repository. Many hands, etc.

Hey, yes, sorry for not responding in a reasonable manner, things have been crazy on my side.
Honestly, I'd prefer not to have to modify the registry and simply assume that that xicore executable is in the same directory, similar to how the swift frontend operates.
I've been thinking about writing a custom build script run a cargo build and copy the resulting binary into the build folder, I just haven't gotten around to it yet.
I'm going to keep this open before I commit to a registry variable, but for the time being I'll push a fix that assumes the xicore executable is in the same directory as the gui executable.

I've push a quick 'fix' in 78ab5daef.


Error when loading not a valid device-id or device-name

Hello, I know I am using the correct ID but I can't get a garage door to show up, I receive the following;
[TuyaWebPlatform] Tried overwriting device config for: "ytqsem3wehsnd7hs3hvc" which is not a valid device-id or device-name.
Any ideas?

Please post the result of the steps under "How to check whether the api this library uses can control your device".
Without this it is hard for me to help you with regards to this specific issue.


Serializing a Message to Any

I'm unable to find any docs on how to create an Any from a Message. Do I need to serialize the message myself and set the typeURL?

+1, I need to deserialize an Any to a known type. In other languages, I see there are pack/unpack methods. I couldn't find such methods on the Any type (https://docs.rs/prost-types/0.6.1/src/prost_types/protobuf.rs.html#964-998 )

Similar to #277.

Subscribed... I know how to go from a type to Any, but not the other way around.

The tools for de-serializing an Any type is already there, you can match on the type URL, and decode the value field using Message::decode.
With a little bit of macro magic you don't have to write a lot of code, I have made an example repo. IMO this is simpler and can easily be adapted to your own needs.

If there is a mechanism for Message to return type_url, it can support generics.
use prost::Message;
use prost_types::Any;

fn to_any<T>(message: &T) -> Any where T: Message {
    Any {
        type_url: T::type_url().to_string(),
        value: message.encode_to_vec(),
    }
}

fn from_any<T>(message: &Any) -> Result<T, DecodeError> where T: Message + Default {
    if &message.type_url == T::type_url() {
        T::decode(message.value.as_slice())
    } else {
        Err(DecodeError::new("Invalid type_url"))
    }
}

Adding a type_url to Message would definitely help. After that it would be straightforward to add pack/unpack methods for messages.

Create a datafusion-proto crate for datafusion protobuf serialization apache/arrow-datafusion#1887

@vbkaisetsu is there a type_url for prost::Message? I don't think there is one if not this will be straightforward

@nyanchor I previously created https://github.com/tokio-rs/prost/pull/535 before to support seamless seriarization, but it needs further discussion.

Yeah, I think supporting this is a good idea. Will need to dig in a bit more to understand how other languages support this feature.

@LucioFranco the simplest way to support it is adding an associated const TYPE_URL: &'static str to either the Message trait (or another specialized trait if adding it to Message is too onerous). Or if there are genuinely good use cases, possibly a static method.
Then prost-build just needs to compute and generate it for each type.

@tarcieri yes, though I don't remember (or couldn't find but didn't look super hard) if there was actually a concrete way to generate that url that isn't google specific etc.

To my knowledge the type URLs take the form:
/<package>.<type name>

e.g.
/foo.bar.baz.MyTypeName

Bumping this one up as it's definitely something that will make my life easier.

Do we have any latest process?

One way to get started would be to add a trait with a carrier for the type URL:
pub trait TypeUrl: Message {
    const TYPE_URL: &'static str;
}

This would avoid the need to immediately populate it for every Message, but provide a common way to implement methods (e.g. on prost_types::Any) for decoding/encoding Message + TypeUrl types from/to Any.
At least initially, the TypeUrl annotations could be added manually, until such a time that prost-build can emit them automatically.

I opened #896 as my latest attempt to implement this feature

@tarcieri would you mind indicating why Name + Pkg needed ? also, would you mind indicating why type_url() isn't exported by default on Prost::Message ?

would you mind indicating why Name + Pkg needed in your implementation

Please see the discussion here: https://github.com/tokio-rs/prost/pull/858#discussion_r1262305092

also, would you mind indicating why type_url() isn't exported by default on Prost::Message

There is currently no prost-build support, so trying to do that would break all code generation in addition to all currently generated code, as it would be a breaking change to Message.
In the future when there is prost-build support, it might make sense to combine Name into Message.

so it’s a problem with the design + impl not any technical decision / desire ?
If that’s the case — prost::Message::type_url can trivially be added to prost-build

It’s something that could potentially be merged into Message in the next breaking release provided prost-build support were implemented, yes

Merging Name into Message might be problematic for custom Message implementations like prost_reflect::DynamicMessage, unless the return type were changed to something like Cow<'static, str> to allow an owned string.

Yeah, the Name trait in its current form is poorly suited to anything dynamic. We could potentially refactor it to make it easier to use for that use case (and potentially trait object safe) in the next release


Use menhir

This PR removes ocamlyacc and fully uses menhir to generate our parser. The positions in the parser actions are changed to use menhir syntax, instead of the rhs/rhs2 module which depended on the Parsing module. This actually improves some ranges (e.g. it fixes #1955).
There also some build system changes. Would be good if @tahina-pro can take a look to check I didn't mess it up.
Note: dune drives the generation of the parser, so no snapshot is checked in. I still need to measure, but it seems like even while generating the parser is fast, the build slows down a bit.
I still need to update some READMEs, posting to not lose track of this.

Everything mostly works, except incremental parsing for some reason. The test in FStar.Tests.Pars fails to parse a single decl, and throws an error positioned at the let after the module Demo. This happens even if there's nothing (EOF) after the module Demo. The same happens for other decls, but not others, assume X : False worked.
So, this reeks of something related to lookahead, but it's not the problem we had before with the parser consuming an extra token that needs to be put back. Here the parser never successfully returns, it fails to parse (incrementally) any prefix of the module.
It's weird that also it seems to accept this, see this run of menhir --interpret
Warning: 5 states have shift/reduce conflicts.
Warning: 6 shift/reduce conflicts were arbitrarily resolved.
Warning: 291 end-of-stream conflicts were arbitrarily resolved.
Ready!

Then write: oneDeclOrEOF: MODULE IDENT, and get
ACCEPT
[oneDeclOrEOF:
  [decl:
    [list(decoration):]
    [rawDecl: MODULE [qlident: [path(lident): [lident: IDENT]]]]
  ]
]

So I'm rather confused. I will look at this later. I disabled interactive tests in this branch for now. We'll probably get a failure on expected output but hopefully no serious problem.


Check sanity of stage files

Since our stage files are yaml files and can be modified by-hand, we need to add some layers of sanity checks to ensure that the format is correct and gracefully report any errors.

Note: probably worth using something like Schema or JSON-Schema.


Add python 3.9 support

Add support for python 3.9 by fixing matplotlib, and sphinx requirements
Add CI testing for python 3.9

Codecov Report

Merging #2020 (abb620a) into master (9232f29) will decrease coverage by 4.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2020      +/-   ##
==========================================
- Coverage   94.59%   89.98%   -4.62%
==========================================
  Files          50       50
  Lines       11210    11102     -108
==========================================
- Hits        10604     9990     -614
- Misses        606     1112     +506

Impacted Files
Coverage Δ

databricks/koalas/plot/plotly.py
10.52% <0.00%> (-85.53%)
:arrow_down:

...bricks/koalas/tests/plot/test_frame_plot_plotly.py
23.89% <0.00%> (-76.11%)
:arrow_down:

...ricks/koalas/tests/plot/test_series_plot_plotly.py
27.83% <0.00%> (-70.23%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
25.66% <0.00%> (-66.65%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
47.82% <0.00%> (-52.18%)
:arrow_down:

databricks/koalas/__init__.py
73.43% <0.00%> (-17.19%)
:arrow_down:

databricks/conftest.py
88.13% <0.00%> (-11.87%)
:arrow_down:

databricks/koalas/spark/functions.py
88.88% <0.00%> (-7.41%)
:arrow_down:

databricks/koalas/accessors.py
86.43% <0.00%> (-7.04%)
:arrow_down:

databricks/koalas/namespace.py
78.12% <0.00%> (-6.08%)
:arrow_down:

... and 21 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #2020 (abb620a) into master (9232f29) will decrease coverage by 4.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2020      +/-   ##
==========================================
- Coverage   94.59%   89.98%   -4.62%
==========================================
  Files          50       50
  Lines       11210    11102     -108
==========================================
- Hits        10604     9990     -614
- Misses        606     1112     +506

Impacted Files
Coverage Δ

databricks/koalas/plot/plotly.py
10.52% <0.00%> (-85.53%)
:arrow_down:

...bricks/koalas/tests/plot/test_frame_plot_plotly.py
23.89% <0.00%> (-76.11%)
:arrow_down:

...ricks/koalas/tests/plot/test_series_plot_plotly.py
27.83% <0.00%> (-70.23%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
25.66% <0.00%> (-66.65%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
47.82% <0.00%> (-52.18%)
:arrow_down:

databricks/koalas/__init__.py
73.43% <0.00%> (-17.19%)
:arrow_down:

databricks/conftest.py
88.13% <0.00%> (-11.87%)
:arrow_down:

databricks/koalas/spark/functions.py
88.88% <0.00%> (-7.41%)
:arrow_down:

databricks/koalas/accessors.py
86.43% <0.00%> (-7.04%)
:arrow_down:

databricks/koalas/namespace.py
78.12% <0.00%> (-6.08%)
:arrow_down:

... and 21 more

Continue to review full report at Codecov.

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

Waiting on conda-forge to release pyarrow 3.0.0 which supports python 3.9

Waiting on conda-forge to release pyarrow 3.0.0 which supports python 3.9

Nice, thanks @gsheni.

Nice, thanks @gsheni.

Great work, @gsheni

Great work, @gsheni

@HyukjinKwon I am have trouble getting the dev dependencies to work with Python <3.9

If I remove the pinning of (sphinx, nbconvert, nbformat), python 3.5, 3.6, 3.7, 3.8 tests fail
If I don't pin those reqs, python 3.9 test fail
What do you suggest doing?

@HyukjinKwon I am have trouble getting the dev dependencies to work with Python <3.9

If I remove the pinning of (sphinx, nbconvert, nbformat), python 3.5, 3.6, 3.7, 3.8 tests fail
If I don't pin those reqs, python 3.9 test fail
What do you suggest doing?

Can you push the changes for Python 3.9 build only in this PR for now? Will take a look how to make it compatible then.

Can you push the changes for Python 3.9 build only in this PR for now? Will take a look how to make it compatible then.

@HyukjinKwon I have pushed only Python 3.9 changes to this build.

@HyukjinKwon I have pushed only Python 3.9 changes to this build.

As conda for py3.9 seems not stable yet, how about adding a metrix for py3.9 build in pip_build?

As conda for py3.9 seems not stable yet, how about adding a metrix for py3.9 build in pip_build?

@ueshin I added a matrix for python 3.9 build but the CI continues to fail:

https://github.com/databricks/koalas/pull/2020/checks?check_run_id=1787967253

@ueshin I added a matrix for python 3.9 build but the CI continues to fail:

https://github.com/databricks/koalas/pull/2020/checks?check_run_id=1787967253

@gsheni Yeah, but at least now we can install dependencies and it's failing the Sphinx build test in lint-python.
Could you take a look at the failure? Thanks!

@gsheni Yeah, but at least now we can install dependencies and it's failing the Sphinx build test in lint-python.
Could you take a look at the failure? Thanks!

or skip to install Sphinx for py3.9 build, then at least we can see if the unit tests pass or not.

or skip to install Sphinx for py3.9 build, then at least we can see if the unit tests pass or not.

@ueshin The following tests failed
FAILED databricks/koalas/tests/test_dataframe_conversion.py::DataFrameConversionTest::test_to_excel
FAILED databricks/koalas/tests/test_dataframe_spark_io.py::DataFrameSparkIOTest::test_read_excel

@ueshin The following tests failed
FAILED databricks/koalas/tests/test_dataframe_conversion.py::DataFrameConversionTest::test_to_excel
FAILED databricks/koalas/tests/test_dataframe_spark_io.py::DataFrameSparkIOTest::test_read_excel

@gsheni As seems not working with including pandoc in requirements-dev.txt or the docker way, shall we go back to install it with pip as you did before for now?
Then could you make the two tests skip when py3.9?

@gsheni As seems not working with including pandoc in requirements-dev.txt or the docker way, shall we go back to install it with pip as you did before for now?
Then could you make the two tests skip when py3.9?


fix occasional crash moving ui root nodes

Objective
fix an occasional crash when moving ui root nodes between cameras.
occasionally, updating the TargetCamera of a ui element and then removing the element causes a crash.
i believe that is because when we assign a child in taffy, the old parent doesn't remove that child from it's children, so we have:
user: create root node N1, camera A
-> layout::set_camera_children(A) :
	- create implicit node A1
	- assign 1 as child -> taffy.children[A1] = [N1], taffy.parents[1] = A1

user: move root node N1 to camera B
-> layout::set_camera_children(B) :
	- create implicit node B1
	- assign 1 as child -> taffy.children[A1] = [N1], taffy.children[B1] = [N1], taffy.parents[1] = B1
-> layout::set_camera_children(A) :
	- remove implicit node A1 (which still has N1 as a child) ->
		-> taffy sets parent[N1] = None ***
		-> taffy.children[B1] = [N1], taffy.parents[1] = None

user: remove N1
-> layout::remove_entities(N1)
	- since parent[N1] is None, it's not removed from B1 -> taffy.children[B1] = [N1], taffy.parents[1] is removed
-> layout::set_camera_children(B)
	- remove implicit node B1
	- taffy crash accessing taffy.parents[N1]

Solution
we can work around this by making sure to remove the child from the old parent if one exists (this pr).
i think a better fix may be for taffy to check in Taffy::remove and only set the child's parent to None if it is currently equal to the node being removed but i'm not sure if there's an explicit assumption we're violating here (@nicoburns).

i think a better fix may be for taffy to check in Taffy::remove and only set the child's parent to None if it is currently equal to the node being removed but i'm not sure if there's an explicit assumption we're violating here (@nicoburns).

Taffy's tree-manipulation methods need a review in general, and probably to moved closer to a browser-like API where there is no "set_parent", only "appendChild" (possibly with a set of paralell "unsafe/unchecked" methods where explicit invariants must be upheld). For the time being, your suggested workaround seems sensible.


Adding back Algolia name and link to seach.

Added back Algolia name and link to search results container. Hid the logo, and made the text small and light color so it's not so noticeable.

Hello @richbachman,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Update recert cmd args

Added args to the recert command to allow the recert tool to update the /etc/mcs-machine-config-content.json file.
Also added --replace option to the podman run commands.

/lgtm

/approve


PyPI Package

Hi, I'm a developer on the METASPACE project.
I'm investigating methods for imzML file recalibration in conjunction with molecular databases (HMDB, lists of theoretical matrix clusters, etc.) for mostly-automatic alignment & recalibration. I'm currently evaluating using MSIWarp as part of this.
Do you plan to publish this package on PyPI so that it can be easily installed with pip?

Hi Stuart,
thank you for your interest!
We certainly want to publish the package to PyPI since it would simplify installation greatly. We're currently in the process of publishing another software package, and when we do, we'll upload both packages to PyPI.
In the meantime, is it possible for you to build it from source? Provided that CMake and a reasonably modern C++ compiler is installed on your system, the readme instructions should work. If not, I'd be happy to assist.

Hi Jonatan, thanks for the response.
Yes, I'm able to build it from source by following the readme.
For distribution I'd like to publish my workflow on PyPI as well, to make it available to some Windows users that won't have C++ compilers installed. I think I can work around not having MSIWarp on PyPI by "vendoring" it (including a pre-built copy as part of my package). However, this is a really hacky approach that obscures the dependency and potentially causes conflicts if both libraries are installed, so I'd like to avoid relying on this workaround in the long term.

We uploaded x64 windows and mac wheels to PyPI now, so hopefully you can install them with pip now.
We'll try to make the pip installation work for linux too ASAP.