waitForModule docs wrong

https://github.com/BetterDiscord/BetterDiscord/blob/9b38e442da7f24024997e0509e3e7dd343507a7c/renderer/src/modules/pluginapi.js#L665
"options" does not actually do anything relating to if more then one module is returned

this happens a few times in the docs

This is fixed in the latest release!


refactor(transformer): import oxc_allocator::Vec as ArenaVec

[!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

#6999
#6998  👈
#6997
main

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

Join @overlookmotel and the rest of your teammates on  Graphite


Fixes #252: Add a list tag for containing tracks

Fixes #252
Demo: https://ongaku-253.surge.sh

[x] This issue is for enhancement/bug fixing.
[ ] This issue is an addition to JSON database.

[ ] Pull Request Title is of form Added [Anime Name].

[ ] This addition is already on ongaku.

[ ] This PR is an improvement to current addition.

[x] I have checked my addition locally and it is working and won't break existing entries.
[x] I have gone through Contribution Guidelines.
[x] This pull request will be closed if I failed to update it even once in a week.

@aashutoshrathi can you please look into why deployment link is non-functional. :angry:


404: Not Found when running curl install command plus....

npm doesn't work, wget doesn't work -> none of the install options which have been presented are working for me. Is it simply because I am using WSL? I don't know why that would make a difference. I am using ZSH.
A clear and concise description of the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Relevant Zsh configuration (.zshrc)
# Your configuration here

Environment
Spaceship version:  (use echo $SPACESHIP_VERSION)
Zsh version:
Zsh framework: <None/oh-my-zsh/prezto/etc>
Zsh plugin manager: <None/antigen/zplug/etc>
Terminal emulator: <iTerm/Hyper/Terminator/etc>
Operating system: <macOS/Windows/Linux>
Screenshot
Provide a screenshot that shows your issue.


chore: rename armosec/kubescape

The Organization name had been changed.
https://github.com/kubescape/kubescape

Thank you always!

✅ Confirmed https://github.com/armosec/kubescape is redirected to https://github.com/kubescape/kubescape .

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.56.0


Provide defaults when building target defines

fix #95

Thanks. It is good job.

LGTM. If we run into users with portability issues we may want to still consider something like autoconf which can help identify which systems have which headers.
But this PR improves a lot and fixes some at least one potential buffer overflows, and should be more reliable by having default behavior. We should merge this in.


Add a code-formatter to the project

Summary
As multiple contributors will write code in their unique style it is no harm in having a code-formatter integrated using Maven/CI which will ensure consistency.
Motivation
It will improve code readability and consistency.
Additional Context
https://github.com/jhipster/prettier-java

Can I be assigned this?

Can I take this?

@mohanarpit can you let us know how to proceed here. What should be the formatting syntax?

Looks like we need something like this Super Linter

how about something like ESLint, Prettier, and pre-commit hook to enforce the coding style (Husky)

@JoelLau We do have a pre-commit hook but looks like it does get run when committing directly via Github.

I would like to contribute on this card if required, please let me know


Workers should support TLS

Need to update the workers to support connecting to the taskstore using TLS

Example code instantiating python grpc clients with TLS
https://github.com/GoogleCloudPlatform/python-docs-samples/blob/26fd536d49798b6dab042078eaeb101f88c22c48/endpoints/bookstore-grpc/bookstore_client.py#L27


Create database for fabric_lakehouse is not permitted using Apache Spark in Microsoft Fabric.

Module: 01
Lab/Demo: 03
Task: Create a managed table
Step: 1
Description of issue
I got below error message
AnalysisException: org.apache.hadoop.hive.ql.metadata.HiveException: MetaException(message:Create database for fabric_lakehouse is not permitted using Apache Spark in Microsoft Fabric. )
Repro steps:
1.Follow all steps upto Create a managed table
2.Run the code

Hello @linusyoung
I have run the lab several times, but couldn´t reproduce the issue. The managed table is created every time without issues.

Can you confirm you are using a Trial license mode for the workspace that you have created?

Also, can you reproduce this error in another workspace? if so, can you please include some screenshots?
Regards,
Mihai

Hi Mihai,
I experienced the same issue from the other tutorial (Lakehouse tutorial introduction). Not sure if I missed any permission settings.
Here is Workspace setting

This looks like a permissions / configuration issue in the tenant (specifically the underlying ADLS store). We can't repro it with a clean setup, and unfortunately with an early preview service like this it can be difficult to track down the specific problem. The issue doesn't appear to be an error in the content, so closing this. If you figure out the cause of the problem in your tenant, please let us know!

@GraemeMalcolm,
Can you point me some direction regarding ADLS storage? In the Trial capacity, I can't find any related settings. I assume the underlying ADLS storage is configured automatically when I created a new workspace. Also, I checked the tenant setting in the Admin Portal, it does enable the Fabric Items.

The storage is indeed set up automatically along with your workspace. This seems to be some sort of issue in your tenant, which could be related to things like the datacenter region where your tenant is defined or some other setting in your subscription. The lab steps themselves work when tried against a newly created tenant, so it doesn't seem like the issue is in the steps. Unfortunately, we can only support the lab content here - not the product itself. You can check regional status for the service and find FAQs for common known issues at https://support.fabric.microsoft.com/en-US/support/, or engage with the community there to try to resolve the issue you're having.

@GraemeMalcolm
Thanks for your response. I will check with our IT department to see if they can provide more information.

@linusyoung , I was running into the same problem and was able to resolve it by going into the Lakehouse Explorer of my Notebook and selected "Remove all Lakehouses" and then I added back the desired lakehouse and I was able to successfully create the delta tables in my lakehouse.  See screenshot for reference.

Thank you, @jcbendernh , your answer is about 1 year old, but the problem still occurs and your workaround still solves the problem!


Using minimum instead of fixed dependency versions.

Resolves #85.

Hi @NF997 and @Grunny , is it possible to finish this merge request?
I have now come across a conflicting dependency on the python package six.


can't find this paper

could you please open your paper link?

Our paper is still under the anonymous review stage, so we have hidden the link. We will release the pdf once after the review.

Our paper is still under the anonymous review stage, so we have hidden the link. We will release the pdf once after the review.

our paper has been accepted for presentation at aaai2021 with the arxiv preprint.


Haskell major version alias

Describe the bug
Currently we can access ghc and haskell packaging using either:

An arbitrary version (the "current" one) using either ghc or haskellPackages
A precise version, including minor version number, using for example ghc.compiler.ghc8103 and haskell.packages.ghc8103.

The problem with this approach is that:

binary caches may only provides cached version for the "latest" minor version. for example, if ghc8101, ghc8102 and ghc8103 exists, there may only be cached version for ghc8103.
user may want to depend on a specific "major" version of GHC, for example ghc 8.10 (i.e. ghc810x) and update "automatically" to the latest "minor" version available.

Expected behavior
I propose to introduce aliases for major ghc version. for example, ghc810x -> ghc8103.
However, a "safety" mecanism must be introduced in nixpkgs in order to ensure that the alias stays in sync.
Notify maintainers
@peti

Dupe of https://github.com/NixOS/nixpkgs/issues/109256 (double-posted by accident?). Only difference is the title.

Dupe of https://github.com/NixOS/nixpkgs/issues/109256 (double-posted by accident?). Only difference is the title.

(Sorry, github chock when posting with a "fatal error", I just go back in history, change the now empty title, and submited again.

(Sorry, github chock when posting with a "fatal error", I just go back in history, change the now empty title, and submited again.


The reply URL specified in the request does not match the reply URLs configured for the application

Hi,
I'm getting this message but i'm 100% sure the URL is the same.
I've tried multiple settings but i can't get this to work, while it has worked in a previous version before it was a HACS component.
I've tried enabling both internal and external URL's, checked the certificate and recreated the azure config multiple times according to readme but no luck.
It it the port number, or is this component still using the base_url for initial configuration?
Here is my config;
#  external_url: "https://home.domain.eu:8123"
#  internal_url: "http://<IP_ADDRESS>:8123"
  internal_url: "https://home.domain.eu:8123"

Here's some messages from my log:
2020-05-29 16:51:19 WARNING (SyncWorker_14) [custom_components.o365.utils] Could not loacte token at .O365-token-cache/o365.token
2020-05-29 16:51:19 WARNING (SyncWorker_14) [homeassistant.components.http] Detected use of deprecated base_url property, use homeassistant.helpers.network.get_url method instead. Please report issue for http using this method at homeassistant/components/http/init.py, line 136: for frame in reversed(extract_stack()):
2020-05-29 16:51:19 WARNING (SyncWorker_14) [custom_components.o365.utils] Could not loacte token at .O365-token-cache/o365.token
2020-05-29 17:27:32 WARNING (MainThread) [homeassistant.components.webhook] Received message for unregistered webhook

Try with alt_auth_flow as described in readme.

Tried that also, and i got the same result. I looks like base_url is used in this proces, which had been deprecated and gives errors when configured.
I'll try again today by removing all setttings at HA and Azure.
Thanks you for the suggestion.

After I've tried with alt_auth_flow it worked for me flawlessly.

After I've tried with alt_auth_flow it worked for me flawlessly.

Remove everything, configured both internal en external url to https:// and recreated the Azure config. Got it working now!
First i thought i wasn't working because i got a blank page after authorizing, but then i remembered a url was requested. Copied the url from the blank MS-page and pasted it in the HA-page in the 0365 component. Instructions can be a little more clearly on that part.


Update documentation: added timeout to params in delete() call

When I try to delete snapshot with huge indices (~5-10gb), delete call raises an HttpConnectionError exception: ReadTimeout


Need clarification for RI discount paired with EA Discount or ACD

Hello- can you please add on here explicitly info that explains the Recommendation values and how the discount is applied there is an underlying EA Discount and/or an ACD %. We really need this info for customers. Microsoft alias is askenned

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

ID: 56bed59a-bfa7-c102-8051-ed914c5d27f9
Version Independent ID: 5810c399-21d1-374e-a6e6-fb32438a54db
Content: How an Azure reservation discount is applied - Microsoft Cost Management
Content Source: articles/cost-management-billing/reservations/reservation-discount-application.md
Service: cost-management-billing
Sub-service: reservations
GitHub Login: @bandersmsft
Microsoft Alias: banders

@moneyclouds
Thanks for your feedback! We will investigate and update as appropriate.

@moneyclouds, thank you for the feedback.
@bandersmsft - could you please review this question and share your thoughts. Thank you

I've updated https://learn.microsoft.com/en-us/azure/cost-management-billing/reservations/reservation-discount-application and https://learn.microsoft.com/en-us/azure/cost-management-billing/reservations/reserved-instance-purchase-recommendations
to add:

The savings that are presented as part of reservation recommendations are the savings that are calculated in addition to your negotiated or discounted (if applicable) prices.

#please-close

@moneyclouds FYI.


Make Prettification optional

Hi, I have added the builder methods withPrettify() and withoutPrettify() to Labels (locally) as my labels value were being corrupted and I couldn't link between Loki and Tempo.
I tested and it would seem that Loki can now (v2.6.1) accept unprettified labels/values.
Do you think this is a good idea?
Paul.

Thank you for telling about this bug.
Here is the specification https://prometheus.io/docs/concepts/data_model:

Label names may contain ASCII letters, numbers, as well as underscores. They must match the regex [a-zA-Z_][a-zA-Z0-9_]*. Label names beginning with __ are reserved for internal use.
Label values may contain any Unicode characters.
A label with an empty label value is considered equivalent to a label that does not exist.

So, I will fix it by prettifying label names only - label values will never be changed.

Done. Uploaded to Maven Central with version 0.3.9.
It will be accessible after few hours.
Now all label values are accepted (still name-value length limits matters).


LongLora fine-tuning support

LongLora is "an efficient fine-tuning approach that extends the context sizes of pre-trained large language models". They propose to fine-tune a model with a sparse local attention while maintaining dense attention during inference. The Shifted-Sparse Attention (S^2-Attn) is depicted in the following (from the paper):

Moreover, the implied modification is relatively simple:
# B: batch size;
# S: sequence length or number of tokens;
# G: group size;
# H: number of attention heads;
# D: dimension of each attention head
# qkv in shape (B, N, 3, H, D), projected queries, keys, and values

# Key line 1: split qkv on H into 2 chunks, and shift G/2 on N
qkv = cat((qkv.chunk(2, 3)[0], qkv.chunk(2, 3)[1].roll(-G/2, 1)), 3).view(B*N/G,G,3,H,D)

# standard self-attention function
out = self_attn(qkv)

# out in shape (B, N, H, D)
# Key line 2: split out on H into 2 chunks, and then roll back G/2 on N
out = cat((out.chunk(2, 2)[0], out.chunk(2, 2)[1].roll(G/2, 1)), 2)

This can be effectively enabled only during the fine-tuning phase while the standard dense attention can be used during inference.
Another thing that should be modified is the padded sequence length, which should be a multiple of the group-size.
If you think that this can be added to lit-gpt, I'm willing to contribute with a PR (I've already something working which I plan to test)
Edit:
I forgot to mention that they also use the Position Interpolation to rescale the position indices. If I'm not mistaken this can be achieved by simply change the rope_condense_ratio to account for the increased contex-size

I've put together something here.
To further reduce the memory consumption I've also added the possibility to remove the last n layers in the model, as specified in "The Unreasonable Ineffectiveness of the Deeper Layers", Sec. 4.4.
I've trained a model on a Lightning Studio on a A10G with the following hyperparameters:
litgpt finetune lora \
--config "/teamspace/studios/this_studio/litgpt/config_hub/finetune/mistral-7b/lora.yaml" \
--lora_r 16 \
--lora_dropout 0 \
--lora_query true \
--lora_key true \
--lora_value true \
--lora_projection true \
--longlora_n_groups 4 \
--longlora_context_length 8192 \
--logger_name "tensorboard" \
--data.pad_multiple_of 4 \
--checkpoint_dir "/teamspace/studios/this_studio/checkpoints/mistralai/Mistral-7B-Instruct-v0.1/" \
--train.micro_batch_size 1 \
--train.max_seq_length 8192 \
--train.remove_last_perc_layers 0.0 \
--train.get_longest_seq_length true \
--train.trainable_params "wte,norm_" \
--precision "bf16-true"

With those settings I have:

model.block_size=8192 with an adjusted rope_condense_ratio=2.0
The maximum sequence length (even though set to 8192 from the CLI) equals to 728, which has been padded to the nearest integer multiple of 4 (LongLora number of groups)
LongLora Shift-Sparse Attention is activated (longlora_n_groups=4), even though is not strictly necessary in this use case since a sequence length of 8192 is never encountered

Those are two generations:
Below is an instruction that describes a task. Write a response that appropriately completes the request.

### Instruction:
Recommend a movie to watch on the weekend.

### Response:
One great movie to watch on the weekend is "The Shawshank Redemption". It's a classic drama with a compelling story, great acting, and a positive message.

Below is an instruction that describes a task. Write a response that appropriately completes the request.

### Instruction:
Recommend a movie to watch on the weekend.

### Response:
One great movie to watch on the weekend is "The Shawshank Redemption". It's a timeless classic with a compelling storyline and excellent performances.

Thanks for sharing and writing-up this thorough description. I saw the paper a few months back but must admit that I didn't have time to read.
Btw I am all in terms of supporting interesting research techniques that help around real & common issues (e.g., high memory requirements, LLMs not being able to handle long contexts, etc.)
In general, something I am wondering about is if it's really LoRA specific, or could it also be used with "full"-parameter finetuning?
The --train.remove_last_perc_layers is also a nice to have. I'd would apply it in a separate PR, and I think it's useful to have.
What do you think @awaelchli @carmocca ?

Hi @rasbt,

In general, something I am wondering about is if it's really LoRA specific, or could it also be used with "full"-parameter finetuning?

Even though in the paper they have specifically designed everything for fine-tuning with LoRA, it's something that I also thought about. The concern that I have is the flow of information between the first and the last token, which is mitigated during the fine-tuning since the pre-training has already been done on a ton of data. The authors ablate this in B.3 and found that it doesn't influence the finetuning. Maybe it can be applied during pre-training by adopting the Variant-2 in B.2, where they use a separate group in the shifted tokens?

The --train.remove_last_perc_layers is also a nice to have. I'd would apply it in a separate PR, and I think it's useful to have.

Sure

Hi guys, I'm catching up here. I've spotted a little bug due to a missing reshape and I've also implemented LongLora for the full-finetune. If it's ok from you I'll open a PR
cc @rasbt @carmocca @awaelchli

To me both would be welcome and valuable contributions :).
I would maybe do both in separate PRs as that would make the code review a bit easier


Where do I find a github like jst template

I managed to generate a HTML but the look is very plain, I would love to have it generate github like README pages. Any help would be very nice.

You can use Bootstrap in a template.
template.html
<!DOCTYPE html>
<html lang="de">
  <head>
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <title>Document</title>
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/css/bootstrap.min.css" integrity="sha384-1q8mTJOASx8j1Au+a5WDVnPi2lkFfwwEAa8hDDdjZlpLegxhjVME1fgjWPGmkzs7" crossorigin="anonymous">
  </head>
  <body>
  <div class="container">
<%=content%>
  </div>
  </body>
</html>

Gruntfile.js
		markdown: {
			all: {
				files: [
					// your files config
				],
				options: {
					template: 'template.html'
				}
			}
		}

Thanks


Is there any provided code for extracting the specific QA pairs?

The output of submit.py --network_path ./configs/cc_openquestion_filter.CC-MAIN-2023-23.json only reaches the filter part mentioned in the paper. The output is a subset of the input file paths, indicating that these files contain QA pairs. Is there any provided code for extracting the specific QA pairs afterwards?

Due to the loss of process files, we do not have the index corresponding to the QA data. We will be releasing the classifier weights soon, which we use to identify potential QA pairs.


Incorrect notification when adding a new role

This issue is imported from pivotal
When adding a new role from mijn.eduid, the pop-up ask you to validate your name. This used to be two different notifications (one for name-validation, one for adding a role).

(Peter Havekes - Nov 16, 2023)


Can not parse macos system binary files.

Version and Platform (required):

Binary Ninja Version: 4.2.6337-dev personal edition
OS: MacOS
OS Version: Sequioia 15.0.1
CPU Architecture: M1

Bug Description:
__text section of Mach-O file is not getting parsed/decompiled
Steps To Reproduce:

Copy  /usr/libexec/watchdogd
Open the file with binja
Change the platform to macos-AARCH64
Go to the __text section or click on any function from the symbol view

Expected Behavior:
The disassembly should be shown.
Additional Information:
The same thing happens with system at least (I have not tried any other) binaries on iOS.
watchdogd.zip

This is working fine for me with both 4.2.6337-dev and 4.2.6390-dev - do you have initial analysis hold set maybe?

I gave it one mo


Advice | how to format correctly the code (right column) vs the explanation (center column)

I noticed that in order to correctly format the code in the right column with the text in the center column the code snippet should go first, e.g.
# My Title

## My Subtitle

\```csharp
///Code snippet
\```

\```java
Code snippet
\```

\```php
<?php
//Code snippet
?>
\```

\```ruby
#Code snippet
\```

Whatever that goes in the center column. Lorem ipsum

Putting the content for the center column first and the code snippet afterwards will cause the code snippet to be aligned with the last line of the center column, if the code snippet goes first, then the code is aligned with the first line of the center column.
Please note for this example I am escaping ``` because otherwise the comment will not display correctly

That sounds right, you should be putting the code first. Is it not working for you?

works like a charm but had to figure that out hehe. Maybe adding this note somewhere in the instructions could help and then we can remove this comment since it is not an issue per se.

Ah, sorry, thought that was documented somewhere. I don't have time in the immediate future to fix, so I'll leave this issue open. Contributions are welcome to both Slate and the wiki!

ill do the wiki then :)

