Preview version of C# language uses incorrect constructor overloading for arrays

Thanks for the great library!

First-class Span Types is currently under development. Recent changes, in particular Prefer converting T[] to ReadOnlySpan over Span in overload resolution, also affect the library.
See the following code for an example:
using System;
using System.Buffers;

class Program {
    static void Main()
    {
        S(ArrayPool<char>.Shared.Rent(16));
        S("abc");
    }

    static void S(Span<char> _) => Console.WriteLine("Span");
    static void S(ReadOnlySpan<char> _) => Console.WriteLine("ReadOnlySpan");
}

Old behavior vs New behavior
That is, by default, it's now easy to get into an invalid constructor overload when passing a buffer from an array. This is not something that will happen sometime in the future, to get problems you just need to enable the preview version of the language for .NET 9. For example, interesting issues: 1, 2

To resolve this issue, all you need to do now is add the OverloadResolutionPriority attribute
using System;
using System.Buffers;
using System.Runtime.CompilerServices;

class Program {
    static void Main()
    {
        S(ArrayPool<char>.Shared.Rent(16));
        S("abc");
    }

    [OverloadResolutionPriority(1)]
    static void S(Span<char> _) => Console.WriteLine("Span");
    static void S(ReadOnlySpan<char> _) => Console.WriteLine("ReadOnlySpan");
}

Demo

It might be more correct to remove overloading with ReadOnlySpan<char>, for example by adding a static method to create a ValueStringBuilder from a given buffer. For example, .NET without it.

Ohhh that is a very very good point and thanks for the kind words! Thanks for bringing this up @nsentinel

It might be more correct to remove overloading with ReadOnlySpan

Maybe - it would be a breaking change for sure, so nothing I would consider for v1. If we move the ReadOnlySpan version to a static method, I fear that we have (maybe???) less discoverability of the API. I have to make myself some thoughts. But I am glad that you brought this up.

Thanks for the quick reply, and I'm glad you found my comments interesting and helpful!

About removing the overloading from ReadOnlySpan: yes, I realize that this will be a definite breaking change for everyone, which is usually something we would like to avoid. I mentioned it specifically in this case because the behavior changes dramatically depending on the argument type, which I think is generally unusual, but since ValueStringBuilder users are unusual people then it's probably not that big of a deal, I'll leave it up to you.
I agree that it is less noticeable, but you can always just call Append right after constructing with the initial value (exactly what is done in the constructor overload), and those who don't want to can use the special constructor option via static method.
Probably the best option from the current user's point of view would be to really just add the attribute and leave it as is, since this solves the problem for everyone and doesn't break anything.  And the nuance of perceiving differences in constructor behavior seems more philosophical to me.

Probably the best option from the current user's point of view would be to really just add the attribute and leave it as is, since this solves the problem for everyone and doesn't break anything. And the nuance of perceiving differences in constructor behavior seems more philosophical to me.

Indeed - for now that might be the most straightforward solution! I will put this "up for grabs" for folks.


getConversationDetails is not working anymore - returning undefined values

Hello,
since the 5th of December the getConversationDetails method is returning an object with conversationId and state undefined.

Sdk version is 1.0.0.
Thank you.

Is this issue still occurring?


Partially covered lines should appear in yellow with a tooltip X of Y branches missed

Currently partially covered lines appear in green.
Green is reserved for full coverage.
Like in jacoco report, it should show partially covered lines in yellow and a tooltip indicating "X of Y branches missed".

Thanks for your report.
I have a pull request I'm working on for partial coverage. It's currently not supported but I lost steam and I haven't made progress on it lately. But if there's interest shown I can consider working on it again.

That's the PR: https://github.com/aconrad/pycobertura/pull/168
If I recall the code supports partial coverage but I still need to update the renderers to make them aware of partial coverage. I think that's the main piece left.

I would also find this feature useful, FWIW.


rewrite fillna3D using pandas

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have added an explanation of what your changes do and why you'd like us to include them.
[ ] I have written new tests for your core changes, as applicable.
[ ] I have successfully ran tests with your changes locally.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Great @leiyangleon, looking good! Go ahead and update the test_scenario files so that those two unit tests pass and then we can merge this in. Thanks!

Great @leiyangleon, looking good! Go ahead and update the test_scenario files so that those two unit tests pass and then we can merge this in. Thanks!

@jlmaurer I updated the two unit test files and CI build was successful. However, the coverage test failed, which I have no idea what happened and how to track the errors by clicking on the link for details. Could you help?

@leiyangleon no worries for the coverage report. It fails when the coverage of the code decreases. It says that new code was added but the current test-suite does not not test all of the new code changes.

@dbekaert this is quite confusing to me; I checked the coverage failure but the only changed file is 100% covered. Also it is not letting me merge the PR while the coverage test is failing.

You requested changes, which needs to be addressed before you can merge ;).
I overruled and merged it.


MVN repository don't have the latest version (WSO2 API Manager Integration New Test Module)

Description
Could not found the v4.2.0 under:
https://mvnrepository.com/artifact/org.wso2.am/org.wso2.carbon.am.integration.test
Still only 4.2.0-beta now.
Steps to Reproduce
Git clone tag v4.2.0 from APIM.
MVN clean install.
Affected Component
APIM
Version
4.2.0
Environment Details (with versions)
4.2.0
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Hi,
We are in the process of releasing nexus artifact from "staging". Once that is done, artifacts will be available in public maven repo.
But this doesn't have an impact on building the product-apim repo [1] locally since the above artifacts is built locally and added to local m2 repo at build time.
If you are using components built in product-apim repo(i.e org.wso2.carbon.am.integration.test), to build some of your other project repos, you may checkout the 4.2.0 tag of product-apim repo and build it first before building your repo locally.
[1] https://github.com/wso2/product-apim/
Thanks,
Samitha

I got the "Could not resolve dependencies for project org.wso2.am:org.wso2.carbon.am.integration.benchmark.test:jar:4.2.0: was not found. It was because the pom.xml inside the integration benchmark test refering to "4.2.0" while in the public repository only have the "4.2.0.beta" and that's why mvn build is not successful.
I don't know how you can build using "mvn clean install -Dmaven.test.skip=true" , can you show me detail steps? Thanks.

The mvn repository was updated to v4.2.0, issue resolved.


FAILED_PRECONDITION: DNN library initialization failed.

Description:
I'm encountering an error when attempting to run the sample Policy RL script. It seems to be related to utilizing CUDA on a GPU. The error message indicates a failure to initialize the DNN library.
UnfilteredStackTrace: jaxlib.xla_extension.XlaRuntimeError: FAILED_PRECONDITION:
DNN library initialization failed. Look at the errors above for more details.
Environment:
OS: Ubuntu 22.04
CUDA Version: 11.8
JAX Version: 0.4.13
JAXlib Version: 0.4.13+cuda11.cudnn86
Python Version: Python 3.8.19
Steps to set up

I created a conda environment and installed the Cuda 11.8 toolkit using conda install cuda -c nvidia/label/cuda-11.8.0
Then I install PyTorch using pip install torch==2.0.0 torchvision==0.15.1 torchaudio==2.0.1 --index-url https://download.pytorch.org/whl/cu118
Then I install jax with GPU support for Cuda 11 using pip install --upgrade "jax[cuda11_pip]" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html
Then, I install the repo dependencies pip install -r requirements.txt

Attempts to narrow issue

I verified that the GPU is accessible using torch.cuda.is_available(): returns True.
I verified that jax can detect the the GPU using 'import  jax' and 'jax.devices': returns [gpu(id=0)]

Stack Trace
...

Hey, I literally have the exact same problem and I was unable to solve it for days so if you find or get any solution please post it here. I dont know if you have tried this but what I found out is that jax is trying to use a lot of VRAM when running the first operation so you can try for yourself if you have any luck with limiting the allocation by either doing "export XLA_PYTHON_CLIENT_PREALLOCATE=false" to completely stop it from preallocating or "export XLA_PYTHON_CLIENT_MEM_FRACTION=.XX" with XX being for example 50 to give it like 50% of the VRAM but in my case it did not solve the issue but maybe you have more luck. Besides that im now trying to make stuff run on python 3.9 since jax works on my python 3.9 environment. But this is really painful and doesnt seem to be easy. Im afraid nobody will really solve this issue here but i can suggest you to look into the JAX github and if you find a solution to it there within the issues list please tell me cause i was unable to find a solution to it yet and i even posted an issue there myself but nobody answers it so maybe you have more luck on that. Sorry thats all i can give you

At ICML this week so haven't had much time to look at issues. This is not a Torch library, installing both Jax and Torch is maybe causing some strange dependency issues. Additionally, the error:

DNN library initialization failed. Look at the errors above for more details.

Looks like a cuDNN error to me. It doesn't seem that Jax can find it on your system. The instructions for installing CUDA in the README are outdated. Try the recommended install: pip install -U "jax[cuda12]" or pip install -U "jax[cuda11]" depending on your version of CUDA.

At ICML this week so haven't had much time to look at issues. This is not a Torch library, installing both Jax and Torch is maybe causing some strange dependency issues. Additionally, the error:

DNN library initialization failed. Look at the errors above for more details.

Looks like a cuDNN error to me. It doesn't seem that Jax can find it on your system. The instructions for installing CUDA in the README are outdated. Try the recommended install: pip install -U "jax[cuda12]" or pip install -U "jax[cuda11]" depending on your version of CUDA.

Hey I can already tell you that this is an issue with JAX where it runs into problems when trying the cudnn initialization but interestingly when im trying to use jax with any python version higher than 3.8 it works completely fine its just not working within Python 3.8 for some reason. Unfortunately the issue i posted about this in the JAX repo is being completely ignored and there seems to be no real solution provided yet unless i just did not find it. I think in Python 3.8 the installation was a little different as I think i ran into some issue that the extra "cuda12" does not exist so the installation I had to go through looked something like this: pip install jax==0.4.13 jaxlib==0.4.13+cuda12.cudnn89 -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html which basically installs jaxlib with cuda12 and cudnn 8.9 support but this ran into the issue mentioned by OP. With the recommended commands from you you get this problem: "WARNING: jax 0.4.13 does not provide the extra 'cuda12'" so this is not an option as the extra does not exist for the latest jax version usable with Python 3.8. I would be really thankful for any further help with this as I can not get it to run. Maybe you could create a new conda environment yourself to test it if that is possible. Its unfortunately really important for my work that this works but i cannot get it to work. Im currently even trying to make the code run with Python 3.9 because jax works there but thats connected to a lot of issues i am trying to fix so its not looking too good.

Small update: there is no way to use Python 3.9 because the advantage of having higher versions of jax brings incompatibilities with functions that were removed by higher jax version so i guess that would only be possible to fix by doing some major changes in the viper source code.

