Feature :: catch backend error

Descrition
Before: when a backend error occurred nothing happens. The user had no feedback or debug message.
Now: we catch backend error and store the last error message in the vuex store.
What does this PR introduce ?
To do so, we introduce a new backend response format call BackendResponse and implemented in lib/backend-response.ts.
Before: There was two possible backend responses:

list of string --> from listCollection
Dataset --> from executePipeline

Now: All backend responses have the same "meta" format. It must be a object with two fields uncompatible fields:

data can be of type DataSet (from executePipeline) or list of string (from executePipeline)
error must be of type ErrorResponse

If the response has field 'error' we now commit in the store a new state properties: backendErrorMessage.
NB : if no error, then backendErrorMessage is null
Finally we use this new properties to display the message in the template. Need design for the integration.
⚠ this PR is not tested yet
Screenshot and additional info

Codecov Report

Merging #276 into master will decrease coverage by 0.13%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master     #276      +/-   ##
==========================================
- Coverage   95.96%   95.82%   -0.14%
==========================================
  Files          76       76
  Lines        1635     1605      -30
  Branches      252      243       -9
==========================================
- Hits         1569     1538      -31
- Misses         66       67       +1

Impacted Files
Coverage Δ

src/store/state.ts
100% <ø> (ø)
:arrow_up:

src/store/getters.ts
100% <100%> (ø)
:arrow_up:

src/store/mutations.ts
92.5% <100%> (+0.39%)
:arrow_up:

src/components/Vqb.vue
77.04% <100%> (+0.77%)
:arrow_up:

src/store/backend-plugin.ts
88.88% <66.66%> (-11.12%)
:arrow_down:

src/components/stepforms/FilterStepForm.vue
90.9% <0%> (-6.07%)
:arrow_down:

src/components/Pagination.vue
100% <0%> (ø)
:arrow_up:

src/components/ActionToolbar.vue
96.15% <0%> (ø)
:arrow_up:

src/components/constants.ts
100% <0%> (ø)
:arrow_up:

... and 4 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 d25f877...c03d754. Read the comment docs.

@adimasci @vdestraitt @rhuille On avait pas décider de déléguer l'affichage au projet qui utilisent le VQB ?

Yes @Dalanir your are right... my bad... I juste corrected it :)


Unlinkify

Adds a function unlinkify, that completely undoes a call to linkify.

Closing as this is no longer merge-able.
Also, what is your use case for this?


Update dmlc-core to get rid of C++17 deprecation warning

@hcho3

I'll merge it as soon as the tests pass


[$250] BUG: DEV - Console error on web app load react-dom.development.js?.. - reported by @aneequeahmad

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch web app
See the error in console.

Expected Result:
No Errors should display
Actual Result:
Console error on web app load react-dom.development.js?61bb:86 Warning: ReactDOM.render is no longer supported in React 18. Use createRoot instead. Until you switch to the new API, your app will behave as if it's running React 17.
Workaround:
unknown
Platform:

Where is this issue occurring?

Web

Version Number: 1.2.12-3
Reproducible in staging?: N
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/196144874-c4565c78-b77f-47be-9bd1-330cc0e9581e.mov

Expensify/Expensify Issue URL:
Issue reported by: @aneequeahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661251204472079
View all open jobs on GitHub

There is an internal issue #11680 which is going to fix it. Thus this is kind of duplicate.

Thanks @parasharrajat - if this is a duplicate, do you recommend closing this one?

@parasharrajat do you think we ought to pay a reporting bonus for this when the internal issue is resolved? cc @roryabraham since you are assigned to the internal issues.
IMO this doesn't get paid and closed until we decide if (1) it's a tue duplicate or not and (2) the problem is actually fixed.

So this was reported on Aug 23 and we created an internal issue on https://github.com/Expensify/App/issues/8503#issuecomment-1261334274.
Seems like eligible for reporting bonus.

Fair's fair – @aneequeahmad beat me to reporting this so I agree with @parasharrajat's last message.

Cool, closing in favor of https://github.com/Expensify/App/issues/11680. I left a comment on that issue that @aneequeahmad should be paid the reporting bonus once it's completed.


[dev] Prune unnecessary toolchain packages

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

We have a fair number of packages in the toolchain. Toolchain packages should be a base set of packages needed to bootstrap a package building environment along with the packages needed to build them. After the recent removal of certain packages (audit, openldap, tcp_wrappers, cryptsetup, lvm2) from the toolchain, we no longer need several of their dependencies to be in the toolchain.
Change Log

Remove golang from the toolchain- not needed after previous removal of audit
Remove libcap-ng from the toolchain- not needed after previous removal of audit
Remove groff from the toolchain- not needed after previous removal of openldap
Remove perl-File-HomeDir from the toolchain- not needed after removal of groff
Remove perl-File-Which from the toolchain- not needed after removal of perl-File-HomeDir
Remove libnsl2 from the toolchain- not needed after previous removal of tcp_wrappers
Remove libtirpc from the toolchain- not needed after removal of libnsl2
Remove rpcsvc-proto from the toolchain- not needed after removal of libnsl2
Remove libpwquality from the toolchain- not needed after previous removal of cryptsetup
Remove json-c from the toolchain- not needed after previous removal of cryptsetup
Remove libaio from the toolchain- not needed after previous removal of lvm2
Remove several perl packages that are part of the groff dependency chain
Move krb5 install to before curl build
Move intltool install to before systemd-bootstrap build
Remove double installs of gperf, libselinux, libsepol

Does this affect the toolchain?

YES
Test Methodology

Full local package build to ensure packages aren't relying implicitly on removed worker chroot packages

I love the changes. Have you also ran an end-to-end pipeline build to double-check we're not breaking anywhere?

I have- the build didn't pass, but the only failures were the unrelated known failures in the dev branch.

Have you checked, if the removed packages are still needed in the following scripts and files (I think they are related to building the 1st stage toolchain):

toolchain_build_in_chroot.sh,
toolchain_build_temp_tools.sh,
toolchain-sha256sums, and
toolchain-remote-wget-list?

The only one I'm unsure about is libcap-ng. I'll be putting that on my plate later to remove from the raw toolchain, since it's not explicitly used in the LFS book. I talked at length with Jon about having packages in the raw toolchain but not produced as RPMs, and we agreed it's something we're willing to do to avoid toolchain bloat.


refactor label.js out of existence

From https://github.com/AmerMathSoc/texml-to-html/issues/451#issuecomment-2208633202

Looking more at label.js, I think I'd rather get rid of that entirely and move its logic into the various parent pieces -- label and title are just too messy.

[x] case: fn  40f8899
[x] case: ref b5aebe2
[x] case: fig etc fe5bde5
[ ] complex cases

Random thought: this might end up somewhat differently - label and title could just create spans with respective data-ams-doc and parents create wrappers and recurse title and label with those.

fn is somewhat of a wrench - it injects a sup element inside the "typical" label span. This will make the idea from the previous comment a bit harder - unless we move sup-creation downstream (or even upstream).

For the record, the tags of label parents:
toc-entry
sec
book-app-group
book-app
app
disp-formula-group
fn
statement
fig
fig-group
table-wrap
table-wrap-group
ref
secheading

disp-formula doesn't really need a figcaption wrapper around the label (or a figure for that matter).


NWPS-1086-Chevron-Menu-Click-Scrolls-To-Top

Please review it

Hi @sankar-shunmuga - Thanks for the quick fix, approved.
@sankar-shunmuga @dawntaroni - Might be worth handling this on the FE via a click event on app/assets/components/nhsuk-hee/nhsuk-hee-header/submenu.js as part of FE tidy-up work:
this.toggleLink.addEventListener("click", (event) => {
  event.preventDefault();
});


"Cenad" is listed in the drop-down menu

Description:
"Cenad" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Cenad" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Cenad, TIMIȘ (Cenad)" and "Comuna Cenad, TIMIȘ".


'Cannot bounce Puppetfile.lock!' error when Forge modules contain duplicated dependencies

Hello,
I use librarian-puppet 1.0.2 on Debian Wheezy 7.5. Puppet version is 3.6.1-1puppetlabs1 (from Puppetlabs apt repository).
My working Puppetfile is
forge "http://forge.puppetlabs.com"

mod "opentable/windowsfeature"
mod "garethr/kibana"
mod "akumria/nullmailer", "1.3.2"
mod "elasticsearch/logstash", "0.4.3"
mod "puppetlabs/apt"
mod "daks/wget",
  :git => "https://github.com/daks/puppet-wget.git"
mod "elasticsearch/elasticsearch", "0.3.0"
mod "dwerder/graphite"

librarian-puppet install do the job normally
If I add
mod "pdxcat/collectd"

I got this error
Cannot bounce Puppetfile.lock!


GUI Interface for cm-rgb-cli

I made a a GUI interface using Python 3 and PyGObject. It's makes it way easier to actually apply colors since you don't need to memorize a hex code.

This is really awesome!
Did you consider cm-rgb-monitor support as well ?
This would really lower entry level for this package and is something that is surpassing original software from CM.

Thank you very much for the support, it's actually the first proper GUI I wrote haha.
As for cm-rgb-monitor support, I haven't really been using it personally but I could definitely look into it. Maybe create a 2nd script dedicated to it. Also I was thinking of adding an install script to lower the entry level even more.

Sorry for the long wait.
I'll try to merge on weekend, I need to think of a way to restructure the project.


Meeting for participants to do a "show and tell" at the end of each day

This was brought up as a suggestion in the retro for the February onboarding week

Between this and the two daily check-ins, we think this would be a bit too much of commitment for facilitators. We're thinking about twice a week -- maybe Wednesday and Friday.

We're going to try this at the end of next week's Onboarding and see how it goes.

@njbennett, how did this go last week?


bug: limit-count plugin not working right.

Issue description
When I use limit-count plugin, I send 3 request in 3 seconds, the limit-remain should be 9999, 9998, 9997, but get the 9999,9999,9999.

And the access log only log one request.

apisix version 1.1 in master:
OS: MacOS

please update your apisix version.
and please confirm only one APISIX service is running.
if you need any help, please leave a message.


Bug: Search Unindexed Items Label is not responsive to window size when using Details view

Description
When searching or browsing an empty tag, the label at the bottom is not responsive to window size.
It's only responsive to the column headers which introduces clipping when you resized the window and the column headers are wider.
This only happens in Details View.

Steps To Reproduce

Search or browse a tag that contains no item
Resize the columns wide enough
Resize the window to see the clipping

Steps to fix

Make the label responsive to the window size.

