Newbie question: How can I consolidate like items (some from third-party apps) into their own submenu?

Just discovered Shell and find it to be incredibly flexible, albeit with a learning curve. Have tried other context menu extensions in the past but Shell seems to be the most customizable of all.
This may seem a newbie question, but: I have applications installed that leave their own various commands in various places throughout the context menu. What I am wondering is how I might consolidate like items from third-party programs (i.e. not Windows) into their own submenu? For instance below, how might I incorporate "Upload to Icedrive" and "Upload to MEGA" into a submenu "Upload to Cloud"? How might I consolidate the various PDF extensions into their own submenu? How might I consolidate the various compression programs into a submenu "Archive Options"? And is there a way to include these submenus as sub-submenus under Shell's own "More Options"?

Programs like ShellMenuEx from the similarly-named NirSoft (I have both, and they are tucked neatly next to each other in the Program Files folder!) allow for picking up the ID of these items in the Registry, and enabling or disabling them, but not to consolidate them into their own subcategory, which I am hopeful that Shell has the potential or capability to do.
Also, if there is a way I can include the standard Cut/Copy/Paste along with third-party items like "TeraCopy," "Unstoppable Copy," and "Bulk Rename" in Shell's own File Manage submenu, that would clean up the context menu quite a bit too.

first create an empty menu:
menu(mode='multiple' title='Upload to Cloud' image=\uE11F)
https://nilesoft.org/docs/configuration/new-items
https://nilesoft.org/gallery/glyphs
move item(s) in the menu:
modify(find='Upload to Icedrive|Upload to MEGA' menu='Upload to Cloud')
https://nilesoft.org/docs/configuration/modify-items
mare help from users: https://discord.com/channels/1106387012707168318/


Feature/infinite scroll both ways, dynamically renders pages on the fly

@atawmicpm, i think this is a pretty good advancement for the project. thanks for the effort.
also, please take a look at https://github.com/emberjs/ember-collection.
... this project is the Glimmer-friendly list view (ancestor of Ember.ListView) that we probably want to be sure we're adopting optimizations from. it's a good project to model in terms of properly triggering node rerendering:
https://github.com/emberjs/ember-collection/blob/master/addon/components/ember-collection.js

@mysterlune you're welcome and thank you very much.  I will take a look at ember-collection...

@mysterlune check out the latest change I made to pdf-page.js.  It now creates an svg out of the PDF instead.  The text selection / highlighting is better, albeit not perfect, and the text is incredibly crisp.  There are some glitches when it comes to embedded images, but overall a much better seeming starting place.

hey @atawmicpm, thanks for the update.
not sure why the direction toward svg in this case. what is the reasoning for changing to that element? does the PDFJS.SVGGraphics somehow add the vector and the text layer in one context?
... also, seems like this PR is getting a lot of different kinds of features and optimizations bundled together. it's easier to review if there are more granular PRs... maybe back the svg one out of this PR and put it on a separate branch for now, so we can discuss more about the reasoning there.

@mysterlune - I've removed the SVG stuff stuff and have updated the PR to address your other comments.  Please let me know if anything else needs to be done for this.

hey @atawmicpm, last request... you think you can throw in a unit or integration test for this project?

@mysterlune - PR now updated with tests. :-)

Thanks @atomiccc...


Maybe Incorrect docstring?

Shouldn't this say base llama as opposed to base granite, given the latter is mentioned here?

Shouldn't this say base llama as opposed to base granite, given the latter is mentioned here?

Both of these should most likely be fixed. First should say llama, second should say gpt_bigcode


Conditions with strings having ' and ' / ' or ' word will be interpreted as natural language and replaced with '&&' / '||' code

as above

since tagui conditions are meant to be interpreted, the presence of ' and ' / ' or ' within strings in conditions will be replaced with && / || respectively, which is conditionally wrong. as it is working as originally intended, if users want to compare strings where there is ' and ' / ' or ' within strings, they can assign to a variable first and compare with that variable. example below -
a = 'yes or no'
if a equals to 'yes or no'
        echo 'yes'
else
        echo 'no'

b = 'yes or no'
if a equals to b
        echo 'yes'
else
        echo 'no'

START - automation started - Fri Jun 15 2018 02:41:47 GMT+0800 (+08)

no
yes

FINISH - automation finished - 0.2s

fyi @Aussiroth @lohvht


await import(...)

Expected:
const module = await import('...')

Actual:
const module = await
import ('...')      ^
      ^

newline between await and import
space between import and ('...')

Webpack uses this syntax for importing modules lazily. Not sure if this is (or going to be) a standard though.

+1

@laggingreflex  #1393 Partially addresses this.


Refactor application logic into use cases

Description of changes:
This changes extract application logic away from the ClientDeviceAuthService by introducing use cases which tailor the flow of the application by stitching together the domain and defining interactions (please not that the use case refactored in this PR will still be improved, the certificateManager won't be a thing in the future).
Some additional changes:

Introduced a class called CDAConfiguration that represents the entire application configuration and composes several configuration object
Introduced use cases.

Why is this change necessary:
Increased complexity makes it hard for us to reason about software. Thinking about use cases (workflows) make it easier for us to develop the application and increase visibility into what is the functionality we support, making it easier to maintain and tailor separate scenarios.
How was this change tested:
Ensure tests are still passing and new tests for use cases.

We identified new issues on unchanged lines of code. Navigate to the Amazon CodeGuru Reviewer console to view the recommendations to fix them.


CachePoolChain implements TaggableCacheItemPoolInterface

Question
Answer

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
yes

License
MIT

replace for #162
thx to @somework

Can i change the doc to:
    /**
     * array[]
     *     ['skip_on_failure'] bool If true we will remove a pool form the chain if it fails.
     *
     * @param array $pools
     * @param array $options (See above)
     */

Codecov Report

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

@@          Coverage Diff           @@
##           master    #169   +/-   ##
======================================
  Coverage    76.5%   76.5%
======================================
  Files          26      26
  Lines        1481    1481
======================================
  Hits         1133    1133
  Misses        348     348

Impacted Files
Coverage Δ

src/Adapter/Chain/CachePoolChain.php
80.88% <ø> (ø)
:arrow_up:

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 159d468...dc5abd5. Read the comment docs.

TODO: Update Changelog.md


data_prep: use argparse for introspector

Use argparse for introspector's module. This fixes a todo and will also make it easier to add more feature to the introspector module.

seems to be unrelated to the changes?

Yes, it is unrelated.
That CI builds a docker image based on the code in the PR and pushes it to an artifact registry so that we can run experiments with it.


The version of sequelize that being used is currently vulnerable to sql injection

and should be using 5.8.11.  Thanks

For flexibility, sequelize is used on this module as a devDependency.
So you should simply update the sequelize version of your project.
Regards


Train model on small dataset

Hello author!
I train an unconditional class image generation model on a small dataset (under 100 images), however the model does not generate multiple types of images, no matter how much the seed is set, it is only a small variation of a single type of image. May I ask what is the reason for this? I would be very grateful for your answer!

check your stats.jsonl
either diffaug doesnt work or you have a "NaN" value for your scores/fake


SKU-1474 #comment Pre-filter on server not working

Filter out used members

Hey @Baaka
You can send your PR to  branch release-3.9 and change:
return used_members.indexOf(o.caption) === -1;

to
return used_members.indexOf(o.obj ? o.obj.caption : o.caption) === -1;

===
Well...
when you click in button Pre-Filter on Server, the return of variable this.available_members is for example:
[0]: {
  caption: 'Drink',
  name: 'Drink',
  uniqueName: '[Product].[Products].[Drink]'
}

When you open Selections Modal, the return of variable this.available_members is for example:
[0]: {
  calc: false,
  obj: {
    caption: 'Drink',
    name: 'Drink',
    uniqueName: '[Product].[Products].[Drink]'
  }
}

so the ternary IF will work ;)


unknown type: envoy.extensions.upstreams.http.v3.HttpProtocolOptions

Bug description
Running the example file found at
https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/grpc_json_transcoder_filter#sample-envoy-configuration
returns the error
unknown type: envoy.extensions.upstreams.http.v3.HttpProtocolOptions
Steps to reproduce the bug
create a file 'test.yaml' from https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/grpc_json_transcoder_filter#sample-envoy-configuration
$ envoy -c test.yaml
Version
$ envoy --version
envoy  version: e98e41a8e168af7acae8079fc0cd68155f699aa3/1.16.2/clean-getenvoy-be6132a-envoy/RELEASE/BoringSSL

How was Envoy installed?
Install followed the procedure described here:
https://www.envoyproxy.io/docs/envoy/latest/start/install#install-envoy-on-ubuntu-linux
Environment where the bug was observed (cloud vendor, OS, etc)
$ uname -a
Linux ubuntu 5.4.0-58-generic #64-Ubuntu SMP Wed Dec 9 08:16:25 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

seems to be duplicate of #14272

This does not seem related to Istio at all, and should be an Envoy issue. Seems one is already open: https://github.com/envoyproxy/envoy/issues/14545. Thanks!

This does not seem related to Istio at all, and should be an Envoy issue. Seems one is already open: https://github.com/envoyproxy/envoy/issues/14545. Thanks!


BS4 Compatibility - Datetime Control

This started pretty innocuous, I just wanted to add the list-inline-item class to the list elements, so that they are propery aligned.
The widget parts required a separate template so that empty parts do not leave empty list entries, as they are affecting the layout, causing additional space before time only controls, and after date only controls. In order to avoid having to create a separate template for each part, and possibly also its error variants, the part and error class are being injected in the overriden _datetimeOptions() method.
The way how an error is being detected might not be overly appreciated, ie checking whether the error class exists in the options. Another way would be checking explicitly for an error via the form context in the control() method, and passing the error state into the options, doing that would also allow using an extra error template (dateWidgetPartError), and/or using a custom/extended datetime widget, which could also be used for injecting the part class.
No tests yet, but the test app has been updated with additional date widget variants.
On a somewhat related note, that's a lot of additional templates, once we have all controls working, we might want to check if we can reduce them, there's quite some duplication.

