System.NotSupportedException: Cake does not support /v2/gotodefinition

I noticed GoToDefinition is no longer working.
the log shows:
[dbug]: OmniSharp.Stdio.Host
        ************ Request ************
{
  "Type": "request",
  "Seq": 42,
  "Command": "/v2/gotodefinition",
  "Arguments": {
    "FileName": "d:\\_dev\\test\\Nils.Cake",
    "Line": 7,
    "Column": 23,
    "WantMetadata": true
  }
}
[fail]: OmniSharp.Stdio.Host
        ************  Response (0.5418ms) ************
{
  "Request_seq": 42,
  "Command": "/v2/gotodefinition",
  "Running": true,
  "Success": false,
  "Message": "\"System.NotSupportedException: Cake does not support /v2/gotodefinition\\r\\n   at OmniSharp.Endpoint.EndpointHandler`2.<HandleRequestForLanguage>d__20.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Host\\\\Endpoint\\\\EndpointHandler.cs:line 233\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Endpoint.EndpointHandler`2.<Process>d__16.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Host\\\\Endpoint\\\\EndpointHandler.cs:line 131\\r\\n--- End of stack trace from previous location where exception was thrown ---\\r\\n   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()\\r\\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\\r\\n   at OmniSharp.Stdio.Host.<HandleRequest>d__14.MoveNext() in D:\\\\a\\\\1\\\\s\\\\src\\\\OmniSharp.Stdio\\\\Host.cs:line 218\"",
  "Body": null,
  "Seq": 666,
  "Type": "response"
}

I'm guessing this came with https://github.com/OmniSharp/omnisharp-vscode/releases/tag/v1.23.13

I'm guessing this issue does not belong here, but rather in OmniSharp/omnisharp-roslyn.
Actually I'm guessing it's a regression from https://github.com/OmniSharp/omnisharp-roslyn/pull/2168 : I think OmniSharp.Cake/**/GotoDefinitionHandler.cs should have been modified as well in that PR.

This has been fixed, as of https://github.com/OmniSharp/omnisharp-roslyn/releases/tag/v1.37.17


-streams not working

Does it matter what location we put the -streams 4? It doesn't seem to be working. Plex is only allowing one stream at a time. Also even when telly is running Plex shows that the tuner is offline, but it's still functional.

First part of your issue: restart Plex when setting -streams
Second part: see #44, it's something we're working on

OK. Awesome. You rock. Where can I shoot donations?


Intervention action at response body phase.

This issue recaps how we currently handle the response body and both short-term and long-term proposals.
Previous discussion: https://github.com/corazawaf/coraza-proxy-wasm/pull/19.
Current Behaviour:

The interruption status returned by ProcessResponseBody is not checked.
ProcessResponseBody is called just to let Coraza log the error/intervention.
https://github.com/corazawaf/coraza-proxy-wasm/blob/main/main.go#L275-L284

Current Problem:

Rules capable of detecting sensitive data leaked are not effectively working: Coraza logs the intervention, but no action is taken in order to prevent the leakage of information.

Current Proposal:

If an intervention is raised during OnHttpResponseBody, we want to drop/replace/alter the response body. The client will receive a 200 status code, but with an empty/altered body. The original response body, containing the detected sensitive information, will not be sent to the client.

Future goal (not addressed in this issue):
The most suitable solution will be to stop delivering the headers until the check of the body completes.
Currently, it is not feasible and requires some work on proxy-wasm. It applies to both request and response flow.

Current Working proposal implementations:

Always remove the content-length header during OnHttpResponseHeaders and, if intervention is required during OnHttpResponseBody, replace the response body with an empty buffer.

	if err := proxywasm.RemoveHttpResponseHeader("content-length"); err != nil {
		proxywasm.LogCriticalf("failed to remove content-length from response headers: %v", err)
	}
	...
	if interruption != nil {
		proxywasm.ReplaceHttpResponseBody([]byte(``))
		return types.ActionContinue
	}

Are there known drawbacks to always sending back responses without the content-length header?

If intervention is required during OnHttpResponseBody, trigger handleInterruption.

	if interruption != nil {
		return ctx.handleInterruption(interruption)
	}

Internally handleInterruption calls SendHttpResponse. The latter should NOT be called during OnHttpResponseBody. Quoting: "You cannot use this call after types.HttpContext.OnHttpResponseHeaders returns Continue because the response headers may have already arrived downstream."
Hacky solution, I don't know the internals, I'm surprised that it is working (The client (curl) receives an empty body and does not hang waiting for a response): HTTP/2 200. curl: (18) transfer closed with 446 bytes remaining to read. I'm wondering about unexpected behaviours.

Replace the response body with random null data without altering the content-length.

	if interruption != nil {
		err = proxywasm.ReplaceHttpResponseBody(bytes.Repeat([]byte(`a`), ctx.contentLength))
		return types.ActionContinue
	}

Yolo solution: useless response and allocation to generate it, but content-length is not altered and the response does not contain sensitive data.

@anuraaga @jcchavezs @mathetake

Are there known drawbacks to always sending back responses without the content-length header?

We should have a tenet that when things are working fine, we don't touch user data. Removing the content-length always even when no problems is getting too invasive I think.
Practically speaking, it will force chunked encoding of the response which will add some overhead and otherwise perform differently then having a content-length.

Replace the response body with random data without altering the content-length.

Any solution that doesn't return 403 is already weird since we're altering the response in unexpected ways (proxy-wasm host should probably support this in the future, though it would still have the tradeoff of requiring buffering so would be an opt-out or opt-in option either way). So I don't see this as a weird solution compared to the others, there doesn't seem to be a reason to prefer sending no response bytes to sending null response bytes.

Curious about what is missing here @M4tteoP?

I was thinking about keeping it open to maintain the visibility of this issue, even if we have an initial solution in place. Removed the misleading (not addressed in this issue), I'm good also about opening a new sibling issue and closing this one.


Fix Spring AOT for ConnectionDetails beans

Fixes gh-861

I think #782 will fix this 😁 I noticed after the PR was merged

@eddumelendez nice, I hadn't noticed that! Without any public classes in the Testcontainers module, I believe the maven publishing task might fail the Maven Central criteria. I got similar issues before with other libraries. It's a bit counterintuitive.


Option to change app icon theme?

Describe your suggested feature
You posted two icons on the discord server with different colors. Please give us an option to switch between in the actual app.
I know this is rather unnecessary but the dark mode one looks cooler
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 0.15.3.
[X] I will fill out all of the requested information in this form.

There is no native way to support icon changing and I'll rather not go with the hack and workarounds.
P.S. The dark icon is for Preview builds

oh that's aight but telegram has it


Playorama 1.0.2 no longer 'seeing' any videos except the piano-cat sample

I updated to Playorama 1.0.2 today via Play.Date sideload, which worked fine. Unfortunately, after installing the update, Playorama no longer sees any videos stored in the Data\user.XXXXX.playorama folder (where XXXXX is my UserID).
I'm testing with both a video that was working nicely under 1.0.0, and with a new video I made and converted today. Neither video clip (just a simple rotating-logo animation) has audio, so I'm just placing the .pdv files into the folder.
Any tips or suggestions on how I can get this working again?

A couple quick updates / additions to the question above:

I also tried adding the videos to a "com.hteumeuleu.playorama" folder, but no difference in behavior
I uninstalled "Game Data" and deleted Playorama from within the Playdate settings and then reinstalled the app before replacing the video files in the folders. No change in behavior
I tried encoding the videos using 1bitencoder on macOS and installing them from there (including the .wav audio outputs too, both as .wav files and I tried renaming them to .pda), no difference in behavior
The log within Playorama only ever says "No error"

That's it, not sure what else I can try here!

Oooh, that makes sense, and also is something I was totally unaware of. I'll try changing the format-name of the data folder now, and (assuming it works), I'd suggest that you might update the instructions for the app to help the next confused beginner like me :-)

Update: that worked! Changing the folder name caused my videos to show up in Playorama.
One odd quirk: the version of the video I created using the online creator works fine, but the version I exported from 1bitvideo on macOS plays for maybe 1/2 second and then the whole Playdate crashes and reboots. Not sure what's up with that, but using the online converter seems to work without issue so I'm happy to proceed as-is.
Do let me know if I can be of any assistance re: the 1bitvideo behavior though, I love your app and I'm happy to help any way I can!

Sure, if you're able to share your pdv files directly I'd love to take a look.

Sure, I'll attach the one that seems to cause my PlayDate to insta-crash.
1bit-samurai.pdv.zip


Add support for primitive data types

I should be able to append primitives like a string. See below
int claimBlocks = 50;
Component text = TextComponent.builder("").append(claimBlocks, TextColor.YELLOW).build();

Didn't we decide against this? /cc @kashike


add devcontainer.json config for .net 6 -Codespaces

Why this change?
Enhance the PR review experience in GitHub Codespaces. Currently, review productivity is hindered because the default Codespaces container only has .NET 8. (While there is work to add .net 8 to dab, this fix enables .net6 (current) compat in codespaces).
Pros:

Everything that is needed is pre-installed in container:

.NET 6 SDK -> allows you to "view definition" and other symbols with intellisense
C# Dev Kit VsCode extension -> view solution , run tests etc.

No need to update this file when DAB is updated to build against .NET 8.

Cons:

Slower to load CodeSpace because container needs to build (1-3 minutes), but once built, easy access for future PR iterations.

What is this change?

Adds folder .devcontainer
Adds file devcontainer.json which

Uses .net 8 sdk image for future compatibility with DAB changes in #2140 .
Installs .net 6 SDK, Runtime, and ASPNETCORE
Installs C#devKit Vscode extension

Actual experience

When loading codespaces you'll see this container setup after selecting which codespace to create or reuse.

Auto install required extensions to make intellisense work:

Solution explorer works

Working intellisense!

You can speed up the codespaces build by pre-building the devcontainer on the repository. This is a trick I often use with various https://github.com/azure-samples repos.

If you need to test the .NET tool locally, you will need to add the .NET tools directory to the PATH which is not automatically done in the SDK repos:
{
    ...,
    "remoteEnv": {
        "PATH": "${containerEnv:PATH}:/root/.dotnet/tools"
    }
}

Please add to the PR description the docs referenced for this change

azp run

/azp run


rails 6 compat

