No include folder after INSTALL project is made

Hi,
I have used the following configurations
1-MS visual studio 2015 64x
2-Qt5
3-VTK9
4-DCMTK (latest version)
VTK and DCMTK already installed on my computer and I configured the CTK to used them.
So far so good, CTK libraries widgets (DICOM and VTK) are make
but when I build the INSTALL project I expect 4 folders: bin, lib,CMAKE and include
to appear in my install destination. Except CMAKE, non of the other folders comes up.
Do you have any idea what should I do?
Thanks

Have you set CTK_INSTALL_..._DIR variables?
Make sure you install the internal (CTK-binary)\CTK-build\CTK.sln project.

thanks

Have you set CTK_INSTALL_..._DIR variables?
Make sure you install the internal (CTK-binary)\CTK-build\CTK.sln project.

Hi,
I encountered the same problem. I can find the installed CTK in the BUILD library but I would like to install it to C:\apps\CTK
For now I'm trying to do it by editing CMakeLists.txt in CTK-master folder as follows:

set(CTK_INSTALL_BIN_DIR "$C:/apps/CTK/bin")
set(CTK_INSTALL_LIB_DIR "$C:/apps/CTK/lib/ctk-${CTK_MAJOR_VERSION}.${CTK_MINOR_VERSION}")
set(CTK_INSTALL_PLUGIN_DIR "$C:/apps/CTK/lib/ctk-${CTK_MAJOR_VERSION}.${CTK_MINOR_VERSION}/plugins")
set(CTK_INSTALL_CMAKE_DIR "$C:/apps/CTK/lib/ctk-${CTK_MAJOR_VERSION}.${CTK_MINOR_VERSION}/CMake")
set(CTK_INSTALL_INCLUDE_DIR "$C:/apps/CTK/include/ctk-${CTK_MAJOR_VERSION}.${CTK_MINOR_VERSION}")
set(CTK_INSTALL_DOC_DIR "$C:/apps/CTK/doc")

Does it correct? Or how to set the install directory to C:\apps\CTK?
P.S. earlier I used to change INSTALL_PREFIX in CMake GUI but now it didn't work :(

You should configure CTK passing these options from the command line (or from a CMake external project like it is done here)
Variables CTK_INSTALL_*_DIR should be set with a relative path not an absolute path
If you would like to install CTK in a specific directory, you should set CMAKE_INSTALL_PREFIX
For example, first you configure:
cd CTK-build
cmake -G "Visual Studio 16 2019" -A x64 -DCMAKE_INSTALL_PREFIX:PATH=C:\apps\CTK  ..\CTK

The,  you build:
cmake --build . --config Release -- /maxcpucount:4

and  finally you install:
cmake --build .\CTK-build --config Release --target INSTALL


Add extension to search files in Google Drive

Description
Adds an extension to fuzzy search files in the locally mounted Google Drive
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @vishaltelangre 👋
Thanks for your contribution 🔥I'm already reviewing #1354, which seems to do the same.
Maybe you and @nametake can collaborate if your version has anything besides that.
What do you think about that?

Thanks, @pernielsentikaer.
From the initial looks, it seems both #1354 and this PR can be merged together (if @nametake is willing to do so).
This extension adds the following major things -

Fuzzy search the absolute path (overrides the default search feature, see the third screenshot above)
Sorts results by match score + file's modified date (so recently modified files appear first)
File details on the right-side detail panel (see the above screenshots)
Preference to show/hide hidden files in the search results
Some error handling

The extension added in #1354 seems to have these features -

Preference to specify the mounted Google Drive's root path
Preference to specify a set of paths to exclude from the search results
File search

I may be biased, but it seems that this PR has more convenient features in quantum if compare with #1354. I like the preference to explicitly configure the mounted Google Drive's root path (can default to ~/Google Drive like in this PR) from #1354. If @nametake is okay with it, maybe we can copy that feature from his PR and add it to this and continue with this PR?
Besides these, I also have a couple of things in mind that we can add further to this extension  -

Ability to star/unstar a result
Either

Divide results into "Favorites" and "Recent files" sections (when the search term is empty)
OR add right-side dropdown filters - "Recent files" and "Favorites"

Build a cache of all the file paths + file details in LocalStorage (recurringly, say daily) so as to speed up the initial boot (useful if one has a Google Drive containing tons of files)

@pernielsentikaer @nametake Wdyt?

Thank you @vishaltelangre and @pernielsentikaer !
I also think #1503 extension is more convenient than #1354.
So I'll close #1354.
If possible, I would like to see the directory exclusion feature implemented.
Thanks!

If possible, I would like to see the directory exclusion feature implemented.

@nametake Absolutely. It would be great if you could contribute that feature (because it is your idea and you had actually implemented it in your version!) once this gets merged in.
--
@pernielsentikaer I think this is ready for the initial review (and for the initial release, feature-wise!). I have added file preview support as well.

@pernielsentikaer Interesting. That setting seems to work as expected for me.

Are you in Slack, catch me there and we can try debug it 😄

Pinged you in Slack!

@nametake Since this has been merged and published, cannot wait for your PR to add the "directory exclusion feature"! Thanks.

Hi there, this extension is pretty pretty important nowdays due to the lack of official support by Apple (spotlight) for Google mounted Drives. So thanks a lot!!!
Just a question/suggestion? Could you add this extension to the possibles fallback extensions? So we could use more easily alike File Search extension?

Thanks a lot! you saved my life!

