Uncaught TypeError: Cannot set properties of undefined (setting 'format')

in case of using this module with VITE
react-dom.development.js:26591 Uncaught TypeError: Cannot set properties of undefined (setting 'format')
at F (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:415:32)
at Object.l (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:432:46)
at Object. (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:433:62)
at n (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:28:24)
at Module. (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:440:9)
at n (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:28:24)
at :3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:55:22
at :3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:56:8
at :3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:20:82
at node_modules/react-moment/dist/index.js (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:21:6)
at __require2 (:3000/node_modules/.vite/chunk-VNVKLEIO.js?v=b732fa8d:45:44)
at node_modules/react-live-clock/lib/Component.js (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:2034:24)
at __require2 (:3000/node_modules/.vite/chunk-VNVKLEIO.js?v=b732fa8d:45:44)
at node_modules/react-live-clock/lib/index.js (:3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:2146:26)
at __require2 (:3000/node_modules/.vite/chunk-VNVKLEIO.js?v=b732fa8d:45:44)
at :3000/node_modules/.vite/react-live-clock.js?v=b732fa8d:2152:32

I have never used VITE. Could you please produce a minimal example (for example on code-pen)?

it's been more than a week, I am closing this due to a lack of response.

Hello @dgastudio, not sure if you found a solve. I was having this same issues, it's a issue with vite (using version 2.9.2) and momentjs (version 2.29.4) the fix I found is adding:
import { viteCommonjs, esbuildCommonjs } from '@originjs/vite-plugin-commonjs';

and
    esbuildOptions: {
      plugins: [esbuildCommonjs(['react-moment'])],
    },

to your vite.config.js file:
import {defineConfig} from 'vite';
import hydrogen from '@shopify/hydrogen/plugin';
import { viteCommonjs, esbuildCommonjs } from '@originjs/vite-plugin-commonjs';

export default defineConfig({
  plugins: [hydrogen()],
  optimizeDeps: {
    include: ['@headlessui/react'],
    esbuildOptions: {
      plugins: [esbuildCommonjs(['react-moment'])],
    },
  },
});

@spacecamp, Thank you for the feedback. I am positive this will help someone in the future.

Hi @anthony0030  and @spacecamp ! The future soy yo!
I just had this issue. I am useing Vite.
Here is the really worling vite.config.ts for a simple React:
`import { defineConfig } from 'vite'
import react from '@vitejs/plugin-react'
import { esbuildCommonjs } from '@originjs/vite-plugin-commonjs';
export default defineConfig({
plugins: [react()],
optimizeDeps: {
esbuildOptions: {
plugins: [esbuildCommonjs(['react-moment'])],
},
},
})
Many thanks!`

Hi @rogeriocassares, I don't have any knowledge of Vite. I really wish I could help. Do you have any suggestions on how we can get this bug squashed?


ENH: add scipy.linalg.convolution_matrix

Reference issue
See #7834
What does this implement/fix?
This implements the convolution_matrix function. It is similar to the Matlab/Octave convmtx function but not derived from their implementations in any way.
There is an additional "mode" argument analogous to that in numpy.convolve.

@ilayn  , @tylerjereddy
I believe all changes have been done except we have not resolved the discussion above about using itertools.product in a single fast test vs. nesting parameterize decorators to make multiple tests.
I tend to favor the itertools approach as I  find it clearer to read and maintain the code, but I don't feel strongly about it.  I defer to your judgement.

bump

Hi @mborgerding instead of range(2,7) for both test cases please use [10, 100] such that at least two order of magnitudes tested. There is not much difference between 3 and 4 in terms of testing behavior. Some problems only reveal themselves when the values are large or arrays size are large.

Hi @mborgerding instead of range(2,7) for both test cases please use [10, 100] such that at least two order of magnitudes tested. There is not much difference between 3 and 4 in terms of testing behavior. Some problems only reveal themselves when the values are large or arrays size are large.

I expanded the range, but rather than [10,100],  it tests the lowest meaningful number for convolution (2) and at least one odd number (9), and 100.

You can install pycodestyle and use that for trivial PEP8 checks or adjust your editor such that it does it for you.

I appreciate the tip, but the default pycodestyle options did not seem to catch some of the commas.
Apparently some of the checks are turned off by default.
I was able to turn on full pedantic mode with "--ignore ''"

Is there anything more to be done for this PR?

If the CI is green this would be ready for 1.4.0. Please review if you would have time. I'll add the label prospectively.

@mborgerding
Just for the future reference, please make separate branches to work on new features and to send PRs such that your standard working repo does not interfere with the PRs you have submitted. Once a PR is merged you can safely delete that branch and keep working on other branches.

@rgommers Have you observed this Python 3.8/Windows Azure timeout issue before?

The ppc64le failure probably isn't related

There are no holes in the line coverage, which is good.

@rgommers Have you observed this Python 3.8/Windows Azure timeout issue before?

no I haven't

I'm bumping the milestone--we've already had more last-minute merges than I'd like & the CI is a little strange here, but this looks close to merging after we branch.

The failures are not related to this PR.

Do you still have time and interest for working on it? Let us know. If not, we can take it over.

@WarrenWeckesser Are you going to have time to take over here before we branch?

@tylerjereddy, thanks for the reminder.  Yes, I'll take this over.

Update pull request is in https://github.com/scipy/scipy/pull/12135.

This PR was continued in gh-12135, and that PR has been merged, so closing.
Thanks @mborgerding for the contribution, and thanks @ilayn and @tylerjereddy for the reviews.


3.x - remove testing/latex_envs

If my previous PR for usability/latex_envs (master) is accepted, I propose to remove testing/latex_envs (3.x).

Great. Thank you.


Duniverse v15.0.0

Maybe there are some better ways to port this, that don't require calling topkg by the packager?

Do you mean checking in the code generated by running this package's build command? What problems are caused by doing this?

The main problem is that the dune port is "further away" from the source tarball. Which is not a problem per-se but usually the closer it is to the original source (e.g. no additional commands that need to be remembered to run) the easier it is to verify it is correct and the simpler it is to port it to the next version.
So sure, this is a valid way and we for sure don't want it to download random files from the internet at build time. I took a look at how this is generated upstream and it makes vaguely unhappy that it downloads and unpacks a random ZIP file, so the approach in this PR might indeed the best possible.

Would it be better to check in the raw xml file (the contents of the zip that gets downloaded) and have dune rules process the xml file into the generated ml files? That way we still avoid downloading a file as part of the build but the result is closer in some sense to the upstream source?
The problem with checking in the xml is that the process of porting the upstream package would still involve running the topkg command to download the file (or a custom curl/wget command I suppose), and if the porting process requires running a custom command and checking in the result anyway then I see no harm in running the custom command that generates the ml files instead (as we currently do) and that has the benefit of simplifying the dune files.

For things like that I would recommend checking in the downloaded source (it's probably in the release tarball no?). If you were upstream you could add a refresh rule with mode promote and alias refresh, so that dune build @refresh would redownload it, but I don't think we're going to update the data files independently of upstream releases.

@Leonidas-from-XIV does anything more need to happen here or is this good to merge?


test: support testing on Windows

package.json_s: use compatible quoting
use posix paths in createTestPackage
convert \r\n to \n before comparing strings

correct naming of snapshot strings; deduplicate write logic

resolve file relative paths with fileURLToPath

This is great, thank you so much! Would you be up for adding a Windows runner to the CI workflow before merging?

I don't know what that is but I Will take a look. Can add another PR after if I don't figure it out straightaway.

This example shows something like what I have in mind: https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/running-variations-of-jobs-in-a-workflow#using-a-matrix-strategy


Keep expanding home in CRYSTAL_CACHE_DIR

Fixes a small regression from #7903
Somebody with more time than me to spend on this might want to go through and evaluate all File.expand_path uses in the compiler  for whether it's safe and beneficial to expand home.

Why do you think this should be expanded? I think it would be unusual to do.

Well I noticed it because I've set mine to CRYSTAL_CACHE_DIR=~/.crystal

I would expect that Dir.mkdir_p, Dir.exists? and others would resolve ~ and not requiring the user to expand it. Am I wrong? Currently, a directory named ~ is created in the current location.

I would expect that Dir.mkdir_p, Dir.exists? and others would resolve ~ and not requiring the user to expand it. Am I wrong?

At least Ruby doesn't expand ~ in those calls.
I believe Ruby and other language never expand ~ unless explicitly asked (in Ruby you have to do File.expand_path). I think we should do the same in Crystal.
The problem is that we recently changed File.expand_path to not expand ~ by default because it's considered dangerous.

And that was a good change! Expansion of  ~  should  always  be  explicit.  We just have to revisit in which places inside the compiler it makes sense to do it. I  believe this is one of them. (CRYSTAL_PATH  might be another obvious candidate).

@bcardiff If we would expand home in all these methods, there would have been no point in disabling it by default in Path#expand. Also they couldn't handle paths with a literal $ which is a perfectly valid directory name in most file systems.
We just need to explicitly expand paths where necessary, for example when they come from environment variables or similarly trusted user input. For command line applications, probably every user input should be considered.

Ok. I thought the intention was to manipulate path while preserving ~, but once we actually use the string to access a path the ~ should be solved always.

I'm pretty sure that very few tools expand *_PATH variables like that. To me this is just messing with the user's intention and has the potential of great confusion.
Namely, there's no way to specify a directory that starts with ~. Not only that, paths that happen to be that (in some dynamic context) will break. Sure, this will never reasonably happen. But security issues don't have to be reasonable.
@jhass's justification is that he relies on this behavior. But worth pointing out that
CRYSTAL_CACHE_DIR=~/.crystal will actually be expanded by the shell and be passed as a full path to Crystal.
The other part of the justification is that it used to be expanded. But that's circular reasoning; it simply used to be wrong.

I would expect that Dir.mkdir_p, Dir.exists? and others would resolve ~ and not requiring the user to expand it. Am I wrong?

Yes, sorry.

What's the harm in doing it?

All the same harm that #7903 is trying to prevent in the first place.

But worth pointing out that CRYSTAL_CACHE_DIR=~/.crystal will actually be expanded by the shell and be passed as a full path to Crystal.

That's a fair point. It's sometimes hard to separate what kind of behaviour come from the shell and what from the actual program.
Maybe it's best to leave this as is and expect home paths to be fully expanded by the shell. I'm not sure. Might be worth looking closer at how other applications behave.
@jhass What's your specific use case? How do you configure the environment variable without the path being expanded by the shell?

As an example, CRYSTAL_CACHE_DIR="~/.crystal" would not be expanded.
Then you could argue that it's nice for Crystal to guess the intent, but it's not a safe guess.
Users deal with this mistake for all other tools (subsequently learning something about the shell) and I'm sure they can deal with it for Crystal.

Well looks like the majority disagrees here, fair.


Connection starting on GoViral should see "Data connection complete" page

Current incorrect behavior: a member that starts an Open Humans connection on the GoViral site (i.e. without the "origin" parameter) is returned to the Open Humans connection page on GoViral.
A member that's started a connection on GoViral (or PGP or American Gut) should see the "Data connection complete" page, as described in the mock-ups. (This is step 3a in the "New flow" on the overview.)
Notes:

this panel should have a warning message (as mocked up) for users that haven't verified their email, but shouldn't display it otherwise
"Continue to Open Humans" should go to the "Welcome" page for now
"Return to GoViral" can go to the GoViral main page

Needs to use the URL+parameter view created by #224

Fixed with the following PR, pending merge: https://github.com/PersonalGenomesOrg/goviral-integration/pull/12


Update .gitignore to new dart_tool pub cache

dart-lang/sdk#32030

Thanks, @bcko !

Thank you for your help @kevmoo ! :)