Fixes version conflict when updating to Rails 6
Bundler could not find compatible versions for gem "thor":
  In Gemfile:
    foreman was resolved to 0.85.0, which depends on
      thor (~> 0.19.1)

    rails (~> 6.0) was resolved to 6.0.0, which depends on
      railties (= 6.0.0) was resolved to 6.0.0, which depends on
        thor (< 2.0, >= 0.20.3)

I highly suggest that you do not include foreman in your Gemfile. I've written up my reasoning here:
https://github.com/ddollar/foreman/wiki/Don't-Bundle-Foreman


revise updateState logic for ReactModalHostView (v2)

Summary:
Remove unneeded code around size calculation and old arch support

updateState was getting called unnecessarily in multiple places --> only call from onSizeChanged()

this is a reliable source for getting the content size area of the dialog used for Modal
remove code checking duplicated update

Old architecture cleanup

Remove Java implementation of ShadowNode

we already have logic to set the node size via UIManagerModule::updateNodeSize(). This  code is now group together in updateState() for both new and old architecture

This fixes issues with resulting from wrong size calculation:

having gaps at bottom when we set statusBarTranslucent to true
Modal cut off at bottom on Android 15 (drawn under bottom nav bar)

Changelog:
[Android][Fixed] - Modal statusBarTranslucent bug, Modal at bottom being cut off in Android 15 (without forced edge-to-edge)
[Android][Deprecation] - Deprecating ModalHostShadowNode and ModalHostHelper classes
Differential Revision: D62286026

Hi @alanleedev. I am running the latest RC (0.76.0-rc.6) and still have some problems with statusBarTranslucent={true} for Android modals.
<Modal
      animationType="slide"
      onRequestClose={onClose}
      presentationStyle="overFullScreen"
      statusBarTranslucent={true}
      transparent={true}
      visible={true}>
      {Modal content rendered here}
</Modal>

The modal doesn't draw under the status bar and additionally gives it a white background (at least that appears to be the case).

After the modal is dismissed, the status bar remains visible.

For reference, I am setting the status bar translucent with the StatusBar component provided by RN. If I instead use the StatusBar component to hide the status bar...
<StatusBar hidden={true} backgroundColor="transparent" barStyle={barStyle} translucent={true} />
...the modal problem does not show itself both when the modal is open and afterward.

Based on commit history, you seem to be the most familiar with this topic so I hope that you are able to look into it and help out! I appreciate any and all help you are able to provide!

Yeah I will try and get it posted tomorrow. The issue arises while the modal is visible and persists after it is closed. Only if I have the status bar hidden does the issue not show itself.

@alanleedev sorry for the alarm. Turns out I was missing a flag when using useAnimatedKeyboard from reanimated that was causing the status bar padding issue. Everything is good on the RN side.


Web Workers Implementation?

Since we do get that 'small' freeze when loading extremely large maps (it's to be expected), not complaining, but just a pet peeve and suggestion.
I was wondering if it's possible to implement something like:
https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Using_web_workers
To load up the map instead so we can free the main thread to get rid of that initial freeze? Possible? Thoughts? :) I'll take a look at it and see what I can do, but no guarantee's w/ my brain :(

Right now (unfortunately) the "freeze" comes from allocation of the sprite and texture objects, not necessarily large calculations. Right now it creates every texture your map has, as well as  Moving the calculations to the web worker still requires us to allocate those large objects on the main thread causing slowdown.
I'm working on v2 right now that has the concept of thin and lazy tiles. Which actually do not allocate textures until they are necessary for drawing.
For the following I used a 2556x256 tiles map on 1920x1080 screen with 16x16 tiles.
v1 would take ~500ms to setup the map textures and screen sprites (the freeze you see):

With v2 that setup takes ~25ms, no noticible freeze:

I'm hoping that by time I finish this up, the setup isn't necessary anymore. Or at least not as impacting. The next step after that is to try and make it a bit more data driven and have optimized renderers for both webgl and canvas that bypass sprite creation all together.

v2 is now released, should clearup most of these issues.


👷 Change build system from Poetry to Hatch

Poetry has a probrem in dynamic versioning even though poetry-dynamic-versioning can cover it.
We have to change version in pyproject.toml in order to reflect the version to a package directly installed from GitHub.
Flit seems nice, but we feel Hatch is becoming popular.
If Hatch becomes de-facto standard, it is good to deal with it as early as possible .

Codecov Report

Merging #21 (05d5941) into master (a2cf0ce) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #21   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          130       130
=========================================
  Hits           130       130

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Bugs when capturing screenshot

As you're selecting what part of the screen you wish to screenshot, it has a tendency to preview the wrong area. This doesn't occur 100% of the time, only sometimes. But when you release your mouse, it will screencapture the correct portion of your screen, just as it's previewing it screws up.
Context:
I have 3 screens @ 1080p, each sitting besides eachother.
CaprturingRegion with no parameters.
Observations: only happens when you're not dragging from top-left to bottom-right.

Fixed, version 1.1 released to nuget.


Feature/issue 1096 fix extra margin on status bar

@rasmus-storjohann-PG thanks for pointing that out. I think I have addressed this issue. I had also tried some of the solutions people were proposing in the thread below, such as setting properties in app.json, but it seemed like adding a Statusbar to the main component was the only one that worked:
https://github.com/expo/expo/issues/7635


Install extra command directories as python packages

These packages should be installed directly to ~/.dodo_commands/default_commands, using
pip install --install-option="--prefix=$PREFIX_PATH" package_name
(see http://stackoverflow.com/questions/2915471/install-a-python-package-into-a-different-directory-using-pip)

Fixed


Add hypsometric extrapolation methods

See discussion in PR #85. Some methods should be implemented to extrapolate hypsometric bins when they are missing at the bottom/top part of the glacier. @rhugonnet suggested the following:

0/ Existing: fill with the mean regional signal at a given elevation (Fanny's paper, and in Bob's paper, I don't like this much and I quote Bob's results here: the global hypsometric methods are furthest from the true values, which is perhaps not surprising in a region with a high variability of glacier elevation changes)
1/ Easy: loop through all local hypsometric pd.DataFrame, normalize those by their hypsometry and apply the area-weighted mean dhdt at the same normalized hypsometry to gap fill: at least a bit more representative of the regional signal independently of the elevation location at the scale of each glacier, but can still be off due to high inter-regional variability
2/ Best: loop through all local hypsometric pd.DataFrame, normalize by both hypsometry and elevation change and derive a mean normalized signal for the region; then, for glacier with local hypsometric (maybe with a criteria covering at least X% of the area), de-normalize by an estimated range of elevation change rate. If the rate at high elevation is not available (accumulation area full of voids), you can use the signal at half of the normalized elevation range, or maybe the best would be to use the mean elevation change rate at highest elevation captured in the region (essentially, you scale the regional hypsometric signal only by the dhdt in the ablation area of each glacier, keeping the accumulation dhdt similar than where measured for others in the region).

For 2/, the best would be a Bayesian approach, but you don't have time for this right now.
@adehecq I think that the detailed steps should be:
a) Normalize all local hypsometric estimation with more than X% coverage in accumulation area by both hypsometry and elevation change rate.
b) Derive area-weighted, mean hypsometric regional normalized signal from everything in a).
c) Derive area-weighted, mean regional elevation change rate at highest elevation from everything in a).
d) For glaciers with more than X% coverage in ablation area, de-normalize the regional signal of b) with the maximum dhdt of the glacier and the dhdt at highest elevation of c).
e) For glaciers with less than X% coverage in ablation area (glacier with pretty much no coverage), use with the mean elevation change rate in the region? (or for a more beautiful visual rendering, fill with the mean hypsometric signal without normalization)

@erikmannerfelt maybe #108 closes this?


Add documentation for Tableau Cloud

Is your feature request related to a problem? Please describe.
The examples here don't show how to install a web connector with Tableau Cloud, the hosted product.
Describe the solution you'd like
I'd like, if nothing else, a yes or no answer to the question: Can WDC's and this repo be used with Tableau Cloud. If the answer is yes, it would be great to include an example showing how to use it.

Hi @palewire - thanks for filing a request.
To the best of my knowledge, you cannot initiate or refresh an extract powered by a Web Data Connector in Tableau Cloud's web UI. Based on my research, you can create the extract and workbooks with Tableau Desktop and then publish the data source/workbook to Tableau Cloud however Tableau Cloud will not be able to refresh that extract without the use of Tableau Bridge: [source]

When you publish to Tableau Cloud, as a security measure, Tableau Cloud can't connect to or refresh an extract created by a web data connector. To refresh some web data connector extracts, you can use Tableau Bridge. For more information, see When to use Tableau Bridge to keep data fresh(Link opens in a new window) in the Tableau Cloud Help.

All that being said, thank you for bringing this to my attention. I will leave this issue open until we improve the documentation in this repo to address Tableau Cloud by name.


Add route to get customers by lastname  44

Codecov Report

Merging #65 (0522ffc) into master (aeeb2b0) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
+ Coverage   98.46%   98.49%   +0.02%
==========================================
  Files           7        7
  Lines         260      265       +5
==========================================
+ Hits          256      261       +5
  Misses          4        4

Impacted Files
Coverage Δ

service/routes.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add Pango lineages to the GenBank ingest

Helps bring the GenBank metadata up to feature parity with the GISAID metadata.

I suppose we could replicated the pattern we use with nextclade for pangolin (and clean this up a bit in the process). @WandrilleD is currently harmonizing these workflows. Elements of this PR could be useful:
https://github.com/nextstrain/ncov/pull/593


Report upper-bound memory usage based on SLM model lib

This PR introduces a command that reports the estimated upper-bound memory usage based on the metadata section of an SLM-compiled model.
Example:
>> python -m mlc_chat.cli.model_metadata /path/to/model_lib.so --memory-only
[2023-12-31 18:40:43] INFO model_metadata.py:49: Parameter size: 3885.14 MB
[2023-12-31 18:40:43] INFO model_metadata.py:58: Temporary buffer size: 7184.15 MB
[2023-12-31 18:40:43] INFO model_metadata.py:71: KVCache size when context/sliding window size is 4096: 512.00 MB
[2023-12-31 18:40:43] INFO model_metadata.py:79: Total memory usage: 11581.29 MB
[2023-12-31 18:40:43] INFO model_metadata.py:84: Tweaking `prefill_chunk_size`, `context_window_size` and `sliding_window_size` to reduce memory usage

Addresses both B1 and B2 in https://github.com/mlc-ai/mlc-llm/pull/1516#issuecomment-1872878113.
Another demo using Python API:
from mlc_chat import ChatConfig, ChatModule, callback
from mlc_chat.support import logging

