Devise not finding reopened User class

I have an app that has centralised business logic that is stored in a single gem. I'm attempting to introduce this into the Rails app so that I can DRY things up.
Having said this devise seems to fall over once this gem is introduced.
The gem includes a basic User class with no dependency to Devise what so ever.

class Foo::User
  include Mongoid::Document
  include Mongoid::Timestamps

  field :username
  field :email
end

The application on the other hand does, so I have reopened the class and add the following.

class Foo::User
  include Mongoid::Document
  include Mongoid::Timestamps

  devise :database_authenticatable, :registerable, :token_authenticatable,
    :recoverable, :rememberable, :trackable, :validatable, :omniauthable
end

After removing the private gem everything passes again.
I have done some digging and it seems the devise finds the Foo::User class first and runs respond_to? on this instead of the version in the models directory.
You can find the stacktrace here: https://gist.github.com/1299110

Devise just uses Rails loading. It does not interfere or change it in any way. In any case, if you want me to take a further look at it, please provide a failing sample app to Github and I will gladly take a look.

Sorry, I could of written that last bit better.
I don't think Devise is changing anything but it does seem to choke as soon as I introduce the gem, which is working in another ruby app.
I tried to reproduce the issue with a basic gem but had no joy but the repo demonstrates the current issue I am experiencing.
http://github.com/baphled/dummyapp
Thanks in advance for looking into this, any light on this would be a great help

I have taken a look at the app, but as you said, I couldn't reproduce the issue. I am closing this but I would gladly reopen if more information to reproduce the issue is provided. Thanks!

I've just come back to this, thought maybe I've a step back would help illiminate the issue, no such luck.
The dummyapp definately reproduces the error perfectly for me, I've tried it on a couple of machine and had the same results.
When mapping.to.respond_to?(:devise) is called the reopened class is not loaded (not sure why), leading to this particular issue.
I'm more than happy to give more information and access to the private repo, though I don't totally understand the loading process enough to be able to figure this out why the reponed class is not being found prior to this check.


Support for module path in Java module system

Referring to the Q&A on StackOverflow - How to fix the issue with Java 9 Modularity after adding a dependency on mongock?
Seems like the upgraded versions in the 4.x.x somehow manage to get away with this using different dependency and improved package structures. But is there a plan around explicitly introducing Automatic-Module-Name under the META-INF/MANIFEST.MF of the library to start off with and then maybe modularise it later?

Hello @namannigam ,
we have plans to make some adjustment to adapt to new versions of Java, but we have some other priorities first. This might be coming after transactions and some other cool features we are working on.
Thanks


Support ARRAY fields

I think the current implementation doesn't support yet passing lists as parameters so we can use PostgreSQL Arrays

Arrays should work, but I now see we have fewer tests for them than I thought https://github.com/erleans/pgo/blob/main/test/pgo_basic_SUITE.erl#L402
How are you trying to use them? Wondering if its a casting issue.

I'm using it with gleam_pgo, which needs something like a pgo.array so I can pass array parameters when inserting into array fields

Is it that it doesn't work if you give PGO a list, or is it that a list function needs to be added to gleam_pgo? If the latter that can be easily done 👍

Gleam_pgo doen't have a pgo.list() and I don't know if it exists on PGO

PGO doesn't have argument functions as it is dynamically typed, one can pass in any value as an argument. If it's just the Gleam function and PGO supports arrays we can move this issue to the Gleam project or just add that function.


cabal2nix unusable from within a nix-shell

