Telescope gives some duplicated results for goto definition and goto references

Describe the bug

Using the goto definition or goto references features of Telescope shows multiple results for the same line under some scenarios.
To Reproduce

Clear pre-existing Neovim config if necessary (rm -rf ~/.local/share/nvim && mv ~/.config/nvim ~/.config/nvim-old)
Install kickstart.nvim as normal so that it becomes the default neovim configuration
Open kickstart's init.lua file and find the map function used for LSP keymaps (/map =<CR>)
Hover cursor on any of the calls to the map function and use the goto definition finder gd to find definitions
Telescope gives 2 results for the map function

Hovering either option gives the exact same results in the "Grep Preview" pane (show below) and pressing Enter on either of them takes you to the same line, so it's not a different identical option off-screen or something like that.

This entry also shows up twice when using goto references with gr.
Desktop

OS: Arch Linux 6.8.2-arch2-1
Terminal: wezterm 20240203-110809-5046fc22

Neovim Version

NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1702233742

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Is this not expected given that you can have multiple lsps installed etc?

Does the default kickstart configuration have multiple Lua LSPs installed or something? I'm not totally sure what you mean by that. My expectation would be that goto definition will immediately jump to the target if there is only one place that symbol could be resolved to in the target scope. This is the only definition of the map function for example, and yet I get 2 results that both point to the same line of code.

Just lua_ls and stylua

Here's the output of :LspInfo as well just in case that matters too

I also verified that I still get the double results in this neovim session, just to make sure that I didn't unknowingly have multiple LSPs attached in the earlier screenshots when I originally made the issue.

Amazing debugging skill @aski !
This looks like it should be an improvement in telescope itself - to merge matches on the same line to one, so perhaps the issue with all these details should be opened in the telescope project.

Interesting behaviour that it captures both the function name in local map = and the actual function syntax function() itself, but looking at it now kind of makes sense it would work that way. Still a bit unfortunate behaviour since variable = function() ... end is a very common pattern in Lua and I personally think it would be better if there's some way to control if multiple entries on the same line could be collapsed to an instant jump (which was the behaviour I expected, and why I thought it was a bug in the first place). This feels like a more natural expectation instead of requiring a decision on whether to jump between points only a few characters away that are very similar logical entities anyways (plus an extra keyboard input or two).
Would it be reasonable for me to make an issue in the Telescope repo as a potential enhancement? Or would it make more sense for one of the kickstart contributors to do it? Or are we thinking this isn't something that should be changed at all?

@jcoffa I think it would not make a difference if you raise this issue or anyone else in this community, so go ahead.


Add C++ Solution for Problem Of The Day | 07 November 2022

@ImSaMPro can you assign this to me

Hi @Shreya111111 ,
Definately you can contribute too.

@ImSaMPro I will try to do it before 3 or 3.30 pm as I have some other important work to do.

@ImSaMPro I have created a pull request for floyd algorithm in cpp and java.. have not done in python yet and added a readme file


Javascript in

Operating System (+Version): Mac OS Sierra 10.12.11
VS Code Version: 1.7.2
beautify Version: Latest

Provide the settings you use:
(VS Code workspace and user settings, .jsbeautifyrc, .editorconfig)
Defaults
Action performed
Format html file that has javascript inside

Thank you for proving that this is a bug. I just spend an hour trying to figure out why I am the only person on this planet where the <script> tags are not getting formatted.
This issue should be marked critical as this brakes one of the main functionalities of a code beautifier.

Hey there. It's not a bug, it's caused by an unfortunate default setting in VS Code:
"html.format.unformatted":
    "a, abbr, acronym, b, bdo, big, br, button, cite, code, dfn,
        em, i, img, input, kbd, label, map, object, q, samp, script,
        select, small, span, strong, sub, sup, textarea, tt, var"

If you remove script from this setting it will work as you expected.

You'll be happy to know that the script tag is being removed from this default eventually. See  https://github.com/Microsoft/vscode/commit/8eb0cce43838ea8531c1bce8f44d835d9c2060d5

Wow, thank you very much for this fast answer and the fix. I actually suspected some config having script excluded as default, but couldn't find one. Thank you for this awesome project!

This should be fixed in the current VS Code version. script has been removed from the default for unformatted.


vim-full: rename from vim_configurable

Supersedes https://github.com/NixOS/nixpkgs/pull/145351.
The word "configurable" has proven to be confusing because it suggests user configuration, not build configuration. I think this is enough to warrant a rename.
Makes vim_configurable and vimHugeX aliases to vim-full.

@ofborg eval

"vim-full" is much better than "vim_configurable". Is there a plan to retire  "vim_configurable" ?

Is there a plan to retire "vim_configurable" ?

I don't know what is usually done for deprecated aliases. I don't think it's an emergency to retire it, since it won't show up in search any more. Maybe in a few releases when everyone has forgotten about it.

We probably shouldn't block this on the vim maintainers.

We probably shouldn't block this on the vim maintainers.

We don't have strong maintainer requirements so really anyone can merge this.

We probably shouldn't block this on the vim maintainers.

I agree with this as a vim maintainer since I don't have time for nixpkgs all the time.
Changes like this are an improvement to the overall consistency of nixpkgs and always highly appreciated.


Remove test username hardcoding

This hardcoding was put in place to avoid failure during test cleanup. It is being removed now.

Codecov Report

Merging #1231 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1231      +/-   ##
==========================================
+ Coverage   60.17%   60.31%   +0.13%
==========================================
  Files         119      119
  Lines       10383    10383
==========================================
+ Hits         6248     6262      +14
+ Misses       3479     3466      -13
+ Partials      656      655       -1

Impacted Files
Coverage Δ

remoteworkitem/scheduler.go
63.41% <0%> (+9.75%)
:arrow_up:

models/transaction.go
70% <0%> (+20%)
:arrow_up:

remoteworkitem/github.go
100% <0%> (+22.85%)
:arrow_up:

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 7166697...9963c94. Read the comment docs.

Good point @alexeykazakov , change addressed.


Performance improvement for preview ensemble algorithms (RF and ET)

Description
This corrects performance problems associated with estimators_ construction in the random forest and extra trees fit methods for regression and classification, and corrects a performance problem associated with a default construction of a sample_weights vector (which is unnecessary). These yield significant performance improvements.
Changes proposed in this pull request:

DFS tree traversal conversion to numpy data arrays occurs at end, rather than intrinsically (in estimator_ construction). This saves costly unnecessary data accesses.
Add the ability to the 'fit' method to operate without a sample_weights vector, which has impacts on branch of code operation on CPU (15-20% performance impact) and in part prevented GPU operation (sample_weights not supported in oneDAL GPU)
Set preview Random Forest Regressor back to using the 'hist' method rather than 'dense'

/intelci: run

/intelci: run

/intelci: run

@mergifyio rebase

/intelci: run

/intelci: run

@icfaust please rebase and enable this tests #1393 , if it's required.

/intelci: run

Common PreCommit job with https://github.com/oneapi-src/oneDAL/pull/2441: http://intel-ci.intel.com/ee459771-cca8-f11a-93c5-a4bf010d0e2e

http://intel-ci.intel.com/ee45d6a3-be9d-f123-adec-a4bf010d0e2e


Implement ServerSideCharacter

Terraria.Main.ServerSideCharacter

Has anything happened with this now that SQLite and MySQL are supported?

Not yet. But it is planned. This whole sql support thing has been a lot of work and it's still not completed. This is a high priority though. We plan on it for the next release, hopefully. But it could change.

Just FYI. If you want to test SSC now, you can! Just grab the latest binaries.

This should now be testable, the only thing left should be buffs.

Updated this to use the entity framework 6, soon I'll send off a few messages in slack when I'm ready for testing.

Yet another update. I believe we have settled now on a Dapper+Fluent Migrator DB solution and will be what we roll in B6.
We are now in the testing phase for SSC when I get our showcase server back online.


Powershell for install servicemap/infrastructure insights - "Cannot find path"

I'm attempting to follow the deployment instructions directly from this page, and running into an error where I cannot run the template deployment because PowerShell thinks the file I created in the previous step does not exist:
New-AzResourceGroupDeployment : Cannot retrieve the dynamic parameters for the cmdlet. Cannot find path 'Azure:/installsolutionsforvminsights.json' because it does notexist.

New-AzResourceGroupDeployment -Name DeploySolutions -TemplateFile ins ...

CategoryInfo          : InvalidArgument: (:) [New-AzResourceGroupDeployment], ParameterBindingException
FullyQualifiedErrorId : GetDynamicParametersException,Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDeploymentCmdlet

Here's the output of ls in the current directory:
PS Azure:> ls
clouddrive  installsolutionsforvminsights.json
The file exists and it's right there, but for some reason PowerShell thinks it doesn't.

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

ID: bc8a12a8-5e16-0e6a-43ae-891d18005212
Version Independent ID: 95f3383d-d3f9-cd50-b4f4-e4e1c135f47c
Content: Deploy Azure Monitor for VMs (preview)
Content Source: articles/azure-monitor/insights/vminsights-onboard.md
Service: azure-monitor
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@sshorkey Thanks for your question. We are checking on this and will respond to you soon.

@MGoedtel Would you have insights on this query?

@sshorkey & @SadiqhAhmed-MSFT  - I just tested the ARM template for installing the solution(s) and it worked as expected.  What I did notice while reviewing the instructions was step 3 is incorrect.  You should not modify the template, and what that step is meant to say is note the workspace ID, resource group name, and workspace location to pass to the respective command line parameters in step 4.  I'm going to bug this and fix it now.

@SadiqhAhmed-MSFT - the article has been fixed and will be released on 5/6 when normal publishing cycle resumes (post-BUILD content publish).  #please-close

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

I'm attempting to follow the deployment instructions mentioned in this page.
But i am getting the below error. Please help me.
PS Azure:> New-AzResourceGroupDeployment -Name parking360Deployment -ResourceGroupName ECE_parking360_nonprod-arm -TemplateFile C:\Users\rameshd\Downloads\template\template.json -TemplateParameterFile C:\Users\rameshd\Downloads\template\parameters.json
New-AzResourceGroupDeployment : Cannot retrieve the dynamic parameters for the cmdlet. Cannot find drive. A drive with the name 'C' does not exist.
At line:1 char:1

New-AzResourceGroupDeployment -Name parking360Deployment -ResourceGro ...