logging.enable_logging()

MODEL="HF://junrushao/NeuralHermes-2.5-Mistral-7B-q4f16_1-MLC"

cm = ChatModule(
    MODEL,
    device="cuda",
    chat_config=ChatConfig(
        sliding_window_size=4096,
        prefill_chunk_size=1024,
        opt="O2",
    ),
)
cm.generate(
    "What is the meaning of life?",
    progress_callback=callback.StreamToStdout(callback_interval=2),
)

>>> MLC_JIT_POLICY=REDO python main.py

Followup on this issue. Because for iOS, android and web, we cannot simply read meta-data from the library(because they are cross compiled to form a different library). Would be great to print out those stats during mlc_chat compile, so we can fill in those numbers.
We can then work on more automated flow that generates the ModelRecord with the necessary information

cc @CharlieFRuan @junrushao @Kartik14 @davidpissarra

https://github.com/mlc-ai/mlc-llm/pull/1655


Fix spanish gerunds in index.html

See the Debian bug: https://bugs.debian.org/821788 from Santiago Vila:

The file doc/es/index.html.in is a translation of doc/index.html.in, but gerunds like "Using Network Printers" have been translated as "Usando [...]" (Spanish gerund, which is not the same as English gerund). English gerund is a noun, so it should be translated as such.

Thanks!


iosevka-bin: 16.8.2 -> 17.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for iosevka-bin is: "Slender monospace sans-serif and slab-serif typeface inspired by Pragmata\nPro, M+ and PF DIN Mono, designed to be the ideal font for programming.\n"
meta.homepage for iosevka-bin is: "https://be5invis.github.io/Iosevka"
meta.changelog for iosevka-bin is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 17.0.1 with grep in /nix/store/sym4zhviy4y5p1s9xip706k80afkfhqs-iosevka-bin-17.0.1
found 17.0.1 in filename of file in /nix/store/sym4zhviy4y5p1s9xip706k80afkfhqs-iosevka-bin-17.0.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
iosevka-bin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sym4zhviy4y5p1s9xip706k80afkfhqs-iosevka-bin-17.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sym4zhviy4y5p1s9xip706k80afkfhqs-iosevka-bin-17.0.1
ls -la /nix/store/sym4zhviy4y5p1s9xip706k80afkfhqs-iosevka-bin-17.0.1/bin

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

1 package built:

iosevka-bin

Maintainer pings
cc @cstrahan @montchr for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for iosevka-bin is: "Slender monospace sans-serif and slab-serif typeface inspired by Pragmata\nPro, M+ and PF DIN Mono, designed to be the ideal font for programming.\n"
meta.homepage for iosevka-bin is: "https://be5invis.github.io/Iosevka"
meta.changelog for iosevka-bin is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 17.0.2 with grep in /nix/store/f3gbycvqf2nfgv4pv7xpqij691rq6qx0-iosevka-bin-17.0.2
found 17.0.2 in filename of file in /nix/store/f3gbycvqf2nfgv4pv7xpqij691rq6qx0-iosevka-bin-17.0.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
iosevka-bin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f3gbycvqf2nfgv4pv7xpqij691rq6qx0-iosevka-bin-17.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f3gbycvqf2nfgv4pv7xpqij691rq6qx0-iosevka-bin-17.0.2
ls -la /nix/store/f3gbycvqf2nfgv4pv7xpqij691rq6qx0-iosevka-bin-17.0.2/bin

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

1 package built:

iosevka-bin

Maintainer pings
cc @cstrahan @montchr for testing.


Use Electron, NodeJs API in Renderer-process

You can see this template 👉 electron-vite-boilerplate
If you are learning Chinese 👉 请教一下vite-renderer.config中的resolveElectron函数. Or are you willing to help translation it. 🥺


Error: client and server don't have same version

After cloning and running vagrant up I get the following error:
vagrant up
Bringing machine 'rancher' up with 'virtualbox' provider...
==> rancher: Importing base box 'coreos-alpha'...
==> rancher: Matching MAC address for NAT networking...
==> rancher: Checking if box 'coreos-alpha' is up to date...
==> rancher: A newer version of the box 'coreos-alpha' is available! You currently
==> rancher: have version '353.0.0'. The latest is version '550.0.0'. Run
==> rancher: `vagrant box update` to update.
==> rancher: Setting the name of the VM: rancher_rancher_1420547237181_66568
==> rancher: Clearing any previously set network interfaces...
==> rancher: Preparing network interfaces based on configuration...
    rancher: Adapter 1: nat
    rancher: Adapter 2: hostonly
==> rancher: Forwarding ports...
    rancher: 8080 => 8080 (adapter 1)
    rancher: 22 => 2222 (adapter 1)
==> rancher: Running 'pre-boot' VM customizations...
==> rancher: Booting VM...
==> rancher: Waiting for machine to boot. This may take a few minutes...
    rancher: SSH address: <IP_ADDRESS>:2222
    rancher: SSH username: core
    rancher: SSH auth method: private key
    rancher: Warning: Connection timeout. Retrying...
==> rancher: Machine booted and ready!
==> rancher: Setting hostname...
==> rancher: Configuring and enabling network interfaces...
==> rancher: Running provisioner: shell...
    rancher: Running: inline script
==> rancher: Unable to find image 'rancher/server:latest' locally
==> rancher: Pulling repository rancher/server
==> rancher: 9ac2845c2bd20bd7978abd4c35353188e939a4d6138b005ad3654901707fff1d
==> rancher: Running provisioner: shell...
    rancher: Running: inline script
==> rancher: Unable to find image 'rancher/agent:latest' locally
==> rancher: Pulling repository rancher/agent
==> rancher: 2015/01/06 12:30:38 Error response from daemon: client and server don't have same version (client : 1.15, server: 1.12)
==> rancher: Please ensure Host Docker version is >= 1.3.2 and container has r/w permissions to docker.sock
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

chmod +x /tmp/vagrant-shell && /tmp/vagrant-shell

Stdout from the command:

Stderr from the command:

Unable to find image 'rancher/agent:latest' locally
Pulling repository rancher/agent

2015/01/06 12:30:38 Error response from daemon: client and server don't have same version (client : 1.15, server: 1.12)
Please ensure Host Docker version is >= 1.3.2 and container has r/w permissions to docker.sock

It appears that you have an old CoreOS vagrant image cached.  Try running 'vagrant box update'

That worked, thank-you.


No-IP language need to be set to English in order for the renewer to work

Hi Simao Silva,
during an unsuccessful attempt to get your renewer working, I noticed that the renew.py is searching for a button with the exact test "confirm" in the hosts list. This only is the case if the language is set to English in the No-IP settings.
After I changed to English language in my personal settings on No-IP homepage, your renewer worked like a charm.
As there might be more people running into the same issue, it might be worth mentioning this in the readme.
BTW: Excellent tool!
BR
Ralf

Instead of restricting exclusively to English maybe you can open a PR with the translated name of the confirmation button. Just add an or in the if condition.

It should now work with any language.


Threads are leaking if disconnecting service that was closed by the server

If reconnection is handled by the client code every call of disconnect() and connecting again creates a new thread while the old one still lives

please rerun the checks

please rerun the checks


Provider revoke grants created outside terraform.

Provider Version
0.30.0
Terraform Version
1.1.7
Describe the bug
Granting resources no longer working properly.
For example, I had a resource snowflake_account_grant created in an earlier version of provider 0.28.8
resource "snowflake_account_grant" "grantCreateDatabase"  {
    roles               = ["SANDBOX_SYSADMIN_ROLE"]
    privilege           = "CREATE DATABASE"
}

I upgraded snowflake provider to version 0.30.0 I modified the resource to compatibly with existing environment:
 resource "snowflake_account_grant" "grantCreateDatabase"  {
    roles               = ["SANDBOX_SYSADMIN_ROLE"]
    privilege           = "CREATE DATABASE"
    enable_multiple_grants = true
}

Unfortunately, regardless of what value I give for the parameter "enable_multiple_grants", each time resources created outside the terraform are selected for deletion.
13:57:23  Note: Objects have changed outside of Terraform
13:57:23
13:57:23  Terraform detected the following changes made outside of Terraform since the
13:57:23  last "terraform apply":
13:57:23
13:57:23    # module.snowflake.snowflake_account_grant.grantCreateDatabase has changed
13:57:23    ~ resource "snowflake_account_grant" "grantCreateDatabase" {
13:57:23          id                = "ACCOUNT|||CREATE DATABASE|SANDBOX_SYSADMIN_ROLE|false"
13:57:23        ~ roles             = [
13:57:23            + "DEVEL_ROLE",
13:57:23            + "DEVEL_SYSADMIN_ROLE",
13:57:23              # (1 unchanged element hidden)
13:57:23          ]
13:57:23          # (2 unchanged attributes hidden)
13:57:23      }

13:57:23    # module.snowflake.snowflake_account_grant.grantCreateDatabase will be updated in-place
13:57:23    ~ resource "snowflake_account_grant" "grantCreateDatabase" {
13:57:23        + enable_multiple_grants = false
13:57:23          id                     = "ACCOUNT|||CREATE DATABASE|SANDBOX_SYSADMIN_ROLE|false"
13:57:23        ~ roles                  = [
13:57:23            - "DEVEL_ROLE",
13:57:23            - "DEVEL_SYSADMIN_ROLE",
13:57:23              # (1 unchanged element hidden)
13:57:23          ]
13:57:23          # (2 unchanged attributes hidden)
13:57:23      }

I came across this too. Looks like the provider is unable to properly map from Terraform state to Snowflake state. Would be happy to pick this up as a first time contributor.

Hello I have the same issue.
Example : a USAGE privilege was granted on a database outside of terraform. Terraform will revoke this privilege.

This is interesting; I'm on Terraform version 1.0.11 and Provider version 0.31.0 and running into a very similar problem, but it runs in the opposite direction – no matter what value I pass to enable_multiple_grants, it seems that Terraform will not revoke grants that took place outside of Terraform.
As a test, I:

Created a role through Terraform called TEST_TERRAFORM_MANAGED_ROLE
Manually granted the role to USER_A and USER_B in the Snowflake web console
Created a Terraform resource block that looks like:

resource "snowflake_role_grants" "test_terraform_managed_role" {
  role_name = "TEST_TERRAFORM_MANAGED_ROLE"
  users = ["USER_C"]
  enable_multiple_grants = false
}