Sounds good! If you manage to come up with a good solution, feel free to close this issue, too.

put the comment in the wiki, in the end I dont think this is a bug, it just needed further explanation :)


Powerline tutorial should mention how to modify settings.json

In the Install a Powerline font section, it doesn't mention how to modify your settings. It took me a while to figure out I should

Open settings
Modify setting.json like this (other ways work too)

{
    "profiles":
    {
        "defaults":
        {
            // Put settings here that you want to apply to all profiles.

            "fontFace": "Cascadia Code PL"
        },
        ...
    }
}

As a new user, I would really expect the documentation could cover that.

Huh, it looks like that paragraph just hasn't been localized yet. In the en-us version of that page:

To set the Cascadia Code PL font for use with PowerLine (after downloading, unzipping, and installing on your system), you will need to open your profile settings in your settings.json file by selecting Settings (Ctrl+,) from your Windows Terminal drop-down menu.

So that doc will probably be updated shortly with the new text, once it's translated.

Thanks for clarifying.


Improve the example with talkback

Adds a talkback and read to the basic example. :)

I'm closing this for now. This was submitted pre tokio / futures-mio merge. There are some examples on tokio-core now.


App crashing in SpeakersDetail Activity

Actual Behaviour
Currently on clicking some speakers in the SpeakersFragment the app is crashing as seen in the log posted below.
Expected Behaviour
Ensuring the SpeakerDetailActivity opens without making the app crashing.
Steps to reproduce it