Files Version
<IP_ADDRESS>
Windows Version
10.0.22623.1028
Relevant Assets/Logs
None

Merging with #10549


Server wont stop...

The server has to be forcibly killed to restart it. The server gets stuck on "Waiting for all replays to finish saving" or whatever it says. How do I solve this? I do not have any players recording or targeted by the mod!

Hi,
This is already covered by Issue 30.
For now, since you aren't targeting any players or recording any regions, I'd suggest you disable the mod by removing it from the mods folder while it isn't in use until this gets fixed.


Pdf to Markdown

Hey,
to increase the readability and accessibility of our documentation, I'm suggesting to favor open file formats like Markdown over closed formats.
Why? This helps in collaboration: If something is different then we want it in PDFs, we need to have the original file, and cannot easily make changes. And once those changes are made, no body understands who made those.
I suggest using markdown, and therefore started with a simple sample.

Hi Mario, +1. In fact this is closed issue for the Commonalities working group -  but as you say the rule needs to be applied throughout CAMARA.  We can use Mermaid syntax for sequence diagrams, use cases etc. within markdown as required.

closing as this is an existing issue (rather than PR) and we've put it into action with recent contributions.


Fix #351 - Added missing import

Please see #351

Coverage remained the same at 92.825% when pulling 469a668f8b55834f41336ca11bb150ebd9ab3238 on webimpress:hotfix/351 into 1f2b0c69d97f9afd331b0092f7d252ee4d1cf161 on ZF-Commons:master.

Coverage remained the same at 92.825% when pulling 469a668f8b55834f41336ca11bb150ebd9ab3238 on webimpress:hotfix/351 into 1f2b0c69d97f9afd331b0092f7d252ee4d1cf161 on ZF-Commons:master.

Good catch! Thanks!


AssetPath source parse fix

Objective
Fixes #11533
When AssetPaths are created from a string type, they are parsed into an AssetSource, a Path, and a Label.
The current method of parsing has some unnecessary quirks:

The presence of a : character is assumed to be the start of an asset source indicator.

This is not necessarily true. There are valid uses of a : character in an asset path, for example an http source's port such as localhost:80.

If there are multiple instances of ://, the last one is assumed to be the asset source deliminator.

This has some unexpected behavior. Even in a fully formed path, such as http://localhost:80, the : between localhost and 80 is assumed to be the start of an asset source, causing an error since it does not form the full sequence ://.

Solution
Changes the AssetPath's parse_internal method to be more permissive.

Only the exact sequence :// is taken to be the asset source deliminator, and only the first one if there are multiple.

As a consequence, it is no longer possible to detect a malformed asset source deliminator, and so the corresponding error was removed.

As a consequence, it is no longer possible to detect a malformed asset source deliminator, and so the corresponding error was removed.

It looks like the error type was not actually removed? I see a removed test but it sounded like you meant to remove an Error enum variant completely.