I then apply this change, at which point I would expect Terraform to have revoked the grant of TEST_TERRAFORM_MANAGED_ROLE from USER_A and USER_B. But when I run:
SHOW GRANTS OF ROLE TEST_TERRAFORM_MANAGED_ROLE;

it shows me that all grants have been left fully intact, i.e. USER_A, USER_B, and USER_C all still have been granted the role.
The only thing that I'm wondering is if this has something to do with the fact that USER_A and USER_B are not actually resources that we manage through Terraform (user accounts are created via an external mechanism), so maybe this is causing the resource to behave unexpectedly? Either way, it seems to me like the provider is not properly respecting the enable_multiple_grants flag.

Came here for the issue described by @TaliaSRTR.
As far as I can read the current ressource implementation it seems that the enable_multiple_grants is completely ignored.
Instead the ressource only considers the granted role/users referenced in. the ressource, and completely discard anything else. I expected enable_multiple_grants=false to enable the opposite behaviour and drop anything that isn't referenced in the ressource.

I also ran into this issue. For security's sake, I expect that if a user/role is granted a role outside of the terraform, then applying the terraform again will revoke grants as necessary to bring Snowflake into alignment with the code.
As of version 0.32.0 of this provider, no matter what I specify for enable_multiple_grants, it refuses to revoke grants given outside of terraform.

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.


[BUG]: Doxygen docs can't be generated

Description
Doxygen Doxyfile is outdated both in terms of doxygen version and in terms of being able to generate any docs. The input files are not correct since the v2 restructuring.
Reproduce
install doxygen and run doxygen Doxyfile from the icicle directory
Expected Behavior
Doxygen docs should be generated

Removed doxygen and don't plan on using it


Fix MyPy CI tests

For #8517

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Unit tests & system/integration tests are added/updated

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8518   +/-   ##
=======================================
  Coverage   59.11%   59.11%
=======================================
  Files         510      510
  Lines       23564    23564
  Branches     3813     3813
=======================================
  Hits        13931    13931
  Misses       8725     8725
  Partials      908      908

Continue to review full report at Codecov.

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


Use $.fn.css instead of $.fn.(show|hide)

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

License
MIT

I'm using jQuery 3.x, and the popup windows just won't show. I looked around google, and found that the solution is just by changing .show() into .css('display','block'). Now it's working perfectly with jQuery 3.x. I have tested it with jQuery 1.12.4 and jQuery 2.2.4, and it's still working normally.

@stash86 thanks for the PR!
Can you provide a jsfiddle to show the problem?
Thanks!

Sure, I'll try to create a jsFiddle to show the problem

Seems like my problem is a localized problem on my production site only. My localhost site and my jsfiddle shows no error using the original js file. You can delete this PR if you want.

Thanks @stash86!


Rotation Gizmo not drawn in Display Overdraw mode

Godot version
3.4.3, 3.4.4.rc1
System information
macOS
Issue description
The rotation gizmo is shown in every mode except Overdraw
Steps to reproduce
Click (E) to enter Rotate Mode, change the viewport to Display Overdraw and the gizmo will be missing

Minimal reproduction project
No response

This likely occurs because the rotation gizmo now uses a custom shader, and the overdraw shader is not designed to render this gizmo correctly.


Demo Lit integration

Demo lit integration once we upgrade to Vaadin 14.5: https://github.com/vaadin/flow/issues/8862

Vaadin 14.5 will only be released next month, which is too late. I'll only implement the lit demo for v19. This is therefore blocked by #14


Codec request - ALC271x with Layout_ID = 13 support

Unfortunately, on Acer Aspire V5-471G (Laptop) my audio layout_id = 13, still doesn't have support for AppleALC. Please, if you kind enough to add support for layout_id = 13. If you need anything particular I can do to help or contribute just mention me here.

If you guys need similar files for troubleshooting. I can refer you here from @insanelydeepak

ALC271x = ALC269. Use layouts from WiKi https://github.com/vit9696/AppleALC/wiki/Supported-codecs for ALC269


Better handling of updated changeset list

It would be great if we could poll the changeset list and see if any change sets arrived since the user last refreshed.

This was added in #67


[Storage][DoNotMerge] Add test for batch request with token auth

This is a test for issue #9534
This should only be merged once a fix has been released in azure-core and these tests pass when run live.

Can one of the admins verify this patch?

/azp run python - storage - tests

/azp run python - storage - tests

/azp run python - storage - tests


Crash during migration

Hi Kryojenik,
Thank you so much for continuing the miniloader in 2.0. I have a big Pyanodons save with maybe more than 10k miniloader entities that failed to migrate with this mod and it crashes with the following error. Could you help us take a look? Thanks again!
The mod Modernized Miniloader Migration (1.0.0) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event miniloader-migration::on_gui_click (ID 1)
Index out of bounds: 4
stack traceback:
[C]: in function 'create_entity'
miniloader-migration/scripts/from-miniloader.lua:113: in function 'replace_miniloader'
miniloader-migration/scripts/from-miniloader.lua:490: in function 'handler'
flib/gui.lua:177: in function 'handler'
core/lualib/event_handler.lua:47: in function <core/lualib/event_handler.lua:45>
You can find the save file here:
https://drive.google.com/file/d/1hNSS3NVG9JMWf3oiWkRz2Mdsdz-7XWed/view?usp=sharing

Thanks for the report and the save file.   I believe this one is squished.

I migrated successfully after the update. Thank you so much and happy holidays!


Unable to load static roles

Hello,
I try to configure and install opendistro 1.13.2(main branch) in a Kubernetes environment thanks to helm chart.
I used Self Signed certificates.
When i try to execute manually the securityadmin by executing the command below in master node terminal, The execution failed
/usr/share/elasticsearch/plugins/opendistro_security/tools/securityadmin.sh -f  /usr/share/elasticsearch/plugins/opendistro_security/securityconfig/config.yml -icl -nhnv -cert  /usr/share/elasticsearch/config/admin-crt.pem  -cacert /usr/share/elasticsearch/config/admin-root-ca.pem -key /usr/share/elasticsearch/config/admin-key.pem
result:
Open Distro Security Admin v7
Will connect to localhost:9300 ... done
Connected as CN=ADMIN
Elasticsearch Version: 7.10.2
Open Distro Security Version: <IP_ADDRESS>
Contacting elasticsearch cluster 'elasticsearch' and wait for YELLOW clusterstate ...
Clustername: elasticsearch
Clusterstate: GREEN
Number of nodes: 3
Number of data nodes: 1
.opendistro_security index already exists, so we do not need to create one.
Populate config from /usr/share/elasticsearch
Will update '_doc/config' with /usr/share/elasticsearch/plugins/opendistro_security/securityconfig/config.yml
SUCC: Configuration for 'config' created or updated
FAIL: 3 nodes reported failures. First failure is FailedNodeException[Failed node [PUi6M4oOTTi6oPt2qSJZDQ]]; nested: RemoteTransportException[[opendistro-opendistro-es-master-0][<IP_ADDRESS>:9300][cluster:admin/opendistro_security/config/update[n]]]; nested: NotSerializableExceptionWrapper[static_resource_exception: Unable to load static roles];
FAIL: Expected 3 nodes to return response, but got 0
Done with failures
Is this a common issue?
What could cause such issue?
Any idea?
Regards

It was a configuration issue.


Possible to expose the client for NextJS SSR?

Nextjs example only shows client side rendering.  Is it possible to expose the client so we can use getServerSideProps(), or is SSR not currently supported?

See
https://github.com/nhost/nhost/releases/tag/%40nhost%2Fnextjs%400.2.0
and
https://docs.nhost.io/reference/nextjs


Option for White Background

Challenge Golf Code has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

function golfScore(par, strokes) {
  // Only change code below this line
  if(strokes == 1){
    return "Hole-in-one!";
  }
  else if(strokes <= par -2){
    return "Eagle";
  }
  else if(strokes == par - 1){
    return "Birdie";
  }
  else if(strokes == par){
    return "Par";
  }
  else if(strokes == par + 1){
    return "Bogey";
  }
  else if(strokes == par + 2){
    return "Double Bogey";
  }
  else{
    return "Go Home!";
  }
  // Only change code above this line
}

// Change these values to test
golfScore(5, 4);

This is not a bug, but a request about the UI of FreeCodeCamp. I am a school teacher using this as part of a web class, and I'm projecting the code to a white board. It is very difficult for students to see the code on a black background.
And anyway, a key preferences that people as coders have is the color background they code with (fully acknowledging that that dark is ergonomically better).
Could you implement a choice of UI to be dark/black or light/white?

@johnrayworth thanks for the issue and sorry to taking so long to get back to you about this. Have you tried night mode? If you go to Settings, you can turn on night mode, which might be helpful when projecting on a white board.

There is also a hotkey combination of g t n that will also toggle night mode. Let us know if this solves your problem.


[Question]: some Deprecated

Please check the following before submitting a new issue.

[X] I have searched the existing issues.
[X] I have carefully read the documentation.

Please select for which platform(s) you need help

[ ] Android
[X] iOS
[ ] Linux
[ ] macOS
[ ] Web
[ ] Windows

Your question
Is there a plan to fix these?

Version
10.1.0

Hi @blondie63,
We are currently working on a large refactor of the plugin and this will definitely be part of the refactor.
Note, that all API's that are marked deprecated are behind if statements testing the iOS version to make sure the API is available. Unfortunately Xcode will not detect this and keeps reporting the deprecated API's.
I'll keep this issue open for reference purposes.


BindlessArray fails to load correct values in DX with AMD GPUs

Hi, the following test code gives wrong result in AMD GPUs but works normally with Nvidia ones.
#include <luisa-compute.h>
#include <dsl/sugar.h>
#include <gui/window.h>

using namespace luisa;
using namespace luisa::compute;