The way how an error is being detected might not be overly appreciated, ie checking whether the error class exists in the options. Another way would be checking explicitly for an error via the form context in the control() method, and passing the error state into the options, doing that would also allow using an extra error template (dateWidgetPartError), and/or using a custom/extended datetime widget, which could also be used for injecting the part class.

Yeah checking for a particular class name to decided if the field has errors is pretty flaky. It would be better to use the context methods instead.

No tests yet, but the test app has been updated with additional date widget variants.

I couldn't get the test app to fetch the updates you have made. After git pull I tried both composer update and compose update but it seems to pull an older commit of BUI. Perhaps the composer.lock or the BUI hash in composer.json needs to be updated? Anyway, just copied the updated FomHelper from this PR and checked.
The display seems fine. Should there be : between the time selects? I don't remember what the core does by default.

On a somewhat related note, that's a lot of additional templates, once we have all controls working, we might want to check if we can reduce them, there's quite some duplication.

Yeah not too thrilled by all the additional templates required for fixing datetime selects display. I am wondering if it's really worth dealing with this. In practice wouldn't majority of the people using bootstrap just use a JS based datetime picker?  At least that's what I do.

I couldn't get the test app to fetch the updates you have made.  [...]

Yeah, sorry, I've never updated the composer requirements, you'd need to require the specific commit that you want to test.

The display seems fine. Should there be : between the time selects? I don't remember what the core does by default.

The core doesn't use any separators, but we could sneak them in if you like?

Yeah not too thrilled by all the additional templates required for fixing datetime selects display. I am wondering if it's really worth dealing with this. In practice wouldn't majority of the people using bootstrap just use a JS based datetime picker? At least that's what I do (through CrudView plugin).

I'm usually using a JS datepicker too, I guess that's true for most people. I don't know, I'd still like to see this working. Given that all the datetime container templates are identical, the date and time ones could be removed, and we could use the datetime ones for all variants, unless the user provides a specific template, hence leaving the ability for the user to use individual templates per variant. That would be 12 templates less.

Ok I just did it and removed the duplicates, please have a look :)
I've tested it with other container templates too, I don't want to speak too soon, because it could easily turn out to be failure, and I've aldready stumbled over one caveat, but it looks like most of them could be removed too.

Yeah, sorry, I've never updated the composer requirements, you'd need to require the specific commit that you want to test.

No problem, I figured.

The core doesn't use any separators, but we could sneak them in if you like?

Nah, we don't need to if core doesn't.

Ok I just did it and removed the duplicates, please have a look :)

Good to see the duplicates are gone. Though there's an issue with display, error message while being generated isn't being shown because it's not matching required CSS selector to set it's display to block.


training COCO on one GPU?