diff --git a/crates/bevy_asset/src/path.rs b/crates/bevy_asset/src/path.rs
index 689913169..9b4b68bb6 100644
--- a/crates/bevy_asset/src/path.rs
+++ b/crates/bevy_asset/src/path.rs
@@ -135,7 +135,7 @@ impl<'a> AssetPath<'a> {
         let mut label_range = None;
         while let Some((index, char)) = chars.next() {
             match char {
-                ':' => {
+                ':' if source_range.is_none() => {
                     let (_, char) = chars
                         .next()
                         .ok_or(ParseAssetPathError::InvalidSourceSyntax)?;

Seems like something like this could fix it, and avoid scanning the path twice.

As a consequence, it is no longer possible to detect a malformed asset source deliminator, and so the corresponding error was removed.

It looks like the error type was not actually removed? I see a removed test but it sounded like you meant to remove an Error enum variant completely.

My bad, you're correct. I've removed the error variant now.

diff --git a/crates/bevy_asset/src/path.rs b/crates/bevy_asset/src/path.rs
index 689913169..9b4b68bb6 100644
--- a/crates/bevy_asset/src/path.rs
+++ b/crates/bevy_asset/src/path.rs
@@ -135,7 +135,7 @@ impl<'a> AssetPath<'a> {
         let mut label_range = None;
         while let Some((index, char)) = chars.next() {
             match char {
-                ':' => {
+                ':' if source_range.is_none() => {
                     let (_, char) = chars
                         .next()
                         .ok_or(ParseAssetPathError::InvalidSourceSyntax)?;

Seems like something like this could fix it, and avoid scanning the path twice.

This still doesn't fix the issue of localhost:80 being an invalid AssetPath. This parse function is called on AssetPath creation, so if you were using something like an http:// asset source as the default, then localhost:80 ought to be a valid path, but it wouldn't be.
In any case, is using find and rfind really a significant performance hit?
They only look for the first instance of a substring, so they could early out once they find it.
I tried looking into the standard library implementation to see if they do, although I couldn't make much sense of it since it is heavily generic.

This still doesn't fix the issue of localhost:80 being an invalid AssetPath.

Does it not? The test you added passes with that change. (If not, another test would be nice)

In any case, is using find and rfind really a significant performance hit?

This code isn't likely to be in a hot path, but most asset paths at the moment use the default source with no label.

This still doesn't fix the issue of localhost:80 being an invalid AssetPath.

Does it not? The test you added passes with that change. (If not, another test would be nice)

I'll add another test to show that case.

In any case, is using find and rfind really a significant performance hit?

This code isn't likely to be in a hot path, so probably not, but most asset paths at the moment use the default source with no label.

Good point, you are right that if an AssetPath uses the default source with no label then the &str will be scanned twice. I could modify the previous algorithm so that it's always one pass. The only reason I opted not to was because it would quite messy.

I wouldn't be opposed to heavy commenting of this entire function

Maybe with some good comments though, it wouldn't matter too much if the function implementation itself is a bit messy.

I could modify the previous algorithm so that it's always one pass.

I thought it was worth discussing, because the single-passness was touted here: https://github.com/bevyengine/bevy/pull/9885 when that code was added.
I'm not personally sure how much it matters. The error being removed doesn't seem super useful, and I'm not sure that the optimization really matters in any real-world uses.
Either way, this seems like a high priority bug to fix, so I'm happy as long as the test coverage is there for future refactoring.

Yeah, I'm much more concerned with correctness and tests than performance here. We can refactor later if it shows up.

Does this branch also close #11271?

Does this branch also close #11271?

Not quite, but it does help. This PR will allow the path portion of the AssetPath to contain anything and will no longer error when it sees a : character. However, interpreting the path is the job of the AssetReader, and the default file AssetReader interprets all paths as relative.
The options for an absolute path would either be to:

Modify the file AssetReader to check the provided path for an absolute path indicator like C:.
Add a new absolute file AssetReader which uses the provided paths as absolute paths.

I've back to a single pass algorithm. It's a bit messy, but hopefully the comments I've added are enough to clarify the behavior.
I've also discovered a couple new error cases:

# in source such as #insource://a/b.test
:// in label such as source://a/b.test#://inlabel

To account for these two error cases, I've forbidden # in sources and :// in labels. I've added appropriate error types, as well as tests for these cases.

Remember to update the PR description with the final set of changes :)

This seems to accept both empty strings "" and pathless strings "source://#label". Is this intended behavior?

This is consistent with the prior behavior. The Path portion of the AssetPath doesn't really have any restrictions on it, and it is up to the AssetReader to interpret it. For example, with the default file AssetReader, an empty path "" would be interpreted as the assets folder directory and could be used with the AssetServer's load_folder method.


How to prevent the keyboard to get dismiss

Hi contributors,
I am sorry that I have to fill an issue for this, but I can't figured out the solution.
I am currently implementing a long press gesture on a component that follows the translation of the keyboard. The problem is that every time I start to touch <LongPressGestureHandler /> on this component, the keyboard get dismissed and the component with the long press gesture will translate with it.
I would like to keep the keyboard open. Any idea ?
Thanks in advance

Ok my bad!
I had some keyboardShouldPersistTaps set to always hidden on an upper layer.
Problem solved.


Codecov has not been set up properly yet

Can we make sure that the integration with Codecov is set up properly?
See https://github.com/marketplace/codecov, then go to Configure access:

When the integration has been set up properly, we should be able to

see comments on pull requests (see https://docs.codecov.io/docs/pull-request-comments)
see details of recent commits (https://codecov.io/gh/joindin/joindin-api)

For an example, see https://github.com/localheinz/composer-normalize/pull/226#issuecomment-545158388.

Just added this.
https://codecov.io/gh/joindin/joindin-api

Thank you, @svpernova09!


Multiple Folder for DAO/DAL

Multiple core/spark module for hive, cassandra and lucene.

I updated the issue details...


only space not working

I want to format code by replace all tab to space
In Eclipse Kepler, Indentation -> tab policy  => use space only
After export xml file, I found two properties
<setting id="org.eclipse.jdt.core.formatter.tabulation.char" value="space"/>
<setting
id="org.eclipse.jdt.core.formatter.use_tabs_only_for_leading_indentations"
value="true"/>

I think they are conflict settings.
There two way to format my code
===========Test1: use formatter in Eclipse===========
Expected output:
all tabs are replace to spaces
Actual output:
all tabs are replace to spaces

Steps to reproduce:
1. mouse right click
2. source
3. format

===========Test2: use maven plugin===========
Expected output:
all tabs are replace to spaces
Actual output:
all tabs are replace to spaces, !!!except leading_indentation ones!!!

Steps to reproduce:
1. run plugin

Plugin version: 0.4
Maven version: 3.1.0
Java version: 1.7_x64
Eclipse version: Kepler
OS: Win7_x64

Additional details:
I have tried to set false value to use_tabs_only_for_leading_indentations.
(in text editor, I can not find a way to do this in eclipse )
The result of the Test2 comes to right.

I guess eclipse will ignore use_tabs_only_for_leading_indentations, if
tabulation.char equals space.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 30 Oct 2013 at 3:02
Attachments:

eclipse-formatter-config.xml

I believe this is fixed in 0.5 version which will be getting a PR shortly.

See merge in #37


Diagnostics

Latency diagnostics are not executed automatically anymore because when they were, sometimes they were causing a delay in the response of the other results. And we don't want to have to wait 30 sec before seeing the results of the main diagnostics.


feat(node): add PORT_HEADER env var for reverse proxies with non-standard ports

Add support for the PORT_HEADER environment variable to @sveltejs/adapter-node that can be used to read the original port of a request from a reverse proxy, which is needed if the reverse proxy is hosted on a non-standard port (e.g. not port 80 for http nor port 443 for https).
This port is normally added as part of a x-forwarded-port header, which is used by many reverse proxies, although it's not as popular as the more common x-forwarded-proto and x-forwarded-host headers.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

It looks like tests for adapter-node have been broken and disabled since https://github.com/sveltejs/kit/commit/ecb423b54d459be8c0693c723799be4171369afd, see https://github.com/sveltejs/kit/blob/336d8ae9266112b2ae938099c2443b0cec0281ea/packages/adapter-node/package.json#L29

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

pnpm test is failing for me on the master branch, but I don't think this is necessary, since adapter-node doesn't currently have any tests. I have linted and checked my changes.

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

For the documentation, though, is there a more agnostic source we can link to about the X-Forwarded-Port header? If not, I'd probably rather just not link to anything than link to an AWS documentation page.

I've removed this link. Unfortunately, the only agnostic source I could find was https://dzone.com/articles/what-are-x-forwarded-headers-and-why-is-it-used, but it looks like that article might have been plagiarized from https://requestly.io/blog/what-are-x-forwarded-headers-and-why-it-is-used/, which is another product specific link.
Searching for "X-Forwarded-Port" on GitHub gives 44k matches, so it's clearly popular, but not popular enough to be on MDN!
I did find this blog post briefly mentioning it, but only in relation to the RFC 7239 Forwarded HTTP header, which makes it outdated (and I see you've already got https://github.com/sveltejs/kit/issues/11148 open for adding support for it to SvelteKit, although I don't know how popular it is with reverse proxies).

thank you!


fix(dependencies): Updating dependencies. 0 vulnerabilities

:tada: This PR is included in version 1.0.7 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


fix: remove unneeded cluster index

Description
Remove cluster indexes that seem not to be required
Fixes #132

https://github.com/rh-ecosystem-edge/ztp-pipeline-relocatable/runs/6902700105?check_suite_focus=true

Could I ask why index will not be required ??? just for curiosity...

Sure, check the linked issue where you can see the commands that return exactly the same result

Sure, check the linked issue where you can see the commands that return exactly the same result

mmm but what happens if we have 2 or more clusters in the config yaml file? TBH even maybe could be the same result, I don't feel comfortable changing this because this info of cluster could be useful in the future to log for example the name in a easy way....I'd like having this in case we need to refer the cluster name in the future or maybe to make it clear ...
WDYT???

Sure, check the linked issue where you can see the commands that return exactly the same result

mmm but what happens if we have 2 or more clusters in the config yaml file? TBH even maybe could be the same result, I don't feel comfortable changing this because this info of cluster could be useful in the future to log for example the name in an easy way....I'd like having this in case we need to refer the cluster name in the future or maybe to make it clear ... WDYT???

The 'more than one' cluster is handled via the '${index}' part in the command as it directly branches from the main '.spokes' section, not from within a cluster itself

hey since this requires a newer version of YQ let's bump fist the version on the repo PR #447

Not really sure as it already passed the CI tests with the previous version at https://github.com/rh-ecosystem-edge/ztp-pipeline-relocatable/runs/6902700105?check_suite_focus=true

hey since this requires a newer version of YQ let's bump fist the version on the repo PR #447

Not really sure as it already passed the CI tests with the previous version at https://github.com/rh-ecosystem-edge/ztp-pipeline-relocatable/runs/6902700105?check_suite_focus=true

On the host version it was not updated. The container version is supporting this change
But for good measure we shold bump it

lgtm

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


docs(sidenav reorg): reorganize sidenav

Changes

Remove glossary and cheatsheet from REFERENCE
Rename REFERENCE to API REFERENCE
Put Glossary under BASICS, for all three languages
Add code links for Class Details

Preview

API REFERENCE https://sidenav-restyle.firebaseapp.com/docs/ts/latest/api/
Glossary under Basics https://sidenav-restyle.firebaseapp.com/docs/ts/latest/guide/glossary.html
Code links for class details: https://sidenav-restyle.firebaseapp.com/docs/ts/latest/api/core/Injector-class.html, for NoProviderError is now a link

@naomiblack @petebacondarwin
CLOSES:
https://github.com/angular/angular.io/issues/1143

this is live and in master!


Class 'Irazasyed\Larasupport\Providers\ArtisanServiceProvider' not found Lumen

Hello there!
I am using lumen 8.0 microservice to implement an mqtt communication, but when I publish a package
php artisan vendor:publish --provider="Salman\Mqtt\MqttServiceProvider"
it shows me this error: Class 'Irazasyed\Larasupport\Providers\ArtisanServiceProvider' not found
I have already added the service provider to bootstrap/app.php file.
$app->register(Irazasyed\Larasupport\Providers\ArtisanServiceProvider::class);
thank you in advance

Hey!
Can you please try the latest version? I've just pushed with some minor changes.

Thank you! It works now


feat(auth): token renewal

add token renew
return TTLs in time.Duration

Hi @hekike, this is now in our latest SDK release! Closing this PR as this functionality has been implemented, thank you!


PHP 8 Support

Are there any known incompatibility with PHP 8?
Is it possible to add PHP 8 to composer.json?

It's a work in progress: https://github.com/nikolaposa/rate-limit/pull/43. Thank you for reminding me by the way, that PR got stuck.

It's a work in progress: https://github.com/nikolaposa/rate-limit/pull/43. Thank you for reminding me by the way, that PR got stuck.

Merged the PR, I'll create a new release tomorrow.

Merged the PR, I'll create a new release tomorrow.


Question: encrypted beacons support

Is there any support for encrypted beacons (encrypted advertisements) coming from Xiaomi LYWSD03MMC with custom ATC firmware (pvvx format)?
And if yes, how should the config look like?

No, it is not supported

Is there any plan for this new feature?
Thanks


Pug bug

Don't know how it was broken but it's really annoying.
I use PostHTML after Pug in Gulp:
.pipe($.pug())
.pipe($.posthtml())

Looks like Pug should make some HTML and PostHTML should process it after pug. But it doesn't. If I use elem attribute into Pug mixin, it's ignored by PostHTML and remains as is.
Expected behaviour
Input:
extends views/layout
block content
	main(block='foo')
		header(elem='bar')
		mixin item
			li(elem='baz') item
		ul
			+item
			+item
			+item

Output:
…

item

item

item

…

Unexpected output
…

item

item

item

…

I created a repository for this bug: https://github.com/Grawl/posthtml-bem-pug-bug

Hi @Grawl
The problem is that li elem='baz' isn't a child of the main(block='foo') element. It's a child of ul. If you add a block='foo' in mixin item it will be work
        mixin item
            li(block='foo' elem='baz') item

I think it can be a bug, but it's more complex issue than it may seem

@rajdee oh the problem was too simple. Maybe we have to mention this limitation in project readme? And it will be good to get rid of it.


Suggestion: moveable space station and downwards laser drill

I have a suggestion to add a drive that enables you to bring your station to specific coordianets on the orbiting planet and that the drive is at the specific cordinates and that the laser drills start coordinates are where the drill is located on the station

So, orbital attack (read:greifing) station? Because unless zmaster
blacklists pretty much all refined forms of stone and wood that's exactly
what will happen.
On Fri, Aug 26, 2016, 1:51 PM BramDeCreep<EMAIL_ADDRESS>wrote:

I have a suggestion to add a drive that enables you to bring your station
to specific coordianets on the orbiting planet and that the drive is at the
specific cordinates and that the laser drills start coordinates are where
the drill is located on the station
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/193, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AJzt6u9Gn9_wgCdQe51HXE76svUt7SLHks5qjzXBgaJpZM4JuXIv
.

rightnow orbital attack is already possible just enter cordinates in laser drill of enemy base and it drills it away i want this so you dont need to change all laser drills everytime you get to bedrock and maybe he can add a shield generator block that blocks the lasers

As of now there is an event system and the drill can be disabled, in theory
it would be easy to integrate with any protection mod, and it really is not
too much different than if someone put a buildcraft quarry over a base
(arguably harder and more expensive).  For survival defence I plan on
adding mirror blocks and mffs should soon have support.  At some point I'll
add a config to enable the drill but not actually destroy the surface,
pulling ores from 'far away' aka nothing similar to MFR'S.  I do like the
idea of moving the station and giving the drill a limited range though.

But griefing is exactly the reason why a config was added to completely
disable the drill (even if somebody hacked the block in it still wouldn't
do anything) while balance with it is flushed out.  For that reason I
strongly suggest disabling it if you don't trust everyone on  the server.

A planet blacklist was added in 0.8.5


Whitelisted Download Feature

For internal needs, we've built a feature to fileBlockService.js that allows downloads of certain, whitelisted filetypes. Would this be interesting for the upstream project? If so, I could make a PR out of our patch.

I don't quite understand the use case for whitelisting file types. Could you give me a case of types some would like to block?
I guess you noticed our domain whitelist. ;)

We have an enterprise environment where we want to allow only certain file types (pdf, zip, odt, ..), but no shell scripts, executable files etc. This is no real security measurement, I am aware of that.
Apart from that: Webconverger does not allow any downloads, did I get that right? We could contribute this as an optional feature.

Thank you Florian for the offer. Wouldn't mind looking at the code in a gist or something, but yes since Webconverger doesn't download any non-Web formats (by design), I don't see its utility.
Please review the existing code. Would like to hear more from you. ;)

All right, so I close this issue.


Fix to the delete-domain to fail early if domain is shared

delete-domain command is not failing early if given name is a shared
domain and delete-shared-domain is not failing early if given name is
owned domain.
Modified the delete-domain and delete-shared-domain commands to check
the domain type and fail early based on domain type.
$ cf domains
Getting domains in org abc as admin...
name                 status
<IP_ADDRESS>.xip.io   shared
abc.shared.com       shared
abc.private.domain   owned
Before Fix :
delete-domain abc.shared.com -f
Deleting domain abc.shared.com as admin...
FAILED
Error deleting domain abc.shared.com
Server error, status code: 404, error code: 130002, message: The domain could not be found: c42e00e4-0c72-4ba4-ba1c-cb4b59ff1bc0
$ cf delete-shared-domain abc.private.domain -f
Deleting domain abc.private.domain as admin...
FAILED
Error deleting domain abc.private.domain
Server error, status code: 404, error code: 130002, message: The domain could not be found: 51a6d739-5134-4dfd-8ed0-6e1d38c5a2e0
After Fix :
$ ./out/cf delete-domain abc.shared.com
domain abc.shared.com is not an owned domain
$ ./out/cf delete-shared-domain abc.private.domain
Deleting domain abc.private.domain as admin...
domain abc.private.domain is not a shared domain
[#68736518]

Hi @camelpunch ,
I have fixed all your comments.
Any suggestions,comments are welcome.
Thanks & Regards,
SriniCH

Hi @camelpunch ,
I have fixed all your comments.
Any suggestions,comments are welcome.
Thanks & Regards,
SriniCH

@goehmen how does the language sit with you? Do both "X is not an owned domain" and "X is not a shared domain" make sense in context?

Yes they do.
Greg Oehmen
Cloud Foundry Product Manager
415.205.6596
On Wed, Sep 23, 2015 at 7:44 AM, Andrew Bruce<EMAIL_ADDRESS>wrote:

@goehmen https://github.com/goehmen how does the language sit with you?
Do both "X is not an owned domain" and "X is not a shared domain" make
sense in context?
—
Reply to this email directly or view it on GitHub
https://github.com/cloudfoundry/cli/pull/596#issuecomment-142575612.

@simonleung8 @goehmen @squeedee
I'd expect this change to result in a nonzero exit code, but:
cli$ out/cf domains
Getting domains in org me as admin...
name                 status
<IP_ADDRESS>.xip.io   shared
cli$ out/cf delete-domain asdfasdf
OK
Domain asdfasdf not found
cli$ echo $?
0

Should this really be a 1 exit?

Looks like the default is a fail, so exit 1 seems appropriate
https://github.com/cloudfoundry/cli/blob/master/cf/commands/domain/delete_domain.go#L72

Hi @camelpunch   @simonleung8 , I think  you mean to  change the below lines
case *errors.ModelNotFoundError:
cmd.ui.Ok()
cmd.ui.Warn(apiErr.Error())
to
case *errors.ModelNotFoundError:
cmd.ui.Failed(apiErr.Error())
both in delete-domain & delete-shared domain commands?  am i correct?
Thanks & Regards,
SriniCH

Sorry I didn't make it clear. I meant changing the modified lines from Say() to Failed(). Since basically what happens here is that the domain exists, but the user is issuing a wrong command, it should exit non-zero.
	case nil:
		if domain.Shared {
			cmd.ui.Say(T("domain {{.DomainName}} is not an owned domain",
				map[string]interface{}{
					"DomainName": domainName}))
			return
		}

Hi @camelpunch @simonleung8,
changes are done as suggested.
Thanks & Regards,
SriniCH

@SrinivasChilveri sorry for the back-forth, but we could really do with test coverage for that last change. For example, a test fails when I change the default case to Say, but not for the nil case that you've added.

HI @camelpunch ,Thanks a lot for your review. changes are done.
Thanks&Regards,
SriniCH


Latest CRC setup fails

I'm running Ubuntu 18.04. I had been using CRC 1.10 then ran into the cert renewal issue yesterday. Reverted to 1.8 yesterday but had other issues. Noticed today that a new version (1.11) is available and downloaded and extracted:
$ ls -l
total 17003000
-rwxrwxr-x 1 eace eace<PHONE_NUMBER> May 31 23:55 crc
-rw-rw-r-- 1 eace eace<PHONE_NUMBER> Jun  2 12:38 crc-linux-amd64.tar.xz

Before downloading, I removed the 1.8 VM with crc delete. Then:
$ ./crc version
crc version: 1.11.0+883ca49
OpenShift version: 4.4.5 (embedded in binary)

Then:
$ ./crc setup --log-level=debug
INFO Checking if oc binary is cached
DEBU oc binary already cached
INFO Checking if podman remote binary is cached
DEBU Currently podman remote is not supported
INFO Checking if goodhosts binary is cached
DEBU goodhost binary already cached
DEBU /home/eace/.crc/bin/goodhosts does not have the SUID bit set (-r-x------)
INFO Caching goodhosts binary
DEBU goodhost binary cached
DEBU Making /home/eace/.crc/bin/goodhosts suid
INFO Will use root access: change ownership of /home/eace/.crc/bin/goodhosts
DEBU Running '/usr/bin/sudo chown root:wheel /home/eace/.crc/bin/goodhosts'
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: chown: invalid group: ‘root:wheel’
FATA Unable to set ownership of /home/eace/.crc/bin/goodhosts to root:wheel:  exit status 1: chown: invalid group: ‘root:wheel’

So:
$ mv /home/eace/.crc/bin/goodhosts /home/eace/.crc/bin/goodhosts.save
eace@ubuntu-desktop:~/codeready$ ./crc setup --log-level=debug
INFO Checking if oc binary is cached
DEBU oc binary already cached
INFO Checking if podman remote binary is cached
DEBU Currently podman remote is not supported
INFO Checking if goodhosts binary is cached
DEBU goodhost binary is not cached
INFO Caching goodhosts binary
DEBU Trying to extract oc from crc binary
DEBU Extracting embedded 'goodhosts-cli-linux-amd64.tar.xz' from /home/eace/codeready/crc to /tmp/crc468885978/goodhosts-cli-linux-amd64.tar.xz
DEBU Uncompressing /tmp/crc468885978/goodhosts-cli-linux-amd64.tar.xz to /tmp/crc468885978
DEBU Copying '/tmp/crc468885978/goodhosts' to '/home/eace/.crc/bin/goodhosts'
DEBU goodhost binary cached
DEBU Making /home/eace/.crc/bin/goodhosts suid
INFO Will use root access: change ownership of /home/eace/.crc/bin/goodhosts
DEBU Running '/usr/bin/sudo chown root:wheel /home/eace/.crc/bin/goodhosts'
DEBU Command failed: exit status 1
DEBU stdout:
DEBU stderr: chown: invalid group: ‘root:wheel’
FATA Unable to set ownership of /home/eace/.crc/bin/goodhosts to root:wheel:  exit status 1: chown: invalid group: ‘root:wheel’

Can you offer some guidance? Thanks

Thank you marcelomrwin, It works.

@msunilkumar This documentation can help you: https://docs.fedoraproject.org/en-US/Fedora/13/html/Virtualization_Guide/sect-Virtualization-Troubleshooting-Enabling_Intel_VT_and_AMD_V_virtualization_hardware_extensions_in_BIOS.html


ctodo: enable build on unix

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

 If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108097 run on x86_64-linux 1

1 package built:

ctodo

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.


Release 1.0.0-rc.0

:+1:

Just for future reference: I suggest to follow the same tagging scheme as Go itself:

bugfix releases have no RCs
thus, this would be 1.0rc1
1.0.1 will be released without rc
the rc for 1.1.0 would be called 1.1rc1

Cf. https://golang.org/dl/

We follow semver tagging, which is well-defined and the scheme applied here.
http://semver.org/

I see... then I have to blame the Go project to break the consistency. Too bad...

I suppose that's, as many things, a derivative of some GOOG internals that shouldn't have made it into the outside world.

I don't think so. I guess they wanted to do semver, but missed that point. So much the better that you did it right and fixed my faulty view of the world.


Flutter version 2.10.1 is not found in upstream origin

Should be now fixed by #9 . @sabaridev Can you please verify on you end?

@ludwiktrammer Having this same issue


Unclear Introspection error message: Error parsing attribute "@id": Fields that are marked as id must be required.

Introspecting SQLite:
CREATE TABLE "playlist_track" (
	"PlaylistId"	INTEGER NOT NULL,
	"TrackId"	INTEGER NOT NULL,
	"id"	INTEGER PRIMARY KEY AUTOINCREMENT,
	FOREIGN KEY("PlaylistId") REFERENCES "playlists"("PlaylistId") ON DELETE NO ACTION ON UPDATE NO ACTION,
	FOREIGN KEY("TrackId") REFERENCES "tracks"("TrackId") ON DELETE NO ACTION ON UPDATE NO ACTION
)

Creates error message:
 ERROR  Oops, an unexpected error occured!
Schema parsing

error: Error parsing attribute "@id": Fields that are marked as id must be required.
  -->  schema.prisma:126
   |
125 |   TrackId    tracks
126 |   id         Int?      @id
   |

Error message does not include name of table/model for context.

Investigation into how SQLite works:
Create a table with this SQL:
CREATE TABLE "testing" (
	"id"	INTEGER,
	PRIMARY KEY("id")
);

Executing INSERT INTO testing (id) VALUES (NULL) creates a column with the id filled with the next auto increment value.
Decision
Integer PKs should therefore be inferred as Autoincrement NOT NULL.

For clarification:
The consequence of inferring Integer PKs as Autoincrement NOT NULL (or as it would look like in the schema.prisma file: Int @id @default(autoincrement())) is that the id would be optional when writing, and non-null when reading.
DECISION: this is excellent!
See also https://github.com/prisma/specs/issues/176#issuecomment-561289686 for context on autoincrement syntax.

I could observe this issue in introspection CI. Re-opening it to re-triage.

This is not observed any longer for simple example like:
CREATE TABLE "testing" (
	"id"	INTEGER,
	PRIMARY KEY("id")
);

However, it is still observed in the following schema: https://github.com/prisma/database-schema-examples/tree/master/sqlite/sql-mysteries

@divyenduz I am unable to reproduce the error in the latest alpha:

It passed through introspection without any error, there are warning for the model which doesn't have an identifier.
Introspected schema result:
datasource db {
  provider = "sqlite"
  url      = "file:./schema.db"
}

// The underlying table does not contain a unique identifier and can therefore currently not be handled.
// model crime_scene_report {
  // city        String?
  // date        Int?
  // description String?
  // type        String?
// }

model drivers_license {
  age          Int?
  car_make     String?
  car_model    String?
  eye_color    String?
  gender       String?
  hair_color   String?
  height       Int?
  id           Int      @default(autoincrement()) @id
  plate_number String?
  person       person[]
}

model person {
  address_number      Int?
  address_street_name String?
  id                  Int                  @default(autoincrement()) @id
  license_id          Int?
  name                String?
  ssn                 Int?
  drivers_license     drivers_license?     @relation(fields: [license_id], references: [id])
  get_fit_now_member  get_fit_now_member[]
}

// The underlying table does not contain a unique identifier and can therefore currently not be handled.
// model facebook_event_checkin {
  // date       Int?
  // event_id   Int?
  // event_name String?
  // person_id  Int?
  // person     person? @relation(fields: [person_id], references: [id])
// }

// The underlying table does not contain a unique identifier and can therefore currently not be handled.
// model interview {
  // person_id  Int?
  // transcript String?
  // person     person? @relation(fields: [person_id], references: [id])
// }

model get_fit_now_member {
  id                    String? @id
  membership_start_date Int?
  membership_status     String?
  name                  String?
  person_id             Int?
  person                person? @relation(fields: [person_id], references: [id])
}

// The underlying table does not contain a unique identifier and can therefore currently not be handled.
// model get_fit_now_check_in {
  // check_in_date      Int?
  // check_in_time      Int?
  // check_out_time     Int?
  // membership_id      String?
  // get_fit_now_member get_fit_now_member? @relation(fields: [membership_id], references: [id])
// }

model income {
  annual_income Int?
  ssn           Int  @default(autoincrement()) @id
}

// The underlying table does not contain a unique identifier and can therefore currently not be handled.
// model solution {
  // user  Int?
  // value String?
// }

Version used:
@prisma/cli          : 2.0.0-beta.6
Current platform     : debian-openssl-1.1.x
Query Engine         : query-engine c48b94aa8d1d9a0e5582dcd14e25202b76303dca (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli c48b94aa8d1d9a0e5582dcd14e25202b76303dca (at<EMAIL_ADDRESS>Introspection Engine : introspection-core c48b94aa8d1d9a0e5582dcd14e25202b76303dca (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt c48b94aa8d1d9a0e5582dcd14e25202b76303dca (at<EMAIL_ADDRESS>

Your output has and optional ID, so, generate should fail (I should have mentioned that sorry)
model get_fit_now_member {
  id                    String? @id

Oh yes, thanks for catching

Introspection delivers the correct representation of the db schema here. Then generate and the schema validation rightfully fail with the proper error that the id needs to be required. I looked at the underlying sql schema for the sqlite db and that id field should really be required I think.
The only improvement I could see here is to comment that model out already during introspection since we cannot handle it anyways. I think only sqlite is affected so I'd see that as a minor bugfix.

This will probably lead to an @@ignore on the model now, but work. To be confirmed.

Scheduling for this release to confirm that @ignore fixes this.

This can only occur on SQLite since only there are optional id fields valid DDL. Models with an optional Id will now be marked as @@ignore during introspection and won't cause a validation error. If a model is handwritten with an optional id and without @@ignore, you'll still get the same validation error which is then clear enough.


.annotate() should raise ValueError when input MMIF is incompatible

Related to https://github.com/clamsproject/mmif-python/issues/163 and #52 .
As the title says, we want a CLAMS app to fail gracefully when the input MMIF is incompatible in terms of the spec version used to generate the input.

This depends on the decision to be made in https://github.com/clamsproject/mmif/issues/14 / https://github.com/clamsproject/mmif-python/issues/163 .

closed via #68.


detectText provide base64

improve the function to able to read base64.
And improve your error message for more specific.

Can you show example code that you'd like to work, as well as the error (and how you got it)?

https://github.com/GoogleCloudPlatform/cloud-vision/issues/112

As best as I can tell, he is referencing this code, which is in of itself quite cryptic.

Google's example got wrong here for "images" but only one content keys in the payload.
The problem might be the "s".

Our detectText function is here: https://github.com/GoogleCloudPlatform/google-cloud-node/blob/a3c455f765822a24d87adb3c2469b5e31b8288f2/packages/vision/src/index.js#L1535 - which as you'll see, is just a wrapper around detect() that specifies the text type.
Your input is fed to findImages_, a helper function which can take multiple forms of input and figure out how to send them to the API. The first case it handles is for Buffer types, which it will send through as base64.
So, I believe we support the feature you're looking for, however, provide the base64 string as a Buffer:
var input = new Buffer(myBase64String, 'base64')

vision.detectText(input, ...

PR sent with an addition to the docs to make this more clear: #2145


WebRTC/RTSP camera support (crowsnest)

I've recently learned about crowsnest (https://github.com/mainsail-crew/crowsnest) which supports alternative methods for serving up the cameras attached to a printer. It allows for webrtc cameras via camera-streamer. After upgrading my printer to use this package I noticed that the camera entities from the moonraker integration in Home Assistant no longer worked correctly - broken icon instead of the camera's image.
I swapped my configuration back to using ustreamer with the /webcam/?action=stream URL and deleted and reloaded the integration for moonraker and that made the camera entities work again.
The URL can be several different variations based on the source. The default for webcamd works in crowsnest if you don't change away from ustream:
####    Port 8080 equals /webcam/?action=[stream/snapshot]      #####
####    Port 8081 equals /webcam2/?action=[stream/snapshot]     #####
####    Port 8082 equals /webcam3/?action=[stream/snapshot]     #####
####    Port 8083 equals /webcam4/?action=[stream/snapshot]     #####

but you can also use camera-streamer and enable rtsp. The URLs are:
/webcam/webrtc for the camera-streamer
####    rtsp://<ip>:<rtsp_port>/stream.h264                     #####
Describe your setup
I have a Voron 2.4 with Klipper, Moonraker, Fluidd and Mainsail running from a Raspberry Pi 4 and Octopus controller.
Describe the solution you'd like
Ideally, the camera entities served into Home Assistant would support all technologies served by the software.

This is a good idea, but it's really tricky for us to implement since we don't have this hardware.
that being said if someone would implement it, we will happily merge that into the integration

There are multiple options. https://github.com/ayufan/camera-streamer/blob/main/docs/streaming.md
I had success with the /video endpoint but there is a delay. Using the RTSP url would return connection timed out in home assistant

This is a good idea, but it's really tricky for us to implement since we don't have this hardware.
that being said if someone would implement it, we will happily merge that into the integration

I don't think one would necessarily have to have this hardware to be able to implement this feature. I could very well be wrong, I'm not a Python expert.
class MoonrakerCamera(MjpegCamera):
It appears that the Moonraker integration currently assumes all video cameras are MJPEG, which is not necessarily true.
One could test by transcoding an MJPEG camer into another format supported by Moonraker/Fluidd/Mainsail (HLS, WebRTC, etc).

If it assumes mjpeg, Camera-Streamer has a mjpeg stream
http://:8080/stream

The Creality camera for my K1 finally arrived. The integrated camera in my K1 works automatically in the HA integration, but the IP camera that is defined in my moonraker.conf file does not work in the integration.
I believe this is supporting evidence to the notion that the HA integration assumes all cameras defined in moonraker.conf are mjpeg cameras.

The integration only supports mjpeg cameras

Im using a Creality K1, rooted and the camera is set up and working in Fluidd in the UI as mjpeg adaptive however the HA integration shows it as a camera but it consistently idle. Do these need to be in the moonraker.conf file to work?

I saw the last update added a way to configure camera links, any info about how to use it and in which cases? does it support webrtc/rtsp?


kernel: Update README with right instructions

Fix the instructions in the README guide
when setting up kernel source code.
Fixes:  #673
Signed-off-by: Nitesh Konkar<EMAIL_ADDRESS>

@GabyCT CI seems stuck

/test
I suspect CI not stuck, just not triggered yet...

/AzurePipelines run


Add Centrify

Thanks for the great tutorial. I was able to apply the OIDC flow to Centrify as well with very little effort. This PR adds Centrify to the list of out-of-box support.

Looks great, thanks!
https://developer.centrify.com/docs/openid-connect-_new


Provide "organize imports"

Need equivalent functionality to Source... Organize Imports.
Maybe we could even provide an improved UI that doesn't ask you which List you want, but let's you fix incorrect guesses?
(Imported from Bitbucket #5 reported by maxg on 2010-03-12)

(Changed responsible from nobody to phy.)
(maxg on 2010-06-22)

Skeleton committed in 8b8694e6837fbb1075b3f83dd427b1f390e05873.
Once the TextEditVisitor in ChangeSetOpIterator can handle InsertEdits and DeleteEdits, this code should already work in the case where no ambiguous names have to be resolved!
When prompting is needed, currently just pops up an alert(), then calls back with null, and the organize imports operation is canceled.
(maxg on 2010-06-22)

P.S. Be sure to review all XXX lines in the skeleton and revise/remove as appropriate. Do keep XXX on e.g. uses of printStackTrace or other places with outstanding issues.
(maxg on 2010-06-22)

(Changed status from open to resolved.)
(phy on 2010-08-26)


Code should not complete when pressing .

VSCode Version: 1.1.1
OS Version: Mac OSX 10.11.4

Steps to Reproduce:

Create a TypeScript file test.ts
Start typing console.log
Then type a . as if you want to type console.log.bind(console)

Current behaviour
It auto completes to console.log(., optionalParams) instead of letting the developer continue with console.log.

Expected behaviour
To be able to continue typing and get function suggestions like bind, apply, call, ...

Extra notes
This only applies in TypeScript .ts files and not in .js files. That's how I recorded the second screen capture.
I'm not sure if this actually is an issue in VS Code or somewhere in another package.

@alexandrudima could we use your cool new - keyboard shortcut feature to unbind . as a completion accept character ?

Yes, starting with the May release it will be:
{ "key": ".",                     "command": "-acceptSelectedSuggestion" }

Cool! Great work.

This started happening again in the latest release and is quite annoying.

I looked into the keyboard shortcuts and I found that the command for the . key was set to ^acceptSelectedSuggestion. Not sure what that means @alexandrudima? I fixed it by changing it back to -acceptSelectedSuggestion in my custom keybindings.json file.

@SamVerschueren The default keybindings always shipped with:
{ "key": ".",                     "command": "^acceptSelectedSuggestion",
                                     "when": "editorTextFocus && suggestWidgetVisible && suggestionSupportsAcceptOnKey && editorLangId == 'typescript'" },

The ^ means execute the command, but let the key code fall through, thus generating an internal type event for ".". In other words, pressing "." (VK_OEM_PERIOD) executes acceptSelectedSuggestion and then results in the character produced by VK_OEM_PERIOD.
My suggestion above was that you can customize your keybindings.json to remove this entry via an easy syntax:
{ "key": ".",                     "command": "-acceptSelectedSuggestion" }

So @SamVerschueren I'm not sure what exactly changed?

Strange, I thought the -acceptSelectedSuggestion shipped with VS Code since May, but I might be wrong here. I never configured it to be -acceptSelectedSuggestion and it never selected the suggestion when I pressed .. Since the latest release it started happening again so I thought that it was reverted for some reason.
It works now when I add use the - sign so I'm fine with how it works now.

Yeah, in my comment above I was saying how one can customize their keybindings.json to remove this keybinding item.
It would not make sense to ship with the -acceptSelectedSuggestions, we might as well ship without that keybinding item to begin with.

It would not make sense to ship with the -acceptSelectedSuggestions, we might as well ship without that keybinding item to begin with.

Good point! Probably something changed locally. Thanks for the explanation :).


Can you make a guide/version for Redhat/CentOS

I wanted to run the Bot on my CentOS server and the wiki didn't had an explanaition for the OS.
Thanks in advance. Yours Unrealrussian
Edit: I don't really know much about scripting and Python in generall

We don't have anyone on the staff that runs that operating system which is why there hasn't been a guide made. You should be able to reference the Ubuntu guide for the packages you need to install and Google how to install them for your system in the mean time though. Your request though is noted and we will see what we can do.

perhaps the most easy way for you is running your bot with docker

I have a machine I used to test CentOS on. If you'd like I can write a guide on it, sometime in the next few days.

@TheHeroBilly Please do write a guide, thanks!

If you really want to do this @TheHeroBilly , then i tell you that you need a really long time.
First get everything you need like python3, git and such. ([http://stackoverflow.com/questions/32618686/how-to-install-pip-in-centos-7] Link to pip if it says that it isn't installed or properly updated)
Then, i would suggest you installing all of this: https://gist.github.com/mustafaturan/7053900
in order for the bot to work properly. (that is the most timeconsuming part)
I am not sure if Opus was installed with it too, so extra link :P
wget ftp://mirror.switch.ch/pool/4/mirror/scientificlinux/7rolling/x86_64/os/Packages/opus-1.0.2-6.el7.x86_64.rpm
Now you should be safe to download the bot and install it.
Make sure that it has proper folder rights or it may error out.
And that should be it. If i missed something or something is unclear, please don't hesitate to contact me :)

@Unrealrussian I have a CentOS install working now.
https://gist.github.com/parkervcp/f598bc9899d4e44dd90c3bae5a3226a5

I made a working systemd unit file for CentOS 7 in case anyone is interested. No screen required. You will need to modify it to your username, install location and what not.
[Unit]
Description=Discord Music Bot
ConditionPathExists=/home/musicbot/MusicBot

[Service]
Type=simple
WorkingDirectory=/home/musicbot/MusicBot/
ExecStart=/usr/bin/python3.5 run.py
StandardOutput=syslog
StandardError=syslog
User=musicbot
KillSignal=SIGINT

[Install]
WantedBy=basic.target

A bit delayed, but a guide was added.
https://just-some-bots.github.io/MusicBot/installing/centos/


records.yaml  - conf_remap update to use records.yaml as input files.

A follow up from the conversion from records.config to records.yaml, now this PR updates the conf_remap  plugin to use YAML files as input files. That is beside the inline variable style which does not change the plugin can now parse YAML files.
The plugin does not support .config files for config variables anymore, YAML should be used instead.
This PR does not change the internal conf_remap structure as it just replaces the legacy config parsing by usintg TSRecYAMLConfigParse API.
Autest is also updated and a new one is introduced.

[approve ci autest]

[approve ci]


clGetPlatformIDs error

OS:Ubuntu  System:RK3399
When I first ran the example cl_convolution i got following error
Can't` load libOpenCL.so: libOpenCL.so: cannot open shared object file: No such file or directory Can't load libGLES_mali.so: libGLES_mali.so: cannot open shared object file: No such file or directory Can't load libmali.so: libmali.so: cannot open shared object file: No such file or directory RuntimeError: clGetPlatformIDs Terminated
Then  I used ran following commands in terminal
sudo apt-get install ocl-icd-opencl-dev ocl-icd-dev
sudo apt-get install cmake libx11-dev xorg-dev libglu1-mesa-dev freeglut3-dev libglew1.5 libglew1.5-dev libglu1-mesa libglu1-mesa-dev libgl1-mesa-glx libgl1-mesa-dev

again ran the program now i getting an runtime error

terminate called after throwing an instance of 'cl::Error'
  what():  clGetPlatformIDs
Aborted (core dumped)

Hi @waveajay
This seems to be a problem with the device installation, for some reason it's not finding any opencl devices.
Please make sure your device is properly setup to run opencl programs.

Hi @morgolock,is it arm compute library installation problem.
Do I have to install opencl separately?

Hi @waveajay
It looks like there is a driver issue, please refer to the device's instructions on how to install opencl.

Closing due to inactivity, reopen if this is still an issue.


Fix initializer detection for weight deduplication

As per title. It is unclear why the CI is green (https://github.com/huggingface/optimum/actions/runs/6050752962/job/16420709454) although there was still this bug cc @xenova

After running a few more tests, decoder-only models seem to work quite well! 👍  massive reductions in size.
One I did run into a problem for was nlpconnect/vit-gpt2-image-captioning, which is an encoder-decoder architecture (encoder=vit, decoder=gpt2). There shouldn't be any weights shared between the two (unlike text2text models), so this should be possible.
Post-processing the exported models...
Deduplicating shared (tied) weights...
Could not find ONNX initializer for torch parameter decoder.lm_head.weight. decoder.lm_head.weight will not be checked for deduplication.
Found different candidate ONNX initializers (likely duplicate) for the tied weights:
	decoder.lm_head.weight: set()
	decoder.transformer.wte.weight: {'decoder.transformer.wte.weight'}
Could not find unique initializers corresponding to the torch tied parameters ['decoder.lm_head.weight', 'decoder.transformer.wte.weight']. Deduplication will be skipped for this group of weights although it should be done. Please open an issue in Optimum repository.
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/__main__.py", line 503, in main_export
    models_and_onnx_configs, onnx_files_subpaths = onnx_config.post_process_exported_models(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/config.py", line 396, in post_process_exported_models
    models_and_onnx_configs, onnx_files_subpaths = super().post_process_exported_models(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/base.py", line 949, in post_process_exported_models
    models_and_onnx_configs, onnx_files_subpaths = super().post_process_exported_models(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/base.py", line 531, in post_process_exported_models
    remove_duplicate_weights_from_tied_info(
  File "/usr/local/lib/python3.10/dist-packages/optimum/onnx/graph_transformations.py", line 94, in remove_duplicate_weights_from_tied_info
    onnx_model = _deduplicate_gather_matmul(onnx_model, tied_groups_to_tie, tied_groups_map, initializer_name_to_idx)
  File "/usr/local/lib/python3.10/dist-packages/optimum/onnx/transformations_utils.py", line 458, in _deduplicate_gather_matmul
    torch_to_initializer = tied_groups_map[tuple(params)]
KeyError: ('decoder.lm_head.weight', 'decoder.transformer.wte.weight')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/content/transformers.js/scripts/convert.py", line 328, in <module>
    main()
  File "/content/transformers.js/scripts/convert.py", line 289, in main
    main_export(**export_kwargs)
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/__main__.py", line 507, in main_export
    raise Exception(
Exception: The post-processing of the ONNX export failed. The export can still be performed by passing the option --no-post-process. Detailed error: ('decoder.lm_head.weight', 'decoder.transformer.wte.weight')

More testing: https://huggingface.co/Xenova/tiny_starcoder_py/tree/main/onnx is producing very different model sizes for merged and unmerged models (68MB difference):

another problem is that the original model  is 164M params, so I would expect the quantized model to be nearer to 164 MB

One I did run into a problem for was nlpconnect/vit-gpt2-image-captioning, which is an encoder-decoder architecture (encoder=vit, decoder=gpt2). There shouldn't be any weights shared between the two (unlike text2text models), so this should be possible.

I am not sure to understand what you mean. Accelerate's find_tied_parameters do find [decoder.lm_head.weight', 'decoder.transformer.wte.weight'] that are shared for gpt2. And inspecting the exported model from optimum-cli export onnx --model nlpconnect/vit-gpt2-image-captioning vit_gpt2 with netron, this weight is indeed shared (and I don't run into the error you have).
Are you sure you checked out on the correct branch?

More testing: https://huggingface.co/Xenova/tiny_starcoder_py/tree/main/onnx is producing very different model sizes for merged and unmerged models (68MB difference):

This is a different issue I believe. Basically the Constant nodes are not deduplicated in the merged decoder since we only deduplicate initializers. For this specific model, it appears that a Slice operator takes an input data as a Constant and not as an Initializer, hence the silent non-duplication.

@xenova The issue for encoder decoders should be fixed, could you give it a try?
Actually the CI was green because we never error out on deduplication failure. Maybe we should.

Will do testing now!

Can confirm this fixes all the above issues! (bart, vision-encoder, decoder, m2m_100 and many more...)
For example, Xenova/nllb-200-distilled-600M's decoder went from 2.91GB -> 1.86GB! 🤯

Just tested t5-small, and I get the following error:
Post-processing the exported models...
Deduplicating shared (tied) weights...
Could not find ONNX initializer for torch parameter decoder.embed_tokens.weight. decoder.embed_tokens.weight will not be checked for deduplication.
Could not find ONNX initializer for torch parameter encoder.embed_tokens.weight. encoder.embed_tokens.weight will not be checked for deduplication.
Found different candidate ONNX initializers (likely duplicate) for the tied weights:
	decoder.embed_tokens.weight: set() --> ignored (may be a parameter from a part of the model not exported)
	encoder.embed_tokens.weight: set() --> ignored (may be a parameter from a part of the model not exported)
	lm_head.weight: {'onnx::MatMul_1430'}
	shared.weight: {'shared.weight'}
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/__main__.py", line 503, in main_export
    models_and_onnx_configs, onnx_files_subpaths = onnx_config.post_process_exported_models(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/base.py", line 949, in post_process_exported_models
    models_and_onnx_configs, onnx_files_subpaths = super().post_process_exported_models(
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/base.py", line 531, in post_process_exported_models
    remove_duplicate_weights_from_tied_info(
  File "/usr/local/lib/python3.10/dist-packages/optimum/onnx/graph_transformations.py", line 99, in remove_duplicate_weights_from_tied_info
    onnx_model = _deduplicate_gather_matmul(onnx_model, tied_groups_to_tie, tied_groups_map, initializer_name_to_idx)
  File "/usr/local/lib/python3.10/dist-packages/optimum/onnx/transformations_utils.py", line 506, in _deduplicate_gather_matmul
    initializer_name = next(iter(torch_to_initializer[i]["initializer_name"]))
StopIteration

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/content/transformers.js/scripts/convert.py", line 351, in <module>
    main()
  File "/content/transformers.js/scripts/convert.py", line 312, in main
    main_export(**export_kwargs)
  File "/usr/local/lib/python3.10/dist-packages/optimum/exporters/onnx/__main__.py", line 507, in main_export
    raise Exception(
Exception: The post-processing of the ONNX export failed. The export can still be performed by passing the option --no-post-process. Detailed error:

Just one thing to check: when converting a >2GB file to ONNX, and then doing the weight deduplication, then the model is <2GB, the original file remains. (https://huggingface.co/Xenova/nllb-200-distilled-600M/tree/main/onnx). Is this intended?

Wow good catch. This is not intended!

I can confirm that t5-small now works! The decoder goes from 233MB → 167MB (-28%)

@xenova https://github.com/huggingface/optimum/pull/1333#issuecomment-1705407400 should be fixed, and this PR good to go!

@echarlaix suggested to use do_constant_folding=False, and use ONNX Runtime optimizer to then remove the duplicated initializers. We may change to that later.


Update upgrade-data-controller-direct-cli.md

Removing extension upgrade part.. not needed anymore.

#sign-off


any plant for play the network resource?

hi,
does you has any plant for play the net work resource?
like mp3 file.
thanks.

dup of #35


fix watch folder failed issue of winodws

if we set "c:\test*.csv" as path of tail plugin, fluentd can't get watch files list.
user need change "/" to "" on windows. this patch is for fixed this problem.

sorry , again. please reject this one. I will submit a new patch into master.


Local Time format not working for time machine API

Describe the issue
As explained in the docs, it should be possible to specify the date/time for the time machine API in the local time zone for the requested location.  The datestring should be in the form [YYYY]-[MM]-[DD]T[HH]:[MM]:[SS] i.e. without any Z at the end (which would signify UTC time format).
Example request:
https://timemachine.pirateweather.net/forecast/my_api_key/47.08,-123.01,2000-01-01T12:00:00?exclude=daily,alerts,flags&extend=hourly&units=si&lang=en
Response:
"Invalid Time Specification"
It is really convenient to be able to specify the date/time in local time, because typically I just want the data for a particular date, so I can specify noon on that date (as per example request above) and be sure that I'm getting the right data without having to think about time zones.
Thanks!
Acknowledgements

[X] I have searched this repository and Home Assistant Repository to see if the issue has already been reported.
[X] I have read through the API documentation before opening this issue.
[X] I have written an informative title.

I tested this for myself and can confirm that the datestring only works if you query it in UTC time. I'm going to ping @alexander0042 to sort out this issue for you.

Great catch here- it's in the docs, so it should work!
I put a fix in (variable was missing) and pushed it live, so let me know if this is working again for you

Haven't heard a response for issue for about a month and I can confirm that this issue is fixed. I'm going to close this issue now but feel free to re-open if it's still not working for you.


RootPanel should be assigned automatically

Have RootPanel automatically pick the pseudo-root, or provide an easier mechanism for the user to create multiple panels without having to manually set the RootPanel.

Use the new MaterialRoot container to automatically set the root. Otherwise, specify manually with Binding.


Don't know why, didn't find any description about this in hypersistence-utils documents.

Try to create a replicating test case that proves the problem.
There are many tests asserting the number of statements that are executed, so you can use that mechanism to demonstrate how the dirty checking mechanism is affected.

@vladmihalcea Added a pr #591


Compile ‘index_max’ fail.

Thanks for your working and new uploads ~
When compiling in file 'index_max_ext', I fail like below,

I cannot find why I fail. Do I have to use this new extension? I remember last time I run this code it does not need to be done.
Thanks and regards.

Thanks for point this out. I have updated the index_max_ext module, which is ok on my system with python3.7+cuda10.1+pytorch1.1. Could you please try again and tell me the result?
Previous the cuda operation is implemented with numba. In the latest commit I remove the dependency of numba and introduce the c++ based index_max_ext to do the job.

It seems to be the same ERROR with your latest code. My system is : python3.7 + cuda 10.0 + pytorch 1.2.

In the last line, it says command 'gcc' failed with exit status 1.
I don't know if there's anything do with gcc's version.  Version of my gcc is 7.3.0. I think it should be my environment problem which I couldn't find out TAT

I also failed to compile in folder 'index_max_ext'. My system is python3.7 + Cuda 9.1 + PyTorch 1.1.

@JeremyCJM @SkyeFromKindergarten  The index_max_ext module has been updated for torch 1.2. Could you guys try again and give some feedbacks? Thanks!

I can successfully compile on my mechine now :)

Please make sure that the cuda environment is configured properly. For example, proper environment variables. For example, you may consider adding these lines to ~/.bashrc
# cuda-10.1
export CUDA_HOME=/usr/local/cuda-10.1
export PATH=/usr/local/cuda-10.1/bin:${PATH}
export LD_LIBRARY_PATH=/usr/local/cuda-10.1/lib64:${LD_LIBRARY_PATH}

Run nvcc -V to confirm that cuda is working properly.
After making sure that cuda and pytorch is installed properly, delete folder dist, build, and the file index_max.egg-info, then run python setup install to install the index_max module again.
Finally, please make sure that the index_max folder is not in the python path. There may be linking problems.

"Finally, please make sure that the index_max folder is not in the python path. There may be linking problems."
You said make sure that the index_max_ext folder ''is not", instead of ''is'' ,in the python path .Did you make a mistake or I misunderstood?

The index_max_ext folder should NOT in the python path, otherwise it may cause confusion on the one you installed into python, or that "un-installed" folder.

Hello, thank you very much for the open source of the code, the paper inspired me a lot. But there are some problems when replicating the code, about "import index_max", the compilation environment I use is pytorch1.12.0+cudatoolkit11.3., but I can't import it when importing index_max? How to solve it?

Hello, thank you very much for the open source of the code, the paper inspired me a lot. But there are some problems when replicating the code, about "import index_max", the compilation environment I use is pytorch1.12.0+cudatoolkit11.3., but I can't import it when importing index_max? How to solve it?

hi, I met this issue too. Have you solved it ?


Selectively disable Cut/Copy/Paste menu

Due to #31 such a menu was added. The options should only be enabled when they are actually sensible.

Does this mean only on OS X?

The reason this issue was marked for OS X is that at the time of creation, Linux and Windows did not have a menu bar at all and as such no items that could be enabled or disabled.

Ideally all text, but dumping output and the input fields should also be made unselectable.


Any plans to add /api/v2/orgs/1/container_workloads API endpoint to illumio-py?

Although not documented in illumio Core REST API Reference documentation, Illumio 22.5 has the following API endpoint which returns a list of configured container workloads - /api/v2/orgs/1/container_workloads. Are there plans to add the above listed API endpoint into future versions of illumio-py?

I'm planning an update with several API changes from more recent PCE versions. I'll add the container_workloads endpoints to the list.


show recent questions and qset contribution count on user profile page

re https://trello.com/c/x3JHLUGf
thx

Closing until conflicts can be resolved.


Updated to new version, cant see any pokemon

Updated to the new version and cant see any pokemon.

What is your version ?

Go app 2.0, iphone 5 ios 9.0.2

Go radar app is 2.0 iphone 5 running ios 9.0.2
Sent from my iPhone

On Aug 29, 2016, at 12:03 PM, Dimitri Dessus<EMAIL_ADDRESS>wrote:
What is your version ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@szlyrhjm GO Radar is not my app.
Please contact : https://twitter.com/miniroo321


Unity SDK Cognito sync count exception mishandled

If the client attempts to update records with a stale sync count, the exception returned from the service is not handled properly.
From Dataset.RunSyncOperation, CognitoSyncStorage.PutRecords is called, which calls AmazonCognitoSyncClient.UpdateRecords.  If the sync count of an updated record is stale, the service responds with an error.  HttpErrorResponseExceptionHandler.HandleException attempts to un-marshall this error and throw an exception.
I am seeing that the un-marshalled error code is "InvalidParameterException" and the un-marshalled status code is BadRequest.  The message is something of the form "Current SyncCount for: blarg is: 27 not: 26".
It appears to me that UpdateRecordsResponseUnmarshaller should be detecting a ResourceConflictException, but it fails to do so because the following logic is trying to match "InvalidParameterException" to "ResourceConflict".
if (errorResponse.Code != null && errorResponse.Code.Equals("ResourceConflict"))
{
     return new ResourceConflictException(errorResponse.Message, innerException, errorResponse.Type, errorResponse.Code, errorResponse.RequestId, statusCode);
}

(It also has a near miss at matching to "InvalidParameter" and throwing an InvalidParameterException.)
Because this match fails (as do all others in UpdateRecordsResponseUnmarshaller), the default case of AmazonCognitoSyncException is thrown.
Later, when CognitoSyncStorage.PutRecords catches this exception and calls its HandleException method, it is unable to generate the DataConflictException that Dataset.RunSyncOperation is expecting.  It does so, because the following logic fails to match (because the exception is an AmazonCognitoSyncException not a ResourceConflictException and because the StatusCode is BadRequest (400) not Conflict (409).
else if (ase.GetType() == typeof(ResourceConflictException)
         || ase.StatusCode == System.Net.HttpStatusCode.Conflict)
{
    return new DataConflictException(message);
}

The initial post in this issue was for iOS.  For some reason, we are seeing slightly different behavior on Android.  Instead of BadRequest / "InvalidParameterException", Android is seeing OK / "ResourceConflictException", which is failing to generate a DataConflictException as well.


Does it work with Tensorflow 2 and Python 3?

I am trying to set up an environment to have the correct versions of necessary packages in Anaconda and the setup.py just installs the latest versions of packages. I am getting some errors and it seems to mostly be version related errors. Which versions of Python and tensorflow are guaranteed to work?

Hi @ChristopherMarais , I am having the same issue. Did you find a solution?

when I try that I get the No module named 'compat'. I am using python 3.9 and tensorflow 2.6

@areiasfi; Sorry I don't have that issue. My versions: Python 3.8.10, tensorflow 2.6.0. (and no conda)


Create test.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Articles/test.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Articles/test.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Articles/test.md
:white_check_mark:Succeeded

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


[release-v1.9] bump to latest of Quarkus

Bumping to latest on midstream...

/hold
not really needed - more for visibility.
Also let's see if we get a newer version...

/close
not needed. on product build, the actual version is adjusted anyways...


Add GA4 tracking to the super breadcrumb

What
Adds tracking to the super breadcrumb.
In order to provide the capability of toggling GA4 tracking on and off, a ga4_tracking boolean is passed to the step_by_step_nav_header component (the component responsible for rendering the super breadcrumb) where it is used to determine whether GA4 data attributes should be included in the HTML. In order for this to work, a ga4_tracking parameter has been added to the page_with_step_by_step_navigation and breadcrumb_selector presenters (without these changes, the template would return an error stating that the incorrect number of parameters had been passed).
Why
Part of the GA4 migration.
Visual Changes
N/A
Trello card: https://trello.com/c/AM5JCKEd/430-add-tracking-to-super-breadcrumbs

Hi @gclssvglx - I've had to make some changes to the Ruby code (details in the PR description, happy to explain further if I haven't made sense!) and was wondering if you'd be able to also give this a look?


double-slash in import doesn't give analysis errors, but fails at compile time

Open lib/main.dart in the flutter_gallery app and update the import to have two slashes:
import 'gallery//app.dart';

There are no analysis errors as a result of this, however running the app will spit out a lot of compilation errors:
Launching lib/main.dart on iPhone XS Max in debug mode...

Compiler message:
lib/gallery//demos.dart:7:8: Error: Error when reading 'lib/gallery/demo/all.dart': No such file or directory
import '../demo/all.dart';
       ^
lib/gallery//demos.dart:99:18: Error: Getter not found: 'ShrineDemo'.
      routeName: ShrineDemo.routeName,
                 ^^^^^^^^^^
lib/gallery//demos.dart:100:51: Error: Method not found: 'ShrineDemo'.
      buildRoute: (BuildContext context) => const ShrineDemo(),
                                                  ^^^^^^^^^^
lib/gallery//demos.dart:107:18: Error: Getter not found: 'FortnightlyDemo'.
      routeName: FortnightlyDemo.routeName,
                 ^^^^^^^^^^^^^^^
lib/gallery//demos.dart:108:45: Error: Method not found: 'FortnightlyDemo'.
      buildRoute: (BuildContext context) => FortnightlyDemo(),
                                            ^^^^^^^^^^^^^^^

I would expect this to either work, and the // is just treated like a / or it should give an analysis issue. I don't know which is correct (@bwilkerson - wdyt?).

I agree that URIs should be handled consistently across tools. I'm not sure which is correct. I'm fairly certain (though I haven't double checked the code) that analyzer just uses the class Uri to do all of its manipulation, and I would also expect the other tools to handle it the same way.
But in any case, I doubt that this is a flutter specific issue, so we might want to move it to the SDK repository.

You're right, I tried again and could repro outside Flutter (I think it might be the ../ prefix whereas I tried with another folder earlier). I'll move there.

I've opened https://github.com/dart-lang/sdk/issues/36337 with a clear repo.


chore(main): release bspwm 1.17.4

:robot: I have created a release beep boop
1.17.4 (2024-08-22)
Miscellaneous

main: release bspwm 1.17.3 (#189) (76e049c)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

bspwm-v1.17.4
:sunflower:


allow lines starting with double-quotes variables

allow double-quotes vars, and allow trailing spaces before array items, ) and comments:
WHATTOSYNC=(
            "/dir1"
            # 'dir2'
            '/dir3'
            )

Good idea, thank you.


fix: Conditionally import dependency and pass RNG as param

Based on some errors building the new version of the client for wasm32-unknown-unknown, I looked at how the features from dependencies were being used for the web crate of the client and found that there were unnecessary imports. After leaving only the correct features, I found that there was a build error for miden-base (which can be reproduced by doing cargo build --no-default-features --target wasm32-unknown-unknown --workspace --exclude miden-bench-tx --features testing). This PR solves those errors.
error[E0432]: unresolved imports `winter_rand_utils::rand_array`, `winter_rand_utils::rand_value`
 --> objects/src/testing/block.rs:6:25
  |
6 | use winter_rand_utils::{rand_array, rand_value};
  |                         ^^^^^^^^^^  ^^^^^^^^^^ no `rand_value` in the root
  |                         |
  |                         no `rand_array` in the root

error[E0599]: no function or associated item named `new` found for struct `SecretKey` in the current scope
   --> objects/src/testing/account.rs:158:34
    |
158 |         let sec_key = SecretKey::new();
    |                                  ^^^ function or associated item not found in `SecretKey`
    |
note: if you're trying to build a new `SecretKey`, consider using `SecretKey::with_rng` which returns `SecretKey`
   --> /Users/ignacioamigo/.cargo/registry/src/index.crates.io-6f17d22bba15001f/miden-crypto-0.10.0/src/dsa/rpo_falcon512/keys/secret_key.rs:73:5
    |
73  |     pub fn with_rng<R: Rng>(rng: &mut R) -> Self {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

With these changes the client builds correctly.

Tested it on my machine, the error stated and client build are fixed, when this gets merged we can update the PR migration to next in miden-client.
On the other hand, when testing the compilation command mentioned in the description, other compilation errors popped up related to the use of some rngs (like StdRng and ChaCha20Rng) and some imports that only work on std. Should these errors be fixed? The normal make build and make build-no-std commands work correctly so maybe it's not necessary.

If we want to fix them maybe we can merge this and fix it in a separate PR so that we can also merge the migration in miden-client.

Yep - that sounds good!


Rename nrfb packages

This PR will rename the nrfb packages:

arch notation will be the same as in infrastructure-agent and newrelic fluent-bit output plugin for consistency as follows:
"amd64" "386" "arm" "arm64" "mips" "mips64" "mips64le" "mipsle" "ppc64le" "s390x"
Version will be removed from the package name so the infrastructure-agent will only need to know the nrfb release version
Windows packages will not include the directory structure to be consistent with linux packages

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


Does Apache Kafka need Apache Zookeeper to run Frontera?

Because I want to install apache kafka, I realize that it depends on apache zookeeper. However, it is not specified in the documentation. Does Apache Kafka need Apache Zookeeper to run Frontera?
if possible would you like to share me how to install apache kafka in Ubuntu 18.04, as i just migrate  from Windows to Ubuntu just for using frontera. Hence, im very new to linux.
In addition, I would like to ask what standard modules or files need to be created to run frontera with apache kafka and apache hbase. In fact, I already know which files and modules should be configured in the Document Cluster Setup Guide, but I want to know the naming and arrangement of it with scrapy.
As far as I know, scrapy can use the scrapy startproject tutorial command to automatically generate the standard files and modules needed for scrapy.
Thanks in advance!

Hi, ZK is mandatory requirement for Kafka, and Frontera is not using it directly. The easiest way to setup Kafka is to use Docker image or binary bundle from Confluent. Scrapy requires Frontera to be installed, and its using middleware and scheduler implementations from Fronteta.

@sibiryakov thanks your suggestion, but i have done installed kafka package in my pc. I have start my db worker module by this command $ python -m frontera.worker.db --config tutorial.frontera.dbw --no-incoming --partitions 0 1 i got the result in below screenshots.

It is an indefinite loop, is that normal? Sorry for this simple question, as im very new beginner in hbase, kafka and frontera.

@sibiryakov  Hi, thanks your suggestion about the kafka. But i have installed it in my pc. I tend to build kafka+hbase crawler.
I have few questions, first when i run this command
scrapy crawl tutorial -L INFO -s SPIDER_PARTITION_ID=0
i got this error
ModuleNotFoundError: No module named 'frontera.contrib.scrapy.middlewares.seeds'

after i removed, i can run the scrapy, but 0 page crawled
SPIDER_MIDDLEWARES = {     'frontera.contrib.scrapy.middlewares.schedulers.SchedulerSpiderMiddleware': 999, ̶ ̶ ̶ ̶'̶f̶r̶o̶n̶t̶e̶r̶a̶.̶c̶o̶n̶t̶r̶i̶b̶.̶s̶c̶r̶a̶p̶y̶.̶m̶i̶d̶d̶l̶e̶w̶a̶r̶e̶s̶.̶s̶e̶e̶d̶s̶.̶f̶i̶l̶e̶.̶F̶i̶l̶e̶S̶e̶e̶d̶L̶o̶a̶d̶e̶r̶'̶:̶ ̶1̶,̶ }

besides, my kafka didnt consume any message

All my config is followed the document cluster setup guide.
Thanks in advance!


Dusk dependencies fails to build on MacOS (M1 apple chip)

Description:
When INSTALL_DUSK_DEPS=true, workspace fails on MacOs using M1 apple chip
Expected Behavior:
Is expected to build workspace correctly
Context information:
Output of git rev-parse HEAD
3d57064466e3232cecc6c47ebdf9ca164b8773c3

Output of docker version
Docker version 20.10.6, build 370c289

Output of docker-compose version
docker-compose version 1.29.1, build c34c88b2

System info: Mac, Windows or Linux. Include which disto/version
MacOs Big Sur 11.4 (20F71) - M1 Apple chip

Steps to reproduce the issue:

Change file .env update line INSTALL_DUSK_DEPS=true,
Rebuild using docker-compose build --no-cache workspace
Pop up the error message

Stacktrace & Additional info:
2021-06-08 12:23:56 (3.37 MB/s) - ‘google-chrome-stable_current_amd64.deb’ saved [83335940/83335940]

dpkg: error processing archive google-chrome-stable_current_amd64.deb (--install):
 package architecture (amd64) does not match system (arm64)
Errors were encountered while processing:
 google-chrome-stable_current_amd64.deb

up


support pulling private images from docker pull

Signed-off-by: Victor Vieux<EMAIL_ADDRESS>

ping @aluzzardi

LGTM


IntelliSense cannot find variables from typing module

Environment data

VS Code version: 1.44.2
Extension version (available under the Extensions sidebar): 2020.5.78807
OS and version: macOS Mojave v10.14.6
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.2, Anaconda
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: typing
Relevant/affected Python-related VS Code extensions and their versions: N/A?
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): Language Server (python.jediEnabled=false)
Value of the python.languageServer setting: Microsoft

Actual behaviour
Trying to navigate to the definition of certain variables defined in the typing module fails. This includes but is not limited to typing.Pattern and typing.TYPE_CHECKING.
This error occurs regardless of whether from typing import <variables> or import typing; typing.<variable> is used.
Using Ctrl-Click to navigate to the function definition does nothing. Using F12 or the right-click context menu to navigate to the definition causes a pop-up to appear that says, "No definition found for ''".
Steps to reproduce:

In a new Python file, type:

import re
from typing import Pattern, TYPE_CHECKING

if TYPE_CHECKING:
    pass

x: Pattern = re.compile("foo")

Try to navigate to the definition of TYPING in if TYPE_CHECKING: or Pattern in x: Pattern = re.compile("foo"). VS Code won't be able to find the definition.
However, you can navigate inside the typing.py file by Ctrl-Click-ing typing in from typing and find the two variables defined there.

Logs
Nothing came up in the logs when trying to navigate to the code definition of the variables mentioned above.

Note: I know this works with Jedi enabled, but I'd like to see this issue fixed with the Microsoft language server.