Failed to execute 'arc' on 'CanvasRenderingContext2D'

When I set the width/height properties of the chart wrapper div via css, there can be a problem with labels sizes and wrapper div size. If the label is long and wrapper div size is small, the following error occurs:

Failed to execute 'arc' on 'CanvasRenderingContext2D': The radius provided (-37) is negative.

I didn't mention I'm talking about Radar chart.

Bug reports must be submitted with an interactive example. E.g. http://codepen.io/pen?template=JXVYzq

Duplicate of https://github.com/chartjs/Chart.js/issues/5107

You can see it here https://codepen.io/anon/pen/KGaXRJ
If you increase the width for example to 600px, then it works fine.


Add Twin Macro Support

Added twin.macro support to this plugin so we can lint code like:
tw.div`flex mt-2`

const hoverStyles = css`
  &:hover {
    border-color: black;
    ${tw`text-black`}
  }
`

Fixes #3

@Idered Can we do something to help this PR to be merged and released?

Kinda sad to see this feature sit, is there a reason this has not been adopted?


Allow addons that require IPC_LOCK capability

Currerntly it's not possible to create an addon for pigpiod (the pigpio daemon) because it uses mmap and required the IPC_LOCK capability when running the addon Docker image.
This PR adds support to home-assistant for this.

Hi @mark9white,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


missing abs during compile?

gcc:
arm-none-eabi-gcc (Arm GNU Toolchain 13.2.rel1 (Build arm-13.7)) 13.2.1 20231009
Copyright (C) 2023 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

host:
Linux buildroot 5.15.0-91-generic #101-Ubuntu SMP Tue Nov 14 13:29:11 UTC 2023 aarch64 aarch64 aarch64 GNU/Linux

error:
max@buildroot:~/stm32mp1-baremetal/bootloaders/mp1-boot$ nano mak^C
max@buildroot:~/stm32mp1-baremetal/bootloaders/mp1-boot$ ls
Makefile   boot_detect.hh     boot_loader.hh        boot_nor.hh  clocks.hh  delay.h         gpt            main.cc   pmic.hh            startup.s              systeminit.c  uboot-port
README.md  boot_image_def.hh  boot_media_loader.hh  boot_sd.hh   ddr        fsbl_header.py  linkscript.ld  norflash  print_messages.hh  stm32mp1xx_hal_conf.h  systeminit.h
max@buildroot:~/stm32mp1-baremetal/bootloaders/mp1-boot$ nano main.cc
max@buildroot:~/stm32mp1-baremetal/bootloaders/mp1-boot$ make
Building startup.s at -O3
Building main.cc at -O3
Building systeminit.c at -O3
Building ../../shared/system/syscalls.c at -O3
Building ../../shared/system/libc_stub.c at -O3
Building ../../shared/system/libcpp_stub.cc at -O3
Building ../../shared/print.cc at -O3
Building ../../third-party/STM32MP1xx_HAL_Driver/Src/stm32mp1xx_ll_usart.c at -O3
Building ../../third-party/STM32MP1xx_HAL_Driver/Src/stm32mp1xx_ll_rcc.c at -O3
Building ../../third-party/STM32MP1xx_HAL_Driver/Src/stm32mp1xx_hal.c at -O3
Building ../../third-party/STM32MP1xx_HAL_Driver/Src/stm32mp1xx_ll_sdmmc.c at -O3
Building ../../third-party/STM32MP1xx_HAL_Driver/Src/stm32mp1xx_hal_sd.c at -O3
Building ddr/stm32mp1_ddr.cc at -O3
Building ddr/stm32mp1_ram.cc at -O3
ddr/stm32mp1_ram.cc: In function 'int stm32mp1_ddr_clk_enable(ddr_info*, u32)':
ddr/stm32mp1_ram.cc:39:19: error: 'abs' was not declared in this scope
   39 |         ddr_clk = abs((int)ddrphy_clk - (int)mem_speed * 1000);
      |                   ^~~
make: *** [../../shared/makefile-common.mk:112: build/obj/obj/ddr/stm32mp1_ram.o] Error 1

Thanks for the report.
I believe this is related to gcc 13's change in what is included with the -ffreestanding flag.
Since abs is the only function in the project that's no longer part of a freestanding environment with gcc 13, probably the most simple way to work around this is just define it ourselves. Another approach would be to remove the flag and change the return type of main, but I'm not certain this would not entail other unforeseen issues.
I'll push a fix for it.

Great! Thanks for this repo/information about stm32mp1 booting and baremetal.
would it be possible to use this bootloader to then boot a real linux kernel? thinking about ditching u-boot

I haven't tried that, but I think it would depend a lot on the kernel configuration. For instance, mp1-boot doesn't support FIT images or compressed kernel images, nor does it handle secure booting. It also doesn't configure a display, USB, or ethernet, nor does it pass any parameters to the kernel. There's probably a lot more to this list...
You could configure a minimal kernel to not need any of those things, or you could implement the ones you need in mp1-boot. I'd be curious to see how it works out if you decide to tackle that.


docs: update aad group name

Which issue this PR addresses:

Fixes the group name in our docs
What this PR does / why we need it:
The old group name was under the old tenant, we are now under a new tenant in our dev subscription

Test plan for issue:

n/a
Is there any documentation that needs to be updated for this PR?
this is the docs

Merged without wawiting for e2e as it is just a single line doc change.


NA dates and errors with existing column names

From the message below, this seems to be triggered when perhaps a column is already named month. I'll dig into this and see what is going on.
After identifying dates and the NAdates screen pops up, it crashes because I think I already had a column named month in my dataset. When I removed that column name (as well as others), it seemed to work well. After the crash, i searched around in the r console and found an error something along the lines of "error duplicate column names: month"

Just tried running a few datasets through with month as an existing column. @jmerkle1 - Was this an error you had or submitted by another user?

