Don't ignore .git folder in docker build

Description
This makes sure our docker build includes the .git folder, which should give us access to the git tag and let us build the version string 🤞
Fixes (hopefully) https://github.com/ConduitIO/conduit/issues/895

IIRC, it was removed to reduce Docker build time. WDYT about getting the version from git and saving it to a file, then sending it to Docker?

I just tried it out to see what difference it makes, I ran docker build -f Dockerfile -t conduit-local . and then removed the image two times in a row (second run caches almost everything excpet loading the context). I ran it on main and on this branch, here are the results:

main first run 79.8s, second run 5.0s
lovro/dockerignore first run 74.4s, second run 6.3s

I wouldn't say that's a noticeable difference and I don't think we should optimize this further if it solves the problem, it's just not worth it. Especially because our docker builds are triggered nightly and on releases, a few seconds don't make a difference there.

Definitely, just one second of difference is not worth it.


Update to kubernetes 1.10

This patch updates the vendor directory after the recent changes to the Gopkg.toml file.

This depends on #73, should be rebased once that is merged.

/lgtm /approve


Feature Request: Add support for cookie headers and parameters in OpenAPI schema

Use case
Original discussion: https://github.com/aws-powertools/powertools-lambda-python/discussions/3609
OpenAPI supports Headers and Cookies and parameters and we must add support for this!
https://swagger.io/docs/specification/describing-parameters/
Cookie Parameters
Operations can also pass parameters in the Cookie header, as Cookie: name=value. Multiple cookie parameters are sent in the same header, separated by a semicolon and space.
Header Parameters
An API call may require that custom headers be sent with an HTTP request. OpenAPI lets you define custom request headers as in: header parameters.
Solution/User Experience
Add Query and Header in https://github.com/aws-powertools/powertools-lambda-python/blob/develop/aws_lambda_powertools/event_handler/openapi/params.py
Alternative solutions
No response
Acknowledgment

[X] This feature request meets Powertools for AWS Lambda (Python) Tenets
[X] Should this be considered in other Powertools for AWS Lambda languages? i.e. Java, TypeScript, and .NET

Assigned to @rubenfonseca


Add base_path setting to atmos config

what

Add base_path setting to atmos config
Add atmos logo
Update README

why

Provide/configure a base path for all components and stacks
Allow controlling stacks and components paths using only one setting or ENV var
Can also be set using ATMOS_BASE_PATH ENV var, or --base-path command-line argument
Supports both absolute and relative paths
If not provided or is an empty string, components.terraform.base_path, components.helmfile.base_path and stacks.base_pathare independent settings (supporting both absolute and relative paths)
If base_path is provided (via ENV var or command-line arg), components.terraform.base_path, components.helmfile.base_path and stacks.base_path are considered paths relative to base_path

test
ATMOS_BASE_PATH=./examples/complete atmos terraform apply test/test-component -s=tenant1-ue2-dev

Found ENV var ATMOS_BASE_PATH=./examples/complete

Variables for the component 'test/test-component' in the stack 'tenant1/ue2/dev':

enabled: true
environment: ue2
namespace: eg
region: us-east-2
service_1_name: service-1
service_2_name: service-2
stage: dev
tenant: tenant1

Writing variables to file:
examples/complete/components/terraform/test/test-component/tenant1-ue2-dev-test-component.terraform.tfvars.json

Is this new atmos logo in creative commons ?

Does this close PR https://github.com/cloudposse/atmos/pull/96 ?

Does this close PR #96 ?

No, it's a diff thing (providing paths to the code via inputs, not reading them from atmos.yaml)


Guidance on storing money values in SQLite

I am seeking some guidance for my project which will need to store money amounts in a SQLite DB from a UWP app.
From my research the .NET type of decimal is the preferred type for money.  The rest of my question assumes this is correct.
My model has a type
public decimal Amount { get; set; }

and as I follow the document getting started for UWP and SQLite I create a migration.
I note that in the migration it appears to tell SQLite to create a column of type decimal.
My migration has the following:
Amount = table.Column<decimal>(nullable:False),

From my research SQLite has no decimal type and many people appear to prefer the use of an integer and do the old */100 to find the decimal in the integer ... now I am showing my age as that is an old COBOL trick.
I would love to just store the money and issue ef statements to sum the money and I should not care where ef decides to execute those statements (in SQL or client). The Utopia :-)
I am concerned that allowing ef to handle my money storage as it may not be accurate ... but then getting involved and rolling my own storage seems silly if I don't have to.
Hence here is my question on guidance?
I do not need to care as ef has me covered or I need to roll my own?
If I need to roll my own what is the consensus?  Use the integer and the */100 trick within my models to expose a decimal?
Regards
Chris ....
Further technical details
Database Provider: SQLite: 1.1.0 Tools: 1.1.0-preview4-final
Operating system: Win 10
IDE: Visual Studio 2015 patch 3

Microsoft.Data.Sqlite (the ADO.NET provider) will convert the decimals to TEXT for lossless storage. This can get a little weird when you perform operations on the columns in queries. See Datatypes In SQLite Version 3 for all the details. At a high-level, it will do its best to perform a lossless numeric conversion (e.g. to the equivalent of ulong or double), but it may have to resort to a lossy conversion.
So basically, if all you're worried about is reading and writing the values, you're good. But if you're performing operations on them in your queries, be aware of the corner cases.

Thanks @bricelam for such a quick response.
I am still fairly new to this .NET and EF world so I would to just spell out my assumption here and if you could confirm this I would appreciate it.
Using my sample code above if I wrote something like this:
using (var db = new myContext()) {    _TotalValue = db.Transactions.Sum(t => t.Amount); }
Then I assume ef will not issue raw SQL but would bring the data into my classes (where the amount is a decimal) and issue some form of Sum against my classes (ie on the client) to ensure correct values?
This would also be true for OrderBy and other methods ... as in ef will be aware of the storage as text and do all it's work on the client side where the data is a decimal and results will be correct.
I think this is the way it would work which means when the day comes that SQLite can handle money types and the provider is updated then ef will automatically start implementing SQL where appropriate of course.
Thank you for your time.
Regards
Chris ...

Sum will be translated to SQL by EF. To ensure it gets evaluated on the "client" (i.e. in .NET), use .ToList() for force EF to hit the database.
    _TotalValue = db.Transactions.ToList().Sum(t => t.Amount);

Comparisons are always be lossless on SQLite. When using .OrderBy(), the rows will be ordered by their TEXT values.
If SQLite ever adds additional primitive types (they only really have four: TEXT, BLOB, 'INTEGER&REAL), we will certainly light them up in Microsoft.Data.Sqlite` and Entity Framework Core.

You can use Logging to see the queries EF is sending to the database.

As mentioned by @bricelam , when using .OrderBy(), the rows will be ordered by their TEXT values on the database.  This produces weird order results for my 'Price' column. Say, $9 will be deemed larger than $13. In my model definition, Price is of type decimal. And in the database, it is converted to TEXT. Do you have a way to make it converted to 'REAL'? Or do you have other solutions to make the sorting happen numerically?
Thanks.

In EF Core 2.2, OrderBy() should be evaluated in .NET giving you the expected results (fixed by PR #11889)

Is there a workaround to fix it in a EF 1.X configuration ?

@jonathanantoine Adding .AsEnumerable() (or ToList or ToArray) before the .OrderBy() will force client-eval.

@bricelam I still have this issue on 2.2.4 : normal ? Here is a repro project : TestProject1.zip

@jonathanantoine Can you clarify? What are you trying and what do you expect to happen?

@bricelam I am trying to store decimals in a DbContext using a Sqlite driver and be able to order them :)

The OrderBy should be evaluated on the client (i.e. in .NET). SQLite doesn't naively support 128-bit floating point operations.

I agree but as you can see in my repro project, it does not seems to be the case : or am I doing something wrong in my project setup ?

...?
Test started...
 Summary: Passed: 1, Failed: 0, Ignored: 0

I am really sorry @bricelam : this was an issue with my IDE which didn't clear my cache and was still using the previous EF version.


[Bug] no protection against moving when there are no XYZ devices

Problem
The XY (or Z) buttons have no protection against being called even if there is no valid device. So if you click them then they will throw this error:
RuntimeError                              Traceback (most recent call last)
File ~/Documents/oss/micro/napari-micromanager/micromanager_gui/main_window.py:497, in MainWindow.stage_y_up(self=<micromanager_gui.main_window.MainWindow object>)
    496 def stage_y_up(self):
--> 497     self._mmc.setRelativeXYPosition(
        self._mmc = <CMMCorePlus at 0x7fb4f961cf40>
        self = <micromanager_gui.main_window.MainWindow object at 0x7fb4f9fa8f70>
        self.xy_step_size_SpinBox = <PyQt5.QtWidgets.QSpinBox object at 0x7fb4f9595280>
    498         0.0,
    499         float(self.xy_step_size_SpinBox.value()),
    500     )
    501     if self.snap_on_click_xy_checkBox.isChecked():
    502         self.snap()

RuntimeError: No device with label ""

Solution
disable those buttons if there is no device available to be moved.

True @ianhi! I just open PR #99 that should fix this.


env: add environment parsing

This enables support for the new syntax described in sourcegraph/sourcegraph#15822.

700 added and ZERO removed?? You don't see that every day :-)


fix redundant file "1"

This fixes the redundant file '1' issue reported by @omidkrad in https://github.com/endjin/git-hooks/issues/2#issuecomment-131958924

Good spot! I just face palmed when I saw your change...

Yeah, I told you! ;)

It's the problem with jumping between C# and PowerShell all the time - so similar, yet almost fatally different!


Identical tests generated by ContestEstimator

Description
There are identical tests generated for io.seata.core.store.StoreMode.valueof in seata-core-0.5.0 with ContestEstimator.
To Reproduce

