Updated auto location logic

Had some issues with the auto-location implementation. This is a bit hacky, but if someone wants to clean up the "use the same result" code and you are ok with adding a new express route, this works a lot better.
Not having global notifications from a node_helper to every frontend module really limits how this can be implemented, and this is currently the best I could come up with.

Warnings

:warning:

Please include an updated CHANGELOG.md file.This way we can keep track of all the contributions.

  Generated by :no_entry_sign: dangerJS

The frontend modules that use the auto-location don't even really need to do it in the node_helpers at this point, they could just refetch if they recieve a 500 from the /location url. I don't have the drive right now to clean this up more though.

Hi Jacob,
The amount of changes necessary for this function feels way to impactful for the number of users that will use it. Most of the code would fit perfectly fine in a separate 3rd-party module which would broadcast the location to any interested module. In that case the default modules only need to listen to a LOCATION notification, which would set the location of the module if a location for that specific module is not provided by the config.
I suggest you take this route, remove all the current auto location code from the MagicMirror² core (effectively reverting this PR: https://github.com/MichMich/MagicMirror/pull/1564) and only add the LOCATION notification listeners for the modules that need it. Then you can create your own module that does the heavy lifting.
This would allow other modules to leverage the 'LOCATION' notification as well, both by broadcasting one, as well as listing to the notification.
Since the current solution is a bit hacky, and doesn't add a lot of functionality to the core, I'll remove your current implementation before I merge develop into master if no alternative solution is provided via a PR.
I hope you understand.
Looking forward to your PR using the above suggested route. Thanks!

FYI: I already created a PR to revert PR 1564: https://github.com/MichMich/MagicMirror/pull/1592


signalbackup-tools: 20230322 -> 20230329

Description of changes
Update to latest version. No changes that effect build (diff: https://github.com/bepaald/signalbackup-tools/compare/20230322...20230329).
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.


react-testing-library changed import

https://github.com/testing-library/react-testing-library
It changed from react-testing-library to @testing-library/react
and it gives a warning when you install using the old name.

https://github.com/hyanmandian/react-etalpreliob/pull/18
The pullrequest fixing it

The image o the warning

Also as a suggestion when you commit to this repository makes sure you change the package-lock.json, otherwise you could possibly give information about your employeer.

closed by #18


Directory completion issue

I'm not sure whether this is a emacs-ycmd or company problem, so reporting this here first:
When completion paths in ycm, there are relevant completions, but when one of them is selected (by pressing enter), the whole path is replaced, leaving only the basename.
For example:

typing
./some/path/dir/
suggestions
file1
file2
selecting one of them by pressing enter
now the whole path (including the leading ./some/path/dir/) is replaced by
file1

(@Valloric fyi)

I cannot reproduce this with latest company, emacs-ycmd and ycmd. For me the file suggestion does not replace the whole path.
Do you have more details on the repro?

~ $ mkdir aaa
~ $ emacs hello.sh
In Emacs:
cd ~/a
aaa -> [Dir]
{ press Enter or Tab }
cd aaa
(note the missing ~/, same works if you use /tmp or another dir)

@r4nt I could reproduce the issue with your example. However I am not able to reproduce it in a "regular ycmd" c++ project with a .ycm_extra_conf.py loaded.

I tried the reproduction steps also in a python buffer and I could not reproduce the bug. It seems it is filetype related.

Yes, this is for bash files. Any reason YCM shouldn't work for shell? (@Valloric)

No ideas, sorry.

One more data point: these kind of path completions work fine in a normal text buffer (.txt), but not in a .sh file. Could it be that YCM is misconfigured just for this particular mode?

It's also happening when editing my .emacs file.

@apresta Are you seeing any suspicious-looking output in *Messages* when you see the bad behavior?

Nothing in Messages. In ycmd-server I see "Received completion request" and "Using filetype completion: False", but nothing when I press Enter to accept the completion (triggering the problem).

OK, thanks for checking. I'm looking into this right now, and I think I'm on the right trail. I'll let you know how it goes...

It appears that we're ignoring ycmd's advice on where to put the completion result. Here's the HTTP traffic involved in one buggy completion:

"{\"file_data\":{\"\\/Users\\/sixtynorth\\/sandbox\\/ycmd\\/hello.sh\":{\"contents\":\"cd ~\\/san\\n\",\"filetypes\":[\"generic\"]}},\"filepath\":\"\\/Users\\/sixtynorth\\/sandbox\\/ycmd\\/hello.sh\",\"line_num\":1,\"column_num\":9}"
HTTP RESPONSE CONTENT

((completion_start_column . 6)
 (completions .
	      [((extra_menu_info . "[Dir]")
		(insertion_text . "sandbox"))
	       ((extra_menu_info . "[File]")
		(insertion_text . "passport 01.small.png"))
	       ((extra_menu_info . "[File]")
		(insertion_text . ".csearchindex"))]))

ycmd is clearly instructing us to start the completion at column 6 – which seems to absolutely correct – yet it's getting inserted at column 4.
Another related clue is that when we provide a prefix to company-mode via company-ycmd--prefix, we respond with the text starting at column 4, i.e. the ~/sa portion of cd ~/sa.

The problem seems to definitely be a mismatch between a) what we report to company-mode for the prefix and b) what ycmd returns for completion. The proximal culprit here in our case is company-grab-symbol which we use to determine the prefix. For example, if my cursor is at the end of this:
cd ~/sa

the prefix calculated by company-grab-symbol is ~/sa. However, ycmd calculates candidates like sandbox and salami rather than ~/sandbox and ~/salami (which is a very important directory.) As a result, company is replacing the prefix with the candidates, and this results in the behavior we're seeing.
I'm not sure how to fix this yet. If I replace company-grab-symbol with company-grab-word, this specific problem goes away, but I'm not sure how it impacts other completions (i.e. in C++, etc.) With that said, I selected company-grab-symbol mostly because it sounded like the most official function to use...it doesn't have any documentation.
One possibility is that we can figure out how to "short-circuit" company mode and simply tell it to insert the completion at the offset supplied by ycmd. This would be great.

With that said, I selected company-grab-symbol mostly because it sounded like the most official function to use...it doesn't have any documentation.

You're basically expected the read the source, and see if it corresponds to your expectations (and expectations of underlying technologies). The framework has no way of anticipating where completions returned by ycmd would start.

@dgutov I wouldn't expect the framework to anticipate the completion insertion points, of course. However, in my case I have the insertion points in hand, and it would be convenient if I could communicate that to company.

@apresta @r4nt I've pushed a change to the fix-prefix branch that hopes to address the issue. Could you try it out and see if it helps? It definitely works in the limited testing I've done here.
The core problem is a mismatch between how company and ycmd determine where to insert completions. This change tries to reconcile that difference. It's a bit hacky (in ways I'm trying to rectify), but it seems to hobble along.