This has now been resolved... The main issue was coming from columns such as lat, lon, month, day, year etc being in imported data but in uppercase (e.g. Lat, Lon etc).
MM was able to create new columns such as lat, lon etc but when importing to the sqllite database, errors were thrown. By now converting all column names to lowercase on import, this is fixed


OneDrive 介绍

URL: https://amazingkenneth.github.io/docs/posts/tutorial/OneDrive 介绍.html
欢迎来到此页面留言、评论、提出问题，这里的评论与 对应网页 上底部的 Gitalk 评论区中的留言互相可见。
Comments and questions are welcome on this page. Comments here can be seen in the Gitalk section at the bottom of the corresponding page.

11

Since the post had been removed, the Gitalk messages issue shall be deleted.


Add user input boxes for...

samples of interest (with typeahead for sample IDs)
home_geo
parameters mutations_per_transmission mutations_from_parent

Note: all of these values are specified in App.tsx / there's only one place to wire all this up to


cleanup: remove all filterOptions usages

Explanation of Change

While working on performance improvements at this PR:

https://github.com/Expensify/App/pull/51954

I noticed that the OptionListUtils needs a few cleanups before I can move on with the performance improvements.
In this PR I:

Removed the unnecessary wrapper function getFilterOptions (which was marked as discouraged to use anyway) and instead getOptions is directly exposed.
Remove the searchValue from getOptions as there was no code path that uses this parameter anymore. For filtering by a search value we should use filterOptions

Due to that I was able to remove one condition where we would order items > that code is actually never reached

Fixed Issues

$ https://github.com/Expensify/App/pull/51954#issuecomment-2464282215
PROPOSAL: https://github.com/Expensify/App/pull/51954#issuecomment-2464282215
Tests

Test that all search features are still working.
Offline tests

QA Steps

Test that all search features are still working.
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 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 used JaimeGPT to get English > Spanish translation. I then posted it 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

https://github.com/user-attachments/assets/98902240-9139-4217-8614-ad37286c9e65

Android: mWeb Chrome

https://github.com/user-attachments/assets/98902240-9139-4217-8614-ad37286c9e65

iOS: Native

https://github.com/user-attachments/assets/98902240-9139-4217-8614-ad37286c9e65

iOS: mWeb Safari

https://github.com/user-attachments/assets/98902240-9139-4217-8614-ad37286c9e65

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/98902240-9139-4217-8614-ad37286c9e65

MacOS: Desktop

https://github.com/user-attachments/assets/98902240-9139-4217-8614-ad37286c9e65

cc @ahmedGaber93

Reviewing today


equipment detail page headers are different than rest of site - prob should be same

I noticed on all the equipment detail pages (like large format printer) are:

missing title/h1
missing image in header

likely unrelated, but the image is real snug on the body text if yer in there fixin stuff:

def setting this one aside for post launch.
touching the equipment pages layout template is like trying to move an image inside a word doc. It's the result of about 7 nested dependency includes I'll need to strategically poke at, so will take a crack next weekend.


The subscription command does not appear in Teams

I have followed all the prerequisite steps and ensured I am an Admin in DevOps and using a public Teams channel. I have linked the DevOps project.
When I type the command @azure boards it only shows one command for "create" to create work items.  One of our service admins has tried and reported he does not get the option to create a subscription either.
Either there is a missing step or prerequisite or the subscription command is no longer supported.

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

ID: def2836e-b8c1-6ca0-0113-c72f53c17e18
Version Independent ID: 58b6cb7c-df71-39c9-325d-b35e696ddf26
Content: Use Azure Boards with Microsoft Teams - Azure Boards
Content Source: docs/boards/integrations/boards-teams.md
Service: azure-devops
Sub-service: azure-devops-integration
GitHub Login: @chcomley
Microsoft Alias: karrg

@darren-kay -- Darren, thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@chcomley -- Chrystal, please look into this potential documentation issue.

Hi @darren-kay, the subscription command is indeed supported. If all the prerequisites are fulfilled, your app is correctly installed, and you're on a public channel, and you still face issues, please reach out to the Developer Community for further assistance. They can help troubleshoot any specific problems related to your setup. Thank you.


Fix errors passing back by Nginx reverse proxy from Mina's GraphQL endpoint

2023-12-20 17:40:45 2023/12/20 15:40:45 [error] 151#151: *629 open() "/usr/share/nginx/html/50x.html" failed (2: No such file or directory), client: <IP_ADDRESS>, server: localhost, request: "POST /graphql HTTP/1.1", upstream: "http://<IP_ADDRESS>:3085/graphql", host: "localhost:8080", referrer: "http://localhost:9042/"

For whatever reason Nginx is trying to render the HTTP 500 received from Graphql on problematic response in HTML format instead of just throwing response back.

https://github.com/MinaProtocol/mina/issues/14751

Check with:

o1js: f5d277e
bindings: 9192d66
mina: 8cb4651

It was already fixed on lightnet side.

Closing for now since the issue is on the protocol side.


build settings - few questions

Hi,
I have some "user defined" build settings that i want to translate in to the project.yml file,
but i could not understand how to do it.
for example, on the project file (level):

or on s specific target:

how do i define user defined settings?
Another setting is this one:

it seems like a 2 levels setting and i could not find any example for it, how do i represent it in the project.yml?
thanks a lot for your help

Hi @menesh2
Project and target settings are defined in the same way with a settings dictionary, with project settings being on the root of the project spec, and target settings living in each target.
Config specific settings can also be specified with a configs map:
An example
settings:
  configs:
    Debug:
      MTL_ENABLE_DEBUG_INFO: YES
    Release:
      MTL_ENABLE_DEBUG_INFO: NO
targets:
  MyTarget:
    settings:
      configs:
        Debug:
          PROVISIONING_PROFILE: ""
        Release:
          PROVISIONING_PROFILE: my_provisioning_profile