Thank you for your comments @Alescontrela @NonsansWD. I managed to solve the problem on my end, but I'm not sure if the solution will apply to you. It's worth giving a try. I set up the environment as I mention above. If you run pip show jaxlib you should get an output specifying "Version: 0.4.13+cuda11.cudnn86". I was installing cudnn incorrectly. I had to replace /usr/local/cuda/ with a different path (I needed to install on a supercloud and within a conda environment). If you're installing this locally, the following should resolve the issue.

Download cuDNN 8.6.0 from https://developer.nvidia.com/rdp/cudnn-archive for your os
Follow installation instructions here: https://docs.nvidia.com/deeplearning/cudnn/archives/cudnn-860/install-guide/index.html

If you want to install it within a conda environment, you can identify your cuda path using which nvcc. The output will be something like your_cuda_path/some_conda_environment/bin/nvcc. So you would replace /usr/local/cuda/ with your_cuda_path/some_conda_environment.
Then export the path to the conda environment to your LD_LIBRARY_PATH. I hope that helps!

Thank you for your comments @Alescontrela @NonsansWD. I managed to solve the problem on my end, but I'm not sure if the solution will apply to you. It's worth giving a try. I set up the environment as I mention above. If you run pip show jaxlib you should get an output specifying "Version: 0.4.13+cuda11.cudnn86". I was installing cudnn incorrectly. I had to replace /usr/local/cuda/ with a different path (I needed to install on a supercloud and within a conda environment). If you're installing this locally, the following should resolve the issue.

Download cuDNN 8.6.0 from https://developer.nvidia.com/rdp/cudnn-archive for your os
Follow installation instructions here: https://docs.nvidia.com/deeplearning/cudnn/archives/cudnn-860/install-guide/index.html

If you want to install it within a conda environment, you can identify your cuda path using which nvcc. The output will be something like your_cuda_path/some_conda_environment/bin/nvcc. So you would replace /usr/local/cuda/ with your_cuda_path/some_conda_environment.
Then export the path to the conda environment to your LD_LIBRARY_PATH. I hope that helps!

You won't believe it but before looking at this comment I managed to solve it too the exact same way xD. For me it was a little different because I built the matching cudnn version on arch but that way it worked. It's also fine to use cudnn 8.9 with cuda 12 as thats what im using now and on another system I'm using cudnn 8.6 with cuda 11 now and it works there too so the only problem is basically the major version update to 9.x for cudnn which cant be supported as jax cannot support it. Also got that hint from a recent reply from today on my issue on the jax github and for the pointers i already have Symlinks in place and bashrc prepared so it works with any environment out of the box but thank you very much for sharing this and I hope anyone who stumbles upon this issue will see our comments and either of those approaches will work for them too. Have a nice time running the code. I hope it wont take too long cause it wants to do like a million steps and starts at 40,000 :D

Thank you for your comments @Alescontrela @NonsansWD. I managed to solve the problem on my end, but I'm not sure if the solution will apply to you. It's worth giving a try. I set up the environment as I mention above. If you run pip show jaxlib you should get an output specifying "Version: 0.4.13+cuda11.cudnn86". I was installing cudnn incorrectly. I had to replace /usr/local/cuda/ with a different path (I needed to install on a supercloud and within a conda environment). If you're installing this locally, the following should resolve the issue.

Download cuDNN 8.6.0 from https://developer.nvidia.com/rdp/cudnn-archive for your os
Follow installation instructions here: https://docs.nvidia.com/deeplearning/cudnn/archives/cudnn-860/install-guide/index.html

If you want to install it within a conda environment, you can identify your cuda path using which nvcc. The output will be something like your_cuda_path/some_conda_environment/bin/nvcc. So you would replace /usr/local/cuda/ with your_cuda_path/some_conda_environment.
Then export the path to the conda environment to your LD_LIBRARY_PATH. I hope that helps!

Quick question tho, Since i fixed it it constantly ran and after 5 days i got to like 380k do you experience something similar or is it signifcantly faster for you? Its still way faster than doing it on cpu but it still takes a long time. Also are the loss curves and images also as bad as mine as described in my recently posted issue?


Update README.md

Added language tag to code snippets.

Thank you @Minitour for you contribution!

@all-contributors please add @Minitour for docs


(3.0.87) Extra click after logout to reenable the login button

Login to Jadeite
Logout
Login button is disabled.
Click on Login window somewhere.
Login button becomes enabled.

Need to eliminate need for extra click.

Fixed in 3.0.88

About one time in 10 or so, the login button doesn't re-enable by itself. However, I haven't yet been able to make it happen on purpose. When I try to setup complicated sets of windows to trigger the not-reenablement... it very reliably works correctly. So reopening, but leaving assigned to myself.

Took another look at this. Think I found a more deterministic fix which seems to work better. Needs further retesting.

Still occurs about the same frequency and equally only after some time of doing stuff and not reproducible.

Have an improved fix. The button's enablement seems to be influenced by what window has focus. If the login window does not have focus, logging out wouldn't enable the login button. Force the login window to have focus before invalidating the button.

Haven't seen this, I don't think, so (tentatively) agree it's fixed.


Do not realize all configurations in a project by default

Fixes #28192
Previously, unlike tasks, Configurations of a project are automatically realized, even if they are 'register'ed or even if the new role-based factory methods are used, which are intended to be lazy.
This happens for two reasons

The base plugin had a configurations.all call in it in order to implement behavior related to the assemble task
When resolving a dependency against a target project, we would previously realize all configurations in the target project, even if they were non consumable.

This commit makes two significant changes

We no longer call configurations.all in the base plugin, but rather lazily add the artifacts of all visible configurations to the assemble task. This prevents all configurations from being realized instantly if the base plugin is applied.

We no longer realize all configurations of a target project when resolving a dependency against it. We now only realize the configurations known to be consumable or with an unknown/mutable role.

We add a new test to WellBehavedPluginTest to ensure it does not add configurations.all calls or other calls to instantly realize configurations We also add a new test suite to verify dependency resolution only realizes consumable configurations or configurations with unknown an unknown role, but does not realize configurations with a known non-consumable type.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test apt

I've triggered the following builds:

All Performance Tests (Trigger)

Build Scan

The following builds have failed:

All Performance Tests (Trigger)

Build Scan

The following builds have failed:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test prf apt

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

All Performance Tests (Trigger)

Build Scan

The following builds have failed:

All Performance Tests (Trigger)

Build Scan

@bot-gradle test prf apt

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

All Performance Tests (Trigger)

Build Scan

The following builds have passed:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have passed:

All Performance Tests (Trigger)

Build Scan

@bot-gradle test this

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have passed:

Pull Request Feedback (Trigger) (Check)

Build Scan


LinefeedFixer - Fix in a safe way

The fixer should not touch all line breaks.

then we need a second fixer (risky one) that will fix all the rest...

then we need a second fixer (risky one) that will fix all the rest...

I think this is a bug fix, so do not need such a fixer for BC reasons. Do you think someone really wants it?

Do you think someone really wants it?

I know at least one: me!

me too .actually most of community. the only thing is that it's risky.
we have exactly the same problem with trailing spaces. PSR requires to not have them in non-empty lines - even if it's part of string var. but then it modifies a content, so it's risky. but still, most of us wanna follow that.

It's bugfix but also shrinking functionality. Now one cannot fix linefeed inside var content. could you prepare the new fixer as well please? it's really important for us ! :)

alrighty, would a config flag work as well or you really want a diff. fixer?

I was thinking about the config as well, but then you need to dynamically say if the fixer is risky or not depends on config.

I'm kind of happy with this PR as is, but if we need another fixer please provide names for both fixers and I'll update this PR :)

as discussed it could be an option. the only thing to keep in mind is risky vs not risky... depends on used options

OK, I treat it as bugfix after thinking about it. Missing functionality is raised here: #2224

:+1:

Thank you @SpacePossum.


SSR + customization + typing

Hi all,
I made a fork with some improvement.
I use this project on my PWA based on Next.js and typescript.
I made some modification to make it work with server side rendering and I add typing.
I also add props to enable developer to use their own icon library.
Are you interested on a PR?

I am unfortunately not able to maintain this project myself at this time.  I'm happy to hear that you're able to make use of it.  If you have the time to keep it moving, I'm willing to add you as a collaborator so that you can make some of these improvements yourself.

Perfect.
i'll pull request my changes
push a new npm version
make a pull request on @types repo
As soon as I have the rights ;)


change rack elevation template to use device.display_name property

Fixes:
The display_name property works better for virtual chassis members. For standalone unnamed devices though, it will show the device type name instead of the device role name. I'm not sure one of those is definitely better than the other. I hope this makes sense though!

Pull requests which alter functionality must be opened against an existing, approved issue.


Website and API monitors require HEAD  / Logging is lacking

Discovered that the website and API monitors require HEAD. I didn't see a mention of this anywhere. I have internal API's that only use GET and POST. If I do not enable HEAD in my API, OneUptime will make a HEAD request and get a 403. When checking the monitor logs, I see the 403 error but no mention of the REST method used. It is had logged that it failed with a HEAD request, I would have immediately known how to fix.
My suggestion:

add the REST method you use to logging
optionally - if HEAD method fails, try the GET or POST anyway.

Fixed here: https://github.com/OneUptime/oneuptime/commit/36746412244df96821713ca5a94384cc1e781afc


Variable resistance selection

Mainly updated https://github.com/ZBT-Tools/pemfc-tkinter-app repository, however the following callback needed some temporary rework to be able to include the activation/deactivation logic of connected entries for the new dropdowns:
https://github.com/ZBT-Tools/pemfc-dash-app/blob/1a7ea57aed3d1c2984af530562491a225682804f/pemfc_dash/callbacks.py#L1376

Seems not to work

did you update your pemfc and pemfc_gui packages?

yes.

--> After restart of Pycharm it works.


Identify and remove unused files

There are some files in the repo that I am unclear about if they are still needed or not, e.g. .landscape.yaml. Identify and remove unused files.

Hi @mariobehling, I'm new to open source and would love to contribute. Could I take on the task of identifying and removing unused files?


The adaptor example isn't working - via Twitter

A quick look through didn't surface the issue. Here's the repro: http://jsfiddle.net/ATAgH/28/

You used adaptors instead of adapt :P

Nice! I was looking on my phone and completely missed that. I thought it was a docs bug, but it looks like it's just a case of slightly confusing options.

Yep. That got me once before while going through adaptor docs.
And you're working on a fone? So hardcore! 🙇

@evs-chris Could you update the docs to link to the fixed jsfiddle on http://docs.ractivejs.org/0.8/writing-adaptor-plugins ?

