Add default storage location or support more file managers

Describe your suggested feature
Add default storage location, for example /sdcard/mihon or /sdcard/Android/data/app.mihon/files/Download, then user can use mihon without DocumentsUI, this should be the easiest way to solve  #386
Or try to use like
https://github.com/zanjie1999/metime/blob/master/app/src/main/java/com/cupinkie/metime/FileUtil.kt
to support more file managers
Example:
https://github.com/zanjie1999/metime/blob/master/app/src/main/java/com/cupinkie/metime/SettingsActivity.kt
Lines 61 to 64 call the file manager
Lines 90 to 110 get the selected content
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.16.5.
[X] I will fill out all of the requested information in this form.

The app you have linked requires reading permission of a singular image while we require writing permission inside a directory. The whole concept is different. We need DocumentsUI because in future Android version SAF (which relies on DocumentsUI) will be the only way to gain those writing permissions.

But the app writes to its own data directory located in /sdcard/Android/data/app.mihon no additional authorization required
(use getExternalFilesDir(null) to get file uri)
Adding a default data directory and allowing skipping this step in the setup wizard will make the app available to more devices

Users expect their downloads and especially backups to stay even after they remove the app. Defaulting to app's internal storage doesn't achieve this

This is indeed the case, but it allows users to choose to use the app's internsal storage at least allowing devices without DocumentsUI to use the app

DocumentsUI is a core part of Android. Mihon is an Android app. Do you see the correlation?

Many Android eink reader remove DocumentsUI, as you can see in #386

Many Android eink reader remove DocumentsUI, as you can see in #386

They do yes. And people should be bashing them for doing so instead of asking us to add workarounds. I have no issue adding small workarounds for small quirks but this is not a small quirk.

The e-book system seem to have broken DocumentsUI

Broken as in it is present but app doesn't save the path or something? Cause that is being handled in beta and will soon come to stable.

I forked and tried to fix this problem, but I didn't succeed because of my poor skills
Then I found that I could use tachiyomiJ2K instead of mihon, which uses the old file api, has a default path, and can also double-page horizontally


Add description to Page.Params

Now it is set on Page.Description -- it would be useful if it was ALSO set in the Params map.
See #1462

I am interested to work on the issue. Can you explain what's exactly is needed ?

See
https://github.com/spf13/hugo/blob/c274e21356826a6aa1e04d875a37838a6fe1a6c6/hugolib/page.go#L508
Just put the same value into the Page.Params map with "description" as key.

@arunk-s do you want to make a Pull request?

@spf13 I've actually created a pull requested quite earlier: https://github.com/spf13/hugo/pull/1522.


Cannot run make requirements from an empty virtualenv for cookiecutter-django-ida

The Makefile target piptools, which installs pip-sync, was removed from cookiecutter-django-ida (but not other cookiecutters) in  [link to line]. This means that make requirements in a clean virtualenv will fail for that cookiecutter's outputs.
The other cookiecutters vary in how they install the pip-tools package.
Acceptance criteria:

[x] There is a pre-requirements target (or similar) that installs pip and pip-tools pin files, as a dependency of the upgrade and *requirements targets.

https://github.com/openedx/edx-cookiecutters/pull/330

[ ] CI checks that make requirements runs properly in the output of all cookiecutters in a fresh virtualenv.

Note that make upgrade installs pip-tools for at least some cookiecutters, so this does need to happen in a separate or clean virtualenv from any tests that also run make upgrade. Not only that, but make upgrade is listed as a post-cookiecutter setup step. Why the extra work? So probably what we want (possibly in another ticket) is:

[ ] make upgrade is run in a post hook in order to generate the .txt files needed by make requirements
[ ] Instructions updated to remove the mention of make upgrade
[ ] ...and then write the tests that ensure that a fresh virtualenv supports make requirements

[ ] Any other requirements handling is standardized while we're in there.

How each repo handles installing pip-tools / pip-sync via make requirements now:

cookiecutter-django-app uses a piptools recipe
cookiecutter-django-ida doesn't install it
cookiecutter-xblock doesn't install it
python-template runs pip install -r requirements/pip.txt and pip install -r requirements/pip-tools.txt in the make requirements recipe

Manually edited shell search:
edx-cookiecutters % git grep -n -A5 '^requirements:' **/Makefile
cookiecutter-django-app/{{cookiecutter.repo_name}}/Makefile:63:requirements: piptools ## install development environment requirements
cookiecutter-django-app/{{cookiecutter.repo_name}}/Makefile-64- pip-sync -q requirements/dev.txt requirements/private.*
--
cookiecutter-django-ida/{{cookiecutter.repo_name}}/Makefile:38:requirements: clean_pycrypto dev_requirements ## sync to default requirements
--
cookiecutter-xblock/{{cookiecutter.repo_name}}/Makefile:45:requirements: piptools## install development environment requirements
cookiecutter-xblock/{{cookiecutter.repo_name}}/Makefile-46-     pip-sync -q requirements/dev.txt requirements/private.*
--
python-template/{{cookiecutter.placeholder_repo_name}}/Makefile:61:requirements: ## install development environment requirements
python-template/{{cookiecutter.placeholder_repo_name}}/Makefile-62-     pip install -r requirements/pip.txt
python-template/{{cookiecutter.placeholder_repo_name}}/Makefile-63-     pip install -r requirements/pip-tools.txt
python-template/{{cookiecutter.placeholder_repo_name}}/Makefile-64-     pip-sync requirements/dev.txt requirements/private.*

Hmm! Yeah, make requirements should be installing the pip and piptools pinning files first, one way or another. (There are a few ways that we have gone about this in various repos.) I also would have expected CI to catch this, as we do have automated tests that use the cookiecutters and then run tests in the new virtualenv. Perhaps that virtualenv is dirty somehow, though? I'll add that to the acceptance criteria.

@pshiu I think the original scope of the issue is complete; cookiecutter-django-ida now has a working make requirements. However, I did want to check on something -- one thing about the cookiecutters is that you need to run make upgrade after using the cookiecutter, which is annoying manual step, and would also result in make requirements not working. Is that something you ran into? I have it listed in the A/C of this issue to look into making that automated (probably as a separate issue), but wanted to check first if that was something you encountered as a pain point.

@timmc-edx Hooray! Thanks for taking on this work!
I never found make requirements inoperable after a make upgrade, but I was so frustrated back then & was so furiously running manual installs of pip that I'm sure I might have missed that being the case.

Filed https://github.com/openedx/edx-cookiecutters/issues/337 for automatically running make upgrade


Verify basic camera connectivity

There is a command that will say "is a camera detected or not".

[x] verify that the camera is detected by the pi
[x] capture a test image with the camera

reviewed at team meeting 3/28


[Bug Report] Automatic song skips

Bug Description
The bot will skip songs automatically when multiple songs are in queue. Happens mostly when I am alone with the bot, occasionally happened with multiple people requesting songs too.
Steps to Reproduce

Request a song (file or YT link)
Request more songs (file or YT link)

Expected Result
The songs should play back like they are in the queue.
Debug Output
System Properties:
  java.version = 11.0.18
  java.vm.name = OpenJDK 64-Bit Server VM
  java.vm.specification.version = 11
  java.runtime.name = OpenJDK Runtime Environment
  java.runtime.version = 11.0.18+10-post-Ubuntu-0ubuntu122.04
  java.specification.version = 11
  os.arch = amd64
  os.name = Linux

JMusicBot Information:
  Version = 0.3.9
  Owner =<PHONE_NUMBER>42899486
  Prefix = !!
  AltPrefix = null
  MaxSeconds = 0
  NPImages = true
  SongInStatus = true
  StayInChannel = false
  UseEval = false
  UpdateAlerts = true

Dependency Information:
  JDA Version = 4.4.1_353
  JDA-Utilities Version = 3.0.5
  Lavaplayer Version = 1.3.78

Runtime Information:
  Total Memory = 380
  Used Memory = 56

Discord Information:
  ID =<PHONE_NUMBER>63625236
  Guilds = 1
  Users = 2

Additional Info
Could it be associated with the fair queue function?
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

Likely duplicate of #885

Without logs showing the errors that happened at skip time it's impossible to debug this


Allow filtering resources to watch based on a label

Both kube-lego and kube-cert-manager have in some form held a concept of a class label that can be added to resources to indicate which instance of kube-lego/kube-cert-manager should process what resource. This is a ticket to discuss the introduction of a similar concept to cert-manager.
This distinction is only relevant if the user wants to run multiple instances of cert-manager, and so for the 'normal case' (ie. user sets up one instance of cert-manager watching one namespace) I'd rather keep the configuration surface at a minimum.
However, there are a few cases where this becomes especially important:

If a user wants one 'global' instance of cert-manager that watches all namespaces, in addition to an instance of cert-manager configured to watch just one namespace. Without this feature, both instances would process the same Issuer/Certificate resources making this a currently unsupported configuration.

If a user plans to run multiple instances of cert-manager within a single namespace, or multiple instances watching all namespaces.

These are potential scenario's for when such functionality may be useful, however I'd still like to hear if these are convincing user stories that we should address. I'm adverse to complicating the configuration surface to users if not required.
One proposed, non intrusive way this could be implemented is by allowing arbitrary key-value pairs be provided to the cert-manager CLI that filter what resources are watched. This way, we are not opinionated about what labels should be added to resources, and it saves us having 'class' as a first-class concept in our API.
Related discussion in #19 and #23
/cc @whereisaaron

Closing this as nobody has come forward with a use case yet.
If you are interested in this feature, please comment here 😄


[WFRP4e System] Possible Token Bug

When a player has its own token assigned as "Owner" he can roll his checks when he clicks his skills. But he can choose the Characteristic himself. I can understand that this is sometimes necessary but the Chatlog wont mention that another chacteristic was used to roll or that the difficulty of the roll was lowered.  I think a player could abuse this and its hard to spot as a GM.From: johnny.jofoundry: 11.315wfrp4e: 7.0.3wfrp4e-altdorf: 4.0.1wfrp4e-core: 4.0.2wfrp4e-starter-set: 4.0.0Active Modules: dd-import, levels, lib-wrapper, roofs, simplefog, stairways, theripper-premium-hub, wall-height, wfrp4e-character-details

This would be covered by https://github.com/moo-man/WFRP4e-FoundryVTT/issues/1730
So I'm closing this unless there's an actual "bug" I'm not understanding


Minimize to tray for any app

