New Valheim update brought an issue to PlanBuild with it

LogOutput.log

The in-game result of this for me was that upon logging in, the blueprints tab was completely gone.  The tools tab and everything that comes with it was there, but the blueprints tab was completely missing.  I was able to solve this by building something quick, small and simple and creating a blueprint out of it.  That actually made my blueprints tab, and all my blueprints show back up. So far I haven't seen any other issues.

having the same problems. also selecting objects for a blueprint makes them stay highlighted even if you cancel the blueprint or remove them from the selection

+1

selection by click and radius is totally fubar
blueprints not showing but physically present (as above)
completely unuseable for saving a plan

+1
same issues as above

+1

having the same problems. also selecting objects for a blueprint makes them stay highlighted even if you cancel the blueprint or remove them from the selection

Yep. Facing the same issue here. Select and unselect work but the graphic doesn't change and remain highlighted in green.

for everyone having the issue of the pieces staying green when they save a blue print, i found that if you just fly far enough away to de-render the build, when flying back to the build the pieces are no longer green!

...2 weeks later...
Is there no update or comment from maintainer yet?
valheim discord said something about 'holidays... back next week' but that was a week ago iirc.

There is a temp plugin posted on thunderstore under a different dev. You can use that until this main one is updated. I haven’t noticed any regressions on my dedicated server

