MatomoTestingTracker is missing the Injectable decorator and causing warnings

I get this warning when using the provideMatomoTesting provider helper:
DEPRECATED: DI is instantiating a token "MatomoTestingTracker" that inherits its @Injectable decorator but does not provide one itself.
This will become an error in a future version of Angular. Please add @Injectable() to the "MatomoTestingTracker" class.

Hi @viceice
you’re right, it’s a known "bug" (it should however work properly).
Feel free to open a PR, it should be ok tu just add @Injectable() to this class

see linked PR 🙃

:tada: This issue has been resolved in version 6.1.3 :tada:
The release is available on:

v6.1.3
GitHub release

Your semantic-release bot :package::rocket:


fresh install fails on icu

Hi,
I would tweets screenshots too so I did a fresh install in some instances. All fails if icu isn't yet installed. On Debian it require a apt-get install libicu-dev before npm install.
At github.com/mooz/node-icu-charset-detector there are some more OSs statements.
A readme update or a setup fix? I bet the first.

Having the same problem on macOS 10.12.6. I used
brew install icu4c brew link icu4c --force
as described here  and it worked for me


Added ALC298 layout-id 9 for Alienware 17 (AKA ALC3266) by Rockjesus

fixed some typo

Hello.
Did you read what i wrote you in previous pull request?
Read carefully please - https://github.com/vit9696/AppleALC/pull/209#issuecomment-335136603

@RockJesus
layout-id 1-10 is reserved, custom layout-id should start at 11 （layout-id 1-10是预留的，自定义layoutid应该从11开始到99，请先看看wiki😌）


Layer's source MUST be an AnalysisModel

Part of https://github.com/CartoDB/cartodb.js/issues/1749. Changes:
Changes to CartoDBLayer and TorqueLayer

These models had a source attribute that was the id of an analysis  node. In this PR we're changing things so that the attribute must point to an instance of AnalysisModel.

Both classes now expose getSource, getSourceId to keep the source attribute "private".

Also, it's not possible now to set a new source via the update method. These two classes will
throw an error when trying to do so and setSource will have to be used for this purpose.