Go to the SpeakersFragment
Now over there click on the speakers to view their detail and it is seen in some the detail screen opens while in some the app crashes.

LogCat for the issue

Screenshots of the issue

Would you like to work on the issue?
Yes

loadSpeakerImage should have never been called when selected speaker is not loaded from the observable. I think in one of the recent commits, someone has assumed the loading of speaker is synchronous. A better approach would be to move the network util listener inside the subscribe method here
This way selectedSpeaker is never null when the photo loading function is invoked

@iamareebjamal
actually this work was done by me. I have resolved it but will have a look at your described method later.

:+1:


generate_worlds.launch is broken again

The vrx-docker README has instructions to generate practice worlds from yaml files: https://github.com/osrf/vrx-docker/blob/master/README.md#preparing-trials-for-a-task
These instructions current produce an error because the prepare_task_trials.bash script calls generate_worlds.launch which eventually generates a xacro file that references macros that xacro is unable to find. For example, the command
prepare_task_trials.bash stationkeeping

generates a file called stationkeeping0.world.xacro that contains the following lines:
<world name="robotx_example_course">
  <xacro:include filename="$(find vrx_gazebo)/worlds/xacros/include_all_xacros.xacro" />
  <xacro:include_all_xacros />
    <!-- === sydneyregatta_minus_scene === -->
      <xacro:sydneyregatta_minus_scene />

However, when xacro is called on this file it is unable to find the macro sydneyregatta_minus_scene, which should have been included by the include_all_xacros macro, which contains the following lines:
  <xacro:macro name="include_all_xacros">
    <xacro:include filename="$(find vrx_2019)/worlds/sandisland.xacro" />
    <xacro:include filename="$(find vrx_gazebo)/worlds/xacros/sandisland_minus_scene.xacro" />
    <xacro:include filename="$(find vrx_gazebo)/worlds/xacros/sydneyregatta_minus_scene.xacro" />

None of the xacro files included by this macro can be found anymore. If they are manually added to the stationkeeping0.world.xacro file, it works fine.
This was tested and working earlier this year, so I'm not sure what changed. I know we have repeatedly had problems with this particular design choice. At least once the reason was that the version of xacro updated and changed the way it scoped included files.
I'm still working on a fix, but it would be helpful if someone could verify that they can reproduce the error, just in case I made some mistake in my setup somewhere.

@caguero or @j-herman would you mind running through the instructions at https://github.com/osrf/vrx-docker/blob/master/README.md#preparing-trials-for-a-task to verify this is broken. The error message you should see is:
unknown macro name: xacro:sydneyregatta_minus_scene

@M1chaelM I was seeing this error on Tuesday, but only sydneyregatta_minus_scene wasn't being found, which seemed even odder since it's in the same folder as others that weren't raising the error.  I tried everything I could think of to troubleshoot, with no success, and it suddenly started working again.  I'll dig back through the history to see if there's anything that might have triggered the change, but I really think it was not something I did.
Try rebooting?

The error is back on my system as well, but I can confirm one-time success since I have the three xacro and world files that were constructed from the new wildlife yaml.  I will document my troubleshooting steps in more detail this time, and hopefully if the error goes away again we'll have a better idea of where to look.

Thanks for confirming. I did try updating and rebooting just in case, but still no luck. I will investigate. I think this will require looking into xacro source code because the project documentation is poor.

Yes, after spending a few hours looking through the xacro documentation and source code, I think it's more surprising that it worked once than that it's broken.
Historical research shows that the fix which you thought had broken something last fall was somehow lost on a subsequent update, which may be why it all started working again that time.  The fix was re-fixed last month.
It will be easier to change how we are including the files than to try to sort out what's going on with xacro.  From the time I spent going through the source and the recent changes, it seems like the file includes live in the top-most level and we shouldn't be having this problem.  There is a way to set the scope for properties, but not for includes.

I think we should file an issue against the xacro repo to at least get clarity on how it's supposed to work. I will do this. I'm not really sure whether our nested include strategy is supported or not.


Test deployment to own device (iOS / Android) with npm run dev

Possible only for builds, please read the documentation.


[DataFactory] Add queryTrigger API and rerun from failure support

Hello @LijuanZ , could you

Please add link(s) to the swagger spec review PR that is related to this code generation in the PR description
Test error is blocking the CI
Please resolve the conflict

Thanks

Hi @isra-fel, thank you for checking. Because of the conflicts and we also have some more changes added to swagger, I'd like to abandon this PR. Changes are included in PR: https://github.com/Azure/azure-sdk-for-net/pull/10140

Abandoning this one, the change is included in another PR


Feature/update to v0.9.2

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

This branch contains commits not ready to be merged. Deleting it.


[ACS][CallAutomation] Migrate test to use playback tests

Issue: https://github.com/Azure/azure-sdk-for-python/issues/31980
This updates to work with new plyaback test to work with asset uploaded here (https://github.com/Azure/azure-sdk-assets/tree/python/communication/azure-communication-callautomation_238e3f10c9)

Updated to use AzureRecordedTestCase
Deleted custom recordings (i.e. incoming events)
Updated to have custom recordings work with asset repo and its playback
Sanitization added for recordings
Updated to work with better event handling
Overall bug fixes and improvements on usuage

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


Panel maximize to the left focus border and dimensions are wrong

VSCode Version: Code - Insiders 1.18.0-insider (8dfa696a1eb6ff67eb68925ddaa9e83d3c56b791, 2017-10-31T16:36:35.429Z)
OS Version: Linux x64 4.13.0-16-generic
Extensions:

Extension
Author (truncated)
Version

EditorConfig
Edi
0.11.1

lorem-ipsum
Tyr
1.0.0

sort-lines
Tyr
1.3.0

vscode-svgviewer
css
1.4.3

tslint
eg2
1.0.14

git-project-manager
fel
1.4.0

md-navigate
jri
0.0.1

theme-monokai-pro-vscode
mon
1.1.2

vscode-scss
mrm
0.6.2

seti-icons
qin
0.1.3

Ruby
reb
0.15.0

vim
vsc
0.10.2

(1 theme extensions excluded)

#36571
Focus is fatter and the wrong shape for the maximize (to the left) icon

Correct:

Duplicate of #37321


How to get stats for different teams?

Hi , I have installed an icebreaker bot in different teams, I want to get the statistics for each team separately, like number of pairs notified, number of users notified, number of users paused @Porkai-Pandian @shashishailaj
1 icebreaker bot and it is installed in different teams need statistics for each team separately.
from Application insights currently, I am able to get data for 1 teams channel, but how can I get it for different teams.

Hi @Nandeesh82,
Thank you for raising the query, as you mentioned the details can be gathered from App Insights, where you can check the number Teams the bot installed as well, however it'll be summary of count and not as per individual team. If you want individual metrics, then you need to go separate deployments for each Teams channel.

Hi @Nandeesh82
Please let us know if you've any other queries, if not we'll proceed with closing this issue.


Increase php memory_limit to 768M

I've just installed fresh Nextcloud 25.0.2 and with default role's values it's getting error: The PHP memory limit is below the recommended value of 512MB.
Current default was 512M, so I've changed it to 768M (added 256M), which solved this issue.
@aalaesar @staticdev What do you think about changing default to new value?

I've just installed fresh Nextcloud 25.0.2 and with default role's values it's getting error: The PHP memory limit is below the recommended value of 512MB. Current default was 512M, so I've changed it to 768M (added 256M), which solved this issue. @aalaesar @staticdev What do you think about changing default to new value?

I see it as a possible bug on nextcloud actually. Maybe we can double check before changing the role.

Hi! Sorry for late response. I've tested it twice again on new, clean VMs and now I cannot reproduce it anymore and I'm not getting any error.

I think you used ansible as intended by changing the default value.

Yes, I did, copied whole file from defaults and tweak it as always did ;)
I meant that default should work out-of-the-box (maybe not fully optimized, but without errors.) Anyway it vanished, so current defaults are good and error/warnings free on new instances. So I'm closing this PR for now :)


ValueError: 'Radiator_73' is not in list