Description of the new feature / enhancement
Like always on top feature, which put any window on topm there should be a minimize to tray option that put any app to system tray
Scenario when this would be used?
Sometimes we need to always run an app but it do not have inbuild option to pin it into system tray i.e. Alexa so we have to minimize it into taskbar, that's why minimize to tray is important for power user.
Supporting information
No response

Duplicate of #47

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


Checkboxes and radio buttons are not responsive

Hey there,
I was just looking at your documentation about checkboxes and radio buttons and noticed that on mobile they are not responsive. I checked them out also in your package and noticed the same thing. What I mean is when I place the buttons on my page the horizontal scroll appears on mobile (the same as on your documentation page). Shouldn't they break in the new line or something?
OS: Android 5.0.2 Lollipop
Browser: Chrome 58.0.3029.83

Can you specify which documentation ? v3 or v4 ?

Sure, it is v4

There's no really good way to design those to be responsive in a way that suites everyone. You have to choose how and where you use certain components for however you build your site/app.


Selection replacement bug

When copying a code or plain text block from brackets to paste into another program or browser, Brackets will - once you return to the Brackets window - replace your selected block with a "-" virtually erasing your selection.
This also happens when saving. If you have a block of text selected and then save the file you're working on, without touching any keys or clicking anywhere else, the block will be replaced with a minus symbol "-" as soon as you click back into Brackets (i.e. click anywhere and this happens).
Ctrl+Z is the only way to get your text block back that I can tell. It happens frequently and seems very persistent. Once a selection is copied, it will be erased as soon as you click anywhere (to deselect for instance) and replaced with a dash "-" text entry. Super bizarre.

Cannot replicate with Brackets Release 1.6 development build 1.6.0-0 (master 239c9375f) myself on Windows 10. @noseGoblin can you post your Health Report (Help -> Health Report) and clarify which platform you are on, thanks!

It's: Release 1.5 build 1.5.0-16538 (release cf9cf4698)
build timestamp: Wed Oct 14 2015 15:08:37 GMT+0100


crypto: provide simple way to add new curves

Feature Request
Summary
currently there is not a way to add new curves to comet unless you fork the software. This causes many forks for simple changes.
Problem Definition
adding new curves forces users to fork the software. (unless im missing something)
Proposal
add simple way to add new curves to be used for consensus key signing
Zondax is proposing an api to help with this in the sdk. we could look at hosting the new api in its own module to enable usage in comet as well

to add on it would be nice to have the file system privval also take arbitrary curves. This way users dont also have to fork privval file once pluggable curves land

I think we would need a method such as RegisterKey at some place.
Now one can implement the crypto.PrivKey and crypto.PrivKey interfaces, but not sure how to set the new key for use by Comet.

i think the pubkey proto type may need some changing as the current design wouldnt allow register key without redoing the proto types

refs https://github.com/cometbft/cometbft/pull/2399


Unrevert Python Compression

This PR to be cherry picked.

Unrelated failure: https://github.com/grpc/grpc/issues/18431

Unrelated failure: https://github.com/grpc/grpc/issues/18637

Just found this Python API that might be the Pythonic way of dynamically adding test cases:
https://docs.python.org/2/library/unittest.html#load-tests-protocol


MoV code clean up

Clean up the code

remove unused parts
standardize inline documentation style

Confirmed no influence to demo 4 notebook. Merging it as it's mostly just clean up


Invalid query crashes plyql server + possible caching issues

Hey,
An invalid query was issued to plyql server from imply data distribution 1.2.1 :
plyql -c 2 -h broker:8082 -i P2Y --json-server 8099

which crashed it :
/usr/local/lib/node_modules/plyql/node_modules/q/q.js:155
                throw e;
                      ^
Error: can not serialize an approximate unique value
    at UniqueAttributeInfo.serialize (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:715:19)
    at DruidExternal.makeSelectorFilter (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:3779:38)
    at DruidExternal.timelessFilterToDruid (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:3833:37)
    at DruidExternal.timelessFilterToDruid (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:3823:37)
    at DruidExternal.makeNativeAggregateFilter (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:4534:30)
    at DruidExternal.applyToAggregation (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:4686:36)
    at DruidExternal.getAggregationsAndPostAggregations (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:4701:22)
    at DruidExternal.getQueryAndPostProcess (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:4978:64)
    at DruidExternal.External.queryValue (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:3170:48)
    at ExternalExpression._computeResolved (/usr/local/lib/node_modules/plyql/node_modules/plywood/build/plywood.js:6788:29)

It was just count query like this :
curl -X POST 'http://host/plyql' -H 'content-type: application/json' -d '{ "sql": "SELECT COUNT(bar) FROM baz" }'

bar is hll field
Isn't this a really critical bug? Server being crashed by such a common query ?
Also what is strange about it is that I was investigating why all of a sudden plyql-server returns empty results which should not be empty. And right after plyql-server  docker container restarted it was all good again.

Hey,
I can confirm that even a simple query like SELECT COUNT(bar) FROM baz crashes docker container started like this plyql -c 2 -h broker:8082 -i P2Y --json-server 8099. Isn't this a really critical bug? Server being crashed by such a common query ?

Wops forgot to update the issue. This has been fixed a long time ago. Should be returning a meaningful error now.


05 Sep 2024 - A11y Theme Builder Meeting Minutes

Date
2024-09-05 - 09:15am EDT
Zoom info

Meeting link: https://zoom.us/j/91013899295?pwd=Qnh5UVN1aUZ4ak9tODgvMVNLNUg5Zz09
Meeting id: 910 1389 9295
Password: 123225
Call-in: +1<PHONE_NUMBER> US (New York) (Check calendar for more options)

Untracked attendees

Fullname, Affiliation, (optional) GitHub username
...

Meeting notices

FINOS Project leads are responsible for observing the FINOS guidelines for running project meetings. Project maintainers can find additional resources in the FINOS Maintainers Cheatsheet.

All participants in FINOS project meetings are subject to the LF Antitrust Policy, the FINOS Community Code of Conduct and all other FINOS policies.

FINOS meetings involve participation by industry competitors, and it is the intention of FINOS and the Linux Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws. Please contact<EMAIL_ADDRESS>with any questions.

FINOS project meetings may be recorded for use solely by the FINOS team for administration purposes. In very limited instances, and with explicit approval, recordings may be made more widely available.

Agenda

[x] Convene & roll call (5mins)
[x] Display FINOS Antitrust Policy summary slide
[x] Review Meeting Notices (see above)
[x] Approve past meeting minutes
[x] Community WG Update (@PaulaPaul)
[x] Mentorship project update (@omesh-omg)
[x] AOB, Q&A & Adjourn (5mins)

Minutes

Community WG Update

Lise has been working on her presentations for Grace Hopper and All Things Open
Lise will be involved with the Duke University Innovation Q&A that is coming up
Lise has been spreading information on the capabilities of Theme Builder to groups inside of Discover

Lise would like to demo her new Figma Token work along with the new Figma release during our 09/19 call

Lise and Cody will be updating the CSS transforms, too

Lise asked some folks interested in seeing Bionic Text integration in Theme Builder to open issues

find more information on Bionic Reading here: https://bionic-reading.com/br-method/

Decisions Made

[ ] Decision 1
[ ] Decision 2
[ ] ...

Action Items

[ ] (from last week) Lise to get Omesh and Aaron CSS and responsive breakpoints and behaviors for #820 design

will provide CSS for framework only for now

[ ] (from last week)Aaron and Evan need to break the Epic that will be worked on by the mentee into detailed and digestable issues
[ ] (from last week) Lise to work with Aaron to update video in mkdocs documentation
[ ] (from last week) we need to find a location to keep Keith's recording.  Waiting on FINOS.
[ ] (from last week) Lise and Paula to meet re: Theme Builder at Grace Hopper Open Source Day
[ ] Cody to look at pre-push hook

Cody Zuschlag - Nearform

Kathleen Burkhardt - Black Rock
Lise Noble - DFS
Aaron Reed - DFS

approved in 09/05/2024 meeting, forgot to close it.

Approved in today's call

omesh kumawat - @omesh-omg


Move from gulp to webpack

We should completely move our build step from gulp to webpack 🕵

https://github.com/wulkano/kap/blob/feature/a-whole-new-world/webpack.config.js ✅

@matheuss I saw your TODO about the devtool what was the value you did set ?

@Khaledgarbaya IIRC I tried every option 😓


Bug In "Text Extractor" Feature

Microsoft PowerToys version
0.64.1
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
TextExtractor
Steps to reproduce
https://user-images.
https://user-images.githubusercontent.com/62754606/202195741-ca31319a-5913-409f-a863-7451161dbabe.mp4
I get a flicker on the screen for a few seconds after extracting the text
✔ Expected Behavior
extracting the text without getting a flicker on the screen
❌ Actual Behavior
extracting the text cause a flicker on the screen
Other Software
No response

it's working fine now.

Please let us know if it happens again. It'd be nice to understand what causes this to start happening.

I'm going to close this issue, if it returns feel free to reopen.


Add bitwise operations

Adds the bitwise operations to calculator.py and updates the relevant menu.
I've used f-strings which you seem to be purging. The warning you're getting is because you've enabled code compatability inspection, and f-strings aren't python 2 compatable. If you want to be able to run on python 2 I'll change them to .format but I can't understand why you would, and I expect that other parts of the code aren't compatible.

Everything looks good here, I will stop using .format and revert back to using f strings as well.


Same image for all in loop using external URLs

Craft 2.6.2967 and Imager 1.5.0
I have a plain text field in an entry that holds an external URL to an image
The pattern for the external image URL looks like this
example.com/news/GetImg.aspx?ArticleID=12345&ImageTypeID=2
When I loop through the images I get the first returned for every one and only one is added to S3 and renamed like so
/made/example_com/news/GetImg_c1e9e2f1c244d11eed18e13b29d61849.aspx
{% for entry in craft.entries.section('news').limit(25).find() %}

{% set marketImage = entry.newsImage %}
{% set transformedImage = craft.imager.transformImage(marketImage, {
             width: 80,
             ratio: 4/3,
            mode: 'crop',
             quality:90
}) %}

{% endfor %}

Any ideas or am I missing some basic Twig trick here?