Hello, Can you share me your parameters for 1 GPU with 12 GB memory in training coco dataset?
I used of this Article(https://arxiv.org/abs/1706.02677) and set it as this:
gpu 8 -> 1
lr 0.02 -> 0.0025
max_iter 90000 ->720000
but i have this error now:
FloatingPointError: Predicted boxes or scores contain Inf/NaN. Training has diverged
i had to choose lr very low as 0.00000125 and it will make different result. bAP is 0.0025 :(

I met the same problem, even if I have 8 GPU and the learning-rate is set as 0.02.
I also have FloatingPointError, I am so confused...
Do you fix this bug?

From this issue, maybe I get it , my  latest version of detectron2 is 0.5, while the default is 0.2.

从这个问题，也许我明白了，我最新版本的detectron2是0.5，而默认是0.2。

Hello, sir! I have the same problem. Can you run normally after you change the version of detectron2?

From this issue, maybe I get it , my latest version of detectron2 is 0.5, while the default is 0.2.
Hello, sir! I have the same problem. Can you run normally after you change the version of detectron2?

From this issue, maybe I get it , my latest version of detectron2 is 0.5, while the default is 0.2.

Hello, sir! I have the same problem. Can you run normally after you change the version of detectron2?

From this issue, maybe I get it , my latest version of detectron2 is 0.5, while the default is 0.2.

Hello, sir! I have the same problem. Can you run normally after you change the version of detectron2?

yes you can try

Thank you @Wei-i , i changed detectron2 version to 0.2.1 , and change parameters as i said in first message, but now i have this error on COCO dataset:
CUDA error: out of memory

From this issue, maybe I get it , my latest version of detectron2 is 0.5, while the default is 0.2.

Hello, sir! I have the same problem. Can you run normally after you change the version of detectron2?

yes you can try

Thank you very much for your reply. This problem has been solved for the time being.Is the version of detectron2 making the calculated AP value very low? Or,when the detectron2 version is 0.5, the trained model is normal, but the detectron2 version leads to errors in the AP test?Is that right?
Moreover,when I was performing Stage 2: Few-Shot Fine-Tuning,I can't find the fast_ rcnn_ R_ 101_ FPN_ ft_ novel1_ 1shot.yaml. do you have any solutions?

Thanks for the answers.
@alphacyp The exact reason is something we have to investigate, but we are planning on upgrading to the newest version of detectron2 (may take some time). The config file can be found in this issue. See my answer there.


update(plugins): bump version of cloudtrail, json, okta, and dummy

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area plugins
What this PR does / why we need it:
This bumps the version of the following plugins: cloudtrail, json, okta, and dummy. These new versions are release candidates to be bundled in Falco 0.32. In all of them, the version increased the minor due to the breaking changes introduced in the plugin API and propagated to the newly-released Plugin SDK Go v0.3.0.
There are no breaking changes in the plugin's code or their UX that motivates a major version bump.
Which issue(s) this PR fixes:
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: e4fb84910a8404d7a7540c313d1193804bb4ba62

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Feature] EPS figure

Is it possible to add feature for supporting to export EPS figure ?

Duplicate of #1540 . Please search before posting in future.

#1540 has been closed without fixing. Please consider re-opening either #1540 or this issue. Thanks!


HostInfo: added field to list available SIMD architectures on host device

Internally uses libsimdpp detection code (BSL), converts to string vector

Can https://github.com/p12tic/libsimdpp be a git submodule?

Sure, I just wasn’t sure if it was worth it for one file, especially since that’s not the library in my SIMD proposal. Simple to do though.

On Jun 28, 2020, at 16:53, Josh Blum<EMAIL_ADDRESS>wrote:

Can https://github.com/p12tic/libsimdpp be a git submodule?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sure, I just wasn’t sure if it was worth it for one file, especially since that’s not the library in my SIMD proposal. Simple to do though.

Its a few files in a directory, easier to keep up to date if its a submodule. It might help with packaging to not include the sources of other projects when possible.
I would tag the repo before adding something like this
Minus the one API header call, it could be implemented as a runtime loaded module, if for some reason that was useful.

Sounds good. I'll start a SIMD branch with this stuff in place. My thoughts are that this branch would have:

This new HostInfo field
The simdpp submodule
The CMake fanciness we're talking about here: https://github.com/pothosware/PothosBlocks/pull/63

PothosBlocks and PothosComms could have next-esque branches with the implementations I've done, and we'll have it call into the new CMake stuff as it develops. And it all comes together for the tag?

Closing. Testing a simpler solution without a full additional submodule.


Request: allowing httpx version 0.28

https://github.com/encode/httpx/releases/tag/0.28.0 comes with some breaking changes, mainly on httpx.AsyncClient's app argument.
Trying to pull it in, I get blocked by ollama because it downpins httpx: https://github.com/ollama/ollama-python/blob/v0.4.2/pyproject.toml#L13
Can we move httpx = "^0.27.0" to httpx = ">=0.27"

Hey @jamesbraza, thanks for bringing this up - we pin the version to not have unintended updates (like even the one you've mentioned) Will check this out a bit more bump the version!

Hello @jamesbraza It also means some incompatibility with jupyterlab as well.

breaks llm-ollama plugin install as well, due to conflicts


New lead for Agile Guild

What we're after:

So we can continue our guild leadership, we want to find a replacement for @porta-antiporta :pour-one-out:

@porta-antiporta as discussed, if you hit me up with the materials we used last time, I can review/revise them and coordinate with Jacqueline on next steps.

I :slacked: you what I have re: guild lead selection.

Picking this back up....

We are currently accepting nominations and will reach out to nominated folks next week to assess their interest and schedule interviews.

I have notified folks who have been nominated and will be scheduling interviews over the next two weeks.

As an update, we have interviews scheduled with candidates next week. I still have not heard back from one candidate to determine whether or not they are interested and will follow up with them.

As an update, we rescheduled an interview for the guild lead role, so we now expect to have the next lead selected by the week after Thanksgiving – the week of 12/3.

Waiting on some TTS administrivia.

@afomi selected as new lead Agile Guild. Closing 👋


Error importing vetiver.handlers when torch not installed

Describe the bug
Because vetiver's TorchHandler class tries to set base_class = torch.nn.Module when it is defined, it cannot be imported without torch. Since it is imported in vetiver.handlers, not having torch causes an error on import.
To Reproduce
From vetiver repo
python -m venv env
source env/bin/activate

pip install .

from python
import vetiver.handlers

output:
>>> import vetiver.handlers
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/machow/repos/vetiver-python/vetiver/__init__.py", line 9, in <module>
    from .vetiver_model import VetiverModel
  File "/Users/machow/repos/vetiver-python/vetiver/vetiver_model.py", line 3, in <module>
    from vetiver.handlers._interface import create_handler
  File "/Users/machow/repos/vetiver-python/vetiver/handlers/_interface.py", line 1, in <module>
    from vetiver.handlers import torch, sklearn, base
  File "/Users/machow/repos/vetiver-python/vetiver/handlers/torch.py", line 13, in <module>
    class TorchHandler(VetiverHandler):
  File "/Users/machow/repos/vetiver-python/vetiver/handlers/torch.py", line 22, in TorchHandler
    base_class = torch.nn.Module
NameError: name 'torch' is not defined

This is my bad, for suggesting the base_class =  approach 😬. If you could set base_class = <actual model OR a dummy one> that should do it. Or maybe some other test than using isinstance in the handler?

The crux of the challenge is that to register with a singledispatch generic, we need to reference the class being dispatched on (e.g. torch.nn.Module).
from functools import singledispatch
from vetiver.handlers.torch import TorchHandler

@singledispatch
def create_handler(model, ptype_data):
    raise NotImplementedError()

# base_class is a torch.nn.Module
create_handler.register(TorchHandler.base_class, TorchHandler)

However, if we don't have torch installed, then we'll fail to import TorchHandler, because it will try to set base_class = torch.nn.Module when its defined.
It seems like there are 3 potential fixes:

(easiest?): use a try/except for the importing of the handler, rather than inside the handler file
define a register(cls, dispatcher) classmethod on handlers, so they can register themselves
define each handler as a namespace package, with an entrypoint

Approach 1: move try/except outward
handlers/torch.py:
# note: no try except in this file anymore
import torch

class TorchHandler:
    base_class = torch.nn.Module

handlers/_interface.py:
try:
    from vetiver.handlers.torch import TorchHandler
    create_handler.register(TorchHandler.base_class, TorchHandler)
except ImportError:
    pass

# additional try/except for sklearn

# additional try/except for xgboost

Alternatively, you could write a function to do the try/except
import importlib

def try_register(dispatcher, handler_module_name:str):
    try:
        mod = importlib.import_module(handler_module_name)

        # note: each handler module would need to define a variable named e.g. handler,
        # that is its handler class, so this function doesn't need to know the class's name
        dispatcher.register(mod.handler.base_class, mod.handler)
    except ImportError:
        pass

try_register(create_handler, "vetiver.handlers.torch")

Approach 2: define a register method on handler
For this approach, each handler's module makes sure it can be imported, even if the dep doesn't exist. The added piece is the handler is responsible for registering itself on create_handler (double dispatch).
handlers/torch.py
torch_exists=True
try:
    import torch
except:
    torch_exists=False

class TorchHandler:
    # no base_class, so won't error if torch isn't installed

    @classmethod
    def register(cls, dispatcher):
        if torch_exists:
            dispatcher.register(torch.nn.Module, cls)

handlers/_interface.py
from torch import TorchHandler

TorchHandler.register(create_handler)

This issue is the result of juggling 2 things.

Class definition that should be aware of a potentially unavailable type
Object instantiation that depends on the type of the argument

No. 2, hints at using singledispatch, though this use of singledispatch is interesting because the class already knows about the type on which it specialises. So when every class is created, it has to be registered/associated with the type that it already knows about. This extra bookkeeping can be avoided, though the solution is metaprogramming!
For this to be worth it the payoff in usability should outweigh the complication of introducing metaprogramming. It should be a few lines of code, but maybe too much magic!

@isabelizimm, here is the development of the metaprogramming solution.

I think your piece may be tackling a second problem: automating registering with create_handler on class creation.
So if I understand, it looks like the metaprogramming proposal:

makes base_class dynamic (a staticmethod here)
uses a metaclass to handle the try/except + create_handler registration

In this sense, I think it's close to the second option I mentioned (define a register method on handler), with the added piece of handling the registration automatically!
One quick thought is that I think you could get away with using __init_subclass__() on the base VetiverHandler, to dodge some of the intensity that metaclasses bring.

One quick thought is that I think you could get away with using __init_subclass__() on the base VetiverHandler, to dodge some of the intensity that metaclasses bring.

Perfect, on the money. I had never used it. I have updated the gist.

Oh wow, thank you both SO MUCH for this very thorough explanation of how to fix this! 😮 This gist is so handy on understanding how everything works (and when it doesn't). I don't mind taking this one up after the monitoring is finished, unless someone is feeling very passionate about it!


Deploy mesos-dns on masters and slaves

This should fix #9

This can be tested with
$ dig leader.mesos


Add --wait to iptables call used to list tables.

Fixes issue 368.

Can one of the admins verify this patch?

Looks good.


Remove requirement for "kotlin = true"

As discussed before, we might not have to force users to indicate the language of their annotated @RuntimePermissions class with kotlin = true. Taking a hint from how the JUnit 5 team solves this issue, we might also be able to detect a Kotlin class by looking at its annotations during the annotation processor phase.
I'll play around with this a little and see if I can produce some results. Would be great to remove this additional step for our Kotlin-based users!

Thx! obviously a way to go!

It seems like the annotation processor doesn't receive the "proper" class when processing the Round Environment with kapt. Unfortunately, we can't detect kotlin.Metadata:
roundEnv.getElementsAnnotatedWith(RuntimePermissions::class.java)
    .sortedBy { it.simpleName.toString() }
    .forEach {
        val rpe = RuntimePermissionsElement(it as TypeElement)
        messager.printMessage(Diagnostic.Kind.WARNING, "RuntimePermissions Element: $it")
        messager.printMessage(Diagnostic.Kind.WARNING, "* Annotation Mirrors: ${it.annotationMirrors} (${it.annotationMirrors.size})")

        val metadataCls : Class<Annotation> = Class.forName("kotlin.Metadata") as Class<Annotation>
        messager.printMessage(Diagnostic.Kind.WARNING, "* Metadata Class: $metadataCls")
        messager.printMessage(Diagnostic.Kind.WARNING, "* Metadata On Cls: ${it.getAnnotation(metadataCls)}")
    }

Output:

w: warning: RuntimePermissions Element: permissions.dispatcher.samplekotlin.MainActivity
w: warning: * Annotation Mirrors: @permissions.dispatcher.RuntimePermissions(kotlin=true) (1)
w: warning: * Metadata Class: interface kotlin.Metadata
w: warning: * Metadata On Cls: null

Do you guys suppose we should raise an issue & ask about this for annotation processing specifically?

@aurae Sorry for the late! Thank you for your investigation and I suppose it's worth to file an issue to improve Kotlin dealing from APT!
I found similar forum thread but there's no response yet... 🤔
https://discuss.kotlinlang.org/t/kotlin-data-classes-inspection-from-annotation-processor/2163

seems there's no way to detect annotated class is Kotlin or not within ATP... 😢
@shiraji any idea?

To recap from the meeting: If the distinction between Java and Kotlin can't be made on a per-class basis, it should be moved "a level higher", and converted into a question of "kapt vs. javac apt". Because we are able to detect kapt during the annotation processor's runtime, we can go the Kotlin route for all annotated classes in this case, modelling the interoperability with Java accordingly (e.g. using @file:JvmName). The last dealbreaker with this approach still remains the lack of a Kotlin version to Google's compile-testing library, with which we can validate the correct behaviour during unit testing. We should be reaching out to JetBrains & ask for the plans for a library like that.

Thx for recapping!
Actually that's so so clever way and I'd like to go with it! So next step is I guess:

Fix generated file template to allow Java to call its methods easily
Create PR

As for unit testing, I've not investigated deeply but I wonder we can do file-base testing with google/compiler-testing like that at least they test manifest file generation in google/auto 📝

After upgrading Kotlin to 1.1.3, I confirmed a processor could see kotlin.Metadata by the same way https://github.com/permissions-dispatcher/PermissionsDispatcher/issues/350#issuecomment-319587938 described.
Though it seems that you already got a conclusion of this discussion.... this is just for information!

@jmatsu Wow!!!

Really? I'm pretty sure I tested it on 1.1.4-2 and it wouldn't show up. Thanks for the info!

I've started work on the migration to @kotlin.Metadata here.

@aurae as for lint issue actually I'm not sure it was addressed already or not(seems no update on youtrack?). But please give it a go!

#331 is the issue

Hey - just a quick question: any note on when 3.0.0 will be released? :)

@dbecher-ito we're trying to release on 8/31 or 9/1 for now! https://github.com/permissions-dispatcher/PermissionsDispatcher/milestone/22

This is considered as part of #358


Fix the issue with feature phones

@smn @miltontony @bruskiza ready for review :)

Thanks @bruskiza you are too fast :))

And I did review actually… not just +1 it :D

On 26 Aug 2015, at 4:07 PM, Saeed Marzban<EMAIL_ADDRESS>wrote:
Thanks @bruskiza https://github.com/bruskiza you are too fast :))
—
Reply to this email directly or view it on GitHub https://github.com/praekelt/molo-tuneme/pull/33#issuecomment-135033024.


@conda-forge-admin, please add bot automerge

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/cfn-flip-feedstock#8.


make FAM scrollable

Hi, I'm implementing the FAM in a way that the user can add more buttons, and if the user adds more buttons than the screen can hold, the FAM must become scrollable, much like a ListView, do you guys already have a way to do this? If not, any thoughts?
Thanks

Anyone?

I don't think anyone will be interested in implementing that. Material design spec explicitly says it's wrong to have more than 6 items in FAM.

As a general rule, have at least three options upon press but not more than six, including the original floating action button target. If you have two options – i.e. your floating action button only flings out one other choice – choose which action is most important. If you have more than six, users may have trouble reaching the furthest option.

I don't know what is your use case, but maybe you should use bottom sheet instead?


Filtering by hash with select

Hi again,
small problem with hash and selectbox, after filterin of color and size, the hash is as follow:
#color=green&size=square
After removing the color as filter the chash is:
#color=&size=square
correct would be:
#size=square
Cheers Chris

Hi - this seems to be related to an issue with the MultiFilter .getFilterGroupSelectors() API which wrongly returns an empty string for a select with no value. I'll get this fixed in a future version.
In the meantime, you can make the following adjustment to the serializeGroupsState() function in the hash demo code to avoid this problem:
/**
 * Serializes a groupsState object into a string.
 *
 * @param   {object}    groupsState
 * @return  {string}
 */

function serializeGroupsState(groupsState) {
    var output = '';

    for (var key in groupsState) {
        var values = groupsState[key];

        if (!values.length || (values.length === 1 && values[0] === '')) continue; // check for empty string in array

        output += key + '=';
        output += values.join(',');
        output += '&';
    };

    output = output.replace(/&$/g, '');

    return output;
}

This has now been fixed in the latest MixItUp MultiFilter version (v3.3.1), which you can download from your account.


fix: change Harbor to Registry in in user logs

Hey there hope your doing good
I'm new in contributing on open source projects so maybe I'm wrong but This PR closes https://github.com/kthcloud/go-deploy/issues/480 issue.

LGTM, thanks for contributing! 🚀
We are currently working on making this project buildable for anyone. So stay tuned for that :)