I am training the model with customdataset of Radiator.
While running the code I am getting the below error:
Generating anchor boxes for training images and annotation...
Average IOU for 9 anchors: 0.83
Anchor Boxes generated.
Detection configuration saved in  Radiator/json/detection_config.json
Training on: 	['Radiator']
Training with Batch Size:  4
Number of Experiments:  200
Training with transfer learning from pretrained Model
/usr/local/lib/python3.6/dist-packages/keras/callbacks.py:1335: UserWarning: epsilon argument is deprecated and will be removed, use min_delta instead.
warnings.warn('epsilon argument is deprecated and '
Epoch 1/200
ValueError                                Traceback (most recent call last)
 in ()
7 # In the above,when training for detecting multiple objects,
8 #set object_names_array=["object1", "object2", "object3",..."objectz"]
----> 9 trainer.trainModel()
10 frames
/usr/local/lib/python3.6/dist-packages/imageai/Detection/Custom/generator.py in getitem(self, idx)
110
111                 # determine the index of the label
--> 112                 obj_indx = self.labels.index(obj['name'])
113
114                 # determine the location of the cell responsible for this object
ValueError: 'Radiator_73' is not in list

An information is missing in your dataset. It can be:

a missing image
a missing annotation file for an image
didn't add object name to  object_names_array in your setTrainConfig function

I used below code :
from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer()
trainer.setModelTypeAsYOLOv3()
trainer.setDataDirectory(data_directory="Radiator")
trainer.setTrainConfig(object_names_array=["Radiator"], batch_size=4, num_experiments=200, train_from_pretrained_model="pretrained-yolov3.h5")
In the above,when training for detecting multiple objects,
#set object_names_array=["object1", "object2", "object3",..."objectz"]
trainer.trainModel()
Also, I checked the data all the images n annotations were present.
Each time when I run the code again, it shows same error but with different no. of image
For e.g.
ValueError: 'Radiator_160' is not in list
ValueError: 'Radiator_26' is not in list
ValueError: 'Radiator_59' is not in list  .. etc.
I am not sure where I am wrong.
I checked the annotations as well.
Is there any specific format for name of images?

Please help with this.

Do anything else has to be added?
I created the dataset for Radiator:
Radiator

train

annotations
-Radiator_1.xml
-Radiator_2.xml
....Radiator_170.xml
-images
-Radiator_1.jpg
-Radiator_2.jpg
....Radiator_170.jpg

validation

annotations
-Radiator_171.xml
-Radiator_172.xml
....Radiator_170.xml
-images
-Radiator_171.jpg
-Radiator_172.jpg
....Radiator_200.jpg
Downloaded the pretrained yolo model and ran the above code

Kindly visit #303 and review the last comments. You might get insights on resolving your issue from there.

I changed the name tag and its working now.
Just one more thing I want to use resnet model instead of yolo.
would it be possible.
I tried using set model type as resnet () but it did not worked.

I am happy to know you have resolved this issue. ImageAI currently support only YOLOv3. Support for training with RetinaNet will be added in future versions.


[MyCollection] Rough in some more UI

This starts adding MVP ui around the various pickers on the create an artwork screen. Nothing is polished in any way but this gets things in there.
Note that I've opened an RFC around the use of react-native-image-picker: https://github.com/artsy/eigen/issues/3473.

#trivial
#skip_new_tests

Warnings

:warning:
Missing Test Files:

src/lib/Scenes/Consignments/v2/Screens/MyCollectionAddArtwork/Components/__tests__/MediumPicker-tests.tsx
src/lib/Scenes/Consignments/v2/Screens/MyCollectionAddArtwork/Components/__tests__/PhotoPicker-tests.tsx
src/lib/Scenes/Consignments/v2/Screens/MyCollectionAddArtwork/Components/__tests__/SizePicker-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS

Going to extract out the photo picker aspect as that RFC is in limbo... one sec.


Better .travis.yml

test on other ruby versions
test with mysql and postgres. right now we test with in-memory sqlite

part 1 done. part 2 will be done as part of #46


Check template should catch only TCLResult

Currently check template catches any expression so when importing OpenCL, you cannot use check anywhere else in your code.
This makes opencl incompatible with NimCuda for example: https://github.com/unicredit/nimcuda/blob/5ca0cbcd962050e522f7ad01de0a8ba585a61eaa/nimcuda/nimcuda.nim#L47-L81
Or CLBlast: https://github.com/numforge/nim-clblast/blob/master/src/clblast.nim#L12-L15.
I suppose several other C library wrappers have a template called "check" to test the return status/error code.

Actually it also breaks the unittest module
Error: ambiguous call; both opencl.check(a: expr): typed[declared in .../.nimble/pkgs/opencl-1.0/opencl.nim(191, 9)] and unittest.check(conditions: untyped)[declared in lib/pure/unittest.nim(563, 6)] match for: (TClResult)


Guidance to replace schema generated VTL resolvers with JS resolvers

Amplify CLI Version
12.10.1
Question
To enhance the flexibility of Appsync, we are investigating the migration from VTL resolvers to JS resolvers. Despite consulting the documentation for instructions, the implementation process remains unclear. My current approach involves using the custom resource feature of the Amplify CLI to substitute VTL resolvers with JS resolvers. This, however, has presented several challenges.
A primary concern is the limitation imposed by the Code.fromInline function, which constrains the scalability of our JS resolvers. An alternative method that avoids the necessity of embedding extensive code within multiline strings in a single file would be helpful.
The main objective is to replace auto-generated VTL resolvers with JS ones. My initial strategy was to use the CDK stack to create new JS resolvers, naming them identically to their VTL counterparts. This resulted in an error indicating that a resolver for the query/mutation already exists. Altering the approach by preventing Amplify from generating resolvers using "@model(queries: { get: null }, mutations: null, subscriptions: null)" and then adding the resolver led to an error stating "The query/mutation does not exist."
A concise guide on employing JS resolvers within Amplify, particularly for replacing generated ones, would be highly valuable. Additionally, I am interested in understanding Amplify's future plans regarding these updates. The Appsync documentation now primarily supports the APPSYNC_JS runtime, as stated on their page. I am eager to know when this will be integrated into Amplify. Thank you for your attention to this matter.

Hey, 👋 thanks for raising this! I'm going to transfer this over to our API repository for better assistance 🙂


Fixed #14 build error resulting due to change in influxdb

The NewValue function was changed to accept
int64 instead of time.Time in this commit
commit 8d70d65a826222800b13cb6be5fc11913976bdb7
Hence, change is needed in migration.go

Come on influxdb: it's been a whole four months:  we need this patch merged....


stairwell.com false positive

Hi,
you've got our corporate domain (stairwell.com) listed.  maybe because we're an information security company and writing about malware indicators sometimes?  Not sure - but either way, appreciate it if you can remove.  Thanks!

I've removed this domain from the blocklist, I apologize for the false positive. Thanks for reporting.


Adding a new created blob container to Quick Access shows as broken

Storage Explorer Version: 1.8.1_20190523.2
Platform/OS: Linux Ubuntu/macOS High Sierra/Windows 10
Architecture: ia32/x64
Regression From: 1.8.1 release
Steps to reproduce:

Expand one normal account -> Create a new blob container.
Try to add the new created blob container to Quick Access.
Check the status of added blob container in Quick Access.

Expect Experience:
The blob container shows well after adding it to Quick Access.
Actual Experience:
The blob container shows as broken after adding it to Quick Access.
More info:

This issue doesn't reproduce for file shares/queues/tables.
This issue also reproduces when adding one renamed blob container to Quick Access.

Fix checked into master


RetryException doesn't have Retry-After value

according to Error handling section of documentation, HTTP error 429 provides retry time in Retry-After header
RetryException doesn't have this information, so when API call throws RetryException there are no way to know when its legal to do the next call.

In v2.3.6 we introduced a subclass of RetryException called RateLimtException which contains a RetryAfter field. Client can do proper backoff based on this value. For general RetryExcepton which mostly caused by transient failure, the sdk already does a certain number of retries and it's up to the client to decide whether continue retrying or not.


Idea: Replace dependent: :delete/dependent: :nullify with FK on_delete:/on_update:

Hey @djezzzl ! :wave:
Cascade deletion is sometimes a very expensive operation.
In case the association is defined as dependent: :delete, it is possible to rely on the database to perform the deletion instead.
See :on_delete/:on_update.
Disclaimer: I clearly understand that dependent: :destroy should remain ignored, as destroy also runs callbacks.
Would it make sense to detect this and suggest changing the FK in such a way that the DB handles cascade deletion/nullification?
Sorry for not being verbose. I'd love to validate the idea with you first, and elaborate if you support it.
Originally posted in the wrong repo: https://github.com/toptal/database_validations/issues/59

I'm up to send a PR with a new checker that would detect dependent: :delete/dependent: :delete_all/dependent: :nullify and suggest changing FKs to use on_delete: :cascade/on_delete: :nullify instead. Is this a good starting point? Should it be an AssociationChecker subclass?
Optional: recommend on_delete: :restrict instead of dependent: :restrict_with_exception.

Rails skips validations on associated records

https://guides.rubyonrails.org/association_basics.html#options-for-has-many-validate

If you set the :validate option to false, then associated objects will not be validated whenever you save this object. By default, this is true: associated objects will be validated when this object is saved.

I meant this. I guess it's only for the newly created associated records, though. Wondering how validate: false works:
class School < ActiveRecord::Base
  has_many :students, validate: false
end

class Student < ActiveRecord::Base
  validate :name, presence: true
end

school.students << Student.new
school.save!

Won't that new student's name be validated for presence? 😱
Additional reason to consider on_delete FK option:

Note that :dependent is implemented using Rails' callback system, which works by processing callbacks in order. Therefore, other callbacks declared either before or after the :dependent option can affect what it does.

Related: https://rails.rubystyle.guide/#callbacks-order https://docs.rubocop.org/rubocop-rails/cops_rails.html#railsactiverecordcallbacksorder (but it does not consider dependent: option.
Related: https://rails.rubystyle.guide/#before_destroy

dependent => :destroy deletes children before "before_destroy" is executed

Choosing between delete and destroy: https://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#module-ActiveRecord::Associations::ClassMethods-label-Delete+or+destroy-3F

I'm up to send a PR with a new checker that would detect dependent: :delete/dependent: :delete_all/dependent: :nullify and suggest changing FKs to use on_delete: :cascade/on_delete: :nullify instead. Is this a good starting point?

Please do so :+1:

Should it be an AssociationChecker subclass?

I think yes. We check an association and it's option, if the precheck is true meaning it has dependent: option we:

raise an error if there is no on_delete
raise another error if there is on_delete

Optional: recommend on_delete: :restrict instead of dependent: :restrict_with_exception/dependent: :restrict_with_error.

I can understand why we need on_delete vs delete but this one I'm not yet sure, probably it should improve the consistency as callback is avoidable. Let's have it too then :+1: Could be part of the same checker.

Won't that new student's name be validated for presence? scream

Seems like that's the case, didn't check though.

Hi @pirj,
I have finally released this in 1.5.0. Please try it out and let me know if it works for you.
Thank you for your suggestions! Keep it going, please!


Path

Where does token grabber build

it automatically opens the folder. But the location is .../Blank Grabber/env/Scripts/dist/


Add docker compose deployment

What this PR does / why we need it:
Enable developers deploy control plane on local single machine using docker-compose
Which issue(s) this PR fixes:
Fixes #4129
Does this PR introduce a user-facing change?:
Support deploy control plane by using docker-compose

LGTM with a nit. Any reason we have to store the docker-compose file in root? It mays confused our users that we only have docker-compose as the installer for pipecd. Do you guys have any idea to resolve that? 🤔

LGTM with a nit. Any reason we have to store the docker-compose file in root? It mays confused our users that we only have docker-compose as the installer for pipecd. Do you guys have any idea to resolve that? 🤔

I think We have two ways

To store the docker-compose file in a directory under project root

docker-compose name the project based on directory name that includes docker-compose.yaml, so we must name the directory a name such as "pipecd-control-plane"

To store the docker-compose file in root and edit Makefile
Many developers use Makefile to deploy control-plane, so they do not take care of the files in root. But we should write rules to deploy via docker-compose in Makefile.

@kevin55156 okay, got it. Let's go with option 2 (since one more dir named pipecd-control-plane makes thing even harder to understand)

Temporary close; we need to reconsider making this part of the main repo or pushing to a different one.

How about posting the example on PipeCD's official blog?

I made PRs.  These are useful for users.

Examples: https://github.com/pipe-cd/pipecd/pull/4822
Blog: https://github.com/pipe-cd/pipecd/pull/4823

@kevin-namba
I posted a blog based on your gread idea and codes, thank you so much!
https://pipecd.dev/blog/2024/03/14/control-plane-on-local-by-docker-compose/


Print AllowTeamsConsumer in the output

The status of the property AllowTeamsConsumer is not listed when you run the cmdlet.
[Enter feedback here]

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

ID: b929fe13-9ac8-ce73-f331-5c4ce1d08640
Version Independent ID: 4042cf42-2b7b-f3f1-34a6-fc615c78229b
Content: Get-CsTenantFederationConfiguration (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Get-CsTenantFederationConfiguration.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

Both AllowTeamsConsumer and AlowTeamsConsumerInbound show for me in version 2.6.0

@BJGGut3, thank you for your feedback and the confirmation that you can see these parameters now.

@dariomws Thank you very much for the contribution and sharing this explanation. @ParaGlideRocks Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


3 libs needs to update

1.simpleini =>needs to update ports/simpleini/portfile.cmake
file(COPY ${SOURCE_PATH}/simpleini.h DESTINATION ${CURRENT_PACKAGES_DIR}/include)
file(COPY ${SOURCE_PATH}/SimpleIni.h DESTINATION ${CURRENT_PACKAGES_DIR}/include)
as linux can't detect simpleini.h
2.cpp-redis needs to update from  https://github.com/Cylix/cpp_redis to https://github.com/cpp-redis/cpp_redis
3.tacopie needs to update from https://github.com/Cylix/tacopie to https://github.com/cpp-redis/tacopie
thanks.

@whidbey, thanks for reporting this issue, I have summit a update.

The fix has merged.

thanks,but still not work
this is the error log ,fullly pull the lastest version;
[root@localhost vcpkg]# ./vcpkg install simpleini
The following packages will be built and installed:
simpleini[core]:x64-linux
Starting package 1/1: simpleini:x64-linux
Building package simpleini[core]:x64-linux...
-- Using cached /root/vcpkg/downloads/brofield-simpleini-fe082fa81f4a55ddceb55056622136be616b3c6f.tar.gz
-- Using source at /root/vcpkg/buildtrees/simpleini/src/be616b3c6f-d6d01d7f61
CMake Error at ports/simpleini/portfile.cmake:12 (file):
file COPY cannot find
"/root/vcpkg/buildtrees/simpleini/src/be616b3c6f-d6d01d7f61/Simpleini.h".
Call Stack (most recent call first):
scripts/ports.cmake:94 (include)

cpp-redis is ok
simpleini is fault.


.NET 9 preparation

Move suppressions to .ruleset file.
Set NuGetAuditMode=direct.

Cherry-picked from #1358.

The discussion for this CVE indicates that this version is unaffected
https://github.com/advisories/GHSA-hh2w-p6rv-4g7w
There's a suggestion to add NoWarn="NU1903" to the package reference to suppress this, which sounds reasonable, at least until the CVE is updated/fixed

Yeah, I just haven't got around to circling back and fixing this PR yet as I've had a bunch broken this preview.


Add interface for creating tables in Python

Should be able to do db.new_table(column_names, rows). Motivating use case is combining bounding boxes from multiple Scanner runs into a single set, and then running a tracker.

Added this in some commit I can't remember.


Check Home Assistant configuration has version 3.3.0 but in log it says 3.2.9

When you start the add-on in the log it says: starting version 3.2.9
But installed version of this add-on is 3.3.0
Small thing, but maybe worth mentioning.

Hi there @formadelfo!
Could you please share the name of the add-on in question?
Also, please share the logs as shown.

It is the 'Check Home Assistant configuration' add-on.
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] udev.sh: executing...
starting version 3.2.9
[11:39:22] INFO: Update udev information
[cont-init.d] udev.sh: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[11:39:22] INFO: Setup udev devices
[11:39:23] INFO: Don't worry, this temporary installation is not overwriting your current one.
[11:39:23] INFO: Installing Home Assistant: latest...
[11:39:23] INFO: Please be patient, this might take a few minutes...
[11:41:06] INFO: Installed Home Assistant 0.110.2
[11:41:06] INFO: Making a copy of your configuration for checking...
[11:41:49] INFO: Checking your configuration against this version...
[11:45:17] INFO: Configuration check finished - no error found! :)
s6-svscanctl: fatal: unable to control /var/run/s6/services: supervisor not listening
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.

