runner/lib: use uint64_t for dev_size

Use uint64_t for dev_size to prevent issues with 32 bit setups.
Attached is the kernel patch that will also be pushed.
dev-size64-target-pending.txt

bump

Attached is the correct patch for the kernel.

Did you maybe try to send this patch to linux mainling lists?

Attached is the correct patch for the kernel.

Did you maybe try to send this patch to linux mainling lists?

The kernel patch is merged upstream already. It was only attached in this PR to make it easier for people to test.
The userspace parts in this PR are not merged so it's still open. I don't work on this project anymore, so feel free to take up the issue/PR/code.

Well, would be cool if someone would maintain this software, but it seems dead :/

Your commit does loads of other things, not only uses uint64_t for dev size.
Could you please split it and name it correctly?

@lxbsz what do you think about this PR? It seems to fix some issues I've stepped into on 32-bit arches, but would need rebase and also probably spliting into more commits.

@lxbsz what do you think about this PR? It seems to fix some issues I've stepped into on 32-bit arches, but would need rebase and also probably spliting into more commits.

I am okay with it. If you can test on the 32-bit arches, please go ahead to take it.
I will test it after that.


Feature-Request: URL-Schema

The blog software I use, uses only an index.php with the page/article names behind a question mark.
Take a look at the following examples:

https://www.deimeke.net/dirk/blog/index.php?/archives/4023-Linkdump-352020.html
https://www.deimeke.net/dirk/blog/index.php?/pages/about_me.html

Umami shows only the index.php and not the informations behind that.
Please allow statistics for pages following an URL-schema ("like 'index.php?/.*.html')

Ok, I can make it so ?/ is recognized as a page in the combined filter. In the meantime, you can click More and filter by Raw which will show you all the URLs.

@mikecao This would be perfect for me. Thanks for the hint to use "More" :-)

Thanks a lot @mikecao, that works perfectly well.


feat: new resolutions subpage

Resolves #179

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


Refined error management.

All async function could return error.
async_recv complete with error always started closing process. But it could intentional cancel (operation_aborted). In this case, no longer start closing process. Just report error.
async_send needs to care about packet_id.
Basically, packet_id ownership is moved to the library if async_send() is called. So if error happens, it should be released by async_send() if releasing is needed.

PUBLISH

If packet is stored, packet_id shouldn't be released
If packet is not stored but send error happened, the packet_id should be released because the counter part never return acknowlege packet.

PUBREL

packet_id should be owned by async_mqtt until PUBCOMP is received. So it shouldn't be released.

SUBSCRIBE, UNSUBSCRIBE

If send error happened, the packet_id should be released because the counter part never return acknowlege packet.

Codecov Report
Attention: Patch coverage is 94.59459% with 2 lines in your changes missing coverage. Please review.

Project coverage is 94.19%. Comparing base (95e0b73) to head (83f7371).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #286      +/-   ##
==========================================
+ Coverage   94.17%   94.19%   +0.02%
==========================================
  Files         150      150
  Lines        8921     8935      +14
==========================================
+ Hits         8401     8416      +15
+ Misses        520      519       -1

Files
Coverage Δ

include/async_mqtt/endpoint.hpp
100.00% <ø> (ø)

include/async_mqtt/impl/endpoint_recv.hpp
70.85% <100.00%> (+0.24%)
:arrow_up:

include/async_mqtt/impl/endpoint_send.hpp
86.62% <92.30%> (+0.66%)
:arrow_up:


New version: Telerik.Fiddler.Everywhere.Insiders version 4.6.2-insiders

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:


PR OpenAPI Generator support

This PR supports OpenAPI Generator

Thank you for your contribution!
LGTM. I have tested some examples as well.
I will rebase these commits and merge it later.

Merged.


Windows Terminal refusing to open with no visual feedback

Windows Terminal version
1.18.2822.0
Windows build number
10.0.22000.1848
Other Software
No response
Steps to reproduce
I downloaded it through winget (using CMD). Command I ran : winget install Microsoft.WindowsTerminal
Expected Behavior
Run with no issues, open normally
Actual Behavior
Not opening, not saying anything at all, no error popups, no confirmation dialogs, not "opening and closing immediately". Just does not do anything at all.
When I checked "Windows Event Logs" I could see the cause, the log looks as follows:
Faulting application name: WindowsTerminal.exe, version: 1.18.2310.9002, time stamp: 0x65248b0f
Faulting module name: dcomp.dll, version: 10.0.22621.1778, time stamp: 0x830223ed
Exception code: 0xc0000005
Fault offset: 0x0000000000092244
Faulting process id: 0x0x250C
Faulting application start time: 0x0x1DA0FFE088FD00A
Faulting application path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.18.2822.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
Faulting module path: C:\Windows\System32\dcomp.dll
Report Id: 7b41473e-1bfe-40d5-bde1-c837d3335246
Faulting package full name: Microsoft.WindowsTerminal_1.18.2822.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: APP

Can you try to get this repro'd with the Feedback Hub taking a recording? Alas, I can't find your dump on the backend from what you've shared, but the feedback hub link should help.
/feedback
FWIW there is a dcomp.dll crash tracked in MSFT:40378391. That one was fixed in 2022, but all the details in there are very Xbox specific. Maybe this is a fresh crash bucket in the Terminal that is wrongly getting bucketed to that bug.