CategoryInfo          : InvalidArgument: (:) [New-AzResourceGroupDeployment], ParameterBindingException
FullyQualifiedErrorId : GetDynamicParametersException,Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDeploymentCmdlet


1.20.1

Hello, what is the status on a 1.20.1 version?

It should work. I haven't tested though, let me know when you test the 1.20 file :)

The problem I'm having is that since there is no release file under 1.20.1 in Modrinth, modloaders can't detect that it is installable.

Ah gotcha. Just updated it give it a go now

Cam confirm it's working for 1.20.1


Added forceFloatLabel property to input

Fixes #465
Added forceFloatLabel as mentioned in issue to force label to float if user wishes to use null. Default value is set to false and does not interfere with current behaviour.

Build failed due to Redbox-react (1.2.5 installed since package.json had ^1.2.4) peer dependency requirement (react ^15.1.0). Updated it and didn't see any odd behaviours.

Please, see #520!


Fails to build with Temurin JDK 17

Build scan link

Plugin version
1.25.0
Gradle version
8.3
JDK version
17
% java --version
openjdk <IP_ADDRESS> 2023-08-24
OpenJDK Runtime Environment Temurin-<IP_ADDRESS>+1 (build <IP_ADDRESS>+1)
OpenJDK 64-Bit Server VM Temurin-<IP_ADDRESS>+1 (build <IP_ADDRESS>+1, mixed mode)

