Add wide angle camera sensor

🎉 New feature
depends on:

#168
#172
https://github.com/ignitionrobotics/ign-rendering/pull/490
https://github.com/ignitionrobotics/sdformat/pull/744

Summary
Added a wide angle camera sensor. The bulk of the work for generating wide angle camera data is done in  https://github.com/ignitionrobotics/ign-rendering/pull/490. This PR adds logic to package the image data and publish them through ign-transport and event callbacks.
Test it
added a wide_angle_camera integration test. Run the test with ogre render engine from your colcon workspace:
 RENDER_ENGINE_VALUES=ogre ./build/ignition-sensors7/bin/INTEGRATION_wide_angle_camera

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Sensor is working as expected and the tests passes for me!


Keyboard shortcut for saving resource

What would you like to be added:
Add a keyboard shortcut for saving resource manifest (Ctrl + S)

Environment you are Lens application on:

Kubernetes distribution: any
Desktop OS: any

Thanks for the suggestion. We will consider this in the future.

Closing in favour of https://github.com/lensapp/lens/issues/7583 as one unified request


OpenAI chain crashes due to missing content key

To replicate:
Make hundreds of simultaneous calls to AzureAI using gpt-3.5-turbo. I was using about 60 requests per minute.
About once every 3 minute you get a response that is empty that has no content key. There is an easy fix for this. I pushed a PR that solves the problem: https://github.com/hwchase17/langchain/pull/4139

What error do you get when it crashes?


New package: Adobe.Acrobat.Reader.ALSD version 23.008.20421

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
Resolve #166451

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Both URL are proved valid.

URL: https://ardownload3.adobe.com/pub/adobe/acrobat/win/AcrobatDC/2300820421/AcroRdrALSDx64_2300820421_all_DC.msi
Status Code: 200
(Automated message - build 887)

/azp run

[!NOTE]
Closed because https://github.com/microsoft/winget-pkgs/labels/URL-Validation-Error https://github.com/microsoft/winget-pkgs/labels/Validation-SmartScreen-Error


Moves MemoryManager to global scope

For multi-threading support. This also fixes #288.

@VocalFan can you test if this fixed your issue?

I'll test when I get back from work

++++++++++++++++++ W [00:00:00:01:247]:0x0000000000021088:kernel\src\syscalls\mod.rs:68
Syscall 608 failed: invalid argument.
++++++++++++++++++ P [00:00:00:01:247]:0x0000000000021088:kernel\src\syscalls\mod.rs:83
not yet implemented: int 0x44 at at 0x000000000002b13a on /system/common/lib/libkernel.sprx

Yep! Now reaches 608.


Is decomposition of layers possible?

Hi
If I have a pdf, about which I know it has just one page, contains a set of layers, and there is color data associated with each layer, would it be possible to extract those layers and, for example, save them to disc as tiff or jpg files? Or to database as some sort of text file, which I could then read for further processing?

It would be technically possible, but pdf-reader doesn't provide anything out of the box that meets your needs.
You'd have to write a custom receiver class that processes each page (see PDF::Reader::Page#walk), renders the bits you care about and writes them to your disk or database.


Binder error

I think that there is an error in the JupyterLab MyBinder session since we moved the files out of the root:

Possible solutions I can think of are:

have all the files in the root in a new branch, which would be edited here:

https://github.com/courtois-neuromod/anat-processing-book/blob/b2b2046823ec75a17c697252e27adf4154d3d558/_config.yml#L33

have them in the root of main like they were

ooops! 😬  Sorry.
Is that something that you can change on your Binder config?  For example, when I compile locally with the sources moved, I just point to the source folder:
jupyter-book build source/

Also, what do you use Binder for?
I changed it because I found it cleaner to enclose the source of the book at one location (ie: separate from the jupyter book config files). Other projects seem to also do it:

https://github.com/executablebooks/quantecon-example
https://github.com/YannDubs/Neural-Process-Family
https://github.com/INRIA/scikit-learn-mooc

No worries, we use Binder for reproducing the plots from the notebooks. We don't have to use it, we can use Google Colab instead. I have both linked 😄

I fixed the error when running the notebook on Binder. I added the path of the book inside the config!
https://github.com/courtois-neuromod/anat-processing-book/blob/150eea141e7cb34d07059b6e6c10332127770dae/source/_config.yml#L49
I will close this issue.


changed three parameters: sample_rate win_length and hop_length

