refactor(add): abstract questionary dependency from API

If we ever decide to use something other than questionary, abstracting the dependency will allow us to change the PromptChoice implementation without changing the API.

Codecov Report

Merging #83 (9750e4f) into master (97321dc) will decrease coverage by 7.23%.
The diff coverage is 95.00%.

@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
- Coverage   98.29%   91.05%   -7.24%
==========================================
  Files          44       44
  Lines        2644     2651       +7
==========================================
- Hits         2599     2414     -185
- Misses         45      237     +192

Flag
Coverage Δ

integration
?

network
?

unit
91.05% <95.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.


Attempting gateway doc import from previous repo

Importing gateway .md files from: https://github.com/AdobeDocs/commerce-extensibility

This comment is to contextualize my latest commit: https://github.com/AdobeDocs/graphql-mesh-gateway/pull/7/commits/dd50db74edccbc886af52c6dff97f48e772db045
I added all files to the side nave. I used the header navigation because it looks the cleanest for now.
I added a .env file, which is necessary for the Edit in Github and Log an Issue buttons to work.
Since we can add html elements, we could use  to collapse a section. I used one in the create-tenant.md page to keep the code from obscuring the process. I also added a  block to fix some spacing.
I went through and resolved the issues and display errors that I could figure out, here are some remaining issues:

The first paragraph of every page is in a slightly larger font with extra space after the paragraph.
Adobe Graph is listed twice in the breadcrumbs. Modifying the top "Pages:" entry in the Gatsby config seems to change both of them.
Lists and indentations seem to have general issues. Having two lists back to back causes them to appear as double spaced. Lists appear very close, vertically, to the sentence that precedes them.
Separate lines that were defining key value pairs somehow wrapped to a single line. I'm guessing it has to do with the way the JSX parser looks at code blocks.
Using a note in a list breaks the numbering. Also, indenting the note usually prevents the note from displaying properly. Following what I could find in PWA, I replaced notes in lists with a simple "NOTE: ..." option.
Not sure where to customize the Console and "Was this helpful" Yes/No buttons


Open gigs in another tab instead of clicking the share button

[Need to groom user story]
Currently, it is unclear how to open a post/gig to see the full description. Currently a user has to click the share icon in order to be able to view the full description or the URL. As a user, it would feel more intuitive to me to have a way to click on the title or gig itself to open it in a new page or see the full description.

It make sense to make title clickable and open the post full description page. It's a worth doing good first issue.


change the port number of the local registry for macOS Montray

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice catch! Thank you.
/lgtm

Here you go
/approve


Fail to pull local proxy list ffrom http://<IP_ADDRESS>:8000/proxy.txt

Initially I need to check my local proxy list. So I set "http://<IP_ADDRESS>:8000/proxy.txt/" in my providers list. But ProxyBroker returns 0 proxies.

Are you sure that you should check http://<IP_ADDRESS>:8000/proxy.txt/ instead of http://<IP_ADDRESS>:8000/proxy.txt?


Removing (git rm) has/had threading issues

handleRemoveButton() in SessionController was threaded in the "old way" and I don't think it has really been tested because when called, asserts for being in the FX thread fail. Without spinning off a new thread it seems to work fine, but that could still use testing.
Currently, the threading code is commented out and it is accompanied with a TODO message. I don't think any multithreading is happening when that method is called anymore.

Bugs #546 and #561 are also about remove; it makes sense to look at this all at the same time.

Remove is fast enough that I'm just going to take the threading out. If someone finds a use case later on where it's slow and bothering someone, they can file that again as a later bug. I've made the fix already regarding the "are you sure" check in #546. The key remaining issue, which is the hard one, is #481 .


MAINT: Deploy docs

@banesullivan can you try working out the deploy key business in the docs branch I just created in this repo:
https://github.com/pyvista/pyvistaqt/blob/f126d6d597ea2eeafbef09c4ac12d769ff9de5e8/.github/workflows/docbuild.yml#L33-L43
The doc build appears to work, in theory "just" the deploy key stuff needs to be sorted out. Feel free to push directly, set org or repo level secrets, etc. Once it actually pushes some docs, I'll open a PR to merge the docs branch into main and then delete it from this repo, and also set the docs only to be deployed when a tag is pushed rather than for every push.

... you can see the expected failure here because the deploy key is not actually set up for the pyvistaqt-docs repo:
https://github.com/pyvista/pyvistaqt/actions/runs/5180508643/jobs/9334776061
Relevant docs for the action (but feel free to switch to a different one if you want):
https://github.com/peaceiris/actions-gh-pages#-set-ssh-private-key-deploy_key

Do we need a dedicated repo to host the docs for this package? Why not just gh-pages branch of this repo? AFAIK, these docs are not massive in size (like PyVista proper and the justification for it having its own repo)

(by comparison, creating a repo or org level key is very easy I think -- if you can make me admin of the Qt docs repo I can do it)

Okay I created a SSH deploy key, added the private key as a repo secret ACTIONS_DEPLOY_KEY in this repo, and added the public key as a deploy key in pyvistaqt-docs (where you either gave me or I already had permissions to do it!) and now it's working!

https://github.com/pyvista/pyvistaqt/actions/runs/5189730092/jobs/9355133734
https://qtdocs.pyvista.org/

Fantastic! Thanks for handling all of this!


Latest export of P1 automation scripts for EA

XML file to be imported into EA as reference data. Contains automation scripts (e.g. update decomposition diagram with child elements not currently present).

This is pretty horrible! Is there a way to compile the source code into xml after pulling it down? I'd rather commit real code into here and then compile into xml before using in EA

This is pretty horrible! Is there a way to compile the source code into xml after pulling it down? I'd rather commit real code into here and then compile into xml before using in EA

As far as I'm aware the conversion to and from XML is done solely in EA. The idea is to have users pull latest scripts into EA as easily as possible, will double check with Al if there's a way of replicating this conversion but I think we might need these XML files in source control...

Ok, that's fine for now but without putting raw code in source control you won't get collaborative devops benefit. You'll have to do dev in EA and I can't really review it as it's unreadable in xml


How does this compares to the other projects

Hi, How does this compares to the other projects?

https://github.com/gram-ai/capsule-networks
https://github.com/higgsfield/Capsule-Network-Tutorial
https://github.com/danielhavir/capsule-network
https://github.com/jindongwang/Pytorch-CapsuleNet (this one)

Should you include your project also in this list?
https://github.com/sekwiatkowski/awesome-capsule-networks#pytorch

I did not perform a thorough inspection nor benchmarks but it appears that the logic beneath the implementations is quite the same. Depending on how the author designed the interface/arrange the structures, there might be tiny differences regarding usability.


New version: CrowTranslate.CrowTranslate version 2.10.1

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.2 schema?

Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Executable-Error


Problem: add time_locked information to accounts for our ledgers

This is required by the accounts GQL endpoint to be feature parity with the ME GQL API

Closing since it's a duplicate of: https://github.com/Granola-Team/mina-indexer/issues/748


Exception Occurs When Executing CloseAsync() and Dispose() Method

Based on the API documentation available at
https://vnc-client.marcusw.de/apidoc/api/MarcusW.VncClient.RfbConnection.html#methods,
I attempted to disconnect from the server using two methods bellow, however, both methods resulted in an exception being raised:

RfbConnection.CloseAsync()

RfbConnection.Dispose().

