tests+dialect for MariaDB

Testcontainers has a module for MariaDB.
MariaDB is for the most part just a flavor of MySQL, and we support it via the Vert.x driver for MySQL. But I think that's not quite good enough. There are some (very) minor differences in the Hibernate Dialect, and so we should treat it as a separate case.
So we should set up tests to run on Maria, and make sure we detect it and use the correct dialect by default, instead of making the user specify it explicitly.

@blafond this might be a good issue for you

+1 for the MariaDB specific testing. But I don't understand why you mention the Dialect differences, won't we use the one from core ORM?

+1 for the MariaDB specific testing. But I don't understand why you mention the Dialect differences, won't we use the one from core ORM?

yes we will, but we need to wire it into our stuff in HR.

yes we will, but we need to wire it into our stuff in HR.

I’m going to work on this one if no one else is.

So I have this basically done.
Except: Maria doesn't like how we drop schemas. And that's causing lots of errors in the test suite.

Perhaps we just need to up the wait time.

Also it doesn't like the local temporary table syntax.

Done, and @DavideD has promised to add Maria to the CI build.


Cdk Accordion example

proposal:
I can see that accordion is in cdk folder but there is no example or doc on website. Could anyone provide one?
BTW. Cdk is great tool but needs more and more examples :)

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate


[Bug report]: Fiat rates from the API are wrong

Platform
Desktop
Version
1.6.5
Description
The calculation of each fiat value is correctly implemented in Firefly. However there is a big difference for some fiat values compared to exchanges.
I have checked the IOTA market API and there seems to be a big difference in the fiat rates. In the example below all rates received from the IOTA market API are listed. The fiat rate EUR for example is at 0.8528784648 to the base USD but it should be more like 0.97 at the time of the request.
"rates":{"CAD":1.2607249467,"HKD":7.7742430704,"ISK":126.3113006397,"PHP":48.5296375267,"DKK":6.3431130064,"HUF":309.8251599147,"CZK":22.2968017058,"GBP":0.7267292111,"RON":4.1981236674,"SEK":8.7320255864,"IDR":14524.9978678038,"INR":73.1880597015,"BRL":5.7491684435,"RUB":75.3240938166,"HRK":6.4567164179,"JPY":110.7974413646,"THB":31.2648187633,"CHF":0.9441364606,"EUR":0.8528784648,"MYR":4.1465245203,"BGN":1.6680597015,"TRY":8.2942430704,"CNY":6.551130064,"NOK":8.5249466951,"NZD":1.4301918977,"ZAR":14.7960767591,"USD":1,"MXN":20.512238806,"SGD":1.3448187633,"AUD":1.31445629,"ILS":3.3414072495,"KRW":1129.3731343284,"PLN":3.9665671642}}
Expected behaviour
Fiat rates should be correctly fetched.
Actual behaviour
For some fiat rates there is a big difference.
Can the issue reliably be reproduced?
No
Steps to reproduce the issue
No response
Operating System
Windows
Diagnostics
No response
Developer Profile
No
Profile Type
Software (Stronghold)
Staking
No response
Crash Status
None of the above
Error Messages
No response
Error Logs
No response
Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

[X] I agree to follow this project's Code of Conduct

new version will be released soon with improved fiat


Offcanvas backdrop in navbar remains visible when displayed on larger screens

On certain breakpoint, when offcanvas is triggered on iPad in portrait mode and then the device is rotated, offcanvas disappears as expected. But backdrop remains visible. Same happens when browser is resized - when offcanvas gets hidden, backdrop then is visible on all larger breakpoints.
Suggested fix is to add
.offcanvas-backdrop {
    display: none !important;
}

to .navbar-expand responsive classes. Not sure though if this solution is correct, because backdrop element still exists in the DOM.

Hi Kopyov,
this is a known issue. Offcanvas navbar differs from default offcanvas because it‘s placed inside the navbar component and uses navbar-expand-* classes to show or hide offcanvas.
But an open offcanvas do more than open offcanvas and showing a backdrop, it locks body scroll as well.
In case of resize through the breakpoints, backdrop is still visible and scrolling locked.
That must be fixed in JavaScript. In can be done by general hiding offcanvas on resize if component is in navbar or hiding it if offcanvas touches a breakpoint on resize or maybe via a data-attribute.
There will be a solution for this
in furter updates.

Closing as duplicate of #35594. This should be fixed with the JS changes coming in #35736.


[PE-D][Tester E] Duplicate Weddings can be made.

Duplicate weddings can be made by adding whitespaces between each word although each wedding is essentially the same.
Steps to recreate:

Run "create-wedding w/Test Wedding"
Run "create-wedding w/Test   Wedding"

Labels: type.FunctionalityBug severity.Low
original: kohkakohla/ped#8

invalid

I think its invalid also.

Declare limitations of duplicate detection in UG commands that look for duplicates + state as planned enhancement


TypeError: Cannot read property 'getContent' of null

Hello,
I integrated this and everything seems to be working fine except that on the edit page where I get all the data, I'm getting the following console error.
TypeError: Cannot read property 'getContent' of null

Any suggestion to remove this error?

Hello, Thanks for issues  ♥
It seems to be a bug!
Please put your code here for check the problem.

I'm getting this error too.  What exactly is the work-around?

@alphatrust-dev Hello, can you put your code here? And please tell me which version of TinyMce and Vue .js is used.
Or send me an email<EMAIL_ADDRESS>

@alphatrust-dev Hello, can you put your code here? And please tell me which version of TinyMce and Vue.js is used?
Or send me an email<EMAIL_ADDRESS>


Native Notebooks + Custom Editor Error?

Got the following customer error from survey.
"CustomEditor and NativeNotebook experiments cannot be opened together" - this is the error i get everytime i try to open an .ipynb file.  I undestand bugs happen, but when they do, a detailed explanation of what went wrong should be available (most of us vscode users are devs ourselves, when something breaks, we like to know why, so we might address the issue)."
Can someone clarify what logic would trigger this error to a user? Would like to update the error to be more clear + potentially provide location to resolution instructions.

We believe we have resolved this issue in the recent point release.
Hence closing this issue.
If this comes up again we can re-visit it.


Readonly for entities

Fix https://github.com/jhipster/jhipster-vuejs/issues/472

Please make sure the below checklist is followed for Pull Requests.

[ ] Travis tests are green

[ ] Tests are added where necessary

[ ] Documentation is added/updated where necessary