@jasny done! I didn't even see the link on the first look though :smile: That was a very out of date fiddle.


Make validation observable and occur across all fields

Useful if, e.g., you need to disable a save button until a valid cc is entered.
Also allows the user to go back and edit anywhere (e.g. an invalid expiration date) and immediately get a callback when that's entered correctly again (instead of having to get to the end of the CVV field.

Shoot, sorry, meant to make a PR to my own fork


Add s3 trigger

Issue #, if available:
Closes #265
Description of changes:
Document how to start workflows from the command line and from an s3 trigger.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This pr only effects documentation so I'm skipping the code review.


docs: json schema update json/json-schema-v0.2.1

Updates schema to include https://github.com/open-feature/flagd-schemas/pull/131.
Fixes: https://github.com/open-feature/flagd/issues/1205
@austindrenski I imagine this is cached quite heavily by IDEs, etc. Not sure how long it will take to clear.
Preview link: https://deploy-preview-1215--polite-licorice-3db33c.netlify.app/schema/v0/targeting.json

@toddbaert could you check whether this need https://github.com/open-feature/flagd/pull/1212 to be there ?

@toddbaert could you check whether this need #1212 to be there ?

Ya, I've merged that, it's a good idea to do that first so the validation in flagd matches the schema for this fix.


Can't resize JPEG while MozJpeg Encoder is enabled

Is this a bug or just the way it works normally?

Noticed that too, not sure what's going on, will investigate


Visual Studio Code won't save any file

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.9.1
OS Version: Linux cas12-GemiBook-Pro 6.5.0-35-generic #35~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Tue May  7 09:00:52 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

Steps to Reproduce:

open any file,
make changes
Save changes ctrl+S or via vim(:w)

expected:
changes are made
actual

chanes are seen in the vscode editor but has not been saved
version:

I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


App not compiling.

Describe the bug
Will not compile app.
Link to the blitz that caused the error
https://stackblitz.com/edit/github-poeerw?file=package-lock.json
Steps to reproduce
npm run dev
Expected behavior
error - Failed to load next.config.js, see more info here https://nextjs.org/docs/messages/next-config-error
[TypeError: Failed to fetch dynamically imported module: file:///home/projects/github-poeerw/next.config.js]
Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 95.0.4638.54
Major version = 95
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.54 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 82370438,
  "usedJSHeapSize": 76909330,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = eb19bc088249a645473767848622a1d98c51b8f3

Additional context
No response

This issue should be fixed by now. Thanks for the report @MatthewGHub2


Wrong lisp is selected in lightshow tester

In https://lightshow.githubapp.com, when I select "Emacs Lisp (source.lisp)" and submit the form, it always switches to "NetLogo (source.lisp)".

I can confirm this behaviour although I think this is because they both use the same grammar. See the NetLogo entry: https://github.com/github/linguist/blob/master/lib/linguist/languages.yml#L1878
So while this is a bug in the UI I'm pretty sure it's testing against the correct grammar.

Yeah this is just a bug in the presentation of that form. As @arfon says the right grammar is being used. Thanks for letting me know; I'll try to get a fix in place.

I'm tracking this elsewhere. I'll leave a note here when the bug is fixed. Thanks!


feat: 🎸 pass cluster name to container

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (d209d4f) 29.72% compared to head (47b9b72) 31.50%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

cmd/namespace-check/main.go
25.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #20      +/-   ##
==========================================
+ Coverage   29.72%   31.50%   +1.77%
==========================================
  Files           1        1
  Lines          74       73       -1
==========================================
+ Hits           22       23       +1
+ Misses         51       49       -2
  Partials        1        1

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


Error recipes Brave | Signal | Anydesk

Hello!
I am trying to run these recipes without success, each of them presents an error, in searching for a solution on a forum in my country they commented that it might be a pkg2toappimage error.
PS: I'm sorry, did I post in the right place?

Hello, yes, this is the right place to ask.
Can you please post the command that you are using, the last few lines of the exact error message, and the system (distribution and version) you are using?

Hello, yes, this is the right place to ask.
Can you please post the command that you are using, the last few lines of the exact error message, and the system (distribution and version) you are using?

Environment
Fedora 33 (Workstation Edition)
First I did this
wget "https://raw.githubusercontent.com/AppImage/pkg2appimage/master/pkg2appimage"
chmod +x "pkg2appimage"
sudo mv "pkg2appimage" /usr/bin/
command: pkg2appimage Signal
Message console:

sed -ne 's|^([[:blank:]])([a-zA-Z0-9_])[[:blank:]]:[[:blank:]]"(.)"[[:blank:]]$|\1\2\3|p' -e 's|^([[:blank:]])([a-zA-Z0-9_])[[:blank:]][:-][[:blank:]](.)[[:blank:]]$|\1\2\3|p' /tmp/_recipe.yml
awk $'-F\034' '{
indent = length($1)/2;
vname[indent] = $2;
for (i in vname) {if (i > indent) {delete vname[i]}}
if (length($3) > 0) {
vn=""; for (i=0; i<indent; i++) {vn=(vn)(vname[i])("")}
printf("%s%s%s=("%s")\n", "",vn, $2, $3);
}
}'
sed s/_=/+=/g
sed -i -e 's|^_script+=("||g' /tmp/recipe_script
sed -i -e 's|")$||g' /tmp/recipe_script
bash -ex /tmp/recipe_script
mv 'opt/Signal/' usr/bin/
mv: cannot stat 'opt/Signal/': No such file or directory

command: pkg2appimage Brave
Message console:
It just worked now, but another one I tried was Vivaldi

command: pkg2appimage Vivaldi
Message console:

awk $'-F\034' '{
indent = length($1)/2;
vname[indent] = $2;
for (i in vname) {if (i > indent) {delete vname[i]}}
if (length($3) > 0) {
vn=""; for (i=0; i<indent; i++) {vn=(vn)(vname[i])("")}
printf("%s%s%s=("%s")\n", "",vn, $2, $3);
}
}'
sed s/_=/+=/g
sed -i -e 's|^_script+=("||g' /tmp/recipe_script
sed -i -e 's|")$||g' /tmp/recipe_script
bash -ex /tmp/recipe_script
sed -i -e 's|/usr/bin||g' vivaldi-stable.desktop
sed: can't read vivaldi-stable.desktop: No such file or directory

command: pkg2appimage Anydesk
Message console:
*$|\1\2\3|p' /tmp/_recipe.yml

sed -i -e 's|^_ingredients_script+=("||g' /tmp/recipe_script
sed -i -e 's|")$||g' /tmp/recipe_script
bash -ex /tmp/recipe_script
++ wget -q https://download.anydesk.com/linux/ -O -
++ grep -E 'anydesk.*amd64.deb'
++ cut '-d"' -f2
++ head -n1
++ cut -d/ -f2
DLD=anydesk_6.0.0-1_amd64.deb
wget -c https://download.anydesk.com/linux/anydesk_6.0.0-1_amd64.deb
--2021-03-15 15:26:03--  https://download.anydesk.com/linux/anydesk_6.0.0-1_amd64.deb
Resolving download.anydesk.com (download.anydesk.com)... <IP_ADDRESS>, 2a01:4f8:221:2841::2
Connecting to download.anydesk.com (download.anydesk.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 404 Not Found
2021-03-15 15:26:04 ERROR 404: Not Found.

Thank you....

Brave generated the image but it doesn't run, below is the output
[osiel@localhost Signal]$ ./brave-browser-1.21.76.glibc2.18-x86_64.AppImage
/usr/bin/AppImageLauncher: /lib64/libcurl.so.4: no version information available (required by /usr/bin/../lib/x86_64-linux-gnu/appimagelauncher/libappimageupdate.so)
QSocketNotifier: Can only be used with threads started with QThread
"/proc/50448/root"
"/proc/50448/root"
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libgio-2.0.so.0)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libmount.so.1)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libkrb5support.so.0)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libgio-2.0.so.0)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libmount.so.1)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libkrb5support.so.0)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libgio-2.0.so.0)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libmount.so.1)
/tmp/.mount_brave-wCLYd4/usr/bin/brave: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libkrb5support.so.0)
libva error: vaGetDriverNameByIndex() failed with unknown libva error, driver_name = (null)
[50539:50539:0315/152923.131142:ERROR:sandbox_linux.cc(374)] InitializeSandbox() called with multiple threads in process gpu-process.
/proc/self/exe: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libgio-2.0.so.0)
/proc/self/exe: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libmount.so.1)
/proc/self/exe: /tmp/.mount_brave-wCLYd4/lib/x86_64-linux-gnu/libselinux.so.1: no version information available (required by /lib64/libkrb5support.so.0)
[50503:50554:0315/152923.640426:ERROR:nss_util.cc(169)] Error initializing NSS with a persistent database (sql:/home/osiel/.pki/nssdb): /lib64/libsoftokn3.so: undefined symbol: PR_GetEnvSecure
[50503:50554:0315/152923.640481:ERROR:nss_util.cc(127)] Error initializing NSS without a persistent database: NSS error code: -5925
[50503:50554:0315/152923.640488:FATAL:nss_util.cc(129)] nss_error=-5925, os_error=0
/tmp/.mount_brave-wCLYd4/usr/bin//brave-browser: line 48: 50503 Trace/breakpoint trap   (core dumped) "$HERE/brave" "$@"

/lib64/libsoftokn3.so: undefined symbol: PR_GetEnvSecure seems to be the culprit.
Also see

saiarcot895/chromium-ubuntu-build#12
probonopd/linuxdeployqt#34

Just for fun, can you try to run these AppImages on an Ubuntu xenial Live medium?

I performed the following test of what I had on hand. This same appImage generated in Fedora I ran in a notebook with Elementary OS 5.1.7 and the error was this
tati@dell:~/Teste$ ./brave-browser-1.21.76.glibc2.18-x86_64.AppImage
[6891:6891:0315/160752.828142:ERROR:sandbox_linux.cc(374)] InitializeSandbox() called with multiple threads in process gpu-process.
[6864:6876:0315/160753.409573:ERROR:nss_util.cc(169)] Error initializing NSS with a persistent database (sql:/home/tati/.pki/nssdb): libsoftokn3.so: não é possível abrir arquivo compartilhado: Arquivo ou diretório inexistente
[6864:6876:0315/160753.409635:ERROR:nss_util.cc(127)] Error initializing NSS without a persistent database: NSS error code: -5925
[6864:6876:0315/160753.409649:FATAL:nss_util.cc(129)] nss_error=-5925, os_error=0
[6895:6923:0315/160753.432271:ERROR:ssl_client_socket_impl.cc(924)] handshake failed; returned -1, SSL error code 1, net_error -3
/tmp/.mount_brave-BPLwhx/usr/bin//brave-browser, linha 48: 6864 Trace/breakpoint trap "$HERE/brave" "$@"