brq = BestRQ(
sample_rate = 22050,
win_length = 1024，
hop_length = 256,
hi, I changed three parameters, as above,  then the program may have crashed. so  these three parameters can not be changed?

The sample rate parameter should match the sample rate of your source audio — you'll need to resample your audio manually if it doesn't match what you specify when setting up the network. You should be able to modify the window length and hop length (aka stride) to meet your needs, though!

My training samples are all audio files with a sampling rate of 22050HZ, which are taken from LJSpeech-1.1, which are used on many GAN, where the  win_length is 1024 and the hop_length is 256. So i changed the sample_rate the win_length and the hop length. but your program  is not running, and I don't know how to modify it.


Remove pagination because it's completely broken

Navigating result pages does not work at all. Get rid of the pagination, use the Steam website's builtin pagination, and just have the extension display the float values.

I added some visual feedback about page loading on my recent PR #251. This indication works and I now do understand when steam is bitching about too much requests


hotfix: change devpool repo name

Resolves https://github.com/ubiquity/devpool-directory-bounties/issues/10

@pavlovcik related to https://github.com/ubiquity/devpool-directory-bounties/issues/10

@pavlovcik when does it reflect on the UbiquiBot... is it while its on development or main because it still does not work

main goes on the production bot but those usually are rolled out 1-2 times a month.

main goes on the production bot but those usually are rolled out 1-2 times a month.

So the change i made won't actually reflect until it is rolled out, I tried to install a new app just now and it seems its still getting the repo name wrong

rfc @rndquu @0xcodercrane

@devpanther I don't understand how the current PR solves the original issue. This "create PR" script will work both for devpool and devpool-directory because https://github.com/ubiquity/devpool redirects https://github.com/ubiquity/devpool-directory.
This "unexpected" issue was created because the sync workflow run for a closed PR if it was not merged. So we should refactor the sync workflow to run only on push to the development branch.

@devpanther I don't understand how the current PR solves the original issue

It doesn't run anymore and I don't think it's redirecting.. Because new installations don't trigger anymore so this is suppose to help that

And then I'd be able to install on my organization and figure out when the workflow runs.. But like you said it runs on PR closed so that's that issue there
But this repo name still needs to be changed


security: update to containerd 1.4.13-azure-2

What type of PR is this?
/kind bug

What this PR does / why we need it:
Updating containerd to 1.4.13+azure-2, from 1.4.12+azure-3, with fix for recent CVE-2022-24769 (https://nvd.nist.gov/vuln/detail/CVE-2022-24769).
Which issue(s) this PR fixes:
CVE-2022-24769 (https://nvd.nist.gov/vuln/detail/CVE-2022-24769)

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
This update to containerd 1.4.13-2 contains containerd CVE fix for CVE-2022-24769, for kubernetes versions < 1.22 (which used containerd 1.4.12 previously).
none

re-ran the gpu test, hoping that was a flake

re-ran the gpu test, hoping that was a flake

Thanks. The test failed again and this time, it returned this [error]Artifact vanilla-gpu-logs already exists for build 53768549.


Run mne sys_info in post-install script?

The first run of mne sys_info (or, actually, just the first import of MNE and some of its dependencies) seem to take a long time on my machine (macOS). Subsequent runs are usually much faster. This seems to have to do with some library caching / checking / integrity analysis or whatever macOS is performing.
I think if we were to execute mne sys_info as part of the installer (very last step), we could probably speed up the first start of the freshly-installed package, hence improve the user experience.
WDYT?

We have had issues in the past with this crashing, which makes me a little bit wary of doing it. But if we can run it and ignore the return value (e.g., getting a bad one from a segfault) then I agree it would be nice!

You mean the crashes due to issues with 3D / PyVista?

In bash you can set +e then run the command then set -e, but it's easier just to command || true

Fixed by #133


Change the Logo / title / subtitle

Goal
Update your batch's mainpage.

Create a random logo (or use Dall-E!) to generate a batch logo
title: Batch #{batch_number}
subtitle: Whatever you feel appropriate :)

I just created a logo with AI . Do you like it guys?

It looks good👍🏻

Looks good I think, maybe you should create a PR for it @0xzeze and let the maintainer know

Very nice logo! @0xzeze go ahead and create a PR, so that it can be merged


[DOCS] : Improper readme - broken links and images

✏ Description
there are a lot of mistakes in the readme , some links are not working , image is not proper and the teck stack is also wrong
📸 Screenshots
No response

i would like to work on this

go for it


Add 'x-li-format: json' header when creating a share

#1

Coverage remained the same at 100.0% when pulling 80d4f37300fa8f92d8f7cfd2b0ee0efa065dcb87 on smartshowltd:add_header into ceb63460b8b45b7e00766ae3a4c3e361d76c6ffc on emorikawa:master.

Coverage remained the same at 100.0% when pulling 80d4f37300fa8f92d8f7cfd2b0ee0efa065dcb87 on smartshowltd:add_header into ceb63460b8b45b7e00766ae3a4c3e361d76c6ffc on emorikawa:master.


Easy installation for cli

Having issues with automatic installation on CI/CD platform.
I can download the zip file via curl but am unable to successful extract it End-of-central-directory signature not found.
So i had to download it through the UI and check in the executable in order to get it successfully installed on CI/CD platform.

should be something like this
         name: Release
         command: |
           curl -L -o fargate.zip https://github.com/jpignata/fargate/releases/download/v0.2.3/fargate-0.2.3-linux-amd64.zip
           unzip fargate.zip && sudo mv fargate /usr/local/bin```

Related: may be a dumb question. @jpignata is there an automated build for this project?  In other words, where are the released binaries being built?  I didn't see any deploy step here.

Yeah - sorry - I killed it as I thought it served its purpose. Thanks @ksteiny!
@jritsema Nope - not currently. The last few releases I shipped manually. Automated releases are on my growing list :smile:


error when changed the num_classes

Hello I am facing following error when I tried to train the model using custom dataset that has 5 classes. I know the error below would majorly occur because of difference in num_classes, but not sure what is the effective rectification for this:
Traceback (most recent call last):
File "main.py", line 326, in
main(args)
File "main.py", line 275, in main
train_stats = train_one_epoch(
File "/home/vsrikar/engine.py", line 43, in train_one_epoch
loss_dict = criterion(outputs, targets)
File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/home/vsrikar/models/deformable_detr.py", line 342, in forward
indices = self.matcher(outputs_without_aux, targets)
File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, kwargs)
File "/home/vsrikar/models/matcher.py", line 87, in forward
cost_giou = -generalized_box_iou(box_cxcywh_to_xyxy(out_bbox),
File "/home/vsrikar/util/box_ops.py", line 59, in generalized_box_iou
assert (boxes1[:, 2:] >= boxes1[:, :2]).all()
RuntimeError: CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
terminate called after throwing an instance of 'c10::CUDAError'
what():  CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Exception raised from create_event_internal at ../c10/cuda/CUDACachingAllocator.cpp:1387 (most recent call first):
frame #0: c10::Error::Error(c10::SourceLocation, std::string) + 0x3e (0x7f89f1df01ee in /usr/local/lib/python3.8/dist-packages/torch/lib/libc10.so)
frame #1:  + 0x26e61 (0x7f89f1e6ae61 in /usr/local/lib/python3.8/dist-packages/torch/lib/libc10_cuda.so)
frame #2: c10::cuda::CUDACachingAllocator::raw_delete(void) + 0x257 (0x7f89f1e6fdb7 in /usr/local/lib/python3.8/dist-packages/torch/lib/libc10_cuda.so)
frame #3:  + 0x466858 (0x7f89f641c858 in /usr/local/lib/python3.8/dist-packages/torch/lib/libtorch_python.so)
frame #4: c10::TensorImpl::release_resources() + 0x175 (0x7f89f1dd77a5 in /usr/local/lib/python3.8/dist-packages/torch/lib/libc10.so)
frame #5:  + 0x362735 (0x7f89f6318735 in /usr/local/lib/python3.8/dist-packages/torch/lib/libtorch_python.so)
frame #6:  + 0x67c6c8 (0x7f89f66326c8 in /usr/local/lib/python3.8/dist-packages/torch/lib/libtorch_python.so)
frame #7: THPVariable_subclass_dealloc(_object) + 0x2d5 (0x7f89f6632a95 in /usr/local/lib/python3.8/dist-packages/torch/lib/libtorch_python.so)
frame #8: python() [0x5d1908]
frame #9: python() [0x5a978d]
frame #10: python() [0x5ecd90]
frame #11: python() [0x5447b8]
frame #12: python() [0x54480a]
frame #13: python() [0x54480a]

frame #19: __libc_start_main + 0xf3 (0x7f89fa857083 in /usr/lib/x86_64-linux-gnu/libc.so.6)
./configs/r50_deformable_detr.sh: line 10:   997 Aborted                 (core dumped) python -u main.py --output_dir ${EXP_DIR} ${PY_ARGS}
Traceback (most recent call last):
File "./tools/launch.py", line 192, in
main()
File "./tools/launch.py", line 187, in main
raise subprocess.CalledProcessError(returncode=process.returncode,
subprocess.CalledProcessError: Command '['./configs/r50_deformable_detr.sh']' returned non-zero exit status 134.
I changed the following code snippet in deformable_detr.py:
def build(args):
num_classes = 5 if args.dataset_file != 'coco' else 91
when I change the num_classes back to 20, it is working out fine. Please suggest how to handle this issue.

It's a problem from coco in pycocotools.
coco class is assigned a class ID from 1 to N.
In the case of deformable DETR, it is a problem because it allocates from 0 to N-1.

You can try adding "tgt_ids = torch.sub(tgt_ids, 1, alpha=1, out=None)" in the matcher.py.


update MentionUserRenderer.tsx display text

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/38079
PROPOSAL: https://github.com/Expensify/App/issues/38079#issuecomment-1990120462
Tests

Open any room chat.
Mention someone not member in this room with public domain (ie: @gmail.com) and click send.
Wait for "actionable whisper" message Heads up, `@mentionDisplayText` isn't a member of this room.
Verify that @mentionDisplayText display @email not @display name.
Logout then login via email with private domain (ie: @privateDomain.com) , and repeat steps 1, 2.
Mention someone not member in this room and have email with the same private domain you used in login and click send.
Verify that @mentionDisplayText display @username not<EMAIL_ADDRESS>

[x] Verify that no errors appear in the JS console

Offline tests

N/A.
QA Steps

Same as Tests step.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/41129870/c485a26e-f8fb-4fee-bd8c-a5dfe82d6754
https://github.com/Expensify/App/assets/41129870/ee63eb0a-148c-46a8-bb8b-0aebb92e9a6a

MacOS: Desktop
https://github.com/Expensify/App/assets/41129870/046b5216-c756-4eb4-ae47-ae5fb0d316ac
https://github.com/Expensify/App/assets/41129870/016f7cda-9c5b-45bc-80af-f1d4604a45a6

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Tests were passing when I merged and are still passing now.

🚀 Deployed to staging by https://github.com/puneetlath in version: 1.4.59-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@ahmedGaber93 @puneetlath @sobitneupane I believe this PR caused this blocker https://github.com/Expensify/App/issues/39426

I believe this PR caused this blocker https://github.com/Expensify/App/issues/39426

Replied in issue page https://github.com/Expensify/App/issues/39426#issuecomment-2032868846

🚀 Deployed to production by https://github.com/Julesssss in version: 1.4.60-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Client Allocation Request for: Filecoin Discover

Client Allocation Questions
Core Information

Name: Filecoin Discover
Website / Social Media: https://store.filecoin-discover.com/
Region: North America
DataCap Requested: 10TiB
Addresses to be Notarized: f3vi5y7ykd4rlcjfhd6eyjuuvhqn2g52cx7elufvv4xmkfyjbo5epfurf3443ww3jaghy4po2yfjsl53xaysvq
Notary Requested: ann-shin

Hi, nice to meet you. Thanks for your request. Can you kindly add more information here.
1.Are you representing for yourself or your company？
2.If you are representing for your company, Please give the following informations, duration of etablishment, number of employees, business scope, users volume, number of social media followers, data size to be stored, products or projects, and so on. If you are representing for yourself , please introduce yourself,such as number of social media followers, filecoin activities partipation with links, data size to be stored. Provide links to social media/news mentions.
3.The history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time, etc.
Type, size, and source of the data you will be store on Filecoin,e.g. public data or private data, etc. Is there any sensitive or illegal content?
4.How do you intend to store the data on Filecoin, e.g. redundancy, location/miner preferences, etc
5.Have you already apply for DataCap to another notary, and whether it is accepted or rejected. Is there any reliable DataCap left?
6.Would you like to post real transactions on Github?

@ribasushi
Hi, I haven't received your reply for a long time.
The issue will be closed in two days.
If you want to continue, please submit your information as soon as possible.
Looking forward to your reply.


I've already import jszip. I've working this before changing to devextreme-angular 2 and v:16.1

Its a 16.1 breaking Change:
[HTML]

to:
[HTML]

That solves the problem.


heroku buildpack fails, but commits gets accepted

Sometimes my Heroku PHP Buildpack fails, but deis accept the commit, so I have to do an empty commit to trigger a rebuild. A rebuild function would be greate as mentioned in other issues, but should'nt the commit be rejected? My main problem is that it seems that the app is up to date in the git tree, but in reality an old version is running online because the commit never deployed.

+1 This annoyed me the last few days as well when compiling failed because some file could not be downloaded to compile my buildpack...
Although obviously if a compile fails (for any reason, it could be the apps fault) and the commit would not get accepted then you are possibly in for a large upload again...

Yes, the commit should get rejected. I'll cook up a test buildpack and see what I can come up with.

same behavior on nodejs and docker builds

I encounter this problem too using docker builds and dpl...

I can confirm this with a custom heroku buildpack:
><> deis create fail
Creating Application... done, created fail
Git remote deis added
remote available at ssh://git@deis.<IP_ADDRESS>.xip.io:2222/fail.git
><> deis config:set BUILDPACK_URL=https://github.com/bacongobbler/heroku-buildpack-epic-fail
Creating config... done

=== fail Config
BUILDPACK_URL      https://github.com/bacongobbler/heroku-buildpack-epic-fail
><> git push deis master
Counting objects: 80, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (45/45), done.
Writing objects: 100% (80/80), 19.02 KiB | 0 bytes/s, done.
Total 80 (delta 30), reused 75 (delta 30)
-----> fail app detected
-----> failing buildpack with exit code 1...
To ssh://git@deis.<IP_ADDRESS>.xip.io:2222/fail.git
 * [new branch]      master -> master

Source available at https://github.com/bacongobbler/heroku-buildpack-epic-fail


MOD 10,30 & 40 Not match

Hi Team,
May You check again the MOD 10,MOD 30 and MOD 40?
Here with the issue I found.

MOD 10 Migrating Web Applications to Azure - currently link to MOD10 - Options for Building and Running Your App in the Cloud ?

MOD30: Modernizing Your Application with Containers - currently link to MOD30 - Enhancing Web Applications with Cloud Intelligence ?

MOD40: Consolidating Infrastructure with Azure Kubernetes Service - currently link to MOD40 - Debugging and interacting with production applications ?

Got it. Thanks @cheahengsoon I'll fix that.


422 error on delete/write

This is probably user error but I'm getting a 422 (cannot completely) error when trying to delete a file or write to existing file. I'm sure I'm using it wrong.

@lucusp Hi, could you please provide a JSFiddle approximating your code or at least explain what you're doing a bit better? You've provided effectively zero information... :sweat_smile: Thanks!

Ok I will as soon as I can :) sorry for the vague info.

Ok, so my issue is this (using jQuery):
     $.ajax({

		url: githubjs

	}).success(function(data){

		var content = data;

                    repo.write('master', _path, content, 'my commit message', function(err) {

                            $('#tmpdump').text(JSON.stringify(err));

                    });
     })

Looks like it's successful on my page but in the console I get this:
https://api.github.com/repos/lucusp/test/contents/dir/pop.html?1433031143713 Failed to load resource: the server responded with a status of 422 (Unprocessable Entity)
Hopefully this is more clear? It only does this when writing to an existing file in a directory.

Gonna close this issue due to #78, #85, #108 all being similar more practical examples of my issue(s).

@lucusp Cool, thanks! Will look into those issues next chance I get!


RBF Custom Layer in Tensorflow can not learn the centers (mu) parameters

Using Google Colab with TF version 2.5

Bug.zip

from tensorflow.keras.layers import Layer
from tensorflow.keras import backend as K
class RBFLayer(Layer):
def init(self, units, gamma, **kwargs):
super(RBFLayer, self).init(**kwargs)
self.units = units
self.gamma = K.cast_to_floatx(gamma)
def build(self, input_shape):
    self.mu = self.add_weight(name='mu',
                              shape=(int(input_shape[1]), self.units),
                              initializer=tf.random_normal_initializer(),
                              trainable=True)
    # self.gamma = self.add_weight(name='gamma',
    #                           shape=(int(input_shape[1]),),
    #                           initializer=tf.constant_initializer(1),
    #                           trainable=True)
    super(RBFLayer, self).build(input_shape)

def call(self, inputs):
    diff = K.expand_dims(inputs) - self.mu
    l2 = K.sum(K.pow(diff,2), axis=1)
    res = K.exp(-0.5*self.gamma * l2)
    return res

def compute_output_shape(self, input_shape):
    return (input_shape[0], self.units)

Was able to reproduce the issue in TF 2.5. Please find the gist here. Thanks!

@miladtoutounchian
Please post this issue on keras-team/keras repo.
To know more refer to:
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999


sc-288 Mock SendGrid for email testing

This creates a mock for SendGrid which does some email message validation, writes to an in-memory store, and returns a REST response. The way it's intended to be used is to call emails.NewMock() instead of emails.New(), which creates a mocked EmailManager which contains a mocked SendGrid client that contains a running list of sent emails that tests can verify against.
One thing that could be added is writing the email data out to disk on Send, although I wasn't sure if that actually makes the integration tests easier.

@bbengfort I decided to create a test suite for the send email tests following your suggestion, and made other modifications based on your comments.


Machine batch for images

afaik machine_batch works only with machine resource. Any plans to add it for machine_image? What is required to do so?

It's a pretty important one, any info on this question? @tyler-ball

Theoretically, it should be possible - machine_batch just spins multiple threads to create the machines in parallel.  I don't know when we will prioritize this though

@tyler-ball if you give me some directions I can work on it

@tyler-ball,  I am also on to the task that @Fodoj referred. This is very import to us as the time it takes to provisioning machine images serially is very time consuming. Would you be able to give me some pointers how to code for this?


Palettes

Found source code for creating custom palettes on codeplex.
paletteselectors.zip

And another...
KryptonPalComposer.zip

Isn't it already in the Krypton Suite ... https://github.com/Wagnerp/Krypton-NET-4.7/tree/master/Source/Palette Designer

Ahh yes.

Bingo! I've finally worked out how to change colours! Any suggestions?


"go to search page" function is missing or not supported in the new Metrics Explorer for application insights.

This question is specially for application insights.
For the classic metrics explorer in application insights, if click the bar on the chart we add, then it will redirect us to the search page. Screenshots as blow:

But in the new metrics explorer in application insights, after clicking the bar on the chart we add, it does not redirect us to the search page:

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

ID: ea95f9c8-e0ad-93d2-cac2-0b486a684600
Version Independent ID: e03365fb-056d-1388-1cc3-01c009b1d2ec
Content: Azure Monitor metrics explorer
Content Source: articles/azure-monitor/platform/metrics-charts.md
Service: azure-monitor
GitHub Login: @vgorbenko
Microsoft Alias: vitalyg

@imeya Thanks for the Comment. We are actively investigating and will get back to you soon.

@vgorbenko any insights on this please ?

For the purposes of crosslinking, I recently asked this on on Stack Overflow, but currently no answers.
This is the only reason I'm still using the classic Metrics explorer.

@cmeeren & @imeya I have checked with author @vgorbenko internally and got the below update from him.
"This is a known limitation of the new metrics explorer. Unfortunately, I don’t have an ETA when this will be fixed. For now, the user can open Analytics and run the query to get the data he or she needs".
Hope this helps!!

Thanks, but what is meant by "the query"? Using the new Metrics explorer, I can't see any way of getting to the underlying query in Analytics.

@cmeeren It means that at this point you would need to write a query.
Just FYI, @vgorbenko is working on an article that lists all standard metrics with the underlying queries. It should be finished and released very soon.

@cmeeren & @imeya Now we have an article that is live at https://docs.microsoft.com/azure/azure-monitor/platform/app-insights-metrics which explains what queries are running for all standard appinsights metrics.
Regarding the primary question in this thread, you can work with standard metrics in metrics explorer. For advanced cases, you can use log analytics, copy the query from the article, and begin customizing it as needed. There is a plan to implement direct navigation from Metrics Explorer to Log Analytics where the query is carried over (meaning, once it is implemented then you need not refer the doc) in the future but we don’t have an ETA for this yet.
At this point it appears that there's no edits or updates needed for the documentation. Closing this issue for now. If there are further questions regarding this matter or feedback on the documentation, please do let us know and we will gladly continue the discussion.


Get the REAL token addresses for dynamic sale

We need someone to find and verify the addresses for ceur in alfajores and mainnet and write them down here.

[x] get alfajores ceur address`
[x] get mainnet ceur address
[ ] topup developers addresses with test ceur

[x] Artur 0x661Ac71bbe43fe56935c1CA4d62e62ed380950A3
[x] Julien 0x346314781c4D1483bE27fAEA9d698074f7cBa1Be
[ ] Sam

Just to confirm @arturictus We're not using these addresses for mainnet launch.
Also not sure what you want to test on Alfajores with cEur but the faucet is limited to 5 cEUR each time


ARROW-16638: [Go][Parquet] Fix skipping large number of rows in boolean columns

Boolean columns still have an issue when skipping a large number of rows, due to the buffer passed for values not being large enough. If batch size to skip is 1024, the buffer allocated for values when skipping a boolean column is only 128, resulting in an index out of bounds panic.
This PR assumes the buffer passed for values when skipping a boolean column contains 1 byte per 8 bool values, and explicitly creates a bool slice with len(buf)*8 capacity.

@zeroshade note that this PR allocates additional memory to skip, but I don't see a much better way to patch given how the generic column reader allocates scratch space and there not being a good way to convert a []byte buffer into []bool where 8 bool values map to each byte?

@mdepero In the parquet/internal/utils package there is a function BytesToBools which explicitly is an efficient conversion from bitpacked bytes to a []bool. It assumes that the slices are already sized appropriately (len(out) should equal len(in)*8).
That being said, there might be a way we can get around this by changing the implementation slightly.
Since the columnChunkReader is embedded in the typed readers, we don't "technically" need to do cr.columnChunkReader.skipValues(... and could instead do cr.skipValues(.... I only specified the cr.columnChunkReader portion to make it explicit. The benefit there if we convert it to being just cr.skipValues is that we can then override the skipValues function for the BooleanColumnChunkReader to allocate the correct amount of scratch space, it does result in some duplication of code but I think it's a better solution to avoid the extra allocation where possible. As another, forward looking idea, I'd probably want to have the scratch space use a pool of buffers rather than allocating a new scratch space for every skip but that can be done as a later change. Anyways, did that all make sense as something you could do?
Let me know if you have any questions. Thanks again for this!

@zeroshade My understanding of BytesToBools would still require separately allocated memory for bytes and bools, and it just copies the values from byte encoding to []bool's?
Assuming make([]bool, n) only allocates 1 byte per 8 bool values, is it better to do two allocations each of size (n/8) bytes or one allocation of size (n) bytes? I've pushed a new version that retains the scratch space and only allocates once, but that one allocation is now n bytes since the cast from []byte to []bool creates 1 bool per 1 bytes. I do agree it's a bit hacky to simply ignore the buf passed into the skipValues readFn, which approach do you recommend?

@zeroshade SGTM, referenced a new Jira issue in the TODO. Thanks!

@mdepero Perfect! once the Travis builds finish and succeed i'll merge this!
Thanks again!


Fix colorization bug and add regression tests

Fixes https://jira.oncprojectracking.org/browse/BONNIE-168

Adds rake task allowing colorization code to be regression tested

Looks good other than changing the failure output to be more helpful to the user.


Copy/paste of nodes stopped working.

It has taken me a while to track it down, but it's the rgthree nodes that are causing the problem.
Until today I was able to select nodes(s), copy them (ctrl+c) and paste them (ctrl+v) in the same workflow, and even to another tab. I used it yesterday without any problem, I update every day, including this morning. Now, if I try to copy/paste, I can only paste the last node I copied yesterday, even if I right click a node and copy it to clipspace I get the same behaviour. I have tried in private browsing and another browser, but nothing is pasted. I can copy/paste text within nodes, but not the nodes themselves.
Originally opened at: https://github.com/comfyanonymous/ComfyUI/issues/3995

I've resolved this by deleting the rgthree custom node folder and then reinstalling it via the manager. There must be some lingering old code in there that caused it.


Fixes #9185

Summary
Clarify that ASP.NET Core MVC is supported in .NET Core
Fixes #9185

@Rick-Anderson @scottaddie can one of you review this PR? Thx!


Add accountType and employeeId for corpUser entity and ldap ingestion

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

One alternative to adding them as strongly-typed fields across the entire stack (as part of CorpUser models) is to
instead introduce a key-value map on the CorpUserInfo.pdl aspect (see customProperties on DatasetProperties.pdl for > an example) and then populate these properties with any arbitrary information that you may need.
We need to populate this during an ldap ingestion and the current implementation of ldap.py populates only CorpUserInfo, it doesn't populate DatasetProperties. So, essentually an MCE which is generated at ldap.py has no information about anything beyond what was populated into CorpUserInfo.

Ping on this one. I think the team is hesitant to continue to grow the set of typed properties on this model without clear value add on each.

Ping on this one. I think the team is hesitant to continue to grow the set of typed properties on this model without clear value add on each.

@jjoyce0510, I originally discussed this on slack (Dmitry Bryazgin, ingestion topic), but I can provide some details here. A corporate account at enterprise is not always reference to a particular employee, it could be a so called a "service" accounts. Service accounts are usually responsible to run = all kind of scheduled jobs. In fact, ~ 20% of all ldap accounts at our enterprise are "service" accounts. We need a way to distinguish "service" accounts from all others at datahub because they follow different business rules. This is why we wanted to add a new field called "accountType" to the corpUser entity. However, it is clear now that "accountType" is not a standart ldap attribute and nobody else will benefit of this even if they want to have a similar functionality in place. Thus I found a standart ldap field called "employeeType" which could be used as a default mapping. In our case I will remap this to the ldap field that carry the real information we need, all other can have a default value being populated. The same approach could go to employeeId which could be mapped to a standart ldap attribute  employeeID.
If this approach doesn't work, we can go with proposed solution if custom fields are searchable (I have not verified this yet). I have already extended CorpUserInfo by customProperties and added code to populate this custom properties at ldap.py. I was need to add an extra property to ldap ingestion configuration to list all my custom properties, in this case something like this: custom_prop_list: ["employeeId","accountType"]. I need this property because I don't want to hardcode anything specific at ldap.py and to continue to keep it general. If I have a property, I can easily read all custom fields from there and populate MCE correctly.
The next thing left is "Wire up the customProperties map all the way to GraphQL layer". This is where I will need to investigate more because I am not quite clear how to make this all dynamic (if this even possible and needed).  Please, advise if I miss something.

I am closing this PR to create a new one where the above functionality will be implemented through custom properties.


omnibus comment changes

Fixed a few typos I came across.
Updated/moved some comments.

Thanks for this @EvaJanouskova  -- I'll leave this open a little longer in case other small things need to be changed.

@tbhallett, The description at the top of hsi_generic_first_appts.py seems outdated, doesn't it?

Yes it is a bit, you’re right. It should explain that those appointments can run at different levels.

thanks for all these @EvaJanouskova --- is it ready to go in?

thanks for all these @EvaJanouskova --- is it ready to go in?

There will always be smt, but it is all I have right now, so I think it can go in.


addressing issues with docs and readme

Address #13977

@seankane-msft  - please help to fix the following problem:
sample_query_tables_async.py

Suggestion:

Update to:

print("Listing tables:")
async for table in table_service.list_tables():
    print("\t{}".format(table.table_name))

Update to:

print("Queried_tables")
async for table in table_service.query_tables(filter=name_filter, results_per_page=10):
    print("\t{}".format(table.table_name))

Add await table_service.close()
Add await ts.close()

@seankane-msft

sample_create_delete_table_async.py
Suggestion 1:  Add async with table_service_client: for funcion create_table , create_if_not_exists, and delete_table.
Suggestion 2:
Update the code marked in red in the figure below:

        table_service_client = TableServiceClient.from_connection_string(self.connection_string)
        async with table_service_client:
            table_item = await table_service_client.create_table_if_not_exists(table_name=self.table_name)
            print("Table name: {}".format(table_item.table_name))


Use github hosted runners for approval bot

Applicable spec: n/a
Overview
Use github hosted runners for approval bot workflow
Rationale
We currently have a low capacity for edge runners and this check is currently only waiting for available runners.
Workflow Changes

Checklist

[x] The contributing guide was applied
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

Just saw that we can squash and merge our PR's anyways as its not included in the required status checks, but still its requesting edge runner resources so this PR makes sense anyways.


proposal: add Shrinker interface to testing/quick

The ability to simplify failing test cases that are automatically generated by testing/quick has the potential to enhance its usefulness. Often, automatically generated test cases take a long time to debug, as they may be quite complex.
Consider a slice of automatically generated events of length 50. There exists some subsequence that triggers a defect. Without reducing this slice to something more manageable, it may be cumbersome for the user of testing/quick to debug the issue.
My more lengthy comments describing the usefulness of this type of behavior: https://github.com/golang/go/issues/9282#issuecomment-351394008
It is common for quickcheck-style testing libraries to support "shrinking" where a failing testcase may be automatically simplified before being surfaced to the user. It is possible to support this with the addition of a simple interface:
// A Shrinker can produce simplified values of its own type.
type Shrinker interface {
    // Shrink returns a slice of simplified instances of this value.
    Shrink() []reflect.Value
}

When running Check and CheckEqual, when a failing input is discovered, the library may attempt to use the rest of our configured MaxCount cycles trying to Shrink the arguments, for each argument that implements the Shrinker interface. It is possible to mix Shrinker and non-Shrinker arguments, and the library should attempt to Shrink every argument that implements Shrinker, up until the configured MaxCount has been exhausted, even when different arguments require different numbers of iterations to reach a local minima.
The following property fails when any argument of type events contains 3.
type events []int

func doesntContain3(i1, i2, i3 events) bool {
	for _, input := range []events{i1, i2, i3} {
		for _, event := range input {
			if event == 3 {
				return false
			}
		}
	}
	return true
}

The shrinker, which could be made to work as a fallback "default" Shrinker for any slice argument, returns a slice of events with individual elements dropped out:
func (e events) Shrink() (ret []reflect.Value) {
	if len(e) < 2 {
		return
	}
	for i := range e {
		var evs events
		for j := range e {
			if i == j {
				// drop out individual elements
				continue
			}
			evs = append(evs, e[j])
		}
		ret = append(ret, reflect.ValueOf(evs))
	}
	return
}

Let's give it some inputs that we expect it to fail:
var nGenerated = 0
var genCases = []events{
	[]int{0, 0, 3},
	[]int{0, 3, 0},
	[]int{3, 0, 0},
}

func (e events) Generate(r *rand.Rand, _ int) reflect.Value {
	v := genCases[nGenerated%len(genCases)]
	nGenerated += 1
	return reflect.ValueOf(v)
}

When running with the Shrinker interface implemented for events, testing/quick will now simplify each argument passed in before presenting the failure to the user for debugging:
quick_test.go:375: #3: failed on input quick.events{3}, quick.events{0}, quick.events{0}

And when commenting out Shrinker, the pre-Shrinker behavior persists:
quick_test.go:377: #1: failed on input quick.events{0, 0, 3}, quick.events{0, 3, 0}, quick.events{3, 0, 0}

There are several different search strategies that may be employed. Because we have a finite number of iterations to work with, it may be a good idea to try shrinking several arguments simultaneously, and then then become more fine-grained after we overshoot our simplified target. This can be made to be fairly complex, which is at-odds with the current testing/quick implementation, so there is a trade-off to be made between code simplicity and search thoroughness.
Some quickcheck implementations allow the configuration of a second iteration budget to consume while searching for a simplified failing set of arguments, which allows users to generally have a shorter test latency when failures are not detected, but the library is able to spend a bit more time trying to simplify arguments when a failure is detected, to make debugging simpler. I am in favor of adding a field to Config in testing/quick for this purpose, perhaps called MaxShrinkCount, and when left unset simply consuming iterations from MaxCount.
If this new Shrinker interface is not desirable, I would argue that the above shrinking behavior would still be quite useful to apply to failing slice arguments by default for simplifying the information presented to the user, easing the debugging process.

The testing/quick package is frozen and is not accepting new features.

I think that part is relevant :D. I didn't even know testing/quick existed. I love the assertion style personally but the name is a bit clunky anyway.

You may not have heard about it because it's often considered incomplete by people familiar with property testing libraries. The current implementation is good at testing things like codecs, but it breaks down when testing things like user interfaces, distributed systems, services, or generally anything that is nice to test by generating a sequence of events/requests/interactions to find defects. Shrinking is the missing piece to get there.
P.S. @tj in the off-chance you're jumping in to throw sand on work that may benefit a particular competitor, I'm not currently engaged with them.

I'm not sure what you mean by that. I just saw this proposal in my GH feed, checked out what "quick" was and saw that it's deprecated, just wanted to mention that.

testing/quick was frozen to prevent test breakage. I believe the addition of this interface dramatically improves the viability of this library, without causing any breakage. When we can generate and shrink sequences of events/interactions/requests/partitions, it becomes far easier to test complex high-level interactions in systems that engineers are often unequipped to build in a low-defect manner. Given Go's usage for infrastructure and services, which engineers are generally unable to test thoroughly on their laptops, I believe this could be awesome for the ecosystem.

testing/quick was frozen as part of #15557, with the intention of not developing it further. Quoting @bradfitz: "testing/quick can be forked and develop elsewhere on Github, perhaps with versioning."
These, and other, packages aren't frozen to prevent "test breakage", but because it has been decided that they shouldn't be developed further as part of the standard library.
In the case of testing/quick, several people have worked on far more powerful 3rd party solutions that are worth using.


Pages removed or moved

http://schemas.microsoft.com/Start/2014/TaskbarLayout and http://schemas.microsoft.com/Start give the following error:
The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.
Do you happen to know the new locations? Thank you in advance.

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

ID: 140f038c-e307-78f0-5165-309e0cf34ca5
Version Independent ID: b1011259-ee05-21a4-5308-aabc8b0dd74b
Content: Configure Windows 10 taskbar (Windows 10)
Content Source: windows/configuration/configure-windows-10-taskbar.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@nenonix : Is this related to the news you linked to me sometime earlier this year, regarding templates and a hijacked Microsoft site?

Hello, @Anton2020
The issue is under investigation and you will be notified with any changes on the way.
Thank you for providing feedback.


Json deserializing for html with only json throws exception

Hi there,
This is just a question actually. If this is not the correct way to submit a question please let me know.
We have many tests that asserting json response from the website. The json response is wrapped in < Pre > tag and it has always been like this.
However we notice that the tests break for version 0.9.9 (we used 0.9.7 and they worked there). This is because now in 0.9.9 document.Body.InnerHtml contains the < Pre > tag while on 0.9.7 this was not the case.
This seems like the correct behavior in my opinion but I want to confirm that it is. It's a bit hard to see what change in the AngleSharp code that modify the behavior. So questions are:

is this what we want?
if so could you point me to where the change for this?

cheers,
AK

Cannot make a statement with the given amount of information. Please post a MWE.


Access-Control-Allow-Origin missing only in firefox :(

Hi! i am getting this issue only in firefox . I don't have much knowledge just asking is the setup good enough , OPTIONS request is getting 404
Error:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://some.in/api/admin/test. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing). Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://some.in/api/admin/test. (Reason: CORS request did not succeed)
cors.php:
return [
    'paths' => ['api/*'],
    'allowed_methods' => ['*'],
    'allowed_origins' => ['*'],
    'allowed_origins_patterns' => [],
    'allowed_headers' => ['*'],
    'exposed_headers' => [],
    'max_age' => 0,
    'supports_credentials' => true,
];

kernel.php
protected $middleware = [
        \App\Http\Middleware\TrustProxies::class,
        \App\Http\Middleware\CheckForMaintenanceMode::class,
        \Illuminate\Foundation\Http\Middleware\ValidatePostSize::class,
        \App\Http\Middleware\TrimStrings::class,
        \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
        //\App\Http\Middleware\Cors::class,
        \Fruitcake\Cors\HandleCors::class,

    ];

    protected $middlewareGroups = [
        'web' => [
            \App\Http\Middleware\EncryptCookies::class,
            \Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
            \Illuminate\Session\Middleware\StartSession::class,
           // \Illuminate\Session\Middleware\AuthenticateSession::class,
            \Illuminate\View\Middleware\ShareErrorsFromSession::class,
            \App\Http\Middleware\VerifyCsrfToken::class,
            \Illuminate\Routing\Middleware\SubstituteBindings::class,
        ],

        'api' => [
            'throttle:60,1',
            \Illuminate\Routing\Middleware\SubstituteBindings::class,
        ],

    ];

    protected $routeMiddleware = [
        'auth' => \App\Http\Middleware\Authenticate::class,
        'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
        'bindings' => \Illuminate\Routing\Middleware\SubstituteBindings::class,
        'cache.headers' => \Illuminate\Http\Middleware\SetCacheHeaders::class,
        'can' => \Illuminate\Auth\Middleware\Authorize::class,
        'guest' => \App\Http\Middleware\RedirectIfAuthenticated::class,
        'password.confirm' => \Illuminate\Auth\Middleware\RequirePassword::class,
        'signed' => \Illuminate\Routing\Middleware\ValidateSignature::class,
        'throttle' => \Illuminate\Routing\Middleware\ThrottleRequests::class,
        'verified' => \Illuminate\Auth\Middleware\EnsureEmailIsVerified::class,
        //'cors' => \App\Http\Middleware\Cors::class,
    ];

    protected $middlewarePriority = [
        \Illuminate\Session\Middleware\StartSession::class,
        \Illuminate\View\Middleware\ShareErrorsFromSession::class,
        \App\Http\Middleware\Authenticate::class,
        \Illuminate\Routing\Middleware\ThrottleRequests::class,
        \Illuminate\Session\Middleware\AuthenticateSession::class,
        \Illuminate\Routing\Middleware\SubstituteBindings::class,
        \Illuminate\Auth\Middleware\Authorize::class,
    ];

api.php
Route::post('admin-login','CustomLogin@userLogin');
    Route::post('admin/login','CustomLogin@betUserLogin');
    Route::get('logout/{type}','CustomLogin@logOut')->middleware('auth:api');
    Route::group(['prefix' => 'admin/','middleware' => 'auth:api'], function() {
        Route::post('list-user','Admin\AdminController@userList');
    });

but does the route exist?

Sometimes its getting 200 Sometimes 404

Which version of fruitcake/laravel-cors are you using? Are you on the latest 2.x version? If not, can you upgrade?
It seems you're using Cloudflare?

I am using 2.0 version of fruitcake/laravel-cors !
Can the Cloudflare is the reason of this 443 port ??

Cloudflare should Vary on the Origin header, so not sure if it's getting old responses

For somebody who use Windows 10 Maybe fixed.
This is a intel graphics card software related issue on windows 10.
How to fix:
In windows, go to:
Start menu
Type: Services:
Find the service called: "Intel(R) Graphics Command Center Service"
Right click > Properties > Startup type = Disabled
This fixed the problem for me.. :-)

Can you try with the latest 2.0.1 version?

I am getting the same problem like @abiradak with the latest version (2.0.1) in Firefox. In Chrome i am getting another problem:
GET http://localhost/api/posts net::ERR_CONNECTION_REFUSED

That's not the same error?

In Firefox I am getting the same Error:
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://localhost/api/posts. (Reason: CORS header ‘Access-Control-Allow-Origin’ missing)
And in Chrome this Error:
GET http://localhost/api/posts net::ERR_CONNECTION_REFUSED

Well is your API working correctly? Maybe you get an actual error?

Yes, the API is completly working. If I am visiting the site "http://localhost/api/posts" directly, i am getting the data..

@silasbuchwald I was facing the exact same issue with Firefox. All other browsers were working fine. I could discard an API issue as - like in your case - the API is working completely fine when fetching data with any other source than Firefox. I was finally able to solve it by changing a Firefox setting. I hope this helps.
This is my setup:

My local development environment is running on http://localhost:3000 and my API on http://api.localhost/api.
I was getting the error Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at http://api.localhost/api/test. (Reason: CORS request did not succeed).. Additionally I got TypeError: NetworkError when attempting to fetch resource..

To solve the issue I did the following:

Open Firefox and type about:config in the URL field.
Search for network.dns.native-is-localhost. For me the setting was false, which seems to be the Firefox default. Change this to true

After performing the last step the application works like a charm also in Firefox. No API code change needed.

Just recognized that this setting had an unwanted side effect for Firefox: now all non-localhost pages throw an error :-(. I guess for now I will just switch between the settings when testing with Firefox. If anybody figures out how to resolve this, let me know.

It's working here locally, using Firefox for myself. Within the allowed origins key I had to append the local url and protocol.
The following worked:
'allowed_origins' => ['https://site.test']

It's working here locally, using Firefox for myself. Within the allowed origins key I had to append the local url and protocol.
The following worked:
'allowed_origins' => ['https://site.test']

This helped, thanks Ultrono. Had to set allowed_origins = true

I have the same issue with Firefox. works with Chrome. I am using v2.0.1. I added my virtual domain like 'allowed_origins' => ['https://site.test'], but it doesn't solve the issue.

Since today I had the same problem.  Yesterday it worked perfectly.
Firefox:
Blocage d’une requête multiorigines (Cross-Origin Request) : la politique « Same Origin » ne permet pas de consulter la ressource distante située sur https://api.myapp.test/login. Raison : échec de la requête CORS.
Chrome:
ERR_CONNECTION_REFUSED
The api work fine with Insomnia.
@silasbuchwald You found a solution ?

@barryvdh Chrome, Firefox and Safari not working. I try a new fresh laravel install, same problem. Other none laravel api working fine.
I'm using Laravel Homestead up to date.
I'm adding <IP_ADDRESS>   api.myapp.test to my host file.
Any idea ?


Create buffer C++ side

Resolves #1 and resolves #10

Coverage remained the same at 100.0% when pulling 2d349bed504ec6294acb6587805a0fd5e0cce684 on PlasmaPower:master into e7699c68ae4391fa5430ed3db26d458c156b9225 on mgmtio:master.

First time working with C++ in Node! :tada:

I removed {u,}int_to_hex because they aren't used any more. Side note: is it possible that the ulong64 byte order is correct, and it was just being incorrectly outputted before? Fixing that would be a breaking change of course.

Coverage remained the same at 100.0% when pulling e47e7d3d0b1eb80a3ed41fb4e03ad88e46e0a0fe on PlasmaPower:master into e7699c68ae4391fa5430ed3db26d458c156b9225 on mgmtio:master.

adding you as a collab :) not really using htis right now

@jonathanong Thanks! To be honest, the reason I made this PR was just to get the $15 bounty on #1, but I should be able to maintain this :).

oh thank god because i don't know how to refund that money

@jonathanong Yeah, a lot of bounties on bountysource seem to fall apart that way :P

@jonathanong Mind accepting the claim on bountysource? It will auto accept in 2 weeks, but this will be a bit easier. Link to issue on bountysource: https://www.bountysource.com/issues/5524486-create-the-node-buffer-in-c-land

@jonathanong Thanks!

:D


When I disconnect from device and call FlutterBlue.instance.connectedDevices the disconnected device still appears.

// 1) Call device.disconnect();
       await device.disconnect();

// 2) Call connected devices future in periodic stream (as shown in FlutterBlue examples)
StreamBuilder(
    stream: Stream.periodic(Duration(seconds: 1)).asyncMap((_) => FlutterBlue.instance.connectedDevices),
    initialData: [],
    builder: (context, connectedSnapshot) {
        /// ...list open devices
    }
)

If I do this, then the disconnected device still shows up in my stream. This leads me to believe that there is some asynchronous disconnection that we cannot synchronize on by calling await. Am I using the library wrong or is this the current expected behavior?

I have the same issue, although I'm using a FutureBuilder.

Also the device really still connected and I need to turn off Bluetooth to drop the connection.

Facing this same issue. Was anyone able to figure this out?

did someone find a solution for this?


Wrong link on jest blog?

On this page: https://facebook.github.io/jest/blog/2016/07/27/jest-14.html#what-s-next-for-jest
The opening line "Recently Dmitrii Abramov" , has  Dmitrii Abramov name linked to some random girls twitter page lol
I think he changed his name: https://twitter.com/fbjest/status/879057512749826048?lang=en
So this should be the link the url in the name points to: https://twitter.com/aaronabramov_

Mind sending a PR with correct twitter account? :)

Submitted a PR : https://github.com/facebook/jest/pull/5608


Auto-generate tool list for CI

Currently, we are managing these lists manually, which is not a great situation, because people forget to add tool names, merge conflicts occur between PRs, etc.
https://github.com/taiki-e/install-action/blob/93a5b91855ce53b049b56dc66eecf1cf411e6057/.github/workflows/ci.yml#L44-L59
https://github.com/taiki-e/install-action/blob/93a5b91855ce53b049b56dc66eecf1cf411e6057/.github/workflows/ci.yml#L105-L125
The tricky part of generating this automatically is how to handle the differences in tools supported by each platform. Another problem is that some lists contain versions or intentional spaces (for testing https://github.com/taiki-e/install-action/issues/115). For the latter, it may be reasonable to ignore these lists for now and only auto-generate the other parts.

If you reuse the manifest, you already have the list of software that is supported by each platform.
For the specific test for the spacing, it's here to test if the installation script correctly trim the space before and after.
So it could be ignored or hard coded in the script to still have it tested.


Cannot toggling channel details when you package master

For some reason when I run npm run package and run the build on Mac I cannot open the channel details when I click a channel. It works when running npm run dev. Bizarre.
Any ideas @Empact @mrfelton?

Just to confirm I can reproduce the issue. Not sure why that would be though.

I know so weird. No errors or warnings in dev either. I'll have some time in a sec to do some more serious debugging and will report back


Product Roadmap

This PR adds a product roadmap landing page to our marketing site. The primary goal is to keep our users informed about major new upcoming features and releases while still allow enough flexibility around timeline expectations so that we aren't crunching to meet arbitrary deadlines.

Pull Request Test Coverage Report for Build c12d2d48b3bc43c00bed0e4077192d24a6e37348-PR-1740

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 70.679%

Totals

Change from base Build 0b964a1c0787f311b6e97dab8cd060386fb226ae-PR-1733:
0.0%

Covered Lines:
1967

Relevant Lines:
2783

💛  - Coveralls

@AndrewDryga I believe this is ready. Could always be improved, but I think it suffices for a first version so as not to block the announcement.

Ah also, I re-enabled Preview environments so you can actually comment directly on the preview 😎


Impossible to install now

Congratulations!  In trying to "automate" the install process from the old step-by-step version which I had zero issues with, this is now a complete and impossible mess to install.
Following "Option 1" (the steps of which no longer reflect current Puppet documentation) results in errors on Bolt's end and attempting "Option 2" just results in madness as it's very poorly documented.
Oddly enough, a separate project based upon this one is actually easier to install and would actually work if it wasn't crashing all the time on a required "Holfuy" API key that is currently impossible get just to obtain wind data, none of which is actually covered in the vast majority of the United States.
That project is:
https://github.com/ciprian288/PiWeatherRock-Windguru-Holfuy
Now, as for actually replacing Dark Sky API with something else, look at https://github.com/alexander0042/pirateweather as it seems to be a completely drop-in replacement for Dark Sky's soon-to-be-defunct API and was created just for instances like this where devs using Dark Sky can easily jump over to it.
For now it looks like I'm going to have to find an ancient version of PiWeatherRock, install it, and then edit the code myself to use the new PirateWeather API so my Pi display that's been working flawlessly for many years continues to work still.
Any Raspberry Pi project that requires the installation of additional software - like Puppet - on a completely separate PC is a huge no-go when things like Ansible exist that can be easily installed and run on the Pi itself, which Puppet cannot.
I can't think of a single dev or MSP shop I've ever seen - and I've seen a lot of them - that uses Puppet.  They all use Ansible.
I recommend learning and implementing Ansible if you're not going to create a deb package for PiWeatherRock.
I apologize if this sounds harsh but trying to following your instructions to get this re-installed has just wasted hours of my valuable time, time which could have been better spent just updating the old code itself to implement PirateWeather API calls.

Sorry to hear that.  On https://github.com/ciprian288/PiWeatherRock-Windguru-Holfuy I have updated the README. If you don't want the wind page to be displayed, go to the 'Wind screen' section in the web interface for configuring PiWeatherRock and set the pause to 0.


Node version locking and installation

https://github.com/AstarNetwork/swanky-cli/pull/131#issuecomment-1488829123
In node install command, having version locking mechanism, and allow users specify which version to install with command args.

moved to #177 , closing as it's very similar.


Download link for v2.11.1 Mac M1/ARM CLI is broken

The download link for the v2.11.1 Mac M1/ARM IBM Cloud CLI is broken here:
https://github.com/IBM-Cloud/ibm-cloud-cli-release/releases/
That page links to this URL which doesn't exist:
Mac OS X M1/ARM  (as-is)

The link is corrected on the releases page. Apologies for the inconvenience.


lumina-1.4.0: ctrl-q no longer works in qpdfview

After the upgrade to 1.4.0, when I start qpdfview and then press ctrl-q, nothing happens. qpdfview is the current snapshot configured with Qt5. In lumina-1.3.0 patchset 1 I could quit qpdfview by pressing ctrl-c.

Probably because you are no longer using qpdfviewer... :-)
qdfviewer is a completely separate, 3rd-party application that TrueOS installed out-of-box quite a while back, but in recent times that package no longer builds properly on FreeBSD (port has been broken for about 2-3 months now) and we moved to a new default application called "lumina-pdf". Since lumina-pdf is still so new, we are still missing some of the niceties that have existed in more mature applications, but we are gradually adding in them in as time goes on (keyboard shortcuts for lumina-pdf were actually just committed a couple days ago if I recall correctly).

No, I use qpdfview, the current snapshot, configured it with qt5 and compiled myself. I'm on Gentoo Linux (btw, I'm the maintainer of the lumina package in Gentoo, have just committed 1.4.0 patchlevel 1). The same qpdfview binary works as expected in lumina-1.3.0 patchlevel 1; in 1.4.0 it also works, but ignores ctrl-q. Seems mysterious. Does lumina-1..4.0 intersept ctrl-q in any way?
When I start qpdfview (and many other programs), I get
lthemeengine: using lthemeengine plugin
lthemeengine: D-Bus global menu: no
These programs work normally in spite of these messages, only qpdfview ignores ctrl-q.