(Optional) Kotlin and Kotlin Gradle Plugin (KGP) version
(Optional) Android Gradle Plugin (AGP) version
(Optional) reason output for bugs relating to incorrect advice
* What went wrong:
Execution failed for task ':lightwallet-client-lib:compileDebugJavaWithJavac'.
> Could not resolve all files for configuration ':lightwallet-client-lib:androidJdkImage'.
   > Failed to transform core-for-system-modules.jar to match attributes {artifactType=_internal_android_jdk_image, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
      > Execution failed for JdkImageTransform: /Users/lucacampobasso/Library/Android/sdk/platforms/android-34/core-for-system-modules.jar.
         > jlink executable /Library/Java/JavaVirtualMachines/temurin-17.jre/Contents/Home/bin/jlink does not exist.

Describe the bug
What's written above. It's the output of running  ./gradlew buildHealth
To Reproduce
N/A
Expected behavior
N/A
Additional context
Sorry for the lack of the other information, but I believe this has to do mostly with the java JDK.

I think it's missing a jlink there - but I have no idea where to take it from. Is there a setting to change that somehow?

Thanks for the issue. Do you have a reproducer? Also, I note you appear to be describing an Android project but have not provided the AGP version in use. And given that it's an Android project, the Kotlin version may also be relevant.


fix(lint): fix linting issues blocking release

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Summary:
Resolve linting issues that prevent release
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Why this library needs the expo dependencies?

@danielang good point 👍

Looks like this is still an issue. Any progress?

If you're not using Expo, open up the .babelrc (in node_modules/react-native-responsive-ui) and change the presets to react-native:
{
    "presets": [
        "react-native"
    ],
    "env": {
        "development": {
            "plugins": [
                "transform-react-jsx-source"
            ]
        }
    }
}

This is fixed now. However this new version is non backward compatible.


if formation is error,how to resolve it

how to fix the error when code runs.can I try catch this error to resolve it?

flow is not a runtime checker, obj.bar is just an invalid property access, it produces no runtime error, it just evaluates to undefined, so there is nothing to catch.
you can suppress the error with // $FlowFixMe


The value of the certificate in service client_certificate does not take effect after modification

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
2.8.1
Current Behavior
I configured a service and set client_certificate. When I modify the actual certificate value of this certificate through admin Api, the client certificate provided by the service to the upstream is still the original certificate, and it is not updated to the value I modified.
Service

Operate
Update Certificate content.
Expected Behavior
It is hoped that when the content of the certificate is changed, the service can automatically obtain the latest certificate
Steps To Reproduce
k8s

Anything else?
none

Could you share the setup you are using, for example, hybrid or traditional mode, DBLess or with a DB?
After waiting for a period of time, is the cert still not updated?

@StarlightIbuki Thank you for your reply.
this is my helm configuration, is the traditional mode
  migrations:
    init: false
    preUpgrade: false
    postUpgrade: false
  deployment:
    daemonset: true
    hostNetwork: true
  env:
    log_level: info
    database: "postgres"
    pg_host: xxxxxx
    pg_name: kong
    pg_password: xxx
    pg_port: 1921
    pg_user: xxx
    prefix: "/kong"
    plugins: "bundled,simple-basic-auth"
    dns_resolver: <IP_ADDRESS>
    error_default_type: "application/json"
    nginx_worker_processes: "auto"
    nginx_http_client_body_buffer_size: "4m"
  admin:
    enabled: true
    type: ClusterIP
    http:
      enabled: true
    tls:
      enabled: false
  proxy:
    enabled: true
    type: ClusterIP
    http:
      enabled: true
      servicePort: 80
      containerPort: 80
    tls:
      enabled: true
      servicePort: 443
      containerPort: 443
    stream:
      - containerPort: 7000
        servicePort: 7000
        protocol: TCP
      - containerPort: 9898
        servicePort: 9898
        protocol: TCP
      - containerPort: 7443
        servicePort: 7443
        protocol: TCP
        parameters:
          - ssl
          - reuseport
          - backlog=16384

Could you share the setup you are using, for example, hybrid or traditional mode, DBLess or with a DB? After waiting for a period of time, is the cert still not updated?

The id value in the client_certificate in the Service object is always the same. I modified the actual content of the certificate through the management API, and it still hasn't been updated after a period of time.
And I have deployed the same function in other clusters. The certificate has been updated for many days, but the certificate obtained by the tour is still the old certificate content.

I can only restart the kong pod, the certificate content will load normally.

This sounds like a cache invalidation problem. We will address that soon.

Hi @Murphy-hub,
The issue you mentioned should be resolved by https://github.com/Kong/kong/pull/8934, the patch was included in the 3.0.0 release. You could try to patch the older kong versions with that fix or upgrade to 3.0.

@mayocream Nice, thank you very much


some svgs turn into black box

looks like i may not be reading clipping path correctly

Hi Peter,
unfortunately that is a shortfall of svg2ttf or the fontello project for that part. I tried the erroneous Outline lock.svg glyph at fontello.com. It displays an error hinting to solutions here: https://github.com/fontello/fontello/wiki/How-to-use-custom-images#importing-svg-images But none of the suggested solutions worked.
http://www.glyphrstudio.com/online/ also displays an error. Seems that certain features the icons use in their path attribute are not supported in ttf.
At the moment I am not yet sure what to do about that.
Cheers
Marco

Also happens with the "star" icon when using Rounded style.

yea.. maybe go in and manually edit the blanks in a font editor or something

Yea i've dealt with this problem before, if someone is down to identify all of them i can fix them in illustrator

I created a Figma file where I loaded all of the icons from the four variants to easily identify the black boxes. https://www.figma.com/file/0HPZjnC9OUwUtjzlBmNmiqp6/Material-Design-Icon-Font-Fix?node-id=0%3A1
If anybody wants to jump in the file and add comments wherever you find those black boxes, it would be easier to make a list of all the icons needing fixes.

@petergng, how do you fix those in Illustrator so there are no black boxes?
Is there a better solution to use Material Design on Figma/Sketch/XD?

O damn this is amazing! this is super helpful, let me try to get at this issue in the next couple weeks.

Amazing is what you have done with the fonts. Thank you so much for doing Google's work.
If you tell me how to fix those black boxes in Illustrator I can help with that.

ok, so unfortunately this seems like an issue with some of google's provided svgs which have a black box around them: download the sharp version of "web" and you can see the svg renders as a black box. https://material.io/tools/icons/?search=web&icon=web&style=sharp
The path here would be to redraw source, or i parse out the box they have hoping it's the culprit for everyone...

@geiras can i get access to the figma file? we need to just edit the source by outlining and then removing the outer border and re-exporting the svgs

I think you already got access to the Figma file, right?
If you don't, just use this link and make a copy of the file, if you wish to edit...

Yup, i've highlighted all the messed up ones. If you can help run through and create fixed source for them that can be exported as svg i can re-run the generator. Here's the source i fixed up: https://www.figma.com/file/XgYwTUBgjClPBrQHMbgDvP/Material-Design-Icon-Font-Fix-Source-pn?node-id=0%3A1

I've removed the black boxes from all the highlighted icons on that file. But, as the outlined version was created out of a single text box, the icons are on the same vector shape, making it impossible to export without cutting and pasting the isolated icon.
Given that, I've marked with a green box all the icons that are already isolated and marked with an orange box all the icons that still need to be isolated.
And then there are the missing icons that I'll try to add later, by downloading the original svg source files from Google.

Do you need the new fixed versions of SVGs to have the 24x24px bounding box?

Yup 24x24 with the same file names!
On Tue, May 28, 2019 at 1:47 AM geiras<EMAIL_ADDRESS>wrote:

Do you need the new fixed versions of SVGs to have the 24x24px bounding
box?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/petergng/material-icon-font/issues/2?email_source=notifications&email_token=AAIXLYGZTAALMMWFQHYH3XLPXTWSFA5CNFSM4GGRPOZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWLNLGI#issuecomment-496424345,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIXLYDZLTT2LAJ675MFPQLPXTWSFANCNFSM4GGRPOZQ
.

--
Peter Ng

Can you confirm that the attached example file would attend those needs?
I opened one of the faulty SVG with Illustrator, fixed the Compound Path and saved again.
Outline_star.svg.zip

That should work but i think tracking the source via figma: https://www.figma.com/file/XgYwTUBgjClPBrQHMbgDvP/Material-Design-Icon-Font-Fix-Source-pn?node-id=0%3A1 is a better idea given i think there will be a bit of trial and error and be good to have a source of truth.

The problem with the Figma is that is too slow to delete the compound paths. All icons are a single vector shape which make everything too slow.
For me it's easier to just find the faulty icons on the bin folder, open with Illustrator, fix it, save and close.

ahh ok, that should be fine, can you dump the files to me when you're done?
Peter Ng
On Tue, Jun 4, 2019 at 2:20 AM geiras<EMAIL_ADDRESS>wrote:

The problem with the Figma is that is too slow to delete the compound
paths. All icons are a single vector shape which make everything too slow.
For me it's easier to just find the faulty icons on the bin folder, open
with Illustrator, fix it, save and close.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/petergng/material-icon-font/issues/2?email_source=notifications&email_token=AAIXLYBZAGG6BKGDZ7PAYMTPYYXXLA5CNFSM4GGRPOZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW36ZQY#issuecomment-498592963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIXLYFLZKXPCM2NVEY3GFDPYYXXLANCNFSM4GGRPOZQ
.

Hey... Here is a folder with the fixed version of all of the black box icons identified on the Figma file.
I preserved the original filenames and folder structure.
Can you take a look and tell me if it works?
bin_fixed.zip

Also, created a pen for the chunks of missing icons.
Some are available, others don't.
I can generate a PDF from this Codepen and create SVG versions for those that exist.
https://codepen.io/anon/pen/arxRpv?editors=1000

@petergng is there a solution for the "black box icons"?

I've done "my part" as a designer. I've provided the fixed versions of all SVGs that had the black boxes. Now, on to someone who understands code better than me and can fix it for good.
I'm still here if anybody needs anything else from a designer.
Currently, I'm using https://materialdesignicons.com since it has way more icons than what Google provides.

@geiras no worries. I was asking @petergng.
I've tried working with materialdesignicons.com but I need something that will integrate with XD. Adding SVGs to every component/layout/artboard is labor intensive. Fontawesome offers a font to install and icons appear by selecting the font. They all seem to work without the "black box" issue. Looking for the same solution with Material.io icons.
Not sure what it takes to add the repaired SVGs you provided and compile a new font file...just trying to get there.

fortunately it seems like they've updated their hosted font css here: https://fonts.googleapis.com/css?family=Material+Icons|Material+Icons+Outlined|Material+Icons+Sharp|Material+Icons+Round|Material+Icons+Two+Tone
gonna play around with this and see if it's everything we need or at least helps out this process

@petergng did you have any luck?

i ended up finding the ttf and otfs filed here: https://github.com/petergng/material-icon-font/issues/6
Unfortunately the figma file isn't exportable for me. if someone could name them all and make it exportable as svg, it will be trivial for me to run it and regenerate. otherwise i would use the ttf's i listed above in the interim


Entry not found component

If on detail pages / forms the requested entry not exists, a nice error should be shown.
For this, a "Entry not found" component is needed.

[ ] Create UI screendesign
[ ] Implement it

The comon ux is, to send the user to the listing page with an error toast. Therefore no component is needed.


disable global

Hi,
I have set the following configuration :
config :meerkat_controller, MeerkatController.Scheduler,
global: false,
  jobs: [
    cluster_metrics_generation: [
      schedule: {:extended, "*/10"}, # Runs every ten seconds,
      run_strategy: Quantum.RunStrategy.Local,
      task: {Binarytemple.Metrics.ClusterMetricsJob, :build, []}
    ]
  ]

But I still see lots of messages like this in the logs indicating that ExecutionBroadcaster is running :
17:38:20.001 [debug]<EMAIL_ADDRESS>Continuing Execution Broadcasting at -576458753997 (2019-09-12T16:38:30)

I'm also using a forked version of swarm and I'd like to avoid that dependency, any advice?
Thanks

Tracked via #411


Virtualized Table fails DOM nesting validation

Virtualized Table fails DOM nesting validation
I am using the example in the docs
https://react-table-library.com/?path=/docs/compact-table--virtualized
Warning: validateDOMNesting(...):
 cannot appear as a child of

Warning: validateDOMNesting(...):
 cannot appear as a child of

Warning: validateDOMNesting(...):
 cannot appear as a child of

Warning: validateDOMNesting(...):
 cannot appear as a child of

Warning: validateDOMNesting(...):
 cannot appear as a child of

I will update the docs, but try the following:

Because in a virtualized table, there are div elements between the table elements, but semantically that's not allowed. Therefore we need to transform the whole table to div elements.
Let me know if this helps.

Did it help @MA-MacDonald ?

Closing this, because the solution to this issue is there.


Abort from test_threads() on ESP32-C3

Attempting to run the demo as of 0d085d7, I'm getting an abort from test_threads():
[...]
Rust main thread: Thread { id: ThreadId(1), name: Some("main"), .. }
Main TLS before change: 13
Main TLS after change: 42
This is thread number 0, Thread { id: ThreadId(2), name: None, .. }
Inner TLS: 0
abort() was called at PC 0x40380ff5 on core 0
[...]

A full log is shown in https://gist.github.com/sirhcel/bf55cb77898cb1e45d25e4eb1c66433d. I've seen this behavior with Rust 1.59.0 and 1.58.0 nightlies.
Just commenting out the invocation of test_threads() makes this demo run:

Connects to WiFi
Succesfully pings external host
Reacts on the echo server on port 8080
Serves via HTTP on port 80

Duplicate of #45.


Added pre-commit linting as GitHub workflow

Adds pre-commit checks.
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

We could also drop Travis config in this PR since all checks are now run as GH actions.

@NyanKiyoshi  This new checks still trigger Codecov Report?

@fowczarek yes, that's part of #5787 - svg I don't know but I prefer excluding them as they are generated files


Single WordPress With SubDirectory

set $dir "";
if ($request_uri ~ ^/([^/]*)/.*$ ) {
      set $dir1 /$1;
}
location / {
      try_files $uri $uri/  $dir1/index.php?$args;
 }

Linked #66

Can we get an update on this item?
Single Install Site on a sub Directory....
how does one accomplish it?
i'm using the most recent EE ..

Partially related, but I really like the way Cliff Seal is going about it here: https://youtu.be/pUGrDdvkwtk?t=19m36s

Will be handled in https://github.com/EasyEngine/easyengine/issues/66


Added dot symbol to HTML::getInputId

When you use attributes of the model with dots in it's names, you get wrong id for html field.
Example when you need names with dots is when you use relations withing gridview
https://github.com/yiisoft/yii2/blob/master/docs/guide/output-data-widgets.md#working-with-model-relations

Merged, thanks!


convenience -all flag for moving files

It would be nifty to have a shorthand for datashuttle test-shuttle upload_data --experiment_type all --sub_names all --ses_names all, perhaps sth like datashuttle test-shuttle upload_data --all. This would essentially be similar to the git push functionality.

implemented as a new convenience function upload_all or download_all (for API and CLI)


camera_qcom: move opencl stuff into class Rgb2Gray

Looks clean. Can you change the class name to LapConv or something? (the kernel does rgb -> gray -> conv, sorry for the bad naming)

Can you rebase? Should be just deleting a few duplicate lines


merging of #218 breaks the build

#218 didn't account for a number of file path fixups that were necessary. The pkg-config files are needing fixups as well.

resolved in #236


VS Code extension doesn't work with v1

I'm trying to upgrade a codebase to v1, but it seems that this extension doesn't work anymore.
Looking at the code, I'm assuming that this should be using @twind/intellisense under the hood, but it's not. Does this mean that the stable release of twind doesn't have intellisense except for when we use twind.run?
Is there any way around this?

Same in here!

Broken for me as well, sadly.

This is a separate VSCode Twind intellisense extension that I developed, which only supports v1. It is still a work in progress. If you're interested, feel free to check it out.


update all dependencies in one

replace all individual pyup-bot PRs into a single one.

Coverage remained the same at 96.517% when pulling 864cf3487a1a58179d142d7b5a7daa24812ea834 on ixje:master into c1288c3c49fadb2f2a9d9e4c915dbaec741f953a on CityOfZion:master.


While entering discount code pressing [Enter] triggering PlaceOrder Action [UX - Enhancements]

While entering discount code pressing [Enter] triggering PlaceOrder Action.
So, it's a kind of user behavior and expecting Apply to Discount Code, but Placing Order.

This is correct. This needs to be fixed. We need to invalid formik form when there is a data present in the coupon field. I will incorporate this when I get time.


Merge Two Sorted Lists Python

You are given the heads of two sorted linked lists list1 and list2.
Merge the two lists in a one sorted list. The list should be made by splicing together the nodes of the first two lists.
Return the head of the merged linked list.
https://leetcode.com/problems/merge-two-sorted-lists/

Hey @arungudelli , Please assign this to me.

Hey this is me again! Commenting here as asked to. would like to do this one . Kindly assign!

@bhoomitri4 Read the guidelines
https://github.com/arungudelli/python.shiksha/blob/f2e70e57e90988b3db839041a472ea6db3a1b262/CONTRIBUTING.md
And look at the sample article
https://python.shiksha/problems/string-anagram-python/
Let me know if you can able to explain the soultion

@JatinSingh08 Are you ok work on https://github.com/arungudelli/python.shiksha/issues/49 ?

Do you want me to explain the solution in simple english of the main problem statement of merging the lists here in the comments itself?

You need to write the explanation in the article.. read the contributing guidelines i have provided the sample article

ok I have read and will do so !!

@arungudelli Please assign me. I would like to work on this

I did what was asked to do. Please review that and merge the PR if all is good!! thank you.

Please assign me. I would like to work on this.


spring-cloud-zookeeper have error when setting of instance status OUT_OF_SERVICE

1. I create two simple project of spring-cloud-zookeeper
Below is the basic configuration of my project.
fiirst :
spring.cloud.zookeeper.connectString=localhost:2181
spring.application.name=spring-cloud-zookeeper-server
server.port=8811

Main Class:
@SpringBootApplication
@EnableDiscoveryClient
public class ServerApplication {
    public static void main(String[] args) {
        SpringApplication.run(ServerApplication.class, args);
    }
}

Controller critical code  :
List<String> services = client.getServices();
        for (String service : services){
            List<org.springframework.cloud.client.ServiceInstance> instances = client.getInstances(service);
            for (org.springframework.cloud.client.ServiceInstance instance : instances){
                Collection<ServiceInstance<ZookeeperInstance>> zkInstances = serviceDiscovery.queryForInstances(instance.getServiceId());
                for (ServiceInstance<ZookeeperInstance> zkInstance : zkInstances) {
                    ZookeeperServiceRegistry serviceRegistry = new ZookeeperServiceRegistry(serviceDiscovery);
                    ZookeeperRegistration registration = ServiceInstanceRegistration.builder()
                            .uriSpec(zkInstance.getUriSpec())
                            .address(zkInstance.getAddress())
                            .port(zkInstance.getPort())
                            .name(zkInstance.getName())
                            .payload(zkInstance.getPayload())
                            .build();

                    if(null == zkInstance.getPayload()){
                        System.out.println("unknow");
                    }else{
                        Object status = serviceRegistry.getStatus(registration);
                        if(zkInstance.getName().equals("spring-cloud-zookeeper-server2")){
                            serviceRegistry.setStatus(registration, StatusConstants.STATUS_OUT_OF_SERVICE);
                        }
                    }
                }
            }
        }

second:
spring.cloud.zookeeper.connect-string=localhost:2181
spring.application.name=spring-cloud-zookeeper-server2
server.port=8822

the Main class ditto.

the phenomenon:
when execute the code.

serviceRegistry.setStatus(registration, StatusConstants.STATUS_OUT_OF_SERVICE);
java.lang.Exception: Service not registered: ServiceInstance{name='spring-cloud-zookeeper-server2', id='7fe212c3-2453-4121-9642-afb626c3fb59', address='QJLCPJMMQ4IOW96', port=8822, sslPort=null, payload=ZookeeperInstance{id='application-1', name='spring-cloud-zookeeper-server2', metadata={instance_status=OUT_OF_SERVICE}}, registrationTimeUTC=1526031092312, serviceType=DYNAMIC, uriSpec=org.apache.curator.x.discovery.UriSpec@6c2ac0dc, enabled=true}
	at org.apache.curator.x.discovery.details.ServiceDiscoveryImpl.updateService(ServiceDiscoveryImpl.java:201) ~[curator-x-discovery-4.0.1.jar:na]
	at org.springframework.cloud.zookeeper.serviceregistry.ZookeeperServiceRegistry.setStatus(ZookeeperServiceRegistry.java:110)

location

 private final ConcurrentMap<String, Entry<T>> services = Maps.newConcurrentMap();

the services properties length is always zero of ServiceDiscoveryImpl class.
public ServiceDiscoveryImpl(CuratorFramework client, String basePath, InstanceSerializer<T> serializer, ServiceInstance<T> thisInstance, boolean watchInstances)
    {
        this.watchInstances = watchInstances;
        this.client = Preconditions.checkNotNull(client, "client cannot be null");
        this.basePath = Preconditions.checkNotNull(basePath, "basePath cannot be null");
        this.serializer = Preconditions.checkNotNull(serializer, "serializer cannot be null");
        if ( thisInstance != null )
        {
            Entry<T> entry = new Entry<T>(thisInstance);
            entry.cache = makeNodeCache(thisInstance);
            services.put(thisInstance.getId(), entry);
        }
    }

    @Override
    public void updateService(final ServiceInstance<T> service) throws Exception
    {
        Entry<T> entry = services.get(service.getId());
        if ( entry == null )
        {
            throw new Exception("Service not registered: " + service);
        }
        synchronized(entry)
        {
            entry.service = service;
            byte[] bytes = serializer.serialize(service);
            String path = pathForInstance(service.getName(), service.getId());
            client.setData().forPath(path, bytes);
        }
    }

in addition, I read the  DefaultServiceDiscoveryCustomizer class  No pass parameters ServiceInstance.
public DefaultServiceDiscoveryCustomizer(CuratorFramework curator, ZookeeperDiscoveryProperties properties, InstanceSerializer<ZookeeperInstance> instanceSerializer) {
		this.curator = curator;
		this.properties = properties;
		this.instanceSerializer = instanceSerializer;
	}

	@Override
	public ServiceDiscovery<ZookeeperInstance> customize(ServiceDiscoveryBuilder<ZookeeperInstance> builder) {
		// @formatter:off
		return builder
				.client(this.curator)
				.basePath(this.properties.getRoot())
				.serializer(this.instanceSerializer)
				.build();
		// @formatter:on
	}

How to solve this problem?

@spencergibb  Do you have time to help with this?

I think that Spencer will reply once he has some time. Can you create a failing test in Zookeeper's codebase to prove your point?

ok, this is my project address.
address : https://github.com/zhongzunfa/spring-cloud-zookeeper-register.git

Is this still a problem with the latest Finchley.SR2?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Tidy up the client python file

Was looking to use this, realized there was a lot of cleanup that could be done to the python client script.

@rpavlik Looks cleaner and more organized. Thanks for putting this together.


ci: bump github actions

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bumps GitHub Actions to their latest major versions. Main change is they're now using Node 16 instead of 12, which is EOL at end of month.

Changelog for actions/checkout can be found here
Releases for actions/setup-node can be found here

This change is

Thank you, merged


[Bug]: API call inside a JS is displayed as an error when bound to a widget

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Calling an API/datasource query in a JS object and binding this to a widget throws an error stating Api1.run is not a function
Steps To Reproduce

Go to app at this location
Observe the binding on the table widget display an error stating mockAPI.run is not a function

Public Sample App
https://release.app.appsmith.com/app/git-connected-app-with-js-objects/page1-62668caa01f360366e25e4d2/edit
Version
Cloud

Related to #12179
Duplicate of #13288

Error message needs to be improved and documentation enhanced on usage of async functions
@Rishabh-Rathod will raise a seperate task for this

https://github.com/appsmithorg/appsmith/issues/13320 created an issue for improvement in docs here.
@ramsaptami Can you create an issue for improvement of the error message?

Hey Rishabh, I've added the error tag on the issue and updated the description to state the same. Hope the works


Add support for aliases

Currently, fastavro does not support Avro aliases: http://avro.apache.org/docs/current/spec.html#Aliases
Aliases are required for renaming fields, which is a frequent type of schema evolution. At read time, when a field is absent from the writer schema, the field name is looked up among the field aliases.
If you agree this is a feature worth having, I can start working on a PR.
EDIT: simplified the description of this issue to reduce the scope of aliases.

I second this request - useful if one maintains technical & "business" field names. It is not clear to me if doc attribute is supported at field-level either.
If not, @regisb I like to piggyback on your PR as adding support for both attributes would be in the same place.

@yiga2 I'm happy to see some interest in this feature! I'm going to be too busy to work on it in the next couple weeks, but I should be able to open a PR before the end of the year. I'm not sure if doc is already supported; if not, I will add it to the PR.

Feel free to submit a PR. Thanks!

@yiga2 I just opened a PR to implement aliases, but I did not work on doc attributes. It seems to me that they are supported. Which use case did you have in mind?

To store more descriptive information about the record (table) and
column/field. Aliases are useful as well to have a technical and business
name for a field but have a long text to explain further (=doc attribute)
is even better as we are dealing with many flavors of the same data,
depending on the customer. Maintaining a separate data dictionary would be
unnecessary.
I appreciate you reaching out but I will give spavro a try or even avsc.
The latter is for node.js but is superfast, which is critical in my use
case: encoding of 2GB csv files in less than 5min as I want to use
serverless functions (AWS Lambda or G/Az Functions).
Thanks anyways.
On Thu, Nov 16, 2017 at 2:15 PM, Régis B<EMAIL_ADDRESS>wrote:

@yiga2 https://github.com/yiga2 I just opened a PR to implement
aliases, but I did not work on doc attributes. It seems to me that they
are supported. Which use case did you have in mind?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tebeka/fastavro/issues/87#issuecomment-345032272, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AYDkSVuCMsh0J4BTF6fZEf7u2ySgIxsuks5s3InNgaJpZM4OpzHd
.


Validation states such as 'touched' not applied

The touched property is not set to true after clicking into and out of the input element.
I'm using validation messages on the page which rely on these properties.
Any chance you could add support for this?
Thanks. Rich.

Friend I'm going through the same problem, I still have not found the solution. I'm also interested in this adjustment. I sent an email to Caesar, but there was no return.

Thanks - the other properties seem to be working ('dirty' etc.), just not 'Touched'.
Many thanks
Rich

@cesarrew muito obrigado, você irá me ajudar muito.

Hi, I'm also having the same issue - "touched" property not updating on input change - which bypasses validation.
Thanks

@cesarrew Has any solution been developed for this problem?

To get around this issue I had to move to "text-mask-addons"
https://www.npmjs.com/package/text-mask-addons - it is not as clean as the currency-mask, it returns a string value so you will need to convert this in @HostListener function but it works.

@richyclarke @heidermatos @RodyKirwan1
A new version was released and should fix this issue.

Many thanks @cesarrew , I will update here on my project and test.


Support py3.4 and py2.7 both by same source code.

It works by 2.7 or 3.4 .
Notes:
Please create sysdic with Python 2.7.
Because Python3.4 create sysdic for 3.4 but not 2.7

Thakns PR!
Obviously, this is the desirable way. I'll try it soon.


feather-format now available on conda via conda-forge

conda install feather-format -c conda-forge
for osx/linux-64

This worked on OS X, but I did receive a warning during usage:
/Users/me/anaconda/lib/python2.7/site-packages/pandas/computation/init.py:19: UserWarning: The installed version of numexpr 2.4.4 is not supported in pandas and will be not be used
From the installation:
Fetching package metadata: ......
Solving package specifications: .........
Package plan for installation in environment /Users/A/anaconda:
The following packages will be downloaded:
package                    |            build
---------------------------|-----------------
numpy-1.10.4               |           py27_0         3.0 MB
numexpr-2.4.4              |      np110py27_0         103 KB
scipy-0.16.0               |      np110py27_1        11.8 MB
pandas-0.18.0              |      np110py27_0         5.8 MB
feather-format-0.2.0       |           py27_0          90 KB
------------------------------------------------------------
                                       Total:        20.8 MB

The following NEW packages will be INSTALLED:
feather-format: 0.2.0-py27_0

The following packages will be UPDATED:
numexpr:        2.4.3-np19py27_0   --> 2.4.4-np110py27_0
pandas:         0.18.0-np111py27_0 --> 0.18.0-np110py27_0
scipy:          0.15.1-np19py27_0  --> 0.16.0-np110py27_1

The following packages will be DOWNGRADED:
numpy:          1.11.0-py27_0      --> 1.10.4-py27_0

Proceed ([y]/n)? y

is the message not clear? that's from pandas. you version of numexpr is a really buggy one. it won't be used. you can either downgrade numexpr after, or create a new env with numpy 1.11 which will use the latest of numexpr.
you can also conda install feather-format -c conda-forge --no-update-deps but you may need to upgrade conda before that (IIRC its a bit broken on some older versions)

From what I understood of the message, the install of feather both upgraded numexpr from 2.4.3 to 2.4.4 and downgraded numpy to 1.10.4 from 1.11.0.  From your comment, I upgraded numpy to where it was, and now pandas is happy.  Thanks Jeff.
$ conda install numpy
Fetching package metadata: ....
Solving package specifications: .........
Package plan for installation in environment /Users/me/anaconda:
The following packages will be downloaded:
package                    |            build
---------------------------|-----------------
numexpr-2.5.1              |      np111py27_0         109 KB
scipy-0.17.0               |      np111py27_0        12.9 MB
------------------------------------------------------------
                                       Total:        13.0 MB

The following packages will be UPDATED:
numexpr: 2.4.4-np110py27_0  --> 2.5.1-np111py27_0
numpy:   1.10.4-py27_0      --> 1.11.0-py27_0
scipy:   0.16.0-np110py27_1 --> 0.17.0-np111py27_0

Proceed ([y]/n)?

BTW I have written a build script to generate manylinux1 wheels for all feather versions:
https://github.com/matthew-brett/manylinux-builds/blob/master/build_feathers.sh
All tests pass. It would be great to upload the resulting wheels to PyPI to help increase adoption.
There is also a script to generate manylinux1 wheels for all pandas versions in the same repo:
https://github.com/matthew-brett/manylinux-builds/blob/master/build_pandas.sh


Diversify default RNG seed

Description
Fix fixed seed value for random number generators in MXNet (fixes https://github.com/apache/incubator-mxnet/issues/21053).

Someone may want to have repeatable results and I guess it was designed this way. So I am not sure if this change should applied.

In other tools like numpy by default "random is random" and to have repeatable results one needs to explicitly set the seed. I believe we should have it in the same way as we are somewhat following numpy's design.

@mxnet-bot run ci [unix-cpu, unix-gpu]

In numpy by default "random is random" and to have repeatable results one needs to explicitly set the seed. I believe we should have it in the same way as we are somewhat following numpy's design.

So, we can allign to the NunPy  behaviour and make random seed as default.

@mxnet-bot run ci [unix-cpu, unix-gpu]

@mxnet-bot run ci [unix-cpu, unix-gpu]

@mxnet-bot run ci [unix-cpu, unix-gpu, windows-gpu]

@mxnet-bot run ci [unix-cpu]

@mxnet-bot run ci [windows-gpu]

@mxnet-bot run ci [windows-gpu]


[release-4.10] Bug 2108692: Check all batches for upgrade complete

Cherrypick from 78947ce43d6ec9bee80572b94ab29598afc58ddd #257

Check all batches for upgrade complete

Proper error handling for isUpgradeComplete

/bugzilla refresh

/hold

/unhold

/hold

/unhold

/approve

/retest


Build a CI Workflow

CONTEXT
I had a meeting with @edisonz0718  on Friday discussing some CI work we can automate. Also, we discussed the possibility to use Argocd for CD work.
DONE WHEN
1/ build a GHA workflow to lint/build/push images (2 in total)
2/ finalize the decision about image tagging

@AndyBoWu Is this done so that we can close?


error

What's happening?

How can it be reproduced?

Build details?

No details provided by reporter.

No details provided by reporter.


Palindrome Linked List

Hello I'm a participant of Hacktoberfest 2021. I'm very new to open source. I want to contribute in your repo DSA-Linked List by adding this problem with a solution in C++. Please assign me this issue.
Problem Statement :
You have been given a head to a singly linked list of integers. Write a function check to whether the list given is a 'Palindrome' or not.
Consider:
While specifying the list elements for input, -1 indicates the end of the singly linked list and hence, would never be a list element.
Sample Input 1:
9 2 3 3 2 9 -1
Sample Output 1:
true
Sample Input 2:
0 2 3 2 5 -1
Sample Output 2:
false

@paltannistha  go ahead :)
Follow for more such repo to contribute

Hello, I would like to work on this issue in Java. Could you please assign it to me?

@paltannistha go ahead :) Follow for more such repo to contribute