OK so I solved my problem. Likely an edge case.
https://github.com/aelvan/Imager-Craft/blob/master/imager/models/Imager_ImagePathsModel.php#L153
_getPathsForUrl was not accounting for the query after parsing external URL. So I just built that in.
General gist and inclusion of $pathQuery
   $pathQuery = null;
    if (parse_url($convertedImageStr, PHP_URL_QUERY)){
        $pathQuery = $urlParts['query'];
    }

    $hashRemoteUrl = craft()->imager->getSetting('hashRemoteUrl');
    $hashPath = craft()->imager->getSetting('hashPath');

    if ($hashPath) {
        $targetFolder = '/' . md5($pathParts['dirname'] . $pathQuery);
    } else {
        $targetFolder = $pathParts['dirname'] . $pathQuery;
    }

    if ($hashRemoteUrl) {
        if (is_string($hashRemoteUrl) && $hashRemoteUrl == 'host') {
            $parsedDirname = substr(md5($urlParts['host']), 0, 10) . $targetFolder;
        } else {
            $parsedDirname = md5($urlParts['host'] . $pathParts['dirname'] . $pathQuery);
        }
    } else {
        $parsedDirname = str_replace('.', '_', $urlParts['host']) . $targetFolder;
    }

I feel like this should be part of the main build. I've had the same issue when loading images up via an API where the URL contains a query string.
I've fixed this a few times using @john-henry fix - But it keeps getting overwritten when I update the plugin :(

I agree, this should probably be taken into account when processing external urls. Does anyone have a couple of public urls that uses this format? It's probably more work setting up a test case with an image server that returns images dynamically, than actually implementing the fix. ;)

I can certainly send you some, but possibly not on here as they are API requests (That we then hash etc)

Ok, can you PM them to me on Slack?

This was fixed in 1.5.3.

Just installed Imager v2.1.2 on Craft 3.0.33 and getting this exact issue. Anyone experienced this with the same versions?

@JonnyTurnerUK Try using the useRemoteUrlQueryString config setting.
When enabled, any query strings in external urls will be included in the transformed filename.

Thanks for the super quick reply @john-henry. I've tried that as I thought that would be the option to use but no joy. Here's my latest config / attempt:
{% set images = value|split(',') %}
{%- for image in images -%}
    {% set newBaseImage = craft.imager.transformImage(image, {
        width: 740,
        height: 494,
        format: 'jpg',
        jpegQuality: 70,
        useRemoteUrlQueryString: true,
        cacheEnabled: false,
        cacheRemoteFiles: false
    }) %}
    {{ newBaseImage.url }}{%- if loop.last == false -%},{%- endif -%}
{%- endfor -%}

@JonnyTurnerUK I would have thought
useRemoteUrlQueryString: true,
cacheEnabled: false,
cacheRemoteFiles: false

needs to go into your craft/config/imager.php (renamed config file from plugin folder). They are config variables and not transform parameters

Yep! That'd be it. I just completely over looked it, thanks again - really appreciate it.


Cortex requirements

I'm looking for a clarification.
I initially ran the below on my host, as it didn't refer to running it inside a docker.
How should this actually be run?
before/after the docker-compose up?
which docker / host?
for I in $(find Cortex-Analyzers -name 'requirements.txt'); do pip2 install -r $I; done &&
for I in $(find Cortex-Analyzers -name 'requirements.txt'); do pip3 install -r $I || true; done

Hi @mgaulton,
Thanks for pointing out. The commands must be run on the Cortex container on the path Cortex-Analyzers is mounted to. The instruction on Wiki has been updated accordingly.


  Start two node services using forever.  Try to stop one using the process ID.    Do you not have regression tests?  This is a critical bug.

Very odd indeed, this should not have gotten past tests. @Tjatse this seems to be related to your work around stop, could you look into this?

Sorry about the confusions, the test suit does not coverage this, try to manually update your local codes like below:
https://github.com/foreverjs/forever/blob/master/lib/forever.js#L232-L236
      procs = forever.findById(target, processes)
        || forever.findByIndex(target, processes)
        || forever.findByUid(target, processes)
        || forever.findByPid(target, processes);

Actually, this is an unfinished work, there still have too much PRs from my repository, but have no time to wait for approving.
Any way, glad to help solve this, and again, sorry for the bug, guys!
My forked repository has been deleted, so, anyone could help to send a PR will be appreciated.

Hah. Sorry you're not sorry? We'll figure it out. So we should expect more bugs from your other PR?

