[Storage] Share a single httpClient among clients

Currently child clients obtained via get*Client() share their parent
client's pipeline. But when the pipeline doesn't have a httpClient
specified, a DefaultHttpClient will be created for each of the
clients.
There's no reason why we cannot share the http client instance among
storage clients.  This change create a default http client instance
for parent client's pipeline if none is specified.

Is it better to just have one single globally cached http client for all storage clients?  Any concerns on that?

/azp run js - storage-blob - tests

Looks good. If default http client instance doesn't have internal states.

as far as I know the http clients don't store anything specific to a request or response.

#6375


[gui error report] Error: Initializing daemon...

👉 Please describe what you were doing when this error happened. 🤷🏾♂ just seeding… ?
Specifications

OS: darwin
IPFS Desktop Version: 0.22.0
Electron Version: 19.0.8
Chrome Version: 102.0.5005.148

Error
Error: Initializing daemon...
Kubo version: 0.14.0
Repo version: 12
System version: amd64/darwin
Golang version: go1.18.3

Error: lock /Users/a/.ipfs/repo.lock: someone else has the lock

    at errorTemplate (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/migration-prompt.js:78:49)
    at loadWindow (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/migration-prompt.js:115:26)
    at module.exports (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/migration-prompt.js:120:3)
    at startIpfsWithLogs (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/daemon.js:165:9)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async module.exports (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/daemon.js:209:15)
    at async startIpfs (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/index.js:47:17)
    at async restartIpfs (/Applications/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/index.js:101:5)

@futuristudios can you please follow the steps at https://github.com/ipfs/ipfs-desktop#i-got-a-repolock-error-how-do-i-resolve-this to unblock yourself? Let us know if you need further help


Use HubL to Generate grid CSS

Part of #8 basically. The gist is let's update the css to use HubL to loop through generating the classes. Otherwise this is going to be tedious to update/maintain.
https://github.com/TheWebTech/CrankShaft/blob/master/CrankShaft/code/cs/cs-grid.css#L441-L2532

Example code
{%- for i in range(1, 12) -%}

  .col-{{i}} {
 -webkit-box-flex: 0;
  -ms-flex: 0 0 8.333333%;
  flex: 0 0 8.333333%;
  max-width: 8.333333%;
  }
  {%- endfor -%}

You'd just need to have it then calculate that %.  Which is somewhat simple math.
bootstrap has a 12 col grid.
100% = 12
You'll know you have it right, if it matches the code you already see there.

why not use the span1 - span12 class so when you use the drag and drop it still works
You can customize it for other situations.

Maybe I wasn't clear, that IS the plan, but we're going to support both class names. as the spans are specific to one screensize.
I have that as a separate issue #32
Sorry I'm trying to break down all the steps into smaller chunks so that people have clear todo's on how to help. Otherwise telling people "okay let's build the framework" is an abstract daunting task. This one maybe I drilled down too much. idk.

Its cool.
I was just going issue by issue.

helped Andreas Wessoly with the math for it so here's the math for generating the widths
{% set colwidth = (100/12) * i  %}
width: {{colwidth}}%;
*width:{{colwidth}}%;

going to close this issue, we have some further related changes to make in Hublizing the grid but for the sake of keeping the issues single topic, I'll create new issues for those.


Added backend for compiling to FlipperZero🐬

I'm going to add some stuff that calls everything for ufbt so it can automagically be deployed to the flipper from the Sage compiler


⚠ Samsung has degraded performance

In b08373a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2457 ms

Resolved: Samsung performance has improved in d48c1dd.


Snakemake 7 does not create SLURM dependencies when --immediate-submit flag is used

First of all: Thanks for Snakemake! I love it and it makes my life so much easier!!!
Snakemake version
Snakemake 7.
Describe the bug
I have been using Snakemake with an adapted SLURM cluster profile with the --immediate-submit flag, so Snakemake submitted always all the required jobs with the respective dependencies at once.
With Snakemake 7 the profile and job submission still work fine but no dependencies are provided hence all jobs that depend on running jobs fail/don't start immediately. Repeated iteratively until no unfinished jobs are left works as well, but is of course no solution.
Additional context
SLURM profile I am currently using: https://github.com/epigen/unsupervised_analysis/tree/main/config/slurm.cemm
original SLURM profile cookiecutter: https://github.com/Snakemake-Profiles/slurm
If you have a solution, an idea, or know someone who is working with Snakemake and/or SLURM in more depth, please let me know.

I'm having the same problem. Do you know what was the last version that worked? That might help pin point the cause.

I do not know the exact last version but I know for sure that it works with 6.0.3.
Hope this helps! Please let me know if you figure out how to make it work.
Thanks & Cheers,
Stephan

@SichongP

I'm having the same problem. Do you know what was the last version that worked? That might help pin point the cause.

My analyze may help https://github.com/snakemake/snakemake/issues/1865
(maybe I should close that issue and move it here)

Hi @Hocnonsense,
thanks for your analysis.
Did you get it to work? Does your PR fix the issue?
Thanks & Cheers,
Stephan

Hi, a easy way to fix it, is add some line to https://github.com/snakemake/snakemake/blob/dc45ccb9ee94e8fcdf386c8598bbae57e319ba10/snakemake/executors/init.py#L1272-L1281 like this:
https://github.com/snakemake/snakemake/pull/1851/commits/6649305b88d4bdc38a226e4d178f845836ff37ed#diff-438f3317205fd7130727d0589d2fc1a6c2e1f6fc48c2c04d354a8a09b91ba2f4
This could quickly fix this problem. However, after I review the code and noticed PR 41a5071, I revert this fix as it make thing more complex.
I'm still puzzled about the purpose of this PR, so I have no new PR now.
Do you have any idea? Thanks!

Hi,
I do not have any new ideas as of now, sorry.
Cheers,
Stephan

Hi folks. I've just merged PR #1851. If that fixes your issue, please close it. If not, feel free to keep it open. Note that I might forget to check this routinely, so feel free to ping me via discord.

Hi @johanneskoester,
I installed the latest Snakemake version (7.15.2.) and it worked!
Thanks a lot, @Hocnonsense for the PR, you made my day!
Cheers,
Stephan


OnConfigurationChanged is never called and OnDestroyed is still called even when adding ConfigChange attributes to Activity in Xamarin-Forms Android App