(looks like the bot's dead?)
Hi there!Can you please send us feedback with the Feedback Hub with this issue? Make sure to click the "Start recording" button, then reproduce the issue before submitting the feedback. Once it's submitted, paste the link here so we can more easily find your crash information on the back end?Thanks!

Confirming the issue.
When I try to open the Terminal from Power Shell by running wt, I get
PS C:\Users\pianykri> wt
Program 'wt.exe' failed to run: The file cannot be accessed by the systemAt line:1 char:1
+ wt
+ ~~.
At line:1 char:1
+ wt
+ ~~
    + CategoryInfo          : ResourceUnavailable: (:) [], ApplicationFailedException
    + FullyQualifiedErrorId : NativeCommandFailed

From the Run Window:

The wt.exe file is on my PATH (located at %USERPROFILE%\AppData\Local\Microsoft\WindowsApps)

The issues was resolved by reinstalling the Windows Terminal. The current version is 1.18.2822.0 and it opens as intended.


Change RPC endpoint for POA.network

Closes #1925
I have deployed this PR to:
https://rstormsf.github.io/etherwallet/

thanks @gamalielhere
Can  I know when it  will be live on official URL?


Example page minor fixes and ignore warnings

Changelog

Font Size for description
Spacing between use cases
Disable warning messages

@YashKumarVerma Fixed!

@anku255 Warnings is removed from the App and these will still be shown on terminal

@chirag-singhal Oh! that makes sense.


Provide role session name in STSProfileCredentialsProvider::GetCredentialsFromSTSInternal()

Issue #150
Description of changes:
Add missing role session name when building the AssumeRoleRequest object in STSProfileCredentialsProvider::GetCredentialsFromSTSInternal. Without it, I am getting this error in the outcome:

1 validation error detected: Value null at 'roleSessionName' failed to satisfy constraint: Member must not be null with address : <IP_ADDRESS>

Check all that applies:

[x] Did a review by yourself.
[ ] Added proper tests to cover this PR. (If tests are not applicable, explain.)
[x] Checked if this PR is a breaking (APIs have been changed) change.
[ ] Checked if this PR will not introduce cross-platform inconsistent behavior.
[x] Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

[x] Linux
[ ] Windows
[ ] Android
[ ] MacOS
[ ] IOS
[ ] Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

It has been merged. https://github.com/aws/aws-sdk-cpp/commit/54dbe8f59c31f310e976ca2fa2d01d0abf41807a
Thank you for your contribution!


Using strict with Key Vault values

The section 'Use Azure Key Vault with GitHub Action' of this document shows an example how strict mode can be used while updating app settings and secrets in two separate steps. Correct me if I'm wrong, but in this example the first step will delete all the KeyVault references while the next step re-adds them. That means there will be a brief moment where there are no secret keyvalues and if any error were to occur on the second step this could permanently break applications depending on KeyVault references.
Is my assumption correct and is there an alternative that doesn't result in keyvalues being temporarily removed?

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

ID: 9d098a84-8ff2-2166-9e93-a84a21f589b7
Version Independent ID: de3ca7f9-9b67-d015-d188-d1ad1f6158f9
Content: Sync your GitHub repository to App Configuration
Content Source: articles/azure-app-configuration/concept-github-action.md
Service: azure-app-configuration
GitHub Login: @maud-lv
Microsoft Alias: malev

@audrius-freemarket Thanks for reaching out with your question. We have assigned it to author for review and will update you.

Hi audrius-freemarket! Thank you for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled. If you feel that this issue still needs to be addressed, you can still share feedback here: provide feedback for Microsoft Learn content. #please-close


Google Photos stuck on 'Setting up your photo library' on non-rooted device using MicroG RE

Google Photos shows "Setting up your photo library" and does not sync when using MicroG RE, but works perfectly with GmsCore.

True. Same problem with me.


Test cupertino/nav_bar_test is flaky

The test flutter/test/cupertino/nav_bar_test.dart is flaky on the monorepo CI, and fails about 20% of the time.
The test either passes or fails exactly 24 tests, with the same errors each time.
The 24 failing tests have different errors and stack traces, but these errors are consistent between failing runs.
The test does not seem to fail on Flutter CI, which seems to run the test in the same way, on the same browser version.
Both builders run
bin/dart dev/bots/test.dart

with shard web_tests and subshard 4, using the Chromium for Testing browser 119.
The failing monorepo test builder is
https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/flutter-linux-web-tests-4
with an example failing run
https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/flutter-linux-web-tests-4/4127/overview
The passing Flutter CI builder running the same tests is
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux web_tests_4
The failing runs seem to be clustered in time, where 2-5 consecutive runs will fail.
But I checked which bots they run on, and found one bot that ran a passing build, then a failing build, then a passing build again, so it doesn't seem to be that a bad bot will always fail.

Running the command
flutter test -v --platform=chrome --web-renderer=html test/cupertino/nav_bar_test.dart

locally does not seem to reproduce the failures , using Chrome 121.
Neither does running deb/bots/test.dart locally with shard web_tests subshard 4.

Consider disabling on web?

Has anything changed/been updated here? Based on https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/flutter-linux-web-tests-4, it doesn't look like this test has failed in the last 100+ attempts.

It doesn't look like this test is flaky anymore, so I'll go ahead and close the issue for now.

Correct, the test stopped flaking on the monorepo CI on Feb 1.


fix/bump node js runtime to 20.x

while running terraform apply, AWS Lambda APIs returned an error: the nodejs 18.x environment is not supported and they recommended to upgrade to nodejs 20.x
This will unblock future depoloyements

Hey @giovanni-riva, thanks for this! I'll check that the upgrade isn't breaking anything and will approve.
But are you sure that the error said that nodejs18.x is not supported? because it seems like it is supported according to this page

Hey @ChengaDev
sorry for the confusion, there is a typo in my message.
Node nodejs14.x is not supported.
You are correct and node18.x is supported and we could deploy the lambda with such version

Hey @giovanni-riva, we have made this update in another PR with some other changes
Thanks for raising this flag.


Add option to install extra gem groups to brew

Closes #35
I ended up trying to implement the first option, because brew install-bundler-gems doesn't seem to have any way to specify a custom install directory for the gems (they are always expected to live in the vendored bundle directory).
This is kinda hacky, but at least seems to work for e.g. brew livecheck. Some other things like brew style might require more tweaking but this is a starting point to at least ensure the dependencies are installed.
With this configuration:
{
  nix-homebrew = {
    enable = true;
    enableRosetta = true;
    inherit user;

    bundlerGemGroups = [ "livecheck" "style"];
    # TODO: Declarative tap management
  };
}

I was able to run brew livecheck:
$ brew tap homebrew/cask --force
==> Tapping homebrew/cask
Cloning into '/opt/homebrew/Library/Taps/homebrew/homebrew-cask'...
remote: Enumerating objects: 1073648, done.
remote: Counting objects: 100% (3871/3871), done.
remote: Compressing objects: 100% (1246/1246), done.
remote: Total 1073648 (delta 2842), reused 3629 (delta 2625), pack-reused 1069777 (from 1)
Receiving objects: 100% (1073648/1073648), 396.96 MiB | 15.93 MiB/s, done.
Resolving deltas: 100% (788997/788997), done.
Tapped 2 commands and 7041 casks (7,180 files, 436.1MB).
$ brew livecheck vlc
vlc: 3.0.21 ==> 3.0.21

Edit: and with the latest commit I can also style and audit my custom tap:
$ brew audit --tap ian-h-chamberlain/dotfiles && brew style ian-h-chamberlain/dotfiles
Inspecting 7 files
.......

7 files inspected, no offenses detected

Sorry for the late response, but performing downloads inside derivations introduces impurity and isn't a good idea. It just so happens to work because Nix sandboxing is disabled by default in macOS installs.

Sorry for the late response, but performing downloads inside derivations introduces impurity and isn't a good idea. It just so happens to work because Nix sandboxing is disabled by default in macOS installs.

Gotcha, that makes sense and I actually didn't realize sandboxing was disabled on macOS!
I guess a more correct implementation would use something like https://github.com/nix-community/bundix ?
I haven't been doing as much homebrew development recently as I've moved to Windows for work, so I guess I'll just abandon this PR meanwhile, but the issue itself might still be useful in case someone wants to pursue a different approach.


Uncaught SyntaxError: Unexpected token '{' setting up tailwind with postcss

I am trying to set up tailwind with 11ty and postcss
The css is compiled successful using both postcss-cli and parcel but when I load the css in the browser, I keep getting an error: Uncaught SyntaxError: Unexpected token '{'
Reproduce issue:

repo https://github.com/dephiros/11ty-tailwind
Run ./Taskfile
Open http://localhost:8094 and see the error in the console

Appreciate any help!

Is it possible to build without minifying so we can actually see where the error is?

Thanks Adam for the quick response. I didn't think of that(actually tried and reformatted the response in vscode)
Let me take a quick look

If you pull the latest master, I uncommented cssnano in postcss so should no longer minify. Note: you might have to remove cache before rerun: rm -rf dist .parcel-cache && run.
Chrome points to html { which does not makes sense to me. I attached the css file and screenshot here too
index.css.zip

I run the file through csslint.net and it seems to point to a bunch of errors starting at line 11865:
.grid-cols-1 {
  grid-template-columns: repeat(1, minmax(0, 1fr));
}

.grid-cols-2 {
  grid-template-columns: repeat(2, minmax(0, 1fr));
}

.grid-cols-3 {
  grid-template-columns: repeat(3, minmax(0, 1fr));
}

Found the issue! My silly self included the stylesheet through a script tag instead of a style tag 🤦
Sorry for the noise and thank you for your help Adam!


Revert "refactoring jp.co.yahoo.yosegi.util"

Reverts yahoojapan/yosegi#21
remove refactoring

LGTM


[$250] Bug: "Hmm... it's not here" view is showing up with back and dismiss button while the page already has those buttons reported by @Puneet-here

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Login at new dot and create a workspace
Open new tab > Go to old dot > log in > delete newly created workspace
Go back to new dot and navigate to Reimburse expenses

Expected Result:
Back and dismiss button shouldn't be shown twice
Actual Result:
Back and dismiss button are shown twice
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.2.12-3
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/195203194-1c72fd0e-7466-44fd-9725-4407d5b366b5.mp4
https://user-images.githubusercontent.com/43996225/195203211-4bba1e86-4664-4b74-a240-d56eaf8968a1.mov
Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1665509133039339
View all open jobs on GitHub

Proposal
We need to remove FullPageNotFoundView from this component
https://github.com/Expensify/App/blob/4ff239ef81b0b06b94f00efd4026881de1465055/src/pages/workspace/reimburse/WorkspaceReimburseView.js#L192
instead use it here to wrap https://github.com/Expensify/App/blob/d99d7c81ca9b02edf6c0151a311f881f4eea9633/src/pages/workspace/WorkspacePageWithSections.js#L97

I was able to reproduce:
https://user-images.githubusercontent.com/31258516/196668634-4487633c-1044-46bb-b643-b91d53f09a73.mp4

WorkspaceReimburseView is a child of WorkspacePageWithSections which contains a header
https://github.com/Expensify/App/blob/b1e0bd64b6d99ba626f1fd9490a0a17bb729051c/src/pages/workspace/reimburse/WorkspaceReimbursePage.js#L25-L33
At WorkspaceReimburseView we check if this.props.policy is empty (it will be empty when it's deleted) and showing FullPageNotFoundView  which has it's own header so that's why we are seeing two headers https://github.com/Expensify/App/blob/b1e0bd64b6d99ba626f1fd9490a0a17bb729051c/src/pages/workspace/reimburse/WorkspaceReimburseView.js#L192
we can also create a prop and use it FullPageNotFoundView to hide the header but I believe when a user clicks at any section inside a deleted workspace they should see not found page not only at members page and Reimburse View

Looks good to me! @NicMendonca can you create an upwork job so that @Puneet-here can apply? Thanks!

Opps, sorry! Job post here: https://www.upwork.com/jobs/~011a3d442964c96c36

@Puneet-here how's it looking?

I was waiting for the upwork job. I have applied to the job and raised the PR.

@Puneet-here @rushatgabhane hired :)

@Puneet-here is working on PR - this isn't overdue

I am about to go OOO for a week, so re-assigning.

Hey @Puneet-here , can you please provide an update on your progress? Thanks!

OOO, reassigning.

Sorry for the delay everyone! Everyone has been paid out for this one, and the Upwork post is closed.

@sakluger, It's also eligible for the reporting bonus. I only received $250 for the fix.

@sakluger, can you please look at the comment above?

Bump @sakluger

Will look into this for ya Puneet!

@Puneet-here sent you the bonus! Sorry for the delay here!

@sakluger can we make sure to get the checklist here completed when ya get a chance! TY! 🙇♂

Thank you @dylanexpensify


update ?

Hello,
Currently, the program doesn’t compile with python>3.10 and numpy>1.26 (and I am not sure about cuda12).
Would it be possible to update it ?

After some tries, I was able to use it with cuda 12.
For the compilation under python3.12, I had to modify the two setups.py files :
# cpp_subsamplin/setup.py:
from setuptools import setup, Extension
import numpy as np

# Adding OpenCV to project
# ************************

# Adding sources of the project
# *****************************

SOURCES = ["../cpp_utils/cloud/cloud.cpp",
             "grid_subsampling/grid_subsampling.cpp",
             "wrapper.cpp"]

module = Extension(name="grid_subsampling",
                    sources=SOURCES,
                    extra_compile_args=['-std=c++11',
                                        '-D_GLIBCXX_USE_CXX11_ABI=0'])

setup(ext_modules=[module], include_dirs=np.get_include())

and
# cpp_neighbors/setup.py:
from setuptools import setup, Extension
import numpy as np

# Adding OpenCV to project
# ************************

# Adding sources of the project
# *****************************

SOURCES = ["../cpp_utils/cloud/cloud.cpp",
             "neighbors/neighbors.cpp",
             "wrapper.cpp"]

module = Extension(name="radius_neighbors",
                    sources=SOURCES,
                    extra_compile_args=['-std=c++11',
                                        '-D_GLIBCXX_USE_CXX11_ABI=0'])

setup(ext_modules=[module], include_dirs=np.get_include())

didn’t try to launch a training with this yet. But there’s no reason.


Remove byebug since it is not in dependency

byebug dependency is defined here:
https://github.com/jasonm23/story_branch/blob/217b5fd6a695d02ca77dbe572e6ab823011121e7/story_branch.gemspec#L28
If we only use gem install story_branch, it won't be able to run.

Good catch, you can also bump the version patch number for this PR
Thank you Lincoln

Done


Remove hook.data.payload

This removes hook.data.payload because it’s too easy to open a security hole when implementing custom authentication solutions.  hook.params.payload is the only default mechanism to customize the JWT, now.  A hook can be used to restore this functionality, if needed.

Agreed. I think this was an oversight on my part and was a remnant left over from before I figured out how we should be returning the payload from the auth plugins. See this line as an example.
This may be a breaking change for people. I'm not sure if anyone has been using that but my guess is this probably should be a major release. Thoughts?
This is totally a :shipit: though.

This might be a breaking change for one or two people.  In order to hit the vulnerability you had to use a custom verifier and not pass any payload out of the verifier.  Otherwise, the params.payload always overwrite the data.payload, anyway.  I think we should get it into the current release.

Yeah let’s ship it. Technically it should be a major since we are removing functionality. But that functionality shouldn’t be there and unless you were really monkeying with stuff in a way you shouldn’t be, that PR shouldn’t have an effect.
:shipit:


Use speedtrap as printer

Hey, this is more of a question then a issue, sorry.
I'm trying to configure speedtrap from the command line instead of from the config file.
Mainly since I want it to be a option per run not per every run.
this option doesen't seam to work.
--printer <printer>       TestListener implementation to use.

Could not use "JohnKary\PHPUnit\Listener\SpeedTrapListener" as printer.

I don't think the --printer option does that, that is the equivalent of the printerClass attribute on the root tag in the XML

no, this shall not be done as a printer.
printer would require us to actually print whole printout, like inherit it from one of the printers, but that would kill other printer formats.
as a listener, user might print in any format, and still get speedtrap executed.
IMO, this one shall be closed.

@keradus @REBELinBLUE Thank you for your input!
@Neppord If you discover any way to code this type of feature please open a PR. Based on feedback above we will close this issue.


Manual approval required for workflow run<PHONE_NUMBER>

Workflow is pending manual review.
URL: https://github.com/pushpendrasingh3/manual-approval/actions/runs/7847503746
Required approvers: [pushpendras21 pushpendrasingh3]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

approved

approve

All approvers have approved, continuing workflow and closing this issue.


Omit references

Previously, model.get(['a', 'b']) where b is a reference would result in references in the json output. This change instead results in emitting nothing where a reference would previously exist. JSONG remains unchanged.

Coverage increased (+0.006%) to 91.802% when pulling 8c86c381f48acee9dbbbe5e7a11d6937f2af6fc8 on steveorsomethin:omit-references into 2c978de9dacc884e6bb582c1a98a39bd2b0c135f on Netflix:master.

Coverage increased (+0.006%) to 91.802% when pulling 4f7d6fd846bfa7f500d9b767f45281195982504b on steveorsomethin:omit-references into 2c978de9dacc884e6bb582c1a98a39bd2b0c135f on Netflix:master.


Xilinx tests failing with AttributeError: module 'prjxray.tile' has no attribute 'OutpinTiming'

https://source.cloud.google.com/results/invocations/3344eb49-e07d-44c6-b30a-0287ba31bcc4/log
100% (2837 of 2837) |####################| Elapsed Time: 0:00:00 Time:  0:00:00
Loading database from '/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7/build/xc7/archs/artix7/channels.db'
2019-06-05 14:28:40.578516: About to load database
Dumping database to '/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7/build/xc7/archs/artix7/channels.db'
Traceback (most recent call last):
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7/xc7/utils/prjxray_form_channels.py", line 1651, in <module>
    main()
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7/xc7/utils/prjxray_form_channels.py", line 1630, in main
    import_phy_grid(db, grid, conn, get_switch, get_switch_timing)
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7/xc7/utils/prjxray_form_channels.py", line 390, in import_phy_grid
    db, write_cur, tile_types, site_types, tile_type, get_switch_timing
  File "/tmpfs/src/github/symbiflow-arch-defs-presubmit-xc7/xc7/utils/prjxray_form_channels.py", line 294, in add_wire_to_site_relation
    if isinstance(site_pin.timing, prjxray.tile.OutpinTiming):
AttributeError: module 'prjxray.tile' has no attribute 'OutpinTiming'
xc7/archs/artix7/CMakeFiles/file_xc7_archs_artix7_channels.db.dir/build.make:247: recipe for target 'xc7/archs/artix7/channels.db' failed
make[3]: *** [xc7/archs/artix7/channels.db] Error 1
make[3]: *** Deleting file 'xc7/archs/artix7/channels.db'
CMakeFiles/Makefile2:55746: recipe for target 'xc7/archs/artix7/CMakeFiles/file_xc7_archs_artix7_channels.db.dir/all' failed
make[2]: *** [xc7/archs/artix7/CMakeFiles/file_xc7_archs_artix7_channels.db.dir/all] Error 2
CMakeFiles/Makefile2:142: recipe for target 'CMakeFiles/all_xc7.dir/rule' failed
make[1]: *** [CMakeFiles/all_xc7.dir/rule] Error 2
Makefile:199: recipe for target 'all_xc7' failed
make: *** [all_xc7] Error 2

I thought I saw something go by to fix this?

https://github.com/SymbiFlow/symbiflow-arch-defs/pull/727/commits/0e52a41a83b19c0d5539108edd5f7175c9f6638d

FYI, you broken master by merging https://github.com/SymbiFlow/symbiflow-arch-defs/pull/805

Also, with the router lookahead still behaving poorly has resulted in increased runtime: https://github.com/SymbiFlow/symbiflow-arch-defs/issues/716#issuecomment-499276865


scrollToPosition not working

hi, scrollToPosition method not working for call galleryLayoutManager.scrollToPosition(position). It only works with galleryLayoutManager.attach(recyclerView, position) method.

this method can be used instead,
smoothScrollToPosition(RecyclerView recyclerView, RecyclerView.State state, int position)


Adoption by Drupal & i18n considerations

This code is being considered for adoption in Drupal 8:  https://drupal.org/node/1497290
There are concerns though about how English focused the dictionaries are.
It would also be good if the lists of strings could be updated regularly.
I assume too that there is support for all UTF8 characters.

This concern also came up in https://github.com/pypa/warehouse/pull/2542#issuecomment-367169819 .


create navbar for small devices

Checklist

[X] I've looked at the documentation to make sure the behavior isn't documented and expected.
[X] I'm sure this is an issue with Leaflet, not with my app or other dependencies (Angular, Cordova, React, etc.).
[X] I've searched through the current issues to make sure this hasn't been reported yet.
[X] I agree to follow the Code of Conduct that this project adheres to.

Steps to reproduce
create Navbar
Expected behavior
adding navbar
Current behavior

Minimal example reproducing the issue
No response
Environment

Leaflet version: 1.8.0-beta.0
Browser (with version):  Firefox 98.0.1
OS/Platform (with version):  Windows 10 laptop

Do you think I can work on this?

This looks like a feature add, not a bug fix to me. Links to different locations of site already exist on mobile version.

sure @HappyViki
I want to work on this issue ....assign me

Can I work on this please...

Closing this issue as I don't think this will add a lot of value right now, and we're already stretched thin. The navigation is already responsive, and putting it behind a navbar doesn't really feel like a massive enhancement. We will eventually have to get to a possible refresh of the site, so this could be a part of it in the future.


4.4.0 RC1 regressions

I am testing 4.4.0 RC1. Globally amazing :D
Just found a few issues

First there is some unwanted animations while zooming out. As you can see in the video the roads is shortly showing only the outline and not the road fill are not animated the same way. I am not sure what s happing so i attach the style.
osm.zip

https://user-images.githubusercontent.com/655344/121360295-9037f200-c934-11eb-9eab-605137a5791c.mp4

changing a style parameter is much much slower than before. It happens with any prop like enabling / disabling buildings. On my phone it usually takes around 3s

Great feedback, thanks! I have reproduced the style parameter changing issue, will fix this in RC2.
Regarding the unwanted style animations: SDK 4.4.0 now applies transition animations for appearing/disappearing vector tile layers automatically. Perhaps this is unwanted in some cases and should be disabled. Will look into this.

@mtehver your latest commits fixed the issue with style parameter change!! Thank you

@farfromrefug I tried to reproduce the issue with unwanted animations. I noticed some strange transitions between zoom levels, but these seemed to be result of the CartoCSS rules. For example, take the  following rule for 'primary' roads:
::casing {
	[class=primary][zoom>=12] {
		line-color: linear([view::zoom], (12, @primary-low-zoom-casing), (13, @primary-casing));
		line-width: linear([view::zoom], (12, @primary-width-z12), (13, @primary-width-z13), (15, @primary-width-z15), (17, @primary-width-z17), (18, @primary-width-z18), (19, @primary-width-z19));
	}
}

::fill {
	[class=primary] {
		[zoom>=8][ramp!=1],
		[zoom>=10] {
			line-color: @primary-low-zoom;
			line-width: linear([view::zoom], (8, @primary-width-z8), (9, @primary-width-z9), (10, @primary-width-z10), (11, @primary-width-z11));
			[zoom>=12] {
				line-color: @primary-fill;
				line-width: linear([view::zoom], (12, @primary-width-z12 - 2 * @major-casing-width-z12), (13, @primary-width-z13 - 2 * @major-casing-width-z13), (15, @primary-width-z15 - 2 * @major-casing-width-z15), (17, @primary-width-z17 - 2 * @major-casing-width-z17), (18, @primary-width-z18 - 2 * @major-casing-width-z18), (19, @primary-width-z19 - 2 * @major-casing-width-z19));
			}
		}
	}
}

This rule has a sharp transition when zoom=11 tiles are replaced with zoom=12 tiles. Perhaps it would be better to replace with something like this:
::casing {
	[class=primary][zoom>=12] {
		line-color: @primary-low-zoom-casing;
		line-width: linear([view::zoom], (12, @primary-width-z12), (13, @primary-width-z13), (15, @primary-width-z15), (17, @primary-width-z17), (18, @primary-width-z18), (19, @primary-width-z19));
	}
}

::fill {
	[class=primary] {
		[zoom>=10] {
			line-color: linear([view::zoom], (11.5, @primary-low-zoom), (13.0, @primary-fill)); // 11.5 and 13.0 key values may need tweaking
			line-width: linear([view::zoom], (8, @primary-width-z8), (9, @primary-width-z9), (10, @primary-width-z10), (11, @primary-width-z11), (12, @primary-width-z12 - 2 * @major-casing-width-z12), (13, @primary-width-z13 - 2 * @major-casing-width-z13), (15, @primary-width-z15 - 2 * @major-casing-width-z15), (17, @primary-width-z17 - 2 * @major-casing-width-z17), (18, @primary-width-z18 - 2 * @major-casing-width-z18), (19, @primary-width-z19 - 2 * @major-casing-width-z19));
		}
	}
}

This is simpler and does not have the issue.

@mtehver i tried with your change but i still see the issue. I ll dig deeper.
I found another visual "bug" with latest (i build from your latest changes).
As you can see the hillshade layer is like having a weird redraw effect. Feels like some parameters are quickly changing.
I am wondering if it comes from my build. Can you confirm you see it too?
https://user-images.githubusercontent.com/655344/124512619-1aab2e80-ddd9-11eb-9fbd-31dbff40ea7e.mov

@mtehver thanks ! https://github.com/CartoDB/mobile-sdk/commit/ce8e0c8af79a7f7c24a1a3fa98128175c8d906ec fixed the issue with hillshade layers!

@farfromrefug Thanks for reporting the hillshade issue. You are correct, the referred change was made to fix the issue.


Dataflow tooltips are not working on touch devices

This is a "general" D3.js issue, but would it be possible to fix this for d3-sankey specifically, since it's quite vital for readability? Demo e.g. http://bl.ocks.org/xaranke/9ada4c74a87b57ae7308

This issue is not specific to d3-sankey or even to d3; it’s an issue with SVG and accessibility. The energy flow example uses SVG title elements. You can use JavaScript for different tooltips if you prefer.
Please use Stack Overflow tag d3.js to ask for help. Although I make an effort to assist everyone that asks, I am not always available to provide help promptly or directly. Stack Overflow provides a better collaborative forum for self-help: tens of thousands of D3-related questions have already been asked there, and some answered questions may be relevant to you.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗


runtime: building Go on windows/arm64 hits inconsistent test failures

What version of Go are you using (go version)?

$ go version
go version devel +280c735b07 Fri Mar 5 16:02:44 2021 +0000 windows/arm64

Does this issue reproduce with the latest release?
N/A, cloned from master (280c735b07af9ea313d73049b0031f466e8d1000)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
C:\Users\u\code\go\src>..\bin\go env
set GO111MODULE=
set GOARCH=arm64
set GOBIN=
set GOCACHE=C:\Users\u\AppData\Local\go-build
set GOENV=C:\Users\u\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=arm64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\u\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\u\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Users\u\code\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Users\u\code\go\pkg\tool\windows_arm64
set GOVCS=
set GOVERSION=devel +280c735b07 Fri Mar 5 16:02:44 2021 +0000
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=0
set GOMOD=C:\Users\u\code\go\src\go.mod
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=C:\Users\u\AppData\Local\Temp\go-build4225962944=/tmp/go-build -gno-record-gcc-switches

What did you do?
Trying to build Windows arm64 go on a Surface Pro X.
Cloned latest go source, ran src\all.bat with a bootstrapped version of go cross compiled from my macbook and copied over. First run I hit a test failure of:

##### GOMAXPROCS=2 runtime -cpu=1,2,4 -quick
--- FAIL: TestGcLastTime (0.00s)
    gc_test.go:126: bad last GC time: got<PHONE_NUMBER>000702600, want [1614963630099702800,<PHONE_NUMBER>000702600]
FAIL
FAIL    runtime 21.884s
FAIL
2021/03/05 09:00:48 Failed: exit status 1

Running again I hit a different test failure:

--- FAIL: TestClientCallsCloseOnlyOnce (21.49s)
    client_test.go:2048: dial tcp <IP_ADDRESS>:57933: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
FAIL
FAIL    net/http        22.909s

On the third run everything passed successfully, and I appear to have a working go build.

@rsc this is probably of interest to you.

First fail seems related to low time resolution on Windows.
https://github.com/golang/go/blob/912f0750472dd4f674b69ca1616bfaf377af1805/src/runtime/gc_test.go#L120-L125
There is should be
t0 >= last || last >= t1

instead of
t0 > last || last > t1

Similiar failure
https://build.golang.org/log/b9491de2f3352bc70be88748959c5e1a06f5e35f

Different tests often fail due to different underlying causes; please file separate issues for test failures that are not obviously correlated.
I've filed the TestGcLastTime issue as #48072.

The TestClientCallsCloseOnlyOnce failure reproduced on the Go builder; filed as #49454.
(Closing this issue as redundant with the two specific ones.)


juniper_warp: panic when connection reset

Describe the bug
thread 'tokio-runtime-worker' panicked at 'Websocket receive error: WebSocket protocol error: Connection reset without closing handshake', ....cargo\git\checkouts\juniper-aa5e857cd28538c7\cab6dec\juniper_warp\src\lib.rs:495:46
To Reproduce
Not sure. But I guess reset a connection when sending websocket frame could reproduce.
Expected behavior
Don't panic because one connection is reset.
Additional context
https://github.com/graphql-rust/juniper/blob/6e4bc127b05dc29f7832cbf468c9fb2bc18f78ec/juniper_warp/src/lib.rs#L495

@nWacky

@spacemeowx2,
I looked it up.
For now it's unclear how to handle errors when dealing with channels. For now, it creates a new channel which returns all messages to end user and keeps executing logic inside ForEach future. Because ForEach expects return type of Future<Output = ()>, it's not possible to return errors in there => it's not possible to return errors for now (a potential workaround could be by implemeting custom future that drives iterator to completion).
For now, I can change juniper_warp's subscriptions impl so that it ignores errors

Would an optional dependency using https://github.com/tokio-rs/async-stream help?

What about using fold instead of for_each?

fold worked, didn't think of it.
#591 should resolve the issue


Loses mapped fields from parent

Hi
You neglect to add that if you do this on a child form on create from the parent, any mapped field values are lost as the form is refreshed,
Please confirm and add - would be great to keep any mapped values from the parent. This needs to be addressed
Pete

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

ID: 71065911-65c7-1464-97f9-d5c6a673c808
Version Independent ID: 6573c9a2-6933-a8fe-eb70-a047990e3312
Content: setActiveProcess (Client API reference) in model-driven apps - Power Apps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/formContext-data-process/activeprocess/setActiveProcess.md
Service: powerapps
GitHub Login: @adrianorth
Microsoft Alias: jdaly

Thanks for your comment, @PeterHale53. This issue has been assigned to the doc author for review.

@PeterHale53
Can you elaborate more on what you expect to find in the documentation about this?
Are you describing a bug, or is there some missing information that will help people get the desired outcome?
A new child record may be initialized with some default values set because the application uses the InitializeFrom Function and these values should be set in the form.
Seems like you are describing that the mapped values are lost when the setActiveProcess method is used. Am I understanding you correctly?

@PeterHale53
Thanks for your kind words.
Dev team confirmed that setActiveProcess will re-load the form, which will discard any unsaved changes, which includes the data from the mapping. That is something we can document but isn't likely to change.
The Create form contains a record that hasn't been saved yet, so I'm not sure how a BPF could be associated to a record with no id yet. The fact that this seems to work for system administrators is something I can't explain.
Have you tried just saving the record? Does it re-load with the BPF in this case?

Hi Jim
Thanks for confirming the reload and loss of mapping. Agree, not sure anything can be done about it, but all I was really asking for was a warning on the page saying that if you do change the BPF in the OnLoad on a create you will lose the mapping as you have suggested can be done.
As for my issue, if the record is saved and closed and then reopened the bpf appears fine. Strange. Others seems to have had the issue over the years, but it appears very intermittent although in our case it is consistent. Dev and Test in our tenant and then when we move to customer environment for UAT it is the same. We haven’t provisioned Production at the customer yet (probably tomorrow or Friday).
Anyhow, thanks for your time. Keep up the good work.
Pete
From: Jim Daly @.>
Sent: Wednesday, 29 June 2022 6:46 AM
To: MicrosoftDocs/powerapps-docs @.>
Cc: Peter Hale @.>; Mention @.>
Subject: Re: [MicrosoftDocs/powerapps-docs] Loses mapped fields from parent (Issue #3471)
@PeterHale53https://github.com/PeterHale53
Thanks for your kind words.
Dev team confirmed that setActiveProcess will re-load the form, which will discard any unsaved changes, which includes the data from the mapping. That is something we can document but isn't likely to change.
The Create form contains a record that hasn't been saved yet, so I'm not sure how a BPF could be associated to a record with no id yet. The fact that this seems to work for system administrators is something I can't explain.
Have you tried just saving the record? Does it re-load with the BPF in this case?
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/powerapps-docs/issues/3471#issuecomment-1169218859, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC72CEHQGI2G4JP2PF6ONNLVRNQCHANCNFSM5Z5CSPCA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

I've updated the topic to include a mention of the fact that calling setActiveProcess will reload the form.
I also updated the description with help from the dev team to try and bring more clarity, I think it got scrambled a bit over time with terminology changes.
Should go live this evening


Try yet again to fix Mac build

After guessing that removing the conflicting symlinks might solve the issue of not being able to install SDL2 on Mac after running brew update, then I found that this confirmed my guess:
Relates: actions/setup-python#577


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
11 seconds
sanity-inspect-image

✅ Succeeded
10 seconds
deprecated-base-image-check

✅ Succeeded
15 seconds
clair-scan

✅ Succeeded
57 seconds
clamav-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
16 seconds
sanity-label-check

✅ Succeeded
16 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary


Contribution guidance

It closes the issue #2 I added contribution guidance

Thanks For giving the contribution guidelines to our project.


ibus-engines.rime: 1.4.0 -> 1.5.0

Motivation for this change
Update ibus-engines.rime to 1.5.0
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 128181 at 41bdfe56 run on x86_64-linux 1

2 packages built successfully:

ibus-engines.rime
rime-data

@SuperSandro2000 Ping.


#82 Add jasmine to eslint rules

Associated Issue: #82
Summary of Changes

added jasmine env rule to eslint rules
added .idea to gitignore (Webstorm / PHPStorm auto generated folder)

Pull Request Test Coverage Report for Build 172

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.789%

Totals

Change from base Build 164:
0.0%

Covered Lines:
97

Relevant Lines:
107

💛  - Coveralls

Thanks!


demos ipython folder?

Prediction
Training and config
tfrecords?

better to just focus on reproducible examples in the docs
https://deepforest.readthedocs.io/en/master/getting_started.html#prediction


HAWQ-1062. Remove ASF incompatible Perl JSON module

Remove JSON Perl Module (Artistic license) module (files and
directory) from source tree.  The user wanting to run these Perl JSON
module dependent scripts (tidycat.pl, calico.pl, caqluniqdef.pl) are
instructed to download it from www.cpan.org if it is not found in dev
environment.  Here is error message received if JSON Perl Module is
not found:
Fatal Error: The required package JSON is not installed -- please
download it from www.cpan.org
JSON Perl Module versions 2.27 through 2.90 are known to work.

Update LICENSE file and remove licenses/LICENSE-json.txtfile.

Good to have this fix. +1

LGTM! +1

@xunzhang - I feel the error message received when using the tools is sufficient enough.  It informs the user the require JSON package is not installed and where to get it from (www.cpan.org).  That should be sufficient for a developer to install the module for use with the Perl utilities.  It is what I used when manually updating my system with the JSON module on my system in order to run the tidycat.pl utility.  Is that sufficient?

I think we could add more details  on wiki to save dev's time although dev do not call these perl script often. ed, you have had the successful experience so you can share on that page.
I just tried to install JSON perl module but failed, following the steps on http://www.cpan.org/modules/INSTALL.html
cpan App::cpanminus
cpanm Module::JSON

@edespino Cool, I missed the help info, that is enough. Seems Travis-CI failed, does this change affect building process?

@xunzhang - This is building the <IP_ADDRESS>-incubating branch and this is known to not pass.  The changes do not impact the build process.
@paul-guo- - I have added a section Running catalog tidycat perl modules to the build page https://cwiki.apache.org/confluence/display/HAWQ/Build+and+Install

Hi @edespino , I checked out the Travis Failure from https://s3.amazonaws.com/archive.travis-ci.org/jobs/161283621/log.txt. I think you should also modify lines around here together with this pull request~

You are correct Hong.  I will also be updating it to fix the travis based
builds.
-=e
On Tue, Sep 20, 2016 at 4:48 AM, Hong Wu<EMAIL_ADDRESS>wrote:

Hi @edespino https://github.com/edespino , I checked out the Travis
Failure from https://s3.amazonaws.com/archive.travis-ci.org/jobs/
161283621/log.txt. I think you should also modify lines around here
https://github.com/apache/incubator-hawq/blob/master/src/backend/catalog/caql/Makefile#L175
together with this pull request~
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-hawq/pull/918#issuecomment-248278820,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABN-GjeE6C5EIREP7Bf3FR8tI4EXRt2Dks5qr8gqgaJpZM4KBUed
.

--
Ed Espino
Director, Pivotal Technical Staff
*Cell: *925.389.4640
WeChat ID: EdEspino


Adds a code of conduct.

:+1:

:+1: – and ping @nateabele @petebacondarwin and @PascalPrecht who belong to this org and I happen to know.
"Angular UI" can use the official Angular Organization's as a starting point: https://github.com/angular/code-of-conduct
Let me know if I can help at all. This is important stuff.

:+1:

Thanks, @CoralineAda!

Hey, thanks. This is a super important issue, unfortunately I'm in the middle of a family crisis this week, so I can't give it the attention it deserves right now.
In the meantime, I'd be interested to hear feedback from the community. For a lot of people this is a no-brainer, but I've heard it can also be kinda contentious...
What say you, UI Router community?

I don't believe there is anything contentious about clarifying how to behave in an appropriate manner when interacting with a project.

@btford :( no mention for me

First, thank you @CoralineAda for putting in the effort for this. We want more people helping.
I'm not going to lie, this is a boring CoC that I don't think most people will read. Even I had trouble getting through it (kind of skimmed some of it). I'm not saying this to be disrespectable. I wish there was a fun code of conduct that people would actually want to read. Most are so dry and boring that I don't think most people read them, at least not all of it anyway. I know its a serious topic and I'm not trying to make light of it in anyway. But I'd rather have people have a good laugh or smile when reading it as long as they are reading it and taking it to heart and practice.
It makes me sad that we actually have to have things like this. Sometimes it really is culture though and people just don't know better. Other times they forget there are actual people on the other end of the keyboard. I always try to imagine what would happen if I said the things I put online in person. If I would get punched in the face then its probably not the best thing to say.
I guess I'm saying I am mixed on this. It is good for the people who will read it but the people who it is intended for probably won't care anyway. I am ok with adding it or adding a different one or just adding a note to the readme or somewhere else that says something like "Treat people like they are standing in front of you and can punch you in the face if you get out of line". Grossly oversimplified and doesn't cover all cases but thats the point: nothing ever will. But I suppose something is better than nothing.
p.s. :( to no trolling: https://github.com/angular-ui/ui-router/pull/2309/files#diff-a1ee87dafebc22cbd96979f1b2b7e837R11
/rant

@petebacondarwin well said. :)

Alright. I had a chance to sleep on this for a couple of nights, sort out my own thoughts, and consult some trusted friends who I admire for their leadership positions in open, positive, inclusive communities that I've observed or been a part of. I've also heard privately from community members who were understandably reluctant to air their views in public.
First of all, thanks for kicking off this discussion, and thanks for addressing the inappropriate comment that led to it. If nothing else, I'm proud of the fact that it was the first instance of inappropriate behavior that I've observed in 2½ years of being a maintainer on this project, and that it was resolved quickly and smoothly once raised. Hopefully the conclusion of this will be that the whole community is clear on the boundaries, and if any future inappropriate comments do occur, they won't sit unaddressed for months. :smile:
This discussion has forced me to sit and think about the type of community and culture that I want to create, the intentionality of which is something that's clearly overdue for a project at this level of visibility, but we'll get to that in a minute.
I have to say I'm a bit taken aback by the fact that the people primarily championing this have, so far as I can tell, had no prior involvement in the project. I'm not entirely sure what to make of it, but I will proceed on the assumption that you're all here in good faith.
To address the proposed CoC itself... after having read it carefully a number of times, I can't say I'm comfortable with the language. In particular, I'm uncomfortable with being made to be responsible for things outside of my purview, and I'm uncomfortable with the idea of having to mediate things that I can't observe or verify. Overall, it seems to be phrased primarily in the negative. I also carefully read the FAQ. I really don't love the patronizing generalizations it makes about its intended audience. I also read the CoC linked by @btford. While it's definitely phrased in a more positive way, I think we can do better.
I sat and thought about what a CoC for this project would look like, and came up with the following wish list:

Focus primarily on how to treat people, instead of on how not to treat people
Address any unprofessional, discourteous, or dishonoring comments, or any other comments that take focus away from the work
Address sarcasm and passive-aggression
Be expressly apolitical; this includes any and all mention of arbitrary group identifiers
Create a space to collaborate and get work done; political discussions or causes are expressly forbidden
In the presence of ambiguity, assume good intent
Use instances of inappropriate behavior to teach, not to shame
Absolutely forbid allowing or enabling offenses to live beyond correction & apology, in any public medium
Be an ideal for people to aspire to, not a stick to beat them with

This certainly doesn't mean there will be no explicit boundaries or repercussions, I just have yet to see a policy that strikes a balance between the one-dimensional and the ad-hoc. A friend pointed me to the Open Code of Conduct, which I plan on poking through shortly. In the meantime, consider the foregoing to be the ground rules for the project going forward. Infractions can be reported to me directly. My email address is on my profile. I'll set up a community@ account for the project at my earliest opportunity.
This informal policy has the full unanimous consensus of the project team. Thanks for your time, thoughtful consideration, and input. I'd love to have further input from the existing UI Router community and user base.

@nateabele Beautifully stated.

I have to say I'm a bit taken aback by the fact that the people primarily championing this have, so far as I can tell, had no prior involvement in the project. I'm not entirely sure what to make of it, but I will proceed on the assumption that you're all here in good faith.

I don't think there's anything wrong with developers who like to contribute to open source suggesting to add a code of conduct so they feel they can contribute without harassment. However, since this seems to be controversial, i'm going to unsubscribe myself from this thread. Good luck.

The Contributor Covenant is the default CoC for Ruby gems built with bundler and has been adopted by over 4700 projects. I strongly advise against trying to write your own CoC; documents like the CC have built on the collective, real-world experiences of marginalized people in tech.You ignore them at your own peril.

I don't think there's anything wrong with developers who like to contribute to open source suggesting to add a code of conduct so they feel they can contribute without harassment.

That's totally fair. I mentioned it because the events leading up to this discussion pattern-match to other incidents that I've seen go Downhill Fast™. To be clear, that is in no way a reflection on you.

The Contributor Covenant is the default CoC for Ruby gems built with bundler and has been adopted by over 4700 projects.

Please don't take my comments as anything other than my assessment that the CC doesn't fit exactly with the culture I aspire to create for this project. This isn't a judgement of CC in the abstract, nor is it a suggestion that the CC isn't perfect for others.

I strongly advise against trying to write your own CoC

I'm only slightly less interested in writing my own license (...that is, I am vehemently disinterested in writing my own license). :smile: However, just as there are a variety of licenses to meet a variety of project needs and intents, it follows that there would be a variety of CoCs to address the many different styles and flavors of Open Source cultures and organizations. This is a topic I still don't know enough about and am interested in exploring. The above is a beginning, not an end.

I'm curious as to what that peril might be.

Adding another one for future reference: https://indiewebcamp.com/code-of-conduct

@nateabele I am very happy that you are taking this seriously and I am impressed that you are putting so much thought into the culture that you wish to cultivate in ui-router. I am sure that the team will agree on an acceptable code of conduct. Really the key is that people should know what to expect and how to act and that there is a clear path to dealing with indiscretions quickly and with the least amount of wasted time and upset for all those involved!

Not involved with the project (my intentions are good :relaxed:) but I stumbled upon this thread and just wanted to encourage and praise you for what you're doing here @nateabele

@petebacondarwin Totally. The other part is, I haven't always treated people here the way I want to, so if I'm setting a standard of behavior that I know people will hold me accountable to in a constructive, encouraging way, that helps me, too.
@kvz Thanks buddy. :smile:

Just to add my thoughts - I agree wholeheartedly with the CoC. We implemented in our Angular 2 UI Bootstrap work, as can be seen here & should have it in UI Bootstrap soon as well, and personally believe it is only a good thing.


feat!: rewrite with URL ponyfill

Codecov Report

Merging #4 (e15c01d) into main (d15b6f0) will decrease coverage by 12.26%.
The diff coverage is 87.73%.

@@             Coverage Diff              @@
##              main       #4       +/-   ##
============================================
- Coverage   100.00%   87.73%   -12.27%
============================================
  Files            1        5        +4
  Lines           47      106       +59
  Branches        14       35       +21
============================================
+ Hits            47       93       +46
- Misses           0       13       +13

Impacted Files
Coverage Δ

src/ufo.ts
77.77% <77.77%> (ø)

src/parse.ts
93.33% <93.33%> (ø)

src/encoding.ts
100.00% <100.00%> (ø)

src/index.ts
100.00% <100.00%> (ø)

src/utils.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

(updating docs later because to release faster)


Can no longer block file type

The file type can no longer be blocked.
All file type are now allowed.
Stephane

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

ID: a3a0cebd-7e82-6631-bdee-031b3b84f74d
Version Independent ID: ee04db29-a6db-2874-2573-d80f0b415308
Content: Block syncing of specific file types - OneDrive
Content Source: OneDrive/block-file-types.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@StephaneSeguin Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

All file types are allowed in SharePoint sites, however you can still control file types when syncing to OneDrive For Business, which is different and it's what this article is referring to. No change needed.

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Evaluate whether it is more convenient to use a configuration file system for the Task class

The Task class takes as input > 10 parameters. We will evaluate if the package of configuration management Dynaconf can help in the setting of the many Task class' parameters.
By using Dynaconf, the loaded config (which is a variable) is importable and thus can be accessed across different functions/classes/modules without passing it as a parameter (Detail see example Using Python Only).

https://github.com/ANNUBS/annubes/pull/37#issue-2146219429:

Thinking about it, I am not convinced. It makes no sense to create a class whose only function is to create a few attributes in the init, and from what I understand this would be the case. Maybe a dictionary passed to the Task class would be more than enough for this case. I also opened an issue (https://github.com/ANNUBS/annubes/issues/36) for evaluating Dynaconf - The configuration management package, which I still think is the best way to go.
Thoughts? @DaniBodor

I agree that a settings dictionary could be a simpler way of doing it compared to a dedicated class. On the other hand, dataclasses do offer a bunch of advantages over dictionaries. For example, it would be a bit convoluted to create a dictionary where some values are explicitly given by the user and others loaded from default. Other things that would be funky is type hinting the dictionary.
In the end, the dataclass can easily be loaded into Task as a dictionary (which I agree would be the way to go) using vars(TaskSettings) (or whatever the name of that dataclass would be).
All that being said, if there's a better way of doing it, I'm all for it!
I don't know the Dynaconf package, but if you think it's a robust system and a better way to store the settings then let's do it that way. The only usage of configuration files I have seen in the past were very ad hoc and error prone, but if that is not the case with this system then let's go for it.
Ultimately, this might be a thing where we try both ways and see which one we think is better in practice rather than theorizing about it for a long time. If you want we can divide it up, where one of us does Dynaconf and the other does dict/dataclass.
Let me know what you think.

Okay let's proceed with the dataclass option for now, which is the fastest, and let's keep this issue for the future. In the meantime, I'll understand better how Dynaconf works in the NPLinker project, and then I will have a better idea if it can work for us as well.

We've decided to implement the functionalities originally planned for the annubes package within the NeuroGym package instead, which represents the current state-of-the-art. For now, this is not relevant anymore. In case we will open the issue again.


redis lock with sentinel missing masterName

according to https://github.com/redisson/redisson/wiki/2.-Configuration#27-sentinel-mode
we should call setMasterName() here: https://github.com/pavelhalabala/conductor/blob/master/redis-lock/src/main/java/com/netflix/conductor/locking/redis/config/RedisLockModule.java#L75
checking the https://github.com/pavelhalabala/conductor/blob/master/redis-lock/src/main/java/com/netflix/conductor/locking/redis/config/RedisLockConfiguration.java, i don't see a property defined for master name using sentinel.

Hey @zodiac1214, Thanks for reporting this. Does adding it to the Sentinel configuration work for you? If so, can you send us a PR with the same.

Hi, I believe adding a property under server.properties with other redis.lock.server properties fixes this.
Also we need to add this property while creating the reddison config


Added the ability to set a css stylesheet & added valid html tags to pages

Added the ability to set a css stylesheet & added valid html tags to pages so that firefox doesn't leave random Unicode characters all over the place.
If this goes ahead, I'm happy to update the documentation to reflect this change

Thanks for offering these changes @bsncubed but I'm not really looking to increase the scope or functionality of these scripts much wider to keep maintenance reasonable, they're really here to provide some simple starting points or examples for various scenarios.
Feel free though to extract your more developed work into it's own repository for it to then be added to the "Community Projects & Scripts" list in the project readme.


Add analyser prefix customisation and output directory to command

Resolves https://github.com/ppy/osu-localisation-analyser/issues/28
Resolves https://github.com/ppy/osu-localisation-analyser/issues/29
For osu!, we'll move the resource files to osu-resources. This adds an editorconfig option to adjust the "prefix" value, which is currently wired to the namespace of the Strings class.
For osu!, this will be set to dotnet_diagnostic.OLOC001.prefix_namespace = osu.Game.Resources.Localisation.
It defaults to {AssemblyName}.Localisation as previously.
I've also added an --output option to the to-resx command, in order for the resource files to be easily uploadable to crowdin:
[smgi@smgi net5.0]$ ./LocalisationAnalyser.Tools to-resx /home/smgi/Repos/osu/osu.Game/osu.Game.csproj --output ~/Desktop/test
Converting all localisation files in /home/smgi/Repos/osu/osu.Game/osu.Game.csproj...
Processing ButtonSystemStrings.cs...
  -> /home/smgi/Desktop/test/ButtonSystem.resx
Processing ChatStrings.cs...
  -> /home/smgi/Desktop/test/Chat.resx
Processing CommonStrings.cs...
  -> /home/smgi/Desktop/test/Common.resx
Processing NotificationsStrings.cs...
  -> /home/smgi/Desktop/test/Notifications.resx
Processing NowPlayingStrings.cs...
  -> /home/smgi/Desktop/test/NowPlaying.resx
Processing SettingsStrings.cs...
  -> /home/smgi/Desktop/test/Settings.resx

Are the CI failures valid on this one?

Ah, yep.


oxygenctl-action@v4

This PR:

Introduces a v4 to oxygenctl-action and updates the action's oxygenctl binary to v3. This breaking change removes the shop_id, storefront_id and workflow_run_id inputs, as they as no longer necessary. See https://github.com/Shopify/oxygenctl/pull/70 for more details.
Changes the format of the workflow ID to be a combination of the workflow_run_id and the workflow_run_attempt. This is done so that a unique ID is generated on every GitHub workflow re-run, which causes every rerun to be treated as a new deployment. We'll start depending on the v4 tag at the same time as a change to use the new workflow ID format to query Oxygen for deployments.

/shipit

/shipit


Замахаев Влад. Lecture04 (Cache)

Coverage increased (+11.7%) to 62.42% when pulling 5ca2a179b8e531e4d63306dc2554addf55b438ae on zamakhaevV:lecture04 into 1f005511cdff2990e3c484daf5158496734cc1e5 on rybalkinsd:lecture04.

Coverage increased (+10.6%) to 61.25% when pulling 0a463e6e6631969cbfe13e2414c562a95eeeeb2e on zamakhaevV:lecture04 into 1f005511cdff2990e3c484daf5158496734cc1e5 on rybalkinsd:lecture04.

+


Send different email updates to different types of ticket holders, fix #283

Instead of a different body for each ticket type, better to have a dropdown for ticket type on the event update form, and just one subject and message box.

I can propose that to handsup, but at present I've implemented it the way they described in meeting.

Instead of a different body for each ticket type, better to have a dropdown for ticket type on the event update form, and just one subject and message box.

I can propose that to handsup, but at present I've implemented it the way they described in meeting.

I would strongly suggest to them that they do it this way. It'll be much easier to use, and more flexible - e.g. if you only want to email one group, the current solution wouldn't work.

Amira has asked to try this solution before deciding.

I absolutely disagree with merging code like this.
If we start doing things like
payload_json = await self.request.json()

And intentionally avoiding validation, but keeping the validation in the codebase in order to confuse a future developers (it's not doing anything else), the codebase will be unusable in a matter of months.
You don't need to merge PRs to deploy them for testing.
This is a route to errors and getting hacked.
@pro-Nate please revert this pull request.


Adjust json parser to new format

Motivation
Adjust parser to new format

@wtrocki why two examples files? (assets/mobile-services.json and assets/dummy-mobile-services.json)

@secondsun Can we say why we have 2 example files? They both are used for unit test purposes.

They are used for unit tests.

@pb82 I have added 2 different JSON conf file to test the setConfigFileName. See MobileCoreTest.java

@secondsun @danielpassos thanks, was only curious.


dependency issue

Because every version of video_thumbnail_generator depends on http ^0.12.2 and  depends on http ^0.13.4, video_thumbnail_generator is forbidden.

Any updates on this issue?

Is there any alternative for this library?
I found a hint on stack overflow showing videoplayer for extracting thumbnail. But the code was not complete.
https://stackoverflow.com/questions/52838120/how-to-make-video-thumbnail-in-flutter

not really - I've been thinking of getting it done from someone on UpWork (it will be for my own use though, as this will likely be on Heroku, as was this service)

Does not matter - this package is dead as the link https://video-thumbnail-generator-pub.herokuapp.com/generate/thumbnail link is a web service that is now down.

I didn't expect it to be a picture that needs to be uploaded to the network and read back, hahahahaha

video-thumbnail-generator.zip
Got this developed as a paid project on UpWork. You can host this on Heroku (instructions included). Set a key in the .env file and use this when calling this way:

If it is worth your while, you could sponsor a coffee direct to the developer - let me know

视频合成器.zip
您可以在 Heroku的付费上使用文件中设置一个键，并以这种方式调用时设置一个键。
http.Response response = await http.post( Uri.parse(HerokuVideoThumbnail.domain), headers: {"Content-type": "application/json"}, body: '{"videoUrl" : "${widget.videoUrl} ", "key": "${key}"}';

video-thumbnail-generator.zip
Got this developed as a paid project on UpWork. You can host this on Heroku (instructions included). Set a key in the .env file and use this when calling this way:
http.Response response = await http.post( Uri.parse(HerokuVideoThumbnail.domain), headers: {"Content-type": "application/json"}, body: '{"videoUrl" : "${widget.videoUrl}", "key": "${key}"}';

very troublesome

video-thumbnail-generator.zip
Got this developed as a paid project on UpWork. You can host this on Heroku (instructions included). Set a key in the .env file and use this when calling this way:
http.Response response = await http.post( Uri.parse(HerokuVideoThumbnail.domain), headers: {"Content-type": "application/json"}, body: '{"videoUrl" : "${widget.videoUrl}", "key": "${key}"}';

very troublesome

@InTheClodus could you explain? I've taken the trouble to build and share code, so let us discuss.
If you read the code, you will see that there is a size that you may need to change (I have set it to 114 x 64)
let cmd = ffmpeg -i ${req.body.videoUrl} -ss 1 -frames:v 1 -s 114x64 ${filename};
What is the "trouble"?

@InTheClodus could you explain? I've taken the trouble to build and share code, so let us discuss.
If you read the code, you will see that there is a size that you may need to change (I have set it to 114 x 64) let cmd = ffmpeg -i ${req.body.videoUrl} -ss 1 -frames:v 1 -s 114x64 ${filename};
What is the "trouble"?

The video-thumbnail-generator project is a plugin for flutter to get thumbnails of online videos, but now I want to use it, and I need to build a nodejs project first. Isn't that troublesome enough?

@InTheClodus the package was using a free cloud function, that has been pulled down - cloud functions cost money, so its not surprising that someone shut it down. I'm not aware of accomplishing this with pure Dart or native Flutter capabilities.
All you need is host this - the instructions are simple for any developer. I was able to get this hosted on Heroku in a few minutes -  there is a free tier too, check the license terms. Or host on Google Cloud, which is a pay as you go.


Culling buggy when many part-models loaded

Describe the bug 📝
Possibly related to #209.  I have 18 files which are all seperate building stories from the same project.  As i orbit around the model, whole sections of the model dissappear, even though they shouldn't be culled.  Unfortunately, I couldn't test this on that open viewer, because of #209, but the screens below demonstrate the behavior.

Files have been privately shared to<EMAIL_ADDRESS>Reproduction ▶
No response
Steps to reproduce 🔢
No response
System Info 💻
openbim-components

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

Update: I changed the guids myself so that I could load the part models onto https://platform.thatopen.com/app.  The same issue is persistent (even worse?).

File privately available here.


Hamburger menu dark mode background not changing.

Describe the bug
The Hamburger menu background doesn't change when in dark mode yet the text color does change making reading the links an impossible task. I've tested on multiple browsers and the same issue.
To Reproduce
Steps to reproduce the behavior:

Go to Hamburger Menu in DarkMode
Click on it
Scroll down to  look at the open nav menu from hamburger
See White Box with no apparent links.

Expected behavior
Hamburger menu, when opened on Dark mode will appear as a white box with no apparent text.  Links are active and you can click them.
Screenshots

Desktop (please complete the following information):

OS: MacOs 10.15.7
Browser: Chrome, brave
Version: 102.0.5005.61

Smartphone (please complete the following information):

Device: Pixel 6/ One+ 9
OS: latest
Browser chrome/ brave/ Vivaldi
Version latest

Additional context
No additional context is needed I believe.  I love this theme and your work!

I have a fix for this in #347


Create a global ranklist of all the users.

A global ranklist should sort the users based on a few parameters like, how much questions he/she has solved, his/her accuracy, no. of submissions in each question. This can only be resolved after creation of a question database as user can only submit after that.

I wish to work on this issue. Please assign. Till the completion of question database,
I can work on algorithm and interface and test it with trial inputs

@rapperdinesh good luck. Start with the issue


Allowed memory exhausted.

Here what I get when I use realText() in a loop:
PHP Fatal error:  Allowed memory size of 268435456 bytes exhausted (tried to allocate 4096 bytes) in C:...\faker\fzaninotto\faker\src\Faker\Provider\Text.php on line 91

What's your locale? How many iterations have you done when the memory is exhausted?

Local fr_FR and two successful and then error.

I tried to place the call into a function hope the memory would be freed upon return, but changed nothing.

You could try invoking gc_collect_cycles at the start of the function to force cleanup of any pending cycles left after the previous invocation.
Not a solution but a workaround of course.

Thank you very much, apparently that do the trick.

@fzaninotto perhaps if the text generator is that memory hungry this should be part of the provider wrapper?

Got same issue here in vendor/fzaninotto/faker/src/Faker/Provider/Base.php.

I fixed this issue temporarily by extending the php memory size.

I'm sorry, but why it's closed? Is it fixed?

Hi,
I closed it because I got an answer which fixes my problem. Now I don't know if @fzaninotto has or is about to fix it.

Okey, got it. Increasing memory helped me too. But it's interesting, am I wrong with using it or it's a bug connected with memory leaks?

I think it creates all models, then saves. Because when it fails here is no data added there. Or it uses transaction.
Dividing model creation into parts works:
$count = \App\User::count();
for ($i = 0; $i < 10; $i++) {
    $this->command->info("Adding $count answers");
    factory(\App\UserAnswer::class, $count)->create();
    gc_collect_cycles();
    $this->command->info("Current answers: " . ($count * ($i + 1)));
}

Increasing memory doesn't help me if I run unit test more than 3-4 times, so I had to manually add gc_collect_cycles() before each call to text generating method.
Is there any solution for this issue? :)

Like I said in https://github.com/fzaninotto/Faker/issues/1125#issuecomment-268825965 I'd recommend adding the gc_collect_cycles call to the wrapper.

Ran into this problem. Changed $faker->realText(100) to $faker->sentence(10). Getting similar fake data but without error (Can't be sure if I'll get the error in a future when adding more tests tho)

I tried running it in a loop but there is no such thing as increasing memory or a leak. The usage is stable at MEM: 38.50390625MB
$faker = Faker\Factory::create( 'cs_CZ' );
for ($i=0;$i<1000;$i++) {
    $faker->realText(10000).PHP_EOL;
}

Please note that you should not create a new Faker instance on the factory for each loop which will consume a lot of memory as intended where it will keep it all in memory.

I will close this issue for now since the fact that we can not reproduce the issue and it is pretty old already. In case more is needed, just let me know

@pimjansen any comments on https://github.com/fzaninotto/Faker/pull/1730#issuecomment-619435477 ?

it in a loop but there is no such thing as increasing memory or a leak. The usage is stable at MEM: 38.50390625MB

tried running the loop 1.000.000 and got the memory exhausted in a loop of about less than 200.000. seems like you running it in a very short loops.
just fyi, factory created at outside of the loop. even after i called gc_collect_cycles() at each loop still have the same error.


app.exception_handler does not work when used in separate file

Discussed in https://github.com/awslabs/aws-lambda-powertools-python/discussions/1191

Originally posted by arunbhati April 29, 2022
Hi,
I am using APIGatewayRestResolver and I'm having trouble working with @app.exception_handler. When I have APIGatewayRestResolver object and @app.exception_handler decorator in same file then it works but if I create a separate file for exceptions(exceptions.py) and do the import of APIGatewayRestResolver then it doesn't work.
Following are steps to reproduce.

Create a lambda function in Python with Lambda Layer for AWS Powertools from here.
Attach this Zip file to lambda function - Zip file: Archive.zip
Use following input for Lambda request

{
  "resource": "/proxy",
  "path": "/proxy",
  "httpMethod": "GET",
  "headers": null
}

Lambda function should fail with following error

{
  "errorMessage": "",
  "errorType": "ValueError",
  "requestId": "24f06853-16fc-4657-9360-40409f0726be",
  "stackTrace": [
    "  File \"/var/task/lambda_function.py\", line 10, in lambda_handler\n    return axis_controllers.main(event, context)\n",
    "  File \"/var/task/src/abc/axis_controllers.py\", line 11, in main\n    return app.resolve(event, context)\n",
    "  File \"/opt/python/aws_lambda_powertools/event_handler/api_gateway.py\", line 498, in resolve\n    return self._resolve().build(self.current_event, self._cors)\n",

Now move ValueError exception method from exceptions.py into init.py file in same folder.
Rerun test with same input and now lambda output will show expected result(shown below).

{
  "statusCode": 400,
  "headers": {
    "Content-Type": "text/plain"
  },
  "body": "Invalid request",
  "isBase64Encoded": false
}

I don't have much expertise on how decorator works so any help here would be appreciated.
Thanks.

hey @arunbhati thanks for sending over an archive to help reproduce it quickly! The issue is that exceptions.py code is never executed, because it isn't being imported anywhere else.
To confirm it, I've added a dummy constant within exceptions.py and imported it from your lambda_function.py. I'm gonna close this and create a feature request to support defining exception handlers within in separate files via Router instances - this will make the  process more easily in the future (after our pause).
lambda_function.py
import src.abc.axis_controllers as axis_controllers
from src.abc.exceptions import HELLO   # HERE!

def lambda_handler(event, context):

    return axis_controllers.main(event, context)

event = {
  "resource": "/proxy",
  "path": "/proxy",
  "httpMethod": "GET",
  "headers": ""
}

print(HELLO)
lambda_handler(event, {})

exceptions.py
from aws_lambda_powertools.event_handler import content_types
from aws_lambda_powertools.event_handler.api_gateway import Response
from aws_lambda_powertools.event_handler.exceptions import NotFoundError

from src.abc import app

HELLO = "WORLD"

@app.exception_handler(ValueError)
def handle_value_error(ex: ValueError):
    metadata = {"path": app.current_event.path}
    print(f"Malformed request: {metadata}")

    return Response(
        status_code=400,
        content_type=content_types.TEXT_PLAIN,
        body="Invalid request",
    )

@app.not_found
def handle_not_found_errors(exc: NotFoundError) -> Response:
    # Return 418 upon 404 errors
    print(f"Not found route: {app.current_event.path}")
    return Response(
        status_code=418,
        content_type=content_types.TEXT_PLAIN,
        body="I'm a teapot!"
    )

I'm gonna close this and create a feature request to support defining exception handlers within in separate files via Router instances - this will make the process more easily in the future (after our pause).

Hi @heitorlessa did you actually create this feature request? I'm having trouble finding it. Was it every implemented, if so?

Hey @jasmarc yeah, sorry I didn't update this back (my bad) - it's already live: https://awslabs.github.io/aws-lambda-powertools-python/2.7.1/core/event_handler/api_gateway/#specialized-router-types

PR: https://github.com/awslabs/aws-lambda-powertools-python/pull/1824

Hi @heitorlessa, has this issue been solved with the mentioned PR: https://github.com/aws-powertools/powertools-lambda-python/pull/1824? With newest version of powertools (2.25.0) I am not able to define exception handlers in a separate file with Router. Is there some other way to achieve it?

oh hey, seeing this only now as it was closed -- That's something else, that would be a feature request to add exception_handler to the Router class. It was an oversight. This issue was for App not Router.
@rajalokan if you've got the bandwidth, it'll need two changes (we need a feature request issue):

Add exception_handler method in the Router
Merge registered exception handlers from Router into App - it'd be a dict merge (self._exception_handlers)

Any update on this feature @heitorlessa  // @rajalokan ?

Reopening this issue as using @router.exception_handler still doesn't work.
Sending a PR in few minutes.

Thank you for commenting here and bringing this issue to our attention @alecrevangelista!


[WIP] Adding support for java projects.

[x] sous detect working with pom.xml
[x] sous dockerfile generating something sensible.
[ ] use the java-specific base image / or install maven
[ ] support tags with the format - (currently warning with 0.0.0)
[ ] support multi-module projects

//cc @samsalisbury @stevenschlansker

Thanks for looking at this!  Hopefully my comments are helpful.

Closing in favor of https://github.com/opentable/sous-buildpacks/pull/1


no sound in youtube video unless the audio .webm gets ffmpeg-ed into .ogg with "-c copy"  first

youtube-dl --user-agent "Mozilla/5.0 (X11; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0" -k https://www.youtube.com/watch?v=DYGc-31mEec
That pulls a *.f136.mp4 video, a *.f251.webm audio , and they get converted into a .mkv but with vlc there's no sound (either on the .webm or the .mkv files, in fact the sound card goes to sleep, even though statistics say that audio gets decoded just fine).
If I run opusinfo *.webm I get some errors which say that's not a valid ogg. Then if I convert the webm into ogg via ffmpeg(this shaves 453,728 bytes) then there's audio on the resulting .ogg; this ffmpeg command:
ffmpeg -i "file: ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.f251.webm" -c copy "file:stuff.ogg"
I tried renaming webm to ogg but there's still no audio, so that ffmpeg conversion is needed apparently. Maybe's my vlc acting out? (latest git, but config options may matter)
EDIT: forgot to mention youtube-dl version, since it's not latest git:
$ youtube-dl --version
2015.09.28-gentoo_no_offensive_sites

Looks like for extracting audio only, the sound works fine(in vlc player) because it gets transformed from webm into opus by youtube-dl via this command:
ffmpeg -y -i "file:Epic Action _ Path of Aroha - In The Shadow of a Colossus - EpicMusicVN-BNyuHH7kUJk.webm" -vn -acodec copy "file:Epic Action _ Path of Aroha - In The Shadow of a Colossus - EpicMusicVN-BNyuHH7kUJk.opus"
Command I used:
youtube-dl --user-agent "Mozilla/5.0 (X11; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0" --extract-audio https://www.youtube.com/watch?v=BNyuHH7kUJk
And if I pass the -k param, it gets the video file too, but vlc-ing the webm file (or the resulting mkv) still no sound! (good thing it does an opus too, with the -k arg, although maybe it shouldn't pull the video? guess that's another issue though)
$ downaudio -k https://www.youtube.com/watch?v=BNyuHH7kUJk
[youtube] BNyuHH7kUJk: Downloading webpage
[youtube] BNyuHH7kUJk: Downloading video info webpage
[youtube] BNyuHH7kUJk: Extracting video information
[youtube] BNyuHH7kUJk: Downloading DASH manifest
[youtube] BNyuHH7kUJk: Downloading DASH manifest
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[download] Destination: Epic Action _ Path of Aroha - In The Shadow of a Colossus - EpicMusicVN-BNyuHH7kUJk.f136.mp4
[download] 100% of 29.82MiB in 00:08
[download] Destination: Epic Action _ Path of Aroha - In The Shadow of a Colossus - EpicMusicVN-BNyuHH7kUJk.f251.webm
[download] 100% of 4.36MiB in 00:01
[ffmpeg] Merging formats into "Epic Action _ Path of Aroha - In The Shadow of a Colossus - EpicMusicVN-BNyuHH7kUJk.mkv"
[ffmpeg] Destination: Epic Action _ Path of Aroha - In The Shadow of a Colossus - EpicMusicVN-BNyuHH7kUJk.opus

Post the full output of youtube-dl when run with --verbose option.

$ cd /tmp;downvideo -k --verbose https://www.youtube.com/watch?v=DYGc-31mEec
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--user-agent', u'Mozilla/5.0 (X11; Linux x86_64; rv:39.0) Gecko/20100101 Firefox/39.0', u'-k', u'--verbose', u'https://www.youtube.com/watch?v=DYGc-31mEec']
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2015.09.28-gentoo_no_offensive_sites
[debug] Python version 2.7.10 - Linux-4.1.7-hardened-r1-g45b4b78-x86_64-AMD_A6-3400M_APU_with_Radeon-tm-_HD_Graphics-with-gentoo-2.2
[debug] exe versions: ffmpeg 2.8, ffprobe 2.8
[debug] Proxy map: {}
[youtube] DYGc-31mEec: Downloading webpage
[youtube] DYGc-31mEec: Downloading video info webpage
[youtube] DYGc-31mEec: Extracting video information
[youtube] DYGc-31mEec: Downloading DASH manifest
[youtube] DYGc-31mEec: Downloading DASH manifest
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on 'https://r1---sn-h5bt5guxax3vh-qxce.googlevideo.com/videoplayback?id=0d819cfb7d6611e7&itag=136&source=youtube&requiressl=yes&mn=sn-h5bt5guxax3vh-qxce&mm=31&ms=au&pl=20&mv=m&ratebypass=yes&mime=video/mp4&gir=yes&clen=281392417&lmt=1443527915463881&dur=2028.392&sver=3&mt=1443862895&upn=YpMeCBHRxjw&signature=3C75E272EF86A991C4B4D4E5CD82F3A7F7DB2131.4E7A9E5F3F45272D93E6E9C82E6F7EBB484E094A&fexp=9405998,9406852,9407116,9408710,9409069,9415365,9415485,9416023,9416126,9416729,9416838,9417707,9417815,9418153,9418391,9418448,9419743,9420348,9421013,9421252,9421351&key=dg_yt0&ip=<IP_ADDRESS>&ipbits=0&expire=1443884631&sparams=ip,ipbits,expire,id,itag,source,requiressl,mn,mm,ms,pl,mv,ratebypass,mime,gir,clen,lmt,dur'
[download] Destination:  ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.f136.mp4
[download] 100% of 268.36MiB in 01:13
[debug] Invoking downloader on u'https://r1---sn-h5bt5guxax3vh-qxce.googlevideo.com/videoplayback?expire=1443884631&dur=2028.401&ipbits=0&sver=3&key=yt6&mime=audio%2Fwebm&gir=yes&signature=5F5F8CF2852A20A5A5804E03D7BA8B318347AEC8.45155720EC4DDA897517BD6B270239A8F1A95284&clen=34031062&itag=251&upn=EsLdmunxJdc&source=youtube&initcwndbps=902500&lmt=1443338875369980&keepalive=yes&sparams=clen%2Cdur%2Cgir%2Cid%2Cinitcwndbps%2Cip%2Cipbits%2Citag%2Ckeepalive%2Clmt%2Cmime%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Crequiressl%2Csource%2Cupn%2Cexpire&mv=m&mt=1443862895&ms=au&fexp=9405998%2C9406852%2C9407116%2C9408710%2C9409069%2C9415365%2C9415485%2C9416023%2C9416126%2C9416729%2C9416838%2C9417707%2C9417815%2C9418153%2C9418391%2C9418448%2C9419743%2C9420348%2C9421013%2C9421252%2C9421351&requiressl=yes&ip=<IP_ADDRESS>&mn=sn-h5bt5guxax3vh-qxce&mm=31&pl=20&id=o-APZzVMD8hSghX6adxuJzu6QsOfBULxljI48-K4IAbDdX&ratebypass=yes'
[download] Destination:  ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.f251.webm
[download] 100% of 32.45MiB in 00:08
[ffmpeg] Merging formats into " ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.mkv"
[debug] ffmpeg command line: ffmpeg -y -i 'file: ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.f136.mp4' -i 'file: ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.f251.webm' -c copy -map 0:v:0 -map 1:a:0 'file: ASMR Ear (Cleaning) Groomers & Spa Roleplay  (unisex)-DYGc-31mEec.temp.mkv'

real	1m28.506s
user	0m21.053s
sys	0m14.987s

Works fine for me with vlc 2.2.0 and mplayer. So it's probably something wrong with either your ffmpeg or vlc.

Thanks for looking into this.
I can confirm it works with mplayer.
Sorry for the trouble.
$ mplayer audio.webm
Creating config file: /home/zazdxscf/.mplayer/config
MPlayer SVN-r37440 (Gentoo)-5.2.0 (C) 2000-2015 MPlayer Team

Playing audio.webm.
libavformat version 56.40.101 (external)
libavformat file format detected.
[lavf] stream 0: audio (opus), -aid 0, -alang eng
Clip info:
 encoder: google
Load subtitles in ./
==========================================================================
Opening audio decoder: [ffmpeg] FFmpeg/libavcodec audio decoders
libavcodec version 56.60.100 (external)
AUDIO: 48000 Hz, 2 ch, floatle, 0.0 kbit/0.00% (ratio: 0->384000)
Selected audio codec: [ffopus] afm: ffmpeg (FFmpeg opus)
==========================================================================
AO: [alsa] 48000Hz 2ch floatle (4 bytes per sample)
Video: no video
Starting playback...
A:  26.6 (26.6) of 2028.4 (33:48.4)  1.6%

Exiting... (Quit)

Can you look in vlc Tools->Messages(Ctrl+M) in the Modules Tree tab and say whether the demux is "avformat" or "ogg" while the audio of the .webm is working for you? That's under libvlc->playlist->input->demux...
Looks like for .opus is demux "ogg" but for .webm is demux "avformat" and in the latter case Audio decoded/played remains 0 (for me). (in Tools->Media Information(Ctrl+i) Statistics tab)
whether I'm using vlc 2.2.0 or the git version.
With or without ffmpeg compiled in. I've no idea how to fix it currently. But clearly since with mplayer works, it must be something on my side.

This is for 251 webm:

Amazing! I didn't have --enable-mkv which is because gentoo has the matroska USE flag disabled by default, and now it works as it does for you, with demux "mkv" and audio works fine! It previously was demux "avformat" (with no audio being heard)
If any gentoo users stumble upon this no audio (no sound) issue on (some of) the .webm and .mkv files, just remember to USE=matroska when emerging vlc.
Thanks a lot for the help @dstftw ! Much appreciated!


theme upload errors out on layout/theme.liquid

Similar to https://github.com/Shopify/themekit/issues/78, theme upload layout/* works, but theme watch errors out when editing layout/theme.liquid. If I remove the offending lines causing syntax errors, a different error pops up. Uploading via admin also works as expected.
>> theme upload layout/*
Successfully performed Update operation for file layout/theme.liquid to carson-blank.myshopify.com
~/Documents/Projects/shopify-themes/Venture

>> theme watch
Spawning 2 workers for carson-blank.myshopify.com
carson-blank.myshopify.com Worker #0 ready to upload local changes
carson-blank.myshopify.com Worker #1 ready to upload local changes
Received Update event on layout/theme.liquid
Could not upload layout/theme.liquid:
	Liquid syntax error (line 477): 'if' tag was never closed

@csaunders

Is this causing the application to crash? It seems like getting the
feedback you are getting back from the attempted upload is correct.
When you make the appropriate fix, are you still getting errors?
Could you include replication steps similar to these guidelines?
https://developer.mozilla.org/en-US/docs/Mozilla/QA/Bug_writing_guidelines#str
?
On Wed, Sep 9, 2015 at 10:07 AM, Carson Shold<EMAIL_ADDRESS>wrote:

Similar to #78 https://github.com/Shopify/themekit/issues/78, theme
upload layout/* works, but theme watch errors out when editing
layout/theme.liquid. If I remove the offending lines causing syntax
errors, a different error pops up. Uploading via admin also works as
expected.

theme upload layout/*
Successfully performed Update operation for file layout/theme.liquid to carson-blank.myshopify.com
~/Documents/Projects/shopify-themes/Venture

theme watch
Spawning 2 workers for carson-blank.myshopify.comcarson-blank.myshopify.com Worker #0 ready to upload local changescarson-blank.myshopify.com Worker #1 ready to upload local changes
Received Update event on layout/theme.liquid
Could not upload layout/theme.liquid:
Liquid syntax error (line 477): 'if' tag was never closed

@csaunders https://github.com/csaunders
—
Reply to this email directly or view it on GitHub
https://github.com/Shopify/themekit/issues/79.

--
Chris Saunders // Developer @ Shopify

App does not crash, just shows the error. The error listed isn't actually an error, so there's nothing to fix. Strangest part, randomly removing a line and re-saving sometimes uploads it properly during theme watch, and other times says there's an error. Gif might help explain a bit:

I can reproduce by following these steps:

Run theme watch
Remove a line of code and save. Seems to be any line, liquid or not

Expected result:
Upload file (assuming there are no liquid errors)
Actual result:
Sometimes uploads the file, sometimes says there is a liquid error

++ I also run into this issue.  Haven't managed to catch it with mitmproxy though :-(

I just started developing theme for shopify based on Timber and I encountered this issue as well . For me its happening only if I use themekit on Ubuntu. It works fine without any issues on Windows machine.
So themekit for linux needs to be fixed.

I'm constantly having this issue as well. It seems to be that the files are only being partially uploaded to Shopify. When I see an error in SCSS files it's most commonly because the uploaded file is missing a significant chunk. I can confirm this by opening the online file editor and viewing the SCSS file. The same seems to be happening with liquid template files. The errors are thrown because the server only got a piece of the liquid file. Would sure like to get a fix on this one! It happens all the time to me.
Here's a fine example that just now occurred. With 'theme watch' running I saved a theme.scss.liquid file and got the "Successfully performed upload" response. The file locally  is 4499 lines long. If I view that file with the online editor it is only 593 lines and ends with:
/*================ Responsive Text Alignment Helper ================*/
@mixin responsive-text-align-helper($grid-breakpoint-type:"") {
  .#{$grid-breakpoint-type}text-left {
    text-align:

That'll break it..


Add command to generate README on roles argspec

We add a new subcommand to generate README files for roles based on argspec definition.

I used it for the latest release of the dalibo.essential collection
This is functionnal