Weird tone :(
This is totally unexpected, I just wanna to help to make this tool better, if you think I am doing a wrong thing, and curse me for that, I think I deserved.
I've figure it out, and always glad to do that. I still do the jobs but not only say Sorry, right?

No matter what happens with this issue, but someone should immediately pull the release 0.14 from npm (if that's possible) before more people upgrade and kill all their running processes!!!!

@Tjatse I don't believe @indexzero understands your tone here either. It seems you submitted unfinished work that was finished in a later pull-request. This creates mismatched expectations of what a patch consists of if it was expected to be unfinished. What he is trying to build here is trust. He likes that you are enthusiastic about helping but patience and attention to detail are crucial in a project that has many people who depend on it. A thorough review process is required and when there is not time for that, pull-requests can sit for a while. I understand this is difficult as the contributor as you want to help and see your patches land, but trust needs to be built in the beginning. Things do not always operate at a rapid pace, and that can be for good reason. If trust can be earned through the learning process that happens through open source contribution, things can move faster, but its obviously not at that point yet.
@derEremit It has been unpublished

Any news about the release ?

Somehow this re-surfaced, this time only when  is zero.

When pid is zero it terminates all processes, a serious bug, i ended up terminating all processes.

It appears that the commands stop 0 and restart 0 function exactly like stopall and restartall. This is very critical.

The problem has not gone away yet.

I can confirm that I am seeing the same behavior. Executing forever stop 0 results in all running processes being terminated.

+1
Here too, when running forever stop 0 all processes stop.

This has been fixed in the main codebase (see pull request https://github.com/foreverjs/forever/pull/756) but there hasn't been a release since then. My companies current deployment system is using a custom build including this fix which we deploy via:
npm install -g git://github.com/JamieMcNaught/forever.git

You can do similar with the current dev branch of forever using something like:
npm install -g git://github.com/foreverjs/forever.git

However with both solutions, bear in mind that either of these branches could be horribly broken when compared to a proper release (albeit my branch has been used in production for a while now and I don't intend to make any changes to it).

+1 Still seeing this,
forever stop 0 is stopping all processes
any updates?

@stevenmyhre  - have you tried with of the git repo's mentioned in my previous comment (Dec 02 2015)? I've been using this successfully for a number of projects.

No, I'll wait for the official release

This was fixed in 0.15.2


1051 Make Embed layout more dynamic and add toggle buttons on mobile …

…and web
This uses the old default value of hideList if you want the list to be hidden by default but it can be open and closed regardless so the old defaults are removed and there will only be 1 option available to people who want to embed as you can just toggle both views.

Following up on this issue, would be great to have it implemented and would improve our usability of the embeds!


Report all data load errors when not in tests

Follow up from artsy/metaphysics#1936
The above issue was tricky to debug because the error was being swallowed by renderWithLoadProgress.
renderWithLoadProgress does invoke console.error but only for errors with a specific shape. It turned out that our error yesterday didn't match that shape. So I moved the console.error invocation to make it more inclusive for all errors 🌈
#trivial

#MergeOnGreen


hook the notify function for the screenshare #71

I just tested this on my device and works as expected. +1


Replacements for api.pre_update and api.pre_create events

Hi,
in my application, I need to replace data prior writing entities to the database. Previously, I used the api.pre_update and api.pre_create events to do so.
Are there any replacement events in master I can use?
Verbose use case:

Users upload images to the application. They are created as TemporaryUploadedFile entities and passed back to the frontend
The frontend attaches those TemporaryUploadedFile entities to the objects which are about to be created/updated
The frontend sends the new object including the TemporaryUploadedFile entity to the server
The server used to listen to all create and update events and replacing the TemporaryUploadedFile entities with the correct file entities

The design goal here is to don't create "floating" file entities, which would have to be removed afterwards if the user decides not to save the entity.
Thanks
Felicitus

Can't you register two listener, before and after the ManagerViewListener in order to do so ?

I don't really understand your use case, but still, if you use Doctrine, you can still implement event listeners  => http://doctrine-orm.readthedocs.org/en/latest/reference/events.html#implementing-event-listeners

@ogizanagi In fact, one listener was enough. I will post my solution later.
@dupuchba The issue is that I can't submit the file data itself. So the user uploads a file and receives a reference to the uploaded file. Directly creating an attachment to the entity is bad practice, because the user should be able to cancel editing any time.
Example: I have a Product which may have images. The user retrieves the product with no images. The user adds an attachment by uploading it to the server and receives a ´TemporaryUploadedFile. Now when the user clicks "save", the frontend transmits the changes Producthaving theTemporaryUploadedFilein theimagescollection. However, theProductentity has a relation which only allowsProductImagesin the collection, causing Doctrine to throw an error. Thus my listener replaces anyTemporaryUploadedFilewith actual instances of the typeProductImage`.

My solution can be found here:
TemporaryFileEventListener.php
config.yml
@dunglas A flowchart of program execution could be helpful for the documentation, especially with the event system. A few hours, I was unaware of that DunglasApiBundle currently hooks in via the event system, and users can actually modify the serializer results prior doctrine takes over. What do you think?

@felicitus I agree and I'll do it but I prefer to wait the end of development cycle of the v2 because big change will still occur.

The event system has been entirely rethought and is now documented: https://github.com/api-platform/docs/blob/master/core/events.md


openmp 16.0.0.rc1

Follows https://github.com/conda-forge/llvmdev-feedstock/pull/195; osx didn't get built yet due to failing tests, but will continue with the rest of the arches in the meantime.
Builds on #70, which didn't get merged in the 15 cycle, so the rc branch is still at 14...

So, compared to the errors about PRIxPTR (which apparently isn't available in the glibc 2.12 inttypes.h[^1]) before https://github.com/conda-forge/openmp-feedstock/pull/77/commits/4918835bcd46de1356a72c1da16dc6fa0244d636, we now get undefined reference to<EMAIL_ADDRESS>I think this is due to this. We may be hitting the limits of what's possible with glibc 2.12 here.
[^1]: the compiler says note: 'PRIxPTR' is defined in header '<cinttypes>'; did you forget to '#include <cinttypes>'?, but it's already included; one file contains the superset #include <cstdint>, and the other contains what should be the C-equivalent #include <inttypes.h>

@conda-forge-admin please rerender

i think this helped.

Thanks @hmaarrfk !
Doesn't this mean that we'd need a run-dependence on the newer sysroot here?

Doesn't this mean that we'd need a run-dependence on the newer sysroot here?

@isuruf, maybe you could give some guidance here?

Just add -Wl,--allow-shlib-undefined to link flags when linking. We do this in the compiler activation flags

Just add -Wl,--allow-shlib-undefined to link flags when linking. We do this in the compiler activation flags

Thanks, that did the trick! Anything else here you'd like to review/comment on? (use of sysroot 2.17, move to cos7, ...)
Otherwise I'm planning to merge this later today (it's only for the rc-channel anyway, we'll still have a deeper review once 16 goes GA).


Create Change task

Thanks team on working on my last request (Create change request), I must be more demanding, if possible can you please create same for change tasks as well

Hello I tried and changed the config can you please check, it is working for me

Hello Team,
I am new to github as well as to programming, Do we have any template to create the change task in service now?

What about just Get-ServiceNowChangeTask ?
I don't see any method to get information about CTASK items. Change requests are prefixed with CHG for the number. Change tasks are prefixed CTASK for the number.

What about just Get-ServiceNowChangeTask ?
I don't see any method to get information about CTASK items. Change requests are prefixed with CHG for the number. Change tasks are prefixed CTASK for the number.

What about just Get-ServiceNowChangeTask ?
I don't see any method to get information about CTASK items. Change requests are prefixed with CHG for the number. Change tasks are prefixed CTASK for the number.

So I believe I figured out some bit of RestAPI with a proof of concept to "Get-ServiceNowChangeTask" (change XXX to your environment):
#CTASK number
$num = "CTASKXXX"
#Set variable for credentials
If (-not($cred))
{$cred = Get-Credential}
#Specify endpoint uri for ServiceNow
$uri = "https://XXX.service-now.com/api/now/table/change_task?sysparm_query=number=$Num"
#Specify http method
$method = "get"
#Send http request
$output = (invoke-restmethod -method $method -uri $uri -ContentType "application/json" -Credential $cred).Result
#Print Response
$output

Get-ServiceNowRecord can be used to get table info on any table if there isn't a specific function to do so.  For example, Get-ServiceNowRecord -Table change_task -Filter @('number', '-eq' $num)

I'd like to get away from having a different Get- function for every table.  For 1, there are hundreds of tables and it feels like a losing battle.  And 2, the only difference between each function is the table name.  I'd like to use Get-ServiceNowRecord with a tab-ahead perhaps or something similar for -Table to make it easier for folks and have everyone use that 1 function.  Thoughts?

Thank you @gdbarron - I have converted my scripts to use this new logic Get-ServiceNowRecord -Table instead.

This should be marked as closed or resolved.

To be fair to @MorningsWait and @Chandresh88, the original intent of this issue was to 'create' a change task, not 'get'.  I should have suggested we create a new one for 'get', but failed to do so.  I'll keep this open until the 'create' aspect is complete.

I just used the following to create a CTASK and it worked fine.
`
#Variables
$short_description = 'Short Description here'
$description = 'Description here'
$assignment_group = 'assignment group here'
$assigned_to = 'assigned to here'
$requested_by = 'requested by here'
#Splatting the variables to the Cmdlet
$Params = @{
short_description           = $short_description
description                 = $description
assignment_group            = $assignment_group
assigned_to                 = $assigned_to
requested_by                = $requested_by
}
$Record = New-ServiceNowRecord -Table 'Change Task' -Values $Params -PassThru
Write-Output -InputObject $Record.number
`

I believe this still needs to be worked on as well as we can do all 3 tasks get/modify/Delete

@MorningsWait, do you have a suggestion as to which change task fields are most used so we can include them in a New-ServiceNowChangeTask?  According to the dev instance, looks like short description and description are required and I assume we'll need the change request number.  What else are you thinking?

@MorningsWait following up...

I am sorry for responding late, Short Description, Description, Parent CHG, Assignment Group & Assigned To the


Plugin is not working

Hello,
first, thanks for taking the effort upon you to come up with a plugin to connect those two great apps. I really appreciate it.
Unfortunately, it does not work for me. I am running:
Windows 11 (Build 22631.3155)
npcap 1.79
SteelSeries GG 58
Touch Portal v4.1 build 2
Connector 1.1.0
I have all the actions ( but no Events) and are able to configure the buttons (sliders and mute button) as usual. I used the button state to toggle between a muted and unmuted icon.
Unfortunately, the sliders and buttons do nothing (except changing the icon). I was able to configure a keyboard shortcut and configure it on the mute button. That works just fine.
Maybe I am overlooking something, but as far as I can tell, anything should be setup correctly, therefore I am asking for help.
Thanks a lot in advance!

Hello,
Have tried restarting Touch Portal and

Hello,
Have you tried to restart Touch Portal and/or the plugin ?
When installing the plugin, did you click on "Always Trust" ?
If you try to add a dynamic text to a button, like the volume, does the volume appears ? Or a long weird text ? In that case, the plugin is not properly installed or running, so try to reinstall the plugin
Finally, if you use the streamer mode of SteelSeries GG, make sur to use streamer mode sliders.

So, I don't receive any gibberish, but the button is just empty. I went with this:

The slider setting looks like this:

Those are the available options I have with buttons:

I just did a reimport and made sure that I hit "Always trust", restarted TP and tried to create another touch button - no success.
I am running both TP and Sonar on German settings. Might this be an issue here?

Actually, it's normal that the button remains empty, you need to look at the button on your phone/tablet.
But when you use the sliders in streamer mode, the volume should change but because of a bug, the sliders in SteelSeries don't move.
Can you try using the classic mode ?

<- Stupid.
While crafting my VERY elaborate message on my findings I tried to fix the issue by rebooting. Guess what happens to my response... :-/
So, I was lacking a .NET runtime. I found that in the logfile after doing a reimport of the plugin. So maybe you want to add a link to https://dotnet.microsoft.com/en-us/download/dotnet/7.0/runtime?cid=getdotnetcore&os=windows&arch=x64 as well.
Now I am receiving the following message:

Directly after the import, I saw some text in the touch portal button on my tablet. So there is some connection established and listening seems to work.
I hope this helps.

For whatever reason, the plugin stops working in between and the touch buttons for the chat track doesn't seem to work at all.
Redirections won't work for Chat and Microfone.
When using the plugin for a while (<10 minutes)
I am able to provoke this, which will terminate the connection:
12:11:00.438 - [ERROR] [Plugin Instance] Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
12:11:00.439 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesPluginMain.BooleanToMuteState(String state) in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesPluginMain.cs:line 250
12:11:00.439 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesPluginMain.OnSteelSeriesEventHandler(Object sender, OnSteelSeriesEventArgs eventArgs) in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesPluginMain.cs:line 241
12:11:00.439 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesAPI.SteelSeriesHTTPHandler.<>c__DisplayClass8_0.<StartSteelSeriesListener>b__1(Object s, PacketCapture e) in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesAPI\SteelSeriesHTTPHandler.cs:line 109
12:11:00.439 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.SendPacketArrivalEvent(PcapHeader header, Span`1 data)
12:11:00.439 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.PacketHandler(IntPtr handlePtr, IntPtr header, IntPtr data)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.LibPcapSafeNativeMethods.pcap_dispatch(PcapHandle adaptHandle, Int32 count, pcap_handler callback, IntPtr ptr)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.CaptureThread(CancellationToken cancellationToken)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.Capture(Int32 packetCount)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.Capture()
12:11:00.440 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesAPI.SteelSeriesHTTPHandler.StartSteelSeriesListener() in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesAPI\SteelSeriesHTTPHandler.cs:line 142
12:11:02.941 - [ERROR] [Plugin Instance] Plugin Connection (1) has been incorrectly closed...
12:11:02.942 - [LOG  ] [Plugin Instance] Connection has been closed... Plugin: steelseries-gg

For whatever reason, the plugin stops working in between and the touch buttons for the chat track doesn't seem to work at all.
Redirections won't work for Chat and Microfone.
When using the plugin for a while (<10 minutes) I am able to provoke this, which will terminate the connection:
12:11:00.438 - [ERROR] [Plugin Instance] Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
12:11:00.439 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesPluginMain.BooleanToMuteState(String state) in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesPluginMain.cs:line 250
12:11:00.439 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesPluginMain.OnSteelSeriesEventHandler(Object sender, OnSteelSeriesEventArgs eventArgs) in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesPluginMain.cs:line 241
12:11:00.439 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesAPI.SteelSeriesHTTPHandler.<>c__DisplayClass8_0.<StartSteelSeriesListener>b__1(Object s, PacketCapture e) in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesAPI\SteelSeriesHTTPHandler.cs:line 109
12:11:00.439 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.SendPacketArrivalEvent(PcapHeader header, Span`1 data)
12:11:00.439 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.PacketHandler(IntPtr handlePtr, IntPtr header, IntPtr data)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.LibPcapSafeNativeMethods.pcap_dispatch(PcapHandle adaptHandle, Int32 count, pcap_handler callback, IntPtr ptr)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.CaptureThread(CancellationToken cancellationToken)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.Capture(Int32 packetCount)
12:11:00.440 - [ERROR] [Plugin Instance]    at SharpPcap.LibPcap.PcapDevice.Capture()
12:11:00.440 - [ERROR] [Plugin Instance]    at TPSteelSeriesGG.SteelSeriesAPI.SteelSeriesHTTPHandler.StartSteelSeriesListener() in D:\Dev\touchportal-plugins\TPSteelSeriesGG\src\SteelSeriesAPI\SteelSeriesHTTPHandler.cs:line 142
12:11:02.941 - [ERROR] [Plugin Instance] Plugin Connection (1) has been incorrectly closed...
12:11:02.942 - [LOG  ] [Plugin Instance] Connection has been closed...
I'll take a deeper look later this afternoon


Adds sensors and services to RainMachine

Description:
Adds several (binary) sensors to the RainMachine component:

Binary Sensor: Extra Water on Hot Days
Binary Sensor: Freeze Protection
Binary Sensor: Freeze Restrictions
Binary Sensor: Hourly Restrictions
Binary Sensor: Month Restrictions
Binary Sensor: Rain Delay Restrictions
Binary Sensor: Rain Sensor Restrictions
Binary Sensor: Weekday Restrictions
Sensor: Freeze Protect Temperature

Additionally, this PR adds several services:

start_program
start_zone
stop_all
stop_program
stop_zone

Related issue (if applicable): N/A
Pull request in home-assistant.github.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.github.io/pull/5322
Example entry for configuration.yaml (if applicable):
rainmachine:
  ip_address: <IP_ADDRESS>
  password: abc123

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:
~- [ ] New dependencies have been added to the REQUIREMENTS variable (example).~
~- [ ] New dependencies are only imported inside functions that use them (example).~
~- [ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.~
~- [ ] New files were added to .coveragerc.~

FYI: been running this in my local HASS for a week or so and everything seems to work fine.

@balloob I’m sure you’re slammed, but now that 0.70 is out, can we get this stitched up and out the door? Let me know what else you’d like done on this one.

Thanks @MartinHjelmare! I don't recall if that's the only thing I did in that commit. At any rate, chalk it up as a lesson going forward.

@MartinHjelmare FYI, I tried adding a single commit to the base of this branch in which I moved homeassistant/components/rainmachine.py to homeassistant/components/rainmachine/__init__.py (and then rebasing this PR on top of that); no change, unfortunately.


DependsOn

I may just be having trouble, but it would be great to have "DependsOn" working with serverless for the more complicated deploys.
For instance, if I wanted to create a LambdaRole and Lambda I could say
Transform:  AWS::Serverless-2016-10-31
Resources:
Lambda:
  DependsOn: LambdaRole
  Type: AWS::Serverless::Function
    Role: !Ref LambdaRole
LambdaRole:
  Type: AWS::IAM::Role

Right now I'm getting a "Circular Dependency" issue. I'm sure it's supported but I wanted to check with you to make sure I'm not doing anything wrong.

Agreed. DependsOn is a good feature to have. But I don't see how this template would create a circular dependency issue. Can you provide the full template?

I take it that AWS::Serverless::Function does not support DependsOn At this time?  In my CloudFormation stack, I have a section like
  LambdaRole:
    Type: "AWS::IAM::Role"
    Properties:
      AssumeRolePolicyDocument:
        Version: "2012-10-17"
        Statement:
          -
            Effect: "Allow"
            Action:
              - "sts:AssumeRole"
            Principal:
              Service:
                - "lambda.amazonaws.com"
  LambdaRolePolicy:
    Type: "AWS::IAM::Policy"
    Properties:
      PolicyName: "LambdaRolePolicy"
      PolicyDocument:
        Version: "2012-10-17"
        Statement:
          -
            Effect: "Allow"
            Action:
              - "logs:CreateLogGroup"
              - "logs:CreateLogStream"
              - "logs:PutLogEvents"
            Resource:
              - "*"
      Roles:
        -
          Ref: "LambdaRole"
  Lambda:
    Type: "AWS::Serverless::Function"
    Properties:
      Role:
        !GetAtt LambdaRole.Arn
      Handler: lambda_function.lambda_handler
      Runtime: python2.7
      Timeout: 15
      CodeUri: "s3://..."
    DependsOn: LambdaRolePolicy

When I run my CloudFormation stack, the Lambda starts creating immediately after the LambdaRole finishes creation, and before the LambdaRolePolicy event starts it's creation, thus, I get errors like
Error Message: User: arn:aws:sts::***:assumed-role/LambdaRole-***/Lambda-*** is not authorized to perform: logs:CreateLogGroup on resource: arn:aws:logs:us-west-2:***:log-group:/aws/lambda/Lambda-***:log-stream:```

Is there a work around to make the lambda function wait until the policy has been attached to the role?

Alternatively, you could make use of Managed Policy and not have an extra resource in here:
    LambdaRole:
      Type: "AWS::IAM::Role"
      Properties:
        # Managed Policy include required Lambda Execution Role permissions
        ManagedPolicyArns:
            - "arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
        AssumeRolePolicyDocument:
          Version: "2012-10-17"
          Statement:
            -
              Sid: "AllowLambdaServiceToAssumeRole"
              Effect: "Allow"
              Action:
                - "sts:AssumeRole"
              Principal:
                Service:
                  - "lambda.amazonaws.com"

    Lambda:
      Type: "AWS::Serverless::Function"
      Properties:
        Role: !GetAtt LambdaRole.Arn
        Handler: lambda_function.lambda_handler
        Runtime: python2.7
        Timeout: 15
        CodeUri: "s3://..."

Removed DependsOn in Serverless::Function as it's not supported and merged Policy and Role by making use of AWSLambdaBasicExecutionRole that does exactly the same that IAM policy above.
That should allow Lambda to assume just fine and add additional policies right after if you need to. Give it a try and let us know here :)
Hope it helps

Thanks @heitorlessa, I also discovered an alternative method.
The following is also equivalent to what I put in originally:
  Lambda:
    Type: "AWS::Serverless::Function"
    Properties:
      Handler: lambda_function.lambda_handler
      Runtime: python2.7
      Timeout: 15
      CodeUri: "s3://..."

This works because the AWS::Serverless::Function actually generates a number of CloudFormation resources. In the case above, it will generate a role for the function, because one is not explicitly assigned, and it will attach the AWSLambdaBasicExecutionRole to it.
Further, for anyone else facing a similar issue, if you need to attach custom permissions, you can do so using:
  Lambda:
    Type: "AWS::Serverless::Function"
    Properties:
      Handler: lambda_function.lambda_handler
      Runtime: python2.7
      Timeout: 15
      CodeUri: "s3://..."
      Policies:
        -
          Version: "2012-10-17"
          Statement:
            -
              Effect: "Allow"
              Action:
                - "s3:GetObject"
              Resource:
                - "arn:aws:s3:::<some_bucket>/*"

This will create a custom role, with the AWSLambdaBasicExecutionRole Policy attached, and also attach inline policies. In the case above, this will create an inline policy that grants the lambda access to get objects from <some_bucket>.
Internally, it appears that AWS::Serverless::Function seems to use AWS::IAM::Role to generate the role for the function, and passes in your defined list of Policiesto the rolesPolicies` document, and waits for it to complete before creating the lambda function that uses it.

What would be a good way of attaching multiple managed policies to this?

@jrstarke  that's right, AWS::Serverless::Function does create implicit resources including an IAM Role (and Lambda Permissions in case you define an API). Best way to capture these resources as documentation gets updated is to simply describe resources that Cloudformation created.
My suggestion on having an IAM role within the template is that it allows you the flexibility to add Managed Policy (multiples) and Inline policies at the same time since it's a cloudformation resource and not a SAM translation, of which also helps in maintainability for those reading and using that SAM template - In fairness, it's a judgement call as both get the job done :)
@ktruckenmiller Following the example above you can add multiple managed policies as follows - Simply include the full ARN whether that be customer or AWS managed policy:

    ServiceXLambdaRole:
      Type: "AWS::IAM::Role"
      Properties:
        # Managed Policy include required Lambda Execution Role permissions
        ManagedPolicyArns:
            - "arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
            - "arn---policy2"
            - "arn---policy3"
        AssumeRolePolicyDocument:
          Version: "2012-10-17"
          Statement:
            -
              Sid: "AllowLambdaServiceToAssumeRole"
              Effect: "Allow"
              Action:
                - "sts:AssumeRole"
              Principal:
                Service:
                  - "lambda.amazonaws.com"

AWS::Serverless::Function can take multiple Managed Policy names. Like this:
Type: AWS::Serverless::Function
Properties:
    Runtime: nodejs4.3
    Policies:
        - AWSS3FullAccess
        - AWSDynamoDBFullAccess

SAM will automatically add LambdaBasicExecution role to the permissions.
Above will work for AWS Managed Policies. If you are attaching your own Custom Managed policy, you need to follow @heitorlessa's example above. Create your own IAM role and attach the role to SAM function

To get the conversation back on track: without DependsOn support there is no real way to build bootstrapping actions in SAM. This needs to be implemented and should be marked as a bug not a question.

Yup. Supporting DependsOn is a feature request. We will get it prioritized

Hi @jrstarke, You may also create a cloudformation stack with the Role and export it to use it in another Cloudformation stack where you will create your Lambda Function
First cloudformation stack
Outputs:
  FunctionRole:
    Description: Role to be used.
    Value:
      !GetAtt LambdaRole.Arn
    Export:
      Name: !Sub "${AWS::StackName}-Role"

In the second cloudformation stack
Parameters:
  LambdaRoleStackName:
    Description: Name of an active CloudFormation stack that contains the Role, that will be used in this stack.
    Type: String
    MinLength: 1
    MaxLength: 255
    AllowedPattern: ^[a-zA-Z][-a-zA-Z0-9]*$
    Default: SampleCrossStack

Resources:
  LambdaFunction:
    Type: 'AWS::Serverless::Function'
    Properties:
      Role:
        Fn::ImportValue: !Sub "${LambdaRoleStackName}-Role"

However, I prefer your workaround ;)

+1 for DependsOn support.
Also, some documentation on the implicit resources created would be helpful (I just got tripped by a MyLambdaMyLambdaSchedulePermission generated by a MyLambda Serverless::Function with MyLambdaSchedule as an Event)

Perhaps it belong to another issue, but Conditional creation of Function should be supported as well.

SAM resources should now supports DependsOn. Can you check?
Condition is coming soon as well.

I didn't get an error using the DependsOn attribute, but it didn't seem to be effective.

@ShaunEgan  What do you mean? Did the DependsOn property get preserved in the transform?

To other people who ends up here, "DependsOn" works, just put it after "Type".

@sanathkr I tried DependsOn but could not get it to work.  Are you sure the DependsOn is also copied to the LambdaFunction Role resource?
When the role creation depends on creation of another resource, that resource is not created before the lambda function role is created.

So the DependsOn is not propagated to the Role resource. Do you want to create some resource first, followed by the IAM Role, then the Lambda function?
If the IAM role contains !Ref or !GetAtt on the resource, then CloudFormation will implicitly add a dependency between them. Is this not the case?

To create the generated Role, the cloudformation role needs to have the iam:CreateRole policy.
I am adding that policy to the cloudformation role in the template, just before the serverless function.
With the dependency, the Lambda role is created before cloudformation role has the policy to do that.
Thanks for the hint aboud !Ref and !GetAtt. There may be a way to create the !Ref in the Lambda Policy definition without actually using it.

By "CloudFormation role", do you mean the role you use to deploy the stack with? More precisely, the IAM user calling the CloudFormation deploy command would have this role. If this is the case, you should move this role outside of the stack. Otherwise it is impossible to create a least-privilege IAM user that can update your app stack.
Best practice is to create a "infrastructure stack" that creates the IAM Roles/users necessary to manage your "application stack". App stacks contain Lambda function/api gateway etc. With Stack Policies, you can allow only certain users to manage your infrastructure stack. This is mostly administrators. Application developers can be given full access to manage app stacks, without needing admin permissions.

Yes. In our situation the deploy command is and shall only be excuted by CodePipeline. In understand your point, but the content of the stack template is tightly coupled with the required permissions of it's deployment role. things become quite unmanageable if they are dislocated.  To restrict persimssons you do want a separate role for each stack I presume?
I am trying to find a way to implement your proposed 'best practice' in a scalable way.
Scernario: an application developer adds a lambda function to an application template.
He needs to add a whole range of lambda persissions to the ExecuteChangeSet service role used by the codepipeline and it needs to be automated.
I guess you suggest there'd be another stack managing that role. However, the template for that stack would be in yet another repo the needs to be redeployed before the application stack can be deployed?


[v0.4.2] Release Tracker

We will be cutting a 0.4.2 release on top of the 0.4.1 branch.
It will contain the new C++ video reader backend, which improves the speed for decoding videos significantly. Due to packaging issues of ffmpeg, for users to be able to benefit from this faster version they will need to compile torchvision from source (and they need to have ffmpeg available in their system).
By comparing all the PRs that were merged since 0.4.1 (link available at https://github.com/pytorch/vision/compare/v0.4.0...master), here is the proposed list of PRs that will be integrated in the 0.4.2 release.

[video reader] inception commit (#1303) 31fad34
Expose frame-rate and cache to video datasets (#1356) 85ffd93
Expose num_workers in VideoClips (#1359) 02a8c0a
Fix randomresized params flaky (#1282) 7c9bbf5
Video transforms (#1353) 64917bc
add _backend argument to init() of class VideoClips (#1363) 7874374
Video clips workers (#1369) 0982395
modified code of io.read_video and io.read_video_timestamps to intepret pts values in seconds (#1331) 17e355f
add metadata to video dataset classes. bug fix. more robustness (#1376) 49b01e3
move sampler into TV core. Update UniformClipSampler (#1408) f0d3daa
remove hardcoded video extension in kinetics400 dataset (#1418) 929c81d
Fix hmdb51 and ucf101 typo (#1420) b13931a
fix a bug related to audio_end_pts (#1431) 1258bb7
expose more io api (#1423) e48b958
Make video transforms private (#1429) 79daca1
extend video reader to support fast video probing (#1437) ed5b2dc
Better handle corrupted videos (#1463) da89dad
Temporary fix to remove ffmpeg from build time (#1475) ed04dee
fix a bug when video decoding fails and empty frames are returned (#1506) 2804c12
extend DistributedSampler to support group_size (#1512) 355e9d2
Unify video backend (#1514) 97b53f9
Unify video metadata in VideoClips (#1527) 7d509c5
Fixed compute_clips docstring (#1543) b438d32

cc @stephenyan1231 for review

Here is the cherry-picked branch for the aforementioned commits: https://github.com/pytorch/vision/tree/v0.4.2
I had to fix a few minor conflicts on the way. Here is the list of commits that had conflicts:

[video reader] inception commit (#1303) 31fad34
add _backend argument to init() of class VideoClips (#1363) 7874374
Temporary fix to remove ffmpeg from build time (#1475) ed04dee
Unify video backend (#1514) 97b53f9
Unify video metadata in VideoClips (#1527) 7d509c5


SEO Issue with tags and categories.

Hi all,
I'm having an SEO issue with my umbraco articulate blog and seo. I'm finding that Google is caching my tags and categories too, which causes duplicate title and description meta tags as the main article page.
Any ideas as how to prevent the caching. I don't know how I can incorporate this into the robots.txt since it would be redundant and arduous to continuously add each new individual tag and category to the robots.txt file after a new one is created.
I also do not see it as useful to add the meta do not follow since there are no individual tag pages - they are created from the search in articulate.
Any ideas?

What you do is disallow in your robots.txt like so:
disallow: /articles/tags/
disallow: /articles/ categories/


Fix Heroku deploys

Probably due to a Postgres update, the database now is more restrictive when it comes to changing a column's type. The Heroku deployment failed for this reason. I have already confirmed this fix to be working by manually applying it to the currently deployed version. You'll see it running over here: http://workshopportal.herokuapp.com/
Things left to do:

[ ] Verify this doesn't break SQLite migrations. (i.e. destroy your SQLite database and see if rake db:migrate works afterwards)

Things out of scope of this PR:

[ ] Update README links
[ ] Setup deployment for dev branch as well
[ ] Verify SendGrid mailing works
[ ] Remove old errbit bindings (see config/initializers/errbit.rb)

Changes Unknown when pulling 18e0dfe57c62265c472efc5de6af3cf0e4f4569f on fix-heroku into ** on dev**.

Brilliant catch! 💾
rake migrate doesn't work at the moment due to the error 2 is out of range for ActiveRecord::Type::Integer with limit 0. I can fix this later.
I opened an issue for the remaining tasks, #655.

Coverage remained the same at 100.0% when pulling afff9a42994c2a61353c2a841b0abe67dc31e01f on fix-heroku into 288494c3bd04460e741dd1adae93a5ad5cc87cd3 on dev.

Coverage remained the same at 100.0% when pulling 614864bd36a43919ef8e52e4dcc87eb230f03a88 on fix-heroku into 288494c3bd04460e741dd1adae93a5ad5cc87cd3 on dev.

Should we really run migrations on deployment? Shouldn't we rather use db:setup in the release version? If we want to see whether it succeeds I'd rather write a test (if possible) that tries migrating with postgres.

Oh, right!
Bjarne Sievers<EMAIL_ADDRESS>schrieb am So., 28. Mai 2017,
20:08:

@bjrne commented on this pull request.
In app.json
https://github.com/hpi-schuelerklub/workshop-portal/pull/654#discussion_r118846308
:

@@ -1,6 +1,7 @@
{
"name": "workshop-portal",
"scripts": {

"postdeploy": "bundle exec rake db:schema:load db:seed"

Depending on when this setting applies we might want to add
db:populate_sample_data
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hpi-schuelerklub/workshop-portal/pull/654#pullrequestreview-40677089,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACC7RL_GOJoq_AP3CAR5EjBXXKN9_7oiks5r-bgGgaJpZM4NoryF
.

--
Sent from my phone.

Coverage remained the same at 100.0% when pulling 538f7f17bf163b17c79f8a4529d3c3bea35c47d2 on fix-heroku into 288494c3bd04460e741dd1adae93a5ad5cc87cd3 on dev.

Coverage remained the same at 100.0% when pulling 79ec56d6040dc790f75cb6eb119c95330775cbb5 on fix-heroku into 87ef77560d0fa1f6b658e07800dfa913f37d598a on dev.

Coverage remained the same at 100.0% when pulling 9e65724fcc66f92362e15b3266e3f02c32640328 on fix-heroku into 2f5695c361979fdd51ef40226012b9fef08ce10d on dev.


Simplify Initialization of Modelica.Thermal.FluidHeatFlow.Interfaces.Partials.TwoPort

The initialization of extended models from Modelica.Thermal.FluidHeatFlow.Interfaces.Partials.TwoPort would be much easier if the variable h get as modification of the start attribute
Modelica.SIunits.SpecificEnthalpy h(start=medium.cp*T0) "Medium's specific enthalpy";
Ad least the example Modelica.Thermal.FluidHeatFlow.Examples.PumpAndValve would benefit from the additional information.

@FrenkelJ can you open a ticket on the MSL issue tracker?

This is the MSL issue tracker. trac.modelica.org is read-only for MSL components.

@casella Both the MA Webmaster, @AHaumer, @MartinOtter and me sent several mails that the MSL is now developed on GitHub, including issues. It also was tweeted by https://twitter.com/modelica/status/820906213806374912 two weeks ago.

Sorry folks, the management of the Modelica Conference papers obviously hit hard on me. Please be patient with me and allow me to catch up again soon.
That said, I agree with @FrenkelJ proposal

Also cherry-picked to maint/3.2.2 branch.


Add a navigation bar

As the portfolio gains more information, it may make sense to add a navigation bar. This bar should include the logo on the left side but only once the page has scrolled down a bit.

@khattakdev we would like to work on this issue :)


What video course do you want to see next?

Please 'thumbs up' the relevant comment below, thank you.
Note: This is for my next video course in 2020

Update: There's now 4 hours of video content on ▶  ModernDevTools.com!

Learn Browser Automation
In this video course, you will learn browser automation from the ground up. We'll cover the following, and more, using Node.js and JavaScript:

The basics of how to automate Chrome, Firefox and Safari, using tools like puppeteer
End-to-end browser testing
Advanced browser testing techniques, e.g. how to verify your website really sent that 'password reset' email
How to scrape data from other websites and services
How to perform automated performance testing
How to perform automated accessibility testing
Automating browsers on different infrastructure, like GitHub Actions, a $5 VPS, your own machine and a Raspberry Pi
Common pitfalls with browser automation

Modern Developer Workflows
Speed up your developer workflows from start to finish. This structured course includes:

Automating and optimising workflows from your terminal
Tips and tricks for your code editor (how to organise projects, which plugins to use, setting up snippets)
Development workflows worth adopting (live reload, browser sync, hot module reloading)
Rapidly develop Node.js-based apps
Debugging workflows using DevTools

DevTools Back to Basics
While ModernDevTools.com has some more intermediate level concepts, this new course helps you pick up the absolute basics of DevTools.
This course will take you through each panel, one by one, so you can learn exactly what it does, but more importantly how to use it for your own websites and projects.

Modern CSS
This is structured course on modern layout techniques with CSS, and some other relatively new features:

CSS Flexbox
CSS Grid
Other new CSS features like position: sticky, aspect-ratio, :is(), :where(), the CSS Paint API and more

Build and deploy Node.js apps
Do you want to learn how I rapidly build and iterate on Node.js-based projects? If so, this course is for you. For more context, see my talk Practical Mini-Projects in Node.js
This course covers different Node.js applications which you can build and deploy on a VPS, or other similar services:

A database powered visualisation
Automating a headless browser to scrape a website
A Node.js app to handle payments via Stripe
A Node.js app to interact with third party APIs
Deployment strategies using Git hooks and/or GitHub Actions
Lessons learned with working on Node.js projects

Dynamic web applications without a JavaScript framework
Create a dynamic web application without a JavaScript framework.
JavaScript frameworks are great - they make our lives easier and help with not re-inventing the wheel. But I've identified a potential need in the community for learning how to architect complex web applications, using vanilla JavaScript. E.g.

How can you update 3 parts of the page, at the same time, using vanilla JavaScript, without turning it into spaghetti code
How can you modularise your frontend codebase
JavaScript design patterns to adopt instead of using a framework

Offline Websites with Service Works
This courses teaches you Service Workers from the ground up, so you can ensure your website is offline friendly. There are many pitfalls when designing for offline, so this course includes lots of recipes you can quickly use to get started.
Service workers also includes capabilities for background sync and push notifications, which this course will cover.

Modern JavaScript
Learn the latest JavaScript APIs and how they can help you build faster, and more feature rich products. This course includes Modern JavaScript in both the browser, and Node.js.


Update notes_tie.lua

Amending plugindef() to fix multi-line error and incorporate GroupName and GroupDescription.

Same thing here as with https://github.com/finale-lua/lua-scripts/pull/301.
It's not that I don't think these are worthwhile changes (they are). I'm just closing because I don't want someone to accidentally merge these and mess things up.

I'm just closing because I don't want someone to accidentally merge these and mess things up.

Of course @Nick-Mazuk. There was such mis-matching going on in my local GItHub that I deleted the whole local repo and started again. But the multiple mis-matching continues and I don't know how to fix it. Happy to start again from scratch!

I see. It looks like it's not your local clone on your computer, but rather the fork on your profile.

I believe there should be a button that says "Fetch upstream" or something on the fork's page? I won't be able to see the button but you should.
https://github.com/cv-on-hub/lua-scripts


Fixed some typos

@Pieter-Veenstra,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks Pieter, we appreciate your efforts.


Add New Audience Items

We would love to have an option to create our own Audience items for the drop-down to match our Employee's Roles.  This would allow our users to then find training materials related to them.

@luvsql, Thank you for your suggestion. Please allow us some time, we will provide you update on this ASAP.

@luvsql That is a great suggestion. We weighed this when we were architecting the solution. While we could technically make this possible we elected to not do this due to potential performance impacts. This is an open source project so if you are a developer or have access to a development team you are welcome to try and incorporate that. I would be happy to provide some guidance on this if you wanted to give this a try and submit a pull request.

If we can't add, can we edit the existing descriptions?

You can't edit the descriptions on the Microsoft provided content unless you make a copy of the playlist. That is certainly supported but know that if you do that and Microsoft updates that playlist you will not get the updates automatically.

Here is why it's so crucial for people.  When we post a bunch of training videos, not all Employees need to watch all of them.  We'd like to avoid having to create unique Categories per Role (because then the content is just repeated).
You have one category listing all the videos and if I'm say a Project Manager, I select my role which is the audience and all the videos I need to watch are then filtered.  Currently we will have to list out in an email all the titles they need to watch per role.

You can create a separate playlist for each role (eg project manager training) and create a SharePoint page for each role. On that page add the learning pathways webpart and set it to only show the project manager playlist. This way you can have them only see the trainings you want them to see. You can even host that page on a different SharePoint site like the PMO site. You can do what your asking for just not with a custom audience.

I'm also going to be creating an app in Teams for this single site so I'm not sure if adding all these other SharePoint pages will cause an issue.
How does the user invoke the different pages from the main site i.e. when do they choose their role that then filters the pages?

I don't know your use case but you can customize the learning pathways site so you can add a landing page with links to the difference role pages right on the home page or trade out the links in the hero web part. It's up to you. If you create a Teams app then I would create a landing page with all the roles on it and have that be the main page for the Teams App.


Incomplete Bank vans detection

Add missing BankVanAI to correctly determine vehicle type. Affects vehicle restrictions and junction restrictions features - was detected as not eligible to use turn on red feature or not following Service Vehicles restriction

where is the test for this?
how can I reproduce previous failure.
how do I test this is working now?

test passed


Science Module Complete

Hello,
Raising a pull request with the code added for the Science module #158
With this module added, the user will be able to use Science.h as a include file and generate data for 2 additional user-defined datatypes 'ChemicalElement' and 'Unit'
Just like the other modules, Science::chemicalElement() function will return a random element from the periodic table.
Also using function Science::unit() function will return a random fundamental unit for measuring distance, time, mass, temp, current.
I have also added functions that can generate units specifically for distance, time, mass, temp, current respectively.
Regards,
Shardul

Not sure why a Unit Test from Vehicles module is failing. Trying to fix it. Any idea why this might be happening for clang compiler?

I ran tests again and it passed

Please add @code and @endcode lines in examples: check other api files please

Thanks for letting me contribute to this project. Learned a lot about open source contribution.


Configure script to automatically package GitHub release

Should contain:

BlueprintCore src folder in the project
BlueprintCore.dll
BlueprintCore.pdb
BlueprintCore.xml

This way I don't have to manually copy / bundle every time I push a release :P


Parsing/Preprocessing: add the concept of 'warning'.

So far, all messages in the lex/preprocessing/parse phase have been
of 'error severity'. However, there are issues whose severity can
be lower e.g. warning about re-definition of a macro.
In this change

Make RejectedToken contain a severity (name of that object
is still the same, might need some re-name later)
Change printing of rejected tokens to include the severity
and don't talk about a 'rejected token', but just the token
a hand; that does sound more user-friendly (the 'rejected'
sounds more like an artifact of parsing).
CAVE: this might trip up scripts of users that grep for
messages.
Surface preprocessing re-definition as warning message.
Do not include the preprocessing warning yet in the output
of the parser; merely LOG(INFO) it at this point.
(Reason: might be too noisy as the preprocessor has limited
understanding of `ifdef branches).
At least, this provides a properly formatted filename + line-number
info in the log.

Added a few TODOs for further cleanups.
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Example of the LOG(INFO) output of the preprocessor issues it sees (redifinitions due to incomplete ifdef handling)
I20220317 13:46:25.170456 770932 verilog_analyzer.cc:258] /tmp/picorv32.v:30:31: preprocessing warning at token "" : Re-defining macro 'debug'
I20220317 13:46:25.170563 770932 verilog_analyzer.cc:258] /tmp/picorv32.v:38:25: preprocessing warning at token "(* keep *)" : Re-defining macro 'FORMAL_KEEP'
I20220317 13:46:25.170578 770932 verilog_analyzer.cc:258] /tmp/picorv32.v:40:24: preprocessing warning at token "" : Re-defining macro 'FORMAL_KEEP'
I20220317 13:46:25.170589 770932 verilog_analyzer.cc:258] /tmp/picorv32.v:42:31: preprocessing warning at token "empty_statement" : Re-defining macro 'assert'