Ok, that must be something in qpdfview then. The Lumina theme engine is just a style plugin for the application instance and does not touch anything related to input management.
It is also possible to setup Fluxbox to intercept the Ctrl-Q shortcut and have it close the currently-active window (usually closing the application). That will work across all applications/windows (not just Qt5 applications).


[docs] Removed references to Grid's hidden property

The hidden property of Grid was removed a couple days ago. See: https://github.com/mui-org/material-ui/pull/11348
So the references to it in the docs should be removed as well :)

It's a great first pull request on Material-UI 👌🏻. Thank you for giving it a shot!


Auto-regressive generation implementation

I have found the loop for batches/transformer layers but did not manage to find the loop for generating the outputs at different steps autoregressivey in TransformerDecoder for trainig and it seems to me there is only one pass while generating the decoder outputs.
Could you please guide me where the auto-regressive generation is implemented?
Thanks a lot

I just found that all ground truth labels are fed to the decoder in one pass during training. Sorry to bother.


[rv_dm] havereset may assert at incorrect time

Description
It appears havereset asserts when an NDM reset begins:
https://github.com/lowRISC/opentitan/blob/1fd378caf2d17239ee884c5f2742011b4554d59e/hw/vendor/pulp_riscv_dbg/src/dm_csrs.sv#L514-L517
This does not follow spec. havereset should only assert once the reset completes, and it may be used as a synchronization point for the debugger. It's not used in openocd upstream yet (where there is a bug with merely checking allunavail), but it is properly checked in riscv-openocd.