int main(int argc, char* argv[])
{
	Context context{ argv[0] };
	auto device = context.create_device("dx");
	auto stream = device.create_stream();
	auto device_image1 = device.create_image<float>(PixelStorage::BYTE4, 1280, 720, 0u);
	auto bdls = device.create_bindless_array();
	auto buffer = device.create_buffer<float4>(4);
	std::vector<std::array<uint8_t, 4u>> d{ 1280 * 720 };
	std::vector<float4> a{ 4 };
	a[0] = { 1, 0, 0,1 };
	a[1] = { 0, 1, 0,1 };
	a[2] = { 0, 0, 1,1 };
	a[3] = { 1, 1, 1,1 };
	stream << buffer.copy_from(a.data())<<synchronize();
	bdls.emplace(0, buffer);
	stream << bdls.update() << synchronize();

	Kernel2D kernel = [&]()
	{
		Var coord = dispatch_id().xy();

		UInt i2 = ((coord.x / 15) % 4);
		auto vertexArray = bdls.buffer<float4>(0);
		Float4 p = vertexArray.read(i2);
		device_image1.write(coord, make_float4(p));
	};
	auto s = device.compile(kernel);
	stream
		<< s().dispatch(1280, 720)
		<< device_image1.copy_to(d.data())
		<< synchronize();

	Window window{ "Display", uint2{ 1280,720 } };
	window.set_key_callback([&](int key, int action) noexcept
	{
		if (action == GLFW_PRESS && key == GLFW_KEY_ESCAPE)
		{
			window.set_should_close();
		}
	});
	auto frame_count = 0u;
	window.run([&]
	{
		window.set_background(d.data(), uint2{ 1280,720 });
	});
}

When running this code you should be able to see RGB + white color bars, however if you run with an AMD gpu you can only see red, black, green.
From what I've been experimenting with, this seems due to READ_BUFFER(ba11,l12,l9,bdls0) in generated HLSL code. When indexing variable (l9 here) it cannot give expected result, however if we change it to like READ_BUFFER(ba11,l12,2,bdls0) it works.
Any clue on this bug? Thanks.

We have tested this instance in next-branch with new dx backend implementation. Under Radeon RX 6900 XT the result is exactly same as under RTX 3080. In next-branch, we are using new version DXC: https://github.com/microsoft/DirectXShaderCompiler/releases/tag/v1.7.2212
So we may need more experiments to find out whether the problem is from hardware,, driver, or compiler.

@MaxwellGengYF Thank you for the information.
Sorry I forgot to mention we were using master branch for dev at the moment. I also tried with the latest dxc but no difference. Do you think we should switch to the "next" branch? There seem quite some changes so I am a bit hesitating.

Firstly master-branch will definitely become deprecated in the future. But the next-branch is under massive refactoring. We are revising some APIs and currently only DirectX backend is functional (other backends still require some time to repair).. BTW, python binding is currently only supported by xmake compiling workflow: https://xmake.io/

Yes that's the problem stops us from sticking with latest commit. Even we just started to move to luisacompute within one month we've achieved quite some good result. Switching to next-branch might need some extra effort to us. Anyways, is it possible for you to check the instance above with master branch and see if it can be fixed easily? If not, we might have to start adopting next-branch.

Yes, we are looking into this. The main difference between the branches is that we switched to Byte Addressed Buffers to implement the Bindless Array. We will investigate if this causes the different behaviors…

Hi, @swfly
We switched to the ByteAddressedBuffer implementation in 6fa4cdb and it seems to solve the issue on our AMD hardware (6900XT). Would you please try if it works on your side?

@Mike-Leo-Smith Wow that's amazing, didn't expect it to be done so quickly. Thank you so much.
I am running the test code on a 7900xt and it works well. However, when I go back to the actual renderer, it's still buggy (while the behavior is different from that before this modification), and driver crashes. I think there could be some other problem to be solved, not sure if it's still related to bindless array though. Please let me confirm first and then come to close this issue.


refactor(FormControl): update FormControl to use CSS Modules behind flag

Closes https://github.com/github/primer/issues/3812

Changelog

New

Changed

Update FormControl and shared internal components to use form control behind feature flag

Removed

Rollout strategy

[x] Minor release

https://github.com/primer/react/pull/5438/commits/ab12bdea817581a1606265cce91c91d4b51a47fb adds the missing sx prop that was not being passed in for vertical layouts


Raise a Warning or Error when lon/lat in grids are not monotonically increasing

As shown in https://github.com/OceanParcels/parcels/issues/1360#issuecomment-1738535942, Parcels can throw a confusing OutOfBoundsError when there are negative gradients in the longitude or latitude grids.
It would be good to capture these on FieldSet creation, so that users know that they can expect erroneous behaviour.