with:
RfbConnection = await _connectionManager.ConnectAsync(parameters, cancellationToken).ConfigureAwait(true);
(Source: https://github.com/MarcusWichelmann/MarcusW.VncClient/blob/9d9fbf7b97b390ca9f0dff68ba52d2b9a85aed09/samples/AvaloniaVncClient/ViewModels/MainWindowViewModel.cs#L93)
I'm uncertain whether my implementation is incorrect or if the two methods mentioned above are not fully implemented yet.

That an OperationCanceledException gets thrown may happen in this case. But if you catch it, it should be fine.

That an OperationCanceledException gets thrown may happen in this case. But if you catch it, it should be fine.
@MarcusWichelmann

You can't just catch the exception, because you use ThreadStart to start the background thread. So the background thread will always just throw and break your entire application whenever you call CloseAsync().
You'd have need to catch the exception from your code.


Remove --debug and --verbose from Interactive Mode Completions

Since you can make all the commands run in the interactive mode run with --debug or --verbose with
az interactive --debug --verbose
az>> vm list

will run with debug and verbose

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3464   +/-   ##
=======================================
  Coverage   71.04%   71.04%
=======================================
  Files         419      419
  Lines       25993    25993
  Branches     3930     3930
=======================================
  Hits        18468    18468
  Misses       6378     6378
  Partials     1147     1147

Impacted Files
Coverage Δ

...zure-cli-interactive/azclishell/gather_commands.py
37.71% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


feat(prettier-xacro): update printWidth from 120 to 200

To be consistent with launch.xml.

Rationale for printWidth=200:

People often write tag attributes in one line and the current files are written in the style.
Half-window on 4K display can show 200 characters.


Add unit's  actions

Add unit's actions:

Attack (if possible)
Items
Status
Take (if possible)
Trade (if possible)
Wait

Cancel action:
https://github.com/rootasjey/cycles/commit/9bed996859bae9426aba8578002b81cdbb4253e0
Wait action:
https://github.com/rootasjey/cycles/commit/e71f802e970676d38541474bc9c8d860628bc22b


PLFM-6636: Add worker to stream file handles changes to S3

Depends on stack builder resources: https://github.com/Sage-Bionetworks/Synapse-Stack-Builder/pull/371


Choice icon alignment

Adds a new setting for Choice (modifies Tabs as well as a result) that allows for radio and icon to be aligned to the top of the choice when desired.

Also removes the nowrap setting that was present on Choice
Testing and Impact
See new section on documentation page for Choice in docs site and new setting in React.

Adds new setting for aligning icons in Choice elements. No impact on current uses. Validate the following work as before:

[ ] Website > Navigation > + New in a menu > See Choice card in the Modal that appears
[ ] People > Import Contacts (new) > first page has Choice cards for selecting import modes

@philschanely was looking at this one but it looks like the base branch is master and not develop so there are lots of changes. Would you mind updating that?


Variables Left & Right are undefined. Switch to min & max

To create the truncated normal data frame the code returns undefined for the variable names left & right.
# Create velocity distribution. It's a truncated normal distribution, to avoid freakish speeds.
a = (**left** - mean)/stdev # This should use the variable min
b = (**right** - mean)/stdev # This should use the variable max
speeds = truncnorm.rvs(a, b, mean, stdev, size=num)
df = pd.DataFrame (speeds, columns = ['v1'])

Whoops, no idea when I did that. Now fixed/


EMISSION_CHECK not an allowed condition based service

Describe the issue
I am getting the following error:
'EMISSION_CHECK' not an allowed condition based service (ConditionBasedService(service_type='EMISSION_CHECK', state=<ConditionBasedServiceStatus.OK: 'OK'>, due_date=datetime.datetime(2062, 9, 1, 0, 0, tzinfo=datetime.timezone.utc), due_distance=ValueWithUnit(value=None, unit=None)))
HA tells this error appears only once during the startup process.
Expected behavior
My BMW is a Diesel mild hybrid (320d G21), I am not sure if EMISSION_CHECK is reserved to cars with different engine.
Which Home Assistant version are you using?
2022.10.x
What was the last working version of Home Assistant Core?
No response
What is your region?
Rest of world
MyBMW website

[X] I can still successfully login to the BMW MyBMW website and the car status is available there.
[X] I have MyBMW enabled for my vehicle.

Number of cars

[ ] I have 2 or more cars linked to the MyBMW account.
[ ] I have a Mini vehicle linked to my account.

Output of bimmer_connected fingerprint
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response

Thanks for the report! This shouldn't break anything except not giving details about this specific condition based service.

Yep, I do confirm nothing seems to be broken. Maybe the code could check if the function is available or not before trying to retrieve those data.


Test of hotfix!

Fixed the title!

@JavierRueda @agnerh this was a hotfix branch. It was merged into master. Then, in #2, it was also merged into develop


Could wg-triage get write access on rust-lang/rust repo?

Currently, triage access doesn't allow us to edit issues. It means we have to ping someone who has write access or so when we want to edit issues. Personally, I wouldn't like to ping them each time only to edit it.
for example:

update outdated examples in OP
put a checkmark
add backticks for snippet to be displayed correctly

So, could we get write access?
cc @Mark-Simulacrum and @pietroalbini

I would like @pietroalbini's opinion here in particular, but generally lean towards allowing the access even if that's not great. Ideally we'd have triage access be sufficient here but for whatever reason GitHub has decided that they don't want a role that provides just editing of the issue tracker I guess.

Yeah, I'd be ok for this since most of the triagers are trusted now.

So, I'd say that this request is approved, thanks! Could @Mark-Simulacrum or @pietroalbini give us the access?

Updated the permissions.


OpenAI API key disabled

API key disabled after inserting it into the chat or after using it in the playground.

This can happen when you have actively been using an API key in a different website/domain and for the first time attempt to use it in on a different website such as www.deepchat.dev. This change of use can raise a warning on the OpenAI servers and your API key may get removed.
To clarify - the www.deepchat.dev website does not use your API keys - it is the Deep Chat component that makes the requests to your chosen services so you are safe to use your key in this website.
To overcome this problem we recommend you to simply generate a new key.


test(benchmarks) Improve life & wagon, create wasmer

Use Life's benchmark cases to improve all the benchmarks.

bors try

bors try

bors r+


Minor nitpicks for PTH210

Summary
Some fairly inconsequential nitpicks I had locally and was about to push before @MichaReiser merged #14779 😆
Test Plan
cargo test -p ruff_linter --lib

Any idea why there's an ecosystem change?

Any idea why there's an ecosystem change?

No clue. Neither of the differences being reported is relevant to the changes being made here.


No map-files in Cloudflare

As the title suggests there seems to be no map-files on Cloudflare CDN:
https://cdnjs.com/libraries/aos
Could you generate two map-files and upload them?

I removed auto-generated links to source maps in 2.3.0, so you should no longer get this error @orjanb-ne


compile issue

Describe your problem here.
its says source/psychlua/CallbackHandler.hx:55: characters 18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
export/release/windows/haxe/ApplicationMain.hx:153: characters 5-36 : ... Called from macro here
C:/HaxeToolkit/haxe/lib/flixel-ui/2,5,0/flixel/addons/ui/FlxUICursor.hx:564: characters 9-11 : Warning : Potential typo detected (expected similar values are flixel.addons.ui.SortMethod.ID)
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

first two is real error, third is just warn about code and dont stop compiling
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Libraries-versions

Every libraries that you need all in this page
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Libraries-versions
Follow the version that this page provided you,run
haxelib install [library] [version]
or
haxelib set [library] [version]
I think this will help ya
If in the complie,you saw the WARNING message,ignore it,only the ERROR message should care about it

it still won't compile
On Sat, Mar 9, 2024 at 4:50 AM Pandaman @.***> wrote:

Every libraries that you need all in this page
https://github.com/ShadowMario/FNF-PsychEngine/wiki/Libraries-versions
Follow the version that this page provided you,run
haxelib install [library] [version]
or
haxelib set [library] [version]
I think this will help ya
If in the complie,you saw the WARNING message,ignore it,only the ERROR
message should care about it
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/14271#issuecomment-1986823007,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7SHQUW4KBUBMUFYYY5SG23YXLSOTAVCNFSM6AAAAABEKNEAMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWHAZDGMBQG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Update ur lua

and how do you do that?
On Mon, Mar 11, 2024 at 11:13 PM Ender69 @.***> wrote:

Update ur lua
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/14271#issuecomment-1990224793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7SHQUVGEQ6N2MXUUESDMV3YXZ6EZAVCNFSM6AAAAABEKNEAMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJQGIZDINZZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

so i did some stuff i now i get the
error C:/HaxeToolkit/haxe/lib/flixel/5,6,2/flixel/system/macros/FlxDefines.hx:104:
characters 61-65 : Unsupported Haxe version! Supported versions are 4.2.5
or newer (found 4.105).
On Tue, Mar 12, 2024 at 7:12 PM Maverick Boyles @.***>
wrote:

and how do you do that?
On Mon, Mar 11, 2024 at 11:13 PM Ender69 @.***> wrote:

Update ur lua
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/14271#issuecomment-1990224793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7SHQUVGEQ6N2MXUUESDMV3YXZ6EZAVCNFSM6AAAAABEKNEAMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJQGIZDINZZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

nvm i am back to this source/psychlua/CallbackHandler.hx:55: characters
18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
export/release/windows/haxe/ApplicationMain.hx:153: characters 5-36 : ...
Called from macro here
C:/HaxeToolkit/haxe/lib/flixel-ui/2,5,0/flixel/addons/ui/FlxUICursor.hx:564:
characters 9-11 : Warning : Potential typo detected (expected similar
values are flixel.addons.ui.SortMethod.ID)
On Thu, Mar 14, 2024 at 6:18 PM Maverick Boyles @.***>
wrote:

so i did some stuff i now i get the
error C:/HaxeToolkit/haxe/lib/flixel/5,6,2/flixel/system/macros/FlxDefines.hx:104:
characters 61-65 : Unsupported Haxe version! Supported versions are 4.2.5
or newer (found 4.105).
On Tue, Mar 12, 2024 at 7:12 PM Maverick Boyles @.***>
wrote:

and how do you do that?
On Mon, Mar 11, 2024 at 11:13 PM Ender69 @.***>
wrote:

Update ur lua
—
Reply to this email directly, view it on GitHub
https://github.com/ShadowMario/FNF-PsychEngine/issues/14271#issuecomment-1990224793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7SHQUVGEQ6N2MXUUESDMV3YXZ6EZAVCNFSM6AAAAABEKNEAMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJQGIZDINZZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

i got it to work


Can you set the Comment attribute on a field name in Postgres?

Is there a way to set the Comment attribute on a Postgres field when defining it in JSON?
I've tried without luck:
attributes: {
		name: {
			type: 'string',
			required: true,
			comment: 'This is the user name'
		},
		...
}

Hey, that's a great question.  You might want to file this question in the sails-postgresql repo.  That said, I don't believe this is supported.  But it would be a nice feature to add!

Thanks!


Unable to start Web-app because of diagnostic logging in Opentelemetry SDK.

Package Name: azure-monitor-opentelemetry
Package Version: 1.6.1
Operating System: Linux
Python Version: 3.11
Describe the bug:
I am deploying a python based web-app on Azure App service. This is docker image which is being pulled from Azure container registry.
The problem is coming from inside the Azure Monitor OpenTelmetry SDK. The sdk creates log files for diagnostic logging within the following directory : /var/log/applicationinsights/ (for linux based web apps). This can be seen from the following code from the above sdk. This File path has been hardcoded in the sdk and is not configurable. I start the app using an app_user, which has access to the app related volumes in the container but not to the above directory.
if not exists(_DIAGNOSTIC_LOG_PATH):
       makedirs(_DIAGNOSTIC_LOG_PATH)
   f_handler = logging.FileHandler(
       join(
           _DIAGNOSTIC_LOG_PATH, _DIAGNOSTIC_LOGGER_FILE_NAME
       )
   )

This was added as part of the following bug: Adding diagnostic warning when distro detects RP attach (https://github.com/Azure/azure-sdk-for-python/pull/34971)
Due to this the App does not start with the following stacktrace:
2024-10-25T04:33:34.877284066Z File "/usr/src/app/doc_generator_api/main.py", line 19, in <module>
2024-10-25T04:33:34.877309466Z configure_azure_telemetry(option="basic")
2024-10-25T04:33:34.877361967Z File "/usr/src/app/doc_generator_api/dependencies/instrumentation.py", line 11, in configure_azure_telemetry
2024-10-25T04:33:34.877378367Z configure_basic_azure_monitor()
2024-10-25T04:33:34.877383767Z File "/usr/src/app/doc_generator_api/dependencies/instrumentation.py", line 25, in configure_basic_azure_monitor
2024-10-25T04:33:34.878324584Z configure_azure_monitor(
2024-10-25T04:33:34.879013896Z File "/agents/python/azure/monitor/opentelemetry/_configure.py", line 98, in configure_azure_monitor
2024-10-25T04:33:34.879018996Z _send_attach_warning()
2024-10-25T04:33:34.879022896Z File "/agents/python/azure/monitor/opentelemetry/_configure.py", line 222, in _send_attach_warning
2024-10-25T04:33:34.879026897Z AzureDiagnosticLogging.warning(
2024-10-25T04:33:34.879030297Z File "/agents/python/azure/monitor/opentelemetry/_diagnostics/diagnostic_logging.py", line 87, in warning
2024-10-25T04:33:34.879034197Z AzureDiagnosticLogging._initialize()
2024-10-25T04:33:34.879037997Z File "/agents/python/azure/monitor/opentelemetry/_diagnostics/diagnostic_logging.py", line 68, in _initialize
2024-10-25T04:33:34.879041897Z f_handler = logging.FileHandler(
2024-10-25T04:33:34.879883312Z ^^^^^^^^^^^^^^^^^^^^
2024-10-25T04:33:34.879896112Z File "/opt/conda/lib/python3.11/logging/__init__.py", line 1181, in __init__
2024-10-25T04:33:34.879905312Z StreamHandler.__init__(self, self._open())
2024-10-25T04:33:34.879910112Z ^^^^^^^^^^^^
2024-10-25T04:33:34.879914512Z File "/opt/conda/lib/python3.11/logging/__init__.py", line 1213, in _open
2024-10-25T04:33:34.879919112Z return open_func(self.baseFilename, self.mode,
2024-10-25T04:33:34.879923612Z ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-10-25T04:33:34.879928513Z PermissionError: [Errno 13] Permission denied: '/var/log/applicationinsights/applicationinsights-extension.log'

Expected behavior: There should be a way to disable diagnostic logging in Azure monitor.

Thanks for the feedback, we’ll investigate asap.


Scrolling diagonally on a trackpad no longer works. Also the traditionalScroll prop does not work.

Describe the bug
Scrolling diagonally on a trackpad no longer works since version 0.22.0. Additionally the traditionalScroll prop does not work.
It seems like it was introduced in https://github.com/namespace-ee/react-calendar-timeline/pull/427 in the src/lib/scroll/ScrollElement.js file.
To Reproduce
You can observe this behaviour in the regular codesandbox: https://codesandbox.io/s/w6xvqzno4w
Gif from library version 0.21.0 (commit 9d0d3fdfd7382d7770af6a8462c11bff3d44acb8)

Gif from current library version 0.23.0

Steps to reproduce the behavior:

Go to codesanbox
Try scroll diagonally on a trackpad

Expected behavior
The ability to scroll diagonally when using a trackpad.
Library Version
0.22.0 and 0.23.0
Desktop (please complete the following information):

OS: iOS
Browser: Chrome 71.0.3578.98 and Safari 12.0.2

Hi can I help????


Config files

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@08aede8). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #1   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         1
  Lines             ?         2
  Branches          ?         0
==========================================
  Hits              ?         2
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

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


[14.0] ISPN-14969 Elytron 1.20.4.Final

https://issues.redhat.com/browse/ISPN-14969

@karesti I've added a commit to fix the Jakarta issue

Failures are unrelated


Added choosable formatting filesystem for LVs

LXD has hardcoded ext4 as the filesystem to be used when LVM storage backend is choosed, I don't like very much ext* filesystems and prefer xfs over them so I've done a little patch to be able to use it with LXD.
To be able to choose preferred filesystem a key was added to the configuration parameter of LXD: storage.lvm_fstype, for now it's value can be only ext4 (default, so I don't break anything) and xfs, but it's trivial to add other filesystems.
For now the filesystem has to be choosen at LXD installation time, in the future I think it will be better to let the users to choose filesystem at container creation time so it'll be possible to have different filesystem for different container and so migrating a container from one host to another will be possible even if they have different default filesystem.

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Hi,
Can you sign-off your commit? (see CONTRIBUTING.md)
Also, why is it necessary to call xfs_admin all the time for that uuid? isn't that done for you at mount time?
If it is, then you could use the generic mounting function from lxd/devices which detects the filesystem type for you.

Hi, Stéphane, thank for your comment.
Generating a new xfs'UUID when creating a container, an image or a snapshot is necessary because isn't possible to mount two xfs volumes with the same UUID.
root@lina:~# xfs_admin -u /dev/lxc/w3
UUID = 834ea86b-f8a8-4076-b04d-16f0b3d4a898
root@lina:~# xfs_admin -u /dev/lxc/w4
UUID = 834ea86b-f8a8-4076-b04d-16f0b3d4a898
root@lina:~# mount /dev/lxc/w3 /tmp/p
root@lina:~# mount /dev/lxc/w4 /tmp/q
mount: wrong fs type, bad option, bad superblock on /dev/mapper/lxc-w4,
           missing codepage or helper program, or other error

       In some cases useful info is found in syslog - try
       dmesg | tail or so.
root@lina:~# dmesg | tail -1
[1116904.345407] XFS (dm-11): Filesystem has duplicate UUID 834ea86b-f8a8-4076-b04d-16f0b3d4a898 - can't mount
root@lina:~#

Okay, I guess that makes sense. I see two cases where it's indeed needed (new container from an image and mounting a snapshot), it's however not needed when creating a new image (commented).
I'd also prefer if you made the xfs uuid re-generation a separate function which you call from the two places where it's needed.
Thanks!

Oh and can you also update specs/configuration.md with the new config key?

Hi Stéfan, I've made a commit with your suggestions.

Cool, just one tiny change to specs/configuration.md and we'll be good to test and merge this!

I changed it.

Great, lets throw that at Jenkins.
jenkins: test this please

Looks like all the LVM tests are failing, so there's something wrong with this pull request, looking at the diff again...

Oops, the problem was pretty obvious, I should have thought of it on first review :)

Stupid me that I didn't do it on first commit...

jenkins: test this please

Hi, I didn't have time for a full review, but wanted to leave a drive-by note about FS features required by LXD/LVM:
xfs should be fine in terms of support for fs_freeze, which is necessary to make the snapshotting and copying work safely.
However, it might be worth looking into whether xfs has any similar issues to what we found with ext4, where the first snapshot was the expected small size, but subsequent snapshots of the same origin LV were larger and larger. In LXD terms, this meant that the first container you created from an image used a tiny amount of additional space, but subsequent containers created from the same image would start getting bigger, counter to what you'd expect from a COW snapshot.
This was apparently due to some background initialization (see the lazy_itable_init etc in the mkfs.ext4 call) that meant that the origin LV was still changing as the initial containers were being created, causing inefficient allocation.
We noticed this by tracking usage as we created a large number of containers with the LVM backend, using basically the https://github.com/lxc/lxd/blob/master/test/extras/speedtest_create.sh script.


Does not support win32 or win64?

Does magellan support windows OS? I tried installing it to my local project and also cloned the git example given at the documentation.  It runs the sample tests in the example nightwatch project but still gets this error.
[INFO] [Magellan] ============= Suite Complete =============
[INFO] [Magellan]      Status: PASSED
[INFO] [Magellan]     Runtime: 19.8s
[INFO] [Magellan] Total tests: 4
[INFO] [Magellan]  Successful: 4 / 4
[WARN] [Magellan] executor teardownRunner error: Error: Operation unsupported on Windows
(node:17160) UnhandledPromiseRejectionWarning: Error: Operation unsupported on Windows
at getTree (C:\nw-magellan\boilerplate-nightwatch\node_modules\testarmada-tree-kill\index.js:141:13)
at checkForZombies (C:\nw-magellan\boilerplate-nightwatch\node_modules\testarmada-tree-kill\index.js:21:16)
at Object.getZombieChildren (C:\nw-magellan\boilerplate-nightwatch\node_modules\testarmada-tree-kill\index.js:85:3)
at Object.module.exports [as processCleanup] (C:\nw-magellan\boilerplate-nightwatch\node_modules\testarmada-magellan\src\util\process_cleanup.js:23:20)
at Promise.all.then.catch (C:\nw-magellan\boilerplate-nightwatch\node_modules\testarmada-magellan\src\cli.js:443:29)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
(node:17160) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:17160) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
PC Specs:
Processor: Intel(R) Core(TM) i7-7600U CPU @2.80GHz 2.90GHz
System type: 64-bit Operating System, x64-based processor

Also having this issue.

Happening on both Windows 7 x64, and Windows 10 x64.
node: v10.15.3
npm: v5.3.0
yarn: v1.15.2
testarmada-bloop: v0.0.4,
testarmada-magellan: v^10.1.0,
testarmada-magellan-browserstack-executor: v^1.0.0-beta4,
testarmada-magellan-local-executor: v^2.0.0,
testarmada-magellan-nightwatch-plugin: v^7.0.0,
testarmada-magellan-saucelabs-executor: v^5.0.0,
testarmada-nightwatch-extra: v^5.0.0,

Any help with this, would be greatly appreciated.


Login Flow improvements

[ ] Select Account (Remove possible)
[x] User Icon
[ ] Bach to select account after choosing one

I will close this issue since it is to generic


[WARNING] sqlite3-binding.c: In function ‘sqlite3SelectNew’

Hi, while I'm developing an appliation using gorm I noticed new messages shown in my terminal and said there is an issues in this package:
# github.com/mattn/go-sqlite3
sqlite3-binding.c: In function ‘sqlite3SelectNew’:
sqlite3-binding.c:128049:10: warning: function may return address of local variable [-Wreturn-local-addr]
128049 |   return pNew;
       |          ^~~~
sqlite3-binding.c:128009:10: note: declared here
128009 |   Select standin;
       |          ^~~~~~~

I hope this message helps you out to fix the issue. Thanks

#803

Does anyone know of a solution to fix the warning messages?

@miguelmota That depends what you consider to be a fix. As mentioned in the linked issue, this warning is a false positive that could be addressed by changing how SQLite itself is implemented, but it is unclear whether such a change will be made. Alternatively, that flag could be added to this wrapper library so you don't need to manually set an environment variable.
https://github.com/mattn/go-sqlite3/blob/aa77c03e2fcb7ed283920d09a0fb6b873488b5be/sqlite3.go#L12-L24
#cgo CFLAGS: -Wno-return-local-addr

@rittneje that worked perfectly, thanks!


select2 focus is not working for tab order

Expected behavior and actual behavior
From input if I use tab it will focus on select2. On focus, content message should change. but content message is not updating.
https://jsfiddle.net/Lupj6oaw/

This is a duplicate of https://github.com/select2/select2/issues/3876.


Support 10-step RGB gamma

Request: Allow user to specify RGB gamma similar to HDTV's in multiple steps (amount of steps can be defined by the user, e.g. 1 like now or 2, 5, 10, ...) and use the (closest) gamma level that matches the pixel's/regions brightness for calculation of the RGB values for the LED strip.
Why?
Correct color reproduction on a HDTV requires the following, else the resulting picture will not be as expected (I am restricting the list to what is important for my request):

Calibrating the RGB values on the HDTV so that they "match" specific color ( WRGBCMY / white, red, green, blue, cyan, magenta, yellow)

White balance using RGB gamma calibration (how do colors "mix")

I've done the calibration on my HDTV with calibration hardware & software and the result is astonishing and was well worth the costs and effort for me. However, hyperion is not matching the colors of what I see on the screen and that is very irritating.
Hyperion allows calibrating RGB values for the required colors, resulting in a newly calculated RGB color - this is done by displaying an WRGBCMY image.
Hyperion also takes a gamma value to adjust the final RGB values displayed on the LED strip - this is done by displaying a white image.
The problem: gamma is restricted to one value for each R, G and B. This means that no matter how bright a calculated RGB color is (value 0-255), the same gamma value will be used for the calculation.
As a result, displaying an image which shows a single RGBCMY color in different brightness next to each other results in hyperion creating incorrect colors: At some brightness level the LED strip just shows one color as the RGB values cannot go any higher.
Additionally, the RGBCMY (no W) may be way off from what it should be as gamma calibration happened only on W.
HDTV televisions on the other hand allow defining multiple RGB gamma values, e.g. in 10 brightness steps (close to black till maximum). This allows defining the RGB gamma in 10 ranges, splitting up a single R/G/B color from a single range 0-255 to 10 ranges (0-24, 25-49, ... -255). This allows perfect RGB calibration for every WRGBCMY color at every brightness level. This is what any calibration hardware & software target. Without this, the colors cannot match.
Every LED strip I've used in the last years was never able to show the correct colors at a given brightness. I can change the gamma when I see red so that red has a nice brightness increase visible in an image with 10 ranges - perfect! But then blue is wrong, or green. A yellow suddenly becomes brownish. Changing gamma for yellow results in perfect yellow matching the TV in the middle of the range, but is still brownish in the higher brightness.

Closed due to ignoring request template.
Pleas use our request template.


Canvas becomes laggy when adding many polygons

I'm loading around 2000 small polygons with ~20 points per polygons on a 1000x1000 pixel image. When loading this dataset, the canvas becomes really laggy/sluggish. This is in itself not very surprising, there has to be a limit somewhere. However, when zooming in and panning around, the canvas stays laggy, even though only a few polygons are visible.

I can also load the polygons on a html canvas using the fabric.js canvas framework. Here, the zooming and panning is also relatively laggy, although a lot less already than the cvat canvas implementation:

When zooming in, the panning becomes really smooth, because only the objects which are present on the current subsection of the canvas are actually rendered and the rest is ignored.
Is there a way to mimic this behavior in cvat?

Git hash commit (git log -1): 7456a1
Docker version docker version (e.g. Docker 17.0.05): 19.03.13
Operating System and version (e.g. Linux, Windows, MacOS): Ubuntu 20.04

Hi Boris, thanks for looking into this. I attached a profile file, you can see the frame rate drop to around 8 fps.
profile.tar.gz

Hi,

From profiling you provided I actually see that fabric.js (I am not familiar with it, but I suppose that it works based on canvas) works faster than svg.js (works based on svg technology).

Speaking about fabric.js, I expect that it renders all the content once to a single image and image navigation (scrolling, moving, etc) should not cause lagging at all. But as profiling says renderAndReset is called after these events what takes some time to be done (I suppose it can be related with svg -> canvas converting), but in browser it finally becomes a html canvas (right?). And canvas technology significantly stricts our abilities related with interaction with the objects, so, we cannot use it.

Viewing a profiling file for svg.js implementation we can see, that the must of time is required to rerender DOM (for example changing position of all the elements on the screen, not only visible I suppose, but it is not a question of CVAT I believe). And according to profiling, to be honest, the way CVAT works with these objects is quite optimized (at least in some previous versions it could take much more time :) ). Speaking "optimized" I mean, that we try to avoid extra rendering when it is possible. But I do not see ways to influence browser render time somehow (maybe try to use some css properties like shape-rendering: optimizeSpeed, but I do not think it will help significantly)

Hello, thanks for your answer.
You are right, fabric.js uses the html canvas element to draw on, and does not depend on .svg graphics. I don't exactly know how fabric.js works internally, but objects are select-able, movable and resizable, just like with the .svg implementation here. I think that all items are redrawn on every zoom, pan, etc. as well in fabric.js, and  that it's not just an image which you can zoom/pan into.
I see that it would be a major undertaking to convert to using a canvas as a main drawing object, and do not expect you to move to this approach. I just wanted to use it as an example to show how quick the polygons could be rendered on a canvas.
I did some more testing today, and did notice that after collapsing the side bar listing all of the objects on the right side of the screen, the frame rate increased from 8 fps to circa 20 fps with the same items on screen (see profile). This definitely seems like a place where performance gains are relatively easy to make.
A typical frame with uncollapsed sidebar:

A typical frame with a collapsed sidebar:

The profile file:
profile_yes_no_sidebar.tar.gz
What do you think about this?

Hi again @joannanosek ,
Thank you for you experiments, the results are interesting. You are right, migrating to canvas is a huge task and it is out of our scope at this time. Let's add "enhancement" label and push the issue to backlog. Maybe somebody from community will be interested in this implementation.
I suppose the main reason why you have significantly increased performance when the side panel is hidden is that all these elements have css property display: none and browser doesn't need to re render them. In spite of these elements aren't changed during just moving the canvas, looks like browser rerenders them too. I am not sure can we optimize it or not, but it might be a way to optimizing.


gnome3.webkitgtk: 2.20.3 -> 2.20.4

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/webkitgtk/versions.
meta.description for gnome3.webkitgtk is: '"Web content rendering engine, GTK+ port"'.

Checks done (click to expand)

built on NixOS
/nix/store/6avrvnv9dsinnw3nkapczagpnwnv32c9-webkitgtk-2.20.4/bin/WebKitWebDriver passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 2.20.4 with grep in /nix/store/6avrvnv9dsinnw3nkapczagpnwnv32c9-webkitgtk-2.20.4
directory tree listing: https://gist.github.com/15d22d023b00f6d2d297a65348685770
du listing: https://gist.github.com/33cfea385f907b43817d452cf8dc4269

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout ba8d69ccec0f04a641dee1e7619af4be31ddfc8f && nix-shell --pure -I nixpkgs=. -p gnome3.webkitgtk

The postConfigure can be removed now, 2.20.4 fixed the issue: https://webkitgtk.org/2018/08/06/webkitgtk2.20.4-released.html
cc @ryantm is it easy to make small mods to these PRs, or should I create a new one?

Done.


Cannot override server.port with JVM parameter in Spring Cloud Client

Application server port cannot be overridden by JVM parameter -Dserver.port if the property is present in Spring Cloud Config.
I'm using Spring cloud Greenwich.SR3 version and Spring Boot 2.1.9.RELEASE
The sample application could look like this
@SpringBootApplication
public class RandomPortApplication {

	public static void main(String[] args) {
		SpringApplication.run(RandomPortApplication.class, args);
	}

}

with bootstrap.yml like
spring:
  application:
    name: random-port
  cloud:
    config:
      discovery:
        enabled: true

and with the pom.xml
<dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>org.springframework.cloud</groupId>
      <artifactId>spring-cloud-dependencies</artifactId>
      <version>Greenwich.SR3</version>
      <type>pom</type>
      <scope>import</scope>
    </dependency>
  </dependencies>
</dependencyManagement>

<dependencies>
  <dependency>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-web</artifactId>
  </dependency>
  <dependency>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-config</artifactId>
  </dependency>
  <dependency>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-netflix-eureka-client</artifactId>
  </dependency>
</dependencies>

<build>
  <resources>
    <!-- make sure that configs are not packaged and read them from Cloud Config Server -->
    <resource>
      <directory>src/main/resources</directory>
      <excludes>
        <exclude>application*</exclude>
      </excludes>
    </resource>
  </resources>

  <plugins>
    <plugin>
      <groupId>org.springframework.boot</groupId>
      <artifactId>spring-boot-maven-plugin</artifactId>
    </plugin>
  </plugins>
</build>

This requires Eureka server running with default config and Cloud Config server having the following application.yml for our service
server:
  port: 0

If I run such service with
mvn spring-boot:run -Dserver.port=7777 -Deureka.client.serviceUrl.defaultZone="http://localhost:8761/eureka/"

The service uses random port instead of the provided one!
The things that I notice

if I move server.port=0 to bootstrap.yml, the port can be overridden
or if I add dependency org.springframework.cloud:spring-cloud-config-server (although does not make sense to have this dependency) it works fine, I can override the port

Try setting spring.cloud.config.override-system-properties=true. See https://cloud.spring.io/spring-cloud-static/Hoxton.RELEASE/reference/html/spring-cloud-hoxton-configprops.html

Hello @spencergibb tried with override-system-properties  and also doesn't work.
I created a sample repo here: https://github.com/dome313/random-port with only Cloud Client and Cloud Config which shows that is not possible to override port with JVM property -Dserver.port

spring.cloud.config.override-system-properties=true needs to be in a property source returned from config server.

Adding
spring:
  cloud:
    config:
      override-system-properties: false

to a property source returned from config server did the trick.
I updated my git repo to reflect that.
Thanks @spencergibb. You can close the ticket.


Add a user-agent when using Quilt Meta (and update dependencies)

As requested by @Akarys42 in #16

I'm not free at the moment, I'll merge master later.

@CheaterCodes @EnnuiL Please do have a look

The reason I'm liberally cloning the client is that it's an Arc internally so it's very cheap.

I guess this is why glob defaults aren't ideal.
Anyways, @CheaterCodes it should work now. I also made the error handling in main() a little neater.


[#92205218] clean up some js syntax

This PR cleans up some javascript syntax.
@ramen74 for your perusal :bow:

LGTM

LGTM


Truncated text within flex containers breaks out rather than being contained/truncated

I've found an interesting bug and a partial solution. I have text that is truncated inside a flex media object (and flex grids). In FFOX and IE, the truncated text doesn't truncate (Chrome, Safari and Edge work as expected). The truncated text takes up its full length, rudely smashing anything in its path. I created a minimum test case here: http://codepen.io/stefsullrew/pen/Kpyarb
I was able to fix the FFOX problem by adding min-width:0 to both the .col of my grid and the body of the media object (and I understand that's probably ok and Chrome is about to implement the same default now). My trouble remains with IE 11.
Philip, you'll see the code of your flexi media object in my Pen. My first problem was, I had no text showing at all (with the truncation class) in IE. I changed the flex:1 on the media object's body to flex: 1 1 auto; and that was fixed. But IE still wants a pixel-based max-width on the truncated text, not a percentage-based, to truncate. Very aggravating since this needs to be responsive and my designer doesn't want to add a width.
(I am able to truncate in IE in grids that have explicit percentage widths set on the columns.) I'd love to hear the thoughts of you wise flexperts here. :)

Started working on this, I think ultimately you're hitting some of the changes to min-width (Edge/FF/Canary has these), also you're hitting an interop issue between MS Edge/FF and Canary (somewhat similar to this one but I can't be sure at this point). I have gotten it closer to working in IE11, I need to get the ellipses working but this exposes the interop issue stated above.
In this use case you're trying to create, you'll need roughly three workarounds to make this work which is sub optimal, especially due to the complexity of flex and current changing in implementations (so you may break). My honest suggestion would be to not use flex for this entire use case, but try to simplify the nesting of the flexes.
I'll continue to try and find a work around that works in IE11+/FF/Chrome/Canary but there are so many interop issues between the four of those that it might be impossible :)
@philipwalton If you're able to take a glance as well, this will need a third pair of eyes and I would appreciate it as I'm pretty slammed ATM (thanks!!!)

So I think this is a genuine IE11 bug, and it appears to be fixed in Edge, so that's good. I'm not entirely sure if this is just one bug or a combination of things, so it's hard to know how to phrase it for others to avoid.
I think the Firefox issue you were noticing is actually Firefox doing it correctly because it implements the new min-width: auto part of the spec (Edge and Chrome Canary too). However, for your use-case, setting min-width: 0 is probably what you want since a single line of text larger than the container would cause overflow.
As for solving your immediate problem, I think you can do that by approaching the layout from a different direction. Here's what you have currently:

You have two columns, the left one has a computed flex value of 0 1 auto and the right one 0 0 auto (aka none).
Neither column is growing, but they're spaced apart by setting space-between on the container.

From a layout perspective, this is basically the same as the following:

You have two columns, the left one has a computed flex value of 1 1 0% and the right one 0 0 auto (aka none).
When the content in the left column is sparse, there will still be space between the columns because the left column is set to grow.

I put together this codepen using this second approach and it's working in FF/Chrome/IE11/Edge/Safari for me. I hope that helps!
p.s. I really wish codepen had support for diffing two pens, so you could easily see what I changed!

Sorry, was on PTO yesterday for my birthday. Having a look now. :) And yes, I agree — after reading through the bugs and changes a couple days ago, FFOX seems to be doing it right and Chrome is about to implement the same it appears: https://code.google.com/p/chromium/issues/detail?id=426898
What I'm working on is an actual framework (that people will use in a variety of ways), so I'm trying to work things out without being too precious. If the rule needs to be "no flexbox in flexbox" that could be quite a frustration for the teams using this since grids are flexbox (and can be nested), media blocks are flexbox and horizontal lists are flexbox (with fallback for oldies). I may need to rethink and pull some of these items back into their older form. :sigh:

So @philipwalton solution that works cross browser won't work for this? In regards to flex updates, FF/Chrome still need to implement the abspos changes, but at that point Canary/Edge/Firefox will be at the same spot. The min-width change is actually in Canary now, but the interop issue you've found is a potential bug, but the overall feature is there. Let us know if your saying that the markup and layout has to remain as you set it up and we'll try to do what we can, but as stated there are bugs in these since they're so new and just now starting to be used which reveals edge cases which at times can be hard to test for.

Thanks @gregwhitworth - I'm testing Philip's solution in the real component right now: http://codepen.io/anon/pen/Xbzgzw?editors=110
Since this was a highly simplified proof of concept, I have to test whether this solution will work in my actual components. I shall report back.

It's not working. I'm making a new, more complex CodePen (since I boiled it down to the least code where the problem still showed up). Stay tuned.

@gregwhitworth - The CodePen you shared has overlapping columns with no truncation at all in IE11: http://www.screencast.com/t/OuXtygYS6zn (And yes, with the min-width: 0 I was able to solve my FFOX truncation — and I understand that other browsers will be making the same spec change.)
However, nothing I do is solving the IE11 issue. I'm just going to take the media object out and inline-block the three items that need to stay together (which I believe will also get rid of a nested grid row). Like I said, what I'm building will be used by other people, so I can't get "too hacky"... Thanks for trying to assist though. Not sure exactly what the IE11 bug is but seems it should be called out? (I don't know that there's a workaround though.)

Completely agree, I'm not sure if there is a workaround either, it just seems like a straight up bug. Regarding my codepen, it wasn't finished, was just trying to get them to where they could at minimum overlap, I was going to try to figure out how to cause the truncation inside of the same markup, but was waiting to see if @philipwalton workaround would work. More than likely as you stated it may require too much hacking, especially for your purposes.

Ok, I'm going to close this for now. @stefsullrew feel free to reopen if you're able isolate your specific issue.

Hey @philipwalton and @gregwhitworth - Even though this is closed, I'm going to leave the solution here for others. I've commented the new pen I forked from my first one. The solution was a couple of things.
First, I took Philip's advice and changed from using justify-content: space-between:... to simply setting the second .col not to flex (which has the same effect).
Second, I reverted my .Media-body back to flex:1; as it had originally been, — but left the min-width:0; which has become important due to the changes in the flexbox spec.
Third, I added a class that we're finding very useful since we truncate loads of things and use a lot of flexbox. We call it .has-flexi-truncate, and it goes on the .col that contains the nested flexi-object that has the truncation inside (in this demo, it's the media object).
Hope this helps someone. I made notes in the new pen indicating what I changed since there's not a compare feature: http://codepen.io/stefsullrew/pen/WvWWqJ?editors=110


Error when building app - Could not resolve all files for configuration ':react-native-month-year-picker:implementation'

When running the app on android, I get this error: "Could not resolve all files for configuration ':react-native-month-year-picker:implementation'", here's the screenshot:
My RN version is 0.70.3 and react-native-month-year-picker version is ^1.8.2.

I was able to fix it when removing this line from android/build.gradle but it should work without changing anything from the package:

same error

+1

Hello! Thanks for using the library! I've been working on this one and a couple of other issues reported. I'm hoping to have a new version by EOW. Regards!

@gusparis and update on this?

@gusparis I have same issue
`* Where:
Build file 'C:\Users\sazza\Projects\ReactNative\taxi\node_modules\react-native-month-year-picker\android\build.gradle' line: 115

What went wrong:
A problem occurred configuring project ':react-native-month-year-picker'.

Could not resolve all files for configuration ':react-native-month-year-picker:implementation'.
Could not resolve com.facebook.react:react-native:+.
Required by:
project :react-native-month-year-picker
Cannot choose between the following variants of com.facebook.react:react-native:0.70.5:
- debugVariantDefaultRuntimePublication
- releaseVariantDefaultRuntimePublication
All of them match the consumer attributes:
- Variant 'debugVariantDefaultRuntimePublication' capability com.facebook.react:react-native:0.70.5:`

Fixed on #140. Can you guys check?


python3Packages.atomman: disable python 3.10, disable failing tests

Description of changes
ZHF: #172160
According to Hydra, some tests are failing. Checking the package on PyPI, it is not intended to build on Python 3.10.
Something to consider is that the tests won't run if using pytestCheckHook because of this error:
...
Executing pytestCheckPhase
============================= test session starts ==============================
platform linux -- Python 3.9.12, pytest-7.1.1, pluggy-1.0.0
rootdir: /build/source
collected 20 items / 14 errors

==================================== ERRORS ====================================
_____________________ ERROR collecting tests/test_root.py ______________________
ImportError while importing test module '/build/source/tests/test_root.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/nix/store/01kia41csjia67pry1rv828i9pvnnqfq-python3-3.9.12/lib/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/test_root.py:6: in <module>
    import atomman as am
atomman/__init__.py:17: in <module>
    from .core import *
atomman/core/__init__.py:2: in <module>
    from .dvect import dvect
E   ModuleNotFoundError: No module named 'atomman.core.dvect'
...

This is why pytest is ran manually.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 172638 run on x86_64-linux 1

1 package built:

python39Packages.atomman

@ofborg build python3Packages.atomman


Move to SDP pipeline

Docs Build status updates of commit c139fba:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docs/api/overview/openxml/index.yml
:x:Error

Details

:x:Error

Details

docs/api/overview/openxml/index.yml

[Error] Error transforming model "W:\4d5a-o-unknown\raw-\pszuseuc.dag\rawmodel\api\overview\openxml\index.raw.json" generated from "api/overview/openxml/index.yml" using "ManagedReference.html.primary.js". Error running Transform function inside template preprocessor:

[Error] Error transforming model "W:\4d5a-o-unknown\raw-\pszuseuc.dag\rawmodel\api\overview\openxml\index.raw.json" generated from "api/overview/openxml/index.yml" using "ManagedReference.html.primary.js". Error running Transform function inside template preprocessor:

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

Docs Build status updates of commit 89f8460:
:clock10: Pending: waiting for processors (39 builds ahead of you)
:warning: Docs Build is busy, currently there are 39 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 89f8460:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:x:Error

Details

:x:Error

Details

.openpublishing.publish.config.json

Line 290: [Error] Restore of dependent packages failed. Please check your .openpublishing.publish.config.json?on the configuration of the dependent packages or ask for help?in Docs Support teams channel. Please also include the following error code for our team to troubleshoot: 1

Line 117: [Error] Restore nuget package W:\0aou-s\.optemp\dependentPackages\Microsoft.DocAsCode.ECMA2Yaml\packages.config in folder:W:\0aou-s\.optemp\packages failed, possibly due to transient errors from nuget servers. Please retry building your content. If the issue still happens, open a ticket in http://SiteHelp and include the build report and the following error message for our team to troubleshoot:  Errors in packages.config projects     Unable to find version '1.0.669' of package 'Microsoft.DocAsCode.ECMA2Yaml'.       D:\Windows\system32\config\systemprofile\.nuget\packages\: Package 'Microsoft.DocAsCode.ECMA2Yaml.1.0.669' is not found on source 'D:\Windows\system32\config\systemprofile\.nuget\packages\'.       https://www.myget.org/F/op/api/v2: Package 'Microsoft.DocAsCode.ECMA2Yaml.1.0.669' is not found on source 'https://www.myget.org/F/op/api/v2'.

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

Docs Build status updates of commit 89f8460:
:clock10: Pending: waiting for processors (12 builds ahead of you)
:warning: Docs Build is busy, currently there are 12 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 89f8460:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

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

@v-alje Would you be able to merge this so that this repo can be SDP-enabled? I believe we have a bug that is happening because it hasn't gotten the SDP updates.

@v-alje Would you be able to merge this so that this repo can be SDP-enabled? I believe we have a bug that is happening because it hasn't gotten the SDP updates.

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

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

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

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

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

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


added 'reversed' orientation

I've added 'reversed' style to your fab-menu only editing css styles and 'position' property

Thanks for contributing.


Sync eng/common directory with azure-sdk-tools for PR 7623

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/7623  See eng/common workflow

/check-enforcer override


[Bug] ACLs with exact ip addresses don't work

This works:
"acls": [{
	"action": "accept",
	"src": ["group:fastcup-ml"],
	"dst": ["<IP_ADDRESS>/20:5432"],
}]

But this doesn't:
"acls": [{
	"action": "accept",
	"src": ["group:fastcup-ml"],
	"dst": ["<IP_ADDRESS>:5432"],
}]

The users in group:fastcup-ml don't see the server which announces <IP_ADDRESS>/20 subnet in their tailscale status output.

Can you try with "dst": ["<IP_ADDRESS>/32:5432"], (adding /32 after the IP address), please?

Can you try with "dst": ["<IP_ADDRESS>/32:5432"], (adding /32 after the IP address), please?

Already tried. Doesn’t work.

Hi @maxpain, we have an issue template asking for a series of information and proposing debug information for us to be able to debug this quicker and making our life easier as maintainers. Please follow it and fill it out.
We need to see the output of certain things to figure out what is happening and it saves us a great deal of time if you help us since you have the setup already instead of us having to replicate it.
If you would have followed the template, it would have proposed to use tailscale debug netmap to help us with info we need.

Please go back and fill out all the information and provide a netmap dump from the relevant clients with the two different settings, the working one and the broken one. If not all the information is filled out, we will have to close the issue, thank you.

@kradalby, sorry about this. I've updated the issue.


Add option to respect current tab size

Editing a JSON with a 2 space width for tabs and your global settings are for 4? Well now the whol e file is in 4. There should be a separate command to either respect the already, or maybe override with the default.

Closed in 2021.1


MM-30377: Cypress/E2E: Automate backlogs - Messaging > Channel and Post Links (6 test cases)

Summary
Cypress/E2E: Automate backlogs - Messaging > Channel and Post Links (6 test cases)
Ticket Link
https://mattermost.atlassian.net/browse/MM-30377

/update-branch


Generating Swagger Specification of a WebAPI 2.2 service generates error about "Could not find value type of dictionary type 'System.Collections.Hashtable'"

I've attempted to use both the Cake add-in, the command line, and the NSwagStudio.
When I select my Web API Assembly, I get the error: Could not find value type of dictionary type 'System.Collections.Hashtable' I've scanned my Web API projects and cannot find a usuage of a Hashtable in the code.
After I get the error message "Object reference not set to an instance of an object" twice in NSwagStudio, but not the Cake add-in or the command line....

You can ignore the second exception as it occurs because the swagger spec could not be generated... The first exception occurred because you have a non-generic hashtable somewhere in the public API (i.e. in one of the web api controllers or a dto class). Can you locate it? Just replace it with Hashtable...

Use the NSwagStudio to find out which controller has the problem, the check if the controller classes exposes a "special" .NET Framework class (ie ActionResult). If we know the used class, we can implement a special treatment for it...

Btw: ActionResult is handled correctly, it must be something like that...

If possible, post a sample controller where this problem happens..

Added support for non-generic dictionary (e.g. Hashtable): https://github.com/NJsonSchema/NJsonSchema/commit/cce3f6998650cdc80ff2109382f93708f6638979
(generates Dictionary<string, object>)

Sorry it took so long to get back to you on this... I just downloaded the latest version of NSwagStudio and ran it against the code... The original exception is gone and the second has been replaced with the "No Specification" message. But, now I am getting a different exception... "The JSON property 'ItemOf' is defined multiple times on type 'System.Xml.XmlAttributeCollection'.
The class that is generating this...
public class QuestionController : ApiController
{
    private IRepository<QuestionTemplate, int> _templateRepository;

    public QuestionController(IRepository<QuestionTemplate, int> templateRepository)
    {
        _templateRepository = templateRepository;
    }

    public QuestionController()
    {
    }

    [ValidAccessTokenRequired]
    public HttpResponseMessage Get(int id)
    {
        using (var repository = _templateRepository ?? new QuestionRepository())
        {
            var entity = repository.FindById(id);

            if (entity == null)
            {
                return new HttpResponseMessage(HttpStatusCode.NotFound);
            }

            return new HttpResponseMessage
            {
                Content = new StringContent(entity.Template.OuterXml, Encoding.UTF8, "application/xml")
            };
        }
    }

    [SuperUserRequired]
    public IHttpActionResult Post([FromBody]XmlDocument xmlDocument)
    {
        var entity = new QuestionTemplate(xmlDocument.OuterXml);

        using (var repository = _templateRepository ?? new QuestionRepository())
        {
            repository.Save(entity);
        }

        var location = Url.Link(
            "DefaultApi",
            new
            {
                id = entity.Id.ToString()
            });

        return Created(location, entity);
    }

    [SuperUserRequired]
    public IHttpActionResult Post(string xmlContent)
    {
        var xml = new XmlDocument();

        try
        {
            xml.LoadXml(xmlContent);
        }
        catch (Exception)
        {
            return BadRequest();
        }

        return Post(xml);
    }
}

I can create a new issue if you want and we can work from there...

The problem is, that XmlDocument is not supported... what JSON do you expect here?

Unfortunately, the requirement of the web service is to return/accept XML to/from the client instead of JSON... so I'm not really sure what the appropriate output would be in this case...

Currently XML schemas are not supported... but for now I think we should treat XmlDocument as string...

Can you create a new issue for that?

@rsuter sure...


Respond to https://hypothes.is/a/TitMIKKBEeqiPo9geMbr3Q

Respond to hypothes.is annotation TitMIKKBEeqiPo9geMbr3Q on https://jrnold.github.io/r4ds-exercise-solutions/tibbles.html (user<EMAIL_ADDRESS>created: 2020-05-30T14:24:43.306101+00:00, updated: 2020-05-30T14:24:43.306101+00:00)

Exercise 10.5.2

Link: https://hyp.is/TitMIKKBEeqiPo9geMbr3Q/jrnold.github.io/r4ds-exercise-solutions/tibbles.html

No changes required. Replied with comment https://hyp.is/TitMIKKBEeqiPo9geMbr3Q/jrnold.github.io/r4ds-exercise-solutions/tibbles.html
Relates to #115


Document changes from #1099 of dapr/components-contrib

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Read the contribution guide
[x] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
The new GCP Pub/Sub configuration value "enableMessageOrdering" is documented alongside all existing configuration for this service. Additionally, the new requirements for roles assigned to a service account to guarantee ordering under certain circumstances have been captured.
Issue reference
This PR will close: #1742

@LLKennedy any reason why this is still a Draft?

@LLKennedy any reason why this is still a Draft?

No, I didn't intentionally create it as a draft and I'm not actually sure how to change it.


Support custom speeds for bike + ped

https://replicahq.atlassian.net/browse/RAD-6446
https://replicahq.atlassian.net/browse/RAD-6445
Adds custom speed support for bikes + pedestrians. Both cases follow a very similar model to custom speeds for cars, so supporting these new modes was relatively straightforward with the existing architecture.

Testing-wise, I added unit tests that match existing tests for other custom speeds scenarios. I decided to only test road closure situations with bike/ped, as it seems a lot more likely we'll hit those in production settings than speed changes, but I'm happy to add additional tests if it seems warranted.
Outside of unit tests, I locally built a router that included both the default foot + bike profiles, as well as foot + bike profiles where Baseline Road outside of Sac is closed (ie the road closure we test in unit tests). I was able to confirm that in the returned routes, only the default foot/bike profiles took the section of Baseline Road that was closed:

I've left comments on other areas that might need a bit of clarification

cc @rslassman @rregue

Thanks for the review here @bryanwilliams1025 ! Yes to both high-level questions, I plan to bring these changes over to the model repo/TN CDP as a fast-follow to this PR, and I'll make sure the docs get updated by EOD!


Remove "almighty" appearances from code.

As a followup we have to rename this repo
from https://github.com/almighty-test/almighty-test-unit
to https://github.com/fabric8-test/fabric8-test-unit
or something alike and adapt the places in tests accordingly.

[test]

Why not remove the word from comments as well?

https://github.com/fabric8-services/fabric8-wit/pull/1796/files#diff-acc8d3974d80a8260ecd28803a0b064aR159
https://github.com/fabric8-services/fabric8-wit/pull/1796/files#diff-acc8d3974d80a8260ecd28803a0b064aR171
https://github.com/fabric8-services/fabric8-wit/pull/1796/files#diff-acc8d3974d80a8260ecd28803a0b064aR194


[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

не зрозуміла відповідь ужгородської ода
28 вересня 2021, 12:09:04, від "Justin Kenyon" @.***>:
Closed #2548.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
--
Mapsler GID HAB


Add controller access component

Create branch from main
issue-90-add-controller-access-component


Add a migration to fix Blockbook endpoints.

I told @placer14 that the new BB endpoint were ready prematurely. We're still working on them and had to make them unavailable for a while. This migration reverts the endpoint change back to endpoints that are stable.
We'll continue to support the existing endpoints for upgrade window and after the new endpoints are fully ready we can migrate back to them. The transition should be seemless for existing nodes.

Coverage increased (+0.3%) to 33.907% when pulling 0c93b8dd583d06381c4ba84e41a0d9cc957ef1ef on TS_fix_wallet_endpoints into d476856905a847dac59238d3e467fb11fab05ed0 on master.


Google maps component - programmatic script/lang change

Feature Description
Right now you would have to add key, lang etc into index.html manually.
What it should allow, is to add a init service that can be used to inject the API with params (key, client, language, libraries).
It should also handle wiping all map scripts + loaded library to trigger language change.
Right now this has to be implemented manually, but it should be part of the package.
Use Case
Allows loading maps from configurations that may come from the server-side (key, libraries etc).
Allows changing language without page reload (transloco for example).

This is definitely something I'd like to implement. I will look into this.

This is how I currently deal with dynamic lang + script. Init is called from angular app initializer.
async init(): Promise<void> {
    // Trigger location prompt if user has not accepted yet
    await navigator.geolocation.getCurrentPosition( () => {});

    // Reinitialize on language change
    this.preferencesService.getLanguage().subscribe(lang => {
      this.setGoogleMapsApiScript(lang).then( () => {
        // Maps API callback
        window['mapInit'] = () => {
          this.mapInitialised$.next(true);
        };
      });
    });
  }

  private async setGoogleMapsApiScript(lang: string): Promise<any> {
    // Delete any existing maps api instance and script to reload language
    document.querySelectorAll('script[src^="https://maps.googleapis.com"]').forEach(v => {
      v.remove();
    });

    // Delete existing loaded map scripts
    if (typeof google !== 'undefined' && typeof google.maps !== 'undefined') {
      delete google.maps;
    }

    const appConf = this.configurationService.getConfig();
    // Init new maps api
    const script = document.createElement('script');
    script.id = 'googleMaps';
    script.async = true;
    script.defer = true;

    script.src =
      'https://maps.googleapis.com/maps/api/js?client=' +
      appConf.google_maps.client_id +
      '&key=' +
      appConf.google_maps.key +
      '&callback=mapInit' +
      '&libraries=' +
      appConf.google_maps.libraries.join() +
      '&language=' +
      lang;

    document.head.appendChild(script);
  }

This requires another step - I have to listen for the init and inject the google-maps component dynamically so I can refresh the view - can likely be handled within the maps component itself.

While this approach is good, it has it's pitfalls too. Google Maps API should loading as soon as it is needed, Thus if it is in the initial view, waiting for JS to parse before starting would be quite a slowdown. Also if someone decides to include it in the initial view, we should look if it is present to either warn xor prevent loading.

Yes, this is a relatively quick version of this to get it working.
APP_INITIALIZER should be called before bootstrapping the AppModule and since the script is async, it would load while the rest of the app is bootstrapped.
I also used the mapInitialised$ observable that can be listened to inject the map (had to use dynamic comp to reload the map on lang change - but the map refresh should be part of the component).
It's just unfortunate google maps still does not allow passing lang, key, clientid and libraries to the API after the first script is loaded.
But they still rely on the poor user experience of having to full reload or hacks like this to reload scripts.

@Ketec

Yes, this is a relatively quick version of this to get it working.
APP_INITIALIZER should be called before bootstrapping the AppModule

Yes, and no... It should only run, if (at least) one of google maps components is being used.
I think that there should be a service, that will have ReplaySubject with google map API. This observable will look if google.maps is defined, and if not check scripts, if it is loading, and if either, start downloading on its own. This way you can have google maps script inline in head if you want while being able to fall back.
Also, why not use dynamic import for the map?

Dynamic import to replace what, dynamically injected component?
Also checking the existing script only works for one-time load where parameters to the API do not change.
Like language or libraries - for those you need to delete the scripts, delete any loaded js for maps and reinject with new parameters.
They do not support loading/changing those through API.
And reloading the entire page on language change is a ancient,  anti-user experience way to do things.

Wow... That's irratating. Woundt be better to create iftame with API and get cross site reference, or something like that, it should work as long as it is on same domain

I did go for a bit simpler approach instead of dynamic component -
I am calling ngOnInit() again after I have reloaded gmap scripts to refresh this component.
It's a bit too much but the required parts (like event manager) is private.
If the component had a method to call to reinitialize, this would be better.
Ideally, this reinitialize step should also be handled inside the component.
But the question is how it should all be solved.

Script insertion should be dynamic.
It should support dynamic parameters - API key, clientId, libraries and language.
Language can change during runtime without page reload (ex transloco or ngx-translate) - the component/service needs a lang input for that.
To update lang, existing scripts and loaded maps API need to be deleted and recreated.
Option to use service from angular initializer or lazy when needed.

Why does it actually need to be deleted? Won't Google map override the previous language?

If you look at the maps API during load - it injects a number of new scripts into your HTML by itself.
If you don't delete them - you would get duplicates added, bloating and potentially causing conflicts.
And if you don't destroy the old instance it seems it won't properly clear it itself:
You have included the Google Maps JavaScript API multiple times on this page. This may cause unexpected errors.


MRG: Add search and generalization lights

We need to have simple transformer objects to apply search lights and what could be called 'generalization' light (or whatever name you find more useful) that are dedicated to efficient parallel processing on independent data subsets.
The aim is to be able to do:
X = epochs.get_data
y = epochs.events[:, 2]

clf = make_pipeline(SpatialFilter(), SearchLight(RandomForest())
clf.fit(X, y)
scores = clf.score(X, y)

plt.plot(epochs.times, scores)

The TimeDecoding and GeneralizationAcrossTime classes would be high level classes to deal with data extraction (provide epochs, automatically pick channels etc) and plotting.
The main motivation is that we can then

nest these objects in with sklearn.model_selection.cross_val_score and sklearn.pipeline
apply search light across freqs/ time freq / sources / sensors etc.

To-do

[X] base transformers
[x] documentation
[x] tests
[x] scoring: here we need to discuss, I'm not sure what is the best design

What won't be done in this PR:

refactor GAT and TimeGen
deprecate GAT and TimeGen advanced method
add Delayer()allow multiple time points per estimator

cc @agramfort @kaichogami @dengemann

add Delayer() allow multiple time points per estimator

Unintuitive name IMO.

Unintuitive name IMO.

Let's keep this for later

@agramfort @dengemann do you understand exactly how the scoring API works in sklearn?
I think we may want something that automatically adjust the prediction method when the scorer requires continuous versus discrete y_pred. e.g.

SearchLight(scoring='roc_auc').score(y, clf.decision_function(X)) when scoring is roc_auc_score
SearchLight(scoring='accuracy').(y, clf.predict(X)) when scoring is accuracy_score
avoid doing roc_auc_score(y, clf.predict(X))

In particular, I'm not sure whether this is to be solved at the base_estimator level, or at the SearchLight level. For example, cross_val_score has a scoring param.
The general idea is that I want to avoid dealing with this kind of param as much as possible and delegate this to sklearn objects.

I think we may want something that automatically adjust the prediction method when the scorer requires continuous versus discrete y_pred

Please implement this. Right after raw.to_submission(journal='nature_neuroscience').

Ready for review if Travis is happy.
What remains to be done (in other PR):

[ ] Refactor GAT for high level channel picking and plotting function
[ ] add scoring similar to cross_val_score to intelligently vary the type of prediction as a function of the scorer
[ ] optimize scoring (ATM: it duplicates the entire dataset in parallel because it does not internally call its parallel _transform. This is related to the above point)
[ ] Make new example with other types of search light

cc @jasmainak I hope you'll be happy with that, you seemed eager to get a clean API at some point ;)

[Current coverage][cc-pull] is 86.66%

Merging [#3381][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.01%

@@             master      #3381   diff @@
==========================================
  Files           335        337     +2
  Lines         57635      57852   +217
  Methods           0          0
  Messages          0          0
  Branches       8782       8819    +37
==========================================
+ Hits          49959      50137   +178
- Misses         5001       5028    +27
- Partials       2675       2687    +12

[][cc-pull]

Powered by Codecov. Last updated by [0623de9...57993c4][cc-compare]
[cc-base-branch]: https://codecov.io/gh/mne-tools/mne-python/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/mne-tools/mne-python/compare/0623de9142aa1aeabfde66f065618ef78e613ee5...57993c4c6c71b6a7cf3f8bdbd3486106372cd88c
[cc-pull]: https://codecov.io/gh/mne-tools/mne-python/pull/3381?src=pr

I've gone over it and I don't see any problems.

thx @jona-sassenhagen ; anyone else to review? @jasmainak ?

@dengemann can you give it a review and merge if applicable?

@agramfort @dengemann ready for review
@Eric89GXL the ci error is related to #3443. Shall I wait until merged and rebase?

Merged, yes please rebase

@dengemann ping ;)

@kingjr other than my few remarks on code comments it looks good.

I don't manage to parse the ci error. What's wrong exactly?

It seems to think the build couldn't be merged, can you try rebasing and push-forcing?

@dengemann ready for merge

Thanks JR!!


Getting the error "no required module provides package " on windows

my question is...
( Getting the error "no required module provides package " on windows )
My import system is as follows,
package main

import (
	"log"
	"os"

	"github.com/urfave/cli/v2"
)

Running go build main.go returns the following error.
main.go:7:2: no required module provides package github.com/urfave/cli/v2: working directory is not part of a module

I am new to GO, and I have tried setting GO111MODULE=on as most sources have suggested, both by using go env -w GO111MODULE=on and set GO111MODULE=on. I have even used $env:GO111MODULE="on" with no luck.

Comment by @rliebz is the proper way to go


Agile Rates Card(s) - Ability to view in dashboard upcoming rates

Feature request.
Would it be possible to have an entity that showed the upcoming rates which could then be used in a card on a dashboard.
What I ideally want is something similar to what https://github.com/markgdev/home-assistant_OctopusAgile has done.
The dev no longer maintains that project and only caters for Import not Export/outgoing.
I can see you provide both import and export rates in the Atributes of Current Rate but I am looking for a nice way to view these rates on a dashboard, ideally like the card provided in the other project that is no longer maintained as that has configureable colour coding when the rate is low, medium or high (Green, Yellow, Red). If the sensor and attributes were the same between the 2 I could just point that card at your rates entity but there is significant differences.
The other uses the format:
2022-11-09T22:30:00Z
17.16
2022-11-09T22:00:00Z
23.35
2022-11-09T21:30:00Z
23.15
2022-11-09T21:00:00Z
27.56
2022-11-09T20:30:00Z
23.71
2022-11-09T20:00:00Z
24.48
2022-11-09T19:30:00Z
20.71
2022-11-09T19:00:00Z
31.75
2022-11-09T18:30:00Z
35
While yours uses the attributes in a sub attribute called rates and in the format of:
Rates

from: '2022-11-08T00:00:00+00:00' to: '2022-11-08T00:30:00+00:00' rate: 8.1585 - from: '2022-11-08T00:30:00+00:00' to: '2022-11-08T01:00:00+00:00' rate: 11.025 - from: '2022-11-08T01:00:00+00:00' to: '2022-11-08T01:30:00+00:00' rate: 10.8885 - from: '2022-11-08T01:30:00+00:00' to: '2022-11-08T02:00:00+00:00' rate: 9.8805 - from: '2022-11-08T02:00:00+00:00' to: '2022-11-08T02:30:00+00:00' rate: 10.8255

So not trivial to convert (for me anyway).
This is the only thing I use the other HACS plugin for, the rate card as your plugin provides everything I need other than that, and your update and sensor values remain even after a reboot so using your values would be advantageous.

Screenshots for reference. Thank you for your epic work!

I would love thus too. Have the same situation where I just use the rate card from the old integration!

Oh and Blue for negative pricing such as tonight (not seen that before till today)

I'm not planning on changing the current setup because it would be a breaking change for existing users. If someone would like to port the card across (licencing/permission from author permitting) and make a pull request, I'm happy to take a look at the PR and include it.

I'd like this feature as well... The mentioned tariff integration does not work for me (I'm not an advanced user)

That feature would be amazing  - I have just moved to Octopus Agile and I am scratching my head how to get a graph or a table of upcoming/past rates

I've modified the Javascript from https://github.com/markgdev/home-assistant_OctopusAgile/tree/master/custom_cards to work with this integration and added a couple of new features (ability to hide rates before $now and show the day of week next to the time, since this integration retrieves a lot more rates than the other one)
I'm more than happy to share it, the question now is logistics...
@BottlecapDave are you familiar enough with HACS to say whether you can have a custom card and an integration in the same repo? From the reading of their docs it seems like no? In which case I'll publish my own HACS repo for this custom card using your integration as the data source.

Okay - I published this in my own repo because I think you can't join Integrations and Cards anyway.
https://github.com/lozzd/octopus-energy-rates-card
Please feel free to give it a shot and let me know how it goes.... feedback welcome (in that repo I guess)
@BottlecapDave let me know if you want me to PR the README in here to link to it, or want to keep them totally separate. Then you can close this :)

OMG! At last. Thanks a million Laurie!
On Wed, 1 Feb 2023, 15:33 Laurie Denness, @.***> wrote:

Okay - I published this in my own repo because I think you can't join
Integrations and Cards anyway.
https://github.com/lozzd/octopus-energy-rates-card
Please feel free to give it a shot and let me know how it goes....
feedback welcome (in that repo I guess)
@BottlecapDave https://github.com/BottlecapDave let me know if you want
me to PR the README in here to link to it, or want to keep them totally
separate. Then you can close this :)
—
Reply to this email directly, view it on GitHub
https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/88#issuecomment-1412259577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4ZZMQRUS2ZT247H7IL4SI3WVJ65VANCNFSM6AAAAAAR2RR324
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
.com>

I second this.  This is brilliant.
On 1 Feb 2023, at 21:29, slip83 @.***> wrote:

OMG! At last. Thanks a million Laurie!
On Wed, 1 Feb 2023, 15:33 Laurie Denness, @.***> wrote:

Okay - I published this in my own repo because I think you can't join
Integrations and Cards anyway.
https://github.com/lozzd/octopus-energy-rates-card
Please feel free to give it a shot and let me know how it goes....
feedback welcome (in that repo I guess)
@BottlecapDave https://github.com/BottlecapDave let me know if you want
me to PR the README in here to link to it, or want to keep them totally
separate. Then you can close this :)
—
Reply to this email directly, view it on GitHub
https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/88#issuecomment-1412259577,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4ZZMQRUS2ZT247H7IL4SI3WVJ65VANCNFSM6AAAAAAR2RR324
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
.com>

—
Reply to this email directly, view it on GitHubhttps://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/88#issuecomment-1412753377, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM5C5J4JYDJO5SQCGMQFWE3WVLIRVANCNFSM6AAAAAAR2RR324.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@lozzd Thanks for taking this upon yourself! I'll update the readme soon with a link to your implementation :)

I've added this to the readme, so closing now.