TypeError: '>' not supported between instances of 'LooseVersion' and 'Version'

The following error happens on Linux (flatpak version) using inputstreamhelper 0.6.1:
  File "/home/user/.var/app/tv.kodi.Kodi/data/addons/plugin.video.amazon-test/resources/lib/playback.py", line 471, in PlayVideo
    suc = _IStreamPlayback(asin, name, streamtype, isAdult, extern)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.var/app/tv.kodi.Kodi/data/addons/plugin.video.amazon-test/resources/lib/playback.py", line 342, in _IStreamPlayback
    if not inputstream_helper.check_inputstream():
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/__init__.py", line 420, in check_inputstream
    if self.drm == 'widevine' and not self._supports_widevine():
                                      ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/user/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/__init__.py", line 163, in _supports_widevine
    if parse_version(config.WIDEVINE_MINIMUM_KODI_VERSION[system_os()]) > parse_version(kodi_version()):
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '>' not supported between instances of 'LooseVersion' and 'Version'
-->End of Python script error report<--

@emilsvennesson same errors appears about two hours ago when i try to use my DAZN account for watch Juventus Napoli (whit my valid login).
I have report the error here on addons fork:
https://github.com/Maven85/plugin.video.dazn/commit/fb45f98ec7e16299cb80aaac4aa27f52077eec28#commitcomment-147020701
but i have tryed to (Re)Install the Widevide CDM this errors appears:
`2024-09-21 20:12:23.637 T:894745 warning : [script.module.inputstreamhelper] Recursively create directory '/home/diabolik/.var/app/tv.kodi.Kodi/data/userdata/addon_data/script.module.inputstreamhelper/temp/'.
2024-09-21 20:12:23.769 T:894745   error : EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <class 'TypeError'>
Error Contents: '>' not supported between instances of 'LooseVersion' and 'Version'
Traceback (most recent call last):
File "/home/diabolik/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/default.py", line 8, in
run(sys.argv)
File "/home/diabolik/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/api.py", line 15, in run
widevine_install(choose_version=params[2])
File "/home/diabolik/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/api.py", line 48, in widevine_install
Helper('mpd', drm='widevine').install_widevine(choose_version=choose_version)
File "/home/diabolik/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/init.py", line 34, in clean_before_after
result = func(self, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/diabolik/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/init.py", line 219, in install_widevine
if not self._supports_widevine():
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/diabolik/.var/app/tv.kodi.Kodi/data/addons/script.module.inputstreamhelper/lib/inputstreamhelper/init.py", line 163, in _supports_widevine
if parse_version(config.WIDEVINE_MINIMUM_KODI_VERSION[system_os()]) > parse_version(kodi_version()):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '>' not supported between instances of 'LooseVersion' and 'Version'
-->End of Python script error report<--
2024-09-21 20:12:29.210 T:21       info : CActiveAESink::OpenSink - initialize sink
2024-09-21 20:12:29.210 T:21       info : CAESinkALSA::Initialize - Attempting to open device "default"
2024-09-21 20:12:29.217 T:21       info : CAESinkALSA::Initialize - Opened device "default"`

@Paco8 if you want to try to back to use your addons try to disable the "inputstream helper" from setting in the debug section.

Can you test if https://github.com/emilsvennesson/script.module.inputstreamhelper/pull/571 solves it?

Can you test if #571 solves it?

How i can install this ? I can download the zip manually from branch and install manually ?

Can you test if #571 solves it?

Yes, it fixes the problem for  me.

Can you test if #571 solves it?

Yes, it fixes the problem for me.

How you have installed that ?

I downloaded this:
https://github.com/emilsvennesson/script.module.inputstreamhelper/tree/fix-version-parsing

I downloaded this: https://github.com/emilsvennesson/script.module.inputstreamhelper/tree/fix-version-parsing

simply download as zip and install on kodi ? or it's need to do other ?

Yes, download as zip and install.

Yes, download as zip and install.

@Paco8 i thinks at right now also the master branch as the #571 applied... try to install this :)

@emilsvennesson strange issue.. i have installed the master branch zip for resolve the issue.... now kodi say to me it's exist an update for this addons and suggest to me to upgrade to, apparently, same version from kodi official repository.
It's a newest 0.6.1 has been released recently ?

Fixed on 0.7.0.

@emilsvennesson in the meantime i have understand because, after the installation of master branch zip manually, my kodi keep to continue to found an update:
Current 0.6.1 on Kodi main repo it's "named 0.6.1+matrix.1"  if i remember. The master zip branch in addons.xml simply 0.6.1... for kodi the +matrix.1 it's an updated addons :)
Thanks for 0.7.0.... need to wait as been push on main kodi repo.
Or you have a your personal repo ?

@DjDiabolik It is awaiting approval in the Kodi repo, but you can download it from https://github.com/emilsvennesson/script.module.inputstreamhelper/releases/tag/v0.7.0 in the meantime.

@DjDiabolik It is awaiting approval in the Kodi repo, but you can download it from https://github.com/emilsvennesson/script.module.inputstreamhelper/releases/tag/v0.7.0 in the meantime.

Exactly...That's the first thing I thought of too. Thanks for fixing this issue :)


segment erro:tensorflow/tensorflow/examples/label_image/main.cc

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):source
TensorFlow version (use command below):r1.13
Python version:3.6
Bazel version (if compiling from source):0.23
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:9.0/7.0
GPU model and memory:1080ti/11gb

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
Describe the expected behavior
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
tensorflow/tensorflow/examples/label_image/main.cc
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
i think the reason is "TF_RETURN_IF_ERROR(env->NewRandomAccessFile(filename, &file));"
https://github.com/tensorflow/tensorflow/blob/5fad2f50d610fdb765f4f99c7e0531b7a3ddbe94/tensorflow/examples/label_image/main.cc#L100

I make a mistake
here
https://github.com/tensorflow/tensorflow/blob/5fad2f50d610fdb765f4f99c7e0531b7a3ddbe94/tensorflow/examples/label_image/main.cc#L103

If I commente out this ，it's ok.

@allendred Is it resolved or still an issue? Could you provide more details on the issue and provide a code to reproduce the bug? Thanks!

@allendred Is it resolved or still an issue? Could you provide more details on the issue and provide a code to reproduce the bug? Thanks!

I changged to another demo(detection),abondon the function ("read_entirefiles"),so it works

@allendred Thanks for the confirmation. I will close the issue. Thanks!


zeus console loads pry instead of irb

zeus console loads pry instead of irb
when running zeus c, it loads pry console by default, instead of irb console. Is there a way to load irb by default instead of pry?
System details

uname -a:
Linux dev 3.13.0-65-generic #106-Ubuntu SMP Fri Oct 2 22:08:27 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

ruby -v:
ruby 2.1.2p95 (2014-05-08 revision 45877) [x86_64-linux]

Steps to Reproduce

Ensure you have pry gem installed

2)zeus console in a new rails project
Observed Behavior

[1] pry(main)>

Expected Behavior

Loading development environment (Rails 3.2.19)
cfs-web(dev)>

Option 1: Remove pry from your Gemfile.
Option 2: Add require: false to pry in your Gemfile.


Possible bug in binary classification calibration_error

🐛 Bug
In calibration_error(), the accuracies in the binary classification setting are not correctly computed I think. It just returns the targets. I am guessing this should rather return target == preds.round().int() or something similar? Am I missing something?
Code example
import torch
from torchmetrics.functional.classification import calibration_error

preds = torch.tensor([0.01, 0.001, 0.005])  # The raw sigmoid output
targets = torch.tensor([1, 1, 1])
calibration_error(confidences, targets)
# This returns: tensor(0.9947)

The model confidently predicts the wrong class, but is rewarded with a near perfect calibration score.
Environment

TorchMetrics version (and how you installed TM, e.g. conda, pip, build from source):

Version 0.9.1
Installed with mamba

Python & PyTorch Version (e.g., 1.0):

Python: 3.9.13
PyTorch: 1.11.0.post202

Any other relevant information such as OS (e.g., Linux):

I am on Ubuntu, Linux.

Added a little example to better illustrate my point.
By the way, using just a 0-vector would have been a simpler example, but it turns out the preds can't be 0 exactly due to how the binning is done. It could make sense to clamp the predictions in the binning process to prevent this. E.g.:
torch.clip(confidences, 1e-6, 1.0)