Website is out?

Hello! Seems the domain of the site deosn't exist anymore, or isn't linked to anything! I cannot acess any part of the website nor link any image hosted on it! Is it just me?

The VPS provider had a technical error which corrupted the hard drives (which were in RAID), the server is being rebuilt from backups. The wiki and forums will be back when this is complete.

Thanks for answering :D
On Thu, May 2, 2019 at 1:30 AM rubenwardy<EMAIL_ADDRESS>wrote:

Closed #169 https://github.com/minetest/minetest.github.io/issues/169.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/minetest/minetest.github.io/issues/169#event-2313191738,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEG5URTUJVGYZR67NZKSHVLPTIRZTANCNFSM4HJK6Y5A
.


Add a delay for page-load in e2e tests

@srawlins this is a hacky fix for #2360 until we can figure out what's going on here...

This might be a webdriver issue: https://github.com/google/webdriver.dart/issues/253


fix(ci): Inline parser tests.

swc_ecma_parser:

Inline parser tests.

+4,538,299 −2
This is too much. Closing...


cURL command in API section is not accurate

Bug Description
the example's output_type value should be quoted.

Reproduction
setup a flow , and try to call it.
Expected behavior
copy the command and paste it should be OK to run.
Who can help?
No response
Operating System
Docker image v1.0.15
Langflow Version
v1.0.15
Python Version
None
Screenshot
No response
Flow File
No response

Issue was recently fixed by https://github.com/langflow-ai/langflow/pull/3337

#3337


fix powershell -version and help