Yeah thank you.

Hello, I would like to work on the issue in C++. Could you please assign in to me?

hey, I would like to work on this issue in python in hacktoberfest-22 can u pls assign me this issue

this repo is no more under hacktoberfest  , though you can contribute


Error in running 'python -m vllm.entrypoints.openai.api_server '

Your current environment
python -m vllm.entrypoints.openai.api_server \
--served-model-name Llama-3.1-8B-Instruct \
--model "/data/hf_models/Llama-3.1-8B-Instruct/" \
--tensor-parallel-size=4 \
--trust-remote-code

I want to deploy the llama model, ‘/data/hf_models/Llama-3.1-8B-Instruct/’ is the model path. However, when running this script, it will generate an error:
ImportError: cannot import name 'build_regex_from_schema' from 'outlines.fsm.json_schema', and I cannot find  build _regex _from _schema' in json_schema.py actually.
Enviroment: vllm=0.5.0.post1, outlines=0.1.11
How to solve this problem?
How would you like to use vllm
I want to run inference of a Llama-3.1-8B-Instruct. I don't know how to integrate it with vllm.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Please check your package compatibility. For vllm==0.5.0post1, you should have outlines >= 0.0.43 according to the requirements.txt file here: https://github.com/vllm-project/vllm/blob/v0.5.0.post1/requirements-common.txt