Run the 'utbot' project in IntelliJ Idea 2022.2.4
Open utbot-junit-contest/src/main/kotlin/org/utbot/contest/ContestEstimator.kt
Change javaHome property to full local path to JDK 8
In fun main change: projectFilter = listOf("seata-core-0.5.0")
Run fun main - make sure to set JRE in Edit Configurations to JDK 8.
When run is finished, check utbot-junit-contest/build/output/test/seata-core-0.5.0/io/seata/core/store/StoreModeTest.java

Expected behavior
Tests are supposed to be minimized - if several tests are covering same branches, only one should be present in resulting tests class.
Actual behavior
There are three identical tests on IllegalArgumentException for io.seata.core.store.StoreMode.valueof
Trying to call enum's method valueOf with abc, ab, h - while enum values are file, db.
Visual proofs (screenshots, logs, images)
Here is the generated test class:
package io.seata.core.store;

import org.junit.Test;

public final class StoreModeTest {
    ///region Test suites for executable io.seata.core.store.StoreMode.valueof

    ///region

    @Test(expected = IllegalArgumentException.class)
    public void testValueof1() {
        StoreMode.valueof("abc");
    }
    ///endregion

    ///endregion

    ///region Test suites for executable io.seata.core.store.StoreMode.valueof

    ///region

    @Test(expected = IllegalArgumentException.class)
    public void testValueof2() {
        StoreMode.valueof("ab");
    }
    ///endregion

    ///endregion

    ///region Test suites for executable io.seata.core.store.StoreMode.valueof

    ///region

    @Test(expected = IllegalArgumentException.class)
    public void testValueof3() {
        String string = "h";

        StoreMode.valueof(string);
    }
    ///endregion

    ///endregion

    ///region Test suites for executable io.seata.core.store.StoreMode.valueof

    ///region

    @Test(expected = IllegalArgumentException.class)
    public void testValueof4() {
        StoreMode.valueof(null);
    }
    ///endregion

    ///endregion
}

Environment
Windows 10 Pro
IntelliJ IDEA 2022.2.4
JDK 8

@korifey @EgorkaKulikov regarding Summaries.
Please check the test generated above.
Regions should be:

either joined together, only one should be left for same method, without imbedded regions inside
or removed at all


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

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


Update gluon tutorial link

Description
(Brief description on what this PR is about)
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[x] Feature1, tests, (and when applicable, API doc)
[x] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Job PR-483/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-483/1/index.html

Job PR-483/3 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-483/3/index.html


organizations card style improved.

Organizations card style improved for better visibility

Good work on improving the card style for better visibility! Keep it up!


Use latest version of grafana/ui and grafana/aws-sdk

In a recent change in Grafana, the dependency to grafana/aws-sdk was moved from grafana/ui. This means that any plugin that support SIGV4 auth need to depend on the grafana/aws-sdk and pass a SIGV4 editor as prop to the DataSourceHttpSettings. To get details on why we're doing this change, please refer to this PR in grafana.
This means we'll have to upgrade this plugin to use latest version of grafana/ui and grafana/aws-sdk and release a new major version of this plugin. Any existing version of this plugin will be incompatible with grafana 9.0.0 and later.
My goal was to put up a PR with a working example, but since this plugin runs 7.5 of grafana packages there's simply too many necessary changes that are unrelated to the breaking change in the DataSourceHttpSettings so I'll need help. I also lack context in this domain as well as ways of testing this data source.
This draft PR includes changes necessary for rendering SIGV4 correctly.

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

Hi @sunker,
if okay for you, I close this PR in favour of https://github.com/grafana/opensearch-datasource/pull/48.
Best
@svennergr


Don't force inlinable constructors to delegate in non-resilient code

This restriction came from wanting to make resilient and non-resilient code follow the same rules whenever possible, but after thinking about it a bit more we realized there was no reason why you wouldn't just mark your structs @_fixed_layout in non-resilient libraries anyway. Since that (currently?) doesn't affect what you can do with the struct across module boundaries, and since the layout of the struct is available anyway in a non-resilient library, there's no real downside, which means it's a meaningless restriction.
The same logic doesn't quite apply to classes, since classes are normally much more flexible than structs. (For example, you could add a stored property to a class without recompiling clients, as long as no initializers are inlined.) But it's close enough that we don't want to put in the restriction at this time.
All of this is about attributes that haven't been finalized yet anyway (hence the leading underscore), but it's still useful information.
rdar://problem/37408668

@swift-ci Please test

@jckarter, can you review this since both Slava and Doug are out? You can check out the Radar for a bit more background, too.

So, the correctness condition here is "can all modules that can inline this constructor access the type non-resiliently?".  It's not totally clear to me that the new condition actually establishes that; if we're adding a constructor to a type outside its defining module but with non-resilient access to it, is it possible that someone might see our constructor that doesn't have such access?

No, this only applies to constructors defined within the same module. Constructors across modules are still subject to SE-0189, because otherwise adding a new field is a source-breaking change.
For constructors defined in the same module, I think that condition is sufficient, yes.

Ah, I see.  Yes, in that case I agree.


Emotes don't seem to be downloaded after editing config and running twt

Describe the bug:

Emotes don't appear in the terminal or in ~/.cache/twt/ after following the configuration instructions.
To Reproduce:

sudo pacman -Syu twitch-tui
run twt in kitty, complete the config wizard, then exit with q
set twitch_emotes and betterttv_emotes to true in ~/.config/twt/config.toml
twt then connect to any chat
Alt+e to bring up the emote menu: it is empty
send "PogChamp" in the chat: the text "PogChamp" appears and not the emote

Expected behavior:

I expected either the menu to contain emotes, or that the emote PogChamp would appear in the terminal when sending "PogChamp". I also expected ~/.cache/twt/ to have been populated at some point in the process, as indicated in the documentation.
Screenshots:

System:

Twitch-tui version: 2.6.10-1

Operating System: Arch Linux w/ kernel 6.9.3-arch1-1

Terminal: kitty 0.35.1

Shell: Bash 5.2.26(1)-release

Additional context:
I also tried uninstalling twitch-tui with pacman, removed the config and cache directories, reinstalled with cargo (twitch-tui version 2.6.10), and ran the exact same steps, with the same results.

Thanks for the issue. I'll try to diagnose what's going on within the next few hours.

I haven't been able to reproduce the issue as of now, although I am running the same system as you. I'll do a deeper dive in the near future.

Thanks for the quick replies! Feel free to ask for any details/screenshots/screen recordings, or if you need me to try things, when you get to it. I couldn't find similar reported issues, so I'm not excluding the possibility that I just did something silly on my end; I'll report back if it ends up being the case.

Could you send your entire config? Without the token, of course.

Could you send your entire config? Without the token, of course.

[twitch]
username = "luky75"
channel = "luky75"
server = "irc.chat.twitch.tv"
token = "redacted"

[terminal]
delay = 30
maximum_messages = 500
verbose = false
first_state = "Dashboard"

[storage]
channels = false
mentions = false

[filters]
enabled = false
reversed = false

[frontend]
show_datetimes = true
datetime_format = "%T"
username_shown = true
palette = "pastel"
title_shown = true
margin = 0
badges = false
theme = "dark"
username_highlight = true
state_tabs = false
cursor_shape = "user"
blinking_cursor = false
inverted_scrolling = false
show_scroll_offset = false
twitch_emotes = true
betterttv_emotes = true
seventv_emotes = false
frankerfacez_emotes = false
favorite_channels = []
recent_channel_count = 5
border_type = "plain"
hide_chat_border = false
right_align_usernames = false
show_unsupported_screen_size = true

Odd, that works for me as well. Did you modify your kitty config in any way that you're aware of?

I just changed the fonts and colors in kitty.

How long did you wait before exiting the terminal after typing PogChamp? I'm having a feeling it could be due to connection issues to twitch servers, which has caused problems in the past.

Not very long, maybe a minute. I just tried now and I'm keeping the window open for a bit

Neither, I followed those steps from Streamlink's docs:

In order to get the personal OAuth token from Twitch's website which identifies your account, open Twitch.tv in your web browser and after a successful login, open the developer tools by pressing F12 or CTRL+SHIFT+I. Then navigate to the "Console" tab or its equivalent of your web browser and execute the following JavaScript snippet, which reads the value of the auth-token cookie, if it exists:

document.cookie.split("; ").find(item=>item.startsWith("auth-token="))?.split("=")[1]

Then I would connect to my channel with twt and get an "Improperly formatted auth" error message from Twitch. So I prepended oauth: to the string I got with the above steps, and then I could successfully connect and send messages.
(Still no change since my previous comment, by the way)

Ah, that's the issue. I went the streamlink way, and I was able to replicate your issue.
You need to use one of the two websites that I posted above. The first gives more scopes if you want them, described in the default configuration. However, I suggest using the second link, twitchapps.com/tmi, as it gives all scopes you need for basic usage, including emotes.

Yup, I got a new token with https://twitchapps.com/tmi and everything works as intended now. Thank you so much for your help!

No problem. Have a good one!


Finalize EDA

[ ] add comments and text
[ ] write functions for code that is repeated
[ ] move code/functions into Python scripts

[ ] use same labelling (e.g. dementia/Alzheimer)
[ ] similar colour codes

[ ] combine OASIS and ADNI in one notebook
[ ] highlight differences in the two datasets (also in images) and the need to consolidate the two
[ ]

OASIS numerical and image data final EDA completed @tamap94


Support for darwin_arm64

Hello! And thank you for the provider!
Do you think it'll be hard to add support for darwin_arm64 platform?

Hey @allomov 👋 - thanks for opening up an issue. This should be fixed in the 1.9.1 release of the provider.


hsa-runtime_DIR is not a valid identifier

If I attempt to set the hsa-runtime64_DIR environment variable, this fails in bash version 4.4.23 with the message that this is not a valid identifier. This post seems to suggest environment variables with dashes are not valid in POSIX.

Please use CMAKE_PREFIX_PATH to specify the installation directory of hsa-runtime.


Some small optimizations in core/math/geometry.h.

Optimize get_closest_point_to_segment by combining scalar factors.
More coming (please comment with suggestions / wishes).