That is another program starting there (udev), which is not related to the add-on version. I see the confusion, however, there is not much we can do about it.

Ah ok, LOL. Sorry. Nevermind then.


Expose GetFileResponse to stream directly to file

This exposes GetFileResponse as an advanced method so there's a chance to directly access the response.Content before it's read.

Hi @ngbrown ,
I'll merge it and release a new version.
Thanks for the PR.

A version 2.5.0 is available in NuGet.

@skazantsev thanks for your library and releasing a new version.


could'nt recieve integers at master from slave

well in examples the character array works pretty good..., but when modified to send integer there is no serial print ... gets struck at some unknown point

Hi  @DeadSHoT555
This place is for issue with BoardManager, so do not see any link with board manager.
I guess you have an issue with the core, so please submit issue here
https://github.com/stm32duino/Arduino_Core_STM32/issues
Also be more precise about your issue (core version, sketch example,...), you can follow the issue template to help us to help you.


iOs duplicate group

In iOs 9.3 the CardDav synchronization creates a duplicate group, which is of one group only. e.g.:

Test Group 1
Test Group 2
Out of place group (duplicate)
Out of place group (duplicate)
Out of place group (duplicate)
Out of place group (duplicate)
Out of place group (duplicate)
Out of place group (duplicate)
Out of place group (duplicate)
Test Group 3
Test Group 4
Test Group 5
Test Group 6

These duplicate groups are all empty.
I have no idea where they're coming from, and this only occurs on an iPhone. Tried to reproduce the problem on an iPad with iOs 9.2 (13C75) but this would sync just fine.
The weird thing also is that the user does not have access to the duplicate group, however it still shows up but is completely empty.

Just updated the iPad to iOs 9.3.2 and the problem does not exist on the iPad, just on the iPhone.

Have you looked at the actual data?
iOS (unlike macOS) supports multiple addressbooks in one CardDAV account, but displays them in the same user interface where it also displays groups, which sometimes ends up showing quite confusing results.

I'm closing this ticket because it's been a while since the last comment.
If this is indeed still an issue, feel free to comment here so we can continue discussing.


Nextcloud sync

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
I've started to migrate across from Evernote to a private cloud hosted on nextcloud/qownnotes. Problem is, the migration is happening piecemeal and I'm worried that some of my changes might not be carried across.
Describe the solution you'd like

Consistent markdown conversion between tusk/nextcloud/qownnotes
ability to sync Evernote/nextcloud/qownnotes (with an ability to reconcile differences in files via name / update/ accept differences between versions of the same file)

A possibility for having the notes taken to sync on my nextcloud server. That would be huge. I am migrating away from Evernote, because of data ownership / privacy concerns - but the functionality is missing. Since this is already an open source client, it would make sense to integrate Nextcloud self hosting :) - would be a unique selling point, haven both - if you look at the market.
Maybe you could make a campaign, if reaching a specific amount of github sponsors for this solely purpose.


[fix](Variant Type) Add sparse columns meta to fix compaction

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Add unit-testing

Here's a first pass on adding a unit test for populate_instructions.
Not certain about some details but thought this would be a good place to start.
Closes #12

Nice! Can we also run the tests as part of the github workflow file ? https://github.com/bergercookie/asm-lsp/blob/master/.github/workflows/lint_and_test.yml#L28
Also, not related to the code of the PR but just noticed you merged the master into this branch. Could we instead rebase feature branches like this on top of latest master instead of merging master into them? The latter often leads to noise when viewing the git tree of commits

@bergercookie I'll try looking into adding the tests to the workflow file now, although I'm not super familiar with github workflows.
Sorry about the merge! I'll make sure to rebase in the future.

Sorry about the merge! I'll make sure to rebase in the future.

No worries, you can always cherry-pick pick the commits of this branch on top of latest master and force-push to this branch.
I can do it for you in this case if you prefer

That would be great, thank you :)

That would be great, thank you :)

Just did.
Actually, no need to cherry-pick. git pull --rebase origin master did the trick for me and then I squashed the commits together with git rebase -i HEAD^^ :)


Remove leading slash from Retrofit interface.

Otherwise since the base URL has to end with a slash, but the retrofit interface has a leading slash, URLs result in https://baseurl:9000/my/path//api/v0... which of course strips out /my/path.

See https://github.com/square/retrofit/issues/907
Ran into this in some internal testing.

@aash good to merge?

Not quite yet, need confirmation this is being used properly for uploads and don't have it yet.  Partially because resource-staging-server doesn't have request logs: https://github.com/apache-spark-on-k8s/spark/issues/311

rerun integration test please

We confirmed on an internal product that the upload is succeeding, but not yet the download.

And now have confirmed that the download succeeds as well -- merging


[Issue]The frequency of P cluster is not consistent with the value of powermetrics

The key for frequency and residency is 'PCPU' or 'ECPU'. But I tried 'PCPM' then the frequency of P cluster is consistent with the value of powermetrics. 'PCPU' key seems not suitale for frequency.