CC @GregAC

Oh, this is a duplicate, it turns out.
Same issue as the following:
https://github.com/lowRISC/opentitan/issues/15123
https://github.com/pulp-platform/riscv-dbg/issues/118


Add the framework to a Swift Cocoa project

Hello there,
I'm trying to add the project to my cocoa project .
Here is my podfile :
source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '8.1'
pod 'GoogleMaps'
pod 'Polyline', '~> 2.0'
use_frameworks!

It works perfectly in the simulator but the project can't be launched on a real iPhone.
The "Polyline" framework is not found. What am I doing wrong ?
I tried to add the framework manually too but it still does not work.
If you have advices it would be great :sunglasses:

I just updated Polyline to v3.0.0.
Can you check if it fixes your issue ?

I made a test with an empty project and this podfile:
source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '8.1'                                                                                pod 'Polyline', '~> 3.0'
use_frameworks!

and it builds and run fine on both simulator and device (iPhone 6, iOS9.1).

I'll test it, thanks :D

Hmmm this is working like a charm now even with Xcode 7.1 :) Thank you very much


Other timm backbones

Is it possible to use other timm backbones?

Some work is needed to implement the new backbone.
Which backbone are you interested in?
Although I currently don't have the bandwidth to add new models, I might be able to implement it if I become available.