@abingham Why not use (- (point (length prefix)) directly later, instead of saving the value to a property?

That's a very good question. If there was a good reason for it, I don't remember it now. Probably just my being tired...I'll take care of that soon.

If there are cases when this fails, I'd like to know about them. Some other actions might get called from other positions in the buffer (like annotation, although that might be something to fix, too), but not candidates.

The fix I put in place for this issue seems to be working fine, so I've merged it into master hereeffbc7a343551fe01614c1658551d4b21ede1a93. Let me know if you see any problems.


[Visual] Dropdown menu overlaps text in Widescreen view

GoAccess is built after commit dfbbe8b66f22dbda9ceddc5005d0114529d7d582

Good point. The widescreen was added mostly for 4K dashboards so I'm not sure it's worth moving the dropdowns.
I'll close this and if it becomes a bigger issue I can look into possible solutions. Thanks though.


Use require-style for uproxy-obfuscators

Started here: https://github.com/uProxy/uproxy-obfuscators/tree/iislucas

Found a way to use have uproxy-networking use it as is for now.

Seems like a duplicate of:
https://github.com/uProxy/uproxy/issues/1250


Control pinned tabs location

Testing https://github.com/microsoft/vscode/issues/98019
The workbench.editor.openPositioning setting makes me wonder if we should also control where pinned tabs are placed: left or right. This would work great for those users who place the sidebar on the right.

Setting to candidates to see if more people ask for it, I think the idea makes sense.

Great idea, It could look like this eg.:


[Win10] Ahoviewer launches minimized and stays minimized

Basically, what the title says.
I tried updating to the newest release, no cigar. I tried uninstalling and installing the newest release – no cigar.
When hovering over the icon on taskbar I can see the miniature of the window, but when hovering over the miniature it simply shows my uncovered desktop.
Edit: Uninstalled it again, with Revo, deleted both registry keys, all files in appdata, every single trace of Ahoviewer on my disk. Installed it again. The UI isis broken, it seems.

Open %LOCALAPPDATA%/ahoviewer/ahoviewer.cfg and set the x = # and y = # to 0.
There was a problem in an older version that would save those values to -32000 and cause ahoviewer to never appear on your screen again until you edit the config as described above.
The bug has been fixed since version 1.6.0.

The entire ahoviewer appdata folder was nuked. Only then I installed the newest available release. Screenshot in my opening post shows the result.


DirbScan Module issue

I believe the DirbScan Module is not replacing %EXT% and is just appending it to the end of the directory. Would be nice if I could skip this module now because its getting nothing but 400 errors. I believe I can fix this myself, but I assume everyone else is also having this problem.
[*] Waiting for DirbScan module
+ http://redacted.com/Searchadminbox.%EXT%.php (CODE:400|SIZE:242)
+ http://redacted.com/SiteServer/Admin/commerce/foundation/driver.%EXT%.php (CODE:400|SIZE:242)
+ http://redacted.com/SiteServer/admin/findvserver.%EXT%.php (CODE:400|SIZE:242)
+ http://redacted.com/Sites/Knowledge/Membership/Inspiredtutorial/Viewcode.%EXT%.php (CODE:400|SIZE:166)
+ http://redacted.com/about.%EXT%.jsp (CODE:400|SIZE:166)
+ http://redacted.com/Sites/Samples/Knowledge/Push/ViewCode.%EXT%.php (CODE:400|SIZE:242)
+ http://redacted.com/aboutus.%EXT%.jsp (CODE:400|SIZE:242)
+ http://redacted.com/access_admin.%EXT%.jsp (CODE:400|SIZE:166)
+ http://redacted.com/account.%EXT%.jsp (CODE:400|SIZE:242)
+ http://redacted.com/actions_admin.%EXT%.jsp (CODE:400|SIZE:242)
+ http://redacted.com/addadmin.%EXT%.jsp (CODE:400|SIZE:166)

@j3ssie Nice tool by the way! I love it!

Thank you for the feedback!
Will be fix with #88


chore(deps): update dependency zone.js to ~0.13.0

This PR contains the following updates:

Package
Type
Update
Change

zone.js (changelog)
dependencies
minor
~0.12.0 -> ~0.13.0

Release Notes

angular/angular
v0.13.0
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/zone.js
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| ~0.12.0 || ~0.13.0" from<EMAIL_ADDRESS>npm ERR!   node_modules/@angular/core
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/@angular-material-components/datetime-picker
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     10 more (@angular/animations, @angular/cdk, @angular/common, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ~0.12.0" from<EMAIL_ADDRESS>npm ERR! node_modules/ngx-markdown
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/zone.js
npm ERR!   peer<EMAIL_ADDRESS>|| ~0.12.0" from<EMAIL_ADDRESS>npm ERR!   node_modules/ngx-markdown
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /tmp/renovate/cache/others/npm/_logs/2023-03-06T21_38_23_415Z-eresolve-report.txt

npm ERR!


Finish drawing a geometry while freehand is active cause error on next pan

http://openlayers.org/en/latest/examples/draw-and-modify-features.html
Start drawing a polygon
Hold SHIFT to enable freehand mode
Double-click to finish the geometry
Try to pan, it throws an exception : Cannot read property 'getGeometry' of null

Fixed with #5769.


422 error found duplicate series for the match group

Description
Container Query gives this error if it finds duplicate record
{
  "status": "error",
  "errorType": "execution",
  "error": "found duplicate series for the match group {namespace=\"cadvisor\", pod=\"cadvisor-pgf6n\"} on the right hand-side of the operation: [{namespace=\"cadvisor\", pod=\"cadvisor-pgf6n\", prometheus=\"monitoring/k8stage\", prometheus_replica=\"prometheus-k8stage-1\", workload=\"cadvisor\", workload_type=\"daemonset\"}, {namespace=\"cadvisor\", pod=\"cadvisor-pgf6n\", prometheus=\"monitoring/k8stage\", prometheus_replica=\"prometheus-k8stage-0\", workload=\"cadvisor\", workload_type=\"daemonset\"}];many-to-many matching not allowed: matching labels must be unique on one side"
}

This error occurs in Prometheus when two sets of data with overlapping labels are involved in a query that requires unique matches on one side. Specifically, it indicates that there are duplicate series on the right-hand side with the same labels {namespace="cadvisor", pod="cadvisor-pgf6n"}.
Fixes # (issue)
query is resulting in a "many-to-many matching not allowed" error because both kube_pod_container_info and namespace_workload_pod:kube_pod_owner:relabel have overlapping label sets that prevent unique matching on the right-hand side of the * operation.
Here’s how you can refine it to avoid the conflict:
Adjust the Labels for Unique Matching: Use on and group_left() correctly to ensure the labels specified for matching (pod and namespace) are unique across one side of the operation. You can try refining the query by removing unnecessary labels that may create duplication issues.
Aggregate by prometheus_replica (if present): If duplicate series with the prometheus_replica label are causing the conflict, you can aggregate over this label by using avg or sum functions to consolidate the data, removing the prometheus_replica dimension.
sum by (container, image, workload, workload_type, namespace) (
  avg_over_time(kube_pod_container_info{container!=""}[15d])
  * on (pod, namespace) group_left(workload, workload_type)
  avg by (pod, namespace, workload) (avg_over_time(namespace_workload_pod:kube_pod_owner:relabel{workload!=""}[15d]))
)

Type of change

[x] Bug fix
[ ] New feature
[ ] Docs update
[ ] Breaking change (What changes might users need to make in their application due to this PR?)
[ ] Requires DB changes

How has this been tested?
Please describe the tests that were run to verify your changes and steps to reproduce. Please specify any test configuration required.

[ ] New Test X
[x] Functional testsuite

Test Configuration

Kubernetes clusters tested on:

Checklist :dart:

[ ] Followed coding guidelines
[ ] Comments added
[ ] Dependent changes merged
[ ] Documentation updated
[ ] Tests added or updated

Additional information
Include any additional information such as links, test results, screenshots here

@msvinaykumar Can you please try
sum by (container, image, workload, workload_type, namespace) (
  avg_over_time(kube_pod_container_info{}[15d]) *
  on (pod, namespace,prometheus_replica) group_left(workload, workload_type)
   avg_over_time(namespace_workload_pod:kube_pod_owner:relabel{}[15d])
)
``
and compare what is the performance difference between this query and query mentioned in the code.

@msvinaykumar Can you please try
sum by (container, image, workload, workload_type, namespace) (
  avg_over_time(kube_pod_container_info{}[15d]) *
  on (pod, namespace,prometheus_replica) group_left(workload, workload_type)
   avg_over_time(namespace_workload_pod:kube_pod_owner:relabel{}[15d])
)

and compare what is the performance difference between this query and query mentioned in the code.

Done. Not much performance impact


Fix/33006

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/33006
PROPOSAL: https://github.com/Expensify/App/issues/33006#issuecomment-1854655489
Tests

[x] Verify that no errors appear in the JS console
Navigate to settings -> profile -> DOB
change months by arrow key click the forward arrow of the month (>) until you get to January, note year is changed now  open year picker and verify the changed year is highlighted
click the backward arrow of the month (<) until you get to december, note year is changed now open year picker and verify the changed year is highlighted

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console
Navigate to settings -> profile -> DOB
change months by arrow key click the forward arrow of the month (>) until you get to January, note year is changed now  open year picker and verify the changed year is highlighted
click the backward arrow of the month (<) until you get to december, note year is changed now open year picker and verify the changed year is highlighted

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/104348397/e60632af-d0a2-4c40-bd52-0b1c323757d8

Android: mWeb Chrome
https://github.com/Expensify/App/assets/104348397/84fce44a-388b-485f-ab2e-d2bb33777a83

iOS: Native
https://github.com/Expensify/App/assets/104348397/eeb28cdc-a359-4c05-8082-9138aca3a5fc

iOS: mWeb Safari
https://github.com/Expensify/App/assets/104348397/e0022b8f-768e-4c3e-b5ea-e2831ca36cad

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/104348397/79171fdb-7f9d-4e69-9421-ed0001b65877

MacOS: Desktop
https://github.com/Expensify/App/assets/104348397/c4e9fc80-b42d-4b38-ae54-2178b5e84b26

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@ishpaul777 Could you update the checklist with new one https://raw.githubusercontent.com/Expensify/App/main/.github/PULL_REQUEST_TEMPLATE.md? Thanks!

Sorry for the delay, I was OOO. Checking now.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/danieldoglas in version: 1.4.23-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.23-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


bg-blue/50 + bg-opacity-100 doesn't change opacity

I understand that these classes generate different outputs.

bg-blue/50 generates { background-color: rgb(0 0 255 / 0.5) }, which ignores the Tailwind variable --tw-bg-opacity
While bg-opacity-100 generates { --tw-bg-opacity: 1 }

However, this limitation is not documented.
Should it be documented? Or is this a bug?

Hey @hmaesta! This is a limitation of Tailwind CSS but not tailwind-merge. tailwind-merge only decides how different classes in strings are merged together, but doesn't alter the resulting CSS.
Or am I misunderstanding what you mean? 🤔

This is a limitation of Tailwind CSS

That's what I thought, but I just wanted to be sure. I was hoping tailwind-merge could find a way around that limitation somehow. 😅

Yeah tailwind-merge can't change the CSS produced by Tailwind, only Tailwind CSS can do that. I suggest to open an issue in the https://github.com/tailwindlabs/tailwindcss repo.


AndroidX support

Fixed #205

Please merge this PR and publish a new version with AndroidX support!

@mandybess  and @thoughtbot for the great library. I am updating  new version on fork repo  for AndroidX support. Let me know If it will create any licence issue or some other issue
Please check my fork repo https://github.com/tank777/expandable-recycler-view
Download
	allprojects {
		repositories {
			...
			maven { url 'https://jitpack.io' }
		}
	}

ExpandableRecyclerView:
implementation 'com.github.tank777.expandable-recycler-view:expandablerecyclerview:1.6@aar'

ExpandableCheckRecyclerView:
implementation 'com.github.tank777.expandable-recycler-view:expandablecheckrecyclerview:1.6@aar'

is this PR still open? What version did this go into?

This PR is still open. I've been marking PRs as approved so we can fast-track those through and cut a new release.

Why are these PRs sitting around?

Closing this as this library is deprecated: https://github.com/thoughtbot/expandable-recycler-view/pull/235


cv::BFMatcher::match crashes graphics driver

I found a problem in the GPU brute force matcher. When a certain amount of points/data is used, the program crashes the graphics driver with the following exception:
OpenCV Error: Assertion failed (clEnqueueReadBuffer(q, (cl_mem)u->handle, CL_TRU
E, 0, u->size, alignedPtr.getAlignedPtr(), 0, 0, 0) == CL_SUCCESS) in cv::ocl::O
penCLAllocator::map, file ......\opencv\sources\modules\core\src\ocl.cpp, line
4612
......\opencv\sources\modules\core\src\ocl.cpp:4612: error: (-215) clEnqueueRe
adBuffer(q, (cl_mem)u->handle, CL_TRUE, 0, u->size, alignedPtr.getAlignedPtr(),
0, 0, 0) == CL_SUCCESS in function cv::ocl::OpenCLAllocator::map
This is caused by cv::BFMatcher::match and occurs with both 3D points and descriptors of higher dimensions and might be related to issue #4876. I tested this on 4 different machines and 5 different graphics adapters (Geforce 260, 280, 480, 660 and 970, all with newest drivers). The only machine where this did not happen, was the one with the GeForce 480. The crash always occurs, if a certain amount of points is used. However, all cards have enough memory. The following code reproduces the problem with 3D points (adjust the number of points accordingly):
unsigned int numPointsA = 100000;
unsigned int numPointsB = numPointsA;

cv::Mat pointsA((int) numPointsA, 3, CV_32FC1);
cv::Mat pointsB((int) numPointsB, 3, CV_32FC1);

for(unsigned int i = 0; i < numPointsA; i++)
{
	pointsA.ptr<float>(i)[0] = (float) (rand()%100);
	pointsA.ptr<float>(i)[1] = (float) (rand()%100);
	pointsA.ptr<float>(i)[2] = (float) (rand()%100);
}

for(unsigned int i = 0; i < numPointsB; i++)
{
	pointsB.ptr<float>(i)[0] = (float) (rand()%100);
	pointsB.ptr<float>(i)[1] = (float) (rand()%100);
	pointsB.ptr<float>(i)[2] = (float) (rand()%100);
}

cv::BFMatcher bfMatcher(cv::NORM_L2, false);
vector<cv::DMatch> matches12;

cv::UMat samplesFixed1U = pointsB.getUMat(cv::ACCESS_READ);
cv::UMat pointsMoved1U = pointsA.getUMat(cv::ACCESS_READ);
bfMatcher.match(samplesFixed1U, pointsMoved1U, matches12);

The source of this problem seems to be Window's timeout detection and recovery (TDA) feature. The OS checks, if the graphics driver is responding and restarts it after a timeout. I found out, that TDA was disabled on the machine, on which the example program worked. The bug thus is not directly related to OpenCV. However, this case should be addressed somehow.


test only - do not merge

Why:

Closes [issue link]
What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

rachmari-patch-1

Why:
Closes [issue link]
What's being changed:
Check off the following:

[ ]  I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ]  For content changes, I have reviewed the localization checklist
[ ]  For content changes, I have reviewed the Content style guide for GitHub Docs.


I can not extract *.tar.gz file

this is a problem which I encounter:
tar -zxvf conll2012_model.tar.gz
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
This problem is really confuse me, I would appreciate if someone help me.

After downloading it from the google drive it, the extraction works as expected.


npm reports high severity vulnerability with @adobe/jwt-auth

Expected Behaviour
When installing, npm should not report any high severity vulnerabilities.
Actual Behaviour
When installing, npm reports a high severity vulnerability:
Steps to Reproduce
Environment: node v19.3.0, npm v9.2.0

Run npm i @adobe/reactor-releaser.

Platform and Version
All
Logs taken while reproducing problem
# npm audit report

jsonwebtoken  <=8.5.1
Severity: high
jsonwebtoken has insecure input validation in jwt.verify function - https://github.com/advisories/GHSA-27h2-hvpr-p74q
jsonwebtoken's insecure implementation of key retrieval function could lead to Forgeable Public/Private Tokens from RSA to HMAC - https://github.com/advisories/GHSA-hjrf-2m68-5959
jsonwebtoken vulnerable to signature validation bypass due to insecure default algorithm in jwt.verify() - https://github.com/advisories/GHSA-qwph-4952-7xr6
jsonwebtoken unrestricted key type could lead to legacy keys usage  - https://github.com/advisories/GHSA-8cf7-32gw-wr33
No fix available
node_modules/jsonwebtoken
  @adobe/jwt-auth  *
  Depends on vulnerable versions of jsonwebtoken
  node_modules/@adobe/jwt-auth
    @adobe/reactor-releaser  *
    Depends on vulnerable versions of @adobe/jwt-auth
    node_modules/@adobe/reactor-releaser

4 high severity vulnerabilities

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

This issue is caused by https://github.com/adobe/jwt-auth/issues/60

@adobe export issue to Jira project PDCL as Story

#24


improve quickstart time 2:05 -> 1:15

it looks good, very readable
however, it didn't improve my build times as significantly:
pre-changes:
./scripts/quick-start.sh --recreate --local  9.67s user 8.58s system 16% cpu 1:51.32 total
./scripts/quick-start.sh --recreate --local  9.63s user 8.60s system 15% cpu 1:54.19 total
./scripts/quick-start.sh --recreate --local  9.64s user 8.49s system 16% cpu 1:51.76 total

with changes (first run built the cache):
./scripts/quick-start.sh --recreate --local  18.06s user 24.29s system 35% cpu 1:58.84 total
./scripts/quick-start.sh --recreate --local  16.09s user 19.78s system 35% cpu 1:40.75 total
./scripts/quick-start.sh --recreate --local  16.53s user 20.30s system 34% cpu 1:46.11 total
./scripts/quick-start.sh --recreate --local  16.42s user 19.69s system 36% cpu 1:40.03 total

still some improvement though


Abide Live Validation

I was working on modifying the live validation on a form to be less in your face the first time through the form. Although in some instances live validation can be handy it can be intimidating. We wanted our form to live validate errors only after the user submits the form. This way if they tab through the form before filling it out they aren't inundated with errors.
It might make sense to change live validation to 'always, after or never' instead of 'true, false' so that validation only occurs after the initial post to assist with he correction process not the initial filing.
Here's a thread on the forum http://foundation.zurb.com/forum/posts/1543-disable-abide-form-validation-on-blur-the-fields

We'll address the live-validation for this version. @Ethanhackett You're suggestion makes sense and we'll see if it's something we can implement as well. Either way....
We added this to our "known issues" section in our planning board for the next version. A PR would be valuable to the community, otherwise It will be addressed then. You can see and vote on the proposed updates here: https://trello.com/b/QBPKlQqf/foundation-sites

Thanks @rafibomb

@Ethanhackett If I understand you correctly, you mean the following behaviors to correspond with the value of live validate as follows-
"always" : behavior as it stands now. Form will validate each section per changing fields.
"after" : after submission of the form
"never" : never validates the form
Is this correct?

Yes that's what I was thinking. And maybe a fourth which would be on command? But I guess that could be never or until command?
Ethan Hackett

On Feb 24, 2015, at 6:31 PM, jeaniec<EMAIL_ADDRESS>wrote:
@Ethanhackett If I understand you correctly, you mean the following behaviors to correspond with the value of live validate as follows-
"always" : behavior as it stands now. Form will validate each section per changing fields.
"after" : after submission of the form
"never" : never validates the form
Is this correct?
—
Reply to this email directly or view it on GitHub.

Maybe "never" could be renamed to "manual," which would basically mean that you can do some sort of selector event or 'on command' sort of thing. Essentially, it won't validate any element until otherwise triggered.

I think that's excellent that would allow for some very flexible validation procedures. I think manual is a great label.
Ethan Hackett

On Feb 24, 2015, at 8:37 PM, jeaniec<EMAIL_ADDRESS>wrote:
Maybe "never" could be renamed to "manual," which would basically mean that you can do some sort of selector event or 'on command' sort of thing. Essentially, it won't validate any element until otherwise triggered.
—
Reply to this email directly or view it on GitHub.

Hi all, I've made a commit where there's an option for different levels of control when it comes to validation. As this would be a breaking change (i.e. we're holding off until Foundation 6 to make this MAINSTREAM), I've still included the previous live_validate and validate_on_blur options.
You can, however, use this option when you initialize Foundation, like so:
$(document).foundation({
abide: {
validate_on: 'manual'
}
})

etc.
If you go with 'manual,' you can trigger your custom validations in this sort of manner:
$('#myInput').on('keydown', function() {
  $(this).trigger('validate.fndtn.abide');
});


Add setting for sidebar visible

I always want to toggle sidebar and activitybar at once, I can toggle multi property at once with extension toggle. but I can't find setting about sidebar visible.

Similar to #3742

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable.
Thanks for understanding and happy coding!


Precompilation

For now, the framework is limited to parse <gray>{slot}</gray> whenever a style with this nanomessage representation is being used. This is not the most performant way, since static parts of the string are being parsed over and over again.
To reduce redundant parsing, static messages could be cached. Also, maybe self-closing tags could be parsed into "future components" that need applying values to complete. Then, static components around placeholders could be parsed once.
page <page/>/<pages/>
became
text("page ").append(placeholder(String key, Queue<String>)).append(text("/")).append(placeholder(String key, Queue<String>));

Then traverse and for each placeholder insert appropriate value.

MiniMessage is fast enough so that compilation should not be necessary


Added Interview Question

Cool example! Can you add a bit of explanation about why it works like this? This way the question would be much more useful

@ganqqwerty Thank you for the suggestion, as per your suggestion I have added an explanation for the question.

Closing this as #89 is created with all these changes


Add ability to remove expired tokens

Currently there is no way to remove expired tokens from the database.
Some options:

Add a ModelAdmin for managing preview links.
Add a dev task for removing expired tokens.

https://github.com/silverstripe-labs/silverstripe-sharedraftcontent/pull/28


Loading checkpoint shards:   0% 0/2 [00:00<?, ?it/s]^C

Can't run it on free colab T4 with # Advice for 1 Tesla T4 15GB python video_chatgpt.py --load "VideoCaptioning_cuda:0,ImageCaptioning_cuda:0,ModelscopeT2V_cuda:0",  getting this error:
Loading checkpoint shards:   0% 0/2 [00:00<?, ?it/s]^C

I also ran into this problem when running
`from modules.blip import ImageCaptioning
tool_blip2 = ImageCaptioning("cpu")
frames_captioning = tool_blip2.inference # input: f"{video_path}", output: caption (str)`
in inference.py, suspect it is a duplicate of https://github.com/huggingface/transformers/issues/22064

Sorry, this was our oversight. When we tried to reproduce this issue using the free Colab T4, it alerted us that we had used all the RAM, resulting in a session crash. Therefore, it may be necessary to use a machine with a larger cache. We have already removed this instruction from the readme.


/usr/share/oem -> /oem

CI: http://jenkins.infra.kinvolk.io:8080/job/container/job/packages_all_arches/1695/cldsv

Looks good.
Though AFAIU we should merge this PR when flatcar/mantle#423 is ready as well.

Thanks for the review. And yes, it will be merged only when all four PRs are ready. (mantle, scripts, init and bootengine)

The mount unit should take care of creating the OEM mount point folder if it does not exist but when it's not activated during boot (I think that shouldn't be the case but who knows) it would be nice to ensure that it still exists, or?
How about defining it like /boot in https://github.com/flatcar/baselayout/blob/flatcar-master/tmpfiles.d/baselayout.conf#L1?

The mount unit should take care of creating the OEM mount point folder if it does not exist but when it's not activated during boot (I think that shouldn't be the case but who knows) it would be nice to ensure that it still exists, or? How about defining it like /boot in https://github.com/flatcar/baselayout/blob/flatcar-master/tmpfiles.d/baselayout.conf#L1?