Interpreter README: mode is selected by an ENV var

Indeed, thanks! I naïvely thought it was a command line argument when writing the README.


The sample application code on GitHub not found

The repository "https://github.com/Azure-Samples/storage-blobs-go-quickstart" not found.

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

ID: 3402293d-ac39-abd4-55ae-a3fc55180883
Version Independent ID: 832abbb0-5588-fadc-b413-4ad3f2a870bc
Content: Azure Quickstart - Create a blob in object storage using Go
Content Source: articles/storage/blobs/storage-quickstart-blobs-go.md
Service: storage
GitHub Login: @seguler
Microsoft Alias: seguler

@ToruMakabe Thanks for the feedback! We are currently investigating and will update you shortly.

Resolved.


missing parameter error when using invoke  transfer with neo3-preview4

Hello,
I created a NEP17 contract under neo3-preview4 (following) the example.
The command transfer works.
However using either neo-cli or curl, it constantly failed because of a 4th parameter which is missing (from, to, amount and something else missing).
I tried every json combination, and I always end up with an error.
here an example:
curl -H 'Content-Type: application/json' -v -d '{"jsonrpc":"2.0","method":"invokefunction","params":["0x1415cac91819029f329aec275c937c8158ee64b4","transfer",[{"type":"Hash160","value":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c"},{"type":"Hash160","value":"
0x8c5d07e39d3d5947c31da98b797c5f322e1bc9af"},{"type":"Integer","value":"100"}]],"id":1234}' localhost:13332
end I get this answer:
{"jsonrpc":"2.0","id":1234,"result":{"script":"AGQMFK/JGy4yX3x5i6kdw0dZPZ3jB12MDBR8BHrMoJe1J0\u002BwvRUc8Y1LTgwcYhPADAh0cmFuc2ZlcgwUtGTuWIF8k1wn7JoynwIZGMnKFRRBYn1bUg==","state":"FAULT","gasconsumed":"0.0099942","exception":"Method transfer Expects 4 Arguments But Receives 3 Arguments","stack":[]}}
so the question what is that 4th parameter and how my json request including that parameter should be written
(or alternatively, I would suggest to add the "transfer " method to RpcServer)

you need to pass 4 parameters, the fourth param is object type, so you can just pass string, or other object-type.
curl -H 'Content-Type: application/json' -v -d '{"jsonrpc":"2.0","method":"invokefunction","params":["0x1415cac91819029f329aec275c937c8158ee64b4","transfer",[{"type":"Hash160","value":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c"},{"type":"Hash160","value":"
0x8c5d07e39d3d5947c31da98b797c5f322e1bc9af"},{"type":"Integer","value":"100"},{"type":"String","value":"random"}]],"id":1234}' localhost:13332

you need to pass 4 parameters, the fourth param is object type, so you can just pass string, or other object-type.
curl -H 'Content-Type: application/json' -v -d '{"jsonrpc":"2.0","method":"invokefunction","params":["0x1415cac91819029f329aec275c937c8158ee64b4","transfer",[{"type":"Hash160","value":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c"},{"type":"Hash160","value":"
0x8c5d07e39d3d5947c31da98b797c5f322e1bc9af"},{"type":"Integer","value":"100"},{"type":"String","value":"random"}]],"id":1234}' localhost:13332

Hi,
thanks a lot, it worked (I tried that before but I might have had an error).
However the problem is that the transfer doesn't not seem to happen.
Here what I tried:
curl -H 'Content-Type: application/json' -v -d '{"jsonrpc":"2.0","method":"invokefunction","params":["0x92049d6721839fbed9d6b770ed90d271afae9747","transfer",[{"type":"Hash160","value":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c"},{"type":"Hash160","value":"0xe9fb13f75c1e9d6365f90a22d4d24fb6275d4d15"},{"type":"Integer","value":"1000"},{"type":"String","value":"truc"}],[{"account":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c","scopes": "CalledByEntry","allowedcontracts":[],"allowedgroups":[]}]],"id":1234}' localhost:13332
and the answer:
{"jsonrpc":"2.0","id":1234,"result":{"script":"DAR0cnVjAegDDBQVTV0ntk/S1CIK\u002BWVjnR5c9xP76QwUfAR6zKCXtSdPsL0VHPGNS04MHGIUwAwIdHJhbnNmZXIMFEeXrq9x0pDtcLfW2b6fgyFnnQSSQWJ9W1I=","state":"HALT","gasconsumed":"0.1148962","exception":null,"stack":[{"type":"Integer","value":"1"}],"tx":"ADDAxxtUUa8AAAAAAHbWEgAAAAAAXvkAAAF8BHrMoJe1J0\u002BwvRUc8Y1LTgwcYgEAXAwEdHJ1YwHoAwwUFU1dJ7ZP0tQiCvllY50eXPcT\u002B\u002BkMFHwEesygl7UnT7C9FRzxjUtODBxiFMAMCHRyYW5zZmVyDBRHl66vcdKQ7XC31tm\u002Bn4MhZ50EkkFifVtSAUIMQCVr4eTQNWhRERovoJf0rtYZXtrG7i0iEmIzncr8ZznUR00wiiuGXDGwTkILyO9VhmZaueDP\u002BkYcw1PeSH9dnIEpDCECzlsSgLaaFH0RaUxjeHjxsf67qBwQXcVgKfkKZorspOkLQZVEDXg="}}
But when checking the balance, the account didn't receive the tokens.
Do I need something else to sign and relay the transaction ?
Thanks for your help.

Hi,
thanks a lot, it worked (I tried that before but I might have had an error).
However the problem is that the transfer doesn't not seem to happen.
Here what I tried:
curl -H 'Content-Type: application/json' -v -d '{"jsonrpc":"2.0","method":"invokefunction","params":["0x92049d6721839fbed9d6b770ed90d271afae9747","transfer",[{"type":"Hash160","value":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c"},{"type":"Hash160","value":"0xe9fb13f75c1e9d6365f90a22d4d24fb6275d4d15"},{"type":"Integer","value":"1000"},{"type":"String","value":"truc"}],[{"account":"0x621c0c4e4b8df11c15bdb04f27b597a0cc7a047c","scopes": "CalledByEntry","allowedcontracts":[],"allowedgroups":[]}]],"id":1234}' localhost:13332
and the answer:
{"jsonrpc":"2.0","id":1234,"result":{"script":"DAR0cnVjAegDDBQVTV0ntk/S1CIK\u002BWVjnR5c9xP76QwUfAR6zKCXtSdPsL0VHPGNS04MHGIUwAwIdHJhbnNmZXIMFEeXrq9x0pDtcLfW2b6fgyFnnQSSQWJ9W1I=","state":"HALT","gasconsumed":"0.1148962","exception":null,"stack":[{"type":"Integer","value":"1"}],"tx":"ADDAxxtUUa8AAAAAAHbWEgAAAAAAXvkAAAF8BHrMoJe1J0\u002BwvRUc8Y1LTgwcYgEAXAwEdHJ1YwHoAwwUFU1dJ7ZP0tQiCvllY50eXPcT\u002B\u002BkMFHwEesygl7UnT7C9FRzxjUtODBxiFMAMCHRyYW5zZmVyDBRHl66vcdKQ7XC31tm\u002Bn4MhZ50EkkFifVtSAUIMQCVr4eTQNWhRERovoJf0rtYZXtrG7i0iEmIzncr8ZznUR00wiiuGXDGwTkILyO9VhmZaueDP\u002BkYcw1PeSH9dnIEpDCECzlsSgLaaFH0RaUxjeHjxsf67qBwQXcVgKfkKZorspOkLQZVEDXg="}}
But when checking the balance, the account didn't receive the tokens.
Do I need something else to sign and relay the transaction ?
Thanks for your help.

The invokefunction RPC method does not change the state of the blockchain. It only tests the transaction. The Neo docs say: "This method is used to test your VM script as if they ran on the blockchain at that point in time. This RPC call does not affect the blockchain in any way." (see here).
To actually persist the change to the blockchain a transaction needs to be built and transmitted through the blockchain network.
(1) One way to do this is by using the neo-cli. You could use the transfer method for this, as you already now, i think.
For example: transfer 0x4961bf0ab79370b23dc45cde29f568d0e0fa6e93 NZNos2WqTbu5oCgyfss9kUJgBXJqhuYAaj 100, which transfers 100 NEO to the address NZN...
(2) Another way is to use the sendfrom RPC method in case you have an open wallet on the neo-node. For example:
{
  "jsonrpc": "2.0",
  "method": "sendfrom",
  "params": [
		"3e1c7c20b1ddbb998b1048061e7665c426b85b14", "NZNos2WqTbu5oCgyfss9kUJgBXJqhuYAaj", "NX8GreRFGFK5wpGMWetpX93HmtrezGogzk", 0.1
	],
  "id": 1
}

Where the first parameter is the contract hash, the second the source account, the third the receiving account, and the fourth the amount to transfer.

The invokefunction RPC method does not change the state of the blockchain. It only tests the transaction. The Neo docs say: "This method is used to test your VM script as if they ran on the blockchain at that point in time. This RPC call does not affect the blockchain in any way." (see here).
To actually persist the change to the blockchain a transaction needs to be built and transmitted through the blockchain network.
(1) One way to do this is by using the neo-cli. You could use the transfer method for this, as you already now, i think.
For example: transfer 0x4961bf0ab79370b23dc45cde29f568d0e0fa6e93 NZNos2WqTbu5oCgyfss9kUJgBXJqhuYAaj 100, which transfers 100 NEO to the address NZN...
(2) Another way is to use the sendfrom RPC method in case you have an open wallet on the neo-node. For example:
{
  "jsonrpc": "2.0",
  "method": "sendfrom",
  "params": [
		"3e1c7c20b1ddbb998b1048061e7665c426b85b14", "NZNos2WqTbu5oCgyfss9kUJgBXJqhuYAaj", "NX8GreRFGFK5wpGMWetpX93HmtrezGogzk", 0.1
	],
  "id": 1
}

Where the first parameter is the contract hash, the second the source account, the third the receiving account, and the fourth the amount to transfer.