@james77777778 caformer


Feat: Community Spotlight Page issue

Describe the feature
Adding similar header and footer like main page.
Improving UI to match website theme.
Making page responsive too.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@huamanraj @anuragverma108 Sir,
Can you please assign this issue to me under GSSoC'24. Thank You!

assigned 👍


Add select multiple component

one jasmine spec has failed and you should update the styleguide.

does it allow blank?

Yep.

:+1:

You mean empty array, right?

no, I mean to allow user to choose nothing.

:+1:


KYLIN-3912 fix CI

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

retest this please

Codecov Report

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

@@            Coverage Diff            @@
##             master     #590   +/-   ##
=========================================
  Coverage          ?   25.27%
  Complexity        ?     5805
=========================================
  Files             ?     1374
  Lines             ?    81503
  Branches          ?    11430
=========================================
  Hits              ?    20603
  Misses            ?    58887
  Partials          ?     2013

Impacted Files
Coverage Δ
Complexity Δ

...apache/kylin/source/hive/CreateMrHiveDictStep.java
0% <ø> (ø)
0 <0> (?)

Continue to review full report at Codecov.

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


[MXNET-1188] Maven Publishing Test Environments

Description
This PR is the second part of the maven Jenkins publishing that adds testing of the produced packages on 16.04, 18.04, and centOS. It also creates dedicated publishing dockerfiles (on 14.04) and testing dockerfiles that are built without any dependencies to ensure that the static linking worked properly.
Because the packages built on this do not yet use static linking, it instead tests against the snapshot repository.
Note that this PR depends on (and is rebased on top of) #13450. When reviewing the files, please look at only the changes in https://github.com/apache/incubator-mxnet/commit/d4ac689f1be50a1c5f519bce5eb7be0973fa1bad.
Please review so the pipeline can be put into production:
@nswamy @lanking520 @andrewfayres @piyushghai @marcoabreu @yzhliu
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@mxnet-label-bot add [CI, Java, Maven, Scala, pr-awaiting-review]