NanoMQ process crashes when reloading configuration file with 'munmap_chunk(): invalid pointer' error

Describe the bug
When using nanomq start --conf /etc/nanomq.conf followed by nanomq reload to reload the configuration file, the NanoMQ process crashes and reports an error message:
munmap_chunk(): invalid pointer
Aborted (core dumped)

Expected behavior
I expected the NanoMQ process to reload the configuration file without crashing.
Actual Behavior
The NanoMQ process crashes and reports an error message: munmap_chunk(): invalid pointer.
To Reproduce

Run nanomq start --conf /etc/nanomq.conf
Run nanomq reload

Environment Details

NanoMQ version: 0.17.5
Operating system and version: Linux ccdecc0ea4a2 5.13.0-52-generic #59-Ubuntu SMP Wed Jun 15 20:17:13 UTC 2022 x86_64 GNU/Linux
Compiler and language used: N/A
Testing scenario: N/A

Client SDK
N/A
Additional context
N/A

Fixed, plz verify and reopen issue if you found any new problems


unresolved interferes with the log widget

 Issue by sigmundch
Originally opened as dart-lang/sdk#21640

hides the log widget that shows warnigns and errors. To fix this, we think the log widget should move the 'unresolved' attribute down to every child of body (except the widget of course).
We probably want to include 'unresolved' in the list of blacklisted attributes as well.

 Comment by lrhn

Added Area-Pkg label.


[BE] allow description in APNS notifications

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1796f1e). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #34   +/-   ##
=========================================
  Coverage          ?   81.98%
=========================================
  Files             ?       31
  Lines             ?     1049
  Branches          ?        0
=========================================
  Hits              ?      860
  Misses            ?      189
  Partials          ?        0

Impacted Files
Coverage Δ

universal_notifications/backends/push/apns.py
80.95% <100%> (ø)

universal_notifications/notifications.py
95.7% <100%> (ø)

Continue to review full report at Codecov.

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


Summary and detail consistency for campaign:update

The summary says to use campaign:update while the detail talks about campaign:rebuild as being required.

This is the wrong repository


Output parameters

Greetings Aphros team!
We are using Aphros to research foam. We encountered some problems and hope you can help us.

We would like to know which parameters of the foam can be obtained if the foam is studied. For example, the size or distribution of the foam over time. We would appreciate it if you could tell us in detail what foam parameters are available or how to set up to access the parameters that are available
We have run some examples and imported the resulting .vtk file into paraview, and we found that only the information with the names cl, l and c could be visualized, and we wanted to know what each of these three parameters meant. And we found that not all examples have the three parameters in the output files (.vtk) and . We would like to know how these parameters are set up for export to .vtk files, are they defined in a file? Please explain in more detail
We look forward to your response!
Thank you

Hi，Dr.karnakov. Thank you for your answer, it was very helpful. We still have some questions that we are looking forward to your answers.

We are a bit confused about extent.
(1) Is extent built from (0.0.0) ?
(2) Regarding the waterfall example, the supplementary materials  of the 11th reference you provided (Computing foaming flows across scales: From breaking waves to microfluidics) describes domain width=100mm, but in the source code example the extent is 200mm, and there is no wall set in the width direction to limit it, so should the domain width be 200mm?
(3) Regarding the Clustering example, the supplementary materials describes the domain height=20 mm , while the extent in the source code example is 30 mm. We only find the centre of the box in the y-direction is 20 in the b.dat file. What should we understand by domain and extent?
We have tried to simulate waterfall and Clustering in the example, if we use the settings of m=32 32 32 and np=4 (in computer A), there will be a situation where the liquid will appear in the upper part of the computational domain and the liquid above will fall into the lower part of the liquid and interfere with the lower part of the liquid, while with the number of grids you provide and increasing the number of np (in the server B) it will not appear.  What is the reason for this. We have included pictures here so that you can better understand our problem. The AB in the file name represents the two cases of running in the computer AB respectively.

3.Thanks again for the answers to our last questions. We have studied the traj_*.csv file and we understand that cl, p, vf, v, x, y, z are the colour value, pressure inside the bubble, volume fraction, velocity and bubble position respectively, but we don't quite understand what xx, xy, xz, yz, zz mean, what does each mean?
We look forward to your answers again. Thanks again!

Hi，Dr.karnakov. Thank you for your answer, it was very helpful. We still have some questions that we are looking forward to your answers.
1、We are a bit confused about extent.
(1) Is extent built from (0.0.0) ?
(2) Regarding the waterfall example, the supplementary materials of the 11th reference you provided (Computing foaming flows across scales: From breaking waves to microfluidics) describes domain width=100mm, but in the source code example the extent is 200mm, and there is no wall set in the width direction to limit it, so should the domain width be 200mm?
(3) Regarding the Clustering example, the supplementary materials describes the domain height=20 mm , while the extent in the source code example is 30 mm. We only find the centre of the box in the y-direction is 20 in the b.dat file. What should we understand by domain and extent?
2、We have tried to simulate waterfall and Clustering in the example, if we use the settings of m=32 32 32 and np=4 (in computer A), there will be a situation where the liquid will appear in the upper part of the computational domain and the liquid above will fall into the lower part of the liquid and interfere with the lower part of the liquid, while with the number of grids you provide and increasing the number of np (in the server B) it will not appear. What is the reason for this. We have included pictures here so that you can better understand our problem. The A and B in the file name represents the two cases of running in the computer A and B respectively.

3.Thanks again for the answers to our last questions. We have studied the traj_*.csv file and we understand that cl, p, vf, v, x, y, z are the colour value, pressure inside the bubble, volume fraction, velocity and bubble position respectively, but we don't quite understand what xx, xy, xz, yz, zz mean, what does each mean?
We look forward to your answers again. Thanks again!