ok thanks a lot, I thought that sendfrom was only for neo/gas
actually (just found out) it is also possible to use invokefunction, then send the generated tx with sendrawtransaction. But sendfrom is definitely easier.
Actually I am trying to find a way to automatize all smartcontract management.

ok thanks a lot, I thought that sendfrom was only for neo/gas
actually (just found out) it is also possible to use invokefunction, then send the generated tx with sendrawtransaction. But sendfrom is definitely easier.
Actually I am trying to find a way to automatize all smartcontract management.

If you want to automate interaction with your smart contract and the Neo blockchain in general, you could use one of the many libraries that provide the required functionality for this, e.g., neow3j for Java, neo-one for TypeScript, neo-go for Go.

If you want to automate interaction with your smart contract and the Neo blockchain in general, you could use one of the many libraries that provide the required functionality for this, e.g., neow3j for Java, neo-one for TypeScript, neo-go for Go.

neow3j would probably a good option, does it support neo3-preview4 ?

neow3j would probably a good option, does it support neo3-preview4 ?

Yes, from version 3.6.0 neow3j supports preview4. The latest version at this time is 3.6.1.
Next to the documentation at neow3j.io there is an example repository that provides many example usages of the library.
Please don't hesitate to open issues or discussions if you get stuck or find inconveniences in the library.

Yes, from version 3.6.0 neow3j supports preview4. The latest version at this time is 3.6.1.
Next to the documentation at neow3j.io there is an example repository that provides many example usages of the library.
Please don't hesitate to open issues or discussions if you get stuck or find inconveniences in the library.

@djm34  so this is not an issue, can you close?

@djm34  so this is not an issue, can you close?


prometheus: Use hashstructure package for input checksum

Instead of marshalling all StatefulSet inputs to JSON and hashing that
string, use hashstructure [1] package. In addition do not generate new
rule config maps if underlying rules did not change.
Related to https://github.com/coreos/prometheus-operator/issues/1659.
[1] github.com/mitchellh/hashstructure

@squat replaced all config map with ConfigMap in the entire repo. Thanks for the hint.

Of course, sorry for distracting from the actual PR target. We can pair any
time on this one.
Max Inden<EMAIL_ADDRESS>schrieb am Do., 2. Aug. 2018, 19:51:

@mxinden commented on this pull request.
In pkg/prometheus/operator.go
https://github.com/coreos/prometheus-operator/pull/1737#discussion_r207316963
:

@@ -969,18 +968,22 @@ func (c *Operator) sync(key string) error {
}