@frankfliu - You may be interested in this PR


Problem in Vcalendar#getBaseComponent

Hello,
I think there is a bug in the getBaseComponent when the filter by component name is triggered :
...(line 251)
     if ($componentName) {
            foreach ($this->select($componentName) as $child) {
                if ($isBaseComponent($child)) {
                    return $child;
                }
                return null;
            }
        }

     // Searching all components
...

IMO, the return null should be out of the loop.
For instance if componentName is set to 'VEVENT' and the vcalendar only contains a VTODO, the code will not enter the foreach and will return something because it will search all base components.
(I can issue a PR if you agree there is a problem here)

I think you're right, this does indeed look broken. This function is for the most part for single-component objects (such as in caldav, imip) so in most cases the base component appears first, but you are right that it would immediately fail if this is not true.
So a PR would definitely be appreciated.

Please, could this change be included in the current released version of vobject ? It would allow us to use the new sabre 3.1.0 in our project

If you can wait a week or two I'll do another vobject release. In the meantime, if you are not already using composer to install sabre/dav, do this... because then you can specify to use dev-master of vobject instead of the latest stable.

Ok thanks for the tip


Updates CONTRIBUTING with notice about integration testing.

This should help people with running integration tests better.

Filed #588


SPAdes: init at 3.13.1