Hi,
I checked this issue as an bigger refactor (see this issue https://github.com/Lightning-AI/metrics/issues/1001 and this PR https://github.com/Lightning-AI/metrics/pull/1195) and it seems that our calibration error is computing the right value.
First, in the example provided the metric is giving a score of 0.9942. As the metric is an calibration error the optimum would be 0 and not 1 and it therefore seems correct that the metric is giving a high score as the example is clearly not well calibrated.
Secondly I ran the example through an third party package https://github.com/fabiankueppers/calibration-framework which gives the same result as our implementation (we are actually using it for testing now).
Therefore, there does not seem to be an error in the implementation.
Closing issue.

I also have the same problem.
def _binary_calibration_error_update(preds: Tensor, target: Tensor) -> Tensor:
confidences, accuracies = preds, target
return confidences, accuracies
How could the target be possible equal to accuracy?
The target is ground truth label and accuracy is preds == target.


[bug] log file not cleared on reload-window (MacOS)

See 5 messages here:
https://github.com/bhoov/manim-notebook/pull/57#issuecomment-2444095357

Is the old log file ever cleared?
I just reloaded the window,
opened the log file,
but all my old logs are still there..
(I thought the README says to reload the window - in order for the old logs to be removed?)

wanted behavior is described in this video

@VladimirFokow I've opened an issue for this in the vscode repo here. Feel free to subscribe to it should they have MacOS-specific questions.

To make this issue ready for v0.1.0 NOW:

describe in the Troubleshooting section:
If you're on MacOS: first use manim command to open the log file,
then delete everything from it,
then reload window (reloading the window will log to this file your system info),
(then - the rest of the steps)

move the Troubleshooting section into the GitHub wiki
(because: we don't want to put these distracting details on the main page of the extension which all first-time users see)
(just can add a link to it in the "Links" section of the README)

Afterwards, whenever the technical limitations are solved by the VSCode maintainers,
our workflow can be improved accordingly.
But we don't need to wait for them now - to make this extension usable comfortably.

The new way is awesome!
This issue will be closed by it


Examples/Documentation for replacing component local state?

Hi,
I like the look of hyperapp but i can't find any info/best practices on how to replace local component state?
For example, i would love to know how would one implement an extremely basic expand/collapse component? In React the component would simply track state.isExpanded and return either props.children or something else based on the value of it. This is good because you don't have to worry about creating state.expanderXIsExpanded at the root component for every instance of the component in your app!
Many thanks.

Hello!
Hyperapp doesn't support local state, so everything has to be added to the single state object.
The term "Component" in hyperapp refers simply to a view function, similar to the stateless functional components in React.

@rssfrncs
@skaterdad is only partially correct. It's true that hyperapp itself does not implement support for local state in anyway. That does not mean that stateful hyperapp components arent possible though. See https://zaceno.github.io/hypercraft/stateful-components/ for an explanation, or https://github.com/zaceno/hyperapp-nestable for the lib I use (occasionally). Just know that when, if ever, it is a good practice, is a matter of some debate ;)

I'm not an expert on this at all, but doesn't it just require a mindset shift that everything is controlled in the one single state? The state can contain nested objects, so you could manage the state of all your components in a nested object called 'components'. ie:
const state = {
  // other state stuff
 components: { componentA: { expand: true },
                           component B { expand: false }
  }
}

This nested object would then allow you to track the different states of each individual component.
Alternatively you could switch this round and put the focus on the state of being expanded or not instead:
const state = {
  // other state stuff
 expanded: { componentA: true,
                      component B: false
  }
}

Not sure if that helps or not...

@rssfrncs There is no local component state, but we have a feature usually known as slices, but better explained IMO as simply nested actions, that lets you modularize your state and actions within our single state tree architecture.
It's basically a way to break up your state and actions, so that actions can update a specific section of the state, instead of the whole state.
See: https://github.com/hyperapp/hyperapp#nested-actions

Seems like we can close here.


[BUG] Avatar broken on every home page after page 1

Describe the bug
The avatar image is broken and not displaying on pages after the first page in the home page.
Expected behavior
I expect the avatar image to load on all pages of the home page.
Screenshots
Here is the first page of my home page, with the avatar loading correctly:

Here is the second page, with the avatar broken:

Build Environment

OS: MacOs
Theme version/commit: I'm not sure how to find the theme version, maybe you can advise me on this, but it is 2.X.X.
Hugo version [e.g. 0.69.0]: hugo v0.82.0+extended darwin/amd64 BuildDate=unknown

Please try to use Hugo extended version before opening the issue.
Preview Environment

OS: [e.g. Windows 10, macOS, Ubuntu, iOS, Android] MacOS. Bug is visible on preview and live server.
Browser [e.g. Chrome, Safari, Firefox]: Brave and Safari

Additional Information
Configuration files or front matter code...

I do not have such a problem, so it is worth attaching either a minimal project illustrating the problem, or at least a configuration

I figured out the issue. I had the avatarURL as "images/headshot.jpeg", and once I added the leading backslash and changed the file path to "/images/headshot.jpeg" it fixed the issue. Thanks!

Yes, not at all, I'm glad you figured it out. You can contact with me any time


Out of chart cpu usage after update

HA core version
2024.9.2
Current integration version
0.5.9
Last known working integration version
0.5.7
Device type and model

Working mobile app

The description of problem
Hi,
I have a microcomputer used a server with a ryzen 5 5500U (6C/12T).
After the update to the version 0.5.8 and 0.5.9 the cpu usage for home assistant jump from 4% to 100% (due to python limitation, home assistant cannot use more than 1 cpu unit).
The rest of the home assistant instance became quite unstable (automation don't start or their execution got cut, unresponsive device etc).
I have the cprof and the out file from the profiler integration if you want it. They are quite eavy file to read, so I'm not sure if you want them. To sum up the important point of them, they reported over 35M call for 2 minute log to device:663:_check_heartbeat, using 49% of the total cpu time, and 729 calls to selector:445:select using 54% of the total cpu time. I don't know if these are from your integration or the are totally normal numbers.
The logs
No response

@Rihan9 could you double check with your installed version?  or just reinstall v0.5.9 and reboot your HA to confirm again
v0.5.9 should fix the high cpu usage issue, and v0.5.8 should have this high cpu usage issue.
thanks


add docs

Codecov Report

Merging #4 (04cdff4) into main (32d51a9) will increase coverage by 0.49%.
The diff coverage is 100.00%.

:exclamation: Current head 04cdff4 differs from pull request most recent head ccb4694. Consider uploading reports for the commit ccb4694 to get more accurate results

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   85.71%   86.20%   +0.49%
==========================================
  Files           3        3
  Lines          28       29       +1
==========================================
+ Hits           24       25       +1
  Misses          2        2
  Partials        2        2

Impacted Files
Coverage Δ

internal/dog/interact/service.go
100.00% <100.00%> (ø)

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 32d51a9...ccb4694. Read the comment docs.


story(global-styles): remove default scss mode and make shared default

Feature Description
We want to remove the default scss files and make the shared scss file the default/only variant.
Additionally we want to provide and use a global (s)css file for consumers and our storybook.
Definition of Done

[ ] Component is implemented
[ ] Tests are implemented/adapted
[ ] E2E Tests are implemented/adapted
[ ] Storybook stories are implemented/adapted

Closed by #1281


[Don't submit]: add test for SMP bug

Summary
[Don't submit]: add test for SMP bug
Impact
test only
Testing

This PR is labeled [Don't submit]. Do you want to mark it as Draft to avoid accidental merge?


LowercaseConstantsFixer - Fix case where fixer would incorrectly chan…

…ge class properties after the object operator.

Hi @srathbone ,
Thanks for this PR, looks like a good bug fix! Just wondering, is this bug also present on 2.12 that you know? If so it would be great to retarget this PR to that branch so we've the bug fix on the lowest support version of the project :)

Hi @SpacePossum,
New PR is open, can I close this PR?

FYI, you can change the target branch in the PR itself: https://help.github.com/articles/changing-the-base-branch-of-a-pull-request/

Closing as replaced by #4127.


[Bugfix] Save Pending Keyword Input

This PR is intended to try and address my suggestion here: https://github.com/AmruthPillai/Reactive-Resume/issues/1720
Preamble
For each of the dialog components that allow a Keyword to be input, including:

custom-section
interests
projects
skills

If an input was typed by the user, but Enter was not pressed, their input will be lost.
Disclaimer
If the suggested code changes are antithetical to the code style of this codebase, I am open to suggestions on how to better architect this suggestion, if it is a desired fix.
Before PR Code Changes

If the user went through the process of inputting a value into the Keyword field, I reckon they intend for it to be saved.
After PR Code Changes

Thank you very very much for taking the time to make the screen recordings, helps immensely. I'll get this merged onto the app ASAP! Thank you for your contribution! :)


[react-union] Consider use of react-loadable

Instead of
export default {
	path: 'hero',
	getComponents: (cb) => {
		loadWidget((module) => cb(module.default));
	},
};

we could do something like
import Loadable from 'react-loadable'
import Loading from './Loading'

export default {
	path: 'hero',
	loadable: Loadable({
		loader: () => import('./components/Root'),
		loading: Loading,
	}),
}

I find this solution cleaner, because now we needn't create the .widget.js files and the user can use their own Loading component easily. It is suggested here. What do you think? Can you see any issues with it?

I think both solutions are compatible... ".widget.js" feature is just out-of-the box functionality that user of react-union-scripts can use. But if he wanna use react-loadable library with import-syntax he surely can.

It surely is compatible, but you would need to wrap it in another function (which wouldn't be necessary – it would also clean up the code base a little bit, because we wouldn't need to care about loading the components).


Using Select2 and enum with large data sets

Hi!
I've read the docs, and from what I see, it is hard, if at all possible to create an editable field with a large remote dataset using Select2.
Here is my use case. I have a table with about 200000 locations from geonames.org. I need the user to select and assign location ID to an object property. The property is on integer type.
There are two problems. Problem one is validation (the ID needs to be a valid location ID). This is beyond the scope of this editor and, for that matter, JSON schema. Problem two is actually selecting that location ID using Select2.
I think using a remote data source with Select2 is a useful feature, and I'd like to help implement it. I think to solve my problem I need to do the following:

Force json-editor to display a select control, even though field type is integer. I'm not sure if it's possible right now.
Declare a data source url for Select2. Hopefully this will be a cleaner solution than to pass select2_options.ajax function.

What do you think? Looking for your soon reply!

This would be highly useful feature. Have you finished it?

Funny that, we're looking at trying to plug into Geonames as well.
@beowulfenator did you find a solution?

Nope, sorry.


inside  of Next.js, refresh the page on navigation to another route.

NextUI Version
2.2.9
Describe the bug
<Link href="/auth/login?user_type=jobseeker">
    <Button color="primary" type="button" radius="sm">
            Login
      </Button>
</Link>

So basically what i wanted to do was wrap the  component of Next UI inside the  component of the Next.js and when trying to click, the page would refresh and navigate to the respective route.

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
import Link from "next/link";
import { Button } from "@nextui-org/react";

function Component(){
<Link href="/home">
          <Button color="primary">
                   Home
            </Button>
 </Link>
}

export default Component;

Expected behavior
Since, Next.js provide seamless routing, i didn't expect page to refresh when wrapping Next UI Button inside Link of next.js
Screenshots or Videos
No response
Operating System Version
Windows
Browser
Chrome

Try using this instead.
<Button
    as={Link}
    href="/auth/login?user_type=jobseeker"
    color="primary"
    type="button"
    radius="sm"
>
    Login
</Button>

Did you solve this? Having the same issue

Try using this instead.
<Button
    as={Link}
    href="/auth/login?user_type=jobseeker"
    color="primary"
    type="button"
    radius="sm"
>
    Login
</Button>

Didnot work.

Did you solve this? Having the same issue

Not yet. What about you?

Did you solve this? Having the same issue

yeah try the above-mentioned solution by @CaseyBlackburn

Closing - see the solution here.

Ehh I'm necroing this, still an issue specifically for other elements that  you want to turn into buttons, for example:
export default function ArtistCard({ user }) {
    return (
        <Link
            href={user.username ? {
                pathname: "/[username]",
                query: { username: user.username, push: true },
            } : {
                pathname: "/user/[user_id]",
                query: { id: user.id, push: true },
            }}
            as={isExport() ? undefined : user.username ? `/${user.username}` : `/user/${user.id}`}
        >
            <Card as={Button} fullWidth className="px-2" onPress={() => { }}>
                <CardBody className="gap-2">

                        <UserAvatar user={user} size="lg" />

                                {user.full_name || user.username}

                                <MapPinIcon className="size-4 text-secondary" />

                                {user?.city}

                            {user?.shop && (

                                    <BuildingStorefrontIcon className="size-4 text-secondary" />

                                    {user?.shop}

                            )}

                            <CalendarIcon className="size-6" />

                                <AutoTranslate tKey="next_available">
                                    Next available:
                                </AutoTranslate>

                                4/20/24

                    {user?.style && (
                        <>
                            <Divider className="mt-1" />

                                    <AutoTranslate tKey="style">
                                        Style:
                                    </AutoTranslate>

                                    {user?.style}

                        </>
                    )}
                </CardBody>
            </Card>
        </Link>
    )
}

Andddd here is my solution for anyone who runs into this.
button-link.js
const ButtonLink = forwardRef(({ ...props }, ref) => {
    return (
        <Button ref={ref} as={Link} {...props} />
    )
})

export default ButtonLink

Hi, I think this problem is back but it's because of the way nextj doesn't allow to have a  as a child. Before the recommendation given above worked for me, but with the latest updates nextui it stopped working. Here is the button:
 <Button
        key="main-dashboard"
        as={NextLink}
        prefetch={false}
        href="/dashboard"
        className={classnames("rounded-sm bg-transparent data-[hover=true]:bg-zinc-200 text-sm", {
              "bg-green-700 text-zinc-50 data-[hover=true]:bg-green-700": activeMainLink,
     })}
      size="sm"
      variant="solid"
   >
     { "Main"}
 </Button>

As I said before it was working, but when upgrading nextui from 2.4 to 2.6,  nextjs ( 14.2.18) started complaining that I'm using a  as child.
nextjs version 14.2.18
nextui 2.6.4
this is the error message from log:
Error: Invalid <Link> with  child. Please remove  or use <Link legacyBehavior>.
Learn more: https://nextjs.org/docs/messages/invalid-new-link-with-extra-anchor

as can take a function that returns a component... for example
as={(props) => <NextLink legacyBehavior {...props} />}

Will that fix it?

So I'm not having any problems with this:
               <Button
                    as={NextLink}
                    href="/settings?tab=notifications"
                    variant="light"
                    onPress={() => setIsOpen(false)}
                >
                    Settings
                </Button>

            Only difference is I'm on Next 15

Hi @daveycodez , solved Button with Link using passHref props. But Links inside DropdownItem i cant solved with this

Yea I had to move all of my DropdownItem links over to onPress with router push while they are working on that, I made a separate issue for the Dropdown links

Thx for the tip, now using router push too.


Don't template when: conditions

I'm never going to figure this quoting out. I thought you had to quote
in ansible v2, but it doesn't want to work that way.
Signed-off-by: Jamie Lennox<EMAIL_ADDRESS>

Build failed.

http://logs.bonnyci.org/check_github/BonnyCI/hoist/259/1490658167.6/bonnyci-run-check-multinode : FAILURE in 1m 12s

recheck


feat: add support for custom server links

yes

oops, doesn't compile lol

Fixed in 94100db

That's why I did not add it, yeet :P
error[E0716]: temporary value dropped while borrowed
   --> pumpkin\src\world\mod.rs:256:39
    |
256 |                 links.push(Link::new(&Label::TextComponent(TextComponent::text_string(key.to_string())), value));
    |                 -----                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^         - temporary value is freed at the end of this statement
    |                 |                     |
    |                 |                     creates a temporary value which is freed while still in use
    |                 borrow later used here
    |
    = note: consider using a `let` binding to create a longer lived value

For more information about this error, try `rustc --explain E0716`.
error: could not compile `pumpkin` (bin "pumpkin") due to 1 previous error

I think you forgot to look at the note below it... Will add a let binding in a second in a second.


Support manually-defined WeMo devices

This is extremely useful if you want to support wemos that are on
another subnet or across a VPN. It also lets you sidestep the discovery
process, which is problematic for a lot of people and situations.
In order for this to work, we need to bump the pywemo requirement to
0.3.10, which includes my changes to make this possible.
WeMo devices can be manually configured by adding a static section to
the config, like this:
switch:
platform: wemo
static:
- <IP_ADDRESS>
- <IP_ADDRESS>

cc: @pavoni

Bueno :dolphin:

Sorry, ignore that linkage to the numeric-state patch. Too many tabs open :/

Doc update for this:
https://github.com/balloob/home-assistant.io/pull/207


MissingPluginException (MissingPluginException(No implementation found for method checkPermissionStatus on channel flutter.baseflow.com/permissions/methods))

💬 Questions and Help
For questions or help we recommend checking:

The Flutter tag in Stack Overflow

onTap: () async {
                                    List<Contact> contacts =
                                          await ContactsPlugin().getContacts();
                                      print(contacts);
                                    },

same with me


Visual artifacts when using sprites

Describe the bug
I was trying to make use of this library and noticed some visual artifacts when using my own sprites. I wasn't sure if it was a problem with my code, so I tried the sprite sheet example and modified the index to always be 12 and it shows for me. Haven't looked into whether this is caused by some floating point issue or something else yet.
To Reproduce
Modify the sprite sheet example to use index 12 for all sprites and run the example.
Expected behavior
I expect there to not be visual artifacts from the tiling.
Screenshots Or Code snippet

OK, I've read more about this problem and seems to be a more general problem of floating-point issues/texture bleeding. Going to close this out as it's non-actionable in this crate.


[Python] Table.from_struct_array function

Following up on this stack overflow question: https://stackoverflow.com/questions/74430949/table-from-nested-list-struct
Would it be worth introducing a Table.from_struct_array function?
I'm actually not 100% sure it is needed or if it make sense.
For example, I'm not sure how it should behave when receiving a ChunkedArray (of type StructType) vs receiving a StructArray. In which case it's behaviour feels a bit overloaded and maybe it belongs to the compute package.
Also it's already possible to do such a thing (as illustrated in the SO question). Maybe it's more a problem of documentation. An example like this could be added to the cookbook.

Reporter: &res / @0x26res
Note: This issue was originally created as ARROW-18330. Please see the migration documentation for further details.

Given that this already exists on RecordBatch, I think it makes sense to add it to Table as well.

I'm not sure how it should behave when receiving a ChunkedArray (of type StructType) vs receiving a StructArray.

In both cases it would return a Table. I would say it is actually natural to convert a Chunked StructArray to a Table, since each column in the table is chunked anyway.

Fixed by @judahrand


localdev: fix kind helm install shell function

the cilium-helm-install shell function must cd into the
install/kuberenetes directory before generating the templates.
Signed-off-by: Louis DeLosSantos<EMAIL_ADDRESS>

I am OK with these changes, but I wonder if they will break existing setups for other contributors who already accounted for these discrepancies?

contrib scrips to OK to skip CI.


GTS is broken

Repro steps:
❯ EMBER_CLI_PNPM=true npx ember-cli@beta addon gts-broke -b @embroider/addon-blueprint --typescript --pnpm

then,
cd gts-broke

create a gts-broke/src/test.gts file.
place the text:
import { on } from '@ember/modifier';

<template>
  <button {{on 'click' @foo}}></button>
</template>

and then save or run prettier.
the file contents will be removed.

Resolved by: https://github.com/embroider-build/addon-blueprint/pull/180

Also resolved by: https://github.com/embroider-build/addon-blueprint/pull/185

@mansona it was both gjs and gts

Anyone, the fix is released in 2.4.0


Fixes #830 Upload toast is hiding the "Upload All" button

Description
Now the toast message will appear at the centre of screen
Related issues and discussion
#830
Make sure you've done all the following (You can delete the checklist before submitting)

[x] Apply the AndroidStyle.xml style template to your code in Android Studio.
[x] Code is well documented
[x] Include unit tests for new functionality
[x] All user-visible strings are made translatable
[ ] Code passes Travis builds in your branch
[x] If you have multiple commits please combine them into one commit by squashing them.
[x] Read and understood the contribution guidelines .

Hi @Siddharth-Singhs, I still see some messages hiding the upload button; could you take a look?

Okay.


[Addition]: Calm Lemonade Jellyfish is spoofing the location from Germany to France with over 500km

Hotspot b58 Addresses
112CWKNVF8aBnMVVx5hoUPnTseKJEFvqWN7odUNj5MeYeczmwEyb
Discord Handle
xCarlitos #5639
Reason(s)
Calm Lemonade Jellyfish witnessing Miners in Germany especially in the location of Ludwigshafen, Mannheim and surrounding area. But the Miner seems to be placed in France. It is very suspicious since the Miner only witnesses Beacons from the mentioned area and nothing in France.

wahrscheinlich umgezogen, der macht doch keine großen umsätze...?

Then the location should be changed... The Miner seems to never have witnessed one in France. So probably it was never placed there physically.


Position no longer available

After updating to 1.9.0, the entity device_tracker.audi_a1_sportback_position is no longer available and i cannot get the position of my car.
This entity is no longer being provided by the audiconnect integration. If the entity is no longer in use, delete it in settings.

https://github.com/audiconnect/audi_connect_ha/issues/428

Can you provide any logs? Or do you have the same issues like in #428 ?

I guess it's the known bug. :)


DoomFist Visual Bug: Brackets around Slam Damage Counter - Useless Fix

Created by: DpsDoomMain
• Currently Doom’s Slam Damage Counter has no brackets around the numbers
• Previously Doom’s Slam Damage Counter had slanted brackets around the numbers

moved from ow1_emulator_backup#492


No such host is known

I'm trying to mine ZCASH with nheqminer version 05.C. This it the command I use:
nheqminer.exe -l stratum.zcash.nicehash.com:3357 -u (ZCASH ADDRESS) -t 3
And I keep getting same error: NO SUCH HOST IS KNOWN.
Where could be the problem? Thanks

i was having the same problem, try using an older version.

with nheqminer_v0.3a  i'm getting same error with all servers
stratum | Connecting to stratum server equihash.asia.zclassic.miningpoolhub.com:20575.nicehash.com:3357
[2018-02-04 18:34:56.327837][0x0000107c]: stratum | resolve: No such host is known
How to solve this...? please suggest
Thanks.


use product overrides in web

@sandy081 I wonder if I need to adopt @vscode-test/web as well
https://github.com/microsoft/vscode-test-web/blob/6d4145d916ca746f9af5fe1bc0f8c15da965d55f/views/workbench.html

@aeschli Adopted in vscode-test-web module.

@bpasero Thanks for the feedback. Sorry for not giving the context.
This is to help developers easily override product configuration without touching the product.json in sources. Existing methodology is a quite bit of hassle especially making sure the changes done to product.json locally for debugging/testing purposes are not checked in. Hence this new methodology - having a product.overrides.json in sources that is git ignored and is loaded only in dev - will help a ton for developers.
Regarding your feedback on changes, I liked your idea of using existing productConfiguration embedder API. Updated the PR accordingly. Please check


feat: ssr add more context

Summary
In some situations, ssr need some information from req, res.
So we need support server.ssr.unsafeContext.
we support more information put window._SSR_DATA
Related Links

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

!canary


[Autocomplete] How to submit data in functional tests

If I have a functional test using WebTestCase, I can submit a form from the test. This was working fine for my own home-grown autocomplete solution, but today I upgraded and replaced my antiquated solution with UX Autocomplete.
The first hurdle was changing the form field in the test from:
enquiry[contact]
to
enquiry[contact][autocomplete]
I understand that there is a subscriber which re-maps the child back on to the parent, so this makes sense.
However, when I then run my tests again, with valid ID values for entities that can be returned by the autocomplete, I get errors such as:
InvalidArgumentException: Input "enquiry[contact][autocomplete]" cannot take "1" as a value (possible values: "")
Clearly this is something to do with the form not knowing what values are permissible, but how do I get around this and allow the test to submit the form successfully without Javascript?
Thanks

Hi, this is for easy admin, but it should work here too. You need to disable validation for the autocomplete field before submitting the form.
$location = LocationFactory::createOne(['name' => 'My new location']);

$form = $crawler->filter('button[type="submit"][value="saveAndReturn"]')->form([], 'POST');
// @fixme without js autocomplete selects are empty, so we need to disable validation
$form['entity[location][autocomplete]']->disableValidation();
self::$client->submit($form, [
    'entity[location][autocomplete]' => $location->getId(),
]);

I'm curious is that works. But also, if that's what's needed, that's gross (this is not a comment about your code @1ed  - just the fact that you would NEED that code).
It's weird enough that we need to change from to add the [autocomplete], though that is likely unavoidable (but should probably be documented). But you should be able to, I would think, just put "1" into the field... as that's exactly what the frontend JS does. So I would have tried exactly what @benr77 did... and so I'm surprised it didn't work.
Btw, we actually DO have a test case in core that, I believe, does exactly what @benr77 is doing, and that works fine. Here it is: https://github.com/symfony/ux/blob/5315ca00c42c035d89a6db5b33114dfab896add5/src/Autocomplete/tests/Functional/AutocompleteFormRenderingTest.php#L56-L60
Cheers!

@weaverryan the validation done the the Crawler (which validates that the user input is supported by the HTML, or something like that), but $browser uses a different method (just creates a POST request). Why the [autocomplete] field is needed? I don't remember.

I can get it working by following the same method as used in the test that Ryan has pointed out. However, then I'm immediately falling foul of CSRF protection as we're not sending the token in the post() request.
This also means I can't use any of the default form field values, and have to specify all of them in the test. This is not ideal with a big form - I only want to specify the fields that the test is interested in.
I have managed to get it working by using the DOM to change the value of the placeholder option to the one I want to submit, and then submitting the form (which then automatically sends all the other default values).
->use(function(Browser $browser, Crawler $crawler) use ($newId)
{
    $crawler->filter('#id-of-select-field')->each(function (Crawler $crawler) use ($newId)
    {
        /** @var DOMElement $option */
        foreach ($crawler->children() as $option)
        {
            $option->setAttribute('value', $newId);
        }
    });
})
->click('Save Changes')

I guess you could also add a new <option> and give it the selected attribute, rather than change the placeholder option.
I have no idea if this is the best way of doing it but it seems to work.
Any thoughts?

BTW I'm using AJAX lookups for this - so all the options are not presented in the HTML on page load.

As far as I am aware this issue is not yet resolved.

I think the CSRF issue has been resolved in a recent release, but as for the general issue I think it's still unresolved.
Please keep the issue open for now.

@benr77 Check https://github.com/symfony/ux/issues/1334#issuecomment-1896659252 for a working example


license

Good Morning. Is it allowed to use deep zoom format on my imaging site? I have seen that this format has a proprietary license.

Please don't cross-post.
https://github.com/jcupitt/libvips/issues/597


79931 Log events

DevOps#79931
Add history events, the same as for Preservation. Added ObjectGuid to IPO

Is it always true that Constructor should add the AttachmentUploadedEvent?
@janid1967
I think so. Did you see any problems with this, or other times it should be added?

Is it always true that Constructor should add the AttachmentUploadedEvent?
@janid1967
I think so. Did you see any problems with this, or other times it should be added?


Merge "Exists" and "Must exist" policy rule validator ( #446 )

Signed-off-by: Guillaume Roques<EMAIL_ADDRESS>

Fix https://github.com/MediaArea/MediaConch_SourceCode/issues/446


Change the way get "index"

Codecov Report

Merging #13261 (58b6586) into master (1f32bdb) will decrease coverage by 26.55%.
The diff coverage is 0.00%.

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

@bzheng888 Thanks for contributing! Looks like our PR-EMAIL check failed
@alluxio-bot
PR-EMAIL — Email not linked to a valid Github account

can you help check if your account is linked with an email address?

@LuQQiu @apc999 Thank you for the help. I'll help @bzheng888 to resolve the issue about email.

@bzheng888 Thanks for contributing! Looks like our PR-EMAIL check failed
@alluxio-bot
PR-EMAIL — Email not linked to a valid Github account

can you help check if your account is linked with an email address?

alluxio-bot, merge this please


Store Octokit instance at org.api

Redo of #534, part of #482.

I'm stuck on mocking octokitWithRetries and getClient.test.ts.

Note to self: update not in one place in the test suite but at all the getClient calls.

Passing locally, not sure what's up, this error usually means something is drastically wrong like a syntax error or something. 🤔 =

Error: @jest/core tried to access pretty-format, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.
😳

Retry in 691ce4a51faee940342adc9951d5b318aa465c12.

Okay so not locally but in CI yarn/jest is choking on throw inside constructor.

Okay, I figured it out. The throw 'string' was masking the issue. throw Error('string'); does better. We're instantiating GitHubOrg in config/index.ts and there's no GH_USER_TOKEN in the test environment. Working as expected. :)
This stuff gets refactored (with tests) in 0ecf5e29122105546eda4cdd5f418109ee027586 on #541. Let me see if it makes sense to pull some of that up.


Remove version specification for public_suffix, add compat layer

I'm so sorry to do this to you, @blatyo, especially after reading your commit message here: https://github.com/trevorturk/page_rankr/commit/eea359c386e4ce9b7a76d0c18d96b405c5feceac
This is an attempt to solve the public_suffix versioning problem for good by:

removing the public_suffix  version specification entirely, and
adding a compatibility layer to accommodate public_suffix versions 1.x and 2.x

If you'd like, I'd be happy to join up as a maintainer. I don't think I'll be using the library much, but I'm happy to pitch in where I can. Thank you!

Thanks for making the effort to submit this. However, I've moved on from this project. If clearbit is interested in taking ownership of it, we can discuss that.

Gotcha -- thanks for the quick reply!


VersionBot: 'invalid username or password' intermittent error

Example: https://github.com/resin-io/resin-api/pull/308#issuecomment-311463051
Worked fine after adding and removing the label a little later (after an unrelated build hiccup), with no other changes. Nothing visible in the github status stats.

Seen this occasionally, since we started. I think it's a GHism, which for some reason fails the clone and then happily accepts it after. I'll add a retry into the mechanism for cloning, similar to how the GH Service itself works.


Implement MultiLevel (or aggregate) cache

A cache implementation that can for example have in-memory cache + redis distributed cache.
cc  @davidfowl @DamianEdwards @sebastienros

or just use http://cachemanager.net

I had success using redis pub/sub for synchronizing local memory cache with redis, but I'm afraid it adds too much network overhead if working with a large number of keys.

It may be a good idea to combine this one with CacheProfiles and add a way for user to specify what is the preferred location for cached data - either in local memory, or in Redis, or both since different data may have different usages and availability requirements. I've created such implementation for our scenarios and it had been used for some time with success.


Cross-model slugs

Hello,
Right now the package is designed to work with slugs withing single model. We are able to define FindSimilarSlugs scope and/or WithUniqueSlugConstraints on a model and it does the job of finding duplicated slugs.
But I currently can't find a way how to make my slugs unique across all models. So let's say we have an "Article" and a "Product" models (both sluggified). And we want the routing to be prefix-less.
In other word, if particular article is being sluggified with "foo", then it could be loaded in the browser at http://domain.ltd/foo. At the same time, if particular product is being sluggified with "bar", then it could be found at http://domain.ltd/bar.
This brings uniquness problem. If slugs for article and product will match, only one of them could be loaded/found.
To fix this, I would like to be able to define an FindSimilarSlugs or WithUniqueSlugConstraints scope  (or other mechanics) which could look into all sluggable-enabled models to find out if the new slug is already being used on some model.
Is it possible with current design?
Of cource, I can solve this by having an prefix for both models, so that first one will be "articles/foo" and second would be "products/foo", but I would really want to avoid this.

Off the top of my head, I can't see an easy way to look for all "sluggable" models.
What you might want to try is a new model that is is sluggable, instead of Articles and Products, and define a relation between that and your other models.
I'd have to fiddle for a bit to figure out the mechanics, and I'm on my phone, but i bet you could come up with something that worked that way.

On Oct 16, 2016, at 12:52 PM, Pavel Sheiman<EMAIL_ADDRESS>wrote:
Hello,
Right now the package is designed to work with slugs withing single model. We are able to define FindSimilarSlugs scope and/or WithUniqueSlugConstraints on a model and it does the job of finding duplicated slugs.
But I currently can't find a way how to make my slugs unique across all models. So let's say we have an "Article" and a "Product" models (both sluggified). And we want the routing to be prefix-less.
In other word, if particular article is being sluggified with "foo", then it could be loaded in the browser at http://domain.ltd/foo. At the same time, if particular product is being sluggified with "bar", then it could be found at http://domain.ltd/bar.
This brings uniquness problem. If slugs for article and product will match, only one of them could be loaded/found.
To fix this, I would like to be able to define an FindSimilarSlugs or WithUniqueSlugConstraints scope (or other mechanics) which could look into all sluggable-enabled models to find out if the new slug is already being used on some model.
Is it possible with current design?
Of cource, I can solve this by having an prefix for both models, so that first one will be "articles/foo" and second would be "products/foo", but I would really want to avoid this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

After investigating this further, I see 2 options available here.
Option A: Use Slug model and have relations to it from all business models.
Quick spec:

Create database which will store all slugs. Spec:

id (autoincrements)
model_id (polymorphic)
model_type (polymorphic)
slug (string).

Create Slug model for this table.
Slug model will have Sluggable trait.
Slug model will define sluggable() method which would load related (business) model and pull it's config.

Problems:

The logic becomes non-trivial, as we need to store config in business models, and pull it from Slug model.
We lose ability to transparently generate slugs on business models with an observer, because business models are not sluggable now (sluggable trait is used only on Slug model).
The getExistingSlugs method doesn't work with this approach, because the results returned by the query are keyed by primary key of the Slug model and not business model.
It is non-trivial to store all slugs in one table, because package supports multiple slug attributes per model, so we would need to support storing this in one table as well. This may be solved by additional attribute (string) column, but package code is not designed to pick up this change.

Option B: Store slugs in their respective models and override SlugService to handle cross-model slug uniquness.
Quick spec:

All business models are sluggable.
Slugs are stored in the respective business model's tables.
SlugService should allow override of getExistingSlugs so that developer may provide own implementation of it, which will look into all models to decide if slug is already used.
Optional: have centralized "read-only" table which will list all slugs and their models. This just to make querying slugs easier.

Problems:

Additional effort needed from developer, but I guess it is ok, as we are all geeks ;)
Right now SlugService is not bound to the container, making it impossible to override implementation.

For my project, I decided to go with Option B, as level of efforts is a lot lower. I will create an PR to fix problem where SlugService implementation is not overridable.

@qodeboy do you have this implemented yet? Would you be willing to provide your implementation?

I'm on vacation right now, so not in the mind-set to do anything code-related.  But if someone wanted to update @qodeboy 's MR #322 which binds the SlugService to the container, and resubmit ... that is likely the easiest way to get cross-model slugs working.


Build linux version

Hi, it's possible to compile warden for using under linux?

hello, sorry did see your comment
I will do it in February


"Open a pull request" feature

After pushing a branch, my current workflow is to then open a browser window, navigate to the Github repo, then click the "Compare & pull request" link when it pops up.
It looks like the URL is pretty simple:
https://github.com/ORGANISATION/REPO/compare/BRANCHNAME?expand=1
Maybe we should even automatically offer to do this when pushing a branch for the first time:

"No upstream branch set, want to set one?"
Yes, push...
"Want to open a pull request?"

I'm not sure I want to be asked this every time I push a branch to remote, as most of my time is spent outside of github.
I do however consider "open pull request" as a valid feature idea. My current flow is to either navigate via browser, or use git pull-request to open a PR on github from terminal.

Ok, good to know (and thanks for the link to hub - didn't know that existed!). Maybe a slightly different approach would be to have "Open pull request" start by pushing. (Would you ever not to want to push your latest changes before opening the pull request?) Then your flow would look something like

commit
"github: open pull request"
"No upstream defined, where do you want to push to?" > origin/myrepo
Pushing...
Pull request browser window opens.

I think that we should create GitHubSavvy where we can create all kind of views that is related to GitHub. Maybe a GitHub: pull request-interface, where we can open diff, write comments, etc ...
Also there are many user that don't use Github. There are many users that like GitLab, BitBucket or just a git server(no GUI).

Well, all the Github commands are already quite well segregated in the
interface and in the code - no reason there couldn't be GitLab, BitBucket
etc features also included.
Also, GitSavvy can tell by looking at the remotes which tools are
applicable - no need to propose a Github pull request for a remote which is
hosted on GitLab?
The problem with separating things out into separate plugins is
discoverability - most users will probably have no idea that there even is
a GithubSavvy, and won't think to search for one.

We can recommend installing one of these in messages/install.txt. This file is we be opened when someone install GitSavvy

Awhile ago, we were throwing around ideas for high-level GitHub integration - things like managing issues from Sublime, and opening and commenting on PRs - stuff like that.  After exploring the possibilities for awhile, I ultimately realized that I was trying to re-invent GitHub.  And unfortunately, it seemed that the UX would inevitably be worse.
The current strategy is to focus on the local dev experience, and let GitHub be GitHub (BitBucket and GitLab is still on the list to be added).  Asking "Does GitHub do this already?" can serve as a litmus test for whether an integration feature adds value or not.  Also, not everybody uses GitHub, so I've intentionally left those commands separate from the core git: ... commands.
As far as this ticket is concerned, here's what makes the most sense to me:

user opens command palette and types github: open pull request
if an upstream is not configured, prompt to configure
if the local branch is ahead of the upstream, prompt to push
open the browser

Yep, I'm pretty happy with the "launch a Github browser window" approach.
To spell out what about GitSavvy gives me the most value: reducing the friction of interacting with each project's Github repo. In particular:

doing everything I need to do from within my editor, using the keyboard
getting rich context about the current situation to increase confidence I'm doing the right thing
predictable keystroke combinations for standard tasks to minimise cognitive load
hints and prompts as to what I probably want to do next

So this approach meets most of those needs pretty well.
(And of course someone can still write a Github plugin that moves that support directly into Sublime if they want...)

https://github.com/temochka/sublime-text-2-github-tools has a similar feature. It should be great to implement something similar.

Alrighty, this seems like a good way forward.  @randy3k, feel free to open a ticket (or a PR) for the other feature.  I'd prefer the more explicit github: show history for current file.

Now, it is easy to create simple pull requests in Sublime Text in 4 steps.

git: checkout new branch for the pull request
git: quick stage and commit current file and type the commit
git: push and choose the upstream (assume that the remote is set)
github: create pull request and submit the PR via the browser

PS: There is a related feature that we are still missing: the ability to fork a repo, something similar to git fork.

@stevage, is the github: create pull request feature that @randy3k introduced good enough for your purposes?  Or would you like a little more polish somewhere before this ticket is closed out?

Oh! Sorry I didn't see this earlier. Just tested now and it works well.
I guess the only other useful "polish" would be a choice of where to PR to, but you can make those choices through the Github UI, so it's really fine as is. Thanks!


Update maintainers list

Update https://github.com/dapr/community/blob/master/MAINTAINERS.md due to

https://github.com/dapr/community/issues/516
https://github.com/dapr/community/issues/525
https://github.com/dapr/community/issues/518
Mauricio Salatino - Maintainer of the Dapr Shared project - See https://github.com/dapr/community/issues/474

Give access rights to Discord, 1Password, added to the CNCF mailing list and the CNCF Dapr Maintainers list along with updating https://github.com/dapr/community/blob/master/MAINTAINERS.md
Tasks

[x] Discord (@marcduiker)
[X] Remove maintainers from 1Password Maintainers Vault (@marcduiker )
[x] Invite @WhitWaldo and @salaboy to Dapr 1Password (@msfussell)
[x] Update role in Dapr org teams
[x] Update Maintainers doc on Google Drive (@marcduiker)
[x] Update Maintainers.md in this repo (@marcduiker)
[x] Update CNCF project-maintainers.csv (@marcduiker) https://github.com/cncf/foundation/pull/778
[x] Message CNCF about emails to add/remove for mailing lists (@marcduiker) https://cncfservicedesk.atlassian.net/servicedesk/customer/portal/1/CNCFSD-2255?created=true

@msfussell I can't update 1Password access, can you do that please?

@marcduiker - can we include https://github.com/dapr/community/issues/553 in this issue?

All completed


fix(transaction notification): fix that state access was off the mark

And make the test more robust against unexpected state.

Tested ACK bfd1383


Call out TABL tab name changes

Updated spoof to Spoofed Sender
Updated sender to domains and email addresses
As per recent TABL tab name changes as per UR

@chrisda or @MSFTTracyP could you please approve this for me?

Docs Build status updates of commit 6ce0af6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3624e56:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:warning:Warning

Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md

Line 12, Column 1: [Warning: yaml-duplicate-key] Key 'ms.date' is already defined, remove the duplicate key.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3d871d3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:warning:Warning

Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md

Line 12, Column 1: [Warning: yaml-duplicate-key] Key 'ms.date' is already defined, remove the duplicate key.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit e96f757:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:warning:Warning

Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md

Line 12, Column 1: [Warning: yaml-duplicate-key] Key 'ms.date' is already defined, remove the duplicate key.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3d97ab5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/manage-tenant-allow-block-list.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Move tx invocation inside TransactionMethod [ECR-4042]

Overview
Moved invocation of Transaction methods inside
TransactionMethod to achieve better encapsulation
of the MethodHandle.
Removed a class-merge artifact.

Review 15b0a49

See: https://jira.bf.local/browse/ECR-4042
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage decreased (-0.05%) to 86.267% when pulling 15b0a4952031040982eb5f6afae3614857b67dbd on dmitry-timofeev:refactor-tx-method-ECR-4042 into 0fa1979de32da5bcea112a05dc270618a44bc9ab on exonum:master.


Add Linux CPU load support

Linux CPU load support is retrieved from /proc/stat, parsed using the nom crate.
Disclaimer: I did my best to ensure that the BSD versions still compile, but I'm unable to test them.

Nice. Since you're adding nom, could you also rewrite the memory and battery info parsing to use it? :)

If you can give me an example of what the battery info files look like, then sure (I don't have a laptop to hand which I can check with)

My laptop runs FreeBSD :)
Anyway, looks like battery is just reading files that only contain one number https://github.com/myfreeweb/systemstat/blob/a8bc4bb45f0df494f67558f8039c636e39c37d65/src/platform/linux.rs#L113-L118
I was talking more about memory https://github.com/myfreeweb/systemstat/blob/a8bc4bb45f0df494f67558f8039c636e39c37d65/src/platform/linux.rs#L68-L70 it's currently done with find split_at trim trim_left_matches, I think nom might have better performance than that :)

Ah OK, single numbers are simple. And the more complex parsing of /proc/memstat is fairly easy to do in nom too.

Yeah I mean there's no need for nom for single numbers, only use it for memory

Memory parsing is tested and works

Thanks!


Drop older ruby 2.1/2.2 support

Fluentd v1 now works on ruby 2.1 or later but it has several drawbacks.

ruby 2.1/2.2 are dead version. ruby team doesn't maintain it.
Need dig_rb gem for dig method: See https://github.com/fluent/fluentd/pull/2372

Removing 2.1/2.2 is good for the further development. td-agent and docker images use ruby 2.3 or later so it doesn't break popular environment.
However removing support without the transition period may break user environment. So support 2.1/2.2 until the end of 2019 is better.

Thanks for opening this issue.
Maybe it's also better to discuss future support roadmap, I mean "when we drop support with Ruby2.3, 2.4, 2.5... for fluentd in the future" policy. Not fixed one but a rough consensus is good, IMO.

Maybe it's also better to discuss future support roadmap, I mean "when we drop support with Ruby2.3, 2.4, 2.5... for fluentd in the future"

One year support after EOL is good for me but several popular distributions use EOL version with own patches. So fixed drop timeline is not good for community.

Of course, we should care about distributions, and we can predict distros EOL thus suggest "plan" to the community, right?

distro
ruby version
date

Ubuntu 14.04 LTS
ruby 1.9.1
EOL in this April

RHEL6 (CentOS6)
ruby 1.8.7
November 2020

Ubuntu 16.04 LTS
ruby 2.3.1
April 2021

Debian9 LTS
ruby 2.3.3
June 2022

Ubuntu 18.04 LTS
ruby 2.5.1
April 2023

Debian10 LTS
ruby 2.5.5
May? 2024

RHEL7 (CentOS7)
ruby 2.0.0
June 2024

RHEL8 (CentOS8)
?
? 2029

So, I suggest you

drop ruby2.1 and 2.2 support in this summer (2019)
drop ruby2.3 and 2.4 support in July 2022
drop ruby2.5 support - no plan ;)

note: distribution EOL: Ubuntu / Debian /  RHEL /  CentOS

If current release supports 2.1 and so on, why can’t existing users continue to use it? Why do you need to support forward compatibility for LTS? Surely it’s enough to have a stable branch which backport security issues etc. LTS Indicated some desire for stability so do user want to use latest release or stick with known good version?

updated: https://www.fluentd.org/blog/drop-schedule-announcement-in-2019

Thanks for the update. I will probably remove 2.3 support too.

Nice work!


Remove dictionary_keyword table

keyword_dictionary is no longer used, dropping this table.

For some reason I had a vaguely impression that this table is later used by preset feature?

For some reason I had a vaguely impression that this table is later used by preset feature?

Let's confirm in tomorrow's meeting.


Correct manufacturer link (previous was to GL-SD-001)

Correct manufacturer link for
Gledopto Switch Module | GL-SD-002
Previous one directed to page of GL-SD-001.

thanks a lot


STRM-662: Added getGroups methods for KV

Should be added a new method that uses a new endpoint GET API [/_fabric/{fabric}/_api/kv/{collection}/groups](https://akashj-dxb.eng.macrometa.io/api#/Key%20Value/get__fabric__fabric___api_kv__collection__groups) Get list of groups in collection
The previous part of the GroupId feature was implemented in PR https://github.com/Macrometacorp/c84j/pull/122

@dlazarenko-macrometa - Please put the correct year into the headers.

@StoyanTc - Unfortunately, I cannot find classes that I need to fix a year. All new classes with 2023


A sugestion: could we name files as ${filename}-${date}, or even  ${filename}-${counter}

just in the title

@ndharari The latest release supports ${filename} in image name. Have a try.

its not working for me. i'm using as setting ${filename}-${date} and It does not let me paste any image

working perfectly!


[Clarification request / cross-posted] How to use OTP-Authenticator with mod_authn_otp

see https://github.com/archiecobbs/mod-authn-otp/issues/34
I want to use my OTP Authenticator with a server running https://github.com/archiecobbs/mod-authn-otp . Please can you explain (in detail) the needed steps, OTP Authenticator is already installed, and also https://github.com/archiecobbs/mod-authn-otp .

The undelying problem is that mod_authn_otp does not offer a module to generate the shared secret. In the meantime I managed to write a generator.
Code will be published separately. otp-authenticator as such works with it, when a QR of the secret is generated. https://github.com/kckrinke/onetimepad is a fork of https://github.com/0xbb/otp-authenticator which - as an alternative - allows to enter the secret via the keyboard.


fix(report): coverage updates for xsl:merge and descendants

Fixes #1959.
Note: In the expected coverage report, line 26 has a CSS class of "unknown". That's correct, though it won't render distinctively because there is no style for that class yet in this repo's master branch. #1974 adds a CSS style for the "unknown" class.

Cc: @birdya22

As per your changes to the test xsl files in #1974, can you add a comment on line 26 of xsl-merge-01.xsl as follows <!-- Expected unknown-->.

I downloaded the coverage-compute-status.xsl file to run it and look at the output. Based on what I see for #1974 and #1975 I assume that both sets of changes will be merged together when you commit the changes  both sets of changes (#1975 does not contain the changes from #1974).

As per your changes to the test xsl files in #1974, can you add a comment on line 26 of xsl-merge-01.xsl as follows <!-- Expected unknown-->.

Yes: b90542d2d2d82b9872e8191c4a2508f7d67c060f . Thanks for catching that.

I downloaded the coverage-compute-status.xsl file to run it and look at the output. Based on what I see for #1974 and #1975 I assume that both sets of changes will be merged together when you commit the changes both sets of changes (#1975 does not contain the changes from #1974).

Yes. If #1974 is ready to be merged first, then as soon as I merge that pull request, this pull request will become outdated. To bring it up to date, I'll push a "merge commit" to this pull request, which will then contain all the changes from #1974. On the other hand, suppose this pull request is ready first and we can tolerate the 'unknown' class not rendering distinctively yet. In that case, as soon as I merge this pull request, #1974 will become outdated and need to be brought up to date by a merge commit that brings in all the changes from this pull request.
(In case you are interested: Instead of starting this pull request from the master branch, I could have started from the 1974 changes. Had I done that, this pull request would list all the commits from #1974 in addition to the commits that are distinctive for this pull request. Your download would have also included changes from both pull requests. A down side of that approach is that if #1974 gets held up, e.g., due to finding a bug or having a long discussion, then this pull request would have to either wait or be reconstructed to be independent.)

as soon as I merge that pull request, this pull request will become outdated. To bring it up to date, I'll push a "merge commit" to this pull request, which will then contain all the changes from #1974.

FYI: This pull request now contains all the changes from #1974. So does the master branch.


Explore other scan aggregation methods

Right now the only method implemented is using a colorized octomap_server. However, there are other methods that could be explored.

[ ] Texture mapping - Save all images in scan to a directory and then stitch them back together at the end.
[ ] Custom octomap server that integrates detections more robustly. The octomap server colorizer just averages the current voxel with the incoming color to get the new color.

See PCL Texture Mapping example for one method


added multiplayer options upto 8

It doesn't work.

If after this, I blow up the first red, then the next turn should be of blue one, but it shows orange one.

@ketanhwr I have made the changes and its running fine. Please have a look. Thanks.


Replaces $ variable name with lock

Warns at level 4, seemed nicer to follow spscqueue.h

-Wdollar-in-identifier-extension


Clear addresses from Mail (Personalization) object

I have a scenario where I would like to re-use the a Mail object but change the subject and to address.
I only see methods for adding to already set emails. I do not see any methods for replacing or clearing the already set email addresses.
It would be nice if there was a way to modify the addressees so that a Mail object could be reused.

Hello @a61829437,
If you are comfortable, could you please share your use case? Perhaps there is a work around.
Also, this issue has been added to our internal backlog to be prioritized. Pull requests and +1s on the issue summary will help it move up the backlog.
Thanks!
With best regards,
Elmer

In my case I need to send the same email with same content and same attachment to multiple users but with a modified subject per user.
I was wanting to create a Mail object I could re-use with set content and attachment. Then loop a list of users and set unique subject per user and send to that user.

Are you using dynamic templates for this? And can you share sample code of what you're trying to accomplish? There may be a solution already.

Not using templates.
Something along the lines of
$mail = new \SendGrid\Mail\Mail();
$mail->setFrom(...)
$mail->addContent(...)
$mail->addAttachment(...)
...
...
...
foreach( $users as $user )
{
    //  clear previous "to" email in mail object...

    $mail->addTo( $user->email )
    $mail->setSubject( $this->getSubject( $user ) );

    $sendgrid = new \SendGrid(...api key...);
    $sendgrid->send( $this->mail )
}

Thanks.
This way puts all the to email addresses in the to field for each recipient though. I am needing to send individual email to a user without them knowing about the other users or seeing other email addresses it was sent to. I don't  think I clarified that part.
Users receive an individual email (unaware other users it was sent to) with personalized subject, but the content and attachments are the same for all emails.

Yup, I understood the use case; just the code didn't do exactly what I expected (I don't really like that way it's set up and will probably fix it all later).  This should work though:
$mail = new \SendGrid\Mail\Mail();
$mail->setFrom(...)
$mail->addContent(...)
$mail->addAttachment(...)
...
...
...
foreach( $users as $user )
{
    // $mail->addTo(new To($user->email, null, null, $this->getSubject($user)), null, null, null, new Personalization());
    // or easier to read:
    $personalization = new Personalization();
    $personalization->addTo($user->email);
    $personalization->setSubject($this->getSubject($user));
    $mail->addPersonalization($personalization);
}

$sendgrid = new \SendGrid(...api key...);
$sendgrid->send($this->mail)

Thanks, that works. The first one  $mail->addTo( $user->email .... still had all recipients in the to field of the email. But the second way with a To object worked:
foreach( $users as $user )
{
    $to = new To( $user->email );
    $personalization = new Personalization();
    $personalization->addTo( $to );
    $mail->addPersonalization( $personalization );
}

$sendgrid->send( $mail )

I didn't read the API documentation fully before using this library so missed part on personalizations...
I think this seems like a better solution than my thought of clearing the to addresses from the Mail object.
Maybe it would work to just add an example of this to the use cases https://github.com/sendgrid/sendgrid-php/blob/main/USE_CASES.md ?

After reviewing the code this appears to really be a bug in that 'to' addresses with subjects are not placed in separate personalizations. The example linked above would work but only when the 'tos' had substitutions as well.
I've submitted #1001 to fix this.


JFXSnackbar appears again after calling JFXSnackbar.close()

Hello,
I noticed that, after calling close on a snack bar, it appears again immediately, and closes after timeout.
I assume it's a non intended behaviour?
Kind regards

Hello,
Thanks for pointing out the issue 👍 , sure it's no an intended behavior.
Regards,

If you want to fix this bug: save the animation created in lines 223 and 284 (JFXSnackbar.java) in a private variable (e.g. private Animation animation_save). Then call animation_save.stop(); in method close (line 283) before doing anything else.


add into std::io::Error for EspIOError

Why is this necessary?

Why is this necessary?

EspIOError implements the embedded_io::Error trait, and EspIOError also implements std::error::Error in std, so I think it is also reasonable to implement Intostd::io::Error for EspIOError.

Why is this necessary?

EspIOError implements the embedded_io::Error trait, and EspIOError also implements std::error::Error in std, so I think it is also reasonable to implement Into <std::io::Error> for EspIOError.

Why is this necessary?

EspIOError implements the embedded_io::Error trait, and EspIOError also implements std::error::Error in std, so I think it is also reasonable to implement Into <std::io::Error> for EspIOError.

embedded_io::Error and std::error::Error are traits, while std::io::Error is a struct, so I'm not sure I see the analogy here.
Also, if we go this route, why don't we implement From<EspError> for std::io::Error as well?
Isn't here the general wisdom that you should create your own error enums, e.g.
enum MyError {
    StdIo(std::io::Error),
    EspIo(EspIOError),
}

... either manually, or with the thiserror crate?

Why is this necessary?

EspIOError implements the embedded_io::Error trait, and EspIOError also implements std::error::Error in std, so I think it is also reasonable to implement Into <std::io::Error> for EspIOError.

Why is this necessary?

EspIOError implements the embedded_io::Error trait, and EspIOError also implements std::error::Error in std, so I think it is also reasonable to implement Into <std::io::Error> for EspIOError.

embedded_io::Error and std::error::Error are traits, while std::io::Error is a struct, so I'm not sure I see the analogy here. Also, if we go this route, why don't we implement From<EspError> for std::io::Error as well?
Isn't here the general wisdom that you should create your own error enums, e.g.
enum MyError {
    StdIo(std::io::Error),
    EspIo(EspIOError),
}

... either manually, or with the thiserror crate?

In fact, I want to write adapters to adapt the request and response of the http server in esp-idf-svc to std::io::Read and std::io::Write. I write wrapper as follows:
struct StdReadWrapper<'a, R>
where
    R: embedded_io::Read,
{
    inner: &'a mut R,
}

impl<'a, R, E> std::io::Read for StdReadWrapper<'a, R>
where
    R: embedded_io::Read<Error = E>,
    E: Into<std::io::Error>,
{
    fn read(&mut self, buf: &mut [u8]) -> std::io::Result<usize> {
        self.inner.read(buf).map_err(|e| e.into())
    }
}

struct StdWriteWrapper<'a, W>
where
    W: embedded_io::Write,
{
    inner: &'a mut W,
}

impl<'a, W, E> std::io::Write for StdWriteWrapper<'a, W>
where
    W: embedded_io::Write<Error = E>,
    E: Into<std::io::Error>,
{
    fn write(&mut self, buf: &[u8]) -> std::io::Result<usize> {
        self.inner.write(buf).map_err(|e| e.into())
    }

    fn flush(&mut self) -> std::io::Result<()> {
        self.inner.flush().map_err(|e| e.into())
    }
}

in such case E is EspIOError

This would work, does not require From<EspIOError> for std::io::Error and would work for ANY embedded_io::Read trait, not just for traits which happen to implement E: Into<std::io::Error> that you are trying to artificially impose on the trait impl:
impl<'a, R, E> std::io::Read for StdReadWrapper<'a, R>
where
    R: embedded_io::Read<Error = E>,
{
    fn read(&mut self, buf: &mut [u8]) -> std::io::Result<usize> {
        self.inner.read(buf).map_err(|e| std::io::ErrorKind::from(e.kind()).into())
    }
}

... and is already implemented so you don't have to do it.

Thanks, you are right


Group cache

@emanuelschuetze I added a lot of logic to make OpenSlides faster. Please test if everything works before we merge this. You should look that users with restricted permissions can not see the content and you should test this with the redis cache and the normal cache.

Ok, please rebase.

@normanjaeckel Please one last review of the last commit. Then I rebase and wait for @emanuelschuetze

Ok, rebase.

done

@emanuelschuetze ping

@ostcar Sorry for my late reply. I'll test it. Do you have some specific instructions what I should test? Something more than "test if everything works"... ;-)

@emanuelschuetze I added a lot of logic to make OpenSlides faster. Please test if everything works
before we merge this. You should look that users with restricted permissions can not see the content and
you should test this with the redis cache and the normal cache.

Should I test it with users with restricted permissions only?

@emanuelschuetze I added the permissions to the cache and created a function has_perms() (as replacement for the method user.has_perms()) that uses the cached permissions. I think, that everything should work. And there is no "special" case in one app. So if it works for motions, it should also work for other apps.
So what you could test is, to create users with different permissions for one app and see if it works. As I told before, I think that it should work. So of cause, we can merge this right know and let any master user test it :)

@emanuelschuetze done.
@normanjaeckel please see the two new commits

Ok, @ostcar please rebase.

done


Android - Custom Marker component only renders partially sometimes

Summary
We're using custom markers to render circles on the map.
The issue only seems to happen on android.
We are changing the array of markers based on user input, and sometimes only a quarter of the circle gets rendered.
Here's a screenshot of the issue:

It's supposed to be a full circle:

I haven't been able to pin-point the exact scenario when it happens, it seems to be completely random.
Reproducible sample code
//circles are rendered like this:
//currentData changes when the user taps on a circle - this causes a fetch, returning new markers

const markers = useMemo(() => {
    return currentData?.map((marker, index) => {
      const smallestUserCount = Math.min(
        ...(currentData?.map(m => m.uniqueListeners ?? 1) ?? []),
      );
      const largestUserCount = Math.max(
        ...(currentData?.map(m => m.uniqueListeners ?? 1) ?? []),
      );

      const opacities = [0.4, 0.6, 0.8];

      //map marker.uniqueListeners to a percentage based on the smallest and largest user count, then map that percentage to a range of opacities
      const opacity = (userCount: number) => {
        const percentage =
          (userCount - smallestUserCount) /
          (largestUserCount - smallestUserCount);
        const opacityIndex = Math.floor(percentage * (opacities.length - 1));
        const opacityEntry = opacities[opacityIndex];
        if (opacityEntry === undefined) return opacities[2];
        return opacityEntry;
      };
      return (
        <Circle
          key={marker.shortName}
          marker={marker}
          opacity={opacity(marker.uniqueListeners ?? 1)}
        />
      );
    });
  }, [currentData]);

//the map renders them like:
return (
    <>
      <MapView
        provider="google"
        maxZoomLevel={ZOOM_LEVELS[level]}
        zoomEnabled={false}
        ref={mapRef}
        style={{
          flex: 1,
          justifyContent: 'flex-end',
          alignItems: 'center',
        }}>
        {markers}
      </MapView>
    </>
  );

Steps to reproduce
It's not clear how to reproduce - it doesn't happen very frequently.
Expected result
The markers should be displaying properly
Actual result
The markers are sometimes rendered incorrectly (only the top-left quarter of the circle gets drawn)
React Native Maps Version
1.3.2
What platforms are you seeing the problem on?
Android
React Native Version
0.69.4
What version of Expo are you using?
Not using Expo
Device(s)
Pixel 3
Additional information
No response

same issue for us

This happened to me as well and I fixed it by setting width and height for the Marker element. So something like this:
<Marker
   style={{ width: 100px, height: 100px }}
   {...otherProps}
>
   <MyCustomMarker />
</Marker>

Basically it seems that on Android the marker has some predefined size and everything outside is cut off.

Thanks for the suggestion, for us this did not fix it.
However - for anyone else who might read this - after some testing, I figured out that this happens when I try to change the styling of an existing marker on the map.
In our use case, we are rendering different 'levels' of markers on the map, the user can click on a marker to go one level deeper. Sometimes, on the next level, the marker is in the same location as the previous level, so using the lat/long as keys for the marker, the same marker gets re-rendered with new styling, and this causes it to break.
My fix was to figure out a unique key for every scenario, meaning we're rendering new markers rather than editing the same one.

We have an application with a user journey which causes this issue reliably.
We experimented with a bunch of things:

Changing tracksViewChanges after a delay
Forcing tracksViewChanges to always be on
Calling the redraw function on the marker ref in a setTimeout
Using an SVG image instead of the native <Image /> component in our marker

None of these worked.
What we instead found was that there is a hard coded value in the createDrawable function here which is being used instead of the correct size for the rendered custom marker.
  private Bitmap createDrawable() {
    int width = this.width <= 0 ? 100 : this.width;
    int height = this.height <= 0 ? 100 : this.height;
    this.buildDrawingCache();

    // Do not create the doublebuffer-bitmap each time. reuse it to save memory.
    Bitmap bitmap = mLastBitmapCreated;

We found that if we changed the 100 to a larger value then the issue went away.


Next.js 14 compatibility

Next.js 14 was released today, and this module does not install with it:
Found<EMAIL_ADDRESS>ERR! node_modules/next
ERR<EMAIL_ADDRESS>from the root project
ERR!
ERR! Could not resolve dependency:
ERR! peer<EMAIL_ADDRESS>|| ^12.0.0 || ^13.0.0" from<EMAIL_ADDRESS>
Could you check it out @BitPatty? 🙏🏻

Thank you for the quick support @BitPatty! It is working flawlessly with 0.12.0-rc1 🎉

Thanks for your feedback :)


Add a setAlarmMode(newValue) method to allow setting the alarm mode f…

…rom CoRE.

Check out this stackoverflow
http://stackoverflow.com/questions/18149102/groovy-method-with-optional-parameters
looks like you can use
def setAlarmMode(def newValue = null)
to DRY this up a little bit.
I'm not 100%, but I'd be willing to give this a shot.

Thanks! I'm a Java guy, so Groovy is only mostly familiar to me. Let me try to revise it.


Lazyload event isn't broadcasted

Current code below. I do have logs from init, but do not form the lazyload-related events.
Slideshow works correctly, though I need to know where the first image is loaded, to retrieve the real slideshow height.
var slideshow = $('.slideshow');

slideshow.on('init', function(){
  console.log('init'); // TEST LOG
});

slideshow.on('lazyLoaded', function(event, slick, image, imageSource){
  console.log('loaded', image, slider_ready); // TEST LOG
});

slideshow.on('lazyLoadError', function(event, slick){
  console.log('error'); // TEST LOG
});

var slideshow_settings = {
  autoplay: true
  ,autoplaySpeed: 5000
  ,appendArrows: '.header-slideshow-box'
  ,focusOnSelect: true
  ,pauseOnFocus: true
  ,responsive: [
    { breakpoint: breakpoints.large, settings: "unslick" } // breakpoints < defaults.js
  ]
}

function start_slick(){
  slideshow.slick(slideshow_settings);
}

start_slick();

Ok, no, I didn't use data-lazy img attr.


allow to display Elissa from any view, make icon optional

Let's make sure we create a new version in the trunk before merging ;)

In my use case I have to have multiple visible Elissas at the same time. So Elissa can't be a Singleton and multiple showElissa calls will create multiple Elissa instances.
That's also the reason why func showElissa does not have a discardable result. Here you have to keep the Elissa instance to be able to dismiss it later.
Maybe we have to add some documentation to make it clear that TabBar Elissas are Singletons and UIView Elissas are not?

Maybe also update the Example project?

Ok, I understand why a Singleton is not an option. Just wanted to make sure you're not surprised when multiple instances will be displayed in the same position on screen and tapping one won't remove all of them from their superview.
Updating the example project/documentation sounds like a good idea. Maybe as long as it is not stable you can go with
pod 'Elissa', :git => "https://github.com/KitchenStories/Elissa", :branch =>  "3.0"


error rendering PNGs in parralel for dash

Faced the same issue.
Grafana v6
grafana reporter latest
Error output:
error rendering PNGs in parralel for dash {Title:Ceph - Cluster Description:Ceph cluster overview VariableValues: Rows:[] Panels:[{Id:21 Type:singlestat Title:Health Status} {Id:43 Type:vonage-status-panel Title:OSDs} {Id:47 Type:singlestat Title:Capacity used} {Id:53 Type:graph Title:PG States} {Id:66 Type:graph Title:OSD Latencies} {Id:41 Type:vonage-status-panel Title:Monitors} {Id:68 Type:vonage-status-panel Title:Client connections} {Id:45 Type:graph Title:Cluster I/O} {Id:62 Type:graph Title:In-/Egress} {Id:55 Type:heatmap Title:OSD Capacity Utilization} {Id:59 Type:heatmap Title:PGs per OSD} {Id:64 Type:graph Title:Recovery Rate}]}: error getting panel {Id:43 Type:vonage-status-panel Title:OSDs}: Error obtaining render: 500 Internal Server Error

Does it work if you remove that panel from the dashboard?
Id:43 Type:vonage-status-panel Title:OSDs

yes,
had to remove all of the vonage status panels.

Could you render those panels using the Grafana UI "Share > Direct link rendered image" feature. Often the third party panels don't support that correctly.
On Wednesday, July 3, 2019, 11:16:35 AM GMT+2, Harish<EMAIL_ADDRESS>wrote:
yes,
had to remove all of the vonage status panels.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

This error is more on Grafana backend side, you should see the log.
I am getting dozen of them, what could be good is to do the report even with such error and work in best effort


fix: Optimize unconstrained array sets by fixing starting ref counts to be at 1

Description
Problem*
This is an alternate fix for https://github.com/noir-lang/noir/pull/6577. Where that PR re-aligns ref counts by starting them at zero so that the initial inc_rc increments them up to 1, this PR removes the inc_rc in a few cases known to be correct.
Making this PR since https://github.com/noir-lang/noir/pull/6577 is running into issues with DIE removing the initial inc_rc leading to an invalid starting RC of zero.
Summary*
Starts reference counts at 1 instead of 2 for array literals. Instead of changing brillig to start ref counts at zero, avoid issuing the initial inc_rc instruction if the rhs of a let binding is known to be an array literal (and should thus be moved, not shared)
Additional Context
Since this PR also lets us mutate arrays directly more often, it is also susceptible to https://github.com/noir-lang/noir/issues/6680 like #6577 is. That's where the CI failures are currently coming from.
To fix the above, we no longer deduplicate arrats willy-nilly in brillig. We:

Only deduplicate MakeArray instructions which have not been stored to or used in an ArraySet
When we deduplicate we insert an inc_rc to that array to increase the reference count. Although there should be no mutations before that point, there may still be some after so it needs to see that now the ref count is higher.

Additionally, I've added a new optimization: we don't issue inc_rc instructions for the first mutable array parameter of each type (but we do for each subsequent one). This means if you have fn foo(a: &mut [Field; 3]), a is considered mutably borrowed and we don't need a inc_rc since you can only access it through a anyway. If you add another &mut [Field; 3] parameter (with any number of nested references to it), then we issue an inc_rc for the next instance so that the rc is 2+. In practice this leads to pretty good improvements with any function that mutates arrays. You can see this in our opcodes executed even though most arrays in our tests tend to be small, we still copy them considerably less now.
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This PR should be ready to merge now. It has decent opcodes executed improvements with the optimization I mentioned (reference_counts is only larger because I expanded the test).
This PR may or may not require https://github.com/noir-lang/noir/pull/6700 - but either way the expected rcs in the reference_counts test will need to be updated once that PR is merged.


cloud_firestore: Unsupported value: FIRDocumentReference

Similar to #12468 the cloud_firestore package crashes the app if you get a document with a value of type 'Reference'.
Steps to Reproduce
Add cloud_firestore: ^0.0.3 as a dependency in pubspec.yaml and try to grab a document containing a value of type 'Reference'
Logs
Unsupported value: <FIRDocumentReference: 0x600000823fa0> of type FIRDocumentReference
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000109bc81cb __exceptionPreprocess + 171
	1   libobjc.A.dylib                     0x00000001090dcf41 objc_exception_throw + 48
	2   CoreFoundation                      0x0000000109bcd362 +[NSException raise:format:arguments:] + 98
	3   Foundation                          0x0000000108b81089 -[NSAssertionHandler handleFailureInMethod:object:file:lineNumber:description:] + 193
	4   Flutter                             0x0000000106d8af31 -[FlutterStandardWriter writeValue:] + 2108
	5   Flutter                             0x0000000106d8ae57 -[FlutterStandardWriter writeValue:] + 1890
	6   Flutter                             0x0000000106d8ace2 -[FlutterStandardWriter writeValue:] + 1517
Lost connection to device.
	7   Flutter                             0x0000000106d8ae57 -[FlutterStandardWriter writeValue:] + 1890
	8   Flutter    <…>

Can be closed, via flutter/plugins#429


Short urls must redirect to canonical url

Each project by default have these urls:

slug.readthedocs.org
slug.rtfd.org

When I go to slug.rtfd.org I've got redirected to slug.readthedocs.org and that's correct. However if I've specified project's canonical url to point to something else then slug.readthedocs.org then this isn't taken into consideration during redirects.
I think that both above mentioned urls must redirect to specified canonical url (if user have specified one of 2 shorts urls as canonical ones, then don't hit the redirect loop).

Closing this because I don't believe we will ever default redirect top-level short domains.