https://github.com/probonopd/linuxdeployqt/issues/35
https://github.com/saiarcot895/chromium-ubuntu-build/issues/12

Excuse my ignorance, but I read the links and it is still not clear what the solution would be on Signal Brave and Anydesk.
I figured the problem would be the recipe or something because they are old I thought the link changed or whatever.
Ps I'm using a translator... Sorry for any mistakes or anything out of context (rsrs)

Please contact the authors of Brave | Signal | Anydesk for officially supported AppImages. Thanks.

Ok. Thanks


[1.12] Automated Bump DC/OS UI v2.26.12

High-level description
What features does this change enable? What bugs does this change fix?
Corresponding DC/OS tickets (required)

DCOS-ID JIRA title / short description.

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

I will add CHANGES.md tomorrow.

there should be a comment making the change way easier 🤔

@juliangieseke , what comment?


[Question] Why the LlaVA Next output nothing?

Question
I used transformers=4.44.2 , and load the model llava-hf/llama3-llava-next-8b-hf with the following code:
from transformers import LlavaNextProcessor, LlavaNextForConditionalGeneration
processor = LlavaNextProcessor.from_pretrained(model_path)
model = LlavaNextForConditionalGeneration.from_pretrained(model_path)

and then, I used batch inference with batchsize = 4. The prompts are in proper formats as far as I concerned like:
USER: <image>\n<prompt text here>\nASSISTANT:

three items return the expected answers but only one in the same batch return nothing. Here is my code on inference:
inputs = processor(prompts, images, return_tensors = "pt", padding = True)

inputs = inputs.to(device)
outputs = model.generate(**inputs, max_new_tokens = 10000)
descriptions = processor.batch_decode(outputs, skip_special_tokens=True)

the prompts, images are lists.
If anyone has the same problem or has any clue to solve it, please email me or respond to this issue. Thanks a lot!

btw I set the max_new_tokens = 300. The prompt may be long but I don't know if it will influence the generated codes.


[feature] presentation service link

Purpose

Allows the presentation service to be accessed by users directly from the OSF.
Addresses #1418 and #2115

Changes

Adds the link to the presentation service to the Explore tab in the navbar.

Nice work @caseyrollins !!


Infererd type of Data is wrong with the axios-typescript example

Bug report
Description / Observed Behavior
I think the returned data property from the axios typescript example should be a union type of Data | undefined. It's always Data, even with suspense set to false
Expected Behavior
Union type of Data | undefined is returned
Repro Steps / Code Example
https://codesandbox.io/s/swr-axios-ts-example-bp4w6s?file=/src/index.tsx

Additional Context
SWR version: 2.1.5
Edit: with the exact same versions of axios, typescript and swr in a react-native project, the type is inferred correctly

It seems working properly with next.js locally, could be sth wrong with the reproduction on codesandbox? I'll close this issue for now as it's not a problem of the library. Can carry on in the discussion


Map dragging is slow when using more than 15 000 markers in a high zoom level

Hey,
I have (sometimes) much more than 15 000 markers. When i drag the map in a high zoom level e.g. above an area that has only like 200 (clustered) markers it is very slow although I have a Galaxy S3. Someone knows how to make dragging fast in high zoom levels?

Is there any practicable solution for this issue yet?

Any word on this?  I have about 8k markers and the lib is unusable at that point unfortunately.  And I say unfortunately with great respect as this is a beautiful lib as far as UI, very easy to use and documented well.  Really hoping I can use this soon :(
At a high level zoom things are pretty good.  However as I start to zoom in performance degrades.  I assume this is due to markers and clusters off the viewport being drawn/redrawn as I zoom.
Then once I am zoomed all the way in if I zoom out quickly I notice a huge hit.  All the markers are unclustered (as I was all the way zoomed in before).  I keep zooming out until I can see the entire map.  I can see all (unclustered) markers and it takes 20-30 seconds to loop through all 8K markers and place them back into a clusters.
All of this probably relates to the fact that off screen markers and clusters are being drawn.  However I also wonder if this is slow due to the animations.  In cases like this it might make sense to be able to turn off animations and just remove and redraw all clusters and marker on zoom change.

I made the following changes locally to get the library to work reasonably
well with lots of markers:

Disable animation completely.  There is a constant you can change in the
existing code for this.

When camera motion stops, use an AsyncTask to remove all items from the
ClusterManager, and then re-add only those visible on the screen.  (I do
this only if there's a large number of markers.)

In ClusterManager, I imported the
mClusterTask.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR) change from
upstream (I had an older version of the library).

I added a cancel() method on Algorithm, and a periodic check in
NonHierarchicalDistanceBasedAlgorithm.getClusters, so that I could cancel
previous clustering operations when starting a new one.

I like this library but I wish the team were more responsive to obvious
problems like this and
https://code.google.com/p/gmaps-api-issues/issues/detail?id=5313
On Mon, Apr 6, 2015 at 6:44 PM, Billy Newman<EMAIL_ADDRESS>wrote:

Any word on this? I have about 8k markers and the lib is unusable at that
point unfortunately. And I say unfortunately with great respect as this is
a beautiful lib as far as UI, very easy to use and documented well. Really
hoping I can use this soon :(
At a high level zoom things are pretty good. However as I start to zoom in
performance degrades. I assume this is due to markers and clusters off the
viewport being drawn/redrawn as I zoom.
Then once I am zoomed all the way in if I zoom out quickly I notice a huge
hit. All the markers are unclustered (as I was all the way zoomed in
before). I keep zooming out until I can see the entire map. I can see all
(unclustered) markers and it takes 20-30 seconds to loop through all 8K
markers and place them back into a clusters.
All of this probably relates to the fact that off screen markers and
clusters are being drawn. However I also wonder if this is slow due to the
animations. In cases like this it might make sense to be able to turn off
animations and just remove and redraw all clusters and marker on zoom
change.
—
Reply to this email directly or view it on GitHub
https://github.com/googlemaps/android-maps-utils/issues/29#issuecomment-90308332
.

@akirmse agreed, wish issues like this would get more attention.  Of course I am not contributing so I don't have much to complain about ;)
Is this being actively worked by contributors, or has anyone forked, maybe a possible PR for this?

For more than two years the clustering element of this library is totally unusable. Clustering more than 100 items even on a big area brings huge performance issues. This should be either adressed or removed completely from the library.

Heres my hacked version:
Update getClusters to

public Set<? extends Cluster> getClusters(double zoom, LatLngBounds bounds)

Add the following if to the QuadItem candidate for each loop

if(!bounds.contains(candidate.mPosition)) {
continue;
}

Replace ClusterManager.cs onCameraChange function with the following:

mMap.clear();
mPreviousCameraPosition = mMap.getCameraPosition();
currentBounds = mMap.getProjection().getVisibleRegion().latLngBounds;
Set<? extends Cluster> results = mAlgorithm.getClusters(mPreviousCameraPosition.zoom, currentBounds);
for(Cluster cluster : results)
{
// Strip out DefaultClusterRendered.cs perform method and render each cluster accordingly here
}

@akirmse can you show your example in code?

@broady any update on this one? Your clustering looks very cool, the animations are great, but it just cannot handle more markers. And this is a big problem, since the point of clustering that we have lots of markers. I am currently using this algorithm to only cluster the visible markers, but this also has drawbacks (clusters change a lot, user have to wait a little bit when cluster show up after changing camera).
Are you planning to implement robust solution? Thanks in advance for your answer, and for this great library in general! :)

I had to modify this library to my needs to get it working.  I have about 50k+ markers and it works great. It will only show markers in the visible region.

Hey folks.
I've actually left the Maps team, so everything I do on this project is in my spare time.
Here's what I think should happen:

DefaultClusterRenderer implements GoogleMap.OnCameraChangeListener.
Somehow pipe camera change events to DefaultClusterRenderer.
On the first camera change event, set a flag that changes the renderer to only render markers inside the visible bounds.
Update visible markers when the camera changes and/or clusters change.

Unfortunately the loose coupling design of the library isn't conducive to this kind of stuff. I think it was a mistake.
@jfschmakeit @markmcd in case they have spare cycles.

Anyone on maps team supporting this or is it left to die?

I've just written down the exact steps of my preferred solution. Please discuss, send a pull request, etc! :)

@broady it is sad you no longer work on this officially, and no one else took over. This is an important project, even referred from the official doc. Thanks for still managing the project, your time is much appreciated!
I already tried to implement your solution, but only in a very naive way. ClusterManager automatically calls mRenderer.onCameraChange if the renderer implements OnCameraChangeListener, so piping that is already done. This is my onCameraChange method in the renderer:
@Override
public void onCameraChange(CameraPosition cameraPosition) {
  LatLngBounds bounds = map.getProjection().getVisibleRegion().latLngBounds;

  for (Marker marker : clusterManager.getClusterMarkerCollection().getMarkers()) {
      if (bounds.contains(marker.getPosition())) {
          marker.setVisible(true);
      } else {
          marker.setVisible(false);
      }
  }
}

There are two problems with this approach:

when panning/rotating, the movements are slow because this loop blocks the drawing thread for too much time by iterating over all the clusters and checking they are in the bounds or not
this does not help at all when zooming, because the markers are added asynchronously that time

So a more complicated solution is needed, but i could not figure that out.

+1 for this issue

Lol, French people are famous for complaining a lot so keep this in mind, plus I'm not contributing either, but for a library that is referenced on the official documentation I see it as a "fuck you" from the Maps team to not support this at this time.
I mean, look at the performances of the clustering libraries for iOS... Any random app can have blazingly fast map with hundred of points.

With help of stephenmeyer's hack I implemented my version of viewport only clustering which is lightning-fast.
Here is a detailed description of all my changes.
Because my time was limited and I had to reuse the viewport bounds I created a tiny class to store the value in. Quick & dirty but it doesn't hurt anyone.
So, create a class in android/clustering. I called mine bvb.java and this is the content:
package com.google.maps.android.clustering;
import com.google.android.gms.maps.model.LatLngBounds;

public class bvb {
    public static LatLngBounds mapClusterViewportBounds;
    public static boolean infoWindowShowing = false;
}

This holds 2 variables that can be accessed anywhere within the clusterManager.
The mapClusterViewportBounds variable will contain the coordinates of the part in the map that is visible to the user.
The infoWindowShowing variable should be set to true if you are using an infowindow popup when a user clicks on a marker. In the usual code, the clustering is only recalculated when the zoomlevel changes. If a user clicks on a marker, it's centered and the map doesn't zoom. The infowindow shows and stays in place. But with viewport clustering implemented, panning should also recalculate clusters or else you'd not see anything until you zoom. If a marker is now clicked, the infowindow shows, the marker is centered which now triggers reclustering and the infowindow is removed again.
To prevent this, whenever you are showing an infoWindow, add bvb.infoWindowShowing = true;
When an infoWindow is closed, add bvb.infoWindowShowing = false;
Note that you should also add bvb.infoWindowShowing = false; whenever you load new data on the map as the infoWindow is also removed when the map is cleared.
Now edit ClusterManager.java
In the import section add the following:
import android.util.Log;
import com.google.android.gms.maps.model.LatLng;
import com.google.android.gms.maps.model.LatLngBounds;

Find the onCameraChange method and change it to the following:
    @Override
    public void onCameraChange(CameraPosition cameraPosition) {
        // Added this to store the viewport bounds and increase slightly for user convenience
        LatLngBounds b = mMap.getProjection().getVisibleRegion().latLngBounds;
        LatLng bNE = b.northeast;
        LatLng bSW = b.southwest;
        double bAdjustLat = (bNE.latitude - bSW.latitude) * 0.15; // 15% increase
        double bAdjustLon = (bNE.longitude - bSW.longitude) * 0.15; // 15% increase
        bNE = new LatLng(bNE.latitude + bAdjustLat, bNE.longitude + bAdjustLon);
        bSW = new LatLng(bSW.latitude - bAdjustLat, bSW.longitude - bAdjustLon);
        bvb.mapClusterViewportBounds = new LatLngBounds(bSW, bNE);

        if (mRenderer instanceof GoogleMap.OnCameraChangeListener) {
            ((GoogleMap.OnCameraChangeListener) mRenderer).onCameraChange(cameraPosition);
        }

        if (bvb.infoWindowShowing) {
            Log.i("Cluster", "Not reclustering as we have an infoWindow");
            return;
        }
        cluster();
    }

We first load the viewport bounds into a variable. I increased the viewport with 15% so that the corners also show markers/clusters and panning shows at least some data to the sides. Just 15% extra won't hurt performance. You can play with this number and increase/decrease it to your liking.
Then I check if there's an infoWindow set. If so, it will not recluster until it's closed and the onCameraChange method is called.
Now edit algo/NonHierarchicalDistanceBasedAlgorithm.java.
In the import section add the following:
import android.util.Log;
import com.google.maps.android.clustering.bvb;

In the getClusters method find the line "for (QuadItem candidate : mItems) {
Underneath that is an if section about visitedCandidates.contains(candidate)).
Below that if-section, add this following code (for me it's on line 99).
if (bvb.mapClusterViewportBounds != null && !bvb.mapClusterViewportBounds.contains(candidate.mPosition)) {
                    continue; // Added to exclude markers outside viewport
}

While iterating over all map-points, this doesn't process any point outside your viewport + the 15%.
Which is what make the clustering perform quite well.
Then edit algo/PreCachingAlgorithmDecorator.java.
Find the getClustersInternal method.
Uncomment the following line:
mCache.put(discreteZoom, results);
The cluster-results are cached per zoomlevel. Since we now have viewports-clusters, this no longer works. I thought about caching by zoomlevel and viewport-coordinates, by the chance of hitting the exact same combination multiple times is slim and since it already performs pretty well I decided to disable caching completely. I didn't back-out the caching-code but went for the quick approach of just disabling saving to cache so it will never find any cache.
Here is an archive with the changes files.
Bo

As one solution to slow animation you can now choose to disable the animations, which was added in 27f7a5b

I've a map with 4k markers on it. I've tried the VisibleBasedAlgorithm but the performance remains very very slow.
When I zoom out the map, I've to wait some seconds before the clustering ends.
When I zoom in, the cluster disappear very slow, so the user is confused.
My experience:

I've disabled the animations;
I've used the https://github.com/zamesilyasa/android-maps-utils (branch issue-82)
I've added further optimization on ClusterRenderer class

Is there a way to avoid the cluster expansion when the markers aren't visible on maps bounds? I think that this could be only solution.

@CeccoCQ, you can try this optimised variant of Screen Based algorith https://github.com/zamesilyasa/android-maps-utils/commit/8694da38220d6394e4d9216b2a359d536afe9c40.

Hi @Gary111,
I've finish the experiments with your code just now and I've seen some improvements.
By your experience, is there a way to increase the bitmap drawing performance?

Are you sure you are  using as small bitmaps(thumbnails) for markers? if bitmap is large, it will slow rendering.

@CeccoCQ, If you mean drawing markers, I use disk and memory LRU cache for this. So I generate icon only once and save it in cache, next time I just extract it form cache. It increases performance a little.

Hi, thanks for your responses.
This is my code used to create the bitmap:
public LruCache<Integer, BitmapDescriptor> mIcons;
.....
int sizeSelected = Math.round(TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, 50, context.getResources().getDisplayMetrics()));
Bitmap selectedBitmap = BitmapFactory.decodeResource(context.getResources(), mso.markerSelected).copy(Bitmap.Config.ARGB_8888, true);
Bitmap selected = Bitmap.createScaledBitmap(selectedBitmap, sizeSelected, sizeSelected, true);
BitmapDescriptor descSelected = mIcons.get(mso.markerSelected);
if (descSelected == null) {
    descSelected = BitmapDescriptorFactory.fromBitmap(selected);
    mIcons.put(markerTypeId, descSelected);
}