Thanks!


WIP tests: content negotiation

Fixes #421
@XVincentX FYI
Let me know if https://github.com/stoplightio/prism/pull/323 fixes this.
If so then let's make sure we have the coverage aand we can close this PR

@chris-miaskowski I did not really look at the tests in depth but I do understand how the logging PR can solve these problems 🤔 — were you referring to another PR?

I'm closing this in favour of the harness


error in running extension HTML5 Boilerplate

Unknown language in contributes.html5-boilerplate.language. Provided value: njk
Unknown language in contributes.html5-boilerplate.language. Provided value: django-html
Unknown language in contributes.html5-boilerplate.language. Provided value: njk
Unknown language in contributes.html5-boilerplate.language. Provided value: django-htm

not working


fixes #452

@thinkingserious please take a look @ this
The README.md might need some work, I'll do it once I'm sure the approach I've taken is correct.

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

@jaydp17 we have not forgotten you! We are just a little behind on getting to every single PR that has come in so far this month. We will review and get back to you as soon as possible!!

Hello @jaydp17,
Are you still interested in completing this PR?
Thanks!
With Best Regards,
Elmer

@thinkingserious really sorry, completely forgot about this.
I've added it to my TODO list, but I'm not sure I'll be able to start working on this immediately if someone else is ready to start working on it instantly, I'd be happy to hand over.
Else I'll take some time but it'll be done.

Thanks for the update @jaydp17!
It's all yours unless someone comments that they want to take this on and you approve.

Fixed by https://github.com/sendgrid/sendgrid-nodejs/pull/1112


Something Rustles nearby, but no Pokemon appears.

Expected Behaviour
One bot:
[13:17:05] Something rustles nearby!
[13:17:05]  A Wild Pidgey appeared! [CP 188] [Potential 0.07]
Actual Behaviour
Another bot:
[13:17:16] Something rustles nearby!
[13:17:22] Moving towards fort Decorative Colonnade, 0.06km left
[13:17:28] Something rustles nearby!
[13:17:33] Moving towards fort Decorative Colonnade, 0.05km left
[13:17:39] Something rustles nearby!
[13:17:44]  Now at Pokestop: Decorative Colonnade
Steps to Reproduce
Config file is literally exactly the same, don't understand what is going on.
Other Information
OS: OS X
Git Commit: a90a77abe0f6a59f1adc330ea7db40eb24470a7d
Python Version: Python 2.7.10

Appears to be because the pokemon bag is full. However no warning is issued.


AKS ingress xip.io url not being generated

Rancher versions:
rancher/rancher:v2.0.2
Steps to Reproduce:
Create an AKS cluster
Create a workload
Create an ingress to workload using xip.io URL
Results:
After 30 minutes URL for ingress is never generated, stays as the default xip.io

Duplicate of #13088


Point to correct circuit lib location

Summary
circuit_library is directly under apidocs
Details and comments

Finally done right.


RTL support

After changing direction to rtl scrollbars not changing accordingly. Is there anything that I am missing or is it not supported?

It will be supported in version 0.3.0, currently in development

@gromo  Ohh :( okay. Thanks for reply. I am waiting for version 0.3.0. Gr8 work


Fixed broken links in serving component

Ref : issue 1683

In https://www.kubeflow.org/docs/components/serving/overview/#multi-framework-serving-with-kfserving-or-seldon-core

Click doc  --> page does not exist

In https://www.kubeflow.org/docs/components/serving/seldon/

language specific model wrappers --> page does not exist

This change is

/assign @sarahmaddox

Thanks!
/lgtm
/approve

Thanks @sarahmaddox !


Port to pure ASGI Middleware

Hi, I am a Starlette maintainer. We are trying to get folks moved away from BaseHTTPMiddleware because it has several unfixable bugs and performance issues. One of the highest impact ways of doing this is going to be porting downstream packages (like this one) to pure ASGI middleware.
Taking a look at the package, it seems like it would be pretty straightforward to port the middleware itself to a pure ASGI middleware. The part that is going to be tricky is that the API sends Request/Response objects into user's functions. That part can't really be changed. What I'd proposed is creating a Request object with a placeholder receive that raises an error if called (i.e. you can't access the Request body) and maybe a Response that can't be run (i.e. Response.__call__ raises an exception).
To be clear I am volunteering my time to do this, but I will need review, some discussion about the above issues, etc.
Does this sound reasonable @trallnag?

Hey @adriangb, sounds good. But beware my lack of knowledge regarding Python ASGI. Also check out this pull request: https://github.com/trallnag/prometheus-fastapi-instrumentator/pull/139. I think @Kludex has already (partially?) worked on replacing the middleware

Oh I missed that, Marcelo is always one step ahead 😄. I'll close this in favor of #139


Equal Hight in Table Grid not working

Any idea why the equal height isn't working?
Looking at the Style Guide, I see you don't need the style attribute here.  WET will add it with the min-height, but not the right one.  Even if I remove it it still doesn't work
Originally posted by @smellems in https://github.com/canada-ca/ore-ero/pull/1333#discussion_r568610431 - to be merged

After some investigation I've found the problem... although unsure how to solve yet.
The issue is the equalization happens on the tr element (which does work) however the style is implemented against each td element - so the style isn't showing the equalization.

Just and update on some experiments
I attempted to use the eqht-trgt element on the description td, this in most ways worked, but there was outliers, if the "heading" td in one "panel" (tr) was larger than the rest then the calculation didn't work for that "panel". It also took into comparison all of the "panels" not just the 3 in the "row" - so some "rows" had lots of extra white space.


PHP version

Is PHP 5.5 really necessary in composer.json?

Yes, Payum 1.0.0 requires php5.5 - https://github.com/Payum/Payum/blob/master/composer.json#L40
https://github.com/Payum/Payum/issues/387

Ok, an update was due on my servers anyway


Cheap inversion with 32 bytes helper value

Alongside with pubkey and signature transaction caller may provide an inverted value then we can check the inversion (a * a_inv == 1) mod n. So that modexp call can be skipped

Good idea, but since this would increase call data used it would increase the L1 calldata fee for users of the contract on L2s; Since that's the primary use case ATM we probably won't implement this.

Single 32 byte helper would suffice If it is multiple signature verification case with batch inversion technique. But I'm not sure it would worth to touch too many places if modexp cost is not that high


Make the number of validators and full nodes configurable in E2E tests

Summary
The current numbers are specified directly here
We should allow the number of validators and full nodes to be configurable on a per test basis. This could be done with more env vars or some sort of other configuration method.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

can this be closed?

I think so yes 👍


chore: tasks linting

Approving but I do want to understand why we have a dependency on the Microsoft.VisualStudio.Threading library

@darrelmiller this dependency brings the analyzers as well as a bunch of helpful Task management feature. Especially JoinedTaskFactory.Run which is much better than GetAwaiter().GetResult(). Once we fix the Task mess we'll be able to add just the analyzer

Approving but I do want to understand why we have a dependency on the Microsoft.VisualStudio.Threading library

disregard my last comment, I had added a dependency on the wrong package. The new dependency is private and does not impact downstream packages.


[Low] Notification cancel button is not aligned

JDN v 3.1.145
Back-end v 2022-02-12-11.36.19
Chrome

open any webpage
open JDN tab
new page object
generate
edit/delete/restore locator

AR:

ER: Cancel button in notification should be center aligned

Version: 3.1.178


error: Failed to parse org auth token {token}  caused by: invalid type: null, expected a string at line 1 column 35

Environment
"vue": "^3.3.2"
"vite": "^4.3.5"
"@sentry/vite-plugin": "^2.7.1"
Steps to Reproduce

npm run build

Expected Result
What you thought would happen.
Actual Result
some build success info...

error: Failed to parse org auth token {token}
  caused by: invalid type: null, expected a string at line 1 column 35

Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
Please attach the full debug log to all bug reports.
[sentry-vite-plugin] Info: Successfully uploaded source maps to Sentry

vite.config
sentryVitePlugin({
      url: "http://xxx",
      authToken: "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",
      org: "xxx",
      project: "xxx",
    }),

Hi, can you check which version of Sentry CLI is installed in your node_modules? Thanks!

sentry-cli 2.20.7
Sentry 23.9.0.dev0

I cannot reproduce. Can you double check that you have that cli version installed and not an outdated one in a different location?
Other than that we would need a full reproduction example (git repo/stack blitz) in order to debug this further.

This is my project
vue3.zip

@noberumotto Can you double check that you correctly copied your auth token? Thanks!

Additonally, you I recommend not sharing this auth token. It should stay private.

Same error with me!
error: Failed to parse org auth token {token}
caused by: invalid type: null, expected a string at line 1 column 35
Add --log-level=[info|debug] or export SENTRY_LOG_LEVEL=[info|debug] to see more output.
Please attach the full debug log to all bug reports.
"vite": "^4.4.9",
"vite-plugin-sentry": "^1.3.0",
sentry-cli 2.20.7

I found out the solution.
We are using the wrong auth token.
Please reference the document to get the right auth token instead.
https://docs.sentry.io/product/integrations/integration-platform/#auth-tokens
[sentry-vite-plugin] Info: Successfully uploaded source maps to Sentry

@ekisstherain Thanks!It's worked!

Yo this was an actual issue for auth tokens generated on self-hosted sentry and should be fixed with version 2.8.0 of the plugin. Let me know if it works!


router-outlet in loaded component

In my application I´m using router-outlets in multiple levels. In one of the components i want to load with dynamic hooks has router-outlet. The problem is that it takes the first routing component instead of the last, so it kind of tries to start the first component again. Is there a way around this issue or is it not supported?

I'm not sure I understand fully. There is a router outlet inside one of the dynamically-loaded components and that one is not working correctly or causing the parent outlet to bug out?
Is there a way you could post a minimal example with the steps to reproduce the bug? Or post the relevant code that is causing this issue?

Yes, its the router-outlet inside the dynamically loaded component that bugs out, showing incorrect content. It kind of starts over from the beginning of the router chain.

