Start DateInput

Bug: T273002

This is really great! 💯
There's a Reference error that prevents the Basic story to be displayed in Chromatic. Adding some comments here after checking the components in the All story:

For some reason, the IntroText style is not being applied to the default prompt:

The variants in the All page should have some more vertical space between them (see the All story of the ExtendedNumberInput for reference). I believe that the spacing is achieved by containing the variants in paragraphs.

In the design specification, it was indicated that text elements should be contained in 24px high "bounding boxes" spaced by 4px. Checking the current implementation, I believe that probably that's not necessary, but that definitely the vertical spacing between elements should be increased to 8px (dimension.spacing.small.value). I wasn't sure where to suggest the change

Charlie made a good point regarding the default active state of the DateInput component that made me think the interaction flow could be improved in the following way:
Current: Once the user focuses on the DateInput, the extender menu opens to display the prompt text.
Desired: Once the user focuses on the DateInput, instead of just the prompt, users also see the resultsIntroText. In the following way:

Since the component is still in the last stages of development, I thought we still are on time to make this improvement 🙏

This is really great! 100
There's a Reference error that prevents the Basic story from being displayed in Chromatic. Adding some comments here after checking the components in the All story:
* For some reason, the IntroText style is not being applied to the default prompt in Chrome & Safari on MacOs (the style looks good in Firefox/Linux, though):

* The variants in the All page should have some more vertical space between them (see the All story of the ExtendedNumberInput for reference). I believe that the spacing is achieved by containing the variants in paragraphs (which is probably also not ideal).

Added <br> in between.

* In the design specification, it was indicated that text elements inside the Extender menu should be contained in 24px high "bounding boxes" (aka divs) verticall-spaced by 4px. Checking the current implementation, I'm not sure if that's applied or even if it's necessary, but definitely the vertical spacing between elements should be increased, to probably 8px (dimension.spacing.small.value). I wasn't sure where to suggest the change

Isn't it $wikit-InputExtender-padding-vertical?

This still needs a line in the CHANGELOG.md, I think :)


bau: Make epdq order classes singletons

Where subclasses of EpdqPayloadDefinition have no state, these can be made singletons
WHAT YOU DID
A brief description of the pull request:
How to test

How should it be reviewed?
What feedback do you need?
If manual testing is needed, give suggested testing steps.

Code review checklist
Logging

[ ] only emit log lines at ERROR level which require immediate attention from a support engineer. These will trigger a zendesk alert.

Documentation

[ ] Updated README.md for any of the following ?

Introduced any new environment variables / removed existing environment variable
Added new API / updated existing API definition

Ugh 🤦 you're right

@alexbishop1 updated


Update requirements.txt

torchvision is required for the gradio app demo to work

+1 faced the same problem of torchvision being missing!


Add manual overrides for container user namespaces.

Support manual overrides of the container user namespace via the CROSS_CONTAINER_USER_NAMESPACE environment variable. If not set or set to auto, it will use the default value for the container engine (host). If none is provided, no --userns flag will be used. If any other value is provided, that will be the value passed to --userns.
This is required for using lima/nerdctl, which currently does not support the --userns flag.
Related to #888.

I don't know if we should explicitly support this. Maybe instead it should be more that the ContainerEngine is custom (i.e any random binary)

I don't know if we should explicitly support this. Maybe instead it should be more that the ContainerEngine is custom (i.e any random binary)

Maybe, we should probably at least support changing the user namespace, which would be necessary for lima support right now. But you're probably right: there's too many niche things here that can be covered in a more general case. Just keep the user namespace manual overrides, get rid of everything else?

Just keep the user namespace manual overrides, get rid of everything else?

yep, sounds good to me

bors r=Emilgardis


Image color changes when processed by sharp

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?
  System:
    OS: macOS 13.4.1
    CPU: (8) arm64 Apple M2
    Memory: 162.55 MB / 8.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.16.1 - /opt/homebrew/opt/node@18/bin/node
    npm: 9.5.1 - /opt/homebrew/opt/node@18/bin/npm
  npmPackages:
    sharp: 0.32.1 => 0.32.1

What are the steps to reproduce?
Process a sample image with sharp and it will change its color especially red.
What is the expected behaviour?
Check the red color. It should not change.
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem
import sharp from "sharp";

let transformer = sharp("./input.jpeg").toFile("./output.jpeg",() =>{});

Please provide sample image(s) that help explain this problem
Linked here.

The second image uses 4:2:0 subsampling:
$ identify -verbose 249775478-6037f8f7-ac03-4376-b7ce-569e11dedbd0.jpeg | grep JPEG
  Format: JPEG (Joint Photographic Experts Group JFIF format)
  Compression: JPEG
  JPEG-Quality: 80
  JPEG-Colorspace: 2
  JPEG-Colorspace-Name: RGB
  JPEG-Sampling-factors: 2x2,1x1,1x1

whereas the first image does not:
$ identify -verbose 249775446-066c23cd-2229-41c7-a2bf-48d821b580fb.jpeg | grep JPEG
  Format: JPEG (Joint Photographic Experts Group JFIF format)
  Compression: JPEG
  JPEG-Quality: 86
  JPEG-Colorspace: 2
  JPEG-Colorspace-Name: RGB
  JPEG-Sampling-factors: 1x1,1x1,1x1

Anyone any luck?

Hi @lovell any luck with this? If you can provide some workaround, it would be best. One of our customer is stuck because of this.

In addition to the use of chroma subsampling, the first image contains an embedded P3 RGB profile and the second image is profile-less sRGB. Do you only see a "difference" on a wide-gamut display?
Use withMetadata to specify an output profile, plus experiment using pipelineColourspace with a 16 bpp colourspace when working with wide-gamut images.

Thanks!! This worked. Yes we can re-produce this only on wide-gamut display.
We now have to identify input image based on ICC profile and then make sure we process them correctly.


Adds render cross reference in links

This PR Adds render cross reference in links and fix #2

@emmanuelmathot this makes sense to me but I wonder if this spec should be in the web-map-links extension rather than here, since the change is in the links object, not the renders object.

@emmanuelmathot this makes sense to me but I wonder if this spec should be in the web-map-links extension rather than here, since the change is in the links object, not the renders object.

The change specifies an extension to link but specific to render so this should be set here. If the logic discussed here: https://github.com/radiantearth/stac-spec/issues/1284 similar to this proposition changes, we will adapt to the logic but for the moment, we will add the possibility to link a render in a link.


Upgrade cdk

Good morning,
it would be nice to upgrade the cdk to 214 so we can disable standby replicas in the OpenSearch vector db. This will help to save some costs during development:
https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_opensearchserverless.CfnCollection.html
https://aws.amazon.com/blogs/aws/vector-engine-for-amazon-opensearch-serverless-is-now-generally-available/

#266 Merged


Docs: Disabling default appender

In the docs:

When running in an environment where local file logging is not available, or to completely replace the file logger, disable the default rails file logging by setting:
config.rails_semantic_logger.add_file_appender = true

I believe this is supposed to be:
config.rails_semantic_logger.add_file_appender = false

Good catch, want to submit a pull request against the gh-pages branch with this fix?

Fixed with https://github.com/rocketjob/semantic_logger/commit/77679f1b1e9c16c7ac2825cae9e17846deaeb59a
Thank you for the feedback.


Can't access private group

I filled in user, API token but couldn't change to private group. Refreshing doesn't work.

It looks like you are authenticated. Can use use the group dropdown to select a private group?

No. I can only see All and Public

Hmm, sorry, I'm a bit stumped.
You don't need the usename, that's only for search not authentication, the token alone provides authentication.
Have you used this token elsewhere successfully?
All I can think of now is:

try a different browser