See the documentation here for more info
https://github.com/yonaskolb/XcodeGen/blob/master/Docs/Usage.md#configuring-build-settings
https://github.com/yonaskolb/XcodeGen/blob/master/Docs/ProjectSpec.md#settings

For the sdk specific settings you have to use Xcode's special xcconfig conditional syntax. You can learn more about that here https://pewpewthespells.com/blog/xcconfig_guide.html
What's in your screenshot would look like this:
targets:
  MyTarget:
	settings:
      configs:
        Debug:
          CODE_SIGN_IDENTITY: iPhone Distribution
		  CODE_SIGN_IDENTITY[sdk=iphoneos*]: iPhone Developer
        Release:
          CODE_SIGN_IDENTITY: iPhone Developer
		  CODE_SIGN_IDENTITY[sdk=iphoneos*]: iPhone Distribution

Closing this for now. Let me know if you have any other questions


Only display 4 string labels - Axis X

Hello !
I have 30 or more labels (1 for each day) but i only want to display 4 of them on my line chart.
How is it possible ?
Thank you !!!

Hi there, you can use labelInterpolationFnc and return null to skip labels / grid lines to be rendered. Check this example: https://gionkunz.github.io/chartist-js/examples.html#line-scatter-random


protocol Jason.Encoder not implemented for embedded resource