:boom: This will break BUILDER so we need to start using setSource in that project :boom:
Changes to Map#createCartoDBLayer and Map#createTorqueLayer
These two methods were expecting an analysis id as the source attribute. These methods now assume they will get an object and will raise an error if something other than an instance of AnalysisModel is supplied.
:boom: These methods are being used by BUILDER so we need to update the code that uses them :boom:
Support for sql option in CartoDB and torque layers
sql option for CartoDB and torque layers in viz.json will still be supported (see https://github.com/CartoDB/cartodb.js/issues/1754 for more info), but instead of using that attribute internally, the library will generate a "source" type analysis when the viz.json is loaded and layer models are generated.
Changes to Dataview models
At some point in time, dataview models could not have a "source" and they would "inherit" the source from the layer they were associated too. This is no longer true and we can assume that deep-insights.js will always set a valid "source" when a dataview is created. This PRs removes some of that logic (I couldn't resist myself).

@IagoLast I addressed your feedback in https://github.com/CartoDB/cartodb.js/pull/1755/commits/385f7b0a95f74442f103e222b2811766e345052c. I didn't mention AnalysisModel in the error message because it's not part of the actual API (analysis models are created via vis.analysis.analyse method) and also because we'll have to introduce a new class for "source" analyses). Thanks!

@IagoLast I pushed some changes since I asked you for the review (sorry!). I've updated the description of this PR to better reflect everything we're changing here. Thanks!

LGTM

TODO: https://github.com/CartoDB/cartodb.js/pull/1755#pullrequestreview-63920277

There are lots of errors testing deep-insights.js: https://github.com/CartoDB/deep-insights.js/tree/layer-sources.
@alonsogarciapablo @IagoLast could you take a look?

Also 41 failures in Builder (layer-sources branch). I think we need to move it to development

Deployed in ded10

It works without errors!


Unexpected results with subclasshook

When using a __subclasshook__ with equinox modules, it may happen that issubclass returns True on pairs of classes where normally (if they weren't equinox modules) this would result in False. For example, in the following code, the print statement at the end will return the same boolean as set in with_eqx.
with_eqx:bool = False

if with_eqx:
    import equinox as eqx

    Module = eqx.Module
    StatefulModule = eqx.nn.StatefulLayer
else:
    class Module:
        pass

    class StatefulModule(Module):
        pass

class RegularLayer(Module):
    pass

class MaybeStatefulLayer(StatefulModule):
    pass

class SpecificRegularLayer(RegularLayer):
    pass

class SpecialModuleCollection(Module):
    @classmethod
    def __subclasshook__(cls, maybe_subclass):
        if issubclass(maybe_subclass, (RegularLayer, MaybeStatefulLayer)):
            return True
        return NotImplemented
    pass

class SpecificSpecialModule(StatefulModule, SpecialModuleCollection):
    pass

print(issubclass(SpecificRegularLayer, StatefulModule))  # False if with_eqx is set to False, otherwise True

Oh wow, now that really is super weird.
This actually doesn't look to be a bug in Equinox, but a bug in Python's abc itself. Here's a reproducer that doesn't use Equinox at all:
import abc

with_abc: bool = False
if with_abc:
    class X(metaclass=abc.ABCMeta):
        pass
else:
    class X:
        pass

class Y(X):
    pass

class Z(X):
    pass

class Magic(X):
    @classmethod
    def __subclasshook__(cls, maybe_subclass):
        if issubclass(maybe_subclass, Z):
            return True
        return NotImplemented

class Magic2(Y, Magic):
    pass

print(issubclass(Z, Y))

(FWIW using Python 3.11.9 in my case.)
I'd suggest verifying if this is still present in recent Python releases, and if so then opening a bug against Python itself.

Thank you for your reply! Indeed, the bug seems to still exist in Python 3.13, so I'll file a bug report there.


August 30 - September 3  - Work with the Django admin site

Event Information

Category
Details

Reactor Topic
Coding, Languages and Frameworks

Workshop Topic
Work with the Django admin site

Workshop Duration
60 mins

Title
Work with the Django admin site

Description
The most popular feature of Django is its built-in admin site, which allows internal users to manage data without having to create a special utility. When you create applications, you'll eventually want to create individual permissions for site access. Creating an admin interface for employees or clients to manipulate content can be a cumbersome task. Once again, Django comes to the rescue with a simple admin feature that's automatically created in projects. In this workshop, we'll continue our journey through the Django platform by activating the admin site, adding user permissions, and exploring another view of the database. It is recommended that you have a basic understanding of Django to attend this workshop.

Level
Beginner

Content Link
https://docs.microsoft.com/learn/modules/django-admin-site

Workshop(s) details

Date
Local Start Time
Location
Speaker
SharePoint ID
Platform

August 31, 2021
4:00 PM PST
US/CA/AU
TBD
13553
----------

August 31, 2021
6:30 PM
Bangalore
TBD
13554
----------

September 1, 2021
7:00 PM
Sao Paulo
TBD
13556
----------

September 1, 2021
7:30 PM
Shanghai
TBD
13557
----------

September 1, 2021
12:00 PM
EMEA
TBD
13555
----------

Event Checklist
10 Weeks Prior

[ ] Reactor Content Team (RCT) opens a new issue for an upcoming local workshop filling in the Event Information and Workshop details tables. RCT member who owns content (content owner) this week will be the assignee.
[ ] RCT and content owner discuss details of the event in the comments of this issue. RCT adds a confirmed-event label and local-digital-event tag to the issue and adds event to the Local Digital Events project. Issue will appear in the Upcoming Streams column
[ ] RCT adds a ready-for-marketing-edit label so the RCT can review title and description

8 Weeks Prior

[ ] RCT does a review of title and description and removes the ready-for-marketing-edit label
[ ] RCT moves the issue to the Ready for SharePoint column
[ ] RCT informs the Reactor Marketing Team of the new event information
[ ] RCT confirms the speaker
[ ] RCT provides speaker with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT moves the issue to the Content Preparation In Progress column

5 weeks Prior

[ ] Content owner opens a PR with the new content for the workshop on the customer-facing Reactor repository and adds the content-development and local-digital-event tag to the PR.
[ ] RCT reviews, approves, and merges the changes
[ ] RCT shares content with the vendor/instructor
[ ] RCT provides vendor/instructor with assets (e.g. slides, overlays) and the survey data for the event
[ ] RCT ensures that instructors have received an invite to the workshops and that PM team works with instructors to get setup in the platform (e.g. Teams, Teams Live Event)

1 Week Prior

[ ] Connect vendor/instructor with the PM(s) for the locations where this event has been marketed
[ ] Check-in with vendor/instructor that they are prepared for the workshop
[ ] RCT moves the issue to the Ready to Stream column

During and After Stream

[ ] Instructor (and optional moderator) run the stream
[ ] _(Optional)_RCT submits recording to be uploaded to the Reactor YouTube and to be made available on LearnTV
[ ] RCT moves issue to the Stream Complete column

@meaghanlewis - this looks great to me!


Raidboss overlay missing Timeline bars

Hello,
Since updating to 0.13.0 and changing over to the NGLD version of OverlayPlugin I have been unable to get Raidboss to actually function.
The first bit that seems odd is that the "demo" overlay for resizing and such is missing the timeline bars that normally show:

This is accompanied by Raidboss failing to pick up anything when I attempt to actually use the /bow or /countdown tests in Summerford Farms.
Here is my overlay setup:

There are no errors or warnings in the log that I can see except for the current last line:
[11/26/2019 1:38:54 PM] Info: WS: Warn: The current logging level has been changed to Info.
Any assistance would be greatly appreciated

What's your load order? Try loading OverlayPlugin and then Cactbot immediately after the FFXIV plugin.

My current load order has a discord callout plugin before overlayplugin.  I'll change the order and see if that fixes the issue.
Thanks.

Definitely worked, thank you!


LLVM Prebuilt Binary Download Fix

On windows I've been experiencing this problem where I can't seem to download the tar.gz without returning an error, this fixed it for me.

Disabling SSL verification is really dangerous (docs say this too), so i'd much rather not, i think you should verify what problems you are having with SSL and fix them on your side.


Be able to pass an additional callbacks argument to the train method of Sequence/Classifier object

It would be super useful in particular when you use delft as a library and want to implement a progress indicator during the training phase.
In practicle it could be:
def train(self, x_train, y_train, x_valid=None, y_valid=None, callbacks=None)
callbacks should be a list of valid Keras callbacks and be propagated to the underlying Trainer object and then concateneted to the already defined delft callbacks before calling the fit() method of the Keras object

I guess this is also solved with #75 ?

Yes, the ticket can be closed
Merry Xmas
Le lun. 23 déc. 2019 à 16:26, Luca Foppiano<EMAIL_ADDRESS>a
écrit :

I guess this is also solved with #75
https://github.com/kermitt2/delft/pull/75 ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kermitt2/delft/issues/67?email_source=notifications&email_token=ACEOBG4W6ZUACWZJD5CQ6NDQ2DKDLA5CNFSM4JZ2XVDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHRK4XI#issuecomment-568503901,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEOBG7RHC57MT7RDSCXO6DQ2DKDLANCNFSM4JZ2XVDA
.

🎄 Merry Xmas! 😄


Add functions to check Range, IP and CIDR. Included test and docs.

I accidentally closed #59 while cleaning up my forked repo 😓. All the commits have been squashed into a single commit for now.

IPv6 Consideration
My mind can't stop thinking about IPv6 implementation, so it's  better to write it here 😊
Looking back to the eval code
fun eval(formula: String, env: JsonObject): Float {
    try {
        return when (val result = RuleGrammar.parseToEnd(formula).eval(env)) {
            is Boolean -> if (result) 1f else 0f
            is Number -> result.toFloat()
            is String -> result.runCatching { toFloat() }.recoverCatching { 0f }.getOrThrow()
            else -> 0f
        }
...
}

If it was designed to convert any result to Float which is then be used in the isEnabled function.
I think we should reconsider using a separate function to process both IPv4 and IPv6 in 1 function, and this new function will return either 1f or 0f
Why?
Converting IPv6 can't be done with a single variable, but in this case, using an array is not possible either. Because the eval will always return Float.
And currently, I can't find a way to accurately represent 2 Long variables into 1 Float 😁
So adding IPv6 support in the current ip function seems to be incompatible with eval
Proposal
We can create one function to support both IPv4 and IPv6, i.e ipmatches("2001:4860:4860::8888/32", "2001:4860:4860::8888")
I know... we'll be back to the first proposal ipv4matches 😅
But, this single function will accept both IP formats, without needing to revamp the whole eval concept

About the PR itself, I left a couple of follow-ups on #59 since they were inline. Please have a look.
In general, I think we could merge IPv4 support and continue discussing IPv6 in a new ticket, but I'm fine to combine both together if you feel that's best. I'm enjoying thinking this through and ensuring we have a great design and implementation. Thanks for the back-and-forth!

If it was designed to convert any result to Float which is then be used in the isEnabled function.

Something to note about eval is that it's the last operation of all. There are intermediate representations of data (string, boolean, number, array, etc.) and that's OK. In fact, many functions expect to work with certain data structures that are not floats.
It's also worth noting that the existing functions -- eq, lt (+ all others), contains -- don't make many assumptions. As long as something understands equality and implements Comparable, it should work. For example, if ipv6 returned an internal class backed by 2 longs that implements Comparable and equals/hashCode, and cidrv6 returned a range between 2 of those instances, the rest should mostly work out of the box.
We could also look into supporting both formats under ip and cidr. I don't think it would require many changes, but to be fair, I haven't prototyped it.

And currently, I can't find a way to accurately represent 2 Long variables into 1 Float 😁

It's also impossible to represent a single Long in a Float, since a few bits go into exponent precision. The max value is around 2^53, not 2^64! But hopefully, it's clear why that's not a problem -- converting to Float is the very last step, and we're actually just looking to get it within the [0,1] range. 😊

I will address the comments in https://github.com/Doist/ffs/pull/59#discussion_r796191947 to finalize IPv4 implementation
And then I'll open a new issue on IPv6 to keep the discussion relevant 😊

There's also a failing test case:
assertEquals(3232238083f, eval("""ip("<IP_ADDRESS>")"""))

Interesting... this test case was passing when I ran it from IntelliJ (by running the commonTest folder)

If you have alternative ways to run the test suites, please let me know. Would like to make sure if my implementation is bug free 😊

The way CI runs tests is here: https://github.com/Doist/ffs/blob/9b1e61c9538708831155d3ea06efd70e0d8519a9/.github/workflows/ci.yml#L21
I see the failing test in the checks of this PR. Don't you?

The way CI runs tests is here:
https://github.com/Doist/ffs/blob/9b1e61c9538708831155d3ea06efd70e0d8519a9/.github/workflows/ci.yml#L21
I see the failing test in the checks of this PR. Don't you?

Wow, this one got me good! 😅 never should've carelessly converted Long to Float in the first place, let alone created a test for that

It's also impossible to represent a single Long in a Float, since a few bits go into exponent precision. The max value is around 2^53, not 2^64!

Thank you for pointing this out!

The way CI runs tests is here:
https://github.com/Doist/ffs/blob/9b1e61c9538708831155d3ea06efd70e0d8519a9/.github/workflows/ci.yml#L21
I see the failing test in the checks of this PR. Don't you?

Wow, this one got me good! 😅 never should've carelessly converted Long to Float in the first place, let alone created a test for that. All failed tests came from JS platform, I'll make sure to check tests on all platforms before committing

It's also impossible to represent a single Long in a Float, since a few bits go into exponent precision. The max value is around 2^53, not 2^64!

Thank you for pointing this out!

All failed tests came from the JS platform, I'll make sure to check tests on all platforms before committing

Ah, yes, JS is tricky. The entire reason castEval exists is because of JS's leniency with types.

CI is green, we're good to go! There are a few tiny changes I'd like to make, but I'll do them right after. The PR is gold, thanks for contributing!

You're welcome, thanks for all the time you've set aside to review my pull request!
I've opened new issue #65 to start implementing IPv6


Only use HTTPS maven repositories

It has been known for a long time that its important to use HTTPS for maven repositories to avoid attacks on developers, build systems, and end users.
This commit updates repos to use HTTPS and more up to date spring.io domains.
It also removes the objectstyle.org maven repo which doesn't seem to be used, is http-only, and seems likely to go away eventually [1]
[1] http://www.objectstyle.com/objectstyle-org-and-community-roots

Coverage decreased (-0.08%) to 84.889% when pulling 07a3e26209238b30e483a37aca6f691ae25e465a on davidillsley:gradle_sources into caa4aa3ea74df289c5563214009b0846230eda04 on cloudfoundry:develop.


Save in folder file dialog not working as intenden on Windows

The file dialog for saving one or multiple files on batch operations in a folder uses the AcceptSave mode (which linux supports but Windows doesn't)

Issue fixed in commit 2523ce05c853e32a92ff184eed5f263f113599d1 - reverted to AcceptOpen dialog for Windows compatibility


⚠ Samsung has degraded performance

In 166a7f0, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2343 ms

Resolved: Samsung performance has improved in a76012b.


pypi upload

It seems the project is not in the pypi yet. can you upload it?
:skull:

now its on pypi and you can install it with
pip install swproject
https://pypi.python.org/pypi/swproject/0.1.1


CHEBI:32044 wrong charge

CHEBI:32044 has a sodium anion, probably not intended. The charge is not given but importing the SMILES in the editor shows the whole having -2 charge. Please fix.

Now fixed. Thanks


Text is not painted to the Canvas at the correct position on Flutter 3.0.0

Steps to Reproduce

Execute flutter run on the package example app with two different flutter versions:

Flutter 2.10.0
Flutter 3.0.0

Expected results:

Actual results:

For some reason, the text is not painted (painting operation occurs here) at the same position on these two different versions. To paint the background, I use the LineMetrics generated by TextPainter.generateLineMetrics positions. They are the same for both versions (you can run the tests and see), so something may be wrong on paiting the text into the Canvas.

Code sample

https://github.com/bdlukaa/rounded_background_text/blob/master/example/lib/main.dart#L142-L159

Logs

Analyzing example...
No issues found! (ran in 41.3s)

[√] Flutter (Channel stable, 3.0.0, on Microsoft Windows [versÆo 10.0.22000.675], locale pt-BR)
    • Flutter version 3.0.0 at C:\Users\bruno\Documents\flutter\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (11 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\bruno\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.7)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31911.196
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.67.2)
    • VS Code at C:\Users\bruno\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.40.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [versÆo 10.0.22000.675]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 101.0.4951.67
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 101.0.1210.47

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

This is caused by https://github.com/flutter/flutter/issues/104153

Marking as duplicate then.


Context menu not wide enough for "Format option or selection with"

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the *Problem Description, *Steps to Reproduce and Version sections below

Problem Description
The context menu option "Format file or selection with [formatter]" appears on top of the keyboard shortcut. This has happened to me since spyder 4 in Windows 10, yet I don't see that problem in my other installation in a GNU/Linux distribution. This is independent of the formatter. Be it autopep8 or black, they all appear the same way.
What steps reproduce the problem?

Open the context menu by right-clicking on any file.
Check the last option regarding formatting.

What is the expected output? (Taken from my spyder installation in a GNU/Linux distro)

What do you see instead?

Versions

Spyder version: 5.0.2
Python version: Python 3.8.10 64-bit
Qt version: Qt 5.12.9
PyQt version: PyQt5 5.12.3
Operating System name/version: Windows 10

Hi @CamiloMartinezM. Thanks for reporting this. I can't reproduce this in Mac either. @dalthviz can you test this on Windows an let me know if you can reproduce it?

@juanis2112 @CamiloMartinezM  I was able to reproduce this on Windows with Spyder 5.0.3 installer version:


Clarification Request: Table.ToNavigationTable's itemNameColumn Param

Under section "Table.ToNavigationTable", that function's itemNameColumn is described as follows:

The name of the column to use to determine the preview behavior. This is typically set to the same value as itemKind.

Could you clarify the specifics of what values are expected to be found in the column that's referenced, and how those values will be interpreted? (Example: If I specify the itemNameColumn as "Col1" from my nav table, what values in that column will Power Query interpret as telling it to delay vs. not to delay previewing).
Per https://github.com/microsoft/DataConnectors/issues/30, it sounds like a good way to test the preview behavior is to add a function to the navigation table. If the nav table display shows input boxes for the function's params, the function had its preview delayed; if no param input controls are displayed, preview was not delayed.
Using the test code below, it appears that previewing is not delayed if this param set to a text value, regardless of whether that value is a column name in the nav table, and previewing is delayed if the param is set to null, a logical or a number.
Clarification on how this param works would be most appreciated! -) Thanks!
Example Tester:
[DataSource.Kind="MyTester", Publish="MyTester.Publish"]
shared MyTester.Contents = () =>
    let
        SomeFunction = (a as text) => Text.Upper(a),
        Base =
            #table({"Name", "Data", "ItemKind", "ItemName", "IsLeaf"},
            {
                {"string: 'Function' [literal value typically found in ItemType column] (Params Don't Preview)", SomeFunction, "Function", "Function", true},
                {"string: 'Non-Existent Column' [invalid column name] (Params Don't Preview)", SomeFunction, "Function", "Non-Existent Column", true},
                {"string: 'Name' [valid column name] (Params Don't Preview)", SomeFunction, "Function", "Name", true},
                {"string: 'Data' [valid column name] (Params Don't Preview)", SomeFunction, "Function", "Data", true},
                {"Null (Params Preview)", SomeFunction, "Function", null, true},
                {"false (Params Preview)", SomeFunction, "Function", false, true},
                {"true (Params Preview)", SomeFunction, "Function", true, true},
                {"123 (Params Preview)", SomeFunction, "Function", 123, true}
            }
        )
    in
        Table.ToNavigationTable(Base, {"Name"}, "Name", "Data", "ItemKind", "ItemName", "IsLeaf");

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

ID: 9e5bc95e-99a6-0d05-b3d5-c064daa417d0
Version Independent ID: dfb79a5a-5419-44dc-389a-f4e46b5247c6
Content: Handling navigation for Power Query connectors - Power Query
Content Source: powerquery-docs/HandlingNavigationTables.md
Service: powerquery
GitHub Login: @cpopell
Microsoft Alias: dougklo

Notes after further testing:
It looks like Table.ToNavigationTable's param itemNameColumn (which maps to metadata field Preview.DelayColumn on the navigation table that's returned) has the following effects:

If, for a given row, the column corresponding to itemNameColumn contains a text value:

That value will be used as the text displayed in the navigation table's Data column.
Example: see red marks, below.

The Data column's value will have its previewing delayed.

If, for a given row, the column corresponding to itemNameColumn contains something other than a text value:

The value from the column identified by itemKind will be used as the text in the nav. table's Data column.
Example: see blue marks, above.
The Data column's value will not have previewing delayed.

Can you confirm/correct the above, and if correct, clarify the docs for this param?
Thanks!

Please ignore: I'm closing and opening this issue to get automation to add tags. #please-close

Hi @bgribaudo
Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Because this issue has been open for more than a year and a half, we are closing this issue for now. However, if you feel that this issue should be addressed in the documentation, you can always open a pull request in the powerquery-docs repo.


Reopen #16221 (Parameter Hints shown inline all the time (like IntelliJ))

Parameter and type hints shown inline in the editor like IntelliJ.
The rust-analyzer extension has them (https://github.com/rust-analyzer/rust-analyzer/blob/master/docs/user/features.md#code-hints) but they do have bugs since this is not a officially supported by vscode.
The issue mentioned #16221 was closed because of the recommendation for using a plugin that only works with typescript.

That issue isn't closed, it's still open to track this issue. It was just locked due to spam.


Route rules with cors: true does not work

Environment
nitropack: 2.9.6

Reproduction
https://stackblitz.com/edit/github-xaqmuq?file=nitro.config.ts
Describe the bug
Setting cors: true to does not work:
routeRules: {
    '/**': {
      cors: true,
    },
},

The browser still throws cors error:

Access to fetch at 'http://localhost:3333/' from origin 'http://localhost:3001' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

With header, too:
routeRules: {
    '/**': {
      cors: true,
      headers: {
        'Access-Control-Allow-Origin': '*',
      },
    },
  },

Error:

Access to fetch at 'http://localhost:3333/' from origin 'http://localhost:3001' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: It does not have HTTP ok status.

Additional context
I already opened an issue at Nuxt that setting corse:true does not have an effect on /api routes, only on /public folder.
Logs
No response

It seems to work when setting the header (as shown above):
routeRules: {
    '/**': {
      cors: true,
      headers: {
        'Access-Control-Allow-Origin': '*',
      },
    },
  },

But it seems like it does not match too nested routes:
await $fetch(/admin/products/${productId}/status, { method: 'PATCH' }); -> Ok
await $fetch(/admin/products/${productId}/images/{$id}, { method: 'DELETE' }); -> Error
await $fetch(/admin/products/${productId}/images/{$id}/order, { method: 'PATCH' }); -> Error
@pi0 Is this an issue with an UnJS package maybe?

Hi @pi0 , I digged deeper into this and found a critical bug with route rules not beeing applied. Maybe a problem with https://github.com/unjs/radix3?
The following rule works for all routes:
routeRules: {
    '/**': {
      cors: true,
      headers: {
        'Access-Control-Allow-Origin': '*',
      },
    },
  },

BUT if a route does not have GET route, the routeRoule is not applied. E.g.:
/products/index.ts
/products/index.patch.ts
/items/index.patch.ts -> Route rule not applied

Therefor the routes without a GET route throw a cors error in my case.
I made a reproduction repo, you can simply run bun install and then bun dev to start a Nuxt app at localhost:3000 and a Nitro app at localhost:3001
https://github.com/MickL/nitro-route-rules-bug

It gets even more weird!
The route rules are applied if there is a GET route, e.g. contact.ts and contact.post.ts BUT if the GET route just throws an error it is NOT applied:
Route rule applied for POST route:
export default defineEventHandler(async (event) => {
   return '404';
});

Route rule NOT applied for POST route:
export default defineEventHandler(async (event) => {
  throw createError({
    statusCode: 404,
  });
});

This is insane. I spent the last 8 hours trying to figure out what was going on with my CORS headers not being set. I added a GET route and it fixed all of my problems.
I'm genuinely regretting using nitro.

Possibly related to #1806

This is definitely a critical bug and an annoying one as well. Maybe it will be fixed with v3? @pi0
But in any case it needs to be fixed for v2 aswell. This also makes problems within Nuxt: https://github.com/nuxt/nuxt/issues/25686

I have created a better sandbox to show Nitro behavior:
https://stackblitz.com/edit/github-mdtiap?file=server%2Froutes%2Findex.ts,server%2Froutes%2Fapi%2Ftest.post.ts
Nitro actually do apply route rules to all methods regardless if there is a handler registered for them or not. (notice that there are cors headers to the post-only API route's response).
The issue is how CORS preflight works in browsers. Browsers use an OPTIONS request to check CORS availability (not even GET) and the reason adding [path].ts (without method suffix) fixes the problem is that it also handles that.
I understand it can be frustrating with CORS issues but it isn't a bug at least with how route rules work.
(...so what we can improve...)
I understand DX is not good and CORS issues, god knows how much frustration they cause every developer, and that when Nitro allows to set a magical route rule cors: true, we expect that it just works, and it should be.
We can improve our route rule handler to auto-respond to OPTIONS even if there is not a handler but the problem is that it is too soon. Route rules are applied before any of the user handlers and if a user wants to actually handle OPTIONS method of a path (to apply their security policy -- the very purpose of CORS functionality) nitro will implicitly override this. So while it might look like an easy fix/feature, it might cause security implications which are worst than a not working CORS call when it matters.
I will be thinking about how we can safely implement this as a fallback approach (nitro error handler could be one place, by checking cors route rule) but i think one first and good step is that we at least document this:
CORS functionality needs API handlers to respond to OPTIONS requests and as a userland fix, one can add server/api/[...path].ts
// server/api/[...path].ts
export default defineEventHandler((event) => {
  if (event.method === 'OPTIONS') {
    return ""
  }
});

Why do we have

https://h3.unjs.io/utils/advanced#appendcorsheadersevent-options
https://h3.unjs.io/utils/advanced#appendcorspreflightheadersevent-options
https://h3.unjs.io/utils/advanced#handlecorsevent-options
https://h3.unjs.io/utils/response#setresponseheadersevent-headers
https://h3.unjs.io/utils/response#setresponseheaderevent-name-value

Do these only work in a get route file as well?

@ArcherScript Preflight request is not GET, it is OPTIONS.
I suggest you read MDN docs about Preflight request and CORS mechanism first.
Adding server/api/test.ts adds an event handler that handles all methods (GET, POST, OPTIONS, etc) and fixes the issue regardless of what it responds even an empty string with no headers is enough because route rule already adds * cors headers for us.
Browsers expect to have an "ok" response when they want to verify CORS (via OPTIONS fetch). If there is no event handler, Nitro (your application), responds with 404 or 405 both unacceptable by browsers.

@ArcherScript Preflight request is not GET, it is OPTIONS.
I suggest you read MDN docs about Preflight request and CORS mechanism first.
Adding server/api/test.ts adds an event handler that handles all methods (GET, POST, OPTIONS, etc) and fixes the issue regardless of what it responds even an empty string with no headers is enough because route rule already adds * cors headers for us.
Browsers expect to have an "ok" response when they want to verify CORS (via OPTIONS fetch). If there is no event handler, Nitro (your application), responds with 404 or 405 both unacceptable by browsers.
You can alternatively use handleCors h3 utility (instead of route rule shortcut that simply adds headers) to have better control over CORS strategy but you still need to add it to a catch all handler (without .post suffix)

Yeah this is what I thought. Thanks for the explanation.
It seems like server/api/route.ts is the ONLY option if you are trying to make a request from the browser. Which is not clear from the docs.
I understand why it is that way though from the explanation.

Should we expand the documentation on handleCors and other cors helper methods so that it explains how these only function in route.ts files?

Surely feel free to contribute to h3 docs if you could successfully use it 🙏🏼  jsdocs is here.
Mind you, nitro route rules cors handler does not use h3 utility btw. it uses simple headers (why? because it can be natively applied to deployment provider route rules).
I think we might also add a new ## cors section to the end of /guide/routing to mention different ways of handling cors but most importantly, that cors needs a catch all handler.

One thing is CORS but this issue actually turned out to a critical bug. Maybe it was not clear enough in the end but it took me some time to figure it out:
Route roules are not applied if a route does not have a GET route. For example:
/products.ts
/products.patch.ts
/items.patch.ts -> Route rule not applied

Or did I miss something?

@MickL please locally run my reproduction from https://github.com/unjs/nitro/issues/2340#issuecomment-2308264327. Route rules always apply regardless of incoming request or if there is even a matching router handelr or not. they are applied on their own. CORS fails not because roue rule cors: true is not adding headers it fails because browser expects an "ok" (like 200) response and if there is no route handler, well it will be 404/405. A catch all handler makes sure of that 200 (and that i plan to make additional behavior to route rules to be able to auto-response, considering we can be sure of security)

But why does it work when adding the get route? Like for PATCH /product it works in my example. Sorry if I miss something.

/xyz.ts <-- This is a catch-all route (handling ALL methods)

Route rule adds headers for paths it is defined for (and works)
If there is no route handler that matches the path + method, the server responds 404 (not found) or 405 (method not allowed).

The browser makes a preflight to the same path with OPTIONS method. It expects 1) headers 2) ok response. route rules does (1), you need to do (2) by adding any kind of catch-all handler that simply gives that "ok".

I see, thanks for all the explanations!

Another issue with this is that non-existent routes return cors error instead of 404.


Got wrike working

I got Wrike working on my repo. I think the problems you were having were:

It expected the data formatted as query parameters
It requires the content-type header

I used my own env variables so they might not be called the same as the ones you used in the stream

By the way, I figured out that those were the problems by checking the curl example in the docs and testing requests on my own newly created free Wrike account using Postman (which I love using when I need to test APIs).
It worked the first time when using an urlencoded body but it didn't work with equivalent raw data, so I figured it probably was something to do with a mime type (I've had this problem before).
There are AFAIK 2 headers that deal with mime types: The Accept header that specifies what do you want your response to be (usually application/json) and the Content-Type header which specifies what is the data that you are sending (usually one of application/json, application/x-www-form-urlencoded or multipart/form-data).
Hope this helps for future debugging.