regenerate the token (if you're not depending on it to be valid elsewhere)

Sorry! If you do figure it out please let me know.

Thank you. I regenerated the token and it works now.

I'm puzzled as to how this happened. The tokens do not expire, I've been using mine for years. But glad to know you're unblocked.


[Snyk] Upgrade: react, react-dom, react-test-renderer

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade multiple dependencies.
👯♂ The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

reactfrom 18.0.0 to 18.2.0
48 versions ahead of your current version
4 months agoon 2022-06-14

react-domfrom 18.0.0 to 18.2.0
48 versions ahead of your current version
4 months agoon 2022-06-14

react-test-rendererfrom 18.0.0 to 18.2.0
48 versions ahead of your current version
4 months agoon 2022-06-14

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Base: 94.02% // Head: 94.02% // No change to project coverage :thumbsup:

Coverage data is based on head (b9e7675) compared to base (7db429a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #220   +/-   ##
=========================================
  Coverage     94.02%   94.02%
  Complexity      475      475
=========================================
  Files            96       96
  Lines          1607     1607
  Branches        122      122
=========================================
  Hits           1511     1511
  Misses           56       56
  Partials         40       40

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Require ContentHandlerError to implement Send and Sync

I tried using rewrite_str with failure, but it doesn't implement Send and Sync:
error[E0277]: `(dyn std::error::Error + 'static)` cannot be sent between threads safely
   --> src/utils/html.rs:47:5
    |
47  |     lol_html::rewrite_str(html, settings).map_err(Into::into)
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `(dyn std::error::Error + 'static)` cannot be sent between threads safely
    |
   ::: /home/joshua/.local/lib/rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libcore/convert/mod.rs:288:1
    |
288 | pub trait Into<T>: Sized {
    | ------------------------ required by this bound in `std::convert::Into::into`
    |
    = help: the trait `std::marker::Send` is not implemented for `(dyn std::error::Error + 'static)`
    = note: required because of the requirements on the impl of `std::marker::Send` for `std::ptr::Unique<(dyn std::error::Error + 'static)>`
    = note: required because it appears within the type `std::boxed::Box<(dyn std::error::Error + 'static)>`
    = note: required because it appears within the type `lol_html::rewriter::RewritingError`
    = note: required because of the requirements on the impl of `failure::Fail` for `lol_html::rewriter::RewritingError`
    = note: required because of the requirements on the impl of `std::convert::From<lol_html::rewriter::RewritingError>` for `failure::error::Error`
    = note: required because of the requirements on the impl of `std::convert::Into<failure::error::Error>` for `lol_html::rewriter::RewritingError`

error[E0277]: `(dyn std::error::Error + 'static)` cannot be shared between threads safely
   --> src/utils/html.rs:47:5
    |
47  |     lol_html::rewrite_str(html, settings).map_err(Into::into)
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `(dyn std::error::Error + 'static)` cannot be shared between threads safely
    |
   ::: /home/joshua/.local/lib/rustup/toolchains/stable-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libcore/convert/mod.rs:288:1
    |
288 | pub trait Into<T>: Sized {
    | ------------------------ required by this bound in `std::convert::Into::into`
    |
    = help: the trait `std::marker::Sync` is not implemented for `(dyn std::error::Error + 'static)`
    = note: required because of the requirements on the impl of `std::marker::Sync` for `std::ptr::Unique<(dyn std::error::Error + 'static)>`
    = note: required because it appears within the type `std::boxed::Box<(dyn std::error::Error + 'static)>`
    = note: required because it appears within the type `lol_html::rewriter::RewritingError`
    = note: required because of the requirements on the impl of `failure::Fail` for `lol_html::rewriter::RewritingError`
    = note: required because of the requirements on the impl of `std::convert::From<lol_html::rewriter::RewritingError>` for `failure::error::Error`
    = note: required because of the requirements on the impl of `std::convert::Into<failure::error::Error>` for `lol_html::rewriter::RewritingError`

error: aborting due to 2 previous errors

failure has a workaround for missing Sync (i.e. a mutex), but not for missing Send. It would be great if ContentHandlerError implemented both. It looks like this is a user-provided error, so this would be a breaking change, but I think the increase in flexibility is worth it.
(Yes, I'm aware failure is deprecated, but docs.rs won't be switching any time soon and this is useful even without failure)

Hi, thanks for the reporting. I believe it's reasonable to add Send trait bound on the custom error type. Though, I believe that adding Sync would be a bit too restrictive (I'm open to be convinced otherwise).

I believe that adding Sync would be a bit too restrictive (I'm open to be convinced otherwise).

If the error has to implement Sync but doesn't, the user can always wrap it in a mutex, the way that failure does. If ContentHandlerError doesn't implement Sync then you have to wrap it in a mutex even if you know that it's thread safe. So it hurts performance (and can lead to deadlocks if the inner error uses a mutex).

I guess if you never use lol_html in a threaded context it would make you add a mutex for no benefit, which isn't ideal. But mutexes are very fast if there is no contention and it would also help to refactor to add threads in the future.

As another point of information, the Rust api guidelines suggest that errors always implement Send and Sync: https://rust-lang.github.io/api-guidelines/interoperability.html#error-types-are-meaningful-and-well-behaved-c-good-err


Update scalatest to 3.1.2

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

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

labels: test-library-update, semver-patch

Coverage remained the same at 100.0% when pulling 311edd1eadfa684b9f8bd4d7fbdda1008dc30643 on scala-steward:update/scalatest-3.1.2 into 6de5bd57f79172b56b7170a8f692dfb4490cbd1d on sangria-graphql:master.

There is already a newer version of https://github.com/sangria-graphql/sangria-rxscala/pull/11


SimpleITK error

I exactly created nifti file from pngs like this video: https://www.youtube.com/watch?v=RkzsgS-sGFw
Then I tried to convert this to dicom then I get this error. https://prnt.sc/EkHk8c_ekzrC
Additionally, After created dicom slices how can I connect them and get 1 dicom file. (Like an MRI)

Same error here. Anyone knows why?

Add this line and it should be fine: https://github.com/amine0110/Medical-Conversions/blob/main/functions.py#L58

You can check our latest library that handles most of the issues you can face with this repo.
Here is the link to the library: https://github.com/amine0110/pycad


Settings hung, then crashed on a mouse click, SysTray icon became unresponsive to any click

Microsoft PowerToys version
0.69.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Settings
Steps to reproduce
Cannot reproduce. I had gone into settings, turned off Quick Accent, and sent focus elsewhere. After a few minutes, I had second thoughts and went to the tray and left-clocked on the icon. No response. I tried right-click, no response. I then noticed that the settings window was open but in Not Responding. I took a dump from ProcExp, which is attached to this ticket (multi-part zip due to Github restrictions, swap the zip and 00n in the file name). The crash report is also attached.
PowerToysReport_2023-05-22-09-31-28.zip
PowerToys 20230522-0921.003.zip
PowerToys 20230522-0921.001.zip
PowerToys 20230522-0921.002.zip
✔ Expected Behavior
A working Settings window.
❌ Actual Behavior
Settings window was non-responsive.
Other Software
No response

was this one-time occurrence? or it happens from time to time? Could you update PowerToys to latest version and see if it happens there as well ? /needinfo

Could you try steps mentioned here https://github.com/microsoft/PowerToys/issues/27126#issuecomment-1649445254


Question in the inference

the required spectrogram form is like [N,C,W].
spectrogram = # get your hands on a spectrogram in [N,C,W] format
could you please explain these three dimensions?
I use the code from this repo: https://github.com/CorentinJ/Real-Time-Voice-Cloning  to produce the mel spectrogram and use diffwave as the vocoder. But I only get the audio full of noises.
generate mel spectrogram
specs = synthesizer.synthesize_spectrograms(texts, embeds)  #len(specs) == 1
spec = specs[0]   #spec numpy.array, float32, shape(80, 314)
spec = torch.tensor(spec)
Generating the waveform
diffwave_dir = "/hdd/haoran_project/diffwave-master/pretrained_models/diffwave-ljspeech-22kHz-1000578.pt"
generated_wav, sample_rate = diffwave_predict(spec, diffwave_dir, fast_sampling=True)
Save it on the disk
filename = "results/diffwave_Elon.wav"
print(generated_wav.dtype, "  ",generated_wav.shape) # torch.float32    torch.Size([1, 87040])
torchaudio.save(filename, generated_wav.cpu(), sample_rate=sample_rate)

N = batch size
C = number of channels (number of mel dimensions)
W = width or time dimension

Thanks you Sharvil, your answer is handy and I can accomplish the high quality of audio to mel then to audio.
But, I still face the problem of voice cloning, seem like the vocoder supplied by this repo https://github.com/CorentinJ/Real-Time-Voice-Cloning and proceed mel spectrogram is not fully match with the input of diifwave, so I get a low-quality generated speech.
Is there any good text-to-speech vocoder that can meet the mel spec requirement of input of diffwave?

Thanks you Sharvil, your answer is handy and I can accomplish the high quality of audio to mel then to audio.
It seem like the preprocess.py file try do some normalize to the mel spec data, like:
spectrogram = 20 * torch.log10(torch.clamp(spectrogram, min=1e-5)) - 20
spectrogram = torch.clamp((spectrogram + 100) / 100, 0.0, 1.0)

But for different kind of mel spec how to make sure these nomalization  hyper-parameter?


Support sampling from model in model

We should later support something like this.
@model a(...) = begin
    # code
end

@model b(...) = begin
    x ~ a(...)
    # other code
end

This is a really nice feature. I have been thinking about this; it may require some substantial work on the internals. Let's spend sometime think about how this might be designed.

I guess the internal are good enough for us the work on this now?


fix(interactive): Support Regex Match for String Values

What do these changes do?
as titled.

Related issue number

Fixes

Codecov Report

Merging #3286 (64e7e07) into main (314fe42) will decrease coverage by 24.17%.
Report is 5 commits behind head on main.
The diff coverage is 100.00%.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #3286       +/-   ##
===========================================
- Coverage   66.18%   42.02%   -24.17%
===========================================
  Files         101      101
  Lines       11013    11013
===========================================
- Hits         7289     4628     -2661
- Misses       3724     6385     +2661

Files
Coverage Δ

python/graphscope/config.py
75.29% <100.00%> (+5.09%)
:arrow_up:

...on/graphscope/tests/kubernetes/test_demo_script.py
66.66% <ø> (+66.66%)
:arrow_up:

... and 65 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


Update dug.es.resx

I review a couple of this values, some doesn't make sense, you should reach out to someone who speaks Spanish :)

Yeah sorry, I just translated them on Google Translate :sweat_smile:
These tweaks you made look good. I assume the ??? are where it wasnt clear what I meant a all? I can remove those then bring it in. Also yeah I should reach out to a Spanish speaker...

@lemurido When you opened this PR my CI did not support building from a forked repo, it should now. If you want to resubmit it, without the ??? I'll read them and merge it :)

Closed for lack of activity


It does not work with angular 8.0.0 and ivy enabled

when trying to build I got error, it is a stopper
WARNING in Entry point 'ng2-ace-editor' contains deep imports into 'brace/theme/monokai'. This is probably not a problem, but may cause the compilation of entry points to be out of order.

ERROR in node_modules/ng2-ace-editor/src/module.d.ts(1,22): error TS-996002: Appears in the NgModule.imports of AppModule, but could not be resolved to an NgModule class

I created a fork from this repo and upgraded it to Angular 9 , it should work
https://www.npmjs.com/package/ngx-ace-editor-wrapper
This is the exact same ng2-ace-editor just pumped to A9

Hello martinpagesaal,
we use the 'ng2-ace-editor' aswell but got the same problems mentioned above after updating to angular 9 and enabling ivy.
I tried to update to the 'ngx-ace-editor-wrapper' but I can't compile the project due to the following errors:
node_modules/ngx-ace-editor-wrapper/dist/src/directive.d.ts:37:18 - error TS2314: Generic type 'ɵɵFactoryDef' requires 2 type argument(s).
static ɵfac: i0.ɵɵFactoryDef;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/ngx-ace-editor-wrapper/dist/src/component.d.ts:49:18 - error TS2314: Generic type 'ɵɵFactoryDef' requires 2 type argument(s).
static ɵfac: i0.ɵɵFactoryDef;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/ngx-ace-editor-wrapper/dist/src/component.d.ts:50:18 - error TS2314: Generic type 'ɵɵComponentDefWithMeta' requires 7 type argument(s).
static ɵcmp: i0.ɵɵComponentDefWithMeta<AceEditorComponent, "ace-editor", never, { "style": "style"; "options": "options"; "readOnly": "readOnly"; "theme": "theme"; "mode": "mode"; "value": "value"; "text": "text"; "autoUpdateContent": "autoUpdateContent"; "durationBeforeCallback": "durationBeforeCallback"; }, { "textChanged": "textChanged"; "textChange": "textChange"; }, never>;

Same problem for me. I tried to update to the 'ngx-ace-editor-wrapper' but I can't compile the project due to the above mentioned errors.

looks like it should compile correctly upgrading to angular 9.1  https://stackoverflow.com/questions/60898133/angular-9-ivy-generic-type-ɵɵfactorydef-requires-2-type-arguments

looks like it should compile correctly upgrading to angular 9.1 https://stackoverflow.com/questions/60898133/angular-9-ivy-generic-type-ɵɵfactorydef-requires-2-type-arguments

I followed the steps from the thread, but I still get the same errors. The only way to compile the project for now is to set "skipLibCheck":true in the tsconfig.json.

Had the same issue with angular 10.0.5 today. Just in case someone else has the same problem: I fixed it by making the following changes to the code:
node_modules/ngx-ace-editor-wrapper/dist/src/component.d.ts:
static ɵfac: i0.ɵɵFactoryDef<AceEditorComponent, never>;
static ɵcmp: i0.ɵɵComponentDefWithMeta<AceEditorComponent, "ace-editor", never, { "style": "style"; "options": "options"; "readOnly": "readOnly"; "theme": "theme"; "mode": "mode"; "value": "value"; "text": "text"; "autoUpdateContent": "autoUpdateContent"; "durationBeforeCallback": "durationBeforeCallback"; }, { "textChanged": "textChanged"; "textChange": "textChange"; }, never, never>;

node_modules/ngx-ace-editor-wrapper/dist/src/directive.d.ts:
static ɵfac: i0.ɵɵFactoryDef<AceEditorDirective, never>;

Hi guys, I made it work for my project and didn't went deeper into it.
this is how I'm using it right now, this is part of my package.json hope it helps
"dependencies": {
    "@angular-devkit/schematics-cli": "^0.901.6",
    "@angular/animations": "9.0.1",
    "@angular/cdk": "9.0.0",
    "@angular/common": "9.0.1",
    "@angular/compiler": "9.0.1",
    "@angular/core": "9.0.1",
    "@angular/flex-layout": "^9.0.0-beta.29",
    "@angular/forms": "9.0.1",
    "@angular/material": "^9.0.0",
    "@angular/material-moment-adapter": "9.0.0",
    "@angular/platform-browser": "9.0.1",
    "@angular/platform-browser-dynamic": "9.0.1",
    "@angular/router": "9.0.1",
...
    "ace-builds": "1.4.3",
...
    "ngx-ace-editor-wrapper": "^9.0.1",
....
  },


Commit with pre-release

Note: the 810-hal beta has not yet been published due to #91.
CC @nrf-rs/nrf52 @bjc
If there are no complaints in the next 24 hours or so, I will plan to release this as-is (after removing the -beta1 suffix)

All HAL crates published. CC @hannobraun


Image and Description for Injuries

Currently, Injuries can be tracked by way of an Item which contains a name and an image. This works well enough, but I feel the functionality could be expanded upon greatly if it had an expandable description field similar to Talents, which could contain information on, especially in the case of Starship injuries, the effects of the damage and the difficulty to repair. Having that information able to be quickly dropped onto the sheet for easy reference would be a massive quality of life improvement.
Additionally, despite there being an image setting for the Injury Item, it does not display on the sheet. It'd be nice if it were to appear, whether simply for aesthetic reasons, or perhaps even to convey information through the image itself.
All in all, this minor change would make a huge difference for me, and my hope is that given the functionality I'm asking for already exists for Talents, it would be a relatively simple change to make. Thanks for your consideration!

In a similar vein, one of my players has asked if it was possible to add a description to the focuses, too.  I think this comes from some of the focuses in the supplement books having a description, and can help answer questions about whether a focus actually fits or not.

This has been added in release 1.2.1, in addition description boxes added to focuses in same release as per consent.

Needs addition of dropdown when clicked from the character sheet.

Ok, so we now have dropdowns for injuries / damage and focus as well as description boxes.
Does that fill the requirement?

Seems so, this covers everything in the request. Thanks!


JSONC has no icon

VSCode Version: 1.19.1
OS Version: Windows 10 Build 16299.125

Steps to Reproduce:

Create JSON file
Set file association to JSON with Comments
See this

Reproduces without extensions: Yes

duplicate of #40244


Add getting started documentation

@mit-mit Thanks for the review. Comments addressed in https://github.com/flutter/plugins/pull/554.


fix(strimzi-kafka-operator): Support creating namespace

What does this PR do?
Makes sure a Kubernetes namespace for the strimzi-kafka-operator add-on is being created if argocd_manage_add_ons = true.
Motivation

Resolves #1309

More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Yes, I have added a new example under examples to support my PR
[ ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[ ] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs require a new example and/or doc page. In general:

Use an existing example when possible to demonstrate a new addons usage
A new docs page under docs/add-ons/* is required for new a new addon

For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

After applying these changes, the namespace is created as expected:
$ terraform state list | grep strimz
module.kubernetes_addons.module.strimzi_kafka_operator[0].module.helm_addon.module.irsa[0].kubernetes_namespace_v1.irsa[0]

$ kubectl get namespace strimzi
NAME      STATUS   AGE
strimzi   Active   99m

As suggested in the linked issue, let's fix the problem in the add-ons repository instead: https://github.com/aws-samples/eks-blueprints-add-ons/pull/131


@storybook/addon-storyshots: Inconsistency in framework string for react-native.

I have been working on putting together a typescript definition file for the storyshots addon, and when tracking through options provided to functions in the code noticed that the 'framework' string for react-native changes from 'react-native' to 'rn' when the react-native loader is used.
Affected lines:
https://github.com/storybooks/storybook/blob/9b3f63b304c381db4457449c32d37f03ce8e0a9a/addons/storyshots/src/rn/loader.js#L6-L9
https://github.com/storybooks/storybook/blob/9b3f63b304c381db4457449c32d37f03ce8e0a9a/addons/storyshots/src/rn/loader.js#L18-L23
Is this string supposed to be different in the Context information to what is required in the initStoryshots function options? It looked like the change in value may have been an oversight from refactoring, so I figured I'd check, as I need to specify the correct list of values in both the Context and the Options framework values for the typescript type definitions.

'rn' is used internally only. It is used to prevent image snapshots to be called for rn.

The output from the load function is used when creating the context information that is passed to the options.test function, so it is leaking out into user code as well.
https://github.com/storybooks/storybook/blob/9b3f63b304c381db4457449c32d37f03ce8e0a9a/addons/storyshots/src/index.js#L39
https://github.com/storybooks/storybook/blob/9b3f63b304c381db4457449c32d37f03ce8e0a9a/addons/storyshots/src/index.js#L56
https://github.com/storybooks/storybook/blob/9b3f63b304c381db4457449c32d37f03ce8e0a9a/addons/storyshots/src/index.js#L82-L90
If the user does not set a test function but does set a custom renderer, it leaks through into the custom renderer via the context information that is sent to the default snapshotWithOptions test function.
https://github.com/storybooks/storybook/blob/9b3f63b304c381db4457449c32d37f03ce8e0a9a/addons/storyshots/src/test-bodies.js#L4-L10
Anywhere where the context value is passed to is receiving the 'rn' value rather than the initial 'react-native' value for the framework field in the context.

I see. So probably changing it back to the  'react-native' will be a good thing.


Remove Subsocial from Nuxt

remove subsocial packages and rest stuff which is connected to it
yarn remove @subsocial/api @subsocial/types @subsocial/utils

pls observe #892

Is there anything more that I need to do here


RFP - liftoff

Checklist

[x] The package I am requesting does not already exist on https://chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL :https://github.com/liftoff-app/liftoff/releases/tag/v0.10.7
Direct download URL for the software / installer : https://github.com/liftoff-app/liftoff/releases/download/v0.10.7/liftoff-v0.10.7-x86_64-windows.zip
Software summary / short description:
Windows client for Lemmy

I've picked this up and pushed a package to the community repository: https://community.chocolatey.org/packages/liftoff


DRAFT Add packageSettings

This is a follow-up to https://github.com/srid/haskell-flake/pull/87#issuecomment-1448903620 (and closes #87 and closes #96).
The module API is slightly diferrent here. While it still uses deferredModule (like #96), the config implementation of this PR expands directly to overrides, thus we can reuse the merge semantics of overrides without worrying about writing one for packageSettings. We also add input.hackageVer as an easier way of doing self.callHackage ....

[ ] Assess how overlay composition order is affected as a result of this
[ ] API is slightly clunky owing to what appears to be limitations of the module system (see diff comments)
[ ] Deprecate/remove source-overrides (since the input option can do that and more)

The name packageSettings may no longer be appropriate/accurate since this option can also specify custom packages from, say, flake inputs (see https://github.com/nammayatri/nammayatri/pull/37). It is pretty much looking like a friendly wrapper for overrides.

Assess how overlay composition order is affected as a result of this

Unfortunately, packageSettings take priority over overrides, irrespective of the order modules list is in.

Assess how overlay composition order is affected as a result of this

Unfortunately, packageSettings take priority (cf. #82) over overrides, irrespective of the order modules list is in.

Treating packageSettings as a distinct overlay solves this; because we control the order in composeManyExtensions: #99
Now we have no choice but to look into the semantics of merging of multiple packageSettings values.

Now we have no choice but to look into the semantics of merging of multiple packageSettings values.

Okay, we can't merge these modules (there will be errors like error: The option broken' has conflicting definition values`).

Closing per https://github.com/srid/haskell-flake/pull/87#issuecomment-1450286992


edge and vertex id processing

Coverage increased (+0.2%) to 43.908% when pulling e3d179796c9adaf72e6ec97bdc6ee129ab0c9f3b on unit-tests into b10b9ba3ae8c0b157e9c9d89cb6b98e37b782d01 on master.

Coverage increased (+1.2%) to 44.89% when pulling db4d7a6952906bfe0c46073bee04f61e26b3fd7c on unit-tests into b10b9ba3ae8c0b157e9c9d89cb6b98e37b782d01 on master.


Clarifies that VelocityComponent has only one child, not children

Documentation fix for #101.

@finneganh Awesome, this clarification would definitely prevent the issue I was having.


Falling back to gUM audio-only request if A+V request failed

Ported from https://github.com/webrtc/samples/issues/447

Closing due to age.


method_missing_with_variables': undefined method _cset'

Hello everyone,
I have a problem with Capifony. I receive this error method_missing_with_variables': undefined method _cset'` when trying to execute any command. I'm running Ubuntu 14.04.1 LTS.
➜ cat Capfile
load 'deploy' if respond_to?(:namespace) # cap2 differentiator

require 'capifony_symfony2'
load 'app/config/deploy'

➜  cat Gemfile
source 'https://rubygems.org'

gem 'capifony', '~> 2.8.2'

➜  bundle update
Fetching gem metadata from https://rubygems.org/........
Fetching additional metadata from https://rubygems.org/..
Resolving dependencies...
Using highline 1.6.21
Using net-ssh 2.9.1
Using net-scp 1.2.1
Using net-sftp 2.1.2
Using net-ssh-gateway 1.2.0
Using capistrano 2.15.5
Using capistrano-maintenance 0.0.3
Using colored 1.2
Using inifile 2.0.2
Using ruby-progressbar 1.4.1
Using capifony 2.8.2
Using bundler 1.7.6
Your bundle is updated!

➜  bundle exec cap -vT
/home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/variables.rb:122:in `method_missing_with_variables': undefined method `_cset' for #<Capistrano::Configuration:0x000000025b2988> (NoMethodError)
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-maintenance-0.0.3/lib/capistrano/maintenance.rb:8:in `block in load_into'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:97:in `instance_eval'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:97:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:83:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-maintenance-0.0.3/lib/capistrano/maintenance.rb:6:in `load_into'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-maintenance-0.0.3/lib/capistrano/maintenance.rb:96:in `<top (required)>'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capifony-2.8.2/lib/capifony_symfony2.rb:3:in `require'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capifony-2.8.2/lib/capifony_symfony2.rb:3:in `<top (required)>'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:152:in `require'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:152:in `require'
	from Capfile:3:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:93:in `instance_eval'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:93:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:180:in `load_from_file'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:89:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:86:in `block in load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:86:in `each'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/configuration/loading.rb:86:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/cli/execute.rb:65:in `block in load_recipes'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/cli/execute.rb:65:in `each'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/cli/execute.rb:65:in `load_recipes'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/cli/execute.rb:31:in `execute!'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/lib/capistrano/cli/execute.rb:14:in `execute'
	from /home/piotr/.rvm/gems/ruby-2.1.5/gems/capistrano-2.15.5/bin/cap:4:in `<top (required)>'
	from /home/piotr/.rvm/gems/ruby-2.1.5/bin/cap:23:in `load'
	from /home/piotr/.rvm/gems/ruby-2.1.5/bin/cap:23:in `<main>'
	from /home/piotr/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `eval'
	from /home/piotr/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `<main>'

I know this is old, but add this to your deploy.rb
def _cset(name, *args, &block)
  unless exists?(name)
    set(name, *args, &block)
  end
end


Update active-directory-aadconnect-topologies.md

Repeated support statement

@RobdeJong : Thanks for your contribution! The author, @billmath, has been notified to review your proposed change.

@RobdeJong : Thanks for your contribution! The author, @billmath, has been notified to review your proposed change.

Merging per Angela's approval in email.


A logo for iceberg rust!

It must be cool for iceberg rust to have it's own logo! Maybe we can have a logo that a crab eating iceberg or the iceberg is rusty.
Notes
Unlike python's logo, the crab itself is not a logo owned by rust foundation. So it's fine for us to adding a cute crab in our logo.

I'd prefer a letter I with the RUST gears surrounding it ⚙

Looks cool!


optimize: datasource auto proxy

Ⅰ. Describe what this PR did
Simplify the logic of datasource auto proxy, reduce redundant code by merging them into one processor.
Ⅲ. Why don't you add test cases (unit test/integration test)?
Added an integration test which simulates 3 major ways how user would register their DataSource bean.
Ⅳ. Describe how to verify it
Add project's spring boot starter to your project dependency, register Datasource bean in the way you like, test GlobalLock or GlobalTransaction to see whether it works. Or just run the new integration test to verify it.

Codecov Report

Merging #3588 (4515c79) into develop (b55d24b) will increase coverage by 0.24%.
 Consider uploading reports for the commit 0ba238b to get more accurate results

@@              Coverage Diff              @@
##             develop    #3588      +/-   ##
=============================================
+ Coverage      52.15%   52.39%   +0.24%
- Complexity      3509     3521      +12
=============================================
  Files            638      637       -1
  Lines          21108    21096      -12
  Branches        2613     2608       -5
=============================================
+ Hits           11009    11054      +45
+ Misses          9014     8953      -61
- Partials        1085     1089       +4

Impacted Files
Coverage Δ
Complexity Δ

...utoconfigure/SeataDataSourceAutoConfiguration.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...ation/datasource/AutoDataSourceProxyRegistrar.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...ion/datasource/SeataAutoDataSourceProxyAdvice.java
70.83% <66.66%> (+70.83%)
3.00 <3.00> (+3.00)

...on/datasource/SeataAutoDataSourceProxyCreator.java
69.44% <69.69%> (+69.44%)
8.00 <6.00> (+8.00)

...g/annotation/datasource/DataSourceProxyHolder.java
75.00% <100.00%> (+75.00%)
3.00 <3.00> (+3.00)


Opensearch notifications permission

Describe the bug
Hi team, i have created an acion group for some users, i want them to have access to the notifications and alerts panels in version 2.4.0
About alerts i have granted the permissions

All go fine with this, but for notifications they dont' have access

After check, i added the cluster:admin/opensearch* permission and then they can see, but this grants permissions over machine learning, snapshots and so on i don't want them to have (even if this permissions doesn't appear in the documentation https://opensearch.org/docs/latest/security-plugin/access-control/permissions/)

I have try to find the permission under cluster:admin/opensearch that grants the access to notifications, but even trying in the most generic way (cluster:admin/opensearch/ml* cluster:admin/opensearch/snapshot_management* cluster:admin/opensearch/observability* ) the access is not granted, only works with cluster:admin/opensearch*
To Reproduce
Create a action group
Add a user without permissions to this group
Set the permissions in the above images and text
Expected behavior
Have a concrete permission that grants access to notifications
Additional context
Version of opensearch and opensearch-dashboards are 2.4.0
Any needed info only ask
Regards

Hi @IvanLM95 have you tried mapping users to the reserved roles for these plugins?
Alerting has 3 reserved roles:

alerting_read_access
alerting_ack_alerts
alerting_full_access

Notifications has 2 reserved roles:

notifications_full_access
notifications_read_access

Reserved roles should already exist when browsing to Security -> Roles where you can map the user there.
You can find a list of reserved roles here: https://github.com/opensearch-project/security/blob/main/config/roles.yml

Hi, even if it doesn't suggest it, is working fine

So much thanks!!


Add item search up from the current item

The problem
Using the closest () method, I cannot search for ancestors that have specific parameter values.
Details
For example, this entry is currently invalid: $.closest("data-test", "message-manager").getAttribute("id")
I want to find an ancestor that has data-test = 'message-manager' and get the id value from it.
And it would also be nice to rename the closest() method to ancestor() according to the xPath terminology (https://www.w3schools.com/xml/xpath_axes.asp).
Tell us about your environment

Selenide Version: 5.19.0
Chrome\Firefox\IE Version: Chrome 89
Browser Driver Version:
Selenium Version:
OS Version: macOS 10.15.7

Code To Reproduce Issue [ Good To Have ]
$.closest("data-test", "message-manager")

Duplicate https://github.com/selenide/selenide/issues/1429


Introduce XSIMD_DEFAULT_ARCH as a way to customize the default archit…

…ecture
One of the use-case is to detect when the default architecture is used by #defining XSIMD_DEFAULT_ARCH xsimd::unsupported.

@JohanMabille note the subtle bugs this patch unveils :-)

This was an easter egg just for you ;)


Remove unnecesary envvar declaration

Looks like there was a typo in template unnecesarry env variable. Doesn't affect anything actually, but I beleive should be cleaned up.
#112 Is the corresponding issue

Closes #112


Implement recursive function to export only the AWS_PROFILE when using source_profile with credential_process

loren
Slack link
now that i have assume working, i have an enhancement request... i think that when using assume with a profile configured with source_profile, and that source profile is itself configured to use granted credential_process, then assume should export only the AWS_PROFILE and not the full set of credential envs...

The recursive function should probably also keep track of the set of profile names in the stack, in order to check for a loop.


Add development brew module

Even though this program is in alpha development stage, I think it would be beneficial to add a brew module so that it can be updated seamlessly. Once the stable version of chunkwm is released a new brew module can be released.

A brew version will be available when I release the first official Alpha. I still consider it pre-alpha for now


Support for older glibc for arm + update actions

Describe your changes
This PR adds support for older glibc (3.4.24) for arm architecture + updates gh actions to use newest versions without deprecation warnings.
It also fixes ubuntu runner version to 20.04, as this is the newest version that easily supports g++-7; It still has ~3 years of support so we should be good. After amazon updates their Linux distros to use newer glibc versions we should think about also updating our runners and dropping older glibc.
Linked issues
Closes https://github.com/software-mansion/starknet-jvm/issues/189
Closes https://github.com/software-mansion/starknet-jvm/issues/171
Breaking changes

[ ] This issue contains breaking changes

Codecov Report
Base: 69.56% // Head: 69.56% // No change to project coverage :thumbsup:

Coverage data is based on head (7857007) compared to base (23a1b1f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #213   +/-   ##
=======================================
  Coverage   69.56%   69.56%
=======================================
  Files          53       53
  Lines        1564     1564
  Branches      111      111
=======================================
  Hits         1088     1088
  Misses        446      446
  Partials       30       30

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Find some questions in the reference book

Hi professor fossen
In your book **Handbook of Marine Craft Hydrodynamics and Motion Control (2011) ** I have some questions

P125 equation (6.76), (6.77) and (6.81), the coefficient maybe should be $2 \pi$ not $8 \pi$ ?
P125 equation (6.78)-(6.80), the coefficient 2 maybe should delete?
P126 figure 6.4, the second figure's legend equation should change into $C^{new}_f=(A_x/S)C_x$ or something. The equation $C^{new}_f=(A_x/S)C^{new}_f$ doesn't make any sense.
P208 equation (8.66), I just want to know how to choose the value of $\delta\omega$ to compute $A_k$?
Looking forward to your reply.

Yes, this is a known typo in the book.
The number 2 is coming from (6.70) such that you obtain 2*  zeta * omega
Yes, you are right
Dividing the wave spectrum into 50 columns is customary. Sometimes 20 works fine. Hence, if you use omega values from 0-2 rad/s to approximate the wave spectrum, we obtain dw = 2/50 = 0.04 for a 50 columns fit.

您好，已收到您的邮件，谢谢！

Thanks for your reply！I'm still confused about questions 2 and 4:
2. From (6.70), $2\zeta \omega_n=(d+k_d)/m$, and $k_d=m/T=d$, so $2\zeta \omega_n=2d/m$, right? So the coeffecient 2 In equations (6.78)-(6.80) should be cancelled?
4. In the equation (8.66), it seems that the value of $A_k$ is proportional to the value of dw, should there be a dw in the left side such that $1/2 A_k^2 dw=S(w_k)dw$ ? Moreover, when it comes to a short-crested wave how to choose the value of $d \beta$ ?

Equation (8.66) is indeed correct. You can check the famous hydrodynamics textbooks of Faltinsen or Newman for details on this.

By the way, my 2023 lecture notes on the Web are more updated on this topic (less typos). I am not sure how to find the best value for dB since I have not coded this myself. This will be trial and failure, depending on your CPU's speed.

Thanks! I guess I got it!
2. Does that mean PD control only works in surge,sway and yaw and does not work in heave,pitch and roll?
4. If the dw value is large in (8.66), the value of $A_k$ will be large but the number N in the equation (8.67)   $\sum\limits_{k = 1}^N $ will be small and vise versa. So the sum value in equation (8.67) will keep almost unchanged. Am I right?
I have read the  lecture notes, but still have trouble understanding these equations.

For marine craft, we use PD or preferably PID in surge, sway, and yaw. These modes do not have a "physical" spring but the P-gain in the control law adds a "virtual" spring such that the closed-loop system is of order 2. Then you can use pole placement with zeta and omega.
The number k = 1,2,...,N are the number of dw you pick. If you dive the wave spectrum into 10 columns you get 10 k-values, one for each column. You compute one A_k for each column (k-value). In (8.67) you sum up your 10 columns using k = 1,2,...,10.

Ok, I think I fully understand these questions, thanks a lot for your patience answer!


Error at process_image, related to member_token

Processing Image Id: 67445994
Title: 初投稿
Tags : オリジナル, 中性的, 筆ペン
Date : 2018-02-24 22:29:00
Mode : bigNew
Image URL : https://i.pximg.net/img-original/img/2018/02/24/22/29/39/67445994_p0.jpg
Traceback (most recent call last):
File "PixivUtil2.py", line 674, in process_image
filename = PixivHelper.makeFilename(filename_format, image, tagsSeparator=config.tagsSeparator, tagsLimit=config.tagsLimit, fileUrl=url, bookmark=bookmark, searchTags=search_tags)
File "/home/user/Documents/PixivUtil2-master/PixivHelper.py", line 149, in makeFilename
nameFormat = nameFormat.replace('%member_token%', artistInfo.artistToken)
TypeError: coercing to Unicode: need string or buffer, NoneType found
Error at process_image(): (<type 'exceptions.TypeError'>, TypeError('coercing to Unicode: need string or buffer, NoneType found',), <traceback object at 0x7fc804113b90>)
Dumping html to: Error medium page for image 67445994.html
Traceback (most recent call last):
File "PixivUtil2.py", line 1913, in main
np_is_valid, op_is_valid, selection = main_loop(ewd, mode, op_is_valid, selection, np_is_valid, args)
File "PixivUtil2.py", line 1684, in main_loop
menu_download_by_image_id(mode, op_is_valid, args)
File "PixivUtil2.py", line 1361, in menu_download_by_image_id
process_image(mode, None, int(image_id))
File "PixivUtil2.py", line 674, in process_image
filename = PixivHelper.makeFilename(filename_format, image, tagsSeparator=config.tagsSeparator, tagsLimit=config.tagsLimit, fileUrl=url, bookmark=bookmark, searchTags=search_tags)
File "/home/user/Documents/PixivUtil2-master/PixivHelper.py", line 149, in makeFilename
nameFormat = nameFormat.replace('%member_token%', artistInfo.artistToken)
TypeError: coercing to Unicode: need string or buffer, NoneType found
press enter to exit.
I tried to download some images today(about one and a half week spent from last use, which had worked properly) but faced this error. I just commented out
nameFormat = nameFormat.replace('%member_token%', artistInfo.artistToken)
part from PixivHelper.py, and than same error occured from
nameFormat.replace('%original_member_token%',imageInfo.originalArtist.artistToken)
part. So I commented out all codes doing something about member_token(3 lines in total), which finally led my download succeed.
I don't know why such an error occurs suddenly, maybe my python got some problems. However I just wanted to notice you and some other guy who may suffer from similar problem about my experience.

What version you are on?

It says "PixivDownloader2 version 20171127". I think I should update mine more frequently..

Yes, it was fixed one beta release ago https://github.com/Nandaka/PixivUtil2/commit/b08b6933cffc15e0efe8e3361d611c85821d0f47

Thank you. UPDATE FIRST WHEN SOMETHING WENT WRONG <- I'll remember.


rbenv install Usage info is missing linebreaks

Not sure if this is a ruby-build issue or an rbenv issue.
voltagex@devbox:~/src/ruby-build$ rbenv install
Usage: rbenv install [-f] [-kvp] <version>       rbenv install [-f] [-kvp] <definition-file>       rbenv install -l|--list

  -l/--list          List all available versions  -f/--force         Install even if the version appears to be installed already  -s/--skip-existing Skip if the version appears to be installed already  ruby-build options:  -k/--keep          Keep source tree in $RBENV_BUILD_ROOT after installation                     (defaults to $RBENV_ROOT/sources)  -v/--verbose       Verbose mode: print compilation status to stdout  -p/--patch         Apply a patch from stdin before buildingFor detailed information on installing Ruby versions withruby-build, including a list of environment variables for adjustingcompilation, see: https://github.com/sstephenson/ruby-build#usage

Very weird. What is your OS, versions of bash, rbenv, ruby-build? Thanks

I don't think I was able to reproduce this on a clean VM - I'll reopen if it pops up again.


How to add intermediate targets before reaching goals?

Hello, very nice environment for MAPF!
I'm current training agents to collect several intermediate targets(waypoints) collaboratively before they arrive targets.
Is there any possibility/interest to support this kind of setup?
I can make a pull request if available.

Hi! I think you’re referring to anonymous MAPF, where any agent can complete any target.  It would be a great contribution to POGEMA, and I’d be glad to help with integrating this code.
However, I don’t think it’s a good idea to mix it with classic MAPF. That said, I’m open to discussion.


Feat/checkbox component

Description

Create the reusable Checkbox component for Storybook #651
Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Tested the checked, unchecked and disabled state along with stories in storybook
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

Amazing @CKanishka! You rock 🙌

Thanks @psanders 👍

@CKanishka, can you ping me on Discord, please?


Show Derived classes in UWP documentation

When looking at a class from UWP, can we please have the UWP docuemntation show the derived classes ?
The WPF documentation shows it, but not the UWP documentation

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

ID: 14f6e0bc-a81e-013e-847b-8eb197a31f60
Version Independent ID: 0b91329e-cd29-2ee5-c6a0-d6ddcb06a738
Content: Block Class (Windows.UI.Xaml.Documents) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.documents.block.yml&version=GBlive&_a=contents
Product: uwp
Git Login: jwmsft
Microsoft Alias: jimwalk

Thank you for the feedback. We have a request to engineering for this feature.


fix: Unmultiply before brightening

The image data read back is in a pre-multiplied state. Convert it back to straight alpha before brightening and then premultiply.
This just makes the colors go to white though:

You could alternatively use the HSV color space and increase the Value (brightness); this would be more subtle and keep the colors correct.

Thank you so much @jtmcdole !
I will merge this as is to Spydon's branch so I can work on it more easily, and then fix the analyzer stuff.
I also want to extract the shared code between brighten and darken.


update libuv to 1.32.0

close #374

Failing tests on Linux/Windows:
Linux:
  { address = { ip = "<IP_ADDRESS>", port = 33627, family = "inet" }, server = uv_tcp_t: 0x1f88e90 }
  { req = uv_write_t: 0x1f9a048, socket = uv_tcp_t: 0x1f77dd0 }
  "client onwrite"	uv_tcp_t: 0x1f77dd0	nil
  "server read"	{ data = "Hello" }
  "client read"	{ data = "Hello" }
  "client read"	{  }
Uncaught Error: ./tests/test-tcp.lua:253: assertion failed!
stack traceback:
	[C]: in function 'assert'
	./tests/test-tcp.lua:253: in upvalue 'fn'
	./lib/tap.lua:48: in function <./lib/tap.lua:46>
	[C]: in function 'luv.run'
	./lib/tap.lua:55: in function <./lib/tap.lua:42>
	[C]: in function 'xpcall'
	./lib/tap.lua:42: in upvalue 'run'
	./lib/tap.lua:136: in function 'lib/tap'
	tests/run.lua:33: in main chunk
	[C]: in ?
UNCLOSED	uv_tcp_t: 0x1f77dd0
  ./lib/tap.lua:70: 1 unclosed handle
  stack traceback:
  	[C]: in function 'error'
  	./lib/tap.lua:70: in function <./lib/tap.lua:42>
  	[C]: in function 'xpcall'
  	./lib/tap.lua:42: in upvalue 'run'
  	./lib/tap.lua:136: in function 'lib/tap'
  	tests/run.lua:33: in main chunk
  	[C]: in ?
not ok 78 tcp - tcp close reset accepted

Windows:
  { server = uv_tcp_t: 0x037e3830, address = { ip = "<IP_ADDRESS>", family = "inet", port = 3110 } }
  { req = uv_write_t: 0x007ede30, socket = uv_tcp_t: 0x037e30a0 }
  "client onwrite"	uv_tcp_t: 0x037e30a0	nil
  "server read"	{ data = "Hello" }
  "client read"	{ err = "ECONNRESET" }
  .\lib/tap.lua:58: Missing 1 expected call
  stack traceback:
  	[C]: in function 'error'
  	.\lib/tap.lua:58: in function <.\lib/tap.lua:42>
  	[C]: in function 'xpcall'
  	.\lib/tap.lua:42: in function 'run'
  	.\lib/tap.lua:136: in function 'tap'
  	tests\run.lua:33: in main chunk
  	[C]: at 0x00835430
not ok 77 tcp - tcp close reset accepted

Note that there were also new API functions added in 1.31.0 that we are missing: uv_fs_statfs and uv_os_environ. Those can be added in a separate PR, though.

@squeek502  Done, with three new api.

Done

Oh, you also might want to update the version in CMakeLists.txt: https://github.com/luvit/luv/blob/master/CMakeLists.txt#L9-L11

Oh, you also might want to update the version in CMakeLists.txt: https://github.com/luvit/luv/blob/master/CMakeLists.txt#L9-L11

Do that in https://github.com/luvit/luv/pull/377


correct spelling

Where is this {directory} placeholder?

Please fix the conflict.

Where is this {directory} placeholder?

It is  just my mistake.
But it is fixed in the latest version.


Load AOT Module Crash

版本 Windows 11
版本 21H2
安装日期 2022/6/17
操作系统版本 22000.739

It should have been fixed by a recent update. Please feel free to reopen this if anything does wrong again :)


how to use it for convert markdown to html?

I want to know how to use, give a user tutorial. Because I test it, but never get a good html.
My usage is  pandoc openapi.md -o openapi.html  --css GitHub.css --self-contained. The result is ```
[WARNING] This document format requires a nonempty  element.
Please specify either 'title' or 'pagetitle' in the metadata.
Falling back to 'openapi'

@wenjuanhenxing , I've now added some installation and usage instructions for the template:
https://github.com/tajmone/pandoc-goodies/tree/master/templates/html5/github#installation
If you think these instructions are sufficient, you can just close this issue. If not, tell me what could/should be added to make them more exhaustive.

get，I succeeded in the experiment.
Thank you very much.


Migrate away from request package

The request module is now fully deprecated and will no longer receive any fixes.
Its latest published version, 2.88.2, adds a deprecation warning whenever it is npm-installed. On that basis, we'll stick on 2.88.0 for the time being otherwise we'll get needless issues from our end-users.
But it does mean we need a plan to migrate off the request package to something else. I don't yet know what that will be, but this issue gives a good list of potential alternatives: https://github.com/request/request/issues/3143
For more info on the deprecation of request, refer to this issue: https://github.com/request/request/issues/3142

Another comparison list for some of those libs mentioned https://github.com/sindresorhus/got#comparison

https://www.npmjs.com/package/axios is currently being used by https://www.npmjs.com/package/ibm-watson

I suggest we switch to axios. I ran into some bugs in got on Mac OS X and node.js v10.x
I switch my nodes to use axios and it seems to be working fine everywhere.
My investigation is posted at
https://github.com/Call-for-Code/node-red-contrib-twc-covid19-tracker/issues/2

To scope the migration work, I did a quick scan of the node-red repo:
$ grep -r request */package.json
node-red/package.json:        "request": "2.88.0",

In the case of node-red core, the migration work is just the HTTP Request node:
$ grep -r 'require("request")' | grep .js | more
<EMAIL_ADDRESS>

I suggest we convert to axios. I ran into some bugs in got on Mac OS X and node.js v10.x
I migrated my nodes to use axios and it seems to be working fine everywhere.
My investigation is posted at
Call-for-Code/node-red-contrib-twc-covid19-tracker#2

It took some deep diving, but I found the source of this issue referenced: https://github.com/TooTallNate/node-agent-base/issues/35
<EMAIL_ADDRESS>a dependency of many a library's dependencies, is monkey patching the https core of NodeJS. Any package depending on it is cursed with this behaviour too. It got removed from v5, but is still present in a lot of packages. As a result, all urls using https are resolved to <IP_ADDRESS> and hardcoded to use port 443 of it. Which results in the seen ECONNREFUSED <IP_ADDRESS>:443. It's certainly messy because of it. Do note that this is not caused by got, but by agent-base patching NodeJS. The developer of agent-base fixed the issue, but released it as major release rather than a minor/bugfix, which means it's not picked up by libraries depending on it. As a result, using got is always a risk in nodes as you won't know if any other nodes have a deeply nested dependency somewhere on<EMAIL_ADDRESS>I agree on moving to axios as much as possible.

Hate it to be a party-stopper, but seems here that Axios doesn't support Digest Authentication (which is needed for the Http-request node). And it doesn't looks like axios-digest will be ever merged into Axios...

Looking at the comparison table here: https://github.com/sindresorhus/got#comparison
Looks like sindresorhus's got might be the most compatible option. Anyone have any thoughts about that? If I get some time, I might try and do a hack of the request module.

Looking at the comparison table here: https://github.com/sindresorhus/got#comparison
Looks like sindresorhus's got might be the most compatible option. Anyone have any thoughts about that? If I get some time, I might try and do a hack of the request module.

Yes, I’ve thoughts about it as I found out the hard way why it was a bad idea. See comment a few messages up (https://github.com/node-red/node-red/issues/2481#issuecomment-627021709)
Specifically<EMAIL_ADDRESS>monkey patches the https functionality in the core nodejs, meaning that if anywhere in your dependency tree something involving it is present, any code relying on core https functions differently. got is amazing, but relies on core https. Which shouldn’t be an issue at all, if TooTallNate didn’t release the fix for the monkey patching as a major change, pushed to<EMAIL_ADDRESS>And agent-base being in millions of packages as a dependency deep down the line makes this problematic.
As such, like I found out, if you have a contrib node using got, and just the core node-red you’re running, it moght work flawless. But if a 3rd part contrib node has a deep dependency on<EMAIL_ADDRESS>your node stops working. Note here that TooTallNate is a former nodejs core dev, and has been very slow in updating his other libraries to use a later version of agent-base, and all those updates come as major version, rather than semantically bug fix releases as they should have been.

But won't that be a problem for all of the options? As I've seen in the past, anything that monkey-patches core or that faffs around with prototypes of core JS is a nasty thing that needs to be avoided at all costs.
A quick check shows that Node-RED itself contains agent-base but at v6. got does not use anything that depends on agent-base.
Maybe I'm being thick - if got doesn't use it, it would not itself cause an issue. We can't stop the development of Node-RED because some other node might use an old version of agent-base in the future can we?

I firmly agree with your last statement, but that means compiling a list of current contrib nodes that will be incompatible because somewhere down the line they rely on<EMAIL_ADDRESS>as using them will turn your core http node unusable if it is based on got. Monkey patching the core should be a very frowned upon thing, it having slipped through so many dependencies makes it a bit of an issue. So you can move to got, if you either force those nodes to make the move away from libraries they rely on that use<EMAIL_ADDRESS>somewhere, or outright drop compatibility with them. That kind of choice however would be unsuitable for a semantic minor version, and if such a decision is made would be more suitable for the next major version of node-RED.

Agree that we appear to be forced to stick with the current request library then until a major release. Though that isn't too far away now is it?
In theory, it would be possible to make a post-install script that would check for agent-base@4 in the users existing install. That might go some way to lowering the risk a little?

In theory, it would be possible to make a post-install script that would check for agent-base@4 in the users existing install. That might go some way to lowering the risk a little?

I do have an idea to kinda figure out the impact, but I’ve no idea how feasible that would be. Going to figure that out first before presenting something that might only be an issue in theory.

The plan is to get off request one way or another for the 2.0 release in April. I'm still not settled in my mind which package we'll replace it with. None seem a perfect fit - so it's a question of minimising the work needed to bridge the gap.
If we can get a better sense of the true impact of the agent-base issue, that would be helpful.
I also don't know how up to date the comparison table provided by the GOT module is - it's clearly in their interest to make themselves look better. Certainly a useful starting point, but not necessarily the final say on the matter.

If we can get a better sense of the true impact of the agent-base issue, that would be helpful.

If my neck will stay cooperative for the rest of the evening, I think I can put together a quick prototype to figure out which/how many nodes would be affected by a move to got. My neck is a big if, just like how I haven’t programmed anything beyond a couple lines in JavaScript in months. But this sounds doable.

That had occurred to me as well Nick but sindresorhus is a well respected and prolific developer so it has that going for it. I will admit that when I threw together the Wiser heating module recently, I simply reached for the module that most people seem to talk about which was axios but if it is missing a bunch of features that request had, then that could be a problem.

Prototype worked in theory, not in practice, as the library underneath appears to (also) depend on requests, but most of all can't bulldozer its way through this without socket hangups, DNS failures, and timeouts. It works for doing a couple nodes one by one, not pushing the catalogue file through it like I tried here. Package used is npm-remote-ls, but someone else might have a better idea. Downloaded the latest catalogue file as input.
const ls = require('npm-remote-ls').ls;
const catalogue = require('./catalogue.json');

catalogue.modules.forEach(function(elem) {
    ls(elem.id, elem.version, true, function(obj) {
        let deps = obj.filter(s => s.includes('agent-base@'));
        //console.log(deps);
        let deps_checked = deps.filter(s => s.split('@')[1].startsWith('4.'));
        //console.log(deps_checked);
        if (deps_checked.length > 0) {
            // This package will be going to run in trouble with `got`. Log it
            console.log(`[+] ${elem.id}@${elem.version}`);
            console.log(`[*] depends on ${deps_checked.join(', ')}`);
            console.log();
        }
    });
});

Some packages that will cause the http-node to fail if it depends on got are node-red-contrib-npm and node-red-contrib-chatbot, as I found out earlier this year and just confirmed. But beyond those I wouldn't have too much of a clue. For those, node-red-contrib-npm causes issues because npm itself depends on<EMAIL_ADDRESS>I'm also spotting node-red-contrib-google-oauth2, because of the underlying googleapis library that has a dependency somewhere on it.
But, the idea to get this figured out is the same, do a deep check on the dependencies of a package, check if agent-base is in it, specifically for major version 4 which is doing the monkey patching of the core. I checked a couple manually, and a few do, but most so far appear to be steering clear.

https://www.npmjs.com/package/johnny-dependency seems to be similar to npm-remote-ls but is a little newer.
If this analysis can't be done, I still think that there is little choice but to make the change for the next major Node-RED release, warn people and maybe give a script that will run the npm ls agent-base command so that people can easily see if they have a dependency that needs fixing. That could be run as a post-install script on Node-RED install.

Having spent a bit more time on this, with 2.0 looming, there's still no easy option here.
The concerns about agent-base aside,got does look like the closest match - but there do remain a number of differences that need coding around.
It won't take much to get the basic function of http requests working. But we also have to get all the behaviour to match with handling redirects, cookies, form-data, authentication, proxy support to name just a few.
We don't have a comprehensive test suite for the node that verifies every possible combination of things the node can do. There is a good chance we'll break some edge case and not know it.

Have the agent-base concerns been mitigated by the passage of time and other updates ?

Have the agent-base concerns been mitigated by the passage of time and other updates ?

Sadly, no it has not. The way it was fixed through a major release rather than bugfix or minor as it really was, means it’s not picked up in versioning updates as easily for simple reasons. As agent-base is a (nested) child dependency of literal millions of packages, there’s a high chance that it might be somewhere in the codebase of a node-red ecosystem that were to use third party nodes.
With the change not being backported as minor/bugfix release on the agent-base package itself, this will continue to be a problem for the future. A potential solution that’s more of a workaround I thought of earlier would be a function that checks for the presence of agent-base of this version in the node-modules and emits a warning. For example, npm itself has agent-base as nested child dependency, but I don’t know from mind which version that was.
In practice, using got as http library works perfectly as long as there’s nothing else loaded in the same ecosystem that has a deep dependency on that offending version of agent-base, as it monkey patches the core nodeJS. So even if your module works fine on its own as well as the system itself, if you were to install a third party node with that dependency somewhere, it’s introducing the issue for all got calls that use https based urls. That is how I first encountered the issue myself. My node ran fine on its own, as well as imported in a clean/core install of Node-RED. But as soon as I installed a third party node with a deep dependency on it, my node stopped working with error messages that were of no use in figuring out what the issue was. I would go as far as stating that agent-base with that version is a danger for any code that relies on the core library to be untampered.

I've got an initial PR for migrating to got. It isn't complete yet, but it is fairly close.
Even with the amount of features got has, it has still been a few days work to get to this point. Moving to pretty much any other library will require significantly more work. I understand the worries about agent-base but we are somewhat stuck between a rock and a hard place. We have to move off request. got is very popular and well maintained. I would hope and expect, with the passage of time, the use of the legacy agent-base will fade away.

I certainly understand. In terms of health, I’m sadly unable to work this out on my own, but would it be an idea to have a post install script or function added to the palette install that checks for the offending agent-base version and warns the user? I believe it was previously suggested by @TotallyInformation  further above in this thread as well. I believe users should be made aware that the presence of this dependency on the node/ecosystem they’re using could cause harm for other nodes including the future core http-request node and the like, in that it becomes unable to connect to any addresses with https protocol.

So we have shipped this and sure enough, had a couple reports of issues thanks to old levels of agent-base.
I am adding some code to the request node that detects the patched module and restores the original code. It then displays this warning:
---------------------------------------------------------------------
Patched https.request function detected. Restoring original code.
This is likely caused by the<EMAIL_ADDRESS>module that has
been installed as a dependency of a contrib node.

You can identify what node is at fault by running:
   npm list agent-base
in your Node-RED user directory (/Users/nol/.node-red)
---------------------------------------------------------------------

(the user dir is inserted dynamically...)
That should keep the core working and help nail down the modules that need updating.

Closing as I think request is firmly in our past at this point.


where is table of "eicu_crd.patient"?

A demo version of the table is here: https://physionet.org/content/eicu-crd-demo/2.0/patient.csv.gz
For the full version of the table, you will need to request access by following the steps in the "Files" section of the following project: https://physionet.org/content/eicu-crd/2.0/


feat: 🎸 add create & delete core layer

Codecov Report
Attention: Patch coverage is 0% with 41 lines in your changes are missing coverage. Please review.

Project coverage is 37.11%. Comparing base (d4c18d5) to head (9b124ec).

Files
Patch %
Lines

pkg/cluster/create.go
0.00%
30 Missing :warning:

pkg/cluster/delete.go
0.00%
11 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #559      +/-   ##
==========================================
- Coverage   37.57%   37.11%   -0.46%
==========================================
  Files          50       50
  Lines        3186     3225      +39
==========================================
  Hits         1197     1197
- Misses       1864     1903      +39
  Partials      125      125

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


Automatic Install - Storage Files

Automatically create all data files in directory that user specifies.

Get Diectory input from user
re-prompt if that directory is invalid
os.path.exists - use it to check if path is valid
populate the empty string in json file with the file path that user entered
check that there isnt already a path in the data_storage. Only get path once.


Fix/issue 2

This fixes the issue where the get_geocode method was being fired on an empty post, which threw undefined index errors. I placed a check for $_POST['ls_address'] and $_POST['ls_city'] before firing the method, since the state and zip can be inferred by Google Maps.

Addresses #2


Detect MEDICAL_LICENSE

A Medical License is an occupational license that permits a person to legally practice medicine. A Medical License contains personal information about a person's healthcare provider, like their name, license number, address, and other personal identifying details. Detecting this PII (Personally Identifiable Information) involves a combination of technological measures, anomaly detections, monitoring, and organizational policies. Medical Licenses must be protected as PII because it helps ensure the privacy, security, and integrity of healthcare provider's personal information. The information contained in the medical license could potentially be used identity theft or fraud if it falls into the wrong person's hand. Treating it as a PII, organization can implement measures to minimize the risk of those.
The main ways hackers can exploit these information's are by:

Identity Theft
Fraudulent Activities/ Gaining Access to private information
Stuffing Attacks
Social Engineering/ Phishing

With access to a person's Medical License, they could use the information to impersonate Heathcare providers, gain access to patient records, or use fraudulent bills. Also, stolen medical licenses information could be sold on the dark web or to groups with exploiting tendencies. These actions can cause a significant risk to healthcare organizations and providers, along with a risk to patients. Safeguarding Medical Licenses is essential for keeping individuals safe.
For more details on safeguarding Medical Licenses, click this link here.

Would a medical license be considered fraudulent if the person assigned to it were to change their last name?


Update docs on AIRR Compliant repositories

Notes from CRWG meeting 2019-08-29 on docs for ADC compliant repositories.
We want to maintain a list of AIRR Compliant repositories on the docs page.
http://docs.airr-community.org/en/metadata-docs/standards/data_submission.html
This can act as our initial registry. The plan is to keep it limited to ADC Compliant repositories, and refer to other AIRR-seq repositories on b-t.cr. Repositories should self validate and request being listed through the normal AIRR CRWG contact email.
We want to get more information for each of the compliant repositories, such as URLs and contact information. ADC Compliance comes from the AIRR CRWG recommendations.

Closing as it appears this is done: https://docs.airr-community.org/en/stable/api/adc.html


Update munit to 0.7.26

Updates org.scalameta:munit from 0.7.25 to 0.7.26.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "munit" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #123 (21dda71) into develop (b252999) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #123   +/-   ##
========================================
  Coverage    52.56%   52.56%
========================================
  Files            5        5
  Lines           78       78
========================================
  Hits            41       41
  Misses          37       37

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 b252999...21dda71. Read the comment docs.


unexpected end + traceback issue

Hi,
Thanks for your work on this project. I am using your scraper for a project where I am trying to scrape posts from a page which goes back until 2009. It's a lot of posts and I understand that this might be causing some troubles. I am facing two issues, I was hoping you could help with:

unexpected end, I am not sure why. It does this after scraping couple of months of post, usually it never reaches 2017 and it already quits with the following message:

2019-04-27 22:16:00 [scrapy.core.engine] INFO: Closing spider (finished)
2019-04-27 22:16:00 [scrapy.extensions.feedexport] INFO: Stored csv feed (417 items) in: jobbik3.csv
2019-04-27 22:16:00 [scrapy.statscollectors] INFO: Dumping Scrapy stats:
{'downloader/request_bytes': 2295668,
 'downloader/request_count': 981,
 'downloader/request_method_count/GET': 979,
 'downloader/request_method_count/POST': 2,
 'downloader/response_bytes': 12077130,
 'downloader/response_count': 981,
 'downloader/response_status_count/200': 979,
 'downloader/response_status_count/302': 2,
 'dupefilter/filtered': 40,
 'finish_reason': 'finished',
 'finish_time': datetime.datetime(2019, 4, 27, 20, 16, 0, 363479),
 'item_scraped_count': 417,
 'log_count/DEBUG': 1438,
 'log_count/ERROR': 8,
 'log_count/INFO': 675,
 'request_depth_max': 98,
 'response_received_count': 979,
 'scheduler/dequeued': 981,
 'scheduler/dequeued/memory': 981,
 'scheduler/enqueued': 981,
 'scheduler/enqueued/memory': 981,
 'spider_exceptions/IndexError': 8,
 'start_time': datetime.datetime(2019, 4, 27, 18, 8, 37, 318807)}
2019-04-27 22:16:00 [scrapy.core.engine] INFO: Spider closed (finished)

There is no obvious error message and it seems that it has ended, whereas in fact it really did not. I was able to circumvent this error by using an adapted version of your script from here.

The second issue is a traceback problem which occurs during the process. The script is not interrupted but I feel like this should still be handled:

2019-04-27 22:12:02 [scrapy.core.scraper] ERROR: Spider error processing <GET https://mbasic.facebook.com/ufi/reaction/profile/browser/?ft_ent_identifier=10156090261316405&refid=17&_ft_=top_level_post_id.10156090262701405%3Atl_objid.10156090262701405%3Apage_id.287770891404%3Aphoto_attachments_list.%5B10156090262436405%2C10156090262491405%2C10156090263331405%2C10156090261941405%5D%3Aphoto_id.10156090262436405%3Astory_location.4%3Astory_attachment_style.new_album%3Apage_insights.%7B%22287770891404%22%3A%7B%22role%22%3A1%2C%22page_id%22%3A287770891404%2C%22post_context%22%3A%7B%22story_fbid%22%3A%5B10156090262436405%2C10156090261316405%5D%2C%22publish_time%22%3A1524336339%2C%22story_name%22%3A%22EntPhotoNodeBasedEdgeStory%22%2C%22object_fbtype%22%3A22%7D%2C%22actor_id%22%3A287770891404%2C%22psn%22%3A%22EntPhotoNodeBasedEdgeStory%22%2C%22sl%22%3A4%2C%22dm%22%3A%7B%22isShare%22%3A0%2C%22originalPostOwnerID%22%3A0%7D%2C%22targets%22%3A%5B%7B%22page_id%22%3A287770891404%2C%22actor_id%22%3A287770891404%2C%22role%22%3A1%2C%22post_id%22%3A10156090262436405%2C%22share_id%22%3A0%7D%2C%7B%22page_id%22%3A287770891404%2C%22actor_id%22%3A287770891404%2C%22role%22%3A1%2C%22post_id%22%3A10156090261316405%2C%22share_id%22%3A0%7D%5D%7D%7D%3Athid.287770891404%3A306061129499414%3A43%3A1514793600%3A1546329599%3A4304607315176871197&__tn__=%2AW-R#footer_action_list> (referer: https://mbasic.facebook.com/JobbikMagyarorszagertMozgalom?sectionLoadingID=m_timeline_loading_div_1546329599_1514793600_8_timeline_unit%3A1%3A00000000001524508080%3A04611686018427387904%3A09223372036854775742%3A04611686018427387904&unit_cursor=timeline_unit%3A1%3A00000000001524508080%3A04611686018427387904%3A09223372036854775742%3A04611686018427387904&timeend=1546329599&timestart=1514793600&tm=AQCM-FSLJ77YlQat&refid=17)
Traceback (most recent call last):
  File "c:\program files\python\python37\lib\site-packages\scrapy\utils\defer.py", line 102, in iter_errback
    yield next(it)
  File "c:\program files\python\python37\lib\site-packages\scrapy\spidermiddlewares\offsite.py", line 29, in process_spider_output
    for x in result:
  File "c:\program files\python\python37\lib\site-packages\scrapy\spidermiddlewares\referer.py", line 339, in <genexpr>
    return (_set_referer(r) for r in result or ())
  File "c:\program files\python\python37\lib\site-packages\scrapy\spidermiddlewares\urllength.py", line 37, in <genexpr>
    return (r for r in result or () if _filter(r))
  File "c:\program files\python\python37\lib\site-packages\scrapy\spidermiddlewares\depth.py", line 58, in <genexpr>
    return (r for r in result or () if _filter(r))
  File "C:\Users\endre\Documents\GitHub\fbcrawl\fbcrawl\spiders\fbcrawl.py", line 221, in parse_post
    reactions = response.urljoin(reactions[0].extract())
  File "c:\program files\python\python37\lib\site-packages\parsel\selector.py", line 61, in __getitem__
    o = super(SelectorList, self).__getitem__(pos)
IndexError: list index out of range

I have changed the language of facebook to Italian. The page I am trying to scrape posts in Hungarian. Maybe that's an issue?
Otherwise I am using the following specifications. Note, the relatively long delay is to avoid fb blocking my account. I also did not want to overdo it with many concurrent requests given the point would be to go back 10 years in this page.
# -*- coding: utf-8 -*-

# Scrapy settings for fbcrawl project
#
# For simplicity, this file contains only settings considered important or
# commonly used. You can find more settings consulting the documentation:
#
#     https://doc.scrapy.org/en/latest/topics/settings.html
#     https://doc.scrapy.org/en/latest/topics/downloader-middleware.html
#     https://doc.scrapy.org/en/latest/topics/spider-middleware.html

BOT_NAME = 'fbcrawl'

SPIDER_MODULES = ['fbcrawl.spiders']
NEWSPIDER_MODULE = 'fbcrawl.spiders'

# Crawl responsibly by identifying yourself (and your website) on the user-agent
USER_AGENT = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.77 Safari/537.36'

# Obey robots.txt rules
ROBOTSTXT_OBEY = False

# Configure maximum concurrent requests performed by Scrapy (default: 16)
CONCURRENT_REQUESTS = 3

# Configure a delay for requests for the same website (default: 0)
# See https://doc.scrapy.org/en/latest/topics/settings.html#download-delay
# See also autothrottle settings and docs
DOWNLOAD_DELAY = 5
# The download delay setting will honor only one of:
#CONCURRENT_REQUESTS_PER_DOMAIN = 1
#CONCURRENT_REQUESTS_PER_IP = 16

# Disable cookies (enabled by default)
#COOKIES_ENABLED = False

# Disable Telnet Console (enabled by default)
#TELNETCONSOLE_ENABLED = False

# Override the default request headers:
#DEFAULT_REQUEST_HEADERS = {
#   'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8',
#   'Accept-Language': 'en',
#}

# Enable or disable spider middlewares
# See https://doc.scrapy.org/en/latest/topics/spider-middleware.html
#SPIDER_MIDDLEWARES = {
#    'fbcrawl.middlewares.FbcrawlSpiderMiddleware': 543,
#}

# Enable or disable downloader middlewares
# See https://doc.scrapy.org/en/latest/topics/downloader-middleware.html
#DOWNLOADER_MIDDLEWARES = {
#    'fbcrawl.middlewares.FbcrawlDownloaderMiddleware': 543,
#}

# Enable or disable extensions
# See https://doc.scrapy.org/en/latest/topics/extensions.html
#EXTENSIONS = {
#    'scrapy.extensions.telnet.TelnetConsole': None,
#}

# Configure item pipelines
# See https://doc.scrapy.org/en/latest/topics/item-pipeline.html
#ITEM_PIPELINES = {
#    'fbcrawl.pipelines.FbcrawlPipeline': 300,
#}

# Enable and configure the AutoThrottle extension (disabled by default)
# See https://doc.scrapy.org/en/latest/topics/autothrottle.html
#AUTOTHROTTLE_ENABLED = True
# The initial download delay
#AUTOTHROTTLE_START_DELAY = 5
# The maximum download delay to be set in case of high latencies
#AUTOTHROTTLE_MAX_DELAY = 60
# The average number of requests Scrapy should be sending in parallel to
# each remote server
#AUTOTHROTTLE_TARGET_CONCURRENCY = 1.0
# Enable showing throttling stats for every response received:
#AUTOTHROTTLE_DEBUG = False

# Enable and configure HTTP caching (disabled by default)
# See https://doc.scrapy.org/en/latest/topics/downloader-middleware.html#httpcache-middleware-settings
#HTTPCACHE_ENABLED = True
#HTTPCACHE_EXPIRATION_SECS = 0
#HTTPCACHE_DIR = 'httpcache'
#HTTPCACHE_IGNORE_HTTP_CODES = []
#HTTPCACHE_STORAGE = 'scrapy.extensions.httpcache.FilesystemCacheStorage'
#FEED_EXPORT_FIELDS = ["source", "date", "text", "reactions","likes","ahah","love","wow","sigh","grrr","comments","url"] # specifies the order of the column to export as CSV
FEED_EXPORT_ENCODING = 'utf-8'
DUPEFILTER_DEBUG = True
#LOG_LEVEL = 'INFO'
LOG_LEVEL = 'DEBUG'
URLLENGTH_LIMIT =<PHONE_NUMBER>99)

Thanks for taking a look!

An update on this: when I do not set the date the script only stops in the beginning of 2014. I think it's due your in-built settings you describe in the read.me of the repo. Any idea how to overwrite that, given the above problems with the date attribute used in the command?

Hi @eborbath thank you very much for your interest and the detailed description of your problem.
The date attribute was in fact wrongly parsed, it should work fine now! If you provide a date in the format -a date='2018-02-13'  the crawling will stop when you reach that date, otherwise it will go as back as much as it can up to 2014. It will stop only if, for some reason, it finds item duplicates, given that DUPEFILTER is on by default (to avoid infinite recursion).
Are you still experiencing the second issue? It looks like the GET contains a reactions page of a post and that fbcrawl tries to parse it with parse_post instead of parse_reactions, so XPATH selector list (reactions) it's empty and thus the IndexError. I'm not sure why this happens, it has never popped up in my trials, it might be due to wrong link joining. Although the crawling process is not interrupted the post is not stored in the CSV so we should look more into it.
Thank you for your testing, feel free to share more suggestions, experiments and weird behaviors!!

Hi! Thanks for fixing the first issue! I am still having trouble with the second issue. Based on your description my tip would be that it's because of posts which were published before facebook introduced reactions and likes were the only way to show "emotions". I had this idea because I get this for older posts and after a certain date it more or less concerns most posts. I think if you try to scrape the page of the political party you will be able to replicate the problem. If my hunch is correct, it might be a systematic issue with older posts across all pages. I have used the following command:
scrapy crawl fb -a<EMAIL_ADDRESS>-a password="password" -a page="JobbikMagyarorszagertMozgalom" -a date="2008-01-01" -a lang="it" -o jobbik.csv

Hi, very nice hint, adding a sanity check on the reaction xpath seems to be enough to solved the issue in my trials. Please try again now and report any incosistencies :)

Hi! Now it works well, thanks very much for fixing it. You can close the issue.


Radium fails if it is not the last wrapper on a component

I have a number of other higher order components that are wrapping my component, and have found that if Radium isn't the last wrapper, it does not work.
Example
I have a fork that updates the Radium example Button component, by adding a buttonWrapper higher order component: https://github.com/thefivetoes/radium/blob/broken-example/examples/components/button.jsx You can toggle the working/nonworking version by swapping the export and commented export at the bottom of the file.
Summary of the example linked above
Using the Radium examples as starting point and adding a new buttonWrapper higher order component:
const wrapButton = (Component) => {
  class ButtonWrapper extends React.Component {
    componentWillMount() {
      console.log('Wrapped Button Will Mount');
    }

    render() {
      return <Component {...this.props} {...this.state} />;
    }
  }
  return ButtonWrapper;
};

"Proper" wrapping order
So the above example works fine if I wrap the Button with Radium, then the entire component with wrapButton
module.exports = wrapButton(Radium(Button));

## Out of order wrapping
...but it fails if the order is switched:
module.exports = Radium(wrapButton(Button));

# Expected Behavior
Ideally, wrapping order would not matter. If another library also insisted on being the first wrapper, we would essentially have to pick between that library and Radium.

Were you able to debug to see why this is the case?

@thefivetoes is this still an issue for you?


Enable-DbaTraceFlag persistent

Hi dbatools!
I love this module for powershell.
I used the cmdlet Enable-DbaTraceFlag recently, and I just wondered how it's possible to set a trace flag permanently, so it's also there after a restart of the sql server service. I figured out it can be set in the registry e.g.
New-ItemProperty -Path 'HKLM:\SOFTWARE\Microsoft\Microsoft SQL Server\MSSQL14.ENT2017\MSSQLServer\Parameters' -Name SQLArg4 -Value '-T6534'
so I suggest to add a persistent parameter to Enable-dDbaTraceFlag cmdlet that will use this logic. I'll be happy to help implementing this for the traceflag dbatools cmdlets.
Regards
Camilla

You would use Set-DbaStartupParameter for persisting the trace flags.
Ref: https://docs.dbatools.io/#Set-DbaStartupParameter


[Task] [OSPP] JAVA implements the native ipmi2 communication protocol

What's changed?
link https://github.com/apache/hertzbeat/issues/1794
I have successfully implemented the core components of a native IPMI protocol in Java, which includes protocol field declarations, serialization/deserialization, cryptographic algorithms, and abstraction layers. This foundational implementation supports extensive extension of custom IPMI commands.
Currently, session management, chassis status queries and sensor list info have been implemented.

Checklist

[x]  I have read the Contributing Guide
[x]  I have written the necessary doc or comment.
[x]  I have added the necessary unit tests and all cases have passed.

Add or update API

[ ] I have added the necessary e2e tests and all cases have passed.

Please link the help document in https://github.com/apache/hertzbeat/blob/master/home/sidebars.json

lgtm!👍 a fantastic job

It was a big task, but you did it very well. Keep it up!👍


Simplify elementCount custom assertions

This PR takes care of three things:

Slightly improves the file comment.
Renames one of two identically named variables that, because of separate scope, are actually pointing to different pieces of memory. The selector in the function passed to the browser is not the same data as the selector in brackets immediately after it.
Calls a callback directly instead of using .call().

Let me know what you think. #2 was especially confusing to me while I was looking at this file for the first time. If you can think of a better argument name, let me know 🙃.

Side note: I see a lot of templating to add semicolons. I'm converting all of my code to AirBnb style, and I think it might be possible to write all of the code in a way that it could be transformed to Standard or AirBnB with a lint --fix command. Have you tried that? It would be a lot cleaner than all of the if AirBnB then insert ; statements.

tbh I'm not super familiar with Nightwatch, but that change for the callback means that this is now undefined in the callback, which the original code probably intentionally avoided.

From what I read in Nightwatch docs, the this binding isn't necessary in the callback. I looked through the history and there aren't any clues to why it was added. I've tested and everything seems to work as expected.
I'm most interested in making the arguments more clear (c3a2181), so if you'd like to keep the bound callback I can remove the commit that changes it.

@vuejs-templates/collaborators Can anybody with more nightwatch experience chime in wether it's imortant to bind this here?

this will point to the browser instance, I believe it's a convention for most of the callbacks in nightwatch. I'd preserve it for the sake of compatibility.

@robwierzbowski There's your answer :)

OK, commit removed. Very small PR now :)