[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

cc @murdos : if you can review it plz. Otherwise, I'm going to merge this, so we can fix step by step the failures on Vue.js blueprint

Thanks for the review @murdos
I will do another PR to improve it


Not working

@ITADORI-15 I will fix it soon ..

Will u make it an anilist tracker?

@ITADORI-15 check latest release bro.

Thanx brooo

Bro sorry but it still isnt working

@kaungsatthe1n

Strange .. Plesse test it with Vpn ..

Ok

does it work?
it's working fine on my device ..

It works with vpn
But it is hard 2 connect 2 vpn every now and then
But yeah it works
Thnx

Also TakoPlay needs to connect with VPN in some country.. Because (Gogo Server Sucks) haha....
I also released new update last day .. Have u enjoyed with this new update ?

Yesss absolutely
This is one of the best app to watch anime
Not kidding

Ig ur from Myanmar
I'm from India ..there's no prob with gogo here as of now

Glad to hear bro .. thanks..


[release-4.12] dashboard: use recording rules for most metrics

Add more recording rules to reduce the load on Thanos querier and Prometheus.
This removes "auto" interval as it can't be cached via recording rules.
Cherrypick of https://github.com/openshift/cluster-kube-apiserver-operator/pull/1611 on release-4.12

/jira cherrypick OCPBUGS-25922

/jira refresh

/lgtm

/label cherry-pick-approved


Cannot verify OTP in a separate request

Hey Guys,
I am generating an OTP in one API call and verifying it in a separate call, but this doesn't seem to work properly. I have tried this by passing correct as well as wrong values but result is same. Maybe I am missing something or this shouldn't be used this way.
# 1st API call
otp = ROTP::TOTP.new('base32secret3232', issuer: 'MyApp', interval: 180)

Now we will use this OTP in a separate call to verify it.
# 2nd API call
otp = ROTP::TOTP.new('base32secret3232')
verified_otp_at = otp.verify(params[:otp]) # this produces nil

I think the issue is the interval isn't set in the 2nd API call. So in this case the window/interval is 30s vs 180s and you'll end up with different OTP's depending on the time you verify it at. Let me know if that fixes it for you.

@mdp Adding interval in the second call resolved this. Thanks.


Add some links to PeakGroup and PeakData results pages

FEATURE DESCRIPTION
Feature Inspiration
Linking across various objects in the database makes browsing data/results intuitive.
Feature Description
PeakGroup

Tissue => Tissue
Enrichment Fraction, Enrichment Abundance, Normalized Labeling => PeakGroup
Peak Group Set Filename => PeakGroupSet

PeakData

Tissue => Tissue
Raw Abundance, Corrected Abundance, Fraction, Median M/Z, Median RT => PeakData

Alternatives Considered
Comment

ISSUE OWNER SECTION
Assumptions

List of assumptions made WRT the code
E.g. We will assume input is correct (explaining why there is no validation)

Requirements

List of conditions to be met for the feature
E.g. Every column/row must display a value, i.e. cannot be empty

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected/Changed Components

Files
Environment variables
External executables
Database tables
Cron job settings
Etc.

DESIGN
GUI Change description
Describe changes the user will see.
Code Change Description (Pseudocode optional)
Describe code changes planned for the feature.
Tests
A test should be planned for each requirement (above), where possible.

Good thought.  Should we take that further?  Maybe clicking anywhere in a row (other than linked values) should go to that record's details page?

I'm fine with whatever implementation seems to make sense. That sounds reasonable.

Assuming some of these are talking about links from advanced search to detail and/or list pages.  Creating a more specific list WRT the direction of these links...
I will assume that in these cases, the adding of these types of links are not implied:

Advanced search results (any format) -> list pages using measured (or other non-unique field) values (e.g. raw abundance) is not implemented
Advanced search results (any format) -> details pages using measured values (e.g. raw abundance) is not specific enough

In the case of the advanced search peak groups format, a peak group column does exist and already links to the peak group detail pages.
In the case of peak data, there is no column to link to a peak data detail page - however there is currently no template for a peak data record anyway.

Linking of values (e.g. a tissue) on details pages to advanced search pages is not implied because the value exists in multiple formats (e.g. tissue is displayed in both peak group and peak data advanced search formats), though perhaps separate links rendered under the details or in new list columns could be added...

The following appear to already exist (if checked) or are not possible (if so indicated):

[x] Links from the tissue values in the Advanced Search Peak Group results to tissue detail pages
X Links from PeakGroup Detail pages to Advanced Search Peak Group pages for Enrichment Fraction, Enrichment Abundance, Normalized Labeling => PeakGroup

These are not possible the way it seems to be implied in the feature description of this issue (though I could be interpreting it wrong). The way the advanced search is implemented, you cannot search using cached properties as search terms.  That also applies to search_basic links to advanced search.

[ ] Advanced Search Peak Group results' Peak Group Set Filename => PeakGroupSet Detail pages
[x] Advanced Search Peak Data results' Peak Group Set Filename => PeakGroupSet Detail pages
[ ] Links from tissue values in Advanced Search Peak Data results to tissue detail pages
[ ] Peak Data details' and list values for Raw Abundance, Corrected Abundance, Fraction, Median M/Z, Median RT => PeakData Advanced Search

As above, a link using cached properties, such as fraction, to advanced search is not currently possible

I still believe there is value in being able to see the individual peak data rows for a given peak group. My thought had been, when viewing a PeakGroup result table, one could get to the PeakGroup detail view (to me, the total abundance makes sense to click on, or perhaps some added row) and that the PeakGroup detail view would show the PeakData rows (it has a link now, but I think should just show the rows (#151). Other ideas on how to allow easy browsing to the PeakData for a given PeakGroup from the PeakGroup list?


[Sept Sprint] New exercise teaching alternative number types

This is part of our September Spring to get the Go Track in shape. Please read the Overview issue for more details. This exercise is unclaimed. To claim it, please comment below.

Before building this exercise (or exercises), please read up on the documentation.
Cars Assemble teaches int and float64. We need an extra exercise to teach the other number types that are commonly used and returned in Go. This can go into greater detail on more advanced numerical concepts.
It is also entirely possible to create two exercises: one on integral numbers, and one on floating-point numbers. This is the approach taken by C#, see the https://github.com/exercism/csharp/tree/main/exercises/concept/interest-is-interesting and https://github.com/exercism/csharp/tree/main/exercises/concept/hyper-optimized-telemetry exercises.

@junedev Any thoughts on what exactly this exercise should teach? I know we should be teaching about float32 and the various integer types other than int, but you mentioned advanced numerical concepts IIRC?
Also, do you have a suggestion for an existing exercise to port? I could use one of the C# exercises, but you might have a better idea.

In JavaScript there is only "number" so I never looked for number type related exercises. I had a quick look now but most tracks are not that far into the syllabus yet it seems.
The integer exercise could work for Go as well but I would have to try it to be sure. That would cover all the integer related types.
Then we could expand the "interest" exercise a little to cover the two floats and introduce the math package https://pkg.go.dev/math.
The special topic I mentioned before was about numeric constants in Go. They can be defined without a type and will take on the type they need to have when used. This can be part of the interest exercise or can be added to the constants exercise.

The interest-is-interesting exercise in C# uses decimal, which is a type specifically designed to be used with money (which the exercise uses) as it offers very high precision. I googled a bit and found that Go has arbitrary-precision numbers too: https://pkg.go.dev/math/big Do you think we would need to introduce that or could we use float64 for the balance?

I would keep big and complex for separate exercises later in the track.


Rank in ci

~closes~ for https://github.com/wellcomecollection/platform/issues/5350

$ jest --queryEnv=candidate --index=works-indexed-2021-11-29a --testId=alternative-spellings --testId=precision --testId=recall --testId=false-positives
--
  | PASS __tests__/eval.test.ts (5.76 s)
  | ✓ works-indexed-2021-11-29a candidate alternative-spellings (170 ms)
  | ✓ works-indexed-2021-11-29a candidate precision (26 ms)
  | ✓ works-indexed-2021-11-29a candidate recall (15 ms)
  | ✓ works-indexed-2021-11-29a candidate false-positives (24 ms)
  |
  | Test Suites: 1 passed, 1 total
  | Tests:       4 passed, 4 total
  | Snapshots:   0 total
  | Time:        5.862 s
  | Ran all test suites.
  | Done in 6.66s.
  |

Should we be running the image rank tests as well?

Should we be running the image rank tests as well?

We should! I've just created an issue for it:

https://github.com/wellcomecollection/platform/issues/5383


Add a beautiful message when our rocket reaches the moon

Add a beautiful message when our rocket, Spacecraft, arrives on the moon.
You can use some ASCII caracteres as we used in #32 and #65

Hi! Can I work on this?

Yeah, for sure! It's yours.
You could also add a check to continue or finish the travel when the rocket arrives the moon. What do you think?

Thank you! I'm a new developer, but I will definitely attempt to add the check as well.

@mccarthykp Awesome! Go ahead. If you have any doubts, feel free to ask here :smile:


Failing to compile when importing reactflow

Describe the Bug
Getting the following error when importing reactflow into existing project. The existing project is using craco.
`Failed to compile.
<EMAIL_ADDRESS>71:17
Module parse failed: Unexpected token (71:17)
File was processed with these loaders:

./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|     position = 'bottom-right'
|   } = _ref2;

if (proOptions?.hideAttribution) {
|     return null;
|   }
`

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

npm install reactflow
Copied template from quickstart into component
Imported component into existing project

Expected behavior
Successful compile
Screenshots or Videos

Platform
Platforms: Chrome
Additional context
Running on craco

duplicate of https://github.com/wbkd/react-flow/issues/2453


New doc theme and docs fixes

Closes #563
@mnojek please check it visually - I can modify pretty much everything through custom css (spacing, padding, colours etc). I switched from rtd theme to Sphinx default alabaster theme. You can build documentation locally with:
tox -e docs

Documentation will be under docs/_build/index.html

Codecov Report

Merging #567 (ff5235c) into master (6c98345) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #567   +/-   ##
=======================================
  Coverage   96.96%   96.96%
=======================================
  Files          23       23
  Lines        2604     2605    +1
=======================================
+ Hits         2525     2526    +1
  Misses         79       79

Impacted Files
Coverage Δ

robocop/checkers/duplications.py
98.54% <ø> (ø)

robocop/checkers/misc.py
97.57% <ø> (ø)

robocop/checkers/naming.py
99.17% <ø> (ø)

robocop/checkers/spacing.py
98.45% <ø> (ø)

robocop/checkers/tags.py
100.00% <ø> (ø)

robocop/rules.py
100.00% <100.00%> (ø)

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 6c98345...ff5235c. Read the comment docs.

In my opinion, this theme looks better than the previous one and is more readable. But I can see an issue on small screens (actually, my laptop has only 14") where the left menu is not properly displayed and even cannot be scrolled down. Here is the screenshot:

I needed to resize the page in browser to 80% to fully see the menu:

I wonder if other themes also have this problem.

@mnojek
It's actually configurable - in conf.py remove "fixed_sidebar": True, and see the difference. On mobile it's always False. The problem that without that option the menu stays at the top of the page so with long page (like rules) you need to scroll way to the top. That of course could be workarounded with some custom css

This theme looks nice, and it have scrollable sidebar: https://setuptools.pypa.io/en/latest/userguide/package_discovery.html

Ok, with "fixed_sidebar": True it looks better but, yeah, now I need to scroll all the way up to see it. Nevertheless, I prefer this one, because it's the only one that makes it available for me :)
All in all, this theme looks better and I think we can use it.

This theme looks nice, and it have scrollable sidebar: https://setuptools.pypa.io/en/latest/userguide/package_discovery.html

Didn't see this comment. Is it also available in sphinx?

This theme looks nice, and it have scrollable sidebar: https://setuptools.pypa.io/en/latest/userguide/package_discovery.html

Didn't see this comment. Is it also available in sphinx? Looks way more user-friendly with those dropdowns in menu and scrollable sidebar!

It's def build on sphinx, looks like https://github.com/pradyunsg/furo though I'm not sure if it's modified or used pure. I can try it out later (though robotidy uses alabaster so I need to think if I want to all our tools to use the same theme for consistent)

@mnojek
Updated theme with furo theme. I also make some changes, for example I've added simple dropdowns where we had bit too much text:

Please check it out

Please check it out

This is just great! I love it!!! ❤

Now I see that there is no element in the menu that navigates to index.html (it's possible to click on the logo, but it's kind of hidden from the users).
Also, there is no visible link to GitHub repo and the badge with amount of stars.

@mnojek
How bad is Github stars not centered?

I'm still fighthing with it - for now it's left aligned and it looks a bit..off. But it's very resistant to any attempt of moving it to center ;)


Feat/react aria components

Reimplements a few UIKit components with react-aria-components
Commits should be quite clear for review, and changes easily testable in storybook

Now includes label as a prop that will render it on the right side of the checkbox
Optionally, labelledby can be passed if only a checkbox without visible label is required
Adds indeterminate as a state, useful for example for a "select all" checkbox on a table with only some rows checked

Styling changed a bit from floating label to a static one above the input box

Moving label is difficult from a11y perspective, as we would need to disable and move it to final position if user has prefers-reduced-motion
Placeholders don't work that cleanly with floating label
Me and Otto like the looks of this one better :stuck_out_tongue_winking_eye:

Error state styled a bit differently

error instead of warn color
Includes a warning triangle so that error message can be differentiated from an input description

Description and error can be shown simultaneously

Reuses stylings from TextInput
Popover list and options can be styled

Distinct styling for currently selected option and focused/hovered option
Can be bikeshed indefinitely :stuck_out_tongue:

Rename prop options -> items to mimic defaults of react-aria(-components)

Newly implemented
Shared styles with Select

Now it should be good by me, do you want a final check or just :ship: ?

LGTM 👍 Let's 🚢


Init SHiP receiver

Integrates the blocks from the SHIP plugin and add it to the microchain!
To enable SHIP in a local chain (as opposed of using DFuse), set the following env vars:
SUBCHAIN_RECEIVER = "SHIP"
SHIP_ADDRESS = "<IP_ADDRESS>"
SHIP_PORT = "8080"

You must have a nodeos running with the eden contract deployed under account eden.gm. Instructions can be found here: https://github.com/eoscommunity/Eden/pull/587

The initial ship request still isn't correct. This use case (transient test chains) won't trip a problem case, so we can leave it as-is for now.
If we ever need to move ship support into production:

The wasm will need to filter out unneeded blocks
The initial block must be either:

One block past last irreversible (does a partial replay), or
Some point after that block, but with have_positions filled in for each eosio block after irreversible

One of:

Verify we don't hit the web socket overrun case (this problem is unique to js websocket libraries), or
Switch from the ws library to one which implements a stream interface to avoid overrun, or
Change max_messages_in_flight to a small number and implement ack

We don't have to deal with overrun now because the transient eosio chain is small.


Mark DockHandler / all dockwidgets as deprecated

See subject.

We've long since removed the actual API but since UI APIs don't support the usual deprecation marking, we're simply going to point people toward:
https://github.com/Vector35/binaryninja-api/blob/dev/python/examples/hellopane.py
https://github.com/Vector35/binaryninja-api/blob/dev/python/examples/hellosidebar.py


Update community files (#3327)

Signed-off-by: Knative Automation<EMAIL_ADDRESS>

OCF Webhook is merging this PR


Dependencies gradle file with full dependency definitions

In order to manage and update dependencies easier, as a developer, I want a .gradle file where the full dependency definitions (org, group, artifact, version) are set to variables which can be used in module build.gradle files.
So instead of this in a build.gradle file:
dependencies {
    compile 'com.android.support:support-annotations:' + SUPPORT_LIBRARY_VERSION
    compile 'org.dmfs:iterators:' + JEMS_VERSION
    compile 'org.dmfs:optional:' + JEMS_VERSION
    compile 'org.dmfs:essentials:' + JEMS_VERSION
    compile project(':contentpal-api')

    testCompile 'junit:junit:' + JUNIT_VERSION
    testCompile 'org.mockito:mockito-core:' + MOCKITO_VERSION
    testCompile 'org.robolectric:robolectric:' + ROBOLECTRIC_VERSION
    testCompile 'org.dmfs:optional-testing:' + JEMS_VERSION
    testCompile project(':contentpal-testing')
}

we could write this:
dependencies {
    compile deps.support_annotations
    compile deps.dmfs_iterators
    compile deps.dmfs_optional
    compile deps.dmfs_essentials
    compile project(':contentpal-api')

    testCompile deps.junit
    testCompile deps.mockito
    testCompile deps.roboelectric
    testCompile deps.dmfs_optional_testing
    testCompile project(':contentpal-testing')
}

This could help in two ways:

easy to add new dependency which is already used in the project with code completion, no need to copy-paste from other file
easy replacement of artifact name if needed
One use-case is switching between maven and jitpack when there is parallel development, it can come useful often to use the latest version with jitpack. Even if we didn't want that in master and wait for a release, it is useful during development.
Something like this could be part of the gradle file:

def jemsGroup = `org.dmfs`
...
...iterators = jemsGroup + 'iterators' + jemsVersion
..

and then we just have to update jemsGroup to 'com.github.dmfs.jems' to switch to jitpack and it would update alll module's all jems dependencies.
The same thing could be done in jems project as well.
What do you think?

Just saw that using gradle file to hold the common properties is a suggested way in official docs as well:
https://developer.android.com/studio/build/index.html (Configure project-wide properties)

ok let's give this a try.

I've just read this post today:
http://alexfu.github.io/android/2017/11/07/experimenting-with-gradle-dependencies.html
Amazing that he mentions the same exact naming (deps.x)..  but the interesting is that he moves it one step forward, although I think it's better not to do that and keep the individual dependencies.
I'd like to suggest that if the ext.deps version disables the IDE hint about newer available versions, then we add the gradle plugin that checks them, and possibly add to the default build, so at least we see them from time to time:
https://github.com/ben-manes/gradle-versions-plugin
What do you think?

First let's see if we can get the hints to work without an additional plugin.

Ok


Convert README.rst to README.md

@mikhail-akimov Currently we use rst as the legacy format that was supported via old PyPI.
But, new PyPI release also supports md. Just like poetry.
So, what we need to do is manually convert README.rst into README.md.
Could you please help me with that?

Closed via https://github.com/sobolevn/jinja2-git/pull/7


Added AzureAD to list of identity providers

Used this as an excuse to get rid of FB from the identity providers list as well, per #140


fix(search): print repo search result in original case

Signed-off-by: Höhl, Lukas<EMAIL_ADDRESS>
closes #10525
What this PR does / why we need it:
Print the name of the chart in the same case like stored inside the index.
Before:
$ helm cm-push MyChart local
$ helm search repo MyChart
NAME         	CHART VERSION	APP VERSION	DESCRIPTION
local/mychart	0.1.0        	1.16.0     	A Helm chart for Kubernetes

After:
$ helm cm-push MyChart local
$ helm search repo MyChart
NAME         	CHART VERSION	APP VERSION	DESCRIPTION
local/MyChart	0.1.0        	1.16.0     	A Helm chart for Kubernetes

If applicable:

[ ] this PR contains documentation
[x] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

any ETA for this fix to be merged?


AtKey terminology confusion

Discuss the reasons and options for renaming AtKey to prevent confusion with encryption keys.

The current terminology that I'm using is atRecord = { atID : atValue }


Retina 2x tiles from Mapbox displayed incorrectly

Is this a bug report?
Yes
Have you read the Installation Instructions?
Yes
Environment
react-native: 0.56.1
react: 16.4.1
react-native-maps: "react-community/react-native-maps"
iOS:  Target iOS 11.2 / iPhone X

Steps to Reproduce

Use UrlTile with Mapbox with 2x retina tiles ala:

<UrlTile urlTemplate={'https://api.mapbox.com/styles/v1/mapbox/outdoors-v9/tiles/512/{z}/{x}/{y}@2x?access_token=pk...'} tileSize={512} />
Expected Behavior
Map uses mapbox tiles with retina resolution.
Actual Behavior
Tiles are jumbled, like only the first 256x256 of the tile is being used and not the full 512x512.

Reproducible Demo
Use the tileOverlay sample  substituting the UrlTile above.

Looks like this PR should fix it. https://github.com/react-community/react-native-maps/pull/2589
Anyway, if you use Mapbox tiles, is not easier to you use their RN implementation ? https://github.com/mapbox/react-native-mapbox-gl

Mapbox stopped maintaining their RN implementation, questions like this one will matter even more now.


Voice messages

User request:
Is it possible to post voice messages?

There is no direct voice message button in the current interface, although on (android) mobile clicking the document attachment (file upload) icon will present a menu with voice recording option. Selecting that, recording and sending by clicking the attachment (paperclip) button posts a voice message to the chat (the voice file will be stored on the room server). Confirmed that on desktop you can open the file and play the recording, though downloading so far only worked by selecting "open in new tab" in the context menu on the file link.

It would be nice if voice messages could be wrapped in html5 audio tags, so you can listen inline.

Added an audio tag for amr and mp3 file attachments: https://github.com/robvandenbogaard/Scylla/commit/46828f12427dd72471df316bf42a61eea3879e55


Setting Log Level

I've been trying to do some grepping, but i've never used logrus before. I've been trying to find SetLevel
Does setting log level exist right now? Can I add it? Preferences on how to set it? cli? env? config file? all 3? etc?

Currently it's not possible to change it because the project is still relatively early days. The logging helps diagnose any issues that come up and are a requirement for getting any amount of support on the thing.
In future it'll be possible to change the log level, but apparently I didn't have a ticket for that yet. Will use this one to track it.

Okay!
I don't mind doing the work, lemme know.

This has unfortunately hampered our ability to debug #310, particularly why the DB isn't working. Would appreciate this being a thing now it's in use in prod.

repo.logLevel now exists - see sample config


Disable splash affect and control animation duration

Would be great to have an option where the button splash affect can be disabled. Another good feature would be to control the duration of the select animation on the button. Is there a workaround?

Hello @MeshariA !
Now i working to implement WidgetBuilder buttonBuilder.
This will help you to create custom buttons inside the widget

In another case, you can create a pull-request to add these fields to GroupButtonOptions


Select2 4.0.7 with Jquery 3.4.1 multi select option only allows to write 2 characters

The select2 version 4.0.7 that's used has a known bug where multi select option only allows to write 2 characters when used with
jquery 3.4.1.
issue reference:
https://stackoverflow.com/questions/56398689/select2-with-multi-select-option-only-allows-me-to-write-2-characters
fix in that link is to bump select2 version to 4.0.8 or higher.

This is real problem. Please update deps.

@pySilver I'm focusing on https://oss.yourlabs.me/autocomplete-light/
May I ask that you open a merge request with the deps upgrade ?

@jpic in package.json the select2 version is v4.0.13, but when installing the package version 3.5.1 select2 v4.0.7 is installed

Perfect, let me release DAL 3.7.0


Improve the reliability of TestGetPinnipedCategory.

This test could flake in some rare scenarios. This change adds a bunch of retries, improves the debugging output if the tests fail, and puts all of the subtests in parallel which saves ~10s on my local machine.
Release note:
This is a test-only change
NONE

Codecov Report

Merging #501 (30a392b) into main (4ab3c64) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #501   +/-   ##
=======================================
  Coverage   78.49%   78.49%
=======================================
  Files         110      110
  Lines        5393     5393
=======================================
  Hits         4233     4233
  Misses        997      997
  Partials      163      163

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 4ab3c64...30a392b. Read the comment docs.


4.x: demonstrate proper use of GlobalConfig in examples and archetypes

This issue depends on issue #7627 to be resolved first. Once it is we need to make sure that documentation, examples and archetypes illustrate the preferred pattern for setting up config. For example:
// This loads default config sources including `application.yaml`.  If you want to create custom config sources you can
// do that here instead of just using the default config.
Config config = Config.create();
GlobalConfig.config(config);

With GlobalConfig we no longer need to pass config to the routing method found in many examples, nor to the HttpService found in many examples. Instead any customer code that needs config can do something like:
Config config = GlobalConfig.config().get("myapplication");

Close thanks to #7655 and #7664, thanks @barchetta


Definition of the KEAttribute class and some methods of this class

Definition of certain classes of the meta-model 2 (MM2)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.008%) to 40.236%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Kendrick-Core/K2Attribute.class.st
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
5835

Relevant Lines:
14502

💛  - Coveralls


Error rendering GeoJSONLayer in IE11

Hello and thank you for this package, it's been very helpful in our project. I was wondering if there are known issues with rendering layers for IE11. I am able to render the map fine, but am seeing an "Access Denied" error when I try to add a GeoJSONLayer component. Has anyone found a workaround for this or understand the root of the issue?
Thanks for your help!!
Error is on Internet Explorer 11, Windows 10
"react": "^16.12.0",
"react-dom": "^16.12.0",
"mapbox-gl": "^1.7.0",
"react-mapbox-gl": "^4.8.0"

we're using GeoJSONLayer in production and it works well on IE11, please share a reproducible broken-on-IE11 example

@mklopets Thanks for the info! I was able to create an example using GeoJsonLayer that works in IE11, so this will help me debug the actual issue in my project. Once I figure out the part of the code that's causing the issue in ie11 I'll post back here.

For anyone who might stumble across this and is having a similar issue, the problem was not the GeoJSONLayer.  We were attempting to load our own images (see code example) instead of using the Layer component's prop for images. I'm not exactly sure why this was causing an issue only in IE11, but removing the loadImage function and using <Layer images={images}>instead  fixed the issue for me.
map.loadImage(imageURL, (err, image) => {
    if (err) {
      onLoad(err, imageId)
      return null
    }
    if (map.hasImage(imageId)) {
      return null
    }
    map.addImage(imageId, image, options)
    onLoad(null, imageId)
  })

instead used this:
  const image = new Image(24, 24)
  image.src = imageSoureURL
  const images = [PIN_IMAGE_ID, image]

<Layer
        id={layerId}
        sourceId={sourceId}
        type='symbol'
        layout={{
          'icon-allow-overlap': true,
          'icon-image': PIN_IMAGE_ID,
          'icon-size': 1.0
        }}
        images={images}
      />


Add a loading state

Hi all, I think it could be nice to add a loading state to the switch.
For example, in my projet, I am using the switch to trigger an Ajax action (using the onSwitchChange event).
But my async action tends to take up to 1sec to process, so I figured it could be interesting to introduce a loading state (disabled state with a loader on it) that I would enable when my call starts and disable when it is done.

I believe you could accomplish this by using the callback on your ajax request.  I.e. bind the following func to your triggering event
function () {
$(switchelement).bootstrapSwitch({disabled:true});
$.ajax) {
url: someurl.html
}).done(function () {
$(switchelement).bootstrapSwitch({disabled:false});
}
}
just a thought...

Hi @ghostofrecon , how can I do this in this code?
 $(function() {

	$("[name='my-checkbox']").bootstrapSwitch();
	$('input[name="my-checkbox"]').on('switchChange.bootstrapSwitch',
	    function(event, state) {
	    	var beforeUpdateState = state;
	        var item = $(this);
	        var id = item.data('id');
	        console.log(state);
	        //item.bootstrapSwitch(toggleState, true)

	        $.post('test.php',{id:id,status:state})
	        .done(function(res){
	        	console.log('res',res);
	        	if(res=='notupdated')
	        		item.bootstrapSwitch('toggleState', beforeUpdateState);
	        });
	});

});

Thanks

Came here looking for an actual loading state, whereby the switch can display a loading icon while a promise is running. Is this a possibility?


Update fs2-core, fs2-io, ... to 2.4.5

Updates

co.fs2:fs2-core
co.fs2:fs2-io
co.fs2:fs2-reactive-streams

from 2.4.2 to 2.4.5.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

labels: library-update, semver-patch

Closed in favor of #261


Secure all Identity-X Application(s) user facing email links

 Links in email should all be HTTPS

Resolved via: https://github.com/parameter1/base-cms/pull/916


Hot fix to add defaults to cloud params

See #2810
Most important fix is resume and suspend rate.
Prior to fix, default rate will cause nodes to be created and destroyed at a reduced rate on large clusters.
Submission Checklist
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cloud HPC Toolkit Contribution guidelines #

Tests that failed are added recently and running in develop branch. So it is safe to ignore failure while merging in main since build files are not present in main atm.

PR-test-slurm-gcp-v6-rocky8 and PR-test-hpc-enterprise-slurm tests succeeded.


Please also release as a Firefox add-on

I've tested this out by cloning, npm installing, and running grunt chrome-extension, then using:

Open "about:debugging" in Firefox, click "Load Temporary Add-on" and select any file in your add-on's directory:

by pointing it to the dist directory.
It seems to work fine and this would be super helpful to those not using Chrome as their daily-driver.
I'd be happy to help in any way.

I've also tested it and it seems to be working fine, why not releasing it as a Firefox Add-on? I've been using Chrome for quite long, but I'm considering on switching to FF, as Quantom works really nice


bws: 0.4.0 -> 1.0.0

Description of changes
BWS CHANGELOG
Note: This un-breaks the package on aarch64-darwin, due to new Rust compiler and this issue: time-rs/time/issues/693
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

1 package built:
bws
Result of nixpkgs-review pr 344760 run on x86_64-linux 1

1 package built:

bws


shard segment reduction op to multi threads

shards the segment reduction op to multi threads when computing with CPU, so as to speed up the op.
In my test, the time may reduce to about half of original version.
Some other codes are modified automatically when I do the code format with:
clang-format-3.9 -i segment_reduction_ops.cc --style=google

So I'm wondering if there's  something wrong with my clang-format.
Env

CPU: 8 cores
Mem: 14G

the benchmark program: https://gist.github.com/shengofsun/cf9b203cb722a290667360770923c55f

@caisq could you please take a look at this patch?

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

@caisq PTAL

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 29 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 44 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 29 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 44 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 59 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 74 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 89 days with no activity and the awaiting review label has been applied.

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 104 days with no activity and the awaiting review label has been applied.

@gbaned I work on XLA and sometimes StreamExecutor, but not TensorFlow.  I get asked to do a lot of TensorFlow reviews and I have to say "no" every time.  Is there something I can do to fix this?

Nagging Reviewer @caisq: You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

@shengofsun could you please resolve the conflicts? Thanks!

@shengofsun gentle ping to resolve the conflicts. Thanks!

@shengofsun Did you get a chance to look on conflicts? Please let us know on the update. Thanks!

It has been 15 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!


FaceTime server issue and feature request

Issue: In the server, I get Failed to start FaceTime server... osascript can't open default scripting component.
I'm running on MacOS Monterey (12.6.5) through VMWare.
Feature request: I haven't seen it mentioned in any PRs or issues, but I know Airmessage has a (somewhat working) feature where their app automatically converts FaceTime calls to FaceTime links that their client apps can then handle. Would this be a possible feature to add to BlueBubblesApp?

@CompuGenius we still don't technically know why this happens. We are just running AppleScripts, which should theoretically be fine, but we've seen this issue before. There is no fix to it outside of rebooting your Mac. And we suggest to disable the Facetime detection feature right after so it doesn't happen again.
We will be looking into ways to mitigate this. Or even improve upon it using the Private API.

There is no fix to it outside of rebooting your Mac.

I have noticed that if I reboot, the error stops for a bit, but does always come back. Oh well 🤷♂
Do you have any plans regarding the Facetime link integration itself?

There is no fix to it outside of rebooting your Mac.

I have noticed that if I reboot, the error stops for a bit, but does always come back. Oh well 🤷♂
Do you have any plans regarding the Facetime link integration itself?

I think we are more likely to try and accomplish it through the Private API to make it more reliable than using UI scripting using Accessibility.
Also, for what it's worth, we've compared our AppleScript code to AirMessage's and they seem very very similar, and essentially do the same thing. We are not sure why it would cause a persistent issue, but not for AirMessage.

I think we are more likely to try and accomplish it through the Private API to make it more reliable than using UI scripting using Accessibility.

Awesome, I use Private API so that's great! I happen to be an experienced developer and am pretty passionate about this, so I'd actually be happy to go through and try to learn what's happening with this entire project so I can help add the facetime functionality, if needed.

I think we are more likely to try and accomplish it through the Private API to make it more reliable than using UI scripting using Accessibility.

Awesome, I use Private API so that's great! I happen to be an experienced developer and am pretty passionate about this, so I'd actually be happy to go through and try to learn what's happening with this entire project so I can help add the facetime functionality, if needed.

If you join our discord, we have channels dedicated to this type of collaboration. So feel free to chime in and give us a shout!

I'm closing this as we will very soon be removing the ability to get the FaceTime status via AppleScript, in favor of doing it via the Private API on Big Sur and newer. The FaceTime detection via the AppleScript compromised the stability of the server and I'm glad we were able to replace it using the Private API.


Fix tsconfig-doc.json to be valid json

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


failed: Access violation in stack frame 5 at address 0x200005ee0 of size 8

Version:
cxp@R9000P:~/solana_learn/xenobunny/xenobunny-new/xenobunny-rust$ rustc -V
rustc 1.75.0 (82e1608df 2023-12-21)
cxp@R9000P:~/solana_learn/xenobunny/xenobunny-new/xenobunny-rust$ solana -V
solana-cli 1.17.22 (src:dbf06e25; feat:3580551090, client:SolanaLabs)
cxp@R9000P:~/solana_learn/xenobunny/xenobunny-new/xenobunny-rust$ anchor -V
anchor-cli 0.30.1

Code:
#[derive(Accounts)]
#[instruction(collection_nft_mint: Pubkey)]
pub struct MintNft<'info> {
    #[account(
        init,
        seeds = [nft_mint.key().as_ref(), WARRITOR_CARD_STATE.as_bytes()],
        bump,
        payer = user,
        space = 8 + std::mem::size_of::<WarriorCardState>()
    )]
    pub warrior_card_state: Box<Account<'info, WarriorCardState>>,
    #[account(
        init,
        seeds = [
            user.key().as_ref(),
            collection_nft_mint.as_ref(),
            WARRITOR_CARDS_USAGE_TRACKER.as_bytes(),
        ],
        bump,
        payer = user,
        space = 8 + std::mem::size_of::<WarriorCardsUsageTracker>()
    )]
    pub warrior_cards_usage_tracker: Box<Account<'info, WarriorCardsUsageTracker>>,
    #[account(seeds = [MINT_NFT_CONFIG.as_bytes()], bump)]
    pub nft_mint_config: Box<Account<'info, NFTMintConfig>>,
    #[account(
        init,
        payer = user,
        mint::decimals = 0,
        mint::authority = mint_authority,
        mint::freeze_authority = mint_authority
    )]
    pub nft_mint: Box<Account<'info, Mint>>,
    // #[account(
    //     mint::decimals = 0,
    //     mint::authority = mint_authority,
    //     mint::freeze_authority = mint_authority,
    //     seeds = [XENOBUNNY_NFT_COLLECTION.as_bytes()],
    //     bump
    // )]
    // pub collection_nft_mint: Account<'info, Mint>,
    #[account(
        init,
        payer = user,
        associated_token::mint = nft_mint,
        associated_token::authority = user
    )]
    pub user_ata: Box<Account<'info, TokenAccount>>,
    /// CHECK: This is a PDA account, safe to skip verification
    #[account(seeds = [MINT_AUTHORITY.as_bytes()], bump)]
    pub mint_authority: UncheckedAccount<'info>,
    /// CHECK: Created by the metadata program and does not require checking
    #[account(mut)]
    pub metadata_account: UncheckedAccount<'info>,
    #[account(mut)]
    pub user: Signer<'info>,
    pub token_program: Program<'info, Token>,
    /// CHECK: This is a PDA account, safe to skip verification
    #[account(address = TOKEN_METADATA_ID)]
    pub token_metadata_program: UncheckedAccount<'info>,
    pub system_program: Program<'info, System>,
    pub associated_token_program: Program<'info, AssociatedToken>,
}