Issue description
cabal2nix uses nix-prefetch-url which is unusable within a shell (or inside runCommand if we're doing something gross)
[nix-shell:~/programming/alphasheets/nix]$ cabal2nix https://github.com/qrilka/xlsx.git
error: Nix database directory ‘/nix/var/nix/db’ is not writable: Permission denied
Initialized empty Git repository in /run/user/1000/git-checkout-tmp-v8NioZ1R/xlsx/.git/
remote: Counting objects: 63, done.
remote: Compressing objects: 100% (58/58), done.
remote: Total 63 (delta 1), reused 22 (delta 0), pack-reused 0
Unpacking objects: 100% (63/63), done.
From https://github.com/qrilka/xlsx
 * branch            HEAD       -> FETCH_HEAD
Switched to a new branch 'fetchgit'
removing `.git'...
error: Nix database directory ‘/nix/var/nix/db’ is not writable: Permission denied
abort: HTTP Error 406: Not Acceptable

^Csvn: E200015: Caught signal
failed to open trace file: [Errno 2] No such file or directory: ':/.bzr.log'
Not checking SSL certificate for github.com.
bzr: ERROR: Not a branch: "https://github.com/qrilka/xlsx.git/".
*** f

Steps to reproduce
Seen above
Technical details
{
  "owner": "NixOS",
  "repo": "nixpkgs-channels",
  "rev": "09c91f5fea73746c2ea85b37834e7ec450923d18",
  "sha256": "1gxcz8hcq57b1gra91skqhcw2h9ifq9y3bgibhykfmszf0wmjsjv"
}

work-around: set NIX_REMOTE="daemon"

This issue is difficult to fix. cabal2nix downloads the source code of those packages it needs to convert so that it can determine the hash of the given package. Now, we use the nix-prefetch-xxx scripts to do that, because these tools have the added benefit of inserting these downloads into the store, therefore they don't need to be downloaded again when the generated Nix expression is eventually built.
The downside is, of course, that these programs don't work without a functioning Nix installation.
If we would like to make sure that cabal2nix can do these things without Nix, then we'd have to implement the appropriate downloaders ourselves and add them to the executable, so that it works stand-alone. Of course, this would also lose the neat caching feature.
Personally, I fee that it's reasonable to assume that someone running cabal2nix has a working Nix installation, so I'm not particularly motivated to make that change. Also, I want to spend my time hacking on this tool worrying about how to map Cabal to Nix. I'd rather not spend my time worrying about how to download remote code archives and repos using half a dozen VCS systems.
Anyhow, these are my 2 cents. I'm not sure how other see this?
Maybe the answer is to document the need for NIX_REMOTE="daemon" somewhere?

Perhaps I should have outlined my use-case as motivation.
Currently we have a Haskell project that developers use stack on. However as we all know, as far as nix is concerned, stack-things are just blobs. Notably if you want to enable developers to use stack but want to use nix for some things (deploys, CI, builds), you have to meet half way somehow.
I met half way like this. Start with an LTS package set matching LTS package set used in project. This is easy enough, nixpkgs does this already. Stack has a feature where you can however stray from this LTS package set and specify packages with specific versions from Hackage. Question is how to accommodate for this without developers having to change nix expressions. My nix solution is as follows.

runCommand a Haskell program that parses stack.yaml, extracts these versions and spits out expressions that are themselves runCommand
in these runCommands call cabal2nix on those packages and make $out the generated expression. Indeed I set NIX_REMOTE = "daemon"; in the env of this runCommand as necessary.
in my package set my overrides just become haskellPackages.callPackage run-command-output-expression

However this is not enough. Even if NIX_REMOTE=daemon, nix-daemon might not be running. In my experience with developers so far, it's not.
There are couple of ways forward here:

document NIX_REMOTE and nix-daemon requirement: I think this is quite bad because cabal2nix just won't work in some use cases without it
modify cabal2nix such that it doesn't use nix-prefetch-url (if given some flag): we lose cache of the few MBs the Haskell package might be but everything works regardless of env
modify nix-prefetch-* so that it does not copy to the store if given some flag and allow cabal2nix make use of that functionality

My favourite is the second option but I understand your reasoning for not wanting to do that too. Last option would be nice but depending on implementation, the most work.
I hope you can at least see why this is a problem: nix-build suddenly becomes "hey why doesn't it work, I didn't read the doc, make it work" or some ugly wrapper scripts (with same issue).
In the end it's your call.

@Fuuzetsu, are you aware of the callHackage function?
There's also callCabal2nix and any other goodies in pkgs/development/haskell-modules/make-package-set.nix.
Maybe that helps?

I did not know of these things, sounds useful, will check it out

So following is true on my NixOS using Nix 1.11.10:
$ nix-shell -p bash --run "echo $NIX_REMOTE"
daemon

it's not true for developers withoutNixOS at least by default

It's not true for developers without NixOS at least by default.

People who install Nix and the nix-daemon on a non-NixOS host system ought to make sure that their user shells are  configured correctly, IMHO, i.e. by means of /etc/profile or other, similar mechanisms. I don't believe that configuring nix-shellis the responsibility of cabal2nix, nor do I believe that cabal2nix should jump through hoops to function in an incorrectly configured environment.

If it's not a usecase you want to support then feel free to close as WONTFIX.

It could at least provide a user message when NIX_REMOTE is not correctly set to instruct what action should be taken.

Well, I guess nix-prefetch-url could suggest setting NIX_REMOTE if it runs into permission problems accessing the store. I don't think cabal2nix should deal with this issue.


Add BMPlayer by @BrikerMan to Video section

Project URL

https://github.com/BrikerMan/BMPlayer
Description

Add BMPlayer by @BrikerMan to Video section
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 8 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

  :no_entry_sign:
  Found 6 link issues

Link issues by awesome_bot

Line
Status
Link

1263
301
https://github.com/mrugrajsinh/MVMaterialView redirects tohttps://github.com/TheMrugraj/MVMaterialView

1780
301
https://github.com/mrugrajsinh/MVAutocompletePlaceSearchTextField redirects tohttps://github.com/TheMrugraj/MVAutocompletePlaceSearchTextField

1974
301
https://github.com/neonichu/fastlane-plugin-appicon redirects tohttps://github.com/KrauseFx/fastlane-plugin-appicon

2247
503
https://www.manning.com/books/hello-swift

2248
503
https://www.manning.com/books/ios-development-with-swift

2249
503
https://www.manning.com/books/anyone-can-create-an-app

  Generated by :no_entry_sign: danger

Thanks~


Add golden file comparison to unit tests

Add golden file comparisons to unit testing.  This allows modifying code related to YAML generation with simplified unit testing to ensure that there have been no unexpected changes.

I'd like to get this in before any of my other changes related to YAML generation, as this will greatly simplify testing.

I am still digesting, but can you briefly explain the _True / _False roles?

I am still digesting, but can you briefly explain the _True / _False roles?

In reference what?

I am still digesting, but can you briefly explain the _True / _False roles?

In reference what?

I just mean I don't understand the _True and _False dir naming for the golden_files, is that related to the ocs_pvc check?

I am still digesting, but can you briefly explain the _True / _False roles?

In reference what?

I just mean I don't understand the _True and _False dir naming for the golden_files, is that related to the ocs_pvc check?

Yes.

I've changed the naming to make it more clear what the _True and _False refer to.

@jhopper please review the doc changes in particular to check whether it's clear what additional step you need to do when you add a workload.

I've changed the naming to make it more clear what the _True and _False refer to.

Would it be possible to limit the _True _False differences to only the handful of yamls that create a PVC? Just thinking reducing the number of golden_files to change if we need to change the golden reference itself.

I've changed the naming to make it more clear what the _True and _False refer to.

Would it be possible to limit the _True _False differences to only the handful of yamls that create a PVC? Just thinking reducing the number of golden_files to change if we need to change the golden reference itself.

There's a single command to regenerate all of them.  I deliberately want to minimize the number of special cases.  Anything in the future we do with storage will need this flag.

Ok I can understand wanting different buckets for storage configs, although keep in mind we don't use (anything beyond ephemeral) storage for uperf, stress-ng.
Docs look clear to me, thanks for that PYTHONPATH note :)

Yes, but when we do fio we will.  The cost is minimal, just the extra files.  Again, I don't want to encapsulate knowledge of the underlying benchmarks in the unit test.

@RobertKrawitz, We dont want to add all this files to unittest. Unittest MUST be a light test for PR and MAIN. Lets discuss it offline.

@RobertKrawitz Lets discuss the rational behind all this files.

Per discussion, the goal is to detect unintended changes to the YAML files easily.  I have updated the documentation to more clearly explain this, along with what actions developers need to take.

/LGTM


Pricinple of attention visualization

Firstly, this is a maningful code. But I want to understand the principle of the code. I do not understand why you divided into patches.
Can you tell me the details of this code? looking forward to your reply!

the reason about "patches", you can refer to vision  transformer where introduce how the image is divided into patches

Hi, please refer to the vision transformers paper for more details about the architecture.
https://arxiv.org/abs/2010.11929


adding support for running/installing chromedriver on MacOS

I am trying to minimize the effort for Flutter for Web engineers for running/writing integration tests.
Since most of the team is using MacOS day to day, I'm adding chrome driver support to MacOS.
I'm using this repository also from felt, I will first fix here before I sent out the felt PRs.

LGTM, but we should start running tests regularly on a CI. I'd be worried depending on it from engine or flutter otherwise.

LGTM, but we should start running tests regularly on a CI. I'd be worried depending on it from engine or flutter otherwise.

FYI: There are other repo's like this one. They are heavily used as utils for main repos (flutter, engine) and they have no tests.

LGTM, but we should start running tests regularly on a CI. I'd be worried depending on it from engine or flutter otherwise.

FYI: There are other repo's like this one. They are heavily used as utils for main repos (flutter, engine) and they have no tests.

added this one: https://github.com/flutter/flutter/issues/52575


Repeat button does not behave how it should

Amplitude.js Version
3.0.0
Issue description
The player on the Amplitude website does not respect the repeat functionality. There are two issues:

The repeat button does not stay on when it is in shuffle mode
The playlist repeats when it is not in repeat mode

Steps to reproduce the issue

Visit https://521dimensions.com/open-source/amplitudejs/
Click play and keep pressing the next button until playlist end

What is expected?

The playlist should stop if repeat is off
With SHUFFLE on and:

REPEAT off: Songs should be shuffled until the end of the list then stop
REPEAT on: Songs should be shuffled forever

Link to where issue can be reproduced
https://521dimensions.com/open-source/amplitudejs/
Additional details / screenshots

VIDEO OF ISSUE: https://www.dropbox.com/s/38fmfyynegy3p7n/AmplitudeRepeat.mp4?dl=0

Im playing now with player:
https://521dimensions.com/open-source/amplitudejs
And I press repeat button but it dont repeat a track


purest -rs --pyargs chempy ERROR

I've been unable to pip install chempy on my Mac running Python 3.7.  After I run the pip install command.   I run purest -rs -pyargs chempy and get the following error:
`====================================== test session starts =======================================
platform darwin -- Python 3.6.6, pytest-3.7.4, py-1.6.0, pluggy-0.7.1
rootdir: /Users/skmcneill, inifile:
plugins: remotedata-0.3.0, openfiles-0.3.0, doctestplus-0.1.3, arraydiff-0.2
================================== no tests ran in 0.00 seconds ==================================
ERROR: file or package not found: chempy (missing init.py?)
`

You say that you're running python 3.7?
In that log it looks like it's running pytest using 3.6.
If you installed it for 3.7 you can run pytest this way:
$ python3.7 -m pytest -rs --pyargs chempy

Does that help?


(PUP-8730) Yumrepo Parameters: report_instanceid and fastestmirror_enabled

Update the Yumrepo type to include support for the parameters report_instanceid and fastestmirror_enabled.
Both parameters are defined by default on Amazon Linux AMIs and docker images.
report_instanceid is a valid requirement.
fastestmirror_enabled is defined in Amazon Linux, and supported by Chef already, but I am not convinced it does anything. I haven't yet found any applicable yum or fastestmirror plugin source code that consumes this parameter.
https://tickets.puppetlabs.com/browse/PUP-8730

Can one of the admins verify this patch?

Hello! Sorry for the delay on this. I have a little bit more testing before I can move this forward, but I wanted to make sure you know I haven't forgotten!


I can't install deep in production. Need check instruction

Bug description
I have errors when run https://gist.github.com/suenot/7f3a505aa385b7e0be7d143c5d2723e4:
{
  "operation": "run",
  "envs": {
    "DEEPLINKS_PUBLIC_URL": "https://deeplinks.app.marketmaker.cc",
    "NEXT_PUBLIC_DEEPLINKS_URL": "https://deeplinks.app.marketmaker.cc",
    "NEXT_PUBLIC_GQL_PATH": "deeplinks.app.marketmaker.cc/gql",
    "NEXT_PUBLIC_GQL_SSL": "1",
    "NEXT_PUBLIC_DEEPLINKS_SERVER": "https://deepcase.app.marketmaker.cc",
    "NEXT_PUBLIC_ENGINES_ROUTE": "0",
    "NEXT_PUBLIC_DISABLE_CONNECTOR": "1",
    "JWT_SECRET": "'{\"type\":\"HS256\",\"key\":\"7QeCG04mzT5fkfIGMRpi4UWLxu6RExtvvu9enhkKQAXbVzFJ9y6ATYRI2+YaR/1mWMw=\"}'",
    "DEEPLINKS_HASURA_STORAGE_URL": "http://host.docker.internal:8000/",
    "HASURA_GRAPHQL_ADMIN_SECRET": "585f276cb1061a0c7708fd61e35185c80611779287b8201e",
    "MIGRATIONS_HASURA_SECRET": "585f276cb1061a0c7708fd61e35185c80611779287b8201e",
    "DEEPLINKS_HASURA_SECRET": "585f276cb1061a0c7708fd61e35185c80611779287b8201e",
    "POSTGRES_PASSWORD": "07609f230b0f87676f39267da1b86d297e45b1f13b7b82d5",
    "HASURA_GRAPHQL_DATABASE_URL": "postgres://postgres:07609f230b0f87676f39267da1b86d297e45b1f13b7b82d5@postgres:5432/postgres",
    "POSTGRES_MIGRATIONS_SOURCE": "postgres://postgres:07609f230b0f87676f39267da1b86d297e45b1f13b7b82d5@host.docker.internal:5432/postgres?sslmode=disable",
    "RESTORE_VOLUME_FROM_SNAPSHOT": "0",
    "MANUAL_MIGRATIONS": "1",
    "MINIO_ROOT_USER": "ac8c40ce924d5a5211c1ffbd30d4294cd4693dd05cbcc586",
    "MINIO_ROOT_PASSWORD": "26c9f5f2239ccab16203eb897aa6607d74e890d3d0375e63",
    "S3_ACCESS_KEY": "ac8c40ce924d5a5211c1ffbd30d4294cd4693dd05cbcc586",
    "S3_SECRET_KEY": "26c9f5f2239ccab16203eb897aa6607d74e890d3d0375e63"
  }
}
CONTAINER ID   IMAGE     COMMAND   CREATED   STATUS    PORTS     NAMES
__dirname /root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/imports
rootDir /root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks
  deeplinks:engine:error Error: connect ECONNREFUSED <IP_ADDRESS>:3006
  deeplinks:engine:error     at AxiosError.from (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/axios/dist/node/axios.cjs:836:14)
  deeplinks:engine:error     at RedirectableRequest.handleRequestError (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/axios/dist/node/axios.cjs:3086:25)
  deeplinks:engine:error     at RedirectableRequest.emit (node:events:529:35)
  deeplinks:engine:error     at eventHandlers.<computed> (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/follow-redirects/index.js:38:24)
  deeplinks:engine:error     at ClientRequest.emit (node:events:517:28)
  deeplinks:engine:error     at Socket.socketErrorListener (node:_http_client:501:9)
  deeplinks:engine:error     at Socket.emit (node:events:517:28)
  deeplinks:engine:error     at emitErrorNT (node:internal/streams/destroy:151:8)
  deeplinks:engine:error     at emitErrorCloseNT (node:internal/streams/destroy:116:3)
  deeplinks:engine:error     at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
  deeplinks:engine:error     at Axios.request (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/axios/dist/node/axios.cjs:3876:41)
  deeplinks:engine:error     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  deeplinks:engine:error     at async _checkDeeplinksStatus (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/imports/engine-server.cjs:205:14)
  deeplinks:engine:error     at async call (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/imports/engine-server.cjs:359:29)
  deeplinks:engine:error     at async /root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/call.cjs:8:3 +0ms
  deeplinks:engine:error Error: connect ECONNREFUSED <IP_ADDRESS>:3007
  deeplinks:engine:error     at AxiosError.from (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/axios/dist/node/axios.cjs:836:14)
  deeplinks:engine:error     at RedirectableRequest.handleRequestError (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/axios/dist/node/axios.cjs:3086:25)
  deeplinks:engine:error     at RedirectableRequest.emit (node:events:529:35)
  deeplinks:engine:error     at eventHandlers.<computed> (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/follow-redirects/index.js:38:24)
  deeplinks:engine:error     at ClientRequest.emit (node:events:517:28)
  deeplinks:engine:error     at Socket.socketErrorListener (node:_http_client:501:9)
  deeplinks:engine:error     at Socket.emit (node:events:517:28)
  deeplinks:engine:error     at emitErrorNT (node:internal/streams/destroy:151:8)
  deeplinks:engine:error     at emitErrorCloseNT (node:internal/streams/destroy:116:3)
  deeplinks:engine:error     at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
  deeplinks:engine:error     at Axios.request (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/node_modules/axios/dist/node/axios.cjs:3876:41)
  deeplinks:engine:error     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
  deeplinks:engine:error     at async _checkDeepcaseStatus (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/imports/engine-server.cjs:219:14)
  deeplinks:engine:error     at async call (/root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/imports/engine-server.cjs:360:28)
  deeplinks:engine:error     at async /root/.nvm/versions/node/v18.20.2/lib/node_modules/@deep-foundation/deeplinks/call.cjs:8:3 +5ms
MIGRATIONS_DIR /tmp/deep

Expected Behavior
No response
Current Behavior
No response
Steps to reproduce
No response
Workaround
No response
Environment
No response

Connected issue: https://github.com/deep-foundation/deeplinks/issues/337


nixos/screen: clean up module

Description of changes

Cleaned up the screen module. This module has not been touched for quite a while, and I thought it would make sense to make it look more like more modern modules.
Changes made:

Added enable option
Added package option for easy overriding
Changed type of screenrc option for semantic reasons
Added example to screenrc option
Added release note, urging users to set programs.screen.enable = true
Cleaned up a bit of spacing, and removed old module comments

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[X] Fits CONTRIBUTING.md.

@tomfitzhenry Yeah, that makes sense. Updated with an assertion


feat: button component with storybook docs

Description

Button component has 3 types:

Pill button - Button that contains text value. It is commonly used for call to action, tags or filters.
Pill button with icons - Button containing icon either on the left or right side of the button.
Icon button - Button is represented by just an icon without any text.

They have 3 variants - Primary, Secondary and Tertiary.
Different states a button can have are as follows: Active, Disabled, SoftDisabled (The button is currently disabled for user interaction; however, it remains focusable.)
Colors:

Positive - Green button
Negative - Red button
Accent - Blue button
Promotional - Purple button
Default - Grey button

We support the following sizes which vary based on the button type. As per our conventions guide, we shall use REM unit for button sizing.

Pill button - 40, 32, 28, 24.
Icon button - 64, 52, 40, 32, 28 and 24.

Tertiary icon button can be of 20.

Links

Tech Breakdown
Component and styling ticket
Storybook ticket

Screenshots
Pill Button

Pill with Icon Button (prefix slot)

Pill with Icon Button (postfix slot)

Icon Button (soft disabled)

Active disabled button

Closing this PR and re-opening separate PRs for component and storybook.


Blog app: add API endpoints

Milestones
In this project, my coding partner and I implemented the following project requirements:

[x] Create an API endpoint to list all posts for a user.

[x] Create an API endpoint to list all comments for a user's post.

[x] Create an API endpoint to add a comment to a post. Remember that the owner of the comment is the user that makes it; it should not be set by passing a parameter.

[x] Your API endpoints should receive JSON and respond JSON as well.


[Funceble] Result of execution

Hello @StevenBlack ,
As you (may) know, I released and still continue to develop Funceble.
I wanted to help, so you can find here the result of my last execution against the following list.
As it's marked as "occasionally" for the update frequency,
I also published this message/issue to AdAway's repository .
adaway.github.io@AdAway  ==> AdAway :x:
Have a nice day/night.
Cheers,
P.S: I plan to test all list of your "Sources of hosts data unified in this variant" section.

Hi there,
Just released the results with the following.
As it's marked as "occasionally" for the update frequency,
I also published this message/issue to AdAway's repository .
hosts.extras--add.2o7Net@FadeMind ==> add.2o7Net :white_check_mark:
No inactive domains found with this list.
Cheers

Hi there,
Just released the results for the following.
As it's marked as "occasionally" for the update frequency,
I also published this message/issue to FadeMind repository .
hosts.extras--add.Dead@FadeMind ==> add.Dead :x:
Cheers

Hi there,
Just released the results for the following.
As it's marked as "occasionally" for the update frequency,
I also published this message/issue to FadeMind's repository .
hosts.extras--add.Risk@FadeMind ==> add.Risk :x:
Cheers

Hi there,
Just released the results for the following.
As it's marked as "occasionally" for the update frequency,
I also published this message/issue to FadeMind's repository .
hosts.extras--add.Spam@FadeMind ==> add.Spam :x:
Cheers

Hi there,
Earlier today I released the following.
As it's marked as "occasionally" for the update frequency,
I also published this message/issue to FadeMind's repository .
hosts.extras--add.Spam@FadeMind ==> add.Spam :x:
Cheers
Badd-Boyz-Hosts@mitchellkrogza ==> Badd-Boyz-Hosts :x:

Hi Nissar @funilrys thanks for this.
Help me understand, can you summarize what does Funceble does?  Are you suggesting any action is required here?  Thanks for clarifying.

Hi @StevenBlack,
Funceble test if a domain or a list of domain is/are still active!
Here's a short summary:

ACTIVE

Date of expiration accessible per WHOIS-Server (whois command)
nslookup don't return "server can't find domain-name.me: NXDOMAIN"

INACTIVE

whois server don't return anything or date is unreadable
nslookup return "server can't find domain-name.me: NXDOMAIN"

INVALID

Domain extension has an invalid format or is unregistered into IANA Root Zone Database.

You simply can exclude/remove the domains under the INACTIVE directory of each list @mitchellkrogza already did for his list :smile:

@funilrys  In your experience... I wonder: how is "inactive" prone to Type II errors –  a "false negative"?
I'm guessing the whois test might not be great, since all whois servers practice some form of rate-limiting on requests.

@StevenBlack That's why I test first whois then test nslookup if it doesn't return anything. I don't rest only on whois.
To make it simple
Whois true && nslookup true ===> active
Whois false && nslookup true ===> active
Whois true && nslookup false ===> active
whois false && nslookup false ==> inactive
But thanks for pointing out the Type II errors, you're right in a certain way...
I think I have to find a third way in order to be sure...

@funilrys @StevenBlack IMO good idea is MERGE this functionality directly in updateHostsFile.py code but main code is now in BASH: https://github.com/funilrys/funceble/blob/master/funceble

@FadeMind It's planned to write a python version but @mitchellkrogza like to say, I'm a bash basher :smile:

In betwenn @FadeMind It's not a good idea at all to merge it cause:

the file you're talking about gonna take too much time to execute if I implement it to it.

updateHostsFile.py don't read the domains one by one.

Hello Steven @StevenBlack,
I may not have the experience you have as you previously said but I wanted to update the representation of the logic of funceble which is much clear now than the one I gave you in March.
I wanted a short expert opinion so can you tell me what you think about it?
Thank you.

Hi Nissar @funilrys
This seems like a good start.  It appears like a reasonable approach.  But I think whois is the weak link in all this.  Not much can be done about that.  Maybe the flow should bypass whois if it times out or returns an error.

@StevenBlack having very good results with funceble on this repo - https://github.com/mitchellkrogza/The-Big-List-of-Hacked-Malware-Web-Sites and run it against very large lists with over 5900 entries and it seems to work like a charm. @funilrys been doing some great work on this and I am sure over time he will perfect it even more.

Thanks for the concrete answer Steven @StevenBlack.
Funceble already has an integrated time out function written for all call of whois which is set by default to 30 seconds but can be changed by the user with the -t argument. For example, for the big list like The Badd-Boyz-Hosts the time out under dead-hosts is set to 1 second instead of 30 seconds.
For the error part, I assume that if we don't match the expiration date there is an error but I may have to review this part...

Thanks for the support Mitchell @mitchellkrogza.
Be sure that funceble will continue to be developed in order to be more efficient and perfect!

Thanks again to both of you.
Have a nice day/night
Cheers,
Nissar


Enhancement: Move BF and GF's voice to vocals track in Tutorial song

BF and GF's voices from the Tutorial song should be move to vocals track, so the instrumental theme will use music without any voices.
tutorial.zip

Already done for now.


Add args_override_self

#34

Thanks @bryceberger! Will include this in next minor version bump later today :]


Screenshot plugin: fix screenshots being taken of player when setting not selected

closes #11184

fixed in edcfeb25fdf99d47d6fce8b9b7c4eaa30f85b687


[BUG] Select component numbers are sent as string when posting to dataProvider

Describe the bug
When using the Select component to select a value from a related table the related item id value is a number. When saving the change on either the Edit or Create page it is passed to the dataProvider through variables as a string.
Steps To Reproduce

Go to edit page for a resource that has a related resource
Use select to select option
Click Save
Monitor variables in dataProvider

Expected behavior
Numbers to be passed to dataProvider as numbers with out quotations ie 42 NOT '42'
Screenshot
https://i.imgur.com/cE1Bd91.png
https://i.imgur.com/EiXo2Yp.png
Desktop

Windows 11
Chrome
"@refinedev/core": "^4.28.2",
"@refinedev/chakra-ui": "^2.5.4",
Custom dataProvider

Mobile
No response
Additional Context
No response

Additional details, when first loading the edit page if save is pressed the related ids are all numbers. If a new option is selected using the select component then the number becomes a string.

I confirmed that using the Ant framework does not exhibit the same behavior. In Ant the id is send to the customData provider as a number not a string.

Hello @silentnyte,
Probably chakra-ui using native HTML select element and HTML select element works with string.
You can change form data before submitting.

This is a pretty typical database design pattern, related table linked by numeric id. Is this something that the Inferencer could detect and add automatically?


[Request]: Show staking information in the explorer

Task description
At the moment it is not possible to see the staking rewards and participation per address in the explorer. It would be great to include the staking rewards and the participation per address.
Statistics about the top staked addresses and the total staked iota amount would also be interesting.
Requirements

Show staking per address
Show participation per address
Show statistics about top staked addresses and total staked iota

Acceptance criteria

Show staking per address
Show participation per address
Show statistics about top staked addresses and total staked iota

Creation checklist

[ ] I have assigned this task to the correct people
[ ] I have added the most appropriate labels
[ ] I have linked the correct milestone and/or project

@luca-moser Wdyt about this ? Staking stats on chrysalis.

I am not sure we want to implement a per address view of how much it is staking/participating for now. A view for the ongoing votes and staking events might be something we want to look into but it doesn't have any priority right now.

Closing as stale.


RCon Support for Ark Survival Evolved

I'd just like an option to be added to enable Rcon for ark. I was able to make-shift get it working, but it would be nice for it to be a standard feature.

You can already i think so.
http://www.ark-survival.net/en/2015/07/09/rcon-tutorial/
Lines 36-37-39
https://github.com/dgibbs64/linuxgsm/blob/master/ARKSurvivalEvolved/cfg/lgsm-default.ini

Is the config file enough to enable rcon, or does it have to be a start parameter ?
I don't own that game so i can't test it out.

Checked into config and it is do-able. Thanks.


JAX JIT does not work

JAX JIT is not supported due to a dynamically shaped array.
ops::project::fwd::xla computes the number of intersections between tiles and Gaussians (num_intersects).
This value is used in ops::rasterize::fwd::xla as the length of a sorted map of intersects to Gaussians (gaussian_ids).
gaussian_ids is also needed in ops::rasterize::bwd::xla, which is where the problem arises: a dynamically shaped array needs to be present in JAX-side code.

gsplat's kernels will probably need to be changed to solve this.
Some discussion on possible solutions: https://github.com/nerfstudio-project/gsplat/issues/175

I think the easiest way to fix this right now is to simply recalculate gaussian_ids in ops:rasterize::bwd::xla.
This means that ops::rasterize::fwd::xla will not need to return gaussian_ids, and ops:rasterize::bwd::xla will only need statically sized inputs.
This does mean duplicate computations will happen on the forward and backward pass though.


The link to the latest Azure CLI install looks like it is incorrect

This page mentions "The deployment commands changed in Azure CLI version 2.2.0" and then links to https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest to have users install the latest Azure CLI version. However, on that page, the latest version is 2.10.1, which does not have these versions of the commands available.
Is the link wrong?

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

ID: f0aca150-4747-7670-f465-7439aecc40e5
Version Independent ID: f836844c-0089-1477-57af-43a76181e5d6
Content: Deploy resources with Azure CLI and template - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/deploy-cli.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@rabrahamOpti  Thank you for bringing this to our notice, we will look into it and get back to you at the earliest.

@rabrahamOpti - I am not sure I fully understand the issue. To be able to follow the article, you need version 2.2.0 or later.  The link provided installs 2.10.1, which is a version later than 2.2.0.  Do you have any problems with running any of the commands?

Ah, that versioning syntax is extremely hard to read. It looks like you're saying 2.10 > 2.20. But it's 2.10 > 2.02
Yes, I was having trouble with the command being recognized before upgrading. I have a different error now, but it's unrelated. I just confirmed that az deployment group create  by itself is recognized, and complains about missing parameters. Feel free to close

#please-close


Remove unused code.

Fix #332

Merged to MasoniteFramework/orator fork :
https://github.com/MasoniteFramework/orator/commit/b4986bee18db06e34477f050d47c260b07abfbd9


github actions are unrunnable in openshift org, remove them

Removes github actions and dependabot(which uses GH actions) because the Openshift orgs payment plan does not support them.
USHIFT-12

/lgtm


#503 - Removed UI code from main scale repo

Completed:

Moved Scale UI code to https://github.com/AppliedIS/scale-ui
Created a Travis CI build that pushes UI artifacts to S3.
Updated Scale Dockerfile to remove all NPM building and curl in artifacts from S3
Reduced Scale image by another 500+ MB by making this change.

Future:

Move appliedis/scale-ui repo to ngageoint/scale-ui
Add support for versioned retrieval of UI artifacts marked at the same version on releases.

I'd argue that we should move the Scale UI into a different repo for the following reasons:

Multiple builds are not going to be possible with Travis and it would still slow everything down ending with no real overall performance gain.
The UI code is entirely separate of any dependency between Scale scheduler / API and so can be built, mocked and tested in isolation. I'd say at this point it gets less overall changes than we are making to Scale and all that is needed is to pull it in on build. I asked @dickmancj-mil to update the deploy step to take a tag, so it will build a tag package whenever we are ready to release.
It sets us up to start splitting components of the system into different Docker images / packages.

My proposal for the ultimate break-down of Scale components would be, by Docker image:
scale-web (UI from scale-ui repo and APIs / docs from scale repo)
scale-logstash (image for elasticsearch log routing from scale-logstash repo)
scale (scheduler and bootstrap of other containers from scale repo)


Memory leak in ViewGroupDescriptor.ElementContext

We keep track of a mapping from View to either the View itself or the Fragment that owns it.
However, we never remove items from the map ... which means we're leaking Views and Fragments until the ViewGroup itself is unhooked.

I'm thinking that a WeakHashMap<View, Object> could work here. In fact, we should be able to have just 1 of these in ViewGroupDescriptor instead of 1 per ViewGroup stored in ViewGroupDescriptor.ElementContext.
And we may want to move this WeakHashMap up into AndroidDOMProvider via the AndroidDescriptorHost interface.


Extension Functions toRequestBody(), toResponseBody()

The Java API is still there, just deprecated.

These feel like they should be asResponseBody / asRequestBody since they encapsulate rather than convert.


MST2_EU_VW_ZR_P0254T CP OFF not supported

support.zip
Hello,
Does not support CP OFF version, SWAP and SWDL works fine.
Regards

Added.


Chrome iOS link bug

Only happening on Iphone 5 in Chrome:
Site: http://asher.kdpreview.com/press
Under White Papers section:
When you click on an external link that's going to a pdf (Cape May Point Zoning) then close the new tab and go back to the page and then try to click on an internal link (Working in Avalon), the link's don't work.
Love this plugin, hopefully someone can fix this random bug.

Not sure if this issue is related, but I encountered an issue with internal links to pdf's that would crash the browser. SmoothState was trying to ajax the pdf document.
I got around this using a negative lookahead in the hrefRegex option:
hrefRegex: '^((?!\.pdf).)*$'


Do a manual run of billing for cloud costs for dedicated clusters on AWS for Jan-Mar 2024

Context
We want to have clear and detailed documentation for how various billing processes can be done manually by someone other than me. This helps remove me as a bottleneck / single point of failure, and paves the way for fuller automation.
This issue is focused simply on AWS and dedicated clusters. There will be another issue for doing the same on GCP.
Proposal

[x] Yuvi writes documentation (a runbook) for manually reporting AWS cloud costs. No process improvements here, just a literal description of the current 'process'
[x] Somoene else goes through this process, and makes changes to the document as they go to make sure it is accurate
[ ] Notify CS&S that this has been updated for January, February, and March
[ ] This document gets reviewed and merged into our documentation

Updates and actions
There's a draft of this in https://hackmd.io/liA3hR2QQmm9FqLE201RVA

This is ready to go from my perspective - the linked hackmd is complete enough for someone to take a shot at this.

Given that march is now complete, I've just updated the title to have this be done for Jan - March.

@yuvipanda I've updated AWS Jan, Feb, March, and GCP March - went smooth. I did some detail updates for AWS.
Question 1
For GCP, what do we do with [Charges not specific to a project] in the exported .csv file?
Question 2

Notify CS&S that this has been updated for January, February, and March

Is this on me or someone else? I need to be onboarded into this process to do it myself I think - otherwise I'll probably fail to convey relevant info in a way they expect to CS&S.

This document gets reviewed and merged into our documentation

I'll assume I'm the driver of this, I'll see if I can find someplace where it could make sense to integrate in our docs.

@consideRatio yay thank you! Thanks for doing March on GCP too.
I think for 'notify CS&S', right now that's:

Go to the #billing channel in 2i2c slack (this is shared with CS&S) (just added you)
Tag Elliot and let her know the spreadsheet has been updated.

That's it.

@consideRatio and re: documentation, https://github.com/2i2c-org/infrastructure/pull/3851 is the equivalent GCP PR that @GeorgianaElena made. Perhaps right below that?


Move library to new organisation

We've been discussing in the Gitter channel about creating a new parent organisation to collect various bits of code (specifically JavaScript at the moment but could be expanded to other languages or extensions of JS like TypeScript) that interact with GitHub, mainly those that use Github.js but not exclusively (Some examples might be github-release-notes — the author, @alexcanessa, has already expressed interest in the plan — and my Axis for GitHub project).
githubjs is taken and apparently active (I've contacted GitHub's support team, asking about claiming that org name due to the lack of visible activity in that org, but apparently it is active to some degree). Thus we've started compiling a list of alternative names:

github-js
github-tools
GitHub4JS
Githug
GitHubFriends

Moving the library to a new org would help enormously with administration (adding webhooks, etc.) and help ensure github.js continues to be supported into the future.
Any thoughts, @michael? Whether we do this is ultimately contingent upon whether you're willing to move the repo to a new org.
That said, for this to be successful, we also need feedback from community stakeholders. Please, if you either use or contrinute to github.js and have a strong opinion, post it here!
Thanks, everyone!

Thank you for this great summary @aendrew!

Any update on this?

We also probably need to come up with a name that doesn't start with github to make their legal team happy. The package js-github-api appears to be free on NPM, thoughts @AurelioDeRosa @aendrew?
Also: I've been chatting with @michael and he says that email is the channel he's likely to respond to (he doesn't monitor mentions/gitter/twitter/etc.)

Hi @clayreimann.
The original idea was to potentially on-board projects not necessarily written in JavaScript (e.g. PHP), that's why "GitHub tools". I appreciate that GitHub might not like the name, so I'm open for discussion. The libraries there (mine and @alexcanessa's one) will release a new update very soon, so it might be a good time to revamp the idea of a team. @andrewholsted might be interested in releasing a TypeScript version of GitHub.js, so there is a good amount of work to do.

Thanks for the summary @aendrew, as @AurelioDeRosa said, our libraries are going to release new important updates.
I'm still thinking it would be very nice to have everything wrapped up in one organisation.
On my side, I'm trying to align to the main githubjs repo every time it is released.
Just remember that github complained to us about using names that start with github.
I'm also very keen to help in any of the projects that belong to the organisation.

@AurelioDeRosa @andrewholsted re: writing a TypeScript version of github.js — as somebody who's used TypeScript a fair bit at this point, and because github.js is now awesome and modern (Huge kudos to all who worked on that btw — I used it a few weeks ago and the experience was excellent), it's probably more worthwhile to just write a TypeScript definition, which can then be distributed with the library. That way there won't be any code divergence, and TypeScript users then only have one version of the library to choose from.

This sounds like a good plan. Will you be able to do it?

@AurelioDeRosa I'd planned to at one point, though this year has been an absolutely insane amount of busy thus far and don't want to commit to anything when somebody else already is considering doing so. If there hasn't been any progress in a couple months I may consider it.

Moving it to github-tools org makes the most sense to me since a number of the collaborators on this repo are also members of that organization. I like that that org's name is not tied to any specific language. If github is not keen on the name, than it's easier to just change the name of that organization with their advice when that time comes.

@AurelioDeRosa @aendrew et. al. if we want to do this we should send Micael an email. He's said that he doesn't really pay attention to GH issues, gitter, or twitter; so if we want to make this happen we'll need to ping him directly.

I think we might want to arrive to a conclusion before sending an email to Michael.

I vote github-tools org since it already exists.

This has been done long ago, so closing this issue.


graphql support

not really sure if this is needed, but if 'foo' in
foo {
  foo1,
  foo2,
  foo3,
  ...
}

should also be highlighted
(arguments) would have to be removed from (selection( field

wups, looks like i made a checkout oopsie


[request] Page(s) with reference snippets for each supported languages

Hi,
it would be awesome if a set of reference pages (or single one if not too much supported languages?) were available.
It could be used as pseudo testsuite (a real testsuite may be even better!) and, at least for me, a great tool to add new theme.
Currently, I have a theme idea that I would write but I have no idea of how/where to find exhaustive snippets for each languages :(
I assume, using the file highlight plug-in, the integration flow would be easy.

I think this can be closed, right? The Examples page is up and running and it's working great.


chore: Bump min supported Flutter version to match stable

Description

Bumps min flutter version to 3.24 (stable is at 3.24.5 already).
This allows us to finally merge flame_3d in.

Checklist

[x] I have followed the Contributor Guide when preparing my PR.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[x] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[x] I have updated/added relevant examples in examples or docs.

Breaking Change?

[ ] Yes, this PR is a breaking change.
[x] No, this PR is not a breaking change.

Should we wait until the color stuff is in stable? I think there should be a Flutter release any day now


Challenge 4 migration to extension

Challenge 4 migrated to an extension following https://github.com/scaffold-eth/se-2-challenges/issues/234
Install from create-eth repo with:
yarn cli -e scaffold-eth/se-2-challenges:ch4-extension-init

closes #241

Thanks Damu! Added small comments
Also found one bug. Steps to reproduce

add solution contract and also uncomment code on deployment file
add eth from faucet
add "1" to "tokenToEth" input. Press "Send"

Result before (text is easy to understand):
Result in this extension:
Decoded signature:
https://openchain.xyz/signatures?query=0xfb8f41b2

Thanks @rin-st for the review and great catch!
I fixed it by adding a package.json to hardhat with @openzeppelin/contracts.
The new @openzeppelin/contracts v5 uses custom errors and our getParsedError is not able to show a nice message in this case. We should add an issue to fix it.

Lgtm! Created an issue to fix that error, see above.
Thank you @damianmarti !

Also just to comment it, I received this message when I tried to yarn vercel. Never saw it before, I needed to do yarn vercel --yes:

Could you check if it asking it for new deployments? Probably you need to create new instance
Also, try it with env variables like in this PR

Could you check if it asking it for new deployments? Probably you need to create new instance, or is it asked just one time?

Also, try it with env variables like in this https://github.com/scaffold-eth/scaffold-eth-2/pull/1012

Just created a fresh instance of another extension and seems like today I'm getting another error, without the telemetry promt:

Tried with PR but still get the message, I needed to add the --yes in order to make it work

from the docs :
The --yes option can be used to skip questions you are asked when setting up a new Vercel project. The questions will be answered with the provided defaults, inferred from vercel.json and the folder name.
So as I understand it should be a question before asking for --yes. Looks like a vercel cli bug.

Could you please try one more thing for the last project:
yarn vercel telemetry disable

and then
yarn vercel

Is it still asking for --yes?

Nice job Damu!! 🙌 The challenge is working great to me.
Just added some comments with nitpick format errors in the Readme from my instance.

Thanks @Pabl0cks for the review!!
I used a clean README and pushed the changes. There was an issue with the previous one due to auto-formatting from my IDE. Please, let me know if the formatting is ok now.

I'm using Vercel CLI 37.4.2 and it's working fine. Maybe some wrong project configuration. Can you please try to do it from a fresh project?

Realized it was only happening to my Git Bash (on desktop). The rest of terminals were working well, even Git Bash on laptop.
Downloaded and installed last version of Git Bash and is working nicely on desktop without --yes prompt🙏


More verbosity when a Weaviate starts with a non-empty volume attached

When starting Weaviate with a large dataset attached, the logs aren't very verbose.
It shows:
{"action":"config_load","config_file_path":"./weaviate.conf.json","level":"info","msg":"no config file specified, u
sing default or environment based","time":"2021-11-05T12:43:44Z"}
{"action":"startup","default_vectorizer_module":"text2vec-transformers","level":"info","msg":"the default vectorize
r modules is set to \"text2vec-transformers\", as a result all new schema classes without an explicit vectorizer se
tting, will use this vectorizer","time":"2021-11-05T12:43:44Z"}
{"action":"startup","auto_schema_enabled":true,"level":"info","msg":"auto schema enabled setting is set to \"true\"
","time":"2021-11-05T12:43:44Z"}

For a long time when data is loaded into memory (in my case about 185G).
Can the output be a bit more verbose on the status of the import? Or maybe even a 503 status code indicating that Weaviate is importing data into memory?

You can set the log level to "debug" and it will print periodic status updates about the size of the vector index. That's a nice way to see how the vectors are being loaded. To do so, set LOG_LEVEL="debug" as an env var.
A status code with a proper response would be even nicer, unfortunately, that's a bigger change because currently, the REST API doesn't even start up until the persistence layer is ready. Ideally, it would start up before but block all requests. But that's something we'd have to build first.

Haha LOG_LEVEL="debug" is very verbose 😆 - thanks though

I'll close it for now

Haha LOG_LEVEL="debug" is very verbose 😆 - thanks though

That's the beauty of structured logging, every log has its own fields describing what it's about. You can simply filter for the ones you care about :) (You even get log aggregation for free if you run on managed K8s)


CPU usage is very high

When I run the pnec_vo, the CPU usage is as high as 2000%, and every frame runs for a long time, around 1000ms. I don't know why that's happening.

I checked the result file and the pose data is all 0


File format flexibility

Make the code flexible to read from different formats including json, xml, etc in addition to CSV

can I get assigned to this?

can i get this assigned

please can I get this assigned?

can i get this assigned

!assign @AdvaySanketi

Kindly assign this.

can i get it assigned

can i get it assigned

!assign @AdvaySanketi 60

!assign @stealthwhizz 60

!assign @PrithviRajDinesh 60

Pls assign this to me

!assign @PrithviRajDinesh 60

Pls assign this to me

assign me this pls

@AlaynaMonteiro please assign it to me

@AlaynaMonteiro can i get assign

@PrithviRajDinesh I would like to know your progress so far

@PrithviRajDinesh Please respond as soon as you can or else I will be de-assigning you soon

!deassign

can you pls assign this?

!assign @SamarthRamesha

!assign @ashwin1809n

!deassign

Hey can I get assigned to this issue

!assign @AdvaySanketi 60

Can I get assigned to this please?


Make channel match between charm config and bundle config

https://github.com/kubernetes/kubernetes/blob/master/cluster/juju/layers/kubernetes-worker/config.yaml#L23
Does not match the default configuration of the bundle, which defaults to 1.6/stable. This can trigger a premature upgrade notice for end users and is confusing at how they entered this scenario if the worker was ever deployed independently of the bundle. The most basic approach to resolving this would be to make the channels match and attempt to keep them in lockstep.

Same will need to be done for kubernetes-master, too

Update master, worker, and e2e.


[Python][C++] Date64 converted to Date32 on parquet

Executing the example below,

import datetime
import pyarrow as pa
import pyarrow.parquet

data = [
    datetime.datetime(2000, 1, 1, 12, 34, 56, 123456),
    datetime.datetime(2000, 1, 1)
]

data32 = pa.array(data, type='date32')
data64 = pa.array(data, type='date64')
table = pyarrow.Table.from_arrays([data32, data64], names=['a', 'b'])

pyarrow.parquet.write_table(table, 'a.parquet')

print(table)
print()
print(pyarrow.parquet.read_table('a.parquet'))

yields

pyarrow.Table
a: date32[day]
b: date64[ms]

pyarrow.Table
a: date32[day]
b: date32[day]   <------- IMO it should be date64[ms]

indicating that pyarrow converted its date64[ms] schema to date32[day]. I used the rust crate to print parquet's metadata, and the value is indeed stored as i32, which suggests that this likely happens on the writer, not reader.
IMO this does not have any practical implication because they are both dates and a 32 bit date (in days) can hold more dates than a 64 bit date in milliseconds, but still constitutes an error as the roundtrip serialization does not yield the same schema.
A broader question I have is why data64 exists in the first place? I can't see any reason to store a date in milliseconds since EPOCH.
Reporter: Jorge Leitão / @jorgecarleitao
Note: This issue was originally created as ARROW-9502. Please see the migration documentation for further details.

When go through the code, I found this problem:
    case ArrowTypeId::DATE32:
      type = ParquetType::INT32;
      logical_type = LogicalType::Date();
      break;
    case ArrowTypeId::DATE64:
      type = ParquetType::INT32;
      logical_type = LogicalType::Date();
      break;

in parquet/arrow/schema.cc. Is this by design? If it's by design, should we mark it in the doc?
@westonpace

oh, it's in the doc: https://arrow.apache.org/docs/cpp/parquet.html
@rok should we close this because it's already documented?

@mapleFU since #13847 was closed and this is documented I'd say it's safe to.


Problem with companion app

Hi,
I'm trying to setup the companion app to bridge my connected trainer.
When I launch zwift, I can see the bluetooth device, and connect to it, but as soon as I live the "devices window" and go on the riding interface, the "watt" becomme grey and it indicate the device is not available. I soon as I reopen the device windows, the powermeter and resistance device reconnect immediatly.... But disconnect again as soon as I close it.
Anyone having the same problem ??
thanks
Anthony.

This is probably not an issue regarding Linux and Zwift, probably a general android/Zwift connectivity issue.
Unless you can state that the bridge solution works with the windows version or some other platform. Then this is not the right place for this issue I'm afraid.


Frequently getting warnings even if the code formatting is perfect.

I simple created react app using 'create-react-app' command generating default boilerplate code, but the linter gives below warning message for below mentioned code below warning message :
[jshint] Unclosed regular expression. (E015)
[jshint] Unrecoverable syntax error. (50% scanned). (E041)
import React, { Component } from 'react';
import logo from './logo.svg';
import './App.css';

class App extends Component {
  render() {
    return (

Welcome to React
 // warning is here underlined in
 closing tag.

          To get started, edit src/App.js and save to reload.

    );
  }
}

export default App;

Thanks for reporting! I think this is a duplicate of https://github.com/Microsoft/vscode-jshint/issues/29.  JSHint doesn't recognize the jsx syntax of the return statement. One workaround for now would be to add /* jshint ignore:start */ before the return and /* jshint ignore:end */ after the end of the return.
Closing this as a duplicate.

@RMacfarlane  'One workaround for now would be to add /* jshint ignore:start / before the return and / jshint ignore:end */ after the end of the return.;
---> add it where ? and why this code is commented out?
ty

Any elegant solution? I think this solution quite dummy as you need to add this code segment at every render function.


Script to generate LAMMPS potential file is incorrect on main

The LAMMPS documentation points toward this script, but it is outdated. It looks like the correct script is on this branch.

Please refer to the README of the lammps plugin directory that I updated last night.
The entire lammps_plugin directory will be reorganized today

No worries - just thought I'd draw your attention to it if it was an oversight!


Touch

Hey @rbelmega,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 988fa3c0-1ab3-11ea-8ac4-55ffd8dc6fb9

Hey @rbelmega,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: a2a08d70-1ac2-11ea-8ac4-55ffd8dc6fb9


Error occurs when shifting (<) nested blocks

Running command: < + 11 + j gives error: DocumentChangeManager: Error applying neovim edits, error: Overlapping ranges are not allowed!
> + 11 + j works however.
    if (foo == bar)
    {
        foo += 1;
        if (baz == bar)
        {
            baz += 1;
        }
    }
    else if (foo == 0)
    {
        bar == 0;
    }

Thanks for this! I've occasionally seen that issue, but haven't been able to find a reliable reproduction. I can reproduce this though!
This will likely be fixed if/when I find time to rewrite the nvim -> vscode sync code.

I encounter this problem too. Just short nested codes can cause this.

Just encountered this myself.  Any suggested work around?

I'm encountering the same issue.


Remove all non system related dependencies

[x] Remove all non system related dependencies like EntityFramework, HttpClient, ...

[x] Release package

[x] Update CS.System.Utilities reference version in the following projects

[x] biz.dfch.CS.Appclusive.Scheduler
[x] biz.dfch.CS.Cimi.Client
[x] biz.dfch.CS.Scs.Cmp.ApiBroker.Client

HINT: remove system utilities stuff in References, web.config/app.config, packages.config and as well in nuspec file

Update of biz.dfch.CS.Appclusive.UI will be tracked by dfensgmbh/biz.dfch.CS.Appclusive.UI#71


Custom vertical scroll controller

Hello Werner,
I have been using your package for a while and I have a question for you. I want to provide a vertical scrollController myself to the Calendar widget. Is there any way to do so, if not then how can I achieve it by modifying current package code?

Hi, so this isn't possible currently so you would need to make changes to the package, however if you are not in a rush I'm in the process of slowly rewriting parts of the package that would make things like this possible.
Just some notes only the MultiDayView uses a scroll controller, the schedule view uses a ItemScrollController from the scrollable_positioned_list pacakge.
To use your own ScrollController

Pass in a ScrollController to the CalendarView
Pass it into the MultiDayView
Pass it into the attach function called here

I would recommend to make it nullable and not required,
as this function is called from all the different views which do not necessarily have scrollControllers.

ViewState? attach(ViewConfiguration viewConfiguration, {ScrollController? Scrollcontroller}) {...}

Pass it into the MultiDayViewState.fromViewConfiguration called here
And use it here


how to solve it

Hello, run go run from the root folder and add CGO_ENABLED=1 before go run:
CGO_ENABLED=1 go run ./examples/client-play-format-h265-convert-to-jpeg

You will also need FFmpeg libraries as written inside the example.


Support F.copy between non-ChainerX and ChainerX devices only if backprop is not required

Currently F.copy only supports transfer between non-ChainerX devices / between non-ChainerX devices.
With this PR F.copy supports trasfer between non-ChainerX and ChainerX, but only if no backprop is required.
(Originally meant for #7330, but it turned out that #7330 does not require this fix.)

Jenkins, test this please

LGTM!

Jenkins CI test (for commit 718f52ba59ee5d53cb084b74e004d0a6125fb66b, target branch master) succeeded!


Integrate CouchDB

Hey @Arkensor,
really appreciate this framework, I'm currently also thinking about a good persistence solution for the Enfusion engine and stumbled across the NOSQL CouchDB, which seems to be a great fit, because it does support HTTP calls directly which comes out of the box in the Enfusion engine now. If you don't mind, I would like to start implementing a CouchDB driver for this framework.
Cheers,
Lukas

Hey @lmarschall,
thank you for looking into this framework. I am currently writing an HTTP base driver that integrates with MongoDB as a test subject. I have discovered a few things I need to do differently in the query building so external APIs have all the info needed for complex query evaluation. I do not know yet when it's ready and it's too heavy WIP to push anything because I change everything every couple of hours >.> Maybe by the end of next week or so I might have it sorted out (even if performance is not the best then maybe). There are some caveats with using the HTTP client which you can save yourself from getting stuck on by waiting a little bit to see how I did in my basic implementation. But some updates to this will make their way into AR soon.
Single insert/delete/get by id is easy, but complex query evaluation is more difficult. You are of course free to say you only want to support the basics that are needed for e.g. the built-on-top persistency framework but a if you have experience with it and are looking maybe for a little challenge too a full implementation would of course be neat.
I am not sure how couch db handles access but keep in mind that we do not have any header params and can not even set the content type, so right now we are forced to either use a proxy to make "proper" requests to the "real" endpoint or configure e.g. couchdb in a way to accept www-form-urlencoded body data.

Hey @Arkensor,
sounds good, there is no need to hurry with your implementation. I just recently started to adopt myself with the Enfusion Engine, departing from SQF. Today I've implemented the basic CRUD implementations, here is the code to create a document:
CreateData createData = new CreateData();
createData.title = "There is Nothing Left to Lose";
createData.artist = "Foo Fighters";
createData.Pack();
Print(createData.AsString());
string contextURL = "http://admin:admin@<IP_ADDRESS>:5984/enfusion-data/";
RestContext context = GetGame().GetRestApi().GetContext(contextURL);
context.PUT(m_callbackExample, uuid, createData.AsString());

I'm still trying to wrap my head around the async await functionality in the Enfusion Engine, so it will take some time until I'm ready to produce some valid code :) I'll keep myself updated with this repo 👍

Hey @lmarschall
after lots of trial and error on the design, I was able to pretty much get everything working with MongoDB that I hoped for. The 0.9.10 or 0.9.11 update will make the RestApi script API to be more useful (fixing the _now variants, allowing for blocking calls - especially useful for unit testing. Also adding the ability to set a few headers ... finally).
But the current prototype works out fine so far so maybe it will be interesting to you to see what I did there:

EDF_WebProxyDbDriverBase.c
https://github.com/Arkensor/EnfusionDatabaseFramework.Drivers.WebProxy.Core
https://github.com/Arkensor/EnfusionDatabaseFramework.Drivers.WebProxy.MongoDB

Hey @Arkensor
thank you very much for the heads up. I'm very happy they finally got the _now endpoints working, I was following your ticket, https://feedback.bistudio.com/T166390 and was hoping to see it resolved. My current time schedule is pretty tight, but I hope to find some spare time over the weekend to take a look at your new implementation :+1:

Hey @Arkensor
one minor question, I was just looking at your code, specificically at this line, https://github.com/Arkensor/EnfusionDatabaseFramework/blob/78dded35d8114dbb2f9a92f32101d5838981454a/src/Scripts/Game/Drivers/WebProxy/EDF_WebProxyDbDriverBase.c#L290 and I was wondering at //%1:%2/%3/ if there is missing the username and password section as described for mongodb as well, mongodb://[username:password@]host1[:port1][,...hostN[:portN]][/[defaultauthdb][?options]] or can we use these as arguments?

That is not the direct connect URL to MongoDB but to the web proxy, as MongoDB servers have no HTTP API on their own.
For your own implementation, this method can be overridden to inject more info into it or build the url differently. But this is a good point, I will make the URL building into its own method so specific drivers that re-use the logic can adjust more specific aspects without the need to override the whole method.

There were changes but we can not access the GET_now result string yet. Sadly.


Proposal to add captor MQ135 and MICS6814 (Grove Multichanel)

Hi,
I have found these libraries elewhere in github (see citation in headers) I have made amendments to make it work with my Edisons. It could maybe help the community...
These libraries have (not intensively) been tested on Edison Arduino
and Edison miniBreakout.

The sensor adapted from the Seeed repo is ok (MICS6814) but unfortunately I cannot touch GPLv3 code (MQ135). Thus unless you can provide your own implementation for it I won't be able to merge.
Also, please split these into 2 separate commits and name them: "mq135: initial implementation" and "mics6814: initial implementation" respectively. Don't forget to add a signoff line using the -s flag when committing as per our contributions guide.
Thanks and let me know if you need more help.

OK
Hopefully I have done a better job, you can refuse this pullrequest and condider the new ones
Thanks

for mq135 I have quoted the origin and the GNU GPLv3 licence that may enable you to add it I suppose


Integrate CP-ITI-1253-06

Vol 3 CP 1253 integration ready for you to review.

looks good to me.

this should be ready to merge. right?

Yes, I think this is ready to merge


Add support for debug wihout additional args

Closing due to the 3.0.0 being in the works and including the changes of this PR.


Atom 1.29.0 editor has crashed on startup in windows 7

Hello
I install atom version 1.29.0 in windows 7 64 bit version. its work fine but now its crashed on startup. Solve this issue i really need this tool.

Thanks for reaching out!  As mentioned below, please fill out the template details and we can re-open and go from there.
In particular, please confirm which debugging steps you've tried and if they had any effect (safe mode, temporarily resetting to factory defaults, etc.).
And just to confirm, Atom worked fine with previous versions but once you installed 1.29 it started crashing on startup?

We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:

open in safe mode is working good but not working in normal mode. I install and reinstall the previous version and new version get same problem here.

open in safe mode is working good but not working in normal mode

This indicates a problem in a community package, or your init script. You will need to debug this yourself (disabling packages until you isolate it), and raise the issue with the relevant repo if you find it.

OK i fix this problem. I delete the folder from C:\user.atom and re launch the application is working fine.

Installed community packages are placed in this directory so deleting it basically uninstalled all the community packages you have.  If you end up reinstalling the packages you had before, you could end up with the same issue - in this case, you need to figure out which package is causing the issue like @Aerijo mentioned (you can disable all packages and enable them one-by-one for example) and then open an issue in that package's repository.


Enhanced Support for Kotlin

Some things are not working properly in Kotlin

AnnotationWrapper annotation can just be placed on PACKAGES which can't be annotated in Kotlin => Workaround via Class

Added method to FilerUtils for kotlin source file creation.
Additionally added TYPE target to AnnotationWrapper annotation. This should just be used in Kotlin code.
So AnnotationWrapper can be used for AnnotationProcessors developed in kotlin as well now.

Created a aptk-kotlin-demo project that demonstrates creation of annotation processors based on aptk and cute as well.
CUTE doesn't currently support compilation of kotlin files, this will be done next to make the aptk stack 100% kotlin compatible.


Get Wallet address and balance by keys

Sorry, earlier I asked this question in FreeTON github, but they gave me no answer. Maybe, you could do this?
I'm a newbie in TON usage. I'm trying to understand how to get my own wallet address, when I generate my wallet phrase? I have phrase, I can get public and private keys, but not a wallet address/balance.(( Could you explain me this with your cool library? Is it possible? Thank you.

@GoNextGroup sorry you should get answer from the freeton team, here we're expecting to have bug reports/improvement suggestions only.


Offer to apply GHC's suggested typo fixes

A rather significant issue with this is the coupling between the elisp code and the version of GHC. One idea is to have a table of regular expressions indexed by the GHC version. I've been testing this with GHC-8.0.1, but the message parsing will surely break with other versions.

Oo I look forward to trying this out when I get home tomorrow!
On 18 Jul 2016 19:40, "Anthony Cowley"<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/commercialhaskell/intero/pull/169
Commit Summary

Offer to apply GHC's suggested typo fixes

File Changes

M elisp/intero.el
https://github.com/commercialhaskell/intero/pull/169/files#diff-0
(47)

Patch Links:

https://github.com/commercialhaskell/intero/pull/169.patch
https://github.com/commercialhaskell/intero/pull/169.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/commercialhaskell/intero/pull/169, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAArCzDWI3FJdYri9cfcrAjZ2V9W4Ey3ks5qW7qrgaJpZM4JO9Pg
.

Not back at desk yet but it looks pretty good to me. I'll merge when I get back. One thing to check is that these rewrites are applied BEFORE adding or removing lines such as in imports and extensions. I can't tell from the mobile app but the loop should come before the other loops obviously. :+1:

A rather significant issue with this is the coupling between the elisp code and the version of GHC. One idea is to have a table of regular expressions indexed by the GHC version. I've been testing this with GHC-8.0.1, but the message parsing will surely break with other versions.

Agreed. I'll make such a variable (GHC version) available.

@acowley I rebased and merged, but changed the algo. to pick up suggestions. Would you mind testing on GHC 8? Hopefully this regex will work on a few at once, and support multiple replacement suggestions.
As a todo item I'll need to create a test suite which will also run Emacs on all the GHC versions, not just intero the executable.

Thanks for the contribution!
I'd like to add another one for Top-level binding with no type signature: main :: IO () when I get round to it. 💃

The new regex isn't actually even close to working for GHC8.0.1. I can probably merge my working one with the one you added to keep it as a single regex, or we can start down the path of having GHC version specific regexs. Any preferences?

If it can be simply handled with one regex I'd prefer just one. But the GHC-specific ones are an option later.


Add author

Codecov Report

Merging #76 (37062df) into release-candidate (3523ded) will not change coverage.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           release-candidate       #76   +/-   ##
===================================================
  Coverage             100.00%   100.00%
===================================================
  Files                     50        50
  Lines                    860       860
===================================================
  Hits                     860       860


chore: update specs to TestBed

NOT to merge until https://github.com/angular/zone.js/issues/404 is fixed.

Fixed, @filipesilva review please.

Have that nit, otherswise LGTM.

Changed that.

I was about to report an issue that the karma tests don't work, but then I saw your changes here, and I integrated them into my workspace.
However, I'm still getting failures when I run npm test from part 1 of Tour of Heroes tutorial, The Hero Editor.
I created a new repo for QuickStart:
git clone git@github.com:angular/quickstart.git
cd quickstart/
npm install
npm test

And npm test runs correctly.
However, once I add the files from part 1 of the Tour-of-Heroes, the tests fail.
Note that I modified the last spec test in app.component.spec.ts to look for "Tour of Heroes".
It appears that there is still some problem with the test harness.
Can you please advise me?
    Chrome 52.0.2743 (Mac OS X 10.11.6) AppComponent with TCB should instantiate component FAILED
[1] 	Error: Template parse errors:
[1] 	Can't bind to 'ngModel' since it isn't a known property of 'input'. ("
[1] 	    <div>
[1] 	      <label>name: </label>
[1] 	      <input [ERROR ->][(ngModel)]="hero.name" placeholder="name">
[1] 	    </div>
[1] 	    "): AppComponent@6:13
[1] 	    at new BaseException (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/facade/exceptions.js:27:23)
[1] 	    at TemplateParser.parse (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/template_parser/template_parser.js:98:19)
[1] 	    at RuntimeCompiler._compileTemplate (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/runtime_compiler.js:246:51)
[1] 	    at eval (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/runtime_compiler.js:174:83)
[1] 	    at Set.forEach (native)
[1] 	    at compile (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/runtime_compiler.js:174:47)
[1] 	    at RuntimeCompiler._compileComponents (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/runtime_compiler.js:176:13)
[1] 	    at RuntimeCompiler._compileModuleAndAllComponents (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/runtime_compiler.js:78:37)
[1] 	    at RuntimeCompiler.compileModuleAndAllComponentsSync (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/src/runtime_compiler.js:52:21)
[1] 	    at TestingCompilerImpl.compileModuleAndAllComponentsSync (/private/tmp/lwpc/quickstart/node_modules/@angular/compiler/testing.js:71:31)
[1] 	    at TestBed._initIfNeeded (/private/tmp/lwpc/quickstart/node_modules/@angular/core/testing/test_bed.js:215:36)
[1] 	    at TestBed.createComponent (/private/tmp/lwpc/quickstart/node_modules/@angular/core/testing/test_bed.js:296:14)
[1] 	    at Function.TestBed.createComponent (/private/tmp/lwpc/quickstart/node_modules/@angular/core/testing/test_bed.js:126:29)
[1] 	    at Object.eval (/private/tmp/lwpc/quickstart/app/app.component.spec.js:18:41)
[1] 	    at ZoneDelegate.invoke (/private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:332:29)
[1] 	    at Zone.run (/private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:225:44)
[1] 	    at /private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:591:58
[1] 	    at ZoneDelegate.invokeTask (/private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:365:38)
[1] 	    at Zone.runTask (/private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:265:48)
[1] 	    at drainMicroTaskQueue (/private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:497:36)
[1] 	    at XMLHttpRequest.ZoneTask.invoke (/private/tmp/lwpc/quickstart/node_modules/zone.js/dist/zone.js:437:26)

Found a solution!
To app.component.ts, I added at the top:
import { FORM_DIRECTIVES } from '@angular/forms';

and within @Component({, I added
   , directives: [FORM_DIRECTIVES]

@psnider I would avoid that solution, because afaik it will not work in the near future. Instead, what you should do is configure the testing module to import the FormsModule.
TestBed.configureTestingModule({declarations: [AppComponent]});

I couldn't get this to work.  I modified app.component.spec.ts:
import { FormsModule }    from '@angular/forms';

and
describe('AppComponent with TCB', function () {
  beforeEach(() => {
    TestBed.configureTestingModule({declarations: [AppComponent, FormsModule]});
  });

What am I missing?

Well the testing module is like a normal module, right? So let's look at the toh-1 app.module.ts:
// #docregion
import { NgModule }      from '@angular/core';
import { BrowserModule } from '@angular/platform-browser';
import { FormsModule }   from '@angular/forms';

import { AppComponent }  from './app.component';

@NgModule({
  imports: [
    BrowserModule,
    FormsModule
  ],
  declarations: [
    AppComponent
  ],
  bootstrap: [ AppComponent ]
})
export class AppModule { }

Basically, AppComponent expects to be ran in an environment like this module. So I think what you're missing is to use the imports property:
 TestBed.configureTestingModule({
  imports: [FormsModule],
  declarations: [AppComponent]
});

Not too sure why the BrowserModule isn't needed though. I hope that'll be covered in the upcoming revision of the testing chapter that @wardbell is working on.

Thanks so much,   imports: [FormsModule], did the trick.
I'm paying close attention to testing, and want to make sure that all of the tests run at the end of each part of the tutorial.
FWIW, the testing support you have with the QuickStart and tutorial is very valuable to me.
But I struggle when I have to change it for another project structure.
I think I have it working now in: https://github.com/psnider/angular2-quickstart-rc5/tree/restructured
which is for a TypeScript-only micro-service project I'm working on:

node.js service for people, with Angular2 web-app
NativeScript mobile-app for people

What bits of it make it struggle, specifically?

The bits that make it a struggle

How are URLs from the browser transformed by each component of the test system?
The karma config and system.js config are complex. I'm confused by what configuration fields mean, and what they affect.
Related config information must be specified in multiple places, but I'm unclear on the differences
What does /base/ (in karma.conf.js) mean?
Many symbol names collide, making it difficult to understand scope and usage.

Details
Trying to configure karma and systemjs has been very difficult.
I would appreciate any references you could provide me.
karma
I can't say I understand how karma works yet, and there doesn't seem to be much of a tutorial.
All I've found is: http://karma-runner.github.io/1.0/intro/how-it-works.html,
which is surprisingly terse. It refers to a masters thesis, which I haven't read yet, but I'm looking for something in between, and which is specific to karma.
The karma configuration seems to be much more complex than what is described in http://karma-runner.github.io/1.0/config/configuration-file.html and http://karma-runner.github.io/1.0/config/files.html.
systemjs
I feel I have a better understanding of the system.js loader.
I found the documentation on the https://github.com/systemjs/systemjs to be easier to understand, but it is still a complex component that will take me some fair time and work to understand.
karma and system.js together with angular
In particular, I'd like to understand the details of how browser URLs are sent to the karma server, and how they are resolved. I don't understand what resolution is performed in the browser, and what is performed by the karma server, which paths are relative, and which are from the root.  The documentation seems to assume that the reader already understands these things. I also don't understand what the special prefix /base/ refers, and I can't find documentation that discusses this.
Another point of confusion is that the same symbol names are used for many things.
I think that using a separate symbols for each concept would help greatly.
In particular, app has many uses, and I believe many people in the community assume that it's use is required, for example this question. I'm used to web-apps not knowing the file system structure of the server, but it seems like many portions of the URLs are expected to match.
changing the project directory structure
I have several components in one project.
You can find my first attempt in the browser-angular2 component of people-service.
This was basically working, but broke when zone.js upgraded, see issue.
Then as I started to work through this, the tutorial was updated to RC5, so I'm redoing the work to incorporate it.

Heya @psnider, we have a revised and very complete Testing guide at https://angular.io/docs/ts/latest/guide/testing.html#!#setup-files. It doesn't answer all of difficulties, but I think it answers most of them.

@psnider I wince as I read your perfectly reasonable questions. Our testing guide does not yet cover re-configuration of karma and systemjs for a different project structure. It's on our backlog.  Your questions here are helpful and will inform our effort to take this necessary next step.  What should you do for now?   Ah ... I'll be back ;-)

@filipesilva @wardbell
Thanks so much!
For now, my all-TypeScript micro-service project, https://github.com/psnider/people-service, is almost complete!
I'm trying to use best practices for everything, especially testing.
Now I'm only missing one thing:

karma testing!
I still have to go over the testing guide you reference above.

I'd greatly appreciate you keeping me up to date with any improvements to the karma documentation!
Please feel free to comment directly on my project, if you like.


[Material] Add property to theme dial label colors on Time Picker

Description
The labels along the dial can now be themed using the new TimePickerThemeData.dialTextColor property.
Tests
I added the following tests:

Updating existing tests to check for this color.

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Started Google testing for this PR

Google testing passed!


Fix legend for stacked + individual chart

Fixes: #1074

@antw this currently keeps the Total, Stacked, Invidividual select box disabled. So if you could merge it any time soon that would be great.


add additional step in release cycle

What this PR does / why we need it:
Adds an additional step in releasing KIC to notify team.

[x ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Minor typo correction

Impressive open source projects deserve to be judged on usefulness, not spelling ;)

@jhwohlgemuth thanks for fix!


Add a Tutorial on how to perform offline debugging with ovs-offline

It should include both K8s and sos-based data collection

/assign @SalDaniele

Absolutely!


We need a "down" state

I find it hard to believe this hasn't come up before: I need god to keep some processes alive and not others.  For those that it doesn't need to keep alive, I'd like it to tell me whether they're up or down.  However, God currently doesn't support a "down" state, or anything analogous.  I'm genuinely confused as to how this wasn't in god from the beginning?

What daemontools? We're Enterprise with RHEL. Serious business, not some ubuntu toy.


Adding flow type checks

Refactored slideInterval tp set by state
Adding flow type checking

#6

Coverage remained the same at 100.0% when pulling 8f7ac53f6eef413d15f288b73daa1ddb2d68c187 on adding-flow-type-checks into c1b887f10b5a2ea05beb424b5c33afbb94be198e on master.


Select event now report each selected feature with associated layer

@elemoine this is ready for review. I think tests made in #3453 covers the code added in this PR so this should be rebased as soon as that is merged.

+1 from me for including layer in the SelectEvent. Amongst other things, it greatly simplifies multi-layer select for deletion.

this one will need rebasing/reworking

Rebased and resolved conflicts.

Rebased and resolved conflicts.

needs rebasing again :-)

Let's add a milestone so it ends up in the search for the next release manager (v3.8.0)

Let's add a milestone so it ends up in the search for the next release manager (v3.8.0)

I'm not sure I agree with this approach.  I don't think we want to put the onus on the release manager to review a bunch of outstanding pull requests - particularly ones with conflicts or ones that depend on other unfinished work.
I know that is sucks to have patches just sit around rotting, but I think we need to come up with a better way to get the work done rather than assuming the release manager will do it all.
This change in particular makes me a bit uneasy.  It sounds like it solves a pain point, but it looks like it would break any applications listening for select events.  So at a very minimum, it needs upgrade notes.  I'd also feel a lot more comfortable about it if I could read some tests related to the changes.  Without this, I didn't feel comfortable taking it on as an additional release task (in particular because it may very well break applications I'm responsible for elsewhere).
So @bjornharrtell, I appreciate your continued work to improve the situation with multi-layer selection.  I hope this can be resolved in a way that others feel good about.
(Also, very minor nit that I have a bias against tickets with No description provided.)

yeah I didn't mean only the release manager (I know I wrote it like that ;-) ), but anybody searching for tickets to review in advance for a release

Agreed on all points. I hope to be able to revisit this in the near future to improve and fix the merge conflicts.
As mentioned I'd like #3453 reviewed and hopefully merged before that because it resolves a long standing selection bug and adds tests that can be useful when working on this PR.

Closing due to lack of activity. @bjornharrtell please reopen if you continue working on it.


Use node-pty for Windows support

Fixes #441

This combined with https://github.com/sourcelair/xterm.js/pull/435 Windows dev should work. This is the fork used in Hyper and VS Code, I will publish a named release soon, see the README.

This is blocked on the gulp build https://github.com/sourcelair/xterm.js/pull/435

@parisk ready for review

Just tested again after rebase, works on Windows!


Add github.com/ynqa/word-embedding to NLP list

Add Word Embedding Tools in NLP list.

Thanks.  LGTM


Component accordion

Adds the accordion web component, updates to run preview with api-viewer based on custom elements manifest schema
some css fixes for card

Request for an enhancements later, can we explore:

using headings instead of text for the "accordion titles"? I think that will add some needed structure.
adding an id to the heading, so links to an accordion open the panel