Codecov Report

Merging #1270 (9dbcfc7) into master (e624224) will decrease coverage by 0.02%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1270      +/-   ##
==========================================
- Coverage   92.83%   92.80%   -0.03%
==========================================
  Files         339      339
  Lines       23508    23527      +19
==========================================
+ Hits        21824    21835      +11
- Misses       1684     1692       +8

Impacted Files
Coverage Δ

common/analysis/file_analyzer.h
100.00% <ø> (ø)

verilog/preprocessor/verilog_preprocess.h
100.00% <ø> (ø)

verilog/analysis/verilog_analyzer.cc
94.40% <25.00%> (-2.00%)
:arrow_down:

verilog/preprocessor/verilog_preprocess.cc
86.44% <60.00%> (-0.85%)
:arrow_down:

verilog/tools/ls/verible-lsp-adapter.cc
97.03% <66.66%> (-0.70%)
:arrow_down:

common/analysis/file_analyzer.cc
92.00% <75.00%> (-2.32%)
:arrow_down:

verilog/analysis/json_diagnostics.cc
84.61% <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 e624224...9dbcfc7. Read the comment docs.


Invalid size checks when units are specified in the size parameter for volumes

acorn version v0.5.1-79-g84d2894c+84d2894c
Steps to reproduce the problem:
Create a volumeClass with size-range .
NAME               DEFAULT   INACTIVE   STORAGE-CLASS      SIZE-RANGE     DEFAULT-SIZE   ACCESS-MODES      DESCRIPTION
pvclass3           *                    local-path-test3   10G-50G        45G            [readWriteOnce]   my first volumeclass