One of my resources has attribute :variants, {:array, MyApp.Feature.EmbeddedType}, allow_nil?: false which is a union type. What's the preferred Ashy way to derive the Jason.Encoder?
     ** (EXIT from #PID<0.3411.0>) an exception was raised:
         ** (Ash.Error.Unknown) Context: resolving data on commit MyApp.Feature.update

     Unknown Error

     Context: resolving data on perform MyApp.Feature.Version.create
     * Context: resolving data on perform MyApp.Feature.Version.create

     ** (Protocol.UndefinedError) protocol Jason.Encoder not implemented for #MyApp.Feature.EmbeddedTypeA<__meta__: #Ecto.Schema.Metadata<:built, "">, id: "025f3cb7-bbb0-419d-b5df-7971a8deef94", key: #Ash.CiString<"true">,  ...> of type MyApp.Feature.EmbeddedTypeA (a struct), Jason.Encoder protocol must always be explicitly implemented.

     If you own the struct, you can derive the implementation specifying which fields should be encoded to JSON:

         @derive {Jason.Encoder, only: [....]}
         defstruct ...

     It is also possible to encode all fields, although this should be used carefully to avoid accidentally leaking private information when new fields are added:

         @derive Jason.Encoder
         defstruct ...

     Finally, if you don't own the struct you want to encode to JSON, you may use Protocol.derive/3 placed outside of any module:

         Protocol.derive(Jason.Encoder, NameOfTheStruct, only: [...])
         Protocol.derive(Jason.Encoder, NameOfTheStruct)
     . This protocol is implemented for the following type(s): Any, Ash.CiString, Ash.Union, Atom, BitString, Date, DateTime, Decimal, Ecto.Association.NotLoaded, Ecto.Schema.Metadata, Float, Integer, Jason.Fragment, Jason.OrderedObject, List, Map, NaiveDateTime, Time

I think we should actually be extracting the values out automatically if the value is an embedded resource.

Fix by https://github.com/ash-project/ash_paper_trail/pull/10


commit only if need commit

There no need connection commit for  misfireCount == 0 on doRecoverMisfires
https://github.com/quartz-scheduler/quartz/blob/a5c4d27e963f51097f9b2777489d310a88897ca4/quartz/src/main/java/org/quartz/impl/jdbcjobstore/JobStoreSupport.java#L3267
And if the no job triggered then no need commit:
https://github.com/quartz-scheduler/quartz/blob/a5c4d27e963f51097f9b2777489d310a88897ca4/quartz/src/main/java/org/quartz/impl/jdbcjobstore/JobStoreSupport.java#L3866

The problem is that even though no rows were altered, the transaction is open (when the connection was first obtained and row locked) and needs a commit or rollback or it will hold others up and/or eventually timeout.


Loss of multiple user interfaces in the latest 0.1.19 docker image

Between image versions datascience-notebook-default:0.1.0 and datascience-notebook-default:0.1.19 we seem to have lost RStudio, Code and XCFE.
This is likely a consequence of the multiple changes made to the repo to allow this to work on non-AMD architectures.
This is no longer a requirement and if it becomes a requirement in future, we will have to rethink how we do this.
Solution: Revert changes back to how they were previously (at 0.1.0) and tag that as the latest version.

Have reverted the change that caused the problem from this branch. We need to be careful about future PR's, from release branches. I forgot that merge conflicts cause all the changes to be reflected in the source branch. Will have to factor this in to the branching strategy moving forward.
After the revert I've checked to see if this has applied correctly and i can see that it was successfully applied:
NAME      AGE   READY   STATUS
jupyter   36d   True      Applied revision: release/0.2.62@sha1:21921b5473351123c90489353cc31b50d218affe

and that the workspace binding that caused the conflict in the first place is there
NAMESPACE    NAME                                                  WORKSPACE                         EXPIRES      STATUS   COMMENTS
jupyterhub   uhmb-generic-workspace-dale-wright                    uhmb-generic-workspace            2030-12-31

I do think that we need to move workspace management out on it's own so that we can manage these separately, this was only supposed to be a stop-gap solution after all.


Add Parrot channel side by side with Repeater channels?

I sometimes find it useful to test my repeater connection before calling using the Parrot channel.  It's pretty standard across DMR repeaters to support it - so I think I can just add it in whenever there is a DMR repeater as well.

Addressed by #23

Not completely - #23 adds the talk groups - but I need a PR to add additional channels...

It works.  You can select a repeater, and then use the menus to switch to Parrot.  I was able to use this to test my connections to the repeater.


License?

What license is this software?

I didn’t concern myself with it because I published the repository so that I could reference it in my other private projects.
Would there be a use case for you where a license is needed?

I'd like to deploy this as part of an open source project https://github.com/internetstandards/Internet.nl/pull/1142.


now-cli doesn't seem to check "now" field of package.json for deployment preference (npm vs dockerfile)

my package.json
{
  "name": "now-test",
  "version": "0.0.0",
  "private": true,
  "engines": {
    "node": ">= 7.4.0"
  },
  "dependencies": {
   ...
  },
  "now": {
    "name": "fanflow",
    "aliases": [
      "fanflow.now.sh",
      "www.fanflow.now.sh"
    ],
    "env": {
      "NODE_ENV": "production"
    },
    "type": "npm",
    "engines": {
      "node": ">= 7.4.0"
    },
    "files": [
      ".env",
      ".env.example",
      "./scripts/*",
      "./build/*",
      "./assets/**/*"
    ]
  }
}

CLI prompt:
~/Code/now-test [master] now -v
𝚫 now 4.0.0
~/Code/now-test [master] now
> Deploying ~/Code/now-test
> Two manifests found. Press [n] to deploy or re-run with --flag
> [1] package.json	   --npm
> [2] Dockerfile	--docker
> Error! Aborted

As I understood https://zeit.co/blog/now-json it seems like running now should check for this preference, no?

I'm getting this as well. Got in my package.json:
"now": {
  "type": "docker"
}

But still get asked (as above) 😞

+1 same problem as @samtgarson

This is a slightly different issue. But here is proof...
`optimized.technology  (master) $ cat now.json
{
"type": "docker"
}
optimized.technology  (master) $ stat Dockerfile
16777217 17111140 -rw-r--r-- 1 noel staff 0 270 "Feb 21 13:44:13 2017" "Feb 18 11:27:21 2017" "Feb 18 11:27:21 2017" "Feb 17 23:18:35 2017" 4096 8 0 Dockerfile
optimized.technology  (master) $ now

Deploying ~/n/optimized.technology
Two manifests found. Press [n] to deploy or re-run with --flag
[1] package.json         --npm
[2] Dockerfile        --docker `

I'm going to remove the functionality of enforcing deployment types, because they're not necessary.
That will solve all of these problems! 😊


Null values for some arguments cause Terraform errors

Found a bug? Maybe our Slack Community can help.

Describe the Bug
Terraform is throwing errors for the var.unauthenticated_priority variable being null -- which is its default value. This is probably due to issue 24142 reported in Terraform.
╷
│ Error: Operation failed
│
│   on .terraform/modules/ci_alb_ingress/main.tf line 180, in resource "aws_lb_listener_rule" "unauthenticated_hosts":
│  180:   priority     = var.unauthenticated_priority > 0 ? var.unauthenticated_priority + count.index : null
│     ├────────────────
│     │ var.unauthenticated_priority is null
│
│ Error during operation: argument must not be null.
╵

Expected Behavior
The default value should be such that no error is thrown.
Steps to Reproduce
Steps to reproduce the behavior:

Add a module using terraform-aws-alb-ingress
omit passing the unauthenticated_priority parameter.
Run terraform apply
See error

Environment (please complete the following information):
Anything that will help us triage the bug will help. Here are some ideas:

OS: Linux Ubuntu 21.04, Terraform 1.0.6
Version 0.24.0

I found a good workaround is to set unauthenticated_priority = -1
Perhaps can fix this bug by making said variable default to -1?

The value of unauthenticated_priority now defaults to null.
https://github.com/cloudposse/terraform-aws-alb-ingress#input_unauthenticated_priority
@januaryjon @mpaluchowski is this still an issue ?

If the default is still null then yes, this is still a problem, because null isn't an allowed value. See my original report.

Perhaps we can default it to -1 instead


Writing wiki pages

I wrote some wiki pages in my repo in an effort to make the documentation more clear:
https://github.com/lii2/flexbox-wiki
*Added an installation page and attributes page
*Updated the sidebar to match
*Added a table of attributes so people could scan through the attributes easily
A question: Am I installing the flexbox layout correct? It seems to be running correctly on my smartphone, but the Android Studio XML design tab is running into rendering problems:
Exception raised during rendering: android.graphics.drawable.VectorDrawable_Delegate.nDraw(JJJLandroid/graphics/Rect;ZZ)I

Thanks for updating your local wiki.
As I also commented in the #13, I'd like to keep the introductory information such as installation, usage and supported attributes in the README instead of writing it in separate wiki pages. Because README is viewed way more than the wiki pages and read first by developers.
I'd like to let developers start using this library by only reading the README as their first experience.
When developers want to know in detail, then wiki pages are good start.
So I want to keep such contents (supported attributes, installation) in the README.
It's really appreciated if you can improve the detailed wiki contents. E.g. I filed a couple of wiki improvement ideas as #84 and #85, thanks.

A question: Am I installing the flexbox layout correct? It seems to be running correctly on my smartphone, but the Android Studio XML design tab is running into rendering problems:

Not sure what caused that, but please try choosing a different API level from the API dropdown in the design tab.

Closing this issue. In favor of other issues related wiki.


Support request for skai.gr

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.03.20.

[x] I've verified and I assure that I'm running youtube-dl 2018.03.20

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Site support request (request for adding support for a new site)

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output:
youtube-dl.exe -v http://www.skai.gr/player/TV/?mmid=303411
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://www.skai.gr/player/TV/?mmid=303411']
[debug] Encodings: locale cp1253, fs mbcs, out cp737, pref cp1253
[debug] youtube-dl version 2018.03.20
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-90399-ged0e0fe102, ffprobe N-90399-ged0e0fe102, phantomjs 2.1.1
[debug] Proxy map: {'https': 'https://<IP_ADDRESS>:3128', 'ftp': 'ftp://<IP_ADDRESS>:3128', 'http': 'http://<IP_ADDRESS>:3128'}
[generic] ?mmid=303411: Requesting header
WARNING: Falling back on generic information extractor.
[generic] ?mmid=303411: Downloading webpage
[generic] ?mmid=303411: Extracting information
[download] Downloading playlist: SURVIVOR
[generic] playlist SURVIVOR: Collected 2 video ids (downloading 2 of them)
[download] Downloading video 1 of 2
[youtube] _jJt22AHqV0: Downloading webpage
[youtube] _jJt22AHqV0: Downloading video info webpage
[youtube] _jJt22AHqV0: Extracting video information
[debug] Default format spec: bestvideo+bestaudio/best
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on 'https://.googlevideo.com/videoplayback?'
[download] Destination: IHT Global Conversation - Democracy Under Pressure (Greek)-_jJt22AHqV0.f134.mp4
[download]   0.1% of 337.24MiB at 751.47KiB/s ETA 07:38
ERROR: Interrupted by user

Interrupted because it detected two irrelevant YouTube videos
youtube-dl.exe -v http://survivor.skai.gr/survivor-episodio-37-19032018/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'http://survivor.skai.gr/survivor-episodio-37-19032018/']
[debug] Encodings: locale cp1253, fs mbcs, out cp737, pref cp1253
[debug] youtube-dl version 2018.03.20
[debug] Python version 3.4.4 (CPython) - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-90399-ged0e0fe102, ffprobe N-90399-ged0e0fe102, phantomjs 2.1.1
[debug] Proxy map: {'http': 'http://<IP_ADDRESS>:3128', 'ftp': 'ftp://<IP_ADDRESS>:3128', 'https': 'https://<IP_ADDRESS>:3128'}
[generic] survivor-episodio-37-19032018: Requesting header
WARNING: Falling back on generic information extractor.
[generic] survivor-episodio-37-19032018: Downloading webpage
[generic] survivor-episodio-37-19032018: Extracting information
ERROR: Unsupported URL: http://survivor.skai.gr/survivor-episodio-37-19032018/
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpbt_jbnlh\build\youtube_dl\YoutubeDL.py", line 785, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpbt_jbnlh\build\youtube_dl\extractor\common.py", line 440, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpbt_jbnlh\build\youtube_dl\extractor\generic.py", line 3161, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: http://survivor.skai.gr/survivor-episodio-37-19032018/

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: http://www.skai.gr/player/TV/?mmid=303411
Single video: http://survivor.skai.gr/survivor-episodio-37-19032018/
Show dropdown list: http://www.skai.gr/tv/show/?showid=66630

Web TV site of Greek TV station Skai

http://www.skai.gr/player/TV/?mmid=303411 => http://videostream.skai.gr/GrCyTargeting/Gr/survivor20180321.mp4.m3u8
http://survivor.skai.gr/survivor-episodio-37-19032018/ => progressive download
http://download.skai.gr/GrCyTargeting/Gr/survivor20180319.mp4
Both streams probably blocked from outside of Greece...


Add Tornjak test

I will create a new PR to create the tests. Due to several re-base actions I corrupted this request to the point that is easier to create a new one.


Cleanup to showcase files

Remove empty fields (author.url, source)
Move source -> url if url is empty
Replace unicode encoded escaped characters with actual characters
Remove extra " by " from author.name
Remove source field for p5.editor urls (redundant)

This is fantastic, thank you so so much!!


flux_bootstrap_git "Plugin did not respond" during terraform import

We are currently on Flux 0.41.2 and terraform flux/fluxcd provider 0.25.3 and wish to migrate to using the new flux_bootstrap_git resource. While running the command tf import flux_bootstrap_git.this flux-system we encounter the following error :
The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ImportResourceState call. The plugin logs may cont
        github.com/hashicorp/terraform-plugin-framework@v1.1.1/internal/fwserver/server_importresourcestate.go:104 +0x462
github.com/hashicorp/terraform-plugin-framework/internal/proto6server.(*Server).ImportResourceState(0xc0002dba20, {0x20611a0?, 0xc002c6aae0?}, 0xc00062fee0)
        github.com/hashicorp/terraform-plugin-framework@v1.1.1/internal/proto6server/server_importresourcestate.go:44 +0x30b
github.com/hashicorp/terraform-plugin-go/tfprotov6/tf6server.(*server).ImportResourceState(0xc000698fa0, {0x20611a0?, 0xc00068da40?}, 0xc00399c1e0)
        github.com/hashicorp/terraform-plugin-go@v0.14.3/tfprotov6/tf6server/server.go:846 +0x1f6
github.com/hashicorp/terraform-plugin-go/tfprotov6/internal/tfplugin6._Provider_ImportResourceState_Handler({0x1c63e80?, 0xc000698fa0}, {0x20611a0, 0xc00068da40}, 0xc000bb4230, 0x0)
        github.com/hashicorp/terraform-plugin-go@v0.14.3/tfprotov6/internal/tfplugin6/tfplugin6_grpc.pb.go:403 +0x170
google.golang.org/grpc.(*Server).processUnaryRPC(0xc0004c41e0, {0x206b378, 0xc0006d61a0}, 0xc0029f7680, 0xc0005162a0, 0x2ff3ef8, 0x0)
        google.golang.org/grpc@v1.51.0/server.go:1340 +0xd33
google.golang.org/grpc.(*Server).handleStream(0xc0004c41e0, {0x206b378, 0xc0006d61a0}, 0xc0029f7680, 0x0)
        google.golang.org/grpc@v1.51.0/server.go:1713 +0xa36
google.golang.org/grpc.(*Server).serveStreams.func1.2()
        google.golang.org/grpc@v1.51.0/server.go:965 +0x98
created by google.golang.org/grpc.(*Server).serveStreams.func1
        google.golang.org/grpc@v1.51.0/server.go:963 +0x28a

Error: The terraform-provider-flux_v0.25.3.exe plugin crashed!

This is always indicative of a bug within the plugin. It would be immensely
helpful if you could report the crash with the plugin's maintainers so that it
can be fixed. The output above should help diagnose the issue.

We have decided to migrate to using the new resource and then eventually upgrade to the newer version of flux.  The datasources flux_install and flux_sync are not yet deprecated in version 0.25.3 Provider

It's not clear how you get this error without a look at your tf files, I have not seen this error before. You mentioned that you want to upgrade to the new version of Flux, I don't know if there is a seamless transition possible between the old and new versions of terraform-provider and their state. I don't think you need to use terraform import - flux bootstrap is idempotent so if you bootstrap over an existing bootstrap, it should overwrite it with the same content.
But the bootstrap_git resource belongs to the new provider - what resources exactly are you trying to import?

@EishanR could you provide me the full Terraform configuration you had previously please so I can best advise on next steps.

Hello @EishanR  👋
I hope all is well with you. I'm just reaching out to you as part of our initiative to ensure the quality and responsiveness of our project's issue tracking. Thank you for your previous contributions by reporting issues you've encountered; your engagement is invaluable to the health and progress of our project.
I've reviewed the issue you reported and noted that it involves using supported resources, which is great. However, I also noticed that there has been little activity on this issue lately. We understand that circumstances change and priorities shift, which may affect the urgency or relevance of an issue.
We're implementing a policy regarding inactive issues to keep our project's issue tracker effective and up-to-date. If there's no further activity on this issue within the next 3 weeks, we will mark it inactive and close it. This approach helps us maintain focus on current and actively pursued concerns, ensuring resources are well allocated.
That said, we are still very much interested in resolving the issue you've encountered. If the problem persists or if you have any new information that could help us address it, please don't hesitate to update the issue or comment. Your input could be crucial in finding a resolution. Alternatively, if the issue no longer affects you or has been resolved, let us know so we can close it with the right resolution.
We're here to support and work with you to overcome any challenges you face using our project. Your continued feedback and participation are essential to our community's success.
Thank you once again for your contribution. I'm looking forward to your update and am here to help with any other questions or issues you might have.
Best regards,
Steve


WIP rr graph lib

This PR is towards completing a baseline python rr_graph library to make custom rr_graphs beyond what arch.xml can do.
Current test suite is something like:
python -m lib.rr_graph.channel: passes
python -m lib.rr_graph.graph: passes
python utils/testarch_graph.py tests/build/testarch/2x4/wire.rr_graph.xml: fails due to missing channels (being investigated)
After stabilizing above to create the expected data structure, a relatively small amount of work needs to be done to add to_xml() functions to assorted XML elements. For the final library there might be a little work related to open VPR PR on mapping pin names to pins in the rr_graph pins (which is ambiguous in current format).

The pull request at https://github.com/SymbiFlow/symbiflow-arch-defs/pull/115 has replaced this one.


Ensure recalculation when a new function is passed in

Context
Coming from other programming languages, the scope of the cached values may be surprising in JavaScript. When looking at community solutions, I was certainly surprised that other folks were not testing whether f had changed between invocations of the closure returned from memoizeTransform().
Change
Add a new test case asserting that different values of f have different caches.

Thank you 🙇


Retry action on next trigger if failed previously

If an action fails, it will be logged but that's all right now. It would be better to retry the action on the next trigger.

Havin though about this, it also has some unwanted side-effects so it should be optional. Closing this for now until there's a better idea.


Typo: takling

desc = "Automatically hides the takling frame when no voice over is playing.",

No part of this string is in the addon anymore. I think this is now
  desc = "Talking NPC portrait will not appear when voice over audio is played.\n\n" ..
                            Utils:ColorizeText("This might be useful when using other addons that replace the dialog experience, such as " ..
                                Utils:ColorizeText("Immersion", NORMAL_FONT_COLOR_CODE) .. ".",
                                GRAY_FONT_COLOR_CODE),```


Questions on values in ISO when document header is parsed as Hash

In https://github.com/metanorma/metanorma.com/pull/219 we can see some values need work.
{
  :agency => "ISO",
  :authors => [],
  :authors_affiliations => {},
  :docnumber => "ISO/NWIP 33032",
  :docnumeric => "33032",
  :docsubtitle => "",
  :docsubtitlemain => "",
  :docsubtitlepartlabel => "Partie ",
  :doctitle => "Cybernetics — Neuro-information interchange interface",
  :doctitlemain => "Neuro-information interchange interface",
  :doctitlepartlabel => "Part ",
  :doctype => "Article",
  :docyear => "2020",
  :draft => nil,
  :draftinfo => "",
  :edition => "2",
  :editorialgroup => ["TC 399"],
  :ics => "XXX",
  :obsoletes => nil,
  :obsoletes_part => nil,
  :revdate => nil,
  :sc => "XXXX",
  :secretariat => "XXXX",
  :stage => "10",
  :stage_int => 10,
  :statusabbr => "NWIP",
  :tc => "TC 399",
  :tc_docnumber => [],
  :unpublished => true,
  :wg => "XXXX"
}

These values not set should be nil to indicate it is not supplied:

docsubtitle is "" => change to nil
docsubtitlemain is "" => change to nil
draftinfo is "" => change to nil
ics is XXX => change to nil
sc is XXXX => change to nil
secretariat is XXXX => change to nil
wg is XXXX => change to nil

This is strange:

unpublished is true => change to published = false

These default values are wrong:

doctype is Article => change to the default of the flavor, i.e. standard for ISO
docsubtitlepartlabel is "Partie ", but this is French?

These values should be fixed?

:doctitle => "Cybernetics — Neuro-information interchange interface", Why isn't   just space, and — just the long dash?
:doctitlepartlabel => "Part ", This is a bit strange with a trailing whitespace?

What does this mean?   :tc_docnumber => [], The tc_docnumber can have multiple values?

This is a request for massive refactoring of things that are not truly broken, and will accordingly be aggressively demoted in priority.
 —  is built in Asciidoctor behaviour as an expansion of " -- ", putting thin spaces around em-dash. It is a legitimate way of dealing with em-dahes: I see no reason to overrrule it, and Asciidoctor would not make it easy.
Subtitle is the "other" language to the Title in ISO; if the document were in French, the subtitle would be in English.
The XXX were there to signal to the user that a mandatory value has not been supplied.
You explicitly requested multiple TC numbers in https://github.com/metanorma/metanorma-iso/issues/230: you had a document with two of them.

Empty strings have already been replaced with nils.
Not strongly motivated to flip unpublished to published; unpublished is just a more generic word for draft, and draft is the exceptional state in formatting.
Setting default doctype to be "standard" in standoc.
:doctitlepartlabel=>"Part 1", properly; the number was absent because of missing data in the sample.


Kanmao@168

_Originally posted by @kanmao in https://github.com/jnv/lists/issues/213 Dj30 mix 070531673 Kanthoeun Trumoney

Bing mix dj30 Stick to Kanthoeun 070531673 trumoney

Join Super96 ID 17113276A Kanthoeun


V09 final json api

While this does not address the new write line protocol, it does bring the JSON v09 API implementation in line with the final v0.9.0 release enough to work.

+1 for merging this to master, I ran into this same problem with "name" being now "measurement".

Thank you very much!


added exit message reminding to close haskelldo for windows users

Whoops, meant to pull this to my private fork 👅


Breakpoint over line number

Hello vscode developers,
I would like the option to have my breakpoints on the line numbers instead of on the side, similar to intellij ide.
Example

My current vs code version: 1.95
Thank you

I think Chromium's look is nice.


Central Package Management example uses Update instead of Include

The Project1 xml code block on the CPM doc page uses <PackageVersion Update= instead of <PackageVersion Include=....
<Project>
  <Import Project="$([MSBuild]::GetPathOfFileAbove(Directory.Packages.props, $(MSBuildThisFileDirectory)..))" />
  <ItemGroup>
    <PackageVersion Update="Newtonsoft.Json" Version="12.0.1" />
  </ItemGroup>
</Project>

Using Update doesn't work and causes nu1008 errors.

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

ID: 2d000f01-60a3-59c2-e06d-c02c8721fa76
Version Independent ID: 2d000f01-60a3-59c2-e06d-c02c8721fa76
Content: Central Package Management
Content Source: docs/consume-packages/Central-Package-Management.md
Service: nuget
GitHub Login: @JonDouglas
Microsoft Alias: jodou

@matt-entrekin, the example isn't fully detailed, but it implies that a PackageVersion was already defined for the package in the imported Directory.Packages.props file from the <Import statement. This is how one subdirectory can use a different version of a package than a different subdirectory. If you were to use Include, rather than Update, then you'd get a NU1506 warning about duplicate items for the same package

@zivkan: how does that compare with using VersionOverride?
Is Update only intended for use in nested Directory.Package.props files, while VersionOverride is only for use in individual projects' csproj files?


perl528: init at 5.28.0

Motivation for this change
new version added.
default perl is still 5.24.x

Success on x86_64-linux (full log)
Attempted: perl528
Partial log (click to expand)
gzipping man pages under /nix/store/a2iqnq9fczi9nq6lmpw543cm7lx3p866-perl-5.28.0-man/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/a2iqnq9fczi9nq6lmpw543cm7lx3p866-perl-5.28.0-man
checking for references to /build in /nix/store/a2iqnq9fczi9nq6lmpw543cm7lx3p866-perl-5.28.0-man...
shrinking RPATHs of ELF executables and libraries in /nix/store/v9n6665gaj6xmhi8f9nwv0nb9rg6jhx4-perl-5.28.0-devdoc
gzipping man pages under /nix/store/v9n6665gaj6xmhi8f9nwv0nb9rg6jhx4-perl-5.28.0-devdoc/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/v9n6665gaj6xmhi8f9nwv0nb9rg6jhx4-perl-5.28.0-devdoc
checking for references to /build in /nix/store/v9n6665gaj6xmhi8f9nwv0nb9rg6jhx4-perl-5.28.0-devdoc...
/nix/store/3phh2cjpdkh3sm7g96wqmbyx1bkl81x4-perl-5.28.0

Success on aarch64-linux (full log)
Attempted: perl528
Partial log (click to expand)
gzipping man pages under /nix/store/9ks1smsksmaadx4aqq5hzr4vaik6bqvr-perl-5.28.0-man/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/9ks1smsksmaadx4aqq5hzr4vaik6bqvr-perl-5.28.0-man
checking for references to /build in /nix/store/9ks1smsksmaadx4aqq5hzr4vaik6bqvr-perl-5.28.0-man...
shrinking RPATHs of ELF executables and libraries in /nix/store/09pwlqjd26jkfr1nc8njfnhy8hgcq249-perl-5.28.0-devdoc
gzipping man pages under /nix/store/09pwlqjd26jkfr1nc8njfnhy8hgcq249-perl-5.28.0-devdoc/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/09pwlqjd26jkfr1nc8njfnhy8hgcq249-perl-5.28.0-devdoc
checking for references to /build in /nix/store/09pwlqjd26jkfr1nc8njfnhy8hgcq249-perl-5.28.0-devdoc...
/nix/store/m6drigivpnhjwia5ib3h2dzlazzp5b5r-perl-5.28.0


Use DRUPAL_CORE as var name in config

The Issue
How This PR Solves The Issue
Manual Testing Instructions
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Needed after https://www.drupal.org/project/gitlab_templates/issues/3441816 was merged 3 weeks ago.


Add https support for JUnit 5

Could we please add support for a consumer https mock server in the JUnit 5 implementation.

4.1.1 has been released

Awesome, thanks!


RigidBody2D cannot rotate after CollisionShape2D is disabled

Godot version:
3.2.2.stable.official
OS/device including version:
Kubuntu 18.04
Issue description:
As described in this issue:
https://github.com/godotengine/godot/issues/30551
After a RigidBody2D disables its CollisionShape2D, it cannot be rotated until the shape is re-enabled.
Steps to reproduce:
This is my player:

This is the change_state method of my player (you can see that INIT and INVULNERABLE disable collision):
func change_state(new_state):
	match new_state:
		INIT:
			$CollisionShape2D.set_deferred("disabled", true)
			$Sprite.modulate.a = 0.5
		ALIVE:
			$CollisionShape2D.set_deferred("disabled", false)
			$Sprite.modulate.a = 1.0
		INVULNERABLE:
			$CollisionShape2D.set_deferred("disabled", true)
			$Sprite.modulate.a = 0.5
			$InvulnerabilityTimer.start()
		DEAD:
			$CollisionShape2D.set_deferred("disabled", true)
			$Sprite.hide()
			linear_velocity = Vector2()
			angular_velocity = 0
			emit_signal("dead")
	state = new_state

And this is the method used to rotate the player:
func _integrate_forces(physics_state):
	set_applied_force(thrust.rotated(rotation))
	set_applied_torque(spin_power * rotation_dir) # this line is not working when shape is disabled
	var state_transform = physics_state.get_transform()
	if state == INIT:
		state_transform = Transform2D(0, screensize/2)
	if state_transform.origin.x > screensize.x + radius:
		state_transform.origin.x = 0 - radius
	if state_transform.origin.y > screensize.y + radius:
		state_transform.origin.y = 0 - radius
	if state_transform.origin.x < 0 - radius:
		state_transform.origin.x = screensize.x + radius
	if state_transform.origin.y < 0 - radius:
		state_transform.origin.y = screensize.y + radius
	physics_state.set_transform(state_transform)

When the player is in the INVULNERABLE state it doesn't rotate. Please note that spin_power * rotation_dir is correctly set to a correct value even in that state.
Minimal reproduction project:

https://bitbucket.org/lukepass/space-rocks/src/master/
Here is the zip: https://bitbucket.org/lukepass/space-rocks/get/7fad1b35ac35.zip
Please the the invulnerability timer to 10 seconds in the Player scene then run the game. Hit a rock and try to turn.
Thanks!

I'm able to reproduce this issue with 23b553ba0603161346526e1821bff5002520173c

I also got some errors on the console after running your code, but it's a matter of deferring spawn_rock on _on_Rock_exploded:
call_deferred("spawn_rock", size - 1, new_position, new_velocity)

Thanks for the additional correction! As a side topic, do you know where can I find more info about why sometimes is necessary to use call_deferrend when calling methods in process?


Question: SAML2 together with other authentication

Hi,
I'm working on a project which started by having the standard user authentication + two factor.
Now I need to integrate the project with SAML authentication as well, where some users will use the SAML protocol and other will use the previous auth + two factor.
Is it possible to have both auths working together?
Thank you in advance.

I'm also looking to a scenario like this and would be interested in the answer. (I see a reference to using separate URLs in an extra view in the README, but IIUC, that's not what I want).

Use a login resources that takes the login endpoint as GET parameters, you Will redirectthe user-agent on what he selected

Sure, it's possible, just make sure to list all your authentication backends in your Django settings.


Redshift: add Redshift DDL for com.unbounce/event_context/jsonschema/1-0-0

This ticket needs to migrate to Iglu Central

Migrated to snowplow/iglu-central#656


Possible memory leak with Promise.each

I'm using bluebird v3.3.4 running on node v4.4.0. Leaky code:
const Promise = require('bluebird');

Promise.each(new Array(3000), () => {
	const longString = (new Array(1000000)).join('1');

	return Promise.resolve(longString);
});

Not leaky code:
const Promise = require('bluebird');

Promise.each(new Array(3000), () => {
	const longString = (new Array(1000000)).join('1');

	return Promise.resolve(longString).then(() => void 0);
});

It seems that the results of the promises are stored somewhere, but they shouldn't be, at least this is what I understand reading the docs of Promise.each:

Resolves to the original array unmodified, this method is meant to be used for side effects.

Issue is happening on earlier versions of bluebird too.

I've noticed, that second snippet is also leaking memory when long stack traces are enabled. Is this desired behaviour?

@tdzienniak this is because each is currently implemented through mapSeries https://github.com/petkaantonov/bluebird/blob/master/src/each.js#L14-L17 - the behavior of each was modified in 3.0 and then reverted when people complained.
Note that this is not a memory leak - it is merely wasteful behavior.

We can implement directly with _iterate and overriding shouldCopyValues https://github.com/petkaantonov/bluebird/blob/283412e2ab2cd45f4c513e9a16b361b348a527d5/src/promise_array.js#L164-L166 - petka, I can take a shot at it but it's probably faster for you to fix it than to me to fix it and you to verify.

@benjamingr yes, I've noticed that it is implemented through mapSeries, which is why with conjunction with long stack traces is wasting a lot of memory. I spent a lot of time trying to figure out why my node process is running out of memory. Maybe a note in the docs describing this behaviour will be helpful for others?

Ping @petkaantonov

@benjamingr feel free to give a shot at fixing it

@tdzienniak if you feel like it - I would appreciate it if you take a look at the code with the fix and see if it fixes the issue for you.

@benjamingr ok, I can test the examples form my initial post.