PowerShell -v behavior updated to align with other tools like git, curl, and bash where args after -v are silently ignored.
Built-in help updated to reflect changes we've made to the console host in PSCore6 removing unsupported parameters.
Doc update aligned with these changes (and general cleanup of parameters we don't support in PSCore6) https://github.com/PowerShell/PowerShell-Docs/pull/1707

The same test didn't fail in AppVeyor. In AppVeyor, the failed test is Version should write an error if a value is present, see https://ci.appveyor.com/project/PowerShell/powershell/build/6.0.0-beta.7-5642#L384
On Travis, a different test failed: Version should ignore other parameters. See https://travis-ci.org/PowerShell/PowerShell/jobs/281514325#L1461
So probably we shouldn't just remove that test.

We still need to remove this test as the change is to return an error if there's anything after -version whereas previously we silently ignored it.  I think the current behavior is correct.

Need to fix the other test and also make a change to the -v handling.  I should have caught this in the code review.  Apologies.

@iSazonov if you're fine with the changes, can you merge so we can unblock other PRs?  If you have specific concerns with the expected behavior, I think we can continue that discussion in a new issue.  Thanks!

One concern is that we show full help in case of an error.
It's very annoying in interactive session if I typo - I'm forced to scroll back to see error message and lost my history before eyes. I'd prefer to see an error message and shot hint "Use powershell -? or -h to get more help" .

@iSazonov you referring to existing behavior and not something I introduced, correct?

Yes, I wanted to say that we need more review and cleanups in the file. I see:

#if !CORE
_ShowHelp=true
maybe formattings

@iSazonov can submit a new PR?  the #if CORECLR I was going to take care of as part of https://github.com/PowerShell/PowerShell/issues/4953

I'll try to review on next week.

@SteveL-MSFT Please clarify. In #4834 we decided to write an error to inform users about changing -Version behavior. In the PR we remove the error. Should we write in #4834 that we reject the request now?

@iSazonov added comment and closed https://github.com/PowerShell/PowerShell/issues/4834

we should be consistent and, in general, use existing norms like bash
we should not output help either
agree with the proposed changes
absolutely!


Fix Search

Migrated from GitLab: https://gitlab.com/meltano/handbook/-/issues/1
Originally created by @afolson on 2021-09-28 01:24:26

Need to move Algolia over to the Handbook and re-enable the search bar.

View 1 previous comment from the original issue on GitLab


Word wrap not working for lines above 1000 characters

VSCode Version: 1.10.0
OS Version: Windows 10 x64

Steps to Reproduce:

Create a new file containing one line that is 1001 characters long. At this point, Alt+Z switches word wrap on and off, as expected. Leave word wrap on.
Click on a different tab or press Ctrl+N to switch to editing a different file.
Go back to the original file. Word wrap is now disabled and Alt+Z doesn't do anything anymore.
Delete exactly one character, so that the line is 1000 characters long.
Switch to a different tab and back again. Word wrap is now working fine.

Can you please share your user settings. I had trouble reproducing this on my machine

Here they are:
{
    "editor.autoClosingBrackets": false,
    "editor.fontSize": 13,
    "editor.mouseWheelZoom": true,
    "editor.renderIndentGuides": true,
    "explorer.autoReveal": false,
    "typescript.check.tscVersion": false,
    "git.confirmSync": false,
    "window.zoomLevel": 1,
    "typescript.referencesCodeLens.enabled": false,
    "java.errors.incompleteClasspath.severity": "ignore",
    "editor.codeLens": false,
    "window.newWindowDimensions": "maximized",
    "editor.wordWrap": "on",
    "workbench.iconTheme": "vs-minimal",
    "editor.minimap.enabled": true
}

Also, here's a short vide demonstrating the issue: https://youtu.be/M2Lk-CLdHcE

v1.10.1
ubuntu 16.04 64-bit
Default settings
No extensions
I'm having a similar problem. Word wrap correctly remains off when typing/editing normally. Even past 1000 chars.
But type 1001+ chars, switch tabs and back, then the text becomes wrapped to viewport. Existing files with 1001+ chars also wrap when opened.

Duplicate of #21552.
TL;DR: We try to detect minified/generated files and enforce wrapping on them. If a file has more characters on lines longer than 1k chars than it has characters on lines shorter than 1k chars, we consider that this is a minified/generated (not source) file. For performance reasons (no virtual horizontal scrolling) we enforce viewport wrapping. In the insiders channel, there is a fix where Toggle Word Wrap can unwrap such files. Read more about it in #21552.


[JetNews, JetSnack, Owl] The release version will crash if navigation-compose is used with minifyEnabled

When launching the release builds of Jetnews, Jetsnack, and Owl
the following crash occurs:
11-02 14:43:30.994 28787 28787 E AndroidRuntime: FATAL EXCEPTION: main
11-02 14:43:30.994 28787 28787 E AndroidRuntime: Process: com.example.owl, PID: 28787
11-02 14:43:30.994 28787 28787 E AndroidRuntime: java.lang.IllegalAccessError: Illegal class access: 'uu0$a' attempting to access 'androidx.navigation.compose.NavHostKt$PopulateVisibleList$1$1$observer$1' (declaration of 'uu0$a' appears in /data/app/~~NgrI5-Z5F9_KRcBP9LmPFA==/com.example.owl-bQbZtxrcUChvaK343C1SXw==/base.apk)

All these apps use navigation-compose and set minifyEnabled to true for release build.
My app also has the same issue.
But it works well when I use navigation-compose older than 2.4.0-alpha08.

Thank you for the report, I can reproduce.
After investigating a bit, it appears this is an underlying R8 issue which only started occurring with changes added that were first released with 2.4.0-alpha09 to navigation-compose.
It looks that this has already been fixed on the R8 side, as updating to a later version of R8 prevents the crash for me:
buildscript {
    repositories {
        maven {
            url 'https://storage.googleapis.com/r8-releases/raw'
        }
    }
    dependencies {
        classpath 'com.android.tools:r8:3.1.34'          // Must be before the Gradle Plugin for Android.
        classpath 'com.android.tools.build:gradle:X.Y.Z' // Your current AGP version.
     }
}

Could you verify if updating R8 fixes the crash for your app?

@alexvanyo Thank you! That works for my app as well.

Closing as obsolete

Closing as obsolete


Perfect number

Do the checklist before submitting the PR:

[x] Have you read the guidelines mentioned in CONTRIBUTING.md?
[ ] Have you squashed your commits?

Q: What version of Node.js you've used in the project? (e.g. Node v10.11.0)
A: v10.15.1
Q: Mention the Issue Number! (e.g. Fixed #8)
A: Fixed #23

Give additional information regarding the PR below:

Codecov Report

Merging #26 into master will increase coverage by 0.16%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   94.01%   94.17%   +0.16%
==========================================
  Files          32       33       +1
  Lines         284      292       +8
==========================================
+ Hits          267      275       +8
  Misses         17       17

Impacted Files
Coverage Δ

src/perfect/index.js
100% <100%> (ø)

Continue to review full report at Codecov.

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

Well done! :tada:  The PR looks fine so I'm merging it, but from next PR onwards try to squash those commits and include ES6+ syntax!


Roll engine 38a646e14cc2..3052dbd79bdc (24 commits)

https://github.com/flutter/engine/compare/38a646e14cc2...3052dbd79bdc
git log 38a646e14cc2..3052dbd79bdc --no-merges --oneline
3052dbd79 SystemNavigator.pop can pop w/o UINavigationController (flutter/engine#6341)
0c096f798 Roll src/third_party/skia b3e48afc936d..227d4e10276c (1 commits) (flutter/engine#6359)
b8c2a17a1 Roll src/third_party/skia cfe1264d7465..b3e48afc936d (3 commits) (flutter/engine#6356)
c589b312a Expose push/popRoute on FlutterViewController (flutter/engine#6347)
075b3fcca Roll src/third_party/skia 5ea41fc89b26..cfe1264d7465 (1 commits) (flutter/engine#6355)
2dd9b99aa Roll Dart to version 808ed6238b9262660e31ea826f7aea6cfa3a3493 (flutter/engine#6354)
5b799381f Dont make any binaries specify an X11 dependency. (flutter/engine#6353)
309ac4e1b V0.8.2 fix compile problem with xcode10 (flutter/engine#6339)
26fdd1e4f Roll src/third_party/skia 5767fc042834..5ea41fc89b26 (3 commits) (flutter/engine#6351)
cc44ca5d0 Perform persistent cache stores on the IO thread outside the frame workload. (flutter/engine#6350)
f2a3df97e Wire up the Skia persistent GPU related artifacts cache. (flutter/engine#6278)
9d4b80a60 Roll src/third_party/skia 17282da3aa94..5767fc042834 (8 commits) (flutter/engine#6348)
ef98dcb11 Add support for counter timeline traces from the engine. (flutter/engine#6315)
a961e96d1 Roll buildroot to pick up updated vs toolchain. (flutter/engine#6346)
d6bb59978 Fixed IsolateNameServer documentation (flutter/engine#6344)
0c854ac01 Roll src/third_party/skia bdf1431686c2..17282da3aa94 (4 commits) (flutter/engine#6345)
e394ea1dd Roll src/third_party/skia c05302bc84da..bdf1431686c2 (1 commits) (flutter/engine#6343)
bcdfc2312 Roll src/third_party/skia 5457141b011e..c05302bc84da (1 commits) (flutter/engine#6342)
a861f37f7 Pass on the new unsafe-package-serialization option (flutter/engine#6331)
5e77a1e46 Roll src/third_party/skia bd03b54cb98d..5457141b011e (1 commits) (flutter/engine#6340)
6332db875 Roll src/third_party/skia f18c297cfb74..bd03b54cb98d (1 commits) (flutter/engine#6338)
a08bc52e8 Roll src/third_party/skia 7ffa40cedbc4..f18c297cfb74 (6 commits) (flutter/engine#6336)
8247ce26b Implement restore functions on Android and iOS (flutter/engine#6322)
2ead5c06c Roll src/third_party/skia 4ef464cd3c2e..7ffa40cedbc4 (10 commits) (flutter/engine#6334)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

AutoRoller is stopped; closing the active roll.


Interest in adding pre-commit?

As it seems that there are a few instances in PRs like PR https://github.com/ssl-hep/ServiceX_frontend/pull/444 where the linting fails, is there interest in adding pre-commit hooks (which are optional) and pre-commit.ci (which would run the equivalent of pre-commit run --all-files on every commit) so that devs don't have to worry about fixing this stuff manually? https://learn.scientific-python.org/development/guides/style/#pre-commit

Hi @matthewfeickert , for now we've put in a pre-commit check but not pre-commit.ci (we'll fix issues as they come up as files get edited - it's almost all just whitespace stuff). Thanks for the suggestion.


NetworkStats should use u64 so 32-bit architectures can report stats of greater than 4GiB

I'm using this library on an armv7 processor which is 32-bit. I hit an issue that once the data sent/received had surpassed 4GiB, the network_stats function began to error out as the number of bytes could no longer fit in a usize.
This PR changes the NetworkStats struct so that rx_bytes and tx_bytes are now ByteSize, and the other fields are now u64.

Thank


Authoring dialog errors and doesn't allow the event handlers to work

Hi,
I am writing a custom component with event handlers (Touch UI - Using AEM 6.2. ) for dialog fields using the example provided. Whenever I launch the dialog on any component, I am getting the error - "coralui2.js:12588 Uncaught TypeError: Cannot read property 'selectedIndex' of undefined" and I believe this stops the events are getting registered or triggered.
Error is appearing when the below highlighted line of code runs -
getValue: function () {
if (this.options.multiple) { // multiple returns array
return this._tagListWidget.getValues();
} else if (this.options.type === 'static') { // static
return this._nativeSelect[0][this._nativeSelect[0].selectedIndex].value;
} else if (this.options.type === 'dynamic') {
return this._valueInput.val();
}
  return null;
},

We don't see this error when loading the page and this highlighted line of code runs fine. We have this error on all component dialog boxes and on all local instance and on our test instance.
Please can you advice if you have encountered this issue before and guide us to the right direction?

The same issue.
AEM 6.5


UnboundLocalError: local variable 'recurse' referenced before assignment

Trying to analyze a unknown partition;
$ sudo binwalk -vMe /dev/sda2

Scan Time:     2016-01-06 20:24:38
Target File:   /dev/sda2
MD5 Checksum:  7b1fb330a481f9d0acf1f14fe76a3aa4
Signatures:    345

DECIMAL       HEXADECIMAL     DESCRIPTION
--------------------------------------------------------------------------------

Signature Exception: local variable 'recurse' referenced before assignment
----------------------------------------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/binwalk/core/module.py", line 566, in main
    retval = self.run()
  File "/usr/local/lib/python2.7/dist-packages/binwalk/modules/signature.py", line 177, in run
    self.scan_file(fp)
  File "/usr/local/lib/python2.7/dist-packages/binwalk/modules/signature.py", line 160, in scan_file
    self.result(r=r)
  File "/usr/local/lib/python2.7/dist-packages/binwalk/core/module.py", line 461, in result
    getattr(self, dependency.attribute).callback(r)
  File "/usr/local/lib/python2.7/dist-packages/binwalk/modules/extractor.py", line 172, in callback
    (extraction_directory, dd_file, scan_extracted_files) = self.extract(r.offset, r.description, r.file.path, size, r.name)
  File "/usr/local/lib/python2.7/dist-packages/binwalk/modules/extractor.py", line 511, in extract
    return (output_directory, fname, recurse)
UnboundLocalError: local variable 'recurse' referenced before assignment
----------------------------------------------------------------------------------------------------
$

Without looking at the code; Looks like that the 'recurse' variable "escaped" some condition...

Yup, latest commit should fix this!

For future reference, it's commit 9882a629c6967e9824aedde94d8107205bc09a42.


readme needs updating

there are still references to sonar.ts.tslint.lcovReportPath although it appears this has been updated to sonar.typescript.lcov.reportPaths
additionally I cant find any documentation about other sonar.ts properties not being referenced as sonar.typescript instead and am having trouble loading my projects tslint.json so I wonder what else has changed along these lines.
the readme should be updated to reflect these changes and I'm happy to open a PR but don't know the full scope of the changes.

Please make sure you use plugin from this repo, and not a official one.

yes it is this package, from my dockerfile: RUN wget 'https://github.com/Pablissimo/SonarTsPlugin/releases/download/v1.1.0/sonar-typescript-plugin-1.1.0.jar' -P extensions/plugins

sorry you are right somehow the other extension was enabled :( my bad

This plugin has the same pluginKey. So if you install this version then go to update center... The official plugin will be installed because it has bigger version number. :)


stuck at 8 fps

Describe your bug here.
idk if it has something to do with my laptop or not but every psych engine mod i try playing runs at 8 fps or somewhere close to that and I don't know how to fix it
plz help
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
N O

its at the highest framerate

Describe your bug here.
idk if it has something to do with my laptop or not but every psych engine mod i try playing runs at 8 fps or somewhere close to that and I don't know how to fix it plz help
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
N O

either turn down the fps cap, or your laptop is just shit, and isn't powerful enough to run Psych Engine

it ran great before

its at the highest framerate

just gotta put framerate at 60 and anti aliasing off in settings
should fix

i did
it work way bett4r but still lags a bit


Update instructions.js

fix file names

Thanks! Actually since Instructions.js is also a component would you mind renaming the file .jsx instead?

Thanks! Actually since Instructions.js is also a component would you mind renaming the file .jsx instead?

Sure.


Make intro note at top of chat view closeable

This PR makes the intro note at the top closeable and shifts the announcement note below it. I've extracted out a common CloseableNote component.
Without announcement:

With announcement:

Both closed:

Thank you for taking the time to open this PR and gah love seeing screenshots! Just as a design decision I'd prefer to leave things as they are. We've created a contributing guide as well to make it clearer what makes sense for a PR or not, and I'd like UX changes to be discussed in the discord before putting in all this work! I apologize


ContainerArchiveOutputPath is missing

Type of issue
Missing information
Description
The argument ContainerArchiveOutputPath of "dotnet publish" from .Net8 is missing in the command documentation itself.
Page URL
https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-publish
Content source URL
https://github.com/dotnet/docs/blob/main/docs/core/tools/dotnet-publish.md
Document Version Independent Id
9439897e-dcb3-bc2e-5cae-35c1ca2e0681
Article author
@tdykstra
Metadata

ID: 211afdc7-c796-202d-239d-823f2a499dff
Service: dotnet-fundamentals

ContainerArchiveOutputPath is an MSBuild property, not a dotnet publish option or argument. The dotnet publish documentation does not document all of the MSBuild properties you can use with it.
@IEvangelist - ContainerArchiveOutputPath currently is documented only in the RC2 what's new -- it would be good to cover it somewhere in permanent documentation. Should it be documented in https://learn.microsoft.com/en-us/dotnet/core/docker/publish-as-container?pivots=dotnet-8-0 ?

If we are on modifying the Containerize a .NET app with dotnet publish page I'd suggest to add a section just before Clean up resources about "using" it.
As a docker beginner it took me time to understand that:

the publish command pushed the image directly inside the "Images" list in Docker Desktop (it is already described in the documentation as a "local registry", but nothing in Docker Desktop says that their Images page is this local registry)
I just have to "Run" the image to both create the container and run the published application (the Configure container execution is a bit unclear about what it does default, and that in fact the default will be enough to make a web app ready to serve its pages and APIs without any action on our part)
I can just click on the link displayed in the "Port(s)" column of the Containers page of docker desktop to open my web app (it handle the port default randomness)

This part may be considered out of scope and very specific to how Docker Desktop works today, but it would cover a gap between this tutorial and the Docker Desktop documentation.
In the end, for real world web app with only 1 executable project, the default configuration is good enough and it looks possible to have a beginner run its application in less time than the Docker Desktop installation takes. Maybe a simplified version of this tutorial (in a different page, or by using accordion blocs in this one to hide all the optional arguments) could help.


⚠ Jenkins Plugins Artifactory has degraded performance

In 4958094, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1390 ms

Resolved: Jenkins Plugins Artifactory performance has improved in fe229c5 after 12 minutes.


index.html is not working on master head

It lacks some dependencies in the repo index.html (mainly angular), so the index.html is not loading.
After loading, with the script tags from the online demo, it does not load the calendar view...
Great calendar plugin, thanks!!

The repo isn't designed to be run as a static server from master, as per the dev instructions you need to run npm install && npm start to start the local server that serves the assets.


update docs to Webpack 2

hey! thanks for this!!
webpack 2 is now stable. here are the changes that apply to these docs
i could send a PR, if you'd like. would you rather update to document just webpack 2, or add webpack 2 docs in addition to what's there already?

Thanks.  a PR is always welcomed. I tend to keep the doc for webpack 1 (of cause we need to make a title for each version) and prepend the webpack 2 version.
I haven't used React myself for a long time. I am not sure if this extension still works for React now.
I will file an issue to see if people still need this extension. If yes, I will keep developing this extension to make it better. Maybe you can make the PR at then. But again. I am also happy to see the PR if you'd like to send it now.

I am not sure if this extension still works for React now.

i got it working! initially, i'd click "start" and it would give no feedback. adding these lines to my project appears to have worked:
if (process.env.NODE_ENV !== 'production') {
  window.Perf = require('react-addons-perf')
}

so i actually didn't go down the webpack route.

I will file an issue to see if people still need this extension.

i discovered this extension in this article by Twitter engineering! i haven't spent much time using it, but it's already been helpful. thanks again! 🌟


Ensuring raster sizes are the same

I have been reprocessing some data and I ran into an error related to the size of the rasters from ariaExtract. Specifically, when I run:
ariaExtract.py -w Extracted -f "products/*.nc" -d download -m download -croptounion -l all
I get the error:
Generating: bPerpendicular - [>                                                 ]Traceback (most recent call last):
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/bin/ariaExtract.py", line 16, in
main(inps)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 757, in main
export_products(standardproduct_info.products[1], standardproduct_info.bbox_file, prods_TOTbbox, inps.layers, inps.rankedResampling, dem=demfile, lat=Latitude, lon=Longitude, mask=inps.mask, outDir=inps.workdir, outputFormat=inps.outputFormat, stitchMethodType='overlap', verbose=inps.verbose, num_threads=inps.num_threads, multilooking=inps.multilooking)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 396, in export_products
finalize_metadata(outname, open_shapefile(bbox_file, 0, 0).bounds, prods_TOTbbox, dem, lat, lon, mask, outputFormat, verbose=verbose)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 501, in finalize_metadata
out_interpolated = mask.ReadAsArray()*out_interpolated
ValueError: operands could not be broadcast together with shapes (18481,7476) (18481,7477)
So it appears that when aria-tools resamples the data it does not ensure that all rasters (in this case the bPerpendicular) are the same size. Would you like me to try to build this fix in by specifying the array size?

@rzinke, hmm strange, I thought I had resolved this inconsistency a while back. Which revision is this though? Can you try again with my latest PR #145?
If this issue persists with the latest PR, could you provide me with a list of products you had used so I can try to duplicate it?

Nicola was having some similar trouble a few weeks ago when he tried to use a small bounding box. It worked fine when he used a bigger box. I was not sure if it was an ARIA-tools bug or pilot error. I will ask him to try that again when this is merged in to the main version. I think it is a little complicated to explain how he could run the PR version.

The only difference is to clone from the PR instead of the dev branch.
The install procedure remains the same.
On Sat, 25 Apr 2020 at 13:29, Eric Fielding<EMAIL_ADDRESS>wrote:

Nicola was having some similar trouble a few weeks ago when he tried to
use a small bounding box. It worked fine when he used a bigger box. I was
not sure if it was an ARIA-tools bug or pilot error. I will ask him to try
that again when this is merged in to the main version. I think it is a
little complicated to explain how he could run the PR version.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aria-tools/ARIA-tools/issues/148#issuecomment-619436493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESZPSJPYXDASTN3NO6VTUTRONB4PANCNFSM4MQ4KLUA
.

Hi @sssangha, I have installed the sss_bugfixes version of ARIA-tools and am rerunning some of the test patches. Track 41 works fine, but track 121 is throwing an unexpected error. Have you encountered anything similar?
For the command:
ariaExtract.py -f "/u/sar-r2/rzinke/Tibet/Track121/AllProducts/2017.nc" -d download -m download -l all -nt 8 -w T121 -croptounion
I get the error:
All (314) interferograms are spatially continuous.
All layers are to be extracted, pass all keys.
Thread count specified for gdal multiprocessing = 8
Traceback (most recent call last):
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/bin/ariaExtract.py", line 16, in
main(inps)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 743, in main
standardproduct_info.products[0], standardproduct_info.products[1], standardproduct_info.bbox_file, prods_TOTbbox, arrshape, proj = merged_productbbox(standardproduct_info.products[0], standardproduct_info.products[1], os.path.join(inps.workdir,'productBoundingBox'), standardproduct_info.bbox_file, inps.croptounion, num_threads=inps.num_threads, minimumOverlap=inps.minimumOverlap)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 281, in merged_productbbox
union_bbox=open_shapefile(outname, 0, 0)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/shapefile_util.py", line 37, in open_shapefile
file_bbox = file_bbox.GetLayerByIndex(lyrind).GetFeature(ftind)
AttributeError: 'NoneType' object has no attribute 'GetLayerByIndex'

@rzinke would recommend to test wrt the dev branch as there are no open PR from @sssangha  at the moment which are note yet merge. To my knowledge this issue has also not been fixed on the dev branch either, nor release 1.1, but we know how to fix it.
Not sure if you missed a wild-card in your command.

I copied the wrong command, I had been running "/u/sar-r2/rzinke/Tibet/Track121/AllProducts/2017.nc" with wildcards.
Switching to branch dev and retesting.

I have seen something like this if one of the files got corrupted or if i was running from a non-cleaned directory.

@rzinke, yes I'd recommend to test in a clean directory as @dbekaert said. In case the products themselves were corrupted (i.e. the download had been interrupted), I'd save some time and just redownload a small subset in time and run a test on that.

Thanks for the suggestions. Re-running after clearing the old directory seems to have solved the error "...AttributeError: 'NoneType' object has no attribute 'GetLayerByIndex'"
However, I am still receiving the error I first reported pertaining to the incorrect raster size, i.e.,
Generating: bPerpendicular - [>                                                 ]Traceback (most recent call last):
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/bin/ariaExtract.py", line 16, in
main(inps)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 757, in main
export_products(standardproduct_info.products[1], standardproduct_info.bbox_file, prods_TOTbbox, inps.layers, inps.rankedResampling, dem=demfile, lat=Latitude, lon=Longitude, mask=inps.mask, outDir=inps.workdir, outputFormat=inps.outputFormat, stitchMethodType='overlap', verbose=inps.verbose, num_threads=inps.num_threads, multilooking=inps.multilooking)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 396, in export_products
finalize_metadata(outname, open_shapefile(bbox_file, 0, 0).bounds, prods_TOTbbox, dem, lat, lon, mask, outputFormat, verbose=verbose)
File "/u/sar-r0/rzinke/python/miniconda3/envs/ARIA-tools/lib/python3.6/site-packages/ARIAtools/extractProduct.py", line 501, in finalize_metadata
out_interpolated = mask.ReadAsArray()*out_interpolated
ValueError: operands could not be broadcast together with shapes (18481,7476) (18481,7477)

@EJFielding, I have addressed this issue now in the latest PR #156, so I'd suggest you and/or Nicola take a look.

Closing this ticket as it is resolved for @rzinke . However need to confirm that #157 is resolved too.


[Bug]: explicitSpread does not include explicit conditional spreading (react/jsx-props-no-spreading)

Is there an existing issue for this?

[X] I have searched the existing issues and my issue is unique
[X] My issue appears in the command-line and not only in the text editor

Description Overview

On occasion I need a property added conditionally. The way I do this is

Button Link

because the following produces invalid HTML since role may not be false

Button Link

Expected Behavior

I expect explicitSpread: "ignore" to cover this use case since the spreading is completely explicit.

eslint-plugin-react version
7.31.11
eslint version
8.28.0
node version
18.12.1

role={actAsButton ? 'banner' : undefined}


[1.12] Spouts use PutFileClient

We need to get rid of the withSpoutFile function since the retry doesn't really make sense when using the PutFileClient (we wouldn't be able to know which file needs to be retried). So, I think you would just want to use the tar reader as the source for the put file calls on the PutFileClient and get rid of the withSpoutFile function wrapping the put file calls and just delete it since it is dead code at that point.

We need to get rid of the withSpoutFile function since the retry doesn't really make sense when using the PutFileClient (we wouldn't be able to know which file needs to be retried). So, I think you would just want to use the tar reader as the source for the put file calls on the PutFileClient and get rid of the withSpoutFile function wrapping the put file calls and just delete it since it is dead code at that point.


Gather Elevation Map Data

Get good quality elevation map for URC area.

This is a good source for elevation data:
https://gis.utah.gov/data/elevation-and-terrain/2018-lidar-southern-utah/
It has 0.25 sqm. pixel resolution (each pixel corresponds to a 0.5x0.5m square). Each data point has vertical accuracy ±0.1m.
To download the data:

go to the interactive map from that page
draw the region you want to fetch
select the 0.5 meter DEM (DSM if available)
download
It will be zipped in GeoTiff format. This is a rastered image file format. You can technically extract data directly, but there is a more practical way to retrieve the raw data:
Download and open QGIS.
Open the .tif in a new project.
Raster>Conversion>Translate
Under the field "Converted," specify the file you want to export to. The program will change the conversion based on the file extension you specify. gdal (the package built-in for conversion) is very finnicky, so it may or may not allow you to make a csv. If .csv is unavailable, .xyz is also very similar. Instead of comma separated, the x, y, and z values are space separated.

I'm not sure what the x and y values are yet. They're definitely in meters, which is probably sufficient for generating the costmap, but we want to have some sense of x-y to coordinates, so we can initialize the pose of the rover.

We already have the set of elevation points (see Issue #99) and NumPy has a vectorized framework to convert discretized elevation data to slope data.
granularity = 0.5 # our current best data has point granularity of 0.5 meters
elevation = np.array() # parse the .xyz elevation data into a NumPy array
x_grad, y_grad = np.gradient(elevation)
x_grad, y_grad = x_grad / granularity, y_grad / granularity
slope = np.sqrt(np.power(x_grad, 2) + np.power(y_grad, 2))
# Square root is not necessary.
# We may even want to transform these values during testing depending on the behavior of the robot.
# Just make sure any transformations are monotone.

Because it's vectorized, this will be faster than any kernelization I would write. We can just use a ROS Python node to parse this data and create the static layer of the 2D costmap. I'm pretty sure we can just directly parse this data into the static layer. But how would that parsing look like? How is the static layer used? How can we use the 2D costmap to make a planned path? If we ever use vision, how do we import that dynamic obstacle layer into this costmap?

costmap_2d subscribes to a topic called "map" which takes an OccupancyGrid message.
At 3.1.2 here, this allows us to initialize the costmap from a user-generated static map. The formatting for the OccupancyGrid message is a little bit unclear. I don't understand the packages that we're using, but different ros documentations say that the data in the data array in OccupancyGrid are in different formats.

uint8 [0-255] representing the cost of a cell
uint8 [0-100] representing the percent likelihood that a given cell is an obstacle
int8 (-1, 0, 1) -1: unknown, 0: not obstacle, 1: obstacle
float [0-1] representing the likelihood that a given cell is an obstacle

I don't know how nav messages work, but this seems like a bit of an issue to me.


Improvements to mojo-executor

We could implement the following improvements to simplify our configuration of the mojo-executor:

Alias handling - currently, we are explicitly looking for certain parameters that have been aliased and renaming them to the original parameter name before passing to mojo-executor
Irrelevant configuration ignore - to be more inline with built-in Maven behavior, we should look into having the ability to ignore irrelevant or invalid configuration rather than throwing an error

This issue is nearly 4 years old and will not be implemented. Closing.


Rework documentation about running ansible (esp.with Vagrant)

Related: https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/379 https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/379 https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/423 https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/681 https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/656
New updates put in under https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1063 to at least make the information a bit more accurate but will likely need a second round of updates :-)

Some updates put in:
https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/828

@sxa555 asked that some documentation on how to install vagrant with KVM be put in.

@karianna Any suggestions on additions or reworking, in particular? :-)

@karianna Any suggestions on additions or reworking, in particular? :-)

I need to go through it from scratch as a new user :-).  I'll try to get that this week.

Also included the basic execution in the new FAQ being added at https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1066

@Willsparker I immediately get:
A host only network interface you're attempting to configure via DHCP
already has a conflicting host only adapter with DHCP enabled. The
DHCP on this adapter is incompatible with the DHCP settings. Two
host only network interfaces are not allowed to overlap, and each
host only network interface can have only one DHCP server. Please
reconfigure your host only network or remove the virtual machine

@karianna Is this following the instructions in ../ansible/README.md ? I can't say I've seen that error message before, was it on vagrant up ?

@Willsparker Yep on the vagrsant up...

https://github.com/hashicorp/vagrant/issues/3083
At a guess, this may be the cause - if it's a fresh install of Virtualbox and no vagrant ups have been ran before on your machine?
The issue says that running
VBoxManage dhcpserver remove --netname HostInterfaceNetworking-vboxnet0  before vagrant up is a workaround to it.
The issue says the error is due to :

When encountering a private network with type: :dhcp, Vagrant sees this HostInterfaceNetworking-vboxnet0 dhcpserver and assumes that it is valid and ready to be attached to vboxnet0. But vboxnet0 was just created and has an IP address of <IP_ADDRESS>.

So the error message comes from VirtualBox yelling that you can't assign a dhcpserver with an IP address outside the network of an adapter.

VBoxManage dhcpserver remove --netname HostInterfaceNetworking-vboxnet0

Aha, the trick is to actually execute:
VBoxManage list dhcpservers first to get the --netname value (mine was slightly different

artijnsMSFTMBP:ansible karianna$ vagrant up
Bringing machine 'adoptopenjdkC6' up with 'virtualbox' provider...
==> adoptopenjdkC6: Checking if box 'generic/centos6' version '2.0.6' is up to date...
==> adoptopenjdkC6: Clearing any previously set network interfaces...
==> adoptopenjdkC6: Preparing network interfaces based on configuration...
    adoptopenjdkC6: Adapter 1: nat
    adoptopenjdkC6: Adapter 2: hostonly
==> adoptopenjdkC6: Forwarding ports...
    adoptopenjdkC6: 22 (guest) => 2222 (host) (adapter 1)
==> adoptopenjdkC6: Running 'pre-boot' VM customizations...
==> adoptopenjdkC6: Booting VM...
==> adoptopenjdkC6: Waiting for machine to boot. This may take a few minutes...
    adoptopenjdkC6: SSH address: <IP_ADDRESS>:2222
    adoptopenjdkC6: SSH username: vagrant
    adoptopenjdkC6: SSH auth method: private key
    adoptopenjdkC6: Warning: Connection reset. Retrying...
    adoptopenjdkC6: Warning: Remote connection disconnect. Retrying...
    adoptopenjdkC6:
    adoptopenjdkC6: Vagrant insecure key detected. Vagrant will automatically replace
    adoptopenjdkC6: this with a newly generated keypair for better security.
    adoptopenjdkC6:
    adoptopenjdkC6: Inserting generated public key within guest...
    adoptopenjdkC6: Removing insecure key from the guest if it's present...
    adoptopenjdkC6: Key inserted! Disconnecting and reconnecting using new SSH key...
==> adoptopenjdkC6: Machine booted and ready!
==> adoptopenjdkC6: Checking for guest additions in VM...
    adoptopenjdkC6: The guest additions on this VM do not match the installed version of
    adoptopenjdkC6: VirtualBox! In most cases this is fine, but in rare cases it can
    adoptopenjdkC6: prevent things such as shared folders from working properly. If you see
    adoptopenjdkC6: shared folder errors, please make sure the guest additions within the
    adoptopenjdkC6: virtual machine match the version of VirtualBox you have installed on
    adoptopenjdkC6: your host and reload your VM.
    adoptopenjdkC6:
    adoptopenjdkC6: Guest Additions Version: 5.2.34
    adoptopenjdkC6: VirtualBox Version: 6.1
==> adoptopenjdkC6: Setting hostname...
==> adoptopenjdkC6: Configuring and enabling network interfaces...
==> adoptopenjdkC6: Mounting shared folders...
    adoptopenjdkC6: /vagrant => /Users/karianna/Documents/workspace/adoptopenjdk/openjdk-infrastructure/ansible
==> adoptopenjdkC6: Running provisioner: shell...
    adoptopenjdkC6: Running: inline script
    adoptopenjdkC6: Loaded plugins: fastestmirror
    adoptopenjdkC6: Setting up Update Process
    adoptopenjdkC6: Determining fastest mirrors
    adoptopenjdkC6: Could not retrieve mirrorlist http://mirrorlist.centos.org/?release=6&arch=x86_64&repo=os&infra=stock error was
    adoptopenjdkC6: 14: PYCURL ERROR 6 - "Couldn't resolve host 'mirrorlist.centos.org'"
    adoptopenjdkC6: Error: Cannot find a valid baseurl for repo: base
    adoptopenjdkC6: Loaded plugins: fastestmirror
    adoptopenjdkC6: Setting up Install Process
    adoptopenjdkC6: Loading mirror speeds from cached hostfile
    adoptopenjdkC6: Could not retrieve mirrorlist http://mirrorlist.centos.org/?release=6&arch=x86_64&repo=os&infra=stock error was
    adoptopenjdkC6: 14: PYCURL ERROR 6 - "Couldn't resolve host 'mirrorlist.centos.org'"
    adoptopenjdkC6: Error: Cannot find a valid baseurl for repo: base
    adoptopenjdkC6: Loaded plugins: fastestmirror
    adoptopenjdkC6: Setting up Install Process
    adoptopenjdkC6: Loading mirror speeds from cached hostfile
    adoptopenjdkC6: Could not retrieve mirrorlist http://mirrorlist.centos.org/?release=6&arch=x86_64&repo=os&infra=stock error was
    adoptopenjdkC6: 14: PYCURL ERROR 6 - "Couldn't resolve host 'mirrorlist.centos.org'"
    adoptopenjdkC6: Error: Cannot find a valid baseurl for repo: base
    adoptopenjdkC6: cp: cannot stat `/etc/ansible/ansible.cfg': No such file or directory
    adoptopenjdkC6: sed: can't read /ansible/ansible.cfg: No such file or directory
    adoptopenjdkC6: ---------------------------------------------------------------------------------------
    adoptopenjdkC6: *** Creating public-private keypair for ansible-playbook... for user 'vagrant' ***
    adoptopenjdkC6: ---------------------------------------------------------------------------------------
    adoptopenjdkC6: *** Public-private keypair for ansible-playbook... for user 'vagrant' has been created in ***
    adoptopenjdkC6: total 20K
    adoptopenjdkC6: 4.0K drwx------. 2 vagrant vagrant 4.0K Feb 11 16:50 .
    adoptopenjdkC6: 4.0K drwx------. 3 vagrant vagrant 4.0K Nov 21 15:07 ..
    adoptopenjdkC6: 4.0K -rw-------. 1 vagrant vagrant  389 Feb 11 16:47 authorized_keys
    adoptopenjdkC6: 4.0K -rw-------. 1 vagrant vagrant 3.2K Feb 11 16:50 id_rsa
    adoptopenjdkC6: 4.0K -rw-r--r--. 1 vagrant vagrant  744 Feb 11 16:50 id_rsa.pub
    adoptopenjdkC6: ---------------------------------------------------------------------------------------
    adoptopenjdkC6: *** Creating 'hosts' file for ansible-playbook... ***
    adoptopenjdkC6: ---------------------------------------------------------------------------------------
    adoptopenjdkC6: *** '/vagrant/playbooks/hosts' file for ansible-playbook contains ***
    adoptopenjdkC6: localhost ansible_connection=local
    adoptopenjdkC6: ---------------------------------------------------------------------------------------
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

Due to the line [ -r /vagrant/id_rsa.pub ] && mkdir -p $HOME/.ssh && cat /vagrant/id_rsa.pub >> $HOME/.ssh/authorized_keys being at the end of the provision script, if the id_rsa.pub file isn't there, it will return an error - however the vagrant machine is still booted and working, i.e. vagrant ssh will still work.
I'll make a PR to stop that, however, I'm going to just have a look at the Docs and update them again - since October, a fair amount has been added, I think :-) I'll ref this issue in the PR.


shutdown / reboot leads to "unexpected power loss" on SSD

After i had lost a raspiblitz instance to a failing HDD and my last SSD had shown some unrecoverable read errors.
I installed the smartmon-tools on my raspiblitz and tried to figure out if there are any metrics, obtainable via smartctl -a /dev/sda, which would help to determine the health status of my SSD.
root~# smartctl -a /dev/sda

smartctl 7.2 2020-12-30 r5155 [aarch64-linux-5.10.92-v8+] (local build)
Copyright (C) 2002-20, Bruce Allen, Christian Franke, www.smartmontools.org

=== START OF INFORMATION SECTION ===
Model Family:     Marvell based SanDisk SSDs
Device Model:     SanDisk SSD PLUS 1000GB
Serial Number:    211702A00D3C
LU WWN Device Id: 5 001b44 8babd6476
Firmware Version: UH4504RL
User Capacity:    1,000,204,886,016 bytes [1.00 TB]
Sector Size:      512 bytes logical/physical
Rotation Rate:    Solid State Device
Form Factor:      2.5 inches
TRIM Command:     Available, deterministic
Device is:        In smartctl database [for details use: -P show]
ATA Version is:   ACS-3, ACS-2 T13/2015-D revision 3
SATA Version is:  SATA 3.2, 6.0 Gb/s (current: 6.0 Gb/s)
Local Time is:    Thu May 12 02:07:43 2022 CEST
SMART support is: Available - device has SMART capability.
SMART support is: Enabled

=== START OF READ SMART DATA SECTION ===
SMART overall-health self-assessment test result: PASSED

General SMART Values:
Offline data collection status:  (0x00) Offline data collection activity
                                        was never started.
                                        Auto Offline Data Collection: Disabled.
Self-test execution status:      (   0) The previous self-test routine completed
                                        without error or no self-test has ever
                                        been run.
Total time to complete Offline
data collection:                (  120) seconds.
Offline data collection
capabilities:                    (0x11) SMART execute Offline immediate.
                                        No Auto Offline data collection support.
                                        Suspend Offline collection upon new
                                        command.
                                        No Offline surface scan supported.
                                        Self-test supported.
                                        No Conveyance Self-test supported.
                                        No Selective Self-test supported.
SMART capabilities:            (0x0003) Saves SMART data before entering
                                        power-saving mode.
                                        Supports SMART auto save timer.
Error logging capability:        (0x01) Error logging supported.
                                        General Purpose Logging supported.
Short self-test routine
recommended polling time:        (   2) minutes.
Extended self-test routine
recommended polling time:        ( 182) minutes.

SMART Attributes Data Structure revision number: 1
Vendor Specific SMART Attributes with Thresholds:
ID# ATTRIBUTE_NAME          FLAG     VALUE WORST THRESH TYPE      UPDATED  WHEN_FAILED RAW_VALUE
  5 Reallocated_Sector_Ct   0x0032   100   100   000    Old_age   Always       -       0
  9 Power_On_Hours          0x0032   100   100   000    Old_age   Always       -       5557
 12 Power_Cycle_Count       0x0032   100   100   000    Old_age   Always       -       23
165 Total_Write/Erase_Count 0x0032   100   100   000    Old_age   Always       -       1089
166 Min_W/E_Cycle           0x0032   100   100   ---    Old_age   Always       -       6
167 Min_Bad_Block/Die       0x0032   100   100   ---    Old_age   Always       -       98
168 Maximum_Erase_Cycle     0x0032   100   100   ---    Old_age   Always       -       24
169 Total_Bad_Block         0x0032   100   100   ---    Old_age   Always       -       583
170 Unknown_Marvell_Attr    0x0032   100   100   ---    Old_age   Always       -       0
171 Program_Fail_Count      0x0032   100   100   000    Old_age   Always       -       0
172 Erase_Fail_Count        0x0032   100   100   000    Old_age   Always       -       0
173 Avg_Write/Erase_Count   0x0032   100   100   000    Old_age   Always       -       6
174 Unexpect_Power_Loss_Ct  0x0032   100   100   000    Old_age   Always       -       13
184 End-to-End_Error        0x0032   100   100   ---    Old_age   Always       -       0
187 Reported_Uncorrect      0x0032   100   100   000    Old_age   Always       -       0
188 Command_Timeout         0x0032   100   100   ---    Old_age   Always       -       0
194 Temperature_Celsius     0x0022   045   063   000    Old_age   Always       -       55 (Min/Max 20/63)
199 SATA_CRC_Error          0x0032   100   100   ---    Old_age   Always       -       0
230 Perc_Write/Erase_Count  0x0032   100   100   000    Old_age   Always       -       779 276 779
232 Perc_Avail_Resrvd_Space 0x0033   100   100   005    Pre-fail  Always       -       100
233 Total_NAND_Writes_GiB   0x0032   100   100   ---    Old_age   Always       -       6425
234 Perc_Write/Erase_Ct_BC  0x0032   100   100   000    Old_age   Always       -       14707
241 Total_Writes_GiB        0x0030   100   100   000    Old_age   Offline      -       5486
242 Total_Reads_GiB         0x0030   100   100   000    Old_age   Offline      -       59842
244 Thermal_Throttle        0x0032   000   100   ---    Old_age   Always       -       0

SMART Error Log Version: 1
No Errors Logged

SMART Self-test log structure revision number 1
Num  Test_Description    Status                  Remaining  LifeTime(hours)  LBA_of_first_error
# 1  Extended offline    Completed without error       00%         4         -
# 2  Extended offline    Aborted by host               90%         1         -
# 3  Short offline       Completed without error       00%         1         -

Selective Self-tests/Logging not supported

After monitoring these for some weeks. I found that the values of attributes 12 Power_Cycle_Count and 174 Unexpect_Power_Loss_Ct increased whenever i rebooted or powered-off my raspiblitz, using the menu via ssh.
As a test I loged in as root via ssh and issued the systemctl stop ... and shutdown -r now commands manually.
This lead to an increase of  12 Power_Cycle_Count but no change in174 Unexpect_Power_Loss_Ct
I will try to find a way to reboot / shutdown / power-off the raspiblitz via menue, which does not lead to an unexpeted powerloss for the SSD.

As a test I loged in as root via ssh and issued the systemctl stop ... and shutdown -r now commands manually.
This lead to an increase of 12 Power_Cycle_Count but no change in 174 Unexpect_Power_Loss_Ct

unfortunately I can not reproduce this....the  174 Unexpect_Power_Loss_Ct increases with each reboot / power-off indepentant if it has benn initiated via menue or in ssh session

After conducting some test, the situation seems as follows (at least for my Hardware / Software setup)
The SSD experience an "Unexpected Powerloss" if shutdown -r now  or shutdown -h now is used.
The only way to prevent this, is using shutdown -P now ... wait for some secondes and manualy powercycle the RaspiBlitz.
I would like to encourage the comunity to contuct thier own tests as different Hardware setups SSD + USB-to-SATA adaptors may behave different.

Thanks for the research on this. Which USB-to-SATA controller are you using? The UGREEN as linked in the RaspiBlitz shopping list: https://github.com/rootzoll/raspiblitz#amazon-shopping-list-buy-parts--build-it-yourself

Thanks for the research on this. Which USB-to-SATA controller are you using? The UGREEN as linked in the RaspiBlitz shopping list: https://github.com/rootzoll/raspiblitz#amazon-shopping-list-buy-parts--build-it-yourself

Yes, exactly
root@DesertRetreatLN:~# lsusb -vvv -t
/:  Bus 02.Port 1: Dev 1, Class=root_hub, Driver=xhci_hcd/4p, 5000M
    ID 1d6b:0003 Linux Foundation 3.0 root hub
    /sys/bus/usb/devices/usb2  /dev/bus/usb/002/001
    |__ Port 2: Dev 2, If 0, Class=Mass Storage, Driver=uas, 5000M
        ID 174c:1153 ASMedia Technology Inc. ASM1153 SATA 3Gb/s bridge
        /sys/bus/usb/devices/2-2  /dev/bus/usb/002/002

We could add a systemd service which runs when shutdown or reboot is triggered like:
[Unit]
Description=Clean Shutdown Service
DefaultDependencies=no
Before=halt.target shutdown.target reboot.target

[Service]
Type=oneshot
ExecStart=/home/admin/config.scripts/blitz.shutdown.sh
RemainAfterExit=yes

[Install]
WantedBy=halt.target shutdown.target reboot.target

from: https://askubuntu.com/questions/1162420/how-to-run-commands-on-shutdown-with-systemd

Interesting details connected to the X825 SATA-USB board:
the Samsung 870 QVO disk does not have the 174 Unexpect_Power_Loss_Ct metric while the Sandisk SSD Plus and WD Blue disks have it.


Typo in Codelabs documentation. Prone to confusion

Affected page: https://codelabs.developers.google.com/codelabs/first-flutter-app-pt1#4
Under "Create boilerplate code for a stateful widget." section
The sentence "The boilerplate code for two classes appears, and the cursor is positioned for you to enter the name of your stateless widget."
should read instead: "The boilerplate code for two classes appears, and the cursor is positioned for you to enter the name of your stateful widget."
(replace stateless for stateful) -> Confused a newbie like me, which is the intended audience of this codelab.

If you don't mind I'll take a shot at this

Hey, Can I fix this? 😄

Fixed. Thanks for the report!


commands: make build podman compatible

Description of the change:
When running 'operator-sdk' build with podman-docker the build fails
with a path error. Removing -f and setting the PATH argument to ./build
allows podman to build the container.
Has been tested against podman 1.12 on fedora 29 and docker-ce  5:18.09.3~3-0~debian-stretch on debian stretch
Motivation for the change:
I want to be able to build the operator container with podman as I don't use docker

@christophefarges I don't think this change will work because the Dockerfile expects the working directory to be <operatorRootDir>, not <operatorRootDir>/build.
I tested your PR with docker-ce 18.06.1, and got the following error:
$ operator-sdk build go-operator
INFO[0000] Building Docker image go-operator
Sending build context to Docker daemon  40.22MB
Step 1/7 : FROM registry.access.redhat.com/ubi7-dev-preview/ubi-minimal:7.6
 ---> 1f8526ca508d
Step 2/7 : ENV OPERATOR=/usr/local/bin/go-operator     USER_UID=1001     USER_NAME=go-operator
 ---> Using cache
 ---> 2027d56c12c5
Step 3/7 : COPY build/_output/bin/go-operator ${OPERATOR}
COPY failed: stat /var/lib/docker/tmp/docker-builder498450201/build/_output/bin/go-operator: no such file or directory
Error: failed to output build image go-operator: (failed to exec []string{"docker", "build", "build/", "-t", "go-operator"}: exit status 1)
Usage:
  operator-sdk build <image> [flags]

Flags:
      --docker-build-args string     Extra docker build arguments as one string such as "--build-arg https_proxy=$https_proxy"
      --enable-tests                 Enable in-cluster testing by adding test binary to the image
  -h, --help                         help for build
      --namespaced-manifest string   Path of namespaced resources manifest for tests (default "deploy/operator.yaml")
      --test-location string         Location of tests (default "./test/e2e")

@christophefarges any traction on those changes? I'd very much like podman support as well.

Sorry for the enduser ping. I also do not use Docker and I would love to see better support for podman.

I don't have an issue using podman in the master branch now. It looks like the work in #1311 fixed the order of the arguments. Granted, this does require the podman-docker package or some other sort of symlink, so it may still be worth adding podman explicitly as an option alongside docker and buildah.

Since the original intent of this PR seems to be resolved, I'll close this.
I've also submitted #1488 to add a podman option to the operator-sdk build --image-builder flag.


Pgsync

Added pgsync to create elasticsearch index from postgres db and keep it synchronized with tables

Steps to set up pgsync from a postgres db dump:

start postgres: sudo docker-compose up postgres
connect to postgres db through psql: psql -h localhost -U postgres -p 5432 -d postgres
delete literature db: DROP DATABASE literature
create empty literature db: CREATE DATABASE literature
restore literature db from dump: pg_restore -d literature -h localhost -U postgres < literature-4003.pg.dump.20211118
docker-compose down
start full stack: docker compose up
wait until pgsync is ready (it may throw a few errors until the db is ready) and then connect to ES API at http://localhost:9200/references_index/_search?q=test

Thanks @adamjohnwright, I've just removed the AWS key entries and changed 'master' to 'primary' in postgres config file


[WIP] webui

RA.Aid/memlog/webui_restructure.md

It looks like your .venv got committed. Can you delete that from the branch? You might have to locally git rm -rf .venv, add it to git ignore, commit/push, then recreate your venv.

@ai-christianson  i just seen ur comment my bad

I got this initially. It was because I had the wrong API key set for OPENAI (but correct one for anthropic). Once I fixed it, I didn't see the error anymore. Might be good to have some kind of validation of the API credentials and give the user a clear message on that.

Once I got the API issue fixed, I was able to submit a query and get a basic response. It seems to not be spawning any agent activity yet, not sure if that's expected.

the CiaynAgent integration needs to be properly handled in the WebUI


Lazy Load Images?

Is there any way to only load the images once someone actually clicks a thumbnail?
Currently every image is loaded as soon as the page loads, which can make it quite a heavy page if you want to have an optional gallery only appear when someone clicks.
Thanks!

I see you added this feature but kept the issue open, does it work? Thanks!

@shaunrobinson Yes, just have some other changes to release alongside it. Will hopefully get chance to publish the new version tomorrow

Thanks, works great!


Argument error in foreach call in /bedrock/web/index.php

We are seeing these consistently throughout the day on the production environment.
E_WARNING
/index.php

10:08 am

TIME

5f06f300e02d(MoJ Intranet Production)

INSTANCE

1

COUNT

Previous error
18 similar errors to this
in the last 30 minutes
Next error
Error message
E_WARNING: Invalid argument supplied for foreach()

Stack trace
…_index called at /bedrock/web/app/themes/mojintranet/modules/api/
page_tree.php (81)
…_index called at /bedrock/web/app/themes/mojintranet/modules/api/
page_tree.php (32)
…:route called at /bedrock/web/app/themes/mojintranet/modules/api/
page_tree.php (11)
…ree_API::__construct called at /bedrock/web/app/themes/mojintranet/
service.php (23)
in Service::load_api called at /bedrock/web/app/themes/mojintranet/service.php (12)
in Service::main called at /bedrock/web/app/plugins/dw-mvc/Controller.php (57)
in MVC_controller::run called at /bedrock/web/app/plugins/dw-mvc/dw-mvc.php (100)
in DW_MVC::action_route called at /bedrock/web/wp/wp-includes/class-wp-hook.php (286)
…WP_Hook::apply_filters called at /bedrock/web/wp/wp-includes/
class-wp-hook.php (310)
in WP_Hook::do_action called at /bedrock/web/wp/wp-includes/plugin.php (515)
in do_action_ref_array called at /bedrock/web/wp/wp-includes/class-wp.php (726)
in WP::main called at /bedrock/web/wp/wp-includes/functions.php (960)
in wp called at /bedrock/web/wp/wp-blog-header.php (16)
in require called at /bedrock/web/index.php (5)

Closed by https://github.com/ministryofjustice/intranet/commit/705578fb54138be8280b0c8a4b64ea7a78bbfbc2#diff-83109c346e8390c8130fa3b3e72c4127


Client Installation Failure

When I attempt to run the first line from the readme setup script:
source <(curl -sSL https://raw.githubusercontent.com/labdao/plex/main/install.sh)
I get the error curl: (23) Failed writing body

Thanks @MylesGearon. We currently don't support plex binaries runnign on i386 chips, but this is a good push for us to update the goreleaser workflow and install script

Solved with #367


Option to disable breakpoint candidates inline

There was a feature request to disablie the grey dots that appear within the text of breakpoint. I first noticed the dots beginning with the release Version: 1.39.2. I think this feature needs to be escalated because the breakpoint dots have become a total nuisance.

Dup of https://github.com/microsoft/vscode/issues/83018


Exclude subgraphs from processing with an argument

fixes #38

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.2%) to 12.658%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

autoagora/main.py
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

autoagora/model_builder.py
1
0%

autoagora/price_multiplier.py
1
0%

autoagora/main.py
2
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
40

Relevant Lines:
316

💛  - Coveralls


Create documentation for all the stats_* functions

Reported by karl smith on 13 Jun 2011 17:47 UTC
All the python external functions created from scipy.stats routines need more complete documentation that what is given from the show function stats* in ferret7.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1858

Comment by karl.smith on 30 May 2012 18:43 UTC
Documentation is now given at
http://ferret.pmel.noaa.gov/Ferret/documentation/pyferret/ferret-functionality/
Leaving this open for now since better descriptions would be desirable.  What is there is essentially the help from SHOW FUNC STATS*

Comment by karl.smith on 21 Nov 2012 00:10 UTC
Closing this out.  More complete documentation probably would be better, but applies to other areas as well, and probably should fall under an update and improve documentation ticket.


@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/cuquantum-python-feedstock#14.


Get rid of userspace proxying

Determine the master IP via ssh over eth1. This allows nodes to find out
the IP of eth0 of the master.
Background is, that DNS probes from nodes to resolve the master IP, in some libvirt/vagrant combinations point to wrong IPs. Therefore, use the well known IP of eth1, to connect  to the master, and get the IP of eth0 from the master directly. This allows kubeadm on the node, to use the right IP to connect to the master. If we don't do that, we have to mess around with Kubernetes and the netork providers, to use eth1, which is error prone, and some people are still seeing deployment issues.

Another attempt to solve #43 for everyone.

Deployment and functional tests work for me.

We also have a confirmation from @cynepco3hahue that this fixes his setup.


Feature request: allow use of host property in @Component if values are strings

I would like a way to allow using the host property in component/directive decorators as long as the keys are being set to string values. For example, this code:
@Component({
  selector: 'my-component',
  host: {
    class: 'some-base-class some-modifier-class other-modifier-class'
  }
})
class MyComponent {}

seems more readable than:
@Component({
  selector: 'my-component'
})
class MyComponent {
  @HostBinding('class.some-base-class')
  someBaseClass = true;

  @HostBinding('class.some-modifier-class')
  someModifierClass = true;

  @HostBinding('class.other-modifier-class')
  otherModifierClass = true;
}

@eppsilon would you open an issue for this in the angular.io repository. Here's a template issue https://github.com/angular/angular.io/issues/1301.

Closing it for now. @eppsilon let me know if the style guidelines change if I don't notice.

@mgechev Seeing that the styleguide issue is closed, would there be any chance we could make this configurable in some way? By adding some 'allow-constant-values' setting to the rule?

I'd prefer to collect some more votes here.

It's super annoying to have to create a boolean set to true, just to bind a static class to a component.  It's so much cleaner to use the host property for a static string binding.  Really feels like I'm fighting the compiler to do something "the right way" and the compiler is forcing me to do it "the wrong way".

I disabled this lint rule since there is 0 argument for avoiding host when binding literals. I often bind tabindex or aria-* o role up there. It makes much more sense than polluting the class with public stuff all over the place just to bind to the host. If we could have what the issue suggests, it would be useful again.

I think the option should be considered. I understand this is prefer, but in the case I have a value I know it won't ever change, having Angular checking if it did change in the check detection cycle seems like a waste of resources for me.


Multiple people have had issues with UI flickering

Multiple people have had issues with UI flickering, it's visible in this https://www.youtube.com/watch?v=4RsbNPE6saA video
Originally posted by @ShadowBrian in https://github.com/apetavern/sbox-fortwars/issues/12#issuecomment-1036732468

I don't know what causes this and everyone seems to have had a different flickering experience, so I can't really do much about this right now


Error: Unexpected Token

When I create a graphql file, there's a rendering in my app js file like this:
const doc = {
  "kind": "Document",
  "definitions": [
    {
      "kind": "OperationDefinition",
      "operation": "query",
      "name": {
        "kind": "Name",
        "value": "author"
      },
      "variableDefinitions": [
        {
          "kind": "VariableDefinition",
          "variable": {
            "kind": "Variable",
            "name": {
              "kind": "Name",
              "value": "id"
            }
          },
          "type": {
            "kind": "NonNullType",
            "type": {
              "kind": "NamedType",
              "name": {
                "kind": "Name",
                "value": "String"
              }
            }
          },
          "defaultValue": null
        }
      ],
      "directives": [],
      "selectionSet": {
        "kind": "SelectionSet",
        "selections": [
          {
            "kind": "Field",
            "alias": null,
            "name": {
              "kind": "Name",
              "value": "author"
            },
            "arguments": [
              {
                "kind": "Argument",
                "name": {
                  "kind": "Name",
                  "value": "id"
                },
                "value": {
                  "kind": "Variable",
                  "name": {
                    "kind": "Name",
                    "value": "id"
                  }
                }
              }
            ],
            "directives": [],
            "selectionSet": {
              "kind": "SelectionSet",
              "selections": [
                {
                  "kind": "Field",
                  "alias": null,
                  "name": {
                    "kind": "Name",
                    "value": "firstName"
                  },
                  "arguments": [],
                  "directives": [],
                  "selectionSet": null
                }
              ]
            }
          }
        ]
      }
    }
  ],
  "loc": {
    "start": 0,
    "end": 69
  }
};
export default doc;

The browser blows up on each export default doc rendered for every graphql file. This rendering seems unusual.
Following your examples, I have my graphql queries as such:
query author($id: String!) {
  author(id: $id) {
    firstName
    lastName
  }
}

Removing the graphql files removes the issue of course. As your docs state, best practice is to put your queries in gql/queries/ rather than the pod structure.
I've tried with the following scenarios:
Scenario 1
ember-cli: 3.0.0
node: v9.8.0
npm: v5.8.0

Scenario 2
ember-cli: 2.8.0
node: v8.9.4
npm: v5.6.0

Scenario 3
ember-cli: 2.8.0
node: v6.6.0
npm: v3.10.3

I solved this not following your examples.

@mdbiscan can you say how and/or send a PR to fix whatever examples might be broken?


warning when compiled with clang 11

/n1/env_centos/7.6/include/csv.hpp:6313:20: warning: explicitly defaulted move assignment operator is implicitly deleted [-Wdefaulted-function-deleted]
[build]         CSVReader& operator=(CSVReader&& other) = default;
[build]                    ^
[build] /n1/env_centos/7.6/include/csv.hpp:6379:23: note: move assignment operator of 'CSVReader' is implicitly deleted because field 'records' has a deleted move assignment operator
[build]         RowCollection records = RowCollection(100);
[build]                       ^
[build] /n1/env_centos/7.6/include/csv.hpp:5984:24: note: copy assignment operator of 'ThreadSafeDeque<csv::CSVRow>' is implicitly deleted because field '_lock' has a deleted copy assignment operator
[build]             std::mutex _lock;
[build]                        ^
[build] /opt/rh/devtoolset-9/root/usr/lib/gcc/x86_64-redhat-linux/9/../../../../include/c++/9/bits/std_mutex.h:95:12: note: 'operator=' has been explicitly marked deleted here
[build]     mutex& operator=(const mutex&) = delete;

Does it matter?

Sorry. Opened in wrong repo.


removeChorus regex is not working (?) here's a fix

Hi, great lib, very easy to use and the doc is very nice too!
Maybe I'm mistaken but the removeChorus option doesn't work? I tried it and it didn't have any effect.
Checking the line removing the chorus there
https://github.com/zyrouge/node-genius-lyrics/blob/1d8a8bbe4ef73ed4c06fdf400f7adadf97ac1414/lib/songs/song.ts#L127
the regex matching the chorus there is:
/^\[[^\]]+\]$/g
This doesn't seem to match anything (testing in regex101 with lyrics from Genius).
After a few tries, I found that this would seems to be a good regex to apply:
/^\[(?:Chorus|Post-Chorus|Pre-Chorus)[\s\S]*?(?:\n\n|(?=[\n]?$(?![\n])))/gm
It matches Chorus, Post-Chorus and Pre-Chorus blocks. I tried replacing the one in the code with this, and it actually stripes the Chorus blocks.
This part : (?=[\n]?$(?![\n])) is needed to match a Chorus|Post-Chorus|Pre-Chorus block that is at the very end of the lyrics (without a blank line after it).
A great option would be to be able to keep one chorus (and post-chorus etc.) but remove the duplicates, maybe just replacing them by [Chorus] only. This could probably be done in Regex too, but I guess doing it using string manipulation in JavaScript might be more readable and more flexible.

Fixed as of v4.4.1.


LSLAutobids TODO

[x] Handle the memory spike problem with the BrainVision Format
[ ] Handle negative voltage issue - only with some recordings
[ ] Writing tests with a sample recording
[ ] Triple push issue to DaRUS (dataverse)

EDF is fine, we don't particulary care about BrainVision - I thnk both are BIDS compatible
negative voltage issue? what was that?
[ ] fix the TSV/CSV issue that dissallows upload CSV/TSV via dataverse https://github.com/datalad/datalad-dataverse/issues/307 somehow...

@behinger the negative onset issue is this one : https://github.com/bids-standard/pybv/issues/110
It is not yet resolved.

Yes EDF fixes it now. I am marking the TODO as done!


Add az appservice and az functionapp references to existing Azure CLI reference summary

This work is for story 1829866.

Docs Build status updates of commit 5e9cb87:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-reference-for-webapps.md
:bulb:Suggestion
View
Details

docs-ref-conceptual/azure-cli-reference-for-webapps.md

Line 84, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 16affd4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-reference-for-webapps.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

FYI @dbradish-microsoft

Docs Build status updates of commit 440f5dc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-cli-reference-for-hosted-apps.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


search endpoint failure correction

when returning an invalid search, a return was omitted triggering a null on the consumer end.
Fixes: #5228
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

/approve
LGTM

/hold
/lgtm
Cancel hold once tests pass.

LGTM but the tests are fighting you.

Same thing: #5282. Needs rebase against current master.
I'm sorry, this was just really rotten timing: two simultaneous PRs that just happened to land within hours of each other without a git conflict but causing a real test conflict.

rebased

Restarting failed tests

ARGH. It's the fedora registry flake which I thought I had fixed. I don't see my "sleeping & retrying" messages and am really confused.
I've restarted the two fedora-registry flakes and am looking into the third (podman diff) error.

/lgtm
/hold cancel


docs improvement tips

Hi,
I like the improved documentation, but since we can always do better, here goes:

on the docs index page I think the 'Table of Contents' in the top right is not meaningful at the moment, with only two main H1-entries and one entry that should better be indented (the license). My advice would be to rename it to "Page Contents" to convey better to the reader that it's about different content than what is shown on the left of the page in the 'Guide contents' widget below the search box.

on that same page, the Appendix promises us to contain licensing information, which indeed is a subsection, and an enumeration of all the different (i.e. various) modules, classes, functions, etc. However, this enumeration is missing.

I think it would be clearer to have some white lines between the link to the tutorial "A simple Paint App" and "API Reference" to make visually the same separation as you intended with the separate "Guides" and "API Reference" links in the top right.

the top right also contains a link "PDF" whose analogon is missing in the list of links (my advice: add a link to "PDF" a few white lines below the link "Weak Method"

You recommend to head over to the Programming Guide after having got started. But this advice is not congruent with the contents of the 'Diving in' page of the Getting Started guide.
Moreover: why is the "User's Guide" listed in between the Getting Started guide and the Programming Guide in the widget on the left? My advice: remove the sentence "Then head over to the Programming Guide".

Then again, the first line of 'Diving in' wants me to get further into kivy by just going back to the index page. Not helpful.

The section "A first App" in the Getting Started guide does not hold up to its promise. It has a link to one of the tutorials, in which you pose as prerequisite knowledge how to run a basic Kivy application (and point to the paragraph 'Create an application' on the 'Kivy Basics' page of the Programming Guide.) But running an application (as explained in the Installation pages) is not the same as creating an application.
Now really, the "Create an application" paragraph should be the meat of the page "A first App". Or alternatively, the application that is found at the bottom of Kivy's home page as the "Usage example" (displaying the "Hello World" button). And in the programming guide we should refer back to this First App and give the background information and customization ideas.

There are about three different installation pages (for sake of brevity, I count all the installation- pages as just one): 1 in the getting started guide, 1 in the user's guide, 1 section in the 'kivy basics' page of the programming guide, and many of them to be found via the downloads section of the kivy website. My advice would be to create a separate "Installation Guide" and refer to this guide from all the above named locations.

The Quick search box would be more obvious with some magnifying glass icon next to it. And, I feel deprived of a Slow search box ;-)

I hope you find these remarks helpful, sometimes it's hard to decide whether an issue is purely personal (and I should just live with it) or whether others might benefit from them as well.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

See also #1749


Missing items: Serpent-Coil Braid

Links to WOTLK Wowhead for each item:
Serpent-Coil Braid

Duplicate of #1399


Allow mapping array values, not just references

This enables operations like [vec![1, 2, 3], vec![4]].map(|v| v.into_boxed_slice()), consuming the old array without copying.
Note: if a is an array value instead of a reference, you may now need to write (&a).map() to get the previous behavior of a.map().

I'm wary of introducing such a breaking change, on the other hand I like the idea of allowing to consume the old array.

I'd love to have benchmarks
Could we rename the consuming map function, for example map_into?


Proof failure not reported correctly

For isaplanner/prop_06.smt2: the property cannot be proven, but summary of proof is still printed, and exit code is 0.

Also reproducible upstream, see danr/emna#3


Need to sort by seeds

Hi, I need to sort by seed all torrent result data, it's possible? Yts result have no seed data
Just check rarbg not supported to search by imdb id, while the rarbg search website can be found by imdb id. Is the search url is different than the search on website?

I will try to implement sort by seeds (this can be implemented on sites which allows sort by seeds / shows seeders)
and search by imdb id is working on torrent galaxy  but its not working on rarbg site
can u please share a working imdb id on rarbg site.
YTS doesn't show seeders / leechers of their torrent.


Row is not cleared during app restart or app goes down.

Hi Lucas, thanks for implementing this lock! I just tried using it(version:0.18.2) in my spring boot app with MS SQL server DB. I want to understand how the lock should behave in the event if the app is redeployed or server node goes down. In such cases, row in the table still exists though the task was already complete. When the app comes back up, it tries to INSERT row one more time and it fails till deleting the row manually.
What is the expected behavior? Have I missed any thing in the implementation?
@SchedulerLock(name = "someUniqueTaskName", lockAtMostFor = 60000)

Hi, can you please describe how the app fails? It always tries to insert the row (and if you have debug log on you can see an error), but if the insert fails it's handled and the row is updated.
The expected behavior is that the row row is not deleted when the tasks ends, but the value in lock_until is set to 'now'.  If the process dies in the middle of the task processing, the lock is held at most for '60000'. What are you describing is fairly common use-case so I guess that the problem is in your app.

App works fine when it is up and running after inserting the first lock. It keeps updating lock_until information. When I deploy the app using CI tool like Jenkins, Shedlock tries to insert the row again as it may not find any thing in the lock cache. But the row in DB is still there and it produces duplicate row error. In this case none of the nodes in the app server start the scheduler. That is my problem.

OK, lets' dive in the code. JDBC access is handled by AbstractJdbcStorageAccessor If the insertRecord method returns false, update is attempted. Now it depends on the implementation you use. If you are using jdbc-template provider we are depending on Spring to correctly resolve duplicate key exception as SQLIntegrityConstraintViolationException. If it does not happen, ShedLock would rethrow the exception and fail to update the lock. In such case please try to switch to pure jdbc-lock-provider which catches all the exceptions so such thing can not happen. If it's indeed the case, we should file a bug to Spring issue tracker so they fix it on their side.

Hi Lucas, thanks for the insight! Indeed it was JdbcStorageAccessor. I extended AbstractJdbcStorageAccessor to my own class similar to JdbcTemplateStorageAccessor just to view exceptions and log them. But during the process I did not suppress DataIntegrityViolationException as done in handleInsertionException() of JdbcTemplateStorageAccessor. Once I did the exact change in my class, locks are being updated. I sincerely appreciate your quick responses. Thank you so much!!

Next time please mention that you are using your own implementation in the original error report. For failing to do that, you are herby fined to send $10 to EFF or MSF


refactor: js plugin hooks

Summary

Checklist

[x] Tests updated (or not required).
[x] Documentation updated (or not required).

!bench

!bench