Thank you! It is amazing that you were able to figure this out so quickly!


Refactor the tests to use CPOClient instead of directly calling the RPCs

The current collections are created/accessed by directly calling the RPCs. Like
K2LOG_I(log::cpotest, ">>> Test2: create a collection");
    auto request = dto::CollectionCreateRequest{
        .metadata{
            .name="collection2",
            .hashScheme=dto::HashScheme::HashCRC32C,
            .storageDriver=dto::StorageDriver::K23SI,
            .capacity{
                .dataCapacityMegaBytes=1,
                .readIOPs=100,
                .writeIOPs=200,
                .minNodes=1
            },
            .retentionPeriod = 1h*90*24
        },
        .rangeEnds{}
    };
    return RPC()
    .callRPC<dto::CollectionCreateRequest, dto::CollectionCreateResponse>(dto::Verbs::CPO_COLLECTION_CREATE, request, *_cpoEndpoint, 1s)
    .then([](auto&& response) {
        auto& [status, resp] = response;
        K2EXPECT(log::cpotest, status, Statuses::S201_Created);
        return seastar::sleep(100ms);
    });

And cannot gurantee that the collection has been assigned. Should change to use the CPO Client's methods for creating/accessing the collections.

As in  #335


[Improvement request] Object in the Input machine