On Sunday, Sep 08, 2024 at 10:02 PM, BETLOG @.*** @.***)> wrote:
...2 weeks later...
Is there no update or comment from maintainer yet?
valheim discord said something about 'holidays... back next week' but that was a week ago iirc.
—
Reply to this email directly, view it on GitHub (https://github.com/sirskunkalot/PlanBuild/issues/96#issuecomment-2336969274), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AD7QNDB34ZLDM3AWHYQRLN3ZVT6STAVCNFSM6AAAAABNFU7CQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZWHE3DSMRXGQ).
You are receiving this because you commented.Message ID: @.***>

fixed in 0.17.0


Provide feedback to users not entering their realm properly

The realm input in the username is properly enforced in the iOS App (as well as an explicit match @realm.tld or otherwise (*.)realm.tld) but no user feedback is visible (it's maybe provided for a very short time).
When this can't be made visible in the dialog (as we do on Andriod), perhaps show it in a popup of some sorts?

Test instructions:

go to "Eduroam Visitor Access (NL)"
type<EMAIL_ADDRESS>type any password
press Log In
now a dialog shows

Tested on:

iPhone XR, iOS 17.6.1

Version:

geteduroam: 2.4 (128)

Test approved.
Provided steps used for testing.


Is the "Device Virtualization" page supposed to be reachable through Bing search?

I was searching for public documentation on some internal functions - specifically HdvCreateDeviceInstance and HdvInitializeDeviceHost - and while i couldn't find anything on MSDN's search tool I was able to find an entire topic on Bing search.  The page looks odd ... it has all the right function links but the descriptions of each function are from the Host Compute Service page.  And the page doesn't seem to show up in the index at all on the left.
I'm curious whether this documentation is supposed to be findable externally.  I checked and the search works and MSDN loads the page even if i'm not signed in with corp credentials.
Just wanted to check in and make sure this is supposed to be reachable.  And if it is, then the text of the page is incorrect.

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

ID: af9c2379-5625-ca71-2ded-bc2526f86e8e
Version Independent ID: 0aa9183c-ef65-9629-1dc3-d34a5722120c
Content: Device Virtualization
Content Source: virtualization/api/hcs/Reference/hdv/DeviceVirtualization.md
Product: virtualization
GitHub Login: @FayMeng
Microsoft Alias: mabrigg

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Update jsonwebtoken crate.

With this, we no longer need to build two versions of 'pem' and 'base64'
crates. Introduces a duplicate version of 'time' crate, though, but it's
still progress.

I'm not sure I understood the thing with expiration claim correctly. @LizardWizzard


Extension stopped working

Issue description
This extension had been working for a few months now, but I noticed that it just stopped working today. I'm using Safari 16.3, macOS 13.2. It doesn't mute nor speeding up ad anymore.

After checking, I believe that
var adOverlay = document.querySelector('.ytp-ad-player-overlay');

is null even though there is an ad in the video.
However, I noticed that .ytp-ad-player-overlay-layout is not null so maybe we could change the line to
var adOverlay = document.querySelector('.ytp-ad-player-overlay-layout');

Yeah, I think youtube has changed something that makes the extension fails.
Have you tried your modification and see if it works?

I just noticed this issue today. I will be testing it, and if it does not cover all cases, I will try to fix it with a wildcard to be less prone to YouTube changes and create a new release. Thanks for the contribution!


Support dynamic request uri, query params and payload (for -d and -D)

Docs in README.md
Tests in requester/generator_test.go
I use hey and needed a way to send dynamic payload (request body) to endpoint so this is the hack that I thought maybe useful to contribute.
While the docs and tests are based on JSON payload, it is agnostic of content type - all that matters is the placeholders are in the body template and it will be dynamically interpolated. The generator is activated only once at the start only if the body template contains at least one placeholder so otherwise it won't get in the way.
(PS: while at it i have refactored the deprecated ioutil usage)

I would also love this feature!
@rakyll what do you think?
Found some related issues, but for request query parameter:
https://github.com/rakyll/hey/issues/35, https://github.com/rakyll/hey/pull/183

i think it is neat and uniform to make query args and path uri dynamic in the same way and semantics as the body payload. a POC that I am working on is looking great, pushing it asap. i hope this will resolve all dynamic issues once and for all

but getting it merged might be difficult

@Ojasv-Singh-Zeotap it isn't merged yet so no live
(seems it won't be ever so closing this PR for good, i was thinking of doing something of my own but can't afford to invest any more time)

Sadly, I believe these changes will never be merged. However, I found them so useful that I forked the project and added them to my fork. Thanks for your work, @adhocore!


E492: Not an editor command: PluginInstall

I 、git clone https://github.com/VundleVim/Vundle.vim.git ~/.vim/bundle/Vundle.vim
2、edit .vimrc
set shell=/bin/bash
"vundle
set nocompatible
filetype off
set rtp+=~/.vim/bundle/Vundle.vim
call vundle#begin()
Plugin 'VundleVim/Vundle.vim'
call vundle#end()
filetype plugin indent on
3、run :PluginInstall in vim ,but show "E492: Not an editor command: PluginInstall"
Can you help me?

What this means is that vim is not picking up your vundle installation. This is atypical, but not unheard of.
To make sure that I fully understand --- make sure that you have closed vim and fully restarted it after modifying your vimrc.
Assuming that you have done this, have you verified that .vim/ exists and is filled as you expect? Sometimes people forget which vimrc they are using. It might be a good idea to make sure that your vimrc is being sourced. (This is particularly relevant if you are a new user of neovim and forget that init.vim != .vimrc).

I have encountered the same problem. In my case I had export VIMINIT=":set nobackup noswapfile encoding=utf8 viminfo=" in my .bashrc, after remove it, vundle works.

What this means is that vim is not picking up your vundle installation. This is atypical, but not unheard of.
To make sure that I fully understand --- make sure that you have closed vim and fully restarted it after modifying your vimrc.
Assuming that you have done this, have you verified that .vim/ exists and is filled as you expect? Sometimes people forget which vimrc they are using. It might be a good idea to make sure that your vimrc is being sourced. (This is particularly relevant if you are a new user of neovim and forget that init.vim != .vimrc).

I find the reason。the error is in this line "set rtp+=~/.vim/bundle/Vundle.vim"
I am not the root user ,so I need to use $HOME/.vim
but I can not source .vimrc。Is there another command to using without reboot system

It is not necessary to be root to use vim or to use vundle. But I think you are confused, as ~ in typical bash expands to the same directory as $HOME. And I don't understand what you are referring to when you mention a system reboot.

Thanks！You are right。Now my question is how to source .vimrc after I change it 。If I use :Source in vim ,It becomes  effective in this file。but if I open another file。I need to use :Source too 。I source it in the shell, it can not work。

@284794507 Assuming Vim is finding your .vimrc file, it will be run when you first start Vim.  Nothing will be lost when you open new files (assuming plugins or file type autocommands aren't interfering).
If your .vimrc changes don't seem to be working when you start Vim, you might want to check the location of the .virmrc file that Vim is loading by typing
 :echo $MYVIMRC

in Vim.

Is your vim a tiny version? If so, see #944

if it's show in after type wrong command in CMP panel, then just press esc and then type again your command, that you want to run. I hope this is fine.


cmake(build):add missing basic and nxlooper cmake script

Summary
add interpreter basic and nxlooper missing CMake build
Impact
fix ci fail in https://github.com/apache/nuttx/actions/runs/11880169863/job/33103008538?pr=14756
Testing
cmake -B build -DBOARD_CONFIG=sim:minibasic -GNinja
cmake -B build -DBOARD_CONFIG=sim:alsa -GNinja

refresh pass

[Experimental Bot, please feedback here]
No, this PR description does not fully meet the NuttX requirements. While it addresses some aspects, it lacks crucial details.  Here's a breakdown of what's missing and how to improve it:
Missing Information in Summary:

Why is the change necessary?  Simply stating "add interpreter basic and nxlooper missing CMake build" doesn't explain the underlying problem. Why were these CMake builds missing? Was it an oversight, a bug, a new feature request?  Explain the motivation.
What functional part of the code is being changed? Be specific. Mention the affected files/modules (e.g., interpreter, nxlooper, CMakeLists.txt).
How does the change exactly work?  Don't just say what changed, explain how it works.  For example, "Added CMake build rules for the interpreter and nxlooper components by including the necessary source files and dependencies in the CMakeLists.txt files. This allows these components to be built as part of the standard NuttX build process."

Missing Information in Impact:

While the CI failure is mentioned, this is more of a testing detail than an impact assessment. Address all the specific impact points:

New feature? Existing feature changed?  If this adds interpreter support, state that.  If it just fixes a build issue, clarify.
Impact on user:  Will users now have access to a new interpreter?  Will any existing user workflows be affected?
Impact on build: YES. Describe how the build process changes. E.g., "The interpreter and nxlooper will now be built by default for the specified configurations."
Impact on hardware: Likely NO.  But state this explicitly.
Impact on documentation:  If the interpreter is a new feature, or if the build process changes for users, documentation updates are likely needed.
Impact on security:  Consider if introducing the interpreter or changing the build could have security implications. Even if the answer is NO, state it explicitly.
Impact on compatibility:  Likely NO, but state it explicitly.

Missing Information in Testing:

Build Host(s): Provide details about your build environment.
Target(s):  "sim:minibasic" and "sim:alsa" are good starting points, but mention the architecture (e.g., sim:qemu-x86_64:minibasic).
Testing logs before change: You need to show the failure that you're fixing. Include the relevant error messages from the CI.
Testing logs after change: Showing that the build now completes successfully is important.  A concise summary of the build output or confirmation that there are no errors is sufficient. Don't paste entire build logs.

Example of an Improved PR Description:
## Summary

This PR addresses a missing CMake build configuration for the interpreter and nxlooper components, which was causing CI failures (https://github.com/apache/nuttx/actions/runs/11880169863/job/33103008538?pr=14756).  Previously, these components were not included in the standard NuttX build process for the `sim:minibasic` and `sim:alsa` configurations. This change adds the necessary CMake build rules to include the interpreter and nxlooper source files and dependencies in the respective `CMakeLists.txt` files.

## Impact

* **New feature added?** YES (if the interpreter wasn't previously available), or NO (if it existed but couldn't be built).  Be explicit.
* **Impact on user:** YES (if new feature) Users can now build and use the interpreter on simulated `minibasic` and `alsa` configurations. NO (if just build fix) No user-facing changes other than a fixed build.
* **Impact on build:** YES. The interpreter and nxlooper are now included in the build for `sim:minibasic` and `sim:alsa` configurations.
* **Impact on hardware:** NO.
* **Impact on documentation:** YES (if new feature). Documentation should be updated to describe how to use the new interpreter functionality. NO (if just build fix) Assuming no functional changes.
* **Impact on security:** NO.
* **Impact on compatibility:** NO.
* **Anything else to consider?**  None.

## Testing

I confirm that changes are verified on my local setup and work as intended:
* **Build Host:** Linux (Ubuntu 22.04), x86_64, GCC 11.2.0
* **Target(s):** `sim:qemu-x86_64:minibasic`, `sim:qemu-x86_64:alsa`

Testing logs before change:

[ERROR] CMake Error at .../CMakeLists.txt: (some error message indicating missing interpreter/nxlooper build)
... other relevant error messages ...

Testing logs after change:

Build completed successfully.
(Or a brief summary confirming successful compilation and linking of the interpreter/nxlooper).

By providing more comprehensive information, reviewers can more easily understand the purpose, impact, and validation of your changes, leading to a faster and smoother review process.


🎨 Admin++

I don't manage to find existing tests for admin part. You can advice @noam-honig ?

There are none at this time
On Fri, Aug 30, 2024 at 9:53 AM JYC @.***> wrote:

I don't manage to find existing tests for admin part. You can advice
@noam-honig https://github.com/noam-honig ?
—
Reply to this email directly, view it on GitHub
https://github.com/remult/remult/pull/482#issuecomment-2320257529, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD65PUZQLLSVINBRBK5EGJDZUAJHLAVCNFSM6AAAAABNLHOE72VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRQGI2TONJSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm not sure what to add :o
Actually, I don't get how they are counting it :/

The coverage is counted by code that runs as part of the "npm run test" code - in that process. anything else is not included in coverage
To conver things, I usually add tests within the regular tests of what can be covered.
As for the decrease in coverage - don't worry, since pull requests can't run the mongo db tests due to environment variables - that is the cause of the decrease in coverage

Good, so good to merge? :D
In next PR, you want me to look at mongo ?

The issue with mongo is:

there are tests that are done with docker - but that version doesn't support transactions.
there are tests that are done with atlas - but that has a database url that is a secret and is not included in the ci of prs.

@jycouet Are you ok with merging this?

Yes, we can


Fixes issue #54: adds explicit width and height to force layout on menu component

In some situations when a component that is passed to the menu prop and is wrapped
within a another Component view that provides no ability to style its view container
the menu view will collapse and hide its contents because the view container may not
have an explicit width and height set.
By adding an explicit width and height to the menu rule it will prevent this from
happening.

Thanks! Good catch!


Disable horizontal scroll for tab

how to disable horizontal scroll for tab menu?

Could you please explain further what you mean by horizontal scroll? There should be no horizontal scroll on the BottomNavigation.

i want to disable horizontal scroll when shifting the screen. so only move to another tab when clicking the tab. what's the solution @timomeh ?. thanks

Sorry, but I don't understand what you mean by "disable horizontal scroll when shifting the screen". The BottomNavigation shouldn't/can't scroll horizontally. Can you please show your behaviour in a gif?

maybe @roofiah mean like this.

can u play this gif?
I also had a problem with button navigation. Is possible to make the screen only change when we tap the button, so if we scroll the screen it wont change or stay in that screen.
thanks before

I can't play the gif. I need some kind of reproduction in order to understand this issue.
Are you using the BottomNavigation with react-navigation or without?

Do you maybe mean, that the Tab changes if you swipe left or right? That's a react-navigation feature. You can disable this behaviour with swipeEnabled: false, see here: https://reactnavigation.org/docs/navigators/tab

OK thanks alot... its work :)


v0.54.0 and below broken due to SPM package resolution

Description
A recent update to swiftui-navigation is not backwards compatible with v0.54.0 and below of this package.
As a result, when attempting to resolve the package graph, we run into:
product '_SwiftUINavigationState' required by package 'swift-composable-architecture' target 'ComposableArchitecture' not found in package 'swiftui-navigation'.
We've resolved this issue by updating to v0.55.0, but it might be worth considering package pinning via exact for SPM to better handle semver violations by underlying packages.
Checklist

[ ] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[ ] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
Tag v0.54.0 should remain functional regardless of changes in dependencies
Actual behavior
Tag v0.54.0 and below no longer work due to a backwards-incompatible API change
Steps to reproduce
No response
The Composable Architecture version information
0.54.0
Destination operating system
macOS 13.5
Xcode version information
No response
Swift Compiler version information
No response

Hi @venkr, yes this was unfortunately necessary due to a bug in Xcode's build system and how they handle libraries with underscores in the name. FWIW we did detail this problem in the SwiftUINavigation release:
https://github.com/pointfreeco/swiftui-navigation/releases/tag/0.8.0

We've resolved this issue by updating to v0.55.0, but it might be worth considering package pinning via exact for SPM to better handle semver violations by underlying packages.

Using exact to pin versions will only cause more problems in the future. It makes it difficult for libraries to evolve naturally and will create lots of resolution errors. FWIW, it is possible for you to perform exact pinnings for all of TCA's transitive dependencies. Simply depend on the libraries explicitly and set their versions to exact. That will force TCA to use those versions of the libraries.
Since I don't think there is really anything else we can do to help with this issue in the library (other than letting people know they need to update their libraries), I am going to close the issue.


Closure fires keypress instead of keydown for Firefox

Gecko tried to not fire keypress for non-printable chars in https://bugzilla.mozilla.org/show_bug.cgi?id=968056. But unfortunately that broke Google Docs and had to be backed out (sort of, a pref was landed and it was set to false in Nightly):
https://bugzilla.mozilla.org/show_bug.cgi?id=1442528
The cause appears to be:
https://github.com/google/closure-library/blob/ccdb3bd5c80094bc65a56053aa3ea782e2711b62/closure/goog/events/keyhandler.js#L282
I'm not sure what the best path forward is right now, especially since the current solution is to check UA strings / versions to know how to behave and we don't have a version with said behavior (because it's unclear what other sites or libraries will break).

I'm a little bit confused about what this snippet does. I think it will end up matching Chrome, Edge and Safari, which is now the per-spec behavior. If that's right, when this is fixed, then maybe it should be inverted to instead detect Firefox older than a certain version as having the bug?
But... what version could be checked, before we know that this change will successfully be made in Firefox?

I'm pretty confused by the Closure code.  Firefox fires keydown for non-printable chars.  That's not changing.  The only change being made is to stop firing keypress for some of them.
Put another way, what would break if this code were changed to:
goog.events.KeyHandler.USES_KEYDOWN_ = true;

?

cc @masayuki-nakano

There's a similar condition in actionhandler.js as well:
https://github.com/google/closure-library/blob/be0326ba32f47ea74af4fbe8df038e579ec35215/closure/goog/events/actionhandler.js#L84-L89

Although I've not read the code around key event handling, I have no idea why the original developer needed to listen to keypress events only for Firefox because keypress events for non-printable keys and key combinations which are fired only on Firefox  are just redundant. So, I mean, any web applications must not need to wait keypress event only on Firefox. Instead, web apps should/can listen to keydown event to handle non-printable key actions.
But if there were other compatibility issue which made the original developer listen to keypress for Firefox, we (Mozilla) need to fix the incompatible points first.
So, I think that if there were no reasons to listen to keypress for Firefox, it just does not make sense to set USES_KEYDOWN_ to false.

The way the code is structured feels like using keydown was a workaround for the "bug" of not firing keypress in the non-Gecko browsers.

Hmm, if so, there should be KeyboardEvent.isPrintable or something which actually indicates if followed by keypress event... (I proposed that but rejected.)

Reading around USES_KEYDOWN_ briefly. Looks like that firesKeyPressEvent should be fixed too. It checks UA string to guess if the keydown event is followed by keypress event. Additionally,  looks like that there should be a check of ignoring unexpected keypress events (i.e., if firesKeyPressEvent returns false for a keypress event, it should've already dispatched closure's event, so, not necessary to dispatch same event again.).

@EiraGe @NavidZ do either of you want to take a stab at updating closure and testing it against FireFox?

This code was written in 2008. At the time, keydown events didn't repeat in Firefox when you held down a key. Has that changed? If so, USES_KEYDOWN_ should change to whatever Firefox version changed this behavior or (even better) be removed entirely.

Appears this bug is still open.. https://bugzilla.mozilla.org/show_bug.cgi?id=91592
I do get multiple keydown's for FireFox on Windows 10, and Debian. @masayuki-nakano please advise.

I do get multiple keydown's for FireFox on Windows 10, and Debian for held keys correctly.

I ge multiple keydowns on Mac as well, for Firefox versions starting with Firefox 4 (Firefox 3.6 and earlier do not fire multiple keydowns for a held key).
Looking a bit, I think the Mac behavior was changed to the current one in https://bugzilla.mozilla.org/show_bug.cgi?id=599887 and the Linux behavior in https://bugzilla.mozilla.org/show_bug.cgi?id=597981 (though that was just dealing with a change in how GTK sends key events; before GTK made that change Firefox on Linux would send repeated keydown events).
Windows repeated keydown all along' see https://bugzilla.mozilla.org/show_bug.cgi?id=91592#c8 from back in 2004.
So I think bug 91592 was fixed by the fixes in bug 597981 and bug 599887, and those fixes shipped in Firefox 4.

Yeah, I fixed a lot around the key repeat behavior. Here is the document:
https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent#Auto-repeat_handling
When I implement KeyboardEvent.repeat on Firefox, I checked all platforms which currently Firefox supports. In any those platforms, I see same behavior except some special cases like OS doesn't use auto-repeat, etc.


Python 3.11 now available

Python base image 3.11 is now available and can be added to the table
mcr.microsoft.com/azureml/curated/minimal-py311-inference:latest
Thanks

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

ID: 1e865f86-3e71-868e-a389-e5fea8e820be
Version Independent ID: 925f6a78-5cb8-5839-cdee-ab74b1ccb60f
Content: Prebuilt Docker images - Azure Machine Learning
Content Source: articles/machine-learning/concept-prebuilt-docker-images-inference.md
Service: machine-learning
Sub-service: inferencing
GitHub Login: @dem108
Microsoft Alias: sehan

@ShaneOsborne
Thanks for your feedback! We will investigate and update as appropriate.

@ShaneOsborne
Thank you for bringing this to our attention.
I've delegated this to content author @dem108, who will review it and offer their insightful opinions.

Thanks @ShaneOsborne , curated environments are specially managed, and this article isn't best positioned to capture those environments/images. Let us create a PR to address this one.


Study Progress

201402413 ImHoJeong Study Progress
To do List

[ ] Markdown Editing
[x] Study Next.js
[ ] familarize React hooks
[ ] Typescript-in-5-minutes

Questions

[ ] When do you primarily use AMP pages?
[ ] Do you use Next.js a lot in the field?

Next.js
Next.js tutorial to Korean

Good job


Output list of subnets, IDs and CIDRs

The first thing some of us would want to do is map the subnets over to an EKS solution. Here's a quick example
module "eks" {
  source  = "terraform-aws-modules/eks/aws"
  version = "~> 19.0"

  cluster_name    = "my-cluster"
...
  vpc_id                   = "vpc-1234556abcdef"
  subnet_ids               = ["subnet-abcde012", "subnet-bcde012a", "subnet-fghi345a"]
  control_plane_subnet_ids = ["subnet-xyzde987", "subnet-slkjf456", "subnet-qeiru789"]

*** Given the above EKS requirements, what's the best way to extract availability_zone, availability_zone_id, (subnet) cidr_block and (subnet) ID? ***
This VPC module makes that fairly difficult. We have to really go digging for attributes; example:
vpc_details {
  "public_subnet_attributes_by_az" = {
    "us-gov-east-1a" = {
...
      "availability_zone" = "us-gov-east-1a"
      "availability_zone_id" = "usge1-az1"
      "cidr_block" = "<IP_ADDRESS>/18"
...
      "id" = "subnet-0d2dc9c4e190e15ae"
}

Please output these elements specifically:

Subnet CIDRs
Subnet IDs
AZ IDs

it's worth noting that the output for "core_network_subnet_attributes_by_az" = {} <- is empty.
something like this in the EKS config would be ideal
  vpc_config {
...
    security_group_ids = [aws_security_group.apps_cluster.id]
    subnet_ids         = aws_subnet.vpc_network[*].id
  }

Or, perhaps work with the EKS module folks and output something like:
output "eks_outputs" {
  value = module.vpc....
}

So we can get outputs specifically for EKS with the rest.

Hello, thanks for opening this issue! We have a slightly different take on module outputs which we detailed here: https://github.com/aws-ia/terraform-aws-vpc#output-usage-examples
The concepts above you mentioned can all be achieved with the current module outputs. Here are some examples how you can munge the data
Given hcl example:
module "vpc" {
  source  = "aws-ia/vpc/aws"
  version = ">= 4.2.0"

  name       = "multi-az-vpc"
  cidr_block = "<IP_ADDRESS>/20"
  az_count   = 3

  subnets = {
    private = { netmask = 24 }
    eks = { netmask = 24 }
  }
}

In this module you can make n number of private subnet groups. Everything is considered a private subnet unless it is a reserved keyword (public, core_network, etc - see docs for full examples). In the above HCL i only have 2 private subnets, you can see how the name labels are organized in the tf console:
$ tf console
> [ for subnet_name, value in module.vpc.private_subnet_attributes_by_az: subnet_name ]
[
  "eks/us-east-1a",
  "eks/us-east-1b",
  "eks/us-east-1c",
  "private/us-east-1a",
  "private/us-east-1b",
  "private/us-east-1c",
]

That means if you wanted the eks subnets to be the control plane and the private subnets to be subnet_ids you can do this:
module "eks" {
  source  = "terraform-aws-modules/eks/aws"
  version = "~> 19.0"

  cluster_name    = "my-cluster"
...
  vpc_id                   = module.vpc.vpc_attributes.id
  subnet_ids               = [ for subnet_name, subnet_attributes in module.vpc.private_subnet_attributes_by_az: subnet_attributes.id if startswith(subnet_name, "private") ]
  control_plane_subnet_ids = [ for subnet_name, subnet_attributes in module.vpc.private_subnet_attributes_by_az: subnet_attributes.id if startswith(subnet_name, "eks") ]

The reason core_network_subnet_attributes_by_az is empty is because core_network is a reserved subnet name referring to a AWS NetworkManager CoreNetwork which is admittedly a confusing name in this context. However, thats the service name :/
I hope this helps! I understand the output concepts are not quite as straight forward as other modules but we believe its easier to maintain and provides the maximum amount of outputs with the lowest amount of code

Hello! Closing this Issue - as Drew provided an answer on how to obtain the outputs :)


Create trentino.txt

@Leodanperez First stage of verification passed, official domain is gob.pe as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.drelm.gob.pe/drelm/contratacion-de-personal/concursos-2021/


Guide: how to customzie SQLA related model appearances in ModelView

I had to go from this:

To this:

Here's how to do it.
The representation is defined by this method of flask_admin.contrib.sqla.ajax.QueryAjaxModelLoader:
class QueryAjaxModelLoader(AjaxModelLoader):
      #skipped lines
      def format(self, model):
            if not model:
                return None
            return (getattr(model, self.pk), as_unicode(model))

There is no way around subclassing a loader.
Subclass it:
class CustomFormatAjaxLoader(QueryAjaxModelLoader):
    def format(self, model):
        if not model:
            return None
        format_func = self.options['format_func']
        return (getattr(model, self.pk), format_func(model))

Notice I am passing a new function using the options dict. It's base I want to reuse the loader, and apply different formatting to different models.
Now that we have a custom loader, we need to make the ModelView use it.
ModelView has a method _create_ajax_loader that calls create_ajax_loader.
We have to override the process with a custom create_ajax_loader function.
Unfortunately the only way to do this that I found was to copy/paste the original create_ajax_loader and replace the loader class because the loader class is hardcoded.
def override_create_ajax_loader(model, session, name, field_name, options, format_func=lambda x: str(x)):
    attr = getattr(model, field_name, None)

    if attr is None:
        raise ValueError('Model %s does not have field %s.' % (model, field_name))

    if not is_relationship(attr) and not is_association_proxy(attr):
        raise ValueError('%s.%s is not a relation.' % (model, field_name))

    if is_association_proxy(attr):
        attr = attr.remote_attr

    remote_model = attr.prop.mapper.class_

    options['format_func'] = format_func
    options['order_by'] = 'name'
    return CustomFormatAjaxLoader(name, session, remote_model, **options)

I am also passing a format_func here.
order_by is hardcoded as a quick fix, without it AJAX queries trigger errors on MSSQL. You can ignore it if you are not using MSSQL.
Finally subclass ModelView and override loader creation:
def format_generation(gen):
    return 'Generation ({}) of serie {}'.format(gen.name, gen.serie.name)
class SerieModelView(ModelView):
    form_ajax_refs = {
        'generations': {
            'fields' :['id', 'name']
        },
    }

    def _create_ajax_loader(self, name, options):
        return override_create_ajax_loader(self.model, self.session, name, name, options, format_func=format_generation)

That's it. Now you control the representation using the format_func. You can pass any callable that returns a string, it should work.
Whole code:
from flask_admin.contrib.sqla.ajax import QueryAjaxModelLoader
from flask_admin.contrib.sqla.tools import is_relationship, is_association_proxy
class CustomFormatAjaxLoader(QueryAjaxModelLoader):
    def format(self, model):
        if not model:
            return None
        format_func = self.options['format_func']
        return (getattr(model, self.pk), format_func(model))

def override_create_ajax_loader(model, session, name, field_name, options, format_func=lambda x: str(x)):
    attr = getattr(model, field_name, None)

    if attr is None:
        raise ValueError('Model %s does not have field %s.' % (model, field_name))

    if not is_relationship(attr) and not is_association_proxy(attr):
        raise ValueError('%s.%s is not a relation.' % (model, field_name))

    if is_association_proxy(attr):
        attr = attr.remote_attr

    remote_model = attr.prop.mapper.class_

    options['format_func'] = format_func
    options['order_by'] = 'name'
    return CustomFormatAjaxLoader(name, session, remote_model, **options)

format_generation(gen):
    return 'Generation ({}) of serie {}'.format(gen.name, gen.serie.name)

class SerieModelView(ModelView):
    form_ajax_refs = {
        'generations': {
            'fields' :['id', 'name']
        },
    }

    def _create_ajax_loader(self, name, options):
        return override_create_ajax_loader(self.model, self.session, name, name, options, format_func=format_generation)

If you know an easier way to do it - please let me know!
I would like this whole process to be less hacky.
Suggestions:

Remove loader hardcoding, add ability to pass loader class somehow.
Add ability to pass a function to format output, for example via form_ajax_refs:

form_ajax_refs = {
        'generations': {
            'fields' :['id', 'name'],
            'format': lambda x: str(x),
        },
}

That's definitely an overkill. All you have to do as to add a __str__ property on your model (python 3, __unicode__ for python 2).

@mrjoes I have a __str__ property on my model.
The problem is my __str__ property is used elsewhere. To be precise I use it to create a string representation of models for logs.
That is my __str__ generates a programmer-readable representation.
But in the admin I want to generate a human-readable representation.
So my models are used not only in the flask_admin utilizing project. If I changed __str__ method of the model to fit flask_admin conventions then I would have to change all of my other projects.
It makes no sense to not be able to control the output format. It makes it impossible to tailor the representation to a single project (or single view even).
Using __str__ is a reasonable default, but I would like a way to override it.

Up


Compatibility with aioboto3 >= 13.0

pytest-aioboto3 currently depends on aioboto3 ^12.0.0. What stops it from running with aioboto3 >=13.0? Can you please bump the dependency?
FWIW, the aioboto3 changelog is here: https://github.com/terricain/aioboto3/blob/main/CHANGELOG.rst

Good point, no strong reason.
It would be nice to be able to add tox into here to make sure everything works with multiple versions of aioboto3, I would be happy to accept additional contributions towards that end.
With the very light touch that I maintain this project as it is, I think you're right that the correct answer is to pin the dependency as >= 12.0.0 so other people can at least use it and report problems.
Thanks for opening an issue!

FYI I tagged 0.3.0 that relaxes the version constraint.
Thanks again for the report!


Implement Campaign Entry Creation Job

Description
When a campaign is created, a job should also be created given the campaign info.
This job creates character entries on the associated characters and GM entries for the GM.

Possible Implementation
User a Campaign observer to manage the job when a campaign is created/edited
Look into Laravel jobs for implementation of dispatching the entry creation

Out of scope

Requirements / Restrictions

Hey team! Please add your planning poker estimate with ZenHub @bcobo341 @JasonGadoury @m-triassi @RobertoNittolo


.Net Agents - History for Prompt Based Strategies being Encoded

Encoded history results in significantly more token usage and can also obscure debugging or diagnostics.
Embedding JSON within a plain text file shoudn't require encoding.
Investigate how this occurring and simplify.
History:
[{"Role":{"Label":"assistant"},"Items":[{"$type":"TextContent","Text":"Hello! How can I help you today?"}]},{"Role":{"Label":"user"},"Items":[{"$type":"TextContent","Text":"I want to upload a file"}]}]

I'm running into this issue. Its a waste of tokens and at present we can't control the history making it worse. So this needs addressing.


Update CGPoint+Angle.swift

Won't build without this line.
Thanks! 😊

Thanks @teameh , apologies I forgot to add it myself. The rebase in legacy-projects should be automatic


ng serve not refreshing after syntax error

Bug Report or Feature Request (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.4.1
node: 7.10.0
os: darwin x64
@angular/animations: 4.4.0-RC.0
@angular/cdk: 2.0.0-beta.10
@angular/common: 4.4.0-RC.0
@angular/compiler: 4.4.0-RC.0
@angular/core: 4.4.0-RC.0
@angular/flex-layout: 2.0.0-rc.1
@angular/forms: 4.4.0-RC.0
@angular/http: 4.4.0-RC.0
@angular/material: 2.0.0-beta.10
@angular/platform-browser: 4.4.0-RC.0
@angular/platform-browser-dynamic: 4.4.0-RC.0
@angular/router: 4.4.0-RC.0
@angular/cli: 1.4.1
@angular/compiler-cli: 4.4.0-RC.0
@angular/language-service: 4.4.0-RC.0
typescript: 2.5.2
Repro steps.
ng serve and generate a syntax error.
The log given by the failure.

Desired functionality.
ng should refresh
Mention any other details that might be useful.

Same issue here. Is it related to the typescript version? I notice you running 2.5.2. I might try running it with a pre 2.4 version and see if that makes a difference.

@then3rdman It tried<EMAIL_ADDRESS>but it doesn't make a difference

I think i am also facing this issue. ng serve stops compiling after i upgraded from 1.3.2 to 1.4.1
Versions:
  " @angular/animations": "^4.2.4",
    "@angular/common": "^4.2.4",
    "@angular/compiler": "^4.2.4",
    "@angular/core": "^4.2.4",
    "@angular/forms": "^4.2.4",
    "@angular/http": "^4.2.4",
    "@angular/platform-browser": "^4.2.4",
    "@angular/platform-browser-dynamic": "^4.2.4",
    "@angular/router": "^4.2.4",
 "devDependencies": {
    "@angular/cli": "^1.4.1",
    "@angular/compiler-cli": "^4.2.4",
    "@angular/language-service": "^4.2.4",
    "@types/jasmine": "~2.5.53",
    "@types/jasminewd2": "~2.0.2",
    "@types/node": "~6.0.60",
    "codelyzer": "~3.1.1",
    "jasmine-core": "~2.6.2",
    "jasmine-spec-reporter": "~4.1.0",
    "karma": "~1.7.0",
    "karma-chrome-launcher": "~2.1.1",
    "karma-cli": "~1.0.1",
    "karma-coverage-istanbul-reporter": "^1.2.1",
    "karma-jasmine": "~1.1.0",
    "karma-jasmine-html-reporter": "^0.2.2",
    "protractor": "~5.1.2",
    "ts-node": "~3.2.0",
    "tslint": "~5.3.2",
    "typescript": "~2.3.3"
  }

Was having the same issue. Downgrading my typescript version from 2.5.2 to 2.4.2 seemed to workaround the issue, as suggested above.

Heya, can you provide me with a reproduction I can follow on a new project to see this happening?

Hey @filipesilva,
I think the way to reproduce is, create a syntax error before ng serve.
if your code is correct the first time, everything works fine, the problem is when you have an error from the start.

BUMP!!!
I experience the same issue after splitting my application into submodules. The application was recompiling fine after any syntax error. Now after a syntax error i constantly get errors.

Thanks for reporting this issue. This issue was originally reported a long time ago and since then we've had many releases, one of which might have addressed this problem. Please update to the most recent Angular CLI version.
If the problem persists after upgrading, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.


[Doc] update Readme CI/CD Section

please add Documentation for Readme
Description
Section CI/CD:

dev branch deployment link is not correct
master branch deployment is missing
euro branch deployment is missing

Tasks:

Change dev link from https://anyway-newsflash-infographics.web.app to https://anyway-infographics-staging.web.app
Change master deployment  from TBD to https://anyway-infographics.web.app
Add euro deployment action and deployment (link  https://anyway-infographics-euro.web.app/
make sure all links work correctly

solved


Add unknown state

nodata will be for lots that have no data (e.g. Gompitz in Dresden)
unknown for those where we don't know any better (e.g. any lot from a provider that doesn't list any further information)

more or less implemented in most cities


getStatus returns 400 with IPDB

For more info see this thread: https://github.com/bigchaindb/js-bigchaindb-driver/issues/28#issuecomment-309396056
It's still to be determined if this is a IPDB or js-driver related bug

just a heads up: when using the release js driver against diminet.bigchaindb.com I get a 400 too. So I think we can rule out IPDB then

But IPDB is actually more helpful in the response, nice:
diminet returns: The browser (or proxy) sent a request that this server could not understand.
IPDB returns: Invalid transaction schema: '1' is not of type 'integer'

This is a BigchainDB server problem.
We'll handle here: https://github.com/bigchaindb/bigchaindb/issues/1576


[REF] pstats_print2list: Using native limit and remove '(' in method print

The limit by default don't set a limit of max number, is a limit similar to hottest


Update Algolia config

This resolves DOC-426:
https://linear.app/buildkite/issue/DOC-426/update-algolia-config-to-reflect-new-html-structure
Testing to be done between Render and the live site to confirm this works as expected.

@lachlanhardy we could probably add an RSpec test that checks for all of the Algolia selectors, right?

@plaindocs Cool idea!

@plaindocs I'll add that test. Meanwhile, I can't find any situations where the results don't line up the same across the two. I can't decide if that's great or not :) At least it means I've not broken it!
Still I'd appreciate testing from the rest of the team, as you'll all know the content better.

@plaindocs and I have been struggling to prove this is working - or NOT working - so we're ditching it in favour of the upgrade to Algolia Crawler.


Update commons-lang3 to 3.15.0

About this PR
📦 Updates org.apache.commons:commons-lang3 from 3.12.0 to 3.15.0
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.commons", artifactId = "commons-lang3" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.apache.commons", artifactId = "commons-lang3" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #941.

Superseded by #948.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CCS, Git and JavaScript with associated notes listed

We did it !


How to avoid to list POST updates (have only POST inserts)

Hi,
how can I exclude POST updates to my resources?
When i expose a resource for POST method, i get two endpoints:
POST /api/resources
POST /api/resources/{pk}

how can i avoid to list the second one?
A concrete example:
http://i.imgur.com/KQPEsAF.png
I use a simple viewset with zero custom definitions
class ReasonViewSet(models.ViewSet):
    """ [comment] """
    serializer_class = ReasonSerializer

You're using a viewset, not a view. You're not using DRF correctly, so that's your issue. :hamburger:


Update Blog “a-wizard-of-earthsea/index”

Automatically generated by Netlify CMS

👷 Deploy Preview for tubular-cheesecake-3ca0e1 processing.

Name
Link

🔨 Latest commit
21a1572dc8482082e2687df5440213666ea7d8e2

🔍 Latest deploy log
https://app.netlify.com/sites/tubular-cheesecake-3ca0e1/deploys/651e3bdfcfb30f0008f25b04


chore: bdd-spring-1583310768 to 0.0.1

chore: Promote bdd-spring-1583310768 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Wrap semidiscretization in RHS passed to OrdinaryDiffEq.jl

This should help with TTFX - if we switch to the standard setup for which they precompile.

Good point. We should definitely compare it with different options, e.g.,

Baseline (main + regular OrdinaryDiffEq)
RHS(semi) + regular OrdinaryDiffEq
main + OrdinaryDiffEq with precomp disabled
...

Before merging.
In any case, thanks for bringing up this issue and a possible solution!


Augment dataframes with metadata from the origin file

Right now, a third party process saves out json files into an S3 bucket for us. The filename looks like <prefix>-<iso_datetime>.json.gz, and each file is the output of an endpoint that specifies a time-value pair. For example, the file might look like:
[{"time": "2024-06-21T00:00:00Z", "value": 1.0}, ...]

We're loading these files in via wr.s3.read_json and ideally we want to be able to take the latest updated value for any particular time. Or in pandas terminology:
we_want = df_dataset.groupby("fileModifiedTime").last()

I don't think this is possible right now, because there is no way to get information from either filename or file metadata using the wr.s3.read_json method.
If we move away from awswrangler, we can do something like this using pyarrow.dataset, as the dataset has a files attribute and you can call pyarrow_dataset.filesystem.get_file_info(pyarrow_dataset.files)
Describe the solution you'd like
It would be great if there was some way to augment the returned dataframe with metadata coming from the files that were loaded, such as the LastModifiedTime.

Under the hood wr.s3.read_json is simply leveraging pd.read_json.  In any case, given how specific your ask is I would recommend to use pyarrow.dataset instead as you suggested


Trouble with "Create a horizontal Line using HR element" lesson

In the Create a Horizontal Line using HR Element lesson, the check fails when the code is correctly formatted following the instructions.
Campers are presented with the following code:
`

<s>Google</s>Alphabet

Google was founded by Larry Page and Sergey Brin while they were Ph.D. students at Stanford University.

`

The instructions are just to add a horizontal rule tag beneath the
 line. Simple enough. However, the tests won't pass unless you first swap the  and
 tags on the following paragraph. The test seemingly just has a typo in the code its checking against.

Browser Name: Chrome
Browser Version: Version 68.0.3440.106 (Official Build) (64-bit) (up to date)
Operating System: Windows 10

@QuincyLarson I saw your request that someone create this issue, and didn't see it in the list, so hopefully I'm not duplicating this request... and after two weeks, hopefully I'm not the first reporting it here!

Seems like it should be an easy enough fix... superficially. I'd like to tackle it, if so... if someone could point me to the right directory(s). Thanks!

@dkreidler Sorry, but this issue already exists in both freeCodeCamp (#18030 ) and the curriculum repos. Its actually fixed and merged in the curriculum. But just not live yet. For a workaround with this issue, interchange the  and  tags.
Next time, feel free to drop in a message in the Contributers chat room to confirm about an issue.

@dkreidler Could you close this issue as it is a duplicate of #18030

Closed as duplicate of #18030


Hyperion on other SoC- or MCU-based  Boards

Hey,
I'm using Hyperion for Ambilight since  long time with my RaspPi2 and now RaspPi3 and OpenELEC. I'm very happy with it.
Now I want to equip my flat with more LED-stripes using Hyperion, but I don't want to buy a RaspberryPi for every LED-strip.
So is there a way to use Hyperion on even cheaper SoC- or MCU-based Boards like Adruino?
Or at least a "light version" without framegrabber which uses the same commads to remotely set colors?

How about an arduino/ESP8266 based led driver that listens to UDP packets?
https://github.com/tvdzwan/hyperion/wiki/Network-and-distributed-features#udp-led-device-support

well, I am using hyperion on openelec/wetek core with DYI adalight like solution using esp8266 (esp-01) + pl2303 serial converter and ws8212b strip. an esp + serial convertor is ~ $5 on ebay, but I'd bet atmega328p nano will do the same job for the same price.

For another project using node.js I wanna develop an extension/service to control hyperion clients. The module I'm using for this is https://github.com/WeeJeWel/node-hyperion-client which uses JSON websocket to send the commands. So the best case scenario would be if the hyperion client would "understand" JSON. ....or I just look for a module to convenietly send UDP packets.

for nodeMCU, i am not sure if this fit your needs. You could use the hyperion fordwarder.
https://github.com/SciLor
a german short tutorial http://www.forum-raspberrypi.de/Thread-hyperion-tutorial-esp8266-nodemcu-addon-wifi-led-controller-udp

Done? Done
/cleanup - for further discussion - forum
https://hyperion-project.org


Want to add airbnb clone as frontend application for Web using Next JS

Go ahead

done @sonumahajan

Good


Keras fit with mixed dataset/ndarray data results in batch_size arg error

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04 & macOS 10.14.5
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.14.0-rc0-55-g1b365ca304 1.14.0-rc1
Python version: 3.6.8 & 3.7.2
CUDA/cuDNN version: none
GPU model and memory: none

Describe the current behavior
Combining tf.data.Dataset with tf.keras.Model fails since 1.14.0rc0 when using a dataset for training but a numpy array for validation data. Logs with errors for both 1.14 and 1.13 below.
Describe the expected behavior
Being able to use a dataset only for training but a numpy array for validation data.
Code to reproduce the issue
import tensorflow as tf
import numpy as np

data = np.random.randn(1000, 10)
targets = np.random.randn(1000, 1)
ds = tf.data.Dataset.from_tensor_slices(data).batch(32)

model = tf.keras.Sequential()
model.add(tf.keras.layers.Dense(32, input_dim=10))
model.add(tf.keras.layers.Dense(32))
model.add(tf.keras.layers.Dense(1))

model.compile(optimizer="rmsprop", loss="mse")
model.fit(ds, steps_per_epoch=10, validation_data=(data, targets), batch_size=32)

Other info / logs
Behaviour using 1.14.0rc1:
Without batch_size=32 set:
% python bug.py
[...]
1/10 [==>...........................] - ETA: 1s - loss: 2.3122
Traceback (most recent call last):
  File "tensorflow_bug.py", line 14, in <module>
    model.fit(ds, steps_per_epoch=10, validation_data=(data, targets))
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py", line 780, in fit
    steps_name='steps_per_epoch')
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 409, in model_iteration
    steps_name='validation_steps')
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 335, in model_iteration
    batches = make_batches(num_samples_or_steps, batch_size)
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/utils/generic_utils.py", line 493, in make_batches
    num_batches = int(np.ceil(size / float(batch_size)))
TypeError: float() argument must be a string or a number, not 'NoneType'

With batch_size=32 set:
% python tensorflow_bug.py
[...]
Traceback (most recent call last):
  File "tensorflow_bug.py", line 14, in <module>
    model.fit(ds, steps_per_epoch=10, validation_data=(data, targets), batch_size=32)
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py", line 652, in fit
    batch_size, steps_per_epoch, x)
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py", line 1873, in _validate_or_infer_batch_size
    raise ValueError('The `batch_size` argument must not be specified when'
ValueError: The `batch_size` argument must not be specified when using dataset as an input.

Behaviour using 1.13.1:
Without batch_size=32 set:
% python bug.py
[...]
1/10 [==>...........................] - ETA: 2s - loss: 3.0703
Traceback (most recent call last):
  File "tensorflow_bug.py", line 14, in <module>
    model.fit(ds, steps_per_epoch=10, validation_data=(data, targets))
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py", line 880, in fit
    validation_steps=validation_steps)
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 364, in model_iteration
    validation_in_fit=True)
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/engine/training_arrays.py", line 301, in model_iteration
    batches = make_batches(num_samples_or_steps, batch_size)
  File "/Users/ahoereth/repos/AutoECG/.venv/lib/python3.7/site-packages/tensorflow/python/keras/utils/generic_utils.py", line 488, in make_batches
    num_batches = int(np.ceil(size / float(batch_size)))
TypeError: float() argument must be a string or a number, not 'NoneType'

With batch_size=32 set:
% python bug.py
[...]
10/10 [==============================] - 0s 21ms/step - loss: 1.0369 - val_loss: 1.1500

I have tried on Colab with TF versions 1.14.0-rc1 and 1.13.1 and was able to reproduce the issue  on 1.14rc1 but not in 1.13.1 as mentioned in the description. Thanks!

Any news on this issue? I think this is an undocumented breaking change?

@ahoereth I modified your code little bit by making validation data as a dataset instead of numpy.ndarray. In this case, you don't need to provide batch_size=32 as an input to model.fit function. Please take a look at the gist. Thanks!

Automatically closing this out since I understand it to be resolved, but please let me know if I'm mistaken.Thanks!


Review 2648

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!


Explain various concepts related to KubeSlice in a dedicated page

Some of the terms like

Slice boundaries
RFC1918 network addresses
Architectural components from this page

could benefit from a page on the website.

I would like to work on it!!

Hello @sayantani11 , please go for it. I've assigned the issue to you :) Thank you!

@divya-mohan0209 One question- where will that page go in the website?

Hello @sayantani11 , it will be the first section after overview.

@sayantani11 Did your PR get merged? If not, @divya-mohan0209 I would like to work on the issue

We have revamped the overview section from version 1.0.0, So, I'm closing this issue.
https://kubeslice.io/documentation/open-source/1.0.0/


Usage without Facade

Do you have an example of using this without using a facade? I usually avoid using facades in favour of dependency injection as it makes unit testing a bit easier, so I was wondering how I can allow the service container to inject the Salesforce object?
I tried injecting Davispeixoto\Laravel5Salesforce\Salesforce and however it throws an error "Call to a member function __setSoapHeaders() on null" from davispeixoto/force-dot-com-toolkit-for-php/src/Davispeixoto/ForceDotComToolkitForPhp/SforceBaseClient.php line 235

I just injected Davispeixoto\Laravel5Salesforce\Salesforce and it's working fine

I achieved this by injecting Davispeixoto\Laravel5Salesforce\Salesforce and adding $this->app->alias('salesforce', Salesforce::class); in my Service Provider boot method

Thx for sharing, @nicklee1990


cc-wrapper does not pass -stdlib=libc++ as part of cxxflags

The Bug
Currently cc-wrapper does add -stdlib=libc++ to libcxx-ldflags but not to libcxx-cxxflags.
This leads to clang appending gcc libstdc++ header search directories to the search path which in turn breaks libc++ headers (#include_next <...> no longer finds libc headers, as intended, but now finds libstdc++ headers).
To Reproduce
nix-shell -p llvmPackages_14.libcxxStdenv.cc --run "echo '#include <string>' | clang++ -c -xc++ -" --pure

^ results in errors that look like this:
Click to expand.
In file included from <stdin>:1:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/string:523:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__functional_base:22:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/exception:83:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/cstdlib:85:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/stdlib.h:93:
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/stdlib.h:38:12: error: no member named 'abort' in namespace 'std'
using std::abort;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/stdlib.h:39:12: error: no member named 'atexit' in namespace 'std'
using std::atexit;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/stdlib.h:40:12: error: no member named 'exit' in namespace 'std'
using std::exit;
      ~~~~~^
In file included from <stdin>:1:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/string:523:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__functional_base:22:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/exception:83:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/cstdlib:85:
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/stdlib.h:148:34: error: unknown type name 'ldiv_t'
inline _LIBCPP_INLINE_VISIBILITY ldiv_t div(long __x, long __y) _NOEXCEPT {
                                 ^
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/stdlib.h:149:12: error: no member named 'ldiv' in the global namespace
  return ::ldiv(__x, __y);
         ~~^
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/stdlib.h:152:34: error: unknown type name 'lldiv_t'
inline _LIBCPP_INLINE_VISIBILITY lldiv_t div(long long __x,
                                 ^
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/stdlib.h:154:12: error: no member named 'lldiv' in the global namespace
  return ::lldiv(__x, __y);
         ~~^
In file included from <stdin>:1:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/string:523:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__functional_base:23:
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/new:327:11: error: no member named 'posix_memalign' in the global namespace
  (void)::posix_memalign(&__result, __alignment, __size);
        ~~^
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/new:338:5: error: no type named 'free' in the global namespace
  ::free(__ptr);
  ~~^
In file included from <stdin>:1:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/string:523:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__functional_base:26:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/utility:236:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/compare:144:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/compare_partial_order_fallback.h:13:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/partial_order.h:14:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/weak_order.h:14:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/strong_order.h:18:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/cmath:308:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/math.h:300:
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:39:12: error: no member named 'acos' in namespace 'std'
using std::acos;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:40:12: error: no member named 'asin' in namespace 'std'
using std::asin;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:41:12: error: no member named 'atan' in namespace 'std'
using std::atan;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:42:12: error: no member named 'atan2' in namespace 'std'
using std::atan2;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:43:12: error: no member named 'cos' in namespace 'std'; did you mean 'ios'?
using std::cos;
      ~~~~~^
/nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/iosfwd:161:38: note: 'ios' declared here
typedef basic_ios<char>              ios;
                                     ^
In file included from <stdin>:1:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/string:523:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__functional_base:26:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/utility:236:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/compare:144:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/compare_partial_order_fallback.h:13:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/partial_order.h:14:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/weak_order.h:14:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/__compare/strong_order.h:18:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/cmath:308:
In file included from /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1/math.h:300:
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:44:12: error: no member named 'sin' in namespace 'std'
using std::sin;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:45:12: error: no member named 'tan' in namespace 'std'
using std::tan;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:46:12: error: no member named 'cosh' in namespace 'std'
using std::cosh;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:47:12: error: no member named 'sinh' in namespace 'std'
using std::sinh;
      ~~~~~^
/nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/math.h:48:12: error: no member named 'tanh' in namespace 'std'
using std::tanh;
      ~~~~~^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.

The Fix
Passing in --std=libc++ inhibits clang from adding the gcc libstdc++ header directories to the search path and fixes this error:
nix-shell -p llvmPackages_14.libcxxStdenv.cc --run "echo '#include <string>' | clang++ -c -xc++ - --stdlib=libc++" --pure

Running with -v lets us compare the list of search directories:

clang++ -c -xc++ - -v:#include "..." search starts here:
#include <...> search starts here:
 /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include
 /nix/store/azb7l7hzicncdyz1phnjwy0pjb97ikjl-libcxxabi-14.0.1-dev/include
 /nix/store/870xakmqvll2paqgn0s2x959wkxd02jm-compiler-rt-libc-14.0.1-dev/include
 /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1
 /nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0
 /nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/x86_64-unknown-linux-gnu
 /nix/store/2lr0fc0ak8rwj0k8n3shcyz1hz63wzma-gcc-11.3.0/lib64/gcc/x86_64-unknown-linux-gnu/11.3.0/../../../../include/c++/11.3.0/backward
 /nix/store/n93g9kfx1sy6mpiy4z7hzjyf9r7qn9jz-clang-wrapper-14.0.1/resource-root/include
 /nix/store/k3d8wqlsnmm5270zd19cbs26g7wifxj6-glibc-2.34-210-dev/include

clang++ -c -xc++ - --stdlib=libc++ -v:#include "..." search starts here:
#include <...> search starts here:
 /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include
 /nix/store/azb7l7hzicncdyz1phnjwy0pjb97ikjl-libcxxabi-14.0.1-dev/include
 /nix/store/870xakmqvll2paqgn0s2x959wkxd02jm-compiler-rt-libc-14.0.1-dev/include
 /nix/store/rd7zvb5ic6agrsfkp06w3bahflzb35vg-libcxx-14.0.1-dev/include/c++/v1
 /nix/store/n93g9kfx1sy6mpiy4z7hzjyf9r7qn9jz-clang-wrapper-14.0.1/resource-root/include
 /nix/store/k3d8wqlsnmm5270zd19cbs26g7wifxj6-glibc-2.34-210-dev/include

This (i.e. adding echo "-stdlib=libc++" >> $out/nix-support/libcxx-cxxflags here) seems like an uncontroversial change to me but I thought I'd open an issue about this first just in case I'm missing something obvious or just not using the wrapper compiler correctly.
Notify maintainers

CODEOWNERS for cc-wrapper points to @Ericson2314; apologies if this is incorrect.
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.63, NixOS, 22.11 (Raccoon), 22.11.20220901.2da64a8`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@rrbutani Could you please file the PR? Thank you!

Looking into this again today; it seems like #151879 was actually a symptom of this issue (libstdc++ paths were being added when trying to use libc++, breaking include_next).
--gcc-toolchain is what causes clang to include these directories; --stdlib=libc++ inhibits this.
Both removing --gcc-toolchain or adding in --stdlib=libc++ are sufficient to fix this specific issue (i.e. compiling C++ code with clang++ with libc++).
As noted in cc-wrapper, dropping --gcc-toolchain is a little fraught and probably is not something we want to do when using libc++ because it also affects compiler-rt vs. libgcc_s:
https://github.com/NixOS/nixpkgs/blob/d23c0dae4f091d206f08c391f86b40e51bea9862/pkgs/build-support/cc-wrapper/default.nix#L312-L325
#151879 was fixed by #153963 which went the route of patching clang to add -nostdlibinc unconditionally to the command line arguments, effectively making it so that --gcc-toolchain does not influence the include directories at all (this doesn't break the clang++ + libstdc++ use case because cc-wrapper handles adding in the appropriate flags for that: 1, 2). This also has the additional upside that clang will be inhibited from searching FHS directories like /usr/lib for libstdc++ headers.
Unfortunately #153963 did have the side-effect of inadvertently breaking the clang + libstdc++ workflow (note: clang, not clang++, for compiling C++ code; this is, unfortunately, something that some build tools like bazel do; we get around this in nixpkgs but it seems better to just have the nixpkgs cc/c++ behave like they do elsewhere) because cc-wrapper only appends the libstdc++ flags to cxxflags. Previously, when using clang and libstdc++, clang would fill in those libstdc++ flags in for us (when building C++ code) via the --gcc-toolchain flag; passing in -nostdlibinc "broke" this.
~(Note that the clang + libc++ flow was actually, amusingly, fixed by #153963; prior to the change, clang would fill in the libstdc++ dirs on top of the libc++ dirs because there was no -nostdlibinc to inhibit it from doing so. Unlike clang + libstdc++ the cc-wrapper machinery would effectively always add in the libc++ include and link dirs (via NIX_LDFLAGS) because of how they're added to cc.)~
EDIT: this doesn't appear to be true; we need to have the include/c++/v1 dir included which only seems to be added here (i.e. for clang++: 1, 2, 3)

I think the bottom line is that we've got a matrix of toolchains and use cases we want to support (gcc or g++ or clang or clang++, libstdc++ or libc++) and a bunch of knobs we can turn (--stdlib, -nostdlibinc, --gcc-toolchain).
Also worth noting is that as of this writing LLVM 15 does not include the fix in #153963 though it probably should; I was mistaken and #151879 is observable with llvmPackages_15.libcxxStdenv though not when using clang instead of clang++ (because -nostdlibinc isn't present and because the include dir for libc++ with include/c++/v1 isn't present, clang ends up using libstdc++ in this case, incorrectly).
I think that:

we do want the patch that implicitly applies -nostdlibinc (#153963) because it also prevents clang from searching /usr/lib (though there may be a way to achieve this with --sysroot instead)
we do want to specify --gcc-toolchain even when using libc++ instead of libstdc++
we should try to supply the C++ stdlib link and include flags (as well as -stdlib=libc++ as part of cflags) to both clang and clang++

I'm hoping this doesn't break any C code; hopefully there isn't any C code out there that's somehow banking on the compiler not having the C++ stdlib include directories
Another option is to mimic the language detection (based on file extension) that clang does and to set isCxx in cc-wrapper based on the files we see (but this seems fraught)

Ultimately cc-wrapper's desire to be in charge of managing libc and the C++ stdlib put it at odds with it's goal of mimicking the behavior of clang/clang++; clang will, upon detecting C++ source files, add in the C++ stdlib directories which cc-wrapper does not. I'm proposing we have cc-wrapper unconditionally include the C++ stdlib dirs, which, while still not an exact replication of clang's behavior, seems less likely to be observable in practice.

Apologies, this turned into a bit of a rant; I'll open a separate issue/PR for this later and try to figure out whether ^ is practical.

We’ve had to disable -nostdlibinc on Darwin because it breaks using an SDKROOT with libclang – effectively an unwrapped compiler – as part of e.g. Rust’s bindgen. After discussion with @pwaller, we’ve settled on moving the flag‐injecting patch to the wrapper instead for now, to get a target‐independent Clang build and bring unwrapped compilers close to a vanilla upstream state rather than adding this override at such a low level.
However, I still think that -nostdlibinc is the wrong approach for this problem. We try to filter out system dependencies at multiple layers in our toolchains but it’s somewhat flaky at every layer… e.g. cc-wrapper’s “purity filter” command‐line parsing hack or this. The only real solution is the sandbox, but of course that doesn’t handle development shells and is not very usable on Darwin. On NixOS there’s no /usr/include anyway, and on Darwin it’d only pick up Homebrew‐installed stuff in /usr/local/include, which I believe would still require SDKROOT to be unusually unset, like using an unwrapped compiler in a strange configuration… basically the only relevant case where all this stuff is buying us something is development shells on non‐NixOS Linux, I think. And those do matter, but… I hope we can find a less blunt implement. Honestly, development shells are kind of weird and leaky already.
Rambling comment, sorry, but I wanted to get my thoughts out. I’m very much in favour of passing -stdlib properly and I hope we can drop -nostdlibinc on all platforms, one way or another.
cc @reckenrode who had opinions on this too


Support Databricks 11.3 [databricks]

Signed-off-by: Tim Liu<EMAIL_ADDRESS>

CI script for issue: https://github.com/NVIDIA/spark-rapids/issues/7352

build

build


Allowlist filter

In #158 you discussed a blocklist filter. It would be nice to have an allowlist filter or some way to accept connections only from a given address and port pair.
In a telco use case in the initiating SDP messages, there is information about the client's sender socket like address and port. After these messages, we want to tell Quilkin to accept connections from that address and port pair and from nowhere else. This way you wouldn't have an open port for everyone, but only for the trusted one.
More info about a possible telco use-case: There is a separate control plane and a media plane. When the offer and answer come in from the caller and the callee we dynamically (via an XDS server) create a media plane pipeline for every single RTP/RTCP stream. We know where we should expect the streams from, due to the SDP messages sent previously on the control plane. We want to accept packets only from that single address port pair on the listeners(not sure how to call them here).

Just a heads up - I added a design to https://github.com/googleforgames/quilkin/issues/158#issuecomment-926967593 to join allow and block filters into a single Firewall Filter. Would love to know what you think!


boot: permission denied

Kernel download always fails with: Permission denied (0216eb3c)
This is caused by Let's Encrypt who changed their root certificates. Bundle certificates needs to be updated.

Fixed via https://github.com/threefoldtech/0-bootstrap/commit/319d6b532bd680d07a282bc830071f2bb4de3aef, https://github.com/threefoldtech/0-bootstrap/commit/3ba4988c1d30bd0aabe3c5d7ca853a2704e805e5

Fixed via https://github.com/threefoldtech/0-bootstrap/commit/319d6b532bd680d07a282bc830071f2bb4de3aef, https://github.com/threefoldtech/0-bootstrap/commit/3ba4988c1d30bd0aabe3c5d7ca853a2704e805e5


Add a noMatchSnapshots option.

See (https://github.com/storybooks/storyshots/pull/93).
Issue:
The idea is that early on in a project, when things are changing a lot, snapshot tests are not particularly useful, but it's still helpful to "smoke test" all stories to ensure they run without error.
What I did
I added a noMatchSnapshots option to just setup a test that runs each story, without checking snapshots.
How to test

Create a project, install storyshots but pass initStoryshots({ noMatchSnapshots: true }).
Add a story, run jest. Should pass.
Change the component's output, run jest again. Should still pass
Make the component error in the render function, run jest again. Should fail.

Notes:
Perhaps it would make sense to just allow passing a custom test function, to cover off cases like this and https://github.com/storybooks/storyshots/pull/76? Then storyshots is more of a generic tool to integrate storybook with Jest.
initStoryshots({
  test(renderedStory) {
     // In my case, just render without looking at the result
     renderer.create(renderedStory).toJSON()

     // For #76, shallow render + snapshot
     const tree = shallowToJson(shallow(renderedStory))
     expect(tree).toMatchSnapshot()
  }
});

@tmeasday FWIW, I prefer your genericized solution to the current proposal.

I'm having problems testing out the change. It's probably unrelated to your change. What I did:
cd storybook
npm install
cd packages/storyshots
npm link .
cd ../../examples/cra-storybook
npm link \@kadira/storyshots
emacs src/storyshots.tests.js
npm run test

With the following contents for storyshots.tests.js:
import initStoryshots from 'storyshots'
initStoryshots({ noMatchSnapshts: true })

I get the error:
  ● Test suite failed to run

    Cannot find module 'react-test-renderer' from 'index.js'

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:151:17)
      at Object.<anonymous> (../../../../../.nvm/versions/node/v6.2.1/lib/node_modules/storyshots/dist/index.js:13:26)
      at Object.<anonymous> (src/storyshots.test.js:1:173)

I tried npm i -D react-test-renderer to add the peer dependency, but still get the same error.
I suspect this is related to Lerna. @tmeasday @ndelangen  @aaronmcadam. Any help would be much appreciated.

I'm having problems testing out the change

Apologies, I didn't test the changes on the monorepo. My bad. Let me have a go at it and I'll ping you when I figure it out.

I'd name this autoUpdate and have it default value be false.

But I'm not really feeling much for this feature myself, since updating snapshots is real easy already anyway.

@ndelangen -- the idea isn't to update the snapshots, it's to "smoke test" your stories but not to snapshot them.
When you are early in development, pretty much every commit changes your snapshots, and so (in my experience) you tend to disable snapshot tests as they are too noisy. However it's pretty useful to know if a commit has broken a component (in the sense that it fails to render for some story). That's the idea of this change.
@shilman -- I tend to agree that we should do the more generic thing.

I rebased this onto https://github.com/storybooks/storybook/pull/971
Should be easy to test now @shilman.
Haven't changed the API yet though, LMK if it works for you and I'll look at that.

Closing this in favour of https://github.com/storybooks/storybook/issues/1034, simple PR to follow


cache_dir parameter for load_from_disk ?

Is your feature request related to a problem? Please describe.
When using Google Colab big datasets can be an issue, as they won't fit on the VM's disk. Therefore mounting google drive could be a possible solution. Unfortunatly when loading my own dataset by using the load_from_disk function, the data gets cached to the VM's disk:
`
from datasets import load_from_disk
myPreprocessedData = load_from_disk("/content/gdrive/MyDrive/ASR_data/myPreprocessedData")
`
I know that chaching on google drive could slow down learning. But at least it would run.
Describe the solution you'd like
Add cache_Dir parameter to the load_from_disk function.
Describe alternatives you've considered
It looks like you could write a custom loading script for the load_dataset function. But this seems to be much too complex for my use case. Is there perhaps a template here that uses the load_from_disk function?

Hi ! load_from_disk doesn't move the data. If you specify a local path to your mounted drive, then the dataset is going to be loaded directly from the arrow file in this directory. The cache files that result from map operations are also stored in the same directory by default.
However note than writing data to your google drive actually fills the VM's disk (see https://github.com/huggingface/datasets/issues/643)
Given that, I don't think that changing the cache directory changes anything.
Let me know what you think

Thanks for your answer! I am a little surprised since I just want to read the dataset.
After debugging a bit, I noticed that the VM’s disk fills up when the tables (generator) are converted to a list:
https://github.com/huggingface/datasets/blob/5ba149773d23369617563d752aca922081277ec2/src/datasets/table.py#L850
If I try to iterate through the table’s generator e.g.:
length = sum(1 for x in tables)
the VM’s disk fills up as well.
I’m running out of Ideas 😄

Indeed reading the data shouldn't increase the VM's disk. Not sure what google colab does under the hood for that to happen

Apparently, Colab uses a local cache of the data files read/written from Google Drive. See:

https://github.com/googlecolab/colabtools/issues/2087#issuecomment-860818457
https://github.com/googlecolab/colabtools/issues/1915#issuecomment-804234540
https://github.com/googlecolab/colabtools/issues/2147#issuecomment-885052636


Update migration.toml to fix invalid conversion

I've been trying to migrate using comby and I've noticed some issues and things to add on.

Awesome, thank you! Ready to merge?

@zth affirmative!


Examine tabmat

Question
https://github.com/Quantco/tabmat
May or may not be useful. Found by Isaac

Seems like it's quite a nice speedup. Nevertheless, this would require too much work for now. Also not sure whether we'd implement this in ehrapy or whether it should be another matrix type that we support in AnnData. Tabmat basically extends Numpy or Scipy arrays


Redundant ldr in cpu_reset ?

https://github.com/AsahiLinux/m1n1/blob/86d75d4dbaed3010e3c3529424413c92211fbe22/src/start.S#L133-L155
The purpose of ldr x2, [sp, #-8] line 142 does not seem clear, it could be redundant. Should it be removed ?

Added a proposed PR #317 that removes the ldr

IIRC that's just a test to make sure the stack pointer is sane and does not fault.


Log of entries/changes to db and display in admin

Have started this via https://medium.datadriveninvestor.com/monitoring-user-actions-with-logentry-in-django-admin-8c9fbaa3f442 , but quite a lot of work required, including:

'content type' display tidy
'change message' display tidy
log recording changes made outside of admin (i.e. via app frontend)

Closing as now active problems with this at the moment. The use of simple history manager has been good for recording changes and not sure anything over and above this required.


Whisper Sample not working / Memory or Filestream isn't clean audio

Describe the bug
The whisper demo doesn't work (on Apple Silicon, Unity 6)
To Reproduce
Steps to reproduce the behavior:

Import the OpenAI-Unity package
Drag Whisper sample in from the github repo
Result from OpenAI isn't very good

Now I've taken the code that used to create an output.wav file ( from the commit ), and the result is very noisy, it sounds like the conversion from AudioClip to wav jumbles the data a lot.
No wonder when sending the jumbled bytes to OpenAI, the Whisper API can't create a good transcription from it.
Expected behavior
The Whisper example works as good as in other toolsets.
Environment (please complete the following information):
Here's the created wav:
output.wav.zip

OS: macOS (arm64)
Unity Version 6000.0.25f1
Plugin Version 0.2.2

My bad, the code is not at fault, turns out when recording via my AirPods the quality sounds like this.
If I play back the audioclip immediately, it sounds as bad as in the attached zip.
If I switch to the builtin Macbook Mic, it works fine.
Must be a problem with Unity and Bluetooth headsets.


JS package that generates human readable market info data for UI

Purpose
Rundown sourced markers will not have any human readable information on them. Just machine readable data (event_id, team_id, etc.). Human readable information is needed for the UI.
Process

[ ] Pull team names and associated IDs from the rundown.
[ ] Identify market strings and outcomes needed for the various market types and or sports.

https://github.com/AugurProject/turbo/pull/155 - adds a task to smart to fetch and create a teams object which we will want to store as json and pull into the UI. This currently simply logs out the results and doesn't do anything with them.


New Chrome Process Getting Executed On Each New Test Run And Using CPU

I am running my tests on Chrome browser on client server and via jenkins my automation test build initiates on daily basis.
my tests are getting failed on random basis and when I debugged i found that each time a new chrome Tab is opened then the previous tab is not getting killed in task manager even after my use of I.closeOtherTabs() in the start of each test.
there are multiple chrome.exe getting opened when each test runs and this causes high CPU utilization and then causes tests to fail random.
Is there any way to kill old chrome.exe when a new test runs so that there are not threads of chrome.exe running and causing high CPU

Provide console output if related. Use --verbose mode for more details.

# paste output here

Provide test source code if related

// paste test

Details

CodeceptJS version:
NodeJS Version:
Operating System:window 10
Protractor || WebDriverIO || Nightmare version (if related)
Configuration file:

# {
  "output": "./output",
  "helpers": {
    "WebDriverIO": {
      "url": "https://formsviewertest.azurewebsites.net",
      "browser": "chrome",
      "windowSize": "maximize",
      "restart": false,
      "keepBrowserState": true,
      "keepCookies": true
    },
    "TabSwitch": {
      "require": "./tabswitch_helper.js"
    }
  },
  "include": {
    "I": "./steps_file.js",
    "loginPagePage": "./pages/loginPage.js",
    "manageCredentialPage": "./pages/manageCredential.js",
    "templatePage": "./pages/template.js",
    "uiHelper": "./pages/uiHelper.js",
    "fvAssert": "./pages/fvAssert.js",
    "listPage": "./pages/listPage.js",
    "fvData": "./InputData/login.js"
  },
  "mocha": {
    "reporterOptions": {
      "codeceptjs-cli-reporter": {
        "stdout": "-",
        "options": {
          "verbose": false,
          "steps": true
        }
      },
      "mocha-junit-reporter": {
        "stdout": "./output/console.log",
        "options": {
          "mochaFile": "./output/result.xml"
        }
      }
    }
  },
  "bootstrap": "./config/start_server.js",
  "teardown": "./config/stop_server.js",
  "hooks": [],
  "tests": "**/fv_*_test.js",
  "timeout": 10000,
  "name": "FV"
}

**Below is screenshot of my server that shows multiple chrome.exe**

![cpu](https://user-images.githubusercontent.com/28402736/35318496-a6ee2e6e-0101-11e8-8df5-f952f131ba14.JPG)

Thanks @reubenmiller


Correct deploy URL in DocumenterVitepress build

Thanks! This was on my list of things to try and fix, so I'm grateful for your psychic powers! 😀


Route traffic through proxies

Too many requests currently lead to temporary IP block by steam

added feature in v1.0.2


editor is changing files when there is no activity

VSCode Version: Code 1.3.1 (e6b4afa53e9c0f54edef1673de9001e9f0f547ae, 2016-07-12T13:31:10.994Z)
OS Version: Darwin x64 15.6.0

Steps to Reproduce:
I realized that the editor is changing files when there is no activity. I run my Node.js code in development using the nodemon --exec command. Sometimes this command restarts after ~10sec (repeatedly) but no changes are made (only the editor is opened, no clicking, writing). When I close the editor the process stop restarting.

I am very sorry to hear you are having issues with VS Code. I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line, execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.

Hum... I can not reproduce this issue anymore. I'll close it and reopen it if I encounter the problem again. Thanks.


Invalid authentication headers

via command-line in Ubuntu 15.10 with dataserv-client 2.1.12 I run on the command:
_dataserv-client --store_path=/media/matteo/IntoTheCloud1/.storj/store --max_size=980200788296 farm_
I get after build finish:
_```
2016-02-24 00:45:10,635 INFO dataserv_client.messaging 47: Query: http://status.driveshare.org/api/address
2016-02-24 00:45:11,235 INFO dataserv_client.messaging 47: Query: http://status.driveshare.org/api/register/Authentication address/Payout address
2016-02-24 00:45:11,526 INFO dataserv_client.api 209: Starting build
2016-02-24 00:45:12,394 INFO dataserv_client.builder 111: Resuming from height 6947
2016-02-24 00:45:12,394 INFO dataserv_client.builder 150: Minimum free
```disk space reached<PHONE_NUMBER>) for '/media/matteo/IntoTheCloud1/.storj/store'.
2016-02-24 00:45:13,211 INFO dataserv_client.messaging 47: Query: http://status.driveshare.org/api/height/Authentication address/6947
2016-02-24 00:45:13,449 INFO dataserv_client.api 223: Current height at 6947.
2016-02-24 00:45:13,450 INFO dataserv_client.api 235: Build finished
Running storj dht node on port 45515 with id 8ddcaf19608c51b290acf6bb9d5a38ac90746265
2016-02-24 00:45:13,494 INFO dataserv_client.api 150: Pinging server 'http://status.driveshare.org' at 2016-02-24 00:45:13.
2016-02-24 00:45:14,562 INFO dataserv_client.messaging 47: Query: http://status.driveshare.org/api/ping/Authentication address_

As I open the links:
http://status.driveshare.org/api/height/Authentication address/6947
or
http://status.driveshare.org/api/register/Authentication address/payout address
I get:
Registration Failed: Invalid authentication headers.
What's wrong?
Thank you in advance, I know that probably I'm the buged one and not the program

The dataserv-client calls the json api of the server. This api call is secured by an authentication header. Without authentication i could set your height to 0 to get more rewards myself...
If you call the api url without the required authentication header you will see the message.
Let me anser your next question as well. If your client works fine you should find your payout address on this list: http://status.driveshare.org/api/online/json
If you have more questions please join us on slack :)

Thanks littleskunk,
I found my Payout address on that List.

Let me anser your next question as well. If your client works fine you should find your payout address on this list: http://status.driveshare.org/api/online/json

If you have more questions please join us on slack :)
I know that GitHub is not to have a talk about this kind of problem, and I would like to join the community on slack, but unfortunately at http://slack.storj.io/ I get:
Service Temporarily Unavailable
Should i think that I'm unlucky tonight?

Yes the invite bot has a problem at the moment. I hope it will be fixed soon. https://github.com/Storj/storj.io/issues/69


Add global usage instructions to README

Useful when you want to use the hooks to validate changes in forks of projects.

Ah, after using this, it looks like some of the Process commands do not work as expected because they specify vendor/bin/ which does not resolve properly when using global installation. Some kind of relative comparison is needed.

Yep, there was mention of using global tools alongside those in vendor/bin in #27.
I see a few options:

[ ] Only use tools in the $PATH
[ ] Expose getProjectBase as a public method on VersionControlInterface
[ ] Pass in the project's base path into the StaticReivew class as configuration

IMO getProjectBase shouldn't be used to determine location of global tools...

maybe simple use composer global show -i to determine if tool is installed globally via composer ?

@shadowhand going to close this for now.

:(

@sjparkinson I'd like to see this happen sooner rather than later. Do you think we could use a comparison of __DIR__ in StaticReivew with getcwd() to determine if we are in global mode?

I'd like to see this happen sooner rather than later. Do you think we could use a comparison of DIR in StaticReivew with getcwd() to determine if we are in global mode?

That's a good idea, I haven't sat down to think about this issue much yet but would love to resolve it.

Then, if we all agreed that this would be nice and we want to have it - why the issue has been closed?

Reopening this, now that we know it is possible.


Add subtle border to the avatar to match the CSS implementation

The avatar was missing a border compared to the css implementation.

The change is best visible in dark modes with @colebemis's avatar.
<Avatar src="https://avatars.githubusercontent.com/u/4608155?s=100&v=4" size={50} />

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Hi!
Do you have access to the chromatic build to make sure all visual changes are intentional: https://www.chromatic.com/build?appId=61a90feace7802003a4d9c45&number=1009

Hey @siddharthkp
I forgot about Chromatic, it's so good! 🥹 The visual changes look good.
I've tried to update the snapshots but I got into an error:

I'm not 100% sure, but I think running npm install and npm build before updating snapshots should fix this


Updates for travis.yml

Update for travis.yml to release to ruby gems
Updated the gemspec with some out of date information
Travis announcing releases to #rackspace, #chef-hacking, #openstack-chef on freenode.

The flip side is we need a green build now. Can we get those spec failures fixed?


Extract time picker typography from theme

/cc @Hixie

@abarth may have opinions, he wrote this

LGTM


Update toc.md

Looks a bit odd to have the full name on the TOC:
https://docs.microsoft.com/en-us/dotnet/articles/visual-basic/programming-guide/concepts/assemblies-gac/

Yes, they need to be handled on a separate PR. But having long titles on TOC is not helpful IMO and it doesn't impact search, so it shouldn't affect the ability of people "distinguishing" them.