then into the onBeforeClusterItemRendered method:
BitmapDescriptor desc = mIcons.get(markerTypeId);

The original bitmap is a png file of 294x294px of 15Kb

The fact this issue has been open for 5 years and the lack of response from anyone suggests this will not be addressed. I’d recommend a 3rd party clustering library if you want something that works.

I can confirm, this https://github.com/mg6maciej/android-maps-extensions/tree/develop can handle a lot of markers without performance issues

Having used android-maps-extensions and google-maps-clustering I would recommend the following:
https://github.com/sharewire/google-maps-clustering
Performance is fantastic and the code is well written and easily extended. Also has nice animations when the map moves and the clusters regroup.


Solution to Fix overflow #13

Made minor changes to format text.
Fix overflow #13

@Samkit-shah thanks! but this has been fixed. I was not completely sure what you had changed as your code auto-formatter has modified the indentation resulting in a very large diff.

Ok thanks
Regards,
Samkit shah
On Fri, Oct 2, 2020, 6:43 PM Shaan Subbaiah<EMAIL_ADDRESS>wrote:

Closed #22 https://github.com/shaansubbaiah/Portfolio/pull/22.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shaansubbaiah/Portfolio/pull/22#event-3833915630, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APUB3NQJFPBZSAUPIEEILW3SIXGWDANCNFSM4SBRJJGA
.


bug: Menu will not allow text to break

What version of daisyUI are you using?
v3.0.3
Describe your issue
I expect menu items to respect word breaks, but they don't. Instead, menu items will always overflow. This used to work before the upgrade from v2 to v3 (words would break according to break-normal by default without the need to even include the class, which in my opinion seems like the most logical default).
What browsers are you seeing the problem on?
Chrome
Reproduction URL (optional)
https://play.tailwindcss.com/TdBkm1eFoU

Gotcha, thanks for clarifying!

But neither approaches (auto-cols-auto and flex) doesn't work for submenus.
https://play.tailwindcss.com/Sy4pCzcY2y


Issue #763 EhcacheActiveEntity unit tests and corrections

This PR introduces unit tests for EhcacheActiveEntity and corrections for issues exposed by @lorban.
Additional unit tests to follow in additional PRs.

Rebased ...

Rebased with adjustments for PR #1050


An opened file doesn't populate its layers, causing a crash when they're clicked in the viewport