Deploy an app with volumes using the above class and setting the size to a valid value (20G in my case)
volumes: {
  if !args.dev {
    "pgdatanew": {
      class: "pvclass3"
      size: 20G
    }
  }

App fails to deploy with following error message -  ✗  ERROR:  App.api.acorn.io "mytest8" is invalid: spec.volumes.pgdatanew.size: Invalid value: "20E": greater than volume class pvclass3 maximum of 50G
Note --> If i set the  size as "20" , then app deployment succeeds and volumes get created with "20G" capacity.

The issue here seems to be bad documentation. The 20G should be quoted in this case: "20G".
Without the quotes, the Cue parser will interpret 20G as a number<PHONE_NUMBER>. When we get that number for the size of a volume, we always add a G to the end of it. This is to allow users to enter 10 for the size of their volumes and that will get turned into 10G or 10 gigabytes.
With the quotes, the "20G" is interpreted as a string and when the volume parser gets this string, it correctly determines that the size of the volume should be 20 gigabytes.

@thedadams Size specified as just 15G (with no quotes)gets set as expected when App gets deployed with a volumeClass that has no size_limit specified. This notion of setting size with units and no quotes  seems to issue only when size range check needs to be enforced.
App deployed with following volume definition, sets "15G" set as expected.
volumes: {
  if !args.dev {
    "pgdatanew": {
      class: "local-path-test4"
      size: 15G
    }
  }
}

%acorn offerings volumeclass local-path-test4
NAME               DEFAULT   INACTIVE   STORAGE-CLASS      SIZE-RANGE     DEFAULT-SIZE   ACCESS-MODES   DESCRIPTION
local-path-test4                        local-path-test4   Unrestricted                                 Acorn-generated volume class representing the storage class local-path-test4
sangeethahariharan@Sangeethas-MBP volumes %

 % acorn volumes
NAME                                       APP-NAME      BOUND-VOLUME   CAPACITY   VOLUME-CLASS       STATUS    ACCESS-MODES   CREATED
pvc-ae01e9c7-c1c7-416c-ac0e-f7cb15a1c001   mysizecheck   pgdatanew      15G        local-path-test4   bound     RWO            13s ago

Thank you for that clarification. I will investigate why this is different in the validation.

Tested with acorn version v0.6.0-rc1+f4c514dd
Able to deploy apps successfully with volumeClass having size_limit with size parameter having units .


Add SSH Tunnel Manager v2.1.3

Thanks. Merged as fc0fa69, with version interpolation and a revised URL.


CCD-1596 Integrate Fortify Scan into Nightly Pipelines - Message Publ…

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CCD-1596
Change description
Integrate Fortify Scan into Nightly Pipelines - Message Publisher
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

re-opening a new one


Show Node counts in Cluster Management's cluster list

counts are shown (ready, pending,etc) are already shown for RKE2 machines
show restricted set of counts for RKE1 nodes
show basic total node count for non-RKE clusters
#3028

@vincent99 I think the way I've done the node pool ready/pending counts makes sense (along with the dropping of outdated and unavailable that were in machine pool). Does that look ok?


⚠ PrivateHD has degraded performance

In 151dc49, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2113 ms

Resolved: PrivateHD performance has improved in b66d889.


Remove populating mtlsExcludeServices in mesh config.

mtlsExcludeServices flag was used to exclude certain services (common use is for API server) from mutual TLS set up on client side. Since 0.8, this (client settings) needs to be defined explicitly with destination rules, this flag is no longer needed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS
pilot/OWNERS
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
527573e470e044a112c73eaa6d9789270a9bc4ea
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@diemtvu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
527573e470e044a112c73eaa6d9789270a9bc4ea
link
/test istio-unit-tests

prow/istio-pilot-e2e.sh
527573e470e044a112c73eaa6d9789270a9bc4ea
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Can you please add description and motivation for this change?

Added PR description. Thanks.

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: andraxylia, diemtvu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [andraxylia]
pilot/OWNERS [andraxylia]
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, chxchx, diemtvu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [andraxylia]
pilot/OWNERS [andraxylia]
tests/OWNERS [chxchx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

test owners please lgtm:
approvers:
@costinm
@sebastienvas
@sebastienvas please update the OWNERS file in your area and add more owners, and remove the people who no longer work in Istio

Great, it is always helpful to link related issues here, thanks @diemtvu


Quests are listed as Quest - Quest -

Addon Version
v0.1.6
World of Warcraft Version
Retail
Describe the bug
In the bag, quests are shown as Quest - Quest -  the phrase "Quest - "i s duplicated.  I also don't think you need the " - " either but that's an aesthetic decision. IMO there's too much white space
Steps to reproduce
Open bag.  Mouse over a quest category.

I agree, Quest - Quest is unique because both the type and subtype are Quest. This should be deduped.


ocm agent should have enough permission to the managedfleetnotificati…

…on and records
What type of PR is this?
(bug/////)
What this PR does / why we need it?
Add permission for the rbac for ocm-agent
Which Jira/Github issue(s) this PR fixes?
Fixes #
Special notes for your reviewer:
Pre-checks (if applicable):

[ ] Tested latest changes against a cluster
[ ] Ran make generate command locally to validate code changes
[ ] Included documentation changes with PR

/lgtm

Codecov Report

Merging #68 (0d604d2) into master (3a66eda) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   63.86%   63.86%
=======================================
  Files          19       19
  Lines        1165     1165
=======================================
  Hits          744      744
  Misses        361      361
  Partials       60       60


hard coded image

Line 39 of Program.cs references a local file on your hdd.

It should be a fairly easy fix, I'll try to fix tonight


Added 'LBRY Community' link

I Added 'LBRY Community (external site)' link to 'Learn' page.
below "LBRY Merchandise Shop" on the bottom left.
I added target="_blank" to open in new tab/window
Is this okay with you?
Or should I write a post on LBRY.io to announce LBRY Community first?
Or maybe I can do that later?
rouse

Header and footer links are for LBRY Inc corporate content - what we need is a community friendly resource to promote partners and LBRY.fund projects.


Upgrades nan for io.js compatibility

Also adds node 0.12 and io.js 1.1.0 and 1.2.0 to Travis configuration.

Cross-reference: iojs/io.js#456

Thanks.

Released with 0.6.0
On Mon Feb 16 2015 at 8:38:53 PM Alex Potsides<EMAIL_ADDRESS>wrote:

Cross-reference: iojs/io.js#456 https://github.com/iojs/io.js/issues/456
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/node-usage/pull/43#issuecomment-74521826.


Directional light for inherited scene preview

When you create a new inherited scene that is some model prefab it won't have a light as child object
1 ) It's hard to figure how it looks and adjust material parameters.
2) the thumbnail when you save the inherited scene won't have lighting for better preview and thumbnail
When making an inherited scene , it needs to have a directional light option that will not save into the scene file, but it's only purpose is better preview material and get a thumbnail with lighting.
Left is inherited scene, right is the same scene with light preview
https://github.com/DevMagicLord/Godot3/blob/master/bugs/download/lightpreviewnew.jpg?raw=true

Wouldn't it be just the fact that a scene has no light by default unless you actually put a Light node? (regardless of it being inherited or not)

You can already add any light node and set it to editor only. Wouldn't that be enough?

You can already add any light node and set it to editor only.

Could you be be more specific? How does it work for you? Without further ado, I see no difference, the (directional) light effect is the same in the editor and in the running project ...
git 0dbe01483a902c49ecedf4fd36b74353424145a5

In Unity when you make a prefab, it has a thumbnail with some sort of lighting or environment ligthing.
It's not a dark color Tumbnail.
Why Godot could not lighten up thumbnails.
It's only a improvment proposal to make thumbnails better in Godot.
(I don't care if it's not improved)

I think some method of setting a default editor-only directional light (maybe in the editor, maybe per-project) and using it in all scenes without a proper light might be quite useful.
Currently editor-only Light nodes are still present in the scene tree in debug, and, AFAIK, in release build exports as well.

Currently editor-only Light nodes are still present in the scene tree in debug, and, AFAIK, in release build exports as well.

@bojidar-bg That's about to be fixed: #23256.

Great.

@akien-mga I mean, they are present in the scene tree even if not registered on the VisualServer. So, you can still get an editor-only light via get_node or get_children.

@reduz suggested on IRC that we can have a default DirectionalLight as part of the Environment resource. It should be visible both in the editor and running project, but it will disappear as soon as a DirectionalLight node is added to the scene. This way, we can get good lighting out of the box while keeping predictable behavior between the editor and running project.

Closing in favor of https://github.com/godotengine/godot-proposals/issues/1599, as feature proposals are now tracked in the Godot proposals repository.


java-runtime: stream client call listener back-pressure

only request for more messages when stream is being pulled instead
of unconditionally request for more through onMessage.

current behavior causes heap to fill up and eventually cause
out of memory exceptions.

add test that showcases we only request more messages when there
is a demand.

Could you request n upfront, and then still one-at-a-time to replenish?  Still not optimal, but simple, but you'd have a better chance of maintaining a buffer until a more sophisticated strategy is created.

Not sure. I am thinking that this PR loss in throughput could be replenished via. stream combinators. I posted a, rather incomplete, suggestion in the gitter channel.

Yeah, all the prefetches are probably a fine interim solution, but we'll still be requesting 1 at a time.  I don't know enough about the runtime to know how much faster it is to request(100) if we can really accept 100.  Your high-low idea could work, as it would be able to request batches.
I think this is fine as is to solve the crash.

I am fine with the loss in throughput now, as downstream using groupWithin makes up for it when considering downstream probably uses IO like db inserts or similar that take longer time. Perhaps my idea on gitter could be amended to this, but it is probably needs more tests.

Thanks Ross :)