@wtndcs I don't recall powermetrics using the _CPM keys when I revere engineered it although I could be wrong. Maybe the discrepancy between my tool and powermetrics is due to my own error in the formatting of the data from IOReport? I had found an error when doing refactoring for the latest commit I just did, but then again, I had started that refactor months ago and just picked it up again, so I don't really know. Kind of disorganized with this thing cause I don't touch it very often anymore.
Also the keys for per core distributions are some kind of _CPUs and no _CPMs and it seems odd for _CPMs to be just for clusters distributions. Especially then on Ultra SoCs, the keys just get super funky.  But idk, could be something weird. 'Tis Apple. But maybe try the latest commit, maybe my changes fixed it for you..

Closing this issue as the values seem to align with powermetrics as of my latest commit


Migrate to pyproject.toml

Codecov Report

Merging #111 (4fa77c1) into main (2b89fbf) will decrease coverage by 3.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #111      +/-   ##
==========================================
- Coverage   90.59%   87.42%   -3.18%
==========================================
  Files           7        7
  Lines         851      851
  Branches        0      117     +117
==========================================
- Hits          771      744      -27
  Misses         80       80
- Partials        0       27      +27

Flag
Coverage Δ

unittests
87.42% <ø> (-3.18%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

gcp_storage_emulator/__main__.py
69.84% <0.00%> (-9.53%)
:arrow_down:

gcp_storage_emulator/handlers/buckets.py
87.75% <0.00%> (-4.09%)
:arrow_down:

gcp_storage_emulator/server.py
85.22% <0.00%> (-3.04%)
:arrow_down:

gcp_storage_emulator/handlers/objects.py
88.40% <0.00%> (-2.90%)
:arrow_down:

gcp_storage_emulator/storage.py
94.14% <0.00%> (-1.60%)
:arrow_down:

Continue to review full report at Codecov.

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


Clean up code base, add pre-commit, add dependencies

Addresses issues raised in #2, but does not close it.
Ping @agstephens @sol1105
Changes

Adds a pre-commit configuration for automated code linting
NetCDF grid downloader has been cleaned up for readability (Note: Cleaning performed with the aid of ChatGPT)
Base Python has been changed to Python3.9 (follows from NEP-0029)

Work needed

There are a few instances where CLISOPS is called. If this is to become a dependency of CLISOPS, we need to be implementing the functions required here to prevent a circular dependency loop.
The reference grid files are very large! If we only require the grids themselves, it's preferable that the variable values as well as the time dimensions be removed from all files. At the current size, this package cannot be hosted with its data on PyPI.

Proposal
Would this library better serve as a module of CLISOPS instead? Are there instances where the tools here would be used independently of CLISOPS? Given the tight focus and coupling with CLISOPS, this seems more like a specialized helper library. If so, we can very easily push this directly into CLISOPS.

@sol1105
Your comments sound good. I'll merge this now and leave you to work on the comments raised here.
All the best!


Comments in controller not parsing

Hello,
i am using swaggo with go chi. The general Api comments are generating specs correctly. However, the operation comments made in my controller endpoints are not parsing. Please help.

//All the routes exposed in the User Controller.
func Routes() *chi.Mux {
	router := chi.NewRouter()

	// GetUser godoc
	// @Summary Get a user.
	// @Description The user specified by the user id provided is retrieved.
	// @Tags user
	// @Accept  json
	// @Produce  json
	// @param user_id path string true "User Id"
	// @Success 200 {object} model.User
	// @Failure 400 {object} httputil.HTTPError
	// @Failure 401 {object} httputil.HTTPError
	// @Failure 404 {object} httputil.HTTPError
	// @Failure 500 {object} httputil.HTTPError
	// @Router /user/{userID} [get]
	router.Get("/user/{userID}", GetUser)

Here is a snippet

Solved. The error was because i was placing my comments on the route instead of on the handler. :)


Document week numbers

Fixes #283

This change is

 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.

Review status: 0 of 1 files reviewed at latest revision, 3 unresolved discussions, some commit checks broke.

docs/vaadin-date-picker-customization.adoc, line 8 at r1 (raw file):

[[vaadin-date-picker.week-customization]]
= Setting Week Numbers

Setting sounds funny as the user doesn't have to set/calculate the week numbers, but just set them to show. I'd replace setting or even remove it.

docs/vaadin-date-picker-customization.adoc, line 12 at r1 (raw file):

It can be enabled by setting the [propertyname]#showWeekNumbers# to `true`, and the [propertyname]#i18n.firstDayOfWeek# to `1`.

Weeks are numerated based on the ISO-8601 standard, so as the first week in the year is the week containing the day 4th of January.

I'd simplify: ..., so the first week of the year contains 4th of January

docs/vaadin-date-picker-customization.adoc, line 17 at r1 (raw file):

[source,html]
----
<vaadin-date-picker show-week-numbers></vaadin-date-picker-light>

rename vaadin-date-picker-light

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 3 unresolved discussions.

docs/vaadin-date-picker-customization.adoc, line 8 at r1 (raw file):
Previously, samiheikki (Sami Suo-Heikki) wrote…
> `Setting` sounds funny as the user doesn't have to set/calculate the week numbers, but just set them to show. I'd replace setting or even remove it.

Done.

docs/vaadin-date-picker-customization.adoc, line 12 at r1 (raw file):
Previously, samiheikki (Sami Suo-Heikki) wrote…
> I'd simplify: `..., so the first week of the year contains 4th of January`

Done.

docs/vaadin-date-picker-customization.adoc, line 17 at r1 (raw file):
Previously, samiheikki (Sami Suo-Heikki) wrote…
> rename vaadin-date-picker-light

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

docs/vaadin-date-picker-customization.adoc, line 2 at r2 (raw file):

---
title: Input Customization

Started to think is week numbers doc in a wrong file or is the title for it wrong. Showing week number doesn't fit into Input Customization. But then again it did not fit to any else better. I'd recommend either renaming this title or creating a new file for week numbers.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

docs/vaadin-date-picker-customization.adoc, line 2 at r2 (raw file):
Previously, samiheikki (Sami Suo-Heikki) wrote…
> Started to think is week numbers doc in a wrong file or is the title for it wrong. Showing week number doesn't fit into `Input Customization`. But then again it did not fit to any else better. I'd recommend either renaming this title or creating a new file for week numbers.

@jouni 's comment in #288 suggest to use this file (or i18n) instead of creating a new menu entry.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

docs/vaadin-date-picker-customization.adoc, line 2 at r2 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
> @jouni 's comment in #288 suggest to use this file (or i18n) instead of creating a new menu entry.

But should this title then be replaced to just `Customization`?

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Detect image type based on content instead of file extension

Description
Inspired by https://github.com/typst/typst/issues/3079, I find that currently, Typst tries to determine the image type by file extension first and then falls back to content-based guessing.
https://github.com/typst/typst/blob/e215f22965760bcc5df3204243597825e99b3fe7/crates/typst/src/visualize/image/mod.rs#L165-L176
However, many other libraries detect the image type based on the content by default, e.g., Pillow in Python. ref

The open() function identifies files from their contents, not their names, but the save() method looks at the name to determine which format to use, unless the format is given explicitly.

This is also a code of UNIX-like systems, i.e., do not rely on the file extension. ref ; file command
Should we do it the same way?
BTW, I'm not sure if it's okay to post here or should be put into Discussion. Please point it out if I miss something.
Use Case

When extension is not correct https://github.com/typst/typst/issues/3079;
Keep consistency with other libraries like Pillow in Python;
Do something similar to the file command in Unix-like systems where file extension is not always presented.

I think a more useful request would be to allow overriding whatever the built-in detection process does, e.g. image(..., format: "png").

Alternatively, one could add content-based detection as a fallback if the extension-based detection leads to an error.

By the way, there is the imagesize crate which seems to be relatively lightweight and has a method to detect the type of an image. Might be a bit more reliable than our current approach of looking at "magic bytes", although I haven't tested it.

By the way, there is the imagesize crate which seems to be relatively lightweight and has a method to detect the type of an image. Might be a bit more reliable than our current approach of looking at "magic bytes", although I haven't tested it.

Right now, we use the image crate. From a quick look imagesize also seems to look at the first few bytes, so what's the difference?

True, nevermind, I looked at the size function instead of the matches function, which also seems to mainly look at the first few bytes. My bad.

Currently, Typst does the auto-detection if it doesn't know the extension. Upon further thought that seems like a backwards incompatibility hazard. If I have a .png JPEG file, it would detect as PNG and fail to decode, but I have a .webp JPEG file, Typst wouldn't know the extension and decode as JPEG. This would then break if we add WEBP support in the future. Only doing content-based detection when no file extension is present would be better in my opinion.
Adding a hint when the extension specifies one image format, but detection determines it as another seems fine, though not useful very often. I think expanding it to any file type is overkill, and would probably also require a new dependency.

fwiw, https://github.com/image-rs/image/issues/2368 is looking to add sniffing to image


Abstraction for ASN.1 encoding subjectPublicKey and privateKey

Goal: add an abstraction and implementation to support ASN.1 encoding of P8/SPKI as specified by Internet-drafts:
https://datatracker.ietf.org/doc/draft-ietf-lamps-dilithium-certificates/
https://datatracker.ietf.org/doc/draft-uni-qsckeys-dilithium/00/
https://datatracker.ietf.org/doc/draft-uni-qsckeys-falcon/00/
https://datatracker.ietf.org/doc/draft-uni-qsckeys-kyber/00/
https://datatracker.ietf.org/doc/draft-uni-qsckeys-sphincsplus/00/
API would contain:

encode: take the "raw" keys used by libOQS internally, return the specified encoding.
decode: take the encoding according to a specification, return the "raw" keys used by libOQS internally.

API should allow to encode/decode private keys with/without optional public key component (#88).
The drafts get their own encode/decode functions and ASN.1 wrapping / encoding / TLV will be hidden behind them. Encoding will be selected by ENV variable. It would make interop to different IETF-drafts and IETF-draft-versions easier.

The code to be integrated is available here:
https://github.com/Quantum-Safe-Collaboration/qsc-key-rfc/tree/master/qsc-key-encoder
It supports Dilithium, Falcon, SPHINCS+ and Kyber and allows several cases:

encode full PrivateKeyInfo / SubjectPublicKeyInfo (PKCS#8 / SPKI)
encode only privateKey / subjectPublicKey parts of PKCS#8 / SPKI
encode private key with optional public key
encode private key without optional public key
encode private key and/or public key
decode private key and/or public key (including decode private key with optional public key part to raw public+private key)

@bhess: Do you intend to integrate the code from the spec repo above here or what's the next step to come to closure of this issue?

Yes, my intention is to integrate this code which implements the API described above (before the next IETF). Either linking to the library from the repo or replicating the code here. Any thoughts?

I was thinking about using cmake's ExternalProject function. It allows to fetch, build and patch (if needed) and therefore include external projects as part of a cmake build. This will be similar to a "copy_from_upstream" mechanism but better integrates with cmake.


support json and live output options

consistent with https://github.com/spiermar/node-stack-convert

This is now supported. I merged the PR #5


Refactoring of TigerRAG Package

Refactoring of TigerRAG
Originally meant to do enhancement of issue 5 (#5), hence the branch name. However, turned out to be a refactoring (sorry).
Major Changes:

Required dependencies are in setup.py instead of a requirements.txt file. Like this they get installed when the tigerrag package gets installed.  Like this the demos can run as is. The idea is that the tigerrag package dependencies should be self contained. If I wanted to use this package I would expect it to work like all other packages (not asking me to install another requirements.txt). Might make sense to go modern and use a pyproject.toml.
Refactoring of basic functionality into the tigerrag package. The current package doesn't really do anything. The demos contain the full code which seems weird. Now, the demos don't need to import any other third-party dependencies. All comes package within tigerrag. Yay!
Proper typing within the packages. I know pedantic but you will thank me later... I hope
I took the liberty to create a structure within the package. May not be what you intended...
Also took the liberty to add a standard .gitignore file. Can't live without one to be honest. It's like a asking people to take of their shoes when entering the house. Keeps things cleaner
Updated README.md files. Should be simpler to set stuff up. Demos worked (did not test langChain demo, as this one failed for me anyways)
The only thing I regret about this pull request is that I added more code than I deleted

This was a "bored sunday" endeavour. Cherry-pick what you like or discard. Up to you. Good luck

Hey @habichta ,
First and foremost, a huge thank you for dedicating your "bored Sunday" to enhancing our project. Your enthusiasm and proactive approach is truly commendable!

Re-structuring the packages: Absolutely stellar work here. This was on our roadmap, and you've laid a great foundation for us. It's precisely the direction we wanted to head in.
Dependencies in setup.py: Transitioning from requirements.txt to having dependencies in setup.py is a thoughtful change. It indeed makes the package more self-contained and user-friendly. Your suggestion to possibly use pyproject.toml is noted, and we'll be looking into it.
Typing & Structure: Your attention to detail with proper typing and structuring is much appreciated. It not only makes the code more readable but also more maintainable in the long run.
.gitignore & README updates: Thanks for these housekeeping updates. A standard .gitignore is indeed like asking visitors to take off their shoes—clean and considerate.

Regarding the langChain demo, I've created a new issue to address its challenges. We'll ensure it runs smoothly moving forward.
Lastly, don't worry about adding more code than deleting. Quality over quantity, and you've certainly delivered quality!
Once again, thank you for your invaluable contribution. We're excited to integrate your changes and look forward to any future collaborations!


Update wasm-go.md 可能在windows中出现的一些编译问题

Update wasm-go.md 可能在windows中出现的一些编译问题

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@zhangjiahaol Please sign the cla

@zhangjiahaol Please sign the cla

Ok


V1.1.10 breaks PackageManager

This is inside a jupyter notebook, running python 2.7 on AWS EMR.
Expected behavior
from pixiedust.packageManager import PackageManager
pkg=PackageManager()

Output:
Pixiedust version 1.1.9

Pixiedust database opened successfully

Warning: You are not running the latest version of PixieDust. Current is 1.1.9, Latest is 1.1.10

Please copy and run the following command in a new cell to upgrade: !pip install --user --upgrade pixiedust

Please restart kernel after upgrading.

Actual behavior
from pixiedust.packageManager import PackageManager
pkg=PackageManager()

Output:
Pixiedust version upgraded from 1.1.9 to 1.1.10

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-1-168d39b97cbe> in <module>()
----> 1 from pixiedust.packageManager import PackageManager
      2 pkg=PackageManager()

/usr/local/lib/python2.7/site-packages/pixiedust/packageManager/__init__.py in <module>()
     15 # -------------------------------------------------------------------------------
     16
---> 17 from .packageManager import PackageManager
     18
     19 #shortcut to packageManager

/usr/local/lib/python2.7/site-packages/pixiedust/packageManager/packageManager.py in <module>()
     45 packageStorage = __PackageStorage()
     46
---> 47 class PackageManager(object):
     48     DOWNLOAD_DIR=os.environ.get("PIXIEDUST_HOME", os.path.expanduser('~')) + "/data/libs"
     49

/usr/local/lib/python2.7/site-packages/pixiedust/packageManager/packageManager.py in PackageManager()
     56                 return True
     57         return False
---> 58     IS_PIXIEDUST_JAR_ON_JAVA_CLASS_PATH = check_pixiedust_jar(DOWNLOAD_DIR)
     59
     60     def check_valid(fn):

/usr/local/lib/python2.7/site-packages/pixiedust/packageManager/packageManager.py in check_pixiedust_jar(download_dir)
     52     def check_pixiedust_jar(download_dir):
     53         pixiedust_jar_path = os.path.join(download_dir, "pixiedust.jar")
---> 54         for path in Environment.javaClassPath.split(":"):
     55             if pixiedust_jar_path == path:
     56                 return True

AttributeError: 'NoneType' object has no attribute 'split'

Steps to reproduce the behavior
Install v1.1.10

@brettc PackageManager works only if your notebook is connected to Spark. Can you please verify that this is the case. I also think that we should do the check programmatically and display an error message if that's not the case.
I'll use this issue to provide a fix for that.

Bear with me, I'm new to spark. All I know (and I show above) is that it works perfectly with 1.1.9, and precisely the same thing breaks with 1.1.10. What you see above is cut and pasted directly from the notebooks.

@brettc ok, in a new cell, can you please run the following code and let me know the output
from pixiedust.utils.environment import Environment
print("Has spark:", Environment.hasSpark)
print("JavaClassPath:", Environment.javaClassPath)

thank you

how do you append to javaclasspath? I am getting an error PackageManager is disabled because /home/amit.dingare/data/libs is not on the Java Class Path.

To append to java Classpath on Mac OS. Go to your home directory and open the .bash_profile file, inside write:
export CLASSPATH=$CLASSPATH:/Users/yourname/Documents/projectfolder/
or wherever you installed it.
I assume this process is similar for other unix systems.
On windows, I believe there is a path option somewhere in the control panel where one can append


docs: Add README badge for acceptance tests

Summary:
This PR adds a pass/fail badge to the README for the acceptance tests, similar to the other CI tests that are run. We missed this in the original PR for acceptance tests.
As discussed in https://github.com/MobilityData/gtfs-validator/issues/1094, acceptance tests are currently broken on forked repos, but there is no indication whether they are passing or failing on the README.
Expected behavior:
Show badge for acceptance tests on README:

Thanks @barbeau, we totally missed this one!


PREMIS profile requirements

Requirements related to preservation metadata are not formally defined. A PREMIS profile needs to be created which allows validators to include the verification of these requirements.

Just a remark that there are no profiles and thus no schema provided by the PREMIS EC for describing the use of PREMIS. Our description of use of PREMIs is in https://github.com/DILCISBoard/CITS-PREMIS


#122 - Make error locations in bucklescript tool window clickable

Add a feature to make error link clickable.

Thank you!

Great to have this included.
It also sort of works in the run window (when running npm script eg start etc...)
However at times it will work only on the first warning/error:

At other times it doesn't work on the first error/warning:

Indeed I missed one of the error message use case. It should be fixed by #145 .


Try to Add New Medicine, but telling me company field required, and i'm not seeing the field

@Yemsquare Did you manage to get it work?

Yes

@Yemsquare If all the system is working fine, can you please share the code changes?

@Yemsquare If you do not remember what files you changed, it is ok. Share the project files. Zip it and send to<EMAIL_ADDRESS>Thanks

Okay,
i will try to upload the corrected repo and send to you.

The success and error message is not showing when submitting the New Medicine Form.
Please help.
Thanks.

We also encountered the same error. Could you send the corrected repo, please?

@Yemsquare Would you please instruct me how to fix the issue of "The requested URL was not found on this server. " after I submit login username/password to reach http://localhost/mad-pharma/login/signIn ? Thank you very much.
My PHP environment:
Ubuntu 20.04
$git clone https://github.com/Devnawjesh/mad-pharma
$mv mad-pharma /var/www/html/
$vi /var/www/html/mad-pharma/application/config/database.php
update database information
$php -version
PHP 7.4.3-4ubuntu2.18 (cli) (built: Feb 23 2023 12:43:23) ( NTS )
$sudo apt install php php-cli php-mysql php-zip php-gd php-mbstring php-curl php-xml php-pear php-bcmath php-intl
$ls /var/www/html/mad-pharma
application  assets  composer.json  composer.lock  contributing.md  database  genpharma  index.php  license.txt  package.json  package-lock.json  readme.rst  system  vendor
(1) Launch Firefox web browser at Ubuntu 20.04 to access
http://localhost/mad-pharma/
(2) Input Username/Password, Click LOG IN.
(3) The web browser was redirected to
http://localhost/mad-pharma/login/signIn
Not Found
The requested URL was not found on this server.
Apache/2.4.41 (Ubuntu) Server at localhost Port 80
I tried to use the following solution to fix Codeigniter issue, but the URL issue was NOT fixed.
https://stackoverflow.com/questions/49997750/codeigniter-the-requested-url-was-not-found-on-this-server

hello @winstonhong the issue is that the developer disables the admin page of the system so you need to trace it from the admin page, and also you need to go to the database and change the login details.
Thanks

Hello @Yemsquare.
Thank you very much for your quick response.
(1) mysql -u root -p
Enter password: password
mysql> show databases;
+--------------------+
| Database           |
+--------------------+
| genpharma          |
| information_schema |
| mysql              |
| performance_schema |
| sys                |
+--------------------+
(2)  $vi /var/www/html/mad-pharma/application/config/database.php
https://github.com/Devnawjesh/mad-pharma/blob/master/application/config/database.php
'dsn'	=> '',
'hostname' => 'localhost',
'username' => 'root',
'password' => 'password',
'database' => 'genpharma',
'dbdriver' => 'mysqli',
(3) Input Username/Password (), Click LOG IN.
No matter<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or<EMAIL_ADDRESS>always receives the 404 error log below.
<IP_ADDRESS> - - [16/Mar/2023:12:53:19 -0700] "POST /mad-pharma/login/signIn HTTP/1.1" 404 488 "http://localhost/mad-pharma/" "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/110.0"
Questions:
(1) https://github.com/Devnawjesh/mad-pharma/blob/master/application/config/config.php
Do I need to change
$config['base_url'] .= preg_replace('@/+$@','',dirname($_SERVER['SCRIPT_NAME'])).'/';
?
(2) https://github.com/Devnawjesh/mad-pharma/blob/master/application/controllers/Login.php
// Checking login credential for admin
$query =$this->login_model->getUser($credential);
if($row->em_role !='SALESMAN')
<EMAIL_ADDRESS>or<EMAIL_ADDRESS>or<EMAIL_ADDRESS>!='SALESMAN'
return 'success';
It seems that  https://github.com/Devnawjesh/mad-pharma/blob/master/system/core/CodeIgniter.php
creates the routing issue.

Hello @Yemsquare
Thank you very much for your quick response.
I have fixed the issue of "The requested URL was not found on this server. ".
The issue was caused by Apache2 server module rewrite.
I attach the Add Medicine GUI for your reference.
PHP version of this project is PHP 7.2, The following link provides the guidance on how to downgrade to 7.2 (see the answer of Ayman).
https://stackoverflow.com/questions/59781992/how-can-i-downgrade-or-use-php-7-2-without-uninstalling-php-7-4-is-it-possible


Check advisories missed by freshmaker

Check if the RHSA in SHIPPED state  were catched by freshmaker.
JIRA: https://projects.engineering.redhat.com/browse/CLOUDWF-555

@mprahl , @gnaponie , PTAL

@mulaievaRH could you squash the commits and add the card reference to the commit message?

Tested locally and it looks good. :+1: @mprahl do you want to review?
BTW I saw there's no testing for the whole cli, I've filed a tech debt card to fix that in the future -- even though I'm not sure we'll ever have the capacity to pick it up. :)

@mprahl , all is fixed. Review please


Wrong title formatting

Thanks for bringing this to attention; it seems that someone performed the same changes in our internal repo quite a while ago.
I bet that this pull request had something to do with it. 😄


Geoms include a table in their parameters section

We should be able to do this in markdown, though the rendered style of docstrings may need to be tweaked.
Edit: it seems like if I add html for a table inside a row, it gets stripped out by quarto? We should check on this behavior.
TODO:

Can quarto include an html table inside a markdown table?
Is there a markdown table format that would led us do a table in a table?
Worst case, should we just just generate the full html for the parameters, including the nested table?

We now generate the table within a table using html. (The behavior can be tweaked in _renderer.py)


Request: Display Limited Resources via Test Budget Object

What problem does your feature solve?
Soroban includes a handful of resources that are limited but cannot be checked via the test environment's Budget object.
These missing fields are:

Ledger Entry read/writes
Read/write bytes

Currently, the budget object only contains the CPU and Memory limits plus their breakdown:
=======================================================
Cpu limit: 100000000; used: 32025734
Mem limit: 41943040; used: 13713432
=======================================================
CostType                 cpu_insns      mem_bytes
WasmInsnExec             144748         0
MemAlloc                 1745559        12976302
MemCpy                   2914660        0
MemCmp                   21318406       0
DispatchHostFunction     20150          0
VisitObject              11163          0
ValSer                   4632           5464
ValDeser                 0              0
ComputeSha256Hash        18870          0
ComputeEd25519PubKey     0              0
VerifyEd25519Sig         0              0
VmInstantiation          5845598        731652
VmCachedInstantiation    0              0
InvokeVmFunction         1948           14
ComputeKeccak256Hash     0              0
ComputeEcdsaSecp256k1Sig 0              0
RecoverEcdsaSecp256k1Key 0              0
Int256AddSub             0              0
Int256Mul                0              0
Int256Div                0              0
Int256Pow                0              0
Int256Shift              0              0
ChaCha20DrawBytes        0              0
=======================================================

What would you like to see?
The ability to check Ledger entry read/write count, and read/write bytes after a contract call in a unit / integration test.
What alternatives are there?
Test against a standalone container, which can be significantly more complicated.

The budget that is printed out is the budget from the Soroban Env.
@graydon @dmkozh @sisuresh Are the stats above available on an object of the Host that we could use in the SDK?

I don't think the host tracks stats before and after contract invocation. Others can comment.
There is a related issue of adding resources for contract tests: https://github.com/stellar/rs-soroban-sdk/issues/1113

Thanks @jayz22 - looks like this is a duplicate of #1113.
Closing this issue in favor of the previous one.


AEs explode on migration to 3.0.1

AEs explode on migration to 3.0.1
https://discord.com/channels/170995199584108546/670336046164213761/1204435692977918092

Can you post the Actor's UUID (right-click the book icon next to the character's name in the title bar), as well as the origin values for each of the Comprehend Languages and Disguise Self effects?

Actor.XbezijMrVTRuOmFc
[
    {
        "name": "Comprehend Languages",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.ozjZadHlZOnEWP4P"
    },
    {
        "name": "Lamp",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.YSqIuD4NIO3jRJmt"
    },
    {
        "name": "Comprehend Languages",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.ozjZadHlZOnEWP4P"
    },
    {
        "name": "Detect Magic",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.YbmATedSdUh1PxqQ"
    },
    {
        "name": "Disguise Self",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.p27xtZCilZ12ixm5"
    },
    {
        "name": "Expeditious Retreat",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.y1eNMIMfG5vx4C5C"
    },
    {
        "name": "Jump",
        "origin": "Actor.XbezijMrVTRuOmFc.Item.LPhQ9s5WsYXRrR1j"
    }
]```

I wonder if this is an issue with the times on the effects. The migration script that detects duplicates does a diff between the effect on the item and on the actor, so if the times have changed then it might result in them not being identified as duplicates.

Midi is about to release 11.4.1 which will have dnd5e 3.0 compatibility.  You are about to get an influx of users who will migrate and have an absolutely large amount of duplicate active effects caused by this bug.  This happens 100% of the time in a world migrated.  In Midiqol there are significantly way more active effects than a core world.  While I understand it a module, this is a migration script bug that is duplicating these ae's.  Is there an eta on fixing this?  You are about to have a pretty hefty chunk of users about to update to 3.0 and its going to basically break worlds because it is really tedious to go through 6000 monsters/actors to clear duplicate ae's.

when you say it does a diff, does that including flags? For non-transfer/passive effects midi/dae will create flags on creation of the effect on the actor, so a straight diff won't work.
It probably makes sense for me to write module specific de-duplication rather than you having to guess what changes if any midi/dae have done.

Well, I am not using MidiQOL or DAE. There is some chance that there is a DDB cause, but I think some AEs affected are hand made.

Surprised that script didn't fail for a lot of users when origin can be null.

It looks just at changes, my guy.

This is the 2.4.1 actor before migration:

This is the 3.0.2 post migration actor(disregard the NaN's that bug is squashed by tposney already)

There are no errors in the migration process at all, I ran the migration in safe configuration mode.

So one potential issue is that the de-duplication migration is only run on actors that haven't yet been modified in 3.0.0, so it checks the _stats.systemVersion and doesn't run anything unless it is older than that

I would have to do further investigation, about maybe having old modules stuff on the AEs, but I am migrating 2.4.latest to 3.0.2 and it still duplicates.

Looks like there is an issue with how deleting effects combines with the persistSourceMigration flag.

PR is up, feel free to test it to see if it fixes migration issues: https://github.com/foundryvtt/dnd5e/pull/3061


How autoplay a playlist?

Wavesurfer.js version(s):
Browser and operating system version(s):
Chrome, Windows
Code needed to reproduce the issue:
    wavesurfer.on('finish', function() {
        setCurrentSong((currentTrack + 1) % links.length);
        wavesurfer.play();

    });

(Please reduce your code as much as possible and only post the minimum code needed to reproduce the issue. A Code pen is an excellent way to share such code)
Use behaviour needed to reproduce the issue:
So if I want to autoplay a playlist. So when the first song finished, it should start the next song. At the moment it starts the next song but there is no sound an the console shows:
webaudio.js:677 Uncaught DOMException: Failed to execute 'stop' on 'AudioScheduledSourceNode': cannot call stop without calling start first.
    at t.value (https://unpkg.com/wavesurfer.js@3.0.0/dist/wavesurfer.min.js:1:13939)
    at ScriptProcessorNode.scriptNode.onaudioprocess (https://unpkg.com/wavesurfer.js@3.0.0/dist/wavesurfer.min.js:1:9407)

I use the exact same code from the playlist example.

ping @entonbiba

@kaibrune Does the following exist in your code? I'm not able to replicate it with the example link below:
https://wavesurfer-js.org/example/playlist/index.html
var setCurrentSong = function(index) {
        links[currentTrack].classList.remove('active');
        currentTrack = index;
        links[currentTrack].classList.add('active');
        wavesurfer.load(links[currentTrack].href);
    };


Fix AllowDynamicProperty for PHP8.2

Since the project uses namespaces, #[AllowDynamicProperties] need to be specified with a forward \.

Yes. Didn't saw that an other PR was created. Also this one also add an other missing #[AllowDynamicProperties] on FutureArray.

Thanks @adoy for your contribution

Just released ver. 1.2.2

Thanks to you.


Please implement marker clustering

GeoFire is great, but would be even more useful with marker clustering:
https://www.mapbox.com/mapbox.js/example/v1.0.0/leaflet-markercluster/
Would this be able to be ported to GeoFire?
https://github.com/Leaflet/Leaflet.markercluster

Mapbox's clustering already use latitude/longitude pairs to provide this feature. Since there is no straightforward solution to make this more efficient with GeoFire I would advise to just let Mapbox handle this by adding the latitude/longitude pairs you receive in the GeoFire callbacks. The combination of both libraries will be more powerful than letting GeoFire just reimplement this feature.