Combination analysis boolean logic expression INVALID expression acceptance

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Currently we have 2 options to create a boolean logic expression

Entering individual i/p and o/ps
Entering expression

This bug is related to 2nd option where while entering an boolean expression

The present code accepts the expression without letting the user know it's an invalid
For a few conditions it is causing while loop console error

Screencast from 01-08-2024 06:20:31 PM.webm
Steps To Reproduce
simulator->tools->combination analysis->enter the expresssion
EX: a+b+++ or a+b(c() or a+b+r+((((((((((
Expected Behavior
So,

we don't have to show the truth table until the expression is valid.
also alert the user by saying which causes the expression invalid in this way they will check the entered expression once again
resolves the while loop error occuring in the console.

Screencast from 01-08-2024 06:21:38 PM.webm
Screenshots
No response
Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
Yes

I'm interested in working on this issue, can it be assigned to me? @Malavi1

@Arnabdaz @Prerna-0202 can anyone please review the issue!?

I am interested to work in this issue.

@Malavi1 i have resolved the issue for the invalid expression  but i am facing some problem in pull request.

@tanmoysrt can I work!?

@tanmoysrt can I work!?

As @umangPokhriyall has raised a PR for the fix. Would you like to pass this issue (if the PR solve that issue, you mentioned) to him ?

@tanmoysrt Yes.

@tanmoysrt Yes., You can pass to him

https://www.youtube.com/watch?v=5FjiGsO-ITw


Shader support?

hello! beautiful mod you did here 🐢
i was wondering if it's possible to add shader support? at least when using sildur's vibrant shaders the portal doesn't render at all.
minecraft 1.19.2, forge 43.2.0

ooh,  thats not good
i can't actually get optifine (i assume you're using optifine) to run with forge let alone my mod, so i unfortunately cant debug it atm :/
thanks for the report :)

oh, sorry; i'm not using optifine, but rubidium + oculus on forge. sorry for the confusion!

hmm i can't get forge running with those either...

it seems rubidium specifically overrides the end portal shader,, i think theres not much i can do without alot of effort,, im not familiar with it


fix: add trailing '/' to uri in ingress

fixes canonical/bundle-kubeflow#436
This fixes the istio ingress/virtualservice routing.
Previously, the rewrite rule of prefix=/mlflow would serve a blank page at INGRESS/mlflow and a dead link at INGRESS/mlflow/.  This appeared to be because the rewrite rules would break the static files that mlflow serves.  With this old rule (no trailing slash in the prefix) the static files get served at INGRESS/static-files/...rather thanINGRESS/mlflow/static-files/...` and are thus inaccessible.
With this PR, the rewrite rule of prefix=mlflow/ (including trailing slash) serves the correct mlflow UI at INGRESS/mlflow/ with the static files correctly being accessed from INGRESS/mlflow/static-files/....  But, this serves nothing at INGRESS/mlflow, which might be counter intuitive.  I tried to find some extra istio virtualiservice config that would get both to work, but I was not successful.
Test instructions:
On a deployment with at least the istio-pilot, gateway, and possibly kubeflow-dashboard, do:
juju deploy mlflow-server
juju deploy cs:~charmed-osm/mariadb-k8s-35 mysql-mlflow
juju deploy minio minio-mlflow

juju relate mlflow-server mysql-mlflow
juju relate mlflow-server:object-storage minio-mlflow:object-storage
juju relate mlflow-server:ingress istio-pilot:ingress

Then browse to INGRESSIP/mlflow/ (including trailing /).  The auth flow should occur and then get you to the mlflow UI

I know the trailing slash isn't ideal, but given how difficult it seems to be to get the Istio prefix rules to work how it seems like they should, this seems like a reasonable workaround. However, I'm confused as to why this issue wasn't caught by the integration test, considering it does use Istio and the ingress gateway address. Should the test be improved to catch this and avoid a regression?

Yes I agree with @johnsca looking at the integration test, this line might not be enough to actually ensure we have a working dashboard, one additional check would probably be enough for now.

Good point on adding more tests.  I get the feeling from this line that it might be a good idea, too


Provide a way to set the default path to lefthook

In repare-commit-msg hook I found next text:
#!/bin/sh

if [ "{$LEFTHOOK}" = "0" ]; then
	exit 0
fi

# lefthook_version: d41d8cd98f00b204e9800998ecf8427e

cmd="lefthook install"

if lefthook >/dev/null 2>&1
then
	exec $cmd
elif bundle exec lefthook >/dev/null 2>&1
then
	bundle exec $cmd
elif npx lefthook >/dev/null 2>&1
then
	npx $cmd
fi

cmd="lefthook run prepare-commit-msg $@"

if lefthook >/dev/null 2>&1
then
  exec $cmd
elif bundle exec lefthook >/dev/null 2>&1
then
  bundle exec $cmd
elif npx lefthook >/dev/null 2>&1
then
  npx $cmd
else
  echo "Can't find lefthook in PATH"
fi

I guess, it may be confusing, if folk uses this tool from Dart, but Lefthook try to use npx, ruby-stuff, etc. Can you provide API to set this commands implicitly by first install? I mean something like this:
lefthook install --install-command="pub global activate lefthook"

Originally posted by @igorkamyshev in https://github.com/evilmartians/lefthook/issues/71#issuecomment-524967878

Hey! Is this issue still relevant? I've slightly changed the logic and moved most of ENV checking things into lefthook executable. Also install implicitly runs if needed in lefthook run ....

There is an option to extend the PATH with rc for lefthook. It sould solve this issue.


Adds sourcemaps to testing; updates dependencies

Adds karma-sourcemap-loader
Adds karma-preprocessor-rollup
Moves testing Rollup building to Karma
Enables sourcemaps in tests
Fixes #2051

This is passing now, but I don't think it's actually posting code coverage properly yet.

c3.js entry in files section should be removed because it's the bundled version and we don't want that file loaded if we build c3 at the start of karma.
I guess we need rollup-plugin-istanbul for instrumentation. Usually you can't use karma-coverage directly if you use bundlers.

@kt3k That is literally exactly what I was about to try next. :joy:
Awesome, will do that and then update the PR. :+1:


DCGM only storing for two days. Please expand the save time.

fixed switched back to prometheus as datasource- not thanos for now- thanos' receiver disk filled a while ago and i didn thave alerts setup yet - datas' there tho
https://graphs.cais.tools/d/Oxed_c6Wz/nvidia-dcgm-real-time-gpu-metrics?orgId=1&from=1690244975084&to=1692906073659&var-datasource=prometheus-main-sanjose&var-instance=All&var-gpu=All


Update howto-mfa-getstarted.md

Added references to ADFS 2019 when necessary.

@marcelodiiorio : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


snipe-it: 6.4.1 -> 6.4.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for snipe-it is: A free open source IT asset/license management system
meta.homepage for snipe-it is: https://snipeitapp.com/
meta.changelog for snipe-it is: https://github.com/snipe/snipe-it/releases/tag/v6.4.2
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.4.2 with grep in /nix/store/xxgwkj6wvmbx8vaq3rlfmcsc152mr9b6-snipe-it-6.4.2
found 6.4.2 in filename of file in /nix/store/xxgwkj6wvmbx8vaq3rlfmcsc152mr9b6-snipe-it-6.4.2

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

snipe-it

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/94b7be0442664d71b8e9fb94664df65a3a131038#snipe-it

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

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

1 package built:

snipe-it

Maintainer pings
cc @yayayayaka for testing.

@ofborg build snipe-it snipe-it.passthru.tests


Set fields values programmatically during form initialization

Hi kjac,
here again with another question :P
I'm building a simple hotel website with a booking-form inside a dedicated page.
In the header of all other website pages there's a fast mini-booking-form with only 3 fields: dateFrom, dateTo, number of persons.
When the fast-form has been submitted a GET request is created against the bookingPage Url.
What I'd like to achieve is set each form field value to the corresponding query string value.
The little problem I have is that SubmittedValue property has protected setter.
Any idea on how could I achieve that?
Thank you,
Alessandro

Hey @alecrt,
Maybe you should consider extracting those values in your form template and passing them along to the field partials in a custom model? In this way the field partials could init their field values from either the SubmittedValue or the extracted values.

Yep, could be a valid solution. Thx!

Closing this now :)


[BUG] Does deepspeed work with torch amp autocast?

During training, fp16 is set to true.
But for some submodules, i want to using fp32.
with torch.cuda.amp.autocast(enabled=False):
x = self.model(x.float())

An error occur, show half vs float type not matching

@lqniunjunlper, are you able to share repro steps for this issue? Thanks


Django integration

Something like https://github.com/roverdotcom/django-inlinecss (template tag) and https://github.com/Dino16m/mailcomposer (CLI command) but simpler

django-inlinecss is not maintained for some time. And the appearance of a template tag for django in css-inline looks promising.
In fact django-inlinecss supports different engines and it takes 5 minutes to put css-inline in there.
https://github.com/roverdotcom/django-inlinecss/blob/aca7106337cdc85a77dfbffb333ae91aaae74e14/django_inlinecss/engines.py#L1-L29

Thank you for :+1: on this :)
It is unfortunate that django-inlinecss is not actively maintained; it would be nice to have a direct integration there indeed. However, it will also be nice to have everything integrated under the same roof. I.e., running tests against all current Django versions, etc., on each change in css-inline.
I was thinking about deriving a design similar to django-inlinecss in this repo and distributing something like css_inline.contrib.django. Maybe also some jinja tag as well.
I don't have any timeline for this, though, but I hope to get back to this somewhere in August / September.
Btw, feel free to open an issue if you miss anything in css-inline :)

Or, I'll be happy to review PRs if anyone is willing to contribute this feature :)

I've written my own component tag which may be a source of inspiration:

`src/app/templatetags/css.py`
# This is free and unencumbered software released into the public domain.

# Anyone is free to copy, modify, publish, use, compile, sell, or
# distribute this software, either in source code form or as a compiled
# binary, for any purpose, commercial or non-commercial, and by any
# means.

# In jurisdictions that recognize copyright laws, the author or authors
# of this software dedicate any and all copyright interest in the
# software to the public domain. We make this dedication for the benefit
# of the public at large and to the detriment of our heirs and
# successors. We intend this dedication to be an overt act of
# relinquishment in perpetuity of all present and future rights to this
# software under copyright law.

# IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR ANY CLAIM, DAMAGES OR
# OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
# ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
# OTHER DEALINGS IN THE SOFTWARE.

# For more information, please refer to <https://unlicense.org>

from typing import List

import css_inline
import lxml.html
import logging
from django.conf import settings
from django.template import Node
from django.template.base import Parser, Token
from django.template.library import Library

register = Library()

_INLINE_CSS_TAG_NAME = "inline_css"

@register.tag(name=_INLINE_CSS_TAG_NAME)
def inline_css(parser: Parser, token: Token):
    """inline_css
    Inlines CSS in a given document, and then strips the link tags from the resulting HTML

    Usage:
        {% inline_css %}
            <html>
                <head>
                    <link rel="stylesheet" href="{% static 'style.css' %}">
                    <link rel="stylesheet" href="https://some.url/to/style.css">
                </head>

                // ...
            </html>
        {% end_inline_css %}
    """
    nodelist = parser.parse(f"end_{_INLINE_CSS_TAG_NAME}")
    parser.delete_first_token()

    return _InlineCssNode(nodelist)

class _InlineCssNode(Node):
    def __init__(self, nodelist):
        self.nodelist = nodelist

    def render(self, context):
        html = self.nodelist.render(context)

        # attempt to inline CSS. If inlining / DOM manipulation fails, return the original html
        try:
            html_with_abs_style_hrefs = self._rewrite_relative_style_hrefs(html)
            html_inlined_styles = self._inline_styles(html_with_abs_style_hrefs)
            html = self._strip_link_tags(html_inlined_styles)
        except (OSError, css_inline.InlineError) as e:
             logging.getLogger(__name__).exception(e)

        return html

    def _rewrite_relative_style_hrefs(self, html: str, /) -> str:
        tree = lxml.html.fromstring(html)
        xpath_pattern = f"//link[starts-with(@href, '{settings.STATIC_URL}')]"
        xpath_result = tree.xpath(xpath_pattern)
        tags = xpath_result if isinstance(xpath_result, list) else []
        script_tags: List[lxml.html.HtmlElement] = [
            x for x in tags if isinstance(x, lxml.html.HtmlElement)
        ]

        # rewrite `/static/[asset]` paths to absolute URLs so that css_inline can resolve
        # the stylesheets
        for tag in script_tags:
            original_href = tag.get("href")
            absolute_href = original_href.replace(
                settings.STATIC_URL, f"{settings.STATIC_ROOT}/"
            )
            tag.set("href", absolute_href)

        return lxml.html.tostring(tree).decode("utf-8")

    def _inline_styles(self, html: str, /) -> str:
        html = css_inline.inline(html)

        return html

    def _strip_link_tags(self, html: str, /) -> str:
        tree = lxml.html.fromstring(html)
        xpath_result = tree.xpath("//link")
        tags = xpath_result if isinstance(xpath_result, list) else []
        script_tags: List[lxml.html.HtmlElement] = [
            x for x in tags if isinstance(x, lxml.html.HtmlElement)
        ]

        # strip all <link> tags - different OS's and OS-level deps require different configurations
        # when reading from the OS, so we eliminate any discrepancies by removing the link
        # tags entirely
        for tag in script_tags:
            tag.getparent().remove(tag)

        return lxml.html.tostring(tree).decode("utf-8")

What it does:

finds all statically resolved stylesheets and replaces their paths with their absolute path equivalents
inlines the modified HTML using css-inline
strips the link tags from the document

The stripping of link tags is quite likely the responsibility of another custom tag, but it suits my use-case when generating PDFs - wkhtmltopdf on MacOSX fails to resolve the absolute URLs with its default config, but succeeds in Docker - so I encapsulated the removal in the inline_css tag.
Loving css-inline - great work!


Instead of using api: '/' we need to allow api: '' to be a valid URL.
I tested this locally and it seems to work if I allow empty string by changing the schema validation from routes.api: joi.string() to routes.api: joi.string().allow('').
Interesting!

We might want to prevent '/' too since that does not work.

When I leave api empty I get error:
[16:27:55] INFO (payload): Starting Payload... [16:27:57] ERROR (payload): There were 1 errors validating your Payload config [16:27:57] ERROR (payload): 1: "routes.api" is not allowed to be empty [nodemon] app crashed - waiting for file changes before starting...
Payload version 1.2.1


Update Travis CI node version

Travis CI has marked the Yarn version used in the project as deprecated, therefore we are upgrading Node to the latest stable version in order to bring in the updated bundled Yarn dependency

Had some issues in finding the right configuration since what's indicated in the official TravisCI documentation won't install the desired yarn version for Angular 12.2.x, nor what's in other official sources.
Also ran into a bug in Node 17 and had to downgrade back to 16. Hence so many commits for such a simple fix


Revisit: default verification method logic in GeneratePresentation command

@sudeshrshetty what is the plan for this? How should this be revisited? What are/were the challenges encountered?

ie. what is the reason for all this defensive logic? https://github.com/hyperledger/aries-framework-go/blob/b74fc05533e2f7bcf0f4d91da4569d7e169c5728/pkg/controller/command/verifiable/command.go#L863-L913

This logic presents clients from reaching verifiable.Presentation.AddLinkedDataProof() which allows any verificationMethod. This logic should have perhaps been implemented there.

There is an inconsistency between the verificationMethods allowed by the REST API and those allowed by AddLinkedDataProof()

Unable to add support for capabilityChain as a proof option when creating credentials.
capabilityChain is required by ZCAP-LD.

ZCAPs != Verifiable Credentials.
If the spec ever matures enough then we should add support for it in the framework as an API separate from VCs and VPs.


Update virtual-machines-allocation-failure.md

#sign-off

Hi @JiangChen79, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (cjiang). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Folders not creating what it suppose to do

Describe the bug
I tried to test about huggingface qwen2.5-coder.  and enter the prompt Design an accounting application with a modern, sleek, and animation-rich user interface. The project should be developed using Next.js and shadcn.  than voila :) and this happens all model local. what am i wrong about.

Link to the Bolt URL that caused the error
http://localhost:5173/chat/accounting-app-nextjs-shadcn
Steps to reproduce
Just selected Hugging face and qwen 2.5 and enter the prompt than you will(i suppose) get the same error.
Expected behavior
Create a right folder structure and add shadcn properly. Thanks for you all great effort.
Screen Recording / Screenshot

Platform

OS: [ Windows]
Browser: [Chrome,Arc,Brave]

Additional context
No response

I´ve got the same problems with nextjs. It creates a app-folder while init, but the further steps / implementations of bolt, not putting it into the app, instead putting it directly in the root.

did the newest version fix this?

Yes thanks for this great effort. Nicely done!


Raxml

Added basic RAxML functionality to q2-phylogeny.

Added single-thread and multi-threaded executable options for RAxML.
Added RAxML-specific tests along with their associated FASTA and Tree test files.
Added RAxML citation.
Added dependency and citations for future IQ-TREE plugin development.

BTW, if you want to run the tests and linting locally, you can do the following (in your dev env):
# one-time, just to install deps into your env
pip install -q flake8 coveralls
pip install -q https://github.com/qiime2/q2lint/archive/master.zip

make lint
make test

@thermokarst Thanks for those last set of comments! I've added _build_rapid_bootstrap_command and its associated test case. Then I added an additional flag to raxml. I am set to go with the merge if these additions check out. 😃

Thanks @mikerobeson!
A handful of upstream packages recently changed, which has created a bit of cleanup work for us on our build system. I am waiting to do the final review and merge after we address those (unrelated problems). Just wanted to give you an update on the hold-up here! Thanks for being patient!

@mikerobeson, now that this is merged, any thoughts on contributing a Community Tutorial?

I'd be happy to contribute to a tutorial!


crash refined pipes +immersive engineering

https://paste.gg/p/anonymous/63d5b80629f74df2a2da40f8abbce523
MC 1.16.5
Forge 26.2.23
rf 0.51
thanks for looking into it

Not a Refined Pipes issue


Add responsive and animated footer

Can I work on this issue please assign it to me

@1dharode please assign me this task under Hacktoberfest 2022

@1dharode please assign me this task as it will give a blooming start to my open source journey


Remove dependencies on polyfills?

In relatively recent versions of Node, including the current LTS, object-assign and nodegit-promise are unnecessary. It'd be great if promisify-node didn't directly depend on polyfill (or polyfill-like in the case of object-assign) libraries and instead required that the environment provide them if necessary.

nodegit-promise is not unnecessary. it has custom promise implementation that allows us to synchronously poll it's state in C++, which is not available with the built in promise implementation.
Object assign probably yeah, although how long has that been in? We're staying supporting node 4 since it's LTS.

FYI, Object.assign is definitely in node v4. Check out node.green for a decent feature support table.


GAS Gateways

Gas is the power that runs Neo network, which Neo itself is the governance token behind it.
In some situations, community members have expressed their desires of simplifying user operations, e.g., having to buy gas, just to transfer a bunch of Neo... this can happen with any NEP-5, or any asset transfer in fact.
This would require buying these Gas on an exchange, or DEX, but even if it's few cents, this would require several compliance measures, KYC, and in countries like mine (Brazil), user would need to present detailed reports for even few cents operations.
Therefore I believe that creating such an integrated feature would be very helpful to users.
Proposal can be something like this:

a method for getratio, which will give ratio of that token/asset (NFT, NEP-5, ...), into GAS.
a method for getgas, which will attach the precise amount of GAS to backup user tx fees.

Note that getgas shouldn't give profit for users, nor any excess, to avoid DEX discussions. This is good for any NEP-5 that implements this NEP-Z, and the only complicated one now is Neo: because it's not fractionary.
Since excess shouldn't come back to users, it would be complicated to spend a single Neo per tx... since Neo already generates Gas, this is more-or-less solved, but if it gets fractional, then it would be able to benefit more from this.
Implementation:
To implement getratio: contract could use embeded Neo oracles, to periodically update gas ratio (ratio could be higher than market, which is good for gas offers, and also for users that get an integrated service)
To implement getgas: execution could read from Sender field, or any other requirement of fees, and authorize transaction upfront (as long as ratio is matched).
To keep flow of gas, listed third-party addresses with funds could be provided on contract, and used to extract funds. A command like registergas could be provided, with some locking mechanism to keep it there for a while.
One interesting point is that this is not a compete approach to DEX, but complementary... it's very likely that DEX would also support some GAS gateways, to back-up their own user operations. As long as this NEP is very simple, we can easily integrate it to native mechanisms of Neo blockchain.
Thanks to @MorganDream @edgedlt @vncoelho and many others that kept discussing this important topic in the past months... let's add other issue references here too. Please let's discuss this carefully.

Something like that would surely help, I've faced an exchange that deals with ERC20 tokens and not with ether and they only need it to pay for the transactions, and even some users that don't trade with ether needs to have it to move their funds.
In NEO we may have the NEP-5 token, and also some NEO but if we don't have GAS we can't make the transaction.
But how to calculate the proper ratio and where the GAS will come from? Would the wallet holders be allowed to lend some GAS to the network?

I think the proposal is great and would cover my current needs where I'd want to pay with my NEP-5 token.
There are usecases I can can think of, where a project could simply say it will cover all transaction costs just to have free token transfers, they earn enough from other things to keep transactions free for the users.
For that the project could fill/refill the gas gateway at all times with GAS. In that case we would not need to require oracles getting a conversion ratio, but simply pay for the transaction.

I guess it's an interesting direction @rodoufu ... users may lend it for a short period of time (few blocks), and re-claim it if not used. Problem is: these operations would also cost to the network, the lending and re-claiming. So, perhaps best option would be to operate with bigger values and let a third-party manage the margins via oracle operations (that would also cost...)


[Bug report] Scroll jumps when removing item after calling standby(isRemove: true)

Version
1.23.0
Platforms
iOS
Device Model
iPhone 16 Pro Max
flutter info
[✓] Flutter (Channel stable, 3.24.3, on macOS 15.0.1 24A348 darwin-arm64, locale en-US)
    • Flutter version 3.24.3 on channel stable at /Users/lobsta/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2663184aa7 (8 weeks ago), 2024-09-11 16:27:48 -0500
    • Engine revision 36335019a8
    • Dart version 3.5.3
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/lobsta/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16B40
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 21.0.3+-79915917-b509.11)

[✓] VS Code (version 1.95.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.100.0

[✓] Connected device (4 available)
    • iPhone 16 Pro Max (mobile)      • 339438FD-89FF-4359-AEC3-B3C707943695 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-18-1 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 15.0.1 24A348 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 15.0.1 24A348 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 130.0.6723.116

[✓] Network resources
    • All expected network resources are available.

• No issues found!

How to reproduce?
Running my example code, scroll down a little bit, press the - button which removes an item from the top, scroll then jumps. If I press the + button it adds an item to the top and the scroll does not jump, unlike removing an item.
I'm removing the item like this:
_chatObserver.standby(isRemove: true);

setState(() {
     _items.removeAt(0);
});

Logs
No response
Example code (optional)
import 'package:flutter/material.dart';
import 'package:scrollview_observer/scrollview_observer.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const IssuePage(),
    );
  }
}

class IssuePage extends StatefulWidget {
  const IssuePage({Key? key}) : super(key: key);

  @override
  _IssuePageState createState() => _IssuePageState();
}

class _IssuePageState extends State<IssuePage> {
  final List<int> _items = List.generate(100, (index) => index);
  final ListObserverController _listObserverController = ListObserverController();
  late final ChatScrollObserver _chatObserver = ChatScrollObserver(_listObserverController)..toRebuildScrollViewCallback = () => setState(() {});

  @override
  void dispose() {
    _listObserverController.controller?.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Issue'),
        actions: [
          IconButton(
            tooltip: 'Remove from top',
            onPressed: () {
              _chatObserver.standby(isRemove: true);

              setState(() {
                _items.removeAt(0);
              });
            },
            icon: const Icon(Icons.remove),
          ),
          IconButton(
            tooltip: 'Add to top',
            onPressed: () {
              _chatObserver.standby();

              setState(() {
                _items.insert(0, _items.isEmpty ? 0 : (_items[0] - 1));
              });
            },
            icon: const Icon(Icons.add),
          ),
        ],
      ),
      body: ListViewObserver(
        controller: _listObserverController,
        child: ListView.builder(
          itemCount: _items.length,
          shrinkWrap: _chatObserver.isShrinkWrap,
          physics: ChatObserverBouncingScrollPhysics(
            observer: _chatObserver,
          ),
          padding: const EdgeInsets.all(8.0),
          reverse: false,
          itemBuilder: (context, index) => Padding(
            padding: const EdgeInsets.only(bottom: 2.0),
            child: ListTile(
              tileColor: Colors.primaries[_items[index] % Colors.primaries.length],
              title: Text('Test ${_items[index]}'),
            ),
          ),
        ),
      ),
    );
  }
}

Contact
No response

This is expected behavior. You removed the first item, and all subsequent items will definitely scroll up.

_chatObserver.standby(); prevents the scroll from jumping when adding, shouldn't _chatObserver.standby(isRemove: true); do the same for removing?

The standby method will at most perform two operations:

Keeping the message position.
To observe whether shrinkWrap needs to be switched (observeSwitchShrinkWrap).

When you add a message, it will perform the above two operations, but if you delete a message, it will only perform the second operation.

The standby method will at most perform two operations:

Keeping the message position.
To observe whether shrinkWrap needs to be switched (observeSwitchShrinkWrap).

When you add a message, it will perform the above two operations, but if you delete a message, it will only perform the second operation.

Alright, I guess this package isn't for my use case then. Thanks for responding.

@dalobstah you can maybe try super_paging for your usecase. It contains a Bidirectional ListView which you check source code for some help.


Possible outdated content

Hello, customer noticed that this referenced procs as well as the use of the deprecated AzureRm module and was wondering if this content is still accurate? They were using the Azure Hybrid Benefit WS SA Count Tool as well. Hoping to verify whether or not that still works.

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

ID: 90e335dd-0bc5-a7e3-650b-e3dd1823176b
Version Independent ID: 18f34ef3-ff61-0473-a64b-ac5a64dbe1d7
Content: Azure Hybrid Benefit for Windows Server
Content Source: WindowsServerDocs/get-started/azure-hybrid-benefit.md
Product: windows-server
Technology: getting-started
GitHub Login: @eross-msft
Microsoft Alias: chrisrin

( also ref. issue ticket #488 from the same page )

Thanks for the feedback. The article no longer references processors or the AzureRm PowerShell module.  The changes should be visible later today.
#please-close


Udev issues when symlinking cameras

I recently swapped out computers that I use to work with my 2 DFK33UX264's. I did a fresh install of Mint 19 and Tiscamera. I noticed there have been quite a few changes in tiscamera and Mint19 vs Mint18 such as the v4l2 controls and the way that they are read out.
I believe I have most of that sorted but I'm having issues with udev symlinking. Previously I did this with a single .rules file containing only the lines:
SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e", ATTRS{idProduct}=="9089", ATTRS{serial}=="#serialN#", SYMLINK+="topcam"
SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e", ATTRS{idProduct}=="9089", ATTRS{serial}=="#serialN#", SYMLINK+="botcam"
I did not use the 80-theimagingsource-camera.rules file. To load the UVC extensions I just dropped the usb3.xml file into the /usr/share/uvcdynctrl/data/199e directory, I notice the 80~.rules file has another way of doing this with a "tcam-extension loader"
Right now for some reason with the tiscamera install and 2 cameras are plugged in I get 4 kernel video devices (video0-3) shown in /dev along with my 2 symlinks, "topcam" and "botcam". On boot they do not always link the correct video device, they randomly link to the 2 phantom devices from time to time.
What I need to do is create a symlink that is based on the serial number of the camera that will always be linked to the botcam/topcam video devices. I'm not sure how to get this working right on this install.

Hi there,
We deprecated the use of uvcdynctrl as the maintainability of extension units was not feasable.
If you have BUILD_TOOLS enabled tcam-uvc-extension-loader will be built and installed.
Within your build directory in data/udev is the generated rules file that contains the absolute paths necessary for it to work. You can simply copy the extension unit command out of it.
As for the double entries:
They are generated by the kernel due to the fact that the camera has 2 descriptor interfaces. This is something that can not be removed.
Since all cameras will always have 2 interfaces take the first one as that will always be the one you want. i.e.  0,2,4 etc.
If you require serial associations for dev nodes look into /dev/v4l/by-id there you can find symlinks.

I've tried the best I can to review your suggestions but It seems to create some new issues that I'm not sure I can resolve all issues without a further suggestion.

They are generated by the kernel due to the fact that the camera has 2 descriptor interfaces. This is something that can not be removed.

That's not something that happened with the same cameras on the old PC install so I assume this second interface is created by Tcam?

Since all cameras will always have 2 interfaces take the first one as that will always be the one you want. i.e. 0,2,4 etc.

I assume you mean to take the kernel name /dev/videoN which is lowest for that camera? That doesn't solve the original problem that the kernel will instantiate physical devices in a manor that is not consistent. The whole purpose of a Udev symlink that was based on serial was to say "serial #1234 links to topcam" and "serial #4321 links to botcam" because regardless serial #1234 may instantiate as /dev/video0 or /dev/video1. Now the case is that serial #1234 may be /dev/video0+1 or /dev/video2+3. The real problem is that both video0 and video1 links will be attached to serial number info but only one is attached to the useable v4l2 instance, the other has no v4l2-ctrl or image attached as far as I can see. The compounding issue is that the V4l2 attachment can be on either video0 or video1 just as inconsistent as the kernel instantiation.
So it appears I can work around this by using the "v4l....by-id-index-0" symlink because it does reference the cameras unique serial and it does seem to be consistent that the v4l2-ctrl usable interface is always attached to the "index-0" symlink. The problem I encounter with that is that OpenCV needs to refer to a device index, and that's a problem for the reasons above. Currently I'm thinking that querying the v4l link with "readlink" in program will expose the current video device but I'm wondering if that's the best approach.
~$ readlink '/dev/v4l/by-id/usb-The_Imaging_Source_Europe_GmbH_DFK_33UX264_28810325-video-index0'
../../video2

upon boot that kernel device could be any one of 2-4 instances, upon re-plugging the camera it almost certainly will change

I think I finally found a way to reliably create a correct symlink.
The rule would look like this:
ACTION=="add", SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e",\
    ENV{ID_V4L_CAPABILITIES}==":capture:", \
    ATTRS{serial}=="17810181",  \
    GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl", \
    SYMLINK+="topcam"

The important line is ENV{ID_V4L_CAPABILITIES}==":capture:",
It looks at the udev attributes and selects the capture interface for the symlink.
To see the most important properties execute:
udevadm info --query=all --name=/dev/video0

For all other solutions you'd have to write your own program and iterate the devices with libudev.

I wrote the following file as a 80....rules file in /etc/udev/rules.d/ copying out what looked necessary in the example .rules file from the tiscamera /data directory. It's not producing the "topcam" and "botcam" symlinks in /dev as the rules I posted above does. Any suggestions?

This file assures that The Imaging Source USB cameras are correctly recognized
if no driver has claimed the interface yet, load uvcvideo
ACTION=="add", SUBSYSTEM=="usb",
ATTRS{idVendor}=="199e", ATTRS{idProduct}=="8101",
RUN+="/sbin/modprobe -b uvcvideo"
add the imaging source VID and PID to the list of devices supported by uvcvideo
ACTION=="add", SUBSYSTEM=="drivers",
ENV{DEVPATH}=="/bus/usb/drivers/uvcvideo",
RUN+="/bin/sh -c 'echo 199e 8101 > /sys/bus/driver/uvcvideo'"
give users permission to interact with cameras
SUBSYSTEM=="usb", ATTRS{idVendor}=="199e", GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl"
load uvc extensions for USB3 33 cameras, requires absolute path
ACTION=="add", SUBSYSTEM=="video4linux",
ATTRS{idVendor}=="199e", ATTRS{idProduct}=="90??",
RUN+="/usr/bin/tcam-uvc-extension-loader --device=/dev/%k -f /usr/share/theimagingsource/tiscamera/uvc-extension/usb33.json"
##specific camera rules here
#topcam
ACTION=="add", SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e",\ ENV{ID_V4L_CAPABILITIES}==":capture:", \ ATTRS{serial}=="28810325", \ GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl", \ SYMLINK+="topcam"
#botcam
ACTION=="add", SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e",\ ENV{ID_V4L_CAPABILITIES}==":capture:", \ ATTRS{serial}=="43710275", \ GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl", \ SYMLINK+="botcam
<
It does appear to load the extensions without the usb3.xml file in /usr/share/uvcdynctrl/data/199e, the cameras list v4l2-ctrls that would be unavailable as a generic V4l2 device.

If you want to have the camera specific rules in one line you have to remove the \.
They are there to tell udev that the rule continues on the next line.

I probably should have known that about the "". Anyway I fixed that in udev, the links are created, but they are pointed at "fuse". I'm not quite sure what that's about.
# This file assures that The Imaging Source USB cameras are correctly recognized

# if no driver has claimed the interface yet, load uvcvideo
ACTION=="add", SUBSYSTEM=="usb", \
ATTRS{idVendor}=="199e", ATTRS{idProduct}=="8101", \
RUN+="/sbin/modprobe -b uvcvideo"

# add the imaging source VID and PID to the list of devices supported by uvcvideo
ACTION=="add", SUBSYSTEM=="drivers", \
               ENV{DEVPATH}=="/bus/usb/drivers/uvcvideo", \
               RUN+="/bin/sh -c 'echo 199e 8101 > /sys/bus/driver/uvcvideo'"

# give users permission to interact with cameras
SUBSYSTEM=="usb", ATTRS{idVendor}=="199e", GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl"

# load uvc extensions for USB3 33 cameras, requires absolute path
ACTION=="add", SUBSYSTEM=="video4linux", \
               ATTRS{idVendor}=="199e", ATTRS{idProduct}=="90??", \
               RUN+="/usr/bin/tcam-uvc-extension-loader --device=/dev/%k -f /usr/share/theimagingsource/tiscamera/uvc-extension/usb33.json"

# specific camera rules here
#topcam
ACTION=="add", SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e", \
               ENV{ID_V4L_CAPABILITIES}==":capture:", \
               ATTRS{serial}=="28810325", \
               GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl", \
               SYMLINK+="topcam"

#botcam
ACTION=="add", SUBSYSTEM=="video4linux", ATTRS{idVendor}=="199e",\
               ENV{ID_V4L_CAPABILITIES}==":capture:", \
               ATTRS{serial}=="43710275", \
               GROUP="video", MODE="0666", TAG+="uaccess", TAG+="udev-acl", \
               SYMLINK+="botcam"

~$ ls -l /dev/topcam
lrwxrwxrwx 1 root root 4 Sep  6 18:37 /dev/topcam -> fuse

~$ readlink /dev/topcam
fuse

Any ideas on this? I realized that the above udev file pretty much links devices randomly. Recently I found that the topcam symlink was actually linked to my NVME drive!

Hello
Sorry for not answering. I needed some time to understand your issue. I have the same on my Ubuntu 18.04: Connecting a camera to the computer and I get
/dev/video0
/dev/video1
This even happens with the USB 2.0 cameras, which definitely have only one interface : UVC only.
I removed the files from uvcdynctrl, but I suppose, that has no effect on the issue at all, because the issue happens somewhere in udev.
I also removed the /etc/udev/rules.d/aravis.rules, but this also does not make an effect. There are still two /dev/video* shown.
We have to continue searching.
Stefan

Hi
I have learned, that the double devices are caused by the Kernel. The second device is for an UVC Meta Data Stream, but this is unused.
Thus you may make your simlinks to /dev/video0 and /dev/video2.
Stefan

Hi,
that doesn't really work because /dev/videoN is instantiated by the kernel at random. So serial#1234 may be video0, right now, on next boot it may be video2. This is OK if you are ignoring /dev/videoN and symlinking to a serial number, but now you must refer back to video0 or video2 because video1 and video3 exist?
This creates a somewhat convoluted issue with simply identifying individual cameras reliably. When there is one video device per hardware camera it is the typical Linux Kernel issue of just udev symlinking based on a hardware feature that is unchangeable. Now with 2 devices per camera a rule must be written on the user's end to enumerate the lower of the 2 kernel instantiated video devices?
My programmer friend's solution to this is to read which link has v4l2-ctrl output on it and simlink that camera but that depends on my original symlink of identifying which set of links is attached to which serial number. You can see this becomes very convoluted to manage. The v4l2-by-id symlink is probably the only thing reasonable to use in this case but the user must be aware to use the lower instance of the by-id link. I think there should be a better way.
If it's a kernel issue I have not seen it in 4.9. I believe the current build is running 4.18 so I suppose this came about between the 2 versions? I can easily test this on a machine running 5.3, but I assume you are saying the issue will present itself on any machine based on kernel version independant of tiscamera?

Regarding Kernel versions:
I use 4.19 and I did not saw this issue on older Kernels, therefore, that must have been invented in between, as you think.
The best way to identify the cameras is using the serial numbers and query the dev node programmatically. The following C++ sample project shows, how to enumerate the cameras, video formats, frame rates and also how to set properties.
v4l2_cameratest.zip
You may pick from it, what you need to open the cameras.
However, this tool will list the cameras twice too.
Stefan


add taurus/bzt/BlazeMeter compatibility

Codecov Report

Merging #53 into develop will decrease coverage by 0.63%.
The diff coverage is 25%.

@@             Coverage Diff             @@
##           develop      #53      +/-   ##
===========================================
- Coverage    92.12%   91.48%   -0.64%
===========================================
  Files            7        7
  Lines          419      423       +4
===========================================
+ Hits           386      387       +1
- Misses          33       36       +3

Impacted Files
Coverage Δ

pytest_play/plugin.py
72.32% <25%> (-1.76%)
:arrow_down:

Continue to review full report at Codecov.

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


Handling events

I would like to possibly add event handling to the serialized object. The patch would have to be able to contain events that need to be added / removed, but it would be up to user code to do the handling.
I'll have to familiarize myself with the format, but what I think is needed is something like: "addedEvents": ["click", "blur"], "removedEvents": ["mouseup"]
Then it could possibly be used something like this (is this a good api?):
function handleEvent(ev) {
  worker.postMessage({
    type: ev.type,
    msg: "something happened..."
  });
}

applyPatch(el, serializedPatch, {
  handleEvent: handleEvent
});

Whether the API is good or not, how feasible is this?

How would the functionality itself be handled? Correct me if I'm wrong, but wouldn't you need to serialize the listener functions as well? Otherwise it seems reasonable to me. :)

It depends on what you are trying to do. In my case I just want to know what type of event happened, send that back to the worker and handle it somewhat globally. I have enough information in the event, I don't need to call the real handler (which is on the worker side).
If you wanted to call the listener function I would imagine it might work like this (not knowing vdom's structure super-well). You would want to know the path to the VNode within the associated tree. Maybe handleEvent would get that info as a second argument:
window.js
function handleEvent(ev, path) {
  let cloned = Object.assign({
    path: path
  }, ev);
  worker.postMessage(cloned);
}

applyPatch(el, serializedPatch, {
  handleEvent: handleEvent
});

worker.js
let tree; // this is my cached VNode tree

onmessage = function(msg){
  let ev = msg.data;
  let propName = `ev-${ev.type}`;

  // Find the VNode within the tree
  let vnode = find(tree, ev.path)

  // Call the function
  vnode[propName](ev);
};

This is pseudocode since I don't know the VNode structure all that well, been a while.  This assumes events are set up using the ev-click convention.

On second thought I wonder if it wouldn't make more sense for you to implement this as a fork of this project. Looking back on the code I believe I started from the assumption that the patch would be extremely simple in order to optimize the patch size as much as possible. I might be hesitant to merge the PR if the resulting changes will balloon the code size.
OTOH I would encourage you to work on this since worst case scenario is that I would kindly ask you to make a forked project. :D

No problem 👍


Update feeds in the JSON files to include feed_contact_email

User Story

As an app developer, I want to know who I can contact with questions about the feed, its data quality, and licensing requirements.

Acceptance Criteria

All the JSON files in the catalogs repo are updated to include data_producer_email when known (this could be purely manual work or a mix of automated scripting and manual work)

Considerations

We added this field to the Google Form  and contributors have been including it in most cases, so there are about 25 feeds with this info right now. Adding more would require scripting to see how many feeds include feed_contact_email in feed_info.txt and do a one time script to update them.

Adding to the catalogs repo instead: https://github.com/MobilityData/mobility-database-catalogs/issues/285


Bad responses: FortunesPostgreSql-Actix, LinkAThonGrpcTrimmedAndR2R, LinkAThonGrpcTrimmedAndR2RSingleFile

Some scenarios return errors:

Scenario
Environment
Last Run
Errors

LinkAThonGrpcTrimmedAndR2R
Linux, Physical, http, KestrelSockets
10/5/19 2:09:05 AM +00:00
54992

LinkAThonGrpcTrimmedAndR2RSingleFile
Linux, Physical, http, KestrelSockets
10/5/19 2:10:47 AM +00:00
404595

Logs

Still have issues

@rynowak I don't believe this is a gRPC issue, more an issue with linking. Can you do the needful?


202 triangular weighted function

Reference issue #202
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

I have a few comments:

I have created a function to do linear triangles, with the same slope either side of the peak. I'm not sure this is what Nigel envisaged based on his pdf document, as he has described a different way of calculating weights. I could implement his non-linear triangular weights in stead of or as well as the linear triangle I've implemented.
I have not implemented the full functionality to redistribute weights if there are missing points in the coordinate. At the moment weights are created for the points in the input cube's coordinate of interest.  I could implement the functionality to redistribute weights that we have for other functions that create weights, but it would involve some refactoring of the utilities Caroline created to do this.
I am concerned about the duplication we have in these weights functions. They basically have the same interface, and just take different kwargs for the parameters needed for the particular function used to create the weights. I feel there are probably better ways round this, and it is coming up elsewhere (e.g. in Neighbourhood plugins). We probably need to spend a bit of time thinking about this, else our CLIs will become unmanageable.
This function assumes we will always be blending over a coordinate, which will hopefully be the case for topography bands in the future


pg-config-executable-not-found - linux

As an ADMIN:
When I try to build the application with bundle, the build fails to locate pg_config, even though it's there and already added to my $PATH.
It should recommend installing the appropriate libraries, but it recommends installing postgresql-server-dev-X.Y, which is not an actual package.
I did some research, and found a StackOverflow article that had an answer that fit this specific situation - based on the order of the setup instructions. What ended up working for me is adding the following packages:
sudo apt-get install libpq-dev python-dev

Additional Information:
cat /proc/version
Linux version 4.4.0-62-generic (buildd@lcy01-30) (gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4) ) #83-Ubuntu SMP Wed Jan 18 14:10:15 UTC 2017

Adding this to CONTRUBUTING.md should resolve this issue. :rocket:

@sethbergman spot on , thank you

Fixed in #626


TextAlign RTL

We probably need to create an RTL-aware version of TextAlign that is used by TextPainter, then use that in RenderParagraph, RichText, and Text so that you can align start/end as well as left/right. Also "justify" needs to define the justification of the last line.

The engine needs to know the directionality to know how to draw text (consider how "eng1 HEB2 eng3" would render in RTL vs LTR).

Fixed by https://github.com/flutter/engine/pull/4001


[Feature] Option to set the elevation only if the token's elevation is lower

It would be useful to have an option allowing the elevation to be applied only if the token's elevation is lower.
For example, in the case of a token that can fly, is 30ft high and passes over a region that is 5ft high. It would be logical for its elevation not to be affected.

If the region is 5' high, then it will not affect a token that is 30' high. You do need to set the maximum height of the region for that to be true, however.

Thanks, it works to set a maximum elevation, I hadn't thought of that!

Thanks, it works to set a maximum elevation, I hadn't thought of that!


Task processing in event sync

Issue

sync event is processing all changes as they happened on task
few small issues breaking event synchronization

Changes

task changes are filtered at one place and cares only if name or typeid changed
ftrack entities are queried by their ids not as parent ids
added query for tasks as they need different data
tasks of created entity is not processed on avalon document creation but are moved to "task changes" processing
task types are queried at once
few formatting changes

thanks. tested on multiple scenarios and seems to be ok again.


[ownership] Update emitIndirectConditionalCastWithScalar for ownership.

This comes up because when we perform mandatory inlining, we perform the
transform as we inline. So the tests for this are in mandatory_inlining
naturally.

@swift-ci smoke test and merge


[DOCUMENTATION] Update README Format

Describe the usability issue
It's a little bit confusing what configurations are needed by which versions of the app.
To Reproduce
Steps to reproduce the behavior:

Go to README.md
If you look at the formatting, it isn't completely clear which versions or version ranges need specific configuration files.

Expected behavior
Please update the README or improve the formatting using heading hierarchies or horizontal rules to differentiate different sections related to config files. Thanks.

Hi @sandeep-ps,
For /assets/configs.json.enc it says "NB: For versions prior to 2.8."
For /assets/organizations.hook.json.enc it says "NB: For version 2.8 and later."
For /assets/organizations.json.enc it says "NB: For version 2.8 and later."
In short, /assets/configs.json.enc is for backward support as the current version is 2.12.
As regards to /assets/organizations.hook.json.enc vs /assets/organizations.json.enc - they both work but currently we have not set up organization definitions anywhere on our backend so we currently use /assets/organizations.json.enc statically embedded for our builds.
I am not sure how heading hierarchies can make this more clear. I have no idea what is "horizontal rules". Could you please give a visual idea how this data could be presented less confusing?


fix mac cover upload when frozen

Fixes #482
Also adds homebrew 7z to PATH on Mac when frozen and removes the included binaries.
Also, do you have a particular reason to not ignore *.spec files? I don't see a reason to include them since they are auto-generated by PyInstaller and are included in the standard Python gitignore: https://github.com/github/gitignore/blob/main/Python.gitignore
And that plist appears to be unused.

Also, do you have a particular reason to not ignore *.spec files? I don't see a reason to include them since they are auto-generated by PyInstaller and are included in the standard Python gitignore: https://github.com/github/gitignore/blob/main/Python.gitignore

spec files are used in the build process and had to be adapted from the auto generated ones.
https://github.com/ciromattia/kcc/blob/217f571f3d8b24fca13667de46e59effb926dc44/.github/workflows/package-windows-with-docker.yml#L28-L42

And that plist appears to be unused.

The file is needed by the bundle normally.
https://developer.apple.com/library/archive/documentation/MacOSX/Conceptual/BPRuntimeConfig/Articles/ConfigFiles.html#//apple_ref/doc/uid/20002091-CJBJIEDH

An information property list file is a structured text file that contains essential configuration information for a bundled executable.

Are you using a bundled binary?
Here you said you are not.

I've never tried running the binary.
Originally posted by @axu2 in https://github.com/ciromattia/kcc/issues/482#issuecomment-1575099852

removing the included binaries will complicate things for those using them with older mac os architectures.
disclaimer: I don't own an Apple computer currently and can't test this PR right now.
A good soul will have to test/review this.
(Or I need to annoy my mother-in-law and borrow hers)

OK, I thought multiple issues were related, but turns out they aren't, so I'll be editing this PR to only focus on the plist.
PyInstaller provides a plist file like in my first post here:

If we attempt to copy in our own plist we get the permission issue. I have no clue why, but I don't think our plist is doing anything functional.

Superseded by https://github.com/ciromattia/kcc/pull/566 due to merge conflicts


fix: json parser

Fix the JSON parser if the content contains escaped characters.
Before:

After:

BTW, is there a straightforwardforward way to localize statement.where.require ?

BTW, is there a straightforwardforward way to localize statement.where.require ?

Emmm... Provide another endpoint to get rule details by the rule type? @tianzhou
For example:
GET /rule/{rule type}

Returns
{
  "title": "",
  "description": "",
  "doc": "",
  // ...etc
}

But seems the localization should always be in English.


[GLUTEN-4326][CH] Fix unsupported operator ShuffleQueryStage for BroadcastRelation

What changes were proposed in this pull request?
When AQE is enabled but spark.sql.adaptive.coalescePartitions.enabled is false, AQE converts shj to bhj in runtime but the ShuffleQueryStageExec of shj has been generated, which causes the exception java.lang.UnsupportedOperationException: Not supported operator ShuffleQueryStage for BroadcastRelation when createBroadcastRelation for it.
(Fixes: #4326)
How was this patch tested?
manual tests, ut seems not easy to construct.

please add a ut

please add a ut

I have attempted to add it, but it's challenging to construct suitable examples.


bigquery: add Uploader.Put example using ValuesSaver

We never show how to upload rows that are just []Value.

I'm sure I can handle this one. I have used almost all possibilities of the old and new versions already :)
Are we talking about a simple implementation of the value.ValueSaver interface? Or use value.ValuesSaver as stated in the title that already implements this interface so that we just show how to pass the schema and it's inserted into a BQ table?
With this info I'm pretty sure I can open a PR so that you can check it and ask me to adjust it. Also, I'll need to know how to add an integration test. Means, if we must set it so that a project ID, dataset and table is passed as an env variable or it is hardcoded in the code like "put-your-project-id-here".

Thanks for offering to help! You should first read CONTRIBUTING.md.
This issue is about adding an example of ValuesSaver, not ValueSaver, to examples_test.go. We have an integration test that uses it already, so no need to write another one. Just write a simple example, like the other ones for Uploader.Put.

Hi again It's the first time that I use Gerrit and I'm not sure if I understood well the committing process. I have set the aliases, did the git change, a new branch was created (tracking origin/master?), did the commit, fix the things that the server was asking and did the git mail. Everything look fine.
A tag has been created in my local but I haven't been able to push anywhere
It was mailed as https://code-review.googlesource.com/#/c/gocloud/+/20650
Sorry for bothering :( I just want to do it well. In fact I use all the code in this repo so much that I could help in many other areas so it's good that I understand the workflow well before bothering too much :)

Mailing is pushing.
I got an email with your CL and commented on it. Next time, include me as a reviewer when you mail it, and it will show up on my gerrit dashboard as well.

Sorry, I mess it up and because I couldn't manage to push a second commit. I could only managed to push a new commit by amending the previous one. I thought it was "the way" but it has created a new change:

Yes, you do need to amend the commit back into 1 total commit, but you need to preserve the magic Change-Id line from the original commit. The Change-Id is the primary key.

Ok. I think I managed to update the proper Gerrit ID. I guess it will appear in jba dashboard. Maybe we can update also the file for contributing with a small example?

Step 6 under Contributing in CONTRIBUTING.md says "Keep amending to the change with git codereview change and mail as your receive feedback. Each new mailed amendment will create a new patch set for your change in Gerrit." Did you do that and it failed to preserve the Gerrit ID?

I'm not sure what I did but I solved it by copy-pasting the Gerrit ID from the UI.
Now I've just submitted the new patch by simply doing a git add bigquery/examples_test.go, then a git commit --amend and git mail. The ID was fine.
Now I just have to know how to add comments so I don't have to keep writing here once I'm working in a change :wink:

Fixed by f66389d56fdc547c6c76f1dd0377065e7a319aa3.


Publish updates from master

Publish the latest changes from master

Perhaps include https://github.com/docker/docker.github.io/pull/11618 (once reviewed)


Missing Effort.CsvTool

Description
In your documentation(https://entityframework-effort.net/export-data-to-csv), it link to Effort.CsvTool but there is 404 not found error.
https://github.com/zzzprojects/EntityFramework-Effort/tree/develop/Main/Source/Effort.CsvTool
Exception
404 error

You are right,
The branch do not longer exist.
However, you can find it in the master: https://github.com/zzzprojects/EntityFramework-Effort/tree/master/Main/Source/misc/Effort.CsvTool
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function


fix: use rustls instead of openssl-sys

This removes the dependency on openssl, which is not installed by default on many Linux distros.

Thank you for the commit


Contradicting guidelines about default aggregation in XML views/ fragments

In Keep Your Views Short and Simple, there is the suggestion to remove default aggregation tags:

Remove Clutter From Your Views
It's easy to save a few bytes and make your code a lot cleaner:

Don't define properties that are set to their default values.
Remove unused namespace aliases.
Omit the content or items tag for controls that define default aggregations.
Use self-closing XML tags for controls that don't define any aggregations.

On the other hand, the Walkthrough Step 11 says the following:

Conventions

Do not make implicit use of default aggregations but always declare the aggregation names explicitly in the view. In the example above, the content aggregation could also be omitted as the Panel control declares it as a default, but it makes the view harder to read.

Would be nice if this contradiction could be addressed internally.
I personally agree with the 1st guideline; omitting default aggregation tags not only reduces the file size, it actually improves readability by reducing levels of indentation. Additionally, the 2nd guideline defies the whole purpose of defaultAggregation in control metadata.

Hi,
this is a good point, we will look into this.
Thanks and best regards,
Oliver

Hi,
we removed the contradicting section from the Walkthrough step. You were right, this was outdated and is not a best practice.
Thanks again for bringing this to our attention!
Best regards,
Oliver


Bad access when producing a pixel buffer

Hi! So I've run into an interesting issue that I found in InternalRecorder.swift. I managed to get it to crash at this line:
https://github.com/gorastudio/SCNRecorder/blob/6ea497e347cab67460a1fcc4a4a739d5d6a79a70/Source/InternalRecorder/InternalRecorder.swift#L278
I can never capture the frame with the actual error but I was able to see it output some sort of EXC_BAD_ACCESS error. This seems to only be happening to me when recording an SCNView.

Hi! Yea, it looks interesting. Do you use Metal or OpenGL based rendering?

I currently use Metal based rendering. Mostly just out of the box SceneKit right now.

OK, thank you! I will investigate for a week and get back to you.

Should be fixed in the new version. I can't reproduce it and there are no memory leaks.


Work with More Than Two Controllers

If you have two pairs of controllers (4 in total), you cannot combine the second set into a second combined controller. Also, currently, the four can not all be added as controllers, instead the last two added are ignored.

this should be pretty easy to add, but combining the second set of controllers might be a bit tricky. I'll probably just set it up to use vJoy devices 3 & 4 if they are detected, but it might be a while before I combine them, ideally I'll make a controller system like the switch has, but that'll take a while.


Don't try to put the repo manifest in the initramfs image.

It isn't very useful there and just takes up space.

Good point! But maybe we even can add it to certain image types only instead of inheriting image_repo_manifest in sota.bbclass?

But maybe we even can add it to certain image types only instead of inheriting image_repo_manifest in sota.bbclass?

I wondered about that. I think this way is probably slightly preferable, though, because we want to encourage this as the default behavior. For example, if someone were to build their own custom image, we'd want them to have this enabled by default so that the repo manifest is available without any extra work.


Frontend state is not updated when engine is restarted

If a run is started and engine is then killed and restarted, the frontend still shows the engine as running and therefore does not allow starting the engine. The state is fixed with a browser refresh but can be very confusing for users, especially when configuring an uod. This may take quite a few engine restarts.
The cause is likely in aggregator which could be missing a few event pushes to the frontend.

Fixed by #425


Request for Documentation and Example on ANR Detection in OpenTelemetry Android Library

Hello OpenTelemetry Team,
I am currently working on integrating the OpenTelemetry Android library into my project and I'm particularly interested in utilizing the ANR (Application Not Responding) detection feature. However, I have not been able to find concrete examples or detailed documentation on how to implement this feature.
Based on the repository description, it is mentioned that the library supports ANR detection among other features. Despite my attempts, I encountered several issues with dependencies and couldn't find the necessary AnrInstrumentation class or relevant documentation to guide me through the implementation.

Hi @stas-rudevitsky, thanks for the inquiry. We're in the process of a major re-shaping of this instrumentation and how it is wired up to this agent and your application. We don't yet have a release build that contains the changes, nor do we have a working example yet. It's coming soon, though!
If you don't mind getting your hands dirty, you can roll up your sleeves and try the 0.7 SNAPSHOT build before then. You'll need to make sure you match the otel android agent version with this instrumentation version. Including it in the runtime classpath should be enough for the ANR instrumentation to be installed at runtime.
The snapshots are here: https://oss.sonatype.org/content/repositories/snapshots/io/opentelemetry/android/instrumentation-anr/0.7.0-alpha-SNAPSHOT/
For ANR, you'll add the maven snapshot repo to your gradle build config, and then use io.opentelemetry.android:instrumentation-anr:0.7.0-SNAPSHOT.
Let us know how it works out, or check back in a few weeks.

Is there is any update on how to implement the ANR, or some of the other features ?

Hi @AhmedMannai10. The top-level agent project includes the ANR instrumentation, so if you depend on it and create an instance of OpenTelemetryRum (like the demo-app does) you should get ANR instrumentation automatically. It will create a span with the name "ANR", although I'd expect that to change when we get semantic conventions defined for this.
We don't yet have the ability to simulate/cause an ANR in the demo app, so this would be a welcome contribution. I'll open an issue for that, so let me know if you'd like to work on it.


Error message after installing in Kirby 5

Hi,
I just installed your plugin on the latest Kirby 5 beta version and I am getting the following error. I am not sure what exactly it means.

Yes, there is no working Kirby 5 version yet.

Ah ok., I thought because of the index.php, that you only support Kirby 5 and higher:
if (
	version_compare(App::version() ?? '0.0.0', '5.0.0-beta.1', '<') === true ||
	version_compare(App::version() ?? '0.0.0', '6.0.0', '>') === true
) {
	throw new Exception('Kirby SEO requires Kirby 5');
}

Do you plan on upgrading it?

Yes, there will be a 5.0 compatible release. However, there's still too many moving parts for it to be finished. Please only consider the actual versioned releases "ready-to-use".


Bug: Mailbox tracking problem

Flitter's message
Okay, in the new version:
1- Hop in the Tank and Training Room still fail to be tracked.
2- Think Clearly is tracked properly. The outside coin is still incorrectly marked inaccessible.
Since the last apworld update there was a new Mailbox found that was added. This could have made some mailboxes ID's mismatch.

Resolved in commit 4a9050336d1d79cf5dd5caefce26fb32f31bc076 (Pushed into version 0.4.0)


CSS reset on hyperlinks causes paragraph spacing issue (Outlook)

How can we reproduce this bug?
Within a column, add two paragraphs - the first containing a link. E.g.
<columns>

If you did not request this change pleasecontact us.

Best regards.

</columns>

What did you expect to happen?
The paragraphs to have the normal, Foundation-defined margin between them.
What happened instead?
The first paragraph has no margin-bottom, so the paragraphs run against each other as if there was only a linebreak between them. This appears to be caused by a CSS reset within the Foundation framework which adds Margin: 0; margin:0; to all 'a' elements - this overrules the 'p' element's margin.
It's possible to remedy it by adding custom CSS but I'm not sure whether this is an adequate solution?
p a { Margin: default; margin: default; }
Here's a screenshot demonstrating what happens:

What email clients does this happen in?
Outlook 2007, 2010, 2013 and 2016.

It looks like the syntax has nested paragraph tags. Can you verify that this syntax still has an issue?
<columns>

If you did not request this change pleasecontact us.

Best regards.

</columns>

Sorry, that's purely my error transcribing it here. Should have copy/pasted directly from the template itself. I've edited the original post to reflect the actual markup.

Ah, thanks! We'll take a look into this! Sounds like that your custom CSS fix might help, would you mind making a PR so we can quickly test it?

I'll certainly try - very new to Github so it may take me a moment to work out how!

Oh awesome! Welcome to GitHub! If you're familiar with making branches it shouldn't be too different.
First you''ll need to make a new branch to work from with this command:
git checkout -b [name_of_your_new_branch] [name_of_your_new_branch] can be any name you want. It's best to be descriptive with the feature you're working on.
Make and commit the changes in that new branch and  run this command:
git push origin [name_of_your_new_branch]
Then you hop over to github and suggest that branch as a pull request. Checkout the documentation here: https://help.github.com/articles/creating-a-pull-request/
There's a lot of cool people around these parts that can help you out as well :)

@antonydjames code fixes this for me; added PR #762


ES unit test failing on x86_64 due to uint32 memory handle

Another case of a uint32 being used to store a memory pool handle, which fails when running the unit tests on a 64-bit build.
Using the CFE_ES_MemHandle_t typedef instead solves the problem.

Imported from trac issue 114.  Created by jphickey on 2015-10-06T10:22:36, last modified: 2019-03-05T14:57:55

Trac comment by jphickey on 2015-10-06 10:28:19:
Pushed branch trac-114-es-u32-memhandle-fix, commit [changeset:f40cfe5] for review

Trac comment by glimes on 2015-10-06 12:26:12:
Recommend accept.

Trac comment by sstrege on 2015-10-19 18:02:21:
approve/accept

Trac comment by glimes on 2015-12-22 15:54:31:
2015-12-22 merged to development

Trac comment by sduran on 2015-10-20 12:47:17:
recommend accept

Trac comment by glimes on 2016-02-25 10:17:32:
these will be fixed in CFE 6.5

Trac comment by jhageman on 2019-03-05 14:57:55:
Milestone renamed


Output coordinate layer should show coordinate value as header, display field in popup, not 0

If you collect coordinates and export to KMZ, the output popup in ArcGIS Earth is headed with "0" values for all coordinates that you click, and if you expand the KMZ item in the Layer List, all of the points have a "0" beside them.  This seems to be coming from the ID field, which is set to "0".

Since we know that all output is going to have a "coordinate" field, we should set the layer for the output symbology to use "coordinate" as the primary display field, like this:

This would match the planned behavior for the Details window (https://github.com/Esri/coordinate-conversion-addin-dotnet/issues/591).
Interestingly, when you export from ArcMap (10.3.1) with the same build, you do get this behavior.

Seems to be just my week, but I'm having problems getting KMZ export to work on Pro 2.4, it seems like a threading issue, but looking into

Tested exporting from Pro and Arcmap and Legend and Pop-up looks good now - created new issue #604 for new crash observed on export at Pro 2.4:

https://solutionsdev.arcgis.com/defense/help/coordinate-conversion/resources/june-2019-coordinate-conversion.htm
https://solutionsdev.arcgis.com/defense/help/military-tools/resources/june-2019-military-tools-for-arcgis.htm
\...\DefenseSolutions\CoordinateConversion\Testing\CTLocalGovTeam-dev\130\
\...\DefenseSolutions\MilitaryToolsforArcGIS\Dev\292\

Doc verified - website and PDFs in (MT 305 and CC 139)


Reference group @id is sometimes a broken link in the catalog

For example, in the catalog entry for LibGit2Sharp <IP_ADDRESS>, the link at entry["references"]["group"]["@id"] is broken.
The value is:
https://api.nuget.org/v3/catalog0/data/2<IP_ADDRESS>.09.32/.json#gpref
I think it should be:
https://api.nuget.org/v3/catalog0/data/2<IP_ADDRESS>.09.32/libgit2sharp.<IP_ADDRESS>.json#gpref
@johnataylor, I think this is in the XSLT that builds the catalog entry.

@joelverhagen @johnataylor Has this one been fixed? Doing some spring cleaning to see what's to be done and has been done :)

Yes, seems to be fixed.


Broken link to Azure DevOps

markdown-link failure:
https://powershell.visualstudio.com/PowerShell/_build/latest?definitionId=32
[+] ***/tree/master/docs/FAQ.md should work 457ms
2020-06-03T20:01:32.8918939Z       [-] https://powershell.visualstudio.com/PowerShell/_build/latest?definitionId=32 should work 60.64s
2020-06-03T20:01:32.9327050Z         RuntimeException: Failed to complete request to "https://powershell.visualstudio.com/PowerShell/_build/latest?definitionId=32". Response status code does not indicate success: 404 (Not Found).
2020-06-03T20:01:32.9335025Z         at <ScriptBlock>, /home/vsts/work/1/s/test/common/markdown/markdown-link.tests.ps1: line 119
2020-06-03T20:01:33.1943488Z

https://dev.azure.com/powershell/2972bb5c-f20c-4a60-8bd9-00ffe9987edc/_apis/build/builds/54596/logs/23

Looks like the fix is https://powershell.visualstudio.com/PowerShell/_build/latest?definitionId=32 ->https://powershell.visualstudio.com/PowerShell/_build?definitionId=32
I will open PR.


examples/bigsim/emulator: Lots of unsafe shared variables in SMP builds

Original issue: https://charm.cs.illinois.edu/redmine/issues/233

Several of the example codes for the bare BigSim emulator API use global variables for various bits of state, such as the size of each unit of decomposition or the number of iterations completed. When run under an SMP build of the runtime with +ppn X (X > 1), these global variables present race conditions that can cause many unpredictable crashes.
Assuming I've diagnosed that condition correctly, I see a few options:

Make the code thread safe with appropriate shared/thread variable setup and/or synchronization
Under SMP builds, shut down at startup, rather than trying to run.

These issues were uncovered by running autobuild SMP targets with +ppn 2.

Original date: 2013-06-17 00:30:44

Gengbin's response to my email:

If it is the problem of variable declaration in jacobi3d program, we definitely should fix the code.
Use  BpvStaticDeclare  for bigsim "core"-level variables, and BnvStaticDeclare  for bigsim "node"-level variables.
Don't forget to call initialize function to initialize the variables properly.

Original date: 2013-06-17 00:43:27

Gengbin, I'm not sure I understand. Many of the variables in question are effectively readonly variables, that simply need to be properly initialized before anything starts to read from them.

Original date: 2013-06-18 02:07:51

Test disabled on stable branch so that we can see more clearly that everything else passed.

Original date: 2013-06-18 22:40:32

Gengbin notes that this became problematic because of the fix to #161. Further notes there.

Original date: 2013-06-19 19:14:03

The variables involved were being assigned idempotently. As Gengbin noticed, the problem was that the initial changes from #161 left an extra argument that confused the emulator code.


Crash with v2.4.0b

http://pastebin.com/Ferj4keC

This appears to be a duplicate of #154, although it is a version newer. I blame DMF :P

I blame Donald Trump!


[UI] Unable to cancel out of node template modal

Version: master-head (7/18/19)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a node template via the "Add Cluster" page, create a cloud credential if necessary. You first need to submit your cloud credential to see node template settings.
Now, try to Cancel out.

Result:
Browser dev console reports that this.cancel is not a function
Clicking cancel button won't do anything in the UI.

@davidnuzik, sorry I cannot reproduce this issue, can you give me more info to reproduce it.
Here are my steps:

Go to Add Cluster - Amazon EC2 page.
Click "Add Node template" button.
Input keys and click the "Create" button.
After showing node setting, click the "Cancel" button.

Result:
Modal cancel success and browser dev console did not report errors.

This should be fixed in master

Version: master-head (8/16/19)
I went through the Add cluster wizards with Chrome and Firefox. First as pointed out using Amazon EC2, but then I performed a verification to the other cloud providers Add cluster wizards hitting Cancel in all the steps (where applicable) before Create.
I also finished creating an Amazon EC2 and DO without hitting cancel. Clusters were successfully. created.


Add lsp-test ⚡

Adds some initial language server tests using lsp-test.
As this is a Haskell library, I've also added https://github.com/haskell/actions to the CI workflow.
Also did a bit of cleanup of the ditto-lsp code 🧹

Note that I initially captured the LSP logs and asserted that they hadn't changed from previous runs as part of the test. This caused two issues:

The order of log messages is indeterminate. Particularly when calling log::info! et al. as part of LSP notification handlers.
The log messages mention URIs (i.e. file paths) that are specific to the current machine. Hence the logs from a CI run would mention different paths to the ones from my local machine.


drill cannot break blocks with enchantment silk touch

drill cannot break blocks with enchantment silk touch
https://github.com/user-attachments/assets/7b839f2e-98c3-4c34-8119-19d35379f4dd

Thank you for reporting this issue!
It seems similar to an issue we encountered before. Based on the problem description, it is likely that the FTB Chunks or another chunk protection mod is preventing the drill from breaking blocks. This happens because the chunk protection mod is blocking actions from FakePlayer, which the drill uses to interact with blocks.
To resolve this, please check the settings of the chunk protection mod (such as FTB Chunks) and make sure to allow FakePlayer interactions in the protected chunks.
You can refer to a similar solution discussed in this issue: #9.
When building issues in the future, please check past issues and include at least the information necessary to reproduce the bug, such as version and mod configuration.

thank you, yes this is chunk protection by PneumaticCraft


How to convert  []Result to json string ?

How to convert  []Result to json string ?

.String()

is the array has .String() method ? @wMw9

Pretty basic stuff but I'll play along.
// Assuming targeting list
var myResults gjson.Result

for _, r := range myResults.Array() {
	myString := r.String()
	fmt.Println(myString)
}

You may need to read up on the result type methods, they will guide you...


Feature Request: Penner Easing Animation

Is it possible to add Penner animation on the list?
That would be awesome!
Thanks.

I think you'd need javascript for this and, you know, animate.css... So it'd be a no.
In some animations you can change the timing function to your needs but it is kinda limited.
If you need complex animations you'd better use greensock, velocity or any other animation library.


bug: Possible prob when listing virt envs

Hi, I just read about your plugin on Reddit and thought I'd give it a try. I use fish shell and also virtualfish to manage my virtual envs. When I do :WhichPy select then it lists double items under ~/.virtualenvs which is the location where the virtual envs are stored.
Here's a screenshot of what I'm getting (I'm using fzf-lua picker, but also happens with the builtin and telescope)

fixed.
thank you.

@neolooong Just so that I don't open another issue, I also noticed that when I choose a virtual env from the fzf-lua picker, it doesn't seem to get activated. I checked with :!python3 --version from Neovim command line. With venv-selector whenever I did :!python3 --version after choosing a virtual env it showed the correct version of Python3 for the corresponding virtual env. In my screenshot above test venv is with python3.10 while Mit is with python3.12.
Another thing I noticed is that if I want to disable the locator with opts.locator.enable = false I get an error
          Failed to run `config` for whichpy.nvim

...azyVimDev/lazy/whichpy.nvim/lua/whichpy/locator/init.lua:26: attempt to index local 'locator_opts' (a boolean value)

# stacktrace:
  - /whichpy.nvim/lua/whichpy/locator/init.lua:26 _in_ **setup_locator**
  - /whichpy.nvim/lua/whichpy/config.lua:87 _in_ **setup_config**
  - /whichpy.nvim/lua/whichpy/init.lua:7 _in_ **setup**
  - ~/.config/LazyVimDev/lua/config/lazy.lua:11
  - ~/.config/LazyVimDev/init.lua:24

it doesn't seem to get activated

Mmm... yes, whichpy doesn't activate venv. The purpose of plugin just want to make LSP work with specific interpreter.
:!python3 don't use you selected is what I want. Imaging a situation, you select a venv which don't contain pip file. Then :!pip install something would messup your environ.

disable the locator with opts.locator.enable = false I get an error

The correct way to disable locator is like this:
require("whichpy").setup({
  locator = {
    poetry = { enable = false },
    conda = { enable = false },
  }
})

Ok, thanks for clarifying. I'm pretty new to programming myself and Python.
Can you elaborate on the purpose of the plugin, because I don't exactly understand it? I thought it was for listing the virtual envs and be able to switch to one, but apparently I'm mistaken. What exactly do you mean when you say "The purpose of plugin just want to make LSP work with specific interpreter"?

After selecting the interpreter path, Whichpy only updates the LSP configuration and notifies the LSP server. This plugin does not handle:

Using :python to execute code
Automatically activating virtual environments when opening terminals inside neovim
etc


load progressive image with fallback

I am trying for a couple of days now to address the following problem:
I am loading a transparent image at first, which gets overwritten by the original image as soon as it's loaded. The problem is that the original image is a progressive image and I was expecting it to be shown as soon as its fist view was fetched. However, the whole image has to be downloaded before the switch takes place.
Question:  POC, Is this possible? If yes, how? Any workarounds, links, etc, are welcome.
Expectation: Load the page with, initially, transparent placeholders ( in order to avoid the "jump" effect) and switch to the original image as soon as the first (incomplete) view is available.
My setup (trimmed down) is as follows:
Markup

<img
  src="{{ _imgFallback }}"
  data-src="{{ _image }}"
  class="lazyload"
/>

<noscript>
  <img
    src="{{ _image }}"
  />

Css
.lazyload {
  opacity: 0;
}

.lazyloading {
  opacity: 1;
  transition: opacity 300ms;
}

// if .no-js, remove the lazy load img and display the fallback one (no script)
html {
  &.no-js {
    .lazyload {
      display: none;
    }
  }
}

JS
window.lazySizesConfig = {
      init: false,
      addClasses: true,
      preloadAfterLoad: true
    };

BTW, very nice plugin and excellent documentation.

You have to create a new image and replace it with your old one. This means a new plugin.
Something like this:
img
  src="{{ _imgFallback }}"
  data-progressive-src="{{ _image }}"
  class="lazyload"
/>

document.addEventListener('lazybeforeunveil', ({target}) => {
    const progressiveSrc = target.dataset.progressiveSrc;

    if(!progressiveSrc){return;}

    const newImg = document.createElement('img');

    target.replaceWith(newImg);
    target.classList.add('lazyloading');

    target.addEventListener('load', function(){
        target.classList.add('lazyloaded');
    }, {once: true});
});

Note: While I thinkt the code could run. I haven't tested it (It didn't even run in any browser) and it uses some very modern JS and DOM features.

That's actually working! Just need to add:
newImg.src = progressiveSrc;

lol the most important thing ;-)

Hmm I hurried up and answered. Image is loading progressively but there is a problem.
As soon as the original image starts downloading, it replaces the transparent one. But, the original image does not have width and height set on its first Bytes thus creating a jump effect in case you are running in slow connection.

is there an event triggered for "actual initial progressive image with width and height set shown"?
are you aware of any work around that?
what if I kill completely the transparent image? Can I wrap the image in somehow responsive divs?

I am investigating the above and I will update if I find any solution but unfortunately I think 1 and 3 have already POC failed :/


Remove survey link

We have enough responses now and we are in the lockdown for improving the documentation.
We can add another "did we improve?" survey after lockdown sometime.
Test Plan:
Went to http://localhost:8079/react-native/docs/getting-started.html (and other pages) and saw that the survey was not in the footer any longer.

By analyzing the blame information on this pull request, we identified @vjeux and @easyCZ to be potential reviewers.

Can you also remove the css and the image from the repo?

(Feel free to ship it when done)

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Keycodes are overloaded by key names for single numbers

I think you have a bug in bind: it recognize both the keycodes mentionned in wev (ex: 64 for Alt_L) and the keynames (Alt_L)
However, the single digit keycodes 1..9 are taken over by the number keys of the same name, this is visible if you want to specify a bind to the actual Escape keycode (9):
 bindr=,9,exec, foot

It gets executed instead executed when you press 9.
A solution would be either a toggle (to avoid having to change config files), or requiring the use of the full names for number keys, so "Nine" not 9 which should unambiguously mean the keycode.
 # Esc (keycode 9) opens a terminal
 bindr=,9,exec, foot
 # The number nine does whatever
 #bindr=,Nine,...

This will be mostly seen with a modifier as the number keys are often accessed directly, but some minimalist keyboard layouts omit the number keys so it could be a problem for them.
Even outside of this specific case, i's not always possible to just use the keynames: for examples, if using the kb_options you can have 2 different keycodes both sending the same keyname (ex: caps:ctrl_modifier: both the caps and lctrl key will send Control_L) which necessitates the use of keycodes in bindr mappings to disambiguate

Wonderful idea! It's even better, because it would clearly make the numbers stand out as a code!
I'd suggest adding a comment pointing to where the codes can be found, like how the keyboard options are found in /usr/share/X11/xkb/rules/base.lst
I'll be eagerly waiting for this fix: now that my caps key is both control and escape, I want to have the old escape key as a quaketerm :)
ty!

codes are found in wev and it's mentioned in the wiki.

done in 463690a27ac9c921d34dad7169a3d2c8cea5b46f


Read methods/functions accept iterables of paths

Allows passing iterables of paths to dd.read_csv/dd.read_table/dd.read_parquet/db.read_text. Fixes #1948.

cc @martindurant for review.

Hah nice - it all just works and gets passed through.
I see no problems here.
I wonder, is it worth saying in the docstrings that the schemas of the files all need to match? Maybe that's obvious, but maybe not.
For the case of parquet, what do you think is the cleanest way to make arrow (which wants a directory to search for data files) and fastparquet (which wants a metadata file, a directory with a metadata file or a list of data files) to cooperate?

For the case of parquet, what do you think is the cleanest way to make arrow (which wants a directory to search for data files) and fastparquet (which wants a metadata file, a directory with a metadata file or a list of data files) to cooperate?

I'm not sure what you mean here.

Arrow works with either a directory structure (and doesn't require a metdata file) or a list of parquet files.
Fastparquet works with a directory structure (and requires a metadata file), or a list of individual parquet files

In the long run it would be nice to have fastparquet support directories without metadata files, and pyarrow to support writing the metadata file.


Forum Search Enginer "boardtracker" provides no service

It seems that http://www.boardtracker.com/ should be removed from the list as the site provides no service.
Forums -> Forum Search Engine -> BoardTracker

Dropped this to my dead tools tracker.  Thanks.


code is not running with raster mode printer

this is not working with TSP100LAN and other raster mode printer. i think this is only for line mode printers and the receipt xml files are provided only works with line mode.
could you please update for raster mode printing?

This would be great if this was possible!

There has been lots of changes and improvements to the star SDK that it’s better to just download the official sdk instead and rewrite the code.


Remove the wrapping from the top menu

Can you  remove the wrapping from the top menu ? See screenshot:

But then you'll lose sight of some links (they will be cut out)?

Hmm. May be create additional popup menu at right side of top menu?
and this popup menu will consist of second line menu items.

I can't think of anything better. I imagine it would work by setting fixed height of the menu and there would be a button for extending the menu to access hidden links. By default, this feature will be disabled, as not everyone has the top menu so wide.
And by the way: check the settings for plugins which have added additional links to this menu. Maybe some of them can be moved to the main menu bar, so they'll make the top one shorter. For example, Backlogs plugin does have such option.

There are two variables for you to set:

$top-menu-collapse must be set to true to enable this feature,
$top-menu-collapse-breakpoint determines the maximum screen width in which expanding button will be shown (so users with a wide screen resolution won't be bothered with this). Default is 1200px

You are awersome! :+1:  :+1:  :+1:


Revert "Release 0.6.0"

merge into master should use per commit merge

Codecov Report

Merging #1107 into master will decrease coverage by 2.75%.
The diff coverage is 66.27%.

@@             Coverage Diff              @@
##             master    #1107      +/-   ##
============================================
- Coverage     41.83%   39.08%   -2.76%
+ Complexity     1361     1119     -242
============================================
  Files           243      221      -22
  Lines         10111     8833    -1278
  Branches       1319     1130     -189
============================================
- Hits           4230     3452     -778
+ Misses         5334     4940     -394
+ Partials        547      441     -106

Impacted Files
Coverage Δ
Complexity Δ

.../src/main/java/io/seata/rm/tcc/TwoPhaseResult.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...java/io/seata/server/session/SessionCondition.java
66.66% <ø> (+19.99%)
2 <0> (-5)
:arrow_down:

...seata/core/exception/AbstractExceptionHandler.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...a/io/seata/core/rpc/netty/AbstractRpcRemoting.java
9.32% <ø> (ø)
4 <0> (ø)
:arrow_down:

.../io/seata/core/rpc/netty/NettyPoolableFactory.java
10.34% <ø> (ø)
2 <0> (ø)
:arrow_down:

...ain/java/io/seata/tm/TransactionManagerHolder.java
69.23% <ø> (ø)
3 <0> (ø)
:arrow_down:

...ava/io/seata/core/constants/ConfigurationKeys.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ommon/src/main/java/io/seata/common/Constants.java
0% <ø> (-50%)
0 <0> (-1)

...rm/tcc/remoting/parser/LocalTCCRemotingParser.java
51.61% <ø> (ø)
4 <0> (ø)
:arrow_down:

...in/java/io/seata/rm/datasource/StatementProxy.java
25% <ø> (ø)
3 <0> (ø)
:arrow_down:

... and 43 more

Continue to review full report at Codecov.

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

Or you close the PR, and I do it.

You can revert the master, and push master -f ,then merge 0.6.0 to master again with the origin commits.
Or you close the PR, and I do it.

after revert the master,why  execute push -f  master?


[FLINK-12433][runtime] Implement DefaultScheduler stub

What is the purpose of the change
This implements the already existing DefaultScheduler stub to wire the following components together: SchedulingStrategy, ExecutionFailureHandler, and ExecutionSlotAllocator. Note that the implementation is not yet complete. For example, high availability support is still lacking.
cc: @tillrohrmann
Brief change log

See commits

Verifying this change
This change added tests and can be verified as follows:

Added unit tests for DefaultScheduler (DefaultSchedulerTest)
Manually configured MiniClusterITCase to use ng scheduler. Most test are passing.
Manually submitted streaming and batch WordCount job to standalone cluster

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks @GJL for working this PR.
It looks good to me, except for the RestartBackoffTimeStrategy loading part where the placeholder can be replaced with RestartBackoffTimeStrategyFactoryLoader#createRestartStrategyFactory now since FLINK-12709 is merged.
All the follow-up tasks have tickets opened to track them.
I think we can merge it once the RestartBackoffTimeStrategy loading part is reworked.

Merging as soon as build is green.


footer not properly centered

need to remove footer elements padding
"ul" element in footer has padding: left attribute that should be removed
there is a random floating comma underneath the footer that needs tot be deleted

fixed in #48