Okay, this behaviour is new to me. But I have a hard time reproducing it. I've tried a minimal setup:

Three top routes
The second top route has the ngx-dynamic-hooks component
It loads a dynamic component from a string that contains a second, named router-outlet
Three nested routes for that named outlet

So we have a nested router-outlet inside a component that is dynamically loaded with ngx-dynamic-hooks. But it works fine on my end. I can navigate between the top routes fine and can navigate between the nested routes fine when on top route 2.
When having (for example) top route 2 + nested route 3 open, and switching top routes, nested route 3 will again render correctly when returning back to top route 2.
So everything works normally for me. Maybe its something else causing the problem? Have you set up everything right for using named outlets? For example:

Route config: { path: 'someRouteName', component: SomeRouteComponent, outlet: 'customOutlet' }
Outlet: <router-outlet name='customOutlet'></router-outlet>
Link: <a [routerLink]="[{ outlets: { customOutlet: ['someRouteName'] } }]"></a>

If its not that and you could perhaps post a minimal version of the code that produces the bug, I could also have another look at it.

Hi! Im not nameing the outlets. But I guess that it would work, so perhaps it’s my workaround.
Before introducing dynamic hooks I used the same component without naming the router outlet without problem.
I can’t send the app I’m working on but perhaps I could make a smaller one with similar logic.

Made a small example of how i get it to load wrong component, switch in parent component between using dynamic hooks and compare with how it works without.
dynamic-route.zip

Hi there,
thanks for being so kind to provide me with an example project! I encountered the same weird behaviour when running it on my machine.
To cut to the chase, it was indeed a bug with my library which has now been fixed. Here are the patched versions depending on your Angular version:
Angular 6-13: ngx-dynamic-hooks 1.7.4
Angular 13+: ngx-dynamic-hooks 2.0.2
Since the example your posted ran with Angular 7, you would need to download v1.7.4 of the library. Working with child router-outlets should now work fine.
The long explanation: Previously, the library was instantiating dynamic components with the root injector by default for dependency injection. Due to this, when injecting ActivatedRoute, it was always pointing to the top route, i.e. the root component of the app, no matter how deeply routed the components actually were.
As <router-outlet>s internally consult ActivatedRoute when determining which routes to load, the child outlets wrongly assumed they were still at the root level instead of already being a child outlet. Thus they loaded the root routes again. Which load the child outlet again. Which loads the root routes again. And so forth ad infinium.
The patch simply made it so that dynamically-created components now load with the same injector as the <ngx-dynamic-hooks>-component (which provides the correct ActivatedRoute).

Hi again! I can confirm as well that it works as it should in 1.7.4.
Thanks a lot for all the help and the quick response!


Autofocus on the first text field of the form to create a new registry

Signed-off-by: Miquel Sabaté<EMAIL_ADDRESS>

Thanks


Some images not displaying - only in Safari

The images just seem to not show up, but only in Safari. The strange thing is that when I inspect those images, they immediately show up.
Do you have any recommendations? Anything would be really appreciated.
You can replicate my issue by:

Go to http://guidedtours.devmoore.com/
Select "Patient", then click "Next"
Select number "5" from the pagination in the bottom right
Compare the functionality to Chrome and it will be clear which animations are missing

This has happened on multiple versions of Safari, but i have must recently confirmed it happening on Version 10.1.1, running Mac OS Sierra.
Thank you so much!

The issue here was with masks being used in AE. Once we tried a different technique without masks, the animations worked as expected.


[Feature] Including the autogenerated name for local checkpoints of our data.

When I run wandb I get a local checkpoint of the run data ins the wandb directory.
The checkpoint looks like: wandb/run-20210404_003352-qwxqi48r
At the same time my checkpoint is uploaded to wandb and has a automatically generated name, e.g. mild-spaceship-1
Is there a way to automatically rename the local checkpoint to wandb/run-mild-spaceship-1-20210404_003352-qwxqi48r
I have looked into the Settings class but it was not obvious to me if it can be done.
If there is no easy way to do this at the moment I would propose to make it a feature.

Hey @somedadaism this is not supported.  You can create a symlink if you'd like after you've called wandb.init.  I'm curious what your specific use case is?  Why do you ever need to go back to that local directory after the run has completed?
import os
wandb.init(...)
os.symlink(wandb.run.dir, wandb.run.dir.replace(wandb.run.id, wandb.run.name))

I think the link between local wandb folder and in browser wandb is difficult to make.
Naming the local folders with the autogenerated names would make it easier to see what is what.

@somedadaism Hello! We have put this in as a feature request with the engineering team! We will update you/the documentation if this behavior is added to the application. In the meantime did the solution proposed by @vanpelt help with your use case? If not I would love to know what your use case is to provide more context for the team!

Hi there, I wanted to follow up on this request. Please let us know if we can be of further assistance or if your issue has been resolved!

Hi linking the directory has worked for me so far.

Hey @somedadaism I'm glad the linking proposed by vanpelt worked! I'll be closing this ticket and will update if the method for accessing local directory after a run has been changed.

Any update on this?

I really want this feature. @ash0ts
Just like @amitabe mentioned, some files are large and it's not proper to log them to the server. Furthermore, even if the wandb server does support logging codes or other files, it might be more convenient to open them locally. At present, users are required to make some efforts to find out which local dir corresponds to the exp name on the server.


sepolia config update - supported chains etc

Description
as title
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

merged to https://github.com/MystenLabs/sui/pull/16427/files


Permit TypeExpression as well as Named Types for input and output specs

On second thought, that's not such a good idea because it makes type usage for inputs a outputs to be different from other usages.


Use newest Picasso version

https://github.com/criteo/android-publisher-sdk/blob/e9148f6a56222daa6c78b1734591e8023f752b93/buildSrc/src/main/java/Deps.kt#L169
Version that you are currently using does not support AndroidX. We would like to get rid of Android Support dependencies to be able to remove Jetifier and improve build times.
https://github.com/square/picasso/tree/2.8

Hi @petras-vinted, thank you for bringing this to our attention. We will try to update Picasso to 2.8 indeed for the next 4.1.0 release.

The change was merged: https://github.com/criteo/android-publisher-sdk/pull/153
It will be released with the next version: 4.1.0.


Adding the Alert Message in AirQualityProjects

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

when we touch on a search button without filling the details enter latitude here in AirQuality projects then its shows the alert message

i already done this.Please assigned me this

I can solve this issue


Publish Pinniped CLI to Krew

Is your feature request related to a problem? Please describe.
Users don't know how to add things to PATH.
Describe the solution you'd like
Publish Pinniped as a Kubectl plugin installable with Krew so they can just do kubectl krew install pinniped.
Describe alternatives you've considered
They can download the binary and put it into the path, or depending on the os, use, aur, brew or other.
A "curl|bash" flow could also do the thing but I don't really like that.
Are you considering submitting a PR for this feature?
No

How will this project improvement be tested?
How does this change the current architecture?
How will this change be backwards compatible?
How will this feature be documented?

Additional context
See https://krew.sigs.k8s.io/docs/developer-guide/distributing-with-krew/

I think this is technically possible if we publish the pinniped CLI as kubectl-pinniped to krew.  We would need to enhance the get kubeconfig command detect this case so that it correctly uses the plugin mechanism to invoke the exec plugin (possibly in a way that figures out the correct path for kubectl?).  We may also want to (dynamically?) update the install hint to handle this case?

Related discussion on the Kubernetes slack.


[Snyk] Upgrade prism-code-editor from 2.2.0 to 2.2.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade prism-code-editor from 2.2.0 to 2.2.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2023-11-11.

Release notes

Package name: prism-code-editor

2.2.1 - 2023-11-11
Replaced contentEditable hack with -webkit-text-size-adjust: none (6bad6af)
Improved some themes while viewing CSS code (8f029b7)
Slightly decreased bundle size of the core

2.2.0 - 2023-11-04Improvements
Including hyphens in the search wasn't wanted for most languages. Therefore a new parameter was added to the highlightCurrentWord extension.
const selector = ".string, .comment, .keyword, .regex"
const filter = start => !getClosestToken(editor, selector, 0, 0, start)
const includeHyphens = position => getLanguage(editor, position) == "css"
editor.addExtensions(
highlightCurrentWord(filter, includeHyphens)
)
Fixes
In TSX, opening tags comming directly after another tag are now highlighted. This was an issue with Prism's TSX grammar.

    from prism-code-editor GitHub release notes

Commit messages

Package name: prism-code-editor

922bfeb Bump version
57e52eb Change website layout
8f029b7 Improve some themes for CSS
3478654 Optimize token stringify
6bad6af Replace contenteditable hack

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Get filter does not give access denied even after denying in nacm.

Hi @michalvasko ,
I do not get access denied in  when requesting even after denying the same in nacm.

get --filter-xpath /o-ran-sync:*
DATA

Normally when allowing o-ran-sync with read permission I will get data. When denying, data is not got and will get  a blank instead of Access denied.
Is this expected?

Yes.

The NETCONF server replies rejecting the protocol operation with an "access-denied" error.
31
33
34 protocol
35 access-denied
36 error
37
38
But this is not seen @michalvasko .
Should I give rwx permission to any yang with nacm to get this rpc reply?

Thanks resolved


TINKERPOP3-978 Native GraphSON Serialization for Gremlin Server

https://issues.apache.org/jira/browse/TINKERPOP3-978
We already had this feature for Gryo but it wasn't built for GraphSON as the original idea was to mostly support serialization over :remote.  That was a bit short-sighted in thinking so I've added the same capability for GraphSON as will be especially useful to non-JVM languages.
you can test with mvn clean install followed by the Gremlin Server integration tests:
mvn verify -DskipIntegrationTests=false -pl gremlin-server

my vote is +1
@PommeVerte @jbmusso @davebshow - this one may be interesting to you and your driver work

ok, so I wanted to be thorough with this. Hence de delay.

