Getting "loop Promises must not throw!" error when running the example

Hi there,
First of all, I would like to thank you for this really awesome library!
I have started playing around with the example. The delay function rejects the promise with a 20% chance. If the promise gets rejected, the error gets printed to the console.
I am considering to choose this framework to manage my async flow as I'm porting my third-party JS application from Flux to Redux.
I can work this around by ensuring that my promises won't throw, but I cannot use this awesome library if it prints errors to the console. That's strictly forbidden in my third-party application.

Thanks for reporting this, and thank you for your interest @danielmohacsi!
Have you changed any of the example code or does it show this message just as it is?

@danielmohacsi I'm not seeing this happen in the example when installed without modifications from this repo. Did you remove or forget a .catch() call on any of your effect promises?

Thanks for getting back to me!
You're right, I have modified the example a little bit. I have changed the const Api (...) definition to the following one:
const Api = {
  shortIncrement: (amount) => (
    delay(10)
      .then(() => Actions.shortIncrementSucceed(amount))
      .catch((err) => {
        console.log(err);  // Logs undefined - OK
        Actions.shortIncrementFail();
      })
  ),

  longIncrement: (amount) => (
    delay(3000)
      .then(() => Actions.longIncrementSucceed(amount))
      .catch(() => Actions.longIncrementFail())
  )
};

Sorry for not stating this in the issue description!

And I found it, I'm not returning any action. Sorry, I'm an idiot.

@danielmohacsi even though it turned out that there wasn't a bug in the example or library functionality, I think we can consider this a bug in the error message in that it wasn't helpful enough to point out where the issue was. I think we can reopen this with the bug tag and I'll improve the error message to include this case where an action wasn't being returned as well as some possible other cases.
Thanks for your help finding this!

That's true, but you've written your comments to the example, I just did not read it.
Anyway, thanks for your help!

Fixed in v2.0.0!


Antibiotic View for beta-lactam remained "loading" status

Antibiotic View for beta-lactam did not return result. It remained "loading" status

This is because "beta-lactam" is not in the "antibiotics" table in solr.

There is some data discrepancy due to the nature of data. beta-lactam is not an antibiotic name, rather it refers a group of antibiotics containing "beta-lactam ring". see more https://en.wikipedia.org/wiki/Β-lactam_antibiotic

Antibiotic View page should handle this case instead of "loading" forever.

Yes, either we block it before making a link or add entries with some reasonable contents.

This is because beta lactam is a generic term referring to a class of antibiotics, hence it can not be linked to a specific antibiotic compound in pubchem. We need to go back to the original source of AMR data and see if they have provided more specific antibiotic name.
Sent from my iPhone

On Apr 3, 2017, at 12:38 PM, hyoo<EMAIL_ADDRESS>wrote:
Assigned #1462 to @mshukla1.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Two options. Either the page detects that the antibiotic name is not available in solr core and show a message.
OR, I can create an empty record for that antibiotic in the solr core with minimum info.
Sent from my iPhone

On Apr 3, 2017, at 1:02 PM, hyoo<EMAIL_ADDRESS>wrote:
Yes, either we block it before making a link or add entries with some reasonable contents.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.


Remove TiedEmbeddingTransformerDecoder (and GemmaTransformerDecoder)

Credit to @pbontrager for this.
To quote from https://github.com/pytorch/torchtune/issues/1447

Also just going to throw this out there: @pbontrager had suggested that we can do away with TiedEmbeddingTransformerDecoder entirely and instead do something like (crappy diff snippet from one of our model builders as an example)
    tok_embeddings = nn.Embedding(vocab_size, embed_dim)

	# How we do things for non-tied-embedding models (e.g. Llama)
    # output_proj = nn.Linear(embed_dim, vocab_size, bias=False)

	# What things would look like for Gemma or Qwen2 instead
    output_proj = lambda x: F.linear(x, tok_embeddings.weight)

    return TransformerDecoder(
        tok_embeddings=tok_embeddings,
        layers=layer,
        num_layers=num_layers,
        max_seq_len=max_seq_len,
        num_heads=num_heads,
        head_dim=head_dim,
        norm=RMSNorm(embed_dim, eps=norm_eps),
        output=output_proj,
    )

to build tied-embedding models directly in our TransformerDecoder class (though possibly without a lambda and maybe with a proper function). Main open question is whether this works with FSDP and checkpointing

To provide some background, currently, for models which have tied embedding-output projection weights such as Gemma and Qwen2, we define an entirely separate TransformerDecoder. This class is identical except for the final lines in the forward signature:
# in TransformerDecoder

        # shape: [b, s, out_dim] - out_dim is usually the vocab size
        output = self.output(h).float()
        return output

# in TiedEmbeddingTransformerDecoder and GemmaTransformerDecoder

        # shape: [b, s, out_dim] - out_dim is usually the vocab size
        output = F.linear(h, self.tok_embeddings.weight).float()
        ...
        return output

Making this change should be conceptually very straightforward, but will touch several parts of the codebase. Off the top of my head, and in no particular order:

Try parameterize the tied embedding using a callable (either a lambda as above, or something like

output_proj = partial(torch.nn.functional.linear, tok_embeddings.weight)

or as a proper function. Just for a single model (e.g. Qwen2 0.5B)
2) Test out a recipe with this builder and make sure things work OK - they should! Also make sure one of our distributed recipes works okay (we can help out here).
3) Extend the change to any other models using either GemmaTransformerDecoder, or TiedEmbeddingTransformerDecoder. All of these component and model builders should now just construct and return a TransformerDecoder.
4) CTRL+F GemmaTransformerDecoder and TiedEmbeddingTransformerDecoder. Eradicate. Docs, tests, __init__.pys. Be ruthless.
5) Probably something else here.
6) $$$

https://github.com/pytorch/torchtune/pull/1547
https://github.com/pytorch/torchtune/pull/1553

PRs landed. Thanks for the issue!! :)

Thank you so much for doing this


Add script to create mongodb provider account

This PR includes,

Script to include MongoDB provider account
Formatting changes
test-variables.yaml file to support provider creation
Resource and keyword file creation
Attached test results -
results_0428.zip

Taking this one down to incorporate more changes


expose interface for reading partial files

Suggested by @d-v-b
Internally, the behaviour for reading truncated data can be controlled internally - "error" just fails, "warn" succeeds (zero-padding) but complains about it, "ignore" zero pads silently - but that interface isn't exposed. An issue with allowing truncated reads is that it's harder to track where "missing slices" are if the slice isn't actually missing; additionally it means the round trip verification will fail (unless handled explicitly). I suppose the HDF5 could store whether it was a truncated read.

dat2hdf5 ... -F 0 fills truncated image data with 0; if not given, an error will be raised.


Cannot upgrade pip: kivy -m pip install -U pip

Kivy3 OSX cannot upgrade pip version shipped with it. It fails for some command line argument parsing reasons, probably due to how the underlying pip command is wrapper.
kivy -m pip install -U pip
2017-01-02 00:18:22.683 kivy[3752:2767258] NSDocumentController Info.plist warning: The values of CFBundleTypeRole entries must be 'Editor', 'Viewer', 'None', or 'Shell'.
2017-01-02 00:18:22.695 kivy[3752:2767258] App did finish launching
Collecting pip
  Using cached pip-9.0.1-py2.py3-none-any.whl
Collecting install
  Could not find a version that satisfies the requirement install (from versions: )
No matching distribution found for install
You are using pip version 7.1.2, however version 9.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

Workaround: use Homebrew installation method.

Try again with --verbose. This should at least print available version or something, not just "(from versions: )".

This issue was moved to kivy/kivy-sdk-packager#34


bug: meltano run doesn't support --environment argument

Meltano Version
2.2.0
Python Version
3.9
Bug scope
CLI (options, error messages, logging, etc.)
Operating System
Windows 11
Description
This command fails:
meltano --environment dev run tap-csv target-postgres
Due to some other limitations in meltano, I have to create multiple environments in my meltano.yml. Therefore, I would like to be able to call meltano run without having to change the env environment variable each time, and it seems I should be able to specify the --environment argument, but it fails with this error:
> meltano --environment dev-id run tap-csv target-postgres
meltano.core.environment.EnvironmentPluginConfig() argument after ** must be a mapping, not str

Looking at meltano --help, it appears this should work:
Options:
  --log-level [debug|info|warning|error|critical]
  --log-config TEXT               Path to a python logging yaml config file.
  -v, --verbose                   Not used.
  --environment TEXT              Meltano environment name.
  --no-environment                Don't use any environment.
  --version                       Show the version and exit.
  --help                          Show this message and exit.

Code
No response

This was an issue with my meltano.yml, not the CLI.
though it would have been helpful if the error message provided more useful information.

@seajhawk thanks for reporting. You're right, that error is caused by an unexpected data structure, but the message is not too helpful.
Perhaps having something like https://github.com/meltano/meltano/issues/2964 could help.

@edgarrmondragon - I appreciate you triaging my issue even though it was closed.
Yes, I agree that the validation is super important. However, that may require you to have schema validation for each of the underlying plugins... hmmm...
Here are a couple of recommendations:

implement the validation as a warning first (don't fail). There may be some things that your schema validation would fail on, but are working today. Erroring on new validation errors should be considered a breaking change.
Validate opportunistically. You can obviously dictate the overall schema of the file, but I believe it will be impossible to dictate the schema of the plugins. Therefore, the schema validation should support a plugin model where each of the taps/targets can provide a schema validation file and if present the schema validation process will use it. If such file is not present for a tap/target then the schema validation process will ignore that portion of the meltano.yml file.
Validate fully. The validation process should validate the entire file and then report all of the issues found rather than erroring and quitting on the first issue.
meltano cli override. The cli needs a flag to either skip validation or just ignore any errors. As this is an open source project, it's entirely likely a plugin contributor could make a mistake in validation schema and it should be super easy for someone running the CLI to ignore errors. Perhaps the best bet to handle #1 and #4 is a new argument --validation with possible values of "disabled" and "strict" where

by default if the argument is not used, validation runs and issues are reported as warnings, but the process continues
"disabled" - skips the validation entirely (doesn't run, no output)
"strict" - issues are reported as errors and the process stops after completing the validation
This should also be supported in the meltano.yml file as default_validation. I don't want to have to remember every time how to a particular project should be run. However, if I use the cli argument or an env variable, it will override the default behavior specified in the file - I believe this is just how default_environment works.

Per plugin override. There also needs to be a way to specify the "--validation" behavior on a per plugin-basis. Just because one plugin has broken validation, I don't want to disable or ignore all other validation. I believe this should be configured in each plugin's section of the meltano.yml.


game crash on start up

the game crashes on start up when it loads the mod
crash-2019-06-30_15.02.43-client.txt

Hi, you need Opencomputers for this mod. Thanks for the heads up, will update the readme.


Add Set Icon and Username ability; add missing onQueue declaration

I have the need to set the slack Display Name, and icon next to it, when my application posts into my alert channel; this code adds it, and updates the README for it.
It also adds a missing declaration for onQueue which was causing my IDE to not know that it was there.

Could you add tests for this one at well?

@freekmurze fixed an icon_url reference and added tests.   I cannot figure out how to run tests against the items in vendor/tests, so am unable to verify them.   But if they run for you, we should be good.   I have the current changes working in my dev environment here.

Thanks!

Thanks!

Awesome, thank you!

Thanks!

Awesome, thank you!


New version: Libtask_jll v0.4.1+0

Autogenerated JLL package registration

Registering JLL package Libtask_jll.jl
Repository: https://github.com/JuliaBinaryWrappers/Libtask_jll.jl
Version: v0.4.1+0

Similar to https://github.com/JuliaRegistries/General/pull/24916, this version can't be installed on Julia 1.5 and hence auto-merging can't be performed (I assume).


Dark mode in Plugins view issue

Looks like there are some areas across the board slight missing colors and all. Lovely theme though!

Great theme!
There is the same issue in theme switcher in bear dark theme

Fixed with latest updated


TestEgress/testEgressClientIP test failed probabilistically

Describe the bug

TestEgress/testEgressClientIP test failed
go test -v "antrea.io/antrea/test/e2e"  -provider=remote -remote.sshconfig=/var/lib/jenkins/ssh-config -remote.kubeconfig=/var/lib/jenkins/kube.conf -run=TestEgress/testEgressClientIP
2021-10-28 06:26:03.031812 I | Creating K8s clientset
2021-10-28 06:26:03.035307 I | Collecting information about K8s cluster
2021-10-28 06:26:06.391186 I | Pod IPv4 network: '<IP_ADDRESS>/16'
2021-10-28 06:26:06.391202 I | Service IPv4 network: '<IP_ADDRESS>/12'
2021-10-28 06:26:06.391206 I | Num nodes: 3
=== RUN   TestEgress
    fixtures.go:165: Creating 'antrea-test' K8s Namespace
    fixtures.go:128: Applying Antrea YAML
    fixtures.go:132: Waiting for all Antrea DaemonSet Pods
    fixtures.go:136: Checking CoreDNS deployment
=== RUN   TestEgress/testEgressClientIP
    egress_test.go:137: Failed to get expected client IP <IP_ADDRESS> for Pod localpod, stdout: <IP_ADDRESS>:40730, stderr: Connecting to <IP_ADDRESS>:8080 (<IP_ADDRESS>:8080)
        writing to stdout
        -                    100% |********************************|    20  0:00:00 ETA
        written to stdout
        , exeErr: <nil>, err: timed out waiting for the condition
    fixtures.go:371: Deleting Pod 'remotepod'
    fixtures.go:371: Deleting Pod 'localpod'
    fixtures.go:371: Deleting Pod 'fakeserver'
=== CONT  TestEgress
    fixtures.go:239: Exporting test logs to '/var/lib/jenkins/workspace/antrea-private-e2e-for-test/antrea-test-logs/TestEgress/beforeTeardown.Oct28-06-27-21'
    fixtures.go:343: Error when exporting kubelet logs: error when running journalctl on Node 'antrea-k8s-122-d-0', is it available? Error: <nil>
    fixtures.go:364: Deleting 'antrea-test' K8s Namespace
--- FAIL: TestEgress (86.45s)
    --- FAIL: TestEgress/testEgressClientIP (48.23s)
=== RUN   TestEgressToServerInCIDRBlock
    fixtures.go:69: Skipping test as it is not needed in IPv4 cluster
--- SKIP: TestEgressToServerInCIDRBlock (0.00s)
=== RUN   TestEgressToServerInCIDRBlockWithException
    fixtures.go:69: Skipping test as it is not needed in IPv4 cluster
--- SKIP: TestEgressToServerInCIDRBlockWithException (0.00s)
FAIL
FAIL	antrea.io/antrea/test/e2e	89.857s
FAIL

controller log:
I1028 03:50:33.285907       1 controller.go:477] Processing Egress egress-9l98w ADD event with selector ({&LabelSelector{MatchLabels:map[string]string{},MatchExpressions:[]LabelSelectorRequirement{LabelSelectorRequirement{Key:antrea-e2e,Operator:Exists,Values:[],},},} nil []})

agent log:
W1028 03:50:40.884772       1 egress_controller.go:729] Failed to start watch for EgressGroup, please ensure antrea service is reachable for the agent
I1028 03:50:40.884886       1 egress_controller.go:712] Starting watch for EgressGroup
W1028 03:50:40.884774       1 networkpolicy_controller.go:641] Failed to start watch for NetworkPolicy, please ensure antrea service is reachable for the agent
I1028 03:50:40.884979       1 networkpolicy_controller.go:632] Starting watch for NetworkPolicy
W1028 03:50:40.885206       1 networkpolicy_controller.go:641] Failed to start watch for AppliedToGroup, please ensure antrea service is reachable for the agent
I1028 03:50:40.885221       1 networkpolicy_controller.go:632] Starting watch for AppliedToGroup
I1028 03:50:40.895917       1 egress_controller.go:733] Started watch for EgressGroup
I1028 03:50:40.896067       1 egress_controller.go:761] Received 0 init events for EgressGroup

To Reproduce

go test -v -run=TestEgress/testEgressClientIP ./test/e2e -provider=kind
Expected

More flexible strategies are needed to ensure successful e2e testing or print out some useful information in test log.
Actual behavior

test failed.
Versions:

all versions may have the issue.
Additional context

The agent pods started slowly and started watching for EgressGroup 8 seconds after the controller received an Egress ADD event, which may be the root cause of testing failure.
A solution to this issue is changing the wait time in assertClientIP or printing out some useful information in the testing log.

@tnqn @liu4480 @antoninbas Could you help to take a look at this issue and PR?


Feature request

Please add support for converting the Poetry scripts syntax to Rye syntax.
The poetry syntax looks like this:
[tool.poetry.scripts]
my-script = "my_package_name:log_revision.py"

If you think this is a reasonable addition, I can probably submit a PR for this.

Thank you for the suggestion! I agree, and a PR would be great. Please go ahead.


merge upstream at https://github.com/prometheus/prometheus/commit/32f4bf5f6842311af8d2a7a0cc619efac4adb8bc

Major change is from native histograms custom buckets work https://github.com/prometheus/prometheus/pull/13662

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 11 committers have signed the CLA.:white_check_mark: zenador:white_check_mark: bboreham:white_check_mark: krajorama:white_check_mark: beorn7:white_check_mark: pracucci:x: liam-howe-maersk:x: rapphil:x: rexagod:x: bwplotka:x: RiRa12621:x: paulojmdiasYou have signed the CLA already but the status is still pending? Let us recheck it.


Dreamshaper api docs, restore dreambooth api docs

The dreambooth api docs were removed in a previous update to the salad cloud docs. they are restored in this update, and the method they are added was changed to be resilient against this in the future.

The dreamshaper 8 recipe is added to reference/recipes.

Since we're undeploying the dreambooth api, I'm going to go ahead and not restore the docs with this pr.


Connection dropping

I have had connections from connect-kdb-q to a remote q process drop after some idle time. The corporate firewall I have to be behind does this after 30 minutes. Is there any way to configure connect-kdb-q so it will send a TCP keepalive packet every x minutes?

Currently no. Try to add
@client.setKeepAlive(true, 1000)
in onConn function (line 611) in c.coffee
to see if it helps.

My network team configured this on the server side, but I did also try your suggestion prior to this and it seemed to work.


Support multiline TODOs

When I code I wrap all my TODOs as follows:
TODO: this is a long TODO
      which takes 2 lines
      actually 3.

TODO: one line TODO

TODO: another one line TODO

Unfortunately TodoReview sees only the first line of the multiline TODO. It would be great if it could take the other lines too.

Hey, saw this and was curious is this still closed to discussion?  I was looking at the engine and think its possible with some hard rules which no doubt wont work for everyone but if thats not possible would it be possible to create a way users could extend the extract() function to fit the multiline to their specs?  not sure how hard it would be but I could try it and PR if you arent totally adverse.  By the way, super great plugin! Im blown away by how useful it is
Would be dope if this could be to extendible with a few other things

@grahamannett I have a couple of problems with this feature:

Language specific: when scanning todos, we need a way to determine if something else below it is also a comment. This might work for something like // where we know that the initial line had that prefix, but consider something like a comment block, we would need language specific rules to determine what constitutes another line in each language, which is something I really don't want to maintain.

Performance: the way the current engine works, we get all the searched keys, concat them to a single regex and text each file, cleanly return those results. If we were to add an addition step where we also needed to compare each line underneath these regex matches, we would be potentially adding a lot of overhead, especially for users that have a significant amount of results. The overhead would come from doing additional regex matches to determine what prefix the initial matched comment had then searching from that.

Display: the current navigation system would have to be completely overhauled for something like this to be implemented. We currently have a very simple and efficient layout and selection system. This is the least of my concerns, but it would be an annoying one to tackle.