Motivation for this change
Bioinformatic tool used into my laboratory
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

+1 for spades, this is an important tool in bioinformatics. I'd love to have this in nix.

Sorry for the mess with merge conflicts that triggered requests for a bunch of reviewers...


Can SelectionArea support text drawn in the canvas of CustomPainter?

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
I have used TextPainter to display some text on the Canvas. Now I want to select it.
Proposal
Can we consider enabling text drawn using TextPainter on the canvas to support selection, similar to a text widget?

I also need this feature


Getting NoSuchMethodError when using extension on dynamic

Given this code:
void main() {
  dynamic timestamp =<PHONE_NUMBER>;
  DateTime convertedTimestamp = timestamp.toDateTime();
}

extension DynamicExtension on dynamic {
  DateTime toDateTime() {
    if (this == null) return null;
    int milliseconds = this is int ? this : null;
    return milliseconds?.toDateTime();
  }
}

extension IntExtension on int {
  DateTime toDateTime() {
    return DateTime.fromMillisecondsSinceEpoch(1000 * this);
  }
}

I'm getting Uncaught TypeError:<PHONE_NUMBER>.toDateTime$0 is not a functionError: TypeError:<PHONE_NUMBER>.toDateTime$0 is not a function
Happens both locally with Dart VM version: 2.9.0-14.1.beta (beta) (Tue Jun 9 10:52:57 2020 +0200) on "macos_x64" and on https://dartpad.dartlang.org/ (not sure how to look which version they use)
Changing the first line from dynamic to int clears the error.

You can't have extension on dynamic - this is by design.
It's unclear to me why declaring extension on dynamic is not a compile time error though
/cc @lrhn - Lasse is this working as intended? Given that we don't resolve extension methods on dynamic, should not we prohibit to declare them to begin with?

Should we reopen this and change it to throw an error on compile then? It's very frustrating not having any errors and then have it break at runtime...
Thanks you!

Based on https://github.com/dart-lang/language/issues/716 this was a conscious decision to permit dynamic there (it essentially behaves as declaring extension on Object?).
Lets wait for @lrhn to provide the judgement here.

We could warn or err if you declare an extension on dynamic, but the on type of an extension is primarily used for super-type checks, and it allows all types, so disallowing dynamic would be an extra complication. It means exactly the same as on Object? or on void except that the type of this is also affected.
In general, you should probably never have on dynamic, on void, or on Never as extension types, but that's a semantic reason, there is no technical problem with using them. It's not very useful, but it works consistently.

If there is no technical problem using them, how come I’m seeing that error?

On Jun 17, 2020, at 7:48 PM, Lasse R.H. Nielsen<EMAIL_ADDRESS>wrote:

We could warn or err if you declare an extension on dynamic, but the on type of an extension is primarily used for super-type checks, and it allows all types, so disallowing dynamic would be an extra complication. It means exactly the same as on Object? or on void except that the type of this is also affected.
In general, you should probably never have on dynamic, on void, or on Never as extension types, but that's a semantic reason, there is no technical problem with using them. It's not very useful, but it works consistently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

There is no technical problem writing
extension X on dynamic {
  void foo() { this.bar(); }
}

It works exactly the same as
extension X on Object? {
  void foo() { (this as dynamic).bar(); }
}

which means that you can do 42.foo();. The on dynamic means that the extension applies to any receiver which has a static type which is a sub-type of dynamic and which does not have a foo member.
Declaring extensions on dynamic works perfectly fine and consistently.
What you can't do is call extension methods on a receiver of type dynamic:
dynamic x = 42;
x.foo();

This doesn't work because, for all we know, x could have a real method named foo, and this code should call it. Dart extension methods are static extension methods. Whether to call them is decided at compile-time, and what x is here is only determined at run-time.
So, a receiver of type dynamic is treated statically like it has all members, and then checked at run-time, and therefore extensions do not apply to it because instance members always take precedence over extensions.
You can't call static extension members on receivers with static type dynamic, no matter what type the object actually is, because we only know that type long after we decided whether to call some extension method.

It seems excessively strong to say "instance members always take precedence over extensions". Instance members that are statically visible always take precedence over extensions, but what's statically visible is not necessarily obvious. Consider
import 'dart:html';

extension Href on Element {
  String get href => 'http://www.example.com';
}

main() {
  document.body.appendHtml('Foo', treeSanitizer: NodeTreeSanitizer.trusted);
  var queried = document.querySelector('.foo');
  print(queried.href);
  var filtered = document.body.children.whereType<AnchorElement>();
  print(filtered.first.href);
  queried = document.body.children.whereType<AnchorElement>().first;
  print(queried.href);
}

which prints different values for what in the program text looks like the same invocation, and in 2 of the 3 cases the extension takes precedence over an instance member. So something that changes the type inference, or even assigning a value to a different variable, can change the program behavior in arbitrary ways.

http://www.example.com
https://www.google.com/
http://www.example.com


ConfigMap size limit prevents storing release

We hit a problem creating a release from a somewhat larger chart:
2016-10-19 09:42:22.348992 I | Updating "v1" (v1) in storage
2016-10-19 09:42:22.464534 I | configmaps: update: failed to update: ConfigMap "v1.v1" is invalid: data: Too long: must have at most 1048576 characters
2016-10-19 09:42:22.464583 I | warning: Failed to update release "v1": ConfigMap "v1.v1" is invalid: data: Too long: must have at most 1048576 characters