mvn clean install passes
mvn verify -DskipIntegrationTests=false -pl gremlin-server passes
wrote some driver side tests and they work properly. but shouldn't config: { useMapperFromGraph: graph } be added to the default configuration as well?

If there's some drawback to adding the above to the default gremlin-server.yaml and the PR should stay as is then:
VOTE: +1
If there isn't any noticeable drawback I'd be tempted to squeeze that in before merging.

Good idea @PommeVerte - updated the packaged config files to include useMapperFromGraph.

Did a clean install, then the Gremlin Server integration tests, then a code inspection.
VOTE: +1

Fixed with the above commit @pluradj - merging


issue #163

Changes the json snippet to use tabs instead of spaces, and updates the styling on the pre tag so it looks the same as before (\t was indented 8 spaces without the style change).
This doesn't handle formatting the tags array because the correct formating is driven by prettier and depends on the length of the array. I think the best way to handle this is to add a formatting job to the github actions that runs before the linter. I can add that action with this pull request as well if desired.

We probably could detect the string length and whether it needed to be split into multiple lines, but this is a good start either way :-)

Thanks! My thought here was that it's probably best not to be reimplementing Prettier functionality because it wouldn't be tied to the lint/prettier config.
If implemented as a github action it could be setup to run only when the json files change. It would also be resilient against any other changes that a person would make when editing the file.

Can a Github action automatically update someone's fork of this repo though?