sorry for a late reply,

We are a bit confused about extent.
(1) Is extent built from (0.0.0) ?

Yes, extent is the largest size of the three (in x,y,z)

(2) Regarding the waterfall example, the supplementary materials of the 11th reference you provided (Computing foaming flows across scales: From breaking waves to microfluidics) describes domain width=100mm, but in the source code example the extent is 200mm, and there is no wall set in the width direction to limit it, so should the domain width be 200mm?

the extent in this case is the length of the domain, which is twice as large as its width

(3) Regarding the Clustering example, the supplementary materials describes the domain height=20 mm , while the extent in the source code example is 30 mm. We only find the centre of the box in the y-direction is 20 in the b.dat file. What should we understand by domain and extent?

here the largest size is the height

We have tried to simulate waterfall and Clustering in the example, if we use the settings of m=32 32 32 and np=4 (in computer A), there will be a situation where the liquid will appear in the upper part of the computational domain and the liquid above will fall into the lower part of the liquid and interfere with the lower part of the liquid, while with the number of grids you provide and increasing the number of np (in the server B) it will not appear. What is the reason for this. We have included pictures here so that you can better understand our problem. The A and B in the file name represents the two cases of running in the computer A and B respectively.

the grid size determines the proportions of the domain (by default m="192 128 192"), so if you want to run on a coarser grid, you should keep the same proportions (e.g. m="48 32 48")

3.Thanks again for the answers to our last questions. We have studied the traj_*.csv file and we understand that cl, p, vf, v, x, y, z are the colour value, pressure inside the bubble, volume fraction, velocity and bubble position respectively, but we don't quite understand what xx, xy, xz, yz, zz mean, what does each mean?

xx, xy and are the second moments of the volume fraction, which can be used to compute the gyration tensor and e.g. fit an ellipsoid to the bubble
they are computed here https://github.com/cselab/aphros/blob/master/src/util/hydro.ipp#L1100

Thank you for your response to our question, we have been able to run it successfully based on your instructions, but we still have some confusion in running it.

In the clustering example, we unset the x,z direction as the periodic boundary, which will result in the presence of water on the surrounding boundary and the water phase in the box will increase, why does this happen when the periodic boundary setting is turned off? Where does the water in this case come from? We have attached the relevant images, showing the states at 0.33s and 3.27s respectively.

About dumplist setting we find that we can set omm and k. What are these three respective quantities? We find the options about dumplist setting in the files base.conf and hydro_post.ipp. What other variables are available about dumplist besides that?
3、We want to simulate the bubble situation at the stern of the boat when it is moving in the water and the gas-liquid situation when an object like a paddle is inserted vertically into the water and rotated and stirred, can these two situations be simulated by this software setting( For example, by setting the boundary conditions, we find that we can set the rotatable wall in init_bc.h. Please give some setup instructions if it is possible)?
4、Can the calculation continue at the stopping point when aphros is stopped? How do I do this?
Looking forward to your reply again


Routing using GenEvent

Dear @slashdotdash , let me ask you why all the routing structure for handling commands and events, wouldn't be easier to use the GenEvent framework instead ?
thanks

GenEvent was not suitable for a few reasons.

For command dispatching in CQRS there must be exactly one registered handler for each command. Not having a handler, or more than one, configured is an error.
GenEvent doesn't guarantee delivery of events. If a subscriber process goes down, it will miss any events published until it restarts.
All GenEvent handlers run in a single process.

In comparison, each Commanded event handler has its own subscription to the event store. It acknowledges every event it receives and successfully processes. This allows it to resume in the event of a process crash, or catch up if it is lagging behind. It also allows new subscribers to be added and they can receive every event published from the event stream origin.

Got it, it's an elegant design :100:
For the reason 1, it's okay, the developer can pay attention if the system is small, and if the system is too big, a consistency test tool can track commands, events and errors, and alert if they are inconsistent with the CQRS rules, or if some command was lost and no event or feedback was emitted.  For the reason 2 and 3, seems that GenStage comes to solve them: http://elixir-lang.org/blog/2016/07/14/announcing-genstage/ . Please, let me know if makes sense.

GenStage could be useful for event handling. I've taken a look at it before. Its approach to broadcast is to wait for every subscriber. This may not be appropriate if you have independent event handlers that you don't want to be blocked by the slowest handler. Back pressure has already been implemented for Commanded's event handlers.

I think that GenStage producers can provide input for any consumer that requests demand.
Besides, there is an experimental PartitionDispatcher which makes a lot of sense for event sourcing.

For the record, I have read at least a couple of articles advising against using gen_event.

Yep, also read, seems we should avoid it.


Update to polymer 3.0.0-pre.8

Polymer version updated to 3.0.0-pre.8
template() methods in Polymer components updated to return an HTMLTemplateElement which is required since version 3.0-pre.4 (Reference)

Hi @manolakis, thank you so much for your PR. BTW this is related to #111.
I've just checked the latest version of the pre and I think they published a new pre.9 just yesterday. Can you please update the version also for this PR?
Anyway I have tested your changes and I think we are good. Thank you again 👍

I really hope so! 🤞


Use a grammar for the input definition

Have you thought of using a grammar to describe the syntax? Would make coping with its evolution a lot cleaner.
I'm guessing you'd like to constrain it to be a regular language so that the syntax highlighting keeps working? Or is that a secondary consideration?

@jimdowning do you have any use for the language in something like BNF?

@ChristianDavis To be honest, I've dropped off from participation and my time available for helping with dev has gone too. I don't think I ever sold the main contribs on the benefits of a grammar to the architecture. In my mind I imagined keeping the canonical runtime data structure in an AST, and being able to generate both the graphical map from that and roundtrip the text encoding of the map from it.
My hope was that this would mean that map alterations could be added simply as AST manipulations, without needing to worry about graphical form and text form alterations at the same time. It would also make it necessary that anything you can express in the graphical form could be expressed in the text form, which I think is needed so that maps can be fully versioned and managed in git without necessarily needing the back end system of OWM.
When I was looking at this, I was leaning towards nearley.js as it seemed a bit more accessible than e.g. ANTLR.
In a nutshell, I can't make use of a BNF grammar, but you're very welcome to pick up and carry the flag for a grammar in this project.

@jimdowning There would be benefits from it. I've never seen nearley.js, that could be a nice in-between step.
@damonsk has done a great job of getting his idea out there and getting it working. Essentially rolled his own scannerless parser and it seems to work well. I had the same initial plan of getting in and generating an AST, but I've changed my mind a bit. There is non-regular syntax in the language ATM that helps simplify things, and @damonsk's heuristic approach doesn't mind. It's funny because I think if the project had started with a formal approach, it would have ran into roadblocks when stuff got complicated and lost steam.
When I lay out the total scope of his intended features the language is almost complete as-is. I argue for starting some formal language work whenever there is interest (or a champion for it) and keep that committed to the repo, but prioritize finishing the features of the language. I like your nearley.js suggestion because it could potentially be worked right into the existing parser to streamline the work there.
I worked on a compiler for a long time and I know that the formal language route is rarely the easiest one 😆

Similar issues #167 #99 #82 #39 #11 on the topic of syntax.
There has been some conversations in the community to attempt to standardise this, however not sure on progress.  As OWM is quite established, this would be a fundamental change to the tool and one that I'd want the community to have input on.


Allow specifying text for button

Instead of (or in addition to) add2calendarBtnTextMap, be able to pass in the text to display as part of the options in the constructor, what do you think?

Agree, will do.

Done - https://github.com/jojoee/add2calendar/commit/b7c15f14b02e9ea9bf03bca040fdfb6977cfe303


Add Simple Icons drawio library

Library repo: https://github.com/mondeja/simple-icons-drawio
Icons collection repo:  https://github.com/simple-icons/simple-icons

Who is the author, what is the copyright?

I'm the author of the library, the copyright is https://github.com/mondeja/simple-icons-drawio/blob/develop/LICENSE.md and the copyright of the icons library is https://github.com/simple-icons/simple-icons/blob/develop/LICENSE.md

in your library repo documentation, the following is stated:
Inside a drawio project, go to File -> Open library from -> Device and select the created file.
However, draw.io does not have a "File -> Open library from"
Your icons look tasty, but unfortunately I have no way to import them.

The button in your case is Open Library because you're using the Desktop version. In app.diagrams.net the button is Open Library From -> URL. I'll update the docs.

Thank you very much Álvaro, and keep up the good work.


python38Packages.elasticsearch: 7.15.1 -> 7.15.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.elasticsearch is: "Official low-level client for Elasticsearch"
meta.homepage for python38Packages.elasticsearch is: "https://github.com/elasticsearch/elasticsearch-py"
meta.changelog for python38Packages.elasticsearch is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 7.15.2 with grep in /nix/store/ya2j3dqg17gvzzigncs528lgr5b9sy29-python3.8-elasticsearch-7.15.2

found 7.15.2 in filename of file in /nix/store/ya2j3dqg17gvzzigncs528lgr5b9sy29-python3.8-elasticsearch-7.15.2

Rebuild report (if merged into master) (click to expand)

32 total rebuild path(s)

17 package rebuild(s)

17 x86_64-linux rebuild(s)
15 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
archivy
elasticsearch-curator
mailman-web
python38Packages.elasticsearch
python38Packages.elasticsearch-dsl
python38Packages.elasticsearchdsl
python38Packages.flask_elastic
python38Packages.hyperkitty
python38Packages.mailman-web
python38Packages.parsedmarc
python39Packages.elasticsearch
python39Packages.elasticsearch-dsl
python39Packages.elasticsearchdsl
python39Packages.flask_elastic
python39Packages.hyperkitty
python39Packages.mailman-web
python39Packages.parsedmarc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ya2j3dqg17gvzzigncs528lgr5b9sy29-python3.8-elasticsearch-7.15.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.elasticsearch https://github.com/r-ryantm/nixpkgs/archive/04b066581453250ea66a65cd968418b385d11025.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ya2j3dqg17gvzzigncs528lgr5b9sy29-python3.8-elasticsearch-7.15.2
ls -la /nix/store/ya2j3dqg17gvzzigncs528lgr5b9sy29-python3.8-elasticsearch-7.15.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

14 packages built:

archivy
elasticsearch-curator
mailman-web (python39Packages.mailman-web)
python38Packages.elasticsearch
python38Packages.elasticsearch-dsl
python38Packages.flask_elastic
python38Packages.hyperkitty
python38Packages.mailman-web
python38Packages.parsedmarc
python39Packages.elasticsearch
python39Packages.elasticsearch-dsl
python39Packages.flask_elastic
python39Packages.hyperkitty
python39Packages.parsedmarc

Maintainer pings
cc @desiderius for testing.

@GrahamcOfBorg eval


workbench.colorCustomizations stopped working in insiders

VSCode Version: 1.32.0-insider commit 393b48d4a5de85b6fa1d2ad0a59588d6715e12ea
OS Version: CENTOS 7
Electron: 3.1.3
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Linux x64 3.10.0-957.1.3.el7.x86_64

Steps to Reproduce:

Add color theme customizations to vscode workspace settins
Launch code-insiders

Does this issue occur when all extensions are disabled?: Yes
This currently works in normal VSCode on my OS. But it is broken on the insiders version

I just tried with the latest insiders and it works for me.
Make sure your settings file doesn't have invalid syntax, or that you have not defined colorCustomizations in the workspace or workspace folder settings.

@aeschli thanks for the reply. But the same settings work correctly on regular vscode. So that leads me to believe there aren't any issues with syntax errors. My settings file was fine. I updated insiders, and then it was broken.

Tried again, all works for me. I need a reproducible case. Maybe you send me the workspace settings file?

Here is my settings.json file for my workspace settings.
{
  "workbench.colorCustomizations": {
    "titleBar.activeBackground": "#09f",
    "titleBar.inactiveBackground": "#09f",
    "titleBar.inactiveForeground": "#ffffff",
    "titleBar.activeForeground": "#ffffff"
  },
  "tslint.autoFixOnSave": false
}

This is code-insiders

This is normal vscode

@jeffshaver Do you have the window.titleBarStyle on insiders set to native by chance? Looking at your two images the title bar rendering with the menu is clearly different with the menu inside on the the normal vscode and the menu below in the code-insiders.
This looks very similar to issues noted by Peacock when dealing with coloring the title bar.


Add interface to compare byte-sized data as decimals

I noticed that CppUTest currently makes no provision for comparing byte-sized values as numbers. In my experience, in embedded development, sint8_t and uint8_t (or equivalent) are more frequently used than 'characters' in the true sense of the word.
This is an attempt at providing functionality for comparing byte-sized numbers. I don't quite like it, but we are already comparing char as characters, so an overload wouldn't do.
There should actually be methods for unsigned char, as well, but I thought it might be good to discuss this first, as is.

Coverage increased (+0.0007%) to 99.874% when pulling 70b1ce4863fb87e8a5d64984d595fac3e723bc97 on arstrube:chars into c92072393011f596cd3dcd003e5388520c36f87b on cpputest:master.

Coverage increased (+0.0007%) to 99.874% when pulling b2697c3970180229d4c3ac3cffa248a4a58ead93 on arstrube:chars into 7ea6f14e2e984d03d4911c65402b30bedc1fd487 on cpputest:master.

Whats the difference between this and BYTES_EQUAL?

BYTES_EQUAL compares bytes as characters, e.g. 'a' or '¶'.
CHARS_EQUAL

Why not have BYTES equal print the value, the hex and the character?

Why indeed not :)
Sound like a much better solution to me.

My code doesn't do unsigned char correctly though. Will need to fix that as well.

Actually, I got confused. Sorry about that. BYTES_EQUAL does unsigned. That's the problem.
The solution I'm trying to find is being able to do both signed and unsigned comparisons properly for byte-sized data.

@basvodde pls have another look. How could this still be improved?

Coverage decreased (-0.05%) to 99.82% when pulling 34007b743a9003d521cd5b345122af44e95e3116 on arstrube:chars into 7ea6f14e2e984d03d4911c65402b30bedc1fd487 on cpputest:master.

Coverage decreased (-0.05%) to 99.82% when pulling a01f1747ed08dd4beec5583afde1259d97afdbc3 on arstrube:chars into 7ea6f14e2e984d03d4911c65402b30bedc1fd487 on cpputest:master.

Coverage decreased (-0.1%) to 99.767% when pulling 302899ceba4901f5436e0e7536b21799ccd89fe5 on arstrube:chars into 7ea6f14e2e984d03d4911c65402b30bedc1fd487 on cpputest:master.

Coverage decreased (-0.05%) to 99.82% when pulling 34007b743a9003d521cd5b345122af44e95e3116 on arstrube:chars into 7ea6f14e2e984d03d4911c65402b30bedc1fd487 on cpputest:master.

Coverage increased (+0.0007%) to 99.874% when pulling 5d9f39b401e24e67ed3fee2c34c00a82508f4a86 on arstrube:chars into 7ea6f14e2e984d03d4911c65402b30bedc1fd487 on cpputest:master.

I got some things mixed up in my comments. The current situation (before this PR) is:

StringFrom(char) will only print literal characters
BYTES_EQUAL will compare unsigned char, so signed values will be displayed wrong
The C interface will compare char as literal characters
Signed 8 bit data cannot be compared with proper failure messages

Normally, we have macros like "LONGS_EQUAL" vs. "UNSIGNED_LONGS_EQUAL.
Here, we have BYTES_EQUAL vs. SIGNED_BYTES_EQUAL, because BYTES_EQUAL was there first, and it does unsigned.
The C interface has two new macros: CHECK_EQUAL_C_UBYTE and CHECK_EQUAL_C_SBYTE. The C interface doesn't mirror all of the new macros we have for C++.

Uhm, but now the output is not "num hex char" like mentioned?

True. But it never included char, only in the C interface. So, if we do that, it will be like a new feature:

do it only for the C interface? How?
do it for BYTES_EQUAL (doesn't real make sense, it's not CHARS_EUAL)
doesn't make sense for SIGNED_BYTES_EQUAL at all (this is numeric by definition

I suggest to give it some more thought and, perhaps, add it later. It would make sense for the C macro though, in which case there would be no new macros.

Uhm, well to me it is just 3 different representations of the same information. As sometimes showing it in a different representation will trigger an AH!, I think it is useful. I would do it for all of them, or perhaps only when it is in the printable ascii range :)

Hmm. So SIGNED_BYTES_EQUAL and BYTES_EQUAL would then display:
expected <-3 0xfd 'X'>
but was  <-2 0xfe 'Y'>

expected<255 0xff 'Z'>
but was  <253 0xfe 'Y'>

where X, Y, and Z would be the actual screen representation of that character. Similar for the C interface (will take a look next week).

Yes, and then we could check whether the character is printable or not, and not print it if it isn't.

I don't have much time right now, and if I do add the character, I will need to change the tests.
Since I want to test negative numbers and 'large' unsigned bytes, I would have to limit printing to printable characters for reasonable tests.
Can we perhaps merge this 'as is', and add printing the character later? (I can add an issue for that).

Coverage increased (+0.0007%) to 99.874% when pulling dd6902fd53368ad6b860ba4854bf85473543a04d on arstrube:chars into a032b73820470c8a8a773fc7ba6656551a786eda on cpputest:master.

Coverage increased (+0.0007%) to 99.874% when pulling dd6902fd53368ad6b860ba4854bf85473543a04d on arstrube:chars into a032b73820470c8a8a773fc7ba6656551a786eda on cpputest:master.

If I display the character along with numeric data, then
   CHECK_EQUAL_C_CHAR()

which displays a literal character, would be redundant.
   CHECK_EQUAL_C_UBYTE()
   CHECK_EQUAL_C_SBYTE()

would handle unsigned char and signed char, respectively. But we need to keep CHECK_EQUAL_C_CHAR(), because it's always been there.
Should we then do:
    CHECK_EQUAL_C_CHAR() // does signed char
    CHECK_EQUAL_C_BYTE() // does unsigned char

or leave all three (with CHECK_EQUAL_C_UBYTE() giving identical output to CHECK_EQUAL_C_CHAR()?


$firebaseArray data not yet available on $loaded.then() on 1.1.1

The following code will not work in AngularFire 1.1.1:
$scope.numbers = $firebaseArray($fbRef.child('numbers'));
$scope.numbers.$loaded().then(function() {
  $scope.numbers.$keyAt(0); // does not work
  $scope.numbers.$keyAt(1); // does not work
  console.log($scope.numbers); // logs FB array with multiple elements, as expected
});

$getRecord() and $indexFor() do not work either. When I rollback to 1.0.0, I can access by $scope.numbers[0] etc., which is working for me for now.

I should mention that the following doesn't work either:
$scope.numbers = $firebaseArray($fbRef.child('numbers'));
$scope.numbers.$loaded().then(function(data) {
data.$keyAt(0); // does not work
data.$keyAt(1); // does not work
});

What version of the Firebase SDK and Angular are we working with here?

Angular: 1.2.x
Firebase: Latest (2.2.5)

Just to humor us, can you try using the latest Angular 1.3.x and see if your problem persists?

I was unable to repro this in 1.3.3 using this code:
    var app = angular.module('app', ['firebase']);

    app.controller('ctrl', DefaultController);

    function DefaultController($scope, $firebaseArray, $firebaseObject, $firebaseAuth) {
      var fb = new Firebase('https://kato-sandbox.firebaseio.com/books');
        $scope.list = $firebaseArray(fb);
        $scope.list.$loaded().then(function() {
          console.log('keyAt', $scope.list.$keyAt(0));
          console.log($scope.list.length);
        });
    }

The output:
keyAt book1
3

That seems to do the trick!! Apparently Angular 1.2 is the problem, it works on 1.3.15 :) Thanks much for your help!

Glad to help. Thanks for submitting an issue.