This is a regression since the File > Open system used to work. Now this is more urgent with the auto-save system enabled, causing this bug to be encountered by default (and potentially causing a broken state where the only fix is to clear the site's IndexedDB storage).

This is I think due to the layerdata not being saved (which it probably should be to save if a layer is hidden etc)

This is I think due to the layerdata not being saved (which it probably should be to save if a layer is hidden etc)

This used to work though. It's a regression.

I forgot layerdata is automatically generated if missing (although you may still want to save it), instead it appears the problem (introduced when folders where added) was that handle_folder_changed which called an ExpandFolder operation recreating the entries was removed. The fix should be to add back the new FolderChanged operation on the root when a new document is loaded.


cannot get shell log when use node-agent

What would you like to be added:
cannot get shell log when use node-agent
Why is this needed:
cannot get shell log when use node-agent

/assign @village-way

/assign @village-way

refer to  https://github.com/kosmos-io/kosmos/issues/544

/close


Installation check should do retries

When multiple packages are released from the same pipeline and one depends on another Azure.Core.Experimental -> Azure.Core I tend to kick off both releases at the same time. This causes the Installation Check to fail for Azure.Core.Experimental because Azure.Core is just being published.
I think we should loop and try to restore for some time (30m?) before failing the step.

@ckairen FYI this is related to the ordering you introduced for java. I think I like the eventual approach @pakrym is proposing here better, since it will make releases easier to kick off in the background.
I think we should see if we can find a way to determine whether there is a package we expect to be published before retrying, so we don't stall the release for no reason and make people wait. This might not be possible as I don't think agents have any way to communicate values with each other while running, and additionally I don't know of a way to have a deployment job auto-retry (e.g. schedule to a new agent which would pick up a new variable context).

@pakrym I'm marking this needs-triage just so it comes across for @kurtzeborn, @weshaggard and I in our weekly triage.

Adjusting tags so that the reports don't yell at me.

@benbp is a retry loop similar to what @ckairen did in python enough here, so retry in the install check and not the full job.


Support multiple chunks of the same type.

Some chunks, like the Application Specific chunk, are permitted to appear many times in the same aiff. Currently, we have a simplistic model where we store chunks in a map keyed on their ID. Change this to be able to store multiple chunks with the same ID.

Done the main part. Added a test for the application chunk.
Need to add a test that it's not possible to add more than one common chunk though.

Checked that there is exactly one common chunk.

Tightened up checks around the permissible number of Sound Data chunks too.


Create Board.md

Add a piece of code i consistently use when building plugin to determine if currently have a board loaded.

Thanks for the contribution. I have merged it in. I use a similar code but
didn't package it nicely into a function. Is the camera check really
needed? I have it in most of my code because I just grabbed it from my
plugin which involved the camera but I think if the board is loaded the
camera definitely should be, right?
Lord Ashes
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Mon, May 24, 2021 at 11:10 AM Justin @.***> wrote:

Add a piece of code i consistently use when building plugin to determine
if currently have a board loaded.
You can view, comment on, or merge this pull request online at:
https://github.com/LordAshes/TaleSpire-CommonCodeSnippets/pull/1
Commit Summary

Create Board.md

File Changes

A Board.md
https://github.com/LordAshes/TaleSpire-CommonCodeSnippets/pull/1/files#diff-bdf2d0412ae56dd44147888ea642c2efc5e6041a3d50b8a68ec6d67c3974b802
(11)

Patch Links:

https://github.com/LordAshes/TaleSpire-CommonCodeSnippets/pull/1.patch

https://github.com/LordAshes/TaleSpire-CommonCodeSnippets/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/LordAshes/TaleSpire-CommonCodeSnippets/pull/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQ5Y6BY7Y36GUX6HKTSSWTTPJT5BANCNFSM45NNCJRQ
.

Thanks for the contribution. I have merged it in. I use a similar code but didn't package it nicely into a function. Is the camera check really needed? I have it in most of my code because I just grabbed it from my plugin which involved the camera but I think if the board is loaded the camera definitely should be, right? Lord Ashes...

I don't think it's really needed though I'm not the most familiar with unity/Talespire in how it handles that.


Cranelift: cfg had unexpected successor(s) whlie using br_table

.clif Test Case
function u0:0() -> i32 apple_aarch64 {
    sig0 = (i64) -> i64 fast
    sig1 = (i64) -> i64 system_v
    fn0 = colocated u0:2 sig0
    fn1 = u0:3 sig1
    jt0 = jump_table [block3, block4]

block0:
    v0 = iconst.i64 104
    v1 = call fn0(v0)
    v2 = load.i64 v1
    br_table v2, block4, jt0

block3:
    v3 = load.i64 v1+1
    jump block1

block4:
    jump block2

block1:
    v5 = call fn1(v3)
    jump block5(v5)

block2:
    v6 = iconst.i64 0
    jump block5(v6)

block5(v4: i64):
    v7 = iconst.i32 0
    return v7
}

Apologies I can't seem to find where to install clif-util so that I can run clif-util bugpoint.
Steps to Reproduce
Create the above IR via cranelift_jit and the FunctionBuilder API.
Expected Results
Expected the IR to be valid without panicking given that it had passed the cranelift_codegen::verifier::verify_function check.
Actual Results
Panics with:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Compilation(Verifier(VerifierErrors([VerifierError { location: block0, context: None, me
ssage: "cfg had unexpected successor(s) [block3, block4]" }])))', src/cranelift_backend/context.rs:188:14

The line in question is:
module.define_function(function_id, module_context).unwrap();

where module: cranelift_jit::backend::JITModule
Versions and Environment
Cranelift version or commit: 0.82.1
Operating system: Mac OSX 12.2.1
Architecture: arm64
Extra Info
This function IR fails at define_function despite passing the cranelift_codegen::verifier::verify_function check.

Apologies I can't seem to find where to install clif-util so that I can run clif-util bugpoint.

You can use cd cranelift && cargo run -- bugpoint.

Create the above IR via cranelift_jit and the FunctionBuilder API.

Can you be a bit more specific? Using clif-util compile crash.clif --target aarch64 works fine for me.

Running cargo run -- bugpoint crash.clif aarch64" gives me Warning: Given function compiled successfully or gave a verifier error.I'm assuming it compiled correctly sincecargo run -- compile crash.clif --target aarch64` compiles successfully for me. I'll work to try to get a more specific example of how the IR was constructed.

I believe I've discovered/fixed my error. I was calling define_function after each function finished codegen rather than only after all functions have finished codegen. So define_function(u0:0) in the above example was being called before define_function(fn0) or define_function(fn1)

define_function needs to be called for each individual function. Afterwards the context needs to be cleared using .clear() (or discarded entirely) before codegenning the next function.

Ah, I believe that was the true error then. .clear() wasn't being called after compiling u0:0.


Capability to render raw html?

First of all, Thank you all for this great project. It is mind blowing how flawlessly it works.
I only have a suggestion which I think may help with the extensibility of elixir livebooks.
Current behavior
It is not possible to render raw HTML cells from the elixir side. It is possible (with a hacky trick) to render markdown or a text using the following snippet from Kino.render/1.
gl = Process.group_leader()
ref = Process.monitor(gl)

send(gl, {:io_request, self(), ref, {:livebook_put_output, {:markdown, "## Alisina"}}})

receive do
  {:io_reply, ^ref, :ok} -> :ok
  {:io_reply, ^ref, _} -> :error
  {:DOWN, ^ref, :process, _object, _reason} -> :error
end

Process.demonitor(ref)

:"do not show this result in output"

Expected behavior
First of all, Would it make sense to export this as a helper function inside livebook? (So you can render images and markdown etc.)
Secondly, Does it make sense to add capability to render raw html? (For example in order to use iframe)
Thanks in advance.

Hey @alisinabh!
You actually don't need this hacking to render Markdown, there is a Kino.Markdown struct that gets rendered as markdown automatically, so you can do:
Kino.Markdown.new("""
## Alisina
""")

Similarly, there is Kino.Image for rendering images.
As for arbitrary HTML, you theoretically can put HTML in Markdown, but it will get sanitized, so things like iframe or JS won't work. We don't allow rendering arbitrary HTML, because this would open us to security vulnerabilities (running a notebook could produce an arbitrary HTML, for instance running some JS).

Thanks for the quick response. I get your point here, and it totally makes sense.
My problem is it seems currently there are no ways to extend the livebook views without modifying this repository. It might be a good Idea to add extensions for livebook (which can ask user to run to avoid security risks) to modify the view. This can add extension support for different chart libraries and also can help making the livebooks more interactive.
I don't know, Maybe this thing is not currently important or in the roadmap but I personally think this is super useful.
Have a nice week. :smiley:

We support interactions via the Kino package, it already includes stuff like dynamic charts and tables. We will further extend it to cover more use cases, but we don't plan to allow custom extensions that modify the page.

We may support custom Kinos in the future but definitely not a priority for
now. :)

Custom rendering would be pretty neat indeed. For instance, I would like to be able to create custom SVG rendering in LiveBook for MIDI (music) structures, static and dynamic.
At this point I am going to hack around VegaLite to "find a way to render what I need", but anything custom would find tons of use!

If the need is svg, we could allow svg to be rendered directly. We could still sanitize it or render it embedded in an image or iFrame.

If the need is svg, we could allow svg to be rendered directly. We could still sanitize it or render it embedded in an image or iFrame.

iFrame support would be awesome too.

SVGs can already be rendered using Kino.Image :)

@jonatanklosko woah, pretty cool. I'll try using that!


dart compile exe from .dill (kernel)  files

Today, when we try to use dart compile exe bin/foo.dill, it doesn't work (on macOS, it consumes memory until it crashes).
It would be great to have the ability to compile to a self-contained native executable from the .dill file since this already accounts for about 50% of the process.

Could you elaborate why you want this? Where are you going to get dill file from? You still need to recompile this dill whenever your source changes.
It is possible to implement this - it is just unclear what value it provides.
[This was previously asked https://github.com/dart-lang/sdk/issues/41046]

I can compile to .dill, then compile to native for each target os (macOS, windows, Linux), reducing CI time.

Dart has a incremental compiler now. Why not use it here?

//cc @bkonyi

Any update?

No updates yet. I might be able to pick this up in the next week or two, but I'm not sure how much work this will be without taking a closer look.

I can compile to .dill, then compile to native for each target os (macOS, windows, Linux), reducing CI/build time.

This isn't really something you should do: We have 2 different kinds of kernel files:

Those that can run on the Dart VM JIT: They are simple kernel AST representation of the input program, they are target-OS independent, they haven't gone through global analysis, haven't been tree shaken.

Those that can be compiled via Dart VM's AOT compiler: They went through global analysis, are tree shaken, are target-OS specific, are annotated with global analysis results (which allow the VM's AOT compiler to unbox parameters, emit devirtualized calls, ...)

So even after @bkonyi 's https://github.com/dart-lang/sdk/commit/5a386442951ba931c8f575ea70b16c0c7f0db2ea, the actual executables you get out of
% dart compile app.dart
% dart compile exe app.dill