func createSSetInputChecksum(p monitoringv1.Prometheus, c Config, ruleConfigMapNames []string) (string, error) {
I went through a couple of iterations on this idea, but ended up bursting
the scope of this PR. I will follow up with you on this @s-urbaniak
https://github.com/s-urbaniak if you don't mind.
what does the extra S mean in this func name?
@squat https://github.com/squat SSet stands for a StatefulSet.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/coreos/prometheus-operator/pull/1737#discussion_r207316963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAW8MKTi4_UqTZWLfWBpDrHA8_cUEe_jks5uMzw5gaJpZM4VsQrk
.


[AutoPR portal/resource-manager] Add Swagger and Examples for shared dashboards to Azure

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5811

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5017


mirgrate chartjs and leaflet library installs from bower to npm.

bower is on the way out.  Should move library imports to npm

complete


Removes the Serilog provider and its tests

The provider has moved to https://github.com/serilog/serilog-framework-logging (many thanks!)
This PR removes the obsolete Microsoft.Framework.Logging.Serilog sample so that users are more likely to track down the shipped/maintained package.
Unfortunately the API of m.f.Logging has changed slightly (BeginScopeImpl()) since VS2015 RC packages were published, so the SampleApp can't yet be built against the RC s.f.Logging package on NuGet.
Having the JSON dump in the sample was a nice way to inspect the structured data on a log event, it can definitely be lived without however we could also:

Add a "JSON dump" provider to the sample to emulate what the Serilog one was doing, or
Reinstate that part of the sample once dev and the NUPKG line up again (I'd happily PR this when the time comes)

Hi @nblumhardt, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

I don't see the actual provider being deleted here... is GitHub just not showing it?

Ooops - that was a somewhat significant omission. Fixed now :-)

Any chance you could squash the commits into one? It'll make for a cleaner merge.

Should be good to go now :+1:

Thanks!


angular cli bug force me to eject

I'm using worker-loader npm module, but it's not working without adding the following lines to the webpack.config.js, so i need to eject, which is not convenient if i want to switch between prod and dev.
"options": {
"tsConfigPath": "src/tsconfig.app.json",
}
Can somebody add this line to the default config? Spent some time, but not able to figure out, where it should goin the angular cli source.
ng build --aot does not solve the problem - there were similar issues raised, but it's not duplicate, as that issue didn't resolve the problem without ejection.
webpack error also has been swallowed at default, i need to use --verbose to look at, which is not nice.
(i saw only an empty line more, when problem happened)

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.


warning: default-features is ignored for arrow,

Describe the bug
warning: /Users/jakevin/code/arrow-datafusion/datafusion/jit/Cargo.toml: `default-features` is ignored for arrow, since `default-features` was not specified for `workspace.dependencies.arrow`, this could become a hard error in the future
warning: /Users/jakevin/code/arrow-datafusion/datafusion/common/Cargo.toml: `default-features` is ignored for arrow, since `default-features` was not specified for `workspace.dependencies.arrow`, this could become a hard error in the future
warning: /Users/jakevin/code/arrow-datafusion/datafusion/common/Cargo.toml: `default-features` is ignored for parquet, since `default-features` was not specified for `workspace.dependencies.parquet`, this could become a hard error in the future
warning: /Users/jakevin/code/arrow-datafusion/datafusion/expr/Cargo.toml: `default-features` is ignored for arrow, since `default-features` was not specified for `workspace.dependencies.arrow`, this could become a hard error in the future

To Reproduce
cargo build
Expected behavior
no warning
Additional context

cargo build cannot repoduce warning on my machine, but cargo +nightly build can.


Error while start master_discovery

Hello, I followed the Technical Report by Sergi Hernandez Juan and Fernando Herrero Cotarelo (Multi Master ROS Systems) and I am facing this issue : Error while start master_discovery: socket.error[19]: [Errno 19] No such device,
is multicast route set? e.g. sudo route add -net <IP_ADDRESS> netmask <IP_ADDRESS> eth0
By running the suggested command, it works fine, but for once. I need to start this command each time and it is annoying. How to fix this issue ?

Hello, you need to set a default gateway or route for multicast in your network settings.

Thx. I saw in the document a way to set a default gateway. I tried to follow the instructions (creating a file in /etc/network/if-up.d) to add a default gateway, but I don't know where to do it, because my setup is the following :

4 drones with a master running on each ;
my ground computer on which I want to synchronize the drones' masters.

There are many ways to set a permanent route under Linux. It depends on your operating system and chosen network configuration.
It is important to define to which network interface the multicast messages should be sent. You have to set this up on each host.


Android App crashes with DownloadManager when Endpoint returns 403

Are there any option to catch 403's from fetch with Android Download Manager?
The problem is that at the moment of the fetch call i'm not pretty sure if my endpoint responses with a valid 200. Do i have to check the endpoint manually before trying to download my binary with the DownloadManager?

@manuelkch , I've added error handling to Android DownloadManager, could you please verify it?
To update the package from branch issue-296
$ npm install --save git://github.com/wkh237/react-native-fetch-blob.git#issue-296


The first two template examples are incorrect

I believe the first two template examples on this page are incorrect. To start they are not valid json (see the additional "{" within the configurationArguments property). Additionally the examples do not comply with the settings schema on this page. Specifically the configurationArguments object.

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

ID: 92491b1b-f7e2-c4d2-f136-9d7afcc1d186
Version Independent ID: a7b77b0e-ba18-9a49-639e-7e9a99cd4017
Content
Content Source
Service: virtual-machines-windows

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

On top of the obvious schema issue, this page does not elaborate on what members the "configurationArgument" object can have in the new schema version as opposed to what the "property" array was supporting in the old schema. It however suggests that configurationArgument is a like for like replacement for property... however property was clearly an array and from the ARM deploy error messages I get I assume configurationArgument is an object.

I was just about to report the same issue! Pretty frustrating :(
Did any of you manage to get it working?

@mgreenegit any progress on this issue?

I did using the default configuration script. I was going to submit a pull request for the updated docs but haven't had a chance yet. I believe the initial example should look like this:
"name": "Microsoft.Powershell.DSC",              "type": "extensions",              "location": "[resourceGroup().location]",              "apiVersion": "2015-06-15",              "dependsOn": [                   "[concat('Microsoft.Compute/virtualMachines/', variables('vmName'))]"               ],               "properties": {                   "publisher": "Microsoft.Powershell",                   "type": "DSC",                   "typeHandlerVersion": "2.72",                   "autoUpgradeMinorVersion": true,                   "forceUpdateTag": "[parameters('dscExtensionUpdateTagVersion')]",                   "settings": {                        "configurationArguments": {                             "RegistrationUrl" : "[parameters('registrationUrl1')]",                             "NodeConfigurationName" : "nodeConfigurationNameValue1"                        }                   },                   "protectedSettings": {                        "configurationArguments": {                             "RegistrationKey": {                                  "userName": "PLACEHOLDER_DONOTUSE",                                  "password": "[parameters('registrationKey1')]"                             }                        }                   }

I am getting an error using the above template " The type property is invalid." for the line "type": "extensions",

@timlt can you take a look into this? @mgreenegit is listed as the author but appears to no longer be contributing here.

Similar to @chrisnewark , here is what I ended up going with:
{
                "name": "Microsoft.Powershell.DSC",
                "properties": {
                  "publisher": "Microsoft.Powershell",
                  "type": "DSC",
                  "typeHandlerVersion": "2.72",
                  "autoUpgradeMinorVersion": true,
                  "protectedSettings": {
                    "Items": {
                      "registrationKeyPrivate":
                        "[parameters('registrationKey')]"
                    }
                  },
                  "settings": {
                    "Properties": [
                      {
                        "Name": "RegistrationKey",
                        "Value": {
                          "UserName": "PLACEHOLDER_DONOTUSE",
                          "Password":
                            "PrivateSettingsRef:registrationKeyPrivate"
                        },
                        "TypeName": "System.Management.Automation.PSCredential"
                      },
                      {
                        "Name": "RegistrationUrl",
                        "Value": "[parameters('registrationUrl')]",
                        "TypeName": "System.String"
                      },
                      {
                        "Name": "NodeConfigurationName",
                        "Value": "[parameters('nodeConfigurationName')]",
                        "TypeName": "System.String"
                      }
                    ]
                  }
                }
              }

Getting the error that type is not defined and once type is defined as below
"type": "extensions",
Getting the error message as type invalid . Are people looking into this issue ?

We are working on getting this correct and will update everyone shortly.

Update
We are working on getting this fixed today.

Is there any update on the issue

Any updates?

@girishgoudar it is taking longer than expected to fix. The team is working on it.

#in-progress @MicahMcKittrick-MSFT can you swap the timlt tag for jpconnock?

#please-close the fix merged and other fixes are in progress, closing. @chrisnewark @girishgouda @tonjohn @pekkaar thank you for your feedback!


Typing for variables partially works.

In the below I am trying to set the type of the variable i in a for loop. the on-hover looks correct when done with @type, but it fails to evaluate correctly when used, however the cast works as expected.
---@class integerA : integer
local list = {} ---@type table<integerA, table>

---@type integerA
for i = 1, 10 do
    list[i] = {} --- This `i` says its type integerA, however the line gives an error of "Can not infer type. LuaDiagnostics.(no-unknown)"
end

for i = 1, 10 do
    ---@cast i integerA
    list[i] = {}
end

I believe the @type method is "better" than cast as it feels a more natural way to define its type, rather than forcing the variable to be viewed as having a set type with cast.

I don't see the issue when running 3.5.0

Maybe it's somehow from my Sumneko extension settings, as I do have a fair few non default.
I am running the 3.5.0 public realease.

My extension settings are below in case any are somehow relevant.
The fact I have strong enabled to flag unknown typed variables is the only one I assume is relevant given the diagnostic warning that was being raised.
settings.json.txt


ERM - Add new "multi" tables to derived table "agreements_custom_property"

There are new additional tables that need to be added to the derived table agreements_custom_property.

All tables with "multi" in the table name. These tables contains data, if you like to have multiple values inside a custom property.

Waiting for new test environment.


Settings creator + structured interface

This builds on the work in #1904 and #1945 by structuring the signature to Settings, as well as #1721 and #1689 by adding a user-facing SettingsCreator class.
Key logic flow is this:

Linker accepts settings_dict (future: also Settings or SettingsCreator)

converts this to SettingsCreator, and immediately to Settings

SettingsCreator handles turning blocking rule / comparisons from dicts to concrete objects, and in particular does so before it instantiates a Settings

Included:

new SettingsCreator class - simple dataclass, with -> dict and -> Settings methods

naming with this and Settings can change, but while this is in flux this seems like a clear choice

Fixed an issue where CustomComparison from a dict forgot about configuration
Renamed some arguments (blocking rule, comparison) to be in line with their serialised counterparts - this needn't be permanent
More stuff is retained when deepcopying a Settings

Not done:

as_dict() doesn't include additional cols to retain, until we have disentangled InputColumn
I have turned off settings validation until this area has settled and it's clearer what is needed (and skipping corresponding tests)

There are a few fiddly things that took a while to untangle, but think should be mostly fine now - all the tests are passing at least. That said I am happy for this to be a 'longer-lived' branch while subsequent refactorings occur, if we want to be cautious.
I've tried to keep this as minimal as possible while still doing something, so there are a few follow-ups to come building on this of various size:

Split up Settings a bit, and don't pass the whole thing around as much (e.g. InputColumn only needs a fraction of information)
Settings gathering some data (e.g. sql_dialect) from bottom-up
Default settings values living in a single canonical place

Automatic updating of json schema?

Make settings truly required for Linker

linker_uid

Just having a play with this and was looking at some options for autocompletion.  This seems nice:


Cleaned up previous pull request; Prefix, comment additions, typos

The commits are now cleaner and not in the master branch.  Please let me know if I should keep to my own fork instead.  Thanks!

golint complains -  comment on exported method AppID.MarshalJSON should be of the form "MarshalJSON ..."
I apologize if I'm misunderstanding something.

Ah, indeed. Maybe:
// MarshalJSON is generated so {{$typename}} satisfies json.Marshaler.

Corrected.

Any updates here?

Sorry I missed this.


batch_size, cpu_num , memory,train_time,

I found that the larger the batch_size is the less the memory will be, why?

when i set more cpu_num it will be stop unpredictable dure trainning

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


RM-6499 Adding new Fields, and WebHooks support to model.

RM-6499 Adding new Fields, and WebHooks support to model.
Update of libs also included.
I commented out the asserts on SupplierCost ... IntegrationTest that where failing as a result of more records being returned, so we'd assert the remaining, which still pass.
The TestHelper.log for Sys.out was failing because the json contained html links that included %D and the formatter was expecting a parameter to replace it with.  Adding log with out format and uncommented it's exclution from the UT.
I added a Data object for Payload, but did not refactor other objects to utilize it, like Index or Show, but they probably could.  Data uses the Type field to identify the Object type when deserializing the object.

Is it really ok to remove the signed boolean from the Attachment model? If you upload a misc. attachment in the ServeManager UI, you can specify that you want to upload it as an affidavit and whether it is signed.

Never mind. I see it was just moved.

Is it really ok to remove the signed boolean from the Attachment model? If you upload a misc. attachment in the ServeManager UI, you can specify that you want to upload it as an affidavit and whether it is signed.

It's not removed, it just moved to DocumentBase to Document, Attachment, and serviceDocument all have it


AttributeError: type object 'BuildExtension' has no attribute 'with_options'

🐛 Bug

To Reproduce
Steps to reproduce the behavior:
~$ python setup.py install
-- Building version 0.6.0a0+da59f59
Traceback (most recent call last):
  File "setup.py", line 86, in <module>
    'build_ext': setup_helpers.BuildExtension.with_options(no_python_abi_suffix=True)
AttributeError: type object 'BuildExtension' has no attribute 'with_options'

Expected behavior
install torchaudio sucessfully

Environment

What commands did you used to install torchaudio (conda/pip/build from source)?
from source
python setup/py. install
If you are building from source, which commit is it?
6c3cc16
What does torchaudio.__version__ print? (If applicable)
Not applicable

Please copy and paste the output from our
environment collection script
(or fill out the checklist below manually).
You can get the script and run it with:
wget https://raw.githubusercontent.com/pytorch/pytorch/master/torch/utils/collect_env.py
# For security purposes, please check the contents of collect_env.py before running it.
python collect_env.py

$ python collect_env.py
Collecting environment information...
PyTorch version: 0.4.1
Is debug build: No
CUDA used to build PyTorch: 8.0.61

OS: Ubuntu 16.04.6 LTS
GCC version: (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
CMake version: version 3.16.0

Python version: 3.6
Is CUDA available: Yes
CUDA runtime version: 9.0.176
GPU models and configuration:
GPU 0: TITAN X (Pascal)
GPU 1: TITAN X (Pascal)
GPU 2: TITAN X (Pascal)
GPU 3: TITAN X (Pascal)

Nvidia driver version: 410.78
cuDNN version: Probably one of the following:
/usr/local/cuda-10.0/targets/x86_64-linux/lib/libcudnn.so.7
/usr/local/cuda-8.0/targets/x86_64-linux/lib/libcudnn.so.6

Versions of relevant libraries:
[pip3] numpy==1.13.3
[pip3] numpydoc==0.7.0
[conda] numpy                     1.16.3                   pypi_0    pypi
[conda] torch                     1.0.1                    pypi_0    pypi
[conda] torchfile                 0.1.0                    pypi_0    pypi
[conda] torchsnooper              0.7.1                    pypi_0    pypi
[conda] torchvision               0.3.0                    pypi_0    pypi

Additional context

Hi @KiAlexander
To use master branch of torchaudio, the latest PyTorch is required. Please upgrade your PyTorch.

To be more specific, the master branch of torchaudio depends on the master branch of pytorch.
I will close this issue, but please feel free to re-open if the issue persists with a compiled version of the lastest pytorch master.


Add info about v2 VPI not support in CSI

What this PR does / why we need it:
Add no support in CSI cinder on cinder v2 API
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Thanks
/lgtm

/approve


Cannot load mysql2 — Discourse Docker

Disclaimer: I don't know ruby/rails.
In a Discourse docker install, I couldn't execute gem install mysql2 as the discourse user (permission issue) so I did it as root. Then I bundle install but the mysql2 gem isn't in the output list.
When using rake import:bbpress it displays :
rake aborted!
LoadError: cannot load such file -- mysql2
(+ stacktrace)

What I am doing wrong?

Found the solution : add - "templates/import/mysql-dep.template.yml" under the templates section in your containers/app.yml.


Participating EOS Members

Something very similar to
https://www.alohaeos.com/vote/proxy/brockpierce1


The spaces of the value which started with spaces disappear after refreshing

OS: Win10/Mac
Build Version: 20200830.1
Repro Steps:

Expand the Web App.
Right click the "Application Settings" node --> Click "Add New Setting..." --> Enter new app setting name --> Enter the value "       1234".
Check the new added setting.
Check it again after refreshing.

Expect:
Keep the value of new added setting consistent after refreshing.
Actual:
The spaces of the value which started with spaces disappear.

This issue has become stale and is at risk of being closed. The community has 60 days to upvote the issue. If it receives 5 upvotes we will keep it open and take another look. If not, we will close it. To learn more about how we handle issues, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Aia ca issuers must have http only

This pull request adds a lint to check that the value of of the id-ad-caIssuers access method is an HTTP URL and outputs an error if the scheme is not http. This work has been made in close cooperation with the D-Trust CA. We would be grateful if you could incorporate this PR in the main project.
Citation (https://cabforum.org/uploads/CA-Browser-Forum-BR-v2.0.0.pdf):
<IP_ADDRESS>.7 Subscriber Certificate Authority Information Acces
id-ad-caIssuers
<IP_ADDRESS>.<IP_ADDRESS>.2 uniformResourceIdentifier
SHOULD
*
A HTTP URL of the Issuing CA’s certificate

How does or should this lint relate to the existing lint https://github.com/zmap/zlint/blob/26ca0f3bed092ef6e6b34f546f68068ae9d808a1/v3/lints/cabf_smime_br/lint_strict_aia_has_http_only.go ?

How does or should this lint relate to the existing lint https://github.com/zmap/zlint/blob/26ca0f3bed092ef6e6b34f546f68068ae9d808a1/v3/lints/cabf_smime_br/lint_strict_aia_has_http_only.go ?
FYI, I'm assuming the answer is "TLS vs SMIME" but just wanted to clarify that.

Exactly :) It is the same requirement and the same implementation but only for the caIssuers method. We could just take the SMIME implementation and re-use it (changing citation, description, test certificates, etc.). The checkApplies implementation is then the only change.
Still, this might arise the question why not have a "bigger" lint in TLS that checks both methods for HTTP scheme, as the SMIME lint does. My impression is that having small lints with (very) narrow focus is a better choice that allows more flexibility, and especially to align implementation with the specification and changes in the specification. For example, to exaggerate a litlle, we could have a big lint that checks everything associated to authority information access, like presence or not, correct criticality, correct values and so on.  But this approach has not been followed in zlint. As a rule of thumb I would say that one sentence (maybe even a subordinate clause?) in a specification with one reserved keyword could be one lint.
I would be willing to update the lint to match the SMIME lint (bright focus option) or create a new one for the OCSP URL (narrow focus option). Let us wait what you and other users say on this and then I can start working on this.
Best,
Vangelis

I agree with your statement on keeping the lints simple and single purpose, and it was something I questioned in the SMIME lint at least internally (as it also used to have an even larger scope than it does). I think the argument for a single lint is likely simply that the check is dealing with fields in the same "bucket" so to speak, and checking the same rules on those fields. However, I'm not convinced that is a compelling enough argument to have a single lint for both... basically I am torn on it :)

One suggestion: can we add a test case in which the AIA Issuers URL is https://?

yes. I just added a new test case. This required a change in the implementation to catch the HTTPS case, if  both http and https are a correct match for: "A HTTP URL of the Issuing CA’s certificate." (https://cabforum.org/uploads/CA-Browser-Forum-BR-v2.0.0.pdf, Section <IP_ADDRESS>.7)

I don't believe that https is a valid case for this. We had a similar conversation on the SMIME version of this lint.

I don't believe that https is a valid case for this. We had a similar conversation on the SMIME version of this lint.

it boils down to whether an https URL falls under "A HTTP URL of the Issuing CA’s certificate.". Would an https URL in a certificate be a violation of the BR requirements?
My proposal is that I place an issue at the github repository of CAB Forum for clarification and wait for some feedback there.

FWIW, I got that information from @CBonnell .

Update: https is a violation see: https://github.com/cabforum/servercert/issues/522 and https://lists.cabforum.org/pipermail/servercert-wg/2024-April/004462.html. PR is updated.


Does not work with current lambda and won't deploy

Lambda's standard version of Node is 12. this app is using version 8. Will there be an update?

This one should be fixed now


feat(event_source): Extend CodePipeline Artifact Capabilities

Issue number: #5447
Summary
Changes
This change adds the ability to upload output artifacts in a simple way, pulling all possible config from the event itself.
User experience
import json

from aws_lambda_powertools import Logger
from aws_lambda_powertools.utilities.data_classes import event_source, CodePipelineJobEvent

logger = Logger()

@event_source(data_class=CodePipelineJobEvent)
def lambda_handler(event: CodePipelineJobEvent, context):
    ...
    output_artifact = do_the_thing()
    event.put_artifact(
        artifact_name="json-artifact",
        body=json.dumps(output_artifact),
        content_type="application/json"
    )

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Regarding the above issues from SonarCloud:
These are all mocked methods of the S3 client and don't match the snake_case convention because boto3 doesn't ;)

Hi @mw-root! Thanks for sending this PR! This week is super busy, but I'll start reviewing it today, ok?

@leandrodamascena glad to do it and no rush at all, thanks for taking it on!

Hi @mw-root , just ran our CI here, and mypy is complaining about the None type here. I saw that self.data.encryption_key can be CodePipelineEncryptionKey or None.
Can you please take a look?
Item "None" of "CodePipelineEncryptionKey | None" has no attribute "get_id"
[union-attr]
            encryption_key_id = self.data.encryption_key.get_id
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
aws_lambda_powertools/utilities/data_classes/code_pipeline_job_event.py:313:31: error:
Item "None" of "CodePipelineEncryptionKey | None" has no attribute "get_type"
[union-attr]
            encryption_key_type = self.data.encryption_key.get_type
                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Thanks! Let me know if you need help with this

@anafalcao
Sorry about that, commited a fix for the mypy issues.
Thanks!

I'll deal with the coverage gap later today.

I've pushed a commit that should take care of codecov.

Hi @anafalcao, just checking in.. I think I caught everything up.. can you confirm?

Hi @anafalcao, just checking in.. I think I caught everything up.. can you confirm?

Hey @mw-root , sorry for the late reply, it was a long weekend in my country. I'm back today and I'll review it now. Thank you very much for your proactivity in solving this quickly

No need to apologize @anafalcao, thanks again and I hope you had a good long weekend!


Fix FIDE rating sync

We are meant to sync ratings from FIDE early in every month.
Doesn't seem to be completely working, and at the least should be expanded to include all Irish players (not just those with a full classical rating).
Also we should sync these to the database for the www site as part of the existing ratings sync process.

resolved by https://github.com/irishchessunion/icu_ratings_app/pull/7


[SQL] Allow to specify size of null

What changes were proposed in this pull request?
Currently, the size() function returns -1 for nulls (array or map types). It restricts usage of results such as applying sum or avg for result column. In the PR, I propose to extend the size() function by option parameter sizeOfNull which allows to set size of nulls. By default, if sizeOfNull is not specified, -1 is used for backward compatibility.
How was this patch tested?
I added SQL tests for null arrays - size(null) and size(null, 0). Also added Scala and Python tests.

The default behavior size(null) == -1 is a big problem for several reasons:

It is inconsistent with how SQL functions handle nulls.
It is an extreme violation of the Principle of Least Astonishment (POLA)
It is not called out anywhere in the Spark docs or even the Hive docs.
It can lead to subtle bugs in analytics.

For example, we discovered this behavior while investigating post-click user engagement in our adtech system. The schema was per ad placement and post-click user engagements were in an array of structs. The culprit was df.groupBy('placementId).agg(sum(size('engagements)).as("engagement_count"), ...), which subtracted 1 for every click without post-click engagement. Luckily, the behavior led to negative engagement counts in some periods, which alerted us to the problem and this bizarre behavior.
For better or worse, Hive compatibility is necessary given Spark's history. That said, the number of future users of Spark far exceeds the number of current Spark users and dwarfs the Hive user base. This PR is necessary because Spark has to make the right design decisions for all its users, including future ones.
However, if we want to allow size(null) to return something other than -1, it is not obvious that the optional parameter specifying that should be of type Int. The most consistent behavior, ignoring the insanity that Hive created in the first place, is for size(null) to behave as length(null), which returns null. This handles the aggregation case with sum/avg, etc. The "return 0" use case can then be easily handled via coalesce/nvl. This would suggest that the optional parameter should be a Boolean, which also eliminates awkward situations like size(null, 123).
When we consider the goal of reducing the likelihood of current and future Spark uses getting bit by the unintuitive default behavior, two other changes are necessary:

A session setting that specifies the return value of size(null). When missing, it will be to return -1 for backward compatibility. This allows Spark users that do not have legacy code the relies on size(null) == -1, which is most Spark users, to fix this issue without changing code.

An explicit mention in the Spark SQL Programming Guide of this issue with a suggestion that, when there is no legacy code, the above setting should be used.


/rbd/pnpm-volume/2b8ad01c-af91-4e4c-abfb-00c65dbb274e/node_modules/.registry.npmjs.org/vscode-jsonrpc/8.0.0-next.7/node_modules/vscode-jsonrpc/lib/common/linkedMap.js:40         return this._head?.value;

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. Chrome, Safari]
Version [e.g. 58]

Smartphone (please complete the following information):

Device: [e.g. iPhone Xs]
OS: [e.g. iOS 14.7]
Browser [e.g. stock browser, Safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

what

I doesn't work lol

werks on my machine lel

What's your discord, so we can talk?

Check my bio


Maintenance mode and time zones problem

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
There's a problem with maintanance mode and time zones (I think). After a fresh startup it's working fine. Next day it's ignoring the maintanance window.

When I'm posting this issue it should be in maintanance mode but it's not.
👟 Reproduction steps
Add a new maintanance task with UTC+2
Wait for that time.
👀 Expected behavior
In maintanance mode it should stop sending me notifications. But it's not entering maintanance mode.
😓 Actual Behavior
While it's time to fire the maintanance mode it's not doing it.
🐻 Uptime-Kuma Version
1.23.0
💻 Operating System and Arch
Debian 10
🌐 Browser
Brave
🐋 Docker Version
Docker 24.0.2
🟩 NodeJS Version
No response
📝 Relevant log output
No response

What is the maintenance window set to exactly? (what options are you using?)
What exactly do you mean by Add a new maintanance task with UTC+2 (How did you add a maintanance period with a timezone? what is the timezone of the server, uptime-kuma and the maintanance period)
why do you think this is a timezone issue? (what timezone are you running in?)

What is the maintenance window set to exactly? (what options are you using?)

What exactly do you mean by Add a new maintanance task with UTC+2 (How did you add a maintanance period with a timezone? what is the timezone of the server, uptime-kuma and the maintanance period)
why do you think this is a timezone issue? (what timezone are you running in?)

It was working fine from the beginning but after some time I changed the time zone to my real time zone and then it stopped working. In the changelog I noticed that there was some fixes for the time zone so I thought that this could solve my problem but not.

Cannot reproduce with 16:00 to 18:00.

Hello @louislam ,
I tested it with :

and it started /stopped in less than 1 sec

server time is the same as the last screenshot

https://github.com/louislam/uptime-kuma/issues/4634 does cover the same issue and has a deeper reproduction why this issue happens => closing as a "duplicate" (despite this beig first)


Fix disable encryption feature

This PR was originally submitted by @monetidesc to fix https://github.com/ungoogled-software/ungoogled-chromium/issues/2751, https://github.com/ungoogled-software/ungoogled-chromium-windows/issues/326, and https://github.com/ungoogled-software/ungoogled-chromium/issues/2773

The changes looks good to me. Have you verified that the patch still applies and works as expected?

Is it possible for this option to decrypt the profile that has been encrypted rather than clearing the data?

It could be possible if there is a way to tell if the profile has been encrypted or not. I haven't looked into the encryption feature so I'm not sure about that.

The changes looks good to me. Have you verified that the patch still applies and works as intended?

It worked as of 123.0.6312.58. Considering there hasn't been a major version change since then, I would presume it still works as intended.


[BUG] Picking Album Name instead of Song Name

What steps will reproduce the bug?

Well sharing the code for merging the functions in a code block in this issue would not be optimal. I'm not sure we're on the same page here. Let me know if later on you want to save time and let me create the PR for merging the functions.
Will open a PR for this issue (Theme Color Extraction) in a few minutes.
On another note, is it just me or does the image sometimes contain the album name instead of the actual song name?

Album Name: Somewhere New
Link to Album:  https://open.spotify.com/album/2tYcFWd25g2Y69VQHiNJwD?si=kpz0IFRgQtqpKUDSKDXTew&dl_branch=1
Song Name: Your Spell
Link to Song: https://open.spotify.com/track/1y6ZCzDYOkXXq5o7BUkbIi?si=a1f1311e917542d9
ID: 1y6ZCzDYOkXXq5o7BUkbIi
Spotify Card for that image:

@delivey thanks for pointing it out. I will open an issue for it. If you wanna work on that too you are welcome 👍
Originally posted by @AmreshSinha in https://github.com/AmreshSinha/Spotify-Cards-API/issues/3#issuecomment-932681111
How often does it reproduce? Is there a required condition?
<--BUG-->
Is there any Package conflicting with other packages?
<--NO-->
What is the expected behavior?
It should have shown the song name instead of the parent album's name.
What do you see instead?
A card with its parent album's name.
Additional information
For info, you can test it out yourself by using the reproducing step.
Also,
Issue #3

Hey, can you assign me to this issue? Should be pretty easy.

Assigned You ✔


[bug] @veramo/key-manager - SES_IMPORT_REJECTED

I encountered this issue when using @veramo/key-manager.
[SyntaxError: Possible import expression rejected at :44396. (SES_IMPORT_REJECTED)]
Related issue: https://github.com/endojs/endo/blob/master/packages/ses/error-codes/SES_IMPORT_REJECTED.md
The issue is due to this.

The problem is that the function name is "import".

@mirceanis this does seem like it could be causing problems. I propose we just rename this function to importKey. Should we rename the other functions to match?

Correct. I'll work on it today so that we can include it in the upcoming 5.0 release


problem with graph_vgg16.cpp

I tried to test this graph on armv8 neon with library V24.02.1 with below build option
scons Werror=1 -j8 debug=1 neon=1 opencl=0 os=linux arch=armv8a
Weights and bias .npy is taken from the graph from the link mentioned in the .cpp file.
Provenance: www.robots.ox.ac.uk/~vgg/software/very_deep/caffe/VGG_ILSVRC_16_layers.caffemodel
case1:
Getting segmentation fault after run the graph

case2: If image option not provided getting other output with segmentation fault

Attached image and label file

Label_clsid.txt
Please let me know why segmentation fault is coming.

Also i wanted to know the runtime for these examples graphs on neon core.

Hi @RavikumarLav
Could you please share the full error log in a text format ?
What device are you using to run the example?
The graph examples only support images in the ppm format. You cannot pass a jpg image directly, you have to decompress the image and convert it to ppm.


fix erlotnib bug and pin erlotnib version

deployment is failing due to erlotnib.
Looks like its due to changes in erlotnib api. @DavAug, can you fix this and then pin the version of erlotnib used to a specific commit?
Traceback (most recent call last):
  File "/pkpdapp/pkpdapp/manage.py", line 29, in <module>
    main()
  File "/pkpdapp/pkpdapp/manage.py", line 25, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.6/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.6/site-packages/django/core/management/__init__.py", line 377, in execute
    django.setup()
  File "/usr/local/lib/python3.6/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/usr/local/lib/python3.6/site-packages/django/apps/registry.py", line 91, in populate
    app_config = AppConfig.create(entry)
  File "/usr/local/lib/python3.6/site-packages/django/apps/config.py", line 90, in create
    module = import_module(entry)
  File "/usr/local/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/pkpdapp/pkpdapp/simulate/apps.py", line 9, in <module>
    import erlotinib as erlo
  File "/usr/local/lib/python3.6/site-packages/erlotinib/__init__.py", line 8, in <module>
    from . import (  # noqa
  File "/usr/local/lib/python3.6/site-packages/erlotinib/apps/__init__.py", line 12, in <module>
    from ._simulation import (  # noqa
  File "/usr/local/lib/python3.6/site-packages/erlotinib/apps/_simulation.py", line 17, in <module>
    import erlotinib.apps as apps
AttributeError: module 'erlotinib' has no attribute 'apps'
The command '/bin/sh -c python /pkpdapp/pkpdapp/manage.py migrate --noinput' returned a non-zero code: 1

deployment is failing due to erlotnib.
Looks like its due to changes in erlotnib api. @DavAug, can you fix this and then pin the version of erlotnib used to a specific commit?
Traceback (most recent call last):
  File "/pkpdapp/pkpdapp/manage.py", line 29, in <module>
    main()
  File "/pkpdapp/pkpdapp/manage.py", line 25, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.6/site-packages/django/core/management/__init__.py", line 401, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.6/site-packages/django/core/management/__init__.py", line 377, in execute
    django.setup()
  File "/usr/local/lib/python3.6/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/usr/local/lib/python3.6/site-packages/django/apps/registry.py", line 91, in populate
    app_config = AppConfig.create(entry)
  File "/usr/local/lib/python3.6/site-packages/django/apps/config.py", line 90, in create
    module = import_module(entry)
  File "/usr/local/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 941, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 955, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 665, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/pkpdapp/pkpdapp/simulate/apps.py", line 9, in <module>
    import erlotinib as erlo
  File "/usr/local/lib/python3.6/site-packages/erlotinib/__init__.py", line 8, in <module>
    from . import (  # noqa
  File "/usr/local/lib/python3.6/site-packages/erlotinib/apps/__init__.py", line 12, in <module>
    from ._simulation import (  # noqa
  File "/usr/local/lib/python3.6/site-packages/erlotinib/apps/_simulation.py", line 17, in <module>
    import erlotinib.apps as apps
AttributeError: module 'erlotinib' has no attribute 'apps'
The command '/bin/sh -c python /pkpdapp/pkpdapp/manage.py migrate --noinput' returned a non-zero code: 1

That is very weird! It runs on my machine, even when I reinstalled the pkpdapp and erlotinib in my venv again...

And the apps module is definitely there? Is it possible that this is a problem with python 3.6, i.e. the module import handling? I am using python 3.7

Fixed via #96


Allow the conversion interface to discover pandas indexes

Allows referencing pandas indexes in .to interface even when Dataset does not declare them as dimensions.

[x] Fixes https://github.com/ioam/holoviews/issues/2717

Ready to merge.

Other than one question in the comment above, this looks fine and I'm happy to merge once you respond.


[WIP] SessionState hiveclient to be used for all the sql's run on hive metastore

Changed code to use hiveClient available in HiveSessionState. This handler is user specific hive client.

Can one of the admins verify this patch?

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2201/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/68/

LGTM


Fixes #6049

Updates the query for bridgehead atoms to resolve the issue.
This also impacts the NumBridgeheadAtoms descriptors, so the version on that gets bumped as well.

@ricrogz sorry to tag you again, but since this one is from @cdvonbargen I thought you'd want to review it

@ricrogz I agree that your description of the conditions for something to be a bridgehead seems correct. and that it is a bit simpler to explain. It's likely also easier to implement. I'll take a look at changing the implementation (and documentation) to this formulation.

Of course it's never that easy. Here's the counter-example for that formulation (this is one of the edge cases in the tests and it broke a number of my previous attempts

Looking around the N: bond pairs (4,6) and (3,6) share a ring, but (3,4) do not (since the six-ring isn't part of the set of small rings)

Ah, right. I didn't take SSSRs into account.

Roger, I expect, would just do a bfs search for # rings a particular atom
is in as opposed to finding all rings.
i.e. something like
int getNonSSSRRingCount(const Atom *atom);

Cheers,
Brian
On Thu, Feb 9, 2023 at 8:47 AM David Cosgrove @.***>
wrote:

I wonder if it would be worth having a method for producing all rings
alongside the SSSR. The SSSR result not being a complete set of rings
caused me problems this week as well. If it weren't for the prospect of
being pilloried by R. Sayle at a future UGM, I would volunteer to write
one.
On Thu, Feb 9, 2023 at 1:43 PM Ric @.***> wrote:

Ah, right. I didn't take SSSRs into account.
—
Reply to this email directly, view it on GitHub
https://github.com/rdkit/rdkit/pull/6061#issuecomment-1424211678, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACGF2FWRATVSUMTCQXWVM2TWWTYBFANCNFSM6AAAAAAUTRI2VI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
David Cosgrove
Freelance computational chemistry and chemoinformatics developer
http://cozchemix.co.uk
—
Reply to this email directly, view it on GitHub
https://github.com/rdkit/rdkit/pull/6061#issuecomment-1424215969, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLOFIHKJZLI4IVJ7RXKEWTWWTYOPANCNFSM6AAAAAAUTRI2VI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>