Hi, I would like to contribute to this issue if it’s still open. I have tried to add such a warning message in my fork in the FieldSet class, but I’m not sure if I understood the task correctly. So I’d be happy if you let me know how you want it to be implemented, and then I can try to work on it. (https://github.com/Anna7651/ParcelsF/tree/lonLatMonotone)

Hi @Anna7651 , thanks for your interest in this issue. I've taken a look at your fork, and I think we can make the diff smaller. I think something along the lines of the below would greatly simplify things. Thoughts?
import numpy as np

x = np.linspace(-5, 5, 100)
y = np.linspace(-5, 5, 100)

X, Y = np.meshgrid(x, y)

def is_monotonic(array: np.ndarray, increasing: bool = True):
    """Checks that a provided array is all monotonically increasing/decreasing along all axes."""
    if increasing:
        def f(arr): return arr >= 0
    else:
        def f(arr): return arr <= 0

    for axis in range(array.ndim):
        monotonic_along_axis = f(np.diff(array, axis=axis)).all()
        if not monotonic_along_axis:
            return False
    return True

assert is_monotonic(x)
assert is_monotonic(X)
assert not is_monotonic(-x)
assert not is_monotonic(-y)
assert not is_monotonic(-X)
assert not is_monotonic(-Y)

Z = X ** 2 + Y ** 2
assert not is_monotonic(Z)

Regarding the "monotonically increasing" argument (cc @erikvansebille ), I'm not 100% sure if this issue is correctly scoped. If we have an arbitrary curvilinear grid, do the values necessarily need to be monotonically increasing? See the image below from the original discussion, where the latitude isn't monotonically increasing

Thanks for this input, @JamiePringle, @VeckoTheGecko and @Anna7651! I agree that perhaps the issue is not that 'non-monotonic' grids are not parsed well, but rather that parcels sometimes has difficulties with 'complicated' grids (however defined). I remember trying to create a simple minimal example that breaks; and couldn't create a grid that would cause the errors in https://github.com/OceanParcels/Parcels/issues/1360#issuecomment-1738535942.
I propose to pause this development fore now; and wait until we either encounter a simple minimal example where this breaks; or get support for unstructured grids into (hopefully!) Parcels v4.


FFMPEG Documentation/Assistance

How set up FFMPEG for Isorender
Drag FFMPEG.exe into your instance folder.
For example, C:\Users\aria\curseforge\minecraft\Instances\Latest Iso will be the place you have FFMPEG.exe
How to use FFMPEG animations using Isorender
View here: https://docs.wispforest.io/isometric-renders/options/#animation-options
You have 2 new options and 2 new buttons:
Animation Frames and Animation Framerate
Export Animation and APNG
Clicking Export Animation will play the animation frames at the desired framerate (configurable).
Clicking APNG will simply change the output format between APNG - GIF - MP4
(I personally have never heard of APNG but hey if it works it works)
Additional Notes
I wrote this in hopes of helping others figure out this entire thing.

Thank you for catching that.


Not detecting right number type

Hello, I'm converting JSON to C# and if decimal number is x.0, it's detected as long instead of double.
I understand that at the moment of the convertion as it has no decimal value or is 0 it's like an integer but the result may be different and therefore it could hold a decimal value.

Interesting! I know why this happens with 0.0.


Linux version

A Linux port would be appreciated.

Sadly this seems to target direct x only and nothing open :(
Maybe it can be run in proton

this is being worked on: #77


/hpm install - show and sort by count of downloads

It would be great to see the most popular plugins at once.

Good idea!, im looking for the way for implementing it, have any idea?

For getting count of downloads: https://api.npmjs.org/downloads/point/last-week/hain-plugin-github

good! thanks! i should be working on it

Its implemented with v0.2.0


Server doesn't start with bcrypt passwords in routes section

[x] Defect
[ ] Feature Request or Change Proposal

Defects
Versions of gnatsd and affected client libraries used:
$ gnatsd -DV
[33] 2018/09/13 14:34:20.615593 [INF] Starting nats-server version 1.3.0
[33] 2018/09/13 14:34:20.615659 [DBG] Go build version go1.11

Steps or code to reproduce the issue:
Create a NATS cluster and configure the routes section inside the cluster section with bcrypted passwords. For example:
cluster {

  listen: localhost:4244 # host/port for inbound route connections

  # Authorization for route connections
  authorization {
    user: route_user
    # ./util/mkpasswd -p T0pS3cr3tT00!
    password: $2a$11$xH8dkGrty1cBNtZjhPeWJewu/YPbSU.rXJWmS6SFilOBXzmZoMk9m
    timeout: 0.5
  }

  # Routes are actively solicited and connected to from this server.
  # Other servers can connect to us if they supply the correct credentials
  # in their routes definitions from above.

  routes = [
    nats-route://user1:$2a$11$xH8dkGrty1cBNtZjhPeWJewu/YPbSU.rXJWmS6SFilOBXzmZoMk9m@<IP_ADDRESS>:4245
    nats-route://user2:$2a$11$xH8dkGrty1cBNtZjhPeWJewu/YPbSU.rXJWmS6SFilOBXzmZoMk9m@<IP_ADDRESS>:4246
  ]
}

Expected result:
The gnatsd server starts successfully.
Actual result:
Gnatsd server does not start properly and it logs error like this:
2018/08/31 03:35:14.616494 [ERR] Error trying to connect to route: dial tcp: address tcp/$2a$10$pu3tmi8HDmqqBF5OJftln.Q5cdft9HI: unknown port
2018/08/31 03:35:15.616674 [ERR] Error trying to connect to route: dial tcp: address tcp/$2a$10$pu3tmi8HDmqqBF5OJftln.Q5cdft9HI: unknown port
2018/08/31 03:35:16.617854 [ERR] Error trying to connect to route: dial tcp: address tcp/$2a$10$pu3tmi8HDmqqBF5OJftln.Q5cdft9HI: unknown port
2018/08/31 03:35:17.619660 [ERR] Error trying to connect to route: dial tcp: address tcp/$2a$10$pu3tmi8HDmqqBF5OJftln.Q5cdft9HI: unknown port
2018/08/31 03:35:18.621928 [ERR] Error trying to connect to route: dial tcp: address tcp/$2a$10$pu3tmi8HDmqqBF5OJftln.Q5cdft9HI: unknown port

On nats urls, the password is the actual password. In the case above the password used is the encoded (armored) version of the password which will never match the password.
Effectively, the route configuration cannot really have an encrypted password for routes, because the same configuration that specifies the encoded password must use it.

Bcrypt does not work that way, it allows the recipient to not store a plaintext password and to attach a 'cost' to checking if a password is correct. In the section you are referencing that is where your server is acting like a client to the remote server, so it needs to be the actual password. I don't like plaintext passwords in config files at all, so you could assign it to a variable or we will introduce nkey support soon.

The solicited route URL cannot embed an encrypted credential in the url. You have to provide the clear text credential. The cluster authorization user credential for the current server is encrypted and will be honored. The only way to not embed the clear text version of the credential is to provide it as an argument as a command line when you start the server. You'll have to be careful or that credential can be leaked as well.

I see! Thanks for the response. The problem passing the password as an argument is that it can be seen with a ps command.

The password can be an environment variable that is de-referenced inside the config file.

Thank you very much!
Solved the route in plain-text problem by creating a environment variable like so:
export ROUTE1=[nats-route://nats-cluster:password@<IP_ADDRESS>:4222]
With the config file as below:
cluster {       # cluster configuration
  listen: localhost:6223
  routes = $ROUTE1
}

Running the server only with the config file:
gnatsd -c cluster1-nats.conf

Great!


Centralize output folder

This PR centralizes the output folder for all jobs that use the driver (+ the TC example).

Looks good to me!

bors r+


Missing Timestamp on current message

When I am in a chat, the messages received does not display a timestamp

This is by design. Each "conversation" gets a timestamp rather than each individual message.

What's your use case for individual message timestamps? Tell us a little more about the issue you're having and we'll figure out a way to solve it.

two thoughts. the grouping has some time setting of how much time passes before we consider it a new group. don't know if that could use some tweaking?
we could perhaps make each comment div have a title attribute with the timestamp which means you'd see it on hover. dunno if that would be irritating or not.

@davidbanham  That's not how "Hangouts" works...Each portion of message gets how long time passed from the time message was sent. Then we can discuss upon "1min ago", respect @ hh:mm.
Skype does the same, Whatsapp too, and any chat client should report this (but that's IMHO)
In any case this is not to be intended as an issue but as an enhancement. So repo's admins may "label" this as enhancement.

apple's message app does it this way. less clutter.

Imho, its important information, but it may be hidden until message will be hovered.


Change pre-event email to markdown and add image

So this was actually solved in cb06343d0eda3c4c028b084db70cf55d01f02569 -- we now have HTML and plain-text versions of all event-related emails. (see https://github.com/StudentRND/Clear/blob/cb06343d0eda3c4c028b084db70cf55d01f02569/app/views/emails/preevent_html.twig)

(I think using markdown to generate HTML makes sense sometimes, but in the particular case of images, if you're loading the text version of the email it looks weird.)


Category admin fails to load once populated

The category admin list/overview page (not the detailed / edit cat page) fails to load category contents once it's populated.
It shows all the normal admin UI components but simply shows a loading animation in place of categories as if requesting content from the server.
I've tried on my project implementation of Sulu and also with a fresh install from sulu-standard (today) - same result.
Environment:
Localhost, Mac Air 2012, El Capitan,
Stock PHP 5.5 install, MYSQL, Chrome,
Sulu-standard
Hope this can be fixed in a future update.

 GROUP BY c0_.id ORDER BY c0_.depth ASC' with params [5]:\n\nSQLSTATE[42000]: Syntax error or access violation: 1055 Expression #2 of SELECT list is not in GROUP BY clause and contains nonaggregated column 'sulu.c1_.translation' which is not functionally dependent on columns in GROUP BY clause; this is incompatible with sql_mode=only_full_group_by"}
wondering if it's my mySQL config...

Yup, not working on sulu cloud either.

In fact I've tried viewing articles I've created - doesn't work, viewing tagged articles I've created and tagged - doesn't work and now getting a bunch of permission denied errors. I thought this bundle was "production ready"?

@camburns which version of mysql do you use? could you try to disable sql mode "only_full_group_by"?

Mysql version:
+-------------------------+------------------------------+
| Variable_name           | Value                        |
+-------------------------+------------------------------+
| innodb_version          | 5.7.11                       |
| protocol_version        | 10                           |
| slave_type_conversions  |                              |
| tls_version             | TLSv1,TLSv1.1                |
| version                 | 5.7.11                       |
| version_comment         | MySQL Community Server (GPL) |
| version_compile_machine | x86_64                       |
| version_compile_os      | osx10.9                      |
+-------------------------+------------------------------+

Solution:
run in MySQL
SET global sql_mode = '';
however you may want to do some reading about sql_mode first:
http://www.mysqlfaqs.net/mysql-client-server-commands/what-is-sql-mode-in-mysql-and-how-can-we-set-it

Thanks @wachterjohannes, however the issue in sulu-cloud still exists in the PGSQL DB - is this something I should address in the sulu-cloud repo? Also concerned this issue may deter sulu newbies - should these queries be rewritten for stricter mysql_modes?

@camburns we use standard sql_mode wich is fully supported by doctrine. if you want to use another sql_mode you have to address issues about that maybe in doctrine.
PGSQL is currently not fully supported because of another bug which was found by @carltongibson in #522

https://github.com/sulu-io/sulu/issues/1797

Closing this issue in favour of https://github.com/sulu/sulu/issues/1797


fix: bump mehari to 0.21.1 fixes effect prediction

@Nicolai-vKuegelgen @xiamaz note that 0.21.1 release is still pending in bioconda so this will only run a bit later today.


Throughput lower than expected

In my test environment I've tested Solrs client for maximum throughput (requests per second).
I got that my upper bound is nearly 1600 req/s.
As a comparison I ran plain Solrj client in the same scenario and I got 1500 req/s. Good.
Now comes the strange part of the story (at least for me).
In my tests, when I load balance requests against multiple instances of Solrj (running in the same JVM) throughput rises - for ex. with 3 solrj instances I get up to 3000 req/s.
On the contrary using multiple instances of Solrs client does not give any performance boost: total throughput is always up to 1600 req/s.
Are there some Solrs configurations I can try to increase throughput ?

Can you share the code for the benchmark, or at least the construction of the AsyncSolrClient instances?
What I'd really like is to have a self contained benchmark that we could analyze/improve together! :-)

This is the base code I use to do my simple tests:
https://gist.github.com/fsantagostinobietti/a51dbd8304f6f5db042d#file-solrs_benckmark-java
It's in Java since I'm quite new to Scala and code base in which I need to use solrs is mainly Java. :)

Thanks! I didn't go though all of it, but AFAICS you're using only a single instance of AsyncSolrClient, and not multiple instances as I'd have expected according to your description. So how exactly are you testing with multiple instances? And how does the test for solrj look like?
I'd try to introduce a common interface for solrs and solrj, so that the same test can be run with both and differences from the test cannot influence the results. Because solrj has a sync interface, you need to make it async - you can just use Future.successful for this.
Regarding solrs throughput, because this should depend on the underlying AsyncHttpClient, you could to use a custom AHC and play with its configuration. You can pass a configured AHC to solrs like this:
AsyncHttpClient ahc = new AsyncHttpClient(new AsyncHttpClientConfig.Builder().build());
new AsyncSolrClient.Builder(new RoundRobinLB(servers)).withHttpClient(ahc).build();

Out of interest: why are you using java for the test? Would you use solrs with java as well?

You are right, I posted only the single instance version of the code. This is the version with multiple solrs clients :
https://gist.github.com/fsantagostinobietti/80b485f5130330b2e0b9#file-solrs_multiple_clients_benchmark-java
this is the same as previous one, the only difference is that an array of solrs clients is created and then each virtual user has one client assigned to (in round robin way).
Another test done, simpler then this, was to run at the same time 3 instances of the first benchmark code (the single instance one). So I had  3 distinct JVMs each running the test.
Again the same result : total throughput does not change.
Thanks for your suggestion about AsyncHttpClient configuration. I'll try it and let you know if I succeed. :-)
PS. In my company code is written mainly in Java. For this reason I want to try solrs with Java, at first.

How does the test with solrj look like? I'd still favor using the same benchmark code for both clients, otherwise it's hard to compare things...
Re Java: perhaps it makes sense to create a java friendly api and provide this as an alternative...?

What I already find suspicuous is this Await - it's always dangerous! ;-)

ok, I will write a unique benchmark for both solrj/solrs clients. So it will be easier for results comparison.
In my tests each virtual user send a request and than waits for the response, then sends a new request, etc. Load is increased by increasing the number of virtual users.
This is done easily with solrj since it has sync api. Whereas in solrs I had to use the Await command.
Do you think it can generate performance issues? What could I use instead?
PS. Java api for Solrs? I like it :+1:  :D

Using solrs makes sense if you can make use of its async api, i.e. you are using some web framework that can handle async responses (like e.g. play framework). If your app is running in a synchronous/blocking context (you must return the result directly, not wrapped in a Future) I don't see that much value in using solrs, and I'd recommend to use solrj. Or what's your motivation to use solrs?
What's the environment (web framework, sync/async) in your case?
What are your throughput requirements?
Regarding the number of virtual users: typically there would not be one solr client instance per user, but only a single instance. A user request is typically represented by a thread, which then accesses the solr client.
To simulate multiple users, I'd say that a ExecutorService of a fixed size N could be used, so that for at max N users queries are simulated.
PS. would you like to submit a PR for a java api, as a basis for discussion? :-)

@fsantagostinobietti I've created a gist that allows to compare solrj/solrs using the same code.
Implementation notes:

starts an embedded solr server
only a single instance of solrj/solrs is used, the number of concurrent queries can be changed (to simulate concurrent users)
solrj.query is wrapped with a Future (Future.successful, so that no additional thread is created)

Usage:

in the solrs project copy to src/test/scala/io/ino/solrs/Benchmark.scala
add connectInput in (Test,run) := true to build.sbt
run sbt
in sbt start the benchmark with test:runMain io.ino.solrs.Benchmark 4 solrj to simulate 4 concurrent users with solrj, use "solrs" instead of "solrj" to test solrs

On my laptop (4 cores with hyperthreading) with 8 concurrent requests I got 8833 req/sec for solrs and 4783 req/sec for solrj (both running for ~30 seconds).
https://gist.github.com/magro/42c32863178bc9cbc126

Thanks for sharing your benchmark code.
I'm trying to compile it, but I have following errors :
object SolrUtils is not a member of package io.ino.solrs
[error] import io.ino.solrs.SolrUtils._
not found: value SolrRunner
[error]   val solrRunner = SolrRunner.startOnce(8888)
...

Am I missing something?
Anyway, I will have to change it a little bit since I need to connect to SolrCloud cluster (it is part of my CDH - Cloudera distribution).
This is my scenario. I need to expose REST APIs for querying CDH cluster. I want the REST service to be completely asynchronous when making requests to back-end (solr and hbase).
REST front-end is a Jboss Wildfly instance configured for handling requests asynchronously. Hbase is accessed with asynchbase library and Solr is queried with your Solrs lib.
Business logic to compose data from hbase and solr is implemented with Akka.
Everything is working fine, but at high throughput rates solrs becomes a bottleneck. That's why I started benchmarking solrs and solrj.
My requirement is to get at least 4000 req/s. With solrs for me max rate was 1600 req/s.
Currently I've replaced solrs with solrj+Future (to make it async), but this cost me a lot of threads. It's not a scalable solution :(
I'm looking for a better solution with solrs, which I think is the right library for the job.

object SolrUtils is not a member of package io.ino.solrs
It seems as if you've placed Benchmark.scala under src/main/scala, this must be src/test/scala...

Hbase is accessed with asynchbase library and Solr is queried with your Solrs lib. Business logic to compose data from hbase and solr is implemented with Akka.
Thanks for sharing this, then infact you can benefit from async libs.

Everything is working fine, but at high throughput rates solrs becomes a bottleneck.
Are you sure that solrs is the bottleneck and not the solr server? Do you have a real life example that shows how solrs is setup and used, and how it's becoming a bottleneck?

@fsantagostinobietti I just updated the gist to use solrcloud: https://gist.github.com/magro/42c32863178bc9cbc126 - this should then be easily changed to use your existing cluster.

Yes, Benchmark.scala was under src/main/scala. Sorry, my fault :(
Now it builds and runs. I'm going to test solrcloud benchmark in my cluster.
Talking about a possible bottleneck: I excluded Solr server since using (multiple instances of) solrj client I get more than 3000 req/s. Thus the server is able to handle all of them.
I still think it can be a misconfiguration/misuse of solrs in my project. For this reason using your Benchmark in my cluster is a precious test. I'll post results asap ...

You can also share code that shows how you're using it (in akka?) so that I can have a look.

Here are the results of your benchmark code running connected to my Solr cluster.

#virtual users
solrs (req/s)
solrj (req/s)
solrj w/ 3 instances (req/s)

5
831
801
2206

10
1570
1260
3342

20
2105
1228
3271

40
1437
1248
3235

100
1641
1237
3302

Solrs client has max throughput (2100 req/s) with 20 virtual users. With more users asymptotically reach 1600 req/s value.
Solrj client has a different behaviour: quickly gets its max throughput (1250 req/s); which is lower than solrs one.
Last test for me is the most important. I ran (on the same host) 3 concurrent solrj benchmark. Throughput is computed summing up each single benchmark throughput. In this scenario max value is 3300 req/s.
These results confirm my previous tests. I have the feeling that solrs could have an higher throughput, but for some reason it cannot. I've no clue about it at the moment.
What do you think of these results ?

Before thinking further about this: can you do the same for solrs, run 3 concurrent solrs benchmarks?

And can you please test using the embedded solrcloud (solrrunner), so that we can compare our numbers?
How many cpu cores do you have btw?

I've updated results table with a new column (solrs with 3 instances)

#virtual users
solrs (req/s)
solrs w/ 3 instances (req/s)
solrj (req/s)
solrj w/ 3 instances (req/s)

5
831
2166
801
2206

10
1570
1439
1260
3342

20
2105
1542
1228
3271

40
1437
1693
1248
3235

100
1641
1838
1237
3302

A clarification about tests with multiple instances: #virtual users is the number of users for each instance. So the total number of users is 3x for tests with 3 instances.

I ran the embedded tests with the same configuration as yours  :
> Running io.ino.solrs.Benchmark 8 solrs
Success rate: 15344 req/sec

> Running io.ino.solrs.Benchmark 8 solrj
Success rate: 4558 req/sec

My PC has 8 virtual CPUs running on 4 cores.

Ok, so you have a rather good throughput on localhost, probably this is cpu bound - you should see a nearly maxed out cpu usage.
Changing to remote solr this is dropping significantly, for both solrj and solrs, but much more for solrs. This seems to be i/o bound.
The main difference between solrj and solrs is blocking io vs non-blocking io.
Therefore I'd look into this layer, e.g. try to use another provider for async-http-client (AHC) than netty, or change the netty configuration.
Are you running linux? Then I'd be interested in the output of 'vmstat 1' while the test is running.
If you're running Windows I'd be interested how a test on linux looks like.

Which jvm are you running? Can you use a current jvm if this is not yet the case?

AFAICS it's possible to activate blocking io for netty: https://github.com/AsyncHttpClient/async-http-client/blob/1.8.x/src/main/java/com/ning/http/client/providers/netty/NettyAsyncHttpProvider.java#L242
Can you try that as well?

I'm running all my tests on Linux with JVM 1.8 (Java HotSpot 64-Bit Server).
https://gist.github.com/fsantagostinobietti/61d464b85dbe60338b1c#file-benchmark-vmstat
Here is the output file of 'vmstat 1' command used when running this benchmark on my cluster :
  Starting with solrs SolrS, running 100 queries in parallel
  ...
  Success rate: 1614 req/sec

I made another test which surprises me a little bit.
It makes no difference if I run concurrent benchmarks on the same machine or on different machines. The total throughput I can get is always the same (and values are exactly those in my last table).
These machines share nothing, they only access the same external solr cluster. Somehow they interfere each other when "talking" with solr servers.
A question about solrs:  your documentation says that current version does not support shard for SolrCloud. What does it mean? Could this affect performances?

Re vmstat: I don't see more from this what might be the issue - your system is more or less idle.
Re shard support: I think I mentioned this because the _route_ parameter is not taken into account, see also Advanced Distributed Request Options. But this cannot be an issue regarding throughput, I'd also assume that you're not using this.
Re throughput is the same from different machines: Is there some traffic shaping going on, or some firewall limiting i/o? Can you run the benchmark on one of the solr machines or from some machine within the same lan?

Maybe now I have a clue on what's going on.
I've made a tcpdump for both solrs and solrj benchmarks. In those dumps I can see that solrj opens a connection for each solr server in the cluster, whereas solrs connects only to 2 severs (don't know how they are chosen).
Our cluster has 18 servers and 9 shards. So in this scenario solrj has much more open connections in which it can send requests.
Hope this info could help. I would like to do some more tests (and the one you suggested me), but tomorrow I will be on vacation for a bit of time :boat: :palm_tree: ;-)

Great that you've found it, now I'll have to check solrs. Can you share the solrcloud setup with servers, collections, shards and replicas? The output of CLUSTERSTATUS should tell everything I need.
Vacation sounds great, mine will start next week :-)

This is the content of clusterstate.json file from SolrCloud admin :
https://gist.github.com/fsantagostinobietti/a584739b5b776c073122#file-cluster_status-json
(I was not able to use the CLUSTERSTATUS action command, but I think info are the same)

Thanks, that's what I need.

@fsantagostinobietti I just pushed 1.3.3, now all servers from all shards and replicas should be used.

Back from vacation I was very anxious to test 1.3.3 ...
Below updated comparison table solrs/solrj :

#virtual users
solrs 1.3.3 (req/s)
solrs 1.3.2 (req/s)
solrj (req/s)

5
842
831
801

10
1617
1570
1260

20
3159
2105
1228

40
5793
1437
1248

100
11681
1641
1237

I'm very satisfied of resulting throughput :+1: ! Great work!
Thanks a lot.

Great, so I'm closing this.
Just another question: we talked about a java api and I'm still thinking about it. Do you have any expectations for this that should be taken into account? Which kind of future would you prefer to get back from query, scala future, guava ListenableFuture or java8 CompletableFuture?

I found it was quite easy to use solrs from Java.
Scala future is ok (now I'm working with java7), but probably with java8 CompletableFuture will be the best option.
I had just one problem with default values during class CloudSolrServers instantiation. My code was something like this (a bit ugly) :
Duration zkClientTimeout = CloudSolrServers.$lessinit$greater$default$2();	// default value 2nd parameter
Duration zkConnectTimeout = CloudSolrServers.$lessinit$greater$default$3();	// default value 3rd parameter
Duration clusterStateUpdateInterval = CloudSolrServers.$lessinit$greater$default$4();	// default value 4th parameter
Option<String> defaultCollection = Option.apply(null); // None value
Option<WarmupQueries> warmupQueries = Option.apply(null); // None value

CloudSolrServers servers = new CloudSolrServers(
			zkQuorum,
			zkClientTimeout,
			zkConnectTimeout,
			clusterStateUpdateInterval,
			defaultCollection,
			warmupQueries  );

Java-friendly constructors could be enough for me.

Ok, thanks, I created issue #10 to track this.


PWM at 12kHz

Observed Castle Creations running PWM at 12k. Verify that PWM can be set to both 8kHz and 12kHz and probably 16kHz should be possible as well (w/o consuming too much CPU on the ISRs). Higher PWM should improve sampling of voltage/back-EMF. This one is probably going to need to wait in line behind e.g. #12

PWM is setup for 10k - see commit 165f17c1abdff90c892b4ccb9f599531f9d2a7c8


Case insensitivity is a lie, fix it.

case_insensitive=True does not work as intended. It appears to not care. This might be due to having a callable for get_prefix but who knows

Case insensitivity works on things implementing the GroupMixin, not prefixes. Look into casefolding prefixes within get_prefix to check things


Invalid characters in sequence's accession？

Hello, I am using the PhaBOX tool's PhaTYP function to analyze my sequence, and I am getting the message "Invalid characters in sequence's accession." What could be the reason for this? How should I adjust my file?

Could you please show the accession? The name after'>' in the fasta file.
BTW, there are some rules when using the webserver:

The contigs' accession should begin with a letter. Only number (such as >1, >2, >3) is not allowed.
Special characters such as '|', '~', '&', '$', ':', and '/' are not allowed in the contigs' accession/ID/name.

This is used to protect the webserver from attack.
If you would like to use your original accession, you can try the local version. Please check the Wiki for further information. The local version will provide users more flexibility and logs of outputs.
Best,
Jiayu


Add keynote Youtube video to JSON object and embed within Day 1

The video is still up for discussion but this should provide a first template for embedding videos recorded during the meeting. Could also be applied to individual demo recordings and other days

Deployed at https://snoopycrimecop.github.io/www.openmicroscopy.org/events/ome-community-meeting-2021/day1/

Looks good. Let's deploy and iterate with new content


work flow question.

Is flow works of this package (hdl_people_tracking)  dependent to the hdl_localization?
In other words, can hdl_people_tracking run without depending on hdl_localization and detect pedestrians?

The standard workflow of this package requires localization. But, in case the sensor is fixed, you can run it by feeding dummy static transforms to the nodelet instead without running the localization.

Thanks for the reply.
Do you have a compatible bagfile with your package that has a static Velodyne?

Here, a test bag for people tracking with a static velodyne.
https://drive.google.com/open?id=1oPj6xJ0VpvGc_u31EQuGAqiVMnbA5Kar
Try to launch "hdl_people_tracking_static.launch" with this bag. The first frame is treated as the background cloud.
rosparam set use_sim_time true
roslaunch hdl_people_tracking hdl_people_tracking_static.launch

rosbag play --clock hdl_500.bag

In the readme file in workflow image in Related work section I see the following procedures:
https://github.com/koide3/hdl_people_tracking/blob/master/data/figs/packages.png
hdl_graph_slam  --> hdl_localization --> hdl_people_tracking

But I could not find hdl_graph_slam name in package.xml ro CMakeList.txt files in hdl_people_tracking and hdl_localization packages.
So what is the hdl_graph_slam role in this package?

This package requires a map cloud ("hdl_localization/data/map.pcd" in the example) to perform people tracking with a moving sensor, and in our workflow, we use hdl_graph_slam to generate the map cloud. We didn't add it to package.xml so that you can use any other SLAM algorithms instead of it.

So is mean that this tracking pedestrian project works only with hdl_400.bagfile with the saved "hdl_localization/data/map.pcd" map? and if we want to test it on other environments we need to the hdl_graph_slam or any other SLAM algorithm?

Yes, you need to create an environmental map with any SLAM algorithm before running this people tracking. In case you want to run it in an unmapped environment, you need to replace the background subtraction code (The package first perfoms background subtraction, and then do clustering to detect human candidates). I saw a guy modified the code so that it filters out ceiling and floor points with a pass through filter, and I think it would be helpful for you.
https://github.com/tzven0/hdl_people_tracking/commit/874cd5e66f8bb612d191185f5730e1ed7c256832

The link for hdl_500.bag failed. Can u give me a new link for it?


New version: TensorKitManifolds v0.6.0

Registering package: TensorKitManifolds
Repository: https://github.com/Jutho/TensorKitManifolds.jl
Created by: @Jutho
Version: v0.6.0
Commit: 6b1f9b1f16370d0cdfeb576b3b4de722f3f23f1d
Reviewed by: @Jutho
Reference: https://github.com/Jutho/TensorKitManifolds.jl/commit/6b1f9b1f16370d0cdfeb576b3b4de722f3f23f1d#commitcomment-44591365

I am confused why 0.6 - 1 is not an upper bounded compat entry, but I will try differently. [noblock]


Add Code of Conduct

I have added Code of Conduct file.

@muthuannamalai12 update Contributions.md wrt your contribution.


[WIP] help: Replace /help/mute-a-topic with "Manage topic notifications" page.

Updates made in connection with the new "Unmute topic" feature. Posting for initial review in case we want to revise the approach, but known remaining todo items are:

[ ] Update page URL to match title
[ ] Update links to this page and add a redirect.
[ ] Add missing "unmute topic" icon.

Blockers for merging:

[ ] Unmute topic feature
[ ] #25127
[ ] https://github.com/zulip/zulip-mobile/issues/5691

Current page: https://zulip.com/help/mute-a-topic
The intro changes also update the intro for https://zulip.com/help/mute-a-stream to match.

Intro screenshot

Instructions screenshots

@timabbott This is WIP for technical reasons, but your review on the content would be appreciated.

@drrosa FYI, since I'll probably ask for your help with the to-do's once we are good to go content-wise. Also, any feedback on the content is appreciated. :)

I pushed changes addressing the feedback above, and updated the screenshot of the intro int he PR description.

I ended up renaming "Mute a stream" -> "Mute or unmute a stream" to make it more parallel; it does cover both topics. What do you think?

The content lgtm.

Great! @drrosa can you take over from here? I think the remaining tasks are:

[ ] Update links to use the new page names ("Mute or unmute a topic", "Mute or unmute a stream")
[ ]  Add missing "unmute topic" icon.
[ ]  Update commit message and squash commits.

Closing in favor of #25242.


收起功能和子项背景该如何做呢？

What kind of issue is this?

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use Stack
Overflow. https://stackoverflow.com/questions/tagged/MultiType

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Please provide
      your device name, device OS version, and describe the reproduction steps.

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

Description:
1、但是我的需求是有展开和收起功能，展开类似抖音但是抖音没有收起，该如何做呢
2、第二个type共用一个背景图该如何实现呢？二级评论有一个背景图

比如红圈部分共用一个背景
求解惑

list.addAll，然后 notify adapter 刷新页面即可。这很容易，但这里不是提供业务需求实现方案的，您需要自行动脑筋。


On-board Location Based Services

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5716
This is an experimental preview for @microsoft users.

PR being submitted for code review only.

Removed custom get_account. Now uses default get function from the SDK.
Removed Public Preview Terms prompt.
Removed docs on custom.py Already supplied on _help.py.

Note on custom get_account
Our default behavior does not return anything (like empty_on_404).
Referring to az storage account show --name nonexistentaccount, I raised a CLIError instead.
az cognitiveservices --name nonexistentaccount does the same, but with a traceback.
All three behaviors are due to the way how RestAPI specs (Swagger) file is defined.
Our RestAPI specs will be updated by next actual PR, so this code was only a placeholder.
My team suggested that we show 404 errors.
But if CLI convention follows otherwise, I can do that, as noted with other modules using empty_on_404. Please let me know which would be an appropriate response.
Note Public Preview Terms prompt
Our service is currently on a Public Preview, but our actual PR will occur on our General Availability.
This prompt was added as a placeholder until General Availability.

Hi @jp94 on the 404, here are some options:

vm show: returns nothing by using empty_on_404. This is (obviously) acceptable and may become the expected standard for the CLI.
storage account show: returns a CLIError. This is less common but currently acceptable.
cognitiveservices account show: stack trace. This is NOT accepted. Anything that throws a strack trace is considered a bug.

Officially the recommendation is to go with option 1 as it allows you to distinguish between "the command succeeded and the thing isn't there (which you can see by the output)" and "the command failed".

Cool. I will use empty_on_404 when we submit the actual PR.


Use a part object as the layout template's scope

Originally reported in https://github.com/dry-rb/dry-web/pull/19, I still think this is a reasonable change to make.

Instead of using a struct with a page accessor as the layout's scope, use a Rodakase::View::Part object like we already do for the non-layout templates.
This allows rendering of partials via == partial_name instead of having to chain them onto the page object (i.e. == page.partial_name).
The general upshot is that it makes authoring layouts much more consistent with authoring templates by making the same partial rendering behaviour apply everywhere.

Fixed in https://github.com/dry-rb/dry-view/commit/a7fd20dc86267668f3dfba58d5de9485fce09f6c


BlockStoreLoop.DownloadBlocks threw an unhandled exception

I freshly merged master and I randomly got this:
BlockStoreLoop.DownloadBlocks threw an unhandled exception
Exception: System.NullReferenceException: Object reference not set to an instance of an object.
  at Stratis.Bitcoin.Features.BlockStore.LoopSteps.DownloadBlockStep.d__6.MoveNext() in StratisBitcoinFullNode\Stratis.Bitcoin\Features\BlockStore\LoopSteps\DownloadBlockStep.cs:line 106
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
  at Stratis.Bitcoin.Features.BlockStore.LoopSteps.DownloadBlockStep.d__4.MoveNext() in StratisBitcoinFullNode\Stratis.Bitcoin\Features\BlockStore\LoopSteps\DownloadBlockStep.cs:line 66
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
  at Stratis.Bitcoin.Features.BlockStore.LoopSteps.BlockStoreStepChain.d__2.MoveNext() in StratisBitcoinFullNode\Stratis.Bitcoin\Features\BlockStore\LoopSteps\BlockStoreLoopStep.cs:line 24
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
  at Stratis.Bitcoin.Features.BlockStore.BlockStoreLoop.d__35.MoveNext() in StratisBitcoinFullNode\Stratis.Bitcoin\Features\BlockStore\BlockStoreLoop.cs:line 178
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
  at Stratis.Bitcoin.Features.BlockStore.BlockStoreLoop.<b__34_0>d.MoveNext() in StratisBitcoinFullNode\Stratis.Bitcoin\Features\BlockStore\BlockStoreLoop.cs:line 151
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
  at System.Runtime.CompilerServices.ConfiguredTaskAwaitable.ConfiguredTaskAwaiter.GetResult()
  at Stratis.Bitcoin.Utilities.AsyncLoop.<>c__DisplayClass8_0.<b__0>d.MoveNext() in StratisBitcoinFullNode\Stratis.Bitcoin\Utilities\AsyncLoop.cs:line 119

Possibly related to #268

@fassadlr is working on this

The issue is that the blockPairsToStore is cleaned but not initialized.
Maybe @fassadlr will bring back the original code or alternatively just remove any instance parameters as the steps should not carry any state.

@Aprogiena @dangershony I am fixing this now.


High refresh monitor support

Hey,
first of all, thanks a lot for this app!
I was wondering if there is a possibility to enable high refresh support? I noticed it's capped at 60Hz

btw, i also noticed that the dark mode setting is not preserved when closing and opening the app

Oh. BTW. Can you tell me your device model and system version? @ignoxx

Thanks!

Oh. BTW. Can you tell me your device model and system version? @ignoxx

Macbook Pro 14" running on MacOS 13.3.1

It seems that 60hz is a limitation of Apple’s WKWebView.

It seems that there is some problem with the refresh rate on Safari, my 144hz monitor also has its fps halved on https://www.testufo.com/framerates#count=3&background=stars&pps=1440

i think for safari you can enable high refresh support in the developer menu, maybe there is a way to do it programmatlically?
https://forums.macrumors.com/threads/enabling-120hz-scrolling-in-safari.2323961/
Anyway thx for the effor


Fixes to Documentation typos in Update validation-basics.md

Edits:
Under "Displaying Errors" section:
"The validation-errors custom attribute implements the ValidationRenderer interface."
"It also automatically adds itself to the controller using addRenderer when its "bind" lifecycle..."

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

thanks @rajajhansi!