Add MVP A11yoop implementation

Description
This provides the initial A11yoop implementation with A11yoopMonitor. It defines the MVP functionality and provides an additional app preview to showcase the library's functionality.
A11oopMonitor
Have you ever wondered which accessibility features users of your app actually have enabled? Do you want to easily be able to prioritise accessibility improvements that should be made to your app based on real data from your user base?
A11yoopMonitor allows you to track which accessibility features users have enabled on their iOS devices while they use your app. It is simple to integrate into an existing app and provides an easy way of tracking status changes as users enable/disable features on their devices. It is also highly extendable, providing a simple API that allows you to do whatever you want with the data that is received.
A note about privacy...
It is worth mentioning that any data tracked by A11yoopMonitor is completely anonymous and there is no way of identifying one user from another. All that is provided is a unified API to determine accessibility feature status changes as they are occur.
Specify which accessibility features you want to monitor
When you instantiate A11yoopMonitor, you have the option to specify which accessibility features you want to monitor. These can either be chosen by you or can be omitted and all observable accessibility features will be monitored by default.
import A11yoop

final class MyClass {

    // In order to only monitor specific accessibility features (eg. voice over and larger text):
    let specificMonitor = A11yoopMonitor(featureTypes: [.voiceOver, .largerText])

    // In order to monitor all observable accessibility features:
    let defaultMonitor = A11yoopMonitor()
}

Query accessibility feature statuses
Once you have instantiated A11yoopMonitor, it is possible to query the current status of an observed accessibility feature at any point. This can be done as follows:
let monitor = A11yoopMonitor(featureTypes: [.voiceOver, ...])

// In order to query current status of voice over accessibility feature
let isEnabled = monitor.isFeatureEnabled(.voiceOver)

Note that it is only possible to query the status of accessibility features that are currently being monitored. Doing the following would therefore always result in .disabled:
let monitor = A11yoopMonitor(featureTypes: [.voiceOver]) // Only monitors voice over

let isEnabled = monitor.isFeatureEnabled(.boldText) // Query an accessibility feature that isn't monitored
/// - false

Finally, if an accessibility feature that you are monitoring is not currently supported on a user's device, the status is returned as .notSupported.
Integrate with Analytics APIs
In order to provide easy integration with third-party analytics SDK APIs, A11yoopMonitor provides a way to convert all currently monitored accessibility features into a params dictionary (of type [String: Any]), which is compatible with most analytics SDK APIs. This dictionary would contain key-value pairs mapping the feature type description to the current feature status description (eg. ["Bold Text": "Disabled"]).
This conversion is possible by using the asAnalyticsParams(prefixedBy:) adapter, which transforms a sequence of A11yFeature into a [String: Any] representation.
A sample use-case would be including accessibility feature statuses in a tracking event:
func sendEvent() {

    let params = self.monitor.features.asAnalyticsParams()
    let eventName = "A11yoop features"

    // Firebase
    Analytics.logEvent(eventName, parameters: params)
}

In order to prevent conflicts, it is also possible to pass a prefix, which will be applied to all parameter keys (eg. resulting in "A11yoop Bold Text" with a prefix applied rather than "Bold Text" without).
Extend functionality with custom emitters
It is possible to write custom emitters in order to react to accessibility feature status changes in whatever way you want.
For example, if you wanted to send individual analytics events specific to each accessibility feature (rather than using asAnalyticsParams(prefixedBy:)), then the best way to achieve this is to write your own custom emitter.
This can be done by extending A11yStatusEmitter and creating your own static configuration that can then be included in your A11yoopMonitor(featureTypes:emitters:) initialisation:

extension A11yStatusEmitter {

    static var analyticsEventEmitter: Self {
        Self { feature in
            // The feature argument is the updated feature. Do with it whatever you want...

            Analytics.logEvent(
                "A11yoop Feature Updated",
                parameters: [feature.type.description: feature.status.description])
        }
    }
}

final class MyClass {