I referred the previous suggestion(https://github.com/coral-xyz/anchor/issues/3089) and Box some account, but it still show
failed: Access violation in stack frame 5 at address 0x200005ee0 of size 8
when I try to using Solana CLI 1.17.22
[dependencies]
anchor-lang = { version = "0.30.1", features = ["init-if-needed"] }
anchor-spl = "0.30.1"
spl-token-2022 = "3.0.4"
solana-program = "1.18.20"
mpl-token-metadata = "4.1.2"

That is my actions
cxp@R9000P:~/solana_learn/xenobunny/xenobunny-new/xenobunny-rust$ solana-install init 1.17.22
  ✨ 1.17.22 initialized
cxp@R9000P:~/solana_learn/xenobunny/xenobunny-new/xenobunny-rust$ anchor build
info: uninstalling toolchain 'solana'
info: toolchain 'solana' uninstalled
error: package `num_enum_derive v0.7.3` cannot be built because it requires rustc 1.70.0 or newer, while the currently active rustc version is 1.68.0-dev
Either upgrade to rustc 1.70.0 or newer, or use
cargo update -p<EMAIL_ADDRESS>--precise ver
where `ver` is the latest version of `num_enum_derive` supporting rustc 1.68.0-dev

This will be fixed in the next version (https://github.com/coral-xyz/anchor/pull/3259). See Stack memory improvements if you want to learn more.

This will be fixed in the next version (#3259). See Stack memory improvements if you want to learn more.
I optimized the program and successfully compiled it.


Add source field to metadata

This is needed if one want to use index.query_with_sources on git files. Without a source field, index.query_with_sources fails with an exception.

hm the name of the field to use for source is a parameter that you can technically change, but it's definitely not straightforward. think a good solution would be to either make it much easier to specify the name of the metadata field to use as the source, and/or to add a typed source attribute to documents, so we're not just hoping it's in metadata. any thoughts @eyurtsev?

I'd go for a fixed name for the source field as you may want to ask a query over multiple document types.
Judging from the other document_loaders, I'd say the fact that no source is set at all (whether it is in metadata or not) in this loader is an oversight and this PR should fix this.
Perhaps your suggestion about making it more uniform for all document loaders is worthy of another issue? I'd be happy to help with that also :-)


Initial Unit Tests

@FriedrichWeinmann I used mstests, but I'm not sure if that works for a person that doens't have Visual Studio installed. How important is it that to us?
I prefer nunit, because I always used nUnit. But if mstest will mean one less binary to store in a repo that we don't want external dependencies, then lets go with mstest.
Changes proposed in this pull request:

Added initial unit test framework.
Wrote a Size Test
Refactored the Size code a little.

How to test this code:

[ ] Make sure I didn't break size.

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[ ] Working/useful help content, including link to command on dbatools web site
[ ] All examples work as advertised
[ ] Does not contain template content
[ ] Does not contain excessive/unnecessary amounts of comments
[ ] Works remotely
[ ] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[ ] Works with named instances
[ ] Works with clustered instances
[ ] Handles offline/read only databases
[ ] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[ ] No un-handled errors which stop the command working with multiple servers

Heya, looked through it and sounds solid to me.
Would you mind also updating gitignore to ignore the build artifacts of the tests?
Other than that, I'll be happy to merge this once Chrissy is done with the current master release (scheduled for Sunday)

I'm behind - We'll do the beta release on Wednesday. Shall we merge @FriedrichWeinmann ?


⚠ Bofrost.fr has degraded performance

In c89d149, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1357 ms

Resolved: Bofrost.fr performance has improved in af270a9 after 9 minutes.


Windows not completetly visible

The status window is not completely visible - please fix

@graham22 I noticed at least 2 people complaining about the window size not fitting their monitor https://github.com/ClassicDIY/ModbusTool/issues/33
I know I recently made the change to fix the window size, but I just realised it's also possible to make the window scrollable. Do you think that's a good idea? If so I'll do a PR.

that would solve my problem


Runtime Upgrade UX

Description
This ticket documents the work done to push version 1.4.1 of Composer with the UX to show a breaking upgrade from v1.4.1 to v2.0 when Composer v2.0 releases.
The changes will be applied on the v1.4.1 branch

Shiproom - migration is a P0 scenario - approved

@sgellock using this ticket to get Composer's main rebased with 1.4.1 commits. Can merge the PR once I get the approval.


For PressableButton + Unity UI prefabs, add far field visual feedback

Describe the problem
With PressableButtonUnityUI.prefab and its variants, far field interactions correctly activate the button but there is no visual feedback.
Describe the solution you'd like
Consider adding an Interactable to the prefab and wiring it up to get far field visual/auditory feedback similar to PressableButtonHoloLens2.prefab.

See #6049 as an example.

@CDiaz-MS Have you seen this? This involves visual feedback states for UnityUI-based elements. Is this possible, or maybe a good candidate for InteractiveElement?

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


DUPLO 17067 TFstate update fix

Overview
Bug fix for tfstate not getting update for allow_global_access schema attribute for duplocloud_duplo_service_lbconfigs resource
This PR does the following:

PR sets value for allow_global_access during read

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[ ✓] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-17067 TF: allow_global_access attribute value shows always false in tfstate


[FEATURE]

Is your feature request related to a problem? Please describe.
I don't know if I've done something wrong but the locale for firefox and chrome always reverts to United Arab Emirates, regardless of what's set on the host system, in the docker compose file or in the browser within neko
Describe the solution you'd like
I would like the ability to change the locale, this would be a useful feature as neko is "usually" installed on a vps. Either through the compose file, as a flag when run through docker or within neko.
Additional context
Using the latest docker image and followed instructions from the official site, I don't think it's a geoip issue as the ip my vps uses is in germany according to maxmind

I ran into the same issue, its a google thing https://www.reddit.com/r/google/comments/3trv4t/google_thinks_im_in_the_uae/ google is useing an out dated ip database. You have to manually set it in the google settings.


Update service creation error parsing messages

... when discovering the service CIDRs. The messages are displayed to the users so make them more friendly. This is a follow-up to https://github.com/submariner-io/submariner-operator/pull/1061.
@mkimuram

@tpantelis
Thank you for improving the error messages. It looks good to me.


Grid (.col) clarification

Hey again,
I noticed that there's .col, .sm-col, .md-col, and .lg-col. When defining a column, is there a difference between:

and

I.e. in that example, is there a reason why .sm-col should be used instead of .col?

Yes, .col, .sm-col, etc. all set float left, but only for the specified breakpoint.
In your first example, the column would always float left, but only have a width defined from --breakpoint-sm.

Cool, thanks for the clarification,


Formtastic removed GroupedCollections included in select_input.rb but Formtastic Bootstrap didn't

In lib/formtastic-bootstrap/inputs/select_input.rb, this line should be deleted
include Base::GroupedCollections

Due to this commit made in Formtastic.

Resolved by #116


ToStringFast does not take Display attribute into account.

Hi @andrewlock .
I noticed that the instructions of using the NuGet package indicates that the Display attribute can be used to "override" the way the .ToStringFast returns the string.
I implemented everything as in the readme, but I cannot seem to get the [Display( Name = "Value" )] to work.
I used the following Enum:
public enum MyEnum
{
    Value1,

    [Display(Name = "Val2")]
    Value2,
}

The code is then generated as follows:
public static string ToStringFast(this MyEnum value)
            => value switch
            {
                MyEnum.Value1 => nameof(MyEnum.Value1),
                MyEnum.Value2 => nameof(MyEnum.Value2),
                _ => value.ToString(),
            };

I have tried both with and without the NETESCAPADES_ENUMGENERATORS_USAGES constant, same behavior every time.
I am using .NET 6.0 and the NuGet package (1.0.0-beta04) is installed in a Blazor Server app.
As indicated by FroggieFrog, the NuGet is still in pre-release, thus does not have the main branch's features yet. However this happens on the main branch (which has this feature) as well.
Am I missing something?

As mentioned in issue #22:
The latest release on NuGet is from November 2021, but the feature was added in April 2022.
So there need to be a new release on NuGet to use this feature.

I am reopening this issue, as I have checked out the main branch and created the NuGet package locally.
The code is still generated as I showed above.

yup same problem here, downlaoded main branch, built nuget for myself and still no luck with Display attribute.

I'm going to close this for now, as it definitely works in the main branch, and I'm going to do a new release soon! Your issues were likely due to caching of the old package when you tested locally 🙂


Error 404: File not found on a basic maven project

Describe the bug
I'm trying to use the frogbot GitHub action and at the end of the build I can't access the report file generated. I get a 404 file not found error.
The same project with the command jf xr audit-mvn generates the expected report.
The project that I am using is:
Basic maven project: https://github.com/ixchelruiz/app
Github action: https://github.com/ixchelruiz/app/actions/runs/2185445703
At the end of the build on "[scan-pull-request]" I get the following error:
[main] INFO org.jfrog.build.extractor.maven.BuildDeploymentHelper - Artifactory Build Info Recorder: publish build info set to false, build info will not be published...
[465](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:465)
  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ------------------------------------------------------------------------
[466](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:466)
  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - BUILD SUCCESS
[467](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:467)
  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ------------------------------------------------------------------------
[468](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:468)
  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Total time:  5.889 s
[469](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:469)
  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - Finished at: 2022-04-18T18:42:34Z
[470](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:470)
  [main] INFO org.apache.maven.cli.event.ExecutionEventLogger - ------------------------------------------------------------------------
[471](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:471)
  Error: 4 [Error] Server response: 404
[472](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:472)
  {
[473](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:473)
    "errors": [
[474](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:474)
      {
[475](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:475)
        "status": 404,
[476](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:476)
        "message": "File not found."
[477](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:477)
      }
[478](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:478)
    ]
[479](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:479)
  }
[480](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:480)
  Error: The process '/opt/hostedtoolcache/frogbot/[RELEASE]/x64/frogbot' failed with exit code 1

To Reproduce
Use
Basic maven project: https://github.com/ixchelruiz/app
Github action: https://github.com/ixchelruiz/app/actions/runs/2185445703
Expected behavior
Report is generated and added in the comments exactly as the report generated on the Jfrog cli
jf xr audit-mvn
Screenshots
If applicable, add screenshots to help explain your problem.

Versions

Frogbot version: v1
Operating system: Apple M1 Max
JFrog Artifactory version:  2.15.0
JFrog Xray version: 3.45.1

My local setup (jfrog  CLI ) same configuration ( tokens ) WORKS
jf config show
Server ID:			Default-Server
JFrog platform URL:		https://ixchelr.jfrog.io/
Artifactory URL:		https://ixchelr.jfrog.io/artifactory/
Distribution URL:		https://ixchelr.jfrog.io/distribution/
Xray URL:			https://ixchelr.jfrog.io/xray/
Mission Control URL:		https://ixchelr.jfrog.io/mc/
Pipelines URL:			https://ixchelr.jfrog.io/pipelines/
Password:			***
Access token:			***
Refresh token:			***
Default:			true

GitHub actions configuration -- DOESNT WORK
Run jfrog/frogbot@v1
[2](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:2)
  with:
[3](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:3)
    version: latest
[4](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:4)
  env:
[5](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:5)
    JAVA_HOME: /opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/11.0.14-1/x[6](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:6)4
6
    JF_URL: ***
[7](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:7)
    JF_GIT_TOKEN: ***
[8](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:8)
    JF_ACCESS_TOKEN: ***
[9](https://github.com/ixchelruiz/app/runs/6066751313?check_suite_focus=true#step:4:9)
Frogbot

Additional context
Add any other context about the problem here.

Hi @ixchelruiz,
I opened this PR on a fork of your project (same action workflow) and it worked as expected against my JFrog platform instance.
Would you like to try to configure the JF_URL and JF_ACCESS_TOKEN secrets again?


uboot: increase rpi kernel size patch

Motivation for this change
When trying to boot my Pi 4B with a fresh sd img from hydra, i got an ERROR: RD image overlaps OS image (OS=0x200000..0x3100000), that would end up in a Cannot load any image error, preventing the boot process altogether. Which seems to be similar to https://github.com/NixOS/nixpkgs/issues/97064
Following @samueldr's recommendation, i've increased the patch's offset by 8MB.
I can confirm compiling it with nix build and moving the resulting u-boot.bin to the firmware partition on my sd card as u-boot-rpi4.bin completely solves the issue on my pi.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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

[x] Fits CONTRIBUTING.md.

Can confirm this patch worked for me for my raspberry pi 3!

Fixed booting my RPi4 after an upgrade to kernel 5.14 broke the boot.
Error looks like so:
U-Boot 2021.04 (Apr 05 2021 - 15:03:29 +0000)

DRAM:  3.9 GiB
RPI 4 Model B (0xc03111)
MMC:   mmcnr@7e300000: 1, emmc2@7e340000: 0
Loading Environment from FAT... *** Warning - bad CRC, using default environment

In:    serial
Out:   serial
Err:   serial
Net:   eth0: ethernet@7d580000
PCIe BRCM: link up, 5.0 Gbps x1 (SSC)
starting USB...
Bus xhci_pci: Register 5000420 NbrPorts 5
Starting the controller
USB XHCI 1.00
scanning bus xhci_pci for devices... Sorry, protocol 2 not yet supported.
5 USB Device(s) found
       scanning usb for storage devices... 1 Storage Device(s) found
Hit any key to stop autoboot:  0
switch to partitions #0, OK
mmc0 is current device
Scanning mmc 0:1...
Found /extlinux/extlinux.conf
Retrieving file: /extlinux/extlinux.conf
2816 bytes read in 21 ms (130.9 KiB/s)
------------------------------------------------------------
1:	NixOS - Default
2:	NixOS - Configuration 626 (2021-10-03 16:30 - 21.11pre-git)
3:	NixOS - Configuration 625 (2021-09-21 05:15 - 21.11pre-git)
4:	NixOS - Configuration 624 (2021-09-21 05:13 - 21.11pre-git)
5:	NixOS - Configuration 623 (2021-09-21 04:28 - 21.11pre-git)
6:	NixOS - Configuration 622 (2021-09-16 05:24 - 21.11pre-git)
Enter choice: 1:	NixOS - Default
Retrieving file: /extlinux/../nixos/gnvsff3y8xiq4zjdrl4pwzjnf20f6pfg-initrd-linux-5.14.9-initrd
9971664 bytes read in 457 ms (20.8 MiB/s)
Retrieving file: /extlinux/../nixos/f3y00llkbcj4skgn7biqy1nciy3rlnws-linux-5.14.9-Image
45439488 bytes read in 1985 ms (21.8 MiB/s)
append: init=/nix/store/ida56m4p6kmf5hnq88q347242vrh8c4h-nixos-system-dione-21.11pre-git/init console=ttyS1,115200n8 loglevel=4
Retrieving file: /extlinux/../nixos/f3y00llkbcj4skgn7biqy1nciy3rlnws-linux-5.14.9-dtbs/broadcom/bcm2711-rpi-4-b.dtb
26385 bytes read in 40 ms (643.6 KiB/s)
Moving Image from 0x80000 to 0x200000, end=2e00000
## Flattened Device Tree blob at 02e00000
   Booting using the fdt blob at 0x2e00000
ERROR: FDT image overlaps OS image (OS=0x200000..0x2e00000)
2:	NixOS - Configuration 626 (2021-10-03 16:30 - 21.11pre-git)
Retrieving file: /extlinux/../nixos/gnvsff3y8xiq4zjdrl4pwzjnf20f6pfg-initrd-linux-5.14.9-initrd
9971664 bytes read in 456 ms (20.9 MiB/s)
Retrieving file: /extlinux/../nixos/f3y00llkbcj4skgn7biqy1nciy3rlnws-linux-5.14.9-Image
45439488 bytes read in 1985 ms (21.8 MiB/s)
append: init=/nix/store/ida56m4p6kmf5hnq88q347242vrh8c4h-nixos-system-dione-21.11pre-git/init console=ttyS1,115200n8 loglevel=4
Retrieving file: /extlinux/../nixos/f3y00llkbcj4skgn7biqy1nciy3rlnws-linux-5.14.9-dtbs/broadcom/bcm2711-rpi-4-b.dtb
26385 bytes read in 41 ms (627.9 KiB/s)
Moving Image from 0x80000 to 0x200000, end=2e00000
## Flattened Device Tree blob at 02e00000
   Booting using the fdt blob at 0x2e00000
ERROR: FDT image overlaps OS image (OS=0x200000..0x2e00000)
3:	NixOS - Configuration 625 (2021-09-21 05:15 - 21.11pre-git)
Retrieving file: /extlinux/../nixos/1rkhivc32jl0zvl61md89bb4whv2ikmv-initrd-linux-5.14.3-initrd
8645773 bytes read in 417 ms (19.8 MiB/s)
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-Image
45447680 bytes read in 2010 ms (21.6 MiB/s)
append: init=/nix/store/zdgkaq6q76fx83zg4bwjili9a7kli6sm-nixos-system-dione-21.11pre-git/init console=ttyS1,115200n8 loglevel=4
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-dtbs/broadcom/bcm2711-rpi-4-b.dtb
26385 bytes read in 41 ms (627.9 KiB/s)
Moving Image from 0x80000 to 0x200000, end=2e00000
## Flattened Device Tree blob at 02e00000
   Booting using the fdt blob at 0x2e00000
ERROR: FDT image overlaps OS image (OS=0x200000..0x2e00000)
4:	NixOS - Configuration 624 (2021-09-21 05:13 - 21.11pre-git)
Retrieving file: /extlinux/../nixos/1rkhivc32jl0zvl61md89bb4whv2ikmv-initrd-linux-5.14.3-initrd
8645773 bytes read in 419 ms (19.7 MiB/s)
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-Image
45447680 bytes read in 2008 ms (21.6 MiB/s)
append: init=/nix/store/h9qwfvf36ngr0xhfd6a0p5h84s1bvrf4-nixos-system-dione-21.11pre-git/init console=ttyS1,115200n8 loglevel=4
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-dtbs/broadcom/bcm2711-rpi-4-b.dtb
26385 bytes read in 41 ms (627.9 KiB/s)
Moving Image from 0x80000 to 0x200000, end=2e00000
## Flattened Device Tree blob at 02e00000
   Booting using the fdt blob at 0x2e00000
ERROR: FDT image overlaps OS image (OS=0x200000..0x2e00000)
5:	NixOS - Configuration 623 (2021-09-21 04:28 - 21.11pre-git)
Retrieving file: /extlinux/../nixos/1rkhivc32jl0zvl61md89bb4whv2ikmv-initrd-linux-5.14.3-initrd
8645773 bytes read in 417 ms (19.8 MiB/s)
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-Image
45447680 bytes read in 2010 ms (21.6 MiB/s)
append: init=/nix/store/425lpxsy5lxdbx6zq3wvcwsg4661cn0z-nixos-system-dione-21.11pre-git/init console=ttyS1,115200n8 loglevel=4
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-dtbs/broadcom/bcm2711-rpi-4-b.dtb
26385 bytes read in 41 ms (627.9 KiB/s)
Moving Image from 0x80000 to 0x200000, end=2e00000
## Flattened Device Tree blob at 02e00000
   Booting using the fdt blob at 0x2e00000
ERROR: FDT image overlaps OS image (OS=0x200000..0x2e00000)
6:	NixOS - Configuration 622 (2021-09-16 05:24 - 21.11pre-git)
Retrieving file: /extlinux/../nixos/1rkhivc32jl0zvl61md89bb4whv2ikmv-initrd-linux-5.14.3-initrd
8645773 bytes read in 417 ms (19.8 MiB/s)
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-Image
45447680 bytes read in 2010 ms (21.6 MiB/s)
append: init=/nix/store/mwhb53zlg702ardp7043jcya3zmzn6fc-nixos-system-dione-21.11pre-git/init console=ttyS1,115200n8 loglevel=4
Retrieving file: /extlinux/../nixos/ilcfy4bi8zdkinfrhn28fxiqnxhvrwr9-linux-5.14.3-dtbs/broadcom/bcm2711-rpi-4-b.dtb
26385 bytes read in 41 ms (627.9 KiB/s)
Moving Image from 0x80000 to 0x200000, end=2e00000
## Flattened Device Tree blob at 02e00000
   Booting using the fdt blob at 0x2e00000
ERROR: FDT image overlaps OS image (OS=0x200000..0x2e00000)
SCRIPT FAILED: continuing...
Card did not respond to voltage select! : -110
Scanning disk<EMAIL_ADDRESS>Disk<EMAIL_ADDRESS>not ready
Scanning disk<EMAIL_ADDRESS>Scanning disk usb_mass_storage.lun0...
** Unrecognized filesystem type **
Found 4 disks
No EFI system partition
BootOrder not defined
EFI boot manager: Cannot load any image
Card did not respond to voltage select! : -110

Platform firmware for ARM platforms are not handled through the generations lifecycle.
We still have an open question about how to provide a good user story for updating platform firmwares.
In your case, you would need to update U-Boot manually by:

Mounting the FAT32 partition at the beginning of the SD card
Backup its contents
Copy the new U-Boot binary
(possibly) updating the dtb files too

Think of U-Boot not as a bootloader (e.g. GRUB), but more like your BIOS or UEFI.


Recommended CPU memory size

Hi, @ziyc
Thanks for your great work.
I have some trouble in training. My CPU memory is insufficient to run it. Could you please advise on the recommended CPU memory size?

I found that when I implement the evaluation, i.e., eval.py, every time an image is rendered, the CPU memory usage will continue to increase. Is there a way to clear the memory every time the rendering is completed?


Bugfix/106845 fix date validation

Apply validation only when leaving the component

@dotcom9 What's the behaviour when tabbing through a required date field without entering anything? It looks like it wouldn't validate now, but it should, similar to how a text or number field would say "this field is required". I think the test for when to validate needs to be based on whether the three fields have ever received focus rather than whether they contain a value.

@dotcom9 What happens when hideDay={true}? It looks like validation will never fire because the value of day will always be an empty string.

@dotcom9 What's the behaviour when tabbing through a required date field without entering anything? It looks like it wouldn't validate now, but it should, similar to how a text or number field would say "this field is required".

True: the AC don't cover a required field. Added this.


docs(widgets): improve ScrollbarState documentation

I created this PR as draft because I almost forgot about it. I will refine it tomorrow

Another thing I observed is that occasionally the scrollbar handle changes its size by one when moving it (looks a bit like a caterpillar crawling 😄).

This is reasonable if you think about it in terms of how the scrollbar should round.
Example. Scrollbar viewport (no arrows) = 10. Showing some amount which maps to [0.2..2.4]. When rounded that will display a bar from [0..2] Move it a little to [0.4..2.6] and now that rounds instead to [0..3], move it a bit more to [0.6, 2.8] and you get [1..3]. Caterpiller effect.
Something we could do to fix that is make the scrollbar smarter and use partial bars. This would only be possible in the terminal with some symbol sets - 1/8th blocks, quadrant blocks, braille, and the symbols for legacy computing block stuff (sextant / octant). This could be worth considering, but it would add a lot of complexity to the implementation. Because of that I'd suggest experimenting with something like that outside of the Ratatui lib first and then bringing it in paying attention to how previous software works.

I think that is a tradeoff whether you prefer to have a constant handle size or accurate representation of position.
I also think half block would be to complex for the supposed benefit, will they even render correctly everywhere?

I think that is a tradeoff whether you prefer to have a constant handle size or accurate representation of position. I also think half block would be to complex for the supposed benefit, will they even render correctly everywhere?

Another thing to note here is that not rounding like this has some more problems with the way that it displays the bar when positioned close to the end of the bar. If you round to a constant size rather than constant position, then it leads to oddness in the last position. There's perhaps a design that would account for both, but the complexity to maintain it seems intuitively like it would be higher than the benefit (unless written really well).


WebpackIndexHTMLPlugin customisation with createCompatibilityConfig

Hey guys,
I'm trying to ship a custom tailored webcomponents polyfill without ShadyCSS (since I don't have shadow DOM).
relevant parts of my webpack config are:
const configs = createCompatibilityConfig({
  input: path.resolve(__dirname, './index.html'),
});

const newConfigs = configs.map((config) =>
  merge(config, {
    module: {
      rules: [
        {
          test: /\.s?[ac]ss$/,
          use: [MiniCssExtractPlugin.loader, 'css-loader', 'sass-loader'],
        },
        {
          test: /\.(woff|woff2)$/,
          use: {
            loader: 'file-loader',
            options: {
              name: '[name].[ext]',
              outputPath: './webfonts',
            },
          },
        },
      ],
    },
    plugins: [
      new MiniCssExtractPlugin({
        filename: '[name].css',
        chunkFilename: '[id].css',
      }),
      new CopyWebpackPlugin([
        {
          from: 'assets',
          to: 'assets',
        },
      ]),
      new WebpackIndexHTMLPlugin({
        multiBuild: true,
        polyfills: {
          coreJs: true,
          regeneratorRuntime: true,
          webcomponents: true,
          fetch: true,
        },
      }),
    ],
  }));

module.exports = newConfigs;

Removing the last plugin, the build works, it ships two versions with the bundled polyfills for legacy.
If I add the last plugin, the build fails with:
TypeError: Cannot read property 'resolve' of null
    at compiler.hooks.emit.tapPromise (/Users/fnunglaub/Code/some-repo/node_modules/@open-wc/webpack-index-html-plugin/webpack-index-html-plugin.js:117:37)
    at _next1 (eval at create (/Users/fnunglaub/Code/some-repo/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:9:17)
    at _err1 (eval at create (/Users/fnunglaub/Code/some-repo/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:26:1)
    at Promise.all.catch.then (/Users/fnunglaub/Code/some-repo/node_modules/copy-webpack-plugin/dist/index.js:90:9)

Any ideas?

You need to pass in a boolean flag which tells the plugin whether you're running the modern or legacy version of the build. https://github.com/open-wc/open-wc/blob/master/packages/building-webpack/modern-and-legacy-config.js#L133
You can pass custom configuration to the index html plugin by passing an options object when you create the config: https://open-wc.org/building/building-webpack.html#common-extensions
Also, it's possible to add custom polyfills so you could load the web component polyfills without shady css support. You can read more about it here: https://open-wc.org/building/webpack-index-html-plugin.html#polyfills

Hey Lars,
thanks for the infos.
I also noticed that merge does not overwrite the default plugin which comes from createCompatibilityConfig. My custom plugin will simply be appended. I replaced the plugins array in both configs manually now and this weird Promise rejection will be gone then. I'll keep you updated.

I managed to achieve what I wanted now :)
Here's my webpack config FYI: https://gist.github.com/funglaub/7d63f2300dd9031027471977318c32e9


Grunt tab should be hidden by default

It currently takes up a lot of space and is intrusive

:+1:

This PR #67 adds that possibility. There is an option in the package settings.

This is now live on master. As soon as @nickclaw publishes a new version you should be good.
If you can't wait that long you can install the latest version from master :)


adds start view and end view to resetEpoch

Closes #180
Adds startView and endView as parameters to the resetEpoch method
Open question: Should there be any other pre-conditions or expectations for these parameters?

@jordanschalm I added the pre-condition and a test in 46734ba


Add support for EXIF in PowerRename

When we combine photos taken from different devices, what usually happens with the image files is they either have:

different file name conventions
different time stamps

It would be nice to support renaming files based on their EXIF properties, like Date Taken.

track in #4564

track in #4564

@crutkas - These are two completely different issues.
#4564 is a request for a brand new PowerToy to REMOVE all EXIF data from ANY file.
#3880 is a request to ADD EXIF data variable patterns to the existing PowerRename PowerToy.
To clarify @n3ps request:
We would like the addition of EXIF Data attributes to the existing list of available renaming variable patters as described in the HELP documentation under "Replace using file creation date and time":
https://docs.microsoft.com/en-us/windows/powertoys/powerrename
Since ALL of these patterns are directly related to file creation date and time, EXIF Data would most likely be it's own section:
Replace using image EXIF data
The EXIF data attributes of an image can be used in the Replace with text by entering a variable pattern according to the table below.
Variable pattern | Explanation
$exif_YYYY | Year Taken represented by a full four or five digits, depending on the calendar used.
$exif_YY | Year Taken represented only by the last two digits. A leading zero is added for single-digit years.
$exif_Y | Year Taken represented only by the last digit.
$exif_MMMM | Name of the month
$exif_MMM | Abbreviated name of the month
$exif_MM | Month Taken as digits with leading zeros for single-digit months.
$exif_M | Month Taken as digits without leading zeros for single-digit months.
$exif_DDDD | Name of the day of the week
$exif_DDD | Abbreviated name of the day of the week
$exif_DD | Day of the month as digits with leading zeros for single-digit days.
$exif_D | Day of the month as digits without leading zeros for single-digit days.
$exif_hh | Hours with leading zeros for single-digit hours
$exif_h | Hours without leading zeros for single-digit hours
$exif_mm | Minutes with leading zeros for single-digit minutes.
$exif_m | Minutes without leading zeros for single-digit minutes.
$exif_ss | Seconds with leading zeros for single-digit seconds.
$exif_s | Seconds without leading zeros for single-digit seconds.
$exif_fff | Milliseconds represented by full three digits.
$exif_ff | Milliseconds represented only by the first two digits.
$exif_f | Milliseconds represented only by the first digit.

https://github.com/microsoft/PowerToys/issues/1585 has this covered then.

OK, he doesn't go into much detail, especially EXIF, so I will add this there as well.
Thanks!

I should've been more explicit then. Can you incorporate features of tools like ReNamer or EXIF Data Changer.

I should've been more explicit then. Can you incorporate features of tools like ReNamer or EXIF Data Changer.

 It's actually for Clint. It was a mistake on my part not to bother checking why @crutkas just closed it months ago.

@crutkas Would you kindly re-open this ticket? It's far more explicit than #1585 in its request and, additionally, it seems the other ticket was abandoned after it didn't receive some requested clarification (if I followed the thread correctly).


Add benchmark for process symbolization

This change introduces a benchmark for the process symbolization functionality. The benchmark measures end-to-end times for the symbolization of five addresses inside the current process.
> Benchmarking main/symbolize :: symbolize_process: Collecting 500 samples in estimated 5.
> main/symbolize :: symbolize_process
>     time:   [970.41 µs 970.62 µs 970.89 µs]

Going ahead with the merge of this trivial change.


5.10.5 Broken Since 5.10.4 fix

Working on 5.10.4, not 5.10.5 Following these instructions:
Download the DisplayLink driver from the official download page. Then run the following commands:
unzip 'DisplayLink USB Graphics Software for Ubuntu 5.3.1.zip'
chmod +x displaylink-driver-<IP_ADDRESS>.run
./displaylink-driver-<IP_ADDRESS>.run --noexec --keep
Get the updated evdi version by typing:
git clone https://github.com/DisplayLink/evdi.git
cd evdi
git checkout v1.7.x
tar cf evdi.tar.gz *
cp evdi.tar.gz ../displaylink-driver-<IP_ADDRESS>
Install displaylink:
cd ../displaylink-driver-<IP_ADDRESS>
sudo ./displaylink-installer.sh install
LOGS ATTACHED
DLSupportTool_Output_2021-01-07T03:19:04.772502.zip

Thanks @robotard.
I can confirm it works on Debian 10:
Linux 5.10.0-1-amd64 #1 SMP Debian 5.10.4-1 (2020-12-31) x86_64 GNU/Linux

Thanks @robotard.
I can confirm it works on Debian 10:
Linux 5.10.0-1-amd64 #1 SMP Debian 5.10.4-1 (2020-12-31) x86_64 GNU/Linux

What is not working?

What is not working?

Working fine now on 5.10.6... What wasn't working was from USB hub over 3.0 to external display... Basically what DisplayLink is for...
I've updated my script which I use to install, which may help explain the process that works for me...
https://github.com/robotard/setup-ubuntu/blob/master/scripts/displaylinkdownloader.sh

Working fine now on 5.10.6... What wasn't working was from USB hub over 3.0 to external display... Basically what DisplayLink is for...
I've updated my script which I use to install, which may help explain the process that works for me...
https://github.com/robotard/setup-ubuntu/blob/master/scripts/displaylinkdownloader.sh

@robotard Should this issue be closed if everything is working now?
Fedora 33 went from 5.9.x to 5.10.6 and it has been working fine for me.  Maybe there was a reason why they didn't switch to 5.10.x right away... Not sure.
Fedora just updated to 5.10.7 and it is working fine here as well.

@robotard Should this issue be closed if everything is working now?
Fedora 33 went from 5.9.x to 5.10.6 and it has been working fine for me.  Maybe there was a reason why they didn't switch to 5.10.x right away... Not sure.
Fedora just updated to 5.10.7 and it is working fine here as well.

Not working in 10.7 and 10.8
Sent from my iPhone

On 17 Jan 2021, at 22:44, Michael L. Young<EMAIL_ADDRESS>wrote:

@robotard Should this issue be closed if everything is working now?
Fedora 33 went from 5.9.x to 5.10.6 and it has been working fine for me. Maybe there was a reason why they didn't switch to 5.10.x right away... Not sure.
Fedora just updated to 5.10.7 and it is working fine here as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Not working in 10.7 and 10.8
Sent from my iPhone

On 17 Jan 2021, at 22:44, Michael L. Young<EMAIL_ADDRESS>wrote:

@robotard Should this issue be closed if everything is working now?
Fedora 33 went from 5.9.x to 5.10.6 and it has been working fine for me. Maybe there was a reason why they didn't switch to 5.10.x right away... Not sure.
Fedora just updated to 5.10.7 and it is working fine here as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Closing. Evdi should support kernels up to 5.13 now.


"Warning: no meaningful features" for very small datasets

Description
For small datasets (e.g. 5 or fewer features, less than 40 observations), Dataset construction says that features are "constant" and and not "meaningful", even when features do have variability and are related to the target.

[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.

Reproducible example
Observed this while using the Python package.
import lightgbm as lgb
from sklearn.datasets import make_regression

for num_rows in range(10, 100, 5):
    print(f"--- {num_rows} ---")
    X, y = make_regression(
        n_samples=num_rows,
        n_informative=5,
        n_features=5,
        random_state=708
    )
    dtrain = lgb.Dataset(
        data=X,
        label=y,
        params={
            "verbosity": 1,
            "min_data_in_bin": 1
        }
    )
    dtrain.construct()

Produces a log like the following:
--- 10 ---
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
--- 15 ---
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
--- 20 ---
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
--- 25 ---
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
--- 30 ---
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
--- 35 ---
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
--- 40 ---
--- 45 ---
--- 50 ---
--- 55 ---
--- 60 ---
--- 65 ---
--- 70 ---
--- 75 ---
--- 80 ---
--- 85 ---
--- 90 ---
--- 95 ---

This is surprising, given that those features are informative for the target.
For example, a plain random forest achieves a good fit to the target even for just 20 observations from that same dataset.
from sklearn.datasets import make_regression
from sklearn.ensemble import RandomForestRegressor
from sklearn.metrics import r2_score

X, y = make_regression(
    n_samples=20,
    n_informative=5,
    n_features=5,
    random_state=708
)

reg = RandomForestRegressor(n_estimators=10).fit(X, y)
r2_score(y, reg.predict(X))
# 0.92602

I'm able to reproduce this behavior with a different dataset in the R package.
R example (click me)
library(lightgbm)

data(ChickWeight)

target_col <- "weight"
feature_cols <- c("Time", "Chick")

for (num_rows in seq(10, 100, 5)){
    print(paste0("--- ", num_rows, " ---"))
    X <- data.matrix(ChickWeight[seq_len(num_rows), feature_cols])
    y <- ChickWeight[seq_len(num_rows), target_col]
    dtrain <- lightgbm::lgb.Dataset(
        data = X
        , label = y
        , params = list(
            verbosity = 1
            , min_data_in_bin = 1
        )
    )
    dtrain$construct()
}

[1] "--- 10 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 15 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 20 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 25 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 30 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 35 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 40 ---"
[LightGBM] [Warning] There are no meaningful features, as all feature values are constant.
[1] "--- 45 ---"
[1] "--- 50 ---"
[1] "--- 55 ---"
[1] "--- 60 ---"
[1] "--- 65 ---"
[1] "--- 70 ---"
[1] "--- 75 ---"
[1] "--- 80 ---"
[1] "--- 85 ---"
[1] "--- 90 ---"
[1] "--- 95 ---"
[1] "--- 100 ---"

Environment info
LightGBM version or commit hash: v3.3.2 and latest master (https://github.com/microsoft/LightGBM/commit/8e721c564d4534820a7b7febf81382ccf247b5c9)
Command(s) you used to install LightGBM
cd python-package
python setup.py install

cd ..
sh build-cran-package.sh --no-build-vignettes
R CMD INSTALL --with-keep.source ./lightgbm_<IP_ADDRESS>.tar.gz

Additional Comments
I discovered this while attempting to answer https://stackoverflow.com/questions/71285022/why-lightgbm-python-package-gives-bad-prediction-using-for-regression-task.
I understand that supporting very small datasets is not a high priority for LightGBM, but I still think investigating this issue is valuable because:

I've observed in issues and Stack Overflow posts that users often use very small datasets to test LightGBM and improve their understanding of it
the root issue here might be one that affects larger datasets as well (it might just be less likely)

@jameslamb did you try to set "min_data=1" ?

Yes, my reproducible examples above have min_data_in_bin=1.

@jameslamb I remember they are different hyper-parameters. min_data is used to pre-filter features, and min_data_in_bin is used in bin finding.

hmmm ok, I'll look through the code.
https://lightgbm.readthedocs.io/en/latest/Parameters.html#min_data_in_leaf says that min_data is an alias for min_data_in_leaf.

I am going to investigate this today by adding more logging to the relevant BinMapper code and running the example given above.

I don't see any [LightGBM] [Warning] There are no meaningful features, as all feature values are constant. warnings with
        params={
            "verbosity": 1,
            "min_data_in_bin": 1,
            "min_data_in_leaf": 1
        }

commit: d4cdbcfe1b1db2d48e95665a8b358028c496cdb2

oh, thanks very much @StrikerRUS !
Sorry @guolinke , I misread the docs and didn't realize min_data / min_data_in_leaf are different than min_data_in_bin.
You're right, setting min_data_in_leaf=1 does resolve this issue.
from lightgbm import LGBMRegressor
from sklearn.metrics import r2_score
from sklearn.datasets import make_regression

X, y = make_regression(
    n_samples=20,
    n_informative=5,
    n_features=5,
    random_state=708
)

reg = LGBMRegressor(
    min_data_in_bin=1,
    min_data_in_leaf=1,
    num_boost_round=20,
    verbosity=1
)
reg.fit(X, y)
r2_score(y, reg.predict(X))
# 0.985

I'm going to propose a PR clarifying that error message. Even I, someone very familiar with LightGBM, was confused about why LightGBM thought the features for this dataset were "not meaningful" and "all constant".

and I'll go and that Stack Overflow post linked in the description of this bug report


Switch Companies.

Hi @Edujugon ,
Is it possible to switch companies in the get request to Odoo via the package?, e.g. If there are multiple companies in the Odoo instance, but we only wanted to get/put/delete data from company id 4? Is this viable?
Andy

Hi @andyslater99 ,
Sorry for my delay in reply..
I never tried that. have you tried with "where" method looking for that company id?