Can't run dev

I was just doing a quick test and downloaded the template to run it from.
Installed everything but npm run dev gets this error:
[webpack-cli] file:///C:/Users/XXXXXXXXX/Projects/template-webpack-main/node_modules/open/index.js:6 import fs, {constants as fsConstants} from 'node:fs/promises';             ^^^^^^^^^ SyntaxError: The requested module 'node:fs/promises' does not provide an export named 'constants'
npm version: 9.6.3
node version: v16.16.0

Thanks @photonstorm I probably should have realised that - I've not worked on that PC recently... After the update, all is fine :)


chore: update to maturin's recommended project layout for rust/python…

Supersedes https://github.com/apache/datafusion-python/pull/694
Rationale for this change
The previous layout leads to an import error when installing with maturin build and pip install ..
This error was common enough that maturin changed the recommended project layout to what this commit does.
A prior PR attempted to solve this by altering lib.name in Cargo.toml, but that did not work for me.

Prior PR
maturin ImportError issue
maturin changes recommended project structure

Maintainer of pyo3 explains the issue on the prior PR

In particular the problem is that maturin develop (or pip install -e .) will use the checkout's Python sources, but pip install . will put a copy into site-packages, which then conflicts with the checkout sources if you're running Python from the repository root. (You'll end up importing from the local sources rather than the installed contents in site-packages.)

I'm sure @andygrove would have the same Q as on previous PR.
@davidhewitt, could you spare a little more knowledge about the effects of changing lib.name?

Does this have any impact on downstream maturin projects that use this crate as a dependency? If they also use the name _internal, will there be a conflict?
Should we replace all references to _internal with datafusion_internal instead?

@andygrove , I have another commit queued up if you want to switch the lib.name and python module from _internal to _datafusion_internal

@andygrove , I have another commit queued up if you want to switch the lib.name and python module from _internal to _datafusion_internal

I think we can undo this last commit and ignore my question/concern. I missed that the maturin project definition was already qualifying _internal as datafusion._internal, so I think there should be no conflict with downstream maturin projects.
Sorry for the noise.

@andygrove, the commit that changes lib.name is not in this PR. It's in a different branch of my fork.
I created my own noise be referencing the suggestion in this PR in that commit message.

Looks like you figured this out already, but yes I agree no need to change lib.name here 👍


Add SSR to turn power on/off

Using http://www.kudom-elec.com/webmanage/edit/UploadFile/KG3RD-EN.pdf
And hook into IRRemote.

Also it would be a good idea to auto set the volume to 0 before it turns on. (and then move to previous volume over 0.5 seconds once on.


Slick settings

Hi!

The slick's plugin is awful on mobile. It should be more clear and not block the slide down.
Slick-dots is very thin.

Is there any chance that you will open access to the slick settings?

@contributorpw ,
i did some modification of this template, please download the latest code. hope you like it.

@Somrat37 Thank you!