    // Include the custom emitter in your `A11yoopMonitor` initialisation
    let monitor = A11yoopMonitor(emitters: [.analyticsEventEmitter, .log())
}

Codecov Report

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

:exclamation: Current head 59f5494 differs from pull request most recent head 058a26c. Consider uploading reports for the commit 058a26c to get more accurate results

@@           Coverage Diff            @@
##             main        #7   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        16
  Lines           ?       234
  Branches        ?         0
========================================
  Hits            ?       234
  Misses          ?         0
  Partials        ?         0

Continue to review full report at Codecov.

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


Minor edit in README.md

Added a reference to the user Tosuma's profile.

This is very thoughtful of you, thank you


Manifest compatibility question

👍 Great job on this plugin!!
We are publishing seeAlso for each canvas with Alto xml available.
Example: https://purl.stanford.edu/fg165hz3589/iiif/manifest
{
          "@id": "https://purl.stanford.edu/fg165hz3589/iiif/canvas/fg165hz3589_1",
          "@type": "sc:Canvas",
          "label": "Page 1",
          "height": 3279,
          "width": 2530,
          "rendering": [
            {
              "@id": "https://stacks.stanford.edu/file/fg165hz3589/fg165hz3589_1.jp2",
              "label": "Original source file (1.49 MB)",
              "format": "image/jp2"
            }
          ],
          "seeAlso": [
            {
              "@id": "https://stacks.stanford.edu/file/fg165hz3589/fg165hz3589_1.xml",
              "label": "OCR text",
              "format": "application/xml",
              "profile": "http://www.loc.gov/standards/alto/ns-v2#"
            }
          ],
          "images": [
            {
              "@id": "https://purl.stanford.edu/fg165hz3589/iiif/annotation/fg165hz3589_1",
              "@type": "oa:Annotation",
              "motivation": "sc:painting",
              "resource": {
                "@id": "https://stacks.stanford.edu/image/iiif/fg165hz3589%2Ffg165hz3589_1/full/full/0/default.jpg",
                "@type": "dctypes:Image",
                "format": "image/jpeg",
                "height": 3279,
                "width": 2530,
                "service": {
                  "@context": "http://iiif.io/api/image/2/context.json",
                  "@id": "https://stacks.stanford.edu/image/iiif/fg165hz3589%2Ffg165hz3589_1",
                  "profile": "http://iiif.io/api/image/2/level2.json"
                }
              },
              "on": "https://purl.stanford.edu/fg165hz3589/iiif/canvas/fg165hz3589_1"
            }
          ]
        },

Based on the requirements, I would expect this to work. Is there something that I am overlooking? https://github.com/dbmdz/mirador-textoverlay#requirements-for-supported-iiif-manifests
Thanks!

Thanks for pointing that out! It's a bug, I wasn't considering multi-valued seeAlso values :roll_eyes:  Fix should be ready in a few minutes!


Launching the app and putting it to background in quick succession leaves the app in splash fragment

Pressing home or recents button as soon as app starts and then later opening the app again leaves the app stuck in splash fragment

Moving https://github.com/Zhuinden/jetpack-navigation-ftue-sample/blob/3ffdd6d695471ca7b2d99d510fd146f74eabce84/app/src/main/java/com/zhuinden/jetpacknavigationdaggersavedstatehandleftueexperiment/features/splash/SplashFragment.kt#L43 to onResume seems like a fix for this issue

WOW. Makes sense, they are ignoring the navigation event after onStateSaved().
Thanks I hate it XD i have to move it to onStart/onStop, then it would work.
I'm used to simple-stack enqueueing this instead of consuming it. Will fix in a moment, thanks for the notice


ng g universal generate wrong path in tsconfig.server.json

Versions
Angular CLI: 6.0.0-rc.5
Node: 8.11.1
OS: darwin x64
Angular: 6.0.0-rc.5
... animations, cli, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, platform-server, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.5.7
@angular-devkit/build-angular     0.5.7
@angular-devkit/build-optimizer   0.5.7
@angular-devkit/core              0.5.7
@angular-devkit/schematics        0.5.7
@ngtools/json-schema              1.1.0
@ngtools/webpack                  6.0.0-rc.5
@schematics/angular               0.5.7
@schematics/update                0.5.7
rxjs                              6.0.0-uncanny-rc.7
typescript                        2.7.2
webpack                           4.5.0

Repro steps

Step 1

ng new my-app

Step 2

ng g universal --client-project

Observed behavior
        "server": {
          "builder": "@angular-devkit/build-angular:server",
          "options": {
            "outputPath": "dist/my-app-server",
            "main": "projects/my-app/src/main.server.ts",
            "tsConfig": "projects/my-app/tsconfig.server.json"
          }

Desired behavior
        "server": {
          "builder": "@angular-devkit/build-angular:server",
          "options": {
            "outputPath": "dist/my-app-server",
            "main": "src/main.server.ts",
            "tsConfig": "src//tsconfig.server.json"
          }

Mention any other details that might be useful (optional)

I observed the same thing

Hi @thanatos915 ,
Thanks for reporting this issue, seems it seems to be a duplicate of #10288.

Please follow #10288.


[WIP] txt-preview: improvments

:heart: Thank you for your contribution!
Description

Fixes horizontal scrolling in preview iframe
Adds option to truncate .txt preview after a specified number of bytes

Fixes invenio-app-rdm#1864
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[x] I'm aware of the code of conduct.
[x] I've created logical separate commits and followed the commit message format.
[x] I've added relevant test cases.
[x] I've added relevant documentation.
[x] I've marked translation strings (for relevant code).
[x] I've followed the CSS/JS and React guidelines (for relevant code).
[x] I've followed the web accessibility guidelines (for relevant code).
[x] I've followed the user interface guidelines (for relevant code).
[x] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[x] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Thanks @fenekku, good thinking on the possible multi-byte errors.
I've added some tests but it looks like test_macros.py fails all tests anyway :( Tom has demonstrated it works however.
It would be good if we could merge this and @tmorrell's #179 into a 1.3.10 release (as well as push these changes forwards into 1.4) - so there is a painless upgrade option for existing installs.

Looping in Karolina from CERN. Once this one and Tom's one are merged let do a release and backport to 1.3.10 like Dan mentioned.


[robots] robots.txt is not being generated in production

Describe the bug
Per the docs (https://nuxtseo.com/nuxt-seo/guides/site-indexing), it is supposed to index the site by default, but there is no robots.txt being generated in production.
In dev, there is a robots.txt file that looks like this:
# START nuxt-simple-robots (indexing disabled)
User-agent: *
Disallow: /

# Dev mode  - View the indexable output by setting ?indexable

# END nuxt-simple-robots

Do I need to do something to enable it in production?
Reproduction
No response
System / Nuxt Info
- Operating System: Windows_NT
- Node Version:     v18.17.1
- Nuxt Version:     3.8.0
- CLI Version:      3.9.1
- Nitro Version:    2.7.0
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      css, build, vite, nitro, modules, extends, sanity, site, ogImage, gtag, imports, runtimeConfig
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -

Hey @davidstackio, thanks for the issue but you'll need to provide some reproduction details.
Afaik it should always generate a robots.txt in all environments (if the module is installed and enabled). The only difference will be whether or not the site is indexed.

@harlan-zw Sure, here's some additional context. Please let me know if anything else would be helpful!
Live site: https://climatelife.org/
It's hosted using Firebase Functions v2.

Thanks, this isn't that helpful though.
I need to know steps I can take to try and reproduce the problem you're having (no robots being generated). I feel like it may be a firebase issue with how the static files are being ommited though.
Can you confirm how you deploy your site (nuxt build or nuxt generate) and your robots config

I deploy the site using SSR to Firebase with nuxt build then firebase deploy.
I have no robots config beyond what is provided by default with nuxt-seo-kit (am I supposed to have a config?).
Relevant parts of nuxt.config.ts:
nitro: {
  preset: "firebase",
  firebase: {
    gen: 2,
    nodeVersion: "18",
  },
},
app: {
  head: {
    titleTemplate: "%pageTitle",
  },
},

// SEO
site: {
  url: "https://climatelife.org",
  name: "Climate Life",
  description:
    "Take a 3 minute quiz and discover your personalized climate change solutions. Then take the actions that work for you and your life.",
  defaultLocale: "en",
},
ogImage: {
  enabled: false,
},

FYI The sitemap.xml file is generated as expected, though I think that may be done on the fly?


Add Error Enum for Parent App ID Retrieval

Purpose

Add the following error enum which represent server error while retrieving the parent app ID of a given shared app.

ERROR_CODE_ERROR_GETTING_PARENT_APP_ID

Closing this issue since the implementation changed for retrieving parent app ID for a given child application.


Fix issue with trimming path separator '/' for buffers()

Follow up of https://github.com/nvim-lua/telescope.nvim/pull/104 that introduced a bug in which if vim.fn.getcwd() returns a string with a trailing path separator /, it causes the bufname path to be wrongly trimmed (e.g. $(pwd)/foo/bar -> /foo/bar instead of $(pwd)/foo/bar -> foo/bar).
This patch just checks the cwd trailing char and applies an offset.
@tjdevries I'm noticing that so far working with paths in lua has been quite hacky which in turn makes the appearence of these bugs easier. Is there a proper way to work with paths?
I have looked at plenary.nvim , and more concretely at  path.lua which according to the projects README it's suppose to be the equivalent of python's pathlib. But from looking at that file, it's still far behind which leaves me for now with manual string handling.

should probably use os_separator, which we've got elsewhere in telescope.
Yes, we should probably be using the path.lua from plenary. It is in need of some more features :laughing:
I also maintain plenary, so if you want to add any features to make path handling easier here, let me know.

( I just added os sep fix. If you're interested, we can work on improving plenary's path module, and then use it here )

( I just added os sep fix. If you're interested, we can work on improving plenary's path module, and then use it here )

Thanks, it was in my TODO but it was just a busy week and it got buried in my GH notifications. I am interested, will look into it and start making some PR's that implement pathlib equivalents.

Also, we should use libuvs stuff for access file system, not  viml wrappers like I had before. Can explain more later.


(cargo-release) version 1.1.0

Codecov Report

Merging #82 (24dfb22) into main (f85e556) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #82   +/-   ##
=======================================
  Coverage   91.80%   91.80%
=======================================
  Files          17       17
  Lines        1440     1440
=======================================
  Hits         1322     1322
  Misses        118      118

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


Troubleshoot cors

Coverage increased (+0.1%) to 84.517% when pulling 78d50cef3516347017be92ab5bb41ea41fa4a586 on troubleshoot-cors into e4f694d3e416e2750a7c4b8f1594cd9d50927823 on master.


[REQUEST] Email block

Hope in future OP have this block, can get mail from imap because TCP block work not good enough. Thank you

What do you mean with "email block"? Do you want to be able to send emails?

search and fetch email, TCP block have many problem with buffer, if we search email and get too much mail id, OP debugger can't show all of them, the fetch mail too, that will make the config can't work correctly ( like this https://imgur.com/a/wcCAepL and https://imgur.com/a/qyDU3UQ ). And fetch email with imap, some imap servers we can show text but some imap servers they just response https://imgur.com/a/O9AuwZa I told with you on the forums and get reply https://imgur.com/a/TcoAzgp I just make a note to remember.

@thanhlouis Can you please send me that config Trying to learn more about TCP Blocks thanks send on discord or telegram INFINITEY#6622 @INFINITEY_1


Add console for debugging

This PR aims to provide a simple way to direct debug output to the TFT display.
It's an ideal application for hardware scrolling.
Requires feature/hardware-scrolling branch of HardwareSPI.
todo:

[x] Implement Priint class which handles printing text to display with appropiate scrolling
* [ ] Implement back-buffer so display can be scrolled up/down, e.g. using touch input
[x] Add pause function so rapid output can be suspended for reading

@slaff I've added a Console class and hardware scrolling support to the Graphics library, and updated this PR to demonstrate its use. Would you give it a try out?

Would you give it a try out?

Works great! Thank you @mikee47.

@mikee47 Can you merge feature/hardware-scrolling into your develop branch and create a PR with the latest HardwareSPI version for Sming?

@slaff HardwareSPI develop was added in 2a703c8d31555d43e4691dc907a15b2023399d9b.

Note that scrolling only works in full-screen. I'm developing it further to support scrolling within a fixed header/footer but that's more involved.


TICKETGRANTINGTICKET table is getting "Locked" CAS 4.1.2

Hi ,
The TICKETGRANTINGTICKET table is getting "Locked" when user tries to login. It does not happen all the time but the frequency is high .We are using the CAS of version 4.1.2 which runs on Tomcat server and we use JPA to connect to the database which is in mysql.The issue happens only on Production environment and we are not able to reproduce it in any other environment.
Caused by: org.springframework.dao.CannotAcquireLockException: could not execute batch; SQL [update TICKETGRANTINGTICKET set NUMBER_OF_TIMES_USED=?, CREATION_TIME=?, EXPIRATION_POLICY=?, LAST_TIME_USED=?, PREVIOUS_LAST_TIME_USED=?, ticketGrantingTicket_ID=?, AUTHENTICATION=?, EXPIRED=?, PROXIED_BY=?, SERVICES_GRANTED_ACCESS_TO=?, SUPPLEMENTAL_AUTHENTICATIONS=? where ID=?]; nested exception is org.hibernate.exception.LockAcquisitionException: could not execute batch
Please treat this as Urgent as we are facing this issue in Production Environment and we do not have any other job running which is using the table. In fact we configured it as clustered environment ,but to debug the issue we disabled the cluster with only one node running and still we have the same issue.
I have attached the application log and mysql lock for more details
Please help us out.
mysql_lock.txt
error.txt

@frett has applied a number of fixes to the 4.1.x release line. Please upgrade and see if the fixes apply to you. You should be able to adjust the locking level on the registry to avoid this sort of problem.
The CAS project does not provide any sort of SLA when dealing with issues. Please contact the mailing lists or a commercial support provider if you need that type of support.


Added theme folder to ignore

Description
This PR adds customer themes to the git ignore list. The resulting behaviour is the same as for customer plugins. It allows to symlink your own theme into the installation whithout having uncommited files.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
-

How to test?
create a custom theme and do a git status

Thanks :+1:


Upgrade socket.io to latest version

The latest version of socket.io is (as of today) v4.4.1. To keep the extension up to date, we would require to update the socket.io.js  file in Source/
Steps

Get the latest socket.io.min.js from here
Rename it to socket.io.js
Replace it with the old one in Source/socket.io.js
TEST that everything is still working for all example projects!
(optionally) create the extension file (.yymps) within the GameMaker IDE and create a new release / or just attach it here and I'll take care of it

Sadly I don't have time to do it by myself, any help is appreciated

I had a GM project connect to my node server successfully, no errors. And then a bit later I was getting errors with (afaik) no changes to any code.
The error message is Unhandled Exception - Uncaught ReferenceError: gml_Script_gmcallback_sio_on_connect is not defined in file http://localhost:51264/html5game/tph_gms2.api.js at line 33
When I go to this URL that function is definitely in there (however my GM is empty of any socket.io related scripts, only the Extension exists)
Is this related to the v4.4.1 upgrade and the gms2.api being out of date? Am I missing the GM script-resource?
Appreciate any help and I'm willing to help in any way I can.

builds are automated now


Convert the remaining visual MainGameScreen parts to NUI

Description
This pull request converts all of remaining the visual functionality of MainGameScreen to use NUI instead. This does not cover auxiliary classes, such as BorderDrawer and ZoneNameAnnouncer but does include WarnDrawer. Most of the not covered classes render rotated images, which is not supported under NUI (and so use UiDrawer directly instead).
The original MainGameScreen class remains for now, due to other dependencies in the code that rely on it, such as input handling.
This pull request also includes a bug-fix for retrieving item examples in ItemManager.
Testing
Warnings

Start a new game or continue and existing one (Imperial Small is recommended for this)

Armour

Open your inventory and un-equip your armour.
A "No Armor" warning should be displayed in orange.

Shield

Open your inventory again and un-equip your shield this time.
A "No Shield" warning should be displayed in orange.

Health

Reduce your health to below 5 or lower (the exact value depends on the ship chosen).
A "Heavily Damaged" warning should be displayed in red.
Increase your health back to 5 or above (again, exact value depends on the ship chosen).
The "Heavily Damaged" warning should fade away.

Enemies

Shoot at your starting station until it becomes hostile
A "Dangerous Enemy" warning should appear in orange.

Stats Bars

Start a new game with the "Loaded Imperial Large" ship

Health Bar

Ram your ship into the starting station.
You should see the value in your health bar go down.

Shield Bar

Fire on the starting station until it becomes hostile.
When the starting station shoots your ship, the value in your shield bar should go down.

Ammunition Bar

The icons on the ammunition bars should correspond to the weapons you have equipped.
Fire your left gun.
You should see the value of the first ammunition bar go down.
Fire your right gun.
You should see the value of the second ammunition bar go down.
Continuously fire your left gun until it runs out of ammunition.
You should see an hour-glass icon appear in the ammunition bar, which will steadily increase until it is full again.

Miscellaneous

When flying near a planet, you should see some dots in the corner, which roughly indicate the direction towards the planet.
Then flying on a planet, you should see a compass (I don't know what it does - points towards true north?).

Potential Future Improvements

The UIWarnDrawer notifications are not the most visually pleasant. Better placement and design is needed but for now I've tried to retain most lf the original appearance and behaviour.
UIWarnDrawer notifications start from the middle of the screen on Android, which shouldn't really be the case.
There are gaps between the status bars in the top-left of the HUD where certain elements are absent. The FlowLayout explicitly does not ignore empty elements, which is why spacing is still applied.

Breaking Changes

The original GameScreens#mainGameScreen variable has been re-named to GameScreens#oldMainGameScreen.
GameScreens#mainGameScreen now points to the NUI version of MainGameScreen instead.

Notes

This depends on #636 and #638.
You can view a clearer diff of the changes at https://github.com/BenjaminAmos/DestinationSol/compare/nui-game-menu...more-nui-main-game-screen.

This was replacewd by #665.


qtags

command to list all existing tags so people have some reference as to what quotes they can load

added in f4e479d82e7616f82d08425db3fbbe8da43f4948


synchronize ecs-logging spec

What
ECS logging specs automatic sync
Why
Changeset

https://github.com/elastic/ecs-logging/commit/d3ffcf1 Adding service.environment to the spec (https://github.com/elastic/ecs-logging/pull/62)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-05-16T12:01:29.969+0000

Duration: 3 min 30 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
13

Skipped
0

Total
13

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Suggested improvements to the Ibex revoker

Currently, the Ibex's revoker has a single control register, which contains the epoch and the running status.  There are a few things that would make this more useful:

[ ] Making the epoch 32 bits and extending it.  We currently wrap the revoker with software that fakes a 32-bit epoch.
[ ] Reporting the running state in the epoch.  The Flute version uses the low bit of the epoch to indicate running status, which lets us use arithmetic for a few things in software that would otherwise require branches.
[ ] Making the response to writing to the 'start' register stateless.  With the current revoker, we have to read the status then compare and then sometimes write back the response.  With the Flute version, we just write to the 'go' register: if revocation is already running, this does nothing.
[ ] Providing an interrupt when revocation has finished.  The allocator currently has to poll for completion of a revocation pass, this would let threads that are blocked on revocation sleep.

The memory mapped register block is actually implemented outside of the ibex core (similar to clic). So we will probably implement the suggested changes when make the arty a7 FPGA release.

Also - note that if we make the "start" bit stateless, firmware will need to clear the bit before the hardware state machine completes, otherwise the state machine will immediately restart after it completes.

On Flute, it isn’t really a register, it’s an edge-triggered signal. Writing to it starts the operation, but reading from it doesn’t do anything.

Ok, I am making it a single-cycle pulse triggered by writing a '1' to the bit location. The pulse will kick off the hardware revoker if it is currently idle, and if the revoker is running the pulse will be ignored

@davidchisnall, shoudl we close this one?

I believe all of these are now implemented.


Upgrade the transport for prometheus metrics collection

The current form of using graphite by default even when sending data to prometheus is not robust. It creates additional complexity for both the prometheus and the grafana integrations. Labels/Tags are necessary semantic features at the protocol and API level in order to simplify and stabilize the metrics reporting interface going forward.
In any case, the method should be possible to automate for push-button standup of metrics collection and dashboarding. This should include both the locally hosted topology as well as hub-spoke or other topologies which may be used in higher scale scenarios. The commands to configure any of these topologies should be simple and obvious.
One of two possible approaches can be used:
"push"
Add a stub listener to a prometheus docker so that the batch interface to prometheus is exposed as a simple push interface, protected by a simple API key mechanism. This means that the test client is allowed to control the timing and provenance of data as it should, given that specific timing semantics are used to enable metrics visibility for advanced analysis methods. The trade-off is that the "last data point" in prometheus will hang around, but, at least for nosqlbench, this doesn't seem particularly different than the usually preferred way of having prometheus drive all data collection.
"poll"
Add a prometheus target service on the nosqlbench client. This would require the nosqlbench client to publish a web service which hosts metrics while it is running. This is closer to the preferred way of providing data sources for prometheus, but it takes control away from users who want to establish their own update frequency for near-time visibility. It also means that updates to a topology for additional data sources would require active reconfiguration of the prometheus instance. Getting around this would require yet another change to the prometheus docker to allow for an external configuration service.

A requirement for discretely labeled data on short sampling windows has bumped this up in priority. Specifically, the ability to sample short time windows and label them accurately with specific properties will enable a new type of performance study.
Recent suggestions and research has yielded a better design for this:

[ ] Adapt metrics to use label sets internally.
[ ] Ensure metrics are labeled with the previous hierarchic values for scenario, activity, etc.
[ ] Bundle victoria metrics orchestration rules into the docker helper.
[ ] Implement an openmetrics push reporter.
[ ] Rewire NBCLI logic to use the victoria metrics path with a client-push reporter and full label support.
[ ] Re-configure [a new] grafana dashboard to use the more direct and simpler labeling scheme.
[ ] Update docs on how to use the new dashboard, what the labels mean, and how to customize them per scenario.


[SPARK-24768][SQL] Have a built-in AVRO data source implementation

What changes were proposed in this pull request?
Apache Avro (https://avro.apache.org) is a popular data serialization format. It is widely used in the Spark and Hadoop ecosystem, especially for Kafka-based data pipelines.  Using the external package https://github.com/databricks/spark-avro, Spark SQL can read and write the avro data. Making spark-Avro built-in can provide a better experience for first-time users of Spark SQL and structured streaming. We expect the built-in Avro data source can further improve the adoption of structured streaming.
The proposal is to inline code from spark-avro package (https://github.com/databricks/spark-avro). The target release is Spark 2.4.
Built-in AVRO Data Source In Spark 2.4.pdf
How was this patch tested?
Unit test

Test build #92824 has started for PR 21742 at commit 16d08af.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/825/
Test PASSed.

Test build #92824 has finished for PR 21742 at commit 16d08af.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  class SerializableConfiguration(@transient var value: Configuration)
  class IncompatibleSchemaException(msg: String, ex: Throwable = null) extends Exception(msg, ex)
  case class SchemaType(dataType: DataType, nullable: Boolean)
 implicit class AvroDataFrameWriter[T](writer: DataFrameWriter[T])
 implicit class AvroDataFrameReader(reader: DataFrameReader)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92824/
Test PASSed.

Test build #92848 has started for PR 21742 at commit 7a45457.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/840/
Test PASSed.

Test build #92854 has finished for PR 21742 at commit 057c3a4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92854/
Test PASSed.

ping @marmbrus @tdas
Should we create API .avro in Spark's DataFrameReader and DataFrameWriter? If so, should we should expose .kafka in DataFrameReader/DataFrameWriter as well?

since we decided to follow sql-kafka here, I think we should not add .avro at the beginning, and decide it later with .kafka together.

Test build #92937 has started for PR 21742 at commit 3979bad.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/899/
Test PASSed.

Test build #92937 has finished for PR 21742 at commit 3979bad.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92937/
Test PASSed.

Merged build finished. Test PASSed.

LGTM
Thanks! Merged to master.
@gengliangwang Please submit the follow-up PRs to resolve the sub tasks and the good comment in the PR review.


Prepare v0.14

@tomhoule Would love to releas v0.14 if that’s okay with you

Finally published v0.14!


Added Github Profiles Android App.

Added the Android Application inside the 'Android projects' directory.

Please add the readme as mentioned in the issue.

@ashishpatel16 the files are not seen in the PR so I was not able to review it, please make the Pull request again with the readme.


Parse Longs only once

~First (baby) step toward removing BigDecimal as a mandatory intermediate when converting from json to GraphQL Int and Double: Remove the default adapters for Float and Long.~
Edit: After some investigation, this pull request keeps the DEFAULT_ADAPTERS for now as that would be a breaking change. It still optimizes the parsing of Longs. There is no need to parse from String again since BufferedSourceJsonReader already parses internally.
Follow up issue about removing DEFAULT_ADAPTERS there: https://github.com/apollographql/apollo-android/issues/2486

I have a question regarding Long, looking into the code of our json reader seems like we read long values https://github.com/apollographql/apollo-android/blob/master/apollo-api/src/commonMain/kotlin/com/apollographql/apollo/api/internal/json/BufferedSourceJsonReader.kt#L545

I have a question regarding Long, looking into the code of our json reader seems like we can read long values

It can indeed but it's unused in the default GraphQL type system. It could be useful in custom scalar type adapters but they should be registered manually and not put in DEFAULT_ADAPTERS?
More generally, what do we need DEFAULT_ADAPTERS for? All the builtin GraphQL types are handled in a different path (readString, readInt, etc...). The only case I could find for DEFAULT_ADAPTERS is to handle the ID
scalar type but that's one in many default adapters.

I added a couple of benchmarks. It turns out that going from Int -> Double -> Int is slower than Int -> BigDecimal -> Int. So ideally, we should avoid using intermediate representations, which make sense.

And I just pushed an optimization that divides by ~2 the time spent on Int parsing. The code that we had from Moshi already parses the Longs as it scans the Json and we were doing it a second time just after.

More generally, what do we need DEFAULT_ADAPTERS for?

They provide more convenience for user, when for instance some custom scalar type mapped to java.lang.Long there is no need to provide adapter for it. Same for Float.
Is there any reason to remove them?

They provide more convenience for user, when for instance some custom scalar type mapped to java.lang.Long there is no need to provide adapter for it. Same for Float.

Got it 👍 . Yea, removing these adapters will be a breaking change for users that rely on them.

Is there any reason to remove them?

There's an argument to be made for API consistency. Some adapters like java.util.Date currently have to be manually added using addCustomTypeAdapter while others like com.apollographql.apollo.api.FileUpload are registered automatically.
We could instead declare the implementations and let the user register them as needed:
// build.gradle.kts
apollo {
   customTypeMapping.set(mapOf("TimeStamp" to "java.lang.Long"))
}

// Apollo.kt
apolloClient = ApolloClient.builder().addCustomTypeAdapter(
                                   CustomType.TimeStamp,
                                   ApolloCustomTypeAdapter.LongAdapter
                             )

We could also add Iso8601DateAdapter, I think this is one of the first custom scalars that users encounter.

@sav007 I restored the default adapters for now.  Follow up issue there: https://github.com/apollographql/apollo-android/issues/2486


"mail filter" wording is not correct here.. its causing confusion

[Enter feedback here]
Document including below lines;
"This issue may occur if the packets are dropped during high traffic bursts. Ensure that there is no traffic filtering occurring (for example, with a mail filter). After eliminating this possibility, gradually increase the number of buffers in the guest operating system and verify."
Here we are telling that; MAIL FILTER can cause a package DISCARDs on network layer. This is not technically possible. On network layer a package is just a package never a SMTP nor HTTP package. It's low level thing yet and SMTP/MAIL filtering software's does not understand package meaning. Also mail filtering software's does not work on network layer, they works on application layer and look at traffic as SMTP protocol traffic. So please update above lines like below;
"This issue may occur if the packets are dropped during high traffic bursts. Ensure that there is no traffic filtering occurring (for example, with a package filter). After eliminating this possibility, gradually increase the number of buffers in the guest operating system and verify."

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

ID: 2144e7f4-708a-c9c9-04e9-48631a8448d3
Version Independent ID: 7acefe4f-73d9-95d1-d2ed-67b96d509355
Content: Troubleshooting cluster issue with Event ID 1135
Content Source: WindowsServerDocs/troubleshoot/troubleshooting-cluster-event-id-1135.md
Product: windows-server
Technology: troubleshoot
GitHub Login: @Deland-Han
Microsoft Alias: delhan

So please update above lines like below;
"This issue may occur if the packets are dropped during high traffic bursts. Ensure that there is no traffic filtering occurring (for example, with a package filter). After eliminating this possibility, gradually increase the number of buffers in the guest operating system and verify."

Fair point. I will open a Pull Request to ask for this correction to be made. Thank you for contributing.

For other contributors who want to go directly to the referenced paragraph, here is the anchor link:

https://docs.microsoft.com/en-us/windows-server/troubleshoot/troubleshooting-cluster-event-id-1135#cluster-installed-in-the-vmware-virtualization-platform

Derp. I was a bit trigger-happy there and pushed my first commit before checking that all my documented changes were present. Found at least one missing, so I start again with a new commit. BRB.

Anyway, at this point in time, I am more than tired of finding issues to fix in that document. Some of them I can't even fix myself.
 PR #5159 is now open for review, comments, and suggestions.


The 'function' keyword is not included in syntax highlighting

the 'function' keyword for pattern matching seems to be forgotten  :)

Fixed in 0.12.10


doc: update FAQ with correct AQA custom target parameter

Checklist

[x] commit message has one of the standard prefixes
[x] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

/thaw


Improve version management & avoid conflict between multiple version

Purpose
To make it's possible to have multiple versions of rust-mbedtls in the dependency tree.
The following works need to be done:
Able to compile together:

[x] Add namespace for mbedtls_malloc to avoid conflicts. #243
[x] Move self test logic (including exclusive C function definitions and exported rust functions) to a separate crate #244

Better version management:

[x] Publish first version of mbedtls-selftest, done by #250
[x] Back-port all above changes to https://github.com/fortanix/rust-mbedtls/tree/v0.7, done by #251
[ ] publish new version 0.7.3
[ ] Back-port all above changes to https://github.com/fortanix/rust-mbedtls/tree/v0.8, #252
[ ] publish new version 0.8.2

Just realize that there also a lot of  rust functions and function pointers are exposed  in lib.rs and threading.rs.
They will also  cause conflict, those functions could be solved by following same path of self test but the function pointers might be difficult to deal with.

Yeah I guess we could make another crate mbedtls-platform-support. Especially for lib.rs I'm not sure if that will work properly for 0.7, because the features & conditional build logic supporting this changed pretty significantly in 0.8. I don't think the function pointers are a big deal, should work the same way?

I'm not sure if that will work properly for 0.7

If it won't, then I think we should give up on this for now, possibly revert the mbedtls-selftest changes as well, and just do a flag day migration across all our projects.

Got it, let me try if it's possible to split the code in 0.7


Converting an IPv4 mapped to IPv4 format

I've been spelunking the unit tests to understand what would be the most elegant way to convert and addr ::ffff:<IP_ADDRESS> to its IPv4 counter part. Thoughts?
Thank you :)

Try the new tov4 function in the just-released 4.1.0. :)
You can use new v6.Address('::ffff:<IP_ADDRESS>').tov4().correctForm() to get <IP_ADDRESS>.


Removed InternalLogger static constructor and added extension method SetupFromEnvironmentVariables

Added LogFactory.Setup().SetupInternalLogger(b => b.SetupFromEnvironmentVariables())
Trying to resolve #4032 and reduce overhead from NLog initialization.

Updated wiki: https://github.com/NLog/NLog/wiki/Internal-Logging about no longer checking environment variables at startup.


PHPSemVerChecker\Comparator\Type::get does not support PhpParser\Node\UnionType

PHPSemVerChecker\Comparator\Type::get method does not check for instanceof PhpParser\Node\UnionType.  toString method is not present on PhpParser\Node\UnionType or any other ComplexType, hence why a change was made here to specifically support NullableType.  This should be updated to support all complex types introduced by PhpParser.

Are you going to reference the commit that fixes this issue?  How dead is this project on a scale of 1-10?


having issue while compiling the project

java: cannot access com.shaft.gui.browser.BrowserActions
bad class file: /C:/Users/KM/.m2/repository/io/github/mohabmohie/SHAFT_ENGINE/5.1.20210219/SHAFT_ENGINE-5.1.20210219.jar!/com/shaft/gui/browser/BrowserActions.class
class file has wrong version 59.0, should be 52.0
Please remove or make sure it appears in the correct subdirectory of the classpath.

Please ensure that you have the latest JDK 15 configured properly.
Here's a link to a similar issue and how to solve it from stack overflow:
https://stackoverflow.com/questions/28180915/class-file-has-wrong-version-52-0-should-be-50-0


Use JSDoc for docs generation when it supports es6 features

...like spread operator and fat arrows.

https://esdoc.org/tags.html#spread


Turn off the built-in syntax highlighting feature?

Please describe the problem you're trying to solve
I want the preview to be in black and white only
But there doesn't seem to be an option to control this behavior
I'm trying to use a simple .tmTheme file to override the colors, but it seems like I can only use #rrggbb and not the 16 colors of the terminal
Also, there seems to be no option to control the cursor shape. I want to use a block cursor instead of a bar cursor even in insert mode.
Would you be willing to contribute this feature?

[x] Yes, I'll give it a shot

Describe the solution you'd like
add option that turns off syntax highlighting
add option that controls cursor shape
Additional context
theme_preview.tmTheme
<?xml version="1.0" encoding="UTF-8"?>
<plist version="1.0">
<dict>
	<key>name</key>
	<string></string>
	<key>settings</key>
	<array>
		<dict>
			<key>settings</key>
			<dict>
				<key>background</key>
				<string></string>
				<key>foreground</key>
				<string>#616161</string>
			</dict>
		</dict>
	</array>
	<key>uuid</key>
	<string></string>
</dict>
</plist>

No response

Why do you want to disable highlighting? In what situations would you need to do this? If you're worried about speed, Yazi's code highlighting is almost as fast as plain text preview, so this shouldn't be a factor to consider.
If you really don't want it, you can easily achieve this by customizing the previewer. You can refer to the code in file.lua, just replace the file command with cat, or use Lua's built-in io.open() to read files for cross-platform compatibility.
See the previewer doc at: https://yazi-rs.github.io/docs/plugins/overview#previewer and https://yazi-rs.github.io/docs/configuration/yazi#plugin.previewers

Also, there seems to be no option to control the cursor shape

This is by design; disabling it isn't supported because the cursor shape indicates the current mode (normal mode/insert mode). Without it, I'm not sure how users would know which mode they are in.

Thank you for your quick and patient response!
I simply don't like too many colors, too many colors can be overwhelming.
As for the cursor shape, since I don't use normal mode, I don't need to distinguish between the two.
Thank you for creating such a great software!
I am migrating from lf and feel that yazi is more stable and fast👍


Parse from/where/select/new

Caveat: current using 'p.x' doesn't work an a refinement expression, so 'where p.x < foo' won't work. After this is landed, I need to add in the scope/type propagation logic so that refiner.ts is aware what the scope types are, then I can add in the scope lookup syntax.

/gcbrun


Add progression events

Hey,
Thanks for your work on this project.
I wonder how I can add an event to show how much time left to the end of the process.
Thanks
Tzelon

Hello,
Currently this is not possible. But this is a good idea, probably for a future version.
Thanks

Is that demand tinkering with the C code?

For a real data yes.

Thanks.
I'm hoping to see that implemented in the near version :)

Work In Progress 133610050f35e7f8446d238a9b179acd6e1772b8

Now available with the version 2.0.0

Better performance
Async
Get percentage with callback


fix build error for nightly

SUMMARY:
To fix following error:
$ make build
python3 setup.py sdist bdist_wheel
Extracting version info from /home/dhuang/llm-compressor/src/llmcompressor
Traceback (most recent call last):
File "/home/dhuang/llm-compressor/setup.py", line 13, in
version_info = extract_version_info(package_path)
File "/home/dhuang/llm-compressor/utils/version_extractor.py", line 38, in extract_version_info
exec(open(version_path).read(), globals(), locals_dict)
File "", line 39, in
File "", line 25, in _generate_version_attributes
NameError: name 'datetime' is not defined
make: *** [Makefile:38: build] Error 1
TEST PLAN:
workflow to automate nightly build

This LGTM. Have we confirmed this works for nightly and release builds?

yep, manually verified and it works for nightly and release.


Fixed #21379 -- Added support for Unicode usernames

Suggested edits for the first commit http://dpaste.com/3G2DXVH, otherwise LGTM.


⚠ Jenkins Plugins Artifactory has degraded performance

In 84aeaf5, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1116 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 3a65dbb after 30 minutes.


Write rights?

https://github.com/sbt/setup-sbt/settings/actions

Nice. we have a PR - https://github.com/sbt/setup-sbt/pull/21


Adding multiple owners to a Winnitron

I noticed the Winnitron 1000 has 2 owners, do we have the functionality to do that as well?

The data model supports it but there isn't a UI for it (yet).


test-branch/execute-job/quoted-authorization/5ac49612-d7df-481d-a471-990f35cd9740

PR for testing 52340b4315d9fb3f9caebde018f043634a74b828

/canonical/self-hosted-runners/run-workflows 52340b4315d9fb3f9caebde018f043634a74b828


Retrieving from ICOSCP - multiple datasets match criteria

What happened?
I've been retrieving data from the ICOS CP, for example:
wao_data_level2 = retrieve_atmospheric(site="WAO", species="CH4", data_level=2)
The problem is that there are three L2 CH4 datasets for WAO on the CP (all with the same inlet height) - the ICOS dataset, the InGOS dataset and the OBSPACK-Europe dataset. The code above returns only the OBSPACK-Europe dataset, and does not appear to have an option to specify an alternative. Most sites have at least two different files with the same species, level and inlet height - it is important that the user is able to choose between them.
After some digging, it seems to me like _retrieve_remote loops through the dobj_urls (starting L270), and overwrites data if "spec" and "inlet" are the same (which they will be for these files). OBSPACK-Europe happens to be the last one listed, but I don't think this behaviour can be relied upon.
I reckon you can get at the information we want to filter on using: dobj.info["specification"]["project"]["self"]["label"]. But I am nowhere near confident enough to implement a fix based on this, and most likely there is a neater way (possibly inbuilt in the icoscp package, although I couldn't find it).
What did you expect to happen?
No response
Minimal Complete Verifiable Example
from openghg.retrieve.icos import retrieve_atmospheric
wao_data = retrieve_atmospheric(site="WAO", species="ch4")
wao_data.data

Relevant log output
No response
Anything else we need to know?
No response
Environment
Python 3.8.8
Openghg - devel commit: ec166689ab080b0458428aaee4c2102e37135fc7

@ag12733 - Something to be aware of.

I'm working on this now.


Add Deptry, split dev into dev and docs recipes, address bandit security issues

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[x] Tests for the changes have been added (for bug fixes / features)

[x] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] CHANGES.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

Adds the deptry package to the linters for performing dependency analyses (unused or missing)
Pinned a few linting dependencies
Splits the dev dependencies into dev and docs (both can be installed with all)
Enables flake8-bandit like checks in ruff, fixes a few security-related checks
xclim.testing.utils now have more secure URL auditing checks

Does this PR introduce a breaking change?
Yes, a new dependency has been added (deptry).
Also, xclim no longer bundles the documentation dependencies with the dev recipe. Splitting this reduces the number of dependencies needed to run the tests and also helps us know which dependencies are needed and where (determined with the aid of deptry).
Other information:
https://deptry.com/
https://docs.astral.sh/ruff/rules/#flake8-bandit-s

coverage: 90.649% (+0.05%) from 90.595%
when pulling 8c83c693d3adeb3dc0673476610e9d95d9901f52 on deptry
into a375d781f3bed79103cd6d54b5c57896ca11e47c on main.

coverage: 90.639% (+0.04%) from 90.595%
when pulling 59fa5c1e06f96adc7da5cd27801504f888ffb20e on deptry
into a375d781f3bed79103cd6d54b5c57896ca11e47c on main.

coverage: 90.649% (+0.05%) from 90.595%
when pulling a2d84f164adef69cfb3d62741f2093f20cbd28ce on deptry
into e8d2f3609e56d418018c764cb995b7cc4cf42962 on main.

coverage: 90.649% (+0.05%) from 90.595%
when pulling 87f35da075424ea13b10a4214b51537b31bf072b on deptry
into 23fd681a0582ba86afdf156cbc8d359fe170a794 on main.


Clearer go installation and support custom package for go install

Clearer go installation
Problem:

with vagrant - ubuntu 16.04. File not found when using copy.
Copy take a lot of time.
Solution: Using mv instead of cp

Custom package for go install
There was time people want to install in custom package. Say: github.com/any_repo/my_client/cmd
So go_install should be:
    go_install:
      # repo is the git clone url, ssh or https.
    - repo: https://github.com/mholt/caddy
      # dest is the namespace
      dest: github.com/mholt/caddy
      # version refers to a tag, or branch.
      version: "master"
      # package is the package you want to build.
      github.com/mholt/caddy/caddy in this case
      package: github.com/mholt/caddy

LGTM, I appreciate the correction in logic! 🙂


nixos/mandoc: misc fixes and improvements

Description of changes
I've made a mistake in the last pr https://github.com/NixOS/nixpkgs/pull/246923:
https://github.com/NixOS/nixpkgs/blob/79fee6027b432958825b0bb83a1538314327a8a6/nixos/modules/misc/mandoc.nix#L180-L190
The ${lib.concatMapStringsSep " " (path: lib.escapeShellArg path) cfg.settings.manpath shouldn't exist because the users can specify any paths they want, but those paths aren't available at build time, and are probably not a part of the system configuration. So let's keep the behavior the same with man-db by just dealing with manpages installed in the system configuration:
https://github.com/NixOS/nixpkgs/blob/79fee6027b432958825b0bb83a1538314327a8a6/nixos/modules/misc/man-db.nix#L65-L70
I've added a new option to make this clearer and describe what has done by makewhatis. Users can still use this option to generate man caches if they are using the settings.manpath approach.
I've also slightly improved the method to add leading slashes.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

I don't a 100% understand this yet… Can you give me an example configuration that would need to set manPath and cachePath to different value?

I don't a 100% understand this yet… Can you give me an example configuration that would need to set manPath and cachePath to different values?

If someone doesn't need the environment variables, but still wants to generate caches.

@ofborg test man


Question: Get user information, missing attributes

I am attempting to read user information from AWS Cognito using the get_user() method, but I am unsure if it works as expected.
What I am doing:
u = Cognito(.....)
user = u.get_user(attr_map={'given_name': 'firstname', 'family_name': 'lastname'})

I would have expected this method call to expose the values of 'firstname' and 'lastname' as separate properties, or as a single "data" property of the object "user", but unfortunately - no.
The values are only exposed via the user._data private property. Is this a bug, or am I missing something?

It seems that you need to swap keys with values in the attr_map dictionary.
>>> user_obj = user.get_user(attr_map={'firstname': 'given_name'})
>>> user_obj.given_name
'Jenkins'

What is in your self._data? What attribute are you trying to get?

self.data exposes the correct list of AWS Cognito attributes I am asking for, with the correct mapping of attribute names, according to the specifications mentioned above.
{
  'email'<EMAIL_ADDRESS>
  'lastname': 'Ols...', 'firstname': 'Jon.....',
  'custom:kit_app2': 'M24',
  'custom:kit_app1': 'DM'
}

My problem is that all these values resides in a "private" object, not exposed via normal properties or "getter" methods.

It should be exposed as an attribute if it's in self._dict or self._metadata. Can you try to debug __getattr__ method?
https://github.com/capless/warrant/blob/d109924e9e2a2cad34eb0f539f7491cef7585dbf/warrant/__init__.py#L64-L68

I am able to get the values via the self.__getattr__() method, but this is still i private method. Shouldn't this method have been a public method of the user object?

self.__getattr__(self, 'attribute_name') is equal to self.attribute_name
3.3.2. Customizing attribute access

Ah, of course, my bad. Thanks!


Kryo5Codec with redisson-3.19.1 clarification

Release notes read:
Breaking change - Kryo5Codec uses own serializators to serialize UUID, URI and Pattern objects

Could you clarify what this means and implies?

It means that if you serialized object which has UUID, URI or Pattern with previous version. You won't be able to decode them with new version. Since serialization method has changed for these object.


Defined new revisions for golang dep: github.com/containerd/ttrpc

Ran component through the scanners since I noticed it had not been harvested yet. PR matches curation site. Closing.


ENH: Add config for pre-commit

this is just a draft which kind of shows what i wanna say for #755
i set it to WIP, so feel free to comment and i will change. @ueshin @HyukjinKwon

Codecov Report

Merging #756 into master will decrease coverage by 0.42%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #756      +/-   ##
==========================================
- Coverage   93.91%   93.49%   -0.43%
==========================================
  Files          32       32
  Lines        5669     5669
==========================================
- Hits         5324     5300      -24
- Misses        345      369      +24

Impacted Files
Coverage Δ

databricks/koalas/missing/__init__.py
92.3% <0%> (-7.7%)
:arrow_down:

databricks/koalas/indexes.py
94.59% <0%> (-2.71%)
:arrow_down:

databricks/koalas/__init__.py
82.5% <0%> (-2.5%)
:arrow_down:

databricks/koalas/exceptions.py
80.95% <0%> (-2.39%)
:arrow_down:

databricks/conftest.py
95.34% <0%> (-2.33%)
:arrow_down:

databricks/koalas/plot.py
93.78% <0%> (-1.13%)
:arrow_down:

databricks/koalas/datetimes.py
97.34% <0%> (-0.89%)
:arrow_down:

databricks/koalas/frame.py
95.53% <0%> (-0.71%)
:arrow_down:

databricks/koalas/groupby.py
89.44% <0%> (-0.46%)
:arrow_down:

Continue to review full report at Codecov.

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

ahh, just checked .travis.yml, and find out it's building py35, but black only runs on py36 or newer, so pip install will fail..

Maybe we don't need to add it as a dependency, but in that case, definitely we have to add some instruction to install the library for contributors.

yeah, agree... have to remove it, i will add some instructions in CONTRIBUTING.md.

To me, I'm okie. Seems worth trying.


set up problem

I followed the setup instructions.
i completed set up tooling.
But when i run script .\install.ps1, then it got error like this

please help me. thank you

Hello @ptthuc26 - can you look at the installer log file to get more details on what exactly is failing for you?
Also, you can use a Cloud9 environment to run the installer if your local workstation is having problems.

Hello @ptthuc26 - can you look at the installer log file to get more details on what exactly is failing for you?
Also, you can use a Cloud9 environment to run the installer if your local workstation is having problems.

hi @brtrvn , this is the installer log file. i tried cloud9 but still the same error. please help me solve this problem.
Thank you.

Hi @ptthuc26 , this looks to be only a section of your installer log file. In particular, we're looking for any error messages in the saas-boost-install.log file that might point to what's going wrong here. Can you explain in detail which step in the startup instructions you are getting stuck on? Are there any instructions that don't make sense?

I was having the same issue. After debugging the code I found that there is an issue with the unit test case located at "installer\src\test\java\com\amazon\aws\partners\saasfactory\saasboost\SaaSBoostInstallTest.java".
Removing the last assert statement i.e.
for (Map.Entry<String, String> entry : expected.entrySet()) {
assertEquals(entry.getKey() + " equals " + entry.getValue(), entry.getValue(), actual.get(entry.getKey()));
}
from the file and running the install.ps1 script solved my issue.
Maybe @PoeppingT you could check what is the issue with the test case "testGetCloudFormationParameterMap" where keyboard input is checked.


add subenv.yml aliases handling

let me know if you want some tests with this.
The `subenv.yml` is a useful file for storing subenv specific
settings that you might need for your project. one common use case
for this file would be to integrate custom aliases specific to your
project. However, this feature wasn't supported.

These changes now make it possible to add an aliases section to
your `subenv.yml` the same way as you would in your default
`engine.yml`.

This is done by navigating to the specific sub-envs path and
loading up the `subenv.yml` inside the engine and merging
both aliases sections when aliases are accessed.

@codeclimate They were identical before, please...

Verified and tested. All good!


Web Interface counter loops at 65535

I believe this is due to the 8 bit nature of the micro controller i have tried unsigned long to store the value but that does not work either.
Maybe a loop count would work suchas, when arp_count = 65536 add 1 too the loop count then when echoing arp_count add the current count + loop count * 65535.

Well I won't consider that as so important. Maybe just stop counting and display ">65535" in the webinterface? If you have sent over 65 thousand packets, I think you'll have other problems than a misleading webinterface ^^
plus: if you only sent every 1,5s a packet (which I set as default) you would need over a whole day to reach that number.

Well aware of that haha,
This was in my initial release, Feel free to close this, Just an issue that may occur if it's left on a network running for a extended period of time ;)


Identity cases for asuint, asfloat, etc.

Subtle errors can arise when users apply, e.g., the asfloat function to a value that is already a float:
float f = ...;
float g = asfloat(f);

The above code will select between the two available overloads of asfloat that are implemented in Slang (which follows what is documented on MSDN):
float asfloat(uint val);
float asfloat(int val);

Supposing that the int variation gets chosen, this means the code compiled like:
float g = asfloat(int(f));

That means we convert f from float to int, and then convert the bit pattern of that int to a float. This is not what the user would expect/want: they just want asfloat on a float to be an identity.
Indeed, it seems that both fxc and dxc support additional overloads of asfloat, etc. to handle the identity cases, and these just aren't documented on MSDN.

@jsmall-nvidia I think you implemented all these, right?

Yes these are in.


Fix player release

I suspect the line got lost somewhere since your function already "removes the player element from the DOM."
Without the fix I am not able to play different songs with a single player instance under the same htmlId

Thank you for the awesome library


Add strict mode for request completion

Currently, the contract for deliver() means that users have no indication if there are failed files beyond an error message printed to the console/notebook. There is no programmatic way of knowing this from user code. (The most extreme example of this is if all jobs in an uproot-raw request fail due to a missing branch in the input file, for example, the user will see an empty list for the return value!)
I would propose to add a "strict mode" which would raise an exception for a transform if there are failed files, rather than returning a partial list of the files that do succeed. I would prefer that this become the default; this would technically be a breaking change, and would actually affect the AGC scripts (which currently refer to nonexistent files). But it's a real data integrity problem if we don't address this.

Resolved by #516


bosh cck restarts jobs in a (random?) order instead of the order of the deployment spec

Just when you might be most stressed - your entire production system has been deleted by someone with dashboard access; or you're preparing a screencast for PlatformCF - bosh cck brings back VMs one at a time.
Applying problem resolutions
  missing_vm 21: Recreate VM using last known apply spec (00:01:32)
  missing_vm 27: Recreate VM using last known apply spec (00:01:32)
  missing_vm 24: Recreate VM using last known apply spec (00:01:52)
  missing_vm 26: Recreate VM using last known apply spec (00:02:00)
  missing_vm 20: Recreate VM using last known apply spec (00:01:30)
  missing_vm 23: Recreate VM using last known apply spec (00:01:39)
missing_vm 18: Recreate VM using... |oooooooooooooo          | 6/10 00:10:16  ETA: 00:06:31

I think it wants to torture me. It loves me. But the love hurts.

Closing. We are slowly converging to having bosh deploy --fix having ability to recover deployments in a irregular state. Ultimately we may even turn bosh cck into a command that just reports problems instead of allowing to interactively fix them.


SNOW-1677888, SNOW-1677890: Add support for DataFrame.tz_convert and Series.tz_convert

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

Fixes SNOW-1677888, SNOW-1677890

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.

Please describe how your code solves the related issue.
Add support for DataFrame.tz_convert and Series.tz_convert.

@sfc-gh-helmeleegy IMO the code in this PR can be added to the tz_localize one as well. I think you can even test both tz_localize/tz_convert by parametrizing on the function in the same test file. Even though the PR line diff it bloated, it will mostly be docstrings/signature and since most of the code is in both places, you will have to rebase anyways.


Use spf13/cobra for docker tag

This fix is part of the effort to convert commands to spf13/cobra #23211.
Thif fix coverted command docker tag to use spf13/cobra
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Thanks @dnephin. I added flags.SetInterspersed(false) though it is still shows:
docker tag a -b
"docker tag" requires exactly 2 argument(s).
See 'docker tag --help'.

Usage:  docker tag IMAGE[:TAG] [REGISTRYHOST/][USERNAME/]NAME[:TAG]

Tag an image into a repository

After some investigation, it seems that ExactArgs() is not passing the expected args.
So I changed
		if len(args) == number

to
		if len(cmd.Flags().Args()) == number

in ExactArgs().
Now the command behaves as expected.
Just want to make sure there is no unintended side effect with this change.

I fixed it in cobra, and @vdemeester should be picking up the fix in one of the PRs, if you want to keep that change for now, maybe add a TODO saying it should be fixed in cobra?

@dnephin I just added a TODO in the commit and updated the pull request. Thanks for the help!

Updated in #23269 :wink:

Oh, sorry @yongtang this one needs a rebase as well

Thanks @vdemeester I will keep an eye on #23269 and remove TODO if it get merged. And thanks @thaJeztah the PR has been rebased.

The pull request has been rebased. It is likely there is a failing test. The failure is caused by an issue in cobra, which has been fixed (not yet merged into the docker yet). Once #23269 is merged the the failure will be fixed.

@yongtang https://github.com/docker/docker/pull/23269 was merged, so you can rebase and fix the failure

Thanks @thaJeztah the PR has been rebased and fixed.

LGTM

LGTM

LGTM 🐵

All green! Merging


[FEATURE] clear_all_callbacks() to removes previously applied bindings

I want to have different bindings depending on what views are active, and I want to be able to clear all bindings between view changes but that is currently not possible.
Add a function to the Cursive type called clear_all_callbacks() that removes all of the callbacks tied to the cursive instance, and then calls reset_default_callbacks() on itself.
Also possible to add a set_pre_event or set_post_event to views so similar behavior can be achieved

Add a function to the Cursive type called clear_all_callbacks() that removes all of the callbacks tied to the cursive instance, and then calls reset_default_callbacks() on itself.

Ah indeed we're missing a function to remove all callbacks, not just these tied to an event. Will add that. EDIT: Latest commit should bring Cursive::clear_all_global_callbacks().

Also possible to add a set_pre_event or set_post_event to views so similar behavior can be achieved

Views can already set pre/post events using a OnEventView ("post-event" is just set_on_event).

I cloned the repository after grabbing a cup of water and you already added the function lol, that was fast thanks
Also didn't know that thanks for clarifying.


How to show time picker with 24hrs and 12 hrs format??

How to show time picker with 24hrs and 12 hrs format??
如何以24小时和12小时格式显示时间选择器？
I can see kHM constants as well. But not getting idea.
Can you put a proper example for this case.

请查看demo中的第8个和第9个示例


[Question] Select Component search when using paging

I am using a select component and using URL to an external source. I get the data just fine so I added limit and skip to the URL. This works fine as well but it seems to break searching inside the select component.
Is there an option to pass through to the external source the string that is being searched for?

I believe this is the option called searchField property within the component JSON.  https://github.com/formio/formio.js/blob/master/src/components/select/editForm/Select.edit.data.js#L324

I see the field in the form but how do I get it in the url?
I have skip={{skip}}&limit={{limit}}  in the URL but if I add searchField={{searchField}} to that then it does not work.

We're currently addressing a backlog of GitHub issues, and as part of this effort, some inactive issues may be marked as closed. This isn't a dismissal, but a step toward more efficient tracking.
If you feel the issue is still relevant, please re-open and we'll ensure it gets the attention it deserves. Your understanding is appreciated as we work to enhance our open-source responsiveness.


Binary data

Do we envisage that this store eventually will store binary data or will that be a job of another store?

Sorry, I think this store should be renamed to the JSON store. As it will not support raw binary data.
Depending on your requirements, and size of the data, you could base64 encode it. Then store it in a JSON object though.
Have you got an example of the binary data you need to store?

Perhaps thermostat data ? Or accelerometer or location data ?
On Tuesday, 8 November 2016, ToshBrown<EMAIL_ADDRESS>wrote:

Sorry, I think this store should be renamed to the JSON store. As it will
not support raw binary data.
Depending on your requirements, and size of the data, you could base64
encode it. Then store it in a JSON object though.
Have you got an example of the binary data you need to store?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/me-box/databox-store-blob/issues/5#issuecomment-259106266,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACy0rytiMm81NqqVrjag8r2TfWEcVStnks5q8FSXgaJpZM4KsOOd
.

--
Hamed
http://www.eecs.qmul.ac.uk/~hamed/

Videos from action cams (e.g. GoPro). Not a problem if this is going to be a JSON only store, just thought I'd check before using it.

Also, photos...
Something like https://git-lfs.github.com/ might well be of use.

yes, I was going to point to LFS too

LFS looks interesting, any thoughts on how a blob datastore might be implemented using it? I have a couple of thoughts and we could maybe have a discussion locally (Notts) about it.

Looks interesting I will take a look at the docs on Wednesday. @dominicjprice I'm in on Thursday or Friday if you want to have a chat ping me an email.


Split application content from mutable content

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: _______
[ ] ACS AEM Commons Version: _____
[x] Reproducible on Latest? yes

Expected Behavior
As showcased in the AEM Archetype and following the typification of JCRVLT-170 packages containing application content (libs & apps) should be split from remaining content. This is in preparation for scenarios using composite node store where the application part can and must be installed during deployment time while content accounting to the mutable part can only be installed after startup (at runtime)
It is fine to nest both packages in a shared container for convenience but it must be possible to declare dependencies on the individual parts (following changes in JCRVLT-336 you can expect mixed content packages to be installed as application packages while the rest of the content would be ignored.
Actual Behavioral
Acs commons is a single mixed content package.
Steps to Reproduce
Check sources.
Links
Links to related assets, e.g. content packages containing test components

@davidjgonzalez & @badvision  please also crosscheck the content being located in etc which is no longer the recommended location. For AEM we relocated these things but established mechanisms to stay backward compatible if content still resides in the old areas. - this is specifically true for content that has rather application characteristic and would be better located under apps (e.g. clientlibs).

We have to maintain backwards compatibility to 6.3, so whatever we do has to work there as well.

To be clear, there is a content folder in the package but it's just an ACL for DAM to support the acs-commons-review-task-asset-mover-service.  Is there any other content you're seeing that needs to be relocated?

FYI - im going to time box this and see how far i get

Proposed project structure:

oak-pal -> no change
bundle -> no change
bundle-twitter -> no change
content -> turn this into a "container package" which contains 2 NEW projects

ui.apps containing

/apps

ui.content containing

/oak:index (ACL)
/content (ACLs)
/etc (Misc content, ACLs)

All clientlibs will be moved out to /apps/acs-commons/clientlibs with allowProxy=true

/home/groups (ACLs)
/home/users (Service Users and ACLs)
/var/acs-commons (various supporting nodes and ACLs)

FWIW I think we can move the /etc content type stuff (anything that renders a page, etc) into /apps/acs-commons/content and change paths to point to /mnt/override/acs-commons/content ...
This is pretty similar to how things like pickers and data sources already work in the product with /lib/cq/content, etc

I broke out everything in /etc more clearly; i think quite a bit could be moved out of /etc into /apps per https://helpx.adobe.com/experience-manager/6-4/sites/deploying/using/repository-restructuring.html

Question to the group, can we change the "content" projects name and still keep it generally backwards compatible? Ideally, we have an "all" or "container" and the "ui.content" and "ui.apps"
An option, though I worry it might be confusing is we could rename all the maven projects but have the final build artifacts (jar file and deployable content package) names remain the same.

Ideally when updating versions, a consumer would just update the version part of the coordinates in their pom (I think @DominikSuess is talking about this from a package manager standpoint, which is fair, but a different question IMO). That's what we've "trained" consumers to do and it seems confusing to change at this point.
So my suggestion would be to keep acs-aem-commons-content as the Maven artifactId for the container and have that embed acs-aem-commons-ui.apps and acs-aem-commons-ui.content at least for the rest of 4.x. IMO changing that to acs-aem-commons-all is a 5.0 thing.

@justinedelson good point

Few questions that have come up during re-org.

[ ] Is the dependency on cq-content (to enforce 6.3 requirements) still best placed in ui.apps? (https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/content/pom.xml#L74) @DominikSuess
[ ] Do we need min packages for everything (content, ui.apps, ui.content) now? @justinedelson is there a simpler approach?
[ ] Do we need full oak-pal configs in both ui.apps and ui.content? @adamcin

@davidjgonzalez Unfortunately, we need min for everything since there are twitter paths in both /apps and /etc :(

@justinedelson TBH I'm not sure how to aggregate ui.apps min and ui.content min into content min package. Any chance you could whip that up in a PR to https://github.com/davidjgonzalez/acs-aem-commons/tree/feature/1975 ?

@adamcin any thoughts on if/how we should split oak-pal checks for ui.apps vs ui.content?

@davidjgonzalez I think the existing checklists can (and should) still be used for both ui.apps and ui.content, but we may want one or two new checks to enforce the separation of apps and content.

@adamcin any chance you could PR the separation checks to? https://github.com/davidjgonzalez/acs-aem-commons/tree/feature/1975

One new check should suffice for requiring only apps/libs or only non-apps/libs nodes. Something like a CompositeStoreAlignmentCheck that generally wouldn't need additional configuration for AEM packages in general. We can configure Paths checks for each individual package to enforce which side of that distinction the package should be restricted to.

@davidjgonzalez I'll submit a PR.

@davidjgonzalez  see https://github.com/davidjgonzalez/acs-aem-commons/pull/5

Just a FYI - I tried to update the cards to use /mnt/overlay/acs-commons/content/page-x.html however this was preventing certain objects from being exposed in the script (namely currentPage) which in turn broke some of the pages to be moved.
I changed these to use the pages' "real" paths /apps/acs-commons/content/page-x.html and everything works.

I was actually going to comment on that {{/mnt/overlay}} thing. Unless I've forgotten something significant, {{/apps} is the end of the overlay chain, so unless we expect AEM to "underlay" this stuff (which would be weird), there's no reason to use {{/mnt/overlay}}.

@adamcin i took a stab at adding the OakPAL checks here: https://github.com/davidjgonzalez/acs-aem-commons/commit/e0b3d7dc6eb828039122adbda8d777767a79efa9
I wanted to add an OakPAL check to the content container project, to ensure that its empty (other than the ACS sub packages) but it seemed to pick up the paths OF the sub packages and fail on them; not sure if there is simple way handle this.
Im very open to feedback on the rules I added - this is my first time OakPALing around!

@davidjgonzalez it looks like a great start. I hadn't considered enforcing the behavior of the container package type, but that could definitely be useful as a reusable check in its own right. I'm working on that new check that I described, and based on my current implementation, it would fail the content package if it imported any of its own content outside of the /etc/packages/ root, AND if, in combination with its subpackages, it affected multiple composite mounts. If it doesn't affect any content outside of /etc/packages, I ignore any cross-mount effects of its subpackages in combination. If any individual subpackage affects multiple mounts, that subpackage will be reported as a violation.

@adamcin sounds good! feel free to PR a complete replacement to my branch (i wont be offended ;))

build_project_issues.csv.txt
Maybe as as followup issue - we should tackle atleast the Criticals in this report.

Code Coverage is blocked because of this error:
INFO] ------------------------------------------------------------------------

[INFO] Skipping ACS AEM Commons - Reactor Project

[INFO] This project has been banned from the build due to previous failures.

[INFO] ------------------------------------------------------------------------

[INFO] ------------------------------------------------------------------------

[INFO] Reactor Summary:

[INFO]

[INFO] ACS AEM Commons - Reactor Project .................. SUCCESS [ 14.668 s]

[INFO] ACS AEM Commons Bundle ............................. SUCCESS [03:02 min]

[INFO] ACS AEM Commons Oakpal Checks ...................... SUCCESS [ 18.670 s]

[INFO] ACS AEM Commons UI.Apps Package .................... SUCCESS [ 21.868 s]

[INFO] ACS AEM Commons UI.Content Package ................. SUCCESS [  2.812 s]

[INFO] ACS AEM Commons Package ............................ FAILURE [  0.542 s]

[INFO] ------------------------------------------------------------------------

[INFO] BUILD FAILURE

[INFO] ------------------------------------------------------------------------

[INFO] Total time: 04:13 min

[INFO] Finished at: 2019-08-11T14:31:15Z

[INFO] Final Memory: 98M/427M

[INFO] ------------------------------------------------------------------------

[ERROR] Failed to execute goal on project acs-aem-commons-content: Could not resolve dependencies for project com.adobe.acs:acs-aem-commons-content:content-package:4.3.1-SNAPSHOT: The following artifacts could not be resolved: com.adobe.acs:acs-aem-commons-ui.apps:zip:min:4.3.1-SNAPSHOT, com.adobe.acs:acs-aem-commons-ui.content:zip:min:4.3.1-SNAPSHOT: Could not find artifact com.adobe.acs:acs-aem-commons-ui.apps:zip:min:4.3.1-SNAPSHOT in adobe-public-releases (https://repo.adobe.com/nexus/content/groups/public/) -> [Help 1]

[ERROR]

[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.

[ERROR] Re-run Maven using the -X switch to enable full debug logging.

[ERROR]

[ERROR] For more information about the errors and possible solutions, please read the following articles:

[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

[ERROR]

[ERROR] After correcting the problems, you can resume the build with the command

[ERROR]   mvn <goals> -rf :acs-aem-commons-content

Done. Your build exited with 0.

I believe this is complete.

There is an issue due to the split as rep:policy for apps are not being applied. The reason is that rep:policy is in ui.apps package and users are in ui.content. Due to the order of ui.apps being installed prior to ui.content, as a result system users have no access to /apps. It happens if you have no previous version of ACS AEM Commons installed because there are no ACS AEM Commons system users. Could you please fix?

@ataimussova can you raise a new issue for this? I guess that it might get ignored otherwise.

@joerghoh, thank you. The issue was raised and fixed #2048.


ja: Make docs/tasks/run-application/_index.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/tasks/run-application/_index.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/tasks/run-application/_index.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.1/content/ja/docs/tasks/run-application/_index.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/tasks/run-application/_index.md
diff between translated and v1.18
https://gist.github.com/fa8324996d08bf2112b835e1b50efef2
Note
Currently, we use dev-1.18-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel

/assign

/close


Suggestions

Following suggestions:

Cleaner requirements.txt: You seem to have the output of pip freeze as requirements.txt, which is not great since pip resolver works differently for different platforms/versions. Instead, use specific versions of dependencies you actually import in your app, especially third party ones (like pandas, imagededup, etc.)
Python version you use is python 3.7, which is no longer supported by imagededup (from 0.3.0 onwards). So, please switch it to python 3.8/9/10.
There are Heroku specific files in there, whereas the Readme instructions seem to be focussed upon running it locally. You could either remove the Heroku specific part (while keeping the docker part anyway), or include instructions to deploy to Heroku as well.

@tanujjain , Thank you for these suggestions.
I have made all the changes.


KDE: most windows don't maximize under wayland anymore

Maybe I can find a workaround

https://bugs.kde.org/show_bug.cgi?id=460628

Ignoring KWallet's Password popup (whatever application that is) might be an idea, if it is not ignored by default because it doesn't seem to maximize in the weird way it did before.

This bug applies to most gtk2/3 applications

window-rule

gtk3-noborder

gtk3

gtk2

https://github.com/KDE/kwin/commit/903e18d88c39c81bd05162fec1d536f6974c15e2


Not working on mac os installed with yarn

$ echo 'i stpid' > f.md
$ proofreader -f f.md
### Results for f.md ###

There are no suggestions for f.md
Using node 7.8.0, yarn 1.3.2

Thanks for reporting. I did a quick debug, and it turns out to be one of the dependencies which introduced a breaking change. I fixed it by including package-lock.json file which locks versions of deps. It was published as v0.6.6 and should be already available on npm.


Issue starting builder

Running on a vm
Traceback (most recent call last):
File "C:\Users\Administrator\Desktop\Rose-Injector-main\roseui\builder.py", line 1, in
from dhooks import Webhook, Embed
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'dhooks'

look it up online

python -m pip install dhooks


Sub Folder Auto Import

Is your feature request related to a problem? Please describe.
Sub Folder Auto Import
Describe the solution you'd like
I have main folder name SVG inside SVG there are many sub folders like SVG>folders. I just want to select SVG & it should auto import other sub folders with icons like iconset
Describe alternatives you've considered
Iconset

I need this too, else I can't use your tool at all. I have many thousands of icons in hundreds of folders. Icon-Shelf has to recursively search all icon files in all subfolders and represent them in the app with the exact same structure to be able to find them again.


icon misalignment in zen mode

Collapsing the left sidebar causes its icon to misalign, but it only happens in zen mode.
(Also, it would be very useful to have a zen hotkey option!)

Thank you for reporting. I have noticed this issue. I am going to try a better way of managing the sidebar collapse buttons ASAP

This problem should be corrected in the latest version (v.0.6.10). Please update and ‼ reopen Obsidian ‼.
If you encounter similar issues, please reopen an issue.