Ability to have input object as collection to be selected for futur usage (ie. input update)
Can be extended to many machines

Since the new version where we are able to define class to subset the input or menu machines the target is reached.
Now those machines can use with callback function, via new or existant object.
Thanks a  lot ;)


Support for Uno Platform WebAssembly

Describe the solution you'd like
Currently it is possible to use AppCenter in almost all Uno Platform targets, but not with WebAssembly. Having it supported would be very valuable for fully cross-platform app analytics.
This might be tackled simultaneously with #1084.
Describe alternatives you've considered
Additional context

Hi @MartinZikmund, thanks for opening the feature request!
At the moment our .NET SDK is heavily coupled with Windows and requires an additional effort to make it WASM compatible. Details:
https://github.com/microsoft/appcenter-sdk-dotnet/issues/1513
I don't have an ETA for it, cause at the moment App Center is focused on performance and reliability, rather than implementing new features or supporting new platforms.


Use slice instead of substring.

Besides being shorter, the semantics of slice are usually preferred in my experience. 🎉

Coverage remained the same at 100.0% when pulling 70128423a8a30cbd405c4a71939ab6780179ea2d on braddunbar:slice into 518a65525a3dbdb707a8499345cc4150e2123255 on developit:master.

Works for me! Need to double check gzipped size (sometimes these things are crazy).

Funky - I'm seeing slice() show up as 1b larger when gzipped. Maybe because substring is used elsewhere?

Changes Unknown when pulling b6fad739feceb00efbe2055f6d9e5115de3de286 on braddunbar:slice into ** on developit:master**.


Add column width and wrap

Resolves #30

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.56%. Comparing base (7fd3da8) to head (5381169).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   99.53%   99.56%   +0.03%
==========================================
  Files          54       57       +3
  Lines         856      921      +65
  Branches      204      220      +16
==========================================
+ Hits          852      917      +65
  Misses          4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Spike: Logging Enhancement for Logging Context/Custom Attributes and additional log level filtering.

Description
Review the feature specs outlined in:
NEWRELIC-6825
NEWRELIC-6826
Please reconcile the ask from the community:
https://github.com/newrelic/newrelic-ruby-agent/issues/1141
Acceptance Criteria

Discover the path to implementation.
Document in the milestone document the stories needed.
Highlight the following in your milestone template:

Design Consideration/Limitations
Why is this the route we should take to achieve our goal?
What can't be achieved within this story?
Dependencies
Do any other teams or parts of the New Relic product need to be considered?
Some common areas: UI, collector, documentation
Estimates
Please provide the initial t-shirt size. S = 1-3 days,  M = 3-5 days (1 week), L = 1-2 weeks (1 sprint)
Solicit from your peers there estimates for each story.

Spike Document


Replay #90 Pull Request due to hard revert

Replay #90.

Codecov Report

Merging #98 (87c628f) into main (216116a) will decrease coverage by 2.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #98      +/-   ##
==========================================
- Coverage   97.50%   95.27%   -2.24%
==========================================
  Files           5        5
  Lines         281      296      +15
==========================================
+ Hits          274      282       +8
- Misses          7       14       +7

Impacted Files
Coverage Δ

service/routes.py
91.13% <0.00%> (-8.87%)
:arrow_down:

Continue to review full report at Codecov.

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


Is it possible to change login url from one place?

From @Aleksej-Shherbak on Saturday, November 16, 2019 5:53:24 PM
I will describe what I mean here.  By default, Identity has the following login URL: /Identity/Account/Login. I would like to change it on just /Login.
And I have it done! For this I have added on the Login.cshtml file the following code:
@page "/Login"

And it works, but after logout I have redirect on the previous URL and 404 error. I have added the following on the Startup.cs to avoid this behavior:
services.PostConfigure<CookieAuthenticationOptions>(IdentityConstants.ApplicationScheme,
                opt =>
                {
                    opt.LoginPath = "/Login";
                });

And it works. Now I have redirection on  /Login after logout.  But if I want change URL again I must replace two places, not one. It's uncomfortable. Is it possible from one place? I tried this:
services.PostConfigure<CookieAuthenticationOptions>(IdentityConstants.ApplicationScheme,
                opt =>
                {
                    opt.LoginPath = "/Login";
                    opt.LogoutPath = "/Login";
                });

But without any effect.
Somebody suggested me to keep login URL in the appsettings.json but, I think, it's nonsense. I think, appsettings.json is the most unexpected place for URLs.
Any ideas?
Copied from original issue: dotnet/core#3822

Maybe related to my feature request: #17216

Identity is designed to be in a separate area. I guess you could fudge a route, but that's really an MVC question
@pranavkm ?

@javiercn would have more context here.

I think the question is more a general one than specifically the Login page:

But if I want change URL again I must replace two places, not one. It's uncomfortable. Is it possible from one place?

I believe he's asking if there is a way to hard code a route once and then use a reference to it wherever relevant instead of having to hard code it everywhere. Is that right @scalablecory ?


LocalCachingAuthenticator Impl

Closes https://github.com/pac4j/pac4j/issues/247
Let's review the design, and once that is consolidated, I'll add tests, etc.

Thanks


Enhanced CMD.exe prompt

Summary
Enhance CMD.exe with features found in modern shells.
Powershell is ok for scripting, but is a step too far for an everyday working shell.  Using bash (either in wsl or via cygwin) causes issues as paths are not as expected when you invoke tools.
For example things missing are

Ability to customize the prompt based on output of a program/script (e.g. powerline)
GNU readline support

This is possible by third party extensions like  https://github.com/mridgers/clink but it is not a supported solution.
Basic example
In the PROMPT I can see at a glance things like git branch/ kubernetes cluster / directory when you have multiple consoles open
Motivation
As a developer I often drop to the CMD.exe console to perform many tasks and have many consoles open at once when debugging / tracing applications.
Being able to efficiently use the CMD.exe benefits me by providing me with more contextual information so I am more efficient, and means I am less likely to delete the wrong kubernetes cluster.

why PowerShell is not suitable for you?

It does bad things like curl is not curl