I am experiencing an issue when i extend a firebaseArray, if use directly, it works as expected. Tried to make into small application below.
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>plunkr: [http://plnkr.co/edit/jFpxUk]

// declare a module
var myAppModule = angular.module('myApp', ['firebase']);

myAppModule.controller('GreetingController', ['$scope', '$firebaseArray', '$q', function($scope, $firebaseArray, $q) {
  var url = "https://todomvc-angular.firebaseio.com/todos";
  var TestExtendArray = $firebaseArray.$extend({
		$$added: function (snap, prevChild) {
			var self = this;

			return $q(function(resolve, reject) {
				var record = $firebaseArray.prototype.$$added.call(self, snap, prevChild);
				resolve(record);
			});
		}
	});

  // THIS DOESN'T WORK
	$scope.list = TestExtendArray(new Firebase(url).limitToFirst(100));
	$scope.list.$loaded().then(function() {
		console.log('keyAt', $scope.list.$keyAt(0));
		console.log('$$ilst', $scope.list.length);
	});

	// THIS WORKS
// 	$scope.list = $firebaseArray(new Firebase(url).limitToFirst(100));
// 	$scope.list.$loaded().then(function() {
// 		console.log('keyAt_1', $scope.list.$keyAt(0));
// 		console.log('$$ilst_1', $scope.list.length);
// 	});
}]);

When i extend the array the loaded() appears to fire before the data is actually loaded. Maybe i am just doing something wrong. Any ideas?

@dreadjr please submit a new issue as this one has already been resolved. Make sure, as with the original issue, that yours is not also a result of using 1.2.x of Angular.

sorry didn't noticed this was closed. Created a new issue #629


HDDS-2600. Move chaos test to org.apache.hadoop.ozone.chaos package.

What changes were proposed in this pull request?
This is a simple refactoring change where all the chaos test are moved to org.apache.hadoop.ozone.chaos package
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2600
How was this patch tested?
Simple refactoring change.

@mukul1987 Please review.


bug When I unzip the zip file and upload it to chrome

Describe the bug
bug When I unzip the zip file and upload it to chrome
To Reproduce
Steps to reproduce the behavior, or file the issue is found in:
open File C:\Program Files\Subtitle Easy Error Manifest file is missing or unreadable
Or when I go through the dict folder and apply them to coursera, the subtitles don't work knowing that there are 2 folders macos and dict
image:

Based on your screenshot and description, it seems that you are using our plugin on Coursera videos. Currently, our plugin only supports Apple Developer and deeplearning.ai, and it hasn't fully supported Coursera yet. However, I'm glad to inform you that we have plans to add support for Coursera within the next two weeks.
In the meantime, I would suggest trying out a Chrome extension called Dualsub, which already supports Coursera. You can give it a try to enhance your experience on Coursera. I hope this suggestion helps! If you have any other questions or need further assistance, please feel free to let me know.


Open all new users in new tabs with click of a button

Description:
Many people like to give gifts to new members. Clicking a single button to open all 10 of the newest users shouldn't be hard.
Implementation:

[ ] Get all (unvisited?) new user links from list of newest users
[ ] Open links in new tab
[ ] Optionally/additionally automatically send gift instead of opening new tab

If user data entry points are added:

[ ] Get 10 most recent users
[ ] Somehow determine if those users are known (remember IDs for 24hrs?)
[ ] Open tabs / give gifts

What is the end-goal of this feature? Is it for gift-giving?
I was thinking of a feature for the Home page that would give gifts to all 10 New Members at click of a button. Should be fairly easy to accomplish with the #76 code reuse.
Depending on intent of this feature, it may be accomplished with a button to gift instead of a bunch of tabs?

It is, as many people give gifts to all users.
However, I don't always like to give to all users, and will pick and choose based on who has been active in their first hours.
There are definitely people who would like an auto-gift-to-all option though. I'm not sure what the ratio is for "People Who Open All New Users" to "People Who Gift All New Users" is.

If we could get JSON endpoint for the User Search feature (to get all users arrived in X days) and then get JSON endpoint into the User Info page for individual users, we could really build something nice to meet both needs. Not sure what they already have for endpoints though...

There is some work being done on user endpoints for some other features (ex. #22). I'll be asking about those endpoints in IRC eventually and can ask about this as well.

I started working on this because I thought the forum solution was going to take a lot longer. I have some things worked out, but essentially am looking for some design advice right now. I want to use the 10 newest members of the Home Page for this implementation until some sort of endpoints are created. I wanted the option to Gift All or Gift One at a time. The buttons for one at a time are not very aesthetic. I have two options:

Dynamically show the gift amount if valid on the button text

Simply show a "$", or the word "Gift" but risk people not understanding that the text box value applies to each person.

Any thoughts or preference?

Got a lucky break and found a funny bug in my code. If the person's name link shows up elsewhere in the home page, the Util.CreateButton I used to look for the href reference finds the wrong one and adds the gift button to wrong spot! :)

Think I got it figured out how to get around it though.

Haha that's a pretty great bug :) Unexpected, obvious, and easy to fix.

Any thoughts or preference?

My initial thoughts would be to drop the individual buttons. The bulk options at the bottom should cover 90% of the use cases; people who want to donate to individual mice will either Open All In Tabs (to check levels of activity, for example) or click on the individual they wish to gift.
If you think adding a gift amount to each individual is worth it, I would say mess around with the CSS styling. This is admittedly tricky, as the script only ever uses the CSS hosted in the repo; you have to temporarily make the changes in the browser dev tools or in the site's "User CSS" setting. Using flexbox/grid to make a column of $'s would probably look decent.

You're probably right, and I have no idea how to find success in CSS. I will go ahead and delete the code for the individual gift buttons.
Looks like there is a built in limiter for gift giving that blocks me from giving gifts to all 10 new users at once. Need to fine tune an appropriate delay and find a loading.gif to use while processing.

Unfortunately fine-tuning the timing means I have to wait for new members to actually join the site since I have already gifted them once... like watching paint dry

I'd recommend making a throttle function in the Util page, as there might be additional use cases for throttling a function. The simplest thing that comes to mind would be a sleep function. Then you can just await Util.sleep() to pause the code, even inside simple loops.

I have a pretty simple function i am using. I will definitely add it to Util. Just finding the milliseconds that works for the timeout. 3seconds too long... 1,1.5, and 2 are all too slow. Going to try 2.5 next... then back to 3 if it does not work.
Another question, I want to make my own GM variable and am setting it to GM_SetValue("mp_newVar", "string"), but found out the hard way that Saving and Settings in MAM+ overwrites my adhoc variable. Where would I add it or exempt it in the code to ensure it sticks around during Preference Changes?

GM values should persist as long as they're unique... You can console log GM_listValues() to see what values are being stored. As long as you're not using "mp_version" or "mp_currentPage" you shouldn't be conflicting.

GM_SetValue("mp_newVar", "string")

Just be sure this is GM_setValue with a lowercase in your code. Same for GM_getValue.

Oops, yes setValue. It's working just fine in execution. I am saving and retrieve and editing and saving as expected. But then I changed my my default points to gift user value, saved, and all of a sudden the variable seemed to have vanished. I'll continue experimenting to see what's happening, just assumed maybe the settings were being wholesale overwritten on update and not accounting for variables not included in preference page.

Unless there's something wrong with your Tampermonkey installation (a possibility) or you're explicitly calling GM_deleteValue, the values should stick around eternally. Not even sure if clearing browser cache/data deletes them, but maybe.

So, its not deleting my variable when saving settings, its overwriting the value to False in the assumption that it will later set the value again based on the GUI fields. Since my ad hoc variable is not in the settings page, it gets wiped with False and is lost for good.
// Loop over all values stored in GM and reset everything
        for (const feature in gmValues) {
			console.log(typeof gmValues[feature] +"  " +gmValues[feature]);
            if (typeof gmValues[feature] !== 'function') {
                // Only loop over values that are feature settings
                if (!['mp_version', 'style_theme'].includes(gmValues[feature])) {
                    GM_setValue(gmValues[feature], false);
                }
            }
        }

Im thinking there needs to be a check to see if the setting is found in the GUI using a getElementByID and if its null/not found, then skip. If found, then set as False to "reset". Not sure why the setting as False is even needed if its all being setValue later, but you would know that better.

Ah, I think you're right. The Settings class is a horrifying monster that works, but in a very inefficient way. Overriding all settings in order to save the updated settings is not how you should do things, but it's what I was able to make work.
Barring a complete re-write of the _saveSettings logic, I'd say style_theme probably should have been named as a mp_ setting, and the loop should exclude all of those (as they are never features).
if (gmValues[feature].indexOf('mp_') < 0){}

or something.

For now, I am naming any of my variables not directly related to the Preferences page as stor_ prefix and added a lined to exclude any stor_ items from setting false. I didnt want to fiddle too much with the settings as they exist. I will continue to use that as needed until there is a rework of the logic. Afterall, you may look at my solution and think its terrible and needs re-write anyway :)
// Loop over all values stored in GM and reset everything
        for (const feature in gmValues) {
			console.log(typeof gmValues[feature] +"  " +gmValues[feature]);
            if (typeof gmValues[feature] !== 'function') {
                // Only loop over values that are feature settings
                if (!['mp_version', 'style_theme'].includes(gmValues[feature])) {
					if(gmValues[feature].indexOf("stor_") !== 0){
						GM_setValue(gmValues[feature], false);
					}
                }
            }
        }

Getting close to finished on this one I think. So open up a Develop branch if you dont have one already.

Afterall, you may look at my solution and think its terrible and needs re-write anyway :)

There are few things worse than the current Settings implementation so I think you're safe :P

I did end up making a few changes to this.

Changed stor_ values to mp_ as I want to limit the number of variations for now. Features have nothing prefixed, internal values have mp_.
Added a firstRun boolean to the gifting section that toggles itself on the first run and delays after that.
Removed the empty checkboxes, as they look like "missing unicode" symbols and I don't think they're strictly necessary.
Because of the above, I had to change how the script decides who & how to gift. It now adds/looks for an mp_gifted class. It also stores the user IDs instead of the usernames, so the names can be modified however you want without causing issues.
Cleaned up a few promises to be easier to read. Primarily, removing Promise.all() where not needed and storing a returned await in a variable instead.
I realized that there is no actual Home module currently, so I created that. All Shout feature run on Home, but not all Home feature run on Shout. Maybe there will be more Home features in the future.

Nice!
I'm still learning promises so just basically copying how I've seen it and trying to adapt. Thanks for cleaning that up, I'll try to look at how you implemented and learn from it.
Other changes make sense as well. I didn't really love the empty checkboxes either, but wasn't really thinking outside the box on how to adapt my code to fit not having them. This one turned out to be a lot of rewrites and changed more than I thought it would.


Spinner selection dialog not closed when we moved to next screen

In my app i have 3 screen .First screen i am implement searchable spinner .Then search some keywords in popup window .shows some searchable result.Then i am not selected any item .Popup still showing.Then i moved to second screen,but spinner popup showing there. Not closed automatically. How to manage it?
Please help.

Use the property   mSpinner.hideEdit();  when advance screen.
If you open Spinner and use the button onback, use hideEdit() in onDestroyView.