Good idea, please see https://github.com/flatcar/baselayout/pull/30


Add ONTO wallet adapter

Add ONTO wallet adapter

What is the status of the wallet's support for transactions on Solana?

Hi Man, thanks for your reply! We support the full functions of Solana Chain on the ONTO APP, and we are supporting Solana Dapp on the APP

Excuse me, Any other questions？ is it ready to merge？

Hi Man, thanks for your reply! I have updated the file you requested. Please review the code. ONTO Wallet have added solana chain, and the solana dapp is developing.  This is ONTO solana web3 JS file : https://github.com/ONTO-Data-Wallet/Solana-Web3-Provider
If there are no problems, please merge ONTO. thanks~!

What is the status of the wallet's support for transactions on Solana?

What is the status of the wallet's support for transactions on Solana?
Hi Man, Have any questions? Please merge our Pull request. Thanks

@PieroDaVinci there are several review comments that are unresolved, and there were several review comments that are shown as outdated but are still unresolved.
I can't merge the PR until the work is done. If you need additional clarification on anything, just ask.

@PieroDaVinci there are several review comments that are shown as outdated but are still unresolved (the code hasn't been changed).
I can't merge the PR until the work is done. If you need additional clarification on anything, just ask.

Thank you for your reply. I have submitted the modified code, please check it. 39b36f02381c6d0a390dab8f765857fb188d1938

@PieroDaVinci there are several review comments that are shown as outdated but are still unresolved (the code hasn't been changed).
I can't merge the PR until the work is done. If you need additional clarification on anything, just ask.

Thank you for your reply. I have submitted the modified code, please check it. 388f5af533dbefff6e834cb875b958faf61a1733

@PieroDaVinci there are several review comments that are shown as outdated but are still unresolved (the code hasn't been changed).
I can't merge the PR until the work is done. If you need additional clarification on anything, just ask.

Thank you for your reply. I have submitted the modified code, please check it. 388f5af

@jordansexton Please check it，Let me know if have any questions

@PieroDaVinci no changes have been made to this PR. Please don't request review again until all the work is done and incorporated into the PR. If the PR is not ready for review, please mark it as a draft PR.

@PieroDaVinci no changes have been made to this PR. Please don't request review again until all the work is done and incorporated into the PR. If the PR is not ready for review, please mark it as a draft PR.

Thanks for your reply. I have submitted the modified code, please check it

I've reviewed the PR 3 times now. The review comments from the first review and the second review have not been addressed.
I'm closing this PR, but you are welcome to publish your own wallet adapter outside of the repo, and if it's compatible it will work without needing to be reviewed here.


Dependencies from package.json are not loaded into .zip package

Hey guys, I am fighting with a problem and I'm not sure if It's exactly a bug.
I have the following package.json
{
  "dependencies": {
    "aws-sdk": "^2.86.0",
    "lodash": "^4.17.4",
    "joi": "^10.2.0",
    "structure": "^1.2.0"
  },
  "devDependencies": { ... }
}

My serverless.yml is like:
service: my-service

plugins:
  - serverless-webpack
  - serverless-offline

custom:
  webpackIncludeModules: true

provider:
  name: aws
  runtime: nodejs6.10

functions:
  ...

My problem is when I try to deploy my functions to AWS, the structure dependency is not recognized there, I got the error module not found, when I look to the zip generated by serverless package I can see that's really does not exists my dependency there, inside node_modules.
any  tip?

I've just found out, you need to put an include option in serverless.yml anyway.
package:
  include:
    - node_modules/**

Thanks for the feedback 👍


⚠ GitHub has degraded performance

In 5b05947, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 144 ms

Resolved: GitHub performance has improved in ea2f993 after 1 hour, 14 minutes.


ONNX conversion error

I am attempting to convert this model to ONNX using the tf2onnx tool. The tool can take a frozen graph or saved model as input and produce an ONNX file. Unfortunately, it fails on aocr, apparently to due the graph being invalid.
I am filing the issue here and not against tf2onnx because the error occurred in Tensorflow while attempting to import the graph that aocr export produced. Here is the error:
tensorflow.python.framework.errors_impl.InvalidArgumentError: Node 'cond/ExpandDims' has an _output_shapes attribute inconsistent with the GraphDef for output #0: Shapes must be equal rank, but are 1 and 0

To reproduce:
aocr export --format=frozengraph ./exported-model
python -m tf2onnx.convert --input=exported-model/frozen_graph.pb --outputs prediction:0 --inputs input_image_as_bytes:0

alternatively,
aocr export ./exported-model
python -m tf2onnx.convert --saved-model=exported-model

What you could try to do is remove the conditional statement on L117-121 of model.py: it's responsible for converting the input to the same shape no matter if it's a single image or an array of images. Removing the statement will only make the model accept an array, but on the flip side, the conversion to ONNX might work.

@mialo-tech I meet the same question. Do you fix the problem?

@mialo-tech I meet the same question. Do you fix the problem?

No


Would Docker DNS support Load Balance?

I have some question about docker network
1.Would Docker DNS support Load Balance?
2.Is Buildin Overlay Driver of Docker ready for Production?

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Output of docker version:
Client:
Version:      1.10.2
API version:  1.22
Go version:   go1.5.3
Git commit:   c3959b1
Built:        Mon Feb 22 22:37:33 2016
OS/Arch:      linux/amd64
Server:
Version:      1.10.2
API version:  1.22
Go version:   go1.5.3
Git commit:   c3959b1
Built:        Mon Feb 22 22:37:33 2016
OS/Arch:      linux/amd64
Output of docker info:
Containers: 10
Running: 2
Paused: 0
Stopped: 8
Images: 57
Server Version: 1.10.2
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 110
Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
Volume: local
Network: overlay bridge null host
Kernel Version: 3.19.0-51-generic
Operating System: Ubuntu 15.04
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 3.77 GiB
Name: leejan-eleme
ID: DGKE:K4DO:EX7B:3WTL:KIBI:6MKQ:ZW67:MISR:K7XZ:5Z4P:74OD:DWDH
WARNING: No swap limit support
Cluster store: etcd://<IP_ADDRESS>:2379/docker-cluster
Cluster advertise: <IP_ADDRESS>:4000
I have some question about docker network
1.Would Docker DNS support Load Balance?
2.Is Buildin Overlay Driver of Docker ready for Production?

To answer your questions:

The DNS currently does not do "round robin", if that's what you're asking. There's plans to add it, but also realize that "round robin" isn't the best way to do load balancing
Absolutely! No need to install other solutions or networking plugins, the built-in networking should all work out of the box for production.

/cc @mavenugo for additional thoughts on your first question

We added the multiple A Records support for 1.11 release. So, when there are multiple containers have the same --net-alias, the embedded DNS's response will container multiple A records pointing to IP-Addresses of the containers matching the same alias. Also, we randomize the list. For clients capable of handling multiple A records, they can handle the load-balancing with its own algorithm. And for clients who can lookup just the first record, the randomized list will act similar to load-balancing.
But in the long-term we will go with a service level abstraction with Virtual-IP based proper load-balancing. But that is for future releases.

Yes. Anything built-in and that is not experimental is typically ready for production use.

@thaJeztah @mavenugo this would really help me

@sqleejan pls try docker master and let us know if that works for you. I think this issue can be closed now.

let me close; but comment if you're still having this issue on 1.11 once that's released


Instructions Do Not Work for EF Core 2

I get so sick and tired of Microsoft not bothering to keep their documentation synchronized with the current environment. I am trying to configure my VS2019 environment so I can do a simple "dotnet ef migrations add..." command -- and am running into failure after failure, despite following the directions on this page (for EF Core 2). The design package was added to the csproj file, I'm in the correct directory within the package manager console...and yet the command "dotnet ef", which should list available commands, pukes up an error that the specified file cannot be found. Really? Thanx, that's very helpful for figuring out what's wrong!

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

ID: 263f21d6-6edc-7582-e0e2-fa1c9670bcc0
Version Independent ID: b3dd8c84-18b1-f98d-64f1-7897d1482692
Content: EF Core tools reference (.NET CLI) - EF Core
Content Source: entity-framework/core/miscellaneous/cli/dotnet.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: bricelam

Some further information...
This is apparently a known problem whenever you install Net Core 3 preview. It's been discussed over on github extensively, with a variety of workarounds offered.
The one I used -- because I'm not currently developing Core 3 apps, I just installed the SDK to experiment/play around with it -- is this:
dotnet tool uninstall --global dotnet-ef
dotnet tool install --global dotnet-ef --version 2.1.0
in the package manager console. The first command uninstalls your current version of the tooling -- which, if like me, you just used the defaults is apparently 2.2+, which will not work properly when Core 3 preview is installed -- while the second installs a downrev version which works. Ghu only knows what functionality you may lose by using an older version, but that's Microsoft for you these days: keep churning away and don't bother to keep developers from running head-on into walls that shouldn't be there.

Duplicate of #1588


Adding the Polish language to the installer

1. Why is this change necessary?
Because we love Shopware.
2. What does this change do, exactly?
Adds language
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-22643
Please use this issue to track the state of your pull request.

Hi @mateuszfl, thank you for your awesome PR and making Shopware better! 💙
If you ever feel like translating the updater as well, we would be very grateful! 😄


add lark_meta dunder

TBH I don't know how to test it better and write more meaningful test.
Any suggestion is more than welcome
FIXES #9

It's a good start, but make sure the meta attributes are filled in correctly. line, column, end_line, etc.

Thanks.
Sorry I forgot to mention, can you also make sure to test it with a Tree instance? (you can just add a rule to the grammar, and check its meta has attributes too)

It's odd that the tests are passing.. how can that be if I didn't merge the Lark PR yet?

It's odd that the tests are passing.. how can that be if I didn't merge the Lark PR yet?

I think this is not odd since I did not send the meta through the Tree. The token has the line etc... but the Tree still has meta empty. So I wonder how can I exercise a test for this scenario

It's odd that the tests are passing.. how can that be if I didn't merge the Lark PR yet?

maybe i got it ... writing a test

ok now it should fail. FYI on my machine it passes cause I have the parse_tree edited as you PR in lark

Looks good! I'll merge it after next Lark release.

@ernestoarbitrio The tests are still failling after releasing 1.1.4, any idea why?

@ernestoarbitrio The tests are still failling after releasing 1.1.4, any idea why?

looking into it

Now it should work


Problem with Context

I tried the constate library to produce Context.
In Suspense component, i modified the GlobalContext.
In another non-Suspense component (higher level), i use useContext to read from global context.
The result is The second component doesn't read the updated Context value which is modified from Suspense component, which results in a mismatched between server and client.
Is this the issue with Context or with this library ?

Hi @revskill10.
I'm not familiar with the constate library.
Can you please post a code example and I'll trouble-shoot this. Hooks are a recent addition to React and I've not tested them yet.

Please also an you check what version of react-dom you have installed? You'd need to make sure it's >= 16.8.1 for hooks to work.

Hey @overlookmotel , sorry, this is not an issue. It was my mistake for NOT using setValue :D
I tested and it worked with Context nicely.

Oh great. Glad you got it sorted.


Swimlanes in TFS 2017

Hi,
This page lists the following product versions:
Azure Boards | Azure DevOps Server 2019 | TFS 2018 | TFS 2017 | TFS 2015
However, I cannot find the Swimlane configuration in TFS 2017.
I do see the field Board Lane, but no setting to configure the use of swimlanes in TFS.

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

ID: f1127f1b-f0d3-172b-6bc9-5e705a195cef
Version Independent ID: 5714c6fa-14fe-b275-4257-3802d0e6b233
Content: Expedite work with swimlanes, use swimlanes to differentiate work items on your Kanban board - Azure Boards
Content Source: docs/boards/boards/expedite-work.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@jawn - Please check that you are selecting the Configuration tool from the Kanban board page. If you are on that page and aren't seeing the swimlane tab, then raise the issue on our developer community page: https://developercommunity.visualstudio.com/spaces/22/index.html. Regards.


TK103: StringIndexOutOfBoundsException

I've temporary got Chinese device, that use TK103 protocol (as I could determine)
Without GPS it sends something like login packet
(087073005534BP00HSO)

And cause a subj error
2016-11-24 13:27:11  INFO: [23F36031] connected
2016-11-24 13:27:13 DEBUG: [23F36031: 5002 < <IP_ADDRESS>] HEX:<PHONE_NUMBER>37333030353533344250303048534f29
2016-11-24 13:27:13  WARN: [23F36031] error - String index out of range: 15 - StringIndexOutOfBoundsException (... < Tk103ProtocolDecoder:116 < ExtendedObjectDecoder:58 < ...)
2016-11-24 13:27:13  INFO: [23F36031] disconnected

I've started to investigate and found strange https://github.com/tananaev/traccar/blob/master/src/org/traccar/protocol/Tk103ProtocolDecoder.java#L116
Am I missing something or there should be id instead of content.substring(0, 15) ?

This code works for me:
            if (type.equals("BP00") || type.equals("BP05")) {
                String content = sentence.substring(16);
                getDeviceSession(channel, remoteAddress, id);
                if (type.equals("BP00")) {
                    channel.write("(" + id + "AP01" + content + ")");
                    return null;
                } else if (type.equals("BP05")) {
                    channel.write("(" + id + "AP05)");
                }
            }

Fixed the problem. Some devices send full IMEI with BP00 message, so I decided that it would be good idea to capture it for identification.
From documentation:

But if we do getDeviceSession(channel, remoteAddress, id) we will get device online as soon as it send handshake. And It will be consistence with other part of parser, there device unique id is first 12 digits.

The problem is that most people don't know that they should be using "0" plus last 11 digits of their IMEI for the device id. So, I implemented this as a fallback option.

May be if it is fallback anyway we can use:
                if (content.length() >= 15) {
                    getDeviceSession(channel, remoteAddress, content.substring(0, 15));
                } else {
                    getDeviceSession(channel, remoteAddress, id);
                }

There is no need for id there... it's only a heartbeat message. For real location data, if device is not identified yet, it will use 12-digit id anyway.


[SPARK-11497] [MLlib] [Python] PySpark RowMatrix Constructor Has Type Erasure Issue

As noted in PR #9441, implementing tallSkinnyQR uncovered a bug with our PySpark RowMatrix constructor.  As discussed on the dev list here, there appears to be an issue with type erasure with RDDs coming from Java, and by extension from PySpark.  Although we are attempting to construct a RowMatrix from an RDD[Vector] in PythonMLlibAPI, the Vector type is erased, resulting in an RDD[Object].  Thus, when calling Scala's tallSkinnyQR from PySpark, we get a Java ClassCastException in which an Object cannot be cast to a Spark Vector.  As noted in the aforementioned dev list thread, this issue was also encountered with DecisionTrees, and the fix involved an explicit retag of the RDD with a Vector type.  IndexedRowMatrix and CoordinateMatrix do not appear to have this issue likely due to their related helper functions in PythonMLlibAPI creating the RDDs explicitly from DataFrames with pattern matching, thus preserving the types.
This PR currently contains that retagging fix applied to the createRowMatrix helper function in PythonMLlibAPI.  This PR blocks #9441, so once this is merged, the other can be rebased.
cc @holdenk

Merged build triggered.

Merged build started.

Test build #44992 has started for PR 9458 at commit c1258a6.

Test build #44992 has finished for PR 9458 at commit c1258a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44992/
Test PASSed.

@mengxr This PR is ready for review and discussion.

Note: The following reproduces this issue on the latest Git head, 1.5.1, and 1.5.0:
from pyspark.mllib.linalg.distributed import RowMatrix
rows = sc.parallelize([[3, -6], [4, -8], [0, 1]])
mat = RowMatrix(rows)
mat._java_matrix_wrapper.call("tallSkinnyQR", True)

Should result in the following exception:
java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to [Lorg.apache.spark.mllib.linalg.Vector;

Sorry for the long delay!  I looked through the discussions, and this LGTM.  I'll run tests once before merging it.

Test build #2207 has started for PR 9458 at commit c1258a6.

Test build #2207 has finished for PR 9458 at commit c1258a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged with master, branch-1.6 and branch-1.5
This might not make 1.6.0, but we'll see.

Great, thanks @jkbradley!


Implement distinct on functionality

This template isn't a strict requirement to open issues, but please try to provide as much information as possible.
Version: Latest
Module: quill-sql
Database: all
Wanted behaviour
Wanted behaviour is described in this issue but it seems to have been dead for over 2 years and this is a rather important feature to have IMO.
https://github.com/getquill/quill/issues/1540
@getquill/maintainers

Possible for an update on this?


Fix trivia to capture both leading and trailing trivia.

Description:
Currently a token/lexeme has only leading trivia, and the trailing trivia is always empty.
Improve this to have both leading and trailing trivia. The separation between two trivia can be a newline/etc.

Fixed with #22205


n_estimators warning

params = {'objective': 'rank:ndcg', 'colsample_bytree': 0.5,
 'eta': 0.325,
 'gamma': 5,
 'max_depth': 8,
 'tree_method': 'exact',
 'min_child_weight': 6.0,
 'n_estimators': 100,
 'subsample': 0.8,
 }

xgb_model = xgb.train(params, train_dmatrix, num_boost_round = 100,
            evals=[(train_dmatrix, 'training'),(valid_dmatrix, 'validation')])

[14:38:45] WARNING: /Users/travis/build/dmlc/xgboost/src/learner.cc:516:
Parameters: { n_estimators } might not be used.
This may not be accurate due to some parameters are only used in language bindings but passed down to XGBoost core.  Or some parameters are not used but slip through this verification. Please open an issue if you find the above cases.
I am getting the above warning when I run the above train method.

n_estimator is part of the sklearn interface, not in xgb.train.   The sklearn interface do some configurations to change n_estimator into num_boost_round before training.


No curl response from pod service

Following the guide in k8s-on-raspbian/GUIDE.md.
When I run:
pi@raspberrypi2:~ $ curl -4 http://<IP_ADDRESS>:31118 -d "test"

The HTTP request is sent but it is forever awaiting a response.
Environment

Docker version 19.03.6
Kubernetes v1.17.3
Operating System: Raspbian 10 (buster)
Board: PI4

All pods are running
pi@raspberrypi2:~ $ kubectl get pods --all-namespaces
NAMESPACE     NAME                                   READY   STATUS    RESTARTS   AGE
default       markdownrender-f9744b577-gsmk9         1/1     Running   0          14m
kube-system   coredns-6955765f44-hsbhx               1/1     Running   0          23m
kube-system   coredns-6955765f44-zzf9w               1/1     Running   0          23m
kube-system   etcd-raspberrypi2                      1/1     Running   0          24m
kube-system   kube-apiserver-raspberrypi2            1/1     Running   0          24m
kube-system   kube-controller-manager-raspberrypi2   1/1     Running   0          24m
kube-system   kube-proxy-9zx54                       1/1     Running   0          11m
kube-system   kube-proxy-st75l                       1/1     Running   0          23m
kube-system   kube-scheduler-raspberrypi2            1/1     Running   0          24m
kube-system   weave-net-qhmps                        2/2     Running   2          11m
kube-system   weave-net-vhq6j                        2/2     Running   1          16m

Node (raspberrypi3) successfully joins
pi@raspberrypi2:~ $ kubectl get nodes
NAME           STATUS   ROLES    AGE   VERSION
raspberrypi2   Ready    master   26m   v1.17.3
raspberrypi3   Ready    <none>   13m   v1.17.3

I'm using the right port:
pi@raspberrypi2:~ $ kubectl get service  markdownrender
NAME             TYPE       CLUSTER-IP      EXTERNAL-IP   PORT(S)          AGE
markdownrender   NodePort   <IP_ADDRESS>   <none>        8080:31118/TCP   17m

Routes show the 31118 port open
pi@raspberrypi2:~ $ netstat -tulpn
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:10248         <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:10249         <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:2379       <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:2379          <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:2380       <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:2381          <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:45009         <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:10257         <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:10259         <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:6784          <IP_ADDRESS>:*               LISTEN
tcp6       0      0 :::10250                :::*                    LISTEN
tcp6       0      0 :::10251                :::*                    LISTEN
tcp6       0      0 :::6443                 :::*                    LISTEN
tcp6       0      0 :::10252                :::*                    LISTEN
tcp6       2      0 :::31118                :::*                    LISTEN
tcp6       0      0 :::10256                :::*                    LISTEN
tcp6       0      0 :::22                   :::*                    LISTEN
tcp6       0      0 :::6781                 :::*                    LISTEN
tcp6       0      0 :::6782                 :::*                    LISTEN
tcp6       0      0 :::6783                 :::*                    LISTEN

Comments
I believe I am missing something very simple. I already tried reset and init and the problem persists.

Just figured it out. You run the curl command from one of the nodes. If you did the same as me you ran it on master.

This is just a thoery, but I believe the issue here is that there's no external ip address associated with the service:
pi@raspberrypi2:~ $ kubectl get service  markdownrender
NAME             TYPE       CLUSTER-IP      EXTERNAL-IP   PORT(S)          AGE
markdownrender   NodePort   <IP_ADDRESS>   <none>        8080:31118/TCP   17m

no external ip, only the cluster ip. You can access it only from the host it's scheduled on.

It has to do with the iptables of the pis that are preventing them from
communicating. They have to be using leagcy binaries these three commands
solved my problem:
sudo iptables -t nat -A POSTROUTING -o wlan0 -j MASQUERADE
sudo iptables -A FORWARD -i wlan0 -o eth0 -m state --state
RELATED,ESTABLISHED -j ACCEPT
sudo iptables -A FORWARD -i eth0 -o wlan0 -j ACCEPT
More on it here ->
https://www.shogan.co.uk/kubernetes/building-a-raspberry-pi-kubernetes-cluster-part-1-routing/
On Wed, Mar 4, 2020, 12:40 AM Justin Abrahms<EMAIL_ADDRESS>wrote:

This is just a thoery, but I believe the issue here is that there's no
external ip address associated with the service:
pi@raspberrypi2:~ $ kubectl get service  markdownrender
NAME             TYPE       CLUSTER-IP      EXTERNAL-IP   PORT(S)          AGE
markdownrender   NodePort   <IP_ADDRESS>           8080:31118/TCP   17m
no external ip, only the cluster ip. You can access it only from the host
it's scheduled on.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/teamserverless/k8s-on-raspbian/issues/24?email_source=notifications&email_token=AKQPGDTYPNOQM22GJZP2YQTRFXZXJA5CNFSM4K2WZLF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENWR4EQ#issuecomment-594353682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQPGDVAGLFJQJWRMY4WLTDRFXZXJANCNFSM4K2WZLFQ
.

That website seems to suggest that those ip table rules are to allow the worker nodes to access the public internet via the primary node's internet connection. This was due to their odd networking configuration where the primary node served as a router for the worker nodes.

You are correct @justinabrahms but they should all be on the sames network in kubernetes so any node should be able to hit a cluster ip which wasn't happening here.
They should all be using one point of contact to the outside world through the master node. Doing this wont create a public ip but will make sure the master is holding that access point should you create one eventually.
From my understanding that is what kubernetes should be doing.

Wait... Those may be the wrong commands though.. One moment.

I'm sorry you are right that was the wrong commands. you want these:
sudo update-alternatives --set iptables /usr/sbin/iptables-legacy
sudo update-alternatives --set ip6tables /usr/sbin/ip6tables-legacy
sudo update-alternatives --set ebtables /usr/sbin/ebtables-legacy
Links:
[0] "Kubernetes Raspberry Pi 4 Cluster" - https://github.com/codesqueak/k18srpi4

@DwayneGit Thanks for your response.  It seems that setting the iptables to legacy is critical  for kubernetes with iptables-1.8.x.
Can someone please update the guide on  k8s-on-raspbian/GUIDE.md to include this?


[FLINK-21090][tests] Add IT case for stop-with-savepoint and final checkpoint (1.14)

This PR picks https://github.com/apache/flink/pull/16773 to release-1.14

@flinkbot run azure

Merged already.


TypeError: To be compatible with tf.eager.defun, Python functions must return zero or more Tensors; in compilation of <function _tf_if_stmt.<locals>.aug_body at 0x7f3d989cfb00>, found return value of type <class 'function'>, which is not a Tensor.

<ipython-input-16-2f54f413d530>:134 call  *
    decoder_outputs = self.odeblock(encoder_inputs)
<ipython-input-16-2f54f413d530>:75 call  *
    outputs = odeint(self.odefunc, x, integration_time)
/usr/local/lib/python3.7/dist-packages/tfdiffeq/odeint.py:68 odeint  *
    tensor_input, func, y0, t = _check_inputs(func, y0, t)
/usr/local/lib/python3.7/dist-packages/tfdiffeq/misc.py:317 _check_inputs  *
    if _decreasing(t):
/usr/local/lib/python3.7/dist-packages/tensorflow/python/autograph/operators/control_flow.py:1163 if_stmt
    _tf_if_stmt(cond, body, orelse, get_state, set_state, symbol_names, nouts)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/autograph/operators/control_flow.py:1210 _tf_if_stmt
    cond, aug_body, aug_orelse, strict=True)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/dispatch.py:201 wrapper
    return target(*args, **kwargs)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/deprecation.py:538 new_func
    return func(*args, **kwargs)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/ops/control_flow_ops.py:1180 cond
    return cond_v2.cond_v2(pred, true_fn, false_fn, name)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/ops/cond_v2.py:89 cond_v2
    op_return_value=pred)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/func_graph.py:995 func_graph_from_py_func
    expand_composites=True)
/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/nest.py:659 map_structure
    structure[0], [func(*x) for x in entries],
/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/nest.py:659 <listcomp>
    structure[0], [func(*x) for x in entries],
/usr/local/lib/python3.7/dist-packages/tensorflow/python/framework/func_graph.py:952 convert
    (str(python_func), type(x)))

TypeError: To be compatible with tf.eager.defun, Python functions must return zero or more Tensors; in compilation of <function _tf_if_stmt.<locals>.aug_body at 0x7f3d989cfb00>, found return value of type <class 'function'>, which is not a Tensor.

Hi,
Gentle Reminder!
Could you please check this

The error mentions to return a tensor, not a function. Can you check your return statement to make you return one or more tensors ?

actually , the error is because of the function : _check_inputs
/usr/local/lib/python3.7/dist-packages/tfdiffeq/odeint.py:68 odeint  *
tensor_input, func, y0, t = _check_inputs(func, y0, t)

Hi,
Gentle Reminder!


I'm not sure how to use this with homemanager?

I'm far from a nix pro, but I wanted to add this to my (flaked) homemanager config so I can get all the nice benefits.
Unfortunately, I can't figure out how to do this.
My flake:
{
  description = "Home Manager configuration of Spott";

  inputs = {
    nixpkgs.url = "github:nixos/nixpkgs/nixpkgs-unstable";

    home-manager = {
      url = "github:nix-community/home-manager/";
      inputs.nixpkgs.follows = "nixpkgs";
    };

    nix-vscode-extensions.url = "github:nix-community/nix-vscode-extensions";
    /*
       flake-utils = {
      url = "github:numtide/flake-utils";
    };
    */
  };

  outputs = {
    nixpkgs,
    home-manager,
    nix-vscode-extensions,
    ...
  }:
  let
    extensions = sys:
      nix-vscode-extensions.extensions.${sys}; # I can't figure out how to put this somewhere I can pass to the modules below
    overlays = sys: [
        (self: super: {
          python = super.python310;
        })
# is there some way I can pass this here, so I can use it with something like pkgs.nix-vscode-extensions?
      ];

    pkgs = sys:
      import nixpkgs {
        system = sys;
        config = {allowUnfree = true;};
        overlays = overlays sys;
      };
  in {
    packages = {
      aarch64-darwin.homeConfigurations = {
<EMAIL_ADDRESS>= home-manager.lib.homeManagerConfiguration {
          pkgs = pkgs "aarch64-darwin";
          modules = [
            ./normandy.nix
            ./common.nix # I want common.nix to have access to the extensions object, but I'm not sure how to do that.
            ./darwin-common.nix
            ./zsh/zsh.nix
          ];
        };
<EMAIL_ADDRESS>= home-manager.lib.homeManagerConfiguration {
          pkgs = pkgs "aarch64-darwin";
          modules = [
            ./endeavor.nix
            ./common.nix
            ./darwin-common.nix
            ./zsh/zsh.nix
          ];
        };
      };
      x86_64-linux.homeConfigurations = {
        "spott@devbox" = home-manager.lib.homeManagerConfiguration {
          pkgs = pkgs "x86_64-linux";
          modules = [
            ./devbox.nix
            ./common.nix
            ./zsh/zsh.nix
          ];
        };
      };
    };
  };
}

I'd appreciate any help, not sure if this is the right place to post this, if not, point me in the right direction.

On way to do it is to use this flake's overlay. You can modify your  overlays to be:
overlays = sys: [
    # Your Original Overlay
    (self: super: {
        python = super.python310;
    })
    # Nix VSCode Extensions Overlay
    nix-vscode-extensions.overlays.default
];

Notice that you don't need to use the system attribute with overlays. You can safely convert overlays to a list instead of a function taking sys as input:
overlays = [
    # Your Original Overlay
    (self: super: {
        python = super.python310;
    })
    # Nix VSCode Extensions Overlay
    nix-vscode-extensions.overlays.default
];

And then you would use it like this:
pkgs = system: import nixpkgs {
    inherit system overlays;
    config = {allowUnfree = true;};
};

Now you can use pkgs.vscode-marketplace.PUBLISHER.NAME to use extensions from within your home-manager config.

Thanks a ton!  That definitely makes sense.


Remove inline styling from js to accommodate standard CSP policies

This line of code violates standard CSP policies due to the inline style being set:
this.passedElement.setAttribute('style', 'display:none;');

What developers can do instead is specify the hiddenState class with one that has a display: none; rule when creating a Choices input. As there is an operation that occurs a few lines before:
// Hide passed input
this.passedElement.classList.add(
   this.config.classNames.input,
   this.config.classNames.hiddenState,
);

Is there a case that requires this styling to be set here that I'm failing to consider?

Link to code - choices.js:2742
Content Security Policy - Unsafe inline styles

Hey - thanks for the info. Would you be able to submit a PR for this off the develop branch at all? 👍

@jshjohnson can we have a release with this change please? It is currently only on the develop branch.


No syntax Highlighted in my liquid file

Hello i have one block that contains a script tag
<script>
// javascript code
</script>

{% schema %}
{
  "name": "BrowserBell",
  "target": "body"
}
{% endschema %}

the Javascript code doesn't have any syntax highlighting and no auto complete in js code , am I missing something ?

sorry it works now


failed update submodule when change submodule url

recently I change repo URL then failed to deploy.
I found out that repository config keeps containing old submodule URL.
I think we need sync before command submodule update

Reopen it is not fixed yet.

I encountered the same bug and the patch worked

The task does

reset
submodule sync
submodule update
...

In my case, resetwas failing because the stored submodule url contained an expired Gitlab $CI_BUILD_TOKEN (inherited from the main :repo_url, set to $CI_REPOSITORY_URL). Somehow, if the reset fails, then the subsequent sync doesn't update the URLs of the submodules, and thus the update fails. Same thing can happen if one changes the repo URL (like OP), I imagine.
I've worked around it by forcing a submodule sync before the git:submodules:create_release, as below. It does solve my case by propagating the new token, but it's not very hard to come up with scenarios in which this would not offer any relief.
# Capfile

before 'git:submodules:create_release', "git:submodules:pre_reset_sync"

namespace :git do
  namespace :submodules do
    desc 'Run git submodules sync to make sure the URLs are correct for the reset that happens in git:submodules:create_release'
    task :pre_reset_sync do
      on release_roles :all do
        with fetch(:git_environmental_variables).merge(
          'GIT_DIR' => repo_path.to_s,
          'GIT_WORK_TREE' => release_path.to_s,
        ) do
          within release_path do
            execute :git, :submodule, 'sync', '--recursive', '--quiet'
          end
        end
      end
    end
  end
end


Replaced deprecated HttpException for types.ConnectionDeny on onConnect (code examples)

Current coverage is 57.38%

Merging #628 into master will not affect coverage as of 511a484

@@            master    #628   diff @@
======================================
  Files           62      62
  Stmts        10244   10244
  Branches      1664    1664
  Methods          0       0
======================================
  Hit           5879    5879
  Partial        555     555
  Missed        3810    3810

Review entire Coverage Diff as of 511a484

Powered by Codecov. Updated on successful CI builds.


Twich vod. Choose format problem

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
Hello. How can I choose twitch vod quality when downloading ? I can only see the list  but not an option how to choose one. If somebody know the command please let me know the how to download the 720p 60 fps one. Thanks

Use -f ...

-f anddd ? I want to choose the 720p60 option how to do it ?

@magorko read the manual.

@89z I already did but on youtube it put a number next to format. But if its twitch vod it doesnt. So I cant choose one.

Why do you assume the format must be a number?

thats how its works

Youve been presented format codes that arent numbers, so why would you still assume that format code must be numbers?

Just please answer what the command should look like if I want to download 720p60 . Thats all

Indeed, -f 720p60 is what's needed.
Format codes (format_id) are generally set by the extractor, often from a value used by the site, as with YT above, or if not, or if duplicated, are generated by yt-dl itself when processing the extracted data (info_dict).

Sooo just: youtube-dl.exe -f 720p60 https://www.twitch.tv/videos/1297056505 ?

Sure.

And can i download only dedicated period of the VOD? If so whats the
command ?
Dňa po 14. 2. 2022 o 2:05 dirkf @.***> napísala(a):

Sure.
—
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/30639#issuecomment-1038515028,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXWCBI5UHP7MIRL2NBYNT7LU3BIOPANCNFSM5OJ4KACA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Possibly use -g --get-filename to get the download link (url) and output filename and then ffmpeg -ss nn -i url filename, where nn is the desired start time in the stream (736 or 12:16 or 0:12:16 to start 736s in); eg
ffmpeg -ss 736 -i "https://d2vjef5jvl6bfs.cloudfront.net/76b7152560853baa86ef_resttpowered_45515852941_1644774470/720p60/index-dvr.m3u8" "UFC NIGHTTTTTTTTTTTTTTTTTTT-v1297056505.mp4"

Add options -hide_banner -v 24 after ffmpeg to make the output quieter.

Just doing --external-downloader ffmpeg --external-downloader-args "-ss 736" should work

I can't recall where in the ffmpeg command line yt-dl puts the options from --external-downloader-args. It needs to be before -i to avoid saving and then skipping the 736, or whatever, seconds, although I agree that your suggested options should work too.

I've looked through the README

Are you sure? There's a whole section about it.


multiple relations (one to many) are not supported

Currently, relations seem to only work for Single relations. rather than Multiple relations, as the relation is always singular instead of being an array for Multiple relations.
In other words, instead of being SomeResponse[], the codegen generates SomeResponse.

fixed in https://github.com/david-plugge/typed-pocketbase/commit/19bce54b2162757534cd9a436f56c0d36959ab4f (version 0.0.5)

use 0.0.6 instead, the actual record types were still wrong in 0.0.5


构建 YOLOV6 的 TensorRT 文件时出了问题，求大佬帮忙

Error Code 1: Cudnn (CUDNN_STATUS_EXECUTION_FAILED)
Traceback (most recent call last):
File "./tools/export.py", line 288, in
main(args)
File "./tools/export.py", line 249, in main
builder.create_engine(args.engine, args.precision, args.calib_input, args.calib_cache, args.calib_num_images,
File "./tools/export.py", line 242, in create_engine
with self.builder.build_engine(self.network, self.config) as engine, open(engine_path, "wb") as f:
AttributeError: enter

@sha-mao 检查一下CUDA CUDNN版本是否匹配