unfortunately changing cmd in any way is a breaking change for someone and so the team responsible for it cannot/ are not interested in adding features to it. (see https://github.com/microsoft/terminal/blob/main/doc/Niksa.md#cmd and https://github.com/microsoft/terminal/issues/10573)


Build error with auto-generated API.swift file

Describe the bug
I am getting a compile error with the API.swift file that was generated
Key path value type 'Snapshot' (aka 'Dictionary<String, Optional<Any>>') cannot be converted to contextual type 'Snapshot?' (aka 'Optional<Dictionary<String, Optional<Any>>>')

Here are two examples of the code where I see the errors
public init(listUsers: [ListUser?]? = nil) {
    self.init(snapshot: ["__typename": "Query", "listUsers": listUsers.flatMap { $0.map { $0.flatMap(\.snapshot) } }]) // error on this line
}

public var listUsers: [ListUser?]? {
   get {
      (snapshot["listUsers"] as? [Snapshot?]).flatMap { $0.map { $0.flatMap { ListUser(snapshot: $0) } } }
   }
   set {
      snapshot.updateValue(newValue.flatMap { $0.map { $0.flatMap(\.snapshot) } }, forKey: "listUsers") // error on this line
   }
}

Steps To Reproduce
Steps to reproduce the behavior:
1. Create an Aurora database on AWS
2. Create a table via query editor
3.  When asked if you have an existing graphQL schema say yes and import your schema
5. Run `amplify push` and when it asks should it auto-generate models type yes.

Expected behavior
API.swift is auto-generated so it shouldn't have any compile errors when adding into the project
Amplify Framework Version
1.12.0
Amplify Categories
API
Dependency manager
Swift PM
Swift version
5
CLI version
5.3.0
Xcode version
Version 12.5.1 (12E507)
Relevant log output
No response
Is this a regression? (i.e. was this working before a version upgrade)
No response
Device
iPhone 8 - Simulator
iOS Version
iOS 12
Specific to simulators
No
Additional context
No response

Closing this issue and reposting this issue over amplify-cli because I feel like it's more of a CLI issue than the amplify-ios framework
https://github.com/aws-amplify/amplify-cli/issues/7995


Add "default_room" to profile settings

Joined on start if set.
I first tried with OwnedRoom::try_from: https://github.com/ulyssa/iamb/commit/e80b86b4087ea096c7f3cee5c03ccbd9ec4da520#diff-42cb6807ad74b3e201c5a7ca98b911c5fa08380e942be6e4ac5807f8377f87fcR136, but I guess IambWindow::find is more flexible.

I fixed an issue with the tests, and added the PR number to the commit message. Once the CI finishes, I'll merge it and cut a release.

Oh I should have run the tests :man_facepalming: thanks for fixing it!

I've just published v0.0.4. Thank you for the change! :)


fixedGame

To fixed the snake game with JS which can correctly move and eat food.

deleted some css for id: canvas.

adding module to organize js files.

updated


How can I visualize the action?

Hi, I want to visualize  the action after training, but I notice the "d4rl/scripts/visualize_dataset.py" writed 'Only MuJoCo-based environments can be visualized', If I want to visulize 'kitchen-complete-v0',how can I do ? thanks very much!

I try to change it ,and I change the code in
d4rl/d4rl/kitchen/kitchen_envs.py
"    def render(self, mode='human'):
return super(KitchenTaskRelaxV1, self).render()"

@csufangyu Does it work?
In my case, gladLoadGL Error happen.

It is because of the attempt to use GLFW as default. You can try to set this if you are running your code on a headless server: MUJOCO_GL="egl". You can look at the Rendering headline at https://github.com/deepmind/dm_control for more details and requirements to use EGL.

I try to use wrapper to modify render function, but it shows "KitchenTaskRelaxV1" is not defined


Reload page automatically after applying throttling

Current behavior

apply throttling
page has to be reloaded new throttled conditions

Expected behavior

apply throttling
page automatically reloaded

Hi, I hope it will fix it https://github.com/denar90/sloth/pull/5


how to avoid banner reloading ads

hi,
i use "Native UI Component Method" to show banner ads, how can i avoid reloading the ads?

If you unmount the AdView, our plugin should automatically destroy the banner and thus stop it from reloading ads. Or are you asking for an API to stop auto-refresh while the banner is on screen?

i dont want to destroy it,but to reuse the adview, can i ?

We can add a new API - such that you can specify whether you want the banner to auto-refresh or not. It'll just be another attribute on the AdView tag, does that work?

oh no.i dont mean auto-refresh.
i need auto-refresh.
but,my problem is "my banner ad will ask applovin for a new ad always, even though it doesnt exceed auto-fresh time"

when i refresh the app page, the banner will ask for a new ad again.
how can i reuse the ad in this page last time?

That's because a new instance of the component is created. So you want the banner to not ask for a new ad once it is re-mounted?

yes yes.

ok we'll have to investigate further. We likely won't be able to get to it until mid Q1 as we have higher priority items for the MoPub transition, so if you want to give it a try before we do that would be awesome. This the related code (Android, iOS).

if i use "Programmatic Method",cant i resolve this problem?

can "Programmatic Method" set the ad banner into the page, instead of on the top of the page?

Yes, if you use programmatic method you can resolve it, but we only accept certain position parameters, such as "top_center", "bottom_center", "centered", "top_left", "top_right", "bottom_left", "bottom_right".

We can add a new API - such that you can specify whether you want the banner to auto-refresh or not. It'll just be another attribute on the AdView tag, does that work?

Were that implemented?


TextInput blocking other fields

Need an autocomplete="off" like property for the TextInput component.
Detailed description

As you can see from above, the autocompletion is blocking my other input fields (the second input needs to be visible as I type into the first input)

Is this issue related to a specific component?

TextInput, Form

What did you expect to happen? What happened instead? What would you like to see changed?

An autocomplete prop that i can set to "off"

What browser are you working in?

Chrome

What version of the Carbon Design System are you using?

^7.18.0

You should be able to add autocomplete="off" on to TextInput and the component will pull it from ...other object and put it on the <input>. Have you tried adding autocomplete="off" yourself?

This issue was moved to carbon-design-system/carbon-components-react#220


Adds Publishing tasks

Added tasks for publishing the processed JREs to Terasology's Artifactory. Fixes #2
Also refactored some of the code.

Tested out - nice and easy 👍


[Bug Report] Console Warning

Hello! I recently updated to latest version of ServerVariables and today I noticed the following error in console: https://pastebin.com/xrgfBejX
I am not sure what it's related to, but I don't have any problems other than that error popping up a few times a day.
Thank you.

I will try to fix this in the next update.


Image dialog enhancements; control size, placement, etc.; edit existing images

Dialog infrastructure improvements:

support more flexible dialog layouts
support checkboxes and selects (popup menus)
onChange callback gives plugins a chance to validate values
dialogController.update() gives plugins a chance to update values
plugins can trigger dialog directly (e.g. by double-click on node)

Image dialog improvements:

specify class, alt, title, width, height properties from dialog
validate size values to be numeric/integer
dynamically constrain size to aspect ratio when appropriate
double-click on image to edit image properties

placement always shows as inline when I double-click on an existing image and bring up the dialog. Should it restore the previous state?

I must have broken that with my most recent "code cleanup" because that was definitely working, and I thought I tested after the refactor as well. 🤔

constrain proportions is always checked when I double-click on an existing image and bring up the dialog

I thought I checked that as well, but maybe not. I'll take a look at both.
Thx

A couple other notes tied to our conversation yesterday:

I mentioned image URL validation happening in the dialog update, but I no longer think that is needed.  The broken image appearing in the editor is more than enough to let the user know that they need to fix their image URL.
Good move to update the dimensions when the source image is changed.  I was unsure what the right path should be if the user modifies dimensions, but later modifies the source image.  I think the current implementation is correct.  Modifying the source image should reset the dimensions to the source image dimensions.

Also very minor, but I think "Image URL" instead of "Source URL" is clearer in the dialog (and more in line with the link dialog).

Should be fixed now.


7215 add analyses and layers source to viz json for named maps

This closes #7215. @alonsogarciapablo could like to try this to check if it's enough for polling to work?

@gfiorav CR this, please.

Apart from a comment, looking sharp ✏

Working like a charm! Thanks @juanignaciosl! 👍

📦 it!

Can one of the admins verify this patch?


Increasing doc page content.

Adding sample use from generate functions.

Hello @tbfreitas nice contribution! Can you update pt-BR docs too?

@all-contributors please add @tbfreitas for docs.

Hello @tbfreitas nice contribution! Can you update pt-BR docs too?

@all-contributors please add @tbfreitas for docs.

Hello @tbfreitas nice contribution! Can you update pt-BR docs too?

Sure I can.
I`m gonna do it as soon as I can.

Hello @tbfreitas nice contribution! Can you update pt-BR docs too?

Done, my friend.
So I also did a little fix on function exposing on general docs.

Sorry @tbfreitas, I accidentally removed all-contributors-rc... Can you revert that change and keep this file equals main branch?
Just to heads up, we have a bot to do these updates, you can check #228 here.

Not problem at all, I already put it back.
I also had forgot to back it to main branch version..

Thanks for your contribution @tbfreitas !


Countdown widget

vote by reacting 👍 or 👎

Yeah, I like the idea of this being a homepage module. Maybe it's part of the module that presents the live stream, since it'd be replaced with the livestream when the counter reaches 0.

feature request:

high precision timing
if reduced motion no millisecond updates
css tip: use tabular nums
when less than 24 hours (tbd) switch to hyper precision


Fix invariant violation when maintainVisibleContentPosition adjustment moves window before list start

Summary:
https://github.com/facebook/react-native/pull/35993 added logic in VirtualizedList to support maintainVisibleContentPosition. This logic makes sure that a previously visible cell being used as an anchor remains rendered after new content is added.
The strategy here is to calculate the difference in previous and new positions of the anchor, and move the render window to its new location during item change. minIndexForVisible is used as this anchor.
When an item change moves the anchor to a position below minIndexForVisible, shifting the render window may result in a window which starts before zero. This fixes up _constrainToItemCount() to handle this.
Changelog:
[General][Fixed] - Fix invariant violation when maintainVisibleContentPosition adjustment moves window before list start
Differential Revision: D47846165

FYI @janicduplessis

Looks good, thanks for fixing this!


Error encoding service control url

Forum:	article "Open.NAT - A NAT Traversal library for .NET and Mono"
Subject:	3 bugs   (View)
Sender:	gunnarD
Date:	Thursday, August 11, 2016 9:53am

Problem in UpnpNatDeviceInfo:
Had to change from this
var builder = new UriBuilder("http", locationUri.Host, locationUri.Port, serviceControlUrl);
ServiceControlUri = builder.Uri;

to this
var builder = new UriBuilder("http", locationUri.Host, locationUri.Port);
ServiceControlUri = new Uri(builder.Uri + serviceControlUrl.TrimStart('/'));

else I got HTTP error 404 Not found (problem with url, /control?WANIPConnection became /control%3FWANIPConnection

it was fixed in commit 790b407481fe3dc876ceee00461626a452f27d32  for issue 22 Error when Service control urls have question marks


Add Markdown linting

Is your feature request related to a problem? Please describe.
The present markdown has numerous issues, so get it fixed
Describe the solution you'd like
Put pymarkdownlnt check into CI so that repo stays in good shape
Describe alternatives you've considered
Tried the Ruby based mdl but found it less flexible.
Additional context
No response

Done with #118


refactor: adds discriminator in CommonModel and interpreters

This is a preparation PR to support union for objects in Java (https://github.com/asyncapi/modelina/pull/1264). It adds discriminator in the CommonModel, and the interpreters set that value when they should.
Description

adds discriminator in the CommonModel
sets discriminator in the interpreters

Related issue(s)

@jonaslagoni , please review. I'll try to make smaller PRs this time to make the review process easier :)


Security - Updating API versions of alerts and pricing types

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node

Can one of the admins verify this patch?

@chlahav does anybody else need to review this or are we good?

@jhendrixMSFT we are good on our side

@AutorestCI regenerate azure-sdk-for-go


retina.js runs automatically despite using import or require

To use retina.js the new-school way, you'll want to require it (or import it if you're using ES6) into your Gulp/Webpack/Grunt/CommonJS/etc application. In this case, the script won't run automatically. Instead, it'll let you determine when you'd like it to run.

When I do import 'retinajs' or require('retinajs') the window.retinajs method runs automatically. Isn't it supposed to only run when I call it manually?
This contradicts what is written in the docs. Am I missing something here guys?

Good catch @fahmilatib. It appears there's a bit of misinformation in the docs. I'll adjust it accordingly alongside an updated release, which should fix the issue.

Any update on this?


#439 Fix not accepted course groupings showing in search result

This PR closes: #439

I'm not sure this will work, because recall there might be multiple accepted versions of a grouping (the old versions are still marked as accepted) and it seems they'd still be included in this query, wouldn't they? If you added in your test method a third grouping that's accepted with version = 2, I think your test would fail.

Yes, but the controller is expected to return a list of course groupings so I thought we have to return all the accepted ones. Unless you want to return only the currently published one?

I'm not sure this will work, because recall there might be multiple accepted versions of a grouping (the old versions are still marked as accepted) and it seems they'd still be included in this query, wouldn't they? If you added in your test method a third grouping that's accepted with version = 2, I think your test would fail.

Yes, but the controller is expected to return a list of course groupings so I thought we have to return all the accepted ones. Unless you want to return only the currently published one? In that case, I could just check the published attribute and get the lastest version.

We'd want a list of 10 groupings, but only the most recent accepted version of each grouping (not all the accepted versions). You should be able to use a very similar query to this one to achieve it https://github.com/JamesPartsafas/ConcordiaCurriculumManager/blob/1801bebe74acb7c3b5b26d1f9fe2116ea0ee537a/Server/ConcordiaCurriculumManager/Repositories/CourseRepository.cs#L76

not all the accepted versions

Is this for GetCourseGroupingsBySchool as well? In that case, why was it returning a list if it getting only the most recently accepted course grouping for a school?


BlobCache.LocalMachine is NULL on Android but only in Release mode

I have quite a frustrating issue when deploying my Xamarin.Forms Android App in release mode. The BlobCache.LocalMachine is null when in release mode but when I run the app in debug mode, there are no issues with it.
I am not doing anything out of the ordinary with the library, do you have any idea why this would be happening?

Try to exclude Akavache from the Xamarin Linker

@flagbug  I just tried this and it didnt work. The LocalMachine property is still null

@DavePilks Did you add Akavache and Akavache.Sqlite3 to the linker exclusion? Also, do you have a project so we can reproduce this?

I added both and then even turned linking off.
Ill try and recreate the problem in a sample app as I cannot add the current project for IP reasons

@DavePilks Any update on this?

If this is still an issue, lat me know and I'll reopen this


Fix typo

Hi @kazufumi-nishida-www,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
http://www.lightbend.com/contribute/cla

I agreed CLA and received Lightbend CLA signed mail :)


Fix/missing dep

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


BLD: Remove traitlets from requirements; it is not used at all.

Our dependency on traitlets went about with global state. We do not used it anywhere
(confirmed with git grep traitlets).
This removes it from requirements.txt and .travis.yml.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #789   +/-   ##
=======================================
  Coverage   85.77%   85.77%
=======================================
  Files          18       18
  Lines        4793     4793
=======================================
  Hits         4111     4111
  Misses        682      682

Continue to review full report at Codecov.

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


Ripme Error JSON Video not found

Ripme version: 1.7.95

Windows 11

I was trying to rip a subreddit community and this error popped up and stopped the rip.
I have tried deleting the appdata and redownloading and it still is not working. The history in the program was clear when I re-opened it.
Does anyone know what is happening and how to fix it?

same here. it now just shows failed to load [url] .json after 1 attempts on each url on reddit

Would love to know the solution or workaround for this issue. I don't want to rip videos, I wish it would just skip them.


add react-virtualized to jsspreadsheets.com

https://jspreadsheets.com/ is one of the best sites to discover spreadsheet components for javascript. It is frequently cited in discussions on stackoverflow and hacker news.
React-virtualized is conspicuous by its absence. The way to add it is here - https://jspreadsheets.com/submit-new.html
I do request the maintainers to add themselves to jsspreadsheets.com

Hi @sandys 😄
Looks like anybody can submit a new grid library via a PR to krzysztofspilka/JSpreadsheets. Would you be willing to add react-virtualized to the list?
I'm going to close this issue since it's not really a react-virtualized issue. It would be cool if someone added the lib to jspreadsheets.com though. 😁


⚠ Samsung has degraded performance

In 43e7381, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2491 ms

Resolved: Samsung performance has improved in 18d2b67.


Add ferial cycles

We should add ferial cycles.
There are two ferial cycles:

I. cycle (every odd calendar year);
II. cycle (every even calendar year).

This would helps us later to determine, what reading, etc are used in a particular ferial Mass (#20), but also with #74, #12). We need to add some strings to the src/locales just to enable the localisation.

Similar to cycle (years A, B and C), the returned object could looks like:
data.meta.ferialCycle: {
    key
    value
}

Where the key/values could be:
odd: 'I. cycle'
even: 'II. cycle'

And better, the values should refer to locale keys, so we can get them in any language.

Also the current data.meta.cycle (for A B C years) should be renamed to data.meta.sundayCycle.

Agreed. I not sure though if it is (by liturgists) called Sunday cycle though; I searched the Net, but have not found a better translation. And also, Sunday cycle is in contrast with ferial cycle, so it might be beneficial from this point of view, too. Therefore you can rename it. :smiley:

Just tried with the master branch and the feature where it shows the "Monday of the 14 week of Ordinary Time" for the date 2020-07-06 does not seem to be there. Can I assume that this will only come in version 2.0?
Currently using the alpha or beta version mentioned in this thread.
Check out what we are trying to achieve at this test site.
https://romcal.sound.sg/iraivarthai/
It will also change colour according to romcal.
The live site is,
https://www.iraivarthai.org/
FYI. Sometime later we will be contributing Tamil translation.

Hello @pvanthony, thanks for your comment and thank you for sharing us your project you are working on!
Yes the next major version will output weekdays by default if the possible memorial for this day is optional.
I plan to make this new major version out for ~ mid-August. But I can't be 100% sure of the ETA (I am still committed to other projects currently).

Done in the _F-TypeScript branch (soon in the dev branch). Please have look to the corresponding documentation: https://github.com/romcal/romcal/blob/_F-TypeScript/docs/data-output.md#weekday-cycle

Addressed on https://github.com/romcal/romcal/pull/250.


Open recent file

Hello,
Not sure where to put this, it's maybe a feature request?
I'd want to have a way to open recent files.
Is there a plugin that can do this? Configuration options? Something else?
Also, is there a sort of catalog of all available configuration? Ideally something that wouldn't require trawling through all source code and plugins, could/does lite-xl have something like introspection (i.e. automatically generating a list based on all exported symbols)?

@costincaraivan  i'm just putting together some code to have recently used commands populated first in "find-command" so the command you just ran is the first one on the list of suggestions.
this is sounds like a similar idea,
you would like to have "find-file" show some suggestions for recently opened files?
what would the suggestion list look like?
would you have it show the files that already have open views? like maybe you forgot one of the files is already open in a tab?
or just  files that you recently opened, but are now closed?
would you want the entire list of suggestions to be ordered by recent use?
that is actually what i ended up doing with the commands.


Issues with alerting (when comparing 2 Prometheus queries)

I am facing an issue when trying to set up an alerting, by comparing expressions e.g.:
pg_stat_activity_count{namespace="my_namespace"} > pg_settings_max_connections{namespace="my_namespace"} * 0.9
alerts are not firing as expected. But If I put a digit to compare with, e.g.
pg_stat_activity_count{namespace="my_namespace"} > 10 , then it works fine.
When I execute my queries in prometheus both pg_stat_activity_count{namespace="my_namespace"}  and pg_settings_max_connections{namespace="my_namespace"} * 0.9 separately give valid results.
So it should work, but for some reason it doesn't. I've tried all possible brackets, quotations whatever, but nothing helps.
Did anyone encounter a similar issue? Am I missing something here?
Thank you in advance!
Prometheus version:  2.20.1
Postgres version: 10.12

This is due how prometheus work with metrics, basically when you try to use diffrent metrics you need to have same labels on both sides. Which there is not by default because pg_stat_acitivy_count return labels as datname which aren't in pg_settings_max_connections
We have multiple instances and using for each psql separate exporter, which is diversed by our instance label in our usecase. You can manipulate labels via sum() or other PromQL functions. This is what we used for connection usage in percetage.
round(sum by (instance) (pg_stat_activity_count{instance=~".+"}) / sum by (instance) (pg_settings_max_connections{instance=~".+"}) * 100,0.1) > 85
Don't just copy paste alerts you see online, you should try them always on prometheus GUI first to see how they behave.

@frenkye Thank you very much for your answer, when I was looking at 'pg_stat_activity_count' and 'pg_settings_max_connections' on prometheus I've seen that both contain 'namespace' label, I thought that it will be the joining point (intersection), but it seems like it's not working like that. Yes we also have multiple instances and I've created dedicated exporter for each one of them, that's why using labels is important to create meaningful alerts.
Your sophisticated query gives a good example, I will try to play around to see if I can adopt it to my environment!
Thanks again!


Is the test split available?

Thank you for your effort in releasing this dataset! It is an invaluable resource to the community.
The README for the parallel reconstruction demos mentions that we should expect to have 3 splits -- train, val, and test link. However, the zenodo link does not have a multicoil_test zip file. Is there another way we can access this file?

Thank you for your effort in releasing this dataset! It is an invaluable resource to the community.
The README for the parallel reconstruction demos mentions that we should expect to have 3 splits -- train, val, and test link. However, the zenodo link does not have a multicoil_test zip file. Is there another way we can access this file?

Thanks for your interest. We are inspecting the motion issue of the test subset (~30 subjects) and will release it in a few weeks. You may use the val subset for test subset for now.

@ad12 Please note we have just released the test set: https://zenodo.org/record/8056074


pull request for e2e test at 01/11/2017 01:26:15

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


[Enhancement] Re-release workflow should use corresponding tag in opensearch-build to do re-release instead of main branch

[Enhancement] Re-release workflow should use corresponding tag in opensearch-build to do re-release instead of main branch.
The current changes in security plugin caused a lot of changes in main branch of opensearch-build: https://github.com/opensearch-project/security/issues/3624. If we bound to re-release older versions of docker images, we would like to use the corresponding tag code to release, instead of introducing new changes from main back to older version, such as config and entrypoint changes.
Thanks.

PRs:

https://github.com/opensearch-project/opensearch-build-libraries/pull/373

2PR with @zelinhao on 20240201 to re-release 1.3.14 and 2.11.1 version.
Confirmed that build is using the corresponding tag instead of main:
Checking out Revision 6444bd923de1057bb1c1d1e585181d2287dbb5b3 (1.3.14)
Commit message: "Replace refs in manifests with tags for 1.3.14 (#4282)"
 > git rev-parse --resolve-git-dir /var/jenkins/workspace/docker-build/.git # timeout=10
 > git config remote.origin.url https://github.com/opensearch-project/opensearch-build # timeout=10
Fetching upstream changes from https://github.com/opensearch-project/opensearch-build
 > git --version # timeout=10
 > git --version # 'git version 2.25.1'
 > git fetch --tags --force --progress -- https://github.com/opensearch-project/opensearch-build +refs/heads/*:refs/remotes/origin/* # timeout=10
 > git rev-parse origin/1.3.14^{commit} # timeout=10
 > git rev-parse 1.3.14^{commit} # timeout=10
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 6444bd923de1057bb1c1d1e585181d2287dbb5b3 # timeout=10
 > git rev-list --no-walk 6444bd923de1057bb1c1d1e585181d2287dbb5b3 # timeout=10

......

Checking out Revision dce080075c219010371c02e699e816dd4df7758f (2.11.1)
Commit message: "Update 2.11.1 tags in OS and OSD manifests (#4258)"
First time build. Skipping changelog.
[Pipeline] withCredentials
 > git rev-parse --resolve-git-dir /var/jenkins/workspace/docker-build/.git # timeout=10
 > git config remote.origin.url https://github.com/opensearch-project/opensearch-build # timeout=10
Fetching upstream changes from https://github.com/opensearch-project/opensearch-build
 > git --version # timeout=10
 > git --version # 'git version 2.25.1'
 > git fetch --tags --force --progress -- https://github.com/opensearch-project/opensearch-build +refs/heads/*:refs/remotes/origin/* # timeout=10
 > git rev-parse origin/2.11.1^{commit} # timeout=10
 > git rev-parse 2.11.1^{commit} # timeout=10
 > git config core.sparsecheckout # timeout=10
 > git checkout -f dce080075c219010371c02e699e816dd4df7758f # timeout=10

Without security admin:admin changes in here: https://github.com/opensearch-project/security/issues/3622
% docker run -it -d -p 9200:9200 -p 9600:9600 -e "discovery.type=single-node" opensearchstaging/opensearch:<IP_ADDRESS>97.20240201
7f347291427ed0db5be0b7da341300056154659c3b2105bb2990cb60a1a980f9

% curl -XGET https://localhost:9200 -u admin:admin --insecure

{
  "name" : "7f347291427e",
  "cluster_name" : "docker-cluster",
  "cluster_uuid" : "rLg_6PcTTI2tHVZ_RJPaMQ",
  "version" : {
    "distribution" : "opensearch",
    "number" : "1.3.14",
    "build_type" : "tar",
    "build_hash" : "21940d8239b50285ef7f98a1762ef281a5b1c7ee",
    "build_date" : "2023-12-08T22:13:08.793451Z",
    "build_snapshot" : false,
    "lucene_version" : "8.10.1",
    "minimum_wire_compatibility_version" : "6.8.0",
    "minimum_index_compatibility_version" : "6.0.0-beta1"
  },
  "tagline" : "The OpenSearch Project: https://opensearch.org/"
}

% docker run -it -d -p 9200:9200 -p 9600:9600 -e "discovery.type=single-node" opensearchstaging/opensearch:<IP_ADDRESS>42.20240201
fcc05ec988405f82eb0b129e39b0e9b3abfcbe0bce4f3f1a28c822410b29cf4c

% curl -XGET https://localhost:9200 -u admin:admin --insecure

{
  "name" : "fcc05ec98840",
  "cluster_name" : "docker-cluster",
  "cluster_uuid" : "pOa8aALvTsGqEYrH-XNv0g",
  "version" : {
    "distribution" : "opensearch",
    "number" : "2.11.1",
    "build_type" : "tar",
    "build_hash" : "6b1986e964d440be9137eba1413015c31c5a7752",
    "build_date" : "2023-11-29T21:43:10.135035992Z",
    "build_snapshot" : false,
    "lucene_version" : "9.7.0",
    "minimum_wire_compatibility_version" : "7.10.0",
    "minimum_index_compatibility_version" : "7.0.0"
  },
  "tagline" : "The OpenSearch Project: https://opensearch.org/"
}

https://build.ci.opensearch.org/job/docker-re-release/10/console
https://build.ci.opensearch.org/job/docker-re-release/11/console
https://build.ci.opensearch.org/job/docker-re-release/12/console
https://build.ci.opensearch.org/job/docker-re-release/13/console

https://build.ci.opensearch.org/job/docker-scan/2820/artifact/scan_docker_image.txt
https://build.ci.opensearch.org/job/docker-scan/2821/artifact/scan_docker_image.txt
https://build.ci.opensearch.org/job/docker-scan/2822/artifact/scan_docker_image.txt
https://build.ci.opensearch.org/job/docker-scan/2823/artifact/scan_docker_image.txt


No need to add "/" at the end of url when url ends with "="

We should not add '/' to the end of url when url ends with '='. This was causing a malformed url to Parameter summary screen when viewing list of Parameters under Cloud Stack.

Malformed URL: http://localhost:3000/orchestration_stack/parameters/10000000000005?show=/10r15
Correct URL: http://localhost:3000/orchestration_stack/parameters/10000000000005?show=10r15
Issue was introduced in commit: 2e6ce4ccd76a724495dc016613418302afbb430c

https://bugzilla.redhat.com/show_bug.cgi?id=1271514
@tenderlove please review, i think this issue might have been introduced during restful routes PR.

screenshots:
before:

after:


Tags to show a list of all pages with that tag

Is your feature request related to a problem? Please describe.
As a user, I would like tags to open a page that contains a list of all the notes with that tag, so I can create some structure in the exported website
Describe the solution you'd like
When I create a tag using '#tag', that tag should be a hyperlink that then opens a new page (in the same window). That page should contain:

A title: The tag name
A list of links in alphabetical order, each of which is labelled with the name of the page, and the href URL is the URL of the page

The list only contains the pages that contain the tag.

This is something I have been thinking about adding, thanks for the suggestion!

(This is also related to issue #67)

Note for future: Once search support is added (#120) clicking on a tag could instead bring up a list of notes in the sidebar as a search for that tag rather than loading a whole new page.

This feature has been added in 1.8.0 as long as you have the search bar enabled. (it will simply search the vault for that tag). I may improve this behavior in the future, but I will close this issue for now. If you have ideas for improvments on this create a new issue or open a disscusion in the discussions tab.
Thanks! :)


Fix incorrect ordering of args in HttpRemoteTaskRunner warning log

Description
I was tracking down a failed ingestion task and noticed that this log is incorrect.
Failing task[<IP_ADDRESS>:8080] because worker[index_parallel_datasource_abcdefg_2024-02-04T00:00:00.000Z] disappeared and did not report within cleanup timeout[PT15M].

Should be:
Failing task[index_parallel_datasource_abcdefg_2024-02-04T00:00:00.000Z] because worker[<IP_ADDRESS>:8080] disappeared and did not report within cleanup timeout[PT15M].

Made a tiny change to fix the typo.
I'm assuming this is a pretty safe change so I've skipped the testing and verification, let me know if I need to be more thorough.

Release note

Fixed incorrect log statement in HttpRemoteTaskRunner

Key changed/added classes in this PR

HttpRemoteTaskRunner

This PR has:

[x] been self-reviewed.
[x] a release note entry in the PR description.

The following IT failure is unrelated and has been failing in master recently:
[n=openjdk8, Cluster Build On K8s) ITNestedQueryPushDownTest](https://github.com/apache/druid/actions/runs/7805521890/job/21292856960?pr=15846#logs)


Feature - Casting Document to any Class and Saving a class object directly to Database.

I am developing using this library on Android. I need to store multiple JSON documents in the database, and I also need to retrieve them.
But every time when I want to save a document in the database. I have to do these steps:

I have to Convert my class object to JSON
Then this JSON to HashMap<String, Object>
and then I need to create a new Document. -> calling database.createDocument();
Then I need to call document.putProperties(HashMap object here..);

and while retrieving document:

I need to change it to JSON.
Then JSON to my class object.
and then return it to  the API user.

But there should be some generic way to do it so that I can get the document which be be able to cast to any class directly.
If still I have to use above procedure, then why not make it a part of Couchbase-Lite-Android-Core API rather than making it a overhead to API user.

HI,
I would like this feature as soon as possible too, like many others..
Atfer 1 year, do we have something ?
BR

Sorry, no progress on this yet.  Some recent discussion here: https://forums.couchbase.com/t/is-there-any-orm-for-android/2501


Remove duplicate entries in history

See:
Remove duplicate entries in a file without sorting
http://www.commandlinefu.com/commands/view/4389

Thanks for your terrific plugin! ❤


Update Luno

re-add logging
Retry insufficient balance due to server update delay
Enhanced getPorfolio
getOrder now return fees

Seems to be working fine! (based on limited tests)


[gfx][colour] Separate surface manipulation functions from colour tra…

…nslation functions.
Reduce code duplication in the future. copyrect, fillrect, and putpixel are identical regardless of the colour representation (8bit rgb vs 8bit mono). Shifting the responsibility of flattening colours away from these functions.

LGTM

Also I saw what you did there.

Did I do something clever without realizing it? :stuck_out_tongue:


Throw runtime exception if pointer to structure is null inside StructureInspector

Was thinking of doing exactly this, thanks!


No compile

Problem

Initially, only source were available, now compile version are available for arm64, amd64 and armhf.

Solution

update source to already compiled binaries

PR Status

[x] Code finished and ready to be reviewed/tested
[ ] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

!testme

!testme

!testme


Fix Avatar Fallback component position on other email clients

Gmail (Chrome)

Outlook 2021 (Win 10)

By the way, I added a dedicated page in the local preview server to facilitate testing:

https://demo.react.email/packages/avatar-image
https://demo.react.email/packages/avatar-fallback


added file structure from activity 23, needs modified to reflect code…

… for our project

great!

Great!!

looks good !!


Handle reading values/schema from all sources

argocd supports:

helm repo
helm repo with auth
git
git with auth

So far, CLaaS handles only helm repo source

Im closing this one and will create a more specific issues for each source


querying/examples.md: Improve examples to work with stock Prometheus

This is currently geared towards node_exporter 0.15.x, will update once @SuperQ releases 0.16.0

@brian-brazil are you OK with merging for now? I will redo the whole thing for master once we have something more valid on the website.

This PR is against master, so should match master. This won't end up on the website until 2.3.0 is out.

Is this still relevant?

Yes, but probably needs some love to be updated.


[Question]:Unused variable 'pathExtension' Warning

What is your question?
I am using share_plus package(v:4.0.7) in flutter application. But when I get build for ios via Xcode, it gives a "Unused variable 'pathExtension'" warning. How can I solve this?

Can you tell us about the flutter version?

It is advisable to attach, the command that will accurately reproduce this problem.

@Chever-John  Hi, I am using flutter version 3.0.0 and only I just press build on Xcode

@BetulComak Got it, let me try this command on my end.

@BetulComak Hi, I have just tried to reproduce the problem.
The result is that everything is fine here.
The problem reproduction process is as follows. Please reply to me directly if you have a different opinion.
Configuration
My machine & env configuration:
macOS: 12.4 (M1)
flutter version: 3.0.1 (Maybe I should try 3.0.0)
Android Studio: Android Studio Bumblebee | 2021.1.1 Patch 2
Xcode: Version 13.4.1 (13F100)

Problem reproduction process
My pubspec.yaml
dependencies:
  flutter:
    sdk: flutter
  share_plus: ^4.0.7

Codes from this repository


Assumption mapping

https://www.youtube.com/watch?v=PyCvsBrKO4w
Possible for after design phase done

This exercise could be useful for multiple PL projects :)

Yeah it's a good one. Well actually all of thse would work probably for PL projects/products.

@autonome This one would be great for a lot of things, not just browser design guidelines, but out of scope for this project so closing
Would be a good kickoff for any additional next phases.


Reference function not working in true path of if statement

Inside a copy function I have the following statement:
"objectId": "[if(equals(parameters('clusterProvisioningKvAccessPolicies')[copyIndex('accessPolicies')].objectId, 'scalesetIdentity'),reference(resourceId(parameters('clusterProvisioningKvAccessPolicies')[copyIndex('accessPolicies')].resourceGroup, 'Microsoft.ManagedIdentity/userAssignedIdentities','scaleset'), '2015-08-31-PREVIEW', 'Full').properties.principalId,parameters('clusterProvisioningKvAccessPolicies')[copyIndex('accessPolicies')].objectId)]"
which causes the failure "The provided arguments for template language function 'if' is not valid: all arguments should be of type 'boolean'. Please see https://aka.ms/arm-template-expressions#if for usage details."
When I change the reference to a hardcoded the guid it passes.

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

ID: 85bba21f-ef02-381c-48bd-0ced44cc513f
Version Independent ID: c56b631f-e9f4-39ea-c2b2-a46132e5c286
Content: Azure Resource Manager template functions - resources
Content Source: articles/azure-resource-manager/resource-group-template-functions-resource.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@johnthcall - I ran a test with reference as the true path for an if expression within a copy, and it worked as expected. For you scenario, I would suggest looking at the error message stating all arguments should be of type boolean. I am not sure why it would state that. Are you using an if expression to assign a value that expects a Boolean? Or, perhaps the syntax is not resolving correctly.

#please-close

@martinoss - sorry, I should have updated this thread. Yes, the product team is investigating this issue. I have not heard an update about a potential fix.

Any update on this?  I'm facing a similar issue, except in my case I'm making a reference to one of my VM's MSI principalId.

I am also facing this issue, can you guys reopen it please?


No longer matching anything after upgrading to Ubuntu 16.04

Hi,
I have upgraded to the new LTS of ubuntu (16.04) and noticed that cmatcher doesn't match anything anymore. Regardless of what I type, "no entries" is returned.
I removed cmatcher completely and reinstalled/re-run the setup scripts:
➜  ctrlp-cmatcher git:(master) python --version
Python 2.7.11+
➜  ctrlp-cmatcher git:(master) ./install.sh
find python2 -> python
running build
running build_ext

but no luck so far.
I would be happy to try to debug it further but I am not sure where/what to look into next.
Thanks!

Same issue here. Only thing I can add is a warning during compile:
fuzzycomt.c:436:16: warning: function declaration isn’t a prototype [-Wstrict-prototypes]
 PyMODINIT_FUNC initfuzzycomt()
                ^

Same problem here. Would be happy to help debugging, but don't know where to start.

@ilikeorangutans @lejoko and for anyone else running into this issue:
I didn't have any luck with getting ctrlp-cmatcher to work for me, but since it was an essential part of my everyday development, I thankfully found a nice replacement for it (in most cases you can't tell the difference, I think): https://github.com/FelikZ/ctrlp-py-matcher

+1

It looks like my vim installation is compiled with -python +python3. If you run vim with -V9, you will see
line 19: sourcing "ctrlp-cmatcher/autoload/matcher.vim"
Error detected while processing ctrlp-cmatcher/autoload/matcher.vim:
line   26:
E319: Sorry, the command is not available in this version: python << ImportEOF
Error detected while processing ctrlp-cmatcher/autoload/matcher.vim:

The solution would be to port the plugin to Python 3 or rebuild vim with +python.


when i used aws-parallelcluster 3.7.2 build-image the  cloudformation  DELETE_FAILED

When I use pcluster build-image, the AMI is successfully created, but there is an error during resource cleanup, I see from cloudtrail that the reason for the error is that the CleanupLambdaRole created by default does not have the getrole, ListAttachedRolePolicies ListRolePolicies and so on.
But I don't have this problem in version 3.6.1, I can reproduce this problem stably with the following commands and configuration file
pcluster build-image --image-configuration image-config.yaml --image-id testpcluster361
(pvenv) [root@AWS-CLI pcluster-test]# cat image-config.yaml
Build:
InstanceType: c5.4xlarge
ParentImage: ami-0b48ec38e7bb9f2d1
SubnetId: subnet-01b7fa5bfe747ecce
SecurityGroupIds:
- sg-0f7e41c74de8b5dd9

Does this have anything to do with me being at BJS?

I have the same problem.

I have the same problem.
I specified a CleanupLambdaRole in the template and the problem could be solved

trust relation
{
"Version": "2012-10-17",
"Statement": [
{
"Effect": "Allow",
"Principal": {
"Service": "lambda.amazonaws.com"
},
"Action": "sts:AssumeRole"
}
]
}
#policy
{
"Version": "2012-10-17",
"Statement": [
{
"Effect": "Allow",
"Action": [
"logs:CreateLogGroup",
"logs:CreateLogStream",
"logs:PutLogEvents"，
"iam:DeleteRole",
"iam:DetachRolePolicy",
"iam:DeleteInstanceProfile",
"iam:DeleteRolePolicy",
"iam:RemoveRoleFromInstanceProfile",
"imagebuilder:DeleteInfrastructureConfiguration",
"imagebuilder:DeleteComponent",
"imagebuilder:DeleteComponent",
"imagebuilder:DeleteImageRecipe",
"imagebuilder:DeleteDistributionConfiguration",
"imagebuilder:DeleteImage",
"imagebuilder:GetImage",
"imagebuilder:CancelImageCreation",
"cloudformation:DeleteStack",
"ec2:CreateTags",
"tag:TagResources",
"lambda:DeleteFunction",
"lambda:RemovePermission",
"logs:DeleteLogGroup",
"SNS:GetTopicAttributes",
"SNS:DeleteTopic",
"SNS:Unsubscribe"
],
"Resource": "*"
}
]
}
But I still don't know why my stack deletion failed

Hi everyone, this is a "known" issue, due to some changes in the ImageBuilder service that they are reverting back.
The issue shouldn't impact AMI creation but only deletion of the stack.
The workaround is to delete it from CloudFormation service, saying to preserve the leftover Role (that doesn't really exist).

Still valid, but resolving as duplicate of: https://github.com/aws/aws-parallelcluster/issues/5914


[BUG] Integration tests failing for tests without security

What is the bug?
Integration tests are failing for tests without security, https://github.com/opensearch-project/opensearch-py/runs/7397649625?check_suite_focus=true

Failure fixed in PR #180