Hmm... Good question. Possibly using a pull_request_target event (https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#pull_request_target), but I don't know for sure.


Dependency vulnerabilities

Describe the bug
yarn audit and Snyk both report a few high severity vulnerabilities in the dependencies for this repo

Thanks! I updated all of our dependencies!


Some enhancements

It's no longer assumed that the indices begin at 1, and an extra function call is avoided.

Changes Unknown when pulling 07b3418a4ffb5b153f77086f3c3c2aeb98d1bc95 on aa/enhance into ** on master**.

@coveralls Who told you you could speak


Donating ebpf based instrumentation

This PR contains a copy of all the files in the https://github.com/keyval-dev/opentelemetry-go-instrumentation repository

Please make sure to follow the attribution guidelines. All source code files should carry the corresponding copyright notice. If you copied any files from other open-source projects that they need to carry the attribution notice.
Also, is ebpf-based/vmlinux.h generated or it is a copy from somewhere else? If generated please include generation instruction, if copied it needs an attribution.

Added. I followed the same licenses that cilium (another eBPF based CNCF project) use for their Go code / eBPF programs.

Added. I followed the same licenses that cilium (another eBPF based CNCF project) use for their Go code / eBPF programs.

@edeNFed Thank you. To help with the review, can you please tell which files are derived/copied from elsewhere and which files are authored by yourself?
For files that are derived/copied we will need to know their original licenses, since not every license is allowed.
For files that you are the author are you the sole author or you are contributing this on behalf of other authors / on behalf of a company?

@tigrannajaryan The files in include/libbpf directory are copied from the linux kernel: https://github.com/torvalds/linux/tree/master/tools/lib/bpf
All the other files are written by myself and are donated on behalf of Keyval.
Please let me know if there are still attribution issues.
@open-telemetry/go-instrumentation-approvers would love to get a review for this PR.

@tigrannajaryan The files in include/libbpf directory are copied from the linux kernel: https://github.com/torvalds/linux/tree/master/tools/lib/bpf All the other files are written by myself and are donated on behalf of Keyval. Please let me know if there are still attribution issues.
@open-telemetry/go-instrumentation-approvers would love to get a review for this PR.

The libbpf is dual-licensed. IANAL but I have no idea how we should say that we are "choosing" the BSD-2-Clause as LGPL-2.1 may be problematic, because of:

Each time you redistribute the Library (or any work based on the Library), the recipient automatically receives a license from the original licensor to copy, distribute, link with or modify the Library subject to these terms and conditions.

Any ideas?

Those files are needed for easy linkage during compilation. I am not sure if this makes the decision easier but I guess we can download them during the compilation script instead of having a copy in this repo.
Anyway, I think there should be a way to use those files as other eBPF based projects in the CNCF (cilium for example) are also dependent on them.

Those files are needed for easy linkage during compilation. I am not sure if this makes the decision easier but I guess we can download them during the compilation script instead of having a copy in this repo.

It is not about "storing" the code but the fact that the library is linked.
Reference: https://www.gnu.org/licenses/old-licenses/lgpl-2.1.en.html

Anyway, I think there should be a way to use those files as other eBPF based projects in the CNCF (cilium for example) are also dependent on them.

That does not guarantee that these repositories are OK from legal perspective 😉
For instance, AFAIK, MIT requires adding a copywrite notice / EULA.

My "perfectionist ego" forced me to create https://github.com/open-telemetry/community/issues/1280 to track
I think we should consider deferring the licensing "issue" I brought (we can create an issue for it).
At this point of time I think it is fine to use libbpf using the "precedence rule".

BSD is also explicitly allowed by Otel provided that the notice is retained which it is is, so we are good: https://github.com/open-telemetry/community/blob/main/CONTRIBUTING.md#copyright-notices

I am approving as I think that this PR is too big to process it in a regular way. I suggest merging it as it is. People can always look at this PR and add comments. We may address comments as separate PRs


For maintainers only: Improve development & release setup

[ ] Should auto-regenerate yarn.lock after releasing new version

Currently yarn version:* commits & pushes some code without refreshing the yarn.lock file leading to the release failing in GH actions

[ ] Enable single tsc --build --watch for entire project as VSC task

[ ] Requires TS option incremental and composite set to true

[ ] Move package specific devDependencies from root package.json to specific package/my-pkg/package.json
[ ] Enable test watching
[ ] Consolidate configs across projects (Jest, eslint, tsconfig, ...)
[ ] Use changesets for releases/changelogs
[ ] Migrate to yarn 3 (in other repos)
[ ] Emit only once from TS ES2018+ and use esbuild for the commonjs output

I'd add migrate to yarn 3

With the update to yarn3 scripts needs to be changed, at the moment otel cannot be build from within a package

Like in https://github.com/Effect-TS/node/commit/b0c120cdf3630a9542c48d7ab4612d62507b6015


Build Utility to Reorder SQL Table Columns via Migration Script

SQL Server does not natively support the ability to change column order for an existing table via an ALTER TABLE statement. As a result, our migrations, when new columns are added, end up having those columns all at the end of the table instead of wherever the developer intended them to be in the table's column order. Generally speaking this shouldn't affect functionality in any way as people aren't using column order to drive behavior. However, this is "messy" and it is important that we be able to control column order in our tables.
For this reason, we will introduce a new utility method within the CodeGen tool that is capable of generating just such a migration script. The idea is that:

Using SQL Server Management Studio (SSMS) a developer will order the columns in their table as desired - this can be done in the SSMS UI.
After this, the CodeGen tool will be run with a specific option where we will provide the table name(s) that we want column reordering scripts built for. There could be better ways to do this ranging from somehow auto-detecting changes to also doing this in another tool outside of CodeGen, but this seems like a logical place to do it, at least to start.
The CodeGen tool will generate a SQL script to a file in a location specified in the config.json file in a new section to be called "table_column_reordering". This new section will have the following properties:

table_names - list of table names to generate scripts for - can be a single table name or comma delimited. Each table name is assumed to be prefixed with a schema name so the string would look something like __mj.Entity, __mj.EntityField etc
output_directory a fully qualified directory path that resolves from the cwd that CodeGen runs from. For example we could provide a relative path to the migrations folder
file_prefix - a string to be prepended onto each file generated by the routine. If omitted the output files will be just column_reorder_tablename.sql where tablename is replaced with the actual table name. Developers will change these files to merge them into one, or into other migration scripts etc and rename those files as desired for migration script purposes.

@jordanfanapour the first draft of the code that does the heavy lifting on this is here: https://github.com/MemberJunction/MJ/pull/631

@jordanfanapour also, at the end of this set of generated SQL commands, we would want to call __mj.spUpdateExistingEntityFieldsFromSchema which will handle updating the metadata to reflect the sequence of columns in the fields in the table. No new fields will be added, this process just reorders the existing columns so this last SP call will ensure the metadata reflects the physical table order.
However, the really important thing that will be needed in addition to this is to also have all of the SPs and Views associated with the entity regenerated. They will be generated by the developer already who has created the new order, and the migration will need to include those new SPs and Views to be in the script, after all of the above is done. We can do this automatically in this script generation by pulling those SPs and Views from the Generated directory and dropping them at the end of this newly created file.


Create BookingController

What is the task/ tasks? please recreate appropriate branch name or point it explicitly in the PR title - what is the task name , which is implemented here


[FEATURE REQUEST] Support GCP logging

Is your feature request related to a problem? Please describe.
A clear and concise description of what you want to happen.
Is it possible to support GCP logging like they have integration with Bunyan and Winston?
When using pino on GCP I see that it's formating logs well to make them supported in Stackdriver and Logging but it's not setting Trace field:( Also there is option to support logs related to request with section "Formatting Request Logs"
Describe the solution you'd like
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi, this question is not related to nestjs-pino because this is only wrapper for pino with some framework's specific.
But, I googled and found pino-stackdriver. I think this is what you are looking for

Thank you


add automated testing on posix platforms

Since you're deleting cleanup function. You should delete this line.
https://github.com/input-output-hk/cardano-shell/blob/0f5950aca2bf0b67fa590612bc51ad7e4bd4465a/app/NodeIPCClient/server.js#L28


[BUG]: Async validation sets error on form but not on field when using sections (v7.2.1)

What is the current behavior?
For version 7.2.1 async validation only sets error on form and not at field level when the field is part of a form section
What is the expected behavior?
Error is set both on field and form.
Sandbox Link
I updated the async validation sandbox example to demonstrate the issue.
sandbox with 7.2.1 bug
sandbox with 7.1.0 with expected behaviour

@Ciogmi this is fixed in 7.2.3: https://codesandbox.io/s/0y0qqq5v4n

It works, great

I can confirm it still occurs in 8.1.0 (current latest).

Managed to solve this throwing a error object considering the FormSection name.
Instead of...
throw new SubmissionError({
    field1: 'error 1 description',
    field2: 'error 2 description',
})

I've used:
throw new SubmissionError({
    section_name: {
        field1: 'error 1 description',
        field2: 'error 2 description',
    }
})


camera plugin fails in --preview-dart-2

Steps to Reproduce
Downloaded camera plugin example and ran it.  Works in non --preview-dart-2, but fails when --preview-dart-2 is enabled.
Logs
E/flutter ( 5351): [ERROR:topaz/lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
E/flutter ( 5351): type 'List' is not a subtype of type 'List<Map<String, String>>' where
E/flutter ( 5351):   List is from dart:core
E/flutter ( 5351):   List is from dart:core
E/flutter ( 5351):   Map is from dart:core
E/flutter ( 5351):   String is from dart:core
E/flutter ( 5351):   String is from dart:core
E/flutter ( 5351):
E/flutter ( 5351): #0      availableCameras (package:camera/camera.dart)
E/flutter ( 5351): <asynchronous suspension>
E/flutter ( 5351): #1      main (file:///Users/terry/Downloads/plugins-master/packages/camera/example/lib/main.dart:170:19)
E/flutter ( 5351): <asynchronous suspension>
E/flutter ( 5351): #2      _startIsolate.<anonymous closure> (dart:isolate/isolate_patch.dart:279:19)
E/flutter ( 5351): #3      _RawReceivePortImpl._handleMessage (dart:isolate/isolate_patch.dart:165:12)```

## flutter analyze
Analyzing /Users/terry/Downloads/plugins-master/packages/camera/example...
No issues found!
Ran in 3.7s

## Flutter Doctor

flutter doctor -v
[✓] Flutter (Channel beta, v0.1.5, on Mac OS X 10.13.3 17D2104, locale en-AU)
    • Flutter version 0.1.5 at /Users/terry/flutter
    • Framework revision 3ea4d06340 (2 weeks ago), 2018-02-22 11:12:39 -0800
    • Engine revision ead227f118
    • Dart version 2.0.0-dev.28.0.flutter-0b4f01f759

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at /Users/terry/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.4.0

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] IntelliJ IDEA Ultimate Edition (version 2017.3.4)
    • Flutter plugin version 22.2.2
    • Dart plugin version 173.4548.30

[✓] VS Code (version 1.21.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Dart Code extension version 2.9.2

[✓] Connected devices (1 available)
    • iPhone X • 4419315F-328F-436C-A1E8-548A1D631E5E • ios • iOS 11.2 (simulator)

• No issues found!

Thanks for reporting. Fix is out for review: https://github.com/flutter/plugins/pull/417


Example on main page

The example for updating entities isn't working
What can I do to read position from this? I am currently trying GetEntity(packet, out entity) and then adding entity to a list then reading from that list, however i can't find a way to just update positions of entities in my list so I end up with 20000 entities in 10 seconds and previous and current positions

First have a look at Entity.cs
Secondly you will need to make a EntityData class that you can fill with the values you want/need.
You should be using a EntityID as an index for a dictionary.
EntityID is located in ProtoBuf.Entity.baseNetworkable.uid
Like this:
Dictionary<uint, EntityData> entities = new Dictionary<uint, EntityData>();
entities.Add(entityID, entityData);

//You can use this to find entities
if (entities.ContainsKey(entityID)) {
    EntityData entityData = null;
    entities.TryGetValue(entityID, out entityData);
    if (entityData != null) {
        entityData.baseEntity.pos = ...;
        entityData.baseEntity.rot = ...;
        entities.Remove(entityID);
        entities.Add(entityID, entityData);
    }
}


(work in progress) Automated tests for kit updating.

I'm leaving this here because I'd really like these tests to work but it's going to take some creativity :)
This PR is to automatically run tests when I want to test different approaches.

Problem solved.  Once this has been released and one more release has been done the upgrade test should be possible too.  The idea is that:

One version of the kit plays the role of the old kit handing over to the new kit
One version of the kit plays the role of the new kit taking over from the old kit
Conceptually they need to be able to either leave instructions behind or pick them up and understand them
The version being tested (either locally or as part of the PR process) is intended to become the next latest version
At a later time in the life-cycle of this version it will become the outdated version

(a good moment to take a breath)

The version of the project being tested needs to be able to perform both the role of the new version being installed and the outdated version being replaced
If it can perform both roles against the oldest compatible release (and each intermittent release has been through the same tests) then there's a very high likelihood that there's an unbroken chain of updatable releases
If it fails at either role it can be responsible for update issues, therefore needs fixing before release

These tests are designed to isolate both roles that need playing.


provide useful message if no match in search box

At present, if I type something that doesn't match a gene name or ID ino the search box, nothing happens when I hit return. There's no feedback.
It would be much more helpful if it went to a message saying "sorry, no match found for x" or words to that effect.
Later, when we/you implement faceted searching, we can add specificity to the messages, e.g. "no matching gene|publication|term found for x".

I've changed it to say "No matching gene" if there is no match.

That works - thanks!


[Meta issue]: Rollups  Enhancements

Meta issue for Rollups enhancements:
1. Count documents from source index

[x] Core change to use _doc_counts metadata field. https://github.com/opensearch-project/OpenSearch/pull/3985
[X] Index management change to write doc counts to the metadata field https://github.com/opensearch-project/index-management/pull/439

2. Allow a dynamic target index in ISM Rollups

[x] Add the ability to index to an alias which may be managed by a rollover policy
[X] Write to a target index based on a mustache template https://github.com/opensearch-project/index-management/pull/435

3. Allow rollup searches to multiple rollup indices if they have the same rollup

[X] https://github.com/opensearch-project/index-management/pull/440

4. Allow query_string queries against rollups data

[x] Enable searching with query string queries in the rollup interceptor

For collecting the number of documents from the source index, the index management code base was prepped for a change in core which would make use of the _doc_counts metadata field to allow for seamlessly aggregating summary documents as we do in rollups. This core change did not come, but the change really belongs in core. This issue is open in the core repo here:
https://github.com/opensearch-project/OpenSearch/issues/3712
We should pick this up as the solution to aggregating document counts from summary docs.

I have updated the individual issues, but to summarize the progress at the time of the 2.2 release:

Doc counts changes are fully complete
Mustache field templates based on source index name is done. Using an alias as a target index will be targeted for the next release. There are some user experience questions related to that feature which I have commented in the issue.
Searching multiple rollup indices is fully implemented. Integration and unit tests have not yet been added and should be added as soon as possible.
There is an open PR for query string queries, but this PR does not include support for translating field names in the query string or the default field value. It is going to take more investigation to figure out how to best parse and then translate the query string.

@downsrob Can you please re-tag this issue with the right version?

Query String Query in Rollups completed : https://github.com/opensearch-project/index-management/issues/320


Add [Criterion] badge

Hi, this badge is for a new service I am implementing for tracking software performance at the function/method level.  It uses criterion-rs, but I hope that in the future other benchmark frameworks will be added to CI/CD flows.
I am adding the uploader and badge to my projects to accumulate data over time so I can add enhancements.
Thank you for considering this PR, I look forward to your feedback.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chmoder!

  Generated by :no_entry_sign: dangerJS against b0b40c806a814b40d60e0824c4c493157e46a207

@chmoder if you've got a Twitter account, I can @ you in a small Tweet about this new badge. ;)

@chmoder if you've got a Twitter account, I can @ you in a small Tweet about this new badge. ;)

That would be awesome!  I have API interfaces to sign up, etc. but am in the middle of making the web app to make registration and management easy.  And of course sweet visualizations, of performance tracking over time.  If you want to wait for that, I can follow up with you, if not go ahead and announce that for me @chmoder there as well.

Let's wait when, ping me whenever!

Hi @PyvesB, sorry it has been so long!  https://criterion.dev is in an alpha state, meaning we can now sign up, log in, upload reports, view graphs, and even get badge markup from your account.  It is not as pretty as we want, and the visualizations will get more features soon, but either way I am ready for new accounts now.  Thanks for the inspiration!

As promised: https://twitter.com/Shields_io/status/1310687916256043015 :)


Fix for url sync issue in SPA

I'm using Instantsearch in the context of a single page application.  Each time the user hits the page with the search field, a new instance of Instantsearch is created, and whenever they navigate away all handlers are destroyed.
This generally works, with the exception of the urlSync feature.  Since the firstRender variable is defined outside of the URLSync class, once it is set to false the code in lines 127 - 129 never executes again, even when I spin up a new Instantsearch instance.  As a result that the onHistoryChange handler is never attached, and the url does not update when the query is changed.  This pr sets the firstRender instead to be an attribute on the URLSync class, so that it will be reset every time the class is re-instantiated.  This revision is one resolution to the problem, but I'd be interested to hear if you all have other suggestions.  Thanks!

@bobylito Great, thanks for the quick response!


1.7.0 ceased to work on Android 4.0

When I try to run DNSCrypt on Android 4.0, it always show FIX ME! notice firstly and then notice Segmentation fault continously. BusyBox has already installed.

dnscrypt-proxy doesn't seem to be explicitly using that function anywhere in the code.
Did it work with previous versions of the proxy? Maybe this is due to increasing the SDK version.
In any case, I don't think we can reasonably keep supporting Android 4 which is really really old now.

OK I got v1.6.1 to run, then it give me FIX ME! notice again, the content look the same, but have additional information iptables: No chain/target/match by that name. Finally show me Segmentation fault once and quit.

Unfortunately I don't think we can support Android 4, especially since testing it on a real device becomes difficult.
Would it be possible for you to use something more recent, if only by using Cyanogen or another Mod?

is there any .apk package available to install directly on android v5.0 and above which should listen on port 53 so that, by using my android phone as a dedicated wifi router, I should be able to use this phone as on the go DNS encryptor as well as mobile router?
I think, just to listen on port does not need root access. Am I correct?

Yes and no, yes you can listen to a port without root, no you can't listen on ports <1024.
You need to setup something like this: https://github.com/jedisct1/dnscrypt-proxy/issues/393#issuecomment-204802390

@jedisct1, I just tested 1.7.0 on Android 5.1, and it looks like it works just fine, so I think the 4.0 is to old. I have one question though. Why isn't dnscrypt-proxy available in Google Play? Is there a way to add it there? If not, would you consider adding it at least to the F-Droid repository?

@morfikov I think add a frontend to F-Droid is enough.

@morfikov I found an app to tun dnscrypt-proxy
https://play.google.com/store/apps/details?id=com.okirat.dnsmanager
This app has an option to run DNSCrypt, but has very limited function to configure.

I've seen that app, but since it requires the root access and is not OpenSource, I'm not going to install it. That's my rule.

@morfikov OK I agree with you, I think create a full featured frontend app release to F-Droid would be better.

See my post above, follow the link and read: it's a bit more complicated without root, you want to control the connection flow ( local VPN like NetGuard), then re-route DNS requests to another app (say Unbound) that forward DNS requests to a running dncrypt-proxy instance.

Actually it's not about root or not root. The dnscrypt-proxy app isn't available in Google Play or F-Droid repo, which is a little bit inconvenient to install it. I can set it up without a problem on my phone, but I have to manually download the zip file and do the other stuff. I'm just asking to add the app to the Google Play or F-Droid repo, that's all.

Currently I run dnscrypt-proxy via BusyBox + Kernel Adiotur + DNS Forwarder, but I think using a client apk performenced as dnscrypt-winclient is better.

see
https://github.com/smarek/android-unbound-dns/issues/18


Updated gradle and dependecy versions

Updated gradle version to 8.0.1 and kotlin version to 1.8.21 and also updated some dependencies

@Yash-Parsana Issue no. is not mentioned in the PR. Also, give proper labels to the issue and PR according to GSSOC'23.


Query params with no value

When a state has configured query params like this
$stateProvider.state('name', {
  url: '/url?param',
});

is you open the url without assigning a value to param the stateParam value is the string "true", is it possible to make this value be undefined, or at least something other then a string? So that /url?param and /url?param=true would give different results?

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


AC10 mitigation: set alerting thresholds for token count

In #559 we started sending uaa token counts to riemann and influxdb for analysis. Next, we need to research appropriate thresholds for alerting.

[ ] Examine token counts in influxdb
[ ] Set alert thresholds using https://github.com/18F/cg-riemann-boshrelease/pull/64 or similar

Closing; we'll track this in #559.
h/t @mogul


Bug 2087042: Rebase cloud-provider-vsphere 15.06.2022

This PR rebases the cloud-provider-vsphere openshift patches on top of the kubernetes/cloud-provider-vsphere master branch after bump dependencies to kube v1.24.1 release.
There are several commits that we carry on top of the upstream cloud-provider-vsphere and the rebase process allows us to preserve those. Here is a description of the process I used to create this PR.
(replicated old @elmiko's process within openshift/kubernetes-autoscaler)
Process
First we need to identify the carry commits that we currently have, this is done against our previous rebase (or fork in this case) to catch new changes. Once identified we will drop commits which have merged upstream and only carry unique commits. (see below for the carried and dropped commits).
Identify carry commits:
git log --oneline --no-merges 556ede7f720b9cc91c22b23b360b6156a1824b29..openshift/master

where 556ede is the last upstream commit in our fork. This is the
list of commits we will need to apply onto the new upstream version of the
cloud-provider-vsphere. ideally, some of these commits can be dropped.
After identifying the carry commits, the next step is to create the new commit-tree that will be used for the rebase and then cherry pick the carry commits into the new branch. The following commands cover these steps:
$ git remote update # make sure we update our refs
$ git checkout 619502ff4276c345966584bb1f412157385ba7c0 # last commit in upstream master
$ git checkout -b merge-tmp # create a temporary branch for our merge commit
$ git checkout openshift/master # we want to be at the tip of the openshift master branch when we run the next command
$ echo 'merge kubernetes/cloud-provider-vsphere 15-06-2022 ' | git commit-tree rebase-tmp^{tree} -p HEAD -p merge-tmp -F - # create a new merge commit for our history
fdce34cafdcb91f3dad263eb624ca02150c21b5f # id of new merge commit
$ git branch merge-15-06-2022 fdce34cafdcb91f3dad263eb624ca02150c21b5f # create a new branch for the cherry-pick work
$ git checkout merge-15-06-2022
$ git cherry-pick <carry commits> # cherry pick the needed commits into the new branch

With the merge-15-06-2022 branch in place, I cherry picked the carry commits which we should carry.
Commits to carry:
These commits are integral to our CI platform, or are specific to the releases we create for OpenShift.
afd0d591 UPSTREAM: <carry>: Run unit tests from pkg folder only
0400604f UPSTREAM: <carry>: Use golang 1.17 to build the component image
bb9b2a3f UPSTREAM: <carry>: Updating and vendoring go modules after an upstream rebase
b3d92d73 UPSTREAM: <carry>: disable staticcheck for OpenShift CI
e965bd94 UPSTREAM: <carry>: Add .ci-operator.yaml file
f4a86607 UPSTREAM: <carry>: fix staticcheck cache in openshift ci
8898a862 UPSTREAM: <carry>: Add vendored dependencies (vendor)
34e82d18 UPSTREAM: <carry>: Remove /vendor folder from .gitignore
d3a5bdc9 UPSTREAM: <carry>: OCP specific build scripts and Dockerfile
eba378c1 UPSTREAM: <carry>: rm OWNERS_ALIASES and upd OWNERS

Changes in commits:
b3d92d73 UPSTREAM: <carry>: disable staticcheck for OpenShift CI
f4a86607 UPSTREAM: <carry>: fix staticcheck cache in openshift ci
d3a5bdc9 UPSTREAM: <carry>: OCP specific build scripts and Dockerfile

Commits above was squashed into a single commit: 1fc9db07 UPSTREAM: <carry>: OCP specific build scripts and Dockerfile
Commit with dependencies vendoring was added on top of all.

@lobziik following on from our discussion today, i think it would be nice to note in the description the commit you are adding and what it's fixing.

/retest

@elmiko - this required more work around than i expected, i will create a follow up pr with envtest update. Lets leave just rebase and revendoring here.

ack, thanks for the update @lobziik

/retest

/approve


[DDW-504] Number format for syncing percentage and stake pools

This PR fixed number format issue on syncing percentage and stake pools panels
Jira Ticket
Todos

[x] Fix number format issue on syncing percentage and stake pools panels

Testing Checklist

Slack QA thread

App

[x] Select wallet and watch syncing percentage notification to make sure percentage value is formatted using selected configuration settings.
[x] Select wallet and watch blocks syncing tooltip to make sure percentage value is formatted using selected configuration settings.
[x] Go to stake pools screen and make sure number of stake pools label is formatted using selected configuration settings.

Test Cases
Scenario 1 - Validate format of Syncing blocks
Given that Daedalus is syncing
When I change the number format in settings
Then the syncing percentage of the blocks is displayed on that format

Scenario 2 - Validate format of Syncing wallet
Given that a wallet is syncing
When I change the number format in settings
Then the syncing percentage of the wallet is displayed on that format

Scenario 3 - Validate format of Stake pools
Given that more than 999 stake pools are displayed on "Staking Center"
When I change the number format in settings
Then the number of stake pools is displayed on that format

Scenario 4 - Validate format of "CPU" on "Daedalus diagnostics"
Given that Daedalus is running
When I change the number format in settings
Then the value of CPU in "Daedalus diagnostics" is displayed on that format

Scenario 5 - Validate format of "RAM" on "Daedalus diagnostics"
Given that Daedalus is running
When I change the number format in settings
Then the value of "RAM" in "Daedalus diagnostics" is displayed on that format

Scenario 6 - Validate format of "Available disk space" on "Daedalus diagnostics"
Given that Daedalus is running
When I change the number format in settings
Then the value of "Available disk space" in "Daedalus diagnostics" is displayed on that format

Scenario 7 - Validate format of "Local time difference" on "Daedalus diagnostics"
Given that Daedalus is running
When I change the number format in settings
Then the value of "Local time difference" in "Daedalus diagnostics" is displayed on that format

Testing Summary
English

Japanese

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

@yakovkaravelov Cases 1 to 3 passed.
I added more cases to validate the changes suggested in this thread (values on Daedalus Diagnostics).

@yakovkaravelov Cases 1 to 3 passed.
I added more cases to validate the changes suggested in this thread (values on Daedalus Diagnostics).

@yakovkaravelov Cases 1 to 3 passed.
I added more cases to validate the changes suggested in this thread (values on Daedalus Diagnostics).

@gnpf @nikolaglumac This is done and ready for review/test.
Thanks.

@yakovkaravelov Cases 1 to 3 passed.
I added more cases to validate the changes suggested in this thread (values on Daedalus Diagnostics).

@gnpf @nikolaglumac This is done and ready for review/test.
Thanks.

Hi @yakovkaravelov. There seems to be an issue with scenario 1 and 2. The values are reversed, when I select comma a dot used, and when i select dot a comma is used. Also when a space is selected a comma is used. See video
The same is also true for Daedalus diagnostics screen for scenario 4, 5 and 6 for Ram, CPU and Disk Space. See video

Hi @yakovkaravelov. There seems to be an issue with scenario 1 and 2. The values are reversed, when I select comma a dot used, and when i select dot a comma is used. Also when a space is selected a comma is used. See video
The same is also true for Daedalus diagnostics screen for scenario 4, 5 and 6 for Ram, CPU and Disk Space. See video

Hi @yakovkaravelov. There seems to be an issue with scenario 1 and 2. The values are reversed, when I select comma a dot used, and when i select dot a comma is used. Also when a space is selected a comma is used. See video
The same is also true for Daedalus diagnostics screen for scenario 4, 5 and 6 for Ram, CPU and Disk Space. See video

@ManusMcCole No, it's totally not right.
RAM size is 16.0GB.
So if you select first option, it should be 16.0GB
If you select second option, it should be 16,0GB
So they are working fine.

Hi @yakovkaravelov. There seems to be an issue with scenario 1 and 2. The values are reversed, when I select comma a dot used, and when i select dot a comma is used. Also when a space is selected a comma is used. See video
The same is also true for Daedalus diagnostics screen for scenario 4, 5 and 6 for Ram, CPU and Disk Space. See video

@ManusMcCole No, it's totally not right.
RAM size is 16.0GB.
So if you select first option, it should be 16.0GB
If you select second option, it should be 16,0GB
So they are working fine.

Sorrry @yakovkaravelov my mistake I was looking at the symbol one place to the side and was mistaken

Sorrry @yakovkaravelov my mistake I was looking at the symbol one place to the side and was mistaken

@yakovkaravelov There is an issue for one of the scenarios:

Test case
Status
Comments

Scenario 6
Failed
The value on "Available disk space" always displays a dot as the decimal separator. Check this screenshot (Observe the rest of the values show comma, as the second format option is selected)

The rest look good 👍

@yakovkaravelov There is an issue for one of the scenarios:

Test case
Status
Comments

Scenario 6
Failed
The value on "Available disk space" always displays a dot as the decimal separator. Check this screenshot (Observe the rest of the values show comma, as the second format option is selected)

The rest look good 👍

@yakovkaravelov There is an issue for one of the scenarios:
Test case	Status	Comments
Scenario 6	Failed	The value on "Available disk space" always displays a dot as the decimal separator. Check this screenshot (Observe the rest of the values show comma, as the second format option is selected)
The rest look good 👍

@gnpf Fixed now. Please test again, thanks.
@nikolaglumac This is finished, please review, thanks.


backend: Consider the base URL for OIDC-callback's redirect URL

#1156 #292
A quick fix for the issue. Sorry for opening duplicated issue.

Thanks for the fix!
I'm not sure the backend test failure is related to this. I heard there might be some problem in the main branch. I'll get back to you about that. So if the test failure doesn't make sense to you don't worry too much, it might be something else :)


Simple Cipher: add missing Ignore attribute

The Ignore attribute was missing from the test case Cipher can double shift encode so I added it.

Great! Thanks a lot. :tada


Game is crashing while trying to open the new cards in Version 1.29, like the bystials and cartesia based cards.

Does the game crash or do you get an error code like this https://github.com/pixeltris/YgoMaster/blob/master/Docs/FileLoadError.md

Does the game crash or do you get an error code like this https://github.com/pixeltris/YgoMaster/blob/master/Docs/FileLoadError.md

I noticed that New Pack Card didn't have animation in the actual game. But in the Online Game, it has. What makes the difference between them, I remember that was unusual in the past updates.

If you see the animation in-game and not with YgoMaster that probably means you have multiple folders inside of LocalData as mentioned in that link. If you make sure all of the folders in LocalData are updated it should work. Or delete the folders which aren't updated. Each folder belongs to a separate steam account and has its own set of data downloaded. YgoMaster doesn't know which is the "correct" one.


Deduplicate lookup from assignments table

Fixes #90 by omitting to append duplicate assignments to the result of fetch_assignments.

My feeling is that it would be better to de-duplicate such data when we are importing/reading it initially. However, we will consider merging this change.

The commit c2689a4 for this PR was moved to another forked branch.


Fancy zones extends below the taskbar, but only for the main screen

Microsoft PowerToys version
0.64.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
Select a Fancy zone with 0px around zones

Apply it to both main and secondary screens

As highlighted, the explorer window is cropped on main screen while not cropped on the other.
The 2 screen have the exact same preset.
✔ Expected Behavior
The lower part of the 2 windows should be visible
❌ Actual Behavior
The lower part is visible on the secondary screen but not on the main screen
Other Software
No response

/dup of https://github.com/microsoft/PowerToys/issues/10903


Fix immutability of tuples

Return a TypeError for all tuple item assignment attempts, regardless of whether the index used is valid or not.
Fixes #327 indirectly (fixing immutability instead of changing the internal type)

I guess the general issue with the test is similar to #320: accidentally getting exceptions (and having execution stop).
One fix could be to make TranspileTestCase append some print statement at the end of all code execution blocks, so that it's clear when any exception is unhandled. Code that expects exceptions would have to catch and preferably print the exception.
There might be a performance hit to the test suite though, since many operation tests raise exceptions and we'd have to make BinaryOperationTestCase etc wrap things in a raise/except block.

I've had the same idea - I just haven't got around to implementing it. I was thinking that the test could just append print("===end of test===") to the very end of whatever content is in the main test script; and assertCodeExecution does a verification that the last line of content is "===end of test===". If that output isn't observed as the last thing, then you can assert that the test terminated early.


fix: extract Cloudflare_CA.pem to temp dir before using it

With package managers like yarn, the cloudflare cert won't be available on the filesystem as expected (since the module is inside a .zip file). This fix instead extracts the file out of the module, copies it to a temporary directory, and directs node to use that as the cert instead, preventing warnings like https://github.com/cloudflare/wrangler2/issues/1136.
Fixes https://github.com/cloudflare/wrangler2/issues/1136

Alright, I think this works. It definitely works with npm so we're not breaking anything. yarn doesn't allow installing from arbitrary urls, so I can't test with the preeleases. I tried locally and it worked, but that doesn't test the .zip nature of things. So let's try landing this and seeing how it goes?

Confirmed with the beta that this works fine with yarn now


T3 in ES2015 or ES2016?

And it is possible to convert the library's code into es2015 or es7 syntax.
What do you think about it?
Thanks,
Barış Güler

I'm actually opposed to this. Current transpilers tend to add a bit of weight to the final output. That's fine for a large project where maintenance gains you get from transpiling out weight the cost of the extra javascript output. The appeal to this library is that it's simple. You don't need ES2015+ for this project.
Plus, this project is very stable. Touching a lot of code to essentially get a little bit cleaner syntax opens up the potential for bugs without actually gaining anything substantial. As a consumer of a library you shouldn't concern yourself with what its written in as much as how well it works and how large it is.
With all that being said you can absolutely use this library with ES2015 today. There's no inherent need for the library to run off of ES2015 to be able to use it effectively with ES2015.

You can still use T3 as it is with ES6-7 without changing over the source code. (Changing over the source code has little benefit but introduces a lot of work and risk.)
Here's an example of how to use T3 with ES6 modules:
import * as T3 from "./t3";

T3.Application.addModule("foo", function() ());

T3.Application.init();

Yes, got it. Is it thought to change the code structure of the sample todo app into ES6 / ES7 to be a reference for the new-comers into T3.js world?

I don't think that's a good idea right now. ES6 is still very new and fully implemented everywhere, so most people are familiar with ES5 only. For right now, I think it's best to keep the examples in ES5.


slideshow autoplay interval corrupts on mousemove

I have no idea why this is happening, but the autoplay interval extends itself somehow when moving mouse pointer. You can see it happen here:
http://mjau-mjau.com/concepts/slideshow/inline/
Click the play button, and if you don't move the mouse, autoplay works nicely and intervals are correct. If you do move the mouse between slides, somehow it extends the interval and messes up the autoplay. I have no clue why mousemove effects the autoplay/interval.

Done in the 4.6.3 :)


Update Travis-CI badges for repo url change

Seems at some point this repository was under github.com/expressjs, however, now it's under github.com/crypto-utils

f00d8d5eefc69d4f0b0daf6d8090c5440983514c


Unexpected input 'note'

I got the above warning when the action is executed. It completes successfully but not surprisingly  the deploy message is again the default one.
If you point me to the place where the issue comes from I will contribute again to fix it.

@josephwynn-sc  I believe the root of the issue is simple because I noticed the deploy hasn't succeeded github action run
I guess this means that the new version is not available which explains why I get the error above.

@josephwynn-sc I am sure you are quite busy and that's why I am posting just a kind reminder on the issue as I feel we are millimetres close to success

@wildlyinaccurate @josephwynn-sc Coming back to the Issue as we are still struggling a lot with differentiate runs on the SpeedCurve dashboards ...

Sorry @v-dimitroff! I have published v1.2.2 which has your change.

Thank you @josephwynn-sc - appreciate it! Though the issue remains for some reason ... Our latests runs show the same warning output after the workflow has finished.

@v-dimitroff are you definitely using v1.2.2? I just tested the note input and it works:

@josephwynn-sc My bad! You are right, I didn't notice I need to point the version as we were using the v1 major version without other specs. It is working now and to save time for others I will push a PR with the updates and a section for the note param.


Update zio-prelude to 1.0.0-RC27

About this PR
📦 Updates dev.zio:zio-prelude from 1.0.0-RC21 to 1.0.0-RC27
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-prelude" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio", artifactId = "zio-prelude" }
}]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #161.

Superseded by #164.


Stop retweeting blocked acccounts.

Is there a way to stop retweeting blocked accounts(which was done manually). In the sense, it should not retweet if the user id is in the block list. Any help appreciated. .: :)

@MichaelCurrin Can you help with the query?

@humblelad Is your blocked list stored in your application or in Twitter?
For the latter try
blocked_users = api.blocks()

Then you can get user.id or user.screen_name on the result.
You might want to get the blocked list at application start (to save regularly checking) and then reuse it for your retweeting.
http://docs.tweepy.org/en/latest/api.html#API.blocks

@MichaelCurrin https://github.com/humblelad/HashTagger-Twitter-Bot/blob/master/hashtagger.py Here is the sample code. I want to stop retweeting accounts which I blocked manually via twitter to stop spams.

As explained, you can use API.blocks or API.blocks_ids and check against the IDs of the users you have blocked, using the user attribute of the Status/Tweet object and the id attribute of the User object.