Add edit task command

#136

Codecov Report
Base: 47.71% // Head: 46.52% // Decreases project coverage by -1.19% :warning:

Coverage data is based on head (ebda050) compared to base (0c5866b).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #151      +/-   ##
============================================
- Coverage     47.71%   46.52%   -1.20%
  Complexity      413      413
============================================
  Files           104      106       +2
  Lines          1949     1999      +50
  Branches        214      220       +6
============================================
  Hits            930      930
- Misses          974     1024      +50
  Partials         45       45

Impacted Files
Coverage Δ

.../seedu/address/logic/commands/TaskEditCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
59.25% <0.00%> (-2.28%)
:arrow_down:

...du/address/logic/parser/TaskEditCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/AddressBook.java
51.78% <0.00%> (-1.92%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
42.51% <0.00%> (-1.75%)
:arrow_down:

src/main/java/seedu/address/model/task/Task.java
0.00% <0.00%> (ø)

.../java/seedu/address/model/task/UniqueTaskList.java
8.62% <0.00%> (-0.31%)
:arrow_down:

src/main/java/seedu/address/model/team/Team.java
0.00% <0.00%> (ø)

.../java/seedu/address/model/team/UniqueTeamList.java
20.00% <0.00%> (-0.84%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Implement lazy evaluation for DynamicMap methods

After a lengthy discussion on how to work with very large datasets with @jlstevens, we realized that DynamicMap callbacks and the lazy evaluation they allow gets us most of the way there but there's some crucial things missing for usability. In particular the methods on DynamicMap are all inherited from HoloMap, which means they work on the assumption that all the data is already loaded. This means that currently when using a method on a DynamicMap it will apply the operation only to the elements that's already loaded, which will in the best case do nothing but will usually result in exceptions when the next Element is loaded.
What I propose is to subclass the methods on DynamicMap to generate closures, which will apply the requested operation when a new key is requested. This will allow slicing, sampling, reducing, aggregating and overlaying operations to be applied at runtime. Even operations could trivially be modified to simply define a closure when passed a DynamicMap.
This will be a huge improvement for usability and make exploration and analysis of large datasets incredibly easy. The user could specify the data transformations to be applied directly on the object rather than having to write complex callbacks.
Here's a small toy example:
Say we have an array of temperatures on disk which is indexed by Date x Atmospheric Height x Latitude x Longitude.
Currently required implementation:
def get_contours(key):
    array, metadata = load_data_from disk(key)
    img = hv.Image(array, **metadata).select(Latitude=(0, 30), Longitude=(50, 80))
    return hv.contours(img)

def get_curve(key):
    array, metadata = load_data_from disk(key)
    return hv.Image(array, **metadata).reduce(Latitude=np.mean)

kdims = [hv.Dimension('Date', ...), hv.Dimension('Atmospheric Height', ...)]
contours = hv.DynamicMap(get_contours, kdims=kdims)
curve = hv.DynamicMap(get_curve, kdims=kdims)
contours + curve

With lazy evaluation:
def get_data(key):
    array, metadata = load_data_from disk(key)
    return hv.Image(array, **metadata)

dmap = hv.DynamicMap(get_data, kdims=[hv.Dimension('Date', ...),
                     hv.Dimension('Atmospheric Height', ...)])

hv.contours(dmap.select(Latitude=(0, 30), Longitude=(50, 80))) +\
dmap.reduce(Latitude=np.mean)

The lazy evaluation example really demonstrates the strengths of this approach, you can define the data loading operation once and then interactively explore the whole dataset without having to define more and more functions. I hope this gets the idea across, I think this is relatively little work for a pretty huge payoff, this would open up a lot of the functionality in HoloViews up to much, much larger datasets.

Sounds like an excellent idea to me.  How will DynamicMap know that get_data returns an Image?  Or does it not need to know that, as it will simply apply whatever operations are requested onto whatever it returns?  And if so, does that mean that get_data can dynamically return different Element types?

How will DynamicMap know that get_data returns an Image? Or does it not need to know that, as it will simply apply whatever operations are requested onto whatever it returns?

Correct, it simply wouldn't care, if the operation you applied doesn't work for the return type it will complain at evaluation.

And if so, does that mean that get_data can dynamically return different Element types?

No, get_data has to consistently return the same Element type because the plotting code needs to initialize the appropriate artist/glyph.

Having outlined the idea there are various methods that are problematic because they can't easily be lazily evaluated:

range at least could be made to work by working in chunks and with some caching, which would allow for our regular normalization options to be applied.

dimension_values for the value dimension would mean returning a 1D array of literally all the datapoints, which isn't really desirable.

groupby is a tough one but could be made to work if we had proper support for constant dimensions, which would also allow you to explode a DynamicMap into a GridSpace or NdLayout of DynamicMaps using .layout and .grid.

collapse is also tough because it would require more chunking so that only a certain amount of the data is in memory at the same time.

Everything else operates on individual elements and should thus be relatively easy to support.

Just a quick note that .relabel with depth > 1  should also lazily evaluate on generated Elements.

As a summary, these are the methods that now support lazy evaluation:
[x] - call
[x] - mul
[x] - groupby
[x] - overlay, grid, layout
Methods that aren't yet implemented:
[ ] - __getitem__/select (deep slicing and deep select)
[ ] - collapse
[ ] - reduce/aggregate
[ ] - sample
[ ] - map
Methods that will never be implemented:

collate
dframe
table
dimension_values for non-key dimensions

Methods that could be implemented but are currently required to be non-lazy:

split_overlay

I would prefer it if these methods that cannot be made lazy simply didn't exist on DynamicMap. Even if they are made to always raise NotImplementedError, the fact they are defined and tab-complete but don't actually work is rather annoying.
Sadly, I don't know a good way of 'undefining' inherited methods (see this stackoverflow post) so the only suggestion is that these methods are introduced as a mixin class for HoloMap only, with a new intermediate class for HoloMap and DynamicMap.
The prospect of more classes isn't one I am particularly pleased with but I do think we should try to solve the issue of these dud methods somehow.

A shared base class seems like the clean way to do this.  Undefining those methods will completely violate the meaning of class inheritance, which is really not a good idea---the subclass then no longer satisfies "is a", making it very hard to reason about.
Coming up with a name for the shared base class seems like the trickiest part, as I don't know what defines the set of things that make sense for both HoloMaps and DynamicMaps.  But having such a name will be useful, because there are already a variety of cases where we need to refer to both of them without making a distinction.  "Map" is the clearly shared bit, but doesn't mean much (which makes both of those names slightly dubious already).

A shared base class seems like the clean way to do this.

Strongly agree, we decided on ViewableMap as the baseclass.

That name sounds reasonable, though of course a Curve is also a viewable map (from x to y).  Can't think of anything better.

We already have a ViewableElement base class of Element which is why I suggested ViewableMap - this scheme seems fairly consistent to me.

Sure.
Fundamentally, what all three of these classes share is that they (a) contain at least one additional value dimension beyond those mapped on to the screen at any one instant, and (b) specifically allow those other dimension(s) to be selected or iterated over or animated to choose the one specific value to be displayed at that instant.  I.e., for dimensions n>s, dimensions 1 to s are mapped onto the screen in some way, and dimensions s+1 to n are selected in some way, whether by widgets or iteration.
So ViewableMap is a bit backwards, since compared to a regular Element, what's different about these classes is that they have mappings that are not viewable at any one time, only if iterated over. So IterableMap or SelectableMap or PlayableMap would also be true.  Anyway, I don't much like any of these names; nothing is really all that accurate or compelling, mainly because the Map bit is not actually the thing that should be shared between all three, as that's not the distinguishing bit compared to Elements or other Containers.

But of course, it's too late to change HoloMap and DynamicMap...

ViewableMap should be read as "Map of Viewables" not as "Map that is Viewable".

That would work if Elements were called Viewables, but they aren't.

They are Viewable types.

Twice I was about to echo Philipp's reply but he beat me to it ;-p

I don't know what you mean; there doesn't seem to be anything called Viewable in the hv source code, apart from one comment:
~/holoviews/holoviews> \grep Viewable *.py */*.py | \grep -v ViewableElement
plotting/renderer.py:        Given a HoloViews Viewable return a corresponding plot instance.

Right, the actual name is ViewableElement.

So we are meant to read ViewableMap as "a map of ViewableElements"?  Then it all makes sense, but I have to say that it's quite a stretch!  You didn't like ElementMap, in that case?

Well, you don't often use ViewableElement anywhere in the documentation, so I would think you could change it to Viewable now.  And then one could replace many of the occurrences of Element in the tutorials to say the base class Viewable instead, i.e. anything that could either be an Element or an Overlay.  At that point, ViewableMap would make sense (though still awkward due to the grammatically surprising use of Viewable as a noun (a la Smalltalk)).
That said, seems to me that a Layout is also viewable, but is not a ViewableElement, which argues for keeping the current name ViewableElement.  The parent class of HoloMap and Dynamic map could then even be ViewableElementMap; it's not a name we need all that often, and it's more obvious how to parse that.

Sure, for me the more descriptive the better, this class should definitely not make it beyond the namespace of the core.spaces module.

For code purposes, that's true, but for the documentation, we do need a term that unifies both HoloMap and DynamicMap, and ViewableElementMap would then be that term.  Still, even though it's clunky, it probably helps explanations be clearer, because when we first introduce it we can explain why it's called that, and that there are two subtypes, which should actually help people understand HoloMap and DynamicMap better.

I agree ViewableElementMap is ugly and descriptive and fine for coding purposes but I'm not sure I would be happy seeing that name leak into our user docs.
The term I would be happy to use to describe both HoloMap and DynamicMap is NdMapping without mentioning ViewableElementMap. We can still use the term 'viewable' if we like without mentioning class names users really don't need to know about...

But NdMapping doesn't convey anything about what is mapped.  It's fine to mention that these are both NdMappings, but it doesn't help explain how they relate to Elements or to other containers, it just explains how they do mapping.
ViewableElementMap does explain what is mapped.  There are ViewableElements, which are things that can be viewed in a single screen-based coordinate system at a single time, of which there are many types (Curve, Image,  etc., and various kinds of Overlays thereof). Then there are ViewableElementMaps, which are collections of ViewableElements that select one coordinate from these mapped extra dimensions to display at any one time.  ViewableElements and ViewableElementMaps share much in common -- both have one screen-based coordinate system, both take up the same space on the screen, and both can be used interchangeably in a Layout, but one is fully rendered on screen, and what's rendered onscreen for the other is just one particular coordinate's data at any one time, from some larger mapping.
To me this makes much more sense than our current documentation, where it's nearly impossible to see or remember how HoloMaps relate to Elements.  Not being as deeply involved in the code and principles as you two, I think I'm actually more trustworthy in this particular case than you two -- I know very clearly what's confusing, whereas you got over your confusion years ago. :-)

So I had a go implementing the final outstanding dynamic method, map and ran into trouble, if I allow a map that applies both to the DynamicMap and its items I ended up with infinitely nested DynamicMaps (well up to the max_recursion limit), here's my current (slightly incorrect) implementation to avoid this:
    def map(self, map_fn, specs=None, clone=True):
        """
        Recursively replaces elements using a map function when the
        specification applies.
        """
        if specs and not isinstance(specs, list): specs = [specs]
        applies = specs is None or any(self.matches(spec) for spec in specs)

        deep_mapped = self.clone() if clone else self
        if applies:
            return map_fn(deep_mapped)
        for k, v in self.items():
            deep_mapped[k] = v.map(map_fn, specs, clone)

        from ..util import Dynamic
        def dynamic_map(obj):
            return obj.map(map_fn, specs, clone)
        return Dynamic(deep_mapped, shared_data=True, operation=dynamic_map)

Will need to think about it more.

I've just done a final review of all the DynamicMap methods, and I think we're very close. Once #1240 is merged I believe all methods that can be made dynamic have been implemented that way. I also don't mind retaining the list of non-dynamic methods that apply to the cache. However there are three methods which absolutely cannot be support, which are the top three in the list below. My suggestion would simply raise NotImplementedErrors for those cases.
# Should be disabled
add_dimension
drop_dimension
reindex

# General methods
dimensions
get
get_dimension
get_dimension_index
get_dimension_type
get_param_values
matches
pprint
params
warning

# Applies to cache (i.e. not dynamic)
collapse
dframe
dimension_values
items
info
keys
reduce
sample
split_overlays
table
traverse
update
values

# Dynamically implemented
__call__
__mul__
__getitem__
collate
grid
groupby
hist
layout
map
overlay
redim
relabel
select

# DynamicMap method
event
reset

One final thing I need to do is ensure that the __call__ is working correctly as I've encountered some weird interactions with it.

Did there not end up being a shared base class so that these methods could truly just not be defined?

Did there not end up being a shared base class so that these methods could truly just not be defined?

These methods are defined all the way down at the MultiDimensionalMapping level, so I don't think there's a clean way not to define them.

Ok, then...

Since merging https://github.com/ioam/holoviews/pull/1262 this is now finally complete.


Implement v4.1 of bot API

Telegram launched their v4.1 bot API as of today:
https://core.telegram.org/bots/api-changelog#august-27-2018
List of changes:

Added support for Telegram Passport. See the official announcement on the blog and the manual for details.
Added support for editing the media content of messages: added the method editMessageMedia and new types inputMediaAnimation, inputMediaAudio, and inputMediaDocument.
Added the field thumb to the Audio object to contain the thumbnail of the album cover to which the music file belongs.
Added support for attaching custom thumbnails to uploaded files. For animations, audios, videos and video notes, which are less than 10 MB in size, thumbnails are generated automatically.
tg:// URLs now can be used in inline keyboard url buttons and text_link message entities.
Added the method sendAnimation, which can be used instead of sendDocument to send animations, specifying their duration, width and height.
Added the field animation to the Message object. For backward compatibility, when this field is set, the document field will be also set.
Added two new MessageEntity types: cashtag and phone_number.
Added support for Foursquare venues: added the new field foursquare_type to the objects Venue, InlineQueryResultVenue and InputVenueMessageContent, and the parameter foursquare_type to the sendVenue method.
You can now create inline mentions of users, who have pressed your bot's callback buttons.
You can now use the Retry-After response header to configure the delay after which the Bot API will retry the request after an unsuccessful response from a webhook.
If a webhook returns the HTTP error 410 Gone for all requests for more than 23 hours successively, it can be automatically removed.
Added vCard support when sharing contacts: added the field vcard to the objects Contact, InlineQueryResultContact, InputContactMessageContent and the method sendContact.

Most of the API changes are already implemented in branch v2, however, the new Passport platform is HUGE, so it will take a bit more time to implement all of it.

Done items:

Added support for editing the media content of messages: added the method editMessageMedia and new types inputMediaAnimation, inputMediaAudio, and inputMediaDocument.
Added the field thumb to the Audio object to contain the thumbnail of the album cover to which the music file belongs.
Added support for attaching custom thumbnails to uploaded files. For animations, audios, videos and video notes, which are less than 10 MB in size, thumbnails are generated automatically.
Added the method sendAnimation, which can be used instead of sendDocument to send animations, specifying their duration, width and height.
Added the field animation to the Message object. For backward compatibility, when this field is set, the document field will be also set.
Added two new MessageEntity types: cashtag and phone_number.
Added support for Foursquare venues: added the new field foursquare_type to the objects Venue, InlineQueryResultVenue and InputVenueMessageContent, and the parameter foursquare_type to the sendVenue method.
Added vCard support when sharing contacts: added the field vcard to the objects Contact, InlineQueryResultContact, InputContactMessageContent and the method sendContact.

WIP:

Added support for Telegram Passport. See the official announcement on the blog and the manual for details.

I am closing this ticket because I made a new ticket for the Passport stuff: #97


Definition of "Limited Information"?

More information about what limited reporting information is provided for both the "Azure AD Premium P1" and "Azure AD Basic/Free" editions?
If this is described elsewhere then links to the relevant articles would be appreciated.
As it stands currently, it is very ambiguous about what you could see on these plans

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

ID: c2e83171-fd05-9fe1-3ba7-0fb20161ee63
Version Independent ID: 2e6b2451-5ff8-a277-90ea-49aa70957977
Content: What is Azure Active Directory Identity Protection?
Content Source: articles/active-directory/identity-protection/overview-identity-protection.md
Service: active-directory
Sub-service: identity-protection
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Jared-Harrington-Gibbs Thank you for your query. We will investigate this and update further.

@MicrosoftGuyJFlo Could you please review this one and see if we can add a detailed section on the reporting data available depending upon the licenses?
@Jared-Harrington-Gibbs We are assigning this to the content authors for review and this thread will be updated accordingly.

You wont get all of the details of what kind of risk was detected and why. You will see information related to the following other risk detections... https://docs.microsoft.com/azure/active-directory/identity-protection/concept-identity-protection-risks#other-risk-detections

Hey @MicrosoftGuyJFlo, Thanks for the clarification. I'll try to update the documentation and submit a pull request whenever I have some time on my hands.

#please-close


Django 1.9 compatibility - importlib

#5 fix

Looks good to me, didn't realize people were creating PRs on this repo!  Merging.


Android andronix & debian illegal instruction

I got Motorola snapdragon with andronix debian( Linux)
Used your guide to get latest version but it says ' illegal instruction" I guess it's cos CPU.  At compile should Include arm? Arc.
I'm no sw expert to compile this by myself and the linuxbrew denied co/operation would do it by myself..

The releases are targeting a lot of architectures. Have you tried this specific debian package?
https://github.com/SpatiumPortae/portal/releases/download/v1.2.3/portal_1.2.3_Linux_arm64.deb

[Detected OS: linux_amd64]

It seems like the OS detection is not doing a great job of detecting your OS, as you say you're running Debian. As you can see, the file fetched is and installed is the following
https://github.com/SpatiumPortae/portal/releases/download/v1.2.3/portal_1.2.3_Linux_x86_64.tar.gz
What I suggest for you to get a working version going is to use this binary instead, as it is the correct one for your OS: https://github.com/SpatiumPortae/portal/releases/download/v1.2.3/portal_1.2.3_Linux_arm64.deb

Yes though so that the CPU architecture was the problem because the error message.
. Just followed the instructions, it is modified version from Google play.
Thanks for the arm link trying that.


syntax error at or near "desc"

Hi!
Why I try to use ormx with my model:
#[derive(Debug, Serialize, Deserialize, FromRow, Encode, Eq, PartialEq, Clone, ormx::Table)]
#[ormx(table = "users.list", id = id, insertable)]
pub struct User {
    #[serde(default)]
    pub id: i32,
    pub login: Option<String>,
    // #[serde(skip)]
    pub password: Option<String>,
    pub name: Option<String>,
    pub subname: Option<String>,
    pub phone: Option<String>,
    pub desc: Option<String>,
    pub access_card: Option<i32>,
    pub owner_id: Option<i32>,
    pub register_date: Option<chrono::NaiveDateTime>,
}

I got an error:
error: error returned from database: syntax error at or near "desc"

I think the problem is that you don't escape columns names and desc is a keyword in PostgreSQL :/.
I think this should be easy to fix?

Yes, column names are currently not escaped.
I'll escape them when I detect a keyword, thanks for raising the issue!

Fixed now. ormx still emits a warning when a reserved identifier is used since there are some things to keep in mind when using reserved keywords as identifier

Nice! Thank you :)


[Event Request] page 344 "Navigat" - Event for procedure SetSourceForPurchase

Can we get the following event for the "Navigat" page: OnAfterSetSourceForPurchase
    local procedure SetSourceForPurchase()
    begin
        if NoOfRecords(DATABASE::"Vendor Ledger Entry") = 1 then begin
            VendLedgEntry.FindFirst;
            SetSource(
              VendLedgEntry."Posting Date", Format(VendLedgEntry."Document Type"), VendLedgEntry."Document No.",
              2, VendLedgEntry."Vendor No.");
        end;

        […]

        if NoOfRecords(DATABASE::"Issued Deliv. Reminder Header") = 1 then begin
            IssuedDeliveryReminderHeader.FindFirst;
            SetSource(
                IssuedDeliveryReminderHeader."Posting Date", Format("Table Name"), IssuedDeliveryReminderHeader."No.",
                2, IssuedDeliveryReminderHeader."Vendor No.");
        end;

        //New Start
        OnAfterSetSourceForPurchase();
        //New End
    end;

    [IntegrationEvent(true, false)]
    local procedure OnAfterSetSourceForPurchase()
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


OSF-4898 Anonymize public project if user is sharing that project through an anonymized link

Purpose
Anonymize public project in view-only link.
Changes
Delete lines in model.py that says "if a node is public, it is not anonymous"
Make the same change to the template file.

Unit tests are failing; please address, then reassign to me.


Central Package Management is not supported in Visual Studio 17.2 or .NET SDK 6.0.300

This documentation states that:

Starting with NuGet 6.2, you can centrally manage your dependencies in your projects with the addition of a Directory.Packages.props file and an MSBuild property.
The feature is available across all NuGet integrated tooling.

Visual Studio 2022 17.2 and later
.NET SDK 6.0.300 and later

This is very misleading.
You can restore packages using Central Package Management in Visual Studio 17.2 and .NET SDK 6.0.300, but you cannot install new packages, which means that you cannot manage your dependencies.
See here for confirmation: https://github.com/NuGet/Home/issues/11826#issuecomment-1147688099
Installing a package in Visual Studio results in this error:
NU1008: Projects that use central package version management should not define the version on the PackageReference items but on the PackageVersion items: MyPackage.
Package restore failed. Rolling back package changes for 'MyProject'.

Installing a package using the .NET CLI results in this error:
error: NU1008: Projects that use central package version management should not define the version on the PackageReference items but on the PackageVersion items: MyPackage.
error: Value cannot be null. (Parameter 'version')

Please change the documentation so that it is clear that Visual Studio 17.2 and the .NET SDK 6.0.300 do not support Central Package Management.

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

ID: 2d000f01-60a3-59c2-e06d-c02c8721fa76
Version Independent ID: 2d000f01-60a3-59c2-e06d-c02c8721fa76
Content: Central Package Management
Content Source: docs/consume-packages/Central-Package-Management.md
Product: nuget
GitHub Login: @JonDouglas
Microsoft Alias: jodou

Hi @reduckted,
Apologies if this is misleading.
While we are working on support to complete the tooling stories for both of these experiences today (the project system in VS & .NET CLI), we still want to make it clear to users as to what version they can expect the CPM feature with restore support to work under.
There's a note at the bottom today letting people know that this feature currently doesn't have full Visual Studio or .NET CLI support such as the management of packages. We can probably put that note up higher in the docs and be more explicit.
Given these features are actively in development, we will update the documentation with these scenarios as they release. So we'll keep this issue open for now to collect more feedback.
https://github.com/NuGet/Home/pull/11849 (dotnet CLI)
https://github.com/NuGet/Home/issues/11826 (project system support)

@JonDouglas We can probably put that note up higher in the docs and be more explicit.

Yes, it really should be directly underneath "The feature is available across all NuGet integrated tooling" instead of "hidden" under the "Warning when using multiple package sources" section.

@JonDouglas we still want to make it clear to users as to what version they can expect the CPM feature with restore support to work under.

That's fair enough, but the problem is there's no mention that those listed tools only support restoring. The way I interpret the documentation is that CPM is fully supported on those tools.

Starting with NuGet 6.2, you can centrally manage your dependencies in your projects with the addition of a Directory.Packages.props file and an MSBuild property.
The feature is available across all NuGet integrated tooling.

I think that section really needs to be clear that only restoring is currently supported by those tools.


[Docs] Translate docs/core-concepts/yurt-controller-manager.md to EN

The docs link you want to translate
zh: https://github.com/openyurtio/openyurt.io/blob/master/i18n/zh/docusaurus-plugin-content-docs/version-v0.6.0/core-concepts/yurt-controller-manager.md
Target language
en: https://github.com/openyurtio/openyurt.io/blob/master/versioned_docs/version-v0.6.0/core-concepts/yurt-controller-manager.md
Note: The images in the documentation are stored at https://github.com/openyurtio/openyurt.io/tree/master/static/img/docs/core-concepts
If you have any questions, please refer to [README.md].

Since version 0.6.0 is the same as the current documentation, the following article also needs to be translated.
The docs link you want to translate
zh: https://github.com/openyurtio/openyurt.io/blob/master/i18n/zh/docusaurus-plugin-content-docs/current/core-concepts/yurt-controller-manager.md
Target language
en: https://github.com/openyurtio/openyurt.io/blob/master/docs/core-concepts/yurt-controller-manager.md

/kind documentation

/pinned

yurt-controller-manager doc translation is fixed by https://github.com/openyurtio/openyurt.io/pull/127


chore(docs): add roadmap

I think this would be useful to guide and prioritize larger new features that should be added to the notebook service.
Let me know what you think. What should be added or removed? Should things be prioritized differently.
I have tried to just focus on bigger new features rather than fixes, maintenance and similar smaller features that extend existing functionality.

It is easiest if you look at the rendered reamde file here: https://github.com/SwissDataScienceCenter/renku-notebooks/blob/chore-add-roadmap/README.md

p.s. I converted the Gant chart to a normal list. It was pretty but not very useful.

I will most likely close this PR and move these into a separate page in the renku-python github project where we have the Backlog, sprint, etc.

Add a page in our github backlog/sprint planning project see https://github.com/orgs/SwissDataScienceCenter/projects/45/views/7


ARROW-10839: [Rust] [Data Fusion] Implement BETWEEN operator

3 of the 22 TPC-H queries use the BETWEEN operator which is syntactic sugar for {value} >= {low} AND {value} <= {high}
This PR implements the BETWEEN operator by rewriting it into the two binary components. I am not sure if this is the correct approach but logically it feels sensible.
I need some help identifying where the tests can be improved.

This looks great. Thanks @seddonm1. Could you resolve the conflicts?

should be resolved

@seddonm1 maybe something went wrong with merging? The changes also show some unrelated changes?

FWIW, it is generally easier to rebase instead of merge in this project, as we use a linear history.

sorry i am not sure what happened but I have had to do a force push to un-ruin the commit history.

@seddonm1 could you rebase again?

rebased.

rebased.

Thanks @jorgecarleitao . more incoming.


[tests] model zoo add torchaudio models

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ ] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

Part of #3049
📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

Add torchaudio models to testing model zoo.
💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[x] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

Can you update the test files in tests/test_fx/test_tracer/test_torchaudio_model?

DONE


Add footer component

Before building this, we need to evaluate how this will render on pages without a side nav, since all pages need the footer.

Something like this I think @jeanservaas
https://www.ibm.com/able/

Or this site also is full width with the footer, https://www.ibm.com/design/racial-equity-in-design/
Looks like the IBM logo just jumps over to the first column, the rest stays the same?


syntactic sugar for simple for-all assignment

Just a suggestion based on my experience in the lsm exercise.
The model below does a very simple assignment but is quite verbose because the relation traversal contains two lists.
        for instance in network_instance.protocols.ospf.instance:
            for area in instance.area:
                area.interface += srospfarea::Interface(
                    interface_name=interface_name
                )
            end
        end

If we were to add some syntactic sugar for an inline for-all, this could be simplified to
network_instance.protocols.ospf.instance[].area[].interface += srospfarea::Interface(
    interface_name=interface_name
)

The syntax here is just an intermediate proposal, no real thought went into it. Formally, it would only be allowed on the lhs, and xs[].y = z would expand to for x in xs: x.y = z.

Discussion thread here

Conclusion: scepticism and no hard use cases.


select with relation return err

By following the has one relationship example from here

// model
type User struct {
	ID int64 `json:"id,string" bun:",pk"`
	Profile *UserProfile `json:"profile" bun:"rel:has-one,join:id=user_id"`
}

type UserProfile struct {
	ID     int64 `json:"id,string" bun:",pk"`
	UserID int64 `json:"userId"`
}

//selection

user := &User{}
err := DBM.NewSelect().Model(user).Relation("Profile").Where("id = ?", 1).Scan(context.Background())

//debug print sql

SELECT "user"."id", "profile"."id" AS "profile__id", "profile"."user_id" AS "profile__user_id" FROM "users" AS "user" LEFT JOIN "user_profiles" AS "profile" ON ("profile"."user_id" = "user"."id") WHERE (id = 1)

the postgres db return err ERROR: column reference \"id\" is ambiguous (SQLSTATE=42702)

See https://bun.uptrace.dev/guide/placeholders.html#in
use code
DBM.NewSelect().Model(user).Relation("Profile").Where("?TableName.id = ?", 1).Scan(context.Background())

You should fix your query as Oleksandr suggested.

Fair to say that the doc on the website provides an example with the same error

It also seems that correct fix is not  ?TableName, but ?TableAlias.

@Emptyfruit is right. it should be ?TableAlias


Moved drain_cap module to processing and changed import line

Should work now?

Moved drain_cap module to processing, but plotting would work too I guess. It's processing for plotting ;)


Sketch: Top level mock wrapper

A top-level mock wrapper seems like something that would be very useful.
https://github.com/dbrgn/embedded-hal-mock/issues/19
I've come up with something that works and have been using my own project to drive its implementation.  https://github.com/tommy-gilligan/ab1024-ega/blob/main/src/test.rs (the tests here are a bit verbose but that is a different matter entirely)
^^ Emphasis on works because it is quite messy. This is just a sketch. I've taken a few shortcuts (deleted eh0, hard-coded SPI to u8 only, made some fields public).
Some goals I've aimed for here:

keep existing API working and just have top level wrapper exist as extra functionality
rely on existing infrastructure ie. Generic

IMHO the main ugliness here is that now there are multiple iterators involved for a single Mock.  A Mock has its own iterator which it will use unless the Mock was created from a top level Hal Mock, in which case it uses that by reference.  I guess this comes from keeping existing API working but trying to keep code duplication to a minimum but there are probably better ways of achieving this.
All in all, this is a direction towards having a top-level mock wrapper.  I'm not sure it's the right direction but hope it helps in working out what that is.  Happy to complete the implementation of a top-level mock wrapper but only once direction is decided.  As such, love some feedback here.
Out of scope:

check identity of resources (if an expectation is set on pin A, it still succeeds if it is pin B that gets used instead)

Thanks, it's an interesting experiment.
I think there are two aspects to a top-level wrapper:

There's a single API for everything.

While this may be nice, I don't think the benefit outweighs the complexity increase in the implementation. Right now every mock is independent, which makes things easier to maintain. Mocks can re-use the generic mock, but if an API is mocked where this doesn't make sense, it can be implemented differently. With a unified transaction API, we're kinda forced to make everything compatible with the top-level API (or vice versa).

Transactions for multiple subsystems can be asserted in one go

This one is cool! However, it does add more complexity to this project. I'm not yet sure whether this is a good tradeoff or not, although I like the API.
I think right now, as you said, the complexity comes from the fact that both APIs are retained. As long as there is a clear and easy migration path, I think breaking changes are still possible and legitimate. How would an API / implementation look like if we give up backwards compatibilty?
Maybe @newAM or @ryankurte have an opinion as well?

1. There's a single API for everything.

While this may be nice, I don't think the benefit outweighs the complexity increase in the implementation. Right now every mock is independent, which makes things easier to maintain. Mocks can re-use the generic mock, but if an API is mocked where this doesn't make sense, it can be implemented differently. With a unified transaction API, we're kinda forced to make everything compatible with the top-level API (or vice versa).

I think that this could be a move in a generally good direction. Maybe this shouldn't come as a single PR, but I think if this is implemented carefully, overall benefits could be great.
We could start with a "top level" traits and make all impls compatible with it. This would also mean reviewing all existing code, possibly reducing code duplication between modules and standardizing some patterns. There are certainly big differences between modules, even though they usually operate similarly. This refactor could be a great first step.
An added benefit is that it could be easier for new developers to understand the code structure and not wonder why each module does the same thing differently (speaking from experience).
Also, an existence of a defined public trait would potentially add a way to provide user defined extensions, that seamlessly integrate with other mocks.
I believe that only when all modules are refactored, then a "top level" mock can be implemented, but that would unlock very powerful use cases for this library, e.g. expecting sequences of "transaction-delay-transaction" or "transaction-action-transaction".

Maybe @newAM or @ryankurte have an opinion as well?

It's definitely cool and I can see the usecase for it.  The complexity of the added code will depend on how ordering is handled.

The top level needs to independent mutable structures that implement the HAL traits.
Something needs to check that the independent structs are expected in the correct order.

I don't yet have a good idea of how complex it will be in the end.  There might be a path to simplifying the code by having a single type that implements all the HAL traits, or that might just be a hopeful wish :thinking:

A top-level HAL mock would also mean PRs like #119 may be a relic, as a sufficient implementation could lend itself to implementing new HAL features across all traits simultaneously.

A top-level HAL mock would also mean PRs like #119 may be a relic, as a sufficient implementation could lend itself to implementing new HAL features across all traits simultaneously.

Not really. Even if you have a single wrapper type, you'd still need to impl all methods.
One way to simplify that would be a macro that implements every async method by calling the sync method. However, I doubt that it's worth the complexity. And it would also be applicable to a "every mock is independent" approach, not just to a unified API.


fix: supervisord should not bring down the whole web container when non-essential service fails, fixes #5358

The Issue
When a command that runs by the Supervisor gets an error or crash, it automatically turns off the web container. Making the supervisor auto restart feature useless.
How This PR Solves The Issue
This PR check when there's an error happen in supervisord whether nginx or apache is working properly or not. if one of them is down, kill supervisord
Manual Testing Instructions
I cannot test this on real use case because I need the file to run when supervisord start. please guide me how to test this in my env.
Automated Testing Overview
I cannot test this because of the same reason above.
Related Issue Link(s)
https://github.com/ddev/ddev/issues/5358
Release/Deployment Notes
this shouldn't affect anything else.

Thanks! This will require a custom image to be pushed, I'll try to do it in the coming days.

Actually it would be good for you to test this manually before we start pushing images. I think you know all the details of what to do, but update the kill_supervisor.py inside the container after it's running, and then test your scenario. You can use docker cp to copy it in there, or you can copy it from a mounted directory.
I thought you would just be making small changes to the supervisord.conf rather than trying to make the kill script fancy.

OK I'll try this now. I'm putting it in web-build & add a Dockerfile to ADD to replace the original kill_supervisor.py

It works well with my current configuration. But I don't know how to make Apache or Nginx errors to ensure the container will turn off when that happens.

You can just ddev exec killall apache2 or ddev exec killall nginx or ddev exec killall php-fpm

Let me know if you're interested in continuing with this, otherwise I'll close it about November 15.

I'm interested in continuing this issue. it's halfway done but I'm currently got no time. I'll try to fix this before November 15th.

Closing this for now, happy to return to it later, it can be reopened. Thanks for this contribution and all the thinking that went into it.


Captive portal plugins access

Plugins ability to register access paths to custom controllers in captive portal config.
For example, I made captive portal plugin for sms-authentication, and clients must have access to my sms-authentication controller.

@evbevz  I'm not sure we should add this, there's no use case for it in the standard product, maybe it's better to look for a template include construction for custom settings so we don't have to extend the standard model (more similar like the proxy/squid approach).

For custom functionality CP template must have access to custom controller. Access to controllers is guarded by authorisation system. I can't find another way to get access to custom controller.

@evbevz for new overwrite options, see https://github.com/opnsense/core/issues/1934. With this you can create and maintain your own templates for the installed ones.

@evbevz This has been rejected and closed a long time ago, however I need to solve the exact same problem and implement some sort of SMS-based captive portal. I posted in the forum a long time ago without any reply from devs (https://forum.opnsense.org/index.php?topic=9749.msg44477#msg44477).
Would you be willing to share some code samples that you used at the time or point me at a way to solve the problem using "overwrites" as mentioned by @AdSchellevis ?
Patrick

Hi, @pzwahlen. My solution has:

access controller with sms-auth logic
settings controller to store sms-gate settings
service controller to get some stats and current state
sms-athenticator implements IAuthConnector
proxy hook
captive portal config hook to get unauthorised access to access controller (it can be made by override, abouth whish @AdSchellevis says)

If you don't need proxy logs, some parts can be skipped.

Hi,
I'm looking for a solution in a private environment (large house with 2 families, kids en friends...). The idea is to avoid sharing main wifi passwords and provide a somewhat bandwidth-limited guest access.
So far, the built-tin template without any authentication ("click to connect") does the job. I just wanted to improve on it, without having to use vouchers or a user DB.
I don't intend to intercept the trafic, so proxy integration won't be used.
My main concern is that the solution needs to survive upgrades, or it will be hard to maintain over time.
Would you be willing to share or can we discuss pricing for a private setup ?
Anyway, thanks a ton for having taken the time to respond.

@evbevz Would you be willing to share your Captive Portal solution ?

@pzwahlen unfortunately I can’t share the sources anymore, as the work was done in the interests of another customer who didn’t consent to share it. sorry.

@evbevz Thanks, this is perfectly clear and fine.


How do you setup HTTPs for Portus in a Docker Container...

Any advice on how to turn on https support for Portus in a container?

+1
Also can you please provide more information how to build the container and then specify the ENV variables to set up the DB connection? Or do I have to create my own Dockerfile and specify the ENV variables there? Seems to me like this is the way to go, but maybe I'm wrong ;-)

Note that docker-compose is meant to be used for development purposes, that's why we haven't setup https in there. Anyways, here you can find an example of it.

I've decide to go the Nginx route for securing the Portus UI.

If you have a working nginx config, then please post it here.  I've been trying to use an nginx proxy to add SSL to portus, and am unable to do so unless I use port 443 as the port numbers keep being removed.
This is what I've been trying:
server {
    listen 443 ssl;
    server_name rancher.container.auckland.ac.nz;
    ssl_certificate     certs/rancher.container.auckland.ac.nz.crt;
    ssl_certificate_key certs/rancher.container.auckland.ac.nz.key;
    location / {
        proxy_set_header Host $host;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_set_header X-Forwarded-Port 8443;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_pass http://portus:3000/;
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection "upgrade";
        proxy_read_timeout 900s;
    }
}

Then starting with
docker run -d --restart=always   --link=portus   --name=portus-ssl \
    -p 8443:443  \
    -v `pwd`/conf.d:/etc/nginx/conf.d  -v `pwd`/certs:/etc/nginx/certs \
    nginx

Superseded by #1027. Feel free to re-open this issue if you think that there is still things to discuss here and that they are not suitable to be discussed in #1027.


Addon does not work in packaged game

Hey!
First of all love the addon, it let me create stairs much quicker and speed up my process of gray boxing so big kudos for that.
I just noticed, however, that when I try to package the game and run it unfortunately the ramps & stairs do not show. This is caused by the use of editor-only types in the node script, which fails to parse due of them.
See screenshot from console of packaged game for more details.

I workaround it locally by removing any strong-typed relation on editor types, this seems to fix it for me, however, I don't think this is necessarily the cleanest approach to the problem... Maybe there is some hidden Godot way to handle it properly that I don't know about.
Repro steps:

Install plugin
Add ProtoRamp to your level
Package game in debug
Launch game
Overserve missing proto ramp from the level and errors in console

In case you'd be interested into looking at my workaround, here is .diff file.
https://gist.github.com/RootKiller/fdfd22bf0c98a584d67c6a3469890482

The issue seems to be related to this https://github.com/godotengine/godot/issues/91713 as such requires dynamic typing for working export until that issue is fixed.

Thank you for reporting the issue, I will look into this as soon as I can.

Thank you for providing a diff file, it looks great! I will create a PR based on your diff file, but feel free to open one yourself, as I am fixing other issues as well.


Fix Darwin compatibility

Remove ncurses from checked modules as CMake cannot find ncurses on Darwin
Add a compatibility header for endian conversions

I would be happy to find an alternative solution to the ncurses issue.


Use mentions for users in mod logs and infraction search results

Fixes #533
Fixes #337
Mentions
Mod logs and infraction searches will now display a mention for users, including actors when dealing with infractions. The username and discriminator are also displayed for users. This ensures someone can still be identifiable at a glance in case the user isn't in the cache and the mention fails.

If the user's name and discriminator isn't available, it is simply omitted. For example, this is the case when a user is unbanned:

Channel mentions
I have changed a few channels to use mentions when I've encountered them. However, all the channel mod logs have been left alone. Should those be changed to use mentions? Keep in mind such a change will mean they're no longer searchable as text, unless their names are included in plain text in addition to the mention.
Misc changes

The code to send mod logs for triggered filters has been refactored to reduce code redundancy.
Multiple active infractions are no longer checked for when pardoning an infraction. The API will already reject a second active infraction of the same type.

@Numerlor I believe a fetch is done to have a clear distinction between users and reasons. Keep in mind, sometimes queries are done on users that are not in the guild. In such cases, relying on the cache will not work.

@Numerlor I believe a fetch is done to have a clear distinction between users and reasons. Keep in mind, sometimes queries are done on users that are not in the guild. In such cases, relying on the cache will not work.

Only case I can think of where there would be a difference between fetching from the api and trying to validate a provided ID is where a valid user ID is provided, but the user doesn't exist. Is such an edge case worth it when all the user infraction data is fetched from the site after that anyway?

How do you propose to distinguish between an ID and a reason that happens to be a number?

The user IDs are snowflakes, if we parse the datetime inside the number and it's a valid creation datetime then it should be safe to assume that an ID was looked up instead of a reason string

I think it's best to save that for another PR. It's more work than it first seems. The group invokes the subcommand, which expects a discord.User object.

After matching it, it should be enough to just let it through an UserConverter and falling back to proxy_user like the command does, but yeah I agree that it is a bit out of scope here if the fetched user is not reused in the listing but the fetching removed.

Invoke uses the callback directly, without processing converters, checks, etc. The ID would need to manually be converted to a User. It's all a bit awkward.


multiple databases at startup

Is it possible to create multiple databases at startup?
docker-compose.yml
db:
    image: postgres
    environment:
        - POSTGRES_DB=db1
        - POSTGRES_DB=db2
        - POSTGRES_USER=user
        - POSTGRES_PASSWORD=pwd
        - POSTGRES_ROOT_PASSWORD=rootpwd

I would probably namespace them. So:
POSTGRES_DB_1="database=X;user=X;password=X"
POSTGRES_DB_2="database=X;user=X;password=X"

Perhaps a limit of 10 and you could do something like:
for i in $(seq 0 10);
do
    tmp='POSTGRES_DB_'${i}
    # And all other variables

    # Array
    arr=$(echo ${!tmp} | tr ";" "\n")

    for x in $arr
    do
        # Do the postgres logic to create that Database
    done
done


Add Ruby 2.7 support and drop Ruby 2.5

With the release of Ruby 3.0 in Dec, Ruby 2.5 is in security maintenance phase (and will EOL soon)
I think it's about time to start allowing Ruby 2.7 and making Ruby 2.6 the baseline version.  Not sure we should update the build itself yet, though.  I'm thinking for Lasker we build with Ruby 2.6 and let developers play around with Ruby 2.7.  Then in the M-release we can consider bumping the build to Ruby 2.7.
cc @jrafanie

Gah, I didn't realize this already had an issue.
Duplicate of #19678


Introduce functionality to support selection of type of chaos via ENV in MySQL data persistence test

Signed-off-by: ksatchit<EMAIL_ADDRESS>What this PR does / why we need it:

Introduce chaos ansible test modules that can be used to inject failures into the MySQL persistence test
Add support to pick chaos types as an ENV variable in the litmus test job
Include node cordon/drain & pod kill chaos using kubectl & pumba respectively
Add tasks (in mysql tests) to get test result in a dedicated JSON file
README updates for MySQL data persistence tests

Notes:
The supported chaos types that can be chosen via the Litmus job are as follows:

APP_POD_KILL/PUMBA : The MySQL pod is terminated via SIGKILL, multiple times over 120s using Pumba
APP_POD_EVICT/KUBECTL : All pods on the application node are forcefully evicted by Kubernetes via resource taints (disk-pressure)
APP_NODE_DRAIN/KUBECTL : The application node is taken down gracefully via cordon & drain

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:
Future enhancements

Chaos types should be made generic, i.e., outside the scope of a test
Increase chaos verification using kubernetes resourceVersions etc.,

Sample test result :

ubuntu@kubeminion-01:/mnt/mysql_data_persistence$ cat result.json
{ "Testname" : "mysql-data-persistence", "status" : "Pass" }

Sample test o/p :

mysql_data_persistence_test.log


Licence when used in game

I'm slightly confused by the licensing for FLmusiclib, am I required to make my game's source available if I use this Godot addon or is that only required for the author of the addon? Otherwise, what am I required to do when distributing me game, is it putting a link to this or the original library in a txt file credit included with my game's distribution?

Sorry, I should've been more specific, at the time I was a bit confused too.
You just have to include the licenses from the Licenses folder somewhere where the user can see them(in-game, or as files on desktop platforms). The code for the addon is public domain while GME is LGPL2 so I can link GME statically and since the addon uses dynamic linking it doesn't apply to your game.
You can link to this repository instead of including the CC0 licence to make sure people have a way to get the source since LGPL2 requires the user to be able to relink the library if it's linked statically, and if you don't need to play formats supported by libopenmpt you only need the licences for GME and minimp3 in the licences folder.

Thanks, that's very helpful! I actually only need the xm playback functionality, although the openmpt playback is even better and I'm thinking of just using that as it's smaller and allows for many more features. Can I just remove the GME parts I don't need or not include the dlls and it'll work okay? Im not actually sure which licence module or openmpt uses.


feat: Trigger release of new marketplace desc

:tada: This PR is included in version 0.78.0 :tada:
The release is available on:

v0.78.0
GitHub release

Your semantic-release bot :package::rocket:


feature request: bypass the cache for only one script name

the --no-cache cli option allows Lage to not use the cache, but this apply to all the tasks it runs.
We need to way to specify in the config file that a given script will always bypass the cache (because it produces some side effects which are not cacheable).

Close as duplicate


Working demo

What more is there to left to say. It worqz 🎉

@danieljcss @rishotics I'm merging now.


PR diff: URL query changing constantly

Hi!
The problem:

The URL to reproduce:
https://github.com/xcp-ng/xcp-ng-release/pull/7/files
Thanks :)

That's probably because of default-to-rich-diff feature.
Just by looking at the code, my assumption is that for deleted svg files GitHub doesn't render a .render-wrapper, causing both the conditions on https://github.com/sindresorhus/refined-github/blob/09d1ab67b8eae588f33d4f8aa31cfeec1f9dac93/source/features/default-to-rich-diff.tsx#L19 to always fail, which in turn causes the interval to keep on clicking the button for every 300 milliseconds.

Indeed this needs to skip deleted files. Thanks for the report!

It's kinda weird that GitHub displays the Rich Diff button even if it won't work

Not having it would me more confusing IMO, having it provides a reason on why the rich diff cannot be rendered.

It can be there, disabled and with a tooltip, like other similar non-working buttons

A bigger bug here is that the page scrolls down to the file. If that happens all the time we probably have to drop the feature

A bigger bug here is that the page scrolls down to the file. If that happens all the time we probably have to drop the feature

Not necessarily, what if we make the request to and update the DOM instead of clicking the button. GitHub currently does innerHTML for this I'm assuming we can use <include-fragment> too.
If this is a big issue, then we can drop it (Also saving serve costs for GitHub 😋).

Yeah this feature needs to be rewritten or dropped because the scroll jump is seen everywhere. I looked into their code, this is what I found:
Starting from here:
https://github.com/elementary/icons/pull/792/files
The first file loads this:
https://github.com/elementary/icons/diffs/0?base_sha=8333483bdfaf300e175c285bed70203a283c2230&commentable=true&head_user=lainsce&pull_number=792&sha1=8333483bdfaf300e175c285bed70203a283c2230&sha2=50415810f666efb4e03f8710999c08e09d441d46&short_path=0321141
That URL is found in the button's form. We can load that with fetchDom and inject it. That pages includes the whole loading gif interface that loads this in an iframe:
https://render.githubusercontent.com/diff/svg?commit=8333483bdfaf300e175c285bed70203a283c2230&enc_url1=68747470733a2f2f7261772e67697468756275736572636f6e74656e742e636f6d2f656c656d656e746172792f69636f6e732f383333333438336264666166333030653137356332383562656437303230336132383363323233302f616374696f6e732f31362f77696e646f772d636c6f73652e737667&enc_url2=68747470733a2f2f7261772e67697468756275736572636f6e74656e742e636f6d2f656c656d656e746172792f69636f6e732f353034313538313066363636656662346530336638373130393939633038653039643434316434362f616374696f6e732f31362f77696e646f772d636c6f73652e737667&path=actions%2F16%2Fwindow-close.svg&repository_id=47588124#69c770a3-32de-4dc2-9497-586d7628242b
So for every SVG it's two HTTP requests. I don't think we can skip this intermediary page, but if we could it'd be great.
At the very least, for every SVG we should load the intermediary page and change the button classes.


Added gatsby image plugin

-Used on
-Home screen
-Event cards covers
-Blog cards covers
-Event main picture
-Blog main picture
Converted svg to png pictures - Gatsby image plugin doesnt accept svg pictures.

Awesome :sparkles: Thanks for the PR @davidgorel :+1:
Here's the deploy preview of the PR.


THRIFT-4994: changed this to self in forEach callback

see: https://issues.apache.org/jira/browse/THRIFT-4994

[x ] Did you create an Apache Jira ticket?
[x ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x ] Did you squash your changes to a single commit?  (not required, but preferred)
[x ] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, add  [skip ci] at the end of your pull request to free up build resources.

Thanks!


Portable mode not picking user-data (settings and keybindings)

Extensions work though.
The folders structure is exactly the same as in the instructions, no typo...
i.e. vscode-portable\data\user-data\...

VSCode Version: 1.27.2
OS Version: Win 10 (1803)

Steps to Reproduce:

Follow the instructions to migrate to a portable folder (https://code.visualstudio.com/docs/editor/portable#_migrate-to-portable-mode)
Open Code.exe
command palette > open settings.json -> "empty". (or drop a folder on Code.exe)

Does this issue occur when all extensions are disabled?: Yes

What's the file path of that settings.json file which opens when you open it via the command palette?


HBASE-26640 Reimplement master local region initialization to better …

…work with SFT

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  9s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 28s
master passed

+1 :green_heart:
compile
2m 18s
master passed

+1 :green_heart:
checkstyle
0m 38s
master passed

+1 :green_heart:
spotbugs
1m 26s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 49s
the patch passed

+1 :green_heart:
compile
2m 16s
the patch passed

+1 :green_heart:
javac
2m 16s
the patch passed

-0 :warning:
checkstyle
0m 40s
hbase-server: The patch generated 3 new + 84 unchanged - 0 fixed = 87 total (was 84)

-0 :warning:
whitespace
0m  0s
The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>.


The quantity is wrong

Only one face in screen but the faces.size() is two. when two faces in screen the faces.size() is four.
And the image can't recognize

When the camera is front, the Rect is Left-Right reversed

Thanks for reporting. That is indeed a bug in our face detection library. Should be possible to fix.
As for mirrored image issue - I opened another ticket #6


Add tax_rate_id and more to the auto created customer

Feature request

Extension name: I am not sure which extension is affected. I think it is ext-firestore-stripe-subscriptions-create-customer

Is your feature request related to a problem? Please describe.
Hey guys, i have setup the firebase stripe extension and everything was working well.
I figured out that we have to add tax_id_data to the customer object. Because our company is based in Austria and we wanna roll out our software in the whole EU.
I did the implementation based on https://firebase.google.com/docs/use-cases/payments. My Problem is that i can't touch the customer object which is created automatically. I wanna add some data to this object like the tax_id_data {type: 'eu_vat', value: 'VatID'} for example.
is this possible?
Describe the solution you'd like
Maybe there is already a solution for this problem but i don't found it.
Describe alternatives you've considered
What would be a other solution? I tried to add the customer over a self programmed firefunction. Problem: The customer was not synced to my database.
i also tried to create the customer over my firefunction and add the customerStripeId to the checkout_session. But Stripe created also a new customer.

After reviewing your extension i found out, that my use case is impossible.
i found a workaround. I create the customer over my own firefunction. Important, you have to set stripeId in the Customer Object. Otherwise the extension will overwrite your customer entry.
everything else is working fine - with the extension.


Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the Swipeout component.

使用一个数组map出4-5个swipeout，在左划时出现了如上的warning。

map函数是在哪里调用的？能看看代码吗？

Having this issue also, i have a flatlist, and use a swipeout menu to reveal multiple buttons including a delete button, when I have remove my item from the list, the list updates, but a reference to the swipeout  seems to persist

Having this issue also, i have a flatlist, and use a swipeout menu to reveal multiple buttons including a delete button, when I have remove my item from the list, the list updates, but a reference to the swipeout seems to persist

Sorted my issue out, I had set stuff up in the constructor which ended up causing issues due to closure


DBSCAN Import Error

I have been trying to run DBSCAN, using Python from command line .. I got this error
ImportError: cannot import name _get_unmangled_double_vector_rdd
Any one can help me regarding this ?

I think I got the issue, the according to this https://fossies.org/diffs/spark/1.1.1_vs_1.2.0/python/pyspark/mllib/stat.py-diff.html, PySpark has changed the MLib implementation, so the _get_unmangled_double_vector_rdd does not exist any more .. Anyone can fix this ??

@kszucs can you help me with this ?

The DBSCAN python API is outdated and will be removed in the upcoming branch. Have You tried using https://github.com/alitouka/spark_dbscan from scala?

Indeed .. Although this implementation works perfectly for the provided sample data, its not working at all for any other data. I have spent around 10 days trying to make it work, I have used portion of my data, with is typically the number of records, and dimensions as the sample data, but it did not work. Although I have tried all the tuning parameters of SparkConf, did not work as well.
This implementation has a bottleneck of aggregation step, in which the partitions are merged, according to the provided wiki https://github.com/alitouka/spark_dbscan/wiki/Density-based-partitioning. I have used monitoring tools to watch what exactly happens, and the algorithm at this certain step is turned into sequential and the computation kept running and never ends.
I have tried to communicate with the author over git and mail and there were no response at all.
Please let me know if you have any suggestion since I need to use DBSCAN to cluster around 1.5G of data.
Thanks in advance

There is another one: https://github.com/irvingc/dbscan-on-spark
Based on the visualization it could be easily inplemented with sparkit-learn via applying scikit's DBSCAN on the blocks.

@kszucs  Thanks a lot .. Can u give me more hints about how to apply it, or similar example if possible ?


Lazy load CommandPalette and AboutDialog

This sets x:Load to false for the two elements.
On my system, with Windows Defender disabled, this reduces CPU
usage by 15ms and the visual delay during launch by 40ms.
Part of #5907
PR Checklist

Ctrl+Shift+P opens command palette ✅
Context menu opens command palette ✅
Context menu opens about dialog ✅

This PR was accidentally merged into my other (stacked) PR. I've reopened this one here: #15203 (same commit hash).


⚠ ClamAV has degraded performance

In e6d4f8e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in a642fd2 after 22 minutes.


Can't switch for a theme called Things

If I select Things as day or night theme, the appearance doesn't change on time. I have to manually select another theme without this issue and then switch back to Things to make it work again.
Not sure if there are another themes with this issue.

I am having trouble recreating this issue.
I installed the 'Things' theme, changed the Day Color Scheme and Theme to Light Things.
The theme changed to Light Things theme after a few seconds.
Can you recreate this consistently? If so, could you please provide steps to reproduce this?

My version is v1.3.5. It's after the update that this issue appeared.
I find this issue behaves weirdly and may be more general than I expected. And sometimes it works, other times it doesn't.
Let's say I'm in light mode based on the schedule:

I manually change to dark mode via "Appearance" section in the menu. Then it sticks with dark mode. I should change to another theme in Appearance to make it work again.
If I change light mode to dark mode in the Plugin Setting, the theme doesn't change. I should also change to another theme in the Plugin Setting or Appearance to make it work again.

That being said, changing the color scheme makes the plugin stop working. When the plugin doesn't work, restarting the application doesn't help.

Same issue on 1.3.5 on mac - I have both themes set to Dark Typewriter (day and night) and it still uses the default appearance/theme that is set (Light Typewriter).

I have released version 1.1.2 with the fix for this issue. Please update and let me know if that fixes the issue.

Works correctly now - thank you!

Thanks! It works fine now!


Creating custom matches

Howdy,
We have been widely using TypeMoq as our chosen typescript mocking library and it's worked well for us! Thank you for all your hard work on the library!
Unfortunately we've frozen on the 0.6 release because we implement some custom Matches and as you know in order to successfully implement a custom Match you need to expose the certain properties that  have the value of some internal constants.
Newer versions of typemoq don't expose this publicly, so it's a pain to implement a matcher (still possible, but not a great development experience).
I understand that there is a predicate matcher, and it's thus possible to implement a matcher using that, however it makes sense from an extensibility stand point to enable developers to create their own matchers should they find an interesting case that isn't directly supported.
Any enlightenment on the design decision would be welcome, or perhaps I am mistaken and you can do this - in which case how? :)

I managed to get one working:
import { IMatch } from 'typemoq/Match/IMatch';

export class Itx {
	public static isAnyFunction<T>(): T {
		let matcher = new MatchAnyFunc();
		return <any>matcher;
	}
}

class MatchAnyFunc implements IMatch {
	public readonly ___id = '438A51D3-6864-49D7-A655-CA1153B86965';

	public ___matches(object: object): boolean {
		return object && {}.toString.call(object) === '[object Function]';
	}
	public toString(): string {
		let res = `It.isAnyFunction()`;
		return res;
	}
}

mock.verify(m => m.log(Itx.isAnyFunction()), Times.once());


proxy store does not update for a numeric state

It seems like the proxy store does not update when the state of the redux store has a numeric state i.e.
const rootReducer =  (state = 0, action) => {
  switch (action.type) {
    case 'INCREMENT':
      return state + 1;
    default:
      return state;
  }
}

export default rootReducer

To fix this, I changed the state to an object by wrapping the reducer using:
export default combineReducers({
  count: rootReducer
})

It works as expected with this reducer.

Hmm strange we should probably support this base case since serializing state is just JSON.parse/stringify. Did you happen to catch the error logged or did it just silently fail @skk2142?

Can confirm that it also doesn't work for null or false, probably all values which are falsy are ignored. (silently btw)


[Game Request] Gakuen Tokkyuu Hotokenser (学園特救ホトケンサー)

Requesting support for unpacking .xp3 files from Gakuen Tokkyuu Hotokenser (学園特救ホトケンサー)
I tried KrkrDump, KrkrExtract, and XP3Viewer and none of them worked.
KrkrDump doesnt pull anything https://i.imgur.com/juaxuXn.png, KrkrExtract and XP3Viewer immediately closes when the .exe is dragged

Works, thanks!


adding | trim to remove whitespace

fixes issue #188 that I logged. simply adds trim in in line with jinja and ansible docs.

Closing as the ansible playbook no longer supports dynamically pulling the token from inital server. A token must now be provided.


Decorators not handled

Sample project can be found here: https://github.com/Cyberlane/fusebox-test
It currently fails on both Linux and Mac, with the following error:

@Cyberlane
You can't use modulesFolder pointed to src. It's not webpack, use aliases instead. And fix you target, it's not server, it's browser.
const {
  FuseBox,
  SassPlugin,
  CSSModules,
  CSSPlugin,
  WebIndexPlugin,
} = require('fuse-box');

const fuse = FuseBox.init({
  target: 'browser@es5',
  homeDir: 'src',
  output: 'dist/$name.js',
  alias : {
    "decorators" : "~/decorators"
  },
  useTypescriptCompiler : true,
  plugins: [
    WebIndexPlugin({ template: 'public/index.html' }),
    [SassPlugin(), CSSModules(), CSSPlugin()],
  ],
});

fuse.dev();
fuse.bundle('app')
  .hmr()
  .watch()
  .instructions('>index.js');

fuse.run();


Support Configuration

It should be possible to configurate the (external) hub url via configuration.

Hey @endeffects
Can you elaborate more on what you want?


I don't understand the install on Vagrant

vagrant@precise64:/vagrant/dragnet-1.1.0$ make test
nosetests --exe --cover-package=dragnet --with-coverage --cover-branches -v --cover-erase
make: nosetests: Command not found
make: *** [nose] Error 127

vagrant@precise64:/vagrant$ cd dragnet
vagrant@precise64:/vagrant/dragnet$ make test
nosetests --exe --cover-package=dragnet --with-coverage --cover-branches -v --cover-erase
make: nosetests: Command not found
make: *** [nose] Error 127
vagrant@precise64:/vagrant/dragnet$

The Vagrantfile specifies a trusty64 image, so how are you getting a vagrant@precise64 prompt?
My prompt inside the VM looks like this:
vagrant@vagrant-ubuntu-trusty-64:~$

And running make test succeeds.

I do a vagrant up and vagrant ssh

How do you get the trusty64 image up and running ?

vola-g@PO30623 MINGW64 ~
$ git clone git@github.com:seomoz/dragnet.git
Cloning into 'dragnet'...
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

vagrant@dragnet:/vagrant$ make test
nosetests --exe --cover-package=dragnet --with-coverage --cover-branches -v --cover-erase
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named numpy) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named lxml) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named numpy) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
Failure: ImportError (No module named blocks) ... ERROR
======================================================================
ERROR: Failure: ImportError (No module named blocks)
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/nose-1.3.7-py2.7.egg/nose/loader.py", line 418, in loadTestsFromName
addr.filename, addr.module)
File "/usr/local/lib/python2.7/dist-packages/nose-1.3.7-py2.7.egg/nose/importer.py", line 47, in importFromPath
return self.importFromDir(dir_path, fqname)
File "/usr/local/lib/python2.7/dist-packages/nose-1.3.7-py2.7.egg/nose/importer.py", line 94, in importFromDir
mod = load_module(part_fqname, fh, filename, desc)
File "/vagrant/dragnet/init.py", line 1, in
from dragnet.blocks import Blockifier, PartialBlock, BlockifyError
ImportError: No module named blocks

The current Vagrantfile already specifies trusty64, so if you use vagrant up in the checked out directory, it should automatically use trusty64.

Typically, you'll need to use the HTTPS method for cloning the repo if you don't have commit access.

vola-g@PO30623 MINGW64 ~/dragnet (master)
$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'ubuntu/trusty64' is up to date...
==> default: There was a problem while downloading the metadata for your box
==> default: to check for updates. This is not an error, since it is usually due
==> default: to temporary network problems. This is just a warning. The problem
==> default: encountered was:
==> default:
==> default: Failed to connect to atlas.hashicorp.com port 443: Connection refused
==> default:
==> default: If you want to check for box updates, verify your network connection
==> default: is valid and try again.
==> default: Clearing any previously set forwarded ports...
==> default: Fixed port collision for 22 => 2222. Now on port 2200.
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
default: Adapter 1: nat
==> default: Forwarding ports...
default: 22 (guest) => 2200 (host) (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
default: SSH address: <IP_ADDRESS>:2200
default: SSH username: vagrant
default: SSH auth method: private key
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
default: The guest additions on this VM do not match the installed version of
default: VirtualBox! In most cases this is fine, but in rare cases it can
default: prevent things such as shared folders from working properly. If you see
default: shared folder errors, please make sure the guest additions within the
default: virtual machine match the version of VirtualBox you have installed on
default: your host and reload your VM.
default:
default: Guest Additions Version: 4.3.36
default: VirtualBox Version: 5.1
==> default: Setting hostname...
==> default: Mounting shared folders...
default: /vagrant => C:/Users/vola-g.SAVOIE/dragnet
==> default: Machine already provisioned. Run vagrant provision or use the --provision
==> default: flag to force provisioning. Provisioners marked to run always will still run.

vagrant@dragnet:/vagrant$ sudo make test
nosetests --exe --cover-package=dragnet --with-coverage --cover-branches -v --cover-erase
Coverage.py warning: Module dragnet has no Python source.
/usr/lib/python2.7/dist-packages/pkg_resources.py:1031: UserWarning: /home/vagrant/.python-eggs is writable by group/others and vulnerable to attack when used with get_resource_filename. Consider a more secure location (set with .set_extraction_path or the PYTHON_EGG_CACHE environment variable).
warnings.warn(msg, UserWarning)
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named scipy.optimize) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR
Failure: ImportError (No module named sklearn) ... ERROR

The instructions in the README for Vagrant do not say to use sudo:
vagrant ssh # these should now pass
$ make test

Using sudo changes the environment variables such that the build won't work.

Thanks. I spent time trying ti install dragnet with Vagrant but I did not succeed.


Add ImportGranularity::ModuleCondensed

Given that imports_granularity is unstable, is this something you would consider merging?
From the docs I added:

ModuleCondensed:
Like Module, but singleton imports are included in parent modules' use statements where it doesn't introduce nested braces.
use foo::b::{f, g};
use foo::{a, b, c, d::e};
use qux::{h, i};

I have long perceived this to be Clippy's convention. I ran this on Clippy's source code and about 2/3 of the files were left unchanged. Since this supposed convention has been enforced by hand, I consider this confirmation.

We don't really have a import convention in Clippy, other than that we don't allow wildcard imports, enforced by one of our pedantic lints.
That being said, I like this convention and would adapt it for Clippy. Especially when 2/3 of Clippy follow it already.

In principle yes. I don't envision us adding any variants ourselves but we're open to supporting others so long as someone submits the PR like this and it doesn't add any undue maintenance burden for us. We're unlikely to be able to spend any review cycles on this for the foreseeable future though.
Haven't looked at the specifics of the proposed change yet, but would suggest including a #[unstable_variant] attr on the new config variant if you haven't already done so. Stabilization of imports granularity is still a few releases out, but it's definitely on the near term horizon and probably better to go ahead and get that attr on the new variant so that we don't accidentally forget

Haven't looked at the specifics of the proposed change yet, but would suggest including a #[unstable_variant] attr on the new config variant if you haven't already done so.

Done. Please let me know what else is needed. Thanks.

@fee1-dead @CosmicHorrorDev - I know you're both busy (with both Rust and life things :smile:), but just tagging this as one for you to potentially review in case you're interested and get around to it before Yacin and/or myself.
I'm in favor of supporting the style associated with the variant, but haven't had a chance to look at the associated implementation

In addition, could you add unit test cases to test this behavior? thanks.

Done. (Apologies, as I hadn't noticed the existing unit tests.)

Sorry for the force push (minor grammatical adjustment). I should be good now.

We're unlikely to be able to spend any review cycles on this for the foreseeable future though.

Point very well taken, but I thought I would check in. Any chance of merging this?

I thought I would check in again. Any chance of merging this?

Also, reorder_imports=true is on by default. can we add test cases where we set reorder_imports=false. I'd want to see what the output looks like in that case.

I added such a test case. I also took the liberty of adding such a test case for imports_granularity: Module. At least for the given input, the results are the same for both imports_granularity: Module and imports_granularity: ModuleCondensed.
Apologies, as I did not realize there was a reorder_imports option and so did not consider how it would interact with imports_granularity. I will give it some thought.

@ytmimi I think imports_granularity: ModuleCondensed now works sensibly when reorder_imports is off.
Also, it appears that imports_granularity has an effect only when either reorder_imports is on, or group_imports is not Preserve (the default). So I updated the reorder_imports: false tests to use group_imports: One.

@ytmimi Would it be possible to give this PR another look?


proto mean substraction in fcn/ segmentation

in the segmentation example it reads

a Power layer was added to shift inputs by -116.0 (in lieu of per-channel mean subtraction)

can anyone explain why -116? should we change this number when training on other datasets?
where to find this number for other datasets created by digits segmentation dataset tool
Also is it exactly equivalent of mean substraction?

Mean image subtraction is impractical for FCNs because this prevents from taking variable-size inputs. Mean pixel subtraction would work better. In DIGITS you can select mean pixel subtraction and remove the Power layer.
Applying a fixed offset of -116 is even simpler and works well enough in practice to keep the data approximately centered around zero. In my experience you usually don't have to tune that parameter to get acceptable accuracy.
Try both methods and see which ones works better for your data.

Thanks, I will give it a try.

Hi @gheinrich , in  [binary-segmentation] example (https://github.com/NVIDIA/DIGITS/tree/master/examples/binary-segmentation) examples, the introduction recommends us to use 'mean pixel' skill. However, I can not find the mean file (xxx.binaryproto) in that certain created dataset job. What's more, the segmentation-model.prototxt does not include "mean_value" in data layer, which denotes mean pixel operation.
It is more amazing that when I use pixel mean in pycaffe, the output is extremely poor. Yet, when I feed the input image into the network directly, the output is closer to those in digits.
I am really confused. Could you give me some some advices? Thank you.


Apply for Grant, Plus: Polkadot js Plus extension

Project Abstract
Plus extension will be a user-friendly wallet to interact with the Polkadot/Substrate based blockchains through a browser. It allows users to access their Polkadot account(s), which can also be used to interact with decentralized apps. It is actually polkadot js extention, plus some new features. We don't want to rebuild the wheel, we just want to perfect it, standing on the shoulders of giants like "polkadot js extension" and make it user-friendlier for users.
The reason why we have decided to work on this project is every day user comments on social medias that complain "why the extension does not show the balance?", "It is too complicated for new/average users to work with", "It is too abstract", " why should we go to another web site to work with the extension", " how could contribute to the crowdloans via extension", and so on. That's why we've decided to work on Plus extension to address such concenrs/requests of the polkadot community. The new functionalities, that will be added to the original polkadot js extension are:

View balances
Transfer funds
Easy staking (auto stake/unstake/redeem funds and edit nominated validator list)
Contribute in crowdloans
View transaction history
View an address as a QR code
Governance (Referendums/Proposals voting, Treasury Tips, councils/motions )

For which grant level are you applying?

Level 1:  Up to $10,000, 2 approvals

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

The deliveries are updated, Thank you.


bug: Textarea Indent -3px

Type: bug
Platform: android 5.x webview
Hi,
I updated Ionic to V1.2.0
There is this new line of css :
.platform-ios textarea, .platform-android textarea {
text-indent: -3px;
}
It was added for the textarea placeholder : https://github.com/driftyco/ionic/issues/2307
But now, when I type in textarea, the first typed letter is cut by the left border of the textarea.
It concerns only the first line of the textarea
It was on an Android phone but later I will try on IOS to see if this is the same thing

+1 for android.
I have tried it on iOS and there is no issue.
I think the solution is to make it an iOS only change.
What do you think @mlynch?

Thanks, that was supposed to be iOS only, fixed!

+1 for android too!


ilab taxonomy diff should account for .yaml and .yml

in ilab taxonomy diff, the command errors out when .yml is used. Both .yml and .yaml file extensions should be accounted for.

@bjhargrave just want to confirm that .yml is supported in the taxonomy?

@cdoern @alinaryan if that's the case this may not be a bug, but perhaps I can add a clearer message?

it is still a bug given that the error for this is pretty unclear

There have been past issues on this topic which have been closed. Also the new taxonomy reading api in the schema repo instructlab/schema#33 should have a better error message for this.

Previous discussion: #1306

Thanks, will close this one


construct and pass in distinct mininput order_col_indices and order_types

https://github.com/risingwavelabs/risingwave/pull/13015/files#r1369627023

Hi, any progress?

Going to work on this today.

finished in https://github.com/risingwavelabs/risingwave/pull/13376


chore(deps): bump parse-diff to 0.8.1

:tada: This PR is included in version 1.1.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


An error occurred while transfer

version：mainnet-<IP_ADDRESS>
os：centos7
In the transaction between accounts, the following error message appears

This is the plug-in that I started:
plugin = eosio::chain_api_plugin
plugin = eosio::history_plugin
plugin = eosio::history_api_plugin
plugin = eosio::chain_plugin
plugin = eosio::http_plugin
plugin = eosio::producer_plugin

A problem with premature transaction expiration was corrected in today's 1.0.4 release.  Please update and try again.

I've done that, but the problem remains.
version：mainnet-1.0.5
os：centos7


[test] fuzz for deep structure conversion for xdr and host values

Originally brought up in https://github.com/stellar/rs-soroban-env/pull/1139#discussion_r1386974435
The host currently have recursion depth limits for xdr (de)serialization (500) and host recursion levels (100, shared by val conversion, comparison, cross-contract calling).
We need to write a fuzz test to verify these default limits are sufficient, by fuzzing against all combination of structures and their conversion paths, using the current limits, and assert all deep structures are caught by ExceededLimits error (no stack overflow is possible under those limits).

I've added some tests for de-serialization within a deep call stack (https://github.com/stellar/rs-soroban-env/pull/1237).
@jayz22  Please let me know if you had some other scenarios on your mind; I'm not convinced fuzzing is necessary here given that we're just worried about the stack depth.

These tests are good. FYI we also have some basic ones in depth_limit.rs that overlap with yours a bit, but I see you've also added more around invocation which is great.
However, the main concern here, (originally discussed in https://github.com/stellar/rs-soroban-env/pull/1139#discussion_r1386974435) is we aren't sure if the depth value choosen (500 for xdr paths, 100 for host paths) are tight enough which guaruntees the Rust stack will not overflow. The default Rust stack size is 2MiB, and it doesn't care about number of recursion levels (I'm not 100% sure about this statement and would appreciate @graydon's input), which means it is all implementation dependent (e.g. how many local variables used). Our depth limit is a proxy for a determinstic stack size and we need to make sure this proxy is indeed a lower-bound of the stack size in all possible paths of execution. So the goal here is to catch any possible paths in the rs-xdr implementation that can cause the current stack depth limit to be insufficient. That's why I was suggesting we use a fuzzer.
Given our schedule and the fact that we have good test coverage around recursion, we can punt this to later and reduce its priority to P2.

The default Rust stack size is 2MiB, and it doesn't care about number of recursion levels

Sure, I realize the recursion depth is not explicitly limited. But combined with the constant (or upper bound) frame size the depth is a sufficient proxy for the stack limit. Given that recursive XDR types are heap-allocated, I think I'm reasonably comfortable with the current test coverage.

Closing as the additional fuzz tests are not planned. The tests added should be good for now.


Add isMemberOf condition

As suggested in https://github.com/rikbruil/Doctrine-Specification/issues/25

Coverage increased (+0.01%) to 99.648% when pulling f4d4ad917ca518a5fa7a1911ce7fe9ab6de1076f on holtkamp:add-isMemberOfCondition into e61fb801c46107b0a9ce22086448557def7fa532 on rikbruil:master.

@rikbruil I am cleaning up my GitHub repositories and encountered this PR, can you consider merging it?


The --condition and the --action-group options need more information.

The --condition and the --action-group options need more information.  I am gettting a "Badrequest" error when setting something which should be valid for action-group and condition doesn't seem to work with an "and".

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

ID: d97968df-72c9-e50c-d5b1-54562c1bdef7
Version Independent ID: 0cde4f15-6464-3106-b787-3ee37c7de0ee
Content: az monitor metrics alert
Content Source: src/azure-cli/azure/cli/command_modules/monitor/_help.py
Service: monitoring-and-diagnostics
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@MyronFanQiu  planned after Ignite.

@carlbrochu Hello. Could you share the detail of --condition? It should work with and.

@carlbrochu Hello. Are you still there? Is it possible to share an example to us so that we can investigate this issue deeply?

I was trying to create an Azure monitor and passing these arguments just doesn't seem to work on the cli portion.  The support for an "And" statement seems to be missing.

@carlbrochu Hi. I just tried the command
az monitor metrics alert create -n alertmytest -g test --scopes /subscriptions/xxxxx/resourceGroups/feng-cli-rg/providers/Microsoft.Storage/storageAccounts/xxxx --condition "avg SuccessE2ELatency > 250 where ApiName includes GetBlob and ApiName includes PutBlob" --description "mytest"

and it works.
and is used in multiple dimensions instead of multiple condition. You can specify extra --condition to add more than one condition in one command.
If you still meet error, can you share the example of your command and I can have a try.

@carlbrochu I cannot produce this issue and it's a little bit out of date. I closed this one first. If you still have any issue, don't hesitate and feel free to open a new one.


[Plex] ...Failed to load library (does connect)

Extension
https://www.raycast.com/mSarheed/plex
Raycast Version
1.77.1
macOS Version
15.0
Description
Hello!
I was able to successfully connect my local Plex server (aside: might be good to give a bit more verbosity to the URL example: for instance I needed the Local network hostname (not <IP_ADDRESS>) and also needed to include the port :32400)
However, after displaying the library sections (Movies, Series, Music, etc) It just says: "Something went wrong" for any of them. Error below...
TypeError: Cannot read properties of undefined (reading 'toString')

S3:sectionItems.tsx:28:35

---
25:   );
26: }
27:
28: function SectionItem({ sectionItem }: { sectionItem: SectionItemsApiResponse["MediaContainer"]["Metadata"] }) {
29:   const markdown = `
30: |Title| ${sectionItem.title} |
31: |--|--|
---

Dr:index.js:8:6799
    at ray-list
dt:index.js:20:24299

Steps To Reproduce
Initial server connection and first attempt to display index.
Only tried on local network, not remotely
Current Behaviour
No response
Expected Behaviour
No response

@raycastbot assign me

Hi @jasonshanks.
Thank you. I will look into it now and publish a new version of the extension that hopefully solves this issue.


Search: More model renames

Apparently missed some model renames with issue #11707

Name
Rename

StandardTokenizerV2
LuceneStandardTokenizerV2

Pretty sure I got that one in my PR


Reader::parse() and Service::parse() return different values

Using sabre with XSD validation I'm getting unexpected result when using the XSD snippet from documentation.
There's actually 2 problems with suggested solution:
$service = new Sabre\Xml\Service(); $reader = $service->getReader(); $validXml = $reader->setSchema('myschema.xsd') if ($validXml) {     $reader->xml($xml);     print_r($reader->parse()); }
will not work as the Schema needs to be set after xml() method call.
Additionally the parse() method is returning root element array with name, value, and attributes keys.
Looking at code this happens because Service::parse() is returning $result['value'] while Reader::parse() is returning $result.
Can you please either correct the code, to always have same result, or at least update the documentation to make it clear what to expect?

where did you found this code?

@staabm
Which code exactly?
Suggested solution is on https://sabre.io/xml/reading at the end of that page.
Reader::parse(): https://github.com/sabre-io/xml/blob/03a31cad79abbcef81e7aff28e56a9d257a0dae2/lib/Reader.php#L56
Service::parse(): https://github.com/sabre-io/xml/blob/03a31cad79abbcef81e7aff28e56a9d257a0dae2/lib/Service.php#L112

are you willing to provide a fix in https://github.com/sabre-io/sabre.io/blob/master/source/xml/reading.md ?

The reader and service having slightly different signatures is definitely a bit strange in hindsight, but definitely by design. I expect most people to use the service, and the service is meant to be most user-friendly frontend for all of this.


Fix lazy install tag in README.md

Oh sorry, I glanced at this and then forgot to actually merge :sweat_smile:
Thank you, I hadn't spotted this mistake!


How to gerate Invoice like this? Or how to add html in print view?

any one help me for gerating a invoice as in attachment.  Specaily Product Description Section
https://images.app.goo.gl/nHAwk2nhLSZXmAbf6

There is no mention of this software in your poise, so I'm currently assuming that this is unrelated to this project.
If this is incorrect, then please explain what you are doing, how escpos-tools is involved, and which issue you have encountered.

Hello,
Hope You are well,
My question is how to add HTML in escope-tool. Mean how to allign item in
same line. Issue for me in attached file
please check it
On Sun, 5 May 2019, 5:59 a.m. Michael Billington<EMAIL_ADDRESS>wrote:

There is no mention of this software in your poise, so I'm currently
assuming that this is unrelated to this project.
If this is incorrect, then please explain what you are doing, how
escpos-tools is involved, and which issue you have encountered.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/receipt-print-hq/escpos-tools/issues/60#issuecomment-489377318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL6MJGMVH66PQMDF34W2SB3PTYWQNANCNFSM4HJGOOVQ
.


Resilient diags

Looks like a good change, but Circle is failing. Good to merge once it's fixed.
Also might be worth writing a diags ST at this point if we don't have one yet.