With all of that being said, I am always open to PRs. This would without a doubt require a large rewrite of the engine and I would have to be pretty strict in the review to ensure the engine still works flawlessly and at the same level of performance. A lot of people rely on this plugin and I would hate to break something in that rewrite. I just want to be up-front with you before you start any work on this, it will undoubtedly take a ton of your time (and accordingly mine to review it).
To specifically address your comment about the extract function being extended by the user, unless I am unaware of something newly added, I do not believe Sublime is capable of doing that at current. The only config options are through the standard JSON config. One could make the argument you could serialize a function then we could eval it in the plugin, but that is just prone to issues in my opinion.


Upgrade to Immer 2.x

This PR is a simple upgrade of the immer dependency. Version 2 of Immer only introduces a single BC break that shouldn't affect createReducer as it has to do with returning promises from a producer function.
From the migration section of the Immer README:

Migration
Immer 1.* -> 2.0
Make sure you don't return any promises as state, because produce will actually invoke the promise and wait until it settles.

Simple enough.  Thanks!


feat: Add HuggingFace runtime out-of-the-box support

Part of https://github.com/kserve/kserve/issues/3393.

[x] Tests
[x] Update Python spec
[x] Regenerate manifests

@terrytangyuan Looks great! can you help add an e2e test?

/lgtm
/approve


Fix Broken URLs

Fixing broken URLs for few documents

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

Docs Build status updates of commit b4546eb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/apps-upload.md
:white_check_mark:Succeeded
View

msteams-platform/overview.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Some characters not working on proposal description

Quotes & underlined text do not work on proposal descriptions.

fixed


RawNightPhoto

from constants.py line 438
RAW_NIGHPHOTO="98"
class RawNightPhoto:
	ON="1"
	OFF="0"

should be?
RAW_NIGHT_PHOTO="98"
class RawNightPhoto:
	ON="1"
	OFF="0"

it should.  189ef64


take out localhost links

The landing page for tutorial1 says

Each tutorial has its own url, e.g. for Tutorial 1
it is http://<IP_ADDRESS>:4001/tutorial1.

We need a global HOSTNAME or ADDRESS that we can use for constructing URLs when we redeploy the system on a different server (e.g. Amazon)

I have factored this out into tutorial1.url which us determined dynamically using window.location.href


Compose RunIf utilities

What does this PR do?
The Trainer's _RunIf calls Fabric's _RunIf
Pytest is kept to the tests directory exclusively.
cc @borda @carmocca @justusschock @awaelchli

I'm suspicious of the fact that this runif gets refactored so often. Now there are four different RunIf. The previous version was definitely easy to understand and adapt for other projects.

I did not change much (I believe) on the internals, mostly moved it to the package to be re-used...


Suraj Bokde

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one. Nice Profile btw 🚀


Sync open buffers after replacing

Hello,
I'm liking this plugin currently over Spectre, but I'm having a problem. When I run a find-and-replace operation, files open in other buffers that are affected are not reload until I manually call for it using :Edit. Would it be possible to add this somehow? Thanks.
grug version: 01f24e5
nvim version: v0.11.0-dev-131+gf55767afe

Hi there 😄
I think you can just set this option to get around the issue:
vim.opt.autoread = true

This works for reloading buffers affected by external changes in general.

I do have autoread as true, and if I modify the file with another editor while open in a buffer, it autoreloads. This isn´t true for modifications with grug

hmm, just re-tested it. If you alt-tab to another app and then back to neovim, it will show the updated text. I guess that's why I never saw the problem before. I'll try to see if there is a way to update the currently open buffers!

turned out I only needed to call vim.cmd.checktime(). Should be fixed now! Lemme know if you still see issues 😄


[issue-325] - Add tests to verify that HealthCHeckResponse has a name when built as recommended on the producer side

SSIA.
Fixes #325

Draft - still running TCKs

I agree with @xstefank, It wouldn't make sense to randomly generate a name for a Health check with no-name.
We should throw an exception if we detect a health check without a name.
+1 on updating the specification, with this detail.


PyPI wheel build for 1.7.1 can cause crash

The crash
python3: LineString.cpp:125: const geos::geom::CoordinateSequence* geos::geom::LineString::getCoordinatesRO() const: Assertion `nullptr != points.get()' failed.
Aborted

Experienced previously in: https://github.com/anitagraser/movingpandas/issues/81
Steps to reproduce the problem.
pip3 uninstall shapely
pip3 install shapely
Run sample code:
import cartopy.crs as ccrs
import geopandas as gpd
import matplotlib.pyplot as plt
world = gpd.read_file(gpd.datasets.get_path("naturalearth_lowres"))
world = world.to_crs(epsg=3395)
ax = plt.subplot(projection=ccrs.epsg(3395))
world.plot()
ax.tissot(rad_km=100, lats=[30,], lons=[-100,])
plt.show()

Operating system
Debian 10
Shapely version and provenance
1.7.1 installed from PyPI using pip3
The crash occurs when Shapely is built from wheel but not when reinstalled with the --no-binary flag; see workaround discovered by https://github.com/anitagraser/movingpandas/issues/81#issuecomment-688989821

Confirmed with Ubuntu using shapely 1.7.1 and 1.8a3 with GEOS 3.9.1, although the error message is slightly different:
/tmp/venv/lib/python3.9/site-packages/cartopy/mpl/geoaxes.py:799: UserWarning: Approximating coordinate system <cartopy._crs.Geodetic object at 0x7fb9905f9a90> with the PlateCarree projection.
  warnings.warn('Approximating coordinate system {!r} with the '
python: LineString.cpp:121: const geos::geom::CoordinateSequence* geos::geom::LineString::getCoordinatesRO() const: Assertion `0 != points.get()' failed.
Aborted (core dumped)

somehow the CAPI flops at https://github.com/libgeos/geos/blob/3.9/src/geom/LineString.cpp#L119-L124
Does anyone feel like tracing to see where the issue is raised in Cartopy? It's not obvious, so it could take a bit of digging/tracing. The issue is possibly an upstream bug, as the CAPI does not normally raise C++ exceptions.

This is sort of a known issue. Cartopy links GEOS. Shapely links GEOS and also dlopens it (which is going away in 2.0). We can't use these packages together unless they are linking compatible versions of GEOS. Even then, GEOS seems to be sensitive to attempts to dlopen after GEOS has been loaded (via the _speedups or _vectorized modules).


access to objects data struct?

Is there a way to access the underlying data struct of results objects?
For example, I want to implement a delete edit function using
override func tableView(tableView: UITableView, commitEditingStyle editingStyle: UITableViewCellEditingStyle, forRowAtIndexPath indexPath: NSIndexPath)
I have indexPath but no array of objects to get the object for deletion.
What is the recommended way of doing this in ABFRealmSearchViewController?

The underlying array of objects for the search results in handled by the internal RBQFetchedResultsController object:
@interface ABFRealmSearchViewController () <UISearchResultsUpdating, UITableViewDataSource, UITableViewDelegate>

@property (strong, nonatomic) RBQFetchedResultsController *fetchResultsController;
...
@end

I didn't make this a public property since I wasn't considering this use-case. You will need this property public so you can get access to the fetchedObjects property to grab the object and delete it from the Realm. However, to then animate the row away, you will have to call performFetch so that the fetchedResultsController will correctly show the object deleted, then you can delete the row from the table view.

thanks for the info - any plans to implement delete as a more native function of the view controller (vs. mucking around in underlying data structs) ?

Hello,
This one needs a public fetchResultsController for subclass use. As Realm doesn't support  cascading deletes, it is not feasible to do the delete in a general way. Each subclass probably needs to do it on his own. I can make a pull request but the change is trivial.
Thank you.


Synthesis failed for nodejs-datastore-kvstore

Hello! Autosynth couldn't regenerate nodejs-datastore-kvstore. :broken_heart:
Here's the output from running synth.py:
nch autosynth-9
nothing to commit, working tree clean
2021-01-07 02:27:57,275 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
DEBUG:synthtool:Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.eslintignore
.eslintrc.json
.gitattributes
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/workflows/ci.yaml
.kokoro/.gitattributes
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/lint.cfg
.kokoro/continuous/node12/samples-test.cfg
.kokoro/continuous/node12/system-test.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/populate-secrets.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/samples-test.cfg
.kokoro/presubmit/node12/system-test.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/publish.sh
.kokoro/release/docs-devsite.cfg
.kokoro/release/docs-devsite.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.kokoro/trampoline_v2.sh
.mocharc.js
.nycrc
.prettierignore
.prettierrc.js
.trampolinerc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
api-extractor.json
renovate.json
Skipping: samples/README.md
2021-01-07 02:27:57,386 synthtool [DEBUG] > Installing dependencies...
DEBUG:synthtool:Installing dependencies...
npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
npm WARN deprecated<EMAIL_ADDRESS>NOTICE: ts-simple-ast has been renamed to ts-morph and version reset to 1.0.0. Switch at your leisure...
npm WARN deprecated<EMAIL_ADDRESS>Use cheerio-select instead
npm WARN deprecated<EMAIL_ADDRESS>"Please update to latest v2.3 or v2.2"
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
npm WARN deprecated<EMAIL_ADDRESS>The package has been renamed to `open`
npm WARN deprecated<EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated

><EMAIL_ADDRESS>postinstall /home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/protobufjs
> node scripts/postinstall

><EMAIL_ADDRESS>postinstall /home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/@compodoc/compodoc
> opencollective-postinstall || exit 0

Thank you for using @compodoc/compodoc!
If you rely on this package, please consider supporting our open collective:
> https://opencollective.com/compodoc/donate

><EMAIL_ADDRESS>prepare /home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore
> npm run compile

><EMAIL_ADDRESS>precompile /home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore
> gts clean

/home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/meow/index.js:61
		throw new Error(`Flag keys may not contain '-': ${invalidFlags.join(', ')}`);
		^

Error: Flag keys may not contain '-': dry-run
    at validateOptions (/home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/meow/index.js:61:9)
    at meow (/home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/meow/index.js:136:2)
    at Object.<anonymous> (/home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/node_modules/gts/build/src/cli.js:30:13)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
    at internal/main/run_main_module.js:17:47
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>precompile: `gts clean`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>precompile script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>prepare script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2021-01-07T10_28_21_721Z-debug.log
2021-01-07 02:28:21,750 synthtool [ERROR] > Failed executing npm install:

None
ERROR:synthtool:Failed executing npm install:

None
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/nodejs-datastore-kvstore/synth.py", line 12, in <module>
    node.install()
  File "/tmpfs/src/github/synthtool/synthtool/languages/node.py", line 167, in install
    shell.run(["npm", "install"], hide_output=hide_output)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['npm', 'install']' returned non-zero exit status 1.
2021-01-07 02:28:21,792 autosynth [ERROR] > Synthesis failed
2021-01-07 02:28:21,793 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at f00d98b docs: add instructions for authenticating for system tests (#256)
2021-01-07 02:28:21,800 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2021-01-07 02:28:21,805 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Removing node_modules/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 354, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 189, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 334, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 65, in synthesize_loop
    has_changes = toolbox.synthesize_version_in_new_branch(synthesizer, youngest)
  File "/tmpfs/src/github/synthtool/autosynth/synth_toolbox.py", line 259, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:

Autosynth passed, closing! :green_heart:


Add notes about fixed issues for the second patch

@bot-gradle test and merge

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

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


UWP icon behaviors are different with the document

[Enter feedback here]
I would like to feedback two points about UWP application icon.

After Install a UWP application, the icon in the app list of the Start menu will be shown as unplated icon on 20H2.
But It will show a target-size(with plate) icon on Windows 10 2004.
The current document does not mention the latest behavior of 20H2.

The current doc:
Target-size app icon assets
In addition to the standard scale factor sizes ("Square44x44Logo.scale-400.png"), we also recommend creating "target-size" assets. We call these assets target-size because they target specific sizes, such as 16 pixels, rather than specific scale factors, such as 400. Target-size assets are for surfaces that don't use the scaling plateau system:
•	Start jump list (desktop)
•	Start lower corner of tile (desktop)
•	Shortcuts (desktop)
•	Control Panel (desktop)

the Shortcut icon of a installed UWP will use scaled icon on the windows 10(include 20H2). It is different with the document.

Current Doc:
Target-size assets are for surfaces that don't use the scaling plateau system:
Start jump list (desktop)
Start lower corner of tile (desktop)
Shortcuts (desktop)
Control Panel (desktop)

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

ID: 0b1d8079-ae5e-e9ef-e02d-7eb634140291
Version Independent ID: 52e77561-71a5-c4e6-810e-30d8a584e7a2
Content: App icons and logos - UWP applications
Content Source: windows-apps-src/design/style/app-icons-and-logos.md
Product: uwp
Technology: design
GitHub Login: @hickeys
Microsoft Alias: hickeys

Is this still an issue?


Write README files for runtime crates

The crates.io pages for the runtime crates are currently bare. We should add readmes to describe what they're for.

fixed in #899


Allow c++ ComponentBatch to be constructed for Vec of types that can be used to construct the inner component

What
As an alternative (or in addition to) https://github.com/rerun-io/rerun/pull/3967 this introduces a pattern wherein any ComponentBatch<C> can be constructed from any Vec<T> where we have implemented an extension for constructing C from T.
Additionally, this introduces a similar extension for constructing linestrips from vecs of Vec3-like things so that users don't event need to pick up a Vec3 dependency if they don't want to.
These two combine to allow:
    std::vector<std::vector<std::array<float, 3>>> points = {
        {{0.f, 0.f, 0.f}, {0.f, 0.f, 1.f}},
        {{1.f, 0.f, 0.f}, {1.f, 0.f, 1.f}},
        {{1.f, 1.f, 0.f}, {1.f, 1.f, 1.f}},
        {{0.f, 1.f, 0.f}, {0.f, 1.f, 1.f}},
    };

    rec.log("segments", rerun::LineStrips3D(points));

To do the correct thing.
I think this is important, relative to the variadic / initializer-list approach since it's more likely to match the kind of patterns users will encounter in the real-world, where they want to build a Vec of Vecs without needing to instantiate rerun types directly.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested [demo.rerun.io](https://demo.rerun.io/pr/{{ pr.number }}) (if applicable)
[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

[PR Build Summary](https://build.rerun.io/pr/{{ pr.number }})
[Docs preview](https://rerun.io/preview/{{ pr.commit }}/docs)
[Examples preview](https://rerun.io/preview/{{ pr.commit }}/examples)
Recent benchmark results
Wasm size tracking

This looks nice!

Looks like I was correct on it breaking error messages. We no longer hit the static assert I wrote for
    std::vector<std::string> strings = {"a", "b", "c"};
    rerun::ComponentBatch<Position2D> batch(strings);

Instead you get now
/usr/bin/clang++ -DLOGURU_STACKTRACES=1 -I/Users/andreas/dev/rerun-io/rerun/build/_deps/logurugitrepo-src -I/Users/andreas/dev/rerun-io/rerun/build/_deps/catch2-src/src/catch2/.. -I/Users/andreas/dev/rerun-io/rerun/build/_deps/catch2-build/generated-includes -I/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src -isystem /opt/homebrew/include -g -std=gnu++17 -arch arm64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.3.sdk -mmacosx-version-min=11.0 -Wall -Wcast-align -Wcast-qual -Wextra -Wformat=2 -Wmissing-include-dirs -Wnull-dereference -Wold-style-cast -Wpedantic -Wpointer-arith -Wshadow -Wswitch-enum -Wvla -Wc++17-compat-pedantic -Wc99-extensions -Wgnu -Wnon-gcc -Wshadow-all -g -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fsanitize=address -MD -MT rerun_cpp/tests/CMakeFiles/rerun_sdk_tests.dir/component_batch.cpp.o -MF rerun_cpp/tests/CMakeFiles/rerun_sdk_tests.dir/component_batch.cpp.o.d -o rerun_cpp/tests/CMakeFiles/rerun_sdk_tests.dir/component_batch.cpp.o -c /Users/andreas/dev/rerun-io/rerun/rerun_cpp/tests/component_batch.cpp
In file included from /Users/andreas/dev/rerun-io/rerun/rerun_cpp/tests/component_batch.cpp:3:
In file included from /Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/points2d.hpp:6:
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../component_batch.hpp:305:24: error: no matching conversion for functional-style cast from 'const std::string' to 'rerun::components::Position2D'
                return TComponent(datum);
                       ^~~~~~~~~~~~~~~~
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../component_batch.hpp:101:52: note: in instantiation of member function 'rerun::ComponentBatchAdapter<rerun::components::Position2D, std::vector<std::string>>::operator()' requested here
        ComponentBatch(T&& input) : ComponentBatch(TAdapter<T>()(std::forward<T>(input))) {}
                                                   ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/tests/component_batch.cpp:234:39: note: in instantiation of function template specialization 'rerun::ComponentBatch<rerun::components::Position2D>::ComponentBatch<std::vector<std::string> &>' requested here
    rerun::ComponentBatch<Position2D> batch(strings);
                                      ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../components/position2d.hpp:23:16: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'const std::string' to 'const rerun::components::Position2D' for 1st argument
        struct Position2D {
               ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../components/position2d.hpp:23:16: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'const std::string' to 'rerun::components::Position2D' for 1st argument
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../components/position2d.hpp:46:13: note: candidate constructor not viable: no known conversion from 'const std::string' to 'rerun::datatypes::Vec2D' for 1st argument
            Position2D(rerun::datatypes::Vec2D xy_) : xy(xy_) {}
            ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../components/position2d.hpp:53:13: note: candidate constructor not viable: no known conversion from 'const std::string' to 'std::array<float, 2>' for 1st argument
            Position2D(std::array<float, 2> xy_) : xy(xy_) {}
            ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../components/position2d.hpp:44:13: note: candidate constructor not viable: requires 0 arguments, but 1 was provided
            Position2D() = default;
            ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../components/position2d.hpp:33:13: note: candidate constructor not viable: requires 2 arguments, but 1 was provided
            Position2D(float x, float y) : xy{x, y} {}
            ^
1 error generated.

I'll try fixing that

oh my god. This was hard. But I figured it out. It even makes sense when you squint 😄
The nice thing is that this solution is pretty powerful in general and preserves the error message. It's just a liiittle bit of personal defeat that it's SFINAE based, but as long as the error is sane who am I to complain:
/usr/bin/clang++ -DLOGURU_STACKTRACES=1 -I/Users/andreas/dev/rerun-io/rerun/build/_deps/logurugitrepo-src -I/Users/andreas/dev/rerun-io/rerun/build/_deps/catch2-src/src/catch2/.. -I/Users/andreas/dev/rerun-io/rerun/build/_deps/catch2-build/generated-includes -I/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src -isystem /opt/homebrew/include -g -std=gnu++17 -arch arm64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.3.sdk -mmacosx-version-min=11.0 -Wall -Wcast-align -Wcast-qual -Wextra -Wformat=2 -Wmissing-include-dirs -Wnull-dereference -Wold-style-cast -Wpedantic -Wpointer-arith -Wshadow -Wswitch-enum -Wvla -Wc++17-compat-pedantic -Wc99-extensions -Wgnu -Wnon-gcc -Wshadow-all -g -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -fno-optimize-sibling-calls -fsanitize=address -MD -MT rerun_cpp/tests/CMakeFiles/rerun_sdk_tests.dir/component_batch.cpp.o -MF rerun_cpp/tests/CMakeFiles/rerun_sdk_tests.dir/component_batch.cpp.o.d -o rerun_cpp/tests/CMakeFiles/rerun_sdk_tests.dir/component_batch.cpp.o -c /Users/andreas/dev/rerun-io/rerun/rerun_cpp/tests/component_batch.cpp
In file included from /Users/andreas/dev/rerun-io/rerun/rerun_cpp/tests/component_batch.cpp:3:
In file included from /Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/points2d.hpp:6:
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../component_batch.hpp:38:9: error: static_assert failed due to requirement 'NoAdapterFor<rerun::components::Position2D, std::vector<std::string, std::allocator<std::string>>>::value' "ComponentBatchAdapter is not implemented for this type. It is implemented for for single components as well as std::vector, std::array, and c-arrays of components. You can add your own implementation by specializing ComponentBatchAdapter<TComponent, T> for a given component and your input type T."
        static_assert(
        ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/src/rerun/archetypes/../component_batch.hpp:101:52: note: in instantiation of template class 'rerun::ComponentBatchAdapter<rerun::components::Position2D, std::vector<std::string>>' requested here
        ComponentBatch(T&& input) : ComponentBatch(TAdapter<T>()(std::forward<T>(input))) {}
                                                   ^
/Users/andreas/dev/rerun-io/rerun/rerun_cpp/tests/component_batch.cpp:234:39: note: in instantiation of function template specialization 'rerun::ComponentBatch<rerun::components::Position2D>::ComponentBatch<std::vector<std::string> &>' requested here
    rerun::ComponentBatch<Position2D> batch(strings);

Another interesting thing about this solution is that it forced me to exclude the adapter on T==TComponent because otherwise it made resolution ambiguous. That's nice because that was one of my criticisms: Previously I wasn't sure if we'd take this adapter in those cases, now it's definitely impossible :)

cargo deny error is known. 🚢  time!


Some may be redundant

As I understand from your blog post, the purpose of the Some class is to simplify
Optional.ofNullable(maybeBean)
  .flatMap(bean -> optional.ofNullable(bean.getSprout()))
  .flatMap(sprout -> optional.ofNullable(sprout.getFoo()))
  .orElse("foo not found");

However, Optional.map already handles nulls in the way you want:
Optional.ofNullable(maybeBean)
  .map(Bean::getSprout)
  .map(Sprout::getFoo)
  .orElse("foo not found")

This is because "map" automatically converts null values to Optional.empty(). (I had assumed, as I think you have, that the above code would throw NullPointerExceptions if getSprout() returned null. But this is not in fact the case).

Thanks for your feedback. You're absolutely right. I've already updated the blog post and will change the example code in this repo as well later.
I'm glad this works out of the box without the need to write a legacy helper class.


Rename pinned folders in sidebar

I have just installed files on windows 10, and I love it, but there is something that I would love to see and its the option to rename the pinned folders in the sidebar only. I have a set of folders inside other folders, and I want to pin the last folder, but with the name of its parent, and right now, I think that the only way to do this, is by renaming the folder and the pin it.
Can you please add this feature, it is available in clover, but I like your explorer a lot more, since its more modern.
Thanks for your help.

Merging with #3422


Use namespaces, better module loading and Soil unloading

This Pull request is rather big but with small changes:

Change navwalker to nav-walker so that it's in line with Soil
Start using Namespaces
Load modules dynamically, based on filename
If theme supports Soil feature and same feature in Sagextras - unset support for soil feature.
Add more content in comment to be in line with Soil
More whitespace for code readability

I've tested it on my beta site and nothing expoded in my face, so I guess it's good enough for now :smile:

Yay! Now to refactor gallery. That's for tomorrow though :wink:


name 'HTML' is not defined

When I'm trying to run the following from fast.ai v3 lesson 3 IMDB notebook:
data_lm = TextDataBunch.from_csv(path, 'texts.csv')

I get the following error:
/opt/conda/lib/python3.7/site-packages/fastprogress/fastprogress.py in on_iter_begin(self)
    160
    161     def on_iter_begin(self):
--> 162         if self.display: self.out = display(HTML(self.progress), display_id=True)
    163         self.is_active=True
    164

NameError: name 'HTML' is not defined

Here are the library versions:

fastai-1.0.52
fastprogress-0.1.21

That is really weird. In which environment are you? Normally you should have seen a warning if HTML wasn't properly imported.

I'm obtained this in the Jupyter Lab environment. However, even if I change from Jupyter Lab to Jupyter notebook by changing /lab to /tree I still get this error.

In jupyter lab, you need to type this after installing ipywidgets
jupyter labextension install @jupyter-widgets/jupyterlab-manager

More generally, it seems there is something broken with your installation of ipywidgets, try reinstalling it?

This has not been addressed in a while, so closing.


Merging: Awesome-Deep-Neural-Network-Compression

The part of FBS has been merged!
https://github.com/densechen/eve-mli/commit/c89a11eb20ea698068554ea51306794cba1d87b1
@csyhhu

The part of INQ has been merged!
https://github.com/densechen/eve-mli/commit/623657ad59a8209209295c45a747ae64172abde8

The part of INQ has been merged!
https://github.com/densechen/eve-mli/commit/623657ad59a8209209295c45a747ae64172abde8

The part of QIL has been merged!
https://github.com/densechen/eve-mli/commit/7661e54444e305e2344f4f8d61fd93bbb5e3ea8b
But bug exists:
If we quan on param, the network will crash with NaN. But if we only quan on act, everything  works well.

The part of QIL has been merged!
https://github.com/densechen/eve-mli/commit/7661e54444e305e2344f4f8d61fd93bbb5e3ea8b
But bug exists:
If we quan on param, the network will crash with NaN. But if we only quan on act, everything  works well.

The part of TTQ has been merged!
https://github.com/densechen/eve-mli/commit/84640b6e5b9337203d5df25b2d8e7c2862b41c8f
Unstable!

The part of TTQ has been merged!
https://github.com/densechen/eve-mli/commit/84640b6e5b9337203d5df25b2d8e7c2862b41c8f
Unstable!

finished

finished


Create Posts “mars-et-avril-dun-coup”

Automatically generated by Netlify CMS

👷 Deploy Preview for shiny-babka-998c7e processing.

Name
Link

🔨 Latest commit
90664d97884166328b2a431c38f2e027743b95c2

🔍 Latest deploy log
https://app.netlify.com/sites/shiny-babka-998c7e/deploys/645ba3392a088d0008bac3fc


Migrate button focus to :focus-visible

Buttons often show false-positive focus-visible ring. It's time we gradually move from our own implementation to native :focus-visible since it is now universally supported. Let's start with buttons.


ToCode Solution: תרגול תחביר

Thank Inon!
I was in Miluin last week, but I will do my best to finish the 5 rest
questions till Sunday evening
Maor.
בתאריך 8 ביולי 2017 09:37 PM, "ynonp"<EMAIL_ADDRESS>כתב:

@ynonp commented on this pull request.
Good work. What about the rest?
In 05_syntax_lab/01.py
https://github.com/tocodeil/python-course-examples/pull/142#discussion_r126287663
:

-is 74, program should print:

Max number = 74
-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:

   maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
In python, the smallest possible number is called minus infinity and you
can reach it with: max_value = float('-inf')
In 05_syntax_lab/01.py
https://github.com/tocodeil/python-course-examples/pull/142#discussion_r126287682
:

-is 74, program should print:

Max number = 74
-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:

   maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
Also, max is a built-in function and it's generally better to use
variable names that don't collide with built-in functions to make your code
less confusing. The name max_value is therefore preferred
In 05_syntax_lab/01.py
https://github.com/tocodeil/python-course-examples/pull/142#discussion_r126287715
:

Max number = 74
-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:

   maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
+print "insert 10 nubers:"
+for i in range(10):
Since you don't use the name i except for the loop, the convention is to
use the name _ - so you can write: for _ in range(10):
In 05_syntax_lab/01.py
https://github.com/tocodeil/python-course-examples/pull/142#discussion_r126287734
:

-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:

   maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
+print "insert 10 nubers:"
+for i in range(10):

x = raw_input()

You can perform the conversion here and then you won't have to remember to
convert to int twice, i.e. x = int(raw_input())
Also you can add a nice message to the user:
x = int(raw_input('Please select another number: '))

In 05_syntax_lab/02.py
https://github.com/tocodeil/python-course-examples/pull/142#discussion_r126287738
:

-Write a python program that asks the user
-to type in her age, and prints the age in months
-Example input:

12
-Example output:
144
-"""

+""" read age from the user and print the age in mounths"""
+print "insert your age (first the years and after the mounth"
+x = raw_input()
+y = raw_input()
+age = int(x)
+mounths = int(y)
+allmounth = (age*12) + mounths
+print "you age in mounths is:", allmounth
👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tocodeil/python-course-examples/pull/142#pullrequestreview-48761774,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aclyq0c1SEqTu9vC4xg9j6kYKNCyumJuks5sL8xogaJpZM4OQ543
.

Awesome. Do notice that you can read all comments in a really nice format
online here:
https://github.com/tocodeil/python-course-examples/pull/142/files
On Sat, Jul 8, 2017 at 9:48 PM, maornesimi<EMAIL_ADDRESS>wrote:

Thank Inon!
I was in Miluin last week, but I will do my best to finish the 5 rest
questions till Sunday evening
Maor.
בתאריך 8 ביולי 2017 09:37 PM, "ynonp"<EMAIL_ADDRESS>כתב:

@ynonp commented on this pull request.
Good work. What about the rest?
In 05_syntax_lab/01.py
<https://github.com/tocodeil/python-course-examples/pull/
142#discussion_r126287663>
:

-is 74, program should print:

Max number = 74
-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:
maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
In python, the smallest possible number is called minus infinity and you
can reach it with: max_value = float('-inf')
In 05_syntax_lab/01.py
<https://github.com/tocodeil/python-course-examples/pull/
142#discussion_r126287682>
:

-is 74, program should print:

Max number = 74
-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:
maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
Also, max is a built-in function and it's generally better to use
variable names that don't collide with built-in functions to make your
code
less confusing. The name max_value is therefore preferred
In 05_syntax_lab/01.py
<https://github.com/tocodeil/python-course-examples/pull/
142#discussion_r126287715>
:

Max number = 74
-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:
maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
+print "insert 10 nubers:"
+for i in range(10):
Since you don't use the name i except for the loop, the convention is to
use the name _ - so you can write: for _ in range(10):
In 05_syntax_lab/01.py
<https://github.com/tocodeil/python-course-examples/pull/
142#discussion_r126287734>
:

-"""

-maxnum = float('-inf')
-for _ in range(10):

num = float(raw_input())
if num > maxnum:
maxnum = num

-print "Max number = %f" % maxnum
+""" read 10 numbers from user and print the bigger one"""
+max = -100000
+print "insert 10 nubers:"
+for i in range(10):

x = raw_input()

You can perform the conversion here and then you won't have to remember
to
convert to int twice, i.e. x = int(raw_input())
Also you can add a nice message to the user:
x = int(raw_input('Please select another number: '))

In 05_syntax_lab/02.py
<https://github.com/tocodeil/python-course-examples/pull/
142#discussion_r126287738>
:

-Write a python program that asks the user
-to type in her age, and prints the age in months
-Example input:

12
-Example output:
144
-"""

+""" read age from the user and print the age in mounths"""
+print "insert your age (first the years and after the mounth"
+x = raw_input()
+y = raw_input()
+age = int(x)
+mounths = int(y)
+allmounth = (age*12) + mounths
+print "you age in mounths is:", allmounth
👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<https://github.com/tocodeil/python-course-examples/pull/
142#pullrequestreview-48761774>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
Aclyq0c1SEqTu9vC4xg9j6kYKNCyumJuks5sL8xogaJpZM4OQ543>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tocodeil/python-course-examples/pull/142#issuecomment-313874283,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAH2UtxQo0Cbvp-lv2zhJWkHr7SDU25gks5sL88UgaJpZM4OQ543
.


Menu Bar drop down fails on Debian 8 Linux

Do you have a question? Please ask it on Stack Overflow with vscode tag
(Use Help > Report Issues to prefill these)

VSCode Version:  1:13:1
OS Version: Linux Debain 8

Steps to Reproduce:

Click Help
Move mouse to any menu item
Menu graphic changes to a black box with a circle.  All menu items are not visible.

Alternate work around is use the ctrl key combinations, but if none are available then the first letter of the sub menu item works for the first item only.  For example, Help, R gives you release notes.  Unable to reach Report Issues.
Interesting how the View menu item will allow you to move the mouse down to sub-menu options.

@owencasaide can you share a screenshot?

Sorry, I  have moved on to another Linux distribution and do not have VS Code installed. I can tell you that the behavior I reported was consistent on all menu items except View. With View you would get the drop down that allowed you to select a sub menu item with the mouse. All the other menu items would present a black graphic with a gray circle positioned near the upper left corner of the black graphic. The circle was not perfectly round and it would change according to the size of the black surrounding box. When you clicked on the menu item it did open the list of sub menu items, but when you moved the mouse into the list to select a sub menu item, the list was replaced with the graphic as I described. With the graphic open you could use the ctrl key options or the first letter of an item in the list with out a ctrl key option, but not the second submenu item with the same first letter.I am just playing around with Linux and would not be offended if you closed the issue. I am starting to use VS Code on Windows which is working okay.  I am so new to Linux that I couldn't tell you how I got VS Code installed. I think I down loaded it and used the extractor function to open the package. Whatever came out of the box worked without much intervention on my part. That's all I remember. 11:48 AM, June 20, 2017, Benjamin Pasero notifications@github.com:@owencasaide can you share a screenshot?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
-- Sent from Yandex.Mail for mobile

Needs more info, closing.


support single atoms for pkg deps, fetch highest version available

Fixes support for a single atom as a dep, like {deps, [cowboy]}.
Additionally simplifies the job of rebar3 pkgs and fixes an issue with not knowing package deps didn't need to be recompiled by using the AppInfo created when doing rebar_app_discover:find/2 instead of the new one created by install deps to be filled in -- however we override the state for AppInfo with the one newly created since that one has the overrides applied for this run.

Would it be worth renaming tup_sort and tup_merge to prop_sort and prop_merge ? Otherwise, bunch of comments.

I sort of prefer tup_sort over prop_sort especially since in the deps case it isn't really a proplist.

+1


Split most arrow specific CI checks into their own workflows (reduce common CI time to 21 minutes)

Which issue does this PR close?
Re https://github.com/apache/arrow-rs/issues/2149
Rationale for this change
Breaking up the CI for each crate into more finegrained workflows is a prerequisite for triggering in a more finegrained way (e.g. not run arrow tests if only parquet is changed)
More rationale os https://github.com/apache/arrow-rs/issues/2149
What changes are included in this PR?
Break arrow specific CI checks into a new arrow.yml workflow. It is still triggered on all PRs
Are there any user-facing changes?
There will be more Ci jobs, but hopefully they will each run in less time

Codecov Report

Merging #2168 (eefdaf3) into master (ec3530d) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2168   +/-   ##
=======================================
  Coverage   82.86%   82.86%
=======================================
  Files         237      237
  Lines       61429    61429
=======================================
+ Hits        50902    50903    +1
+ Misses      10527    10526    -1

Impacted Files
Coverage Δ

arrow/src/datatypes/datatype.rs
63.00% <0.00%> (+0.31%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.

We are slowly getting there -- thanks @viirya


issues on opencv 3.3.0 version

hi Ramiz,
First of all, great tutorial, thank you.
When i (pip) installed the requirement for this tutorial, I've got OpenCV 3.3.0 and it had some issues with your code.
I've done few adjustments to make it work:

Changed the creation of the recognizer from
face_recognizer = cv2.face.createLBPHFaceRecognizer()
to
face_recognizer = cv2.face.LBPHFaceRecognizer_create()

In the predict function changed this
label = face_recognizer.predict(face)
to
label, confidence = face_recognizer.predict(face)
The "predict" returns you a value of confidence also, and that addition fixed the TypeError.

Hello,
But when changed to
face_recognizer = cv2.face.LBPHFaceRecognizer_create()
it gives me an error while execution as:
400: error: (-5) This LBPH model is not computed yet. Did you call the train method? in function cv::face::LBPH::predict
I suppose it means I have to call LBPHFaceRecognizer class..? Even though I have, just in code...
Can this problem be solved?


M365 Developer Tenant Export and Report Sample - Feedback

Capturing feedback on the new sample to make it better: https://pnp.github.io/script-samples/spo-dev-tenant-report-export/README.html

Add inventory of remote event receivers and web hook lists to see what lists have associated to lists and libraries.

Closing - not needed.


Don't implicitly cast values to strings

When saving something that can't automatically be deserialized properly the client should raise an exception instead of coercing the value into a string.
Current behavior:
>>> sample_dict = {'nested': {'dict': 'here'}}
>>> r.hmset('dict', sample_dict)
True
>>> r.hgetall('dict') == sample_dict
False
>>> r.hgetall('dict')
{'nested': "{'dict': 'here'}"}

Fixed in redis-py 3.0


Recipes in Cooking Table not updating

Hey
So i changed the recipe for AA Dough via crafttweaker, but the Cooking Table is showing the old recipe and doesnt work, hasnt been able to find a way to change it in the Cooking Table, please assist
https://github.com/FTBTeam/FTB-Presents-Stoneblock-2/issues/114

I tried to reproduce with the following script:
recipes.removeShaped(<minecraft:bread>, [[null, null, null],[<minecraft:wheat>, <minecraft:wheat>, <minecraft:wheat>], [null, null, null]]);
recipes.addShaped(<minecraft:bread>, [[<minecraft:wheat>, null, null],[null, <minecraft:dirt>, null], [null, null, null]]);

and it's working fine for me.

recipes.remove(<actuallyadditions:item_misc:4>);
recipes.addShaped(<actuallyadditions:item_misc:4>*2, [
  [<minecraft:wheat>, null],
  [null, <minecraft:wheat>]
]);

Is the code i use in Stoneblock there causes the issue


refactor: clean up button-group styles

It's a pretty big support (eg margin-top has 94.5% support, not 100%), all major browsers support it, we already actively use this property in other places, so shouldn't be any problem here.

OK let's do this 👍
margin-top not having 100% is because there's unknown support, though, not because it's actually unsupported


Multiple windows open when only one should open

When I open a folder, i expect a single window to open for that folder.
In fact multiple windows open.
Why?

When I type "code ." in a directory.
VSCODE 1.61.0.
Ubuntu 20.04.
I have "files.hotExit": "onExitAndWindowClose" set, which was recommended on one of the multitudes of other tickets that raised this issue, but makes no difference.

Wow just got three windows instead of two. This really is garbage software.


Add tests on PR

This will be a pretty messy PR and may be closed and re-opened multiple times to see how things run.

I'll self-merge this too so that the automated tests get into the WIP branch. I'll also merge the pr.yaml back to main via a direct push, since it is... faster...


Vitest did not work on GitHub Action

Describe the bug
I activated GitHub actions for testing my code with vitest. I did some changes but still vitest does not work properly.
Reproduction
You can have a look this exception: https://github.com/sametcelikbicak/tsci/actions/runs/3354938512/jobs/5558830441
or my source code: https://github.com/sametcelikbicak/tsci
with branch: https://github.com/sametcelikbicak/tsci/tree/add-build-test-action

System Info
System:
    OS: macOS 13.0
    CPU: (10) arm64 Apple M1 Pro
    Memory: 57.45 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.15.1 - /usr/local/bin/node
    npm: 8.11.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 106.0.5249.119
    Safari: 16.1
  npmPackages:
    vitest: ^0.24.3 => 0.24.3

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Vitest required Node >=v14, your pipeline installs Node 12.22.12


Show validation error message as string instead of list

In cases where the error is shown in the field itself, do not repeat the message in Toast. It also no longer shows the error code, as this is irrelevant to a manager.
Fixes #1868

@sneridagh can we backport this to 17.x? Should I create a PR?


version 1.2.15 does not include 'dist' folder

This leads to error:
Can't resolve 'simple-react-notifications' in '/home/use/project/src/components/ContactForm'

File: src/components/ContactForm/ContactForm.js

Last working version which includes dist folder: 1.2.13

Now the library is not usable because of the dist.

Hello everyone,
As Alex seem busy and not responding anything
i have created a fork of this repo and maintaining it with name "simple-react-notifications2"
https://github.com/rizwan-ishtiaq/simple-react-notifications2
you can use this to get latest version from npm


Issue with rook provisioning

Tried to setup a cluster last night and left all options enabled.
The process stalled during the worker node provisioning and did not progress. After disabling rook and starting over the process worked fine.

I decided to try to apply the rook config manually:
kubectl apply -f manifests/rook/rook-operator.yaml
kubectl apply -f manifests/rook/rook-cluster.yaml

before I can run the third command the second errors with:
namespace "rook" configured
error: unable to recognize "rook-cluster.yaml": no matches for kind "Cluster" in version "rook.io/v1alpha1"

everything I read suggests that's because the operator is not ready but again after several hours this is still not able to be applied and so far as I can tell the operator is running apart from the following log entry being repeated (which to me seems implied by the above):
E0505 01:27:19.102629       5 reflector.go:205] github.com/rook/rook/vendor/github.com/rook/operator-kit/watcher.go:76: Failed to list *v1alpha1.Cluster: the server could not find the requested resource (get clusters.rook.io

I am getting the same issue :(
Any luck figuring it out?
error: unable to recognize "manifests/rook/rook-cluster.yaml": no matches for rook.io/, Kind=Cluster

@nbiles not as of yet I was hoping someone else would have some sort of idea. It's about the only thing not working on my cluster right :(

I'm still looking into it. This is a rather urgent issue for me. I have a demo on Monday and I have more work to do and need rook working.

good luck, let me know if you find something, my linode cluster is just for fiddling luckily lol

well, I was able to install rook operator using helm then running the manifests:
kubctl apply -f manifests/rook/rook-cluster.yaml
kubctl apply -f manifests/rook/rook-storageclass.yaml
kubctl apply -f manifests/rook/rook-tools.yaml

BUT I'm not sure if this is working. Everything looks good except when I try to mount a pvc:
Unable to mount volumes for pod ... : timeout expired waiting for volumes to attach/mount for pod " ... ". list of unattached/unmounted volumes=[mysql-storage]

Not sure what is going on because this helm chart (deployment) was working on a cluster I had up last week.
Have you seen this?


Filter icons in top right not working.

What version of Music Assistant has the issue?
2022.6.6
The problem
I understand the icons in the top right (in my case local files system and spotify) should filter the song listing below. When I hover over it the cursor changes to the finger to suggest it is clickable. The tooltip with the source name appears. However when I click on those icons nothing appears to happen?
How to reproduce
As above
Relevant log output
Not sure if this is relevant?
Logger: music_assistant.music
Source: /usr/local/lib/python3.9/site-packages/music_assistant/controllers/music/__init__.py:121
First occurred: 4:11:45 PM (1 occurrences)
Last logged: 4:11:45 PM

Provider ProviderType.DATABASE is not available

Additional information
No response
What version of Home Assistant Core are your running
2022.6.6
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Please explain with some screenshots what you mean because your report is not very clear yet to me. If you refer to the provider icons (e.g. Spotify) in the top right in the "info" view. Those are not meant for filtering but should bring you to the webinterface (or app) of that provider. Looks like that got broken somehow so that is a bug.
You are talking about filtering however and I have no idea what you want filtered here.

This is the Album view but same issue if I go though the Artist view. In the top right there are the two icons below the vertical ellipsis. What are they supposed to do if anything?
My core issue is I have this album (amongst many) which I have on my NAS in FLAC. I see that MA has also pulled down the songs from Spotify, The unfortunate problem in this situation is that Spotify has called the Album the same name but has appended (Remaster) to all the song titles so everything is appearing duplicated. I would like to hide the spotify entries.

Yes they are the ones. Not 100% sure what they are for?

Those icons should take you to the web interface (or app) of the provider for the item (album in this case) but this is not working (which is a bug).
Can you please check this album on Spotify if this is whole album is considered a remaster ?
In that case there's a bug. If not this is not a bug.
A remaster is not the same version as a non-remaster so the matcher logic doing exactly what it should do.
It simply may not match an original to a remaster unless explicitly told so.
What I find strange is that original (non remaster) tracks are in the same albumlisting now hence my question if you can have a quick look at Spotify if the album is a remaster or not.
Looking at the duration of the tracks these are in fact different versions.

Its probably a Spotify database issue. On Spotify the Alubum name is Abacab. The cover art shown is the original album but the tracks all have either "2007 Digital Remaster" or "Remaster 2007" appended. This remaster was done as part of a boxset https://en.wikipedia.org/wiki/Genesis_1976–1982 so it should have different IDs https://musicbrainz.org/release-group/4488cc99-a9a1-3f10-adcc-043a18c53078

Also I am only new to MA but I see there is a VERSIONS tab which shows 2. Which I go into that view clicking on the two doesn't seem to do anything? Maybe one option to help with this would be to SORT BY VERSION (and then TRACK NUMBER)?

Exactly! So MA is doing exactly what it should do and it doesn't match the remaster with the non-remaster.
Strange that Spotify has catalogued the album wrong then.
I'm thinking out loud how to prevent such cases. The only thing I can think of:

Ability to manually assign the tracks to a different album (or edit the spotify matched album)
Detect if all tracks of an album have a different version and in that case assume the album is a different version.

While 2 may be nice from user perspective the actual error is hidden so maybe a bit dangerous.

Also I am only new to MA but I see there is a VERSIONS tab which shows 2. Which I go into that view clicking on the two different albums doesn't seem to do anything? Maybe one option to help with this would be to SORT BY VERSION (and then TRACK NUMBER)?

Versions tab should show any other different versions of this album (such as Remaster, Live etc.) on all providers.
Sometimes this doesn't make a lot of sensor if it just shows the versions that are already matched and it could in fact be hidden

Maybe one option to help with this would be to SORT BY VERSION (and then TRACK NUMBER)?

That does sound viable as a workaround but it still itches that the tracks of another album version now are in the album listing of another album version... Guess we can't fix Spotify's metadata so maybe we should indeed work around it ?

Hi, music assistant blends songs from various sources into one jukebox. It automatically slices the best quality available. To my understanding these icons represent the sources used. In your case songs from both Spotify and local media.

Hi, music assistant blends songs from various sources into one jukebox. It automatically slices the best quality available. To my understanding these icons represent the sources used. In your case songs from both Spotify and local media.

Erik, see the above conversation. The icons not working is just a very small bug. There is another issue going on that tracks of a remastered album version are listed together with tracks of the not-remastered (original) album.

Thanks @marcelveldt . Maybe you could have it so that if you click on the album in the VERSIONS tab then you see the list of tracks from that ALBUM?

Great idea! That is something easy to implement.

I just double checked my MB ID's (to make sure I had not made the error!) and I am using a release from 1988 (Recording ID dbc137d6-ac33-4503-a9d9-df63e7f841e3 and Release ID f7eb677a-cf39-47d7-b644-37c3cde348ce). Does Spotify have the MB ID's in their metadata? If so can you see what release they are using and is MA using RELEASE ID to match? I ask because if using RECORDING ID then I believe they will match because that number will be the same....

Spotify uses the UPC and that is what MA is using to match... So I think Spotify has the wrong UPC set on the album ?

Great idea! That is something easy to implement.

Hang on I think you have already done this!! I just went to another album "1" by The Beatles. I see there are TWO versions. Now I can see when I click on those that the source icon in the top right changes (that is I only see ONE of them) and when I go back to the TRACKS tab I only see the tracks for the selected album. In this case when I select the SPOTIFY version it is also a remastered release but MA knows it is a different version. So in this case I am only seeing the tracks related to the version I select.
However, in the Genesis example I see two versions but when I click on EITHER of them I still see BOTH source icons. So if MA was matching the same album and then combing all the tracks why is it showing TWO versions? Maybe a good reason I am just talking out loud to you :-)
While I am throwing feature requests at you; I notice I can't tell in the VERSION tab which one I have selected so I don't know beforehand which is the selected version. Could you add an icon like in this image

Spotify uses the UPC and that is what MA is using to match

Ahhh... I see in MB that there is no barcode/UPC for the release I am using...

I adapted the title to reflect the small bug that the provider icons in top right don't work.
Not sure from the discussion if the doubles (original& remastered) is a Spotify mistake or also a MA bug?
@marcelveldt  Please decide on creating an extra issue or spawn a feature request for the test of the discussion

Hang on I think you have already done this!! I just went to another album "1" by The Beatles. I see there are TWO versions. Now I can see when I click on those that the source icon in the top right changes (that is I only see ONE of them) and when I go back to the TRACKS tab I only see the tracks for the selected album. In this case when I select the SPOTIFY version it is also a remastered release but MA knows it is a different version. So in this case I am only seeing the tracks related to the version I select.

Correct, it already works this way but it still auto matches the same album. So if the album 1 on provider X is matched to album 2 on provider Y it will still show the combined (database) view. That's something I can adjust so that if you click the provider-specific album from the versions tab, it may only view that exact provider version/copy.

However, in the Genesis example I see two versions but when I click on EITHER of them I still see BOTH source icons. So if MA was matching the same album and then combing all the tracks why is it showing TWO versions? Maybe a good reason I am just talking out loud to you :-)

Yes, because of what I just told you. The word Version is a bit misleading in this case. It can mean a "provider copy" (the same album on provider X, Y , Z) and/or a "album version" (such as Live, Remaster etc.).
The version tab just queries all providers for the album name and lists them, regardless of provider, version.

While I am throwing feature requests at you; I notice I can't tell in the VERSION tab which one I have selected so I don't know beforehand which is the selected version. Could you add an icon like in this image

Set the view to "list" and that is what you (should) get...

BTW: I just discovered that this issue exists globally on Spotify. They just consider the album the same, regardless if the tracks are remastered or not.
So the solution will be to either sort the track version separate from the track number or detect all tracks are remastered and consider the album to be the remaster.

Thanks the list view is better for me. Yes I am seeing a lot of albums where Spotify has the tracks listed as "remastered".
Also, just to make things (maybe) more confusing for you. I have found an example where the album name is the same and all the tracks except one have remaster in the name. The one one without remaster happens to be the only track I have from that album but it is not matching. So same album, title, disc number, track number and duration but no match. So I don't know what is going on there?

Can you give me the spotify id from that album and track ? You can find these in the Spotify app/webplayer in the "share link" logic. Or wait for me to fix the bug so that the icon links work again ;-)
And a screenshot would be nice of the offending album listing.

Yes. Spotify link https://open.spotify.com/album/31nRnHI1uSDZ0POYPPJcUH?si=7fPKwHviRYik6qpuu9wqXQ It is track number 6 as per this screen shot

That is a strange one indeed. It should have matched this one. Now I'm curious if the isrc of your track on disk is the same (or not) as the Spotify one. This may as well be Spotify's auto track replacement at work. Maybe it just considers a remaster not as a different version and delivers the same isrc for both versions ? That also explains the high number of streams for that track (13 mln)

The ISRC in my file is GBAZP9900012

Ok, I looked at the raw data from Spotify and Spotify has the data correct so the issue must be somewhere within MA's matching logic. I think it's completely ignoring the album version somehow.
So... long story short: we have a bug to solve.
Thanks for the help diagnosing!
I'll let you know once I have something to test.

I fixed the issue with the provider icons in 2022.6.7. The other issue needs some more testing with 2022.6.7, don't think I resolved that one completely now.

Yes thanks @marcelveldt those other issues are resolved. However, this example of The Go-Betweens is still not matching. Please let me know if I can do anything more. I have another (what I think is a different matching) issue that i can raise as well if you would like.

fixed in 2022.6.10 - please reopen if not resolved, thanks!


Ape Staking Vault

Security Checklist

[ ] 1. Re-Entrancy
[ ] 2. Arithmetic Over/Under Flows
[ ] 3. Unexpected Ether
[ ] 4. Delegatecall
[ ] 5. Default Visibilities
[ ] 6. Entropy Illusion
[ ] 7. External Contract Referencing
[ ] 8. Short Address/Parameter Attack (off chain)
[ ] 9. Unchecked CALL Return Values
[ ] 10. Race Conditions / Front Running
[ ] 11. Denial Of Service (DOS)
[ ] 12. Block Timestamp Manipulation
[ ] 13. Constructors with Care
[ ] 14. Uninitialized Storage Pointers
[ ] 15. Floating Points and Precision
[ ] 16. Tx.Origin Authentication
[ ] 17. Address.isContract Re-Entrancy via Constructor

⚠ NOTES ⚠
Make sure to think about each of these exploits in this PR.

Changes merged to https://github.com/para-space/paraspace-core/pull/403


Update documentation to remove references to Java page objects

documentation needs to be updated to reflect that YAML files are used for page objects now.

Section 2.3

2.4.1 needs to have the example file indented.


Segmentation network name error

Hello. The following code line:
https://github.com/prgumd/EVDodgeNet/blob/0b6f6a0fc781203d3bc395c5305ff7c7ffc7535e/code/EVSegNet/RunEVSegNet.py#L291
produces a "NameError" since "global name 'tu' is not defined".

resolved in https://github.com/prgumd/EVDodgeNet/pull/7

resolved in https://github.com/prgumd/EVDodgeNet/pull/7


Bumblebee instantly closes

Using your example in the README.md, the program instantly closes and outputs this:
ENVIRONMENT_IS_NODE /home/pi/bumblebee-wake-word/node_modules/bumblebee-hotword-node/lib/porcupine-v1.8/pv_porcupine.js

Any ideas on how to fix it? Or what's going wrong?

I removed the 'ENVIRONMENT_IS_NODE' console debug output in v0.0.9.  But that might not explain why it's crashing.
Which environment are you on, and which of the examples did you try?

I tried this only
const Bumblebee = require('bumblebee-hotword-node');
const bumblebee = new Bumblebee();
bumblebee.addHotword('bumblebee');

bumblebee.on('hotword', function (hotword) {
	console.log('Hotword Detected:', hotword);
});

bumblebee.start();

Are you on MacOS, Linux, or Windows?  And are you using v0.0.9?
Also, if you run node using NODE_ENV=dev node start.js you'll get more output.  In v0.0.9 I changed how the Porcupine files get loaded and this may be responsible.

Edit: I was mistaken, looks like the "ENVIRONMENT_IS_NODE" is still there in v0.0.9 (i fixed in the web version).
Are you using Windows, MacOSX, or Linux?  It's possible the location of sox is wrong.  I don't think I've actually  ever tried this in  Windows.
I do have another larger voice project in the works, and it does work in Windows and runs DeepSpeech as well https://github.com/jaxcore/bumblebee


[feature request] Search in Directory with a nice intermediate UI

Description
This one thing would really help a lot, currently I know that to search files in a directory we can do lua require('fzf-lua').files({cwd="/mydir/"}) and for live grepping lua require('fzf-lua').live_grep({cwd="/mydir/"})
But with this I'd have to either hardcode mydir or type it again and again,
Would be awesome if it was possible to launch a fzf-lua command such that it first let's me choose a directory via directory search and then search files inside it/live grep inside it in the next step.

How about your own customization?

Uses fd but can be adjusted for find

_G.live_grep_cwd = function()
  require('fzf-lua').fzf_exec('fd -type d', {
    actions = {
      ['default'] = function(sel, opts)
        require('fzf-lua').live_grep({ cwd = sel[1] })
      end
    }
  })
end

-- you can then call it with
_G.live_grep_cwd()

Also, I wanted to understand your workflow with fzf-lua and quickfix list, I recently came across how well fzf-lua can play with the quickfix list, how do you generally use the quick fix list in your day to day work in vim?

I mostly use the quick fix list for refactors and global search and replace.

How does this refactor workflow look like exactly?

I would first find which places need the refactor either with live_grep/ctrl-g or with lsp_refs, etc and then traverse the locations with ]q and ]q which I mapped to :cnext/:cprev, bonus points if it’s a simple refactor I would create a vim macro and apply it to all lines with :cdo normal @q


Ordering the blog content in the content section based on user desired order

Is your feature request related to a problem? Please describe.
The ordering of the content section is based on alphabets, which might not be suitable for everyone.
Describe the solution you'd like
I would like to have content ordering that can be set by users like : order:1, 2,3 on each content to set the orders while listing of the content
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Thanks for raising the concern.
I just want to reiterate what I understood. We have top-level folders and files that can be modified (read more about it here). You want the option to provide custom orders to the subitems as well.
I am working on that feature. For the time being, there is a workaround. Name your files with numbers like 01-file1.md and 02-actual-file.md. This way we can provide custom orders to inner files and folders.

Hi @taragurung
I have added the feature for custom order of inner items. Kindly check out the documentation to learn more about its implementation.
https://celestialdocs.hyperoot.dev/guides/sidebar-navigation/#custom-order

Thanks! for the quick response.
For inner content its working fine with the work-around of renaming file as 01,02

but for the main title how to achieve it
Like in here Basic Command should be after Docker but due to alphabets its getting on top. If we use workaround here, I will get 01 02 in the url so dont want to do that

You can order them as well. I have documented the same here.
If you go to the config.ts file in the root directory, you can place an order for all the pages. For instance,
export const side_nav_menu_order: string[] = [
  "getting-started",
  "guides",
  "guides/pages",
  "guides/table-of-contents",
  "guides/sidebar-navigation",
  "custom-components",
  "reference",
];

This is the order I have for the official documentation site. getting-started is a mdx file whereas others are folders. If you notice, I have mentioned the order for nested pages as well (guides/pages, guides/table-of-contents and guides/sidebar-navigation for instance).

@taragurung
If you have hosted your code somewhere, then share your repo link. I can take a look and help you with your query.


Fix TiKV panic when aggregation expr type is not valid

Signed-off-by: Breezewish<EMAIL_ADDRESS>
What have you changed?
This PR protects TiKV from panic in scenarios like https://github.com/tikv/tikv/issues/5404
What is the type of the changes?

Bugfix (a change which fixes an issue)

How is the PR tested?
Manually, using a TiDB commit that will produce invalid expr type:
$ mysql -vvv --host <IP_ADDRESS> --port 4000 -u root tpch < queries/1.sql
--------------
select
	l_returnflag,
	l_linestatus,
	sum(l_quantity) as sum_qty,
	sum(l_extendedprice) as sum_base_price,
	sum(l_extendedprice * (1 - l_discount)) as sum_disc_price,
	sum(l_extendedprice * (1 - l_discount) * (1 + l_tax)) as sum_charge,
	avg(l_quantity) as avg_qty,
	avg(l_extendedprice) as avg_price,
	avg(l_discount) as avg_disc,
	count(*) as count_order
from
	lineitem
where
	l_shipdate <= date_sub('1998-12-01', interval 108 day)
group by
	l_returnflag,
	l_linestatus
order by
	l_returnflag,
	l_linestatus
--------------

ERROR 1105 (HY000) at line 4: other error: [components/tidb_query/src/aggr_fn/impl_sum.rs:49]: Unexpected return field type LongLong

/run-all-tests

@sticnarf @lonng @mapleFU  PTAL, thanks

/run-all-tests

By the way, as a bug fix, I think there should be a corresponding test.

/run-integration-common-test

/merge


For Arduino shield, write JSON string once and send to I/O only once

i.e.: for serial port, there are many lines for printing on serial.
that is error prone.
build the string and dump it only once.

Fixed in private repo. Will merge shortly.


Click event on HTML Template (Angular).

Can it be possible for click event working for below template & function with param get called ?

And also pass it to drawflow editor.addNode().

I found a way to do this:

execute npm i @angular/elements
Create an Angular component
Define the component as a custom element (https://www.angulararchitects.io/aktuelles/angular-elements-web-components-with-standalone-components/)
Create a service
Use the service to share data between the component hosting the drawflow canvas and the node


changes: remove the useless pcre config

Since the C API ngx_stream_lua_pcre_version was introduced in this module, and Lua FFI no longer directly calls pcre_version, issue #168 is no longer necessary.

https://github.com/openresty/stream-lua-nginx-module/issues/320#issuecomment-1883361098


⚠ Bofrost.be has degraded performance

In 06a62c2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1140 ms

Resolved: Bofrost.be performance has improved in eb91828.