A helm install --dry-run --debug . > manifest produces a 600 KB file, base64 encoding that file gives a 810KB file.
It seems the serialised release somehow still gets larger then 1MB hitting the maximum size limit for a ConfigMap.
Some background: We are trying to install a complete openstack as a helm release and therefore have more then the usual handful of specs in the installed chart. The top-level chart package has a gzipped size of 113KB, unpacked its about 2MB.
I understand that this is pushing the limits of what helm is used for at the moment but in general I think the chart is not unreasonable in size.
One improvement to increase the size limit could be achieved by adding compression to the configmap storage backend. This would increase the practical size limit of release by a factor of 5 easily given that a release contains a lot of repetitive (compressible) data. Would a PR for that considered useful?
Any other ideas to fix our issue (apart from splitting up the release) ?

A PR for that would be very useful, especially if it is backwards compatible. I have no doubt that we'll see this issue again.

While the there are still limits to the chart/release size that can be stored in configmaps #1420 fixed the reported problem so I'm closing this.

Maybe this should be re-opened?  #1420 enabled compression to delay the point at which this problem occurs but didn't address the fact that there is a relatively small limit.  We're running into a similar issue.  It seems Helm tries to store all the content of the chart's directory.  We have cordoned off config for multiple environments, e.g. config/pd/, config/qa/, and each is only 300 KB but Helm fails to record the release due to the 1MB ConfigMap limit which is only reached by including all of config/ even though we are specifying specific values files.  Maybe we should just keep them outside the chart directory?
$ helm install --kube-context pd -n infrastructure -f config/pd/values.yaml -f <(ansible-vault view config/pd/secrets.yaml.vault) --namespace infrastructure .
NAME:   infrastructure
Error: getting deployed release "infrastructure": release: not found

$ kubectl logs --context=pd --namespace=kube-system tiller-deploy-3210613906-x3z3z | grep 'Failed to record release' | tail -n 1
2017/03/24 17:30:17 release_server.go:817: warning: Failed to record release "infrastructure": ConfigMap "infrastructure.v1" is invalid: data: Too long: must have at most 1048576 characters

$ kubectl delete namespace --context pd --cascade infrastructure
namespace "infrastructure" deleted

$ git rm -r config/qa
rm 'kube/helm/infrastructure/config/qa/secrets.yaml.vault'
rm 'kube/helm/infrastructure/config/qa/values.yaml'

$ git rm -r config/dr
rm 'kube/helm/infrastructure/config/dr/secrets.yaml.vault'
rm 'kube/helm/infrastructure/config/dr/values.yaml'

$ helm install --kube-context pd -n infrastructure -f config/pd/values.yaml -f <(ansible-vault view config/pd/secrets.yaml.vault) --namespace infrastructure .
NAME:   infrastructure
...
STATUS: DEPLOYED
...

It looks like in our case we can use .helmignore because the --values are included in the release regardless.

This is giving us trouble as well.  Is it not possible to break this file up into multiple config maps behind the scenes to get around the 1mb limit?

1MB does seem constraining in the 21st Century.  Does the community agree that this limit should be lifted?   What is it that makes it hard to lift this limit to something MUCH higher?    I do think there is value in keeping Helm charts of modest size so lifting this limit should not be construed as encouragement to build ever larger Helm charts.

+1 to re-open this ticket
This limitation is a significant road block for helm to be used in broad spectrum of applications. Usually I package all the individual pieces ( front-end, back-end, database etc.) as a single helm chart to create a single command deployment for our application. I am having way to many issues hitting the limit of storing deployment configmap. In the example below I have only a single application WSO2 which consists of 6 individual pieces. Since I have to provide all the configuration files to all 6, the overall chart size is exceeding a limit of  1048576 characters. Helm is a great tool, but this limit must be addressed in order for helm to be used for all kind of deployments.
$ helm dependency update; helm install -n esb  --set tags.dev-values=true .
Hang tight while we grab the latest from your chart repositories...
...Unable to get an update from the "local" chart repository (http://<IP_ADDRESS>:8879/charts):
        Get http://<IP_ADDRESS>:8879/charts/index.yaml: dial tcp <IP_ADDRESS>:8879: getsockopt: connection refused
...Successfully got an update from the "incubator" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. Happy Helming!
Saving 4 charts
Deleting outdated charts

NAME:   esb
Error: getting deployed release "esb": release: "esb" not found

$ kubectl logs tiller-deploy-2543526833-fltvb   --namespace kube-system

[storage] 2017/10/27 15:41:33 getting release history for "esb"
[storage] 2017/10/27 15:41:33 creating release "esb.v1"
[storage/driver] 2017/10/27 15:41:34 create: failed to create: ConfigMap "esb.v1" is invalid: data: Too long: must have at most 1048576 characters
[tiller] 2017/10/27 15:41:34 warning: Failed to record release esb: ConfigMap "esb.v1" is invalid: data: Too long: must have at most 1048576 characters

$ helm dependency update; helm install -n esb  --set tags.dev-values=true . --dry-run --debug | base64 > /tmp/manifest ; ls -l /tmp/manifest

-rw-rw-r-- 1 robert robert 2314317 Oct 27 11:55 /tmp/manifest

This (1MB configmap limit) is not something that can be directly resolved in helm. The limitation is made upstream, so we can't really do anything to change that limitation.
The better approach would be to discuss alternative approaches in a design document as I described here so that when Helm 3 development begins, we can discuss alternative solutions such as what @jwomeara suggested.

This is the root cause of https://github.com/kubernetes/helm/issues/1996
An alternative approach to storing the entire chart in a single ConfigMap should definitely be created so that Helm is not limited by this well known upstream constraint.
In addition to the storage size, because Secrets are being stored I would recommend that the Chart should not be stored within an "insecure" ConfigMap.

+1 to re-open this as well.
In the mean time, what have people been doing as a work-around?  I'm trying to create 1 helm chart that has dependencies on all the other charts that make up the application (~20 charts).  This puts it dangerously close to the 1MB limit.  Is there a better way of organizing applications made up of many charts?  Should i, for example, have each chart be it's own release?  If so, how do i manage/automate creating a release for all the charts?  Scripts?

Yeah, the limit is hurting us pretty bad. We use helm to deploy environment wide configs, these include different types of service tokens, which are pretty huge in size and don't compress well.

I think that if we could get some consensus on what a good alternative backend would be, we could get some people to write the driver. Drivers are actually pretty simple.

Two more cents here.  I just ran into this problem.  Until I found this issue, I didn't realize that helm was blindly putting all that is in the chart directory into the ConfigMap.  Finding this helped me figure out that the temporary, somewhat large file in my chart directory was causing my issue.
Long story short, maybe helm should only be putting relevant, chart related dirs/files/info into the ConfigMap, not everything it sees in the chart directory.

@paulbsch thank you for your feedback. Unfortunately, "chart related dirs/files/info" is completely arbitrary, as they can be accessed during rendering time via the .Files object. Restricting helm package to certain file extensions, known files and the like could potentially break existing use cases.
That being said, the Helm chart does have a concept of a .helmignore file, which allows users to specify which files to ignore during helm package. Will that help?

Yea.  I know about .helmignore.  I see what you're saying.  I was just thinking that it might be reasonable to limit things to Chart.yaml, values.yaml, .yaml, charts/, templates/* ...  For example.
Maybe some output/debug mode could print out what is being included in the ConfigMap?  (maybe it does that already and I didn't see it?)  That would have helped me in this case.

that would be a great option for helm package --debug or something similar. I agree.

Even just a check that lets a user know sooner would be nice.

I've proposed an extension to Kubernetes for this. Please take a look at the kubernetes issue #88709.
With this extension, you upload your configuration to s3 (or simply git) and reference it in a File resource. Finally the configmap references this file resource.
Feel free to upvote that proposal to raise awareness.

Current etcd version have 1.5mb limit already, with ability to increase it: https://etcd.io/docs/v3.5/dev-guide/limit/

By default, the maximum size of any request is 1.5 MiB. This limit is configurable through --max-request-bytes flag for etcd server.

Does this mean, that we can increase this limit in fresh k8s versions too?

/reopen  Many years past, this is still not resolved, is there a way to split a chart to store in multi configmap to resolve this?

According to the docs, the sql backend can be used if the storage requirements exceed the size limit of etcd (e.g. configmaps or secrets:
https://helm.sh/docs/topics/advanced/#storage-backends

Using such a storage backend is particularly useful if your release information weighs more than 1MB (in which case, it can't be stored in ConfigMaps/Secrets because of internal limits in Kubernetes' underlying etcd key-value store).

I doubt there will ever be on option to split the the release across multiple secrets/configmaps.

I think this issue should be re-opened as it's blocker for the most of Cloud-managed platform (EKS, GKE, ACK), which limit secret/configmap size to 1MB (~ 1048576 bytes). Such popular charts as kube-prometheus-stack won't work. This is fundamental Helm design limitation, which should be addressed.

Sadly, this is still and issue for us.  I guess it's nice to have the option to write the config to SQL, but I would prefer not to need an external dependency to get around this issue.  If you're not willing to split the release across multiple configmaps, would you consider adding the ability to write the config to a file in a persistent volume instead?  That way we wouldn't have to rely on an external dependency like SQL and could work within the confines of kubernetes.

I ran into this limitation, by using Bitnami Keycloak. I have 4 files to import, with a job. The ConfigMap is 1.3 MB, so Im stuck. Im now trying to work around this. First idea, was to create a CRD(that can hold up to 50MB, if Im correct). I did that, now Im trying to mount that config file into my Job and let it consume.
Proposal would be, to have some basic CRD "BigConfigMap"? That could possibly fix the problem, for most of the people.

I also ran into this limitation with multiple charts like the confluent kafka crds