I am using Xamarin-Forms with .NETStandard 2.0.
This page (https://docs.microsoft.com/en-us/xamarin/android/app-fundamentals/handling-rotation) explains how to manually handle orientation changes by providing OnConfigurationChanged handler.
Orientation change is not the only reason Android decides to send an Activity to OnDestroy callback.  Others are config changes such as attaching an external keyboard and Android provides ConfigChange.Keyboard and ConfigChange.KeyboardHidden which can be added to Activity attributes just as the Orientation was added to nicely explain how to handle orientation changes manually in this article.
So, using the same logic, if I add ConfigChange.Keyboard and ConfigChange.KeyboardHidden attribues to my Activity, I would expect to see my OnHandleConfigurationChanges callback to be fired and mine OnDestroy callback NOT to be fired based on this documentation (as it states, adding these attributes will prevent activity from restarting).  Here is my Activity
[Activity(Name="com.my.domain.mainactivity",
   Label="...",
   Icon= "...",
   Theme="...",
   ConfigurationChanges=ConfigChanges.Keyboard
      | ConfigChanges.KeyboardHidden
      | ConfigChanges.ScreenSize
      | ConfigChanges.ScreenLayout
      | ConfigChanges.Orientation)]
public class MainActivity : Xamarin.Forms.Platform.Android.FormsAppCompatActivity
{
  ...
}

As you can see above, I added both attributes for Orientation and Keyboard.  As far as I understand this documentation, this means that my MainActivity OnDestroy callback will not be called since these attributes prevent activity from being restarted AND my OnConfigurationChanged callback will be called to allow me to manually handle the change.  Here is my OnConfigurationChanged:
public override void OnConfigurationChanged(Configuration newConfig)
{
   base.OnConfigurationChanged(newConfig);
   if (newConfig.Keyboard == KeyboardType.NoKeys) {....}
   else {....}
}

The problem is that even though I added these attributes, I still see OnDestroy being called and OnConfigurationChanged is NEVER being called.
Also, I think your document is missing part of what happens if you do not provide override for OnConfigurationChanged and what happens in that case.  Based on android documentation, in that case, following happens:
"If you don't need to update your application based on these configuration changes, you can instead not implement onConfigurationChanged()."
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6f3f45bd-e23b-25a7-01e1-55fc444ad680
Version Independent ID: 0717b153-921b-2033-5968-6e44430829a9
Content: Handling Rotation - Xamarin
Content Source: docs/android/app-fundamentals/handling-rotation.md
Product: xamarin
Technology: xamarin-android
GitHub Login: @conceptdev
Microsoft Alias: crdun

Hi @dinobu are you also seeing the methods not being hit in the sample code?
If not, perhaps there is a product issue here that needs to be raised to the engineering team. You can do so from within Visual Studio, or visit GitHub (Xamarin.Android is open-source).
Thanks for the comment on the documentation - we'll keep that in mind for a future update.

Hi @dinobu are you also seeing the methods not being hit in the sample code?
If not, perhaps there is a product issue here that needs to be raised to the engineering team. You can do so from within Visual Studio, or visit GitHub (Xamarin.Android is open-source).
Thanks for the comment on the documentation - we'll keep that in mind for a future update.

Hi Craig and thanks for your lightening responses.  I havent tried the sample code but I would like to add that I am getting now exactly what I expect, so my initial comment above can be ignored.  I would only suggest adding more to documentation regarding the last part "what happen when the override ONConfigurationChanged" is not provided as that seam to be missing.  Otherwise, this can be closed, it was my PEBKAC.  Appologies.

10 points for PEBKAC! that's one of my favorite acronyms :) Glad you got it working at least - I'll close issue this off, but make a note on our backlog about the doc suggestion. Thanks for the feedback.

Done and thanks again

@dinobu, could you please describe your PEBKAC, since I have the same problem - I added the ConfigurationChanges flags which correctly end up in the manifest, and I also have an ONConfigurationChanged override method, which gets called. But, when adding or removing a bluetooth keyboard, I still get: OnConfigurationChanged/OnPause/OnStop/OnDestroy/OnCreate/OnStart/OnResume.  I want the keyboard action to only do the ONConfigurationChanged - and I don't see what I'm doing wrong.
Thanks!


Pull Request not using test level 2 on AppVeyor

The existing logic using GitVersion is no longer working. We need at least an interim fix for AppVeyor builds.

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on:

GitHub Release
NuGet Package


Local up setup cluster should contains node other than master

Currently，we setup local cluster by kind, and by default it create a cluster with only master node.
In some case, it need to install components like submariner gateway engineer in normal node.

/assign @zirain


TLS Encryption through nginx configuration issue

I have not been able to make the nginx setup work with the instructions provided on the following page: https://prometheus.io/docs/guides/tls-encryption/
In order to make it work I had to adjust two things from those instructions:

In the nginx.conf, instead of
location /prometheus {
proxy_pass http://localhost:9090/;
I have used
location /prometheus {
proxy_pass http://localhost:9090/prometheus;

I have not added the --web.route-prefix="/"

My system is Debian Buster, let me know if you need more details.

It looks like the issue is the lack of a trailing slash on the location. Would you like to send a PR?

oh wow I thought I had tried everything but apparently no.
thanks a lot @brian-brazil !

We still have a minor doc bug that needs fixing.


Preserve Scaling Issue

Preserve Scaling Issue

Description needs update

The description of the Item "When _preserveScaling is false...", not issue.
Just as clarification.


[zh] Localize /pods/sidecar-containers.md into Chinese

Add zh text to this new page:
content/zh-cn/docs/concepts/workloads/pods/sidecar-containers.md

LGTM
Thanks!

/lgtm


Add an other Aqua Scan Format

I have an Aqua Scan Report coming from Azure Devops job. The file has slightly different json structure, having outter square brackets and 2 more lines at the beginning (image_name and results). The rest is the same, I think the part we want is in the 'results' json key. That file gives me 0 findings when I import it like this. Is it possible to add that new format?

Could you add an example file?

sure
AquaSecurity-test-local-202407191.json

See PR @kzzz1


Low Priority: Reduce vertical space between images on Explore/Ecological Communities page

Low priority
Reduce vertical space between the images in the right-hand column on the Explore/Ecological Communities page
Current:

Update:

@dayotte each image is grouped with content on the left. In some cases there maybe multiple paragraphs of text. ( https://github.com/IMERSS/xetthecum-storymap/blob/87633272888368b6f1b8e300a8d8528917da80f3/ecological-values.qmd#L12-L22 ) Because the text and related image are in sort of a row together I can't equally space out all the images. I could make the images start from the top of the related text though if that's sufficient.
If the images aren't really specific to the content and you just want them all together we could reorganize the content of the page and change up the styling.

The images are not specific to the sections of content, and I think they'd look better all together in a column (like in the mockup). When you say reorganize the content and restyle, is that what you mean, or something else?

The images are not specific to the sections of content, and I think they'd look better all together in a column (like in the mockup). When you say reorganize the content and restyle, is that what you mean, or something else?

Yes that's what I mean, and they'd probably appear grouped together towards the top of the page.


fix: datepicker critical issue

Description
Update @blueprintjs/datetime to fix date picker widget issue
Fixes #3942
Type of change

Please delete options that are not relevant.

Bugfix (non-breaking change which fixes an issue)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

@riodeuno please review, this is a critical issue


Issue with asar.unpacked with electron-builder

When using electron-builder, spawn-rx gets copied into app.asar.unpacked, but not all files are copied, resulting in
Uncaught Error: ENOENT: no such file or directory, open  ../app.asar.unpacked/node_modules/spawn-rx/lib/index.js
Not sure if this is a electron-spellchecker, or electron-builder issue.

electron-spellchecker has no control over packaging, please take this to electron-builder (actually, let me recommend https://github.com/electron-userland/electron-forge instead, it's very new but it's going to be The Supported Way To Package) and it does a lot of cool stuff


Enhancement | Default Disabling of Restrictive Network ACLs (NACLs)

Describe the Feature
Within the le-tf-infra-aws-template repository, there's a need to ensure that restrictive Network ACLs (NACLs) are disabled by default. This proactive approach will prevent potential complications, streamline user experience, and ensure that the use of NACLs is only when explicitly required and approved.
Expected Behavior

In the default configuration, NACLs should be disabled, particularly in the shared account base-network layer variables.
Expected variables setup:  manage_default_network_acl    = true
  public_dedicated_network_acl  = false // use dedicated network ACL for the public subnets.
  private_dedicated_network_acl = false // use dedicated network ACL for the private subnets.

Users or tech leads wishing to enable NACLs must undergo an explicit approval process.
Feedback mechanisms should be in place to inform users of the status of NACLs and any associated permissions.
Comprehensive testing should be conducted to ensure that the default disabling of NACLs does not introduce new issues.

Use Case
Given the recurrent challenges and complications associated with NACLs, especially during real-time troubleshooting with clients, a safer default approach is to have them disabled. This ensures a smoother experience for most users while still providing the flexibility to enable NACLs when necessary.
Describe Ideal Solution

Update the default configurations in the le-tf-infra-aws-template repository to have NACLs disabled.
Implement a clear and straightforward process for users or tech leads to request and approve the activation of NACLs.
Enhance the architecture's feedback mechanisms to provide users with clear information about NACL status and permissions.
Strengthen testing protocols to validate the behavior and implications of having NACLs disabled by default.

Alternatives Considered

Maintaining the current setup and addressing NACL-related challenges reactively.
Offering granular NACL controls without changing the default state.

Additional Context

The emphasis is on providing a safer and more user-friendly default setting by having restrictive NACLs disabled.
The proposed changes aim to reduce the frequency of NACL-related issues while still offering the capability to use them when justified.

Hi @exequielrafaela,  just to confirm. The idea is to add these points to the leverage documentation, correct?

Describe Ideal Solution

Implement a clear and straightforward process for users or tech leads to request and approve the activation of NACLs.
Enhance the architecture's feedback mechanisms to provide users with clear information about NACL status and permissions.
Strengthen testing protocols to validate the behavior and implications of having NACLs disabled by default.

@rodriguez-matias Matu I think It could be useful if we have a dedicated VPC NACLs entry with the info in this PR in the doc under the Ref Arch Network section:


Fix filter params

The auto generated view more link was not setting up the filter params correctly. This fixes that.
Testing

Go to http://localhost:8000/policy-compliance/guidance/implementation-guidance/bulletin-supervision-authority-confidential-information/, click "View more" on the sidebar, expand "Filter activities", and confirm the expected tags are there (in this case, Supervision and Mortgages.

Review

@kurtw @kave @rosskarchner

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Cool


Trieste reference

This PR contains the working draft of a Trieste reference that tries to document how everything works. Here is the progress so far (bullets may be added later):

[ ] Token definitions
[ ] Parsing
[ ] Readers, writers, rewriters
[ ] Specifying PassDefs
[x] Direction flags
[x] Pattern matching
[x] Effects
[ ] Error handling
[ ] Lookup, lookdown, symbol tables, generating fresh symbols
[x] Well-formedness specifications
[ ] Fuzz testing

Feel free to add to this PR or suggest changes/topics.

Addition to the checklist:

[ ] Pre and post hooks on PassDefs


Feature Request, Add --customerId and --gaDomain as parameters

Hi there,
Could you add customerId and gaDomain as parameters to the gam command?
This would help using a CSV over multiple domains.
Regards,
Thomas

Thomas,
My Advanced Gam (https://github.com/taers232c/GAMADV-XTD) supports multiple customers/domains see: https://github.com/taers232c/GAMADV-XTD/wiki/gam.cfg https://github.com/taers232c/GAMADV-XTD/wiki/gam.cfg
There are sections in the gam.cfg that identify customers/domains; if you have a column in your CSV file that identifes the section, then you can do whatt you want.
Sample.csv
Section,Header1.Header2
customer1,blah,blah
customer2,blah,blah
gam csv Sample.csv gam select ~Section some command ~Header1 ... ~Header2
Ross
--
<EMAIL_ADDRESS>
On Nov 2, 2018, at 5:25 AM, thomasgcompany<EMAIL_ADDRESS>wrote:
Hi there,
Could you add customerId and gaDomain as parameters to the gam command?
This would help using a CSV over multiple domains.
Regards,
Thomas
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/jay0lee/GAM/issues/815, or mute the thread https://github.com/notifications/unsubscribe-auth/AIU8L_WktP-pwp5gK2bz8tSrE4MohK67ks5urDmggaJpZM4YLj9R.

You can set these parameters with environment variables:
export CUSTOMER_ID=nnnn
export GAM_DOMAIN=nnnn


Clean up loggers config

Motivation

it's impossible to configure some loggers as their levels are missing from the config struct,
there are some leftover logger levels in the config

The way that selecting a level for a logger works now requires that the logger name (defined in node/node.go) matches the logger level field mapstructure tag in config/logging.go
Changes

added/fixed missing logger levels
removed redundant logger levels

bors merge

bors merge

bors merge


Delivery options do not load again after switching shippingmethod

When switching from vast tarief to afhalen in de winkel, the delivery options disappear as expected.

When switching back tot vast tarief, the delivery options do not appear again.

00760742


[Storage] add re-exports for conveniences

We used to re-export types, or have copies of same types from each storage package so customers have the conveniences of not importing them from another package. This PR adds back some missing types that were exported.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-share

Thanks y'all, once there's a new alpha branch I will give these changes a try.


Abandon 1.0.0-beta.23 and Release 1.0.0-beta.23-1 or .24

OS?
MacOS 10.12.1 (16B2657)
Versions.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Repro steps.
Installing above packages, failed caused by #3513
The log given by the failure.
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

@AilisObrian How did you install the .23? I cannot find it on npm.

@vapits try to install from this repository, steps to reproduce in the main README

The PR you linked updates the required version of tslint-loader to ^3.3.0.  You seem to have a mix of dependencies.

Beta 24 has now been released.

Thanks folks!


merge package from arlac77/npm-package-template

package.json

chore(engines<EMAIL_ADDRESS>

:tada: This PR is included in version 1.2.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dataloading with missing videos

Hi @zhaoyue-zephyrus
Thank you for your incredible work!
I’m currently pretraining LaViLa on an 8-A800 (80G) node using the following script (with a chunk length of 60 seconds per video):
CUDA_VISIBLE_DEVICES=0 \
PYTHONPATH=.:third_party/decord/python/ torchrun \
--nproc_per_node=1 \
scripts/main_lavila_pretrain.py \
--root datasets/Ego4D/video_288px_60sec/ \
--root-val datasets/EK100/EK100_320p_15sec_30fps_libx264/ \
--video-chunk-length 60 \
--use-flash-attn \
--grad-checkpointing \
--use-fast-conv1 \
--batch-size 256 \
--freeze-temperature \
--fused-decode-crop \
--fix-lr \
--output-dir $EXP_PATH 2>&1 | tee $EXP_PATH/log.txt

However, the training code gets stuck at this line:
for data_iter, inputs in enumerate(train_loader):

And the training process cannot be started.
I noticed that while several (about 9/128) of my CPUs are running at full load, most remain idle. Could you provide some insights into what might be causing this, or suggest ways to improve data loading efficiency?
Thanks in advance!

I just tried to start the training again and was surprised to find that the issue has resolved itself. Everything is running smoothly now.
Sorry for the confusion and any inconvenience caused!
Thanks again for your great work!

The issue was caused by missing videos in my Ego4D dataset. When the video loader attempted to load a missing video, it would get stuck in an infinite loop at this line, without producing any error message:
while True:
    video_filename = osp.join(root, '{}.{}'.format(vid, ext), '{}.{}'.format(chunk_end, ext))
    if not osp.exists(video_filename):
        # print("{} does not exist!".format(video_filename))
        chunk_end -= chunk_len

Once I filtered out the missing videos from annotations, the training ran successfully. I think uncommenting the error message here would be helpful for others to identify the issue:
print("{} does not exist!".format(video_filename))

Additionally, I have another question:
While creating video chunks, I noticed that a few of them are corrupted, causing Decord's VideoReader to fail when loading the video stream during training. Do you have any suggestions or experience in handling this issue?
Thanks a lot!

Hi @yuezih
This is a great point. This error message was used for debugging. If I remember correctly, I didn't see corrupted videos when truncating the video. That's why the current implementation does not support interruption for this issue.
For your case, I would suggest checking your FFMPEG version and watching out for errors. If the error still happens, simply skipping those video chunks along with their annotated clips would be a solution.  Since the training data is abundant, this won't impact the performance much.
Best,
Yue

Thanks for the prompt reply! My problem is solved and I will close this issue.


Add the 'packaging' package and it's required on some distros

Closes https://github.com/Azure/azure-cli/issues/3800

Codecov Report

Merging #3801 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3801   +/-   ##
=======================================
  Coverage   70.93%   70.93%
=======================================
  Files         424      424
  Lines       26890    26890
  Branches     4092     4092
=======================================
  Hits        19074    19074
  Misses       6573     6573
  Partials     1243     1243

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 855f123...e9961a0. Read the comment docs.


luci-app-advanced-reboot: bugfix: compatible with new luci

Signed-off-by: Stan Grishin<EMAIL_ADDRESS>

Tested on WRT32X, WRT3200, WRT1900ACS all working good,
Thanks for the prompt fix.


Upgrading to consul 0.6.4

Contains some fixes that hopefully make our tests more stable.

pg?

PG231

Merged as a027fac.


grass: GUI fails to launch with "ERROR: wxGUI requires wxPython. No module named wx"

Describe the bug
Launching GRASS fails with exit code 1 and the following output on the terminal:
Cleaning up temporary files...
Starting GRASS GIS...
ERROR: wxGUI requires wxPython. No module named wx
You can still use GRASS GIS modules in the command line or in Python.
ERROR: Error in GUI startup. See messages above (if any) and if necessary, please report this error to the GRASS developers.
On systems with package manager, make sure you have the right GUI package, probably named grass-gui, installed.
To run GRASS GIS in text mode use the --text flag.
Use '--help' for further options
     grass76 --help
See also: https://grass.osgeo.org/grass76/manuals/helptext.html
Exiting...

To Reproduce
Steps to reproduce the behavior:

Runnix run -f channel:nixos-20.03 grass -c grass76 --gui

ornix run -f channel:nixos-unstable grass -c grass76 --gui

Note: This also happens without the --gui option, if you've haven't launched GRASS with --text before.
Expected behavior
The GRASS GUI appears.
Workaround
Force CLI mode, e.g.:
nix run -f channel:nixos-unstable grass -c grass76 --text

When GRASS has been launched in CLI mode before, it seems to remember that, so --text can be omitted on subsequent sessions:
nix run -f channel:nixos-unstable grass -c grass76

Screenshots
N/A
Additional context
N/A
Notify maintainers
@mpickering
Metadata

system: "x86_64-linux"
host os: Linux 5.4.51, NixOS, 20.09pre234242.c71518e75bf (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.7
channels(root): "nixos-20.09pre234242.c71518e75bf, nixos-unstable-20.09pre218613.ae6bdcc5358"
channels(das-g): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
- grass
# a list of nixos modules affected by the problem
module:

I can reproduce that on NixOS stable.
$ nix run -f channel:nixos-unstable grass -c grass76 --gui
[24 copied (343.1 MiB), 103.4 MiB DL]
Starting GRASS GIS...
ERROR: wxGUI requires wxPython. No module named wx
You can still use GRASS GIS modules in the command line or in Python.
ERROR: Error in GUI startup. See messages above (if any) and if necessary, please report this error to the GRASS developers.
On systems with package manager, make sure you have the right GUI package, probably named grass-gui, installed.
To run GRASS GIS in text mode use the --text flag.
Use '--help' for further options
     grass76 --help
See also: https://grass.osgeo.org/grass76/manuals/helptext.html
Exiting...

Right now even the build fails:

[davidak@gaming:~]$ nix run -f channel:nixos-unstable grass -c grass76 --gui
builder for '/nix/store/hm5704d9v2c0mx5nq5ngpixyyzchq0qr-grass.drv' failed with exit code 2; last 10 log lines:
  Errors in:
  /build/source/lib/python/ctypes
  --
  In case of errors please change into the directory with error and run 'make'.
  If you get multiple errors, you need to deal with them in the order they
  appear in the error log. If you get an error building a library, you will
  also get errors from anything which uses the library.
  --
  Finished compilation: Wed Jan 20 17:39:40 UTC 2021
  make: *** [Makefile:70: default] Error 1
[0 built (1 failed), 297 copied (1797.4 MiB), 412.4 MiB DL]
error: build of '/nix/store/hm5704d9v2c0mx5nq5ngpixyyzchq0qr-grass.drv' failed

Right now even the build fails:

[davidak@gaming:~]$ nix run -f channel:nixos-unstable grass -c grass76 --gui
builder for '/nix/store/hm5704d9v2c0mx5nq5ngpixyyzchq0qr-grass.drv' failed with exit code 2; last 10 log lines:
  Errors in:
  /build/source/lib/python/ctypes
  --
  In case of errors please change into the directory with error and run 'make'.
  If you get multiple errors, you need to deal with them in the order they
  appear in the error log. If you get an error building a library, you will
  also get errors from anything which uses the library.
  --
  Finished compilation: Wed Jan 20 17:39:40 UTC 2021
  make: *** [Makefile:70: default] Error 1
[0 built (1 failed), 297 copied (1797.4 MiB), 412.4 MiB DL]
error: build of '/nix/store/hm5704d9v2c0mx5nq5ngpixyyzchq0qr-grass.drv' failed

Right now even the build fails:

[davidak@gaming:~]$ nix run -f channel:nixos-unstable grass -c grass76 --gui
builder for '/nix/store/hm5704d9v2c0mx5nq5ngpixyyzchq0qr-grass.drv' failed with exit code 2; last 10 log lines:
  Errors in:
  /build/source/lib/python/ctypes
  --
  In case of errors please change into the directory with error and run 'make'.
  If you get multiple errors, you need to deal with them in the order they
  appear in the error log. If you get an error building a library, you will
  also get errors from anything which uses the library.
  --
  Finished compilation: Wed Jan 20 17:39:40 UTC 2021
  make: *** [Makefile:70: default] Error 1
[0 built (1 failed), 297 copied (1797.4 MiB), 412.4 MiB DL]
error: build of '/nix/store/hm5704d9v2c0mx5nq5ngpixyyzchq0qr-grass.drv' failed

The build no longer fails, but the wxPython persists.


File a bug against projectlombok/lombok for the apparent lack of @ExtensionMethod support for interfaces

Let's try using Try.sequence as extension method on, say, GitCoreRepository:
return List.narrow(result.sequence().getOrElseThrow(GitCoreException::getOrWrap).toList().sortBy(b -> b.getName()));

If such a class like below is added to the package and then listed in @ExtensionMethod of GitCoreRepository:
package com.virtuslab.gitcore.impl.jgit;

import io.vavr.collection.Seq;
import io.vavr.control.Try;

public class Ext {
  static <T> Try<Seq<T>> sequence(Iterable<? extends Try<? extends T>> values) {
    return Try.sequence(values);
  }
}

then gitCore:jGit compiles. But an attempt to use io.vavr.control.Try directly in @ExtensionMethod ends up in compilation error (sequence method not found).
Would be nice to create a minimal repro (is it about interfaces? or maybe just generic methods in interfaces?) in a separate project, and file a bug against Lombok (unless already opened, check both opened AND closed issues). At least Lombok should mention the case of interfaces in their docs for @ExtensionMethod, if for whatever reason the support isn't possible.

I prepared a repository that shows the problem:
https://github.com/MaciejG604/lombok-bug-report
You can take a look, I will link it in the PR for lombok.

DONE - https://github.com/projectlombok/lombok/issues/3228


Can a new release be cut please to fix default export issues?

My team has been waiting for the changes introduced here to be released: https://github.com/benmosher/eslint-plugin-import/pull/1640
Thanks :-)

There will be a new release when I have time to cut one.

Can someone else release somehow? :-)
been a while
^_^

It's been 3 weeks on a project with one primary maintainer that you get for free during a global pandemic, i think you can be patient.

Thank you @ljharb for the release!
It works amazing :-)


Error: Invalid or corrupt jarfile when installing from node

I run node.js v0.10.30 on Arch-Linux and did npm install protractor -g and then webdriver-manager update.
After that i tried webdriver-manager start and I get:
Error: Invalid or corrupt jarfile /usr/lib/node_modules/protractor/selenium/selenium-server-standalone-2.42.2.jar
According to [this Stackoverflow answer] (http://stackoverflow.com/a/20684302/3085821) it seems to be a corrupt version on npm?

I am seeing SELF_SIGNED_CERT_IN_CHAIN error recently. Is there a way to resolve this issue?


circleci: prepare build matrix

renamed test-executor to linux-executor
updated linux-executor image to ubuntu-2004:202107-02
renaned mac-executor to macos-executor
updated xcode version to 13.0.0
renamed install_rust command to install_rust_linux
added install_rust_macos command
renamed rust_setup to set_rustc_stable
added set_rustc_nightly
added linux-build-stable job
renamed nightly-build-check job to linux-build-nightly
renamed msrv-build-check job to linux-build-msrv
renamed macos-build-check job to macos-build-stable
renamed setup_test to test-checks
renamed build-check to build-checks
renamed daily-checks to nightly-checks
removed todos and unused stuff and stuff we shouldn't ignore

Not sure if we need the MacOS build tests thinking

Yesterday you said, you don't know if it's building on macos.
I can also remove macos nightly and just keep stable. But one job on macos should be in place, don't you think?

I removed mac nightly in 33c0603 and decided to restore msrv of 1.52.1 for now.


Investigate possible chunking issue

See https://github.com/bmeares/Meerschaum/blob/cff0c913e31d5bf95d311fe05077493267d036a5/meerschaum/connectors/sql/_sql.py#L98

@bmeares do you have any detaills?

@Mause Sorry for the late reply, I just saw this. I'm on the road last week and next but will dive in soon when I can. Thanks for tagging the line in Meerschaum for context in case other developers want to investigate.
When I get back on my laptop, I'll post the stacktrace and give updates on what I find.

I threw together some pytest test in test_pandas.py which is in #97 .
pd.read_sql() with a chunksize other than None invokes sqlalchemy.engine.cursor.CursorResult._fetchmany_impl() which calls sqlalchemy.engine.cursor.CursorFetchStrategy,fetchmany(). The CursorResult.cursor object is None which breaks fetchmany().
 I also ran into issues when calling df.to_sql() with if_exists=True, but that's another issue we can look into another time.
I need to spend more time digging into the duckdb_engine source, but I thought this might be helpful for now. I'll post more updates when I can!

@bmeares this is hopefully fixed now - please let me know if you find any further issues 💜

Thank you for working on the chunking! I was doing some tests and was able to pull some chunks, but the chunks were the wrong size (1024 instead of the specified chunksize of 100).
I tried to reproduce the problem, but psycopg2 is getting in the way and breaking with a KeyError. Below is the stack trace:
Python 3.9.5 (default, May 14 2021, 00:00:00)
[GCC 11.1.1 20210428 (Red Hat 11.1.1-1)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pandas as pd
>>> import sqlalchemy
>>> df = pd.DataFrame({'foo': [0,1,2,3,4,5,6,7,8,9]})
>>> engine = sqlalchemy.create_engine("duckdb:///:memory:")
>>> df.to_sql('test', engine)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/bmeares/.local/lib/python3.9/site-packages/pandas/core/generic.py", line 2779, in to_sql
    sql.to_sql(
  File "/home/bmeares/.local/lib/python3.9/site-packages/pandas/io/sql.py", line 601, in to_sql
    pandas_sql.to_sql(
  File "/home/bmeares/.local/lib/python3.9/site-packages/pandas/io/sql.py", line 1411, in to_sql
    table.insert(chunksize, method=method)
  File "/home/bmeares/.local/lib/python3.9/site-packages/pandas/io/sql.py", line 845, in insert
    exec_insert(conn, keys, chunk_iter)
  File "/home/bmeares/.local/lib/python3.9/site-packages/pandas/io/sql.py", line 762, in _execute_insert
    conn.execute(self.table.insert(), data)
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1262, in execute
    return meth(self, multiparams, params, _EMPTY_EXECUTION_OPTS)
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/sql/elements.py", line 324, in _execute_on_connection
    return connection._execute_clauseelement(
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1451, in _execute_clauseelement
    ret = self._execute_context(
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1813, in _execute_context
    self._handle_dbapi_exception(
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1998, in _handle_dbapi_exception
    util.raise_(exc_info[1], with_traceback=exc_info[2])
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/util/compat.py", line 207, in raise_
    raise exception
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/engine/base.py", line 1750, in _execute_context
    self.dialect.do_executemany(
  File "/home/bmeares/.local/lib/python3.9/site-packages/sqlalchemy/dialects/postgresql/psycopg2.py", line 943, in do_executemany
    context._psycopg2_fetched_rows = xtras.execute_values(
  File "/home/bmeares/.local/lib/python3.9/site-packages/psycopg2/extras.py", line 1280, in execute_values
    sql = sql.encode(_ext.encodings[cur.connection.encoding])
KeyError: None
>>>

I've tweaked test_pandas.py in #105 to address some of the issues I ran into.
Thank you again so much for working on this. I hope I'm helping by testing, and I know duckdb_engine is still a very basic sqlalchemy wrapper so bugs are expected. Your work is greatly appreciated!


feat: make session optional in CreateContextOptions (discussion)

Is your feature request related to a problem? Please describe.
Currently we are doing:
type CreateContextOptions = {
  session: Session | null;
};

/** Use this helper for:
 * - testing, so we dont have to mock Next.js' req/res
 * - trpc's `createSSGHelpers` where we don't have req/res
 **/
export const createContextInner = async (opts: CreateContextOptions) => {
  return {
    session: opts.session,
    prisma,
  };
};

This means that when SSGing pages that don't require the session, you need to explicitly pass session: null to createProxySSGHelpers.
Describe the solution you'd like to see
Change it to
type CreateContextOptions = {
  session?: Session | null;
};

Desribe alternate solutions
Leave it as is, make people pass session: null explicitly
Additional information
One thing I don't love about this proposal is you still need to pass an empty object like
const ssgHelper = createProxySSGHelpers({
    router: appRouter,
    ctx: await createContextInner({}), // type error if empty object isn't passed
    transformer: superjson,
  });

So I'm not sure if it's actually better. Would like to hear opinions.

could do
type CreateContextOptions = {
  session: Session | {};
};

But then it's not really all that typesafe.

If the only param of CreateContextOptions is optional, what if opts itself is optional in createContextInner? Something like
type CreateContextOptions = {
  session?: Session | null;
};

/** Use this helper for:
 * - testing, so we dont have to mock Next.js' req/res
 * - trpc's `createSSGHelpers` where we don't have req/res
 **/
export const createContextInner = async (opts?: CreateContextOptions) => {
  return {
    session: opts?.session,
    prisma,
  };
};

could do
type CreateContextOptions = {
  session: Session | {};
};

But then it's not really all that typesafe.

This is not true - the thing that needs to be at least an empty object is not session, but opts. Also {} is not a type.

If the only param of CreateContextOptions is optional, what if opts itself is optional in createContextInner? Something like

This seems to work well if it is assumed that all params of CreateContextOptions are optional. This is true in the template, and probably most projects. If we use this we would have to comment the hell out of it. Will leave this open for a while if anyone has opinions :)

Doesn't really matter from a usage point, you'd still need to use the session?.user for example when accessing it.
If you think the DX is better by allowing it not to be passed I guess but I also like the explicitness and makes you think whether you should get the session or just omit it by passing null 🤷♂

Doesn't really matter from a usage point, you'd still need to use the session?.user for example when accessing properties on it - no matter if we add the ? or not.
If you think the DX is better by allowing it not to be passed I guess but I also like the explicitness and makes you think whether you should get the session or just omit it by passing null 🤷♂

After thinking about it some more I don't feel like the proposed change is meaningfully better. Closing this for now, but anyone is welcome to reopen.


sourcing of activate.sh on OSx Big Sur causes hanging in Terminal

I am using autoenv as part of my project that also employs virtualenvwrapper.  I upgraded to Big Sur recently and am installing my software.  Unfortunately, after installing autoenv using brew, adding the 'source' line to my .bash_profile and then sourcing that file, the command prompt hangs as if it is processing something.
Commenting out the source of activate.sh stops the issue.  I gave it some time, but the hanging continues indefinitely.  When I open a new terminal window, the same hanging occurs.
My system specs are:
Mac OSx 11.0.1 (20B29) (Big Sur)
Autoenv version: stable 0.2.1, HEAD
Python:  3.8.2

I attempted to trace the system calls using dtruss, but received the following:
$ sudo dtruss /usr/local/opt/autoenv/activate.sh
Password:
dtrace: system integrity protection is on, some features will not be available

dtrace: failed to execute /usr/local/opt/autoenv/activate.sh: (os/kern) failure

I took and uninstalled autoenv (install and uninstall both via pip), and re-installed it using brew.   Even after installing with brew, the issue persisted.

This is the same reason as #193 . The brew version is outdated. Install using git clone will fix this.


Zm single relay 16a

Hi @TedTolboom
Can you add support for the heatit zm single relay 16A ?

Thx
Rens Hoekema

I would love to see that as well. Anything you need that I can help with?

Same here, doesn't support it yet. Will there be any answer here?

Support has been added


 That was fine for mwc713.
For mwc-wallet  they need to go into the logs.
Problem that the same code should work with mwc713 and in mwc-wallet.
Normally we shouldn't have println calls.
Example of how this problem is solved is StatusMessage  at
mwc-wallet/libwallet/src/api_impl/owner_updater.rs
Search for 'status_send_channel: &Option<Sender>,'
To see the usage.
There are
start_updater_log_thread   - will write those message into the logs (mwc-wallet case)
start_updater_console_thread - will write those message to the console (mwc713 case)
You can extend it for mqs messages.

commit 1551bd8f6c875bc09ba960dcb2756fbb43a800f0 (HEAD -> master, origin/master, origin/HEAD)
Author: Yang Li<EMAIL_ADDRESS>Date:   Sun Apr 26 21:56:27 2020 -0700
github issue #256 mwc-wallet, mwcmqs, printing messages for the user

To verify, just check  messages are printed to log file mwc-wallet.log, not write to console directly. But console still shows some important message(warn messages).

Great, console is expecting to print logs as well. mwc-wallet toml has
settings about that.
On Tue, Apr 28, 2020 at 9:03 AM yyangli<EMAIL_ADDRESS>wrote:

To verify, just check messages are printed to log file mwc-wallet.log, not
write to console directly. But console still shows some important
message(warn messages).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mwcproject/mwc-qt-wallet/issues/256#issuecomment-620699537,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDOACGPBERG6ZR4FD4S37TRO345NANCNFSM4MM46ZDQ
.

Verified in MWC wallet, log messages are printed in the mwc-wallet.log file, hence closing this ticket.


Feature: custom views

needed all the way down in the REST API
export customized views:
tabcmd export "SuperstoreDashboard/Overview/3f3b03eb-7c77-4925-9247-64920c1d997e/EastRegion" --png
requires https://github.com/tableau/server-client-python/issues/999

This is partially implemented in tsc 0.25 https://github.com/tableau/server-client-python/releases/tag/v0.25


Dark mode

DARK MODE REQUIRED
For now, in application white is a major color with the contrast of black👀
Include the dark mode feature which is the reverse of the theme of the application🔀
Major color = Black⚫
Minor / Contrast color = white⚪

Make sure to make a toggle in the Settings page✳
Make sure you see whether the Settings page is ready or not because this toggle will be situated at the settings page.

I can help with this, feel free to assign the issue to me 🙂

Hey @diegoalvis, I am waiting for this response, If I won't get any response in 1-2 days, I will assign you this issue for sure👍
And till then, you can work on some other issues 🎯

Hey, @diegoalvis, you can start working on this issue!

Added in #45


Questions?

Hello:  Two questions.  Is there a user forum?  Is adding images to the social campaigns on the roadmap?  Thanks,  Rich

Hi Rich,
There's no user forum as of today. What do you have in mind, regarding the type of conversations and the underlying technology?
Images: Yes, it's on our roadmap. We plan to implement this within the upcoming 4-8 weeks. Is there anything specific that comes to your mind that you believe a basic implementation should cover feature-wise?

Hi gronganz!  I have been following campaignchain for a while now and really like the product and the development.  The ONLY reason I have not implemented it is because so much of my social media content is images.  The most important feature beyond the core functionality of simply adding images (regarding posting to social media) would be, to be certain image links are editable (so that the destination after clicking on an image is controllable).  Plus, sites like Linkedin won't accept posts without image links.  AND critical to the success is the ability to edit contents, dates and images without needing to delete and re-enter the whole post.  I assume a buffering/scheduling opportunity would remain like the txt posts...
As for the forums...topics.  Well, Some technical issues are solved by the community and can relieve some of the burden off of you, the developer...then feature requests and other users experiences...on both sides of the equation (open sourced as well as subscribers)  can be shared for a more positive end user participation.
My experience is that the more involved a user community is in forum activity, the easier life is oin the developers and the users.
I hope this has helped in some way!
Be well today!  regards,  Rich


install.ps1 isn't idempotent

I'm rerunning the install.ps1 in a deployment script, but it doesn't appear to gracefully handle if BurntToast has already been installed:
# command
iex ((New-Object net.webclient).DownloadString('https://raw.githubusercontent.com/Windos/BurntToast/master/install.ps1')) ;

Have you thought about putting this into chocolatey?  Its a great tool.

Expand-Archive BurntToast.zip -force should fix it.
Also, you might also want to add a flag to install.ps1 or consider adding the module system wide vs a user profile:
https://msdn.microsoft.com/en-us/library/dd878350(v=vs.85).aspx
"$Env:ProgramFiles\WindowsPowerShell\Modules"
I'm using the notification from a Scheduled Task that is run as SYSTEM, I'm guessing this might be a common scenario.

Strictly speaking, the install.ps1 file is there more for a quick and dirty install for testing if you don't feel like downloading off of the GitHub site manually and copying the contents to wherever.
It is up on the PowerShell Gallery, which is probably the better way to achieve a reliable (and customizable) install: Install-Module -Name BurntToast
Adding the -force parameter should skip any prompts, allowing for an unattended install (though it will also re-install the module if it is already on the system.)
In saying that I'll add some error handling and the like to make it a bit more usable.

Thanks, sounds good, I'll use install-module to deploy.
On 14 Jun 2016 17:11, "Joshua King"<EMAIL_ADDRESS>wrote:

Strictly speaking, the install.ps1 file is there more for a quick and
dirty install for testing if you don't feel like downloading off of the
GitHub site manually and copying the contents to wherever.
It is up on the PowerShell Gallery
http://www.powershellgallery.com/packages/BurntToast, which is probably
the better way to achieve a reliable (and customizable) install: Install-Module
-Name BurntToast
Adding the -force parameter should skip any prompts, allowing for an
unattended install (though it will also re-install the module if it is
already on the system.)
In saying that I'll add some error handling and the like to make it a bit
more usable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Windos/BurntToast/issues/3#issuecomment-226017733,
or mute the thread
https://github.com/notifications/unsubscribe/ABoqS06ZiUynQ2SdIZ3haVQAsTOEzty7ks5qLxkZgaJpZM4I1duF
.


Use Json for each Virtual Object

Handle each object by Json.  Also, we should allow adding models automatically by just adding the asset to Models.scnassets.
Resources:
Apple Project
Documentation
cc: @NordyVlasman @ericviston

Just an idea, loading online models or something. Like you download a zip file from the internet and it shows it in AR. ARKit accepts dae files..


Build wheels for the Windows platform

This might include win32 and win_amd64.

Also, set up CI to run with Windows, as part of this.


Added large document tests to Surreal DB

Note: This PR relies on fixes in #82 to work, merge this one after #82 is completed.
The tests don't currently push past the limit outlined below:
For REST is looks like there is a limit in SurrealDB somewhere between 31238 bytes and 57862 bytes.
{
    "code": 413,
    "details": "Payload too large",
    "description": "The request has exceeded the maximum payload size. Refer to the documentation for the request limitations."
}

For RPC there seems to be a limit somewhere between 17926 bytes and 31238 bytes.
System.Net.WebSockets.WebSocketException
The remote party closed the WebSocket connection without completing the close handshake.
   at System.Net.WebSockets.ManagedWebSocket.ReceiveAsyncPrivate[TResult](Memory`1 payloadBuffer, CancellationToken cancellationToken)

@Du-z Did you mean #95 instead of #82, Things with JSON keys?

@ProphetLamb Yeah your right, when #95 is merged.


[ISSUE #411] Enable CI workflows running on [0-9]+.[0-9]+.[0-9]+** branches

See #411.
It worked in the repo I forked.

See #411.
It worked in the repo I forked.

thanks @SteveYurongSu , could you please refect pull_request_target in greetings.yml to pull_request?

See #411.
It worked in the repo I forked.

thanks @SteveYurongSu , could you please refect pull_request_target in greetings.yml to pull_request?

Sure! But let me do it in anothor PR and link it to #406.


Fecundity and Atresia codes (for SmartDots)

Codes need to be added to Fecundity and Atresia in SmartDots,
it is several vocabulary types.

Stage
Description
Scale
Questions

YV
YolkVesicle
Atresia

YV_YG
YolkVesicle_YolkGranule
Atresia

YG
YolkGranule
Atresia

Neg_grid
Negative_grid
Atresia

Extra
Extra
Atresia

YV_P
YolKVesicle_Profile
Atresia

YV_YG_P
YolKVesicle_YolkGranule_Profile
Atresia

YG_P
YolKGranule_Profile
Atresia

Oocyte_stage_1
Previtellogenic   oocytes
Histological   Screening

Oocyte_stage_2
Early vitellogenic   oocytes <400 um
Histological   Screening

Oocyte_stage_3
Vitellogenic oocytes
Histological   Screening

Oocyte_stage_4
Migratory nucleus   oocytes
Histological   Screening

Hydration_state_1
Early hydration
Histological   Screening
Question suggestion:   Are hydrated oocytes present? Answer: Yes or No

Hydration_state_2
Advanced hydration
Histological   Screening

Hydration_state_3
Late hydration
Histological   Screening

Egg_stage_1
New eggs present
Histological   Screening
Question suggestion:   Are eggs present? Answer: Yes or No

Egg_stage_2
Residual eggs present
Histological   Screening

POFs
Histological   Screening
Question suggestion:   Are POFs present? Answer: Yes or No

spent
Histological   Screening
Question suggestion:   Is the ovary spent? Answer: Yes or No

massive atresia
Histological   Screening
Question suggestion:   Is massive atresia present? Answer: Yes or No

early alpha atresia
Histological   Screening
Question suggestion:   Is Early atresia present? Answer: Yes or No

Discarded
Histological   Screening
Question suggestion:   Should the sample be discarded? Answer: Yes or No

POF_stage_1
Newly formed
POF Staging

POF_stage_2
Folded
POF Staging

POF_stage_3
Vacuolated
POF Staging

POF_stage_4
Compacted
POF Staging

POF_stage_5
Decreased sized
POF Staging

POF_stage_6
Advanced
POF Staging

POF_stage_7
Residual
POF Staging

Two codetypes for - one for Atresia scale/type and one for the Stages.
No underscore and better description. Codes will be invisible so Carlos can do the tests.

Codes implemented, but kept invisible while waiting for better descriptions.
Joana to contact Martia Korta, Cindy and Anders for better descriptions.

POF - post ovulatory folicule

@evabrna will find descriptions in SISP -5,  ask Cindy in the upcoming meeting.

@evabrna to update the descriptions.

Stage | Description | Detailed description | Scale |
-- | -- | -- | -- | --
YV | YolkVesicle | Number of Yolk Vesical Grid   Points in the entire sample. | Atresia_WGMEGS
YV_YG | YolkVesicle_YolkGranule | Number of Yolk Vesical-Yolk   Granule Grid Points in the entire sample. | Atresia_WGMEGS
YG | YolkGranule | Number of Yolk Granule Grid   Points in the entire sample. | Atresia_WGMEGS
Neg_grid | Negative_grid | Number of points for negative   grid count Number of points for negative grid count, i.e. grid points outside   the ovary for the entire sample | Atresia_WGMEGS
Extra | Extra |   | Atresia_WGMEGS
YV_P | YolKVesicle_Profile | Number of Yolk Vesical cells   (point counts). | Atresia_WGMEGS
YV_YG_P | YolKVesicle_YolkGranule_Profile | Number of Yolk Vesical-Yolk   Granule cells (point counts). | Atresia_WGMEGS
YG_P | YolKGranule_Profile | Number of Yolk Granule cells   (point counts). | Atresia_WGMEGS
Oocyte_stage_1 | Previtellogenic oocytes | No white vacuoles   (cortical alveoli) visible. | Histological Screening
Oocyte_stage_2 | Early vitellogenic oocytes <400 um | Smallest vitellogenic   stage. White vacuoles visible. Yolk can be visible as well. Oocytes < 400   μm | Histological Screening
Oocyte_stage_3 | Vitellogenic oocytes | White vacuoles and yolk granules   visible. It is not necessary to     see vacuoles and granules at one time. Oocytes are between 400     and 800 μm. | Histological Screening
Oocyte_stage_4 | Migratory nucleus oocytes | Nucleus is migrating   from the middle of the oocyte towards the     side of the cell. The envelope of the nucleus breaks down, and     the nuclear contents blends with the surrounding cytoplasm. | Histological Screening
Hydration_state_1 | Early hydration | Yolk granules are   beginning to fuse together. Some     water has been taken up by the oocyte | Histological Screening
Hydration_state_2 | Advanced hydration | in the SISP-5 it is   refered as Mid hydrated….A large amount of water has been taken up by   the     oocyte. Interior and some cytoplasm still visible | Histological Screening
Hydration_state_3 | Late hydration | The oocyte is fully hydrated.   Yolk granules & cytoplasm     are no longer visible. The follicle layer is still attached. | Histological Screening
Egg_stage_1 | New eggs present | New eggs present | Histological Screening
Egg_stage_2 | Residual eggs present | Residual eggs present | Histological Screening
POFs | POFs | postovulatory follicle | Histological Screening
Spent | spent | A spent ovary is   characterised by the absence of normal vitellogenic oocytes at the end of   the     spawning period for that individual. Atretic vitellogenic oocytes and POFs   may be visible. | Histological Screening
MassiveAtresia | massive atresia | Massive atresia is   classed as when 90% or more of the vitellogenic oocytes in the ovary are   atretic.     In case of massive atresia, the fish will not spawn anymore that season. | Histological Screening
EarlyAlphaAtresia | early alpha atresia |   | Histological Screening
Discarded | Discarded | Discard sample for further   analyses - yes / no. For reasons other than in columns Oocyte Stage, Egg   Stage or Spent. If Yes, the Notes column will need to be filled. | Histological Screening
POF_stage_1 | Newly formed | different descriptions for horse   mackerel and mackerel | POF Staging
POF_stage_2 | Folded | different descriptions for horse   mackerel and mackerel | POF Staging
POF_stage_3 | Vacuolated | different descriptions for horse   mackerel and mackerel | POF Staging
POF_stage_4 | Compacted | different descriptions for horse   mackerel and mackerel | POF Staging
POF_stage_5 | Decreased sized | different descriptions for horse   mackerel and mackerel | POF Staging
POF_stage_6 | Advanced | different descriptions for horse   mackerel and mackerel | POF Staging
POF_stage_7 | Residual | different descriptions for horse   mackerel and mackerel | POF Staging

Descriptions updated by Cindy (find the file here P:\DP1\projects\Fecundity&Atresia\Documents)

Stage
Description
Detailed description
Scale

YV
YolkVesicle
Number of Grid Points in the subsample using a Weibel grid which touch   Yolk Vesical cells.
Atresia_WGMEGS

YV_YG
YolkVesicle_YolkGranule
Number of Grid Points in the subsample using a Weibel grid which touch   Yolk Vesical-Yolk Granule cells.
Atresia_WGMEGS

YG
YolkGranule
Number of Grid Points in the subsample using a Weibel grid which touch   Yolk Granule cells.
Atresia_WGMEGS

Neg_grid
Negative_grid
Number of Grid Points in the subsample using a Weibel grid for negative   grid count, i.e. grid points outside the ovary wall.
Atresia_WGMEGS

Extra
Extra
Number of Grid Points in the subsample using a Weibel grid which are   extra, i.e. those touching empty areas within the ovary that occur due to   fixation of the sample.
Atresia_WGMEGS

YV_P
YolKVesicle_Profile
Total Number of Yolk Vesical cells (point counts) in the subsample.
Atresia_WGMEGS

YV_YG_P
YolKVesicle_YolkGranule_Profile
Total Number of Yolk Vesical-Yolk Granule cells (point counts) in the   subsample.
Atresia_WGMEGS

YG_P
YolKGranule_Profile
Total Number of Yolk Granule cells (point counts) in the subsample.
Atresia_WGMEGS

Oocyte_stage_1
Previtellogenic oocytes
No yolk vesicles (white cortical alveoli) or yolk granules visible in the   oocyte. Diameter less than 180 μm.
Histological Screening

Oocyte_stage_2
Early vitellogenic oocytes <400 um
Smallest vitellogenic stage, with oocytes < 400 μm. At this stage yolk   visecles (white cortical alveoli) are visible in the oocyte. Yolk granules   can be visible as well.
Histological Screening

Oocyte_stage_3
Vitellogenic oocytes
Oocytes are between 400 and 800 μm in diameter. At this stage yolk   vesicles (white cortical alveoli) and yolk granules are visible in the   oocyte.
Histological Screening

Oocyte_stage_4
Migratory nucleus oocytes
In this stage the nucleus is migrating from the middle of the oocyte   towards the periphery of the cell. Lipids start to concentrate and will fuse   into one drop. Stage ends when yolk granules start to fuse and hydrate.
Histological Screening

Hydration_state_1
Early hydration
The yolk granules in the oocyte are beginning to fuse together anf   hydrate.
Histological Screening

Hydration_state_2
Advanced hydration
The majority of the yolk granules have fused and hydrated to larger   sizes.
Histological Screening

Hydration_state_3
Late hydration
The oocyte is fully hydrated and only one big yolk vacuole is present.   Yolk granules & cytoplasm are no longer visible. The follicle layer is   still attached.
Histological Screening

Egg_stage_1
New eggs present
Complete fusion and hydration of the oocyte and the follicle layer has   disappeared.
Histological Screening

Egg_stage_2
Residual eggs present
Hyaline eggs that remain in the ovary after a batch is spawned. These   eggs show signs of degradation and/or resorption.
Histological Screening

POFs
POFs
One or more Postovulatory Follicle(s) present in the ovary.
Histological Screening

Spent
spent
A spent ovary is characterised by the absence of healthy vitellogenic   oocytes at the end of the spawning period for that individual. Atretic   vitellogenic oocytes and POFs are present and residual eggs may be visible.
Histological Screening

MassiveAtresia
massive atresia
Massive atresia is classed as when 90% or more of the vitellogenic   oocytes in the ovary are atretic. In case of massive atresia, the fish will   not spawn anymore that season.
Histological Screening

EarlyAlphaAtresia
early alpha atresia
Alpha atresia begins when there is a cut through one or both chorion   layers. The break is smaller than twice the width of the chorion thickness in   any of the two chorion layers. The edge of the oocyte is smooth.
Histological Screening

Discarded
Discarded
Discard sample for further analyses - yes / no. For reasons other than in   columns Oocyte Stage, Egg Stage or Spent. If Yes, the Notes column will need   to be filled.
Histological Screening

POF_stage_1
Newly formed
Large POF and cordlike structure well defined.
POF Staging

POF_stage_2
Folded
POF with line structure but with evident loops.
POF Staging

POF_stage_3
Vacuolated
POFs still large, with folds clearly recognisable.
POF Staging

POF_stage_4
Compacted
POFs still large but compact. Folds barely distinguishable.
POF Staging

POF_stage_5
Decreased sized
POFs compact and folds indiscernable.
POF Staging

POF_stage_6
Advanced
POFs size reduced strongly and seem a mass of disordered cells and   vacuoles.
POF Staging

POF_stage_7
Residual
Very small POFs and scarce.
POF Staging

To check if the description changes are implemented.


Saving changes to very large relations takes a very long time

I think iD might have an issue editing very large relations. I made a minor edit to the Lake Huron relation and it took Chrome a very long time (> 1 hour) to finish saving the changes (in fact, it's still chugging away).* While it was in the process of saving Chrome's memory usage for that tab was very high:

Chrome version 60.0.3112.113 on Windows, iD version 2.4.1
*Once my changeset finishes saving, I'll link to it here. I had a similar problem yesterday with this changeset but the one I'm saving today is much simpler and it still is taking a long time to save.

Yes, I'm not surprised.. I think this is #3056

Makes sense! FYI, it looks like my changeset from yesterday took about four hours to finish saving. I'm a little concerned (maybe without justification) that a conflict will crop up while iD is running its checks.

FWIW, my other changeset did successfully save, eventually. Here's a link, in case it helps to have a cleaner example.
Thanks for all your work on iD! I use it just about every day.

Thanks for all your work on iD! I use it just about every day.

Thanks for the kind words! ...

closing as duplicate of  #3056


Issue with ELB annotations when using websockets

This works fine for plain https, but breaks on websockets:
apiVersion: v1
kind: Service
metadata:
  name: istio-ingressgateway
  namespace: istio-system
  annotations:
    service.beta.kubernetes.io/aws-load-balancer-ssl-cert: "arn:aws:acm:us-west-2:xxx:certificate/xxx-xxxx-xxxx-xxxx-xxx"
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: "http"
    service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "https"
    service.beta.kubernetes.io/aws-load-balancer-connection-idle-timeout: "300"

We have to manually change https -> http to ssl -> tcp on 31390 with our cert in the AWS console.
Is there anyway to configure this to come up correctly?

Deleting the service.beta.kubernetes.io/aws-load-balancer-backend-protocol: "http" made everything TCP, but this seems to work so I think all is good.

@mabushey The downside with that solution is that the TCP packet's client-ip is not preserved, but is replaced by the Classic LoadBalancer's internal IP. The CLB won't add X-Forwarded-For header to the packet either because it does not recognize it as HTTP. This might matter to some applications like geoip.
There are only 3 solutions to this while maintaining the use of websockets/non-HTTP protocols:

Use NLB instead of CLB, which preserves client-ip (best solution; but needs eks 1.15 if using istio)
Enrich the X-Forwarded-For header using a separate HTTP proxy/LB that sits in-front of the service's CLB.
Simply use separate LoadBalancers for each protocol, with HTTP one having the following annotation: service.beta.kubernetes.io/aws-load-balancer-backend-protocol: "http"


834 Benefit Enrollment and Maintenance (#1)

Init 834 implementation.
Thought I would contribute back since I am using this gem for the 834.
Hopefully this will help out others who have to implement the Benefit Enrollment and Maintenance spec.

Michael,
Thank you!
Ill review the files and add them to the core codebase.  Next update will unclude them
Marty
Sent from BlueMail
On Dec 3, 2016, 5:37 PM, at 5:37 PM, michael chrisco<EMAIL_ADDRESS>wrote:

Init 834 implementation.
Thought I would contribute back since I am using this gem for the 834.
Hopefully this will help out others who have to implement the Benefit
Enrollment and Maintenance spec.
You can view, comment on, or merge this pull request online at:
https://github.com/mjpete3/x12/pull/11
-- Commit Summary --

834 Benefit Enrollment and Maintenance (#1)

-- File Changes --
M .gitignore (1)
M README.md (2)
M lib/x12/segment.rb (12)
A misc/834.xml (85)
M misc/BGN.xml (12)
M misc/HD.xml (18)
M misc/INS.xml (26)
M misc/N1.xml (8)
A test/tc_factory_834.rb (265)
A test/tc_parse_834.rb (100)
-- Patch Links --
https://github.com/mjpete3/x12/pull/11.patch
https://github.com/mjpete3/x12/pull/11.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mjpete3/x12/pull/11

Forgot to make an issue :smile:
This PR closes #12


Downloading content disposition attachment.

I tried to download a pdf file from a download.php site i got nothing but when i try using pdf viewer it work so that's mean cef doesn't support content disposition http  or what ? it's streamed chunk file ? Any help

I'm not familiar enough with issue to be helpful. Best bet would be to look at CEF/chromium forums/mailing lists under similar topics and once you know the configs/handlers to change to support it, make a pull request here.
Some resources:
https://stackoverflow.com/questions/57465474/content-disposition-headers-to-allow-both-download-and-open-file
https://bugs.chromium.org/p/chromium/issues/detail?id=487906


[Subtask]: Add shaded fastjson project

Search before asking

[X] I have searched in the issues and found no similar issues.

Description
Add shaded fastjson project
Parent issue
#2628 Add amoro shaded project
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Could this issue be closed due to shade package has been introduced ? @klion26

@baiyangtx yes this can be closed for fastjson shaded has been implemented


Updating Butterknife to latest version (7.0.1)

This should be everything to get Butterknife in to the latest version.  I have run the unit tests, and verified they are all passing.  Let me know if there is something that seems off.

Thanks!


chore: more badges

include more badges

No sure but is it possible to setup an automatic push with GHA to docker hub? IIRC the integration has been disabled, correct me if I am wrong.


[#199] If a board is deleted and an user try to access it, show a better message

Waiting for suggested changes so I can approve this

Thanks for the comments, I'll try to get back to it this weekend


fix: precache n-1 and n+1 pages

Close #348
What does this PR do?

Precache n+1/n-1 page for better experience. (n - 1 also has this problem, run yazi and press G)

Test
https://github.com/sxyazi/yazi/assets/9482395/f4d1127e-588a-49e7-bc03-2d62c91c2767

Thanks for your PR!
I've changed it to use a bound as the condition, which should make it more precise. Would you like to give it a review?

Thanks for your help. And I found another case that hasn't been fixed. It seems that the n+1 is not cached after the initial load. I push a new commit for this.
https://github.com/sxyazi/yazi/assets/9482395/90b77ab8-0abf-4315-9f5f-b8e0261ca6b4

So weird, I can't reproduce it, not sure what's going wrong...

Ah, I reproduced it when I enlarged my terminal window. I created a new commit that should fix it.

Tested it locally, and it works as expected. Thank you! :tada:

Okay, let's merge it now. Thanks for your test!


Run azd up  fail

Describe the issue:
When executing azd up according to README.md, the error is as follows:

Repro Steps:

Clone the code to local.
Run azd up.

Environment:

OS: Windows desktop.

Expected behavior:
Run azd up can pass.
@hemarina for notification.

Thank you for the report.
This needs to sync with this change:
https://github.com/Azure-Samples/azure-search-openai-demo/pull/322

@paulyuk Retest when I synchronize this change https://github.com/Azure-Samples/azure-search-openai-demo/pull/322. At first, I got the following error：

But when I execute azd up again, it works fine.

Wow, thank you for trying after a sync.
There is one special workaround I notice they did in main.bicep which makes that error go away:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/21bac5de946669390014b278c5cf3208a06f1a16/infra/main.bicep#L61C39-L61C39
Essentially, they made another reference to the resource group, and created an implicit Dependson relationship from that resource group.  With depends on in place, the RG will always get created before the AI resource.  For some reason, today, in ARM/bicep there is a timing issue!
My PR will have the full fix.  it's coming momentarily.

Beware to anyone testing this issue and hitting the quota issues, you will likely need to purge your deleted resources first to free up capacity before retrying.
https://learn.microsoft.com/en-us/azure/cognitive-services/manage-resources?tabs=azure-portal#purge-a-deleted-resource

Fixed by #4


added research outputs to the CSV download of the plan

A DMPTool user reported that the research outputs were not appearing on the CSV export of the Plan. This PR adds a research outputs section to the end of the CSV.

@briri The research outputs weren't showing, I added the show_research_outputs option that was missing in the as_csv function.
If it's okay for you, I'll merge this PR :)

Thanks @benjaminfaure


APPSODY_WATCH_REGEX is insufficiently documented

Describe the bug
It's unclear from the current documentation how to correctly code anything beyond simple regex's for the APPSODY_WATCH_REGEX env var content.
Eg. Given a change to src/main/test.sh living in the container at /project/user-app/src/main/test.sh where the /project/user-app is the home dir for the project.. what string is being matched against for the regex ? is it test.sh, src/main/test.sh or /project/user-app/src/main/test.sh ?
The documentation should also state which regex syntax is in use, there are many. I'm assuming go, because I know the controller is written in that, but it's not documented as such, and other regex syntaxes are more common & popular.

@neeraj-laad can you describe what we need to add here and then we can pass to someone else?

I think we should document a number of aspects along with relevant examples for stack providers to understand this better:

Explain what is the regex going to be tested against (full path in the container, the path within the watch dir, only file name). Some examples here will be helpful.
Clearly articulate what is allowed and what is not allowed in this regex.
Briefly explain how the appsody watcher deal with file changes, and how stack creators can use some other variables to get optimum file watching experience - watch dir, watch interval, watch ignore dir etc.


precheck(dm): check downstream's privileges

What problem does this PR solve?

Issue Number: close #7601
What is changed and how it works?

add a new checker to check the downstream's privileges

Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/run-verify
/run-dm-integration-test

/cc @D3Hunter @lance6716

/run-dm-integration-test

Codecov Report

Merging #7614 (951fc22) into master (0438dd4) will decrease coverage by 0.0871%.
The diff coverage is 32.9741%.

Additional details and impacted files

Flag
Coverage Δ

dm
52.0831% <14.7058%> (+0.0003%)
:arrow_up:

engine
63.7702% <ø> (-0.0158%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #7614        +/-   ##
================================================
- Coverage   60.0458%   59.9587%   -0.0872%
================================================
  Files           807        808         +1
  Lines         92391      92682       +291
================================================
+ Hits          55477      55571        +94
- Misses        32099      32291       +192
- Partials       4815       4820         +5


Fixed a failure of a travis script iff no github release exists

Issue

Related to #706

Overview (Required)

jq returned null so Bash paniced 🤷♂

Let me merge. 🙏


Unable to drag hammer element

I am trying to implement a drag event for my hammer object but so far it is not eve detecting the touch event . I am not sure what I am doing wrong

Hammer.js 2.x does not include the drag event it was removed.


Feature request

Hey. Love the simplicity of this addon. I would just like to make a couple of suggestions.

I would really like it if it was easy to see at a glance what class the person who died was. Maybe just have their name be in the class colour? Currently there's no way to see that. The test alerts show the class icon but this doesn't show in the actual alerts.

Would it be possible to have an option to display the zone they died in?

Cheers.

Hey @evilcream
Thanks for using the addon and the suggestions.

Agreed. Great idea for the name and class color combination. This would be a good design pattern to follow. Currently, death logs sourced from the /HardcoreDeaths channel don't have all of their details (like class) parsed. Though, it should be doable.

Agreed. Want to add this next. In the future, I'd like to include other details like race, character portrait, etc.

I'll be able to work on these in the new year. Let me know if you have any more feedback. Thanks!


Triggering changeDetection on back navigation

Environment

CLI: 5.3.1
Cross-platform modules: 5.3.1
Android Runtime:5.3.1
iOS Runtime:5.3.1
NativeScript-Angular:7.2.3
Angular:7.2.0

Describe the bug
The change detection is not triggered for the page when we navigate back to it. A similar case is when we have made a change, which requires an update on the content of the previous page. The issue is reproducible only while using OnPush change detection strategy.
Reported via admin. t.1407062

the fix is in both release and master branches


make markdown-link-check fail when broken link is found

Currently using markdown-link-check in a gitlab CI pipeline.
I'am looking for a way to make the pipeline fail if a broken link is found.
In general the pipeline fails if a commands exit code is not 0.
I guess I will get it to work with a mixture of grep-ing the output log created with tee and some additional bash magic.
But having a way to get errors returned directly would be way cleaner.
What do you think about a option like -f --fail-on-error which lets markdown-link-check return exit code 1 on first/any error?

@belorenz Can you clarify it a bit? markdown-link-check does return exit code as (1) when a check fails.
▶ $ cat test.md
# Title

## Next title.

[some link](https://www.google.com)

[wrong link](https://www.google2233222.com)

▶ $ markdown-link-check test.md

FILE: test.md
[✓] https://www.google.com
[✖] https://www.google2233222.com

2 links checked.

ERROR: 1 dead links found!
[✖] https://www.google2233222.com → Status: 0

▶ $ echo $?
1

The issue here is the use of tee in the pipe. Tee always returns 0 and as it's the last command in the pipe like in markdownlink command | tee logfile that enables to have the output on the console and in the log file at the same time (very useful).
So to check if it failed, on workaround is to grep the log file to check for errors, see one of my project as example:
    "test:links:check": "find . -name \\*.md ! -path \"./node_modules/*\" ! -path \"./env/*\" -exec ./node_modules/markdown-link-check/markdown-link-check -v -c common/build_tools/linkchecker/link_check_conf.json {} \\; 2>&1 | tee linkchecker.out",
    "test:links:verify": "! grep -n '[✖]\\|ERROR' ./linkchecker.out",

This is the package.json scripts I use. First one test:links:check runs link checker and pipes in tee but as the output of it will always be 0, then I run test:links:verify on the log file and grep for errors. That's not very nice I know.
On way to prevent that is to use set -o pipefail bash option. It will then prevent the error to be masked by the tee command.
It doesn't require any change on the markdownlink level.
I planned to test it and see how the output is good for my own use. Feel free to tell if it suits your need too.

Thanks for helping out!
I thought I pasted my workaround here, but cannot see it anymore. Anyhow, the reason for not failing was, as you mentioned, the use of tee.
Indeed our workaround is not very beautiful but works like a charm.
Sorry for the noise :)


Request ID should be sometimes undefined

The event argument type passed to the onSubmit callback (and probably others, I didn't check) can be undefined when sandbox mode is enabled. However, the type definition says it's always a string.
Can be seen when using React
<Widget
  onSubmit={(event) => console.log(typeof event.responseId)}
  sandboxMode={true}
/>

event.responseId is undefined when sandbox mode is enabled, but the type definition is string, not string | undefined.
The fix
Update this line to be
  onSubmit?: (event: WithFormId & Partial<WithResponseId>) => void

However, that may indicate responseId is missing, not set to undefined, so you could also update WithResponseId to be { responseId: string | undefined }, but I'm not sure where else that type is used and what the repercussion is.

Hello @jkjustjoshing
Good catch! From the top of my head I dont see any issues with updating this to { responseId: string | undefined }. I think we can update the type.


[OpenAI] [Assistants] Update CHANGELOG.md

Packages impacted by this PR
@Azure/openai-assistants
Issues associated with this PR
Describe the problem that is addressed by this PR
Update changelog for next release
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.


Color and theme

color and theme keeps going back to default after editing profile or disconnecting

Thanks for the issue! This is definitely on the list to fix.


RQueue Poll blocking

Expected behavior
calling RQueue.poll() or RQueue.peek() either returns the object from the queue or null and it should not wait . This issue is noticed only in redisson-all-3.23.2.jar  and above
Actual behavior
calling RQueue.poll() does not return null in case of empty Queue and the call waits.
Steps to reproduce or test case
obj = queue.poll(); // WATING and DOES NOT COME OUT
Redis version
Redisson version
redisson-all-3.23.2.jar and above
Redisson configuration

Unable to reproduce with the latest version.
        RQueue<Integer> queue = getQueue();
        assertThat(queue.poll()).isNull();
        assertThat(queue.peek()).isNull();


Tracing the http request id across the cf apps

Hi
Is there a way we can track the http request going from one app to another app. For example in our product  we have multiple services. When a user performs an action in the UI the request goes from cf app A to B the go router generates a new request Id every time it goes to an app. We would like track the initial request originated from the cf app A  when we monitor the cf logs in app B. Ideally if there is a unique request id  for every request originated from user action, it will be very handy for troubleshooting purpose. Let us know if there is another way we can track the requests. Appreciate any help
Thanks
Kishore

@kishorereflection That definitely sounds like a valid use case. As you say, the GoRouter does not currently support that type of request tracing, perhaps we could think about doing something to help applications in this area though.
The hard part about handling tracing multiple requests in the GoRouter is that we do not know anything about what requests are related or not. Perhaps having an application-specific header that is passed through by your applications would help enable this type of tracing?
@crhino && @fordaz

@kishorereflection Some time ago we merged PR #92 to add generic support for this use case. When deploying cf-release, you may configure a list of headers that gorouter will include in logs if they are present in requests.
This configuration property is: router.extra_headers_to_log
For example, if you configured your BOSH deployment manifest with router.extra_headers_to_log: ["X-My-Trace-Header"], then updated your apps to pass a unique value back and forth for each request, you'd be able to trace request a request across applications.
I'll close this story for now. Please reopen if you have any additional questions.