@afanjul Glad to hear that you are using this extension. It looks like there's no public API to make it act as a fallback search provider. There's already a similar open request (https://github.com/raycast/extensions/issues/1668) that maybe you can upvote to bump up its priority. Thanks!

@afanjul Glad to hear that you are using this extension. It looks like there's no public API to make it act as a fallback search provider. There's already a similar open request (#1668) that maybe you can upvote to bump up its priority. Thanks!

Thanks, I have added a comment and an upvote on https://github.com/raycast/extensions/issues/1668
I hope they release a Fallback API or anything


Allow to change lambda name in api.route transform

We noticed that SST is trying to make sure all resources have a unique name by giving them random names. In some cases it's possible to adjust the names using the transform but for api.route it's not possible the change the lambda function's name.
It would be nice to allow this option.

Relates to the following issue: https://github.com/sst/ion/issues/244
In general I think it would be better to be:

Explicit about naming, you can always provide a name (and it will be prefixed with $app-$stage as usual), if not then the resource name can be used. This allows users to use whatever naming convention they already have.
When linking specify the name to use for linking like: link: [{ userDB: accountDynamoDb}] this would expose accountDynamoDb as Resource.userDB

Regards

Can you all share what changing the names helps you with? Is it for finding things in the AWS Console? Is it a policy at your team?

@jayair Hi, mainly policy and convention. Working for an agency means we have a lot of customers and different developers. If everyone uses the same naming conventions projects are easier to maintain by different developers.
For this issue specifically the route lambda name is not sensible, hence we would like to customize it. It also does not follow our naming convention

@jayair Indeed, both of them to be honest. In general I'm confused by the naming conventions of the resources created by sst. But as long as they can be overwritten it's fine I guess 😉

You can now pass in name for a function - https://ion.sst.dev/docs/component/aws/function/#name
api.route("GET /", {
  name: "my-function",
  handler: "path/to/handler.index",
});


Unable to load CelebA dataset: "File is not zip file" error.

🐛 Bug
I am unable to download the CelebFace Attributes (CelebA) dataset using TorchVision. The error I get when attempting to download is "File is not a zip file."
I have encountered this error every time I've tried to download the file, over the course of the last 24 hours.
To Reproduce
Steps to reproduce the behavior:

Import torchvision
Call torchvision.datasets.CelebA(root='somewhere', split='train', download=True)

A code sample is provided below:
import torch as tc
import torchvision as tv

def get_dataloaders(batch_size):
    transform = tv.transforms.Compose([
        tv.transforms.CenterCrop(108),
        tv.transforms.Resize(64),
        tv.transforms.ToTensor(),
        tv.transforms.Normalize((0.5, 0.5, 0.5), (0.5, 0.5, 0.5))  # scales pixels to [-1, 1].
    ])
    train_data = tv.datasets.CelebA(root='data', split='train', download=True, transform=transform)
    test_data = tv.datasets.CelebA(root='data', split='test', download=True, transform=transform)

    train_dataloader = tc.utils.data.DataLoader(train_data, batch_size=batch_size, shuffle=True)
    test_dataloader = tc.utils.data.DataLoader(test_data, batch_size=batch_size, shuffle=True)

    return train_dataloader, test_dataloader

Here, my stacktrace is provided as well:
/Users/lucaslingle/opt/miniconda3/envs/pytorch181/bin/python /Users/lucaslingle/PycharmProjects/pytorch_dcgan/main.py
3112it [00:00, 2727831.57it/s]
26721026it [00:01, 23197510.91it/s]
3424458it [00:00, 12672950.45it/s]
6082035it [00:00, 17442168.31it/s]
12156055it [00:00, 21823850.99it/s]
2836386it [00:00, 11141828.55it/s]
Traceback (most recent call last):
  File "/Users/lucaslingle/PycharmProjects/pytorch_dcgan/main.py", line 35, in <module>
    dataloader, _ = get_dataloaders(batch_size=batch_size)
  File "/Users/lucaslingle/PycharmProjects/pytorch_dcgan/dataloaders.py", line 11, in get_dataloaders
    train_data = tv.datasets.CelebA(root='data', split='train', download=True, transform=transform)
  File "/Users/lucaslingle/opt/miniconda3/envs/pytorch181/lib/python3.9/site-packages/torchvision/datasets/celeba.py", line 77, in __init__
    self.download()
  File "/Users/lucaslingle/opt/miniconda3/envs/pytorch181/lib/python3.9/site-packages/torchvision/datasets/celeba.py", line 131, in download
    with zipfile.ZipFile(os.path.join(self.root, self.base_folder, "img_align_celeba.zip"), "r") as f:
  File "/Users/lucaslingle/opt/miniconda3/envs/pytorch181/lib/python3.9/zipfile.py", line 1257, in __init__
    self._RealGetContents()
  File "/Users/lucaslingle/opt/miniconda3/envs/pytorch181/lib/python3.9/zipfile.py", line 1324, in _RealGetContents
    raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

Expected behavior
The download just works.
Environment

PyTorch / torchvision Version: 1.8.1 / 0.9.1
OS: MacOS Catalina
How you installed PyTorch / torchvision: conda and pip, respectively.
Build command you used (if compiling from source): N/A
Python version: 3.9.2
CUDA/cuDNN version: N/A
GPU models and configuration: N/A

Additional context
This is a known issue, and is due to a Google Drive quota limit, which returns an error page that is saved in the supposed zip file. The TensorVision developers closed another issue, saying they would wait for someone else to open a ticket complaining, before they found a robust fix. Anyways, here I am. This is a complaint.
My idea of a robust fix would be for the TensorVision developers to get permission from the dataset owners to mirror the data on S3. This was already done by the TensorVision developers for MNIST. Please consider getting permission from the CelebA dataset owners to mirror the data on S3, like was done for MNIST. This will resolve the problem.
For completeness, I have also opened a feature request ticket to address this issue.
cc @pmeier

@lucaslingle Thanks for reporting.
@pmeier I'm able to reproduce using the provided snippet.

This is still in my backlog. It is just really hard to fix, because I need to wait for a day where the quota is exceeded to even work on it. Getting on it now.

Upping to high priority due to user activity. Cc @fmassa


Prevent widget override or other solution for integration with other plugins

Hello,
I've written a small plugin: https://github.com/psprint/history-search-multi-word. It's quite compact and should be harmless. Defines own keymap, binds cursor keys there, calls recursive-edit, that's pretty much all.
When autosuggestions plugin is loaded, it is doing the following during its operation (tracked by wrapping "zle" function):
zle -N history-search-multi-word _zsh_autosuggest_bound_history-search-multi-word
zle -N history-search-multi-word-backwards _zsh_autosuggest_bound_history-search-multi-word-backwards
This creates conflict. Using cursor keys clears my widget's output. Could this be because autosuggestions uses $POSTDISPLAY like my widget? I wonder what could be the solution, is such problem maybe recognized?

I've managed to solve this. Thanks to the fact that I use zle recursive-edit (also switched to .recursive-edit), I stay inside my widget. It is then again called from say itself. So I was able to override myself with a "clean copy" of myself, free of autosuggestions interference:
+(( __hsmw_hcw_call_count ++ ))
+_zhcw_main
+
+_zhcw_simulate_widget() {
+    (( __hsmw_hcw_call_count ++ ))
+    _zhcw_main
+}
+

This override holds only after first call to my widget, when leaving I restore original autosuggestions-modified widget. And everything plays nice. The issue can be maybe closed. Contributed because widget creators might feel overwhelmed by what happens after each key press when using autosuggestions.

Thanks for following up. Glad you were able to get it working!


Dynamics Of GTAD+PGCN :

Hi,
Thank you for this amazing paper and wonderful repository!
While reading the paper, I found that GTAD+PGCN combined model achieves much better results than any other model trained on THUMOS'14.
I have my own data set and would like to use the combined GTAD + PGCN model for training.

From what I understand, the combined model uses extracted features from RGB and Flow frames using a TSN or I3D Feature Extractor (and stored as HDF5 files), GTAD is used to generate proposals, and then the proposals are passed into the PGCN architecture. Is that right ?

If my logic above in point 1 is correct, how can I only use the GTAD repo for generating proposals? Can you guide me through how I can train the combined GTAD + PGCN model on my own data set ?

Dear @lakshaymehra ,
Thanks for your interests. Using GTAD+PGCN can reach a better performance.

Your logic is correct. However, PGCN models (rgb+flow) are provided in their repo. We didn't train them. We use PGCN models to re-score our G-TAD detections.
The gtad_inference.py script will produce the action predictions under a results folder. You can load those results in P-GCN dataset.

@Frostinassiky Thanks a lot for the detailed response!

Dear @lakshaymehra ,
Thanks for your interests. Using GTAD+PGCN can reach a better performance.
1. Your logic is correct. However, PGCN models (rgb+flow) are provided in their repo. We didn't train them. We use PGCN models to re-score our G-TAD detections. Since you have your own dataset, you have to **train PGCN models by yourself**.

2. The `gtad_inference.py` script will produce the action predictions under a `results` folder. You can load those results in P-GCN dataset.

hi, could you please tell me more detail of how to load the results into P-GCN? thanks very much


do not strip intentional newlines in chat transcript items

Partial fix for https://github.com/sourcegraph/cody/issues/3483. Makes it so that the Markdown is rendered literally with newlines as typed, instead of all being jammed on 1 line. (It does not render the user's code in backticks as code.)
Here is how it looks (not ideal because of the lack of rendering, but better than seen in #3483 since the newlines are preserved):

Test plan
Type a message with triple-backtick then newline then multiple lines of code. Confirm that the newlines are preserved when the message is displayed in the transcript.

Compare this to the current behavior on the stable release (the code is rendered):

@kalanchan It's to make it render nicely as well. That is what I'll commit to doing in the next 48h.


390 Added runs

Added runs for assistants. Maybe the polling part should bve included in the service, but I left it open for now. I fe include it in the service (which would be nice) we should probably include a delay when polling and a timeout.

Removed code that didn't work in 1.8

@bjorkman  where is the method abount RunStep? You defined it but now use it, why ?

I will fix it

https://github.com/TheoKanning/openai-java/pull/408


knowledgebase should reconcile fileGroups updates

When i add a new file operator.csv, the controller donesn't seem to handle this file

@Abirdcfly any progress ? If this has been fixed by a merged pr, pls close this issue.


Remove VersionTemplateJava

Fix #2450

Could the template file be renamed (to something like VersionTemplate.txt or similar), so it does not have .java file ending? For reasons:

It is not a real java file (as in something that should be compiled inte a .class file)
it would make life easier for people who use Eclipse and would like to contribute to testng (as explained by @juherr in this issue: #2450)

Could the template file be renamed (to something like VersionTemplate.txt or similar), so it does not have .java file ending? For reasons:

It is not a real java file (as in something that should be compiled inte a .class file)
it would make life easier for people who use Eclipse and would like to contribute to testng (as explained by @juherr in this issue: #2450)

@krmahadevan I updated the PR but I didn't test it for the moment.

@krmahadevan I updated the PR but I didn't test it for the moment.


feat:updated env variable [ECO-270]

Title- Updated .env variable [ECO-270] for Gatsby
Changes-Standardize .env name for Gatsby
Story-https://contentstack.atlassian.net/browse/ECO-270

Need to change env name as per discussion.


Correction for raw affiliation School of Pharmacy and Biomolecular Sciences, Liverpool John Moores University, Liverpool, United Kingdom; Univ Lyon, ENS de Lyon, Univ Claude Bernard, CNRS UMR 5239, INSERM U20, Laboratory of Biology and Modelling of the Cell, 46 Allée d’Italie Site Jacques Monod, F-69007, Lyon, France

Correction needed for raw affiliation School of Pharmacy and Biomolecular Sciences, Liverpool John Moores University, Liverpool, United Kingdom; Univ Lyon, ENS de Lyon, Univ Claude Bernard, CNRS UMR 5239, INSERM U20, Laboratory of Biology and Modelling of the Cell, 46 Allée d’Italie Site Jacques Monod, F-69007, Lyon, France
raw_affiliation_name: School of Pharmacy and Biomolecular Sciences, Liverpool John Moores University, Liverpool, United Kingdom; Univ Lyon, ENS de Lyon, Univ Claude Bernard, CNRS UMR 5239, INSERM U20, Laboratory of Biology and Modelling of the Cell, 46 Allée d’Italie Site Jacques Monod, F-69007, Lyon, France
new_rors: 02vjkv261;029brtt94;04zmssz18;02feahw73;04zfme737;01bj4fd12
previous_rors: 02vjkv261;029brtt94;04zmssz18;02feahw73;04zfme737
works_examples: W2952344515
contact: a86a299fbefab071872b0754b6ab1e78:ed705bb2935c6e07c3d754d9f26c @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


add the missing config files back

Re-add the hal_conf.h files and FreeRTOS config header back in

@NathanielDelgado what was the argument for putting these files in Test again?


FEATURE: Link Budget Activity to Account Transaction

Link activity rows back to their transaction on the accounts page.
Super cool feature, believe me :D
There is currently a bug in YNAB that makes this not work for older transactions that I've submitted to YNAB -- hopefully they fix it:
1. Go to an account page.
2. Show all transactions (clear filters)
3. Navigate to the budget page
4. Navigate back to the account page from step 1.
5. Scroll down
6. Infinite scrolling should occur but doesn't

Trello: https://trello.com/c/Ia8JzScw/26-from-the-budget-screen-allow-clicking-on-transactions-in-the-activity-view-and-go-to-the-corresponding-section-in-the-accounts-s
I have the default as true because this a feature from YNAB4 -- we can change that if we want.

I agree with the true default. This looks awesome. I haven't tested thoroughly yet. The more anyone else can help test, the better!


Bump vendored containers/image, containers/storage, and add ostreedev/ostree-go

This is an updated version of #679, adding the image library's new ostree dependency, and building the native library that it uses when we're in CI environments.
I don't really know what to change in the top level README.md to talk about this, since libostree-dev isn't in the main Debian repositories.  Should it suggest the containers_image_ostree_stub build tag to avoid that, or adding it via a ppa like Alex's?

LGTM, assuming tests pass.  As far as the README.md goes, I'd add something there if someone trying to build Buildah from 'scratch' would have to change something on their machine to get it to compile/build.  Otherwise I'd go with adding to the ppa.  It would probably be good to get lsm's thoughts.

I think adding a link to the ppa should be fine.

Okay, added what I think should cover it.  We could get a lot more verbose about how to build ostree from sources, but I'm hoping that we don't have to because most people will opt for a prebuilt package.

LGTM once tests finish happily.

LGTM
Why are these tests not running on this pull request?

retest

There is some build failure atleast on Fedora.

Logs not very helpful.  :^(

I missed updating crio-integration-playbook.yaml, hopefully updating it to also install ostree-devel fixes those results.

LGTM


Added github workflow to build and publish docker files

This will build the docker images using the hassio builder. At the moment it is only set to build the amd64 image but you can build others too. The first builds to make sure there is no errors. The second will rebuild and publish. Make sure you change the docker username for yours and add a secret for the password to the repo. I have also included a shellcheck but i have commented this out for now as i dont have time to run through the errors. The build will run for commits and PR's where as the publish on runs for commits. If the build job fails the whole process is stopped so it doesn't publish faulty files. Sorry for the messy commits, if i was you i would squash and merge so there is only one commit in the history :). On my repo's i have it so it only publishes the files to Dockerhub when i do a release but as far as i can see you don't do releases so this is why i have done it this way. Any questions let me know.

Hey @Sabuto,
Sorry to review this so late :( I will try to setup my own Docker repository to include your setup and  push automatically any new Docker image. In the meantime, this PR is on hold.

np :), tbf all you need is an account it will create the repository's automatically :D

Hello a few months later 👋
I finally copied the workflow from home-assistant addons-developement repository, which provides a simple way to build all architectures at once. But there is a small issue: GitHub Actions workers doesn't support the ARM architecture, so I will need to find a way around that (maybe by using my own worker).
In the meantime, amd64 and i386 Docker images are now available. Even if I won't merge this PR, thank you so much for the help and for showing me the way! 👍

Hello a few months later 👋
I finally copied the workflow from home-assistant addons-developement repository, which provides a simple way to build all architectures at once. But there is a small issue: GitHub Actions workers doesn't support the ARM architecture, so I will need to find a way around that (maybe by using my own worker).
In the meantime, amd64 and i386 Docker images are now available. Even if I won't merge this PR, thank you so much for the help and for showing me the way! 👍

no problem :) glad i could be of help :D

no problem :) glad i could be of help :D


Mirrormaker consumer sometimes runs slow

Sometimes find that one of the mirrormaker consumers is running very slowly (~20 alerts/minute), likely just on a single partition. Restarting mirrormaker appears to clear the problem. Not seeing anything in the mirrormaker logs.

Problem has not recurred in the past six months or so. Closing as irreproducible.


fix: ascii idents only to prevent idents starting with characters my keyboard can't handle

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


⚠ Bofrost.at has degraded performance

In 47fab53, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1969 ms

Resolved: Bofrost.at performance has improved in 2b9208d after 310 days, 20 hours, 49 minutes.


[AutoMM] Support HPO presets

Issue #, if available:
Description of changes:

Support HPO presets, e.g., high_quality_hpo, medium_quality_hpo, best_quality_hpo, or hpo.
Unit tests.
Support combining preset hyperparameters and user-provided hyperparameters.
Support combining preset  hyperparameter_tune_kwargs and user-provided hyperparameter_tune_kwargs.

from autogluon.multimodal import MultiModalPredictor
predictor = MultiModalPredictor(problem_type="classification", presets="high_quality_hpo")
predictor.fit(train_data=train_data)

Text backbone candidates with the number of parameters:
medium quality

"google/electra-small-discriminator" (13.5M)
"prajjwal1/bert-small" (28.8M)
"albert-base-v2" (11.7M)

high quality

"google/electra-base-discriminator" (108.9M)
"bert-base-uncased" (109.5M)
"roberta-base" (124.6M)
"albert-xlarge-v2" (58.8M)

best quality

"microsoft/deberta-v3-base" (183.8M)
"google/electra-large-discriminator" (334.1M)
"roberta-large" (355.4M)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

We can pick a selection criteria for each catalog, for example, we can measure the model size / training time (ideally we should report a curve about training throughput v.s. performance).
For now, we can consider to adopt the following rules:

medium quality (<50M backbones)
high quality (>=50M,<200M backbones)
best quality (>=200M backbones)

We can thus also consider backbones in https://www.sbert.net/docs/pretrained_models.html, and use other common backbones like

DeBERTA-series: https://huggingface.co/microsoft/deberta-v3-xsmall
MiniLM-series: https://huggingface.co/microsoft/MiniLM-L12-H384-uncased
FLAN-T5-series: https://huggingface.co/google/flan-t5-base


Cancel Drag with Esc

This implements a new plugin and logic to automatically cancel and revert to the original position an element when the Esc key is pressed.
Fixes https://github.com/SortableJS/Sortable/issues/264, inspired by https://github.com/SortableJS/Sortable/issues/138#issuecomment-1114674135

Yes! Be the change!


optimize

optimize base on https://github.com/cytoscape/cytoscape.js-navigator/issues/46

@haohailiang, would you provide a description of your change and the approach that you used?
@canbax, would you be able to evaluate to what degree this makes a difference for your use case?


Add nil check on ServicePrincipalProfile before populating clientsecrets for different frontend routes

Which issue this PR addresses:
Additional changes for https://issues.redhat.com/browse/ARO-6623
Previous PR:- https://github.com/Azure/ARO-RP/pull/3529
What this PR does / why we need it:
Adds a condition on ServicePrincipalProfile before setting the client secret. This required for the MIWI clusters which will have the ServicePrincipalProfile as nil, and setting ClientSecret on the nil pointer would cause an error.
Test plan for issue:
e2e and CI should pass
Is there any documentation that needs to be updated for this PR?
No
How do you know this will function as expected in production?
Additional checks on already tested and working flow.

/azp run ci,e2e

/azp run e2e


Fix SvelteKit pages docs to use ESM config

The current instructions need updating for the current SvelteKit. The main reason is that SvelteKit changed the config to using ESM. See https://kit.svelte.dev/docs#configuration for the new style.
Also, the adapter-node is not included by default so cannot be removed.
This patch fixes both.

I would also like to move the "SvelteKit Configuration" section to before "Deploying with Cloudflare Pages" since it's not obvious that you can't deploy after reading the "Deploying with Cloudflare Pages" section.


update getting-started link in README doc

Coverage remained the same at 0.0% when pulling 2cc04a490dc7e3dde439ee50fe63714548c0e127 on hangzhang925:master into 48848751ab8c93bf5411e1c66fc6d3408c66b3df on linkedin:master.


Support exponent in JSON number & add implement std::error::Error trait

Thins I noticed while making a glTF parser.
Two changes in one :

Support exponent in JSON numbers (e.g. 1e2 == 100). Always floating point. Available in branch json_exponents.
Implement std::error::Error trait for DeJsonErr. Includes Display trait that is the same as Debug. Might make sense to add to other error types as well but this one was annoying me right now. Available in branch std_error.

Putting both changes in one PR because they both add a test which generates a merge conflict.

Thank you!


Rename backend to storageType

What happened?
We've agreed on renaming this it's at least present in the cp list

The term backend is directly (i.e. not as part of content coming from the API) in two places in the UI.

The Mesh wizard has a "Backend name:" field when setting up Logging, Tracing, and Metrics. Those values are then shown in the YAML content in the install step.
The Zones CP overview has a table header "Backend". This corresponds to the /config API response’s store.type value.

I assume only the latter should be renamed. Is that correct?

This is correct


refactor(transformer/nullish-coalescing): avoid repeated symbol lookups

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7273
#7272  👈
#7269
#7268
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

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


Cannot find name 'OffscreenCanvas' and Cannot find name 'GPUBuffer'

Hi
I have been using this library previously since few months ago and it working fine
Today I am trying to use this library after my clean installation at another PC, and I got this error
./node_modules/@vladmandic/Human/dist/human.esm.js:21:70-77 - Warning: Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
./node_modules/@vladmandic/Human/dist/human.esm.js:24:45-52 - Warning: Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
Error: node_modules/@vladmandic/Human/types/human.d.ts:6:53 - error TS2304: Cannot find name 'OffscreenCanvas'.
6 export declare type AnyCanvas = HTMLCanvasElement | OffscreenCanvas;
Error: node_modules/@vladmandic/Human/types/human.d.ts:863:14 - error TS2304: Cannot find name 'GPUBuffer'.
863     buffer?: GPUBuffer;
Version: "@vladmandic/human": "^2.6.4"
I even try to use the latest version and still got the same error
Somehow I manage to solve the OffscreenCanvas issue by installing:
npm i @types/offscreencanvas
and added this to tsconfig.json
"compilerOptions": {
...
"types": ["offscreencanvas"],
...
},
But I still haven't manage to solve missing GPUBuffer error
Please check it out. Thanks in advanced.
(Sorry this is my first time create new issue)

can you share your package.json?
i'm woring on pre-bundling all external typedefs, but for now its best to set "skipLibCheck": true, in your tsconfig.json

Hi, thanks for feedback
Adding "skipLibCheck": true inside  tsconfig.,json really solved my problem and now my program compiled successfully, even the human library running as intended
"compilerOptions": {
  ...
  "skipLibCheck": true
}

Since you already request for my package.json, here is it
{
  "name": "tmda-app",
  "version": "0.0.0",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "watch": "ng build --watch --configuration development",
    "test": "ng test"
  },
  "private": true,
  "dependencies": {
    "@angular-eslint/schematics": "^14.1.2",
    "@angular-material-components/file-input": "^7.0.1",
    "@angular/animations": "~13.1.1",
    "@angular/cdk": "^13.1.1",
    "@angular/common": "~13.1.1",
    "@angular/compiler": "~13.1.1",
    "@angular/core": "~13.1.1",
    "@angular/forms": "~13.1.1",
    "@angular/localize": "~13.1.1",
    "@angular/material": "^13.1.1",
    "@angular/platform-browser": "~13.1.1",
    "@angular/platform-browser-dynamic": "~13.1.1",
    "@angular/router": "~13.1.1",
    "@angular/service-worker": "~13.1.1",
    "@ng-bootstrap/ng-bootstrap": "^12.1.2",
    "@ng-web-apis/common": "^2.0.0",
    "@ng-web-apis/geolocation": "^1.0.4",
    "@popperjs/core": "^2.11.6",
    "@sweetalert2/ngx-sweetalert2": "^10.0.0",
    "@types/jquery": "^3.5.14",
    "@types/webgl2": "^0.0.6",
    "@vladmandic/human": "^2.6.4",
    "@webgpu/types": "^0.1.23",
    "angular-calendar": "^0.29.0",
    "bootstrap": "^5.1.3",
    "date-fns": "^2.28.0",
    "html5-qrcode": "^2.1.6",
    "install": "^0.13.0",
    "javascript-time-ago": "^2.5.7",
    "jquery": "^3.6.0",
    "material-icons": "^1.10.8",
    "ng-click-outside2": "^10.0.3",
    "ng-maphilight": "^1.0.1",
    "ngx-image-cropper": "^5.0.1",
    "ngx-webcam": "^0.4.1",
    "onesignal-ngx": "^1.0.1",
    "runtime-config-loader": "^4.1.0",
    "rxjs": "~7.4.0",
    "sweetalert2": "^11.3.5",
    "tslib": "^2.3.0",
    "zone.js": "~0.11.4"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "~13.0.2",
    "@angular/cli": "^13.1.2",
    "@angular/compiler-cli": "~13.1.1",
    "@types/aframe": "^1.0.5",
    "@types/jasmine": "~3.10.0",
    "@types/node": "^12.11.1",
    "@types/three": "^0.136.1",
    "jasmine-core": "~3.10.0",
    "karma": "~6.3.0",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage": "~2.0.3",
    "karma-jasmine": "~4.0.0",
    "karma-jasmine-html-reporter": "~1.7.0",
    "typescript": "~4.4.3"
  }
}

My package.json is actually messy and I need to add --legacy-peer-deps every time I want to install / uninstall any package
I also want to add another finding (that might caused the problem)
I compared both<EMAIL_ADDRESS>file from my old project and newly created project (before I put skipLibCheck: true), and there is a lot of line different. The highlighted on below is causing the problem (GPUBuffer)

However, after I putting "skipLibCheck": true and run the project, both files become identical (same as my old project)
Thanks again for any feedback


Clean up tune_gemm script from main_perf branch

This PR is a follow up from https://github.com/ROCm/triton/pull/614#issuecomment-2289459234, since Triton's checks related to code formatting have failed. In addition to code formatting according to Triton standards, linter issues were fixed and old files from tune_gemm V1 were removed.
Important notice: This PR doesn't propose any functional changes, just "cosmetic" changes and cleanup. Dear reviewer, if you notice something that modifies tune_gemm behavior please let me known!
Summary of changes per file:

README.md: remove trailing whitespaces, fix minor issues related to CLI arguments
icache_flush.py: code formatting, remove unused imports
matmul.py: ~code formatting, remove trailing whitespaces, remove unused local variable~

removed, it's an old file from tune_gemm V1

matmul_kernel.py: code formatting
one_config.py: code formatting
rocprof_gemm.py: ~code formatting~

removed, it's an old file from tune_gemm V1

tune_gemm.py: code formatting, fix issue with $(1, 1)$ bias tensors, remove star imports, remove unused local variable
tune_gemm.sh: removed, it's an old file from tune_gemm V1
utils/file_generator.py: code formatting, remove star imports, remove unused local variable
utils/utils.py: code formatting

Are you using a python/markdown formatting tool?

Dependency graph of tune_gemm files:
graph TD;
    one_config.py --> tune_gemm.py
    rocprof_gemm.py
    tune_gemm.py --> matmul_kernel.py
    tune_gemm.py --> utils/file_generator.py
    tune_gemm.py --> utils/utils.py
    tune_gemm.sh --> matmul.py
    utils/file_generator.py --> utils/utils.py
    utils/file_generator.py --> icache_flush.py

This is the super cool graph. Can you add it to the README?

This is the super cool graph. Can you add it to the README?

It's a Mermaid chart. I can try to embed it into our README and see what happens. I never tried it before.

This is the super cool graph. Can you add it to the README?

It's a Mermaid chart. I can try to embed it into our README and see what happens. I never tried it before.

It worked, you can see it rendered here.

Thank you Bruno.

You're welcome, I'm happy to help. Thanks for you time reviewing this PR, I really appreciate it.


[PushNotifications] implicit declaration of function 'RCTNullIfNil' is invalid in C99

I am trying to import PushNotificationIOS, but I somehow get this:

Has anyone seen it?
I am using the latest version of PushNotificationIOS.

I have the same error message in other cases, e.g. "RCTJSErrorFromNSError". Currently using master.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out its new home: https://productpains.com/post/react-native/pushnotifications-implicit-declaration-of-function-rctnullifnil-is-invalid-in-c99
ProductPains helps the community prioritize the most important issues thanks to its voting feature.
It is easy to use - just login with GitHub.
Also, if this issue is a bug, please consider sending a PR with a fix. We rely on the community to provide
bugfixes as the core team is heavily focused working on performance.


Thread Blocking in Magnetometer driver

Hello
I'm using the hmc5883 driver in Firmware/src/drivers/hmc5883 off the latest stable branch.
It works fine on the i2c device. But, When i'm trying to set the calibration in the driver, my thread blocks:
In my application:
  int mag_fd = open(MAG0_DEVICE_PATH,0);
  if (mag_fd <= 0) {
    printf("couldn't open %s", MAG0_DEVICE_PATH);
  } else {
    mag_scale magscale;
   /* fill magscale*/
    if (OK != ioctl(mag_fd ,MAGIOCSSCALE,(unsigned long) (&magscale))) {
      printf("failed setting calib %s", MAG0_DEVICE_PATH);;
    }
    close(mag_fd);
  }

Thread blocks at ioctl();
I managed to make the ioctl() call work by commenting out these two lines in hmc5883.cpp:
int HMC5883::check_calibration()
{
    bool offset_valid = (check_offset() == OK);
    bool scale_valid  = (check_scale() == OK);

    if (_calibrated != (offset_valid && scale_valid)) {
    ---->   //warnx("mag cal status changed %s%s", (scale_valid) ? "" : "scale invalid ",
    ---->   //(offset_valid) ? "" : "offset invalid");
        _calibrated = (offset_valid && scale_valid);
    }

    /* return 0 if calibrated, 1 else */
    return (!_calibrated);
}

Any Idea why this blocks my caller thread?
Thanks for you help,
Markus

Strange, so you are saying that the ioctl never returns?
Can you retest after with master which includes this fix: #5062.

Possibly a stack overflow. Can you look at the stack usage using top on the nsh console?

Thanks for you're replies!
from the top i can tell that it blocks the thread. (it does not segfault which would happen on a stackoverflow)
i see the specific thread that was called with the flag waiting on semaphore.
@julianoes i'll try with the mater branch.

This was anyway wrong on that level. Drivers should never print to the console. I fixed that in 256222d44d.


[DIR-1403] Select number of items in a table

Description
Added a field to select the number of items in a table on the events page.
The default setting is a pagesize of 10 items per page. If you change the setting, the change is remembered after reloading the page.

Checklist

[x] Documentation updated if required
[x] Test coverage is appropriate

Checklist Internal

[x] Linear issue linked (e.g. [DIR-XXXX] pull request title)
[x] Has the PR been labeled

@sebxian i'm curious what you think about the wording here. Do you agree with it our would you phrase it differently.

"Show N rows" is not bad at all, "Show N items" would be a bit more neutral but I don't think it makes a big difference.

@sebxian i'm curious what you think about the wording here. Do you agree with it our would you phrase it differently.

"Show N rows" is not bad at all, "Show N items" would be a bit more neutral but I don't think it makes a big difference.

Ok, thanks. I think @julia-zielke can decide then :)


Wrong response when upload fails due to file write stream error

The problem is that if file write stream emits an error we call storeStream.end() before setting notErr to false, so finish event handler on file stream thinks there was no error and sends 226 reply, which makes the client to believe that upload was successful even though it failed.

Thanks for finding and reporting this bug!


feature: support default response, use comma-separated codes to add a…

Describe the PR

Support default response.  #691  override PR #692
Use comma-separated codes to define  the same response for multi-codes in one line.
Use comma-separated codes to add a header for multi responses in one line. #785
Use keyword 'all' to add a header for all responses. #785

Relation issue
#785 #691
// @Success 200 {string} string	"ok"
// @failure 400,401 {string} string	"error"
// @response default {string} string	"other error"
// @Header 200 {string} Location "/entity/1"
// @Header 200,400,default {string} Token "token"
// @Header all {string} Token2 "token2"

@sdghchj Thanks for your big effort on it!

Hi,
It looks like a broken commit were merged into master and everything is failing now...
Is this a work in progress ???


Documentation for verticalLineClassNamesForTime returns only first holiday in array

Describe the bug
The documentation for verticalLineClassNamesForTime provides an example that only returns data for the first holiday that meets the conditions in the holiday array, regardless of date range. This is because the return statement is within the if block. For a new user, this behavior can be extremely confusing.
I spent a full day trying to debug my code, to discover it was only returning the Jan 1st holiday and skipping all later ones in the array.
I suggested in PR #449 that we adjust the sample code to move the return statement outside the if block. This will allow all holidays that meet the conditions to be displayed and will reduce frustration for new users.
To Reproduce
I made a quick sample code explanation of how the location of the return statements affect how much of the data is returned. You can find it here.
In sampleFunction1, once the condition is met the first time, it exits the function completely due to the use of the return statement within the loop. In order to prevent that early exit, in sampleFunction2 I moved the return statement outside the loop. This ensures the entire array is processed before the function returns.
I want to emphasize that the react-calendar-library works wonderfully, it's just the documentation that was causing my problems while using it.
Expected behavior
I expected to be able to use the documentation for verticalLineClassNamesForTime to guide me in adding holidays to my code.
Screenshots
N/A
Library Version
0.19.0
Desktop (please complete the following information):

OS: iOS
Browser Chrome
Version 69.0.3497.100

Smartphone (please complete the following information):
N/A
Additional context
As a new user to this library, I really appreciate all the detailed documentation! That's why when I got stuck and realized there was a "bug" in the documentation, I wanted to help contribute and improve.

@holytoastr could you please provide a sandbox example?
I've seen the discussion you had with @dkarnutsch in the PR and I agree that your fix will only add ``holiday` over and over again.
I really appreciate your effort in helping this library


Add tests that every current model can predict in-sample, out-sample with gap

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Proposed Changes
Look #783.
Closing issues
None.

Codecov Report

Merging #786 (116709e) into master (687a1a9) will increase coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #786      +/-   ##
==========================================
+ Coverage   83.56%   83.81%   +0.24%
==========================================
  Files         123      123
  Lines        6895     6895
==========================================
+ Hits         5762     5779      +17
+ Misses       1133     1116      -17

Impacted Files
Coverage Δ

etna/core/mixins.py
100.00% <0.00%> (+4.34%)
:arrow_up:

etna/models/catboost.py
100.00% <0.00%> (+22.85%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Fix composite build execution when project isolation is enabled

Fixes #?
Context
This PR also adds a bunch of validation to BuildLifecycleController and BuildModelController to check that various methods are called in the correct order (when there are dependencies between them), that the build model is not in some broken state, that configuration or task execution are not requested while already in progress, and that the build model is not used by multiple threads concurrently. Some limited concurrent access is allowed, as this can currently happen in various cases. This door should be closed and a later PR can take care of this.
To fix the issue and also honour some of the new validation, change some direct usages of the mutable models Gradle and Project with their BuildState and ProjectState owners.
Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


JIRA: Support resolutions for False Positive, Risk Accepted, Out of Scope, etc.

Is your feature request related to a problem? Please describe
Discussion here
https://owasp.slack.com/archives/C2P5BA8MN/p1635238884057600
third point
When I change finding to False Positive for ex., and push to jira, Into Jira finding have Done mitigated(not False Positives).  The DD send always "Done" mitigation.
Describe the solution you'd like
If finding is changed to False Positive or Risk Accepted, it should to be pushed Jira with the same resolutions.

Currently JIRA doesn't send any resolution to JIRA, so JIRA uses the default resolution which is 'Done' for default workflows, but can be something else in other workflows/instance. I think it could be helpful if we add resolution support to our JIRA integration.

Same question, if a false positive issue is pushed from Dojo to Jira, will there be a field in the Jira task to distinguish between Done and False Positive?


adds documentation about the npm scripts

(Pick the language of your choice to fill out the following information. / Choisissez la langue de votre choix pour remplir l'information ci-dessous.)
What does this pull request (PR) do? / Que fait cette demande « pull » (PR)?
Adds documentation about the various nom scripts found in the package.json file
I left out the SASS scripts and any of the other build scripts that are dependent by other scripts.

@shawnthompson did you wanted to add the coding character for each title (English and French)? I only see a change for the title I identified but not for all of them.
FYI - An alternative will be to replace those programmatic title by a short meaningful name/title. like updating the current title "dev" for "Running in development mode" | "Exécution en mode développement".

@duboisp I like the idea of replacing them with a small descriptive title. I'll make the change and push it back up to my branch. Thanks!

@duboisp ready for review.

Ready to squash and merge.


implement logger in convert_script

Request
Implement a logger on the converter script.
Implementation
The logger is quite simple. Basic stdout output. No format. Uses the standard logging module.
The logging is made when we append the line to FileTranslator.lines through an overloaded append method in a custom List class.
Use:
If the optional keyword show_log is True in FileTranslator, the resulting converted PyMAPDL is printed to the stdout.
Example:
>>> from ansys.mapdl.core.convert import convert_apdl_block, FileTranslator
>>> apdl_ = """FINISH
... /PREP7
... """
>>>
>>>_ = convert_apdl_block(apdl_strings=apdl_, line_ending='\n', show_log=True)
Converted: '"""Script generated by ansys-mapdl-core version 0.60.dev0"""'
Converted: 'from ansys.mapdl.core import launch_mapdl'
Converted: 'mapdl = launch_mapdl(loglevel="WARNING")'
Converted: 'mapdl.finish()'
Converted: 'mapdl.prep7()'
Converted: 'mapdl.exit()'

Closes #698

It should now work.


Does not respect the targeted Node.js version

Package ecosystem
npm
Config
https://github.com/sindresorhus/to-readable-stream/blob/d2ca3e102351a57b2b6f8f68027ffbb493e9280f/.github/dependabot.yml

I have a package that defines in package.json that it supports Node.js 12 and later: https://github.com/sindresorhus/to-readable-stream/blob/d2ca3e102351a57b2b6f8f68027ffbb493e9280f/package.json#L15-L17
I expect Dependabot to only upgrade dependencies that are compatible with Node.js 12. However, in https://github.com/sindresorhus/to-readable-stream/pull/12, it upgraded a dependency that requires Node.js 14 and later (not 12). (You can see this if you click on "Release notes" there). This obviously breaks CI and wastes my time.

Hey @sindresorhus! Thanks for trying out Dependabot version updates and taking the time to open an issue. I'm sorry this wasn't working as expected for you.
I just cloned sindresorhus/to-readable-stream and tried out a few things:
Running npm update tsd, and it didn't update the downloaded version. This makes sense because no other releases satisfy the requirement ^0.14.0:
$ curl "https://registry.npmjs.org/tsd/" | jq ".versions | keys"
  # ...
  "0.13.0",
  "0.13.1",
  "0.14.0",
  "0.15.0",
  "0.15.1",
  # ...
  "0.22.0",

To update the requirement of the dependency to the latest version, I tried:
$ npm install --save "tsd@*"

Doing this updates tsd to the latest release, 0.22.0, which is what Dependabot proposed in https://github.com/sindresorhus/to-readable-stream/pull/12. However, as you correctly point out, that's the wrong behavior.
In the context of an application, it’s a bit easier to get npm to do what we want. By enabling engine-strict option, it should honor the host engine settings when determining which version to install.
+ 	"engineStrict": true,

In the context of a library, where it’s about the requirements of engines as a meta dependency, I couldn’t find a way to that with npm alone. However, npm-check-updates appears to do the right thing:
$ npx npm-check-updates -u tsd --enginesNode
Using yarn
Upgrading /Users/mattt/Downloads/to-readable-stream-main/package.json
[====================] 1/1 100%

 tsd  ^0.14.0  →  ^0.21.0

As far as Dependabot goes, we’re actively looking into ways to delegate core functionality to dedicated, ecosystem-specific tools. I can’t give you a timeline or roadmap yet, but I’m really excited about the direction we’re taking. I’ve made a note about this specific behavior, and will make sure that it’s addressed as part of these upcoming improvements.

I'm having a very similar issue whereby I have a basically completed library and Dependabot keeps trying to update to packages that don't satisfy my version settings.
E.g.: https://github.com/ElvenSpellmaker/node-service-status/pull/73
I've clearly stated I need Node > 12: https://github.com/ElvenSpellmaker/node-service-status/blob/822a679e0759e9abcc598abbecac5f23d43805f1/package.json#L11-L13

As mentioned above, and documented here, you need to set engine-strict=true in .npmrc for those version constraints to be enforced.
Also as noted above, that isn't a great solution for a library... but I'm not currently aware of any other solution offered by native npm.
Dependabot is focused on being the glue code that fetches manifests then hands them off to native package managers to process/generate the suggested updates, and then Dependabot turns those into PRs. So this is something that needs to be fixed by upstream npm, and then we'll happily pull it in.
I'm going to close as I'm not aware of any alternative here, but if for some reason I'm wrong or missing information please comment and we can reopen.
Also related:

https://github.com/dependabot/dependabot-core/issues/4072


[5.2] Filter out null files for multiple file uploads.

Fixes laravel/framework#12292

This would actually be a a breaking change. It needs to keep the same behavior it had before the file conversion. I will fix it.


Explore runtime config to change SerialConn

Closing this as it is based on potentially outdated assumptions. Please feel free to re-open (with an updated description) based on the current structure.
Considerations for closing this

missing description / use-case / ...
inactivity


[ADnD_2E_Revised] Slim Spells

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
All changes described on the forum https://app.roll20.net/forum/permalink/11334526/

Character Sheet Info Roll20 Internal Use only.


Fix permissions for Update schedule.yaml job

This fixes the permissions required for the job, which fails like: https://github.com/kubernetes/website/actions/runs/10444771187
cc @kubernetes/release-engineering

We avoid Markdown in commit message summaries, so:
/retitle Fix permissions for "Update schedule.yaml" CD job

/approve

/lgtm


Profile Picture gets covered when selecting profile in sidebar

Checklist:

[/] I updated to the latest version available
[/] I cleared the cache of my browser

Release with the issue: v2.1.5
Last working release (if known):
Browser and Operating System: Home Assistant Companion App Android 14 OneUI 6.1

Description of problem:
When you select the current user in bottom left of Home Assistant in the sidebar the profile picture gets covered by the accent colour.

Javascript errors shown in the web inspector (if applicable):

Additional information:

It doesn't seem to be a problem for me on desktop or the Android companion app. Was this an issue in 2.1.4? None of the styles that fix the profile pic layer was touched for this patch.

It doesn't seem to be a problem for me on desktop or the Android companion app. Was this an issue in 2.1.4? None of the styles that fix the profile pic layer was touched for this patch.

Hi,
I didn't use 2.1.4 so I'm not sure. I don't have a profile picture set, may that be why?

It could be the lack of profile picture. I'll test that later today and will fix it if it is.

Fixed in 2.1.6. Nice catch!


Add coveralls support to CI

Changes Unknown when pulling 85cb8fdcdfad9eae7fa4bd68a1995eb79534bbaa on coveralls-travis into ** on master**.


feat(agora): migrate GCT (AG-1602)

Description
Migration of GCT and gene details page
Also migrates charts (except box plot)
miscellaneous fixes (including api)
Existing issues

conversion of components to standalone components and lazy-loading has resulted in a bug in the loading overlay component (to be fixed in a future PR)
box plot chart utilized in monorepo will be incorporated in a future PR

@sagely1 I will go ahead and merge this PR to make sure it's merged before I'm OOO.


Add flags to ignore some files

  -ignore-dirs PATTERN   ignore dirs matching this regexp
  -ignore-files PATTERN  ignore files matching this regexp
  -ignore-gen-files      ignore generated files

Pull Request Test Coverage Report for Build 31

58 of 70 (82.86%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 80.053%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ignore.go
33
35
94.29%

gocover-cobertura.go
22
32
68.75%

Totals

Change from base Build 29:
0.4%

Covered Lines:
301

Relevant Lines:
376

💛  - Coveralls


Verify SSL False not applying - Python SDK

What happened?
At the moment I need to use Lakefs with SSL verification disabled, however this does not seem to work:
import lakefs
from lakefs.client import Client

clt = Client(
    host="http://localhost:8000",
    username="",
    password="",
)
clt.config.verify_ssl= False

repo = lakefs.Repository("example-repo", client=clt).create(storage_namespace="s3://storage-bucket/repos/example-repo")

For some reason it's still trying to verify SSL but since the compute I am using doesn't have the root certificate yet, it will fail
Expected behavior
No response
lakeFS version
1.12
How lakeFS is installed
No response
Affected clients
Latest high level python sdk
Relevant log output
No response
Contact details
No response

This is the error:
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='<redacted>', port=443): Max retries exceeded with url: /api/v1/repositories/bronze/branches (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007)')))

@ion-elgreco Found the issue - I'll send a PR for a fix today


KAFKA-2668; Add a metric that records the total number of metrics

@onurkaraman @becketqin Do you have time to review this patch? It addresses the ticket that @jjkoshy filed in KAFKA-2668.

I think this may cause problems when there are multiple clients. I ran a sample application that makes two KafkaConsumers. Here's what jconsole shows me:
kafka.consumer
| app-info
  | consumer-1
    | Attributes
      | ...
  | consumer-2
    | Attributes
      | ...
| consumer-coordinator-metrics
  | consumer-1
    | Attributes
      | heartbeat-rate
      | ...
  | consumer-2
    | Attributes
      |  heartbeat-rate
      |  ...
| consumer-fetch-manager-metrics
  | consumer-1
    | Attributes
      | ...
  | consumer-2
    | Attributes
      | ...
| consumer-metrics
  | consumer-1
    | Attributes
      | ...
  | consumer-2
    | Attributes
      | ...
| consumer-node-metrics
  | consumer-1
    | node--1
      | Attributes
        | ...
    | ...
  | consumer-2
    | node--1
      | Attributes
        | ...
    | ...
| metrics-stats
  | Attributes
    | metrics-total

Basically, metrics-stats isn't scoped to a client-id while the others were. I think what ends up happening is the two clients might be writing their own metrics count to the same metrics-total mbean.
Below is the same application:
/**
 * Licensed to the Apache Software Foundation (ASF) under one or more contributor license agreements. See the NOTICE
 * file distributed with this work for additional information regarding copyright ownership. The ASF licenses this file
 * to You under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the
 * License. You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on
 * an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the
 * specific language governing permissions and limitations under the License.
 */
package org.apache.kafka.clients.consumer;

import java.util.Arrays;
import java.util.Properties;

public class MetricOfMetricsConsumer {
    public static void main(String[] args) {
        KafkaConsumer<String, String> consumer1 = makeConsumer();
        KafkaConsumer<String, String> consumer2 = makeConsumer();
        consumer1.subscribe(Arrays.asList("t"));
        consumer2.subscribe(Arrays.asList("t"));
        while(true) {
            consumer1.poll(1000);
            consumer2.poll(1000);
        }
    }

    public static KafkaConsumer<String, String> makeConsumer() {
        Properties props = new Properties();
        props.put("bootstrap.servers", "localhost:9092");
        props.put("group.id", System.currentTimeMillis() + "");
        props.put("partition.assignment.strategy", "roundrobin");
        props.put("key.deserializer", "org.apache.kafka.common.serialization.StringDeserializer");
        props.put("value.deserializer", "org.apache.kafka.common.serialization.StringDeserializer");
        return new KafkaConsumer<>(props);
    }
}```

So yeah I think the original intent was to just have this metric for a broker, which made me think it was odd putting this in the Metrics constructor. But it kind of makes sense to have this for consumers and producers too. Maybe you can address this bug with an optional tag.

@onurkaraman Thanks much for taking time to test it! I have updated the patch to address your comments. The updated patch use the client-id in metricsTags in the metricName. And I have tested the patch with broker, consumer and producer. Please let me know if the updated patch looks good.

I think your revised patch did the trick. jconsole now shows:
kafka.consumer
| ...
| metrics-stats
  | consumer-1
    | Attributes
      | metrics-total
  | consumer-2
    | Attributes
      | metrics-total

The two mbean ObjectNames now are:
kafka.consumer:type=metrics-stats,client-id=consumer-1
kafka.consumer:type=metrics-stats,client-id=consumer-2
If we want this also for MirrorMaker (once it switches to new consumer and producer), I'm not really sure how that'd work, but I think this is probably good for now.

Great. Thanks for confirming that the patch works.

Actually, I just took a look at this metric for a broker. It only reports the number of org.apache.kafka.common.metrics and doesn't factor in the yammer metrics, which I think is misleading.
This is okay in the long run since we want to migrate all of the sensors over, but it's unclear what the right thing to do now is.

How about I include two attributes (i.e. metrics-apache-total, metrics-yammer-total) under MBean (e.g. kafka.server:type=metrics-stats ), to denote the total number of new and old metrics respectively? This would solve our problem in the short term with relatively small change.

@onurkaraman I have updated the patch such that the new metric shows the total number of attributes of all mbeans in the MBean server. This may include some MBean that is not directly registerd via Kafka, e.g. java.lang:type=Memory:ObjectName. But the semantics of this API is clear and it should solve the problem that motivates this ticket.

Let me think about this for a bit more - originally I thought we should only count the metrics under the kafka-metrics package. I think we should eventually move away from yammer metrics but I think the latest approach of just reporting the total number of mbeans may subsume that and may be useful by itself.

I think recording the total number of mbeans is probably not right. In terms of debugging, we care about how many metrics are coming from kafka. Recording the total number of mbeans doesn't really help us if there's a ton of mbeans being made outside of kafka.

@onurkaraman I think the total number of mbeans is generally useful even in your use case -- it means something is going wrong if the number goes through roof. Importantly, it addresses the usecase that motivates this patch, i.e. the number of mbeans registered on the server side.
If you think we need to specifically record the number of mbeans registered by kafka threads, we can add a filter in the query to only record those mbeans named kafka. But I think it is actually useful to report total number of mbeans from user's perspective.

I agree that it may be useful to record and report the total number of attributes. The main concern I have in the current approach is that each lookup for this metric (from some metric pulling system) will involve a traversal of the mbean registry which is a bit much if the lookup happens frequently. There are a couple of ways to avoid this/improve it:

Cache the count and avoid updating the count unless a certain period (e.g., a few minutes) has elapsed
Trap mbean registrations/deregistrations and increment/decrement counts there
Don’t count all mbean attributes - instead, only count Kafka metric attributes. i.e., back to your earlier approach.

@jjkoshy I think the updated the patch should address the concern raised by you and Onur. Thanks!

LGTM

Given the dependency constraint, I think the best approach would be to have a kafka-metrics-total and yammer-metrics-total.
kafka-metrics-total can get created within Metrics
yammer-metrics-total can get made in KafkaServer on startup
kafka-metrics-total could use metrics.size()
yammer-metrics-total could use the com.yammer.metrics.Metrics.defaultRegistry().groupedMetrics().values() trick.

@onurkaraman @becketqin Thanks much for taking time to review the patch! I have updated the patch so that we don't introduce yammer dependency to new producer and new consumer. With the latest patch, server will have 2 additional jmx metrics with name kafka-metrics-total and yammer-metrics-total. New producer and new consumer will  have 1 additional jmx metrics named kafka-metrics-total. Old consumer will have 1 additional metrics named yammer-metrics-total.
I have tested the patch with server, new producer and old consumer. The patch works as expected. Can you please have another look?

Test failure is unrelated to this patch.

@guozhangwang Would you have time to review this patch?

Went through the code and it looks good to me overall, but I would like to borrow another pair of eyes from @junrao to take a look as my knowledge in this module is shallow.

Thank you @guozhangwang @jjkoshy for your review. I have updated the patch as suggested.

@junrao My last commit remove the code that explicitly sets the default tags, and instead use metrics.metricName(name, group, description) to create MetricName with default tags.
I still kept the explicit usage of metricTags in Selector because ControllerChannelManager.addNewBroker(Broker) needs the flexibility to changes tags.
Could you have a look and let me know if this is what you originally suggested? Thanks!

@lindong28 Seems some of the indentation does not match check-style rules in the Jenkins job, could you verify?

@guozhangwang I was wondering why the Jenkins test failed without providing failure information. I have fixed indention and remove unused imports. Could you have a look? Thanks!

kafka.api.ProducerBounceTest.testBrokerFailure passed on my machine. There maybe ephemeral error with kafka.api.ProducerBounceTest.testBrokerFailure. But I don't this is caused by this patch.

There is a compilation error when running test.

There are still quite a few places where we create MetricName directly instead from Metrics: ClientQuotaManager, Sender, SocketServer and RecordAccumulator.

@junrao Ah...my apologies. I thought it passed the testcase.. I am fixing these now.

@junrao The patch passed unit tests and the check style test on my machine. All MetricName instances are now created via Metrics throughout the code. And the patch is rebased on trunk.
All existing jmx metrics should have exactly the same name and attributes before and after this patch. I will do another test to validate that this, by creating 2 KafkaProducer and 2 KafkaConsumer with different client.id. I will update the patch if it doesn't work. Otherwise I believe it is ready for review.

@junrao I have validated that existing jmx metric names and attributes are exactly the same before and after the patch. The new metrics are added as expected for server/producer/consumer. The patch can be cleanly applied to the latest trunk. MetricName constructors are added back with deprecated tag.
More detail are available in the other reply to your comment. Thanks for taking so much time to review the patch.

kafka.integration.SaslSslTopicMetadataTest.testIsrAfterBrokerShutDownAndJoinsBack succeeded on my machine. It should be ephemeral failure unrelated to this patch.

The comment at the beginning of MetricName.java also needs to be changed to create MetricName using metrics.MerticName.

@junrao Yeah I should update the comment for the class. I just updated comment for the class, the first constructor, and the example code in the comment that uses the MetricName. Thank you.

Thanks for the patch. Looks good. Just a minor comment. Also, could you rebase?

@junrao Great! Thanks so much for your time and review. I have removed a bunch of unnecessary space and rebased the patch against trunk.

Thanks for the patch. LGTM

@lindong28 I got some warnings when compiling with this patch, could you take a look? Seems "<>" cannot be directly used in comment link:
/Users/guozhang/Workspace/github/guozhangwang/kafka-work/clients/src/main/java/org/apache/kafka/common/MetricName.java:118: warning - Tag @link:illegal character: "60" in "org.apache.kafka.common.metrics.Metrics#metricName(String, String, Map<String, String>)"
/Users/guozhang/Workspace/github/guozhangwang/kafka-work/clients/src/main/java/org/apache/kafka/common/MetricName.java:118: warning - Tag @link:illegal character: "62" in "org.apache.kafka.common.metrics.Metrics#metricName(String, String, Map<String, String>)"
/Users/guozhang/Workspace/github/guozhangwang/kafka-work/clients/src/main/java/org/apache/kafka/common/MetricName.java:118: warning - Tag @link: can't find metricName(String, String, Map<String, String>) in org.apache.kafka.common.metrics.Metrics
/Users/guozhang/Workspace/github/guozhangwang/kafka-work/clients/src/main/java/org/apache/kafka/common/MetricName.java:77: warning - Tag @link:illegal character: "60" in "org.apache.kafka.common.metrics.Metrics#metricName(String, String, String, Map<String, String>)"
/Users/guozhang/Workspace/github/guozhangwang/kafka-work/clients/src/main/java/org/apache/kafka/common/MetricName.java:77: warning - Tag @link:illegal character: "62" in "org.apache.kafka.common.metrics.Metrics#metricName(String, String, String, Map<String, String>)"
/Users/guozhang/Workspace/github/guozhangwang/kafka-work/clients/src/main/java/org/apache/kafka/common/MetricName.java:77: warning - Tag @link: can't find metricName(String, String, String, Map<String, String>) in org.apache.kafka.common.metrics.Metrics

@guozhangwang Thanks for the catch. I didn't notice the warning message.. I have created a minor pull request at https://github.com/apache/kafka/pull/651. Can you take a look?


deploy app from git

Reference Issues or PRs

What does this implement/fix?
Adds functinality to the Deploy App form to pull in an app from a git repo
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[x ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[x ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ x] Did you test the pull request locally?
[ x] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[x ] The content is written with plain language (where relevant).
[ x] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ x] All links describe where they link to (for example, check the Nebari website).
[ x] This content adheres to the Nebari style guides.

Non-text content

[ x] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

@jbouder made the changes to the Radio buttons, put them below the Deploy titles.

I updated the branch to test-branch in the form, but the request is still sent with ref: "main", same for conda-project.yml:

@aktech made updates to repository config object

Thanks, it needs to be passed in the server endpoint as well, it isn't at the moment: /server
    "user_options": {
        // ...
        "repository": {"url": "https://github.com/nebari-dev/jhub-apps-from-git-repo-example.git"}
    }

@aktech Ooops!
Made the update to user_options

The "keep app alive" and "public" are false in the UI toggle, even thogh the API response has true for both of them:

We should display the error message from backend:
As in this case repository is valid, but there was some other error:

@aktech made the Error title more generic, to 'ERROR'
please review PR again.

Looks like all the tests are failing after merge to main, can you take a look: https://github.com/nebari-dev/jhub-apps/actions/runs/11889273170/job/33132014404

Never mind, the test failures are unrelated. Looks like the new PyJWT version released yesterday, introduced some breaking changes. I have created a PR to pin it for now: https://github.com/nebari-dev/jhub-apps/pull/532


Remove explicitly setting agent's reporter type

Tchannel reporter has been removed in 1.18.0
The GRPC reporter was set as default in 1.11.0
This also helps with using OTEL based images since they do not expose --reporter.type flag.

The flag hasn't been deprecated or removed, but there is only one reporter type and no current plans to add another one, hence we can use the default.

From our operator's perspective, sure. But I mean that other people out there might be using this flag as well. Just removing them without deprecating will break them.

How would it break them? The flag is still supported in Jaeger.

Sorry, my bad. I thought that having this flag would cause an error with the new binaries.


DA-4290: use dynamic mapping for bugzilla & bugzillarest

Signed-off-by: Ibrahim<EMAIL_ADDRESS>

I'll build a new SDS on top of this.


[BUG]: Prerequisite of multiple secrets which is not necessary with the same cert and key

Bug Description
Remove the secret revproxy-certs which is duplicate of csirevproxy-tls-secret
kubectl create secret -n  tls revproxy-certs --cert=tls.crt --key=tls.key
kubectl create secret -n  tls csirevproxy-tls-secret --cert=tls.crt --key=tls.key
Logs
https://github.com/dell/csm-docs/blob/ed594b799155c28a7f00ef674d4480db92385084/content/v3/deployment/csmoperator/drivers/powermax.md?plain=1#L200
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
https://github.com/dell/csm-docs/blob/ed594b799155c28a7f00ef674d4480db92385084/content/v3/deployment/csmoperator/drivers/powermax.md?plain=1#L200
Expected Behavior
Only one secret should suffice
CSM Driver(s)
PowerMax
Installation Type
No response
Container Storage Modules Enabled
No response
Container Orchestrator
NA
Operating System
NA

@shanmydell: should this be tagged with v1.12.0?

/sync

PR raised: https://github.com/dell/csm-docs/pull/1364


[FR]: Support to preview database by inserting ![[database.md]] in Notes

Blogger, could you develop a function like the excalidraw plugin that supports inserting ![[database.md]] in notes to preview the database。

Because the current effect can only be this:

（My English is poor, so I don't know if the expression is appropriate.）

Hi @MeerOst thanks for your suggestion.
I will add it to the backlog :)

starting this issue

with 1.5.1 preview mode shows a data view table with the content =)


Package maintenance

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)

Landed in https://github.com/metarhia/config/commit/86f4290dbd886fd8572a7191e6149d4f4579513d, https://github.com/metarhia/config/commit/bb134f2e4305faadcca68efac671b7294b5cddac, https://github.com/metarhia/config/commit/13851ff6a9bd3663f7e7624d05f7c3f26b0ab229 and https://github.com/metarhia/config/commit/74c3b60f0ade706027e7179b38c053a7fef74f99


Fix the pull request template.

Can u give the description about this issue?

One issue at a time @neokd :))

Haan Sure was just excited about it :-)

When we make a PR, we have to add details about the PR on our own. What if we provide a template prior so that easy for both contributors as well as maintainers?

Once you are done with the issue assigned to you, do let me know. I'll assign you this as well.

Can i work on this issue under JWOC?

Assigning this issue to @Priyamakeshwari for time being.

Thanks bhaiya !


Issue with "urlscan.io" in @site/docs/reference/integrations/urlscanio.md

Describe the problem

Page: urlscan.io
Source: https://github.com/demisto/content/blob/master//Packs/UrlScan/Integrations/UrlScan/README.md

Screenshots

Environment

OS: [e.g. Windows]
Browser: [e.g. chrome, safari, firefox..]
Browser Version:

Suggested fix

Hey @09178407400,
What is the issue in the page you mentioned


separate data source configuration into two parts

Description
Currently, we put all settings of a specific connection in one page, which is problematic, the basic settings includes endpoint/auth/proxy are essential for a connection, other settings may or may not depend on essential settings,
like when setting up epic key for jira connection, it requires access to a jira api which returns all customfields in
order to provide a list of options for user to select, which in turn depends on the essential settings
Describe the solution you'd like
Separate connection settings into 2 parts

Essential settings: contains only the very basis settings, endpoint/auth/proxy
Extended settings: all other settings

Has the Feature been Requested Before?
No

implement PATCH method instead of PUT so we dont't need to do 2 api

Closed by #1717

Closed by #1717


Update java-util.

metamx/java-util#28
metamx/java-util#33
metamx/java-util#34
metamx/java-util#35
metamx/java-util#36

:+1: after travis
can we also release druid-api?

Failures were legitimate, NamespaceExtractionCacheManagerExecutorsTest wasn't starting its lifecycle before stopping it. pushed a fix


代码块增加复制功能

Expected behavior (预期行为)
期望代码块的右上角有一个复制按钮，类似  http://v3.bootcss.com/css/
Actual behavior (实际行为)
目前无
Steps to reproduce the behavior (重现步骤)
目前无
NexT Information
Add the  ✔  sign before an item which is affected by this behavior.
NexT Version:

Master ✔
Latest Release
Old version -

NexT Scheme:

All schemes
Muse
Mist
Pisces ✔

Other Information (Like Browser, System, Screenshots)

just like this ?

是的，如何实现的呢

i  used the clipboard.js !
the github : https://github.com/zenorocha/clipboard.js

@zhuzhuyule 请问鼠标放到左侧边栏后的动画效果怎么做出来的呀？

1.头像动画，css 实现
2.背景显示，js 实现
3.时钟，js实现
4.头像背景，canvas+js实现
@heliuphy 尽量本周内把我的代码都贴出来.

好的，谢谢！
2017-05-31 8:41 GMT+08:00 zhuzhuyule<EMAIL_ADDRESS>
1.头像动画，css 实现
2.背景显示，js 实现
3.时钟，js实现
4.头像背景，canvas+js实现
@heliuphy https://github.com/heliuphy 尽量本周内把我的代码都贴出来.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iissnan/hexo-theme-next/issues/1684#issuecomment-305048594,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXJA6kDFWq-Ls6bh92Gy89tIzZVNcTgvks5r_Lc2gaJpZM4NnFhy
.

@zhuzhuyule 关注中

@zhuzhuyule 关注中，加油！

@zhuzhuyule mentioned

@myoss @heliuphy @Jan9th @yafey @bharals @xkcoding
复制功能已经写好 ，可以尝试了，有问题给我留言
HEXO优化之（二）----添加复制功能

赞一个
2017-06-22 0:00 GMT+08:00 zhuzhuyule<EMAIL_ADDRESS>
@myoss https://github.com/myoss @heliuphy https://github.com/heliuphy
@Jan9th https://github.com/jan9th @yafey https://github.com/yafey
@bharals https://github.com/bharals @xkcoding
https://github.com/xkcoding
复制功能已经写好 ，可以尝试了，有问题给我留言
HEXO优化之（二）----添加复制功能
https://zhuzhuyule.xyz/2017/06/12/HEXO/HEXO优化之（二）-添加复制功能/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iissnan/hexo-theme-next/issues/1684#issuecomment-310125423,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXJA6ln0wJ0uK3TE3DzSmh-8zkFwMZfBks5sGT4DgaJpZM4NnFhy
.

@zhuzhuyule 没有\themes\next\layout\_custom\custom.swig这个文件呀？

@heliuphy 已经更新。
更新此处
感谢提出问题。

感谢分享 (＾▽＾)
发自网易邮箱大师
在2017年06月22日 23:05，zhuzhuyule 写道： @heliuphy 已经更新。
更新此处
感谢提出问题。
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/iissnan/hexo-theme-next","title":"iissnan/hexo-theme-next","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/iissnan/hexo-theme-next"}},"updates":{"snippets":[{"icon":"PERSON","message":"@zhuzhuyule in #1684: @heliuphy 已经更新。\r\n更新此处\r\n\r\n感谢提出问题。"}],"action":{"name":"View Issue","url":"https://github.com/iissnan/hexo-theme-next/issues/1684#issuecomment-310408171"}}}

@myoss solved?

这个功能很使用  👍

我的总是显示不出来复制按钮，和hexo 和next的版本有关系吗？

一直显示不成功，提示这个错误
Uncaught ReferenceError: Clipboard is not defined
at createCopyBtns (custom.js:31)
这是什么原因啊？

Feature already implemented in NexT v6.2.0

NexT is rebased into organization repo.
If you want new feature, fix, or support, create new in NexT v6.x repo (desirable in English).
There is instructions on English or Chinese how to update from v5.1.x to v 6.x
You also may read this for details.


Trying to get in touch regarding a security issue

Hey there!
I'd like to report a security issue but cannot find contact instructions on your repository.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

@JamieSlome I see, you have spread your standard message (see above) over dozens/hundreds(?) of github repos.
So I suggest following procedure:

report your issue (ReDoS?) using this communication chanel. You can use this same issue, just add a new comment below.
I will ask you then for a PoC which explains how Mocha is supposed to be affected by this security issue. Just a warning by Snyk or NPM for a dependency of a dependency is not sufficient.
in case of inactivity I would close this issue after a short period of time.

Yes, we get security issue on a regular basis. And in very most cases they are a waste of time for everybody.
Thank you.

@juergba - see:
https://huntr.dev/bounties/1d8a3d95-d199-4129-a6ad-8eafe5e77b9e/

I consider this issue as spam.


Add Technical Info button

Describe the feature you'd like
Add another Action button for a page called "Technical Information".
When clicking the button, a pop-up(?) can provide the technical info of the complete bread crumb summary for the current displayed item (a page in this example):
Item  > Display name  > Slug   [Technical ID]
Shelve  > MyShelve  > /shelves/cpy-myshelve
Book  >  Book One  >  /books/book-one
Chapter  >Chapter 1  >  /books/book-one/chapter-1
Page  > My  Page  > /books/book-one/pages/my-page [209]
Describe the benefits this would bring to existing BookStack users
This is for technical reference only when an editor etc wants to upload an attachment to a specific page, or make updates to an item via a script or even API where the technical ID of such item is required as part of the API or Node string parameters.
It cuts the requirement from the user side to first run a API/ cURL command to find the required ids for the items to be referenced, being a page, chapter, book etc.an
Can the goal of this request already be achieved via other means?
Yes, if one has the knowledge to issue the commands. It also assumes the specific user does have the access to issue the commands correctly
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
0 to 6 months
Additional context
In my case, I've wanted to upload a number of attachments to various pages using the node script to do so, but need page ids, correct slugs and would find it much quicker and easier to get to those pieces of data. I understand not everyone will need this, but thought it might provide some ease-of-use in a more direct user-friendly manner.

Hi @ferdieb,
Thanks for the suggestion.
From this sounds of this, this primarily would provide the ID for the page since the other listed details can be already be found fairly directly.
The ID can already be found fairly quickly via the link/include tag popup as shown in the "Easily Grabbing the Syntax" section fo the following page: https://www.bookstackapp.com/docs/user/reusing-page-content/#easily-grabbing-the-syntax
The URL or include tag shown in this popup includes the page ID.

Due to the above, and since there's been no further discussion, I'm going to go ahead and close this off.


Disk scheduling in Operating Systems in Java

🚀 Feature
Disk scheduling is done by operating systems to schedule I/O requests arriving for the disk. The various Disk scheduling algorithms are FCFS, SCAN, CSCAN, and SSTF. Disk scheduling algorithms are the algorithms to allocate the services to the requests
Have you read the Contributing Guidelines on Pull Requests?
Yes
Please add the DWOC tag and level of the code.
Motivation
(Please outline the motivation for the proposal.)
Pitch
(Please explain why this feature should be implemented and how it would be used.)

Can you close this issue?
The PR is merged.


ShellCommandAgent timeout

Im having few of somehow long shell scripts to complete, running them via agent end up with error in events:
execution expired
Is there way to setup Agent to wait (best would be infinity) or can we fork out script job outside agent so it will only trigger the start and the script will live it own life?

I never used the ShellCommandAgent but can't you run it like script.sh & ?
Alternatively you can increase the "DELAYED_JOB_MAX_RUNTIME" in .env.

Im trying the DELAYED_JOB_MAX_RUNTIME
I did forget about & but im triggering other scripts after the first one - but now Im thinking I could wrap that into one script that start the other one and start the wraped one with & so they will start one after another
Thanks for the tip :+1: will definitly look into changing scripts

@level20peon sadly the trick with & dosn't work

@level20peon it did semiway.
I made my orginal script in exe.sh
I made script.sh exe.sh &
and run script.sh from agent.
it looks like ShellCommandeAgent hook to script even if its started with &

try nohup then?

To be honest, i didnt know about nohup, i could probably use screen also :+1:
I will stick with wraping the main script with other one with &
Im happy with result, and thanks again about both tips!

nope nohup in command line and inside script dont work the intended way

Setting DELAYED_JOB_MAX_RUNTIME in .env should be the solution.  What do you have it set to, and how long is your script taking?

its depend from the url input. It can take 1minut up to 30 or 45minuts - Im using ARM platform so scripts are executing longer.
I would love to fork out of Open3.popen3(command) but no matter what I do I end up with huginn waiting for finishing job or killing it with above error.
Probably I wont bypass this without editing agent rb file, right?

just run it in screen -dm

It's not clear to me if the system is killing the process, if popen3 is timing out, or if you're hitting your maximum delayed_job run time.

@K1773R -dm nor -dmS both get terminated after DELAYED_JOB_MAX_RUNTIME set time.
@cantino definitly get terminated to the DELAYED_JOB_MAX_RUNTIME
But is there any way to jump out of this to bypass this ? because running time-consuming scripts make huginn stuck on them, and setting DELAYED_JOB_MAX_RUNTIME high in long run can also affect huginn.
I know that bypassing DELAYED_JOB_MAX_RUNTIME mechanism could potentialy be risky making a really CPU/time intense task just overflow huginn, but that is the same risk as using ShellCommandAgent

Well, you can increase DELAYED_JOB_MAX_RUNTIME and also run additional delayed_job workers.  Or make a separate process that POSTs to Huginn periodically.

Or make a separate process that POSTs to Huginn periodically. could you elaborate this more?
addtional delayed_job that could work - I forgot about this because of recommandation to stick with 1 on ARM ;-)

You could use a cronjob or another program and have it POST to Huginn's WebhookAgent.

So currently it impossible to run out of ShellCommandAgent without blocking it.

What about creating a ShellCommandAgent to fork a process which outputs to a file and generates a "cookie" file when done, and another to check for the cookie file and dump the output file (and remove the cookie file)?


Clear representation of multiples

When using the WebGUI and providing multiple entries for some fields (e.g. Bands), it becomes very difficult to see the individual entries. Please provide some sort of visual divider

@KathiSchleidt  could you elaborate more please ?

@baloola take a look at the Bands section of the SPARTACUS_v2.1 Dataset. As this dataset has many bands, it becomes very difficult to understand which fields belong to which band. Some sort of divider between these fields, or a frame around each individual band, would be most helpful.
Same logic applies to all concepts where we can provide multiple entries, each entry consists of multiple fields

it looks like the branch was merged (so it is not visible in the editor), and looking at the actions of the pull request I can see it was not ingested successfully, this is a bug within the pipline, I will work on fixing it.
In the meantime I would recommend to revert the PR

@baloola done, here

Thanks @KathiSchleidt, and yes I saw what you are refering to, I will edit the css so that border lines in the divs are visible again.

added some margins for multiple fields in the Webgui


Deprecation notice urllib3[secure]

[x] I agree to contribute to the project under Apache 2 License.
[x] To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with FlexMeasures

Description
pyOpenSSL and urllib3[secure] are deprecated in the upcoming release (1.26.12)
https://github.com/urllib3/urllib3/issues/2680

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.562%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7205

Relevant Lines:
10091

💛  - Coveralls

Thanks for the notice, @umarfarouk98.
The .txt requirement files are generated with pip-tools. I will study what the necessary changes in .in files would need to be.
First result:
We are importing urllib3 via requests, which is needed by tldextract.
And via selenium, which is needed by timely-beliefs and sentry-sdk.
urllib3[secure]==1.26.8
    # via
    #   requests
    #   selenium
    #   sentry-sdk

It seems none of these hard-code a version requirement anywhere, so I believe there is not much we can do here, other than hard-code urllib3 in the .in files ourselves, and see if that works.
But ultimately, it seems our packaging pipeline would pick up on later urllib3 versions removing the [secure] part, by itself.
Good to see that this PR passes our test pipeline already.

You may want to retain the dependencies that were originally installed via urllib3[secure] like `idna, pyOpenSSL, certifi, etc.

requests and sentry-sdk only require urllib3, but selenium requires urllib3[secure]. We opened a pull request for Selenium: https://github.com/SeleniumHQ/selenium/pull/10932. So the best course of action here could be to wait for Selenium to merge and release this, and then run pip-compile again: [secure] will go away at that point.

Thanks, all. I see the Selenium people already merged that issue, so in one of our next releases (we release every 4-6 weeks, roughly, and usually run pip-compile after each release), this will be okay. I'm merging this to honor your help.


Assertion failed during sanity check of build_causality_graph

Backtrace:
Traceback (most recent call last):
  File "sieve.py", line 714, in <module>
    options.phase,
  File "sieve.py", line 498, in run
    phase,
  File "sieve.py", line 439, in run_test
    oracle_config,
  File "sieve.py", line 312, in check_result
    analyze.analyze_trace(project, log_dir, two_sided=two_sided)
  File "/home/tyler/sieve/analyze.py", line 372, in analyze_trace
    causality_graph = build_causality_graph(event_list, side_effect_list)
  File "/home/tyler/sieve/analyze.py", line 320, in build_causality_graph
    causality_graph.sanity_check()
  File "/home/tyler/sieve/analyze_util.py", line 508, in sanity_check
    > self.event_vertices[i - 1].content.id
AssertionError

How to reproduce
Run learning stage for workload scaleup-scaledown-tserver in yugabyte-operator
python3 sieve.py -p yugabyte-operator -d tylergu1998 -s learn -t scaleup-scaledown-tserver

Fixed by commit fc0d583a5ba03fdea18567021ae4f3d744d6493d


I show an error message in Apple M1 chip environment.

Description
After installing protzo in my mac-mini (Apple M1 chip, macOS11.2.3) environment, I show the following error message when creating a new window in my Terminal application.

zsh: locking failed for /Users/ishii/.zsh_history: interrupt

I don't show this error message on another mac (Intel Chip, macOS 11.2.3) which I'm using.
It seems that it occurs in $ HOME/.zprezto/runcoms/zlogin file.
Maybe in this "if" statement.
if [[-s "$ zcompdump" && (! -s "$ {zcompdump} .zwc" || "$ zcompdump" -nt "$ {zcompdump} .zwc")]]; then

Versions

Prezto: v1.14.6
ZSH version: 5.8
OS information: macOS 11.2.3（20D91)

If you can track it down further, we'd happily entertain a PR. But I don't have an M1 so can't repro/debug further.

I'm on an M1 MacBook Air and cannot reproduce this. Can you start with a clean Prezto install and see if that resolves it?

Thanks for the report. Unfortunately without additional information we can't help debug this, so I'm closing it for now. We're happy to reopen this if you're still experiencing this issue and can debug with us though.


Disabling success state

You can globally disable showing the success validation state.

I think success state being the default makes sense, and thus the option should probably be called disableSuccessState. Thoughts?

Fixed

Was about to start working on this feature, glad to see that a PR already exists. :+1:
Could I ask for an example to be added to the README, otherwise no one will know this option exists.


(git) Installation fails

Expected Behavior

I would expect the package to install just fine
Current Behavior

While installing the package, I get an error message and stating that installation of git.install failed.
Possible Solution

Already tried to follow the fix in this other open issue (https://github.com/chocolatey-community/chocolatey-coreteampackages/issues/1378) but it didn't fix the issue.
Could it be related to this (https://github.com/shiftkey/chocolatey-beyondcompare/issues/29) ?
I am using Windows' own CMD as an Administrator whereas everybody seems to be using Powershell...
Steps to Reproduce (for bugs)

Simply:
choco install git -y
Context

This is also happening with other packages like for instance 'audacity'.
Your Environment

Package Version used: chocolatey-core.extension 1.3.4
Operating System and version: Windows 7 Pro SP1 (Spanish ISO)
Chocolatey version: Chocolatey v0.10.15
Install/uninstall gist: https://gist.github.com/john3voltas/976cdc96a4d5f7d9b3ae0c0b5dc0969e

I experience the same error with various packages (only on Windows 7 with Powershell 2.0).

I have verified that this is indeed a re-producible issue.
Seems to be something related to Powershell v2 compatibility (choco uses powershell when running install scripts even when you launch it through cmd).
Not yet figured out where the problem is, so still investigating that.

Closing this as duplicate of #1373

Is there any solution of this issue?

@RanabirChakraborty This issue is closed. Please raise a new issue.