aren't really optimized - they may be significantly bigger & run significantly slower than using the dart compile exe app.dart.
Right now dart compile kernel is producing a kernel file that's intended to run on the Dart VM JIT, not compiled with our AOT compiler. We may in the future be in a situation where our AOT compiler relies on certain information in the AOT-ready kernel that the JIT kernel doesn't have (or vice versa: The JIT may require things in a kernel that a AOT-ready kernel doesn't have).

(Never mind my message (deleted now): It compiles to JIT kernel and then optimizes that to AOT kernel and then AOT compiles it)


misc/cgo/testsanitizers: fails on Ubuntu 16.10

I assume this is our usual friend, PIE-by-default:
ubuntu@yakkety:~/go$ go tool dist test -run testsanitizers

##### ../misc/cgo/testsanitizers
skipping msan tests: -fsanitize=memory not supported
FATAL: ThreadSanitizer: unexpected memory mapping 0x5620a8a10000-0x5620a8ae1000
exit status 66
FAIL: tsan
FATAL: ThreadSanitizer: unexpected memory mapping 0x55a9d17a7000-0x55a9d1878000
exit status 66
FAIL: tsan2
FATAL: ThreadSanitizer: unexpected memory mapping 0x559e976a5000-0x559e97775000
exit status 66
FAIL: tsan3
2016/05/30 02:56:37 Failed: exit status 1
2016/05/30 02:56:37 FAILED

@dvyukov Does TSAN work with PIE?

TSAN works with -pie, but not on newer linux kernels -- they constantly shuffle process address space, we can't keep up. We have the same issue for C/C++ (https://github.com/google/sanitizers/issues/503). This won't happen for 1.7, I have no idea how to fix it.

That suggests something like https://go-review.googlesource.com/#/c/23535/ then (which does fix this issue).


register package with JuliaHub

@JuliaRegistrator register()

Error while trying to register: "Branch master not found"

@JuliaRegistrator register(branch = "main")


Coupling between com.docusign.esign.client.ApiClient and com.sun.jersey.api.client.Client  too tight.

Coupling between com.docusign.esign.client.ApiClient and com.sun.jersey.api.client.Client  too tight. Therefore it is not possible to, for instance, use the API behind a proxy in an easy way.

Hi @pasinski,
Thank you for using DocuSign. We agree that the Jersey client needs to be updated to the current version.  We have added some proxy support but I understand that more is needed for some use cases.
We already are tracking a different issue to upgrade to the current version of Jersey -- Issue #90
I'm glad that you were able to use Feign with the DocuSign API. Please add to this issue if you write up or post about your usage of Feign with DocuSign.
I will close this ticket as a dup of issue #90
Thanks,
Larry

Duplicate with #90


[vm/ffi] structs test on android arm32 in product mode

After landing support for structs in 32 bit (https://github.com/dart-lang/sdk/issues/36334) one of the tests now enabled for 32 bit arm fails. However, it only fails in product mode (log).
/===========================================================\
| ffi/structs_test is new and failed (Crash, expected Pass) |
\===========================================================/

--- Command "vm_compile_to_kernel []" (took 251ms):
DART_CONFIGURATION=ProductAndroidARM /b/swarming/w/ir/pkg/vm/tool/gen_kernel --no-aot --platform=out/ProductAndroidARM/vm_platform_strong.dill -o /b/swarming/w/ir/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_structs_test/out.dill /b/swarming/w/ir/tests/ffi/structs_test.dart --packages=/b/swarming/w/ir/.packages -Ddart.developer.causal_async_stacks=true

exit code:
0

--- Command "adb_precompilation" (took 02.000750s):
Steps to push Dart VM and Dill file to an attached device. Uses (and requires) adb.

exit code:
-6

stdout:
Executing adb -s 06ad7f0e003b6c8b shell rm -Rf /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Stdout:
AdbShellExitCode: 0
ExitCode: 0
Time: 0:00:00.069053

Executing adb -s 06ad7f0e003b6c8b shell mkdir -p /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Stdout:
AdbShellExitCode: 0
ExitCode: 0
Time: 0:00:00.065481

Executing Skipped cached push
ExitCode: 0
Time: 0:00:00.000066

Executing adb -s 06ad7f0e003b6c8b push /b/swarming/w/ir/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_structs_test/out.dill /data/local/tmp/testing/test/out.dill
Stderr:
4235 KB/s (5706568 bytes in 1.315s)
ExitCode: 0
Time: 0:00:01.403831

Executing adb -s 06ad7f0e003b6c8b shell export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/data/local/tmp/testing/test;/data/local/tmp/testing/dart --deterministic --optimization-counter-threshold=50 --enable-inlining-annotations --ignore-unrecognized-flags --packages=/b/swarming/w/ir/.packages /data/local/tmp/testing/test/out.dill ; echo AdbShellExitCode:  $?
Stdout:
WARNING: linker: /data/local/tmp/testing/dart: unused DT entry: type 0x6ffffef5 arg 0xd37c
WARNING: linker: /data/local/tmp/testing/dart: unused DT entry: type 0x6ffffffe arg 0x12808
WARNING: linker: /data/local/tmp/testing/dart: unused DT entry: type 0x6fffffff arg 0x3
../../runtime/vm/object.cc: 18656: error: unreachable code
Aborted
AdbShellExitCode: 134
ExitCode: -6
Time: 0:00:01.141814

--- Re-run this test:
python tools/test.py -n dartk-android-product-arm ffi/structs_test

Unreachable code that is hit:
int64_t Integer::AsInt64Value() const {
  // Integer is an abstract class.
  UNREACHABLE();
  return 0;
}

I have trouble reproducing this locally.
Build command on bot:
tools/build.py -aarm -mproduct runtime_kernel '--os=android' --no-start-goma -j200

Build command locally:
rm -rf out/ProductAndroidARM
tools/build.py -a arm -m product '--os=android' runtime_kernel

I have the same flags, and I do not see a separate gn.py command on the bot.
Test command on bot:
DART_CONFIGURATION=ProductAndroidARM /b/swarming/w/ir/pkg/vm/tool/gen_kernel --no-aot --platform=out/ProductAndroidARM/vm_platform_strong.dill -o /b/swarming/w/ir/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_structs_test/out.dill /b/swarming/w/ir/tests/ffi/structs_test.dart --packages=/b/swarming/w/ir/.packages -Ddart.developer.causal_async_stacks=true

exit code:
0

--- Command "adb_precompilation" (took 02.000470s):
Steps to push Dart VM and Dill file to an attached device. Uses (and requires) adb.

exit code:
-6

stdout:
Executing adb -s 069a6592003ba933 shell rm -Rf /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Stdout:
AdbShellExitCode: 0
ExitCode: 0
Time: 0:00:00.071593

Executing adb -s 069a6592003ba933 shell mkdir -p /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Stdout:
AdbShellExitCode: 0
ExitCode: 0
Time: 0:00:00.051388

Executing Skipped cached push
ExitCode: 0
Time: 0:00:00.000034

Executing adb -s 069a6592003ba933 push /b/swarming/w/ir/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_structs_test/out.dill /data/local/tmp/testing/test/out.dill
Stderr:
5708 KB/s (5706568 bytes in 0.976s)
ExitCode: 0
Time: 0:00:01.070715

Executing adb -s 069a6592003ba933 shell export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/data/local/tmp/testing/test;/data/local/tmp/testing/dart --deterministic --optimization-counter-threshold=50 --enable-inlining-annotations --ignore-unrecognized-flags --packages=/b/swarming/w/ir/.packages /data/local/tmp/testing/test/out.dill ; echo AdbShellExitCode:  $?
Stdout:
../../runtime/vm/object.cc: 18706: error: unreachable code
Aborted
AdbShellExitCode: 134
ExitCode: -6
Time: 0:00:01.212162

--- Re-run this test:
python tools/test.py -n dartk-android-product-arm ffi/structs_test

Test command locally (with an Android 64bit device):
(I'm unsure how to get the same amount of reporting, but if I add prints to the commands I get the following.)
$ tools/test.py -n dartk-android-product-arm -j1 --verbose ffi/structs_test
Test configuration:
    dartk-android-product-arm(architecture: arm, compiler: dartk, mode: product, runtime: vm, system: android, preview-dart-2)
Suites tested: ffi
Found 1 Android devices.
Running "vm_compile_to_kernel []" command: DART_CONFIGURATION=ProductAndroidARM /usr/local/google/home/dacoharkes/dart-sdk/sdk/pkg/vm/tool/gen_kernel --no-aot --platform=out/ProductAndroidARM/vm_platform_strong.dill -o /usr/local/google/home/dacoharkes/dart-sdk/sdk/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_structs_test/out.dill /usr/local/google/home/dacoharkes/dart-sdk/sdk/tests/ffi/structs_test.dart --packages=/usr/local/google/home/dacoharkes/dart-sdk/sdk/.packages -Ddart.developer.causal_async_stacks=true
Running "vm_compile_to_kernel []" command: DART_CONFIGURATION=ProductAndroidARM /usr/local/google/home/dacoharkes/dart-sdk/sdk/pkg/vm/tool/gen_kernel --no-aot --platform=out/ProductAndroidARM/vm_platform_strong.dill -o /usr/local/google/home/dacoharkes/dart-sdk/sdk/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_function_structs_test/out.dill /usr/local/google/home/dacoharkes/dart-sdk/sdk/tests/ffi/function_structs_test.dart --packages=/usr/local/google/home/dacoharkes/dart-sdk/sdk/.packages -Ddart.developer.causal_async_stacks=true
Running "adb_precompilation" command: Steps to push Dart VM and Dill file to an attached device. Uses (and requires) adb.
Executing adb -s 21d9aecb shell rm -Rf /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Executing adb -s 21d9aecb shell mkdir -p /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Executing adb -s 21d9aecb push out/ProductAndroidARM/dart /data/local/tmp/testing/dart
Executing adb -s 21d9aecb push /usr/local/google/home/dacoharkes/dart-sdk/sdk/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_structs_test/out.dill /data/local/tmp/testing/test/out.dill
Executing adb -s 21d9aecb shell export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/data/local/tmp/testing/test;/data/local/tmp/testing/dart --deterministic --optimization-counter-threshold=50 --enable-inlining-annotations --ignore-unrecognized-flags --packages=/usr/local/google/home/dacoharkes/dart-sdk/sdk/.packages /data/local/tmp/testing/test/out.dill ; echo AdbShellExitCode:  $?
Done dartk-vm product_arm ffi/structs_test: pass
Running "adb_precompilation" command: Steps to push Dart VM and Dill file to an attached device. Uses (and requires) adb.
Executing adb -s 21d9aecb shell rm -Rf /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Executing adb -s 21d9aecb shell mkdir -p /data/local/tmp/testing/test ; echo AdbShellExitCode:  $?
Executing Skipped cached push
Executing adb -s 21d9aecb push /usr/local/google/home/dacoharkes/dart-sdk/sdk/out/ProductAndroidARM/generated_compilations/dartk/tests_ffi_function_structs_test/out.dill /data/local/tmp/testing/test/out.dill
Executing adb -s 21d9aecb push out/ProductAndroidARM/libffi_test_functions.so /data/local/tmp/testing/test/libffi_test_functions.so
Executing adb -s 21d9aecb shell export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/data/local/tmp/testing/test;/data/local/tmp/testing/dart --ignore-unrecognized-flags --packages=/usr/local/google/home/dacoharkes/dart-sdk/sdk/.packages /data/local/tmp/testing/test/out.dill ; echo AdbShellExitCode:  $?
Done dartk-vm product_arm ffi/function_structs_test: pass

This also passes the same flags (--deterministic --optimization-counter-threshold=50 --enable-inlining-annotations) locally as on the bot through adb.
/cc @sjindel-google Is there something I missed w.r.t. the ffi-product bot?
Otherwise it might be due to the specific hardware (my phone), and I can try to debug it on the bot.
Looking in to the code without being able to reproduce it locally
There aren't any obvious calls to AsInt64Value which only happen in defined(PRODUCT). The ffi uses it in various places, but none that differ based on mode.

This crash got fixed by enabling the constant update.

Blamelist: 9bbd3192f44f43e3d3091bb03c0e52a28051daf3 Fri Jul 12 12:32<EMAIL_ADDRESS>[dart] Enable constant-update-2018

ffi/structs_test
Crash -> Pass ✔

on configuration dartk-android-product-arm

What was the original problem?

I'm hoping to be able to find that out with the two different behaviors now, but I'm not able to reproduce it locally. If you have any suggestions on how to reproduce it locally, I'm open for suggestions.
I suspect it's something with constants being treated differently in product mode before the constant update. But it might also be a spurious correlation.

One possibility is that we are incorrectly allocating an instance of Integer -- you could add an assert to check for this and see if it catches on the CQ.

RawObject* Object::Allocate(intptr_t cls_id, intptr_t size, Heap::Space space) {
   // ...
  ASSERT(cls_id != Integer::kClassId);

The above does not get triggered on the bot, so it does not look like we actually allocate an Integer.
Commit 629f38cf87980a31e409e11d6f885e6c4f58614d made product-mode hit unreachable again. So it looks like a spurious correlation. Moreover, the debug-mode now segfaults.
I'll see if the debug-mode crash reproduces locally. Otherwise, I'll continue on the build bot.

Another thing you can try is adding an assert to Object::setRaw:
DART_FORCE_INLINE void Object::SetRaw(RawObject* value) {
  NoSafepointScope no_safepoint_scope;
  raw_ = value;
  if ((reinterpret_cast<uword>(value) & kSmiTagMask) == kSmiTag) {
    set_vtable(Smi::handle_vtable_);
    return;
  }
  intptr_t cid = value->GetClassId();
  // Free-list elements cannot be wrapped in a handle.
  ASSERT(cid != kFreeListElement);
  ASSERT(cid != kForwardingCorpse);
  if (cid >= kNumPredefinedCids) {
    cid = kInstanceCid;
  }
  set_vtable(builtin_vtables_[cid]);
#if defined(DEBUG)
  if (FLAG_verify_handles) {
    Isolate* isolate = Isolate::Current();
    Heap* isolate_heap = isolate->heap();
    Heap* vm_isolate_heap = Dart::vm_isolate()->heap();
    uword addr = RawObject::ToAddr(raw_);
    if (!isolate_heap->Contains(addr) && !vm_isolate_heap->Contains(addr)) {
      ASSERT(FLAG_write_protect_code);
      addr = RawObject::ToAddr(HeapPage::ToWritable(raw_));
      ASSERT(isolate_heap->Contains(addr) || vm_isolate_heap->Contains(addr));
    }
  }
#endif
}

The last debug build that crashed reproduced locally.
The test can be boiled down to this:
import 'dart:ffi';

const highAddress32bit = 0xFFFFFFF0;
const highAddress64bit = 0xFFFFFFFFFFFFFFF0;

final int highAddress =
    sizeOf<IntPtr>() == 4 ? highAddress32bit : highAddress64bit;

final Pointer<Int64> c1 = Pointer.fromAddress(highAddress);

final double ten = 10.0;

void main() {
  for (int i = 0; i < 300000; i++) {
    if (i % 1000 == 0) print(i);

    Pointer<Double> field = c1.cast();
  }
}

The bug happened when first a Pointer is allocated, and subsequently a Mint.
The above regression test relies on many allocations to trigger an garbage collection bug. We should improve this test by triggering a garbage collection on the n-th allocation. I'll land the fix today, and make a proper regression test when I get back.


how to set the analyzer for a field when i usespark-elasticsearch saveToEsWithMeta save to es

my question is how to set the analyzer for a field which's name is changed ,
code like as below:
val test ="性别"
val lbs: Map[String, String] = Map("create_time" -> "1491990010000",
"name" -> "年龄",
"values" -> "70后,80后,90后",
"label" -> "{values}",
"measure" ->"0.0",
"tags" ->"用户画像,灰度发布",
test->"性别,男")
sc.makeRDD(Seq((1, lbs))).saveToEsWithMeta("label_sys_label/label")
3q for help!!

the version is

org.elasticsearch
elasticsearch-hadoop
2.4.0

any one can help?

@tonyC2016 Please keep questions like these on the forum until they can be decided on if they are a bug or a new feature needed. We try to keep Github for reproducible bug reports and new feature tracking only.
If you are adding new fields to an index or creating new indices in Elasticsearch from ES-Hadoop you may need to set some sort of index template so that when the new index is created it has the correct mappings and analyzers that you would expect.


Error: Cask 'torbrowser' definition is invalid: Bad header line: parse failed

When I try "brew cask install torbrowser"
give me the following error : Error: Cask 'torbrowser' definition is invalid: Bad header line: parse failed
thanks for advanced

This error is not a bug, it's caused by an update to how homebrew-cask works. Details can be found here.
To fix the issue, please run:
brew update; brew cleanup; brew cask cleanup
brew uninstall --force brew-cask; brew update

If you are still having problems, open an issue with the appropriate template


Add surf option to boilerplate template

Introducing surf to the boilerplate template.
Tested locally on both vite and nextjs templates.

LGTM, left 2 comments

addressed


[git] Add command to create branch from specific ref

Closes #55732.

Thanks! :fireworks:
I've refactored it a bit and simply added it to the Create Branch... command and always show a HEAD option as the default, such that the user can just press Enter to continue creating a branch in the current HEAD.


pod try doesn't work

Hi,
I just push a new release with a fix. Please try again, I will reopen this issue if the problem persist.
Regards


Model and handlers for service member backup contacts [#156435027]

Description
This PR creates a backup contact model and associated handlers.
Reviewer Notes
This is our first attempt at implementing the model authorization that came out of the model ownership whiteboarding session on 4/10. It can be summarized by:

Handlers get models through fetchers that accept an authenticated user, fetchers handle authorization
Fetchers return generic errors, handlers have helper function to convert generic errors into responses
Creator functions are hung from parent models, if available, e.g. serviceMember.CreateBackupContact()

Code Review Verification Steps

[x] All tests pass.
[x] Code follows the guidelines for Logging
[x] The requirements listed in
Querying the Database Safely
have been satisfied.
[x] There are no aXe warnings for UI.
[x] This works in IE.
[x] Any new third party client side dependencies (e.g., login.gov, google analytics, CDN libraries, etc.) have been communicated to @willowbl00.
[x] Request review from a member of a different team.
[ ] (TRIAL) Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

great work!! just a few comments. :)


Show all ctnr fields

This is a re-submission of https://github.com/OSU-Net/cyder/pull/954

I get this error when trying to change the container:
Cannot change container; interface's container and system's container must be the same. Please change the system's container instead.
Also, the container list isn't sorted.

It's not allowing containers to change because of a backend issue... I'm considering whether it's worth it write a workaround, or if I should go ahead and remove containers from interfaces, already.

@akeym That is a separate issue and not one this pull request was made to handle. @murrown I think removing them altogether is the best choice. A lot of tests will need fixing though.

Alright, but I don't want this merged until that is fixed.. can be a different PR. I don't want to confuse our users by putting in an extra field that doesn't work.

@murrown I fixed the issue. The interface ctnr updating was occurring before the system was actually saved.This will have to do until ctnrs are removed from interfaces entirely.

Superseded by https://github.com/OSU-Net/cyder/issues/979


Taiko L2

Several fixes to allow a codex node to run against the Taiko L2 testnet.

Adds CLI parameter to specify a file containing an ethereum private key
Adds deployed address of Marketplace contract on Taiko
Waits on 1 confirmation for the ERC20 approve() call before doing further calls
Adds integration test that can be run with make testTaiko
Updates asynctest and ethers to the latest version

Seeing some failing tests in testSales after rebasing onto master, could be related to #494


Idea - backlinks and outgoing links

IDK but maybe the same functionality for backlinks and outgoing links is possible. Other plugin? On the other side of the page?
Ideas to develop. I love the concept.

The options on the other side of the page can  be set


Create Jupyter book for used methods

We currently have two separate documentation folders:

demo
point_and_click
workflows

I suggest to merge these into a Jupyter book and add another folder glossary. Thus, other notebooks could simply link there when it comes to explaining what certain plugins do. Thus we avoid having to explain the same stuff multiple times and make the documentation more relevant and shorter.

Jupyter book is now available here

@haesleinhuepf what do you think:
Should I move some of the documentation away from the frontpage and just put the link to the gh-pages there? Leave it like it is and just add more links to the correct places in the jupyter-books?

Also (side note): I think cross-links in jupter books are (field1:field2:field3)= - in the Bio-image analysis notebooks there is a dot rather than a colon in the pages. Maybe that's what causes the link to render?


ERROR: terminate called after throwing an instance of 'std::system_error'

Hi,
I suddently experienced this error:
terminate called after throwing an instance of 'std::system_error'   what():  open(/home/nguyen/tmp/tmp_gaq61e3/.torch_distributed_init): No such file or directory
Everything worked fine so far, we went to 5 kimg OK and suddenly met this error. Nothing has been changed as far as environment is concerned and the original StyleGANV2-ADA ran fine as well, same conda environment.

tick 6     kimg 24.0     time 25m 40s      sec/tick 147.6   sec/kimg 36.90   maintenance 0.3    cpumem 5.44   gpumem 5.18   augment 0.000
terminate called after throwing an instance of 'std::system_error'                                     what():  open(/home/nguyen/tmp/tmp_gaq61e3/.torch_distributed_init): No such file or directory                                                                                tick 7     kimg 28.0     time 28m 08s      sec/tick 148.3   sec/kimg 37.08   maintenance 0.3    cpumem 5.44   gpumem 5.18   augment 0.000                                                                 tick 8     kimg 32.0     time 30m 37s      sec/tick 148.7   sec/kimg 37.18   maintenance 0.3    cpumem 5.44   gpumem 5.18   augment 0.000

Weird thing is, the training somehow keeps going, no stopping, I am just worried that this may lead to something bad later.
** System:**

Ubuntu 18.04 w/ conda
4 GPUS GTX 2080ti, 1T RAM plenty free, 10T SSD plenty free
PyTorch 18 w/ CUDA 11.1
Again, everything worked fine with original StyleGANV2-ADA, I just switch to new folder which this repo

Any idea ?
Steve

Haven't seen this error...

did you solve this? I am encountering the same issue right now.


fixing python2 search

closes https://github.com/jupyterhub/binder/issues/84 and closes https://github.com/jupyterhub/binder/issues/82
cc @betatim

Can you help me grok why this fixes the bug? It looks like this does fix a bug, but one that should manifest by a runtime failure trying to run a requirements.txt that (possibly) doesn't exist.

so that was the error I was getting when trying to run the repo originally mentioned in that issue...a file in binder/ would be missed only if the python 2.7 runtime was used (because the location of the requirements.txt file was hard-coded)

Thanks! I thought we were trying to track down why files were being ignored in favour of a Dockerfile.

yeah - I don't have an example of a repo where this problem exists, so a bit hard to debug, but looking through the code I couldn't find a reason why a Dockerfile would be "preferred" over anything in a binder/  folder :-/

In general, we may need to up our game with some new tests now that we're using the buildpacks.

In general, we may need to up our game with some new tests now that we're using the buildpacks.

yeah I think that's fair - think it's worth creating an issue with a list of build pack combinations we'd like a test for? It's a bit of a combinatorial problem but I agree we should have a test for most common build pack combos

I was actually thinking about lower level tests. How long do tests take to run locally for you?

Hmmm, not super long though it's been a while since I've run them locally.
On Travis all builds take roughly 6 minutes. So locally probably like 4
min? (gonna head to bed now btw, if there's other stuff to take care of in
this PR I'll get to it tomorrow AM!)
On Tue, Mar 20, 2018, 10:16 PM Carol Willing<EMAIL_ADDRESS>wrote:

I was actually thinking about lower level tests. How long do tests take to
run locally for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jupyter/repo2docker/pull/277#issuecomment-374775831,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwSHeR5ow_pXP9DNzOPI0lUJarZxBWdks5tgX-0gaJpZM4SyZ6V
.

Fixing a legit problem!